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

[SOUND]: make_sound should check for negative duration #110

Closed
martin-henz opened this issue Sep 14, 2021 · 0 comments · Fixed by #234
Closed

[SOUND]: make_sound should check for negative duration #110

martin-henz opened this issue Sep 14, 2021 · 0 comments · Fixed by #234
Labels
Enhancement [Category] New feature request

Comments

@martin-henz
Copy link
Member

Currently

make_sound(x => 0, -1)

is possible, but it should lead to an error.

@martin-henz martin-henz added the Enhancement [Category] New feature request label Sep 14, 2021
Cloud7050 added a commit to Cloud7050/modules that referenced this issue Sep 6, 2023
martin-henz pushed a commit that referenced this issue Sep 12, 2023
* feat(src): Extract & update CSG's CSS constants

* chore(csg): Remove V1 comments referencing fork issues

* feat(csg): Migrate to BlueprintJS Tooltips

* devfeat(csg): Update samples

* devfeat(curve): Add samples

* docs(curve): Update animation function docs

* ref(csg): Remove default colour

The addition of operation colour preservation and removal of the store/render pattern means that all Shapes now require an explicit colour. Changing the default colour of the wrapped renderer no longer has a visible effect.

* fix(build): Fix `build modules` building no tabs

Revise/clarify the logic of retrieveBundlesAndTabs() and update tests/comment.

* feat(): Enhance & make curve canvases consistent

* feat(csg): Replace hint dots with colons

* ref(csg): Use real js-slang lists

* devfeat(csg): Update colours sample

* devfeat(csg): Add primitives sample

* chore(pkg): Lock modeling version for patch file

* ref(build): Reorder config to match

Left over from tests with explicit tsconfig path when troubleshooting
build errors

* ref(csg): Rename to Operable, no List, move/del funcs

* fix(csg): Fix primitive sample trailing comma

* devfeat(csg): Add Prof's Sierpinski fractal

* FEAT(csg): Overhaul functions, documentation

* devfeat(csg): Update sample import names

* ref(csg): Reorder sample functions

* devfeat(csg): Add rotation sample

* devfeat(csg): Add summary snowglobe example as sample

* docs(csg): Plural categories, bulleted list, tweaks

* ref(csg): Refactor operables

Rename params, non-deep children copy for new Group

* feat(csg): Improve error feedback for pitfalls

* fix(csg): Implement proper ungroup

* feat(csg): Improve error feedback for pitfalls

* feat(csg): Update sample utility funcs

* feat(csg): Update sierpinski sample for new primitive

* fix(rune): Fix low resolution heart

* docs(csg): Link to GitHub samples in summary

* feat(csg): Update sierpinski sample

* feat(csg): Readd custom default colour due to bug #227

* ref(build): Remove leftover from troubleshooting

* docs(csg): Tweak pyramid dimensions description

* feat(sound): Throw error on invalid sound durations

Closes #110

* chore(pkg): Remove unused lodash

* feat(curve): Add Source version to canvases sample

* feat(tabs): Remove enforced spacing of MultiItemDisplay

* ref(csg): Default colour back in constants

* fix(csg): Disallow negative scaling, update docs instead

Based on #206 (comment).

* ref(csg): Rename RGB components to values

They were called components when the params were clamped between 0-1.
Now more fitting to call them values when 0-255.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement [Category] New feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant