added a preliminary classification class

[LuanVietNguyen]

It is just a startup class, Dr Johnson can look at it and decide whether he want to merge it, and then continue to work on it soon.

[stanleybak]

The structure looks generally right. I wouldn't merge it with the main line until there's more implementation, though. For collaboration on experimental features I think you can add a remote repository that another person forked rather than going through the main line.

Some thoughts:

• You're using a bitwise or operator '|' whereas I think you want a logical or '||'
• The enum name should be DynamicsClass or something like that, maybe DynamicsClassification. Dynamics by itself is too broad
• if e is not an Operation (for example, a Constant), the return value is null, which doesn't seem right. Or if it's an unknown Operation. The default value should probably be the most general category (NONLINEAR) or some unclassified value.
• This type of class is easy to test; when we merge with the main line we'd probably want to add a unit test for each class to SmallTests.java
• BaseComponent is discrete if variables.size() == 0, I think. That might make your classification task easier.
• Also, be sure to handle urgent modes (the urgent flag is true of flowDynamics is null.

[ttj]

Ok, we'll commit to Luan's branch for now, hopefully will not cause issues with other things being developed.

[ttj]

Ok, I added some basic classification into discrete or not with some basic unit tests in Luan's branch.

Luan: we'll continue to add changes for your stuff in this branch, then we'll merge them. If Christian start's writing code for this, please get him to pull from your fork.

[stanleybak]

Sounds good. Closing this pull request (issue another one once the implementation is completed, as discussed).