-
Notifications
You must be signed in to change notification settings - Fork 6
WIP:hr_to_mr and mr_to_hr functions (closes #54) #79
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
Open
berrakozer
wants to merge
28
commits into
main
Choose a base branch
from
sd_function_issue54
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
bdb22fe
docstring for hr_to_mr function
berrakozer 2cbf60c
test for hr_to_mr function
berrakozer f5b5f87
hr_to_mr function added to utils.py
berrakozer 8ae23f6
docstring for mr_to_hr function
berrakozer c574b32
test for mr_to_hr function
berrakozer c90c4f0
mr_to_hr function added to utils.py and testfor mr_to_hr edited, test…
berrakozer e2d9d8e
renamed functions and introduced numpy conventions to the docstrings
berrakozer f2600bd
edited the parameters and returns in docstrings of both functions
berrakozer 72b2687
altered hr_to_mr_number_and_esd function and test to accept string in…
berrakozer 4aa333e
edited hr_to_mr function and test
berrakozer 11eb646
edited mr_to_hr function and test
berrakozer 259bbf1
docstring for the (new) function round_number_esd
berrakozer 8b05b71
added test_round_number_esd function
berrakozer 67a0b4a
added round_number_esd function to utils.py, test passes
berrakozer e294ce3
edited the mr_to_hr_number_and_esd docstring
berrakozer 1411959
edited test_mr_to_hr_number_and_esd
berrakozer fb1e2e5
edited mr_to_hr_number_and_esd and the test, test passing
berrakozer f91eef3
Edited round_number_esd function and the test, test passes
berrakozer b5209ec
Edited test_mr_to_hr_number_and_esd, test passes
berrakozer ccd1faf
adjusted the cases where value is smaller than value error in utils.…
berrakozer fc9d266
edited hr_to_mr_number_and_esd function and test
berrakozer fc72a66
moved round_number_esd upwards and used it in hr_to_mr_number_and_esd…
berrakozer 70e6bd5
edited docstring of hr_to_mr_number_and_esd
berrakozer c3e4709
edited docstring of mr_to_hr_number_and_esd, included proper rounding…
berrakozer 94a66a2
Update test_utils.py
sbillinge e4d6f78
Update test_utils.py
sbillinge 52b2288
fixed test_mr_to_hr_number_and_esd, test passes
berrakozer 5bd70fb
Merge branch 'main' into sd_function_issue54
sbillinge File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good! I like it!