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: Fix _cudadecon_version annotation, added support for skewed decon argument #47

Merged
merged 6 commits into from
Sep 13, 2023

Conversation

dmilkie
Copy link
Contributor

@dmilkie dmilkie commented Sep 13, 2023

Fixed _cudadecon_version: tuple[int, ...] = (0, 0, 0) from throwing an exception.

Working on Windows with cudadecon 0.6.2.

@dmilkie dmilkie changed the title Fix _cudadecon_version annotation, added support for skewed decon argument [fix][improve]Fix _cudadecon_version annotation, added support for skewed decon argument Sep 13, 2023
@dmilkie dmilkie changed the title [fix][improve]Fix _cudadecon_version annotation, added support for skewed decon argument [fix][misc]Fix _cudadecon_version annotation, added support for skewed decon argument Sep 13, 2023
@dmilkie
Copy link
Contributor Author

dmilkie commented Sep 13, 2023

I don't know what this "Semantic PR — add a semantic commit or PR title" is trying for. I think it's looking for a [type][scope] in the PR title. Which I think I gave it?

@dmilkie dmilkie changed the title [fix][misc]Fix _cudadecon_version annotation, added support for skewed decon argument [fix][misc] Fix _cudadecon_version annotation, added support for skewed decon argument Sep 13, 2023
@tlambert03 tlambert03 changed the title [fix][misc] Fix _cudadecon_version annotation, added support for skewed decon argument fix: Fix _cudadecon_version annotation, added support for skewed decon argument Sep 13, 2023
@tlambert03
Copy link
Owner

Don't worry about that, I edited the title... but the annotation needs undoing, see comment

Copy link
Owner

@tlambert03 tlambert03 left a comment

Choose a reason for hiding this comment

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

Perfect, thanks!

@tlambert03 tlambert03 merged commit a2e6cbb into tlambert03:main Sep 13, 2023
1 check passed
@tlambert03
Copy link
Owner

released on v0.4.1

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.

None yet

2 participants