261 all chapters black box model comments #285
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Separated out the bullet on testing to emphasise the point about testing pre-trained models
…ical work in the engagement and scoping section.
14 tasks
|
@lmadavies I have reviewed and think it's a good set of ammendments. @rebecca-wodcke-gad @zarbspace - can you review ? |
|
@Hurstharrier I've also reviewed and it looks good to me. Only query is if everyone will understand the term data drift? |
|
OFFICIAL
Hi Rebecca
Good spot on data drift! I will see if I can write something to clarify.
Alec
***@***.***
Alec Waterhouse
Data Scientist
Energy statistics, Analysis Directorate
Mobile: 077 899 47785
3-8 Whitehall Place, London SW1A 2AW
https://www.gov.uk/energysecurity<https://www.gov.uk/government/organisations/department-for-energy-security-and-net-zero>
Follow us on social media:
Twitter<https://x.com/energygovuk> | LinkedIn<https://www.linkedin.com/energysecurity> | Facebook<https://www.facebook.com/energygovuk> | Instagram<https://www.instagram.com/energygovuk>
OFFICIAL
From: rebecca-wodcke-gad ***@***.***>
Sent: 06 January 2025 12:46
To: best-practice-and-impact/aqua_book_revision ***@***.***>
Cc: Waterhouse, Alec (Energy Security) ***@***.***>; Mention ***@***.***>
Subject: Re: [best-practice-and-impact/aqua_book_revision] 261 all chapters black box model comments (PR #285)
@Hurstharrier<https://github.com/Hurstharrier> I've also reviewed and it looks good to me. Only query is if everyone will understand the term data drift?
-
Reply to this email directly, view it on GitHub<#285 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AWLAA5SEU23LD6VV3PKYRID2JJ3JPAVCNFSM6AAAAABTV7WYB2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNZTGA2DCMZUGE>.
You are receiving this because you were mentioned.Message ID: ***@***.******@***.***>>
|
zarbspace
reviewed
Jan 6, 2025
zarbspace
reviewed
Jan 6, 2025
zarbspace
reviewed
Jan 6, 2025
| @@ -45,7 +45,7 @@ Artificial Intelligence (AI) attempts to simulate human intelligence using techn | |||
|
|
|||
| ## Black box models {.unnumbered} | |||
|
|
|||
| Black box models internal workings are not visible, easily understood, or succinctly explained. These models take input and produce output without providing clarity about the process used to arrive at the output. [Artificial Intelligence](#Artificial Intelligence) models (including [Machine Learning](#Machine Learning)) are the most common type of black box models used today. Other forms of black box models may arise in future. | |||
| Black box models internal workings are not visible, easily understood, or succinctly explained. These models take input and produce output without providing clarity about the process used to arrive at the output, this includes proprietary models with protected intellectual property. [Artificial Intelligence](#Artificial Intelligence) models (including [Machine Learning](#Machine Learning)) can often be considered a type of black box models. Other forms of this type of black box models may arise in future. | |||
There was a problem hiding this comment.
Suggest a separate sentence for the proprietary models point (which is well made!). A suggestion: "Proprietary models with protected intellectual property which means methods and processes are not fully transparent are also black boxes."
There was a problem hiding this comment.
No other comments - I think this looks good apart from my nit picking points :-)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Addresses the points made in issue 261 relating to black box models from DESNZ and The Turing Institute. These covered a number of chapters where comments were separately made about black box models and AI.
Please check: