-
Notifications
You must be signed in to change notification settings - Fork 59
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
Hudson Bay #45
Comments
Oh, I found a working example in DiffEqFlux that works with Lux & Optimization. Is DiffEqFlux being kept separate? Or it just works there till it's moved over? Despite this, I get a no method matching... error when I add DiffEqFlux" and try to use ADAM: And even if I try to use PolyOpt, same. But I don't know why that would be an issue, so I will stop here. |
Well, I moved onto LV Scenario 2, and then thought it could be that I didn't add st I have the exact same problem with Scenario 2. No method matching... then Array has no field layer if I try to add st and [1]. |
Also @AlCap23 |
@ccrnn You are getting this error because you are trying to access the parameters as p[1], p[3:end] etc inside the ude_dynamics function. What I suggest you do is the follows. This should work:
|
@RajDandekar Thanks, that works! If I don't put it in that format it will run, seems to work, but the final long term estimate is way out.
|
@ccrnn I will try to work this out on my end.. |
@ccrnn I found some potential errors in your code. Can you try making these changes? You have this:
which should be
p_model definition should be like this:
Also, this is not correct:
You should have p_model, instead of p. |
Thanks, I went to try this, but now even before I get to that point, at the BFGS shooting loss, it errors. There is also a problem with the predict function without X = Xₙ[:, 1], T = t now. I have no idea how this is happening, because I did get it to run this morning?! In general, I think it would be good if the Lux documentation were updated with some examples of how to set this up too. It's not really clear enough how to use it for a beginner atm. I will have a go at FENEP again next. Tbh I'm a little frustrated at this point though. I don't understand why it doesn't work now. |
You don't need to ping every time. Github will email everyone subscribed to an issue, and one is automatically subscribed after joining the thread. |
Try this code below. It should not give any errors:
|
I get the same error on BFGS with this as I was getting with mine this evening... |
I ran the above code on my end again and am getting no error. Where is it you get the error exactly and what's the error? |
I just opened up a new folder and started again. So maybe I need to delete .julia and start from nothing, I guess it means there's some conflict somewhere? The error appears when I run:
Error:
|
On it. ( with a little delay and probably a little more ). |
@ccrnn were you able to run the code? Is it working now? |
I couldn't find an example of multiple shooting in SciMLSensitivity, sorry - it's coming up with the error "UndefVarError: multiple_shoot not defined" when I try to run ADAM.
I also tried FENEP but it seems like there's some issue with Tracked Arrays, here: SciML/SciMLSensitivity.jl#609
And also with Flux, because the original doesn't work with it. I get the error: UndefVarError: TrackedArray not defined
So i'll leave this one.
The text was updated successfully, but these errors were encountered: