Hello,
I apologise for submitting the patch for review without adequate testing.
Actually I got carried away with this piece of description of the bug
"Perhaps the unit conversion is being multiplied instead of divided?". As I
noticed this exact thing happening at one place in the file "observer.py",
I concluded that this should fix the issue. However, I realise the mistake
and assure you that no such event will occur in future.
I am currently facing the following issues while trying to reproduce the
bug :
1. I am unable to build the ksenija2 branch of the repository due to the
following bug : Error when launching PyGTK GUI if gtksourceview2 add-in is
not available on machine ( http://bugs.ascend4.org/view.php?id=573 ). It
would be helpful if someone can suggest a solution for this.
2. The model "branches/ksenija2/models/ksenija/thermostat.a4c model" does
not load correctly in the current trunk and throws a lot of syntactical
errors including errors like "Function EVENT is not defined". My doubt here
is that, is there something extra in the ksenija2 branch which is
compulsory for loading of this specific model?
3. I tried reproducing the bug with a few other models like
"ascend/models/dyn_tank.a4c" but failed to do so which points to the
conclusion that the bug is not always reproducible.
My guess is there is something specifically wrong with the model that
originally produced the bug and it would be very helpful if you could
provide the details of that model for further analysis.
Note : I have built the trunk successfully and tested it on a number of
existing models.
Thanks,
Aditya Jain
On Mon, Mar 23, 2015 at 9:03 AM, John Pye <[email protected]> wrote:
> Hi Aditya
>
> I looked at this patch from you, but it doesn't seem to solve the problem.
> Did you check it carefully? Can you give me the steps that you used to (1)
> reproduce the bug and (2) to prove that it was fixed?
>
> Cheers
> JP
>
>
> On 16/03/15 20:40, aditya jain wrote:
>
> Hello john,
>
> Thanks for the reply. I"ll surely do that when the bug-tracker is up. As
> you already know, I have been thinking about the project titled "Debugging
> tools for DAE modelling" and I request you to please provide your
> valuable feedback/comments on my progress related to the same (in reference
> to the other email thread that is going on). I am working on the proposal
> and your feedback would be helpful in formulating it in a better way.
>
> Thanking you,
> Aditya Jain
>
>
>
> On Mon, Mar 16, 2015 at 2:49 PM, John Pye <[email protected]> wrote:
>
>> Hi Aditya
>>
>> Thanks very much for the patch! I am very pleased to hear that you found
>> a solution. I'm sorry that the bug-tracker isn't editable, but I'll let you
>> know when it is so that you can post your patch file there. Meanwhile,
>> hopefully we'll be receiving a GSOC proposal from you!
>>
>> Cheers
>> JP
>>
>>
>> On 15/03/15 08:03, aditya jain wrote:
>>
>> Hello everyone,
>>
>> This is in reference to the following bug :
>> http://bugs.ascend4.org/view.php?id=584
>> I detected the point in "observer.py" where the unit conversion was
>> being multiplied instead of divided. I have attached the patch for your
>> reference. Since this is my first patch towards ASCEND, it would be helpful
>> if someone could review the same.
>>
>> Thanking you,
>> Aditya Jain
>>
>>
>>
>
>
------------------------------------------------------------------------------
Dive into the World of Parallel Programming The Go Parallel Website, sponsored
by Intel and developed in partnership with Slashdot Media, is your hub for all
things parallel software development, from weekly thought leadership blogs to
news, videos, case studies, tutorials and more. Take a look and join the
conversation now. http://goparallel.sourceforge.net/
_______________________________________________
Ascend-sim-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ascend-sim-users
