Bdanilko / EdPy

A compiler for a subset of python2 to be used on the Microbric Edison robot
GNU General Public License v2.0
18 stars 12 forks source link

Note Ed.NOTE_EIGHTH, typo #9

Open PSLLSP opened 5 years ago

PSLLSP commented 5 years ago

Note Ed.NOTE_EIGHTH has typo and is defined as:

    "Ed.NOTE_EIGHT":     250,

Documentation is correct, it lists Ed.NOTE_EIGHTH. https://meetedison.com/content/EdPy-app-documentation-guide.pdf

It is problem when man uses on-line editor (https://www.edpyapp.com), the smart assistant suggests Ed.NOTE_EIGHTH but compiler later reports error that Ed.NOTE_EIGHTH has no value defined...

Demo:

#-------------Setup----------------

import Ed

Ed.EdisonVersion = Ed.V2

Ed.DistanceUnits = Ed.CM
Ed.Tempo = Ed.TEMPO_MEDIUM

#--------Your code below-----------

Ed.PlayTone(Ed.NOTE_C_7, Ed.NOTE_EIGHT)  # this can be compiled but value is wrong
while Ed.ReadMusicEnd() == Ed.MUSIC_NOT_FINISHED: pass
Ed.TimeWait(1, Ed.TIME_SECONDS)

Ed.PlayTone(Ed.NOTE_C_7, Ed.NOTE_EIGHTH)  # this is correct but compiler reports error
while Ed.ReadMusicEnd() == Ed.MUSIC_NOT_FINISHED: pass
Ed.TimeWait(1, Ed.TIME_SECONDS)
BenMicrobr commented 5 years ago

Hi I'm Ben from Meet Edison

I've added this to our backlog of bugs.

Additionally, this repository is not the best place to report bugs and issues. We do not continuously monitor this repository. For future bugs and issues, please send them directly to us, via our contact form: https://meetedison.com/edison-robot-support/contact-us/