Skip to content

Remove diagnostics from dscim#60

Merged
JMGilbert merged 11 commits into
dscim-v0.4.0from
remove_diagnostics
Jan 18, 2023
Merged

Remove diagnostics from dscim#60
JMGilbert merged 11 commits into
dscim-v0.4.0from
remove_diagnostics

Conversation

@JMGilbert
Copy link
Copy Markdown
Contributor

No description provided.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jan 10, 2023

Codecov Report

Merging #60 (581be0e) into dscim-v0.4.0 (07a74e1) will increase coverage by 14.24%.
The diff coverage is n/a.

@@                Coverage Diff                @@
##           dscim-v0.4.0      #60       +/-   ##
=================================================
+ Coverage         29.00%   43.24%   +14.24%     
=================================================
  Files                30       18       -12     
  Lines              2551     1711      -840     
=================================================
  Hits                740      740               
+ Misses             1811      971      -840     

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@JMGilbert JMGilbert changed the base branch from main to dscim-v0.4.0 January 10, 2023 23:00
@JMGilbert JMGilbert marked this pull request as ready for review January 10, 2023 23:12
Comment thread CHANGELOG.md Outdated
## [Unreleased]
- Remove diagnostics module. ([PR #60](https://github.com/ClimateImpactLab/dscim/pull/60, [@JMGilbert](https://github.com/JMGilbert))
- Remove old/unnecessary files. ([PR #57](https://github.com/ClimateImpactLab/dscim/pull/57), [@JMGilbert](https://github.com/JMGilbert))

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

does it make sense to add mention of pinning the two packages due to the float issue?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'm working to unpin the packages by messing with the code, so I'm hoping that won't be necessary. But I will definitely mention that if I can't figure this out.

@JMGilbert JMGilbert merged commit 57ee5eb into dscim-v0.4.0 Jan 18, 2023
@JMGilbert JMGilbert deleted the remove_diagnostics branch January 18, 2023 16:50
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.

4 participants