Skip to content

Adds original ann2rr function #236

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

Merged
merged 2 commits into from
Jun 12, 2020
Merged

Adds original ann2rr function #236

merged 2 commits into from
Jun 12, 2020

Conversation

Lucas-Mc
Copy link
Contributor

@Lucas-Mc Lucas-Mc commented Jun 12, 2020

Adds the ann2rr function from the original WFDB software package. This version sets the -A option as the default in place of -c. Both the -f and -t options are conserved which specify the desired start and stop time. The -h option can be found using help(wfdb.ann2rr). The -i option is conserved except for the t parameter and specified number of decimal places. All the other options were not conserved since they can be determined using the current framework.

Adds the ann2rr function from the original WFDB software package. This version sets the -A option as the default in place of -c. Both the -f and -t options are conserved which specify the desired start and stop time. The -h option can be found using help(wfdb.ann2rr). The -i option is conserved except for the t parameter and specified number of decimal places. All the other options were not conserved since they can be determined using the current framework.
@Lucas-Mc Lucas-Mc self-assigned this Jun 12, 2020
@Lucas-Mc Lucas-Mc merged commit 18d949a into master Jun 12, 2020
@Lucas-Mc Lucas-Mc deleted the ann2rr branch June 12, 2020 11:25
Lucas-Mc added a commit that referenced this pull request Jun 12, 2020
Converts all relative imports to absolute as per PEP8 guidelines. According to documentation... Absolute imports are recommended, as they are usually more readable and tend to be better behaved (or at least give better error messages) if the import system is incorrectly configured (such as when a directory inside a package ends up on sys.path). I actually encountered the previous error on #236 and had a tough time tracking down the error which prompted me to make this switch.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant