-
Notifications
You must be signed in to change notification settings - Fork 87
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
Radioactive Game Object onto Map #236
base: master
Are you sure you want to change the base?
Conversation
Sorry, it seems to have included the commits from the other branch. I don't know how to not include the previous commits on this new branch. Hopefully it'll remove the redundant ones when the other PR is merged. |
Yes don't worry this will get sorted out
Right now there are no sprites associated with the object, I will need to create those before you can see them on the map. For now you can login and type
Take a look at the lidar specs: https://github.com/simon987/Much-Assembly-Required/wiki/(Hardware)-LiDAR, IIRC you should be able to see a '0A' if the obstacle is there (the '01' means the tile is blocked) |
// of radioactive objects, set the desired amount of radioactive objects to the | ||
// plain tile count | ||
TileMap m = world.getTileMap(); | ||
int plainCount = 0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This exact code block is used twice, we should extract it into its own function (e.g. WorldUtils.countTiles(int tileId))
public class RadioactiveWorldUtils { | ||
|
||
/** | ||
* Generate a list of biomass blobs for a world |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
RadioactiveObstacle
It should be adding the Radioactive Obstacle onto the map from the World Creation Listener event. I don't think the Radioactive Obstacle has artwork, so it doesn't actually display on the map. Makes progress on issue #135.
Questions I have: