Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Transfer other repos as needed. #3

Open
pv33 opened this issue Aug 11, 2020 · 4 comments
Open

Transfer other repos as needed. #3

pv33 opened this issue Aug 11, 2020 · 4 comments
Assignees
Labels
documentation Improvements or additions to documentation enhancement New feature or request

Comments

@pv33
Copy link
Member

pv33 commented Aug 11, 2020

Some other ivaMatlib code might be sourced. Create the mirror repo here in ivaPy and translate the code. Maintain a list in this comment for what those libraries/repos were:

  • XX
@pv33 pv33 added documentation Improvements or additions to documentation enhancement New feature or request labels Aug 11, 2020
@pv33 pv33 added this to the Neon target tracking milestone Aug 11, 2020
@pv33
Copy link
Member Author

pv33 commented Aug 11, 2020

Make sure to include doxygen formatting that tracks the formatting in the Matlab file.
Some things might need to change. We can discuss.

@pv33
Copy link
Member Author

pv33 commented Aug 15, 2020

@ruinianxu
This issue pretty much says that the idea is to mirror Matlab code to python and have ivaMatlibs and ivapy populated with similar code as needed. Can you explain to me what was not obvious about my requests that you did not know where to find things?

@ruinianxu
Copy link

@pv33
Dear Dr. Vela,

I think there was some confusings before. I was added to the repo named [Surveillance] but not in the repo of ivaMatlibs. I thought you want me to transfer code from [Surveillance] to ivay. But I can't find any classes fit the request you want.

I think everything is clear now. The repo where codes actually locate is here. Am I correct?

Best
Ruinian Xu

@pv33
Copy link
Member Author

pv33 commented Aug 20, 2020

The repositories to transfer are explicitly noted in the ivalab.loadLibraries. One exception is that I may not have put "improcessors" in the list since I added later for completeness and consistency with other similar classes.

Did you read the about.md file that I created in the testing subdirectory of overheadTracker?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants