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

Soil/Sediment Visual Metrics #66

Closed
stephpenn1 opened this issue Mar 7, 2023 · 6 comments · Fixed by #94
Closed

Soil/Sediment Visual Metrics #66

stephpenn1 opened this issue Mar 7, 2023 · 6 comments · Fixed by #94
Assignees

Comments

@stephpenn1
Copy link
Member

No description provided.

@amyerspigg
Copy link
Collaborator

Update: will include in future versions of the data package. Will need help standardizing this

@amyerspigg amyerspigg closed this as not planned Won't fix, can't repro, duplicate, stale Apr 6, 2023
@stephpenn1 stephpenn1 reopened this Apr 28, 2023
@stephpenn1 stephpenn1 self-assigned this Sep 13, 2023
@stephpenn1 stephpenn1 changed the title figure out what to do with viz metrics Soil/Sediment Visual Metrics Dec 8, 2023
@stephpenn1
Copy link
Member Author

Update 2023-12-20

All columns have been standardized except cohesion_post_lyopholization and notes:

  • cohesion_post_lyopholization: based on the entries, it seems like this could be broken into two columns (aggregated and easy to break apart), but I'd like to talk to @amyerspigg or @kaizadp about what this column means before I do so

  • notes: good amount of repeats in here from what the other columns address, but since this is a qualitative analyte, the notes seem important here. I'd also like to go through these with AMP or KP and decide whether to remove or clean up.

@stephpenn1
Copy link
Member Author

stephpenn1 commented Dec 20, 2023

Kits that need attention:

@kaizadp
Copy link
Member

kaizadp commented Dec 21, 2023

  • cohesion_post_lyopholization: based on the entries, it seems like this could be broken into two columns (aggregated and easy to break apart), but I'd like to talk to @amyerspigg or @kaizadp about what this column means before I do so

I don't know the value of this column. I say drop, unless @amyerspigg thinks otherwise. Need to fix the spelling - "lyophilization"

@stephpenn1

This comment has been minimized.

@stephpenn1
Copy link
Member Author

@amyerspigg and @kaizadp agree to drop cohesion_post_lyopholization and notes

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 a pull request may close this issue.

3 participants