Skip to content

Some minor issues in Makefile #69

Description

@torfjelde

I had to make the following changes to the setup:

  1. git clone in https://github.com/oxcsml/Rmap/blob/fdbdf5bb2c29e73c23141536926a8cf6d222545f/Makefile#L6 failed because folder already existed.
  2. It seems to me like https://github.com/oxcsml/Rmap/blob/fdbdf5bb2c29e73c23141536926a8cf6d222545f/Makefile#L7 should actually be pip install -e . dataprocessing, no? At least the current setup failed due to not finding setup.py but changing to what I suggest fixes it and make preprocess-data works just fine after.
  3. If make environment fails for some reason, then it won't be possible to execute make environemt again without removing the first line (https://github.com/oxcsml/Rmap/blob/fdbdf5bb2c29e73c23141536926a8cf6d222545f/Makefile#L5). No biggie, but might be something you want to change when making it public.

Seems like data-generation process ran without issues after fixing the above though, so super-nice!

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions