Coder Social home page Coder Social logo

Comments (7)

MajicDesigns avatar MajicDesigns commented on May 28, 2024

Not sure if you are reporting a problem as this has not been seen by anyone else who uses the code?

from md_parola.

MajicDesigns avatar MajicDesigns commented on May 28, 2024

Parola library compiles all examples ok and no response fro OP, closing issue.

from md_parola.

Martin12350 avatar Martin12350 commented on May 28, 2024

I am guessing not a lot of people are trying to read intensity back, but same error here ;)
int intensity = Display.getIntensity();
Compilation error: 'class MD_Parola' has no member named 'getIntensity'.

In source code, You have only:
inline uint8_t getIntensity() { return _intensity; }

And nothing else, while e.g. getPause has void as a parameter and then 2 other functions.
Same for speed, etc.

Can You please check if this really works? Thanks a lot.

UPDATE!
I see where this misunderstanding comes from...
Speed, pause and others return zone 0 when used on MD_Parola.
But intensity is implemented only for MD_PZone.

It is not a bug, but not implemented the same way.
Is there a possibility to update for consistency? ;)

from md_parola.

MajicDesigns avatar MajicDesigns commented on May 28, 2024

Every LED module can be different intensity so it does not make sense to return just the intensity of module 0 in the zone although I can see there is a lack of symmetry with being able to setIntensity().

With the others you mentioned the setting is universal within a zone so if there is no zone specified zone 0 makes sense as there is only one zone being used.

from md_parola.

Martin12350 avatar Martin12350 commented on May 28, 2024

I was just comparing how other functions in the library work. Like pause, You can have each zone with a different pause, but if You call getPause without a parameter, You get zone 0.
Also in my case, when I have only a few modules, I do not use zones, so I call all functions without specifying zone, which automatically results in 0.
Anyway, how You deal with it is up to You. But would be cool to have getIntensity in MD_P.
Keep the good work :)

from md_parola.

MajicDesigns avatar MajicDesigns commented on May 28, 2024

version 3.7.1 makes all this visible and symmetric with other functions.

from md_parola.

Martin12350 avatar Martin12350 commented on May 28, 2024

Awesome, immediately updating lib :) And as expected compiled ok. Will flash asap, but not expecting any problem. Thanks!

from md_parola.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.