Stayrtr throws rtrlib.SendableData is *rtrlib.BgpsecKey, not *rtrlib.VRP

Dear developers of stayrtr
Whenever i start stayrtr 8a3a71e0 in my network, i get the error below:
stayrtr_1      | panic: interface conversion: rtrlib.SendableData is *rtrlib.BgpsecKey, not *rtrlib.VRP
stayrtr_1      |
stayrtr_1      | goroutine 1046 [running]:
stayrtr_1      | github.com/bgp/stayrtr/lib.(*Client).SendSDs.func1(0x1?, 0x0)
stayrtr_1      | 	github.com/bgp/stayrtr/lib/server.go:1059 +0x2f9
stayrtr_1      | sort.partition_func({0xc0031c2e80?, 0xc00733d470?}, 0x0, 0x21f63, 0xc008a77630?)
stayrtr_1      | 	sort/zsortfunc.go:142 +0xaf
stayrtr_1      | sort.pdqsort_func({0xc0031c2e80?, 0xc00733d470?}, 0x0?, 0x62c42?, 0x87d300?)
stayrtr_1      | 	sort/zsortfunc.go:114 +0x20f
stayrtr_1      | sort.pdqsort_func({0xc0031c2e80?, 0xc00733d470?}, 0x7f1e00deed28?, 0x18?, 0xc00006ec00?)
stayrtr_1      | 	sort/zsortfunc.go:121 +0x25a
stayrtr_1      | sort.Slice({0x87d300?, 0xc00733aca8?}, 0x6ec00?)
stayrtr_1      | 	sort/slice.go:26 +0xfa
stayrtr_1      | github.com/bgp/stayrtr/lib.(*Client).SendSDs(0xc00582cd20, 0x4ced, 0x0, {0xc008858000?, 0x62c42, 0x6ec00})
stayrtr_1      | 	github.com/bgp/stayrtr/lib/server.go:1035 +0xb8
stayrtr_1      | github.com/bgp/stayrtr/lib.(*DefaultRTREventHandler).RequestCache(0xc0001a4da0, 0xc00582cd20)
stayrtr_1      | 	github.com/bgp/stayrtr/lib/server.go:84 +0x109
stayrtr_1      | github.com/bgp/stayrtr/lib.(*Server).RequestCache(0xc00a7ce200?, 0x886e60?)
stayrtr_1      | 	github.com/bgp/stayrtr/lib/server.go:520 +0x2b
stayrtr_1      | github.com/bgp/stayrtr/lib.(*Client).passSimpleHandler(0xc00a7cc6f0?, {0x9e4668?, 0xc00733ea3c?})
stayrtr_1      | 	github.com/bgp/stayrtr/lib/server.go:802 +0x71
stayrtr_1      | github.com/bgp/stayrtr/lib.(*Client).Start(0xc00582cd20)
stayrtr_1      | 	github.com/bgp/stayrtr/lib/server.go:877 +0x377
stayrtr_1      | created by github.com/bgp/stayrtr/lib.(*Server).acceptClientTCP
stayrtr_1      | 	github.com/bgp/stayrtr/lib/server.go:554 +0x276

Here's how i am launching it:
  stayrtr:
    image: rpki/stayrtr:8a3a71e0
    depends_on:
      - rpki-client
    volumes:
     - "rpki-client-output:/var/lib/rpki-client/"
    command:
      - -cache
      - /var/lib/rpki-client/json
      - -checktime=false
      - -bind
      - <IP_ADDRESS>:323
    expose:
      - 323
    ports:
      - 323:323

I believe it's worth to note that I have also tried with rpki/stayrtr:v0.5.0 with similar results and that everything was working with rpki/stayrtr:ad3ed83a .
Could you please look into it?
Thank you
Regards

@lamehost Can you try (when CI pushes it) the docker image that is built from b2a79528c5d221f46bdd766ce9c448714f3b62d5

Done.
Here's the result:
stayrtr_1      | panic: interface conversion: rtrlib.SendableData is *rtrlib.VAP, not *rtrlib.VRP
stayrtr_1      |
stayrtr_1      | goroutine 2673 [running]:
stayrtr_1      | github.com/bgp/stayrtr/lib.(*Client).SendSDs.func1(0x596d5?, 0x0)
stayrtr_1      | 	github.com/bgp/stayrtr/lib/server.go:1059 +0x319
stayrtr_1      | sort.partition_func({0xc0053abe80?, 0xc004cb1200?}, 0x0, 0x62cbe, 0x87e400?)
stayrtr_1      | 	sort/zsortfunc.go:142 +0xaf
stayrtr_1      | sort.pdqsort_func({0xc0053abe80?, 0xc004cb1200?}, 0x7faa436011d8?, 0x18?, 0xc00a700000?)
stayrtr_1      | 	sort/zsortfunc.go:114 +0x20f
stayrtr_1      | sort.Slice({0x87e400?, 0xc004cad5f0?}, 0x6ec00?)
stayrtr_1      | 	sort/slice.go:26 +0xfa
stayrtr_1      | github.com/bgp/stayrtr/lib.(*Client).SendSDs(0xc00950e280, 0xe074, 0x0, {0xc00aa80000?, 0x62cbe, 0x6ec00})
stayrtr_1      | 	github.com/bgp/stayrtr/lib/server.go:1035 +0xb8
stayrtr_1      | github.com/bgp/stayrtr/lib.(*DefaultRTREventHandler).RequestCache(0xc000094060, 0xc00950e280)
stayrtr_1      | 	github.com/bgp/stayrtr/lib/server.go:84 +0x109
stayrtr_1      | github.com/bgp/stayrtr/lib.(*Server).RequestCache(0xc005e880e0?, 0x887f60?)
stayrtr_1      | 	github.com/bgp/stayrtr/lib/server.go:520 +0x2b
stayrtr_1      | github.com/bgp/stayrtr/lib.(*Client).passSimpleHandler(0xc004cc8000?, {0x9e6028?, 0xc003ed556c?})
stayrtr_1      | 	github.com/bgp/stayrtr/lib/server.go:802 +0x71
stayrtr_1      | github.com/bgp/stayrtr/lib.(*Client).Start(0xc00950e280)
stayrtr_1      | 	github.com/bgp/stayrtr/lib/server.go:877 +0x377
stayrtr_1      | created by github.com/bgp/stayrtr/lib.(*Server).acceptClientTCP
stayrtr_1      | 	github.com/bgp/stayrtr/lib/server.go:554 +0x276

Ah, Figured it out to be a data race, Can you try now? with the latest image that is building?

Hello,
I am running 28752753 and everything looks OK so far

Wonderful, I'll tag a release up soon with this fix

@lamehost Can you test https://github.com/bgp/stayrtr/commit/62f59527762a908a74664b8ce1fc69304f80f84b ? It should have improved performance significantly

My bad, I mean https://github.com/bgp/stayrtr/commit/5a3782245ccacad6b7d22349ed5392aba4d0a6e2 (test caught me off guard there)

It's running now. Looks good so far

Fixed in v0.5.1


Shouldn't ERC20.sol state variables be internal instead of private?

As stated in the title, I don't see how the private state variables (eg. _totalSupply) can be set in the child contract. Please shed some light.

I'm not 100% sure, but I believe that private state variables like _totalSupply are actually inherited, it's just that they aren't "available," as in, it's perfectly possible to define a setTotalSupply() function and have the child contract use it. I think similar behavior also exists in PHP, it's a little unintuitive haha.
Try running this in Remix, deploy B and test publicSetInteger() and testInheritability(): https://gist.github.com/Luiserebii/2c1bea2e856c2eef79466d358db9ab15
I think the confusion probably comes from the word "visibility". I might have the wrong interpretation of the word "inherit," but honestly it's less confusing that way. The documentation doesn't make it super clear either since there isn't an example where a child contract accesses a private "inherited" variable.
I'm probably wrong in some of my definitions, but running that contract should show some evidence towards what I'm getting at.

@Luiserebii In your example you have a setter in the parent contract. ERC20.sol doesn't have those setters.

Hi @Murukaen! The variables are private because we are encapsulating their modification in higher level functions like _mint and _transfer. They act kind of like setters but they ensure that the ERC20 protocol is respected by emitting the necessary events, and by preserving balances and supply.

@frangio I see that the _mint method can be used to set the _totalSupply, but it also sets the balances. In my case, I am building a minable token where the initial balances are 0. How should I go about that with extending ERC20.sol?

It's not possible to do what you're asking. Our ERC20 is made so that all exciting supply is assigned to some account's balance. Can you explain in a bit more detail why this doesn't work for you?

So, as the tokens will be distributed through some simulated mining mechanism, I need the initial balances to be 0, but having a totalSupply nonetheless. Can the ERC20.sol be relaxed in any way to provide this?

@Murukaen Unfortunately I don't see how we could do something like that while maintaining the constraints that led to the current design.
I'd suggest to mint all initial balance to a smart contract that can later distribute it through your minting mechanism.

@frangio You say to initially mint the totalSupply to the contract's address?


hide errors

I would like to hide all errors as such, I am not using the verbose option -v
what to do?
http: error: SSLError: HTTPSConnectionPool(host='', port=): Max retries exceeded with url: / (Caused by SSLError(SSLError("bad handshake: SysCallError(104, 'ECONNRESET')"))) while doing GET request to URL:

Yes, error messages need more love. See #815


fix: made edestruct a tactic keyword

The edestruct tactic was not highlighted as a tactic.

Thanks. Feel free to add any other missing tactics or keywords that you notice.


No. Trainable variables mismatch with the number in the paper

Hello,
I found that when i print the model summary of EfficientDet-D0 that Number of Parameters is: 4.2

While in the paper EfficientDet-D0 Number of Parameters is : 3.9M
from:  Table2

I used input size with 512x512x3 as stated in paper in Table 1:

Also, Thanks for the awesome implementation.

Thanks for pointing this out, let's wait for the official implementation. #15


bugfix: report "destroy" after all volumes of container destroy

fixes #25766
If a container's AutoRemove is enabled, client will wait until it's
removed after container exits, this is implemented based on "destroy"
event.
Currently an "AutoRemove" container will report "destroy" event to
notify a hanging client to exit before all volumes are removed, this is
wrong, we should wait container until everything is cleaned up.
Signed-off-by: Zhang Wei<EMAIL_ADDRESS>

To reproduce the flaky test easily, we can add one line time.Sleep(1*time.Second) after https://github.com/docker/docker/blob/master/daemon/delete.go#L42
E.g.
	err = daemon.cleanupContainer(container, config.ForceRemove)
+      time.Sleep(1*time.Second)
	if err == nil || config.ForceRemove {
		if e := daemon.removeMountPoints(container, config.RemoveVolume); e != nil {
			logrus.Error(e)
		}
	}

	return err

LGTM. Thanks!

LGTM :cat:


Only receiving initial stationary update, not continuous updates

I've been trying to use this plugin, and while I get initial glimpses of success, I cant get it to work the same way the README suggests. Nothing crashes - but I only get the very first location update.
I'm only testing the iOS side of things at the moment, using iOS 8.2. I've been testing on the simulator (enabling the "freeway drive" test pattern) and on a physical iPhone 5S while actually driving. I'm using Cordova iOS 3.7.0.
I'm using exactly the same intiailization code as provided on the README. It's not literally in the deviceready handler, but it is definitely being called after deviceready - my app has user authentication, and I'm configuring background location tracking on login success. When the configuration occurs, I get the following logs:
2015-04-21 15:44:56.963 TradesCloud[59257:53709186] WATCH LOCATION: configure
2015-04-21 15:44:56.964 TradesCloud[59257:53709186] CDVBackgroundGeoLocation configure
2015-04-21 15:44:56.964 TradesCloud[59257:53709186]   - token: (null)
2015-04-21 15:44:56.964 TradesCloud[59257:53709186]   - url: (null)
2015-04-21 15:44:56.964 TradesCloud[59257:53709186]   - distanceFilter: 30
2015-04-21 15:44:56.964 TradesCloud[59257:53709186]   - stationaryRadius: 20
2015-04-21 15:44:56.964 TradesCloud[59257:53709186]   - locationTimeout: 60
2015-04-21 15:44:56.965 TradesCloud[59257:53709186]   - desiredAccuracy: 10
2015-04-21 15:44:56.965 TradesCloud[59257:53709186]   - activityType: 1
2015-04-21 15:44:56.965 TradesCloud[59257:53709186]   - debug: 1
2015-04-21 15:44:56.965 TradesCloud[59257:53709186]   - stopOnTerminate: 0
2015-04-21 15:44:56.968 TradesCloud[59257:53709186] WATCH LOCATION: start
2015-04-21 15:44:56.968 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation start (background? 0)
2015-04-21 15:44:56.969 TradesCloud[59257:53709186] push_pending_updates
2015-04-21 15:44:56.969 TradesCloud[59257:53709186] Synchronising job updates...
2015-04-21 15:44:56.970 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation didChangeAuthorizationStatus 4
2015-04-21 15:44:56.974 TradesCloud[59257:53709186] Location from Phonegap

The "WATCH LOCATION" lines are my own console debug. When I put the app into the background, I see console logs starting background tracking:
2015-04-21 15:46:10.329 TradesCloud[59257:53709186] onPause
2015-04-21 15:46:10.330 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation suspend (enabled? 1)
2015-04-21 15:46:10.331 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation setPace 0, stationaryRegion? 0
2015-04-21 15:46:10.463 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation didUpdateLocations (isMoving: 0)
2015-04-21 15:46:10.469 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation didUpdateLocations (isMoving: 0)
2015-04-21 15:46:10.470 TradesCloud[59257:53709186] - Acquiring stationary location, accuracy: 5.000000
2015-04-21 15:46:11.471 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation didUpdateLocations (isMoving: 0)
2015-04-21 15:46:11.471 TradesCloud[59257:53709186] - Acquiring stationary location, accuracy: 5.000000
2015-04-21 15:46:12.472 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation didUpdateLocations (isMoving: 0)
2015-04-21 15:46:12.472 TradesCloud[59257:53709186] - Acquiring stationary location, accuracy: 5.000000
2015-04-21 15:46:13.472 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation didUpdateLocations (isMoving: 0)
2015-04-21 15:46:13.472 TradesCloud[59257:53709186] - Acquiring stationary location, accuracy: 5.000000
2015-04-21 15:46:13.472 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation queue stationary
2015-04-21 15:46:13.481 TradesCloud[59257:53712212] - CDVBackgroundGeoLocation#sync
2015-04-21 15:46:13.481 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation createStationaryRegion (37.573616,-122.399510)
2015-04-21 15:46:13.481 TradesCloud[59257:53712212]   type: stationary, position: 37.57361598,-122.39950986 speed: 33.29
2015-04-21 15:46:13.482 TradesCloud[59257:53712212] - CDVBackgroundGeoLocation#fireStationaryRegionListener
2015-04-21 15:46:13.482 TradesCloud[59257:53712212] - CDVBackgroundGeoLocation stopBackgroundTask (remaining t:<PHONE_NUMBER>86231570814527423731704356798070567525844996598917476803157260780028538760589558632766878171540458953514382464234321326889464182768467546703537516986049910576551282076245490090389328944075868508455133942304583236903222948165808559332123348274797826204144723168738177180919299881250404026184124858368.000000)
2015-04-21 15:46:13.483 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation queue current
2015-04-21 15:46:13.483 TradesCloud[59257:53712212] - CDVBackgroundGeoLocation#sync
2015-04-21 15:46:13.483 TradesCloud[59257:53712212]   type: current, position: 37.57439449,-122.40010154 speed: 33.87
2015-04-21 15:46:13.489 TradesCloud[59257:53709186] [js] BackgroundGeoLocation callback:  37.57439449,-122.40010154
2015-04-21 15:46:13.489 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation finish
2015-04-21 15:46:13.490 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation stopBackgroundTask (remaining t:<PHONE_NUMBER>86231570814527423731704356798070567525844996598917476803157260780028538760589558632766878171540458953514382464234321326889464182768467546703537516986049910576551282076245490090389328944075868508455133942304583236903222948165808559332123348274797826204144723168738177180919299881250404026184124858368.000000)

I also get 2 notifications saying that I'm stationary (first, with DF: -1, then DF: 30); the blue "App is tracking your location" bar shows briefly, but then goes away. After that, it doesn't matter how long I wait, or how far I drive (either in the simulator, or in real life), I don't get any location updates.
Can you provide any guidance as to why I don't get continuous background tracking?

Yes you must interact with my plugin within the deviceready callback.

Is "location" enable in background modes?

My plugin hosts a simple SampleApp.  Have you tried it?  See README for
instructions.

On Tuesday, April 21, 2015, Russell Keith-Magee<EMAIL_ADDRESS>wrote:

I've been trying to use this plugin, and while I get initial glimpses of
success, I cant get it to work the same way the README suggests. Nothing
crashes - but I only get the very first location update.
I'm only testing the iOS side of things at the moment, using iOS 8.2. I've
been testing on the simulator (enabling the "freeway drive" test pattern)
and on a physical iPhone 5S while actually driving. I'm using Cordova iOS
3.7.0.
I'm using exactly the same intiailization code as provided on the README.
It's not literally in the deviceready handler, but it is definitely
being called after deviceready - my app has user authentication, and I'm
configuring background location tracking on login success. When the
configuration occurs, I get the following logs:
2015-04-21 15:44:56.963 TradesCloud[59257:53709186] WATCH LOCATION: configure
2015-04-21 15:44:56.964 TradesCloud[59257:53709186] CDVBackgroundGeoLocation configure
2015-04-21 15:44:56.964 TradesCloud[59257:53709186]   - token: (null)
2015-04-21 15:44:56.964 TradesCloud[59257:53709186]   - url: (null)
2015-04-21 15:44:56.964 TradesCloud[59257:53709186]   - distanceFilter: 30
2015-04-21 15:44:56.964 TradesCloud[59257:53709186]   - stationaryRadius: 20
2015-04-21 15:44:56.964 TradesCloud[59257:53709186]   - locationTimeout: 60
2015-04-21 15:44:56.965 TradesCloud[59257:53709186]   - desiredAccuracy: 10
2015-04-21 15:44:56.965 TradesCloud[59257:53709186]   - activityType: 1
2015-04-21 15:44:56.965 TradesCloud[59257:53709186]   - debug: 1
2015-04-21 15:44:56.965 TradesCloud[59257:53709186]   - stopOnTerminate: 0
2015-04-21 15:44:56.968 TradesCloud[59257:53709186] WATCH LOCATION: start
2015-04-21 15:44:56.968 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation start (background? 0)
2015-04-21 15:44:56.969 TradesCloud[59257:53709186] push_pending_updates
2015-04-21 15:44:56.969 TradesCloud[59257:53709186] Synchronising job updates...
2015-04-21 15:44:56.970 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation didChangeAuthorizationStatus 4
2015-04-21 15:44:56.974 TradesCloud[59257:53709186] Location from Phonegap
The "WATCH LOCATION" lines are my own console debug. When I put the app
into the background, I see console logs starting background tracking:
2015-04-21 15:46:10.329 TradesCloud[59257:53709186] onPause
2015-04-21 15:46:10.330 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation suspend (enabled? 1)
2015-04-21 15:46:10.331 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation setPace 0, stationaryRegion? 0
2015-04-21 15:46:10.463 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation didUpdateLocations (isMoving: 0)
2015-04-21 15:46:10.469 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation didUpdateLocations (isMoving: 0)
2015-04-21 15:46:10.470 TradesCloud[59257:53709186] - Acquiring stationary location, accuracy: 5.000000
2015-04-21 15:46:11.471 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation didUpdateLocations (isMoving: 0)
2015-04-21 15:46:11.471 TradesCloud[59257:53709186] - Acquiring stationary location, accuracy: 5.000000
2015-04-21 15:46:12.472 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation didUpdateLocations (isMoving: 0)
2015-04-21 15:46:12.472 TradesCloud[59257:53709186] - Acquiring stationary location, accuracy: 5.000000
2015-04-21 15:46:13.472 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation didUpdateLocations (isMoving: 0)
2015-04-21 15:46:13.472 TradesCloud[59257:53709186] - Acquiring stationary location, accuracy: 5.000000
2015-04-21 15:46:13.472 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation queue stationary
2015-04-21 15:46:13.481 TradesCloud[59257:53712212] - CDVBackgroundGeoLocation#sync
2015-04-21 15:46:13.481 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation createStationaryRegion (37.573616,-122.399510)
2015-04-21 15:46:13.481 TradesCloud[59257:53712212]   type: stationary, position: 37.57361598,-122.39950986 speed: 33.29
2015-04-21 15:46:13.482 TradesCloud[59257:53712212] - CDVBackgroundGeoLocation#fireStationaryRegionListener
2015-04-21 15:46:13.482 TradesCloud[59257:53712212] - CDVBackgroundGeoLocation stopBackgroundTask (remaining t:<PHONE_NUMBER>86231570814527423731704356798070567525844996598917476803157260780028538760589558632766878171540458953514382464234321326889464182768467546703537516986049910576551282076245490090389328944075868508455133942304583236903222948165808559332123348274797826204144723168738177180919299881250404026184124858368.000000)
2015-04-21 15:46:13.483 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation queue current
2015-04-21 15:46:13.483 TradesCloud[59257:53712212] - CDVBackgroundGeoLocation#sync
2015-04-21 15:46:13.483 TradesCloud[59257:53712212]   type: current, position: 37.57439449,-122.40010154 speed: 33.87
2015-04-21 15:46:13.489 TradesCloud[59257:53709186] [js] BackgroundGeoLocation callback:  37.57439449,-122.40010154
2015-04-21 15:46:13.489 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation finish
2015-04-21 15:46:13.490 TradesCloud[59257:53709186] - CDVBackgroundGeoLocation stopBackgroundTask (remaining t:<PHONE_NUMBER>86231570814527423731704356798070567525844996598917476803157260780028538760589558632766878171540458953514382464234321326889464182768467546703537516986049910576551282076245490090389328944075868508455133942304583236903222948165808559332123348274797826204144723168738177180919299881250404026184124858368.000000)
I also get 2 notifications saying that I'm stationary (first, with DF: -1,
then DF: 30); the blue "App is tracking your location" bar shows briefly,
but then goes away. After that, it doesn't matter how long I wait, or how
far I drive (either in the simulator, or in real life), I don't get any
location updates.
Can you provide any guidance as to why I don't get continuous background
tracking?
—
Reply to this email directly or view it on GitHub
https://github.com/christocracy/cordova-plugin-background-geolocation/issues/170
.

--
Snet from Gmail Mobile

Doesn't the plugin itself enable "location" in background modes? If not, it seems like something should be added to the readme.

It does auto add it.
On Wednesday, April 22, 2015, Keith C. Ivey<EMAIL_ADDRESS>wrote:

Doesn't the plugin itself enable "location" in background modes? If not,
it seems like something should be added to the readme.
—
Reply to this email directly or view it on GitHub
https://github.com/christocracy/cordova-plugin-background-geolocation/issues/170#issuecomment-95341207
.

--
Snet from Gmail Mobile

But verify
On Wednesday, April 22, 2015, Chris Scott<EMAIL_ADDRESS>wrote:

It does auto add it.
On Wednesday, April 22, 2015, Keith C. Ivey<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

Doesn't the plugin itself enable "location" in background modes? If not,
it seems like something should be added to the readme.
—
Reply to this email directly or view it on GitHub
https://github.com/christocracy/cordova-plugin-background-geolocation/issues/170#issuecomment-95341207
.

--
Snet from Gmail Mobile

--
Snet from Gmail Mobile

Yes, location is enabled as a background mode. The SampleApp also works - which led me to what appears to be the underlying problem: At some point in the past, I didn't authorise background location tracking (possibly when testing another plugin for background location tracking). When I uninstalled the from the simulator and reloaded it, background tracking started working.
So - it appears that the underlying problem is that when the permission for background location tracking hasn't been enabled, there's no obvious record in the logs (or ability to notify the user) that the cause of the problem is device permissions.

So... am I to take it from this that completely silent failure due to lack of permissions is considered either a feature, or desirable behavior? This really doesn't even warrant a line in a log file?

I have no idea.  Obviously doesn't happen often.  Let me know if you can
recreate the issue.
On Thursday, April 23, 2015, Russell Keith-Magee<EMAIL_ADDRESS>wrote:

So... am I to take it from this that completely silent failure due to lack
of permissions is considered either a feature, or desirable behavior? This
really doesn't even warrant a line in a log file?
—
Reply to this email directly or view it on GitHub
https://github.com/christocracy/cordova-plugin-background-geolocation/issues/170#issuecomment-95557840
.

--
Snet from Gmail Mobile


[common][orga] link to pretalx.org instead of the github page

pretalx has got a fancy website. link to it! :)

So it does!


For Cuda

Hi; Maybe possible to do this encoding/decoding,   for Cudatext [uvviewsoft.com]?

I just download CudaText and have a try. It seems it has native support for CJK encodings (at which this plugin aims) via "Reload as" menu items. I might be able to create a plugin for encoding detecting, but I can't find any API to invoke SetFrameEncoding method.

from Asian it has support for - 6 codepages. Are they all, others no need?
Api: the way to support enc, maybe is to handle on_open, then in on_open convert text to Unicode, then call ed.set_text_all(text).... and handle on_save: do saving.

There are different encodings and variants for each language. For example, there are different encodings for Japanese: SHIFT-JIS, EUC-JP, CP932, ISO-2022-JP.
"Only encodings supported by LConvEncoding are shown." Do you mean this? There are "DBCSToUTF8" and "UTF8ToDBCS" methods, but I can't find their implementation. Not sure whether it supports all DBCS encodings or not.
The way you mentioned is just the way this plugin does, but it is inefficient. Native support is better. :laughing:

DBCStoUtf8 can only do this-
      case CodeP of
        936:
          code := Uni936C[SearchTable(CP936CC, code)];
        950:
          code := Uni950C[SearchTable(CP950CC, code)];
        949:
          code := Uni949C[SearchTable(CP949CC, code)];
        932:
          code := Uni932C[SearchTable(CP932CC, code)];
        else
          code := 0;
      end;

so it is same encodings. so I cannot support more in native code..

I've created another repository for CudaText: https://github.com/seanliang/ConvertToUTF8ForCudaText.
Will create another issue there for further discussion.

Posted some issues at https://github.com/seanliang/ConvertToUTF8ForCudaText


Unable to use autocomplete-plugin with Jodit Pro

Jodit PRO Version:  1.3.40
Browser:  Chrome/Firefox
OS:  Ubuntu 22.04
Is React App:  False (we use VueJS)
Code
<template>
    <div ref="jodit">
        <slot />
    </div>
</template>

<script>
import { Jodit } from "jodit-pro";

export default {
    props: {
        modelValue: {
            type: [String, Number],
            required: false
        },
    },

    emits: ["update:modelValue"],

    data() {
        return {
            instance: null,
        };
    },

    mounted() {
        const textareaElement = this.$refs.jodit.querySelector("textarea");

        const names = [
            '@mary',
            '@jain',
            '@entany',
            '@isaak',
            '@ivan'
        ];

        const options = {
            autocomplete: {
                sources: [names]
            },
            emoji: {
                enableAutoComplete: true
            },
            license: 'XXXXX-XXXXX-XXXX-XXXXX'
        };

        console.log(options)

        this.instance = Jodit.make(textareaElement, options);

        this.instance.events.on("change", newValue => this.$emit("update:modelValue", newValue));
    },

    beforeUnmount() {
        this.instance.destruct();
    }
};
</script>

Expected behavior:
As shown in the docs: https://xdsoft.net/jodit/pro/docs/

Actual behavior:
Below a screenshot of our local development environment:

We are using VueJS and do have Jodit PRO working with NPM, but only version 1.3.40 because we can't get v2 to work with our VueJS-import.
The demo in the docs is using v2.0 but this version isn't working for us, when we install "jodit-pro": "^2.0.0-beta.12" we get:
[commonjs--resolver] Failed to resolve entry for package "jodit-pro". The package may have incorrect main/module/exports specified in its package.json.
error during build:
Error: Failed to resolve entry for package "jodit-pro". The package may have incorrect main/module/exports specified in its package.json.

The directory-structure in the node_modules is different, that seems to cause this issue:

@xdan can you please help us out? What do we have to do to get the autocomplete-plugin working?

Looks like we needed to import the plugins separately, this took us ages... Hopefully this post will help someone in the feature:
import { Jodit } from "jodit-pro/es2021/jodit";

import 'jodit-pro/es2021/plugins/autocomplete/autocomplete';

import 'jodit-pro/es2021/plugins/emoji/emoji';
import 'jodit-pro/es2021/plugins/emoji/emoji.css';


Support YAML specs for 2.0

Pretty simple for pyramid_swagger. Need to get bravado-core to support it (issue link) and then update a small amount of code in pyramid_swagger to accept both json and yaml.

As a heads up, I added yaml support (in Yelp/bravado-core#73) to bravado-core, and I'm working on adding yaml support to pyramid-swagger. Progress is being made! =)

The branch is here if you're anxious to see what it looks like: https://github.com/djeebus/pyramid_swagger/tree/add-yaml-support
I'm waiting on bravado-core to release a new version that we can depend on in pyramid-swagger. taps foot impatiently =)

@djeebus bravado-core 4.1.0 is up on pypi now

@analogue fantastic! =) Thanks, I'll get the yaml PR here soon.

Resolved by #156


Improve Exception messages when working on "leader"

Add leader hint to error message. See #573
Open adjust the message:
https://github.com/metafacture/metafacture-core/blob/eab57cd43bcfbd670a0741210332cf4ed0a71adf/metafacture-biblio/src/main/java/org/metafacture/biblio/marc21/Marc21Encoder.java#L160
https://github.com/metafacture/metafacture-core/blob/eab57cd43bcfbd670a0741210332cf4ed0a71adf/metafacture-biblio/src/main/java/org/metafacture/biblio/marc21/Marc21Encoder.java#L208
Also it would be greate to hint, which leader position or position name which is wrong.

Note: don't use titles like Update Marc21Encoder.java.


Server will give an error message if an empty string is tried to get …

…decrypted or encrypted
What does this PR do?
fixes: https://github.com/a-rmz/JEasyCrypto/issues/1
Encrypt/decrypt do not check for empty/null strings #1
Where should the reviewer start?
if (operation.equalsIgnoreCase("encrypt")) {
How should this be manually tested?
(List of steps to reproduce, corroborate or tests to run. Write this section clear enough so that external users can also follow it and test the fix.)

give an empty string for encryption.
give an empty string for decryption.

Any background context you want to provide?
The bug was now crashing the program on the first place as it was claimed to do

As a suggestion, I'd recommend a title like Encrypt or decrypt empty string or something like that, shorter, but gives enough idea on what the PR is about. Also, about the commit messages, same thing.

Looks good 👍, just one thing, please run git rm --cached JEasyCryptoServer/bin/CryptoServer.class so it removes the cached .class file and push the changes without that file again.


Reverse order slide is shown while slide turn by turn and also the slide which is selected is not shown on large thumbnail

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: EXACT RELEASE VERSION OR COMMIT HASH, HERE.

Live Link or JSFiddle/Codepen or website with isssue: PREFERABLY (IF YOU WANT YOUR ISSUE TO BE RESOLVED ASAP).

What you did
Checked Simple Slider working fine or not
Expected Behavior
Slider must be go on to next slide while user click on button turn by turn
Actual Behavior
Slider going to reverse while user try to click button turn by turn
P.S. Remember, an issue is not the place to ask questions. You can use Stack Overflow
for that.

Slide 1

Slide 2

Slide 3

Slide 4

Slide 5

Slide 1

Slide 2

Slide 3

Slide 4

Slide 5

Before you open an issue, please check if a similar issue already exists or has been closed before.

I've faced the same issue.


Demo data

I've been thinking about some demo data to match each case 🤔

user id
name
admin (company id)
admin (garden id)
only permission (company id)
only permission (garden id)

1
Admin
1,2

2
Gardener-1

1

3
Gardener-2

3

4
Gardener-3

4

5
Gardener-4

company id
name
owns (garden id)

1
Company-1
1,2

2
Company-2
3

3
Company-3

4
Company-4
4

garden id
name
image

1
Garden-1
x

2
Garden-2

3
Garden-3
x

4
Garden-4
x

Please give the marked gardeners this picture and the coordinates.
src: "https://cloud.naturerobots.de/apps/files_sharing/publicpreview/xZj9ytRt8WKr5cw?file=/goeoentueuegs_ibbenbueren_new2.jpg&fileId=28565&x=1920&y=1080&a=true",
top_right: new LatLng(52.31724604719058, 7.630553394556046),
top_left: new LatLng(52.3170773725588, 7.630054503679276),
bottom_left: new LatLng(52.31686606722232, 7.630242593586445),

@dobro929 Do you have to have any special Beds config for the gardens?

@pbrozi I looked into the database structure and created the following figure. Currently, it is not possible to add more than one garden to a company, due to the 1 to 1 relationship.

@pbrozi It is currently not possible to add the same garden to multiple companies, due to the 1 to 0...n relationship between them. Could you look into that?

@jarkenau No, we don't need any special configs, just beds 😆

@jarkenau We never talked about a garden belonging to multiple companies. One garden is owned by exactly one company but you can already give access to a garden to any user you want. Isn't that sufficient? Otherwise many changes would be necessary.

@pbrozi Never mind I misunderstood the tables from Timo, everything is fine :grin:

@jarkenau Im logged in as Gardener-3 and i have access to garden (id: 4), where do i get the company id from?
Request i have access to: companies/4/gardens/4

The endpoint GET: companies/{company_id}/gardens returns all gardens of the requested company (company id is in the URL) the user has permissions on.

@pbrozi Yes, but where do i get the company id from?

To access this endpoint, the requesting user has to be user of the company, otherwise anyone could see which gardens a company has. To get all companies the user has permissions on, use the endpoint GET: companies.

GET: companies 200 but empty array ❌
GET: companies/4/gardens 403 Forbidden ❌
GET: companies/4/gardens/4 200 garden object ✔

Okay, I see. If a user has user permissions on a company, he cannot access any gardens. Garden permissions for each garden need to be set, too. If you only have permission for a single garden, you cannot obtain the corresponding company id.
I would implement the following changes:

Every user of a company can get all gardens.
Add an endpoint GET: gardens to get all gardens the user has access on.

@fmarahre and I just noticed that with the latest updates the endpoint GET: companies/{company_id}/gardens/{garden_id}/beds does not work anymore. Even as Admin we get 403 Forbidden.


fix : syntax problem

In the file client\posts\distinguish-between-left-and-right-mouse-clicks\index.tsx  , Line 22, nested use of ** ` **, resulting in run error ;

Fixed in #141


Build

Codecov Report

Merging #2 (92ce4ff) into master (e4d2bd3) will not change coverage.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##              master        #2   +/-   ##
===========================================
  Coverage     100.00%   100.00%
  Complexity        77        77
===========================================
  Files              4         4
  Lines            183       197   +14
===========================================
+ Hits             183       197   +14

Impacted Files
Coverage Δ

src/CurlDownloader.php
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add terraform tile integration for the Datadog Provider

What does this PR do?

Create a new integration tile for the Datadog Terraform Provider
Motivation

Have an in-app tile for this provider
Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Thanks, I applied all suggestions 🙇

Thanks for all the reviews! Merging


Is it possible that --exiftool runs even with --dry-run active?

Describe the bug
I noticed that while running osxphotos export with --update on a previously exported folder; where I deleted one file for testing. But running with --dry-run it seems exiftool attempts to perform an update on destination file.
To Reproduce
Steps to reproduce the behavior:

What' the full command line you used with osxphotos?

TOML file

[export]
#cleanup = true
db = "/Users/Shared/Pictures/iPhoto Shared Library.photoslibrary"
description_template = "Album:{album,}{newline}Description:{descr,}{newline}Person:{person,}"
directory = "{album[/,.|:,.]}"
dry_run = true
exiftool = true
exiftool_merge_keywords = true
exiftool_merge_persons = true
from_date = 2021-12-20T00:00:00
keyword_template = [ "{keyword}", "{label}", "{searchinfo.activity?activity:{searchinfo.activity},}","{searchinfo.venue_type?venue_type:{searchinfo.venue_type},}", ]
not_hidden = true
report = "/Users/Shared/Pictures/Logs/colibri.csv"
retry = 2
skip_original_if_edited = true
strip = true
touch_file = true
update = true
verbose = true

What was the error output?

2022-02-23 21:12:01.662887 -- Loaded options from file /Users/MSP/Pictures/Pictures/Apps/colibri.toml
2022-02-23 21:12:01.662960 -- osxphotos version 0.46.1
2022-02-23 21:12:01.663749 -- exiftool path: /usr/local/bin/exiftool
Warning: export database '/Users/Shared/Pictures/Logs/.osxphotos_export.db' is in a different directory than export destination '/Volumes/photo-1/Family.Photos'
2022-02-23 21:12:52.377013 -- Using export database /Users/MSP/Pictures/Pictures/Logs/.osxphotos_export.db
2022-02-23 21:12:52.377105 -- Upgraded export database /Users/MSP/Pictures/Pictures/Logs/.osxphotos_export.db from version 4.3 to 6.0
2022-02-23 21:12:52,379 - WARNING - photosdb.py - 110 - WARNING: This module has only been tested with macOS versions [10.12, 10.13, 10.14, 10.15, 10.16, 11.0, 11.1, 11.2, 11.3, 11.4, 11.5, 11.6, 12.0, 12.1]: you have Darwin, OS version: 12.2
2022-02-23 21:12:52.381352 -- Processing database /Users/Shared/Pictures/iPhoto Shared Library.photoslibrary/database/photos.db
2022-02-23 21:12:52.384790 -- Processing database /Users/Shared/Pictures/iPhoto Shared Library.photoslibrary/database/Photos.sqlite
2022-02-23 21:12:52.384853 -- Database locked, creating temporary copy.
(...)
2022-02-23 21:13:29.635211 -- Exporting IMG_20211224_223711.JPG (IMG_20211224_223711.JPG) as IMG_20211224_223711.JPG (53/105)
2022-02-23 21:13:29.635553 -- Writing metadata with exiftool for IMG_20211224_223711.JPG
Error exporting photo (4F0AFFB9-BA9B-489B-9970-1D3612E9D014: IMG_20211224_223711.JPG) as IMG_20211224_223711.JPG: [Errno 2] No such file or directory: '/Volumes/photo-1/(...)/IMG_20211224_223711.JPG'
Retrying export for photo (4F0AFFB9-BA9B-489B-9970-1D3612E9D014: IMG_20211224_223711.JPG)
2022-02-23 21:13:29.636103 -- Writing metadata with exiftool for IMG_20211224_223711.JPG
Error exporting photo (4F0AFFB9-BA9B-489B-9970-1D3612E9D014: IMG_20211224_223711.JPG) as IMG_20211224_223711.JPG: [Errno 2] No such file or directory: '/Volumes/(...)/IMG_20211224_223711.JPG'
Retrying export for photo (4F0AFFB9-BA9B-489B-9970-1D3612E9D014: IMG_20211224_223711.JPG)
2022-02-23 21:13:29.636644 -- Writing metadata with exiftool for IMG_20211224_223711.JPG
Error exporting photo (4F0AFFB9-BA9B-489B-9970-1D3612E9D014: IMG_20211224_223711.JPG) as IMG_20211224_223711.JPG: [Errno 2] No such file or directory: '/Volumes/(...)/IMG_20211224_223711.JPG'
2022-02-23 21:13:29.637514 -- Exporting IMG_20211224_223701.JPG (IMG_20211224_223701.JPG) as IMG_20211224_223701.JPG (54/105)
2022-02-23 21:13:29.639487 -- Writing metadata with exiftool for IMG_20211224_223701.JPG
2022-02-23 21:13:29.798838 -- Exported updated file /Volumes/(...)MG_20211224_223701.JPG
(...)

Expected behavior
Would expect exiftool not to be run with dry-run option.
Screenshots
N/A
Desktop (please complete the following information):

OS: MacOS 12.2.1
osxphotos version (osxphotos --version): 0.46.1

Additional context
.osxphotos_db file was probably updated due to the most recent changes in osxphotos.

If this is happening, it shouldn't be. --dry-run should make absolutely no changes to the file system.
I would have though this code was blocked by this test -- will need to investigate why it's happening.
https://github.com/RhetTbull/osxphotos/blob/9964fd063522dea261bb0066a4afa18ee2751fb2/osxphotos/photoexporter.py#L1306-L1316

It's not exiftool, it's the --touch-file code that's trying to do a stat on non-existent files.  Working on a fix. (But no files were actually getting touched with --dry-run)

I think this is fixed in 0.46.4 which I'm about to release shortly.  The code was trying to stat a file to see if touch times needed to be updated but the file wasn't there (because it was running in dry run mode).  Came from a recent refactoring of the export logic.

Fixed in v0.46.4


Zoom in/out using ctrl

Hi !
I'd like to enable zoom in/out with mouse wheel only when clicking on ctrl key. This in order to prevent zoom in/out when I'm scrolling the web page.
Is there a way to do it ?
Thank's in advance !
Best regards

I haven't implemented anything in the Dash layer, but there seems to be a few JS solutions available,
https://stackoverflow.com/questions/45982643/leaflet-js-use-ctrl-scroll-to-zoom-the-map-move-map-with-two-fingers-on-mob

I haven't implemented anything in the Dash layer, but there seems to be a few JS solutions available,
https://stackoverflow.com/questions/45982643/leaflet-js-use-ctrl-scroll-to-zoom-the-map-move-map-with-two-fingers-on-mob

I haven't implemented anything in the Dash layer, but there seems to be a few JS solutions available,
https://stackoverflow.com/questions/45982643/leaflet-js-use-ctrl-scroll-to-zoom-the-map-move-map-with-two-fingers-on-mob

I haven't implemented anything in the Dash layer, but there seems to be a few JS solutions available,
https://stackoverflow.com/questions/45982643/leaflet-js-use-ctrl-scroll-to-zoom-the-map-move-map-with-two-fingers-on-mob

I found this solution yesterday also, but I did not succeed to implemented it. I read that Jquery DOM events were not compatible with react components.
But I did not insist. I will try harder in the next few days and let you know If I found a working solution.

I found this solution yesterday also, but I did not succeed to implemented it. I read that Jquery DOM events were not compatible with react components.
But I did not insist. I will try harder in the next few days and let you know If I found a working solution.

I re-try to enable zoom in/out using ctrl, but the provided solution need to access elements using the DOM. Unfortunately, it seems impossible to access DOM element on a react application.
So, it seems to be a dead end. If anyone have any other idea ? Maybe this improvement should be implemented first in react-leaflet ?

At a glance, I think the best approach would be to add a new component that wraps,
https://github.com/elmarquis/Leaflet.GestureHandling
It seems to do what you need - and more (:

I just made a quick wrapper. It's used like this,
import dash
import dash_leaflet as dl

app = dash.Dash()
app.layout = dl.Map([dl.TileLayer(), dl.GestureHandling()], style={'width': '1000px', 'height': '500px'})

if __name__ == '__main__':
    app.run_server()

You can try it in out in dash-leaflet==0.1.16rc3 :)

That's work perfectly ! Thank you very much ! :+1:

The feature has been incorporated in the dash-leaflet==0.1.16 release.

@remiOdite Thanks for posting the issue! I have just added GestureHandling components to a few of my apps, and only now I realize how much the default scrolling behavior annoyed me (:


Adding some pictures to website

http://www.stat.washington.edu/mmp/megaman.html
here is a place from which you can extract text. i made this
independently of jake, but now i think it's superfluous and i will
delete it (as soon as you don't need it any longer for inspiration)
--

  Marina Meila

,_ o  ACMS Director         University of Washington       Box  354322
/ //\   Professor of Statistics                  Seattle WA 98195-4322
>>_|<EMAIL_ADDRESS>             phone:<PHONE_NUMBER>
\,     www.stat.washington.edu/mmp      office: Padelford B-321

I saved the HTML file of this for reference so you can delete it if you
want.
I think the "best" way to do this might be to have a second image with the
caption included in the .png file. Then when you click the first image it
re-directs to the one with the caption.
Jake might have a better idea.
-James
On Mon, Mar 7, 2016 at 4:00 PM, mmp2<EMAIL_ADDRESS>wrote:

http://www.stat.washington.edu/mmp/megaman.html
here is a place from which you can extract text. i made this
independently of jake, but now i think it's superfluous and i will
delete it (as soon as you don't need it any longer for inspiration)
--

Marina Meila
,_ o ACMS Director University of Washington Box 354322
/ //\ Professor of Statistics Seattle WA 98195-4322
>>_|<EMAIL_ADDRESS>phone:<PHONE_NUMBER>
\, www.stat.washington.edu/mmp office: Padelford B-321

—
Reply to this email directly or view it on GitHub
https://github.com/mmp2/megaman/pull/32#issuecomment-193514381.

how about a html file including the image (larger) and the caption? when
you click on the small image, you get to this one. if we want to edit the
caption, html is much easier.
Marina
On 3/7/16, 4:05 PM, "James McQueen"<EMAIL_ADDRESS>wrote:

I saved the HTML file of this for reference so you can delete it if you
want.
I think the "best" way to do this might be to have a second image with the
caption included in the .png file. Then when you click the first image it
re-directs to the one with the caption.
Jake might have a better idea.
-James
On Mon, Mar 7, 2016 at 4:00 PM, mmp2<EMAIL_ADDRESS>wrote:

http://www.stat.washington.edu/mmp/megaman.html
here is a place from which you can extract text. i made this
independently of jake, but now i think it's superfluous and i will
delete it (as soon as you don't need it any longer for inspiration)
--

Marina Meila
,_ o ACMS Director University of Washington Box 354322
/ //\ Professor of Statistics Seattle WA 98195-4322
>>_|<EMAIL_ADDRESS>phone:<PHONE_NUMBER>
\, www.stat.washington.edu/mmp office: Padelford B-321

�
Reply to this email directly or view it on GitHub
https://github.com/mmp2/megaman/pull/32#issuecomment-193514381.

�
Reply to this email directly or
view it on GitHub
https://github.com/mmp2/megaman/pull/32#issuecomment-193516347.

I agree that we should make the caption easy to edit. I couldn't make the re-direct work so I'll have to figure out the best way to do this I might just have to work in html instead of the Sphinx syntax.
James

On Mar 7, 2016, at 5:38 PM, mmp2<EMAIL_ADDRESS>wrote:
how about a html file including the image (larger) and the caption? when
you click on the small image, you get to this one. if we want to edit the
caption, html is much easier.
Marina
On 3/7/16, 4:05 PM, "James McQueen"<EMAIL_ADDRESS>wrote:

I saved the HTML file of this for reference so you can delete it if you
want.
I think the "best" way to do this might be to have a second image with the
caption included in the .png file. Then when you click the first image it
re-directs to the one with the caption.
Jake might have a better idea.
-James
On Mon, Mar 7, 2016 at 4:00 PM, mmp2<EMAIL_ADDRESS>wrote:

http://www.stat.washington.edu/mmp/megaman.html
here is a place from which you can extract text. i made this
independently of jake, but now i think it's superfluous and i will
delete it (as soon as you don't need it any longer for inspiration)
--

Marina Meila
,_ o ACMS Director University of Washington Box 354322
/ //\ Professor of Statistics Seattle WA 98195-4322
>>_|<EMAIL_ADDRESS>phone:<PHONE_NUMBER>
\, www.stat.washington.edu/mmp office: Padelford B-321

�
Reply to this email directly or view it on GitHub
https://github.com/mmp2/megaman/pull/32#issuecomment-193514381.

�
Reply to this email directly or
view it on GitHub
https://github.com/mmp2/megaman/pull/32#issuecomment-193516347.

—
Reply to this email directly or view it on GitHub.


Evaluation results

I have noticed the evaluation results for the half-life regression model are slightly different each time I run the code. So I was wondering how the evaluation results presented in the paper are determined.
Are the results given in table 2 of the paper based on one run or are they based on multiple runs (using the best result or the average or something else)?
Thanks!

It is a property of stochastic gradient descent that you will get slightly different answers for each run. However, on this particular data set, results didn't vary more than 1%, and were pretty stable, so we reported only a single pass.
If you're more comfortable doing multiple passes, you can easily modify the code to do that, and average over them...


Forms fixes & improvements

Shortcut: https://app.shortcut.com/cartoteam/story/301431/core-new-size-xsmall-for-iconbutton && https://app.shortcut.com/cartoteam/story/353403/fix-checkbox-and-radio-sizes
[sc-301431][sc-353403]

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 71.245%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2811

Relevant Lines:
3635

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 71.245%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2811

Relevant Lines:
3635

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

3 of 4 (75.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 71.295%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/react-ui/src/theme/sections/components/buttons.js
1
2
50.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
2814

Relevant Lines:
3639

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

3 of 4 (75.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 71.272%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/react-ui/src/theme/sections/components/buttons.js
1
2
50.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
2814

Relevant Lines:
3639

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

3 of 4 (75.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 71.272%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/react-ui/src/theme/sections/components/buttons.js
1
2
50.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
2814

Relevant Lines:
3639

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

3 of 7 (42.86%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.05%) to 71.198%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/react-ui/src/theme/sections/components/buttons.js
1
2
50.0%

packages/react-ui/src/theme/sections/components/forms.js
2
5
40.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.05%

Covered Lines:
2814

Relevant Lines:
3639

💛  - Coveralls


Replaced empty state and loading svgs with Lottie file animations

Subtle animations... so nice....
The it's using a new-ish format called a Lottie file. Very compact.
If you don't like these animations they're easy to tweak.

Hope this works...


Update tutorial-use-metrics-and-diags.md

For updating the cli stick with the new version of the iot extension "az extension update --name azure-iot" rather than the legacy "azure-iot-cli-ext"

@bamajeed : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@robinsh - Could you review the proposed changes?
Thanks.

@robinsh, you can only sign off in the main PR conversation, not in review comments.

#sign-off
I couldn't remember if I can sign off, or if the OP has to do it. Thanks @prmerger10!


Split extremal-python-dependencies out into its own subpackage

With this, extremal-python-dependencies can be installed from another package's CI script.  Hopepfully this will be a step toward allowing us to avoid some code duplication across repositories.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2410

Relevant Lines:
2410

💛  - Coveralls


.github: show diff when gofmt check fails

Rather than just failing the check, also show the diff.

Note: package workerpool doesn't have any external dependency so I actually don't expect this check to ever fail :)

Note: package workerpool doesn't have any external dependency so I actually don't expect this check to ever fail :)

I think it still can fail for PRs where contributors don't run gofmt automatically (e.g. via editor integration) on their changes, no?

@tklauser I yes, good point I missed that!


Include name of downstream users in comments

We can't reasonably copy the name of all users when syncing comments -- the information is too sensitive.
So right now this is getting synced as:

This comment was posted by the user :58fa1ced-171a-4c00-97e8-5d70d442cc4b.
Please ignore – just testing webhooks.

BUT... for team members, it's actually legitimate and reasonable to display the name.
Maybe we could change the comment syncing so that the reference to the user actually points to the "downstream" user?

so something like for this comment ? https://hibernate.atlassian.net/browse/HSEARCH-5257?focusedCommentId=117035
If so, we probably can do the same for the description as well (we already have the assignee and reporter set, but if we update the comment then might as well do the same logic for description too)

ok, so it seems we can do that, yes 😄


Update version to 2.0-SNAPSHOT

It is somehow confusing that a branch for 1.3.x exits, but master is also 1.3.x. Plus, master was at 1.3.2-SNAPSHOT and 1.3.3 was already released.

It is somehow confusing that a branch for 1.3.x exits,

Master contains upgrade to Jakarta EE APIs and OpenTracing 0.33.0. The next version will be 2.0. We do the version bump during the release.
Consider updating the versions to 2.0

Done!

thanks @radcortez


test: Replace "pass" with "skip".

This makes the test output a little less pretty, but more accurately reflects the fact that we're skipping tests of unimplemented behavior.
cc @UniqMartin

Definitely the right thing to do. Thanks for rectifying this!

This makes the test output a little less pretty, […]

It might also create an incentive to do something about this and implement the currently missing tests (though the tests that are being skipped here are definitely not high priority).


Uncaught TypeError: Cannot read property 'path' of undefined

"vue": "^2.0.1",
"vue-router": "^2.0.0",
"vuex": "^2.0.0",
"vuex-router-sync": "^3.0.0"

routes
const routes = [
  {
    path: '/',
    component: Home
  },
  {
    path: '/about',
    component: About
  }
]

has error
import { sync } from 'vuex-router-sync'
import router from './router'
import store from './store'

sync(store, router)

no error
// import { sync } from 'vuex-router-sync'
import router from './router'
import store from './store'

// sync(store, router)

make sure you install<EMAIL_ADDRESS>and not 2.x

The doc should be updated since there isn't a @next branch available on npm now

@Smallpath The README.md says it. I just totally overlooked it. It was last updated 3 months ago so it must have been in there a while. I think this is good to close.

[Uncaught TypeError: Cannot read property 'route' of undefined ]  when i installed 3.0.0

The default install is now compatible with vue-router and vuex 2.x. If there are still errors with the latest version, please open a separate issue.

Uncaught TypeError: Cannot read property 'path' of undefined
"vuex-router-sync": "^4.1.2"
"vue": "^2.2.2",
"vue-router": "^2.3.0",

Same for me
"vue": "^2.2.6",
"vue-router": "^2.4.0",
"vuex-router-sync": "^4.1.2"

please open a separate issue.

....

"vue": "^2.3.4",
"vue-router": "^2.5.3",
"vuex-router-sync": "^4.1.2"
[vue-router] uncaught error during route navigation:
TypeError: Cannot read property 'path' of undefined

I won't repeat my request a third time. Locking this now.


docs(README): put README to root folder

The original README in the root folder is outdated and make people
confused. Consolidating READMEs also help developer update the doc more
easily.

Don't be scared at the big change. I simply moved root/backend/README.md to root/README.md.


Configure dependabot behavior

It is introduced not to watch deprecated images.
ref. https://docs.github.com/en/code-security/supply-chain-security/keeping-your-dependencies-updated-automatically/configuration-options-for-dependency-updates

Moving obsolete recipes should be done in another PR.


task::Context::new is incompatible when std feature is enabled/disabled

// With `std` feature
fn new(map: &'a mut LocalMap, waker: &'a Waker, executor: &'a mut Executor) -> Context<'a> { ... }

// Without `std` feature
fn new(map: &'a mut LocalMap, waker: &'a Waker) -> Context<'a> { ... }

I realise that implementing an executor that's supposed to work for both std and no_std environments is unlikely. But it might be nice for testing purposes to be able to compile it on both without requiring multiple code paths (I have a no_std executor, but I have yet to look into testing it).

Actually, I now realise that when testing it's still fine to compile without futures-core/std enabled. Only the tests themselves will need to be not no_std. Still, having an API add an extra argument when a feature is enabled seems wrong to me.

I think it's basically a hint that you shouldn't try to write an executor for both std and no_std.  Choose one and make it good.
In any case your executor needs to be aware whether it's std or no_std.
(Also: Executor is a trait that doesn't make sense in no_std - it would be empty if you would define it for no_std.  So again your executor needs to be aware whether it's std or no_std.)


AssertionError: Default process group is not initialized

[10/16 15:51:32 pods.engine.trainer]: Starting training from iteration 0
ERROR [10/16 15:51:32 pods.engine.trainer]: Exception during training:
Traceback (most recent call last):
File "/media/sda6/yhh/FCOS/BorderDet/cvpods/engine/trainer.py", line 89, in train
self.run_step()
File "/media/sda6/yhh/FCOS/BorderDet/cvpods/engine/trainer.py", line 193, in run_step
loss_dict = self.model(data)
File "/home/yons/anaconda3/envs/borderdet140-yhh/lib/python3.7/site-packages/torch/nn/modules/module.py", line 532, in call
result = self.forward(*input, **kwargs)
File "/media/sda6/yhh/FCOS/BorderDet/cvpods/modeling/meta_arch/borderdet.py", line 167, in forward
bd_box_delta,
File "/media/sda6/yhh/FCOS/BorderDet/cvpods/modeling/meta_arch/borderdet.py", line 237, in losses
dist.all_reduce(num_foreground)
File "/home/yons/anaconda3/envs/borderdet140-yhh/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py", line 901, in all_reduce
_check_default_pg()
File "/home/yons/anaconda3/envs/borderdet140-yhh/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py", line 193, in _check_default_pg
"Default process group is not initialized"
AssertionError: Default process group is not initialized
[10/16 15:51:32 pods.engine.hooks]: Total training time: 0:00:00 (0:00:00 on hooks)

soft link to Outputs/model_logs/cvpods_playground/detection/coco/borderdet/borderdet.res101.fpn.coco.800size.2x
Command Line Args: Namespace(dist_url='tcp://<IP_ADDRESS>:50152', eval_only=False, machine_rank=0, num_gpus=1, num_machines=1, opts=[], resume=False)
[10/16 16:29:02 cvpods]: Rank of current process: 0. World size: 1
[10/16 16:29:02 cvpods]: Environment info:

sys.platform            linux
Python                  3.7.9 (default, Aug 31 2020, 12:42:55) [GCC 7.3.0]
numpy                   1.18.2
cvpods                  0.1 @/media/sda6/yhh/FCOS/BorderDet/cvpods
cvpods compiler         GCC 5.5
cvpods CUDA compiler    10.1
cvpods arch flags       sm_61
cvpods_ENV_MODULE
PyTorch                 1.4.0 @/home/yons/anaconda3/envs/borderdet140-yhh/lib/python3.7/site-packages/torch
PyTorch debug build     False
CUDA available          True
GPU 0                   GeForce GTX 1080 Ti
CUDA_HOME               /usr/local/cuda-10.1
NVCC                    Cuda compilation tools, release 10.1, V10.1.105
Pillow                  6.2.0
torchvision             0.5.0 @/home/yons/anaconda3/envs/borderdet140-yhh/lib/python3.7/site-packages/torchvision
torchvision arch flags  sm_35, sm_50, sm_60, sm_70, sm_75
cv2                     4.4.0

PyTorch built with:

GCC 7.3
Intel(R) Math Kernel Library Version 2020.0.2 Product Build 20200624 for Intel(R) 64 architecture applications
Intel(R) MKL-DNN v0.21.1 (Git Hash 7d2fd500bc78936d1d648ca713b901012f470dbc)
OpenMP 201511 (a.k.a. OpenMP 4.5)
NNPACK is enabled
CUDA Runtime 10.1
NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_61,code=sm_61;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_37,code=compute_37
CuDNN 7.6.3
Magma 2.5.1
Build settings: BLAS=MKL, BUILD_NAMEDTENSOR=OFF, BUILD_TYPE=Release, CXX_FLAGS= -Wno-deprecated -fvisibility-inlines-hidden -fopenmp -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -O2 -fPIC -Wno-narrowing -Wall -Wextra -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-unused-variable -Wno-unused-function -Wno-unused-result -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Wno-stringop-overflow, DISABLE_NUMA=1, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, USE_CUDA=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON, USE_STATIC_DISPATCH=OFF,

[10/16 16:29:02 cvpods]: Command line arguments: Namespace(dist_url='tcp://<IP_ADDRESS>:50152', eval_only=False, machine_rank=0, num_gpus=1, num_machines=1, opts=[], resume=False)

in cvpods/engine/launch.py
I changed
if world_size > 1:
# https://github.com/pytorch/pytorch/pull/14391
# TODO prctl in spawned processes
to
if world_size >= 1:
# https://github.com/pytorch/pytorch/pull/14391
# TODO prctl in spawned processes
it works

Well, seems you are using 1-GPU during training? Such an error shouldn't happen. Could you please provide command your are using ?

Since the reporter doesn't reply for a week, We close this issue.


Maui support on .NET 9

Issue: NET9 upgrade in Maui creates compile issue
ScottPlot Version: 5.0.45 (in Net9 Maui)
XAML
<Grid ColumnDefinitions="*" RowDefinitions="*" Padding="0,0,0,0" >
    <ScottPlot:MauiPlot x:Name="MauiPlot1" />
</Grid>

Code-Behind
private void SimpleDrawPhasorPlot()
{
    //Create arrow base coordinates used for all vectors
    Coordinates arrowBase = new(0, 0);

    // create arrow 1
    Coordinates arrow1Tip = new(5, 0);
    CoordinateLine arrow1Line = new(arrowBase, arrow1Tip);

    // add a simple arrow
    var arrow1 = MauiPlot1.Plot.Add.Arrow(arrow1Line);
    arrow1.ArrowFillColor = Colors.Red;
    arrow1.ArrowLineWidth = 0;
    arrow1.ArrowWidth = 5;
    arrow1.ArrowheadLength = 20;
    arrow1.ArrowheadAxisLength = 20;
    arrow1.ArrowheadWidth = 20;
    var dl1 = MauiPlot1.Plot.Add.Text("T(1)", arrow1Tip);
    dl1.LabelFontSize = 28;
    dl1.LabelFontColor = Colors.Black;

    MauiPlot1.Plot.HideGrid();
    MauiPlot1.Plot.Layout.Frameless();

    // Preserve X-axis plot scaling during window size changes
    SquarePreserveX rule2 = new(xAxis: MauiPlot1.Plot.Axes.Bottom, yAxis: MauiPlot1.Plot.Axes.Left);
    MauiPlot1.Plot.Axes.Rules.Clear();
    MauiPlot1.Plot.Axes.Rules.Add(rule2);
}

Compile errors

If it matters, please note the change in creating Maui AppShell. The previous method
MainPage = new AppShell();  // deprecated

has been deprecated. New method is
protected override Window CreateWindow(IActivationState? activationState)
{
    return new Window(new AppShell());
}

Some more information during rebuild

Hi @King-Taz, I made a new app targeting .NET 9 following the directions on https://scottplot.net/quickstart/maui/ and it seems to work as expected.
Perhaps your question relates more to how to upgrade Maui apps from .NET 8 to .NET 9? If this is the case, the topic may be out of scope of the issues section for this repository.

Either way, hope it helps!

Thanks for the quick reply and help! This was the issue
<Project Sdk="Microsoft.NET.Sdk">
	<PropertyGroup>
		<WindowsSdkPackageVersion>10.0.19041.34</WindowsSdkPackageVersion>
                ...

I forgot i added that, should have read the instructions! Have a great weekend!


Release v0.5.12

Introduces changes of external-dns v0.5.12

/assign @Raffo

Happy to approve the changelog once the release has landed! (can't see it still)

Should be there now 😄. Thanks @Raffo 🎉 .

Woop woop! 🎉
/approve
/lgtm


Can dependency on core-js (15MB) be removed?

First of all, thank you for creating this package.
I realised that the package has a dependency on core-js which is huge (15MB).
I searched in the code, it seems to be used in only these two lines in src/index.ts for polyfilling the "at" function:
import "core-js/stable/array/at";
...
    if (this.endsWithSoftHyphen && this.items.at(-2)?.type === "glue") {
...
    return isSoftHyphen(this.items.at(-1));

Would it be better if dependency on core-js is removed?
Background: I am trying to use the package in an AWS Lambda function, additional huge depdency made the bundle size exceeds AWS's limit.

Done ✅


ossl_dh_check_priv_key: Do not fail on private keys without q

Fixes #18098
Checklist

[ ] tests are added or updated

I'll add tests later.
@slontis An alternative would be to not call ossl_dh_check_priv_key from the provider if dh->q is NULL. Which one do you prefer?

Since the check is about a known safe prime, we can calculate Q.
But since the check should be about a known safe prime, and the bug report was about a generated safe prime, I don't understand why does this patch make a difference?

Since the check is about a known safe prime, we can calculate Q.
But since the check should be about a known safe prime, and the bug report was about a generated safe prime, I don't understand why does this patch make a difference?

The function as a whole is NOT just about checking known safe primes. And it MUST NOT fail if the parameters are not a known safe prime. So we do not really know q.

There is a part of the code that is abou safe primes and that is changed.

Also, the function that is called does not support calling it with 2^N for a non-approved safe prime. So either we can calculate Q or need it.

I am sorry, Kurt, I do not understand what do you want. The function is called for any DH private keys and it will currently fail for those with no q. As the function should be general (it is not called ossl_dhx_or_known_safe_prime_check_priv_key) it MUST NOT fail on keys without q.
Or do you think the alternative as mentioned in my above comment addressed to Shane is better? Then the function should be documented that it won't work on keys without q and it must never be called from other places if q is not present.

We're calling the ossl_ffc_validate_private_key() function. That function is documented as:
See SP800-56Ar3 Section <IP_ADDRESS>.2: Owner assurance of Private key validity. Verifies priv_key is in the range [1..upper-1]. The passed in value of upper is normally params->q but can be 2^N for approved safe prime groups. Note: This assumes that the domain parameters are valid.
This change breaks the condition in which the function produces correct output, because you pass 2^N for something that is not an approved safe prime.
As far as I know, all DH keys we have in the source are safe primes. If it's a safe prime, you can calculate Q.
If it's not a safe prime, as far as I know, you still need to know Q to be able to do the check.
I think it's wrong for this function to just skip the check if Q is not present and say everything is OK. Either it needs to return that it can't check it, or it should not have been called.

The code seems to assume that if DH_get_nid() says it knows it that it's an approved safe prime. I doubt that all the keys we internally have are approved safe primes. I also wonder if an application can add it's own primes to the list. It at least looks more sane to never use the 2^N.

Kurt, the key is not broken, the EVP_PKEY_check() on it should not fail. What is your alternative proposal, how to handle it.

My requirements:

If you can't tell it's broken or not, don't claim everything is ok and return 1.
Don't call ossl_ffc_validate_private_key() with invalid parameters

So there are several options which you can go with:

This is expected behaviour, calling that function without Q is an error.
If Q is not present, and it's a safe prime calculate Q.
If Q is not present, and you don't know if it's a safe prime, check that it's a safe prime. This might be more expensive than what we want to do, but since you're checking the private key, it means you don't trust the private key and probably shouldn't trust the prime either.
Change the code so it can indicate it's not checked. This is probably an API change. We document -2 as algorithm not supported which doesn't really match, and let the calling code deal with how to handle it.
If you know you can't check it, don't check it.

I'm not sure we have a format in which we can save Q, making it more difficult to use non-standard primes. But you should probably avoid them in the first place.

Breaking the API is clearly a no-go.
Failing EVP_PKEY_check() on keys we generated is clearly a bug.
Checking whether a p is a safe prime would be possible, however that is a little bit expensive check, not sure it is a good idea to do it.

My suggestion is then to not call ossl_ffc_validate_private_key() in case Q is not present, and also use l = dh->length ? dh->length : BN_num_bits(dh->params.p) - 1 and check that it's in that range.

Note that the dh->length might be present in the file. I'm not sure if we save that or not.

The intention of the function was to fail if there is no q.. In terms of a 'proper' key check that is what it should do... A private key without a 'q' cannot really be deemed to be valid..

Then why does the openssl library/command line utility generate such keys?

The intention of the function was to fail if there is no q.. In terms of a 'proper' key check that is what it should do... A private key without a 'q' cannot really be deemed to be valid..

@slontis, that's a nonsense. Why should it be? If the p is a safe-prime, storing the q is just redundant.

The case where q is missing is not (always) for the case where p is a safe prime, see for instance PKCS#3 and RFC 2631. The private key check should match the range we use for generating such keys. I think what we implement in case of a missing q is PKCS#3, so the check should probably be 2^(l-1) ≤ x < 2^l.
We should probably deprecate PKCS#3.

The intention of the function was to fail if there is no q.. In terms of a 'proper' key check that is what it should do... A private key without a 'q' cannot really be deemed to be valid..

@slontis, that's a nonsense. Why should it be? If the p is a safe-prime, storing the q is just redundant.

I was not talking about the case where q can be regenerated, I am talking about the other 'types' such as fips186-4/2.

We should probably deprecate PKCS#3.

This will be difficult for <= TLSv1.2. The DH implementation there does not transmit the "q" value.

This will be difficult for <= TLSv1.2. The DH implementation there does not transmit the "q" value.

If they send a known prime, we internally do know q, and afaik, use it.

If they send a known prime, we internally do know q, and afaik, use it.

But they are not required to send a known prime and I do not think we should break connections with such peers. Anyway this is IMO off topic.

ping for the second review

It needs more tests.

I am not going to spend more time on artificial test cases for this, I am sorry. This is really not good use of my time. Anybody who likes to can add more testcases later.

@slontis please reapprove
also ping for second review

Sorry @t8m don't think I'm competent enough :(

This pull request is ready to merge

Merged to master and 3.0 branches. Thank you for the reviews.


Poor Track Time

Hello! Thank you for this repo, it's been very helpful.
I've installed with the instructions and got both a run_video_slam example and run_camera_slam working, but I'm getting some very poor track times with the default config file in both of these cases, ~1500ms per frame with 2000 keypoints. This goes down to ~800ms per frame at 500 keypoints, so still very poor. This is at an already scaled-down resolution of 960x480. Do you have any idea what the reason for this could be?
I'm running on Intel i7 11800H. The ORB parameters are as follows:
Feature.max_num_keypoints: 2000
Feature.scale_factor: 1.2
Feature.num_levels: 8
Feature.ini_fast_threshold: 20
Feature.min_fast_threshold: 7

Do you have any idea what the reason for this could be?

Sorry, not really – I'm no longer working with any of this stuff.
Unfortunately the original OpenVSLAM project has been terminated, as you probably know. So I guess it might be advisable to use some other library for which somebody is available to answer questions.

No worries, thank you for the response.


UI: Should have some search box for Services page

When there are huge data in Services page, user has to scroll down to find desired service
There should be a way for user to filter services on UI side

Hello, can I be assigned to this issue?

Sure @jozefkopcak
Please go ahead and make the changes and feel free to ping on Slack or comment here for any help/suggestions.


Add support for env-var-based SSL configuration

If I try to specify the SSL config via env-vars in a non-rails app, I get the following error:
Racecar::ConfigError: unknown config variable RACECAR_SSL_CA_CERT

This PR allows ssl_ca_cert, ssl_ca_cert_file_path, ssl_client_cert and ssl_client_cert_key to be passed via env-vars.

Sorry for the late response, I totally missed this!
#27 should take care of this and work for all config variables from now on.

@recurser can you try running against master and see if this works for you?

Hi @dasch - yes #27 fixes the problem 👍 Closing this.
Thanks!


Auto sized Window does not get updated when app window is resized larger

Describe the bug
When having a auto sized Window, texts get squished when the app window is small (see screenshots).
This is to be expected. However, it does not return to normal after the window is resized larger.

To Reproduce
Steps to reproduce the behavior:
0. This is just speculation, didn't actually test if this reprod would work.

Make a auto_sized() Window
Put in some texts
Resize the application window smaller that the text is forced to wrap
(Maybe you need to be on Linux/X11 too?? we can't verify this ourselves)
Undo that resize, i.e. make the window larger
The text doesn't get their wrapping, well, unwrapped

Expected behavior
For it not to do that, not to wrap the text even though there's enough space.
Screenshots

Desktop (please complete the following information):

OS: Arch Linux on X11
Egui Version: 0.27.1

I think this bug was introduced by https://github.com/emilk/egui/pull/4199
I think the fix is: When clamping window size to the viewport size we should not store it in the state.

@msparkles
I think using ScrollArea will help.
        egui::ScrollArea::both()
            .show(ui, |ui| {
                ui.horizontal_wrapped(|ui| {
                    ui.label("test test test");
                });
            });

Potential fixes:

When clamping window size to the viewport size we should not store it in the state.
Ognore the stored size for auto-sized windows

This has nothing to do with #4199.
#4199 limits the maximum size to screen_rect(), and has no effect elsewhere.
However, #4199 has significant limitations, so for future development, I will submit a Pull Request with minimal limitations.

Related #4349


How to backup/restore systems.users

Re: RESTORE
Issue Description
The system.users documentation here doesn't work as expected -> I believe you have to separately backup the system table, and then restore it into and existing DB and table

@dianasaur323, the docs say that every backup includes the systems.users table. Is that not correct? Can you please provide more details about this not working as expected?

@jseldess yes, happy to provide more detail. So this PR https://github.com/cockroachdb/cockroach/pull/19975 removed systems.users and systems.descriptors from backups. This means that if you want to backup systems.users, you have to back it up intentionally. Also, when I tried restoring the users table into an existing DB, I ended up with a primary key violation. So, I think the correct path here is 1) backup system.users, 2) create DB you want to restore into, 3) create new table, 4) restore into new table, 5) delete all the existing rows from the old system.users table, 6) copy rows from new table into old table, 7) drop new table. cc @dt also note that I haven't had time to manually test this myself yet.

Dup of https://github.com/cockroachdb/docs/issues/5002.


Error when associating object from another package

Describe the bug
When a model is in a different package and try to create a relation with KomapperOneToMany() for example you get the following error :  The targetEntity must be annotated with either @KomapperEntity or @KomapperEntityDef.
Expected behavior
When the target is properly annotated it should be properly resolved even if its is in a different package.
Actual behavior
 The targetEntity must be annotated with either @KomapperEntity or @KomapperEntityDef.
Steps to reproduce the problem
package org.test.one

@KomapperEntity
data class MyTestModel(
   val id : Long;
)

Different package:
package org.test.two

import org.test.one.MyTestModel

@KomapperExperimentalAssociation
@KomapperOneToMany(MyTestModel::class, "mymodel")
@KomapperEntity
data class AnotherModel(
   val id : Long,
   val myModelId : Long
)

Versions

Komapper: 1.10.0
Kotlin: 1.8
JVM: 17
JDBC Driver or R2DBC Driver: R2DBC
Database: H2 / postgress
OS:

Thanks for your report.
I will fix it.


Adding 'search qualifications' user journey

Adding the user jourbey for 'search qualifications' and 'qualification details'. Also tweaks to existing journeys and formatting and adding a README file.

Performing an automatic merge as there are no conflicts.


add cargo build to run instructions

Perhaps obvious to veteran rustaceans but not to larvae - need to run cargo build before the other instructions

Thanks!


📃: Haiku generator

🔴 Title :Haiku Generator
🔴 Tech stack : HTML, CSS, JS
🔴 Objective :To create a web application that generates random haikus by combining pre-defined phrases.
🔴 Summary :The Haiku Generator is a simple web application that allows users to generate haikus. Users can click a button to generate a new haiku, which will be created by randomly combining pre-defined phrases. The generated haikus can be copied to the clipboard and shared on social media.
Kindly assign it to me under gssoc'24 ext

@iamrahulmahato Raised PR kindly check it


Notice: Undefined index: REQUEST_SCHEME

Notice: Undefined index: REQUEST_SCHEME in /home/vagrant/Code/mydomain.com/craft/plugins/htmlcache/functions/htmlcache.php on line 25

Hi @KoenRijpstra, if you redownload the master (or manually update the htmlcache/functions/htmlcache.php file), a hotfix is provided

Thanks @KoenRijpstra!


[FEATURE REQ] Easier way to save outputs

Is your feature request related to a problem? Please describe.
Currently, the only way to save an Outputs object to a file (without the CLI) is something like this:
outs = get_history()
with open("filename.csv") as f:
    s = outs.to_csv()
    f.write(s)

This can become a little cumbersome at times.
Describe the solution you'd like
A Outputs.save(filename) function that automatically creates the file and saves outputs to that. The type has to be automatically inferred from the given filename. So that the usage becomes:
outs = get_history()
outs.save("filename.csv")

This can be extended to the CLI too:
browser-history -o history.csv

(note that the above command currently works as expected since the default format is CSV, but it will fail when the filename given is history.json)
Additional context
This might make the CLI code a little simpler (not sure about this).
The CLI feature has been changed quite a bit in #70 which is merged to the feature-bookmarks branch. If you're sending a PR for this, either send it to the feature-bookmarks branch or wait for it to be merged.

Done in #78


Fixes #28

Fix for bug #28
It's not a perfect fix, I know, but it's good enough IMHO.
It's impossible to fix it in a different way without changing the public API.
Could you merge it and release asap?

Thanks, good work.


BigChain data truncated after system reboot

It might sound weird however I observed after a hard reboot of my box all the existing data got truncated.
After reboot I did the following –
Start rethinkDB &
BigChainDB instance using the command “BigChainDB start”.
Have anyone experienced it before ?

Can you provide more details? What do you mean when you say "all the existing data got truncated"?

whatever payload I have pushed into the BigChain for a owner is no longer available after the reboot.

Is there some data still available (e.g. blocks containing transactions), with parts missing, or is there no data at all?
It would be great if you could provide more details, e.g. how did you build the transactions, what was in the payload, how did you send those transactions to BigchainDB for validation & storage, how did you find out what was stored (or not), etc.?

We have developed Python based REST services to insert/push the payload into BigChainDB by using the CREATE asset API. Also there is a REST service (NodeJS) exposed to fetch the data directly from RethinkDB BigChain table. It works fine till I did a hard reboot. When I try to fetch data after reboot from RethinkDB it returns empty payload. Do you require more info please feel free to let me know. Please note we are querying rethinkdb directly as that's the only option we have to get the payload back. Please suggest if there are better ways to fetch the payload based on owner/assetid and filter condition.

From bigchaindb side shutting it down before it finished processing transactions and voting can lead to data loss.
From the rethinkdb side a hard shutdown will probably also lead to data loss if there is data that was not yet committed to disk.
When you say payload do you mean the payload of the transactions? and if so did the entire transaction disappeared?
Also is there any error output from the bigchaindb process?

@sekharkumarroy Are you still seeing this and able to reproduce it?

Please suggest if there are better ways to fetch the payload based on owner/assetid and filter condition.

We're currently working on some features related to assets for this :).

I haven't seen that lately. I don't also have exact steps to reproduce that as well.

Ok, then I'll close this for now. We can reopen it in the future if we encounter it again :).


[Feature]: Add Team names for watching DPC matches

Are you on the latest version?

[X] I'm on the latest version

Is there a fix in progress related to your proposal?

[X] I have searched the existing issues

Is your feature request related to a problem? Please describe the problem.
No response
Describe the solution you'd like
I am not sure it is possible to do. Add Team names in the discord for watching DPC matches when spectating a DPC game. Since TI is coming soon, it is a nice discord status to show if you are watching TI.
e.g.
Watching OG vs PSG.LGD | TI 2022
Radiant 3k | Dire
20:00 elapsed
or
The International 2022
Watching OG vs PSG.LGD
Radiant 3k | Dire
20:00 elapsed
Just a suggestion. Not sure it is possible to do.
Dota 2 Patch
No response
Anything else?
No response

I'll see if that's possible. But team information isn't available that easily.
Some possible solutions could be: Comparing the match ID with information from Liquipedia or getting the player's team name. But I'll search

Hello @mikqsq, sorry for the delay.
Unfortunately the dota client does not expose the names of the teams that are playing using the Game Integration service.
I researched how to get this information from DPC teams and the only place I could get this detailed information was from the Liquipedia website.
One strategy would be to get the steam ids of the players and compare them with the information on the site and then put the names of the teams. But for that, it would be necessary to consume an API or perform a Web Scraping that identifies the teams from the players' ids.
I believe this feature could be added in the future if there is an easier way to get this information.
I'll be on the lookout for projects that retrieve information from the professional dota league (if it takes too long I might even create a project that does that). But for now, I won't be able to add this feature.

Hi @andsfonseca , no worry. It is understandable. I thought game client would expose team names and stuffs because they have built-in DPC standings and supporter fantasy stuffs inside the client. It is ok if there is no easy way to do it.


ING-Diba error

When using ruby_fints with ING-Diba it throws this error:
/usr/local/bundle/gems/ruby_fints-0.0.1/lib/fints/response.rb:28:in `get_summary_by_segment': segment HIRMG (FinTS::SegmentNotFoundError)
	from /usr/local/bundle/gems/ruby_fints-0.0.1/lib/fints/response.rb:38:in `successful?'
	from /usr/local/bundle/gems/ruby_fints-0.0.1/lib/fints/dialog.rb:82:in `send_msg'
	from /usr/local/bundle/gems/ruby_fints-0.0.1/lib/fints/dialog.rb:39:in `sync'
	from /usr/local/bundle/gems/ruby_fints-0.0.1/lib/fints/client.rb:19:in `get_sepa_accounts'

Do you have any idea why this happens? I tried it with the php lib and it worked fine.

Thanks for reporting this. Unfortunately, this library has only been tested with a Sparkasse account and we are unable to debug this ourselves as we don‘t have a ING-DiBa account.
If you have the time and capacity to look into this yourself I‘d be happy to assist though. A good first step would be to test if https://github.com/raphaelm/python-fints works with your account. If python-fints works and this lib does not it would be relatively easy to find out where the discrepancy is.

Hi @mkilling, thanks for your fast reply.
I just tried it with the python library and it works with my ING-DiBa accounts. It returns the list of my sepa accounts. It would be great if you could assist me because I'm not familiar with the FinTs specifications at all.

Very interesting. Can you try what happens when you use this branch? https://github.com/playtestcloud/ruby_fints/tree/ing-diba

@mkilling that works :)

Cool. I suggest you use that branch for now and after the holidays we‘ll think about merging this into master. I‘ll leave this issue open until then.

Yes, as you suggested I'll use that branch you created for now. Thank you very much!
Happy Holidays 🎄


gc: fix cleaning mounts and files

This fixes multiple issues in pods GC. In particular, rkt now tries
harder to clean its environment by:

looping over all pods, even if one of them is faulty (eg. some files are busy or missing)
leaving manifest removal as last step, so next GC run may succeed
chroot-ing to pod before cleaning, to avoid chasing wrong symlinks (eg. ending outside of pod)
chdir-ing to (host) root, to avoid keeping directories busy
loop-unmounting all mountpoints, as single unmount operations may fail (eg. busy mountpoint)
detecting busy mounts, and marking them as detached for lazy unmount (ie. MNT_DETACH)

Fixes #3181, right, because rkt rm will succeed with the lazy umount change?

@euank I think so, but have to go and double-check many similar bugs.
I was using https://github.com/coreos/bugs/issues/1630#issuecomment-267208841 as a testcase, which is close to a pathological worse case as it does:

full propagation of /
in host namespace
nesting /run and /usr mounts under root (which are thus back propagated to host root)
multiple runs, piling over many mounts

@euank I'm now reporting a boolean success status, as it was already expected by rm. For gc, it looks like a bit of rework is needed on WalkPods side (and its consumers) to aggregate and report error status. I'm leaving that out of this PR.

