Skip to content

Set correct virtual model name in NEST-SONATA documentation#2826

Merged
jessica-mitchell merged 4 commits intonest:masterfrom
nicolossus:fix_nestsonata_docs
Jun 23, 2023
Merged

Set correct virtual model name in NEST-SONATA documentation#2826
jessica-mitchell merged 4 commits intonest:masterfrom
nicolossus:fix_nestsonata_docs

Conversation

@nicolossus
Copy link
Member

@nicolossus nicolossus commented Jun 7, 2023

NEST-SONATA uses the spike_train_injector as virtual model (used to be spike_generator). This PR changes the name of the virtual model to the correct one in places that were overlooked when the switch of models took place. The PR also improves the formatting by removing trailing whitespaces.

@nicolossus nicolossus added S: Normal Handle this with default priority T: Maintenance Work to keep up the quality of the code and documentation. I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) labels Jun 7, 2023
@nicolossus
Copy link
Member Author

@jessica-mitchell The failing CI was due to a formatting error in master which was fixed by #2837. Could you retrigger the CI?

Copy link
Contributor

@jessica-mitchell jessica-mitchell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just one minor suggestion

nicolossus and others added 2 commits June 23, 2023 13:11
@jessica-mitchell
Copy link
Contributor

Merging with 1 reviewer as its doc related

@jessica-mitchell jessica-mitchell merged commit fa9eb7f into nest:master Jun 23, 2023
@nicolossus nicolossus deleted the fix_nestsonata_docs branch June 23, 2023 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) S: Normal Handle this with default priority T: Maintenance Work to keep up the quality of the code and documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments