Skip to content

Conversation

@krzysztofrusek
Copy link
Contributor

This pr introduces beam forming and better walls handling in the simulator.

TODO

  • Refactor walls argument
  • Add isotropic antena and free space loss
  • Add monte-carlo walls
  • Improve documentation. In particular we must decide whether to use classical antena gain or relative to the maximum.
  • Add walls visualization

@krzysztofrusek krzysztofrusek added the enhancement New feature or request label Jul 15, 2024
@krzysztofrusek krzysztofrusek requested a review from m-wojnar July 15, 2024 12:33
@krzysztofrusek krzysztofrusek self-assigned this Jul 15, 2024
Copy link
Contributor

@m-wojnar m-wojnar left a comment

Choose a reason for hiding this comment

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

Looks good 👍 For the further steps, I propose to add some class or function wrapper for a whole process. This will allow the user to call just one function instead of preparation of the wall matrix, antena gain matrix, and finally simulator.

Do you possibly know why unit tests are failing? It looks like a tfp or jax bug?

@krzysztofrusek
Copy link
Contributor Author

Looks good 👍 For the further steps, I propose to add some class or function wrapper for a whole process. This will allow the user to call just one function instead of preparation of the wall matrix, antena gain matrix, and finally simulator.

I agree.

My idea is to have simulator as a callable with differentiable parameters.

This callable can be either :

  • a pytree with __call__
  • or a factory function returning differentiable simulator (optax style).

I am not sure right now what is better at this point, as we need to modularize the sim function.

Do you possibly know why unit tests are failing? It looks like a tfp or jax bug?

no, but indeed this looks like tfp issue, maybe we need to bump version.

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants