HEnquist / camillagui

GNU General Public License v3.0
9 stars 1 forks source link

Develop #35

Closed HEnquist closed 3 years ago

HEnquist commented 3 years ago

Here is a mixed list of things that would be nice to sort out for the next release.

HEnquist commented 3 years ago

ping @JWahle

JWahle commented 3 years ago

I'll have a look this weekend.

JWahle commented 3 years ago

Plots use easier numbers on x-scale, but they have too many zeros and get drawn on top of each other. There are more numbers than needed, half can be removed. And the ones above 1000 can be shortened, 1000 -> 1k etc

I already removed a lot of the labels in the gui-style branch. I had a look at it again and changed it to just have labels for 10, 100, 1k, 10k. How about this? (This change is only in the new gui-style branch on this repo) Filter-plot-new

"Config" box on the files tab lists everything in the config dir. Show only .yml and .yaml?

I also thought about this, however I have already uploaded files to the wrong folder (filters in the config folder and vice versa) and I like, that I don't have to ssh to the machine to clean up. Do you see any cases, where it would be beneficial to have some residual files in the config folder? If not, I would suggest, we leave it, as it is.

The upload buttons in File change symbol after an upload, and never change back. I think it's enough that the new file shows up in the list.

Seems reasonable - especially after I changed the GUI to update immediately after an upload. I pushed a change with this to develop.

There seems to be a limit of about 1MB for the files uploaded via the Files tab. This is pretty big, but a long filter in f64 could be larger than that. (when running with "npm start", check a production build too).

I'll have a look at this - not sure, what causes this.

Clicking on a file in the Files tab should open the link in a new browser tab

Fixed in the develop branch.

Saving a config to file should not set it as active. It's perfectly valid to save to file and not having it as active. See also HEnquist/camillagui-backend#22

Will be fixed in the ticket, you mentioned.

HEnquist commented 3 years ago

Took another look now.

All in all it's a huge improvement compared to the previous release, thank you for doing all this!

JWahle commented 3 years ago

You're welcome - thank you for CDSP :)

I increased the 1MB limit to 1GB - this should be enough for "most" filters. The fix is in the backend's develop branch.

The only point still open is HEnquist/camillagui-backend#22 and that is in the backend. So I think, we're ready to merge here.

JWahle commented 3 years ago

@HEnquist please check my latest commits and merge, if you find nothing suspicious. I'll take care of HEnquist/camillagui-backend#22 soon.