Comments (8)
actually I take this back, it looks good to me. What do you say @rythorpe. See here: https://jasmainak.github.io/mne-neuron/auto_examples/plot_simulate_somato.html#sphx-glr-auto-examples-plot-simulate-somato-py
from hnn-core.
What changed to produce this? The simulated waveform looks good, but the scaling still seems a bit off.
from hnn-core.
nothing really. If I use the parameters you sent me, we get this. The problem arose when I tried to use the "diff" -- so maybe not all parameters were updated.
okay let me rename the issue to reflect that the scaling is off. Personally I think the right way to fix these things is with tests rather than trying to figure out what is different.
There are no tests in HNN, so it is really hard to attest a "correctness" to it. Of course, you have sanity checks but it would be hard to do software development without more solid testing.
from hnn-core.
Implementing some formal tests would be good. Let's also keep dialoguing (maybe at a different time and thread) about how to implement "diff" files in a better way.
from hnn-core.
Hi folks. As I mentioned to @jasmainak today, I found a couple of things causing inconsistent results with HNN:
- Refactoring: this fixed the first dipole deviation point:
8f43e18
@jasmainak Mainak, since you know the logic for the refactoring, can you attack this? Alternatively, since this code is technical debt and will be removed by netpyne integration, just ignore 1). - The one fixed in PR #47. Commit: 8dfc00a
*) This change makes me uneasy because I don't understand the logic. The connection seemed to be unique, so I'm not sure why it's "spurious"?
12c2808
from hnn-core.
Thanks for digging into this @blakecaldwell !
regarding 1. let's fix it if you've already put in the effort into it and the code exists (which seems to be the case). I'll add a whats_new.rst page tomorrow so we can start documenting bug fixes with proper credit. A proper netpyne integration will take some time, but this is already useful!
The connection seemed to be unique, so I'm not sure why it's "spurious"?
okay perhaps I need to explain myself a bit here. This commit is part of #22 which was essentially fixing a bug introduced in the refactor of c76a27b. During the refactor I had mistakenly added a connection that should never have been there in the first place
from hnn-core.
Okay, that explanation helps. I wasn't sure if it was introduced by refactoring or was something that seemed ineffectual, so it was removed.
Can you fix the bug by reverting the commit or fixing the refactoring?
from hnn-core.
yep! I'm going to look into it :)
from hnn-core.
Related Issues (20)
- BUG: MPI simulations break for `dt` of certain size HOT 1
- Upload data error - HNN gui HOT 2
- Clean up optimization API
- Improve error messages for adding drives
- rename simulate_dipole -> simulate HOT 3
- Proposed Enhancements for the Current GUI: A Refined Feature List HOT 1
- change name from calcium model to Kohl_2022 HOT 4
- [JOSS Review] HNN-core: A Python software for cellular and circuit-level interpretation of human MEG/EEG HOT 2
- [JOSS] Documentation HOT 1
- Optimization example error HOT 4
- Problems using non-'soma' values for record_isec argument in simulate_dipole() HOT 12
- [JOSS] Software Paper HOT 2
- pre-allocate arrays for storing continuous simulation data in network_builder.py
- issue with GUI install
- GUI does not show dipole plots HOT 2
- installation on m2 mac HOT 1
- [BUG] `plot_dipole` not showing in GUI with `matplotlib>=3.8.0` HOT 9
- tests: add axis data checks for all plots available in GUI HOT 2
- BUG: Deleting drives in GUI after file upload prevents loading of the same file HOT 1
- GUI callbacks error messages are not logged
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from hnn-core.