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

WIP: Add Splitter() element #112

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

WIP: Add Splitter() element #112

wants to merge 4 commits into from

Conversation

lantz
Copy link
Collaborator

@lantz lantz commented Apr 11, 2022

Splitter() is a simple splitter element that currently does
not model coupling loss (although the split percents may
be adjusted to model it as desired.)

  • also remove dodgy intermediate signal state storage in
    OpticalSignal

  • also make assoc_loc_in/out more reliable by requiring at least
    one parameter

  • add tests/splittertest.py

@lantz lantz changed the title Add Splitter() element WIP: Add Splitter() element Apr 11, 2022
@lantz lantz force-pushed the splitter branch 4 times, most recently from d7192ff to 4910056 Compare April 12, 2022 08:00
Splitter() is a simple splitter element that currently does
not model coupling loss (although the split percents may
be adjusted to model it as desired.)

- also remove dodgy intermediate signal state storage in
  OpticalSignal

- also make assoc_loc_in/out more reliable by requiring at least
  one parameter

- add tests/splittertest.py
- force reinstall of mininet-optical package
- detect reused spans/amps in Link and raise exception
- add monitor to Splitter
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.

1 participant