Ticket #10770 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Run SendUsage on startup

Reported by: Peter Peterson Owned by: Nick Draper
Priority: blocker Milestone: Release 3.3
Component: Framework Keywords:
Cc: Blocked By: #10559
Blocking: Tester: Peter Peterson

Description

Now that the algorithm exists. Run it when mantidplot starts.

Should it run when mantid is imported from python?

Change History

comment:1 Changed 6 years ago by Nick Draper

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Nick Draper

  • Status changed from assigned to inprogress

re #10770 send usage data on startup

Changeset: ed22e47338316b062aa0f56e39de5ce4ab311403

comment:3 Changed 6 years ago by Nick Draper

  • Owner changed from Peter Peterson to Nick Draper
  • Status changed from inprogress to verify
  • Resolution set to fixed

to test

A hard one to test unless you can log into the usage data server site. But hte code change is simple maybe a review and smoke test is enough.

  1. Startup mantidplot with both (in mantid.user.properties)
 algSendStartupUsage = 0
 and
 algSendStartupUsage = 1

comment:4 Changed 6 years ago by Nick Draper

solve merge conflict to develop re #10770

Merge branch 'feature/10770_run_send_usage_on_startup' into develop

Conflicts:

Code/Mantid/Framework/Geometry/src/Instrument/Goniometer.cpp Code/Mantid/Framework/PythonInterface/mantid/geometry/src/Exports/Goniometer.cpp Code/Mantid/Framework/PythonInterface/test/python/mantid/geometry/GoniometerTest.py

Changeset: 6468463691e90a65e2f333044c6e394b9f443337

comment:5 Changed 6 years ago by Pete Peterson

Re #10770. SendUsage should always be run.

Changeset: d9b2f71bd2a490fbe61f1fdfa85bd100e592d2a0

comment:6 Changed 6 years ago by Peter Peterson

  • Status changed from verify to verifying
  • Tester set to Peter Peterson

comment:7 Changed 6 years ago by Pete Peterson

  • Status changed from verifying to closed

Merge branch 'feature/10770_run_send_usage_on_startup'

Full changeset: a269c91385501c430016afce5bd35cfbf50c46a6

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 11612

Note: See TracTickets for help on using tickets.