profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/davidMcneil/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.

davidMcneil/rants 69

An async NATS client library for the Rust programming language.

davidMcneil/mnist 8

MNIST data set parser https://crates.io/crates/mnist

davidMcneil/OCR_formula2latex 5

Convert a written mathematical formula to latex.

davidMcneil/courier 2

A simple pub/sub service

davidMcneil/health_insurance 1

Scripts to compare health insurance plans.

davidMcneil/Remote-Control-Tank 1

Arduino based remote control tank using tablet accelerometer.

davidMcneil/Space-Invaders 1

Space invaders replica.

darsnack/DynamicCoreSim 0

A research based project for ECE597: Adv. Computer Architecture at Rose-Hulman Inst. of Tech. Attempts to analyze the performance benefits of a dynamic core processor.

darsnack/ECE333 0

A repo to contain lab work for ECE333 Digital Systems

PR opened mongodb/bson-rust

[De]sserialize u64 as string if > thank i64::MAX

It is surprising that serializing u64 values greater than i64::MAX fails. This MR [de]serializes these values as a string.

+43 -21

0 comment

4 changed files

pr created time in 18 hours

push eventdavidMcneil/bson-rust

David McNeil

commit sha b3f8a19455baf1ffe3768e7e93e74d421e26db0d

[De]sserialize u64 as string if > thank i64::MAX

view details

push time in 19 hours

fork davidMcneil/bson-rust

Encoding and decoding support for BSON in Rust

fork in 19 hours

PullRequestReviewEvent

Pull request review commentdavidMcneil/rants

updated deps and retrofitted to new apis

 exclude = ["rust-toolchain", ".vscode"] # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html  [dependencies]-bytes = "0.5.1"-futures = "0.3.1"-log = "0.4.8"-nom = "5.0.1"-owning_ref = "0.4.0"-pin-project = "0.4.17"-rand = "0.7.2"-serde = { version = "1.0.101", features = ["derive"] }-serde_json = "1.0.40"-tokio = { version = "0.2.1", features = ["dns", "io-util", "macros", "stream", "sync", "rt-core", "time", "tcp"] }-tokio-util = { version = "0.2.0", features = ["codec"] }-uuid = { version = "0.7.4", features = ["v4"] }+bytes = "*"+futures = "*"+log = "*"+nom = "*"+owning_ref = "*"+pin-project = "*"+rand = "*"+serde = { version = "*", features = ["derive"] }+serde_json = "*"+tokio = { version = "*", features = ["io-util", "macros", "sync", "time", "net"] }+tokio-util = { version = "*", features = ["codec"] }+tokio-stream = { version = "*", features = ["sync"] }+uuid = { version = "*", features = ["v4"] }

Hey Matt, this is awesome! Thanks for the PR.

The one change is I generally think it is bad form to have "*" dependencies in a library. Could you change the dependency sections to

[dependencies]
bytes = "1.0"
futures = "0.3"
log = "0.4"
nom = "6.2"
owning_ref = "0.4"
pin-project = "1.0"
rand = "0.8"
serde = { version = "1.0", features = ["derive"] }
serde_json = "1.0"
tokio = { version = "1.9", features = ["io-util", "macros", "sync", "time", "net"] }
tokio-util = { version = "0.6", features = ["codec"] }
tokio-stream = { version = "0.1", features = ["sync"] }
uuid = { version = "0.8", features = ["v4"] }

# Optional dependencies
native-tls-crate = { version = "0.2", optional = true, package = "native-tls" }
rustls = { version = "0.19", optional = true }
tokio-native-tls = { version = "0.3", optional = true }
tokio-rustls = { version = "0.22", optional = true }

[dev-dependencies]
env_logger = "0.9"
hostname = "0.3"
tokio = { version = "1.9", features = ["process", "rt-multi-thread"] }
mwrock

comment created time in 2 months

PullRequestReviewEvent

Pull request review commentdavidMcneil/mnist

Replace certain dependencies (reqwest -> curl + pbr, rulinalg -> ndarray)

 #![cfg(feature = "download")] -extern crate flate2;-extern crate reqwest;- use std::io::{Read, Write}; use std::path::{Path, PathBuf}; use std::{fs, io}; -const BASE_URL: &str = "http://yann.lecun.com/exdb/mnist";-const FASHION_BASE_URL: &str = "http://fashion-mnist.s3-website.eu-central-1.amazonaws.com";+use curl::easy::Easy;+use std::fs::File;+// use std::io::{stdout, Write};

Extra line

quietlychris

comment created time in 2 months

PullRequestReviewEvent
PullRequestReviewEvent