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

[Task] Imp SANS #928: remove FilteredTimeSeriesProperty object #46

Open
9 tasks
glass-ships opened this issue Dec 17, 2024 · 1 comment
Open
9 tasks

Comments

@glass-ships
Copy link
Member

Description:

This class isn't used and is confusing people. Remove it from mantid.

$ rg FilteredTimeSeriesProperty -l
Framework/PythonInterface/mantid/kernel/CMakeLists.txt
Framework/PythonInterface/mantid/kernel/src/Exports/FilteredTimeSeriesProperty.cpp
Framework/PythonInterface/test/python/mantid/kernel/CMakeLists.txt
Framework/PythonInterface/test/python/mantid/kernel/FilteredTimeSeriesPropertyTest.py
Framework/DataHandling/test/LoadISISNexusTest.h
docs/source/api/python/mantid/kernel/StringFilteredTimeSeriesProperty.rst
docs/source/api/python/mantid/kernel/Int32FilteredTimeSeriesProperty.rst
docs/source/api/python/mantid/kernel/Int64FilteredTimeSeriesProperty.rst
docs/source/api/python/mantid/kernel/FloatFilteredTimeSeriesProperty.rst
docs/source/api/python/mantid/kernel/BoolFilteredTimeSeriesProperty.rst
Framework/Algorithms/src/GetAllEi.cpp
Framework/API/test/LogManagerTest.h
Framework/Kernel/CMakeLists.txt
Framework/Kernel/src/PropertyManager.cpp
Framework/Kernel/test/PropertyManagerTest.h
Framework/Kernel/test/FilteredTimeSeriesPropertyTest.h
Framework/Kernel/inc/MantidKernel/FilteredTimeSeriesProperty.h
Framework/Kernel/src/FilteredTimeSeriesProperty.cpp
qt/python/mantidqt/mantidqt/widgets/samplelogs/model.py

List of Tasks:

  • Understand Story: Talk to owner
  • Address Necessary Revisions to the Story
  • Capture Significant Design Decisions
  • Gather All Required Test Data

  • Write & Run Unit Tests
  • Implement Story (Update unit test as needed)
  • Write & Run Integration Tests
  • Submit Pull Request
  • Report Completion During Status Meeting
@glass-ships
Copy link
Member Author

This issue was migrated from code.ornl.gov.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant