Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Teleplot big update #11

Merged
merged 42 commits into from
Nov 7, 2022
Merged

Teleplot big update #11

merged 42 commits into from
Nov 7, 2022

Conversation

Maximilien22
Copy link
Contributor

===== COMMUNICATION =====
-flag 't' added for text (fixes scientific notation bug)
-flag 'g' removed
-if no flag given or no | given, plot by default

  • protocol simplified for shape (using notation with colons rather than false json) + use previous value if property not given or default one if first time

===== 3D =====
---new---

  • add of orbitControl to turn around with mouse
  • add of a grid
    -rotation and position information added below shape name
  • quaternion added and used by teleplot if there isn't any euler rotation given
  • opacity property added and depthWrite change to always see through

---improvements---

  • no more useless shape copy, only one copy before rendering
  • no more iterating through every shape of a widget

---bugs---

  • shape disappear bug fixed

=====IMPORT / EXPORT=====
-updated to work with new structures

=====WIDGETS=====

  • when drop widget while on pause, show last value
  • stats fixed
  • bug when trying to select last value of serie fixed
  • bug for xy chart, don't display with too few values fixed

=====LOGS=====

  • total code rework :
    • creation of Log and LogConsole classes
    • use of a virtual scrolling list
    • no more iterating through all logs
    • sync with telem while mouse hovering working well in both ways

===== OTHER =====

  • python test improved and updated accordingly to new protocol + same for imu
  • telemetry filter search also in unit and type and is not case sensitive anymore
  • uPlot lib updated and two lines changed to work both with log cursor and xy charts
  • better file hierarchy
  • many other code improvements, and bugs fixed ...

maximilien.charmetant added 30 commits October 26, 2022 14:09
… telemetry fixed + bug on curso quit fixed + freeze while using imu fixed
… more useless shape copy made + imu should not freeze anymore + {changing protocol & shape take last value by default if not given} = in progress
…p chart on pause bug fixed + little single value code clean up + big bug with app.telemetries[key].data[2] undefined finally fixed
…p.logs + synced with everything (Binary search...) + consoleLog class creation + clog class creation + other code improvements
…) + app.window time bug introduced by use of miliseconds rather than seconds fixed
@nesnes

This comment was marked as resolved.

@nesnes

This comment was marked as resolved.

@nesnes

This comment was marked as resolved.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
@nesnes

This comment was marked as resolved.

@Maximilien22

This comment was marked as resolved.

@nesnes

This comment was marked as resolved.

@nesnes

This comment was marked as resolved.

server/www/index.html Outdated Show resolved Hide resolved
server/www/index.html Outdated Show resolved Hide resolved
@Maximilien22 Maximilien22 changed the title Teleplot big update draft:Teleplot big update Nov 3, 2022
@Maximilien22 Maximilien22 changed the title draft:Teleplot big update Teleplot big update Nov 3, 2022
server/main.js Outdated Show resolved Hide resolved
@nesnes
Copy link
Owner

nesnes commented Nov 4, 2022

As a general feedback, there are a few lines of commented code in multiple files, it might worth a cleaning

Can you re-check that?

@nesnes nesnes merged commit e176d59 into nesnes:main Nov 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants