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.

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

Compilation error of PowerModelsACDC for Julia1.4 and higher #59

Closed
SabineAuer opened this issue Sep 8, 2020 · 9 comments
Closed

Compilation error of PowerModelsACDC for Julia1.4 and higher #59

SabineAuer opened this issue Sep 8, 2020 · 9 comments

Comments

@SabineAuer
Copy link

We wanted to add PowerModelsACDC to the dependencies of PowerDynamics.jl and we are working on a PR there at the moment. However, for Julia1.4 and Julia1.5 the compilation of PowerModelsACDC fails with [this error message](Travis CI, see here https://travis-ci.org/github/JuliaEnergy/PowerDynamics.jl/jobs/723063577#L894) saying ERROR: LoadError: LoadError: UndefVarError: GenericPowerModel not defined

Can you help us out with this issue?

@ccoffrin
Copy link
Collaborator

ccoffrin commented Sep 8, 2020

Based on this CI run it looks to be working from a fresh install of Julia v1.4.

https://travis-ci.org/github/hakanergun/PowerModelsACDC.jl

I also tested locally on Julia v1.5 and it seemed to work.

My best guess is that you have another related dependency (like PowerModels.jl) and the package manager selected an old version of this package to try and meet some version constraints.

@hakanergun, if you like I can do a little work on your Project.toml file so that more version combinations can be supported.

@hakanergun
Copy link
Collaborator

That would be much appreciated. In a way, I always have trouble in creating the .toml if I try to register a new version, where I fix the version of PM & IM, to avoid having to run in breaking changes. So all info you can give me on that, I gold :).

@hakanergun
Copy link
Collaborator

@SabineAuer It seems that by default, v0.1.0 of PowerModelsACDC is added, which is not up to date anymore, of course. As Carleton pointed out. It seems that an older version of PMACDC is added, if you already have a higher version of PM 0.17.2 (to which the latest version of PMACDC is fixed) in your project.

@SabineAuer
Copy link
Author

thanks a lot for your fast replies. So the solution for me is probably then to fix the version in the compat-section of our Project.toml, right?
What combination of versions works well together then? PowerModelsACDC v0.3.2 and what version of PowerModels?

@hakanergun
Copy link
Collaborator

I guess so. Otherwise it is retrieving probably PowerModels first and ends up having the version conflict. It seems that I have set the PM version rather ambiguous in the v0.1.0 version of PMACDC, not sure if I can change that retroactively (@ccoffrin)?

Also, it's hard to keep up with PM versions sometimes ;).

@ccoffrin
Copy link
Collaborator

@SabineAuer the master branch should resolve your issue. Can you check on your side and report back? If all is good I would recommend theta @hakanergun tag a new patch release.

@SabineAuer
Copy link
Author

I added the master branch of PowerModelsACDC and everything is compiling now properly now. A new patch sounds great, thank you!

@ccoffrin
Copy link
Collaborator

I think v0.3.3 addressed this issue, shall we close it?

@hakanergun
Copy link
Collaborator

Indeed, I am closing this issue. Thanks again @ccoffrin !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants