dali-lab / pine-beetle-backend

Visualizing data and predicting outbreaks of Southern Pine Beetles across twenty US states. Built in the DALI Lab.
https://spbpredict.com
0 stars 0 forks source link

queryFetch should not always use parseInt, should not string join to array #139

Open jeffzyliu opened 1 year ago

jeffzyliu commented 1 year ago

add logic to search for a ., if so, parseFloat instead?

for array thing: we will update frontend to properly send an array of counties so we don't need to do string split anymore bc express and axios hadle that

jeffzyliu commented 1 year ago

https://code.whatever.social/questions/20355876/how-to-send-integers-in-query-parameters-in-nodejs-express-service#67434444

JSON.parse?

EDIT nvm this isn't needed I think

jeffzyliu commented 1 year ago

Also this should be removed/changed in general since https://windix.medium.com/parsing-array-from-get-parameters-in-node-js-express-ee03d8f2ddb1

jeffzyliu commented 1 year ago

revision: specifyqueryfetch needs complete overhaul. queryfetch needs to know how to parseFloat.