Ticket #9727 (assigned)

Opened 6 years ago

Last modified 5 years ago

Review MaxEnt Testing Situation

Reported by: Peter Parker Owned by: Raquel Alvarez Banos
Priority: major Milestone: Backlog
Component: Muon Keywords:
Cc: adrian.hillier@… Blocked By:
Blocking: Tester:

Description

Currently we are providing the MaxEnt algorithm to users via the script repo. This has its benefits, but unfortunately it means that the unit tests we have for it are only ever run manually.

We need a way to have these tests run automatically, since we have already had our first instance where a change elsewhere has had a detrimental knock-on effect for MaxEnt.

A couple of options are open to us:

  • Provide the ability to run script repository tests, though this would require some thought about where data would be stored, etc.
  • Bring MaxEnt in to the main repository, perhaps when we're sure the algorithm is not going to change much and we won't need to keep on adding binary .pyd files.

Change History

comment:1 Changed 6 years ago by Peter Parker

A further problem caused by providing the algorithm via the script repo is that when we make a change that is not backwards compatible with earlier versions of Mantid, we break the algorithm for anybody using old versions. They may even have "auto-update" checked, in which case even if they don't update themselves the algorithm could still break.

comment:2 Changed 6 years ago by Nick Draper

  • Status changed from new to assigned

comment:3 Changed 6 years ago by Nick Draper

Moved to the backlog at the code freeze of R3.3

comment:4 Changed 6 years ago by Nick Draper

  • Milestone changed from Release 3.3 to Backlog

comment:5 Changed 6 years ago by Anders Markvardsen

  • Owner changed from Peter Parker to Raquel Alvarez Banos

Just happen to notice this one. Rachel started maxent help request investigation yesterday from James

comment:6 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 10569

Note: See TracTickets for help on using tickets.