Ticket #10973 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Feature: Initial box splitting of MD Event data

Reported by: Anton Piccardo-Selg Owned by: Anton Piccardo-Selg
Priority: major Milestone: Release 3.4
Component: Framework Keywords:
Cc: Blocked By:
Blocking: #11320 Tester: Owen Arnold

Description

This ticket investigates the initial box splitting of n-dimensional MDEvent data to a grid of size 50n.

This initial splitting will be applied to a box of depth 0. After the first split, the split procedure should continue as usual.

With this ticket we want to avoid large outer boxes obstructing the view onto the workspace when rendering it in the VSI.

Attachments

TestInstruction.docx (15.1 KB) - added by Anton Piccardo-Selg 6 years ago.
boxSplit4D.py (5.9 KB) - added by Anton Piccardo-Selg 6 years ago.

Change History

comment:1 Changed 6 years ago by Anton Piccardo-Selg

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Anton Piccardo-Selg

  • Status changed from assigned to inprogress

Refs #10973 Force initial box split to desired number

Changeset: 2243fa879595b29bf57efee0aa1be48863665e1b

comment:3 Changed 6 years ago by Anton Piccardo-Selg

Refs #10973 Moving box splitting code to convertToMD algorithm

Changeset: 5fae8d8af9c21f041ad5d35bd68710906fa16ba5

comment:4 Changed 6 years ago by Anton Piccardo-Selg

Refs #10973 Fix for splitting

Changeset: f2a8b2805175977c857c46b81f0ccacf94bbe623

comment:5 Changed 6 years ago by Anton Piccardo-Selg

Refs #10973 Added unit tests for initial split

Changeset: ea4a0e19bfc01c87ab7a8d288ec858da339adb27

comment:6 Changed 6 years ago by Anton Piccardo-Selg

Refs #10973 Enable unit tests for box splitting

Changeset: f2e8234efeded37bab4d0131ee8cf88f5dd69865

comment:7 Changed 6 years ago by Anton Piccardo-Selg

Refs #10973 Extending splitting to 4D

Changeset: 575e5eb152b6d4563acb7ad706979ece26f9efbf

comment:8 Changed 6 years ago by Anton Piccardo-Selg

Refs #10973 Extending box splitting tests to 4D

Changeset: 63f873a4e2411266bb711868f6044d56ad977954

comment:9 Changed 6 years ago by Anton Piccardo-Selg

  • Status changed from inprogress to verify
  • Resolution set to fixed

This is being verified as pull request #378.

comment:10 Changed 6 years ago by Owen Arnold

  • Status changed from verify to verifying
  • Tester set to Owen Arnold

comment:11 Changed 6 years ago by Owen Arnold

  • Blocking 11320 added

Changed 6 years ago by Anton Piccardo-Selg

Changed 6 years ago by Anton Piccardo-Selg

comment:12 Changed 6 years ago by Owen Arnold

  • Status changed from verifying to closed

Merge pull request #378 from mantidproject/feature/10973_initial_box_splitting_of_MDEvent_data_sets

Initial box splitting of md event data sets

Full changeset: f939f6de94243b54723b3dc496943da3ef6e6216

comment:13 Changed 6 years ago by Owen Arnold

I've created a follow on ticket for this. Generally code looks OK. Tests are passing.

comment:14 Changed 5 years ago by Nick Draper

Somehow these slipped through without a resolution. Set to Fixed.

comment:15 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 11812

Note: See TracTickets for help on using tickets.