DelayFixed returning a list assignment index out of range due to wrong pointer value

[OResearcher]

I tried to run .mdl file using pysd

#context: when I changed a few parameters in the model, it showed list assignment index out of range, but these values were working #fine in vensim model

!pip -q install pySD
import numpy as np
import pandas as pd
import pysd
import matplotlib.pyplot as plt

vmodel = pysd.read_vensim('Query_file.mdl')
SD_result = vmodel.run()
#the values of LT1, LT2 parameters in the original model are 30 and 3 respectively
#in the next step I tried to change the parameter (LT1, LT2) values to 60 and 6 respectively 

SD_result = vmodel.run(params={ 'LT1':60, 'LT2':6})

#I again tried to change the parameter values to 30 and 3, but it showed me an error: **list assignment index out of range**

SD_result = vmodel.run(params={ 'LT1':30, 'LT2':3})

Query_file.zip
please let me know why is it happening with my code
I have attached vensim model in the zip file for reference

[enekomartinmartinez]

HI @sanketme

Thank you for opening the issue. I will try to reproduce this bug and see if there is something that we need to correct.

[enekomartinmartinez]

Bug found!

self.pointer wasn't being set to 0 in the initialization. I will try to update it in the master soon.

[enekomartinmartinez]

Solved in branch dev.

I want to include some other fixes before the next release, which I will try to do before the new year. Currently, I do not have a lot of free time to work on PySD. Meanwhile, you can download PySD from that branch to make it work.

Please let me know if you need any further support!

[enekomartinmartinez]

We are also missing a large model that produces the Dataframe is highly fragmented warning. Does it matter if we use your model for testing? @sanketme

We already implemented some fixes to make it faster and avoid this warning in the branch dev

[OResearcher]

Hi there You can go ahead and use my model for testing. Thanks for the promt response. I was on leave since the time I posted the query. I will be active from now onwards. Regards

…

On Wed, 20 Dec 2023, 22:22 Eneko Martin-Martinez, ***@***.***> wrote: We are also missing a large model that produces the Dataframe is highly fragmented warning. Do you matter if we use your model for testing? @sanketme <https://github.com/sanketme> We already implemented some fixed to make it faster and avoid this warning in branch dev — Reply to this email directly, view it on GitHub <#427 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AUWXT7SVDUXJN6ED3E773WLYKMJVRAVCNFSM6AAAAABAOJJQ66VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRUHAYTINZXGE> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[OResearcher]

Solved in branch dev.

I want to include some other fixes before the next release, which I will try to do before the new year. Currently, I do not have a lot of free time to work on PySD. Meanwhile, you can download PySD from that branch to make it work.

Please let me know if you need any further support!

How can I download PySD from that branch?

[enekomartinmartinez]

will it work if I run the code with the same code lines?

Yes, but you may need to wait until the new version is released or install pysd from the branch.

For installing from the branch the instruction should be something like:
pip install git+https://github.com/SDXorg/pysd.git@dev#egg=pysd

[enekomartinmartinez]

@sanketme the release is already done, you should be able to install the new version in the regular way (pip install pysd)