-
Notifications
You must be signed in to change notification settings - Fork 29
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 in General #56
Comments
raunakbh92
added a commit
that referenced
this issue
Mar 9, 2020
Pulled in all the code from Records.jl/src except for common.jl. Did not pull in Records.jl/test AutomototiveDrivingModels.jl tests pass locally Still using external Records.jl package
raunakbh92
added a commit
that referenced
this issue
Mar 10, 2020
…ecords.jl as a dependency. Please see branch incorporate_records
exoticDFT
added a commit
that referenced
this issue
Mar 10, 2020
- Adds the Records/src/common.jl file that contains IO functions for Records objects.
exoticDFT
added a commit
that referenced
this issue
Mar 10, 2020
- Removes the external Records import in the ADM package. - Exports all the "records" functions, types, etc. - Removes need for calling "Records" module in interface.jl.
MaximeBouton
added a commit
that referenced
this issue
Mar 20, 2020
[WIP] Issue #56: Bring Vec.jl as submodule.
MaximeBouton
added a commit
that referenced
this issue
Mar 21, 2020
[WIP] Issue #56: Move Records.jl into ADM Package
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It might be worth registering this package to julia General Registry.
I think the main limitation is the dependencies in Vec.jl and Records.jl.
Here are my suggestions:
option 1:
Register Vec to General (under another name), make Records.jl a submodule of AutomotiveDrivingModels.jl, register AutomotiveDrivingModels
option 2:
Make both Vec and Records submodules of AutomotiveDrivingModels.
I prefer option 2, as Vec and Records are not used anywhere else, and it might lead to less confusion for users about where datastructures like
Frame
,Entity
orVecSE2
are defined.Thoughts?
The text was updated successfully, but these errors were encountered: