-
Notifications
You must be signed in to change notification settings - Fork 4
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.
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
Register package #38
Comments
Hi @rkoireng. I'm glad to hear you are finding Seis.jl useful and I understand that it would be useful to have it in the registry. I am currently in the process of deciding whether this is appropriate, because of potential naming confusion between this and other packages which use the 'Seis' prefix but different underlying code, such as SeisIO, SeisNoise, and the SeismicJulia packages SeisMain, etc. Once that is resolved, the new version of 'Seis.jl' will be released under a different name and added to the registry. At that point you would be able to update your code to use the package (with imaginary name import NewSeisPackage as Seis
Seis.read_sac("*.sac", ".")
# ... I will leave this open for now. I would welcome any feedback you have on whether or not you find it confusing that there are several fundamentally different seismic packages whose names start with 'Seis' which are incompatible. |
Thanks for the information.
…On Tue, Sep 5, 2023, 1:17 AM Andy Nowacki ***@***.***> wrote:
Hi @rkoireng <https://github.com/rkoireng>. I'm glad to hear you are
finding Seis.jl useful and I understand that it would be useful to have it
in the registry. I am currently in the process of deciding whether this is
appropriate, because of potential naming confusion between this and other
packages which use the 'Seis' prefix but different underlying code, such as
SeisIO, SeisNoise, and the SeismicJulia packages SeisMain, etc. Once that
is resolved, the new version of 'Seis.jl' will be released under a
different name and added to the registry.
At that point you would be able to update your code to use the package
(with imaginary name NewSeisPackage) like so:
import NewSeisPackage as Seis
Seis.read_sac("*.sac", ".")# ...
I will leave this open for now. I would welcome any feedback you have on
whether or not you find it confusing that there are several fundamentally
different seismic packages whose names start with 'Seis' which are
incompatible.
—
Reply to this email directly, view it on GitHub
<#38 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AKN3ZSMRGUEZTMFS7UKR3ATXYYV5BANCNFSM6AAAAAA4KZJVPE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Hi ,
I am using Pluto notebook and I am not able to load the Seis package directly. It seems that Seis is not yet in the Julia registry. Can you please resolved this ? It would be great if Seis can be easily imported, especially for Pluto notebook users.
Thanks a lot.
The text was updated successfully, but these errors were encountered: