Closed igreen1 closed 3 years ago
Ready for checklisting. Please notify @dondi when ready merge.
The demos worked as expected at first but after a while the nodes would start moving further apart from one another. Also, the higher number of nodes the more nodes would move apart instead of reaching equilibrium and stopping. Moreover, I noticed some of the options in Top/Bottom Dataset in Node column did not appear on the graph and it showed a message saying "Expression Database is loading". Furthermore, as I recall @igreen1 changed the code so that the direction arrows in D-pad would not get selected, but I was able to select on direction arrows.
good catch @ahmad00m that was an interesting bug. the center
function calls a restart on the simulation. In fact, looking at beta, @igreen1 was able to produce a similar bug by toggling 'restrict graph to viewport'. It causes the nodes to jostle for a long period of time.
This bug can be fixed by removing the restart of simulation in the center command, which does not appear to affect the simulation at all. This is fixed in commit 66e21a46d69b599debf5da71b7de40b432dc0ce6
Second commit should make it impossible to highlight for all computer types: commit d4e224950cf9dc27a4c863cfa684f0e5a1fd7ffa
@dondi should I add this 'no-highlight' to nodes as well? I'm editing the file anyways.
@igreen1 have been unable to find the reasoning behind the expression database error; however, it happens on the beta branch too so likely a result of needing access, which only the official website has. Will confirm in meetign
@dondi will look into network/access restrictions to verify whether the external expression database requests are behaving as expected.
There are some loose-end checkboxes in this PR but I’ll go ahead and merge and we can seek to resolve those as standalone issues.
Summary changelog: grnsight.styl changed to fix #905 by reducing size of button so overflow is unnecessary Changed button direction for #888 by moving camera in opposite direction (flipping signs) Changed restrict graph to viewport behaviour in #804. Now when restrict-graph is chosen, movement controls are not available and zoom is locked to 100%.
Pull Request Checklist