Skip to content

bugfix for Invalid speed "G1 F-2147483648"#10944

Merged
Noisyfox merged 1 commit into
OrcaSlicer:mainfrom
Azio-Pantheon:fix/invalid_gcode_generated_from_cooling
Oct 12, 2025
Merged

bugfix for Invalid speed "G1 F-2147483648"#10944
Noisyfox merged 1 commit into
OrcaSlicer:mainfrom
Azio-Pantheon:fix/invalid_gcode_generated_from_cooling

Conversation

@Azio-Pantheon

@Azio-Pantheon Azio-Pantheon commented Oct 3, 2025

Copy link
Copy Markdown
Contributor

Description

Fixes #10498, also might be same issue in #2067

Added non-zero safety check for feedrate calculation in slow_down_proportional and slowdown_to_minimum_feedrate.

In CoolingBuffer::process_layer sometimes we get 2 identical gcode line:
G1 F15000;_EXTRUDE_SET_SPEED G1 F15000;_EXTRUDE_SET_SPEED
This causes calculate_layer_slowdown to produce lines with -nan(ind) feedrate and results in G1 F-2147483648

Tests

invalid_gcod_line_test.zip
image

@Noisyfox Noisyfox left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, thanks

@Noisyfox Noisyfox merged commit 6feb99b into OrcaSlicer:main Oct 12, 2025
12 checks passed
discip added a commit to discip/OrcaSlicer that referenced this pull request Oct 13, 2025
* Bump actions/stale from 9 to 10 (OrcaSlicer#10919)

Bumps [actions/stale](https://github.com/actions/stale) from 9 to 10.
- [Release notes](https://github.com/actions/stale/releases)
- [Changelog](https://github.com/actions/stale/blob/main/CHANGELOG.md)
- [Commits](actions/stale@v9...v10)

---
updated-dependencies:
- dependency-name: actions/stale
  dependency-version: '10'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump actions/github-script from 7 to 8 (OrcaSlicer#10920)

Bumps [actions/github-script](https://github.com/actions/github-script) from 7 to 8.
- [Release notes](https://github.com/actions/github-script/releases)
- [Commits](actions/github-script@v7...v8)

---
updated-dependencies:
- dependency-name: actions/github-script
  dependency-version: '8'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump actions/setup-python from 5 to 6 (OrcaSlicer#10921)

Bumps [actions/setup-python](https://github.com/actions/setup-python) from 5 to 6.
- [Release notes](https://github.com/actions/setup-python/releases)
- [Commits](actions/setup-python@v5...v6)

---
updated-dependencies:
- dependency-name: actions/setup-python
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump tj-actions/changed-files from 46 to 47 (OrcaSlicer#10922)

Bumps [tj-actions/changed-files](https://github.com/tj-actions/changed-files) from 46 to 47.
- [Release notes](https://github.com/tj-actions/changed-files/releases)
- [Changelog](https://github.com/tj-actions/changed-files/blob/main/HISTORY.md)
- [Commits](tj-actions/changed-files@v46...v47)

---
updated-dependencies:
- dependency-name: tj-actions/changed-files
  dependency-version: '47'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Attempt to get the correct PR commit hash (OrcaSlicer#10900)

* Attempt to get the correct PR commit hash

* Add commit hash to flatpak

* Use given hash if `git_commit_hash` env variable is present

* bugfix for Invalid speed "G1 F-2147483648" (OrcaSlicer#10944)

added non zero safety check for feedrate calculation

* Clarify warning about malicious websites

* overflow inital setup screen (OrcaSlicer#11007)

allow overflow on setup

Co-authored-by: Noisyfox <[email protected]>

* [Profiles] Fix Prusa XL overhang speed (OrcaSlicer#10931) (OrcaSlicer#10933)

Fix reverse order of overhangs slowdown settings

* Delete .github/workflows/assign.yml

* Delete .github/workflows/orca_bot.yml

* Update build_all.yml

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Noisyfox <[email protected]>
Co-authored-by: Azi <[email protected]>
Co-authored-by: SoftFever <[email protected]>
Co-authored-by: Daazed McFarland <[email protected]>
Co-authored-by: Aleksandra M <[email protected]>
discip added a commit to discip/OrcaSlicer that referenced this pull request Oct 13, 2025
* Bump actions/stale from 9 to 10 (OrcaSlicer#10919)

Bumps [actions/stale](https://github.com/actions/stale) from 9 to 10.
- [Release notes](https://github.com/actions/stale/releases)
- [Changelog](https://github.com/actions/stale/blob/main/CHANGELOG.md)
- [Commits](actions/stale@v9...v10)

---
updated-dependencies:
- dependency-name: actions/stale
  dependency-version: '10'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump actions/github-script from 7 to 8 (OrcaSlicer#10920)

Bumps [actions/github-script](https://github.com/actions/github-script) from 7 to 8.
- [Release notes](https://github.com/actions/github-script/releases)
- [Commits](actions/github-script@v7...v8)

---
updated-dependencies:
- dependency-name: actions/github-script
  dependency-version: '8'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump actions/setup-python from 5 to 6 (OrcaSlicer#10921)

Bumps [actions/setup-python](https://github.com/actions/setup-python) from 5 to 6.
- [Release notes](https://github.com/actions/setup-python/releases)
- [Commits](actions/setup-python@v5...v6)

---
updated-dependencies:
- dependency-name: actions/setup-python
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump tj-actions/changed-files from 46 to 47 (OrcaSlicer#10922)

Bumps [tj-actions/changed-files](https://github.com/tj-actions/changed-files) from 46 to 47.
- [Release notes](https://github.com/tj-actions/changed-files/releases)
- [Changelog](https://github.com/tj-actions/changed-files/blob/main/HISTORY.md)
- [Commits](tj-actions/changed-files@v46...v47)

---
updated-dependencies:
- dependency-name: tj-actions/changed-files
  dependency-version: '47'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Attempt to get the correct PR commit hash (OrcaSlicer#10900)

* Attempt to get the correct PR commit hash

* Add commit hash to flatpak

* Use given hash if `git_commit_hash` env variable is present

* bugfix for Invalid speed "G1 F-2147483648" (OrcaSlicer#10944)

added non zero safety check for feedrate calculation

* Clarify warning about malicious websites

* overflow inital setup screen (OrcaSlicer#11007)

allow overflow on setup

Co-authored-by: Noisyfox <[email protected]>

* [Profiles] Fix Prusa XL overhang speed (OrcaSlicer#10931) (OrcaSlicer#10933)

Fix reverse order of overhangs slowdown settings

* Delete .github/workflows/assign.yml

* Delete .github/workflows/orca_bot.yml

* Update build_all.yml

* BUG-FIX avoid crossing perimeters ingonore holes <=2mm (OrcaSlicer#10942)

Fix avoid crossing perimeters 2

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Noisyfox <[email protected]>
Co-authored-by: Azi <[email protected]>
Co-authored-by: SoftFever <[email protected]>
Co-authored-by: Daazed McFarland <[email protected]>
Co-authored-by: Aleksandra M <[email protected]>
Co-authored-by: Rodrigo Faselli <[email protected]>
@Azio-Pantheon Azio-Pantheon deleted the fix/invalid_gcode_generated_from_cooling branch October 16, 2025 22:49
kesor pushed a commit to kesor/SuperSlicer that referenced this pull request Nov 21, 2025
added non zero safety check for feedrate calculation

Source: OrcaSlicer commit 6feb9
Original PR: OrcaSlicer/OrcaSlicer#10944
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.

Invalid speed "G1 F-2147483648" when printing overhangs

2 participants