ethz-asl / data-driven-dynamics

Data Driven Dynamics Modeling for Aerial Vehicles
Other
100 stars 15 forks source link

Could not install dependencies #29

Closed Jaeyoung-Lim closed 3 years ago

Jaeyoung-Lim commented 3 years ago

Problem Description I could not install the python dependencies as specified in this package

Steps to reproduce

$ cd Tools/parametric_model
$ pip3 install -r requirements.txt 
Collecting h5py>=3.2.1 (from -r requirements.txt (line 1))
  Could not find a version that satisfies the requirement h5py>=3.2.1 (from -r requirements.txt (line 1)) (from versions: 2.2.1, 2.3.0b1, 2.3.0, 2.3.1, 2.4.0b1, 2.4.0, 2.5.0, 2.6.0, 2.7.0rc2, 2.7.0, 2.7.1, 2.8.0rc1, 2.8.0, 2.9.0rc1, 2.9.0, 2.10.0, 3.0.0rc1, 3.0.0, 3.1.0)
No matching distribution found for h5py>=3.2.1 (from -r requirements.txt (line 1))
manumerous commented 3 years ago

I could not reproduce that error. What kind of system do you try to install the dependencies on? I use ubuntu 20.04 LTS, python 3.8, and pip 20.0.2.

manumerous commented 3 years ago

But I think we might be able to kick that dependency out of the project. That was a library that was used in the file nik gave me in the beginning. Currently it is not used and I don't think at the moment that we will need HDF5 if we stick to yaml files.

Jaeyoung-Lim commented 3 years ago

@manumerous I am on Ubuntu 18.04 I think this is exactly why we need automated tests to verify that the instructions work. Otherwise we need to be explicit on exactly which the system(only) works

Could you document in the readme the setup process + instructions on how to run/simulate a vehicle from a uLog?(including the ulog generation)

manumerous commented 3 years ago

Yes sure it is a good point to document this.

Let's leave this issue open as a reminder. I will do this once I have the model validation. Maybe it would also be a good point to reflect on the structure of the project together at some point.

manumerous commented 3 years ago

Hi @Jaeyoung-Lim did downgrading the version of h5py solve the problem for you? Could you install every dependence afterwards?

Jaeyoung-Lim commented 3 years ago

Nope it fails with pyproj :) I added you as a collaborator you can see the results

Jaeyoung-Lim commented 3 years ago

Was fixed in https://github.com/ethz-asl/data-driven-dynamics/pull/55