Skip to content

Conversation

@asyncbtd
Copy link

@asyncbtd asyncbtd commented Nov 16, 2025

  • The mono/2.5d project has had its scripts corrected to work in godot 4.5.0
  • In the mono/dodge_the_creeps project, the StartGame() and Hit() signal listeners have been added.
  • In the mono/multiplayer_pong project, the collision area of ​​the paddle object was fixed.
  • In the mono/pong project, the collision area of ​​the paddle, wall, and ball objects has been fixed.
  • In the mono/squash_the_creeps project, added a listener for the OnPlayerHit signal. Added audio pause when a player loses a game

Is it compatible with engine version 3x? I don't know, no testing has been done.

Don't treat this as a pull request

In the future, I plan to maintain and create new demos in C#. Do I need to follow Microsoft's formatting guidelines? Or can I format the code however I want?

real_t sin = Mathf.Sin(angle);
real_t cos = Mathf.Cos(angle);
float sin = Mathf.Sin(angle);
float cos = Mathf.Cos(angle);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

real_t is intended to provide support for compiling Godot with double-precision floats with the same codebase, so it should not be replaced with float which would break that ability.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would have kept real_t if it worked out of the box. The problem is that in the official version of Godot, the _Process method accepts delta as a double.

Also, this example is for beginners, and I believe that if a beginner compiles Godot themselves with double precision, they can change the float themselves.

I could fix the current code to use real_t everywhere, but is that really necessary? Other C# examples don't use real_t at all.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Then change delta and leave the rest as-is?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's your repository, you decide. Should I fix it?

@aaronfranke
Copy link
Member

Is it compatible with engine version 3x? I don't know, no testing has been done.

It is not, but also, it cannot, and it does not need to be. Demos for Godot 3.x are maintained on a separate branch.

Do I need to follow Microsoft's formatting guidelines? Or can I format the code however I want?

Follow Godot's guidelines: https://docs.godotengine.org/en/latest/tutorials/scripting/c_sharp/c_sharp_style_guide.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants