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

Update public engagement statements based on collab cafes #272

Merged
merged 7 commits into from
Oct 12, 2023

Conversation

manics
Copy link
Member

@manics manics commented Oct 5, 2023

✅ Checklist

  • This pull request has a meaningful title.
  • If your changes are not yet ready to merge, you have marked this pull request as a draft pull request.

☑️ Maintainers' checklist

  • This pull request has had the appropriate labels assigned
  • This pull request has been added to the SATRE backlog project board
  • This pull request has been assigned to one or more maintainers

⤴️ Summary

Updates wording of the Public Involvement and Engagement, see issues below

🌂 Related issues

Closes

🙋 Acknowledging contributors

@manics manics added WP5 Community & Stakeholder Engagement proposed change A proposed change to the specification pillar: supporting Supporting capabilities labels Oct 5, 2023
@manics manics requested a review from a team October 5, 2023 12:59
Copy link
Member

@JimMadge JimMadge left a comment

Choose a reason for hiding this comment

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

Ah, I see now 😄.

I would prefer to keep the importance options minimal.

This also removes the consistent "You must", "You should", "You could" style of the statements.

Do we think it is reasonable to have the statement include "if"?
I think that came up in the evaluation we ran at the Turing with some requirements being "0, because we don't need to do this or the requirement is n/a".

@manics
Copy link
Member Author

manics commented Oct 5, 2023

I think the status needs to be clear, since we want to combine the scores to get an overall result
https://satre-specification.readthedocs.io/en/latest/evaluation.html#combining-scores
and I think we should move towards a more machine readable/generated spec #254

If we want to keep the statuses simple I vote for making it Recommended. @Katie-RDS how would you feel about making these statements Recommended so we don't need to have a conditional status, but making it clearer in the statement/guidance that it should be considered mandatory for public sector data?

@Katie-RDS
Copy link

I think the difficulty with making them recommended is that if it is a public sector one and they filter by mandatory they could tick all the boxes without that? But appreciate trying to maintain simplicty

@craddm
Copy link
Contributor

craddm commented Oct 6, 2023

Ah, I see now 😄.

I would prefer to keep the importance options minimal.

This also removes the consistent "You must", "You should", "You could" style of the statements.

Do we think it is reasonable to have the statement include "if"? I think that came up in the evaluation we ran at the Turing with some requirements being "0, because we don't need to do this or the requirement is n/a".

I agree that we should keep the importance options minimal - we really want to avoid having lots of caveats in those options. I think it's ok to make clear in the guidance that there are some statements that are more/less important in certain circumstances

@Katie-RDS
Copy link

I think that works that it is mandatory then in the statement it says 'if' so people can mark down N/A if it is N/A for them

@JimMadge
Copy link
Member

JimMadge commented Oct 9, 2023

I think that works that it is mandatory then in the statement it says 'if' so people can mark down N/A if it is N/A for them

This is the way we interpreted requirements when completing the Turing evaluation.

For example, 4.3.1 is N/A for us so we marked it as 2 as there is nothing for us to do there.

@manics manics force-pushed the public-engagement branch 2 times, most recently from 9720ba9 to 87a0535 Compare October 10, 2023 08:12
@manics
Copy link
Member Author

manics commented Oct 10, 2023

I've changed the statuses to Mandatory* (with the asterisk to make it clear there's more to it).

@manics
Copy link
Member Author

manics commented Oct 10, 2023

I've also added #290 for discussion of how to score non-mandatory items.

@manics manics requested a review from a team October 10, 2023 08:22
@manics
Copy link
Member Author

manics commented Oct 10, 2023

Following discussion today there's no consensus on whether we should make things conditionally mandatory, and what exactly "public sector data" means. We'll therefore return to this after the 1.0.0 release.

@drchriscole
Copy link
Contributor

The text for mandatory* should be about "personal data" not "public sector data".

docs/source/pillars/supporting.md Outdated Show resolved Hide resolved
docs/source/pillars/supporting.md Outdated Show resolved Hide resolved
docs/source/pillars/supporting.md Outdated Show resolved Hide resolved
@manics manics force-pushed the public-engagement branch from b08c3dd to 5a050c4 Compare October 12, 2023 14:53
@manics manics marked this pull request as ready for review October 12, 2023 14:53
@drchriscole
Copy link
Contributor

Discussed in detail with Katie, Antony, Chris and Simon.

@drchriscole drchriscole merged commit 8784f3a into sa-tre:main Oct 12, 2023
@manics manics deleted the public-engagement branch October 12, 2023 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pillar: supporting Supporting capabilities proposed change A proposed change to the specification WP5 Community & Stakeholder Engagement
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants