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.

Sign up for GitHub

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

Jump to bottom

[Type] Check instantiation for system-collection and timestepper #405

Draft
wants to merge 12 commits into
base: update-0.3.3
Choose a base branch
from
Draft

[Type] Check instantiation for system-collection and timestepper #405

wants to merge 12 commits into from

Conversation

skim0119
Copy link
Collaborator

@skim0119 skim0119 commented Jun 30, 2024 •
edited
Loading

skim0119 added 12 commits June 28, 2024 22:00
@skim0119
move explicit steppers in experimental. partially resolve GazzolaLab#402
2e834f2
@skim0119
assign bound for generic typing for OperatorGroup
1bb9f3a
@skim0119
use OperatorGroup to handle constrain_values and constrain_rates oper…
76d46f6 
…ations
@skim0119
use OperatorGroup to handle callback operator
d88df67
@skim0119
test: fix unittest according to change in removing operator list
9519294
@skim0119
type: specify more details of the operator parameter spec
cddfa60
@skim0119
move explicit_stepper system, memory, and protocols into experimental
4bca3a0
@skim0119
type: type-check operator group implementation against its protocol
abd5ba5
@skim0119
partially resolve GazzolaLab#402: move analytical.py into tests
3847502
@skim0119
type: check instantiation for system collection and stepper
efcf453
@skim0119
Reduce protocol requirement for system-collection
3a60b38
@skim0119
Merge branch 'wip/401' into type/instance-check
3d7251d
@skim0119 skim0119 self-assigned this Jun 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@armantekinalp armantekinalp Awaiting requested review from armantekinalp armantekinalp will be requested when the pull request is marked ready for review armantekinalp is a code owner

@bhosale2 bhosale2 Awaiting requested review from bhosale2 bhosale2 will be requested when the pull request is marked ready for review bhosale2 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@skim0119 skim0119

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@skim0119