Skip to content

Conversation

ka-sarthak
Copy link
Collaborator

@ka-sarthak ka-sarthak commented Apr 7, 2025

  • Removes the schemas, app, example uploads, tests related to AutoXRD.

These will be implemented in nomad-auto-xrd plugin

@ka-sarthak ka-sarthak linked an issue Apr 7, 2025 that may be closed by this pull request
@coveralls
Copy link

Pull Request Test Coverage Report for Build 14310112011

Details

  • 9 of 9 (100.0%) changed or added relevant lines in 1 file are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage decreased (-3.0%) to 85.366%

Files with Coverage Reduction New Missed Lines %
tests/conftest.py 1 76.0%
Totals Coverage Status
Change from base Build 13723810177: -3.0%
Covered Lines: 35
Relevant Lines: 41

💛 - Coveralls

@ka-sarthak ka-sarthak force-pushed the 63-remove-autoxrd-schemas branch from d74cf33 to 9b00dbd Compare April 18, 2025 17:24
@ka-sarthak ka-sarthak requested a review from Copilot April 18, 2025 17:24
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes all AutoXRD-related code (schemas, models, apps, and example uploads) from the main repository in preparation for its re-implementation in the dedicated nomad-auto-xrd plugin.

  • Removed AutoXRD schema definitions and normalization routines.
  • Removed AutoXRD app and plugin entry point implementations.
  • Updated pyproject.toml to remove AutoXRD-related entry points.

Reviewed Changes

Copilot reviewed 26 out of 39 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/nomad_analysis/example_uploads/init.py Removed AutoXRD example upload entry point
src/nomad_analysis/auto_xrd/auto_xrd_analysis.py Removed schema for AutoXRD analysis
src/nomad_analysis/auto_xrd/auto_xrd.py Removed AutoXRD normalization and model schema
src/nomad_analysis/auto_xrd/init.py Removed plugin entry point definitions for AutoXRD
src/nomad_analysis/apps/auto_xrd.py Removed AutoXRD models app configuration
src/nomad_analysis/apps/init.py Removed AutoXRD app entry point
pyproject.toml Removed AutoXRD-related entry points
Files not reviewed (13)
  • src/nomad_analysis/example_uploads/auto_xrd/All_CIFs/AllCuxPXSx_CollCode22451.cif: Language not supported
  • src/nomad_analysis/example_uploads/auto_xrd/All_CIFs/AllCuxPXSx_CollCode24530.cif: Language not supported
  • src/nomad_analysis/example_uploads/auto_xrd/All_CIFs/AllCuxPXSx_CollCode29705.cif: Language not supported
  • src/nomad_analysis/example_uploads/auto_xrd/All_CIFs/AllCuxPXSx_CollCode412240.cif: Language not supported
  • src/nomad_analysis/example_uploads/auto_xrd/All_CIFs/AllCuxPXSx_CollCode430200.cif: Language not supported
  • src/nomad_analysis/example_uploads/auto_xrd/All_CIFs/AllCuxPXSx_CollCode628635.cif: Language not supported
  • src/nomad_analysis/example_uploads/auto_xrd/All_CIFs/AllCuxPXSx_CollCode628636.cif: Language not supported
  • src/nomad_analysis/example_uploads/auto_xrd/All_CIFs/AllCuxPXSx_CollCode628639.cif: Language not supported
  • src/nomad_analysis/example_uploads/auto_xrd/All_CIFs/AllCuxPXSx_CollCode656975.cif: Language not supported
  • src/nomad_analysis/example_uploads/auto_xrd/All_CIFs/AllCuxPX_CollCode15056.cif: Language not supported
  • src/nomad_analysis/example_uploads/auto_xrd/All_CIFs/AllCuxPX_CollCode16247.cif: Language not supported
  • src/nomad_analysis/example_uploads/auto_xrd/All_CIFs/AllCuxPX_CollCode16841.cif: Language not supported
  • src/nomad_analysis/example_uploads/auto_xrd/All_CIFs/AllCuxPX_CollCode198370.cif: Language not supported

@ka-sarthak ka-sarthak merged commit 0cdfe3b into develop Apr 18, 2025
5 checks passed
@ka-sarthak ka-sarthak deleted the 63-remove-autoxrd-schemas branch April 18, 2025 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove AutoXRD schemas
2 participants