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

[1pt] Add detail/information to plotting capabilities (Dev-plot-update) #327

Merged
merged 13 commits into from
Mar 30, 2021

Conversation

TrevorGrout-NOAA
Copy link
Contributor

@TrevorGrout-NOAA TrevorGrout-NOAA commented Mar 26, 2021

Addresses github issues #325

Creates a csv containing data used to create the scatterplot (comparing two versions)
Creates a directory ('individual') that contains subdirectories of all analyzed sites which contain a barplot and a csv containing the analyzed_data for each site.

To run the plot script an example call may be:

python3 eval_plots.py -m /path/to/metrics/csv -w /path/to/workspace -v fim_3_0_4_1_ms_c fim_3_0_4_4_ms_c -i

…h site (in a directory titled 'individual') and create a csv of data used to plot the scatterplot
@TrevorGrout-NOAA TrevorGrout-NOAA added the enhancement New feature or request label Mar 26, 2021
@TrevorGrout-NOAA TrevorGrout-NOAA self-assigned this Mar 26, 2021
RyanSpies-NOAA
RyanSpies-NOAA previously approved these changes Mar 29, 2021
Copy link
Collaborator

@RyanSpies-NOAA RyanSpies-NOAA left a comment

Choose a reason for hiding this comment

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

I pulled this feature branch locally and generated plots for my bathy test. The new plots and output csv files passed my spot checks. Approve merge

@TrevorGrout-NOAA
Copy link
Contributor Author

@BradfordBates-NOAA @nickchadwick-noaa This is ready to merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants