fix json parse bug in neuralNetworkData loadData

ml5js / ml5-next-gen

Repo for next generation of ml5.js: friendly machine learning for the web! 🤖

https://ml5js.org/

Other

94 stars 23 forks source link

fix json parse bug in neuralNetworkData loadData #207

Closed ziyuan-linn closed 1 month ago

ziyuan-linn commented 2 months ago

This PR fixes the bug in #206.

Changes:

Run formatted on NeuralNetwork
Fix the bug where JSON.stringify was necessarily called
Update the console message for readability

shiffman commented 1 month ago

Thank you for this @ziyuan-linn! I think an older version of ml5.js supported csv data as well? So that is something we could consider for the future. But I don't think it's a priority. If anyone feels it's a need or is excited to work on it, we can open an issue.