tfjob: Add test for copy labels and annotations

Close #542
Signed-off-by: Ce Gao<EMAIL_ADDRESS>

This change is

/assign @jlewi

/lgtm
/approve

Travis tests have failed
Hey @gaocegege,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
2nd Build

     gometalinter --config=linter_config.json --vendor ./...

pkg/controller.v2/controller_tfjob_test.go:153::error: possible formatting directive in Println call (vet)
pkg/controller.v2/controller_tfjob_test.go:156::error: possible formatting directive in Println call (vet)

3rd Build

     gometalinter --config=linter_config.json --vendor ./...

pkg/controller.v2/controller_tfjob_test.go:153::error: Println call has possible formatting directive %v (vet)
pkg/controller.v2/controller_tfjob_test.go:156::error: Println call has possible formatting directive %v (vet)

travis_time:end:1dca9343:start=1528944178151906823,finish=1528944334058777027,duration=155906870204

     goveralls -service=travis-ci -v -package ./pkg/... -ignore "pkg/client/*/*.go,pkg/client/*/*/*.go,pkg/client/*/*/*/*.go,pkg/client/*/*/*/*/*.go,pkg/client/*/*/*/*/*/*.go,pkg/client/*/*/*/*/*/*/*.go,pkg/apis/tensorflow/*/zz_generated.*.go,pkg/apis/tensorflow/*/*_generated.go"

=== RUN   TestAddAccelertor
--- PASS: TestAddAccelertor (0.00s)
PASS
coverage: 8.1% of statements in github.com/kubeflow/tf-operator/pkg/apis/tensorflow/helper, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/validation, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/fake, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/scheme, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha1/fake, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha2/fake, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/internalinterfaces, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/client/listers/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/listers/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/controller, github.com/kubeflow/tf-operator/pkg/controller.v2, github.com/kubeflow/tf-operator/pkg/trainer, github.com/kubeflow/tf-operator/pkg/util, github.com/kubeflow/tf-operator/pkg/util/k8sutil, github.com/kubeflow/tf-operator/pkg/util/signals, github.com/kubeflow/tf-operator/pkg/util/train, github.com/kubeflow/tf-operator/pkg/util/unstructured, github.com/kubeflow/tf-operator/pkg/version
ok  	github.com/kubeflow/tf-operator/pkg/apis/tensorflow/helper	0.068s	coverage: 8.1% of statements in github.com/kubeflow/tf-operator/pkg/apis/tensorflow/helper, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/validation, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/fake, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/scheme, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha1/fake, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha2/fake, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/internalinterfaces, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/client/listers/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/listers/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/controller, github.com/kubeflow/tf-operator/pkg/controller.v2, github.com/kubeflow/tf-operator/pkg/trainer, github.com/kubeflow/tf-operator/pkg/util, github.com/kubeflow/tf-operator/pkg/util/k8sutil, github.com/kubeflow/tf-operator/pkg/util/signals, github.com/kubeflow/tf-operator/pkg/util/train, github.com/kubeflow/tf-operator/pkg/util/unstructured, github.com/kubeflow/tf-operator/pkg/version
=== RUN   TestSetDefaults_TFJob
--- PASS: TestSetDefaults_TFJob (0.00s)
PASS
coverage: 6.2% of statements in github.com/kubeflow/tf-operator/pkg/apis/tensorflow/helper, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/validation, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/fake, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/scheme, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha1/fake, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha2/fake, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/internalinterfaces, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/client/listers/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/listers/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/controller, github.com/kubeflow/tf-operator/pkg/controller.v2, github.com/kubeflow/tf-operator/pkg/trainer, github.com/kubeflow/tf-operator/pkg/util, github.com/kubeflow/tf-operator/pkg/util/k8sutil, github.com/kubeflow/tf-operator/pkg/util/signals, github.com/kubeflow/tf-operator/pkg/util/train, github.com/kubeflow/tf-operator/pkg/util/unstructured, github.com/kubeflow/tf-operator/pkg/version
ok  	github.com/kubeflow/tf-operator/pkg/apis/tensorflow/v1alpha1	0.068s	coverage: 6.2% of statements in github.com/kubeflow/tf-operator/pkg/apis/tensorflow/helper, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/validation, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/fake, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/scheme, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha1/fake, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha2/fake, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/internalinterfaces, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/client/listers/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/listers/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/controller, github.com/kubeflow/tf-operator/pkg/controller.v2, github.com/kubeflow/tf-operator/pkg/trainer, github.com/kubeflow/tf-operator/pkg/util, github.com/kubeflow/tf-operator/pkg/util/k8sutil, github.com/kubeflow/tf-operator/pkg/util/signals, github.com/kubeflow/tf-operator/pkg/util/train, github.com/kubeflow/tf-operator/pkg/util/unstructured, github.com/kubeflow/tf-operator/pkg/version
=== RUN   TestSetDefaultTFJob
--- PASS: TestSetDefaultTFJob (0.00s)
PASS
coverage: 13.7% of statements in github.com/kubeflow/tf-operator/pkg/apis/tensorflow/helper, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/validation, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/fake, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/scheme, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha1/fake, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha2/fake, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/internalinterfaces, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/client/listers/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/listers/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/controller, github.com/kubeflow/tf-operator/pkg/controller.v2, github.com/kubeflow/tf-operator/pkg/trainer, github.com/kubeflow/tf-operator/pkg/util, github.com/kubeflow/tf-operator/pkg/util/k8sutil, github.com/kubeflow/tf-operator/pkg/util/signals, github.com/kubeflow/tf-operator/pkg/util/train, github.com/kubeflow/tf-operator/pkg/util/unstructured, github.com/kubeflow/tf-operator/pkg/version
ok  	github.com/kubeflow/tf-operator/pkg/apis/tensorflow/v1alpha2	0.056s	coverage: 13.7% of statements in github.com/kubeflow/tf-operator/pkg/apis/tensorflow/helper, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/validation, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/fake, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/scheme, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha1/fake, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha2/fake, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/internalinterfaces, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/client/listers/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/listers/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/controller, github.com/kubeflow/tf-operator/pkg/controller.v2, github.com/kubeflow/tf-operator/pkg/trainer, github.com/kubeflow/tf-operator/pkg/util, github.com/kubeflow/tf-operator/pkg/util/k8sutil, github.com/kubeflow/tf-operator/pkg/util/signals, github.com/kubeflow/tf-operator/pkg/util/train, github.com/kubeflow/tf-operator/pkg/util/unstructured, github.com/kubeflow/tf-operator/pkg/version
=== RUN   TestValidate
--- PASS: TestValidate (0.00s)
PASS
coverage: 14.2% of statements in github.com/kubeflow/tf-operator/pkg/apis/tensorflow/helper, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/validation, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/fake, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/scheme, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha1/fake, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha2/fake, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/internalinterfaces, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/client/listers/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/listers/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/controller, github.com/kubeflow/tf-operator/pkg/controller.v2, github.com/kubeflow/tf-operator/pkg/trainer, github.com/kubeflow/tf-operator/pkg/util, github.com/kubeflow/tf-operator/pkg/util/k8sutil, github.com/kubeflow/tf-operator/pkg/util/signals, github.com/kubeflow/tf-operator/pkg/util/train, github.com/kubeflow/tf-operator/pkg/util/unstructured, github.com/kubeflow/tf-operator/pkg/version
ok  	github.com/kubeflow/tf-operator/pkg/apis/tensorflow/validation	0.058s	coverage: 14.2% of statements in github.com/kubeflow/tf-operator/pkg/apis/tensorflow/helper, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/apis/tensorflow/validation, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/fake, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/scheme, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha1/fake, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha2/fake, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/internalinterfaces, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/client/listers/kubeflow/v1alpha1, github.com/kubeflow/tf-operator/pkg/client/listers/kubeflow/v1alpha2, github.com/kubeflow/tf-operator/pkg/controller, github.com/kubeflow/tf-operator/pkg/controller.v2, github.com/kubeflow/tf-operator/pkg/trainer, github.com/kubeflow/tf-operator/pkg/util, github.com/kubeflow/tf-operator/pkg/util/k8sutil, github.com/kubeflow/tf-operator/pkg/util/signals, github.com/kubeflow/tf-operator/pkg/util/train, github.com/kubeflow/tf-operator/pkg/util/unstructured, github.com/kubeflow/tf-operator/pkg/version
?   	github.com/kubeflow/tf-operator/pkg/client/clientset/versioned	[no test files]
?   	github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/fake	[no test files]
?   	github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/scheme	[no test files]
?   	github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha1	[no test files]
?   	github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha1/fake	[no test files]
?   	github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha2	[no test files]
?   	github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha2/fake	[no test files]
?   	github.com/kubeflow/tf-operator/pkg/client/informers/externalversions	[no test files]
?   	github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/internalinterfaces	[no test files]
?   	github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow	[no test files]
?   	github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow/v1alpha1	[no test files]
?   	github.com/kubeflow/tf-operator/pkg/client/informers/externalversions/kubeflow/v1alpha2	[no test files]
?   	github.com/kubeflow/tf-operator/pkg/client/listers/kubeflow/v1alpha1	[no test files]
?   	github.com/kubeflow/tf-operator/pkg/client/listers/kubeflow/v1alpha2	[no test files]
?   	github.com/kubeflow/tf-operator/pkg/controller	[no test files]
FAIL	github.com/kubeflow/tf-operator/pkg/controller.v2 [build failed]
exit status 2: warning: no packages being tested depend on matches for pattern github.com/kubeflow/tf-operator/pkg/apis/tensorflow/helper
warning: no packages being tested depend on matches for pattern github.com/kubeflow/tf-operator/pkg/apis/tensorflow/validation
warning: no packages being tested depend on matches for pattern github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/fake
warning: no packages being tested depend on matches for pattern github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha1/fake
warning: no packages being tested depend on matches for pattern github.com/kubeflow/tf-operator/pkg/client/clientset/versioned/typed/kubeflow/v1alpha2/fake
warning: no packages being tested depend on matches for pattern github.com/kubeflow/tf-operator/pkg/controller
warning: no packages being tested depend on matches for pattern github.com/kubeflow/tf-operator/pkg/trainer
warning: no packages being tested depend on matches for pattern github.com/kubeflow/tf-operator/pkg/util
warning: no packages being tested depend on matches for pattern github.com/kubeflow/tf-operator/pkg/util/k8sutil
warning: no packages being tested depend on matches for pattern github.com/kubeflow/tf-operator/pkg/util/signals
warning: no packages being tested depend on matches for pattern github.com/kubeflow/tf-operator/pkg/version
# github.com/kubeflow/tf-operator/pkg/controller.v2
pkg/controller.v2/controller_tfjob_test.go:153: Println call has possible formatting directive %v
pkg/controller.v2/controller_tfjob_test.go:156: Println call has possible formatting directive %v

