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

cannot import horner macro #3897

Closed
simonbyrne opened this issue Jul 31, 2013 · 1 comment
Closed

cannot import horner macro #3897

simonbyrne opened this issue Jul 31, 2013 · 1 comment

Comments

@simonbyrne
Copy link
Contributor

I want to use the horner macro in a package, however when I try to import it, I get the following error:

julia> quantile(Normal(),0.1)
ERROR: r not defined
 in Φinv at /Users/simon/.julia/Distributions/src/univariate/normal.jl:60
 in quantile at /Users/simon/.julia/Distributions/src/univariate/normal.jl:141

But it works fine if when redefined in the package.
@JeffBezanson
Copy link
Member

Fortunately this is pretty straightforward --- the horner macro was originally designed to be used only in the module where it is defined, so it does not use esc where it should. I guess this is a risk of using a non-exported macro.

IanButterworth pushed a commit that referenced this issue Jun 4, 2024
@DilumAluthgeBot @DilumAluthge
🤖 [master] Bump the Pkg stdlib from ed7a8dca8 to 4e43058c2 (#54666)
4e211eb 
Stdlib: Pkg
URL: https://github.com/JuliaLang/Pkg.jl.git
Stdlib branch: master
Julia branch: master
Old commit: ed7a8dca8
New commit: 4e43058c2
Julia version: 1.12.0-DEV
Pkg version: 1.12.0
Bump invoked by: @IanButterworth
Powered by:
[BumpStdlibs.jl](https://github.com/JuliaLang/BumpStdlibs.jl)

Diff:
JuliaLang/Pkg.jl@ed7a8dc...4e43058

```
$ git log --oneline ed7a8dca8..4e43058c2
4e43058c2 Merge pull request #3887 from carlobaldassi/validate_versions
bc7c3207d abort querying more pacakge for hint auto complete (#3913)
a4016aed2 precompile repl switch (#3910)
a48c9c645 Fixed glitch in the manual (#3912)
d875aa213 Add timeout and new tests for resolver
aeb55f7f0 run artifact selection code with minimal compilation (#3899)
0180a0105 avoid doing some checks if the package will not be showed in status output (#3897)
c6c7ed502 improve precompilation for `st` in the Pkg REPL (#3893)
bffd0633c Add version validation during Graph simplification
c2ad07003 Fix padding in resolve's log journal printing
3eb86d29f Revert #2267, with better log message
acdbb727e Small extra check in Graph's check_consistency
1d446c224 Fix small bug in Graph constructor
3efc3cbff Fix show method for VersionSpecs
```

Co-authored-by: Dilum Aluthge <dilum@aluthge.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@JeffBezanson @simonbyrne