Jump to content


Photo

Faulty commit "eerror.cpp: Optimize log output routine"


  • Please log in to reply
33 replies to this topic

Re: Faulty commit "eerror.cpp: Optimize log output routine" #21 littlesat

  • PLi® Core member
  • 57,330 posts

+708
Excellent

Posted 25 February 2016 - 19:00

Just a suggestion what you can do to continue debugging for the time being...

 

test = {"A": 1, "B": 2 }
for k, v in test.iteritems():
    print k, v

WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W


Re: Faulty commit "eerror.cpp: Optimize log output routine" #22 Huevos

  • PLi® Contributor
  • 4,735 posts

+167
Excellent

Posted 26 February 2016 - 11:03

 

Just a suggestion what you can do to continue debugging for the time being...

test = {"A": 1, "B": 2 }
for k, v in test.iteritems():
    print k, v

Yes in that instance. But you can't even output a "dir(obj)" for some methods. Like I say debuging or writing code is impossible now.



Re: Faulty commit "eerror.cpp: Optimize log output routine" #23 littlesat

  • PLi® Core member
  • 57,330 posts

+708
Excellent

Posted 26 February 2016 - 11:34


Yes in that instance. But you can't even output a "dir(obj)" for some methods. Like I say debuging or writing code is impossible now.

 

I agree you want to do print dict and some debuggings might be less usefull... But you must also agree for long dicts/objects are also helpfull at all.... And you must also understand this is work in progression.... So later it is most likely fixed...

 

For now when you need own debugging you should implement a work-a-round...


Edited by littlesat, 26 February 2016 - 11:36.

WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W


Re: Faulty commit "eerror.cpp: Optimize log output routine" #24 Taapat

  • PLi® Core member
  • 2,345 posts

+121
Excellent

Posted 26 February 2016 - 16:56

@Huevos maybe I miss something, but if you use your enigma build, where is the problem set buf size as large as you need?



Re: Faulty commit "eerror.cpp: Optimize log output routine" #25 littlesat

  • PLi® Core member
  • 57,330 posts

+708
Excellent

Posted 26 February 2016 - 19:24

Too large is also no good... ;).... It is not a real solution... e.g. when you set it at 2048 we can again have lines of >2048 characters... :(...

 

A dynamic solution should be solved without limits it time processing...


Edited by littlesat, 26 February 2016 - 19:25.

WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W


Re: Faulty commit "eerror.cpp: Optimize log output routine" #26 Huevos

  • PLi® Contributor
  • 4,735 posts

+167
Excellent

Posted 26 February 2016 - 19:51

@Huevos maybe I miss something, but if you use your enigma build, where is the problem set buf size as large as you need?

Having a big buffer is not the answer. The previous code used a 1024 buffer without this problem.



Re: Faulty commit "eerror.cpp: Optimize log output routine" #27 betacentauri

  • PLi® Core member
  • 7,185 posts

+323
Excellent

Posted 26 February 2016 - 21:15

For Python output it used a string and not a 1024 byte char buffer (as far as I saw in the code).
Xtrend ET-9200, ET-8000, ET-10000, OpenPliPC on Ubuntu 12.04

Re: Faulty commit "eerror.cpp: Optimize log output routine" #28 mirakels

  • Forum Moderator
    PLi® Core member
  • 7,603 posts

+62
Good

Posted 26 February 2016 - 21:33

Actually, for python output there were no timestamps so were printed directly to screen. It no goes through the eDebug layer that uses buffers because it has been said strings are expensive (and I guess vsnprint() works on buffers and not on strings as destination)


Geen wonder... Had slechts een dm7000, maar wel ook een rotor. eigenlijk al een tijdje ook een dm600 en dm7025. Maar nu kijkend met een et9000 en vuduo

Re: Faulty commit "eerror.cpp: Optimize log output routine" #29 mirakels

  • Forum Moderator
    PLi® Core member
  • 7,603 posts

+62
Good

Posted 26 February 2016 - 21:34

But then again I never actually saw the issue and also cannot find the example that Huevos is giving in the openpli engima code.

 

 

I may have a solution but have no example to test it on.


Geen wonder... Had slechts een dm7000, maar wel ook een rotor. eigenlijk al een tijdje ook een dm600 en dm7025. Maar nu kijkend met een et9000 en vuduo

Re: Faulty commit "eerror.cpp: Optimize log output routine" #30 Huevos

  • PLi® Contributor
  • 4,735 posts

+167
Excellent

Posted 26 February 2016 - 21:40

But then again I never actually saw the issue and also cannot find the example that Huevos is giving in the openpli engima code.

 

 

I may have a solution but have no example to test it on.

If you have a fix I can test it. Just post the CPP file and I'll post the result.


Edited by Huevos, 26 February 2016 - 21:41.


Re: Faulty commit "eerror.cpp: Optimize log output routine" #31 littlesat

  • PLi® Core member
  • 57,330 posts

+708
Excellent

Posted 26 February 2016 - 21:41

Just put print "*" * 2000 or so somewhere in the python code and you have the anomaly....


WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W


Re: Faulty commit "eerror.cpp: Optimize log output routine" #32 mirakels

  • Forum Moderator
    PLi® Core member
  • 7,603 posts

+62
Good

Posted 27 February 2016 - 00:03

ok, did an implementation.


Geen wonder... Had slechts een dm7000, maar wel ook een rotor. eigenlijk al een tijdje ook een dm600 en dm7025. Maar nu kijkend met een et9000 en vuduo

Re: Faulty commit "eerror.cpp: Optimize log output routine" #33 littlesat

  • PLi® Core member
  • 57,330 posts

+708
Excellent

Posted 27 February 2016 - 00:10

:D

WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W


Re: Faulty commit "eerror.cpp: Optimize log output routine" #34 Huevos

  • PLi® Contributor
  • 4,735 posts

+167
Excellent

Posted 27 February 2016 - 15:39

Thanks for the fixes. Tested and working fine now.




7 user(s) are reading this topic

0 members, 7 guests, 0 anonymous users