2018-06-12 Meeting notes
Date
Attendees
Goals
- Review outstanding tasks
- Review any new tasks
- Plan work for week
Discussion items
Item | Who | Notes |
---|---|---|
Outstanding Tasks | Tim Whitlock (Deactivated) |
|
NHSBT | Roger has arranged a meeting with NHSBT on at 10:00am. This is to discuss the NHSBT matching process and fields that are required etc. with an aim to making the process smoother. | |
Coding | We continue to have issues with working code being changed because it has not been understood properly by the individual doing the changes. This has led to time being wasted and broken systems since dependencies could be broken. We need to realise we are all part of a team with different knowledge and communicate directly to ensure we understand what is happening. It is important to ensure you understand why something has been done before deciding to change it especially if it is not your code. This means talking to other team members and maybe delaying some work until this can be done. It was suggested that more use of pulls and branching would help clarify that these changes are being made. It was laso noted the repository for the UKRDC is gaining a lot of comments that say fixing test or flake8 but give no useful help. In general these sorts of issues shoul dbe fixed in local testing before uploading to the repository. foghorn_leghorn (Unlicensed) will add some notes to the coding documentation about this. It was noted that Validation codebase will not currently support his due to the way the current code is built.
|
Action items
- George Swinnerton to sort issues with the nurture dashboard upload database cron jobs
- George Swinnerton to run a script to re load the latest file received by PV from the archive to try and deal with the missing results. These will be sent using PV_Direct.
- foghorn_leghorn (Unlicensed) to add documentation about use of pull and branching and testing before checking in code to reduce risks of breaking code base and creating a repository full of maningless flake8/fix test still comments.
- foghorn_leghorn (Unlicensed) to continue working on QBL counts ready for next validation meeting.
- rapolas (Unlicensed) To finish the work on the survey report code and making the PAM api calls a module
- rapolas (Unlicensed) finish the new Rituximab screens