profile
viewpoint
Bob McWhirter bobmcwhirter JBoss > Red Hat > IBM Wytheville, VA http://projectodd.org/

bobmcwhirter/drools 11

Drools from JBoss.org

bobmcwhirter/arquillian.github.com 3

Arquillian website source (develop branch) and hosting (master branch). Baked with Awestruct.

bobmcwhirter/capistrano 3

Remote multi-server automation tool

bbrowning/ctjr 2

A very simple and crude facsimile of CoolingTower

bobmcwhirter/ballast 2

Test application for JBoss Rails support

bobmcwhirter/cloud-control 2

Tooling to assist in cloud formation

bobmcwhirter/deltacloud-client-java 2

Java Deltacloud Client

fork goldmann/java-operator-sdk

Java SDK for building Kubernetes Operators

fork in 10 hours

issue commentrust-embedded/embedded-hal

[RFC] `digital::OutputPort`, atomically drive several pins

I have another use case for changing the output value of several pins at once: Driving a 7-segment display. Rather than setting 7 pins separately for each number, I would like to use either a match expression for each number that writes a fixed value that deals with all 7 segments at once. Or some kind of lookup table. And due to board layout issues the port pins in use are not necessarily adjacent.

japaric

comment created time in 4 days

created repositoryjboss-container-images/fuse-images

created time in 10 days

issue commentrust-embedded/embedded-hal

i2c trait Error type has no bounds

This should be solved once we land #229

If we ever land that...

nallar

comment created time in 14 days

issue commentrust-embedded/embedded-hal

i2c trait Error type has no bounds

In case anybody else stumbles on this, another alternative is:

pub fn do_stuff<I2C, E>(interface: &mut I2C) -> Result<(), E>
where
    I2C: Read + Write,
    E: From<<I2C as Read>::Error> + From<<I2C as Write>::Error>
{
    interface.write(...)?;
    interface.read(...)?;

    Ok(());
}

This gives the caller maximum flexibility to define whatever error they want, as long as it has From impls. Also, it works out great in the case where both Read::Error and Write::Error happen to be the same: the user can just return that error type without having to define any boilerplate.

nallar

comment created time in 15 days

pull request commentrust-embedded/embedded-hal

Controller Area Network (CAN) Take 3

Great idea! The traits now live at: https://github.com/timokroeger/embedded-can and there is a v0.1 release on crates.io

timokroeger

comment created time in 19 days

pull request commentrust-embedded/embedded-hal

Storage Traits for on and off board storage

embedded-storage sounds good to me. As previously discussed, it is probable that this takes a while to get ready for embedded-hal. It should be noted as well that there are two proposals for storage traits: this one and also #248. Maybe both can live within embedded-storage until it is settled which one is best or so. Adding it to the rust-embedded-community might be possible. I am preparing an overhaul of the org's mission.

tl8roy

comment created time in 20 days

pull request commentrust-embedded/embedded-hal

Storage Traits for on and off board storage

I'm happy to support which ever solution works best for the community. I do have a bit of time this weekend.

tl8roy

comment created time in 20 days

pull request commentrust-embedded/embedded-hal

Storage Traits for on and off board storage

@Sympatron I think that is the right approach! Currently i am forcing all of my crates into separate branches just to use a storage branch here, and it doesn't quite make sense.

Maybe it would make sense to get @eldruin to create it in embedded-rust-community? I think that would be a great fit!

tl8roy

comment created time in 20 days

pull request commentrust-embedded/embedded-hal

Controller Area Network (CAN) Take 3

What do you think about starting with a seperate crate for can traits for now? That way it could already be used by other crates.

timokroeger

comment created time in 20 days

pull request commentrust-embedded/embedded-hal

Storage Traits for on and off board storage

What do you think about starting with a new crate to get this going? I think something like embedded-storage would be fitting. That way people could start using it immediately.

After some time it could still be merging into embedded-hal.

tl8roy

comment created time in 20 days

pull request commentrust-embedded/embedded-hal

Controller Area Network (CAN) Take 3

@marcelbuesing Looks like the error described in https://github.com/dtolnay/thiserror/issues/85. Probably removing #[from] fixes it.

timokroeger

comment created time in 20 days

pull request commentrust-embedded/embedded-hal

Controller Area Network (CAN) Take 3

