Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

0.2 WIP candidate #10

Merged
merged 1 commit into from Jan 17, 2019
Merged

0.2 WIP candidate #10

merged 1 commit into from Jan 17, 2019

Conversation

lilyvelour
Copy link
Contributor

@lilyvelour lilyvelour commented Jan 15, 2019

Breaking changes/removals:

NOTE: Make sure to back up all VFX graphs prior to install! Graphs will probably break, and likely need to be fully rebuilt.

  • Refactored all blocks and nodes. Instead of many different properties among the different steps, Fluid and SolverData VFX types are sent instead. This helps to reduce node clutter substantially
  • Properties are no longer exposed by default. Instead, they are easily editable within the graph. Users can still expose individual properties, or entire fluids
  • Simulation scale has been removed (fixed at 1.0, likely to be fully deleted later). This is no longer needed due to the VFX graph's improved handling of transforms
  • Verlet integration is now the new default (see below). Physical properties for fluids will likely need to be adjusted slightly, but this change should help overall stability

Additions:

Fixes:

  • Particles no longer require a lifetime/alive attribute in order to be compatible with FluvioFX

Misc. changes:

  • Updated solver to use the built-in mass attribute in preparation for multi-fluid simulation (Use the built-in mass attribute #8)
  • Added FluvioFX templates as a part of the install process
  • Moved ReflectionHelpers to its own assembly to get rid of duplicate code
  • Minor README updates

Media:
graph

Breaking changes/removals:

*NOTE: Make sure to back up all VFX graphs prior to install!* Graphs will probably break, and likely need to be fully rebuilt.

- Refactored all blocks and nodes. Instead of many different properties among the different steps, Fluid and SolverData VFX types are sent instead. This helps to reduce node clutter substantially
- Properties are no longer exposed by default. Instead, they are easily editable within the graph. Users can still expose individual properties, or entire fluids
- Simulation scale has been removed (fixed at 1.0, likely to be fully deleted later). This is no longer needed due to the VFX graph's improved handling of transforms
- Verlet integration is now the new default (see below). Physical properties for fluids will likely need to be adjusted slightly, but this change should help overall stability

Additions:

- Added additional integration methods–Stormer-Verlet and Velocity Verlet (#3). Also tweaked the Semi-implcit Euler integration to also integrate positons
- Implemented optional external force solvers, and built-in gravity (#2)
- The default FluvioFX graph now has a bunch of helpful sticky notes to help get started

Fixes:

- Particles no longer require a lifetime/alive attribute in order to be compatible with FluvioFX

Misc. changes:

- Updated solver to use the built-in mass attribute in preparation for multi-fluid simulation (#8)
- Added FluvioFX templates as a part of the install process
- Moved ReflectionHelpers to its own assembly to get rid of duplicate code
- Minor README updates
@lilyvelour
Copy link
Contributor Author

Since this has some sweeping changes, I'll leave this up for a day or two for any public comment before merging.

@lilyvelour lilyvelour added this to Needs review in FluvioFX Jan 15, 2019
@lilyvelour lilyvelour merged commit 6f04491 into master Jan 17, 2019
FluvioFX automation moved this from Needs review to Done Jan 17, 2019
@lilyvelour lilyvelour deleted the feat/0.2-wip branch January 17, 2019 08:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
FluvioFX
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

1 participant