Skip to content

use std::isnan#24

Open
vanossj wants to merge 1 commit into
pyushkevich:masterfrom
bamf-health:fix-isnan
Open

use std::isnan#24
vanossj wants to merge 1 commit into
pyushkevich:masterfrom
bamf-health:fix-isnan

Conversation

@vanossj

@vanossj vanossj commented Apr 2, 2021

Copy link
Copy Markdown

scoping isnan to use the stdlib version fixes #23

Is there a better isnan to use 🤷

@pyushkevich

pyushkevich commented Apr 2, 2021 via email

Copy link
Copy Markdown
Owner

@vanossj

vanossj commented Apr 2, 2021

Copy link
Copy Markdown
Author

It's still broken in the itk5 branch. But changing isnan to std::isnan fixes the itk5 branch too

@sarthakpati

Copy link
Copy Markdown
Contributor

This should no longer be needed.

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.

Errors building in docker

3 participants