jramapuram / hal Goto Github PK

There are quite a few problems using af::array to handle CPU <-> GPU transitions. A swap to ndarray would be cleaner for host data

Remove the unwrap() calls all over the code and change it to a Result<> type

Data loading is slow. To resolve this we need a separate thread(s) that prefetch data into a queue which is used then in the actual model to execute (get_batch_*) calls.

When adding layers, check the sizing and throw a clean error message.

Only change I made was to Csrgo.toml to update spmc as the version in there is gone. ArrayFire-rust alone compiles fine

error[E0277]: (dyn std::ops::Fn(defines::AfError) + 'static) cannot be shared between threads safely
--> arrayfire-rust\src\error.rs:25:1
|
25 | pub static DEFAULT_HANDLE_ERROR: &'static ErrorCallback = &handle_error_general;
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ (dyn std::ops::Fn(defines::AfError) + 'static) cannot be shared between threads safely
|
= help: within &'static (dyn std::ops::Fn(defines::AfError) + 'static), the trait std::marker::Sync is not implemented for (dyn std::ops::Fn(defines::AfError) + 'static)
= note: required because it appears within the type &'static (dyn std::ops::Fn(defines::AfError) + 'static)
= note: shared static variables must have a type that implements Sync