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

Added citation.cff #173

Closed
wants to merge 6 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 22 additions & 0 deletions citation.cff
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
cff-version: 1.0.0
message: "If you use this software, please cite it as below."
authors:
- family-names: Clark
given-names: Andrew
- family-names: Foster
given-names: Michael
orcid: https://orcid.org/0000-0001-8233-9873
- family-names: Somers
given-names: Richard
orcid: https://orcid.org/0000-0002-1101-9722
- family-names: Turner
given-names: Robert
jmafoster1 marked this conversation as resolved.
Show resolved Hide resolved
orcid: https://orcid.org/0000-0002-1353-1404
- family-names: Walkinshaw
given-names: Neil
- family-names: Wild
given-names: Christopher
title: "Causal Testing Framework"
version: 3.1.0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cwild-UoS can this (and below) be automatically updated somehow?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After a fair bit of digging around I can't find any tools or workflows to currently do this. I've found a good discussion on this in another project.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's tricky. Might need to be done by hand for now. Similar problem for ORDA releases.

doi: https://doi.org/10.48550/arXiv.2209.00357
date-released: 2023-03-30
6 changes: 3 additions & 3 deletions tests/specification_tests/test_metamorphic_relations.py
Original file line number Diff line number Diff line change
Expand Up @@ -106,9 +106,9 @@ def test_should_cause_metamorphic_relations_correct_spec_one_input(self):
self.data_collector = SingleInputProgramUnderTestEDC(
self.scenario, self.default_control_input_config, self.default_treatment_input_config
)
causal_dag.graph.remove_nodes_from(['X2', 'X3'])
adj_set = list(causal_dag.direct_effect_adjustment_sets(['X1'], ['Z'])[0])
should_cause_MR = ShouldCause('X1', 'Z', adj_set, causal_dag)
causal_dag.graph.remove_nodes_from(["X2", "X3"])
adj_set = list(causal_dag.direct_effect_adjustment_sets(["X1"], ["Z"])[0])
should_cause_MR = ShouldCause("X1", "Z", adj_set, causal_dag)
should_cause_MR.generate_follow_up(10, -10.0, 10.0, 1)
test_results = should_cause_MR.execute_tests(self.data_collector)
should_cause_MR.test_oracle(test_results)
Expand Down
3 changes: 1 addition & 2 deletions tests/specification_tests/test_variable.py
Original file line number Diff line number Diff line change
Expand Up @@ -133,7 +133,6 @@ class Var(Variable):
var = Var("v", int)
self.assertEqual(var.typestring(), "Var")


def test_copy(self):
ip = Input("ip", float, norm)
self.assertTrue(ip.copy() is not ip)
Expand Down Expand Up @@ -173,7 +172,7 @@ def test_neg(self):
self.assertEqual(str(-self.i1), "-i1")

def test_pow(self):
self.assertEqual(str(self.i1 ** 5), "i1**5")
self.assertEqual(str(self.i1**5), "i1**5")

def test_le(self):
self.assertEqual(str(self.i1 <= 5), "i1 <= 5")
Expand Down