I've re-triggered the CI but this should be just the usual flake captured in https://github.com/coreos/rkt/issues/3477. This should be ready to go otherwise, PTAL.

fwiw I did a cursory review of this the other week and it lgtm

@jonboulle thanks for the postumous feedback, I also got and additional double-check offline before merging.

will it be feasible to add flag to NOT unmount lazily or even consider removing lazy unmount all together? I'd like to have explicit errors in case something is holding resources. the fact that umount/rkt rm succeeds doesn't mean there there is no leak :(

Care to elaborate? If I read this correctly, your usecase would benefit from a --mode=lazy|strict on rkt rm to control specific pod eviction, which sounds reasonable. On the other hand, I'm not sure if your request also touches into GC, where I'd be more reluctant.

@lucab I was  referring to

detecting busy mounts, and marking them as detached for lazy unmount

This bug possibly together with other bugs managed to create 200k entries in /proc/mounts within ten minutes on my test kubernetes controller node. With that amount of mounts/namespaces creating new busybox container start to take tens of seconds. Lazy unmount will report success for the whole rkt gcprocess,  which conceals broken state, but doesn't fix it: these 200k mounts still exist in kernel and occupy resources, the fact that they are not visible/accessible from userspace makes it even worse, as server becomes pretty much unresponsive yet it is very harder to debug why

You are right, but that huge amount of mounts is due in the first place because of other rkt pods being nested or not being properly cleaned up, and shared propagation being in place between them. After this, as the GC pass progress, those will be untangled pod after pod. The lazy part in here is needed to get out of the deadlock without a node reboot or forced unmounts.

The lazy part in here is needed to get out of the deadlock without a node reboot or forced unmounts.

once you got out of deadlock what state system will be? my understanding is that it just sweeps it under the carpet and no real unmounts happening. If lazy unmount enables kernel to resolve  cyclic dependencies or something like that, which makes it do real unmounts, then I am happy with it.

If lazy unmount enables kernel to resolve cyclic dependencies or something like that, which makes it do real unmounts, then I am happy with it.

That's the rationale for this change, see https://github.com/coreos/rkt/issues/3465 for a better explanation of the case at hand.


Dependabot Alert: Sandbox Bypass Leading to Arbitrary Code Execution in constantinople

Describe the bug
Linked to dependabot alert here. Flagging this for manual fixing.

Removed the jade package

Checked with @TusharKatiyar2002 , the file changes were done. Did a quick testing of flow to make sure any dependency change, didn't affect anything. It's working fine

Hi @subhashini-egov  This alert resolved and taken care


TemplateBinding throws an exception when used in a Selector in the ControlTheme

Describe the bug
XamlX throws Unable to cast object of type 'XamlX.Ast.SkipXamlValueWithManipulationNode' to type 'XamlX.Ast.IXamlAstPropertyReference'.
Stack
10:31:07.792 [Error] 9076 UpdateXamlResult error
System.Xaml.XamlException: Unable to cast object of type 'XamlX.Ast.SkipXamlValueWithManipulationNode' to type 'XamlX.Ast.IXamlAstPropertyReference'.
10:31:31.176 [Information] 9076 Stopping previewer process
10:31:31.177 [Information]  Process exited
10:31:31.177 [Information]  Stopping previewer process
10:32:25.478 [Information]  Started AvaloniaDesigner.CreateCompletionMetadataAsync() for "C:\GitHub\Avalonia\samples\Sandbox\obj\Debug\net6.0\Avalonia\references"
10:32:25.479 [Information]  Starting previewer process for '"C:\GitHub\Avalonia\samples\Sandbox\bin\Debug\net6.0\Sandbox.dll"'
10:32:25.483 [Information] 24432 Started previewer process for '"C:\GitHub\Avalonia\samples\Sandbox\bin\Debug\net6.0\Sandbox.dll"'. Waiting for connection to be initialized.
10:32:25.926 [Information] 24432 Connection initialized
10:32:27.424 [Error] 24432 UpdateXamlResult error
System.Xaml.XamlException: Unable to cast object of type 'XamlX.Ast.SkipXamlValueWithManipulationNode' to type 'XamlX.Ast.IXamlAstPropertyReference'.

To Reproduce
Steps to reproduce the behavior:
Add folowing xaml
  <ControlTheme x:Key="{x:Type NumericUpDown}"
                TargetType="NumericUpDown">
        ....
        <Style Selector="^[ButtonSpinnerLocation=Left] /template/ TextBox#PART_TextBox">
          <Setter Property="InnerRightContent" Value="{TemplateBinding InnerContent}"/>
        </Style>

        <Style Selector="^[ButtonSpinnerLocation=Right] /template/ TextBox#PART_TextBox">
          <Setter Property="InnerLeftContent" Value="{TemplateBinding InnerContent}"/>
        </Style>
  </ControlTheme>

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment

OS: Any
Avalonia-Version: ffff19e6f0d89cd1a8ffa48dd5063a0500512022

Additional context
if change like this works:
  <ControlTheme x:Key="{x:Type NumericUpDown}"
                TargetType="NumericUpDown">
        ....
    <Style Selector="^[ButtonSpinnerLocation=Left] /template/ TextBox#PART_TextBox">
      <Setter Property="InnerRightContent" Value="{Binding InnerContent, RelativeSource={RelativeSource TemplatedParent}}"/>
    </Style>

    <Style Selector="^[ButtonSpinnerLocation=Right] /template/ TextBox#PART_TextBox">
      <Setter Property="InnerLeftContent" Value="{Binding InnerContent, RelativeSource={RelativeSource TemplatedParent}}"/>
    </Style>
  </ControlTheme>

TemplateBinding is more limited than Binding for performance reasons. I believe this is by design and if it's not enough, you need to fall back to a normal Binding (like you did already). Note, you are outside ControlTemplate here. I agree that the error message could be improved.

There should be another warning or an error before this one.

I also can't reproduce this issue by adding InnerContent property and these setters.

TemplateBinding is more limited than Binding for performance reasons. I believe this is by design and if it's not enough, you need to fall back to a normal Binding (like you did already). Note, you are outside ControlTemplate here. I agree that the error message could be improved.

Then this shouldn't have worked either {Binding InnerContent, RelativeSource={RelativeSource TemplatedParent}} but only
{Binding InnerContent, RelativeSource={RelativeSource FindAncestor, AncestorType={x:Type NumericUpDown}}}

I also can't reproduce this issue by adding InnerContent property and these setters.

I attach a modified SandBox that simulates the problem
Sandbox.zip


How to use the new EButtonIds for the Knuckles

I'm on legacy Unity plugin, we have a very advanced custom Command <> Binding system in place that frankly is more flexible for our use cases as of now so we cant move to the new Input system for now. (I will create a separate Issue about this later when I have tried the new Input System more).
I have updated openvr_api.cs but calling SteamVR_Controller.Device.GetAxis(k_EButton_Knuckles_JoyStick)  returns a Vector2 that never changes. If I update the plugin to 2.1 the types to query the device is gone.
I understand the usefulness of the abstraction from the Hardware you gusy have made, but sofar our own abstraction is more flexible for our use case.

Apparently you can't access any of the new Knuckles inputs using the old SDK: https://github.com/ValveSoftware/steamvr_unity_plugin/issues/158
I have the same issue with the new abstraction layer. What I ended up doing is making optional actions for each raw input of each controller I have optimized support for and created default bindings for these for the corresponding controller.
It's kinda ugly, but I don't see any other approach right now.

@Frooxius Ah ok, then I rather port it properly. :D I have created a github issue with the issues I have found with the new SDK, I hope they will look into it