PiRSquared17 / stoqs

Automatically exported from code.google.com/p/stoqs
GNU General Public License v3.0
0 stars 0 forks source link

Review STOQS schema change #52

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Branch name: schemachangeResources

Purpose of code changes on this branch:

To evolve the STOQS schema to enable classification of measurements. (See Issue 
49.)

When reviewing my code changes, please focus on:

Utility of new tables and whether additional association tables are needed. 
Schema evolution is something we don't want to do too often, so if there are 
other model changes that would be good please propose them now.

After the review, I'll merge this branch into: 

default

Original issue reported on code.google.com by MBARIm...@gmail.com on 2 Jun 2014 at 3:11

GoogleCodeExporter commented 9 years ago
Specific change set to comment on:

https://code.google.com/p/stoqs/source/detail?r=3477bfa7d7ff5caa13f820054cc3d54f
3b2fdb14&name=schemachangeResources

Original comment by MBARIm...@gmail.com on 2 Jun 2014 at 3:19

GoogleCodeExporter commented 9 years ago
Added ResourceResource table with this change set:

https://code.google.com/p/stoqs/source/detail?r=7b6403973910090a3f95b86d8b26b125
a7e4ec15&name=schemachangeResources

Original comment by MBARIm...@gmail.com on 4 Jun 2014 at 7:12

GoogleCodeExporter commented 9 years ago
After thinking more about how training and test sets are build I think that we 
need to have these association tables:

- MeasuredParameterResource
- SampledParameterResource

rather than:

- MeasurementResource
- InstantPointResource

Every instance I can think of for training, test, and classification data 
requires links to actual data values, not to the spatial or temporal data 
associated with the data values.

Original comment by MBARIm...@gmail.com on 5 Jun 2014 at 5:50

GoogleCodeExporter commented 9 years ago
Implemented above comment with these change sets:

https://code.google.com/p/stoqs/source/detail?r=2f9f6df4dbc9abf24497fd1bf876764d
9ad146c3&name=schemachangeResources
https://code.google.com/p/stoqs/source/detail?r=2f2f6f87169a0ce614e50a2dc270e394
931cf7b2&name=schemachangeResources

Ready now to test with a made up training set of data based on classification 
of sediment/dinoflagellates/diatoms by salinity using this view from the STOQS 
UI:

http://odss.mbari.org/canon/stoqs_september2013/query/?permalink_id=N4IgxgNglmDW
DOIBcBtEUB2AHArgFxTwE8sBTAXgB0QAnAQwBMoB7agXRQzoFsLr4AFswDuDOnjp147FADc6EHHxDww4
vKRrsQAGnTZ8hEsvpNWIDl15UQWOvV4aa8APoAPGfMXKAzABZtPUxcAmIyG1MWGStlOwdSJ1ciTwUlGx
8ARkD9EKNw6kjzSx5Y%2BxLElzAU7xsAVmyAYgZmU2YAAgAjfDxmDAA6eB6AR3gAWh6AcwmIUiQwXolM
eAAKambW6gBKXRzDMOUwAVI4DuYPCxQoBhtBEVH5lqYuDTH4LHujk7Ps4L3jG0Ox1gp3OHCuNyEwnuzE
emHEpDG3HegK%2BoJ2v1C%2FwKjCiFxiNgA8mQMAAZOhETTwPoAYQWNGYED6ZIpNAAysIoHhATQXLUAA
wuDpSUjMynVNLUNoAUVZACUAJJtAlgUh0DA%2FAyY%2FLgT7A74XcHUImkUnkyk0ukMplmtkcrlHHn8l
wYlxtVkm%2BAtNp4ehwaQWdGavIHXUgmSGkDG00sqm0jA%2Bq2i22c7m8gUu730ABWxx6NCgCI1uX2AN
D%2BrB1yNxKTsctjKT7JTDrTzs1rsG4hYGH9bEDxaxOqBYYNlcj1Zttfj9PrNsb9s0LYzrJ4WBmbSwzE
weB7fb%2B2pRetBl1HAC8YdweqQ3Bp470cFgxBoi3uQ0Py8ebFh3vAVzNRhBmDUPAux3Ng9DoMB5hoJg
1WQNB5ggCA6CweBSFGOgcFkHYEKQlC0IYfMADMnGwhlcNQ0ZpiuTRSMQ5CKO4GF8wwCZaPItDBCgLA2P
otDhDoWRSCohgaL0HDeNGdZGGYHi8MklppNk1CAAUykcTRVPiEiwJACQOkQVA0AaN5OwUAAVOh9N0Pkw
KMmgEWYHAaBVCyrJ0GydCMuJyg0tSEk0VzEHc2yQGM95H0kSCEXgZphAwADGD0oKPKMpFJPESKVXgGKR
Hi5hEss5KQoaDQkQUiBAustgdI7Gg8FGUhuAMkAMh8AB2ABOWoMj8WoACY2oADkGjq%2Bj6jI%2BTanY
TQYBqmuQFr2o6gA2Pw%2BWW5a%2Br8PxRtqPa%2FB2Wr6pErA8AEBbRj8MaMmmjBZpOs6FsGq7BuWnZu
GQ%2B4TRIpBQAA%2BLxGQP7emQUZWtqZ6%2BT65a%2Bh6vlntqPQkLwZBtp6jJBoyMa2r8VqAF89DPZh
uGQDIMgJkAPveJCYzgkBpTlRVlVVDAdjdD0vR9CCEDZrmczAPMCyC%2BmO2A3phbdX9SHXTd40QNg8aA
AA

and the attached image.

Original comment by MBARIm...@gmail.com on 6 Jun 2014 at 10:01

Attachments:

GoogleCodeExporter commented 9 years ago
Further comments on this issue will be in Issue 49.

Original comment by MBARIm...@gmail.com on 17 Jun 2014 at 3:13