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

Issue installing FLOWNoise #2

Closed
eversilia opened this issue Sep 4, 2023 · 5 comments
Closed

Issue installing FLOWNoise #2

eversilia opened this issue Sep 4, 2023 · 5 comments

Comments

@eversilia
Copy link

DISCLAIMER: I am very new to Julia, hence it could be that I am missing something and this is not an issue.

I successfully precompiled all the dependencies and I am using Python 3.8.5, but I get the following error when importing FLOWNoise

ERROR: LoadError: ArgumentError: hasproperty of NULL PyObject
Stacktrace:
[1] pyhasproperty(o::PyCall.PyObject, s::String)
@ PyCall ~/.julia/packages/PyCall/ilqDX/src/PyCall.jl:371
[2] hasproperty
@ ~/.julia/packages/PyCall/ilqDX/src/PyCall.jl:377 [inlined]
[3] rc(args::String; kws::Base.Pairs{Symbol, String, Tuple{Symbol}, NamedTuple{(:family,), Tuple{String}}})
@ PyPlot ~/.julia/packages/PyPlot/2MlrT/src/PyPlot.jl:191
[4] top-level scope
@ ~/.julia/packages/FLOWNoise/t5EXx/src/FLOWNoise.jl:38
[5] include
@ ./Base.jl:418 [inlined]
[6] include_package_for_output(pkg::Base.PkgId, input::String, depot_path::Vector{String}, dl_load_path::Vector{String}, load_path::Vector{String}, concrete_deps::Vector{Pair{Base.PkgId, UInt64}}, source::Nothing)
@ Base ./loading.jl:1318
[7] top-level scope
@ none:1
[8] eval
@ ./boot.jl:373 [inlined]
[9] eval(x::Expr)
@ Base.MainInclude ./client.jl:453
[10] top-level scope
@ none:1

If I manually evaluate the expression in @ ~/.julia/packages/FLOWNoise/t5EXx/src/FLOWNoise.jl:38 it works fine.
Any help would be much appreciated, thank you a lot !

@EdoAlvarezR
Copy link
Collaborator

EdoAlvarezR commented Sep 5, 2023

Hi Enrico, I hope you find this code useful!

We ran into this issue a few weeks ago with the newer version of PyPlot. Please see the following issue in a related code LINK and let us know if the same solution helps you here.

P.S. If you are only interested in noise predictions, you might also find this FW-H code useful (developed by Daniel Ingraham, @dingraha): LINK

@eversilia
Copy link
Author

Hi Eduardo, thank you for your reply. Nice work and thank you for the very detailed installing instructions, very much appreciated !
I solved it by removing the calls to PyPlot.rc which looks like is the only one giving troubles . I will also try to change the PyPlot version as suggested in your linked issue.

Right now I am getting accustomed to FLOWUnsteady; I would like to see if it can help me overcome the limitations of the standard VPM that we exploit to simulate a wide variety of helicopter rotors. If you wish, I'll let you know how it goes !

@EdoAlvarezR
Copy link
Collaborator

It would be great if you are able to share any validation studies you might perform, or any new cool applications, but it's understandable if you can't.
@cibinjoseph was working on a rotor + fuselage case simulating the Robin helicopter, which you might be interested in picking up and continuing.

@cibinjoseph
Copy link
Member

cibinjoseph commented Oct 3, 2023

This is the same issue with PyPlot 2.11.2 version as in the FLOWUnsteady repo. It can be resolved by installing the previous version of PyPlot manually using the command:
] add [email protected]
This has to be done right before installing FLOWNoise.

@EdoAlvarezR
Copy link
Collaborator

Permanent fix: I removed some formatting of PyPlot that was being done upon loading the module. For some reason that was the cause of the error in the latest PyPlot version.

Now FLOWNoise is back to being compatible with any version of PyPlot

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