| | 163 | == Progress reports == |
| | 164 | |
| | 165 | == Report for weeks 1 and 2 == |
| | 166 | |
| | 167 | '''1) What did I complete?'''\\ |
| | 168 | The first simple implementation of the history browser panel (peek at https://github.com/OSGeo/grass/pull/3234) had some flaws, mainly in terms of class hierarchy, so a bigger intervention was needed. So, week one was all about making the code neater and more logical. I dived into improving the code quality and logic as part of the PR https://github.com/OSGeo/grass/pull/3342 since in this PR the problematic structure was fully shown. At the beginning of the second week, this PR was successfully merged. Big thanks to Tomas Zigo for the suggestions and testing! |
| | 169 | |
| | 170 | In the middle of the second week, I started to prepare the biggest PR of this project https://github.com/OSGeo/grass/pull/3365 which focuses on two things: first, it changes the way how the new history log is stored - newly it is going to be the JSON format. Second, it shows the info about commands - execution time, runtime duration, status, region settings, and if a mask was set. We originally planned to show this info in a new Command Info dialog, but during the implementations, we decided that it would be cooler to have it right at the bottom of the history browser. |
| | 171 | |
| | 172 | '''2) What am I going to achieve for the next two weeks?'''\\ |
| | 173 | I'm sticking with https://github.com/OSGeo/grass/pull/3365 for the next two weeks. Firstly, the JSON format structure needs revisions. Secondly, I will rework how the info shows up - the info will be shown directly at the bottom of the history browser after calling OnItemSelected event. As this PR is the most important part of the project, it will probably require a couple of iterations. |
| | 174 | |
| | 175 | '''3) Is there any blocking issue?'''\\ |
| | 176 | No, it isn't. |
| | 177 | |
