Skip to content

Added AbstractPeriodicGridDistribution#288

Open
FlorianPfaff wants to merge 3 commits intomainfrom
feature/AbstractPeriodicGridDistribution
Open

Added AbstractPeriodicGridDistribution#288
FlorianPfaff wants to merge 3 commits intomainfrom
feature/AbstractPeriodicGridDistribution

Conversation

@FlorianPfaff
Copy link
Copy Markdown
Owner

No description provided.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Sep 17, 2023

MegaLinter analysis: Error

Descriptor Linter Files Fixed Errors Warnings Elapsed time
✅ COPYPASTE jscpd yes no no 14.86s
✅ JSON prettier 2 0 0 0 0.42s
✅ JSON v8r 2 0 0 3.2s
✅ MARKDOWN markdownlint 2 0 0 0 0.76s
✅ MARKDOWN markdown-table-formatter 2 0 0 0 0.26s
✅ PYTHON bandit 346 0 0 5.23s
✅ PYTHON black 346 5 0 0 8.15s
✅ PYTHON flake8 346 0 0 2.93s
✅ PYTHON isort 346 6 0 0 0.69s
✅ PYTHON mypy 346 0 0 4.92s
✅ PYTHON pylint 346 0 0 105.49s
✅ PYTHON ruff 346 6 0 0 0.06s
❌ REPOSITORY checkov yes 1 no 21.97s
✅ REPOSITORY gitleaks yes no no 6.23s
✅ REPOSITORY git_diff yes no no 0.03s
✅ REPOSITORY secretlint yes no no 6.43s
✅ REPOSITORY syft yes no no 5.77s
✅ REPOSITORY trivy-sbom yes no no 2.28s
✅ REPOSITORY trufflehog yes no no 19.29s
✅ YAML prettier 4 0 0 0 0.51s
✅ YAML v8r 4 0 0 5.65s
✅ YAML yamllint 4 0 0 0.49s

Detailed Issues

❌ REPOSITORY / checkov - 1 error
github_actions scan results:

Passed checks: 263, Failed checks: 1, Skipped checks: 0

Check: CKV_GHA_7: "The build output cannot be affected by user parameters other than the build entry point and the top-level source location. GitHub Actions workflow_dispatch inputs MUST be empty. "
	FAILED for resource: on(Check for version change, add tag, build, and publish)
	File: /.github/workflows/tags-and-publishing.yml:10-17

		10 |       dry_run:
		11 |         description: "Build only; do not push a tag or create/update a GitHub release"
		12 |         required: true
		13 |         type: boolean
		14 |         default: true
		15 | 
		16 | permissions:
		17 |   contents: write

See detailed reports in MegaLinter artifacts

Your project could benefit from a custom flavor, which would allow you to run only the linters you need, and thus improve runtime performances. (Skip this info by defining FLAVOR_SUGGESTIONS: false)

  • Documentation: Custom Flavors
  • Command: npx mega-linter-runner@9.4.0 --custom-flavor-setup --custom-flavor-linters PYTHON_PYLINT,PYTHON_BLACK,PYTHON_FLAKE8,PYTHON_ISORT,PYTHON_BANDIT,PYTHON_MYPY,PYTHON_RUFF,COPYPASTE_JSCPD,JSON_V8R,JSON_PRETTIER,MARKDOWN_MARKDOWNLINT,MARKDOWN_MARKDOWN_TABLE_FORMATTER,REPOSITORY_CHECKOV,REPOSITORY_GIT_DIFF,REPOSITORY_GITLEAKS,REPOSITORY_SECRETLINT,REPOSITORY_SYFT,REPOSITORY_TRIVY_SBOM,REPOSITORY_TRUFFLEHOG,YAML_PRETTIER,YAML_YAMLLINT,YAML_V8R

MegaLinter is graciously provided by OX Security
Show us your support by starring ⭐ the repository

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Sep 17, 2023

Test Results

    12 files  ± 0     12 suites  ±0   4h 21m 58s ⏱️ + 3m 35s
   986 tests + 1    986 ✅ + 1      0 💤 ±0  0 ❌ ±0 
12 116 runs  +12  9 384 ✅ +12  2 732 💤 ±0  0 ❌ ±0 

Results for commit 1a6bafe. ± Comparison against base commit 224177b.

♻️ This comment has been updated with latest results.

@FlorianPfaff FlorianPfaff force-pushed the main branch 3 times, most recently from 9b9c4f0 to a089dfc Compare September 29, 2023 12:45
@FlorianPfaff FlorianPfaff force-pushed the feature/AbstractPeriodicGridDistribution branch from d58718c to 02e4383 Compare January 19, 2024 21:22
@FlorianPfaff FlorianPfaff force-pushed the main branch 5 times, most recently from 022c55e to bb9ada8 Compare June 14, 2024 20:52
@FlorianPfaff FlorianPfaff force-pushed the feature/AbstractPeriodicGridDistribution branch from 02e4383 to 15b04b6 Compare November 14, 2025 17:38
@FlorianPfaff FlorianPfaff force-pushed the main branch 3 times, most recently from cbbee9b to 22baafb Compare December 1, 2025 20:52
@FlorianPfaff
Copy link
Copy Markdown
Owner Author

@copilot fix this

************* Module pyrecest.distributions.abstract_periodic_grid_distribution
pyrecest/distributions/abstract_periodic_grid_distribution.py:1:0: E0401: Unable to import 'pyrecest.distributions.abstract_bounded_grid_distribution' (import-error)


Your code has been rated at 9.99/10

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 14, 2026

@copilot fix this

************* Module pyrecest.distributions.abstract_periodic_grid_distribution
...

Fixed in d844a17. The import was referencing abstract_bounded_grid_distribution which doesn't exist in the codebase. Changed the import to use the existing abstract_grid_distribution module and updated the class to inherit from AbstractGridDistribution.

@FlorianPfaff FlorianPfaff force-pushed the feature/AbstractPeriodicGridDistribution branch from d844a17 to 161ca85 Compare April 21, 2026 06:42
@FlorianPfaff FlorianPfaff force-pushed the main branch 3 times, most recently from 7b21428 to 429a157 Compare April 21, 2026 13:50
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.

2 participants