Skip to content

[ISSUE-7] Moved all model-specific defaults from Global.h to appropriate model classes#910

Merged
CodersRepo merged 2 commits intoDhruvaDevfrom
issue-7-global-model-default-cleanup
Feb 12, 2026
Merged

[ISSUE-7] Moved all model-specific defaults from Global.h to appropriate model classes#910
CodersRepo merged 2 commits intoDhruvaDevfrom
issue-7-global-model-default-cleanup

Conversation

@CodersRepo
Copy link
Copy Markdown

…classes

Closes #7

Description

Moved all model-specific defaults from Global.h to appropriate model classes

Checklist (Mandatory for new features)

  • Added Documentation
  • Added Unit Tests

Testing (Mandatory for all changes)

  • GPU Test: test-medium-connected.xml Passed
  • GPU Test: test-large-long.xml Passed

@CodersRepo CodersRepo self-assigned this Feb 8, 2026
@CodersRepo CodersRepo marked this pull request as draft February 8, 2026 01:34
@CodersRepo CodersRepo marked this pull request as ready for review February 8, 2026 02:03
@CodersRepo CodersRepo requested a review from stiber February 8, 2026 02:04
Copy link
Copy Markdown
Contributor

@stiber stiber left a comment

Choose a reason for hiding this comment

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

Mostly comments, possibly not requiring change.

@CodersRepo CodersRepo requested review from stiber and removed request for stiber February 9, 2026 18:22
@CodersRepo CodersRepo merged commit 0769e6e into DhruvaDev Feb 12, 2026
2 checks passed
W_[iEdg] = edgSign(type) * 10.0e-9;
type_[iEdg] = type;
tau_[iEdg] = DEFAULT_tau;
tau_[iEdg] = AllSpikingSynapses::DEFAULT_tau;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't think this needs the scope specification here.

@stiber stiber deleted the issue-7-global-model-default-cleanup branch March 13, 2026 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants