Runtime independent design for Rust APIs by bharatGoswami8 · Pull Request #17 · eclipse-score/inc_mw_com

bharatGoswami8 · 2025-11-17T04:59:41Z

Primary APIs moved to Runtime traits
Add generic compatibilty for Vehicle Consumer and Producer
Added Provider and Consumer InstanceInfo
Added Async API for get_instance
added Debug trait for All the Sample trait
Added Core instead of std all the possible palce
Added one Gen file and remove runtime specific gen file
Added both Mock and Lola time in same main file
Added VehicleMonitor as Consumer on example
Removed all the feature flag
added impl for Reloc
Removed root toml file
Added ProviderInfo instance in producer struct and API side
Added Example in detail
Subscriber API pass self as refrence
Added eraly fail check for consumer and producer
Added Reloc in seprate file and used as module
Added Type id with interface trait
Added AsMut trait bound with SampleMaybeUninit trait
Validation for Service name
InstanceSpecifier enum added with ANY and specific instance name
Find Service API take FindServiceSpecifier enum as argument
instance id removed and instead of that used instance specifier to identify the instance
Updated check_str validation logic
Updated example file
Removed unused new method
Added Into Trait impl for InstanceSpecifier

* Primary APIs moved to Runtime traits * Add generic compatibilty for Vehicle Consumer and Producer * Added Provider and Consumer InstanceInfo

lurtz · 2025-11-17T10:54:13Z

com-api/examples/basic-consumer-producer.rs

