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

Implement metrics #78

Merged
merged 10 commits into from
Oct 4, 2019
Merged

Implement metrics #78

merged 10 commits into from
Oct 4, 2019

Conversation

radka-j
Copy link
Contributor

@radka-j radka-j commented Oct 1, 2019

See issue

  • implemented the loss_of_separation() metric
  • will implement sector exit metric once it has been fully implemented in bluebird

@radka-j radka-j changed the title [WIP] Add aircraft_separation() metric [WIP] Implement metrics Oct 1, 2019
tallamjr added a commit that referenced this pull request Oct 2, 2019
It was discovered in builds from #79 and #78 a failure has
occurred on Travis

Error:
    TypeError: attrib() got an unexpected keyword argument 'convert'

It appears that pytest seems to have the package attrs as a dependency.

REF:
    https://stackoverflow.com/questions/58189683/typeerror-attrib-got-an-unexpected-keyword-argument-convert

	modified:   requirements.txt
@radka-j
Copy link
Contributor Author

radka-j commented Oct 3, 2019

TO DO:

  • add tests
  • skip test_async_request() in Travis

@radka-j radka-j changed the title [WIP] Implement metrics Implement metrics Oct 4, 2019
@radka-j radka-j merged commit 17dded7 into master Oct 4, 2019
@radka-j radka-j deleted the add_metrics branch October 11, 2019 12:21
tallamjr referenced this pull request in project-bluebird/bluebird Oct 16, 2019
It was discovered in builds from https://github.com/alan-turing-institute/dodo/pull/79
and https://github.com/alan-turing-institute/dodo/pull/78 a failure has occurred on Travis

Command:
    $ pytest -rs --ignore=./tests/integration ./tests

Error:
    TypeError: attrib() got an unexpected keyword argument 'convert'

It appears that pytest seems to have the package attrs as a dependency.

REF:
    https://stackoverflow.com/questions/58189683/typeerror-attrib-got-an-unexpected-keyword-argument-convert

    modified:   requirements.txt
tallamjr referenced this pull request in project-bluebird/bluebird Oct 16, 2019
It was discovered in builds from https://github.com/alan-turing-institute/dodo/pull/79
and https://github.com/alan-turing-institute/dodo/pull/78 a failure has occurred on Travis

Command:
    $ pytest -rs --ignore=./tests/integration ./tests

Error:
    TypeError: attrib() got an unexpected keyword argument 'convert'

It appears that pytest seems to have the package attrs as a dependency.

REF:
    https://stackoverflow.com/questions/58189683/typeerror-attrib-got-an-unexpected-keyword-argument-convert

    modified:   requirements.txt
tallamjr referenced this pull request in project-bluebird/bluebird Oct 16, 2019
It was discovered in builds from https://github.com/alan-turing-institute/dodo/pull/79
and https://github.com/alan-turing-institute/dodo/pull/78 a failure has occurred on Travis

Command:
    $ pytest -rs --ignore=./tests/integration ./tests

Error:
    TypeError: attrib() got an unexpected keyword argument 'convert'

It appears that pytest seems to have the package attrs as a dependency.

REF:
    https://stackoverflow.com/questions/58189683/typeerror-attrib-got-an-unexpected-keyword-argument-convert

    modified:   requirements.txt

[SQUASH] Moving pytest bump to requirements-dev instead

Since pytest is only a requirement for development, it should reside in
'requirements-dev.txt' file instead of the main 'requirements.txt' file
	modified:   requirements-dev.txt
	modified:   requirements.txt
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