Skip to content

Add short#218

Open
Jodasue wants to merge 6 commits intodevelopfrom
add_short
Open

Add short#218
Jodasue wants to merge 6 commits intodevelopfrom
add_short

Conversation

@Jodasue
Copy link
Contributor

@Jodasue Jodasue commented May 12, 2022

This is just a simple add so there is a new category for short bsuite runs.

Copy link

@rvinaybharadwaj rvinaybharadwaj left a comment

Choose a reason for hiding this comment

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

Flake8 tests are failing.

Jodasue added 2 commits May 13, 2022 14:29
2. Adding option to specify partition at end of sbatch command
3. Adding max steps per benchmark
@codecov
Copy link

codecov bot commented May 17, 2022

Codecov Report

Merging #218 (dd4a1b9) into develop (43aaafe) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #218   +/-   ##
========================================
  Coverage    11.64%   11.64%           
========================================
  Files           80       80           
  Lines         8578     8578           
  Branches      1278     1278           
========================================
  Hits           999      999           
  Misses        7512     7512           
  Partials        67       67           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 43aaafe...dd4a1b9. Read the comment docs.

2. -A option wasn't being passed through in bsuite_launch
Copy link

@rvinaybharadwaj rvinaybharadwaj left a comment

Choose a reason for hiding this comment

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

Did you unintentionally delete Hadrec_dir?

@Jodasue
Copy link
Contributor Author

Jodasue commented May 19, 2022

Sorry, I was switch back and forth branches. I didn't realize it was part of develop. I added it back. Maybe double check I did it right since I think it is a sub module...

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