Skip to content

SRS test - #124

Closed
manny118 wants to merge 3 commits into
masterfrom
SRS_test
Closed

SRS test#124
manny118 wants to merge 3 commits into
masterfrom
SRS_test

Conversation

@manny118

Copy link
Copy Markdown
Contributor

No description provided.

@manny118 manny118 changed the title Srs test SRS test Jul 24, 2022
@lantz

lantz commented Aug 1, 2022

Copy link
Copy Markdown
Collaborator

How is this related to #123?

Also the tests are not passing.

@manny118

manny118 commented Aug 3, 2022

Copy link
Copy Markdown
Contributor Author

This is a test for the "srs_effect_model()" which we discussed some time ago. Yes, the tests fail since it needs the SRS flag in the link.py class to be on.

@manny118 manny118 closed this Aug 5, 2022
@manny118

manny118 commented Aug 5, 2022

Copy link
Copy Markdown
Contributor Author

This has been replaced with request.

@manny118 manny118 mentioned this pull request Aug 5, 2022
@lantz

lantz commented Aug 8, 2022

Copy link
Copy Markdown
Collaborator

@manny118 Remember that you don't need to make a new PR unless you want to. You can update this one by pushing to it (with --force if necessary.) Usually this is easier than creating a new PR and it also makes it easier to keep track of.

@lantz
lantz deleted the SRS_test branch August 8, 2022 21:07
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