Plant-Tracer / webapp

Client and Server for web-based JavaScript app
GNU Affero General Public License v3.0
0 stars 2 forks source link

Create calculate_results_circumnutation #160

Open sbarber2 opened 9 months ago

sbarber2 commented 9 months ago

Calculate maxAmplitude and rate given vector of [x,y] points and frame rate, return as dictionary

See https://github.com/Plant-Tracer/legacy_ios_app/blob/main/www/app/controllers/Graph.js for specfics of calculation

sbarber2 commented 9 months ago

Here is a draft of the specification for the functionality covered in this issue:

The specifics of the result calculations will be analogous to those in the showResult() function of https://github.com/Plant-Tracer/legacy_ios_app/blob/main/www/app/controllers/Graph.js

Please feel free to further elaborate, correct, and/or revise the function specification as more details about the calculations are understood.

simsong commented 9 months ago

We probably want to move all of the image processing modules into a subdirectory.

Evantso commented 7 months ago

May I know what I should do with the y-distance factor? In showResults() of Graph.js, I don't see how y-values are directly used in the calculations for circumnutation.

simsong commented 7 months ago

Since we now have actual data from actual plant videos in the database, I could create a better description here if you wish.

Can you run the app locally? You can also download a csv file from demo.planttracer.com

Evantso commented 7 months ago

Yes, can I have a new description? I'm not able to access demo.planttracer.com because I don't have a course key to register an account.

simsong commented 7 months ago

Yes, can I have a new description? I'm not able to access demo.planttracer.com because I don't have a course key to register an account.

Hi. You did not understand the page, which means that the page is not clear.

A screenshot of demo.planttracer.com is below. Why did you think that you needed a course key? Why did you think that you could not list on "list demo movies"? Would it be better for us to delete the second part? Please suggest text that would make this more clear.

Thanks.

image
Evantso commented 7 months ago

Thanks for updating the homepage. I can use the demo on my local device. It works.

simsong commented 7 months ago

I didn't make any changes to the page. I'm asking you if the current wording is clear, or if it should be clarified. What do you think? What is your opinion?

Evantso commented 7 months ago

I have been set up for automatic login and have access to all the features on the website, but I would be directed to the previous default screen when accessing the web app from an incognito window. To avoid confusion, perhaps it would be better to change the wording of "list demo movies" to "Try Demo?"

simsong commented 7 months ago

I can do that. I'm confused what you mean by "I have been set up for automatic login." Can you clarify that?

Evantso commented 7 months ago

What I meant is I'm not sure why but on my device I was not asked to log in but am able to access all the features of the webapp.

simsong commented 7 months ago

That's the whole point of demo mode!

Evantso commented 7 months ago

Oh, I see you might have verified me.

2024-01-25_20 39 21
Evantso commented 7 months ago

May I know what is currently being worked on for the web app? Can I help with any UI updates or the machine learning side (if it's currently being developed)? Did Mr. Barber discuss anything recently?

simsong commented 7 months ago

May I know what is currently being worked on for the web app? Can I help with any UI updates or the machine learning side (if it's currently being developed)? Did Mr. Barber discuss anything recently?

Hi. If you have questions, please open new issues. Thanks.

Evantso commented 7 months ago

Got it thanks.