Comments (5)
So even with screen real estate being larger, very long lines can be unreadable and can get messy are allows people to nest in more if block, for loops, etc. It also can become more difficult for side by side comparison, especially for 3-way merges (but these are rare to be fair).
We have a lot of equations and long variables names (which makes code more readable) that could really do with some more room, as they are becoming less readable by the time your forced to break them up over multiple lines. We also typically have the bulk of code in class methods, so lose 8 characters before you even get started.
There's always a balance to be had, but I think increasing to 99 characters would improve readability.
If we did make a change, I'd suggest we don't make one massive code change but allow the project to transition naturally as we make code changes.
from stone-soup.
from stone-soup.
I don't really have a problem with this. Can a local flake8 configuration file be distributed with the project, so that it passes the flake8 testing? Or include instructions for how to manually configure this ourselves,
from stone-soup.
@DaveKirkland Yes, the flake8 configuration can be committed, if that what is wanted. I assume CircleCI would honour it (otherwise, we might have to configure that separately).
from stone-soup.
closed as addressed in #166
from stone-soup.
Related Issues (20)
- Extend 2d density plotter to support more state types
- Plotter & Plotterly figures do not display automatically when scripts/functions are ran from the command line/IDE HOT 2
- Handle errors with certain error ellipses in plotly based plotter
- Possible issue with lru cache, _pickle.PicklingError: HOT 5
- Rotation matrix calculation
- Using a 1D measurement model noise covariance matrix doesn't produce an error message HOT 1
- Typo in constant derivative transition model process covariance formula HOT 1
- GM-PHD Rectangular Object Tracker
- Document optional dependencies/extras
- SimpleMeasurementInitiator new state definition HOT 3
- Store ParticleState weights as float array HOT 1
- Sensor clutter appears outside sensor's FOV
- improve solution for R-tree key error
- ground_truth_plots range limits HOT 1
- Measurement model inverse functions aren't vectorised
- Could not import "Particles" HOT 2
- MultiTargetMixtureTracker handling of unassociated detections
- lru_cache on instance methods HOT 1
- Aircraft Heading Markers not working HOT 2
- AnimatedPlotterly doesn't plot multiple GroundTruths when they start at different timestamps HOT 2
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 stone-soup.