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.

Sign up for GitHub

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

Jump to bottom

Parser: to_int() return None instead of np.nan #569

Merged
merged 2 commits into from
Feb 20, 2025
Merged

Parser: to_int() return None instead of np.nan #569

merged 2 commits into from
Feb 20, 2025

Conversation

ywang103-amd
Copy link
Contributor

…parser.py

@ywang103-amd ywang103-amd requested a review from koomie as a code owner February 18, 2025 21:41
@ywang103-amd ywang103-amd requested review from coleramos425 and feizheng10 and removed request for koomie February 18, 2025 21:41
@ywang103-amd ywang103-amd changed the title Fix SWDEV-515801: change ways of using None type to fix type recognition error Parser: to_int() return None instead of np.nan Feb 18, 2025
@ywang103-amd ywang103-amd merged commit e5803eb into ROCm:develop Feb 20, 2025
9 checks passed
vedithal-amd pushed a commit that referenced this pull request Feb 24, 2025
@ywang103-amd @vedithal-amd
Parser: to_int() return None instead of np.nan (#569)
03ea7f2
vedithal-amd pushed a commit that referenced this pull request Feb 24, 2025
@ywang103-amd @vedithal-amd
Parser: to_int() return None instead of np.nan (#569)
29e8fe9
Swathi9494 pushed a commit that referenced this pull request Feb 25, 2025
@ywang103-amd @Swathi9494
Parser: to_int() return None instead of np.nan (#569)
3f606e8
vedithal-amd pushed a commit that referenced this pull request Feb 26, 2025
@ywang103-amd @vedithal-amd
Parser: to_int() return None instead of np.nan (#569)
24dd003
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@coleramos425 coleramos425 coleramos425 approved these changes

@feizheng10 feizheng10 feizheng10 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ywang103-amd @coleramos425 @feizheng10