-
Notifications
You must be signed in to change notification settings - Fork 190
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
Fix/tables #284
Fix/tables #284
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Alright, this also fixes #280. Here was my method:
length = 2048
dat = []
out = open("out.txt","w")
t = np.linspace(0,2*np.pi,num=length,endpoint=False)
out.write("\t")
for i in range(len(t)):
#out.write(str(round(127*np.sin(t[i])))+", ") # for the sin
#out.write(str(round(-127+127*2.*i/(length-1)))+", ") # for the saw
# for the tri
if (i<=length/4):
dat.append(i*127./length*4)
elif (i<=length*3/4):
dat.append(127-(i-length/4)*(127*2)/(length/2))
else:
dat.append(-127.+127.*(i-length*3/4)/length*4)
if (np.amin(data)==-128):
data = (data-np.amin(data)) * (np.amax(data)-(np.amin(data)+1)) / (np.amax(data) - np.amin(data)) + np.amin(data)+1 where What do you think? |
This comment was marked as outdated.
This comment was marked as outdated.
Rewrote tables with standardized script Added standard tables generating script
d1ea293
to
8fcb269
Compare
For reproducibility purposes, I added the script I used to regenerate the "standard" tables. Plus the tables automatically generated. |
This comment was marked as outdated.
This comment was marked as outdated.
Memory usage change @ 860c186
Click for full report table
Click for full report CSV
|
Any opinion? Can I merge this? |
Great work Tom. OK by me to merge. |
Fix #279