Closed Somfic closed 3 weeks ago
I'm finding that when working with the codebase in this repository the default cargo formatter modifies a lot of code, which makes staging small changes difficult. Are you following a specific style guide I can opt in to? I'd like to be on one line with how we format the code
Baseline of this PR:
Species | Test cases | False negatives | False positives |
---|---|---|---|
Aleoida Arcus | 35 | β | π¨ 57.14% |
Aleoida Coronamus | 56 | β | π¨ 73.21% |
Aleoida Spica | 148 | β | π¨ 88.51% |
Aleoida Laminiae | 148 | β | π¨ 100% |
Aleoida Gravis | 12 | β | π¨ 83.33% |
Bacterium Aurasus | 345 | β | π¨ 12.17% |
Bacterium Scopulum | 6 | β | π¨ 83.33% |
Bacterium Acies | 107 | β | π¨ 12.15% |
Bacterium Vesicula | 208 | β | π¨ 7.69% |
Bacterium Alcyoneum | 170 | β | π¨ 16.47% |
Bacterium Tela | 33 | π¨ 100% | β |
Bacterium Omentum | 8 | β | π¨ 75% |
Bacterium Cerbrus | 229 | β | π¨ 25.33% |
Bacterium Verrata | 2 | β | π¨ 50% |
Roseum Brain Tree | 6 | β | π¨ 100% |
Gypseeum Brain Tree | 6 | β | π¨ 100% |
Ostrinum Brain Tree | 6 | β | π¨ 100% |
Viride Brain Tree | 6 | β | π¨ 100% |
Lividum Brain Tree | 6 | β | π¨ 100% |
Aureum Brain Tree | 6 | β | π¨ 100% |
Puniceum Brain Tree | 6 | β | π¨ 100% |
Lindigoticum Brain Tree | 6 | β | π¨ 100% |
Cactoida Cortexum | 68 | β | π¨ 100% |
Cactoida Lapis | 148 | β | π¨ 99.32% |
Cactoida Vermis | 4 | β | π¨ 75% |
Cactoida Pullulanta | 68 | β | π¨ 92.65% |
Cactoida Peperatis | 148 | β | π¨ 66.89% |
Clypeus Lacrimam | 16 | β | π¨ 68.75% |
Clypeus Margaritus | 16 | β | π¨ 62.5% |
Clypeus Speculumi | 3 | π¨ 66.67% | π¨ 33.33% |
Concha Renibus | 72 | β | π¨ 70.83% |
Concha Aureolas | 148 | β | π¨ 92.57% |
Concha Labiata | 257 | β | π¨ 80.16% |
Concha Biconcavis | 15 | β | π¨ 100% |
Crystaline Shards | 6 | π¨ 100% | β |
Fonticulua Campestris | 175 | β | π¨ 34.29% |
Fonticulua Lapida | 15 | β | π¨ 26.67% |
Fonticulua Digitos | 88 | β | π¨ 61.36% |
Frutexa Flabellum | 148 | β | π¨ 97.97% |
Frutexa Acus | 269 | β | π¨ 96.65% |
Frutexa Metallicum | 126 | β | π¨ 61.9% |
Frutexa Flammasis | 148 | β | π¨ 70.27% |
Frutexa Fera | 269 | β | π¨ 100% |
Frutexa Sponsae | 4 | β | π¨ 100% |
Frutexa Collum | 9 | β | π¨ 100% |
Fumerola Carbosis | 2 | π¨ 100% | β |
Fumerola Nitris | 2 | π¨ 100% | β |
Fumerola Aquatis | 1 | π¨ 100% | β |
Fungoida Setisis | 239 | β | π¨ 50.21% |
Fungoida Stabitis | 72 | β | π¨ 98.61% |
Fungoida Bullarum | 182 | β | π¨ 91.76% |
Fungoida Gelata | 72 | β | π¨ 44.44% |
Osseus Fractus | 68 | β | π¨ 61.76% |
Osseus Discus | 4 | β | π¨ 50% |
Osseus Spiralis | 148 | β | π¨ 52.7% |
Osseus Pumice | 281 | β | π¨ 92.17% |
Osseus Cornibus | 68 | β | π¨ 97.06% |
Osseus Pellebantus | 68 | β | π¨ 95.59% |
Recepta Umbrux | 9 | β | π¨ 44.44% |
Recepta Deltahedronix | 9 | β | π¨ 88.89% |
Recepta Conditivus | 8 | β | π¨ 62.5% |
Albidum Sinuous Tubers | 6 | β | π¨ 100% |
Caeruleum Sinuous Tubers | 6 | β | π¨ 100% |
Lindigoticum Sinuous Tubers | 6 | β | π¨ 100% |
Stratum Excutitus | 232 | β | π¨ 100% |
Stratum Paleas | 380 | β | π¨ 82.11% |
Stratum Laminamus | 106 | β | π¨ 100% |
Stratum Araneamus | 205 | β | π¨ 97.07% |
Stratum Limaxus | 232 | β | π¨ 84.48% |
Stratum Cucumisis | 243 | β | π¨ 99.59% |
Stratum Tectonicas | 363 | β | π¨ 52.62% |
Stratum Frigus | 243 | β | π¨ 98.77% |
Tubus Conifer | 138 | β | π¨ 94.93% |
Tubus Sororibus | 20 | β | π¨ 60% |
Tubus Cavas | 140 | β | π¨ 50% |
Tubus Rosarium | 29 | β | π¨ 68.97% |
Tubus Compagibus | 140 | β | π¨ 97.86% |
Tussock Pennata | 17 | β | π¨ 94.12% |
Tussock Ventusa | 27 | β | π¨ 51.85% |
Tussock Ignis | 71 | β | π¨ 92.96% |
Tussock Cultro | 148 | β | π¨ 97.97% |
Tussock Catena | 148 | β | π¨ 23.65% |
Tussock Pennatis | 269 | β | π¨ 100% |
Tussock Serrati | 51 | β | π¨ 90.2% |
Tussock Albata | 35 | β | π¨ 88.57% |
Tussock Propagito | 269 | β | π¨ 25.28% |
Tussock Divisa | 148 | β | π¨ 93.92% |
Tussock Caputus | 56 | β | π¨ 89.29% |
Tussock Triticum | 12 | β | π¨ 91.67% |
Tussock Stigmasis | 9 | β | π¨ 88.89% |
Tussock Capillum | 266 | β | π¨ 95.86% |
Not really sure yet what number weβre starving towards for the false positives, as even when all spawn conditions match; a species is not guaranteed to spawn in-game (right?).
I'm finding that when working with the codebase in this repository the default cargo formatter modifies a lot of code, which makes staging small changes difficult. Are you following a specific style guide I can opt in to? I'd like to be on one line with how we format the code
I usually just run cargo fmt
and let it format it for me. I usually run it when finalizing things for a new release, so you don't really have to worry about it too much as long as PRs don't take multiple releases to merge.
Looking through the https://github.com/Silarn/EDMC-BioScan rulesets, it also specifies region as a spawn condition. Looking at heatmaps here https://edastro.com/mapcharts/codex.html#codex it might also be something that could be tested. For example looking at this one:
It seems that Sinuous tubers spawn in specific regions or near nebula (at least that is what I guess are those spots in the other regions)
That kinda sucks, as the region is only able to be extracted from a CodexEntry
event, which only triggers after scanning a exobiological signal.
@klightspeed has come to the rescue! https://github.com/klightspeed/EliteDangerousRegionMap even has Rust files!
Can probably add this to the project in the Region
enum, something like Region::from_pos(pos: [f32; 3]) -> Region
@Somfic can I take this over or do you still want to work on this?
Hi, sorry lately I have been busy with finishing up uni and other projects, so feel free to take over this PR. The biggest change was the introduction of a few macros to make defining spawn conditions a bit smoother. Was working on updating the spawn conditions for each species, the commit messages should indicate which ones I finished.
No problem. IRL things go first after all. Thanks for the work you've already put in!
Continued in #70
This PR closes #57.
Results: