Skip to content
This repository has been archived by the owner on Jul 14, 2024. It is now read-only.

remove path munging #53

Closed
murphyk opened this issue May 17, 2022 · 1 comment
Closed

remove path munging #53

murphyk opened this issue May 17, 2022 · 1 comment
Assignees
@gileshd

Comments

@murphyk
Copy link
Member

murphyk commented May 17, 2022

https://github.com/probml/JSL/blob/main/jsl/lds/kalman_filter_test.py#L3

sys.path.insert(0, os.path.abspath(os.path.dirname(__file__)))

is bad style. Why is it needed?
@gileshd
Copy link
Collaborator

gileshd commented May 17, 2022

Yeah my bad sorry not necessary! Forgot to remove it after fiddling around with some testing stuff locally.

I'll make a PR to get rid of it and make the import use the absolute path.

This was referenced May 17, 2022
Merged
Open
@gileshd gileshd closed this as completed May 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@gileshd gileshd

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@murphyk @gileshd