travis_time:end:04e2e5e4:start=1528944334064526520,finish=1528944358124390724,duration=24059864204

Coverage remained the same at 55.595% when pulling 1a917b466f1e8a22588c26717aed6ca148bc671e on gaocegege:copy into 9271daba0c32ec1023530b284ee9d2cc7bb25e08 on kubeflow:master.

Add LGTM manually since it is a rebase change.


[BUG] The metric http restful API has problem

In document fe-metrics, it shows "You can get Json format by visiting http://fe_host:fe_http_port/metrics?type=json".
But I got prometheus format response with the url.
I read code in httpv2/rest/MetricsAction.java, I found json format returned when type equals 'agent', not 'json'.
In http/rest/MetricsAction.java, I found json format returned when type equals 'json'.
So I guess it's a bug.

This is not a bug. When the type parameter is json, json data is returned
http://<IP_ADDRESS>:8030/metrics?type=json
[   {     "tags": {       "metric": "jvm_heap_size_bytes",       "type": "max"     },     "unit": "nounit",     "value":<PHONE_NUMBER>   },   {     "tags": {       "metric": "jvm_heap_size_bytes",       "type": "committed"     },     "unit": "nounit",     "value":<PHONE_NUMBER>   },   {     "tags": {       "metric": "jvm_heap_size_bytes",       "type": "used"     },     "unit": "nounit",     "value":<PHONE_NUMBER>   },   ......   ,{     "tags": {       "metric": "node_info",       "status": "is_master"     },     "unit": "nounit",     "value": 1   } ]


Hardware compatibility falied

We are using PX4 for our drones with this hardware.Everything works like a charm with version 1.6.5.  But the board stops working if I upload the firmware from the current master branch. I also obtain the following message in QGroundControl at about 5 minutes after I connect the board.

Is this a bug or PX4 stopped supporting this specific hardware? It will be great if the release 1.7 ensures hardware compatibility!

Here is a bit more insight (Thank you @jlecoeur for helping me)!
The master branch continues to work on this specific hardware just by reverting the commit 553c64925292d0fc99b46d9514eb6646472033e7.
After this being done, when I issue the command fmu sensor_reset 500 the entire systems reboots again. This signifies hardware limitation. I hope this issue will be considered for the upcoming release!

@davids5

The hardware that you're using is not FMUV2 compatible ( it may be an assembly issue). The  sensor power supply reset is wired incorrectly. When the sensors are reset it's actually powering down the MCU.
The manufacturer has checked recent production versions and the problem is not present. I would suggest you contact them and raise the issue.

Hey, this issue has been closed because the label status/STALE is set and there were no updates for 30 days. Feel free to reopen this issue if you deem it appropriate.
(This is an automated comment from GitMate.io.)


Scrubber break out

It's in a helper file now. named scrubData (lower case for helping)

@JohnBinning

🏆  nice work


PopUpAnchor - Nested Elements Not Accessible Via Id In MXML

Below's an example app that results in TypeError: Cannot set property 'width' of undefined
TypeError: Cannot set property 'width' of undefined
<?xml version="1.0" encoding="utf-8"?>
<s:Application creationComplete="init()" xmlns:fx="http://ns.adobe.com/mxml/2009" xmlns:s="library://ns.apache.org/royale/spark">
	<fx:Script>
		<![CDATA[

		private function init():void
		{
			myContent.width = 20;
		}
		]]>
	</fx:Script>
	<s:PopUpAnchor>
		<s:Group>
			<s:Group id="myContent"/>
		</s:Group>
	</s:PopUpAnchor>
</s:Application>

@aharui, do you want to take a look?

It's worth noting that the group containing myContainer is accessible.


Finished within 1 second, no results found

Hi, my docker installation on my Synology seems okay. However, when I run a scan it's done within a second, and no duplicates are found. I'm scanning my photo folder incl. subfolders, which have for sure duplicates.
Does anybody have an idea what I'm doing wrong?

Closing this issue.  Please re-open if needed.


Baseline the project at version 0.1

Description
Updates the project to version 0.1.0.
After the update:
ls ~/.m2/repository/org/opensearch/spring-data-opensearch/0.1.0-SNAPSHOT
_remote.repositories                              spring-data-opensearch-0.1.0-SNAPSHOT-sources.jar spring-data-opensearch-0.1.0-SNAPSHOT.pom
maven-metadata-local.xml                          spring-data-opensearch-0.1.0-SNAPSHOT.jar

Issues Resolved
Resolves #7
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Thanks @dlvenable , the CodeQL should be fixed by https://github.com/opensearch-project/spring-data-opensearch/pull/16

I merged that, @dlvenable rebase pls?

Rebased and it builds successfully.


Add readthedocs template

Add readthedocs template

Codecov Report

Merging #84 into sudo-black-travis will not change coverage.
The diff coverage is n/a.

@@                Coverage Diff                 @@
##           sudo-black-travis      #84   +/-   ##
==================================================
  Coverage              63.81%   63.81%
==================================================
  Files                     10       10
  Lines                   1459     1459
==================================================
  Hits                     931      931
  Misses                   528      528

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2073b6e...98087e3. Read the comment docs.


How do you call alert or console.log

I tried but it never returns. Any of this call will work once then it just stops working.
protected override async Task OnInitAsync()
{
await JS.InvokeAsync<object>("alert", "Hello World");
await JS.InvokeAsync<object>("console.log", "hello world console");
}

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

 This should be documented.
@page "/"
@inject IJSRuntime JS

<button onclick="@ConfirmAsync">Confirm</button>

@functions
{
    protected override async Task OnInitAsync()
    {
        await JS.InvokeAsync<object>("console.log", "hello world console");
    }

    async Task ConfirmAsync(){
        var result = await JS.InvokeAsync<bool>("confirm", "Do you confirm?");
        await JS.InvokeAsync<short>("alert", $"Result {result}");
    }
}

When I remove the console.log from OnInitAsync, everything will work just fine.

Hello @dodyg ... This topic will be updated on https://github.com/aspnet/Docs/pull/11314. We'll either look at this for that PR or the next PR (Preview3) right after it.

This is a bug tracked by https://github.com/aspnet/AspNetCore/issues/8274


Add static highlight

same as PR #14 with some added changes. I was supposed to update that PR but I made a mistake in branch name and so I had to create a new PR. Still commits by @Yardanico are still correctly author so I will merge this and close the other.

I forgot to update README (description on how highlight is done), I will do it later when I revise the whole README


Upgrade the core-lro to the latest in Purview Catalog rest client

In this thread  we notice the LRO implementation  needs to be reviewed in purview catalog service so we fixed the core-lro to 2.2.4 in this pr.
We need to undo the pinning once the service fixes the behavior on their end, so I create this issue to track undo thing.

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Azure.Core:0.30677328,Service Bus:0.07100483,Docs:0.051514395'

Hi @yifan-zhou922 , is there any update of this?

LRO for catalog has been corrected.

Close this issue as resolved in this pr: https://github.com/Azure/azure-sdk-for-js/pull/23784.


New o1 models doesn't suppport stream

Frequently Asked Questions

[X] I have read the FAQ before opening the issue.

Workflow version
2024.15
Alfred version
5.5 [2257]
macOS version
12.7.4
Debugger output
[Unsupported value: 'stream' does not support true with this model. Only the default (false) value is supported.]

More details
No response

Thank you for the report. Looks like this is a specific limitation of the betas and is not the only unavailable feature. Streaming is a core part of how the workflow works and changing it to support non-streaming would require a number of changes. I’m wary of doing that and messing with the stability and usefulness of the workflow (not streaming is already slow as molasses, and the new models even more so) only to support what looks to be a temporary limitation on a beta that a large number of people won’t have access to, so for now I’m removing the models.


Maximum Zoom In causes improper node rendering

Reproducing Steps.

make couple of nodes ( good example is image loader & display node )
scale up one of the display node to occupy half of window from default
zoom in the view to maximum and scroll.

bug : it doesn't properly render the big display node and the image area looks a proper size, but node widget size is getting smaller to hold the image area and it doesn't the node properly.

Exactly, the biggest node doesn't draw half where is bottom end.

anybody figured out how to fix this problem?

I think I'm hitting the same bug and I've been investigating.
It seems to be connected to the caching of QGraphicsItems. Specifically, it's directly affected by the call to setCacheMode in the NodeGraphicsObject constructor.
From playing around with an ImageShowModel node from the samples, it seems that when you zoom a big node, so that its actual size is bigger than the window would be, it gets clipped. I suspect there's a limit on how big a cache can be and the size for the DeviceCoordinateCache case is around the size of the window.
Notes:

If you comment out the setCacheMode line, the problem goes away, but the performance of large scenes will suffer.
If you change the mode to ItemCoordinateCache, then problem goes away, but rendering quality suffers.
A cache size is ignored in the case of DeviceCoordinateCache.

I suspect this may be a Qt bug and that the proper behaviour is that the cache should not be used if it is not big enough. I'm not 100% sure and I don't have time to take it any further tonight. I'm just going to leave this comment here for now.

The following screenshot shows a node zoomed in the Images sample. Notice that the node background is cut off in the top right and bottom left corners:

I did a little more investigation into this bug. I continue to believe it is a Qt problem.
I was able to reproduce it with a trivial set up and it seems to be an interaction between DeviceCoordinateCache and QGraphicsDropShadowEffect: When a QGraphicsItem which is larger than the view widget has a cache type of DeviceCoordinateCache, a drop shadow effect will crop the item to the size of the view widget.
If you don't use the drop shadow effect, then the problem goes away. A possible workaround for nodeeditor is to stop using a DropShadow effect. This bug is quite visible in my project, so I would regard this as a small cost.

