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

v0.3.1 #197

Merged
merged 52 commits into from
May 22, 2024
Merged

v0.3.1 #197

merged 52 commits into from
May 22, 2024

Conversation

jiajic
Copy link
Member

@jiajic jiajic commented May 21, 2024

GiottoClass 0.3.1

bug fixes

  • allow passing of additional params with setGiotto() with ...
  • spatShift() can now perform z shifts when start spatLocsObj has no z information
  • fix bug in joinGiottoObjects() after v0.3.0 where it looks for the now non-existent @largeImages slot
  • fix bug in .update_image_slot() after v0.3.0 where a NULL @largeImages slot will result in an error
  • fix bugs in spatShift() and rescale() methods for giotto when setting a default spat_unit and feat_type

enhancements

  • joinGiottoObjects() extent detection and xshift defaults now depend on ext() of the gobject instead of any images (when available)
  • joinGiottoObjects() now has a dry_run param for previewing where datasets will be spatially located after the join

new

  • as() conversion from giottoLargeImage to array
  • as.matrix() method for spatLocsObj()

jiajic and others added 30 commits May 13, 2024 11:51
replace wrap_msg -> print to avoid error printing list
- allow NULL inputs, which are interpreted as 0 shifts
- allow z manipulations even when z values are not originally in these objects to facilitate z stacking.
- add col ordering before outputs for zstacking
- update to be compatible now that @largeImages slot is gone
- object extent detection and xshift defaults no longer depend on images, but `ext()` of the `gobject`
- update xy shifts to use `spatShift()`
- fix image slot updates when no giottoLargeImages in object
- fix a print statement in createGiottoObjectSubcellular
- also fix error when `joinGiottoObjects` only has one spatLocsObj and a list input is expected
- fix get_giottoLargeImage() and get_giottoImage_MG
- GiottoData 0.2.11.0 changes the image name of the he image from he back to 'image'. Update the examples that have already been switched back to that name.
Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lintr found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

@jiajic jiajic merged commit 559135c into main May 22, 2024
2 of 3 checks passed
@jiajic jiajic mentioned this pull request May 22, 2024
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.

2 participants