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

@test macro with rtol #383

Closed
cnaak opened this issue Sep 8, 2020 · 3 comments
Closed

@test macro with rtol #383

cnaak opened this issue Sep 8, 2020 · 3 comments
Labels
expression explorer Figuring out assignments and references in a cell other packages Integration with other Julia packages

Comments

@cnaak
Copy link

cnaak commented Sep 8, 2020 •
edited
Loading

In the following, rtol is just a parameter of the infix operator ≈ (approx), but Pluto thinks an rtol variable has been created:

@test A ≈ B rtol = eps(Float64)

Side effects include not being able to create an rtol variable in another cell, which errors with: "Multiple definitions for rtol:"
@fonsp fonsp added the expression explorer Figuring out assignments and references in a cell label Sep 8, 2020
mcabbott pushed a commit to mcabbott/Pluto.jl that referenced this issue Sep 13, 2020
add at-test too?
796244d 
Closes fonsp#383
@mcabbott mcabbott mentioned this issue Sep 14, 2020
Closed
mcabbott pushed a commit to mcabbott/Pluto.jl that referenced this issue Sep 14, 2020
add at-test too?
66af479 
Closes fonsp#383
@fonsp
Copy link
Owner

fonsp commented Sep 24, 2020

#196 #407

@fonsp fonsp closed this as completed Sep 24, 2020
@fonsp fonsp reopened this Sep 24, 2020
@fonsp fonsp changed the title Infix operator parameter syntax fools Pluto @test macro with rtol Sep 25, 2020
@fonsp fonsp added the other packages Integration with other Julia packages label Sep 26, 2020
@jcmkk3
Copy link

jcmkk3 commented Oct 17, 2020

I'm not sure if it would be useful, but the presentation for Salsa.jl was referencing some issues very similar to this.

@fonsp fonsp mentioned this issue Nov 19, 2020
Closed
@fonsp
Copy link
Owner

fonsp commented Nov 19, 2020

Fixed in c6fa171

@fonsp fonsp closed this as completed Nov 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
expression explorer Figuring out assignments and references in a cell other packages Integration with other Julia packages
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@fonsp @cnaak @jcmkk3