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

Add tests for GVI-usage + IRS-usage #113

Closed
tph-thuering opened this issue Jun 18, 2015 · 6 comments
Closed

Add tests for GVI-usage + IRS-usage #113

tph-thuering opened this issue Jun 18, 2015 · 6 comments
Assignees
Milestone

Comments

@tph-thuering
Copy link
Contributor

As #56 is now implemented, write tests for it.

@tph-thuering
Copy link
Contributor Author

  • Implement changes in scenario converter 33_to_34

or

  • make usage an optional parameter.

The following tests are failing currently and need updates:
10 - 2ITNs (Failed)
15 - IRS30 (Failed)
23 - Rach5IC (Failed)
26 - VecFullTest (Failed)

Update: These tests work now since 4ddba28

@tph-thuering
Copy link
Contributor Author

Tests are not necessary to be changed, as they are optional now. see 4ddba28

@dhardy
Copy link
Contributor

dhardy commented Jun 20, 2015

I thought the add tests title was more about testing the new functionality than fixing compatibility issues?

@dhardy dhardy reopened this Jun 20, 2015
@tph-thuering
Copy link
Contributor Author

@dhardy You think they are needed? At first I did make <usage> mandatory (XSD: "minOccurs=1"), but changed it to optional (in XSD-context: minOccurs=0), so existing tests don't need to be changed and v33 scenarios won't need to be translated for this specific element.

So I think changing the tests are now obsolete in this case (I even chose this title).

Do you think we should create additional tests specifically for GVI- and IRS-usage?

@dhardy
Copy link
Contributor

dhardy commented Jun 22, 2015

Making them optional is a good idea; it'll save a bunch of complaints about my XML is broken!

In theory there should be tests for every feature, but to do this properly you need (a) to do a mini-experiment (probably at least pop 10000 and a few repetitions) to check the feature seems to behave in the right way, and (b) including the feature in some test scenario with a small pop size (new or existing) to test the feature doesn't get broken by future code changes. But for simple features I'm often lazy; it can be a lot of work.

@tph-thuering
Copy link
Contributor Author

I know there are a few test experiments which have <usage>0.8</usage> for ITN. I trust this behaves correct already and IRS + GVI-usage elements behave the same as the code looks the same and scaling of intervention effects is at the same code location for IRS + GVI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants