fix/S3 uploads and new browser tabs handling

[rugeli]

Problem

What is the problem this work solves, including
our current conditional statement if env.config_data.get("upload_results", env.config_data.get("number_of_packings", 1) <= 1): on main prioritizes checking upload_results with its default value set to False. So the check will be skipped if upload_results is not explicitly set to True in the config file

Solution

What I/we did to solve this problem
changed the if-statement to evaluate upload_results and number_of_packings individually

with @meganrm

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)

[codecov-commenter]

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (fee423f) 98.52% compared to head (baf613d) 98.52%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #203   +/-   ##
=======================================
  Coverage   98.52%   98.52%           
=======================================
  Files          16       16           
  Lines         476      476           
=======================================
  Hits          469      469           
  Misses          7        7           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[github-actions]

Packing analysis report

Analysis for packing results located at cellpack/tests/outputs/test_spheres/spheresSST

Ingredient name	Encapsulating radius	Average number packed
ext_A	25	236.0

Packing image

Distance analysis

Expected minimum distance: 50.00
Actual minimum distance: 50.01

Ingredient key	Pairwise distance distribution
ext_A

[mogres]

Looks good to me! Having the upload_results set to False by default would not have worked with the previous check.