Revert "Revert "Merge branch 'feature/dev-o'""

This reverts commit aad564f.

README.md

@@ -1 +1,15 @@
+## Documentation
 https://ibllib.readthedocs.io/en/latest/
+
+## Requirements
+Requires Python 3.6 or higher.
+
+Requires Matlab R2014b or higher.
+
+## Development practices
+Gitflow.
+
+-   run tests
+-   flake8
+-   build documentation
+

docs/conf.py

@@ -17,6 +17,7 @@
 sys.path.insert(0, os.path.abspath('.'))
 sys.path.insert(0, os.path.abspath('../python/alf'))
 sys.path.insert(0, os.path.abspath('../python/openneurodata'))
+sys.path.insert(0, os.path.abspath('../python/openneurodata/oneibl'))
 sys.path.insert(0, os.path.abspath('../python'))
 print('Python %s on %s' % (sys.version, sys.platform))
 

python/openneurodata/oneibl/params_secret.py

@@ -0,0 +1,3 @@
+# TODO: not very happy with this state of affairs of having unencrypted passwords (keyring ?)
+ALYX_PWD = 'ThisIsNotTheProductionPassword!'
+HTTP_DATA_SERVER_PWD = 'ThoseAreOnlyForGitHub!'

python/openneurodata/oneibl/tests/test_one.py

@@ -7,6 +7,7 @@ class TestLoad(unittest.TestCase):
 
     def setUp(self):
         # Init connection to the database
+        # TODO force credentials on test server
         self.One = ONE()
 
     def test_load(self):
@@ -65,7 +66,6 @@ def test_list(self):
         self.assertTrue(isinstance(l[0], list) and isinstance(l[0], list))
 
     def test_search_simple(self):
-        # Test when the dataset type requested is not unique
         myone = self.One
         # Test users
         usr = ['Morgane', 'miles', 'armin']

python/requirements.txt

@@ -0,0 +1,3 @@
+dataclasses
+numpy
+requests

readthedocs.yml

@@ -0,0 +1,7 @@
+# .readthedocs.yml
+
+build:
+  image: latest
+
+python:
+  version: 3.6