Difference between revisions of "Talk:GRASS GIS Community Sprint Prague 2019"

From GRASS-Wiki
Jump to: navigation, search
(Participants and Reports)
(Also discussed)
 
(17 intermediate revisions by 5 users not shown)
Line 1: Line 1:
 
= Participants and Reports =
 
= Participants and Reports =
 +
 +
== Decisions made ==
 +
 +
* New startup: New GUI startup procedure and roadmap for implementing it
 +
** We spent many hours over the course of the sprint discussing different alternatives and finding a best way to implement the one we have chosen.
 +
* Migration to GitHub: Keep current tickets on track open, but open new issues on GitHub.
 +
** Keep current tickets on track open (i.e., valid and to be used), but disable opening new tickets. People involved in old tickets can continue discussing there.
 +
** Open new issues on GitHub and direct all users there.
 +
** We just need to clarify on mailing list that if you opened or are following an issue on Trac, you should continue there.
 +
** No migration of issues to GitHub is needed which makes this plan simple to execute.
 +
** Everybody is encouraged to be more "aggressive" about closing old tickets on Trac which are now invalid or no longer relevant. Opening new updated issue after that on GitHub is welcome.
 +
 +
== Also discussed ==
 +
 +
* Windows builds:
 +
** In the long run, we need CMake build with subsequent MSVC (Microsoft Visual C++) compatibility changes for smooth CI/CD builds on Windows and easier packaging on Windows.
 +
** We need to test Azure Pipelines for Linux build (needed for both Windows and conda-forge builds).
 +
** conda-forge packaging is need as an alternative (other related projects are already using it, current macOS builds are already based on Conda packages).
 +
** Standalone Windows installer should include Jupyter (Jupyter Notebooks and/or JupyterLab).
 +
* Python scripting: Python API needs to be consolidated before 8.0.
 +
** The pygrass naming needs to be resolved.
 +
** Current grass.pygrass.modules need to be strictly separated from ctypes dependencies.
 +
** The duplication between run_command() family and grass.pygrass.modules needs to be resolved.
 +
** The concept of running in different sessions needs to be explicitly supported and expressed in API.
 +
*** Use response to an unknown method call (__getattr__()) in implementation of a session object which then (in __getattr__()) passes the right env to the called function.
 +
*** This applies not only to module call wrappers, but to all functions which call modules in the background.
 +
*** ctypes-based API probably always need to use the current session.
 +
*** Temporal libraries are unclear.
 +
* Release terminology: Perhaps we should call releases just releases, not adding "stable" like now. After couple of minor versions (when the branch is widely tested), the specific minor release and subsequent patch releases would be called stable. For example, 7.8.0, 7.8.1, and 7.8.2 would be simply releases while 7.8.3 would be marked stable and each subsequent release such as 7.8.4 and 7.8.5 would be stable release as well.
 +
* Terminal started with GUI: Terminal should be removed or at least not shown by default on Windows. Other platforms need to be discussed, but it should be at least possible to configure GRASS GIS like that or start it with nohup ({{Trac|3295}}).
  
 
== Vaclav Petras ==
 
== Vaclav Petras ==
  
