GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Architecture rewrite by caelunshun · Pull Request #7 · feather-rs/feather

Conversation

@caelunshun

@caelunshun caelunshun commented Jun 30, 2019 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

The architecture branch contains a rewrite of the Feather architecture using an ECS-based design.

https://kyren.github.io/2018/09/14/rustconf-talk.html is being used as a resource.

This pull request aims to resolve #2.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
caelunshun added 17 commits June 30, 2019 13:30
@caelunshun
Prepare for architecture rewrite
88e817e
@caelunshun
Implement generational indices
5bd7445
@caelunshun
Reimplement World
6b546d6
@caelunshun
Run rustfmt
0cfee20
@caelunshun
Fix genindex to properly use Options
e6ba0ef
@caelunshun
Start work on network component
a439011
@caelunshun
Further work on network system implementation
c5565f2
@caelunshun
Work on InitialHandler
b16d377
@caelunshun
Finish InitialHandler implementation
a02f3df
@caelunshun
Fix compiler warnings
9546aa3
@caelunshun
Run rustfmt
e20054c
@caelunshun
Implement new connection management
e75a31b
@caelunshun
Fix InitialHandler to allow clients to connect
b7631e0
@caelunshun
Improve error messages in InitialHandler
914d641
@caelunshun
Send chunk data packets. Architecture update now has feature parity w…
f16e9ae 
…ith the master branch
@caelunshun
Fix panic caused by world not allowing chunks to load if they're alre…
8d8441e 
…ady loaded
@caelunshun
Send keepalives
950a8fa
@caelunshun

caelunshun commented Jul 1, 2019

Copy link
Copy Markdown
Member Author

This PR also fixes #8.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@caelunshun

caelunshun commented Jul 1, 2019 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member Author

Should be ready to merge. It seems like this branch is more stable than master as well, looking at #8 and #9.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@caelunshun
caelunshun merged commit 69cf8fe into master Jul 1, 2019
@caelunshun
caelunshun deleted the architecture branch July 1, 2019 05:25
This was referenced Jul 1, 2019
Player count incorrect #9
Closed
Server sometimes panics when a player joins with "called Option::unwrap() on a None value" #8
Closed
@caelunshun caelunshun mentioned this pull request Nov 13, 2019
Legion/Tonks rewrite #170
Merged
Tracreed pushed a commit to Tracreed/feather that referenced this pull request Sep 26, 2021
@Miro-Andrin
Merge pull request feather-rs#7 from Tracreed/refactor
d87f78b 
cargo fmt
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Improve overall architecture

Uh oh!

There was an error while loading. Please reload this page.

1 participant

@caelunshun
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.