Comments (9)
Appreciate this change!
from horizons.jl.
Fantastic! Looks great to me, thanks!
from horizons.jl.
Thanks for this suggestion; I agree we can make this a bit less verbose going forward. Feel free to submit a PR with the suggested change! 😄
from horizons.jl.
@david-macmahon this was fixed in principle by #42; feel free to reopen if it's not working as expected.
from horizons.jl.
Unfortunately, it's only a partial victory. The warning does not appear when using HORIZONS
, but it also does not appear on the console when doing ]add HORIZONS
. It seems that Pkg.build
is very successful at capturing stderr/stdout and logging it to build.log
unless run in verbose mode. I tried a few different hacks, including @ccall write(2, ...)
and open(/dev/sterr, "w")
, but they all got captured and redirected to build.log
.
I think it would be OK to revert this change (or decide to remove the warning altogether and/or move it to README.md
). If we really want to show a warning on only the first using HORIZONS
, we could use LocalPreferences.jl
or Scratch.jl
to track whether the warning has been displayed, but it hardly seems worth it.
This warning is most problematic (if one can really call it a "problem") when using HORIZONS .jl
with Distributed.jl
:
julia> using Distributed
julia> ws = addprocs(4);
julia> @everywhere using HORIZONS
┌ Warning:
│
│ # Breaking change
│ Starting from v0.4.0 HORIZONS.jl connects to JPL via a HTTP API.
│ Previous versions used the telnet command line utility as an external dependency.
└ @ HORIZONS ~/.julia/dev/HORIZONS/src/HORIZONS.jl:35
From worker 4: ┌ Warning:
From worker 4: │
From worker 4: │ # Breaking change
From worker 4: │ Starting from v0.4.0 HORIZONS.jl connects to JPL via a HTTP API.
From worker 4: │ Previous versions used the telnet command line utility as an external dependency.
From worker 4: └ @ HORIZONS ~/.julia/dev/HORIZONS/src/HORIZONS.jl:35
From worker 2: ┌ Warning:
From worker 2: │
From worker 2: │ # Breaking change
From worker 2: │ Starting from v0.4.0 HORIZONS.jl connects to JPL via a HTTP API.
From worker 2: │ Previous versions used the telnet command line utility as an external dependency.
From worker 2: └ @ HORIZONS ~/.julia/dev/HORIZONS/src/HORIZONS.jl:35
From worker 5: ┌ Warning:
From worker 5: │
From worker 5: │ # Breaking change
From worker 5: │ Starting from v0.4.0 HORIZONS.jl connects to JPL via a HTTP API.
From worker 5: │ Previous versions used the telnet command line utility as an external dependency.
From worker 5: └ @ HORIZONS ~/.julia/dev/HORIZONS/src/HORIZONS.jl:35
From worker 3: ┌ Warning:
From worker 3: │
From worker 3: │ # Breaking change
From worker 3: │ Starting from v0.4.0 HORIZONS.jl connects to JPL via a HTTP API.
From worker 3: │ Previous versions used the telnet command line utility as an external dependency.
From worker 3: └ @ HORIZONS ~/.julia/dev/HORIZONS/src/HORIZONS.jl:35
from horizons.jl.
Given that there has not been lots of complaints about moving on from telnet, I think it makes sense to move the warning to the README, so I'd propose to do just that.
from horizons.jl.
Can you confirm if the problem is solved in v0.4.8?
from horizons.jl.
Thanks, that's great! Yes, the problem is solved/eliminated in v0.4.8. While checking README.md
I noticed that it says the current stable release is v0.4.1, which is a bit out of date now. One other suggestion would be to switch the breaking change warning to a "GitHub markdown alert" (kind of like a Julia markdown admonition). This would display better on GitHub, but not quite as nicely on Julia (though not bad).
> [!WARNING]
> Be careful!
...looks like...
Warning
Be careful!
on GitHub and like │ [!WARNING] Be careful!
in Julia. This warning will be going away eventually, so this is not a high priority IMHO, but I thought I'd mention it anyway. Thanks again!
from horizons.jl.
@david-macmahon thank you for the suggestions! I went for adding the current release badge from JuliaHub, which will updated by itself in the future, and also added the GitHub-style markdown warning. I'll close this issue now (do let us know, though, if there's anything else needed to be done here).
from horizons.jl.
Related Issues (10)
- (test issue)
- Info about upcoming removal of packages in the General registry HOT 7
- Tag v0.1.0 HOT 2
- Tag v0.2.0 HOT 1
- TagBot trigger issue HOT 10
- Geophysical properties from major bodies HOT 5
- Add support for new HORIZONS APIs HOT 1
- Implement the rest of JPL APIs
- Tests failing on julia 1.11.0-alpha1 HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from horizons.jl.