Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prepare release 1.6.1 #130

Merged
merged 59 commits into from
May 23, 2022
Merged

Prepare release 1.6.1 #130

merged 59 commits into from
May 23, 2022

Conversation

razman786
Copy link
Collaborator

@razman786 razman786 commented May 9, 2022

Tigger version 1.6.1 with support for Ubuntu 22.04:

Proposed steps:

  1. Process PR 37 for tigger-lsm first (Fixes angle error with mosaics tigger-lsm#37). PR 37 for tigger-lsm already includes a refined PR 36 (Fixes issue #35 - FITS CUNIT 'M/S' header. tigger-lsm#36) for convenience. This is what the newer features and testing of Tigger v1.6.1 has been worked upon.
  2. Please note I've not looked into or tested PR 34 for tigger-lsm (Add LSM to Reg converter tigger-lsm#34)
  3. Currently the setup.py of Tigger v1.6.1 expects PR 37 (at minimum) to form tigger-lsm v1.7.1 for processing mosaic and FITS files with header issues. Therefore release v1.7.2 or use tigger-lsm PR 37 for this PR.
  4. Process PR Closes issue #128 - FITS headers after arithmetic operations #132 .
  5. Process this PR for Tigger v1.6.1

N.B. you can test ARM64 Ubuntu 22.04 via qemu on x86 - it's slow and painful but does work, eventually.

EDIT: Fixed typo

Changes:

  • Supports Ubuntu 22.04
  • Beta support for Ubuntu 22.04 ARM64 (with PyQwt deb pkg) - it should allow a Mac M1 user to use Ubuntu 22.04 for ARM64 with UTM virtualisation on Mac OS X
  • Dependent on the latest tigger-lsm (i.e. PR 37)
  • Fixed live profile selection when using mosaic images (and tigger-lsm PR 37)
  • Fixed float errors with updated library API's
  • Re-enabled splash screen
  • Fixes venv pip3 bug
  • Updated URL's
  • GitHub Actions are now compatible with 'act' for running locally
  • Actions now test VENV installations on Ubuntu 20.04 and 22.04
  • Improved Action tests and logging
  • Failed Actions are processed by an error reporting step
  • Copyright notice years updated and redundant UTF-8 file declarations removed
  • Code was checked for consistency with v1.5.1 as part of Image arithmetic not handing NaNs / blanks correctly #129 and PR Closes issue #128 - FITS headers after arithmetic operations #132
  • Image zooming refined (trackpad and mouse)
  • Dockable widget handling improved, window and image sizes should remain better
  • Minor file formatting
  • Updated README.rst
  • Updated CHANGELOG.rst

…e install builds PyQWT from source on 22.04.
QPoint's have been used for drawImage. Sizing and indexes also cast to
ints where needed.
* Actions are now compatible with 'act' for running locally
* Added support for Ubuntu 22.04
* VENV installations are tested for Ubuntu 20.04 and 22.04
* Tests now confirm that the installation was successful
* Tests also include the loading of FITS and model files
* Flake8 tests were repositioned to the forefront
* Logging output was reduced and log files are shown upon error
* Added logging
* Added failed jobs summary and output
* Simplified check installation test
@razman786 razman786 added this to the 1.6.1 milestone May 9, 2022
@razman786 razman786 self-assigned this May 9, 2022
@razman786
Copy link
Collaborator Author

razman786 commented May 17, 2022

As a note, the recent failing tests are not due to Tigger, but the Actions VM's failing for 18.04 and 22.04 when downloading and installing deb packages via apt. Tests pass locally using 'act' as was the case with 20.04.

@razman786 razman786 changed the title WIP: Prepare release 1.6.1 Prepare release 1.6.1 May 22, 2022
@bennahugo
Copy link
Collaborator

ok to test

@bennahugo
Copy link
Collaborator

bennahugo commented May 23, 2022

Trying to load in an image with the menu I run across:

Welcome to Tigger 1.6.1
Please wait a second while the GUI starts up.
Detected screen size 1200x1920, usable area 1200x1890
Traceback (most recent call last):
  File "/home/hugo/workspace/tigger/TigGUI/Images/Manager.py", line 161, in loadImage
    layout.addWidget(_flabel, 0, 3)
AttributeError: 'NoneType' object has no attribute 'addWidget'

It does not happen when you load in images using the command line

@razman786
Copy link
Collaborator Author

Trying to load in an image with the menu I run across:

Welcome to Tigger 1.6.1
Please wait a second while the GUI starts up.
Detected screen size 1200x1920, usable area 1200x1890
Traceback (most recent call last):
  File "/home/hugo/workspace/tigger/TigGUI/Images/Manager.py", line 161, in loadImage
    layout.addWidget(_flabel, 0, 3)
AttributeError: 'NoneType' object has no attribute 'addWidget'

It does not happen when you load in images using the command line

I cannot replicate it. What version of Ubuntu are you testing with?

@bennahugo
Copy link
Collaborator

Running Ubuntu 18.04 with my previous testing environment

@razman786
Copy link
Collaborator Author

Ah ok that explains it. My Ubuntu 18.04 setup passed away along with the laptop it was on :( But I was able to replicate the issue. It seems that some Ubuntu's use the native file dialog by default and others don't... Give the latest commit a spin, it should work, and if the widget layout cannot be customised it will not show the preview pane.

@bennahugo bennahugo merged commit bdfe29a into master May 23, 2022
@bennahugo
Copy link
Collaborator

Ok thanks @razman786 I've run through it for a bit and I can't see anything majorly amiss. I will accept and merge this one!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants