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.

Sign up for GitHub

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

Jump to bottom

added closure checks also when geochemical data that is in ppm or ppb #476

Merged
merged 1 commit into from
Dec 17, 2024
Merged

added closure checks also when geochemical data that is in ppm or ppb #476

merged 1 commit into from
Dec 17, 2024

Conversation

jtlait
Copy link
Collaborator

@jtlait jtlait commented Dec 13, 2024

Added closure checks also when units are ppm or ppb.

@jtlait jtlait requested a review from nmaarnio December 13, 2024 08:40
@jtlait jtlait linked an issue Dec 13, 2024 that may be closed by this pull request
Coda Transformations - Data Closure (ppm, ppb) #474
Closed
nmaarnio
nmaarnio approved these changes Dec 17, 2024
View reviewed changes
Copy link
Collaborator

@nmaarnio nmaarnio left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@nmaarnio nmaarnio merged commit b8ae3de into master Dec 17, 2024
3 of 4 checks passed
@nmaarnio nmaarnio deleted the 474-Coda-Transformations-Data-Closure-ppm-ppb branch December 17, 2024 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nmaarnio nmaarnio nmaarnio approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Coda Transformations - Data Closure (ppm, ppb)
2 participants
@jtlait @nmaarnio