jherr029 / graphing-calculator Goto Github PK

Should know which line belongs to each function. Color code them with a key on each panel.

The function 'x' will keep itself on the graph after deletion, despite being deleted from both mFunctionList (belongs to FunctionAdapter mFunctionAdapter), and functions (belongs to GraphHandler graph). Tried in the most recent branch and it's not there anymore, but I'm still a little worried.

How to reproduce error:

1. Enter 'x' as the first function in the list.
2. Enter another function as the second function in the list.
3. Delete the first function.

If you try this and it turns out it's being removed properly, please test it with differing conditions. Like, different bounds, multiple graphs, with polar graphing (would be Θ in this case), and other stuff. I just need someone to verify if it's actually gone or not.

In the settings menu, the user should be able to toggle whether to use degrees or radians. The buttons are there, and change graphics, it's just they don't actually do anything to the graph and functions.

I also mentioned this to Tianyou and said he would work with AJ on it.

The step size input field for both X-Axis and Y-Axis don't do anything in the settings menu.

We agreed we can just let this pass, but if anyone wants to pick this up, feel free to do it.

Whenever you try to pinch the graph to zoom it, despite zooming and scaling being disabled on the GraphView, the graph resets its boundaries to a range unspecified in the class.

I mentioned this to Tianyou, and he said he would let AJ know. Don't know what's going on, but I'll leave this here for record-keeping.

The app will crash upon entering any of the symbols in the function list. The cfg parser will accept them as a valid function, since they are valid math symbols, and will crash at the ExpressionEvaluator. Since they're just constants, I assume they will be converted as so somewhere in the back end. I'll let you guys handle it.

How to reproduce:

1. Create a valid function with abs(), π, or 'e' in it.

The checkboxes in the settings menu should do the following things:

1. Grid: Make the grid on the graph visible or invisible.
2. X-Axis: Make the X-Axis on the graph visible or invisible.
3. Y-Axis: Make the Y-Axis on the graph visible or invisible.

Again, something we don't have to do, but could be done with someone looking for something to do. I actually don't know if this is possible with our GraphView API.