pickhardtpayments icon indicating copy to clipboard operation
pickhardtpayments copied to clipboard

- integrated the property `bee_line_distance` to the `Channel` class

Open RohitRathore1 opened this issue 3 years ago • 3 comments

  • integrated the property latency to the OracleLightningNetwork class

RohitRathore1 avatar Jun 08 '22 14:06 RohitRathore1

Hey conceptually this is exactly what will be needed to add latency as an additional feature and thanks a lot for your suggestion. I am a bit reluctant to merge at this point:

  1. adding an additional field to the ChannelFields class breaks compatibility with c-lightning which we currently have
  2. I would love to see tests first, that the bee line distance does actually work well to predict latency.

Once we have such results I will be very happy to merge this and integrate this into the library.

renepickhardt avatar Jun 08 '22 23:06 renepickhardt

Hey conceptually this is exactly what will be needed to add latency as an additional feature and thanks a lot for your suggestion. I am a bit reluctant to merge at this point:

  1. adding an additional field to the ChannelFields class breaks compatibility with c-lightning which we currently have
  2. I would love to see tests first, that the bee line distance does actually work well to predict latency.

Once we have such results I will be very happy to merge this and integrate this into the library.

@renepickhardt Thanks for the feedback. These two changes are fine till now? Am I right? Now I will focus on bee_line_distance. When it will work correctly then we can move forward but before bee line distance I wanted to confirm for above these two properties that I have done correctly or not.

RohitRathore1 avatar Jun 09 '22 04:06 RohitRathore1

Yes if those Features work one could add them like this. The actual feature engineering would later Happen in the uncertainty Networks class though

renepickhardt avatar Jun 10 '22 11:06 renepickhardt