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

Fix Finding Extrema Functionality #288

Merged
merged 7 commits into from
Aug 5, 2024
Merged

Conversation

brsm3129
Copy link
Collaborator

@brsm3129 brsm3129 commented Jul 23, 2024

  • Tickets addressed: MAXGNC-1015
  • Review: By file
  • Merge strategy: Merge (no squash)

Description

By associating all curves with a run number, we allow our selves to transpose the dataframe to find outliers in a time range.

Verification

Tested using the monte carlo examples

Documentation

No Documentation invalidated but getExtremaRunIndices only works with data formatted using curve_per_df_column

Future work

We would like to create a class that centralized plotting routines that standardizes all plotting for every variable.

@brsm3129 brsm3129 self-assigned this Jul 23, 2024
@brsm3129 brsm3129 changed the title Feature/fix extrema runs Fix Finding Extrema Functionality Jul 23, 2024
@brsm3129 brsm3129 force-pushed the feature/fix-extrema-runs branch 8 times, most recently from 589281d to 5df1b3c Compare July 25, 2024 05:53
@brsm3129 brsm3129 marked this pull request as ready for review July 25, 2024 05:53
src/utilities/fswSetupRW.py Outdated Show resolved Hide resolved
@brsm3129 brsm3129 added bug Something isn't working and removed bug Something isn't working labels Jul 29, 2024
@brsm3129 brsm3129 force-pushed the feature/fix-extrema-runs branch 3 times, most recently from ef66360 to 88dac6b Compare August 1, 2024 18:35
@brsm3129 brsm3129 force-pushed the feature/fix-extrema-runs branch 2 times, most recently from ee006ba to 2343b56 Compare August 2, 2024 23:06
@patkenneally patkenneally merged commit 93307a7 into develop Aug 5, 2024
3 checks passed
@patkenneally patkenneally deleted the feature/fix-extrema-runs branch August 5, 2024 18:24
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.

2 participants