Skip to content

Conversation

@znichollscr
Copy link

No description provided.

@znichollscr
Copy link
Author

@rhoesly I think you were writing with CRLF endings, which was causing the CI to break. This will fix it (check is here PCMDI#173).

@znichollscr
Copy link
Author

Once this is merged, then we can merge PCMDI#169

@znichollscr znichollscr changed the title Fix up line endings Fix up line endings and other minor fixes Dec 22, 2024

For use in setting aerosol size distribution and additional speciation (if desired), an auxiliary dataset providing emissions from solid biomass combustion is also provided. Note that these are a subset of emissions in the main files. These data, therefore, should NOT be added to the emissions in the main files. (Note that no data files for CO2 emissions from solid biomass are released, as CO2 emissions from CEDS are from fossil fuels only.)

Supplementary checking `.csv` text files that provide total global mass for each sector, month, and year are also available as well as global seasonal diagnostic plots can be found [here](https://zenodo.org/records/14145000).
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As a note, checksum generally refers to the checksum of the file so I would use different language to avoid confusing people (although I get that the idea is similar).

@durack1
Copy link

durack1 commented Jan 3, 2025

Looks like this is on a forked version of the repo where the changes need to go...

@znichollscr
Copy link
Author

Looks like this is on a forked version of the repo where the changes need to go

Yep. Merge this. Then that will cause PCMDI#169 to update, then we should be good to go.

@durack1
Copy link

durack1 commented Jan 7, 2025

@rhoesly you'll need to hit the green "Merge pull request" button below before we can pull this across and sync in PCMDI#169

Neither @znichollscr or I have repository permissions which allows us to do it

@rhoesly rhoesly merged commit 19f7b8f into rhoesly:slcf-documentation Jan 7, 2025
@rhoesly
Copy link
Owner

rhoesly commented Jan 7, 2025

@rhoesly you'll need to hit the green "Merge pull request" button below before we can pull this across and sync in PCMDI#169

Neither @znichollscr or I have repository permissions which allows us to do it

Ah thank you, I missed that. Done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants