Skip to content

Download fourier data#23

Open
MuLx10 wants to merge 3 commits into
Jezzamonn:mainfrom
MuLx10:download-fourier-data
Open

Download fourier data#23
MuLx10 wants to merge 3 commits into
Jezzamonn:mainfrom
MuLx10:download-fourier-data

Conversation

@MuLx10

@MuLx10 MuLx10 commented Oct 18, 2019

Copy link
Copy Markdown

Added Download button for draw zone. It would help the user to recreate the diagram from the Fourier series data.
Data is saved in the below described format in json.

{"amplitude":[],"phase":[],"frequency":[]}

Capture

fourier-data.json

Added Download button for draw zone. It would help the user to recreate the diagram from the fourier series data.
Remove zero frequency.
@Jezzamonn

Copy link
Copy Markdown
Owner

Hi! First of all, nice job in digging into the code and getting an understanding of how it works. I like the idea of letting people be able to access this data!

Some high-level questions to be resolved though is:

  • How do we make sure this doesn't interfere with reading the page? I'd rather not have a button in a prominent place as I think that might be a bit distracting when you're reading the article. Maybe there's a better way to trigger this?
  • What's the best file format for the data (is there a reason you picked that format?)
  • How do we set this up for the other languages (this page is not just in English!)

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