I've filed this as: https://bugreports.qt.io/browse/QTBUG-77400

I was thinking that I am too slow in addressing the issues. But Qt is not much faster.

If someone does not raise the version to a newer one (it's marked 15.2), often the issue get lost in the hyperspace. Already happened with the Qt crew


[TASK] Add ngrok.io to trustedHostsPattern TYPO3 configuration

with the implementation of the "ddev share" option accessing the page via *.ngrok.io
throws an exception and the trustedHostsPattern must be altered manually.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Benjamin Riezler seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Close as escaping is wrong. New PR follows

Just FYI, you can always push new commits to an existing PR. It's much easier.


Turn map(replicate(n) . f, ...) into transpose(replicate(n, (map(f, ...))))

I think this would enable better memory usage.  This crops up in one function in CalibVolDiff:
fun *[[real]] setPayoff(real strike, [real] myX, [real] myY) =
    let n = size(0, myY) in
    copy(map(fn [real] (real xi) => replicate(n, max(xi-strike,0.0)), myX))

The above is a fairly natural way to write the function, but for the compiler, it would be nicer to have transpose and replicate on the outside.

Probably not worth it.


Enterprise implemenation

Hi, thanks guys for a good app.
I am playing with your app and planning to implement it in our Development Flow.
We have multiple teams, which should be fully isolated from each other, except a few shared packages. We also have a vendor, which has multiple private packages, which should be shared for everyone authenticated.
Every developer has a personal OpenLDAP account, but we use OpenID Connect to authenticate (SSO).
At the bottom there is my list of things  I need to have to launch in production., can do you have any plans to add such a feature?

I can split teams as a different organizations in the current situation. Can a user be in multiple organization/teams?  We need this.
LDAP Users
OpenID Connect
Sync vendor private repository
Disable LDAP Auth ( It can be useful to prevent using LDAP credentials through the web UI when an alternative such as OpenID Connect is preferred. This allows LDAP to be used for group sync, while also allowing your OpenID Connect identity provider to handle additional checks like custom 2FA. and protects from auth by deleted user.
Does it sync dev branches?
org.repo.repo.corp.xxx.xxx  - it is a very long domain name. Can we use folders per organisation? Can repo domain prefix be configured?
Sync support of a hosted bitbucket?

It is already possible (with team members)
This has already been reported, you would have to specify exactly how it would look/work (#58)
This is very easy to implement, see other Oauth (for example, gitlab or gitub)
It is already possible.
You can partially do such things through the instance configuration, but further work will definitely be required.
Yes.
Moving to folders will not be possible because composer allows authorization on individual domains / subdomains. The folder alone is not enough to separate individual access tokens. If the domain name is too long for you, you can always set some proxy or alias in dns.
I haven't tested, but it should work.

If there are any questions I will gladly help.

Good news !

How ?  A Vendor Composer Repo.
Bitbucket Cloud + Hosted Bitbucket

There is actually one more feature we need. To store composer tag zips on S3/Swift.

In that case, I suggest opening a separate issue on each of these topics in which we could discuss the implementation. If something else is not clear then we can finish the discussion here.

@akondas

How do add Vendor Composer Repo?

This is not possible at the moment. But I would like to specify here whether it is one package or if someone else shares their packages using (for example) Satis.

Can we use Bitbucket Cloud and Hosted Bitbucket together?

This is actually possible. Bitbucket Cloud is supported (you only need to add the oauth key for the standalone version). As for Hosted Bitbucket, you could add a repman instance public key (to your bitbucket server) and configure the network so that repman can connect to your bitbucket. Then adding and synchronizing repositories will be possible (by entering the repository url in repman url). I don't have a Hosted Bitbucket instance to test it, but if it works like Gitlab it should work :wink:

Hello @luckyraul, is there anything else to discuss here or can we close this issue for now?

We can close, however I have one last question:
Do you have any plans to mix your app with satis?
Mostly our needs are to grant personal access(oauth/oidc) base on team to restricted repos without touching them in bitbucket, and redistribute vendor private satis.
I can help implement.

We don't have anything like this in the backlog, but contributions are welcome. How would the flow of such functionality look like and what exactly would this integration consist of?

Repman periodically dumps specific packages from every vendor's packagist via personal auth.json ( I assume to vendor named team in organization ). These packages are accessible to users, joined these vendor teams. Normal Satis flow.
From bitbucket cloud we add huge amount of projects grouping them by team. ( Normal Repman Flow )
Person, authenticated via OAUTH/OIDC, joins team(s) automaticly, via JSON/JWT field.
Can use their personal token in local auth.json to download

Looks like a very specific case. We certainly do not have something like this in the backlog, but I can help with the implementation and review pull requests.

If you have any questions or concerns, please create a separate issue in this matter. Cheers :beers:


cl22: Compiler - updated extension list for compiler_defines_for_exensions

Please see https://github.com/KhronosGroup/OpenCL-CTS/pull/55

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Maintanance of ODESystem methods .lhs and .rhs

What exactly do you want to be done here?
Both methods work correctly. Do you want to implement setters?

I am changing status of this issue for now and waiting for more  instructions.

Call of lhs and rhs was added to ODESystemRepresentationComponent:
from dynpy.utilities.components.ode.en import ODESystemRepresentationComponent
from dynpy.models.odes.linear import LinearFirstOrder

ode_test = LinearFirstOrder.from_reference_data()

ODESystemRepresentationComponent(ode_test);


"The option '--flat' is not registered with the generate command"

Please provide us with the following information:

OS?
$ cat /proc/version
Linux version 3.13.0-96-generic (buildd@lcy01-09) (gcc version 4.8.4 (Ubuntu 4.8.4-2ubuntu1~14.04.3) ) #143-Ubuntu SMP Mon Aug 29 20:15:20 UTC 2016

Note: running on Vagrant VM under OS X 10.10.5; directory is shared with host OS
Versions.
$ ng --version
Could not start watchman; falling back to NodeWatcher for file system events.
Visit http://ember-cli.com/user-guide/#watchman for more info.
angular-cli: 1.0.0-beta.18
node: 6.9.1
os: linux x64

Repro steps.
This is an Angular 2 project generated by the CLI, and worked on extensively by me. In the past, the command:
ng generate component myComponent --flat

would generate the files for myComponent without embedding it into a myComponent\ directory. However, today, in the same environment, I got a warning, and the --flat option was ignored:
$ ng generate module --flat utilities
The option '--flat' is not registered with the generate command. Run `ng generate --help` for a list of supported options.
Could not start watchman; falling back to NodeWatcher for file system events.
Visit http://ember-cli.com/user-guide/#watchman for more info.
installing module
  create src/app/utilities/utilities.module.ts
installing component
  create src/app/utilities/utilities.component.css
  create src/app/utilities/utilities.component.html
  create src/app/utilities/utilities.component.spec.ts
  create src/app/utilities/utilities.component.ts

The log given by the failure.
No log was generated.
Mention any other details that might be useful.
Again, I've successfully done this before on this project (I even checked my history). The problem isn't specific to the module name "utilities"; other names give the same results. I've tried restarting the Vagrant VM: no change. Something must have changed or become broken, but I have no clue what.
Interestingly, if I try using ng generate class with the --flat flag, I get the same warning, but the class is NOT put into its own directory:
$ ng generate class --flat anotherClass
The option '--flat' is not registered with the generate command. Run `ng generate --help` for a list of supported options.
Could not start watchman; falling back to NodeWatcher for file system events.
Visit http://ember-cli.com/user-guide/#watchman for more info.
installing class
  create src/app/another-class.ts

Thanks! We'll be in touch soon.

class and module don't have flat mode. For module I don't think it makes sense, and class is flat by default.
Closing this as working as intended, but we can keep the conversation going. What was the behaviour you were expecting?

For reference, see class and module.

If this is "working as intended", then that's fine, but the documentation is misleading. The ng generate <type> command is documented as having a --flat option; there's no hint that --flat is type-specific. See https://cli.angular.io/reference.pdf . If only components are ever put into their own directories (is that true?) then the doc should state that, and that --flat only works with components (and, in fact, --flat shouldn't work with classes, as it's a NOP).
(thanks...)

It's also important to notice that the embedded help: ng help generate and the wiki also suggests that ng g module has a --flat option, which does not seem to work (option is silently ignored) as for ng version 1.0.0-rc.1.
Whether this option for the module generator makes sense or not, I am not sure, but the docs are surely misleading. For instance, if the feature directory gets manually created:
mkdir my-module
cd my-module
ng g module my-module --flat
    create src/app/my-module/my-module/my-module.module.ts

*Notice the doubled directory my-module


Add Windows Support

Instructions
We need to add support for using the extension on Windows platforms.
Issue To Be Solved
Currently VSCode Cadence is not functioning properly on Windows.
Solution Steps

[x] Setup Windows test environment with Parallels
[x] #202
[x] #204
[x] #206
[x] #212
[x] https://github.com/onflow/flow-cli/issues/759
[x] #208
[x] #215
[x] #210

Release v1.0.6 with full Windows support


[BUG] Broken link to BEP guidelines

Describe your problem in detail.
Hi everyone!
Looking in the BIDS-specification at https://bids-specification.readthedocs.io/en/stable/07-extensions.html:

the link onto "BEP guidelines Google Doc" leads to https://docs.google.com/document/d/1pWmEEY-1-WuwBPNy5tDAxVJYQ9Een4hZJM06tQZg8X4/
this link brings us to a Google doc that turns mentions "The content of this document has been moved to this page: https://bids.neuroimaging.io/bep_guide.html"
the link is 404 file not found.

Describe what you expected.
A link to the BEP contributor guidelines. Any help is very welcome!
BIDS specification section
https://bids-specification.readthedocs.io/en/latest/...

a horde of dead links 🧟 🧟♀ 🧟♂
here are the correct ones:
repo: https://github.com/bids-standard/bids-extensions
doc: https://bids-extensions.readthedocs.io/en/latest/

AFAICT this is already fixed on latest:

stable: https://bids-specification.readthedocs.io/en/stable/07-extensions.html
latest: https://bids-specification.readthedocs.io/en/latest/07-extensions.html

you can switch spec version directly via the URL or
by selecting from a menu at the bottom right:

Maybe we don't do enough bug-fix release between minor release, so I usually have "latest" as my default bookmark.

Thanks for the pro-tip ;)


fix: cannot open vscode when use vscode-win32-x64 in Windows

When executing npm run gulp vscode-win32-x64 on the Windows platform, the bundle-vscode task is triggered. However, this task does not handle the path issues well, resulting in the contents of the bootstrap-window.js file not being injected into workbench.js and processExplorer.js.
The reason why VSCode does not encounter this issue is that the code compilation (compile-build) is performed on a Linux machine, while the Windows machine only executes the application build (vscode-win32-x64-ci will not trigger bundle-vscode).
PTAL @deepak1556

@bpasero Done

Thanks, we are actually checking in the related JS file as well, so can you run a npm run compile in the build folder?

@bpasero Completed.


Github Actions fails with test_ray_simulator(plot=True)

Test test_ray_simulator() with plot=True passes locally, but fails on Github Actions.

Code was removed during the implementation of Numba.


[vcpkg] Give hints for yasm with brew and apt

Fix #3772.
yasm is one of those "well-known tool" that is dealt with in vcpkg_find_acquire_program. But the error message on Linux and MacOS is not precise enough, for the moment. This PR fills BREW_PACKAGE_NAME and APT_PACKAGE_NAME so that the build can fail with a better error message line:
Building package mpir[core]:x64-linux...
-- Using cached /home/lrineau/Git/vcpkg/downloads/mpir-3.0.0.tar.bz2
-- Using source at /home/lrineau/Git/vcpkg/buildtrees/mpir/src/3.0.0-a3f571144a
CMake Error at scripts/cmake/vcpkg_find_acquire_program.cmake:293 (message):
  Could not find yasm.  Please install it via your package manager:

      sudo apt-get install yasm
Call Stack (most recent call first):
  ports/mpir/portfile.cmake:27 (vcpkg_find_acquire_program)
  scripts/ports.cmake:94 (include)

Error: Building package mpir:x64-linux failed with: BUILD_FAILED

Without this PR, the message is shorter, and less visible:
Building package mpir[core]:x64-linux...
-- Using cached /home/lrineau/Git/vcpkg/downloads/mpir-3.0.0.tar.bz2
-- Using source at /home/lrineau/Git/vcpkg/buildtrees/mpir/src/3.0.0-a3f571144a
CMake Error at scripts/cmake/vcpkg_find_acquire_program.cmake:291 (message):
  Could not find yasm.  Please install it via your package manager.
Call Stack (most recent call first):
  ports/mpir/portfile.cmake:27 (vcpkg_find_acquire_program)
  scripts/ports.cmake:94 (include)

Error: Building package mpir:x64-linux failed with: BUILD_FAILED

@lrineau would you mind editing your first comment?
This PR fills BREW_PACKAGE_NAME and BREW_PACKAGE_NAME so that
->
This PR fills BREW_PACKAGE_NAME and APT_PACKAGE_NAME so that

You are right. I edited it.


#240: Comma Whitespace Checks #4

Resolves #240.
Adds comma whitespace checks for the following constructs

capture list items
parenthesized expressions
identifier list
switch case item list

Current coverage is 90.85%

Merging #297 into master will increase coverage by +0.03% as of 207a533

Powered by Codecov. Updated on successful CI builds.

:+1:

:+1:


Updated the device layer with a few improvements

Overview
Updated the device layer with a few improvements
Changes

Added Oculus touch support (tested)
Added basic controller detection Support (still needs work)
Added faux touch support to the MR Trigger
Renamed the Vive controllers to be more aligned with that SDK
Added better error reporting to the Interactionmapping, when a mapping type is not supported by the controller

Let's split this up into smaller PRs

Any final comments, we need to get this, plus the reorg PR's merged asap as it's holding up other work

Just one last CR and it'll be approved https://github.com/SimonDarksideJ/MixedRealityToolkit-Unity/pull/17

Last tweaks in, ready to merge @StephenHodgson / @davidkline-ms / @keveleigh


Use gulp-cssnano only in production

Why don't you use gulp-cssnano only when '--production' arg is used so we don't have to minify the styles on 'watch' task. It takes 6-7 additional seconds on 'styles' task when we use gulp-cssnano

for parity between dev and prod, which has been discussed a bunch if you search gh issues/our discourse. we're also no longer making any changes to sage 8 as we're now focused on making changes to the front-end workflow in sage 9


cannot swipe if PagerTabStripView in another PagerTabStripView

Is your feature request related to a problem? Please describe.
My use case is my page have 2 levels, one for main-categories: A, B, C,
and one for sub-categories: B1, B2, B3
something like that:
A is some view
B is PagerTabStripView with 3 pages: B1, B2, B3 (both are some view)
C is some view
I can swipe from A to B, but I cant swipe from B1 to A, and
I can swipe from C to B, but I cant swipe from B3 to C
Describe the solution you'd like
I can swipe from B1 to A, and
I can swipe from B3 to C
Describe alternatives you've considered
none
Additional context
none

Could you please attach the code where the problem occurred?
I would like to understand your issue.
Thanks ! 😄

Hi!
We are working in a solution for this behavior.

On master branch is implemented a solution for this behavior (available for iOS 16+).
In the Readme file is available this documentation to work with that:
The user can also configure if the swipe action is enable or not (the swipe is based on a drag gesture) and setup what edges have the gesture disabled.
Params:

swipeGestureEnabled: swipe is enabled or not (default is true).
edgeSwipeGestureDisabled: is an HorizontalContainerEdge (OptionSet) value where the array could have this options: .left, .right, .both or be empty (default is an empty array).

What is the importance to have this parameter?
Regarding the next PagerTabStripView example in MyPagerView2: if the pager is in the first page and the user try to swipe to the left, is possible to catch a parent view gesture (where this pager is embebbed) instead of catching the actual pager swipe gesture because it is disabled with the edgeSwipeGestureDisabled paramenter.
struct MyPagerView2: View {

    @State var selection = 1

    var body: some View {
        PagerTabStripView(swipeGestureEnabled: .constant(true),
			  edgeSwipeGestureDisabled: .constant([.left]),
			  selection: $selection) {
            MyFirstView()
                .pagerTabItem(tag: 1) {
                    TitleNavBarItem(title: "Tab 1")
                }
            ...
            ..
            .
        }
    }
}

Also is available an example in the Example App (in SegmentedView).
Here is the actual code:


Electrolytic separator energy usage issue

Hello,
I am playing this mod (and loving it), version <IP_ADDRESS>, in the modpack "The 1.7.10 modpack":
http://www.technicpack.net/modpack/the-1710-pack.453902
I noticed after playing in a new world that the electrolytic separator has a base usage that is ten times what it should be. It requires 1600 RF/t when it should be 160 RF/t. From the Mekanism wiki page I could see that it should require 400 J/t. It looks like there are some mishaps in the energy values conversions. Since it was working in my previous worlds with this modpack I wonder if something has changed recently.
I tried to play with the configuration file of Mekanism, Mekanism.cfg but had little luck of getting anywhere:
THIS DETERMINES ELECTROLYTIC SEPARATOR USAGE
D:HydrogenEnergyDensity=2000.0
D:JoulesToEU=10.0
D:JoulesToMJ=25.0
D:JoulesToRF=2.5
B:LogPackets=false
D:MJToJoules=0.04
B:MachineEffects=true
I:ObsidianTNTBlastRadius=12
I:ObsidianTNTDelay=100
I:OsmiumPerChunk=10
D:RFToJoules=0.4
Changing "D:JoulesToRF=2.5" had no effect. Changing "D:RFToJoules=0.4" to D:RFToJoules=4.0" bumped the energy usage to 16000 RF/t (no good), and changing the value to 0.4 reduced it to 160 RF/t (thought I got it). Unfortunately I could not see any difference in the way the machine was pumping the energy from the energy cells. Also I could see that the generators such as the gas burning generator were producing ten times less energy as well. From 800 RF/t to 80 RF/t...
Thanks for looking into this.
Ceruleann.

Wiki is outdated and/or has errors. Electrolitic separator does use 4000 J/t.
Usage is determined by HydrogenEnergyDensity. The same settings determines how much power Gas-Burning generator produces from burning Hydrogen.
The other settings are conversion ratios for other mods power networks.

If that is the case then it makes the cost of running the electrolytic separator way too high. I don't see how it is viable to feed 4000J/t or 1600 RF/t just to get from the tier 1 ore processing to the tier 2. I would have to use a Big Reactor or a fair size or a huge amount of other reactors. :(

Change HydrogenEnergyDensity to 200 - that's been the default for quite some time now. It'll reduce ES usage by a factor of 10.

@Ceruleann You hook it up in a closed loop with a Gas-burning generator. It will then run at half speed indefinitely producing oxygen constantly. Granted all the Hydrogen will be burned.
Good enough in the for a start anyway.


How to send a file?

I'm trying to send a .txt file but everytime I try it's return an error (which I don't understand).
Does someone know how to send a file?

What is the error you're receiving? What's the code you're using to send a .txt file?

Oh sorry I didn't saw the comment.
Here's the error:
`node:internal/deps/undici/undici:11118
Error.captureStackTrace(err, this);
^
TypeError: fetch failed
at Object.fetch (node:internal/deps/undici/undici:11118:11)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
cause: TypeError: object2 is not iterable
at action (node:internal/deps/undici/undici:5969:39)
at action.next ()
at Object.pull (node:internal/deps/undici/undici:6017:52)
at ensureIsPromise (node:internal/webstreams/util:205:19)
at readableStreamDefaultControllerCallPullIfNeeded (node:internal/webstreams/readablestream:2229:5)
at node:internal/webstreams/readablestream:2319:7
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
}`

Can you please also provide the code you used to send your .txt file?


use version 1.4 of php-odm only as 2.0 does not exist

When doing first usage i relized that i made a mistake with versions.

I will merge now, to get a working reciepe at the end of the week.


Update sbt to 1.4.6

Updates org.scala-sbt:sbt from 1.4.4 to 1.4.6.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

labels: library-update, semver-patch

Superseded by #47.

Superseded by #47.


tlsconfig tests should use generated keys and certificates

Made changes to allow using of generated keys and certificates by tests in tlsconfig_test.go rather than regenerating them every time, as pointed out by https://github.com/docker/go-connections/pull/31#discussion_r102535446
The new key and cert files (key.pem, cert.pem, and multi.pem) under the new 'fixtures' directory, were generated using by the same code as tlsconfig_test.go.
Signed-off-by: Arash Deshmeh<EMAIL_ADDRESS>

@tiborvass @cyli

Thank you so much! LGTM

Awesome, thanks for simplifying this!  LGTM!


Add reshuffling method [WIP]

Please ensure your pull request adheres to the following guidelines:

[ ] Tests for new functionality/models or Tests to reproduce the bug-fix in code.
[ ] Updated documentation of features that add new functionality.
[ ] Update CHANGELOG.md with major/minor changes.

Codecov Report
Patch coverage: 34.92% and project coverage change: -0.24 :warning:

Comparison is base (a479a8d) 73.29% compared to head (cc8aeab) 73.06%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1244      +/-   ##
==========================================
- Coverage   73.29%   73.06%   -0.24%
==========================================
  Files          75       75
  Lines       12535    12578      +43
==========================================
+ Hits         9188     9190       +2
- Misses       3347     3388      +41

Impacted Files
Coverage Δ

HARK/ConsumptionSaving/ConsIndShockModel.py
83.90% <34.92%> (-3.12%)
:arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Yay!  Very exciting to see this!
On Sun, Mar 19, 2023 at 7:59 PM wdu9 @.***> wrote:

Please ensure your pull request adheres to the following guidelines:

Tests for new functionality/models or Tests to reproduce the bug-fix
in code.
Updated documentation of features that add new functionality.
Update CHANGELOG.md with major/minor changes.

You can view, comment on, or merge this pull request online at:
https://github.com/econ-ark/HARK/pull/1244
Commit Summary

36d7cbf
https://github.com/econ-ark/HARK/pull/1244/commits/36d7cbfe880ff9e042c1b711badd6f932dfc4341
add reshuffling method

File Changes
(1 file https://github.com/econ-ark/HARK/pull/1244/files)

M HARK/ConsumptionSaving/ConsIndShockModel.py
https://github.com/econ-ark/HARK/pull/1244/files#diff-7f09d28a3d4136ae35d8835cd1352f060da6e0114fe617c34374c8b57d41a57e
(162)

Patch Links:

https://github.com/econ-ark/HARK/pull/1244.patch
https://github.com/econ-ark/HARK/pull/1244.diff

—
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/pull/1244, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKCK73WWLCO2EYRVSVXRX3W46MWJANCNFSM6AAAAAAWANH23M
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Sent from Gmail Mobile

Hi @wdu9
I'm looking over this PR and trying to understand the parts of it. I'm seeing three things:

passing self.reshuffle to the exact_match argument when shocks are drawn from a distribution
A test to make sure that the the AgentCount is an exact multiple of the MinAgentCount (which I guess is the number of equiprobably discretized parts of the distribution)
A third thing that I'm not sure I understand, having to with unemployment draws?

https://github.com/econ-ark/HARK/pull/1244/files#diff-7f09d28a3d4136ae35d8835cd1352f060da6e0114fe617c34374c8b57d41a57eR2273-R2301
Am I reading this right? Can you tell me if anything else is happening that I'm missing?
Any explanation of this could go into the docstrings to improve the API documentation, which is part of the checklist for merging PRs.

On Thu, Aug 10, 2023 at 5:03 PM Sebastian Benthall @.***
@.***> wrote:
Hi @wdu9 https://github.com/wdu9

I'm looking over this PR and trying to understand the parts of it. I'm
seeing three things:

passing self.reshuffle to the exact_match argument when shocks are
drawn from a distribution
A test to make sure that the the AgentCount is an exact multiple of
the MinAgentCount (which I guess is the number of equiprobably discretized
parts of the distribution)

My strong preference is that, when using the reshuffling method, the user
is required to provide an integer-valued input variable named replicants or
multiples or something; AgentCount should be generated as AgentCount =
MinAgentCount * multiples

A third thing that I'm not sure I understand, having to with
unemployment draws?

Unemployment is effectively one more possible value of the transitory
shock; so, if the unemployment rate is 5 percent, you’d need to have
MinAgentCount_unemp
= 20 agents in order to have exactly one person unemployed every period

https://github.com/econ-ark/HARK/pull/1244/files#diff-7f09d28a3d4136ae35d8835cd1352f060da6e0114fe617c34374c8b57d41a57eR2273-R2301
Am I reading this right? Can you tell me if anything else is happening
that I'm missing?
Any explanation of this could go into the docstrings to improve the API
documentation, which is part of the checklist for merging PRs.
—
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/pull/1244#issuecomment-1673917282, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKCK75RMAY7ZVGJ24OT4QTXUVEDTANCNFSM6AAAAAAWANH23M
.
You are receiving this because you commented.Message ID:
@.***>

--

Chris Carroll

Hi @sbenthall ,
Yes you are exactly correct. Much of the code is written to check whether the number of agents is the correct multiple needed to implement the reshuffling method. If the number of agents is appropriate then reshuffle calls exact_match which does most of the heavy lifting by allowing draws to be drawn "perfectly".
The other method that was altered was sim_death to ensure that the number of agents that die each period is exactly (1-LivPrb).
I plan to make the code clearer as the current code is crude in how it determines if AgentCount is the correct multiple. It does work however.


Remove duplicate .sass-cache

why did you abandon this?

@jekyllbot: merge +minor

@jekyllbot: merge +dev.
Thanks @DirtyF! Sorry for the delay here.


docs: add samples for NUMERIC type

Adds samples for the new NUMERIC data type in Cloud Spanner.
(Do not merge yet)

LGTM once the casting sample is removed

I've removed the casting sample.

@olavloite I think we can close this PR since the NUMERIC-specific samples are already merged in. We may need this in the future but hopefully we can just reopen the PR then.


Nav Menu cut off when in Arabic language

What version of Carbon for IBM.com are you using?
1.15.0
What offering/product do you work on? Any pressing ship or release dates we
should be aware of?
It is in production. So it would be nice to get fixed whenever possible.

Steps to reproduce the issue

go to web site: https://proxy1.marketplace-test.ibmcloud.com/profile/dataprivacypreferences/ae-ar
Shrink the window
Click on "persona" icon
you will see the menu is cut off by the screen

Additional information

Screenshots or code
Notes

installing the rtlcss npm package as mentioned in our rtl docs here: https://ibmdotcom-react.mybluemix.net/?path=/docs/overview-enable-right-to-left-rtl--page might solve the issue

@yellowdragonfly  Did the suggestion by Anna fix this problem?

@yellowdragonfly we are closing this issue, assuming the that the issue has been resolved as a result of Anna's comments above.


Crash fixes

https://github.com/BeardedManStudios/ForgeNetworkingRemastered/commit/17c1374c9f4257037eb433b504ccd49bdc546563 adds a null check on application quit for the server in MultiplayerMenu. without this, if the player launches the scene and then closes the application without actually connecting to a server, a null reference exception is thrown.
https://github.com/BeardedManStudios/ForgeNetworkingRemastered/pull/148/commits/e51e611eb4349d1f0b41bea3e05d6c0cb2adbd07  modifies the MultiplayerMenu scene file to remove the default ip address in the NAT Punch host field.  Discovered by @NFMynster . This fixes issue #139 although the true cause is likely related to #136.

Thanks 👍
Are you on Discord? If so what is your username?


Remove  MSBuild properties

Run CIs to see what breaks

API change check
API changes are not detected in this pull request.


Getting error on deleting Newsletter Subscriber.

Bug report
Title
Getting error on deleting Newsletter Subscriber.
Issue Description
Getting error on deleting Newsletter Subscriber.
Preconditions
Please provide as detailed information about your environment as possible.
1. framework Version.-0.1.8

Steps to reproduce
1. step1- Try to delete any newsletter subscriber.

Expected result
Customer should delete from Subscription
Actual result
Getting Exception

Framework Version: 0.1.8

Fixed


[RFE] Log Viewer show timestamps & previous container logs

https://jamesdefabia.github.io/docs/user-guide/kubectl/kubectl_logs/
eg: kubectl logs -p <pod_name>
eg: kubectl logs --timestamps <pod_name>

@fondoger Do you mean we should make sure that the timestamps are being shown in the log viewer?


Make necessary deployment updates for production release of this repository

We need to confirm the steps involved in making this repo the primary source for technical documentation that's under https://docs.metamask.io
Tasks (TBD):

[ ]

@BelfordZ can you share what you had started on this one with @shanejonas ?


[Java] auto-format Maven pom.xml

What feature or improvement would you like to see?
As we did it in apache/arrow#41173, Maven pom.xml files should be automatically formatted by spotless using Apache Maven style

I tried to also apply google-java-format to the codebase, and the changes are minimal so we may actually do both pom.xml and java sources file at the same time without the patch being large

We already apply google-java-format via pre-commit so there should hopefully have been no differences


SQLServerBulkCSVFileRecord should accept an InputStream as one of the constructor overloads

SQLServerBulkCSVFileRecord takes a path to a file on the local filesystem. This works fine when you are doing a bulk upload of a local file. However, this is problematic if the file exists in a remote filesystem, for eg, S3 or HDFS. In that case, we have to copy the file locally, then upload it. Yes, there is the interface, but implementing it requires implementing 95% of what the SQLServerBulkCSVFileRecord class provides out of the box (handling metadata, and parsing input from stream according to that metadata).
All this will be solved if there was an overload available that took an InputStream.
Also, since some of this code will be needed by implementers of the interface, it should be put in a common utility class so that it can be leveraged. I am specifically talking about  the follwoing methods:
SQLServerBulkCSVFileRecord:addColumnMetadataInternal()
SQLServerBulkCSVFileRecord:getRowData()

Hi @ferozed. Just tested using Azure file services. We are able to bulkcopy from csv in remote filesystem. Can you specify what problems you face using S3 or HDFS for instance? Do you get any error messages?

@ferozed just wanted to confirm that you are correct. Right now, the driver fails to load a csv file providing a url to the csv. Thanks for filling out this issue. We will mark it as enhancement.

Hi @ferozed. Could you try testing #318 and see if this is fixing your issue with a file on the remote filesystem? Thanks.

Hi @ferozed . Do you have any updates on pr #318 and if that helped you with your issue with a file on remote filesystem?
Also, could you open another issue for the second part of your question? This way we can track it down easier. Thanks! :)

Fixed by pr #318 . Thanks @ferozed for testing it.


use double loop for operator-/+

Summary
Fixes the linear access bug found in #2213 by using a loop over the rows and columns
Tests
Adds tests that check add/subtract(x, x.transpose()) has the correct adjoint
Side Effects
Nope
Release notes
Checklist

[x] Math issue #2213

[x] Copyright holder: Steve Bronder
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[x] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
dependencies checks pass, (make test-math-dependencies)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[x] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

@bbbales2 Blocks do not support linear indexing, so you can not traverse them using a single loop. So the linked file will not work with blocks.

@SteveBronder I want to get this specific fix in so #2213 can move again, so I went to investigate the scalar + matrix comment here: https://github.com/stan-dev/math/pull/2223#pullrequestreview-544131675
I added two tests to add, subtract, elt_multiply, and elt_divide, and I updated a thing in the tests where we were doing linear indexing there and shouldn't be.
This is an easy way we can test that functions work with transposes:
TEST(MathMixMatFun, add_transpose_test) {
  auto f = [](const auto& x) {
    return x + x.transpose();
  };

  Eigen::MatrixXd x(2, 2);

  stan::test::expect_ad_matvar(f, x);
}

Maybe that should be just the x.transpose() as well. Your tests were already good enough for operator+ and operator-, but I added these tests for elt_multiply and elt_divide too.
The other test I added was:
TEST(MathMixMatFun, add_transpose_test_scalar) {
  auto f = [](const auto& y, const auto& x) {
    return stan::math::add(y, x.block(0, 0, 2, 2));
  };

  Eigen::MatrixXd x(3, 3);

  stan::test::expect_ad_matvar(f, 1.0, x);
}

And I get compile errors. It seems like arena_t and such are not quite working right with block types yet. This is a test that should show a bug with: https://github.com/stan-dev/math/pull/2223#pullrequestreview-544131675
Anyway, I pushed that stuff up here. I would like to get the original fix you had in place for #2213 (just getting operator+ to work), so that that pull can move along too.
So feel free to remove whatever bits that I added that you don't want in this pull, and we can do that later (for instance, we don't have to fix block types for varmat here if they are messed up). The elt_divide and elt_multiply x + x.transpose() tests should stay, and the test_ad changes should as well though.

have to fix block types for varmat here if they are messed up

And if block types are messed up and there is no easy fix, then that also means the test for https://github.com/stan-dev/math/pull/2223#pullrequestreview-544131675 won't work, which means that that issue can't be verified and fixed, so if we don't fix that here, make a note in #2245

I took a hard look at this and there were a couple underlying issues

I missed a loop (yeesh!)

One nice thing about var<matrix> is that when we do arena_t<> on something like a var_value<Eigen::Block<>> we just get back the var_value<Eigen::Block<>> so no copies get made. But this is bad for return types where we don't want our functions to be returning back something silly like a var_value<Eigen::Block<>> (I think this would fail anyway). So I added a return_var_matrix_t<> that returns plain types and changed promote_var_matrix_t<> so that it will pass through var_value's with inner expression types. return_var_matrix_t<> will always return a var_value with an inner plain type while promote_var_matrix_t<var_value<Eigen::Block<>>> just returns back var_value<Eigen::Block<>>.

I also made a specialization for plain_type_t<> so that when it sees a var_value<Eigen::Block<>> it runs plain_type_t over the inner type to get back a var_value<Eigen::Matrix>. I went through a bunch of the functions and standardized / fixed up anywhere this could happen

@SteveBronder sounds good, I'll get a review for this tomorrow

@SteveBronder looks like the tests failed with some sort of RowMajor/Column Major problem:
stan::math::test::type_name() [Arg = stan::math::var_value<Eigen::Matrix<double, -1, 1, 0, -1, 1>, void> returns a  stan::math::test::type_name() [Arg = stan::math::var_value<Eigen::Matrix<double, 1, 1, 1, 1, 1>, void> but should return  stan::math::test::type_name() [Arg = stan::math::var_value<Eigen::Matrix<double, 1, 1, 0, 1, 1>, void>

@rok-cesnovar @serban-nicusor-toptal the build here is saying gelman-group-linux went offline. Is that something that can be worked around or do we need to reboot a computer somewhere?

Needs a physical reboot I am afraid.
We are working on enabling OpenCL tests on the Windows Columbia machine so we have two machines that can run this test.

Oooo, and this one lives on campus? Is this a Goodrich thing or a G-man thing?

I believe Nic usually writes to Ben G., other than that I have no idea where it resides.

Okeedokee I will write the Newyorkerz

Hey, linux machine is back online! Sorry for the trouble.

@serban-nicusor-toptal what version of g++ are we using on jenkins?
I'm seeing
g++ -Werror  -std=c++1y -pthread -D_REENTRANT -Wno-sign-compare -Wno-ignored-attributes      -I lib/stan_math/lib/tbb_2019_U8/include   -O0 -I src -I . -I lib/stan_math/ -I lib/stan_math/lib/eigen_3.3.9 -I lib/stan_math/lib/boost_1.72.0 -I lib/stan_math/lib/sundials_5.5.0/include -I lib/stan_math/lib/benchmark_1.5.1/googletest/googletest/include -I lib/stan_math/lib/benchmark_1.5.1/googletest/googletest -I lib/stan_math/lib/benchmark_1.5.1/googletest/googletest/include -I lib/stan_math/lib/benchmark_1.5.1/googletest/googletest      -DBOOST_DISABLE_ASSERTS       -DGTEST_HAS_PTHREAD=0 -DGTEST_HAS_PTHREAD=0  -c -o /dev/null -include test/test-models/good/function-signatures/distributions/univariate/continuous/wiener/wiener_log_27.hpp test/test-model-main.cpp
g++: internal compiler error: Terminated (program cc1plus)
Please submit a full bug report,

Integration linux runs on linux tag so both aws and gelman-linux.
gelman-linux: g++ (Ubuntu 9.3.0-17ubuntu1~20.04) 9.3.0
aws: g++ (Ubuntu 5.4.0-6ubuntu1~16.04.12) 5.4.0 20160609


Add hostname substitution

it's tested, going to merge, since it's not really a very useful change which we probably are going to roll-back


Add support for an ~/.itermocil directory

We should now have our own directory as there are enough non-teamocil users using this.

For me running --edit sample creates the file on the teamocil folder. Is that a bug?


Move events group name to component params.

Move config group name for events to component params.
'yiisoft/event' => [
    'eventsGroup' => 'events-web',
],

ListenerCollection::class => static fn (Factory $factory) => $factory->create($params['yiisoft/event']['eventsGroup']),
Additional info
https://github.com/yiisoft/yii-event/blob/d3f830018d86dfd5e7d15fc4a6997d9cc8fa1659/config/di-web.php#L11

Need to don't forget add docs to readme about this feature.

Done by #62


Add backend support for gene advanced search

Coverage remained the same at 57.605% when pulling 44e9efda80881122fb6ab510f7855d3a1f5db8de on add-gene-advanced-search into e7cd7f921abf8c3b1446eae21e2f98f0d7c3badc on master.


Cannot prove equality on recursive function with values in Type

I am trying to prove an equality on a recursive function. The following attempt fails:
noeq type list_type: Type u#1 =
| TCONS: ((t: Type0) -> (a: option list_type) -> list_type)

let rec fun_of_list_type (l: list_type): Tot Type0 =
  match l with
  | TCONS t a -> if Some? a then (t -> Tot (fun_of_list_type (Some?.v a))) else t

let fun_of_list_type_some t l:
  Lemma (fun_of_list_type (TCONS t (Some l)) == (t -> Tot (fun_of_list_type l)))
= ()

Using assert_norm to help circumvent fuel issues also fails. In practice, this means that I cannot do anything with this recursive function...
However, surprisingly, replacing Type0 with nat works well:
noeq type list_nat: Type =
| CONS: ((t: nat) -> (a: option list_nat) -> list_nat)

let rec sum_of_list_nat (l: list_nat): Tot nat =
  match l with
  | CONS t a -> if Some? a then (t + sum_of_list_nat (Some?.v a)) else t

let sum_of_list_nat_some t l:
  Lemma (sum_of_list_nat (CONS t (Some l)) == t + sum_of_list_nat l)
= ()

Does it mean that, whenever a recursive function has its value in Type, I can never reason about it?
More generally, how similar is this bug to #496?

It really seems to be about the fact that the recursive function has values in Type, rather than the fact that the data constructor contains type arguments. Indeed, the following works well:
let rec size_of_list_type (l: list_type): Tot nat =
  match l with
  | TCONS t a -> if Some? a then 1 + size_of_list_type (Some?.v a) else 0

let size_of_list_type_some t l:
  Lemma (size_of_list_type (TCONS t (Some l)) == 1 + size_of_list_type l)
= ()

Hi Tahina, thanks for the report, using assert_norm or tactics to do the unfolding makes the proof go through:
module Test

open FStar.Tactics

noeq type list_type: Type u#1 =
| TCONS: ((t: Type0) -> (a: option list_type) -> list_type)

let rec fun_of_list_type (l: list_type): Tot Type0 =
  match l with
  | TCONS t a -> if Some? a then (t -> Tot (fun_of_list_type (Some?.v a))) else t

let fun_of_list_type_some t l:
  Lemma (fun_of_list_type (TCONS t (Some l)) == (t -> Tot (fun_of_list_type l)))
  //by (norm [delta; iota; zeta]) -- this also works
= assert_norm (fun_of_list_type (TCONS t (Some l)) == (t -> Tot (fun_of_list_type l)))

With just smt, we get into hashconsing issues of arrow types, which is a known, and more basic issue. I am closing this issue, please reopen if needed. Thanks!


Dnx project start up error - dnx design time process. No connection could be made

Ok i've tried everything including:

dnvm upgrade or  dnvm update-self

Repaired Asp.Net 5 RC1 Update 1

Deleted all nuget packages from .dnx folder.

But all invain. Although initially dnvm upgrade and dnvm update-self worked temporarily but after they lost their effectiveness, I tried the other two options but they also didn't work. Is their a genuine fix for this or Should I revert to previous versions for my current web app. Please help!
P.S. I really want to use cross platform capabilities on my ubuntu & VS code.

Could you please give us a screenshot/stack trace/complete error message?

Updated @MaximRouiller (see above) Thanks!

/cc @BillHiebert

Ok, After changing global.json from:
{
  "projects": [ "src", "test" ],
  "sdk": {
    "version": "1.0.0-rc1-update1"
  }
}

to:
{
  "projects": [ "src", "test" ],
  "sdk": {
    "architecture": "x64",
    "runtime": "coreclr",
    "version": "1.0.0-rc1-update1"
  }
}

vs project opens perfectly well.
P.S.:- I've only opened the project a couple of times.

3rd time no luck :'(

Hello Anyone here?! @MaximRouiller @davidfowl  @BillHiebert

We are here. Still thinking.

@MaximRouiller Any solution yet? :'(

Have you tried clearing your .dnx folder and reinstalling it?
What about repairing your WebTools installation?
What else have you tried?

i've tried the following:
•dnvm upgrade or dnvm update-self
•Repaired Asp.Net 5 RC1 Update 1(Web Tools?)
•Deleted all packages from .dnx folder.
Right now re-install dnx as per your advice.

Re-installing dnx and web tooling repair also didn't work. @MaximRouiller

Any Luck?! @MaximRouiller @davidfowl @BillHiebert

No luck on my end. I would rely on the team to help debug this. I haven't encountered this issue nor have been able to reproduce.

Argh! Its annoying as I've to restart Visual Studio every time. But, thank god that the error now disappears after restarting visual studio.

Occurs in .Net Core preview 1.0.0 too. Just for the info.


Content-Type and  Authorization set failed

const httpOptions = {
headers: new HttpHeaders({
'Content-Type':  'application/json',
'Authorization': 'my-auth-token'
})
};
return this.http.post(url, body,httpOptions)

This seems like a bug to me. I'm sorry I haven't been back here to see your report in so long.
I cannot understand why the headers on RequestInfo are other than the headers from the original req, supplemented with the 'Content-Type': 'application/json' header. I believe that was the original intention.
The original headers ARE preserved in RequestInfo.req.headers. You can work around with those.
I hope to fix it soon.


Flash attention for cross attention

Dear Authors,
Thank you very much for the open source code.
I have a cross attention module for which I would like to use flash attention. But unfortunately, I get the following error:
*** RuntimeError: k must have shape (total_k, num_heads, head_size)
Is it possible to use flash attention when shape of q and k are different width, although they have similar resolution everywhere else?
Here is my problem setup:
batch_size = 1
seq_len = 512

Shape of q is: [batch_size, seq_len, 32, 16]
Shape of k and v are: [batch_size, seq_len, 32, 128]
 b,c,h,w = q.shape
 bb, cc, hh, ww = k.shape

 q = rearrange(q, 'b c h w -> b (h w) c')
 k = rearrange(k, 'bb cc hh ww -> bb cc (hh ww)')
 v = rearrange(v, 'bb cc hh ww -> bb cc (hh ww)')

 batch_size, seq_len, _ = q.shape

 from flash_attn.flash_attn_interface import flash_attn_unpadded_func
 q_unpad = rearrange(q, 'b s (h d) -> (b s) h d', h=self.heads)
 cu_seqlens_q = torch.arange(0, (batch_size + 1) * seq_len, step=seq_len, dtype=torch.int32, device=q_unpad.device)
 max_seqlen_q = seq_len

 k_unpad = rearrange(k, 'b s (h d) -> (b s) h d', h=self.heads)
 v_unpad = rearrange(v, 'b s (h d) -> (b s) h d', h=self.heads)
 cu_seqlens_k = torch.arange(0, (batch_size + 1) * seq_len, step=seq_len, dtype=torch.int32,device=q_unpad.device)
 max_seqlen_k = seq_len

 output_unpad, _, _ = flash_attn_unpadded_func(q_unpad, k_unpad, v_unpad, cu_seqlens_q, cu_seqlens_k, max_seqlen_q, max_seqlen_k,0.0, return_attn_probs=True)

 out = output_unpad.reshape(batch_size, seq_len, self.heads * self.dim_head)
 out = rearrange(out, 'b c (h w) -> b c h w', h=h)

I'm not sure I understand how attention is supposed to be computed if q and k don't have the same head dimension.
How would the code look in Pytorch?


Flaky Test: replace reflect.DeepEqual with "github.com/google/go-cmp/cmp"

Description
The existing test in cspc_generator_test.go, try to replace reflect.DeepEqual() with "github.com/google/go-cmp/cmp" package to fix the flaky test.
Here on line number L187 tests uses the reflect.DeepEqual() which can be replaced with  cmp.Equal(...) to handle the comparison of interface types , once changes are done PR should be raised in https://github.com/openebs/upgrade repo.
https://github.com/openebs/upgrade/blob/2eb9a3d3e3c707f34ff1939312d6bea2e5c1efac/pkg/migrate/cstor/cspc_generator_test.go#L187
for example :
https://github.com/openebs/upgrade/blob/2eb9a3d3e3c707f34ff1939312d6bea2e5c1efac/pkg/migrate/cstor/cstorvolumepolicy_test.go#L203
Expected Behavior
Above change should fix the flaky test
Current Behavior
Tests gets failed in some test runtimes to above mentioned issue.

I would like to work on this issue :)

@prateekpandey14 Can I contribute to it?

@infiniteoverflow  are you still intrested to work in this issue.

Yes i am interested. Thank you 💯

@prateekpandey14 I have raised a PR for the same : https://github.com/openebs/upgrade/pull/58

This issue was addressed in the PR https://github.com/openebs/upgrade/pull/58


Render::HTML.new(hard_wrap: true) + two trailing spaces => extra

Two extra trailing spaces + hard_warp: true => extra  inserted:
markdown_renderer = Redcarpet::Markdown.new(
  Redcarpet::Render::HTML.new(hard_wrap: true),
)
markdown_renderer.render("hello  \nworld")

=>"
hello

\nworld
\n"
With just one extra trailing space, or with hard_wrap: false, this does not occur.

Hi @bkazez, not a maintainer but I noticed this issue locally. If you pass the arguments through to the Redcarpet::Markdown class, you'll see the behavior you're looking for.
# Your Example, passing options into HTML.new
markdown_renderer = Redcarpet::Markdown.new(
  Redcarpet::Render::HTML.new(hard_wrap: true),
)
markdown_renderer.render("hello  \nworld")
=>  "
hello

\nworld
\n"

# My Example, passing options as arg to Markdown.new
markdown_renderer = Redcarpet::Markdown.new(
  Redcarpet::Render::HTML.new,
  hard_wrap: true),
)
markdown_renderer.render("hello  \nworld")
=>  "
hello
\nworld
\n"

Thanks @dlinch! I tried this and don't get any hard wrapping at all - things like\nthis aren't translated into like
this.

@bkazez I found a combination of the hard_wrap and the lax_spacing options generally worked the way I expected.


apps/mbedtls-alt: Provide system alternative implementation for mbedtls

Summary

provide aes-cmac alternative implemetation.
The computing process and software algorithm of AESCMAC are already provided in the crypto module of NuttX, and the mbedtls-alt interface is provided for the convenience of the upper layer, which can be used to directly replace the implementation of the mbedtls interface. Drive capability is used imperceptibly, and that replacement has gone through the MBEDTLS internal selftest case.
provide hardware random for PSA.
There is a conflict between using the PSA random number interface provided by MBEDTLS and trying to use AES to drive the acceleration function. This is due to the fact that the software random number source of the PSA is a global variable, and the driving ability of AES is exposed through fd, and there will be cross-process call errors. So there is a need to provide the ability for PSA to also use random number driven.

Impact

The NutTX encrypted driver capability can be used to independently implement the MBEDTLS interface, which can provide a general interface without using the driver capability imperceptibly
Provides the ability for mbedTLS to use hardware random number drivers, but does not directly verify driver performance. In order to solve the conflict between the PSA global software random number entropy source and the MBEDTLS AES using cryptographic drivers

Testing
ci & mbeselftest

Please update PR Testing section with test run logs.

thanks, 3 and 4 i will verify in local and update after.

Thank you @ThePassionate :-)


Fix pybind11 empty directory error for Docker

Docker build now successful
Solved #103

This one should be merged quickly!


GetScreenWidth and GetScreenHeight returning wrong values

Hi, i've noticed that, after i resize my window with SetWindowSize(), the values that these two functions return is wrong, and references the previous window size.

Hey,
Once you've changed the windows size, you'll have to wait until the next game loop (Raylib.WindowShouldClose).
Then the values are updated and correct.
I think this is not Raylib-cs related, because these methods are direct interop with the native library.


Add Evmos Mainnet and Testnet

Please add Evmos Networks:
https://github.com/safe-global/safe-deployments/pull/66
https://github.com/safe-global/safe-deployments/pull/83

@datradito as I've said in a previous PR, please clean the commit history, it's very confusing to see 10 commits for this small change. Next one I will not merge unless it's clean


Records named constructor in infinite loop in async code with web workers

Hi everyone.
Browser: Chromium
Dart info:

What is happening:
An infinite loop happens when running a Records constructor in the web platform in async code performed by a web worker.
Log of the error:
records_loop.log
Context:
I'm using Drift database to persist data locally in the web version of the Flutter app I'm working on. The persistence is handled separately by a web worker. More information can be found in Drift documentation.
@override
Future<void> saveNode(OCBaseNode node) async {
  await into(oCNodesDrift).insert(
      OCNodeDrift(node.id, node.labels, node.inRelationshipsIds,
          node.outRelationshipsIds, node.createdDate),
      mode: InsertMode.insertOrReplace);

  if (node.properties != null) {
    for (var prop in node.properties!.entries) {
      final literals = DriftConverters.cypherLiteralConverterToDrift(
          type: prop.value.parseTreeResult as OCType,
          key: prop.key,
          nodeId: node.id);

      await into(oCPropertyKeysDrift).insert(literals.key.toInsertable(),
          mode: InsertMode.insertOrReplace);

      await batch((batch) => batch.insertAll(
          oCPropertyValuesDrift, literals.values.map((e) => e.toInsertable()),
          mode: InsertMode.insertOrReplace));
    }
  }
}

In a snippet of code above hard work is done in the async calls by drift web worker, the Records constructor is inside the method  DriftConverters.cypherLiteralConverterToDrift.
static ({OCPropertyKeyDrift key, List<OCPropertyValueDrift> values})
    cypherLiteralConverterToDrift(
        {required OCType type,
        int index = 0,
        String? key,
        OCPropertyKeyDrift? propKey,
        String? nodeId,
        String? relationId,
        LiteralStruct? literalStruct = LiteralStruct.none}) {
  assert(key != null || propKey != null, "At least one should be provided");

  propKey ??= OCPropertyKeyDrift.init(
      key: key!,
      literalStruct: literalStruct!,
      nodeId: nodeId,
      relationId: relationId);

  if (type is OCList) {
    return cypherLiteralListConverterToDrift(type, propKey);
  }

  switch (type.runtimeType) {
    case OCInt:
      return (
        key: propKey,
        values: [
          OCPropertyValueDrift.init(
            index: index,
            keyId: propKey.id,
            intValue: (type as OCPropertyType).value,
          )
        ]
      );

    case OCDouble:
      return (
        key: propKey,
        values: [
          OCPropertyValueDrift.init(
            index: index,
            keyId: propKey.id,
            doubleValue: (type as OCPropertyType).value,
          )
        ]
      );

    case OCString:
      return (
        key: propKey,
        values: [
          OCPropertyValueDrift.init(
            index: index,
            keyId: propKey.id,
            stringValue: (type as OCPropertyType).value,
          )
        ]
      );

    case OCBoolean:
      return (
        key: propKey,
        values: [
          OCPropertyValueDrift.init(
            index: index,
            keyId: propKey.id,
            boolValue: (type as OCPropertyType).value,
          )
        ]
      );

    case OCNull:
      return (
        key: propKey,
        values: [
          OCPropertyValueDrift.init(
            index: index,
            keyId: propKey.id,
          )
        ]
      );

    default:
      throw const FormatException("Invalid Property type");
  }
}

The error is thrown in the returns inside the switch of the above function.
Workaround:
I replaced Records with Lists and the code worked as expected.

Is it possible to get a more minimal and compilable repro here?

I will try to post a simpler snippet ASAP.

Sorry for the number of files, I deleted all the code I could. The project can be run in native and web, it only doesn't work in web. If the Records are traded for Lists, it work in the web.
Versions:

Code tested in a new empty flutter application.
Repository: https://github.com/AlexandreAndrade00/dart_web_records_bug

This looks like a duplicate of https://github.com/dart-lang/sdk/issues/54375 where it was found that creating a record with a named element "values" would produce a stack overflow. It has since been fixed.
@AlexandreAndrade00 You should be able to confirm the fix in the current Fluter beta release.


Merge compatible and berkeley back to develop

This is compatible merged via berkeley back to develop. This builds upon the berkeley PR https://github.com/MinaProtocol/mina/pull/14145.

!ci-build-me


Fix CMake circular dependencies issue

Issue was related to the fact I am adding a custom command with the file name as a target

I messed up something with Jenkins on the status notification...


Remove interpolated strings

No functional changes; just not quite ready to require C# 6 features in the source code to support legacy users.

I don't want to argue but I'm not sure I understand your point 100% since currently NATS is compiled to .NET 4.5 which has full C# 6 support. Also like I said, C# 6 is a compiler only "thing" and most of it can be used and run in older .NET code base. String interpolation, for example, is compatible up to .NET 3.5 like most of the C# 6 features. You get compiler errors if this feature is not supported on the targetted runtime framework.
Also, all Microsoft supported version of Windows should have 4.5.2 framework at least by now since it was pushed through Windows updates a long time ago.

I do see your point.  We do have at least one user who's deployed NATS into production who at the time required a 4.0 compatibly source code base due to their restrictions - they may wish to remain anonymous, but it's related to https://github.com/nats-io/csnats/issues/49.  I think it'd be sufficient for most users (or the NATS team) to compile down to the compatibility level needed, and that's something to look at in the future.  I'll poll the user community to see if we can remove this restriction in the next release.


Publish 12/09/2021, 10:30 AM

Docs Build status updates of commit c3e22cc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-AutoSensitivityLabelRule.md
:white_check_mark:Succeeded
View (exchange-ps)

skype/skype-ps/skype/Get-CsOnlineApplicationEndpoint.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/New-CsOnlineApplicationEndpoint.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Remove-CsOnlineApplicationEndpoint.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsOnlineApplicationEndpoint.md
:white_check_mark:Succeeded
View (skype-ps)

teams/teams-ps/teams/Get-CsTeamsShiftsConnectionErrorReport.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Get-CsTeamsShiftsConnectionInstance.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Get-CsTeamsShiftsConnectionOperation.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/New-CsTeamsShiftsConnectionBatchTeamMap.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/New-CsTeamsShiftsConnectionInstance.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Remove-CsTeamsShiftsConnectionErrorReport.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Remove-CsTeamsShiftsScheduleRecord.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Set-CsTeamsShiftsConnectionInstance.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Implementing all_gather to datacopy signaling

What's changed
Note: this is a re-opened version of this PR: https://github.com/tenstorrent/tt-metal/pull/11231 , which had to be reverted because of a typo causing build failure on main.
Checklist

[ ] Post commit CI passes
[ ] Blackhole Post commit (if applicable)
[ ] Model regression CI testing passes (if applicable)
[ ] New/Existing tests provide coverage for changes

Post-commit: https://github.com/tenstorrent/tt-metal/actions/runs/10393107179


Dependency installation order regression

Hi
We were just hit by an old bug.
In https://github.com/buddycloud/webclient we require both coffee-script and formatdate. The latter has a dependency on coffee-script as well. It now happens that npm tries to build formatdate before coffee-script. That fails because the coffee-script dependency is unmet, but formatdate requires it to build.

I'm not sure of the best approach here.
If we just say that deps get nested, period, and installed multiple times if necessary, then that sucks in some cases (which we found quite some time ago.)
If we have B wait until A is finished, then you get into a problematic situation when you have cyclic dependencies, which can then deadlock one another.
Suggestions?

More complications:
- A
- B
  `-- A

Let's say we decide to have B wait for A.  That means we have to do the analysis up front to determine whether or not waiting for A will cause a deadlock.  (That is, does A depend on [anything that depends on] B, and is this dependency going to be met by the B that we're considering pausing, and is the thing depending on B not yet built.)
If having B wait for A would cause a deadlock, then you could try to install another copy of A underneath B.  However, if it would cause a deadlock, then A can't be installed, because it will require B, which is not built yet, and the one above it is not built yet!
Also, this approach can reduce performance pretty badly.  Consider this:
a -> c
c -> d
d -> e
e -> b
b -> a

Now, say you want to install A and B.  We can only determine that this will deadlock after downloading and parsing c, d, and e.  That's minimum 6 http requests, which cannot be parallelized, while B could be getting installed.  Even if we didn't have the eventual cycle, it's still potentially very slow.
So, let's say that we pursued the approach of bundling A into B if it's a parallel dep that isn't finished installing yet.  We'd end up with duplicates of not just A, but also C, D, and E, and it would still fail because E depends on B, which isn't built yet!  Except, now we have two things failing -- the E under A, and the E under the A that's under B.
Here are some workarounds:

You're compiling coffee script into javascript.  There's no reason you need to do that in an install script.  Do it in a prepublish script instead, and just ship javascript.  It's not like you need to compile it on the target platform or anything.  Seriously, if there was ever a case of an unnecessary install-time complexity, that's it.  scripts.install will become increasingly irrelevant in the future, and may eventually disappear, once binary deployment becomes more of a reality.  Windows support is pushing this issue.
List coffee-script in the "bundleDependencies" list, and just ship a known-good copy so that it's already there ahead of time.  This is a less good solution.  The better solution is to publish javascript instead of coffee-script.

Note that the issue doesn't arise with actual compiled modules, because in practice, modules with a compile step are typically always a single binary module created from C source files, with no external javascript dependencies.
I think we should punt on this issue, unless someone has some idea to solve it that doesn't introduce deadlocks or performance bottlenecks.

Perhaps another approach would be to have postinstall for every installed package run after the entire process is finished.  But really, you should just skip the install-time complexity.  Make your coffee script compilation a pre-publish step, list coffee-script as a devDependency, and ship js.

Do the analysis beforehand to get it right. Throw an error on cyclic dependencies, they shall not happen unless there's a compelling reason. Is there?
On performance: you have to download all the dependencies anyway. Having the installation phase wait for the download phase to finish is probably not going to increase the total time by orders of magnitude. It just removes some concurrency, which has been there as a non-critical optimization.

  They're almost impossible to prevent, and failing on that is just setting up unnecessary land-mines in the future.
Assume that "install" scripts will not have dependencies installed already.  Bundle them, or do it as a prepublish step.


TfLite. Fix of issue 79317

TfLite. Fix of issue https://github.com/tensorflow/tensorflow/issues/79317. Solves unresolved external symbol linker error when your application uses the regular static TfLite library and a function prefixed with preprocessor macro TFL_CAPI_EXPORT.

Hi @majiddadashi , Can you please review this PR? Thank you !

Hi @junjiang-lab , Can you please review this PR? Thank you !

@keerthanakadiri Thanks for your commitment, because I indeed was wondering what the status was for my PR. I'll await the reviews.

Hi @mattsoulanille, Could you please review this PR?  Thank you!