Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
    Long Vu
    @tlvu
    @nikola-rados In addition for notebooks to play the role of integration tests (all components working together, aka how a typical real user actually interact and experience the system), the notebooks also serve as tutorial notebooks, so free tutorial and testing the tutorials means they are ensured to always be up-to-date. See https://github.com/Ouranosinc/PAVICS-e2e-workflow-tests#description for more infos.
    So basically view the unit tests as service for the developers while the notebooks as service for the real end users. They are complementary, not duplicate of each other.
    Nils Hempelmann
    @nilshempelmann
    @tlvu OK. I got it. well the unit test will focus on the function not on the processes anymore, that will be covered by the notebooks than
    Nik Radoš
    @nikola-rados
    @tlvu ok cool, thanks!
    Nils Hempelmann
    @nilshempelmann
    @tlvu I included a test-notebooks in the flyingpigeon make file
    If I got it right, that the client side birdy needs to be installed in the flyingpigoen env as well to run the notebook tests. Is there a way to keep server-side and client-side environments seperated?
    Long Vu
    @tlvu
    @nilshempelmann Finch have a requirements_dev.txtwhere additional dev related dependencies are specified: https://github.com/bird-house/finch/blob/4e9d2baaad0166a76c3e49481934c1eb56af007a/requirements_dev.txt#L7-L9
    Nils Hempelmann
    @nilshempelmann
    @tlvu D'accord.
    Nils Hempelmann
    @nilshempelmann
    @tlvu I made it with the notebook tests... almost :-)
    Do you have a trick to avoid this error:
    assert reference_output == test_output failed:
    
      '<img src="ht...width="400"/>' == '<img src="ht...width="400"/>'
      - <img src="http://127.0.0.1:8093/outputs/STATUS_FILE/tmpirlo_k7d.png" width="400"/>
      ?                                                        ^^  ----
      + <img src="http://127.0.0.1:8093/outputs/STATUS_FILE/tmplwz7aplo.png" width="400"/>
      ?                                                        ^^^^^^
    different output filenames on purpose. To keep the notebook also as the original idea for documentation and examples, the files should be shown.
    @tlvu @cehbrecht We should integrate the notebook-test also in the main documentaion:
    https://birdhouse.readthedocs.io/en/latest/dev_guide.html#writing-tests
    Long Vu
    @tlvu

    @nilshempelmann

    @tlvu I made it with the notebook tests... almost :-)
    Do you have a trick to avoid this error:

    assert reference_output == test_output failed:
    
      '<img src="ht...width="400"/>' == '<img src="ht...width="400"/>'
      - <img src="http://127.0.0.1:8093/outputs/STATUS_FILE/tmpirlo_k7d.png" width="400"/>
      ?                                                        ^^  ----
      + <img src="http://127.0.0.1:8093/outputs/STATUS_FILE/tmplwz7aplo.png" width="400"/>
      ?                                                        ^^^^^^

    I'll push an regex update soon.

    Long Vu
    @tlvu
    @nilshempelmann new regex pushed.

    @tlvu @cehbrecht We should integrate the notebook-test also in the main documentaion:
    https://birdhouse.readthedocs.io/en/latest/dev_guide.html#writing-tests

    LOL that Writing Tests section is empty for now. Happy to chime in about testing notebooks if someone start that section.

    Nils Hempelmann
    @nilshempelmann
    @tlvu @huard I understand more and more how you did the notebooks integration tests.
    will start writing the doc section. Good moment to fill this gap :
    bird-house/birdhouse-docs#46
    Nils Hempelmann
    @nilshempelmann
    @tlvu Are you familliar with Failed to open UDUNITS-2 XML unit database. even if udunits2 is available in the environment.
    Long Vu
    @tlvu
    @nilshempelmann no, I think you added that udunits2 library so you probably have more knowledge than me.
    MacPingu
    @cehbrecht
    @nikola-rados Is you thunderbird working now? I just checked the master branch and tested “hello” with birdy … works. But … you need to use Python 3.7 or Python 3.6. PyWPS is not working with Python 3.8:
    geopython/pywps#508
    I would use the conda environment for heavy dependencies like netcdf etc.
    I have pinned Python 3.7 in the conda environment, like in Emu:
    https://github.com/bird-house/emu/blob/master/environment.yml
    MacPingu
    @cehbrecht
    @tlvu do you know the GeoHealthCheck monitoring service?
    https://geohealthcheck.org/
    Long Vu
    @tlvu
    @cehbrecht wow that GeoHealthCheck presentation looks great !
    Long Vu
    @tlvu
    LOL I am the first WPS demo on the demo site https://demo.geohealthcheck.org/resource/297?lang=en, it's Finch on my test server.
    My Finch failed the monitoring because of this existing bug bird-house/birdhouse-deploy#14
    Nik Radoš
    @nikola-rados
    @cehbrecht master branch doesn't really have anything on it atm. Its just the cookiecutter with the name thunderbird.
    Nils Hempelmann
    @nilshempelmann
    @nikola-rados if you want, join the next VC: https://github.com/bird-house/bird-house.github.io/wiki/Meetings#next-vc
    it is interesting to know what you are doing and easier to support in terms of questions
    Nik Radoš
    @nikola-rados
    @nilshempelmann thank you for the invite!
    Nils Hempelmann
    @nilshempelmann
    @/all please update the doodle for the upcomming VC: https://github.com/bird-house/bird-house.github.io/wiki/Meetings#next-vc
    Nils Hempelmann
    @nilshempelmann
    @/all I fixed the time_date for the next VC:
    Monday 04.MAY 15:00-16:00h UTC (https://doodle.com/poll/2d43eytbsqnc47n3)
    David Huard
    @huard
    I'd like to set a milestone for July 1st. Suggestions on a name ?
    Nils Hempelmann
    @nilshempelmann
    @huard You mean a City name for the release notes?
    matrixbot
    @matrixbot
    Long test from Riot
    Long Hi all, I am simply trying to consolidate all my messaging (Slack, Gitter) into one client Riot/Matrix, to see if it works well. So this matrixbot is posting for me when I send my message from Riot client.
    David Huard
    @huard
    @nilshempelmann Yes, and start thinking about what we want to achieve in the next release.
    Nils Hempelmann
    @nilshempelmann
    @huard In the not so long array of suggestions is 'Niamey'. Capital of Niger and location of ACMAD, with whom we started a fragile seed of exchange.
    David Huard
    @huard
    Good for me !
    In this spirit, we could include translations to french for some of our processes.
    Nils Hempelmann
    @nilshempelmann
    tout a fait, c'est une autre lien avec Niamey :-).
    MacPingu
    @cehbrecht
    I have added “Niamey” as summer release:
    https://github.com/orgs/bird-house/projects/16
    … and there is Quebec for autumn:
    https://github.com/orgs/bird-house/projects/15
    David Huard
    @huard
    Thanks !
    David Huard
    @huard
    @cehbrecht When testing PyWPS processes, the object created by "client_for" does not seem to be able to fetch files from the localhost server. The solution so far has been to configure the test server with the CFG_FILE, which replaces the localhost by file:/// We then open the file by skipping the file:// section. I think it would be easier for testing if we could use owslib.wps.WPSExecution objects to parse outputs. Everything is there, it's only missing support for local paths in retrieveData. Makes sense ?
    David Huard
    @huard
    MacPingu
    @cehbrecht

    @huard We can do this … using owslib to retrieve the local files. In test mode pywps does not start a service … so there is no localhost. The default output url in pywps is a file://:
    https://github.com/geopython/pywps/blob/e942ada5a74d51fdeef38bbc65fe5092f06042ce/pywps/configuration.py#L78

    Maybe this gets overwritten by the default emu config.

    MacPingu
    @cehbrecht
    The gotomeeting link to the birdhouse VC was updated:
    https://app.gotomeeting.com/?meetingId=751237301
    David Huard
    @huard
    @nilshempelmann @cehbrecht In flyingpigeon, the setup.cfg for pytest hard codes the tests directory:
    [tool:pytest]
    addopts =
        --strict
        --tb=native
        tests/
    Could I use instead
    testpaths = tests
    MacPingu
    @cehbrecht
    I did not try … when we have a better configuration we can also update the template.
    Nils Hempelmann
    @nilshempelmann
    @huard starting finch results in:
    WARNING:PYWPS:not implemented: phase
    WARNING:PYWPS:not implemented: phase
    WARNING:PYWPS:not implemented: phase
    WARNING:PYWPS:not implemented: phase
    WARNING:PYWPS:not implemented: phase
    ...