Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to devel #193

Merged
merged 30 commits into from
Oct 1, 2024
Merged

Update to devel #193

merged 30 commits into from
Oct 1, 2024

Conversation

alexdanjou
Copy link
Contributor

Pull Request Template: delete as needed and mark appropriate check boxes [ ] with an x

Description:

One or two line description of change. Reference any relevant issues or other PRs using #XX e.g. #123 (use "closes #XX" to close the issue)

Type of Change:

[ ] Bug fix

[ ] New feature

[ ] Code-breaking change

Checklist:

[ ] Code documentation has been updated if needed

[ ] If new tests are needed, they have been added

[ ] All tests pass

[ ] No merge conflicts

[ ] /CHANGELOG.md has been updated if change is significant

Other Comments:

Any additional comments to reviewers not accounted for above

brendan-m-murphy and others added 29 commits June 14, 2024 16:23
"countries" was already used in the signature of the function
- changed `np.abs` to `pt.abs`
- added small amount to prevent 0 stdev in likelihood
including mu_bc and "epsilon", which will be used
for model error calculations
These need to be updated once the RHIME output
contains separate repeatability and variability
variables
To hold all concentration formatting
code
Don't want the same name as `get_obs_error`

This will be unnecessary once the RHIME outputs are fixed
Allows to choose the resolution to which the CAMS boundary conditions are stored : 3-hourly ("3h"), daily and monthly available.
…c-updates

Iss176 paris formatting conc updates
Merge BC_CAMS and BC_CAMS_CO2 from Eric's high_freq_bc branch.
instead of Yerror, since it is possible for
some files that Yerror is zero...
…rror-fix

Updated sampling for "no model error" case
Removing "The " from the beginning of ISO long names
doesn't change the number of distinct countries, so
this is done to try to match names from the East Asia
county file correctly.
Allow for formatting without adding aggregated-countries regions.
…ferentiating-countries

Ignore "The..." when looking for country codes
@alexdanjou alexdanjou marked this pull request as draft October 1, 2024 12:29
@alexdanjou alexdanjou closed this Oct 1, 2024
@alexdanjou alexdanjou reopened this Oct 1, 2024
@alexdanjou alexdanjou marked this pull request as ready for review October 1, 2024 12:32
@alexdanjou alexdanjou merged commit ede6099 into addTCCON Oct 1, 2024
0 of 2 checks passed
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