rtoy / maxima

A Clone of Maxima's repo
Other
0 stars 0 forks source link

Error due ratprint occurs more than once in a variable list #2489

Open rtoy opened 4 months ago

rtoy commented 4 months ago

Imported from SourceForge on 2024-07-06 10:01:47 Created by valeryramirez on 2022-01-30 13:25:19 Original: https://sourceforge.net/p/maxima/bugs/3930


Hi all, how are you?. I'm having an error in maxima with ratprint, the error is exactly the one that appears in the title, all the content of the error is in the following github issue

https://github.com/wxMaxima-developers/wxmaxima/issues/1623

Can aneone help me with that?

rtoy commented 4 months ago

Imported from SourceForge on 2024-07-06 10:01:48 Created by willisbl on 2022-01-30 16:26:08 Original: https://sourceforge.net/p/maxima/bugs/3930/#9a25


I looked at the wxMaxima link you provided, but I couldn't understand what you had done. Could you post exactly the commands that you used? Additionally, you might try using the COMA package that comes with the current Maxima--that would make it easier for other to duplicate your bug.

rtoy commented 4 months ago

Imported from SourceForge on 2024-07-06 10:01:51 Created by macrakis on 2022-01-30 17:49:53 Original: https://sourceforge.net/p/maxima/bugs/3930/#3236


a) Next time, kindly submit a self-contained bug report on this bug reporter, rather than on a different one, with a pointer. b) Kindly provide a reproducible example of the problem. c) For bugs in outside packages like coma.mac, it's probably best to start by reporting a bug to the package maintainer.

As it happens, in this case, the bug is easy to find, and it is indeed on line 438 of coma.mac, which is not in function poles (which starts on line 499), but in function root_locus, and is exactly what the error message said it was:

block(
  [wxplot_size:assoc(dimensions,coma_defaults),
   i,range,nnticks,fac,pars,nn,pl,poli:[],gpoli:[],t,t1,t2,defs,hli,fli,h,
>>>>>>>>>>>>>>>>
   ratprint:false,startpoints,endpoints,ratprint:false,wxx],
<<<<<<<<<<<<<<<<
  opts:flatten(opts),
  defs:[aspect_ratio=-1,trange=[0.001,100],nticks=500],

I suggest you redefine this function without the duplicate ratprint:false and notify the maintainer of this package.

           -s

PS The version of coma on the site you provided is 1.902. The version distributed with Maxima is 2.1, and does not have this problem. Just use load("coma") to load that version. I don't know why http://www.austromath.at/daten/maxima/index.htm has an out of date version on it.

rtoy commented 4 months ago

Imported from SourceForge on 2024-07-06 10:01:55 Created by macrakis on 2022-01-30 17:50:25 Original: https://sourceforge.net/p/maxima/bugs/3930/#0929


rtoy commented 4 months ago

Imported from SourceForge on 2024-07-06 10:01:58 Created by valeryramirez on 2022-02-01 11:55:48 Original: https://sourceforge.net/p/maxima/bugs/3930/#5cfa


Hi all, thanks for all your help. In that time I didn't know that coma was included in the last version of maxima, now I know and I tried to reach who I think is the mantaine of the package but he didn't answer and because of the error I first published in maxima's github page and someone suggested me that it could be a bug in maxima (that is why I published here).

On the other hand I tought that that was the line since I opened the file with vscode and found that code on 438 after quiting all the comments, may be a noob analysis so I'm sorry for that.

rtoy commented 4 months ago

Imported from SourceForge on 2024-07-06 10:02:02 Created by macrakis on 2022-02-01 14:46:03 Original: https://sourceforge.net/p/maxima/bugs/3930/#5cfa/d41e


No problem with reporting a problem (whether it turns out to be a Maxima bug or not). That's why we're here.

Line number counts include comment lines normally.