Skip to content

Fixes #787 Move bImportData & bProcessData#788

Merged
MichaelStritt merged 5 commits intodevelopfrom
minor-#787_MoveChecks
Aug 19, 2021
Merged

Fixes #787 Move bImportData & bProcessData#788
MichaelStritt merged 5 commits intodevelopfrom
minor-#787_MoveChecks

Conversation

@MichaelStritt
Copy link
Contributor

Linked issue

#787

@MichaelStritt MichaelStritt requested a review from jan-petr August 18, 2021 12:37
@MichaelStritt MichaelStritt changed the title #787 ExploreASL_...: Minor revamp Fixes #787 Move bImportData & bProcessData Aug 18, 2021
@MichaelStritt MichaelStritt self-assigned this Aug 18, 2021
@MichaelStritt MichaelStritt added the minor improvements Fixes that don't change main pipeline functionality label Aug 18, 2021
@MichaelStritt MichaelStritt linked an issue Aug 18, 2021 that may be closed by this pull request
@MichaelStritt
Copy link
Contributor Author

Testing

Everything looks okay so far 👍

grafik

Copy link
Contributor

@jan-petr jan-petr left a comment

Choose a reason for hiding this comment

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

Very nice for ExploreASL_Master, but somehow weird in ImportMaster - see my comments...

@MichaelStritt MichaelStritt requested a review from jan-petr August 18, 2021 17:21
Copy link
Contributor

@jan-petr jan-petr left a comment

Choose a reason for hiding this comment

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

Looks perfect now!

@MichaelStritt MichaelStritt force-pushed the minor-#787_MoveChecks branch from e624e90 to 7ef05c3 Compare August 19, 2021 07:26
@MichaelStritt MichaelStritt merged commit 7ef05c3 into develop Aug 19, 2021
@MichaelStritt MichaelStritt deleted the minor-#787_MoveChecks branch August 19, 2021 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor improvements Fixes that don't change main pipeline functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move bImport and bProcess to ExploreASL_Master

2 participants