Skip to content

move iir to new dsp crate#174

Merged
bors[bot] merged 2 commits into
quartiq:masterfrom
matthuszagh:dsp
Nov 22, 2020
Merged

move iir to new dsp crate#174
bors[bot] merged 2 commits into
quartiq:masterfrom
matthuszagh:dsp

Conversation

@matthuszagh

@matthuszagh matthuszagh commented Nov 22, 2020

Copy link
Copy Markdown
Contributor

As mentioned here.

@matthuszagh matthuszagh mentioned this pull request Nov 22, 2020
4 tasks

@jordens jordens left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

A couple small cleanups.

Comment thread .gitignore Outdated
@@ -1,2 +1,2 @@
/target
**/target

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Not needed here and yet. And it is a bit broad.

Comment thread Cargo.toml Outdated
nb = "1.0.0"
asm-delay = "0.9.0"
enum-iterator = "0.6.0"
dsp = { path = "dsp", version = "0.1.0" }

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

no need for version.

Comment thread dsp/Cargo.toml Outdated
authors = ["Robert Jördens <rj@quartiq.de>"]
edition = "2018"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Remove boilerplate.

@matthuszagh
matthuszagh requested a review from jordens November 22, 2020 18:35
@jordens

jordens commented Nov 22, 2020

Copy link
Copy Markdown
Member

bors merge

@bors

bors Bot commented Nov 22, 2020

Copy link
Copy Markdown
Contributor

Build succeeded:

@bors
bors Bot merged commit c91e395 into quartiq:master Nov 22, 2020
@matthuszagh
matthuszagh deleted the dsp branch November 22, 2020 18:44
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