+
+fn main() {
+    let runtime_builder = RuntimeBuilderImpl::new();
+    let runtime = Builder::<MockRuntimeImpl>::build(runtime_builder).unwrap();


I expected the non mocked runtime is used here

Yes , both can be used.
Added in next commit.

lurtz · 2025-11-17T11:04:35Z

com-api/com-api-concept/src/lib.rs

+    /// Reading from the received pointer before initialization is undefined behavior.
+    fn as_mut_ptr(&mut self) -> *mut T;


should it be unsafe used if undefined behavior can happen?

No, we don't, as the method itself will not introduce UB - using the pointer could, though. However, any operation that needs dereferencing is anyway unsafe on the pointer itself, so that's already how it should be.

lurtz · 2025-11-17T12:00:10Z

com-api/com-api-concept/src/lib.rs

+{
+}

+pub trait Subscriber<T: Reloc + Send, R: Runtime + ?Sized,> {


is T the payload of the event?

lurtz · 2025-11-17T12:23:35Z

com-api/examples/com-api-gen/com-api-gen-lola/src/lib.rs

I am surprised that generated code for both the mock and the lola binding exist. At the moment it looks even identical.

I was expecting that if we use generics the same generated code can be used for both bindings.

Yes. @bharatGoswami8 can we remove both lola and mock generated crates and create top level crate com-api-gen-types ? If you need smoene doing this, let me know and we can add commits.

Yes, now we have only one gen file with generic R

pawelrutkaq

We are heading to implement iceoryx2 and see if we have any issues with other backends.

pawelrutkaq · 2025-11-17T12:33:16Z

com-api/com-api-concept/src/lib.rs

+    type Publisher<T: Reloc + Send>: Publisher<T>;
+    type ProviderInfo: Send + Clone;
+    type ConsumerInfo: Send + Clone;
+


General note. Can we suffix here the types with Type suffix or any other name that is different than the bounded Trait So we don't have type Publisher: Publisher. This is really confusing later in the code and easy to miss.

@pawelrutkaq

As of now we don't have any guideline for naming in s-core coding guideline .... https://eclipse-score.github.io/score/main/contribute/development/rust/coding_guidelines.html

we may need to get the naming guideline from s-core rust community.

Can we at least don't name different things with exactly same name?
As Paweł pointed we have type Publisher with assigned struct Publisher that implements trait Publisher.
I don't think we need explicit rule for that.

pawelrutkaq · 2025-11-17T12:33:43Z

com-api/com-api-concept/src/lib.rs

+        &self,
+        _instance_specifier: InstanceSpecifier,
+    ) -> Self::ServiceDiscovery<I>;
+


I guess async api we can add in next PR.

added in latest commit.

pawelrutkaq · 2025-11-17T12:36:26Z

com-api/com-api-concept/src/lib.rs

+    fn check_str(_service_name: &str) -> bool {
+        // For now, accept any non-empty string as a valid service name
+        // In a real implementation, this might validate the format
+        true


This means that we will have to specify the str layout in S-CORE as it has to be runtime agnostic.

Either we can call the backend compatible APIs or we can have validation in com-api.

I think we should use backend API so that if communication medium or channel change we no need to modify validation logic in com-api side.

Thats a point ofc. But this would mean that if you change a backend (runtime) then You must change InstanceSpecifier across code base. This would contradict the whole idea of current work.

InstanceSpecifier is definitely meant to be backend agnostic. All backend specific stuff has to be buried inside ProducerInfo or ConsumerInfo.

Then I guess here we will just do like allowed letters , numbers and / ie and thats it.

Decision: Enforce format in concept level (backend agnostic) as described in doc of InstanceSpecifier

pawelrutkaq · 2025-11-17T12:41:47Z

com-api/com-api-concept/src/lib.rs

+/// The string shall describe where to find a certain instance of a service. Each level shall look
+/// like this
+/// <InterfaceName>:my/path/to/service_name
+#[derive(Clone)]


missing Debug, PartialEq, PartialOrd, Ord,Eq

We will add once we finalize on #17 (comment)

pawelrutkaq · 2025-11-17T12:54:53Z

com-api/com-api-concept/src/lib.rs


 unsafe impl Reloc for () {}
 unsafe impl Reloc for u32 {}



We shall add reloc for all u*, all i*, float*, bool,

added in latest commit.

Missing:
usize
u128
i128
isize
f64
all atomics ?
[T] where T: Reloc
[T; N] where T: Reloc
core::mem::MaybeUninit where T: Reloc
tuples from T1, to T5 ie (so variants from 1 elem to 5 elems) where TX: Reloc

I would move reloc trait with this to separate file reloc.rs since we will also need to add the Reloc derive proc macro and reexport it from there.

pawelrutkaq · 2025-11-17T13:49:10Z

com-api/examples/basic-consumer-producer.rs

+    sample.send().unwrap();
+}
+
+fn main() {


we shall have the main running the same code for MockRuntime and LolaRuntime. So we can see all that works together.

added in latest commit.

pawelrutkaq · 2025-11-17T13:50:21Z

com-api/examples/basic-consumer-producer.rs

@@ -67,56 +74,19 @@ mod test {
    fn create_producer() {


We need to add an example with composition.

pub struct SomeClass { // Keeps VehicleConsumer, or its part }

So we can write UT that shows the SomeClass works with SomeRuntime and MockRuntime and is testable due to it.

Added VehicleMonitor , just added for Consumer.
Will extend this with producer also.

pawelrutkaq · 2025-11-17T13:51:54Z

com-api/com-api-concept/src/lib.rs

 use std::fmt::Debug;
+use std::future::Future;
 use std::ops::{Deref, DerefMut};
 use std::path::Path;


change whats possible to core instead std.

[workspace.lints.clippy] std_instead_of_core = "warn" alloc_instead_of_core = "warn"

and enable warnings as errors or use err here.

added in latest commit.

pawelrutkaq · 2025-11-17T13:53:15Z

com-api/com-api-concept/src/lib.rs

+    type Subscriber<T: Reloc + Send>: Subscriber<T, Self>;
+    type ProducerBuilder<I: Interface, P: Producer<Self, Interface = I>>: ProducerBuilder<I, P, Self>;
+    type Publisher<T: Reloc + Send>: Publisher<T>;
+    type ProviderInfo: Send + Clone;


Yes ProviderInfo added as instance info of Producer , I think this will be use with offer service method but need to check feasibility.
This is added just to notify we have two different type for Producer and Consumer.

pawelrutkaq · 2025-11-17T13:55:20Z

com-api/com-api-concept/src/lib.rs

Should we make all samples implement Debug by requiring it for T ? later is's hard to debug in code without it.

added in latest commit.

bharatGoswami8 · 2025-11-18T16:59:30Z

Working on the review comments...

* Added Async API for get_instance * added Debug trait for All the Sample trait * Added Core instead of std all the possible palce * Added one Gen file and remove runtime specific gen file * Added both Mock and Lola time in same main file * Added VehicleMonitor as Consumer on example * Removed all the feature flag * added impl for Reloc * Removed root toml file

piotrchodorowski

I implemented Iceoryx integration on top of those changes and added comments with all my findings.
I didn't add any comments in runtime part as those changes should results from proposed changes in concept part.

piotrchodorowski · 2025-11-19T11:37:21Z

com-api/com-api-concept/src/lib.rs

+#[derive(Clone, Debug)]
 pub struct InstanceSpecifier {
-    pub specifier: String,
+    specifier: Option<String>,


Why removed pub? I think we need it

Why? If you want to access the string, you can just as well use AsRef<str>. We can also add Into<Option<String>> if that's what would fit your purpose.

But without pub I cant access this field within runtime e.g:

| 280 | let specifier = instance_info.instance_specifier.specifier.clone().unwrap(); | ^^^^^^^^^ private field

We can access using instanceSpecifier instance rather then making specifier field as public.
We have new() which is returning InstanceSpecifier.

Okay, it is clear to me now how to use it

piotrchodorowski · 2025-11-19T11:38:57Z

com-api/com-api-concept/src/lib.rs

    type Interface: Interface;
-    type OfferedProducer: OfferedProducer<Interface = Self::Interface>;
+    type OfferedProducer: OfferedProducer<R, Interface = Self::Interface>;



We will need new here

Suggested change

fn new(instance_info: R::ProviderInfo) -> Self;

The chain here should be Runtime::producer_builder().build(). Would that fit as well?

Yes, it was needed to be able to crate Producer with instance_info from ProducerBuilder.

piotrchodorowski · 2025-11-19T11:39:55Z

com-api/com-api-concept/src/lib.rs

+    type SampleMaybeUninit<'a>: SampleMaybeUninit<T> + 'a
+    where
+        Self: 'a;
+


We will need new here

Suggested change

fn new(identifier: &str, instance_info: R::ProviderInfo) -> Self;

The chain here should be Runtime::producer_builder().build().offer().

Yes, it is needed to be able to crate Publisher with identifier and instance info passed.
In current implementation there is no way to provide service name for Publisher.

piotrchodorowski · 2025-11-19T11:41:56Z

com-api/examples/com-api-gen/src/lib.rs

+
+pub struct VehicleProducer<R: Runtime + ?Sized>
+{
+    _runtime: core::marker::PhantomData<R>,


Suggested change

_runtime: core::marker::PhantomData<R>,

_runtime: core::marker::PhantomData<R>,

instance_info: R::ProviderInfo,

piotrchodorowski · 2025-11-19T11:42:19Z

com-api/examples/com-api-gen/src/lib.rs

+impl<R: Runtime + ?Sized> Producer<R> for VehicleProducer<R> {
+    type Interface = VehicleInterface;
+    type OfferedProducer = VehicleOfferedProducer<R>;
+


Suggested change

fn new(instance_info: R::ProviderInfo) -> Self {

VehicleProducer {

_runtime: std::marker::PhantomData,

instance_info,

}

}

piotrchodorowski · 2025-11-19T11:44:55Z

README.md

+For mock test:
 ```
-inc_mw_com/com-api$ cargo run --example basic-consumer-producer --features "lola"
+inc_mw_com$ cargo test --test basic-consumer-producer-test 


This should be called within com-api folder

will update on latest commit.

piotrchodorowski · 2025-11-19T11:46:06Z

README.md

 ```

-For LoLa build:
+For mock test:


It is for all interfaces now

will update on latest commit.

piotrchodorowski · 2025-11-19T11:46:13Z

README.md

+inc_mw_com$ cargo test --test basic-consumer-producer-test 
+```
+
+For mock Build and Run:


It is for all interfaces now

will update on latest commit.

piotrchodorowski · 2025-11-19T11:47:00Z

com-api/examples/basic-consumer-producer.rs

+    pub fn monitor_tire_data(self) -> Result<String> {
+        let subscribed = self.consumer.left_tire.subscribe(3)?;
+        let mut sample_buf = SampleContainer::new();
+


Why removed logic with sending and receiving n samples?

piotrchodorowski · 2025-11-19T11:52:25Z

com-api/examples/basic-consumer-producer.rs

+    sample.send().unwrap();
+}
+
+fn run_with_runtime<R: Runtime>(name: &str, runtime: &R) {


I recommend to change this example logic to something like:

let offered_producer = create_offered_producer(&runtime); let consumer = create_consumer(&runtime); let monitor = VehicleMonitor::new(offered_producer, consumer); for _ in 0..SAMPLES_NUM { monitor.produce_tire_data().unwrap(); monitor.consume_tire_data().unwrap(); }

pawelrutkaq · 2025-11-20T09:39:48Z

@darkwisebear I made a check on trait issues we discussed and doing impl<T: Reloc> ZeroCopySend for T {} is not possible since T is too general and fails into orpchan rules. But I think if we agree that we add in Reloc doc that user shall not implement Reloc by hand, instead use #[derive(Reloc)], and we would use that in generated code we would be fine. I would provide macro implementation for that. And later, we can seamlessly patch that macro if we need to also implement ZeroCopySend when Reloc is there (out of S-CORE) - this would make all this invisible to user and then I hope all are happy ;)

piotrchodorowski · 2025-11-21T09:01:43Z

@bharatGoswami8 One small note: after some additional investigation I see that it would be good (or even it could be necessary) to make all new() calls returning Result instead of just Self. Including of course the two new()'s proposed by myself before.

bharatGoswami8 · 2025-11-21T11:00:11Z

@bharatGoswami8 One small note: after some additional investigation I see that it would be good (or even it could be necessary) to make all new() calls returning Result instead of just Self. Including of course the two new()'s proposed by myself before.

As discussed in this comment thread #17 (comment) , we will update new method with returning Result.

bharatGoswami8 · 2025-11-21T11:02:45Z

Working on pending review comments fix, I will share latest commit next week start...

darkwisebear · 2025-11-24T19:27:31Z

@darkwisebear I made a check on trait issues we discussed and doing impl<T: Reloc> ZeroCopySend for T {} is not possible since T is too general and fails into orpchan rules. But I think if we agree that we add in Reloc doc that user shall not implement Reloc by hand, instead use #[derive(Reloc)], and we would use that in generated code we would be fine. I would provide macro implementation for that. And later, we can seamlessly patch that macro if we need to also implement ZeroCopySend when Reloc is there (out of S-CORE) - this would make all this invisible to user and then I hope all are happy ;)

Sounds ok. We could add a feature flag so that the derive macro will generate the impl, guarded by the optional dependency on the crate that offers ZeroCopySend.

* Added ProviderInfo instance in producer struct and API side * Added Example in detail * Subscriber API pass self as refrence * Added eraly fail check for consumer and producer * Added Reloc in seprate file and used as module * Added Type id with interface trait * Added AsMut trait bound with SampleMaybeUninit trait * Validation for Service name * InstanceSpecifier enum added with ANY and specific instance name * Find Service API take FindServiceSpecifier enum as argument * instance id removed and instead of that used instance specifier to identify the insatnce

pawelrutkaq · 2025-11-27T07:19:26Z

com-api/com-api-concept/src/lib.rs

    fn find_service<I: Interface>(
        &self,
-        _instance_specifier: InstanceSpecifier,
+        _instance_specifier: FindServiceSpecifier,


we shall remove _

Yes, will remove in next commit.

pawelrutkaq · 2025-11-27T07:22:50Z

com-api/com-api-concept/src/lib.rs

+            && service_name.split('/').all(|parts| {
+                !parts.is_empty() && parts.bytes().all(|part| part.is_ascii_alphanumeric())
+            })
    }


Validation does not match format from doc <InterfaceName>:my/path/to/service_name. Since we put InterfaceName into TYPE_ID of Interface, I guess doc shall be my/path/to/service_name . I would suggest to start also with leading /

updated the document and start path with leading slash.

pawelrutkaq · 2025-11-27T07:23:36Z

com-api/com-api-concept/src/lib.rs

@@ -152,6 +154,11 @@ impl AsRef<str> for InstanceSpecifier {
            .unwrap_or("[ANY]")


InstanceSpecifier now have no Any. So the self.specifier cannot be optional anymore and there is no [ANY] case

Ok in that case we need to return default " " string if error -> .unwrap_or("")

Just remove the Option in the type definition. It's not needed anymore. An oversight on my side in the internal review.

pawelrutkaq · 2025-11-27T07:26:47Z

com-api/com-api-concept/src/lib.rs

+    Specific(InstanceSpecifier),
+    Any,
+}



We shall add Into<FindServiceSpecifier> for InstanceSpecifier.

Yes this will add in next commit.

pawelrutkaq · 2025-11-27T07:27:20Z

com-api/com-api-concept/src/lib.rs

-unsafe impl Reloc for () {}
-unsafe impl Reloc for u32 {}
+// Reloc trait and its implementations have been moved to reloc.rs
+// Use `use reloc::Reloc;` to import it here and in other files.


this comment is not needed or ?

Yes , removed it was added to notify the reviewer.

pawelrutkaq · 2025-11-27T07:29:46Z

com-api/com-api-concept/src/lib.rs


 pub trait ConsumerDescriptor<R: Runtime + ?Sized> {
-    fn get_instance_id(&self) -> usize; // TODO: Turn return type into separate type
+    fn get_instance_identifier(&self) -> String;


This shall return &InstanceSpecifier and not String

pawelrutkaq · 2025-11-27T07:30:36Z

com-api/com-api-concept/src/reloc.rs

+
+unsafe impl Reloc for () {}
+unsafe impl Reloc for u32 {}
+unsafe impl Reloc for u64 {}


we are missing u8,u16 i8,i16

Added missing type

pawelrutkaq · 2025-11-27T07:32:09Z

com-api/com-api-concept/Cargo.toml

 publish = ["common"]
 license = "Apache-2.0"
+
+[dependencies]


Updated on latest commit.

pawelrutkaq · 2025-11-27T07:32:16Z

com-api/com-api-runtime-lola/Cargo.toml

 [dependencies]
 com-api-concept = { workspace = true }

+


Updated on latest commit.

* Added Reloc u8,u16 and i8 ,i16 * Removed comment

piotrchodorowski

Minor changes request.

piotrchodorowski · 2025-11-27T09:58:51Z

com-api/examples/basic-consumer-producer.rs

-    use_producer(runtime);
-    use_consumer(runtime);
+
+    let monitor = VehicleMonitor::new(use_consumer(runtime), use_producer(runtime));


Suggested change

let monitor = VehicleMonitor::new(use_consumer(runtime), use_producer(runtime));

let producer = use_producer(runtime);

let consumer = use_consumer(runtime);

let monitor = VehicleMonitor::new(consumer, producer);

How about making producer first to make sure consumer will find suitable one?

yes in this example we can move it., but both should be independent of each other.

Thats not correct I think.
We can't just create Consumer, we can only create Consumer for previously discovered Producer so correct order of operation is mandatory.

piotrchodorowski · 2025-11-27T10:01:02Z

com-api/com-api-runtime-lola/src/lib.rs

    type Subscription = SubscriberImpl<T>;
-    fn new(identifier: &str, instance_info: LolaConsumerInfo) -> Self {
-        Self {
+    fn new(identifier: &str, instance_info: LolaConsumerInfo) -> com_api_concept::Result<Self> {


pub fn new() is no longer needed

new is not pub it is from SubscribableImpl trait.

piotrchodorowski · 2025-11-27T10:01:12Z

com-api/com-api-runtime-lola/src/lib.rs

+    where
+        Self: 'a;
+
+    fn new(_identifier: &str, _instance_info: LolaProviderInfo) -> com_api_concept::Result<Self> {


pub fn new() is no longer needed

it is not public , it is from Publisher trait.

Sorry I put that comment (and the second one in SubscribableImpl) in misleading place.
pub fn new() is no longer needed since we have fn new(_identifier: &str, _instance_info: LolaProviderInfo)
Same for SubscribableImpl.

piotrchodorowski · 2025-11-27T10:56:46Z

com-api/examples/basic-consumer-producer.rs

 // Example struct demonstrating composition with VehicleConsumer
 pub struct VehicleMonitor<R: Runtime> {
    consumer: VehicleConsumer<R>,
+    producer: VehicleOfferedProducer<R>,


Suggested change

producer: VehicleOfferedProducer<R>,

producer: VehicleOfferedProducer<R>,

tire_subscriber: <<R as Runtime>::Subscriber<Tire> as Subscriber<Tire, R>>::Subscription,

Subscriber should be created before sending data to make sure those wont be dropped

piotrchodorowski · 2025-11-27T10:57:09Z

com-api/examples/basic-consumer-producer.rs

-    pub fn new(consumer: VehicleConsumer<R>) -> Self {
-        Self { consumer }
+    pub fn new(consumer: VehicleConsumer<R>, producer: VehicleOfferedProducer<R>) -> Self {
+        Self { consumer, producer }


Suggested change

Self { consumer, producer }

let tire_subscriber = consumer.left_tire.subscribe(3).unwrap();

Self { consumer, producer, tire_subscriber }

Subscriber should be created before sending data to make sure those wont be dropped

* Updated check_str validation logic * Updated example file * Removed unused new method * Added Into Trait impl for InstanceSpecifier

pawelrutkaq

LGTM

Rust::com Runtime independent design for Rust APIs

924b673

* Primary APIs moved to Runtime traits * Add generic compatibilty for Vehicle Consumer and Producer * Added Provider and Consumer InstanceInfo

bharatGoswami8 changed the title ~~Rust::com Runtime independent design for Rust APIs~~ Runtime independent design for Rust APIs Nov 17, 2025

lurtz reviewed Nov 17, 2025

View reviewed changes

pawelrutkaq requested changes Nov 17, 2025

View reviewed changes

piotrchodorowski suggested changes Nov 19, 2025

View reviewed changes

pawelrutkaq requested changes Nov 27, 2025

View reviewed changes

Rust::com Review comment fix

cc3cc73

* Added Reloc u8,u16 and i8 ,i16 * Removed comment

piotrchodorowski suggested changes Nov 27, 2025

View reviewed changes

Rust::com review comment fix

4dd0b65

* Updated check_str validation logic * Updated example file * Removed unused new method * Added Into Trait impl for InstanceSpecifier

bharatGoswami8 force-pushed the runtime_generic branch from f982cd5 to 4dd0b65 Compare December 3, 2025 14:06

pawelrutkaq approved these changes Dec 3, 2025

View reviewed changes

piotrchodorowski approved these changes Dec 3, 2025

View reviewed changes

bharatGoswami8 self-assigned this Dec 4, 2025

darkwisebear approved these changes Dec 4, 2025

View reviewed changes

qor-lb merged commit eafd7e1 into eclipse-score:main_api_rs Dec 4, 2025
1 check passed

		/// Reading from the received pointer before initialization is undefined behavior.
		fn as_mut_ptr(&mut self) -> *mut T;


	fn new(identifier: &str, instance_info: R::ProviderInfo) -> Self;

	_runtime: core::marker::PhantomData<R>,
	_runtime: core::marker::PhantomData<R>,
	instance_info: R::ProviderInfo,

+fn new(instance_info: R::ProviderInfo) -> Self {
+        VehicleProducer {
+            _runtime: std::marker::PhantomData,
+            instance_info,
+        }
+    }

		@@ -152,6 +154,11 @@ impl AsRef<str> for InstanceSpecifier {
		.unwrap_or("[ANY]")

-    let monitor = VehicleMonitor::new(use_consumer(runtime), use_producer(runtime));
+    let producer = use_producer(runtime);
+    let consumer = use_consumer(runtime);
+    let monitor = VehicleMonitor::new(consumer, producer);

	producer: VehicleOfferedProducer<R>,
	producer: VehicleOfferedProducer<R>,
	tire_subscriber: <<R as Runtime>::Subscriber<Tire> as Subscriber<Tire, R>>::Subscription,

	Self { consumer, producer }
	let tire_subscriber = consumer.left_tire.subscribe(3).unwrap();
	Self { consumer, producer, tire_subscriber }

Conversation

bharatGoswami8 commented Nov 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

pawelrutkaq left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

bharatGoswami8 Nov 19, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

pawelrutkaq Nov 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

bharatGoswami8 commented Nov 17, 2025 •

edited

Loading

bharatGoswami8 Nov 19, 2025 •

edited

Loading

pawelrutkaq Nov 25, 2025 •

edited

Loading