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

Expand transparency and explainability mitigations #32

Merged
merged 1 commit into from
Jan 8, 2024
Merged

Conversation

anssiko
Copy link
Member

@anssiko anssiko commented Jan 8, 2024


Preview | Diff

@anssiko
Copy link
Member Author

anssiko commented Jan 8, 2024

(Please note WebSHAP already links to this Ethical Principles document. Here we create a backlink.)

@anssiko anssiko merged commit 022a486 into main Jan 8, 2024
1 check failed
@anssiko anssiko deleted the explainability branch January 8, 2024 08:47
@anssiko
Copy link
Member Author

anssiko commented Jan 8, 2024

@dontcallmedom CI deploy failed https://github.com/webmachinelearning/webmachinelearning-ethics/actions/runs/7445084552/job/20252666527, can you look into this? Builds clean AFAICT. Thanks!

@dontcallmedom
Copy link
Collaborator

@anssiko
Copy link
Member Author

anssiko commented Jan 8, 2024

Much thanks! Is https://www.w3.org/pubrules/ still the best tool to test for TR compliance?

Note to self: Roll up a CLI command to that takes a .bs file as an argument to test against the Specberus REST API. Unless someone has made that already.

@dontcallmedom
Copy link
Collaborator

At least in this case, the deploy failure wasn't linked to pubrules per se and had been detected by the CI at the Pull Request level https://github.com/webmachinelearning/webmachinelearning-ethics/actions/runs/7444503961/job/20251010838

That being said, it looks like spec-prod can be configured to do more pubrules checking pre-deployment if you want to make sure more errors get caught.

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