Skip to content

Conversation

@bernardnormier
Copy link
Member

This PR adds a new IceStorm demo in C#.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

PR Overview

This PR introduces a new IceStorm Weather demo in C#, demonstrating a simple pub‑sub system where weather sensors send periodic readings and weather stations receive and display these reports.

  • Added a sensor application that publishes random weather readings via IceStorm.
  • Added a station application that subscribes to the weather topic and prints received reports.
  • Included supporting components such as a custom AtmosphericConditions override, a ConsolePrinter servant, and updated documentation.

Reviewed Changes

File Description
csharp/IceStorm/Weather/Sensor/Program.cs Implements the sensor logic that publishes weather readings.
csharp/IceStorm/Weather/Station/Program.cs Implements the station logic that subscribes to the weather topic.
csharp/IceStorm/Weather/Station/ConsolePrinter.cs Provides a servant that prints sensor reports to the console.
csharp/IceStorm/Weather/Station/AtmosphericConditions.cs Adds a custom ToString method for better display of weather data.
csharp/IceStorm/Weather/README.md Updates documentation to guide building and running the demo.

Copilot reviewed 11 out of 11 changed files in this pull request and generated no comments.

Copy link
Member

@pepone pepone left a comment

Choose a reason for hiding this comment

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

Very refreshing demo. No more clock!

@bernardnormier
Copy link
Member Author

One issue with clock is while a clock "ticks", you don't send ticks with the current timestamp to your clock: the interface didn't make sense.

@bernardnormier bernardnormier merged commit 7e76c5f into zeroc-ice:main Mar 7, 2025
8 checks passed
@bernardnormier bernardnormier deleted the icestorm-cs branch June 18, 2025 22:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants