-
Notifications
You must be signed in to change notification settings - Fork 24
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
Renames NeuroField to NFTsim #146
Renames NeuroField to NFTsim #146
Conversation
@stuart-knock I didn't find the reference to NeuroField on the top-level README. Which line? |
@pausz The wiki reference, line 68: https://github.com/BrainDynamicsUSYD/nftsim/wiki/How-to-contribute-code-to-NeuroField |
@stuart-knock Thank you -- I completely missed that, I only did a case sensitive search for neurofield (searching for references to the old repo name). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All instances of neurofield have been removed (from source code, documentation, binaries and matlab documents). Seems ok to merge.
I added documentation in the +nf functions; fixed some minor errors and updated TODOs. See #147 for future +nf tasks. |
#141
Completes the internal renaming of NeuroField to NFTsim.
With one exception, there should be no remaining use of any form of neurofield in the repository.
The only remaining use of NeuroField is a reference to a wiki page in the top level README. The wiki should be renamed before we make this change.