pip install missing resources package

[j-faria]

pip install missing resources package

[njcuk9999]

Done.

I've never got this ModuleNotFoundError so I guess you aren't using: pip install -U -e . as per the readme instructions, is that correct?
My guess is that when the you install via pip and keep the git repo editable this doesn't matter (as pip doesn't move the directory to the site-packages)

[j-faria]

Thank you! 👍🏼

My guess is that when the you install via pip and keep the git repo editable this doesn't matter (as pip doesn't move the directory to the site-packages)

Exactly, with the -e it still works because it picks up the "local" directory. I was using

pip install https://github.com/njcuk9999/lbl/archive/main.zip 

[j-faria]

Oops, closed the issue too soon.
The data_str_readme.md file still doesn't get moved to site-packages which makes the copy_readme function fail.

Maybe it would be easier to hardcode the "data structure read me" in lbl_misc.py?

[njcuk9999]

I don't want to hard code this it isn't the point. There should be a way to write a manifest of files that are copied over.

Just out of curiosity is there a reason you can't use pip install -U -e . for now?

[j-faria]

I don't want to hard code this it isn't the point. There should be a way to write a manifest of files that are copied over.

Agree!
I think it might be working with the new manifest file.

Just out of curiosity is there a reason you can't use pip install -U -e . for now?

No strong reason, but I'm using lbl within a CI / action and wanted to specify it as a dependence instead of git cloneing and pip installing locally.
More generally, requiring a user to clone the repository and have a local copy may not be feasible in some cases (although I'm very much guilty of that myself!). Since it's relatively easy to pip install from the repo directly, I thought it would be nice to have the option.

[njcuk9999]

Yer it was the plan eventually to use pip install only but we are still in development and not having the git easily updatable is a pain (especially when we don't change the version for every small change, which means pip wont update lbl).

The MANIFEST.in should work now though.