Skip to content

cgen cache tiny changes #1

Open
AlecTroemel wants to merge 46 commits into
pepe:mainfrom
AlecTroemel:cgen-cache-tweaks
Open

cgen cache tiny changes #1
AlecTroemel wants to merge 46 commits into
pepe:mainfrom
AlecTroemel:cgen-cache-tweaks

Conversation

@AlecTroemel

@AlecTroemel AlecTroemel commented Aug 26, 2022

Copy link
Copy Markdown

Probably wont spend too much more time on this until after the weekend, but thought I'd show what I got from hacking on your branch.

  • removed rogue (print-result) line in stress test
  • moved built cache.c to inside src dir
  • switched ecs.janet to use the c lang cache

NOTE: Theres a bug somewhere in cache/clear, I get this when running the stress test

> janet test/performance/stress-test.janet
lets create components and systems

lets create a ton of entites
:a
error: expected iterable type, got 1.86519e-316
  in cache/clear
  in create-entities [test/performance/stress-test.janet] on line 48, column 1
  in _thunk [test/performance/stress-test.janet] (tailcall) on line 49, column 14

Nothing stands out as the obviously culprit.

AlecTroemel and others added 30 commits July 27, 2022 22:38
@AlecTroemel
tiny implementation of sparse set on integers
5c1b99a
@AlecTroemel
changes to sparse set to handle components, implimentation of views
8f170af
@AlecTroemel
integrate sparse set into ecs!
e2f9020
@AlecTroemel
remove testing file
3826b22
@AlecTroemel
use identity fn and dont run system if all queries are empty
a9c2c8c
@AlecTroemel
move sparse set into ecs file to work around my non understanding of …
7defce4 
…macros
@AlecTroemel
bunch of bug fixes
b7a00fd
@AlecTroemel
remove dead code
b07589e
@AlecTroemel
tweak comments
1048662
@AlecTroemel
split out sparse set into its own file again, fix imports
a58d7cb
@AlecTroemel
add simple cache of views
9a66041
@AlecTroemel
invalidate caches when relevant components change
6fe8d71
@AlecTroemel
remove redundant :as
06c5345
@AlecTroemel
mark cache functions as private, remove cache in name and just use _ …
17c917d 
…as needed
@AlecTroemel
unquote sparse-set/init in add-component macro
a17f630 
no longer need to expose the sparse set in junk-drawer.janet
@AlecTroemel
no longer need to import sparse set in timers
5fc1adc
@AlecTroemel
reuse id's that have been removed
e0e16c4
@AlecTroemel
split out max entity count from id, actually quote the ss init fn
1d0c5ed
@AlecTroemel
remove max entity count limitation, allow passing capacity through wo…
21d6227 
…rld init
@AlecTroemel
fix function call in example
b637fe3
@AlecTroemel
put the view function in the world object, use it to fix example 6
f8b71aa
@AlecTroemel
add docs strings in proper english to all ECS related functions
f3679cb
@AlecTroemel
add a complex stress test that touches most of the ECS
62484be
@pepe
refactor stress test macros
be0cab2 
Some code design changes:
- remove upscopes
- do not quote result arrays
- change quasiquote ~ to quote ', where it is not needed
- make ALPHABET constant
- change log prints from pp to print and add final one
@pepe
Refactor lambdas
6532053
@pepe
One more inc
0762d2c
@pepe
Remove false macro claim :-D
8eee382
@AlecTroemel
Merge pull request #22 from pepe/sparse-sets
c44b51b 
refactor stress test macros
@AlecTroemel
switch import back to "use"
034ad08
@pepe
It is Janet!
7f69f1c
AlecTroemel and others added 16 commits August 5, 2022 08:09
@AlecTroemel
add testament library and tests for cache
c9cc625
@AlecTroemel
fix some bugs in delete/clear sparse set fn, add tests for sparse set
1520f6c
@AlecTroemel
use janet profiling for stress test
ec7fc8b
@AlecTroemel
Merge pull request #23 from pepe/janet-attr
fc6c3e1 
It is Janet!
@AlecTroemel
switch to spork for both unit tests and performance timing
6bdb0bd
@AlecTroemel
Merge pull request #24 from AlecTroemel/tests
1ccac9b 
Add unit tests for sparse-set and cache, use spork/timeit for more granular...timing
@AlecTroemel
Merge pull request #21 from AlecTroemel/sparse-sets
fea7bf3 
Refactor ECS to use the Sparse Set data structure
@AlecTroemel
add install example
d805035
@AlecTroemel
spork doesnt need full url
a4deb4b
@pepe
Add cgen cache
8377664
@pepe
Remove Janet cache impl, fix native location
a1ffc20
@pepe
Fix unit test cache import
bf316d4
@pepe
Return janet cache implementation for now
27db2ba
@pepe
Add next method to cache, clean up cgen
12e1e08
@AlecTroemel
remove rogue function call to library no longer being used
a785584
@AlecTroemel
switch over to the cgen built cache
7f479f1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@AlecTroemel @pepe