I'm attempting to use the fork for an ISOTP implementation but I'm stuck with Error not enforcing being std::fmt::Debug. Any ideas ;)?

use thiserror::Error as ThisError;

#[derive(ThisError, Debug)]
pub enum Error<TCan> where TCan: Can, <TCan as Can>::Error: std::fmt::Debug {
     // ...
    #[error("CAN error")]
    Can(#[from] TCan::Error),
}
#[derive(ThisError, Debug)]
   |          ^^^^^^^^^
   |
   = note: conflicting implementation in crate `core`:
           - impl<T> std::convert::From<T> for T;
   = note: this error originates in a derive macro (in Nightly builds, run with -Z macro-backtrace for more info)
timokroeger

comment created time in 20 days

pull request commentrust-embedded/embedded-hal

Embedded-hal 1.0.0-alpha.4 release

I have now tagged this and published to crates.io.

eldruin

comment created time in 21 days

created tagrust-embedded/embedded-hal

tagv1.0.0-alpha.4

A Hardware Abstraction Layer (HAL) for embedded systems

created time in 21 days

PR merged rust-embedded/embedded-hal

Reviewers
Embedded-hal 1.0.0-alpha.4 release S-waiting-on-review T-hal

Once we have a new alpha release we can move forward with merging https://github.com/rust-embedded/linux-embedded-hal/pull/44 I also added the html_root_url, which is annoying but necessary according to the Rust API guidelines.

+6 -2

3 comments

3 changed files

eldruin

pr closed time in 21 days

push eventrust-embedded/embedded-hal

Diego Barrios Romero

commit sha fdda3795627d8d7bc78aefebebbcbc6d1b6c4e2e

Add html root url

view details

Diego Barrios Romero

commit sha 060c75736bb964008f29d95b1b447d299d951e2b

Prepare 1.0.0-alpha.4 release

view details

bors[bot]

commit sha d45af5f9bf25e7e29029db578aaa3c10c35da424

Merge #261 261: Embedded-hal 1.0.0-alpha.4 release r=ryankurte a=eldruin Once we have a new alpha release we can move forward with merging https://github.com/rust-embedded/linux-embedded-hal/pull/44 I also added the `html_root_url`, which is annoying but necessary according to the Rust API guidelines. Co-authored-by: Diego Barrios Romero <eldruin@gmail.com>

view details

push time in 21 days

push eventrust-embedded/embedded-hal

Ryan Kurte

commit sha c344847cee1ce0f55d5f5bf88f8dd917b7ce3a10

improved digital compatibility docs

view details

Roma Sokolov

commit sha c91e84ee5c1d0363dfe685e7b8f8359d1ab39ec4

Provide v2_compat for ToggleableOutputPin fixes #137

view details

bors[bot]

commit sha b2bbae4e25e2882ed0418efe4df9bd1c32edaf35

Merge #138 138: Provide v2_compat for ToggleableOutputPin r=ryankurte a=little-arhat fixes #137 Co-authored-by: Roma Sokolov <sokolov.r.v@gmail.com>

view details

Ryan Kurte

commit sha 79ae58c0ace8407f1615c8fd6e18663b1f8381d5

added capitals

view details

Danilo Bargen

commit sha 50e9e01f7f23f6035a62744899beb8940ab2b5b3

Remove #[deny(warnings)]

view details

bors[bot]

commit sha 5b0e5268c2841975cd28282c204d6fb358f19d8e

Merge #143 143: Remove #[deny(warnings)] r=ryankurte a=dbrgn It's an antipattern, see https://github.com/rust-unofficial/patterns/blob/master/anti_patterns/deny-warnings.md and https://github.com/rust-embedded/linux-embedded-hal/pull/17 Co-authored-by: Danilo Bargen <mail@dbrgn.ch>

view details

bors[bot]

commit sha a5c0e3c63e4e431807d491d35dd67d1d43c07c92

Merge #136 136: improved digital compatibility docs r=therealprof a=ryankurte Co-authored-by: Ryan Kurte <ryankurte@gmail.com>

view details

Vadim Kaushan

commit sha 640a87e7471f39d1b9b852d1a68d3e93cb7c7b36

Use Infallible error type instead ()

view details

Vadim Kaushan

commit sha 7432e4af7fcf0f0bcc21ddd2584870de78f66fb8

Replace Void with core::convert::Infallible

view details

Vadim Kaushan

commit sha c57928106c5963c9b9aba630f155dd8ee932d02a

Fix tests

view details

Boris Vinogradov

commit sha 193202156dc1f8dbd82bba62158d06f6f3bd9af5

Update dev dependencies to actual version

view details

bors[bot]

commit sha 7dd907cfabad7993e481d679901bb4201327daa3

Merge #160 160: Update dev dependencies to actual version r=therealprof a=no111u3 I clean update developing dependencies to buildable state. Co-authored-by: Boris Vinogradov <no111u3@gmail.com>

view details

bors[bot]

commit sha a3990806c1367142c7aee7f10a2d0dbc82889f2b

Merge #150 150: Fix error types r=therealprof a=Disasm `core::convert::Infallible` was stabilized in 1.34.0. This PR replaces `Void` and `()` error types with `Infallible`. Note that this is a breaking change. Co-authored-by: Vadim Kaushan <admin@disasm.info>

view details

Boris Vinogradov

commit sha 44ff5ab65e38d1e12e007ba48f2534e1cf011775

Change repository link to current

view details

bors[bot]

commit sha 8d520a7144e28ed819f12518577b7a75185c5aa5

Merge #161 161: Change repository link to current r=therealprof a=no111u3 Co-authored-by: Boris Vinogradov <no111u3@gmail.com>

view details

Nils VAN ZUIJLEN

commit sha 1dad061022e1b1f069e28c1795bbbc8a331bc5d7

Makes the digital::v2::InputPin trait proven See #41 for more details about this trait. Removes the feature flag from v2 implementation Keeps it on v1 because it would not make sense to prove a deprecated feature.

view details

Nils VAN ZUIJLEN

commit sha bea9bb6577c5f30d28b212d25882d93b770fe50b

Make the digital::v1::InputPin proven Side effect: make the compat traits proven

view details

Nils VAN ZUIJLEN

commit sha d9d1052c0967929b7185840a0f5585ce86780faa

Update changelog

view details

bors[bot]

commit sha db09d73f71879950c6d537b97f654751924772a4

Merge #164 164: Make the digital::{v1, v2}::InputPin traits proven r=therealprof a=nils-van-zuijlen See #41 Previous attempt : #102 Co-authored-by: Nils VAN ZUIJLEN <nils.van-zuijlen@mailo.com>

view details

Austin Glaser

commit sha 61877ce7fae416262474185277bcae35811e98a9

Add nonblocking RNG trait

view details

push time in 21 days

delete branch rust-embedded/embedded-hal

delete branch : staging.tmp

delete time in 21 days

push eventrust-embedded/embedded-hal

Diego Barrios Romero

commit sha fdda3795627d8d7bc78aefebebbcbc6d1b6c4e2e

Add html root url

view details

Diego Barrios Romero

commit sha 060c75736bb964008f29d95b1b447d299d951e2b

Prepare 1.0.0-alpha.4 release

view details

bors[bot]

commit sha 5c84b3798aabd3eac61cb5fe6a0bb09546eb94a5

[ci skip][skip ci][skip netlify] -bors-staging-tmp-261

view details

push time in 21 days

create barnchrust-embedded/embedded-hal

branch : staging.tmp

created branch time in 21 days

pull request commentrust-embedded/embedded-hal

Embedded-hal 1.0.0-alpha.4 release

bors r+

eldruin

comment created time in 21 days

pull request commentrust-embedded/embedded-hal

Embedded-hal 1.0.0-alpha.4 release

r? @ryankurte

(rust_highfive has picked a reviewer for you, use r? to override)

eldruin

comment created time in 21 days

PR opened rust-embedded/embedded-hal

Reviewers
Embedded-hal 1.0.0-alpha.4 release

Once we have a new alpha release we can move forward with merging https://github.com/rust-embedded/linux-embedded-hal/pull/44

+6 -2

0 comment

3 changed files

pr created time in 21 days

pull request commentrust-embedded/embedded-hal

[0.2.x] Use nb version 0.1.3 for compatibility with nb 1.0

Indeed. Should be fixed now.

eldruin

comment created time in 22 days

pull request commentrust-embedded/embedded-hal

[0.2.x] Use nb version 0.1.3 for compatibility with nb 1.0

oooh, Pin API change in nightly?

---- src/lib.rs - (line 385) stdout ----
error[E0061]: this function takes 1 argument but 0 arguments were supplied
  --> src/lib.rs:442:31
   |
58 |         Pin::new(&mut blinky).resume();
   |                               ^^^^^^- supplied 0 arguments
   |

help: expected the unit value `()`; create it with empty parentheses
   |
58 |         Pin::new(&mut blinky).resume(());
   |              
eldruin

comment created time in 22 days

delete branch rust-embedded/embedded-hal

delete branch : staging.tmp

delete time in 23 days

push eventrust-embedded/embedded-hal

Diego Barrios Romero

commit sha 6e8c8efa760881a39894b206a32bd7ff554bef6d

Fix adding dyn

view details

Diego Barrios Romero

commit sha 0c8f46681af3e062fea9f1ee19aa9fac42209775

Remove #[deny(warnings)]

view details

Diego Barrios Romero

commit sha 936fd16d4a222161fcc16b17b3f895367d91e8f6

Deny warnings on CI

view details

Diego Barrios Romero

commit sha 05ddb7553fc0594196ae4eb2268530ca7e00af34

Update stm32f30x dependency

view details

bors[bot]

commit sha 4c07da50c5b5eee7561261aeb7941558d9664248

Merge #214 214: [0.2.x]: Fix warning, remove #[deny(warnings)] but keep check on CI r=ryankurte a=eldruin This fixes #213. Additionally I removed #[deny(warnings)] but added a check for it on CI. I think we should cherry pick #138 to the 0.2.x branch as well and release a 0.2.4 version, since 1.0 will still take a while. I can do that in a separate PR. Co-authored-by: Diego Barrios Romero <eldruin@gmail.com>

view details

Roma Sokolov

commit sha 5708d979d8e0f7ea3bfaa4f07fea0a56d026817a

Provide v2_compat for ToggleableOutputPin fixes #137

view details

Ryan Kurte

commit sha 45324110519eb61e26f810580092593294e47b20

improved digital compatibility docs

view details

Ryan Kurte

commit sha b1046883289984a769c53d62e76c5dd0c226cda0

added capitals

view details

Diego Barrios Romero

commit sha 59f33ca1a13d7da3f55eaa59a0ae656e9d819108

Fix using InputPin instead of OutputPin Taken from https://github.com/rust-embedded/embedded-hal/pull/199

view details

Diego Barrios Romero

commit sha 55461c3bf751a7f3961b543e6f2cfcd2e86ccb53

Update changelog with text from #199

view details

bors[bot]

commit sha 20fad3ee56b060b1c344f6f3c149828112732488

Merge #220 220: [0.2.x] Fixes to digital compatibility shim r=ryankurte a=eldruin I cherry-picked #138 and some improvements to docs and included #199. Co-authored-by: Roma Sokolov <sokolov.r.v@gmail.com> Co-authored-by: Ryan Kurte <ryankurte@gmail.com> Co-authored-by: Diego Barrios Romero <eldruin@gmail.com>

view details

ryan kurte

commit sha 27ccfb2602e62688a4c4805d8849c6a95865b9c4

Updated cargo version and changelog for release v0.2.4

view details

Ryan

commit sha 689b2db345e2e9946b1598e85311a413eb8fb7f5

Updated changelog Fixed links, Removed empty section

view details

Ryan

commit sha f3c191e6728eafb68d7804000acf6d5a806e1a69

Fix dates

view details

Daniel Egger

commit sha 384b4934a88a7939604ca7cf4048c49a4cd07c16

Merge pull request #218 from rust-embedded/v0.2.4-updates v0.2.4 release

view details

Diego Barrios Romero

commit sha d77ba9232ed5a9567c51c912b7a8870a77ea3da5

Use nb version 0.1.3 for compatibility with nb 1.0

view details

bors[bot]

commit sha 9eb740c9de8ffe631b8992115f0fdba6b17f48a7

Merge #253 253: [0.2.x] Use nb version 0.1.3 for compatibility with nb 1.0 r=ryankurte a=eldruin With `nb` < `0.1.3` other crates cannot use `nb` `1.0` as the types will not match. A new release of 0.2.x including this would be good. Co-authored-by: Diego Barrios Romero <eldruin@gmail.com>

view details

push time in 23 days

more