* Creating notes for New Startup discussion (https://trac.osgeo.org/grass/wiki/wxGUIDevelopment/New_Startup)
+
* Creating notes for new wxGUI startup discussion ([https://trac.osgeo.org/grass/wiki/wxGUIDevelopment/New_Startup#GRASSGISCommunitySprintPrague2019 wxGUI New Startup])
 +
** Created new wxGUI startup Roadmap: [https://trac.osgeo.org/grass/wiki/wxGUIDevelopment/New_Startup#PragueRoadmap New Startup Prague Roadmap]
 +
* Testing CMake build of GRASS GIS by rkanavath ([https://github.com/rkanavath/grass-ci github.com/rkanavath/grass-ci])
 +
* PR for CONTRIBUTING file ([https://github.com/OSGeo/grass/pull/237 GitHub PR 237])
 +
** Update Trac wiki Submitting/General ([https://trac.osgeo.org/grass/wiki/Submitting/General?action=diff&version=17 Submitting/General])
 +
* PR for sync keywords of r.null & comp. ([https://github.com/OSGeo/grass/pull/239 GitHub PR 239])
 +
* Participating in discussions and notes on these discussions in ''Decisions made'' and ''Also discussed'' above
  
 
== Martin Landa ==
 
== Martin Landa ==
Line 10: Line 46:
  
 
{{fig|osgeo4w-wxpython-4.0.7}}
 
{{fig|osgeo4w-wxpython-4.0.7}}
 +
 +
* GRASS 7.8.2RC1 [https://launchpad.net/~ubuntugis/+archive/ubuntu/ubuntugis-experimental/+packages?field.name_filter=grass&field.status_filter=published&field.series_filter=bionic UbuntuGIS Expr package] published
 +
 +
* GUI welcome screen discussion
 +
 +
{{fig|wxgui-startup-screen-discussion-prague|ext=jpg|wxGUI startup screen discussion|size=500}}
  
 
== Anna Petrasova ==
 
== Anna Petrasova ==
 +
* welcome screen discussion
 
* bug fixing
 
* bug fixing
 
** https://trac.osgeo.org/grass/ticket/3937
 
** https://trac.osgeo.org/grass/ticket/3937
 
** https://trac.osgeo.org/grass/ticket/3934
 
** https://trac.osgeo.org/grass/ticket/3934
 
** https://trac.osgeo.org/grass/ticket/3981
 
** https://trac.osgeo.org/grass/ticket/3981
* r.sim improvements - dynamic allocation
+
** [https://github.com/OSGeo/grass/commit/931490c3691c00c94b4388e3cfa81440ba342a08 wxGUI: fix GCP manager for py3]
 +
** [https://github.com/OSGeo/grass/pull/242 PR for wxGUI: fix WMS]
 +
* [https://github.com/OSGeo/grass/commit/f5adfb6bb4fd704bed17853b50715fd2936c4372 r.sim improvements - dynamic allocation]
 +
 
 +
== Ondrej Pesek ==
 +
 
 +
* README refactoring: https://github.com/OSGeo/grass/pull/231
 +
* bug fixing
 +
** https://github.com/OSGeo/grass/pull/234
 +
** https://github.com/OSGeo/grass/pull/241
 +
* Updating artificial neural network modules docs to correspond with GRASS 7.8 (see [https://github.com/OSGeo/grass-addons/compare/master@%7B12/02/2019%7D...master@%7B12/07/2019%7D github commits] overview)
 +
* Working on g.gui.gmodeler PyWPS export (see [https://github.com/ctu-yfsg/2017-d-grass-modeler-pywps/compare/master@%7B12/01/2019%7D...master@%7B12/07/2019%7D github commits] overview)
 +
 
 +
== Markus Neteler (remote) ==
 +
* https://github.com/OSGeo/grass/pull/236 (i.vi)
 +
* testing of https://github.com/OSGeo/grass-addons/pull/58 by Anika Bettge
 +
* some GitHub team management

Latest revision as of 10:41, 5 May 2020

Participants and Reports

Decisions made

  • New startup: New GUI startup procedure and roadmap for implementing it
    • We spent many hours over the course of the sprint discussing different alternatives and finding a best way to implement the one we have chosen.
  • Migration to GitHub: Keep current tickets on track open, but open new issues on GitHub.
    • Keep current tickets on track open (i.e., valid and to be used), but disable opening new tickets. People involved in old tickets can continue discussing there.
    • Open new issues on GitHub and direct all users there.
    • We just need to clarify on mailing list that if you opened or are following an issue on Trac, you should continue there.
    • No migration of issues to GitHub is needed which makes this plan simple to execute.
    • Everybody is encouraged to be more "aggressive" about closing old tickets on Trac which are now invalid or no longer relevant. Opening new updated issue after that on GitHub is welcome.

Also discussed

  • Windows builds:
    • In the long run, we need CMake build with subsequent MSVC (Microsoft Visual C++) compatibility changes for smooth CI/CD builds on Windows and easier packaging on Windows.
    • We need to test Azure Pipelines for Linux build (needed for both Windows and conda-forge builds).
    • conda-forge packaging is need as an alternative (other related projects are already using it, current macOS builds are already based on Conda packages).
    • Standalone Windows installer should include Jupyter (Jupyter Notebooks and/or JupyterLab).
  • Python scripting: Python API needs to be consolidated before 8.0.
    • The pygrass naming needs to be resolved.
    • Current grass.pygrass.modules need to be strictly separated from ctypes dependencies.
    • The duplication between run_command() family and grass.pygrass.modules needs to be resolved.
    • The concept of running in different sessions needs to be explicitly supported and expressed in API.
      • Use response to an unknown method call (__getattr__()) in implementation of a session object which then (in __getattr__()) passes the right env to the called function.
      • This applies not only to module call wrappers, but to all functions which call modules in the background.
      • ctypes-based API probably always need to use the current session.
      • Temporal libraries are unclear.
  • Release terminology: Perhaps we should call releases just releases, not adding "stable" like now. After couple of minor versions (when the branch is widely tested), the specific minor release and subsequent patch releases would be called stable. For example, 7.8.0, 7.8.1, and 7.8.2 would be simply releases while 7.8.3 would be marked stable and each subsequent release such as 7.8.4 and 7.8.5 would be stable release as well.
  • Terminal started with GUI: Terminal should be removed or at least not shown by default on Windows. Other platforms need to be discussed, but it should be at least possible to configure GRASS GIS like that or start it with nohup (trac #3295).

Vaclav Petras

Martin Landa

Osgeo4w-wxpython-4.0.7.png
  • GUI welcome screen discussion
wxGUI startup screen discussion

Anna Petrasova

Ondrej Pesek

Markus Neteler (remote)