How to deploy on two host or more host?

I have a based problem, how to deploy on two host?
e.g. the one host to run netmaster service and netplugin service,
another one to run netplugin service only?
I not use vbox + vagrant , I use "go install" command to compile.

@xiaodan1 : May I know which orchestration engine (swarm or Kubernetes) you are planning to use ?
also which docker version , you are installing ?
@gkvijay Could you please respond to this ?

@xiaodan1, are you trying to run the netplugin/netmaster binaries? Do you have etcd running? Please also let us know version of docker, OS and orchestration.

thanks,
OS: Centos7
docker-engine version: 1.12.5
OVS version: 2.5.0
I use netplugin/netmaster binaries to run , and also run etcd, but The etcd to running not etcd`s cluster,(only one etcd service.)
I modify netplugin/netmaster and netplugin/netplugin code ,  from <IP_ADDRESS>:2379 or localhost:2379  to  :2379, and then compile to binaries.
I not use orchestration engine (swarm or Kubernetes) . only use docker-engine.

@gkvijay My environment following:
node1(<IP_ADDRESS>/24):  running netmaster service,  netplugin service , netctl (from code compile                                           binaries)
etcd container(docker run -itd -p 2379:2379 elcolio/etcd /bin/bash )
The /etc/hosts  file  have  "<IP_ADDRESS>  netmaster"
for docker daemon add option "--cluster-store=etcd://<IP_ADDRESS>:2379"

node2(<IP_ADDRESS>/24): running netplugin service , netctl (from code compile binaries)
not running etcd, but i modify the netplugin/netplugin/{netd.go,skydns2.go}:
and then again compile netplugin/netplugin to get binary of netplugin:

The /etc/hosts file have  "<IP_ADDRESS>  netmaster"
for docker daemon add option "--cluster-store=etcd://<IP_ADDRESS>:2379"

I not use orchestration engine (swarm or Kubernetes) . only use docker-engine.
The result is :
on node1:   "netctl net create -s <IP_ADDRESS>/24 -e vlan -p 20 contiv-vlan20" ,
and "netctl net ls" is ok, "docker network ls" is ok,
but  on node2 "netctl net ls" is ok,  "docker network ls" is error,(have not 'contiv-vlan20').
on node2:    "netctl net create -s <IP_ADDRESS>/24 -e vlan -p 30 contiv-vlan30" ,
and "netctl net ls" is ok, "docker network ls" is error,(have not 'contiv-vlan30').
but  on node1 "netctl net ls" is ok,  "docker network ls" is ok .

@xiaodan1: Could you recheck your environment states and configs? The same scenario works for me.
Also, you don't have to modify netplugin/netmaster to pass the etcd IP. You could use the -cluster-store option in both netmaster and netplugin as you did for dockerd.

@xiaodan1: I ran into your issue when docker on node2 was not started before etcd was started on node1.
Try the following sequence:

Start docker daemon on both node1 and node2 with cluster-store option
Start etcd on node1 with -p 2379:2379
Start netmaster and netplugin services and execute netctl commands (you can use cluster-store option here as well)

execute netctl commands (you can use cluster-store option here as well)  ??
How to use --cluster-store option when i execute netctl commands ???

You are right, It`s all ok!!
on node1:

on node2:

Is really the cause of the startup sequence.
Thank you very much.


rmse -> distance correlation

Ping @lazappi @mvinyard . Closes #746

Going to merge this in a hurry to get results in for the new run. @lazappi @mvinyard if you disagree with how it was done lmk and we can fix post-hoc.

Code looks fine to me. I'm not sure if anyone has checked with Dan if he had an argument for keeping RMSE. I think he added it originally.

@dburkhardt ^


request for extrinsic LiDAR-image calibration tool

It would be great to have an inbuild robust extrinsic calibration tool, the extrinsic value can be embedded as an attribute for each object ID's
Example:
[
{
"id": 0,
"name": "pc0",
"type": "pointcloud",
"extrinsics": {
"location": [
-0.7956064878711493,
0.06994663796442704,
0.07554082295382114
],
"rotation": [
0.06168062793771913,
0.01200528651139976,
-0.26920656196175347
]
}
},
{
"id": 1,
"name": "image2",
"type": "image",
"intrinsics": {
"focal": [
721.5377197265625,
721.5377197265625
],
"center": [
609.559326171875,
172.85400390625
],
"skew": 0,
"radial": null,
"tangential": null
},
"extrinsics": {
"location": [
-0.0027968167666705956,
-0.0751087909738869,
-0.27213280776889376
],
"rotation": [
0.011902515224334161,
-1.5603441219160237,
1.548328901848571
]
}
},
{
"id": 2,
"name": "image3",
"type": "image_3d",
"intrinsics": {
"focal": [
721.5377197265625,
721.5377197265625
],
"center": [
609.559326171875,
172.85400390625
],
"skew": 0,
"radial": null,
"tangential": null
},
"extrinsics": {
"location": [
-0.5327253937721252,
0.0,
0.0
],
"rotation": [
-1.5723664220969806,
0.07224191126404556,
-1.8406363281435645
]
}
}
]

We'll let you know as soon as it launches.

We'll let you know as soon as it launches.


Emulate 'createExternalLoadBalancer' support with default 'publicIPs' list

question/possible-enhancement:
Currently in the docs (in various places), I read that if your cloud provider doesn't support 'createExternalLoadBalancer', you have to modify your Service definitions to contain a list of 'publicIPs'.
I'm in the process of testing k8s in various deployment scenarios, and it's a little annoying to have to change my service definitions (or template them and token-replace) for each cluster environment.
Is there any way I could provide a default list of "publicIPs" for my cluster as a fall-back for lack of 'createExternalLoadBalancer'-support?

Good news, I hope. PublicIPs is deprecated, in part because it only sort of
limps along as you experienced.
Instead, set your Services with spec.type = "NodePort". This will allocate
a port (not your Service's port, so no conflicts) that is exposed on every
node. You can then set up your own LB to target that port on one or more
nodes. Or you can use DNS-RR with SRV.
Tim
On Jun 5, 2015 8:53 AM, "Matt DeBoer"<EMAIL_ADDRESS>wrote:

question/possible-enhancement:
Currently in the docs (in various places), I read that if your cloud
provider doesn't support 'createExternalLoadBalancer', you have to modify
your Service definitions to contain a list of 'publicIPs'.
I'm in the process of testing k8s in various deployment scenarios, and
it's a little annoying to have to change my service definitions (or
template them and token-replace) for each cluster environment.
Is there any way I could provide a default list of "publicIPs" for my
cluster as a fall-back for lack of 'createExternalLoadBalancer'-support?
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/9320.


Get this error: SPXERR_GSTREAMER_NOT_FOUND_ERROR

When I'm trying to recognize an alaw audio in a server, Ubuntu Server 18.04, a got this error, but in my local machine running Ubuntu 18.04 runs without errors. I need to install something else in the server to work?
[CALL STACK BEGIN]

/tmp/speech-sdk-native-214117773185900690/libMicrosoft.CognitiveServices.Speech.core.so(+0x197a6d) [0x7f29b6e77a6d]
/tmp/speech-sdk-native-214117773185900690/libMicrosoft.CognitiveServices.Speech.core.so(+0x2ab078) [0x7f29b6f8b078]
/tmp/speech-sdk-native-214117773185900690/libMicrosoft.CognitiveServices.Speech.core.so(+0x228291) [0x7f29b6f08291]
/tmp/speech-sdk-native-214117773185900690/libMicrosoft.CognitiveServices.Speech.core.so(+0x22670f) [0x7f29b6f0670f]
/tmp/speech-sdk-native-214117773185900690/libMicrosoft.CognitiveServices.Speech.core.so(+0x226d9a) [0x7f29b6f06d9a]
/tmp/speech-sdk-native-214117773185900690/libMicrosoft.CognitiveServices.Speech.core.so(+0x227726) [0x7f29b6f07726]
/tmp/speech-sdk-native-214117773185900690/libMicrosoft.CognitiveServices.Speech.core.so(+0x14f7c6) [0x7f29b6e2f7c6]
/tmp/speech-sdk-native-214117773185900690/libMicrosoft.CognitiveServices.Speech.core.so(recognizer_create_intent_recognizer_from_config+0xe9) [0x7f29b6e28f29]
/tmp/speech-sdk-native-214117773185900690/libMicrosoft.CognitiveServices.Speech.java.bindings.so(+0x186f75) [0x7f29b7738f75]
/tmp/speech-sdk-native-214117773185900690/libMicrosoft.CognitiveServices.Speech.java.bindings.so(Java_com_microsoft_cognitiveservices_speech_internal_carbon_1javaJNI_IntentRecognizer_1FromConfig_1_1SWIG_10+0xb7) [0x7f29b76e7c67]
[0x7f29ffccc970]
[CALL STACK END]```

Do you have GStreamer installed? Check with this command in bash:
dpkg --list | grep gstreamer

The following packages are needed:
libgstreamer1.0-0
gstreamer1.0-plugins-base
gstreamer1.0-plugins-good
gstreamer1.0-plugins-bad
gstreamer1.0-plugins-ugly

Do you have GStreamer installed? Check with this command in bash:
dpkg --list | grep gstreamer

The following packages are needed:
libgstreamer1.0-0
gstreamer1.0-plugins-base
gstreamer1.0-plugins-good
gstreamer1.0-plugins-bad
gstreamer1.0-plugins-ugly

I downloaded those plugins, really most of them were not installed. But now I got this error:
[CALL STACK BEGIN]

/tmp/speech-sdk-native-5290572440647486173/libMicrosoft.CognitiveServices.Speech.extension.codec.so(+0xfb0d) [0x7fef684bcb0d]
/tmp/speech-sdk-native-5290572440647486173/libMicrosoft.CognitiveServices.Speech.extension.codec.so(+0xc3b4) [0x7fef684b93b4]
/tmp/speech-sdk-native-5290572440647486173/libMicrosoft.CognitiveServices.Speech.extension.codec.so(+0x8943) [0x7fef684b5943]
/tmp/speech-sdk-native-5290572440647486173/libMicrosoft.CognitiveServices.Speech.core.so(+0x2aac48) [0x7fef69209c48]
/tmp/speech-sdk-native-5290572440647486173/libMicrosoft.CognitiveServices.Speech.core.so(+0x228291) [0x7fef69187291]
/tmp/speech-sdk-native-5290572440647486173/libMicrosoft.CognitiveServices.Speech.core.so(+0x22670f) [0x7fef6918570f]
/tmp/speech-sdk-native-5290572440647486173/libMicrosoft.CognitiveServices.Speech.core.so(+0x226d9a) [0x7fef69185d9a]
/tmp/speech-sdk-native-5290572440647486173/libMicrosoft.CognitiveServices.Speech.core.so(+0x227726) [0x7fef69186726]
/tmp/speech-sdk-native-5290572440647486173/libMicrosoft.CognitiveServices.Speech.core.so(+0x14f7c6) [0x7fef690ae7c6]
/tmp/speech-sdk-native-5290572440647486173/libMicrosoft.CognitiveServices.Speech.core.so(recognizer_create_intent_recognizer_from_config+0xe9) [0x7fef690a7f29]
/tmp/speech-sdk-native-5290572440647486173/libMicrosoft.CognitiveServices.Speech.java.bindings.so(+0x186f75) [0x7fef699b7f75]
/tmp/speech-sdk-native-5290572440647486173/libMicrosoft.CognitiveServices.Speech.java.bindings.so(Java_com_microsoft_cognitiveservices_speech_internal_carbon_1javaJNI_IntentRecognizer_1FromConfig_1_1SWIG_10+0xb7) [0x7fef69966c67]
[0x7fefb091c970]
[CALL STACK END]

It was missings thos libs.

Do you have GStreamer installed? Check with this command in bash:
dpkg --list | grep gstreamer

The following packages are needed:
libgstreamer1.0-0
gstreamer1.0-plugins-base
gstreamer1.0-plugins-good
gstreamer1.0-plugins-bad
gstreamer1.0-plugins-ugly

Thanks!


Pivot capability

Imagine you have a schema : quarter - product - sales with data
("quarter1",  "wine",   100)
("quarter1",  "beer",   220)
("quarter1",  "coffee", 550)
("quarter2",  "coffee", 5)

And you want to pivot by product and sales - with newschema = "wine" , "beer" , "coffee" defaultvalue=0 resulting into data
quarter      wine     beer   coffee
quarter1      100     220      0
quarter2        0       0      5

Also be able to unpivot the data

I've never really understood pivot tables. It seems you're replacing the
column names with values of a different column, so in this case the type of
drinks are the new column headers. Is it sufficient to just say
frame.pivot(columnName), or do you need to specify two, and if two, why.
On 27 January 2016 at 11:11, Antonios Chalkiopoulos <
<EMAIL_ADDRESS>wrote:

Imagine you have a schema : quarter - product - sales with data
("quarter1",  "wine",   100)
("quarter1",  "beer",   220)
("quarter1",  "coffee", 550)
("quarter2",  "coffee", 5)
And you want to pivot by product and sales - with newschema = "wine"
, "beer" , "coffee" defaultvalue=0 resulting into data
quarter wine beer coffee
quarter1 100 220 0
quarter2 0 0 5
Also be able to unpivot the data
—
Reply to this email directly or view it on GitHub
https://github.com/sksamuel/eel/issues/15.

I think this gives some basics analytics capabilities into the lib
One column should be ok

Ala https://issues.apache.org/jira/browse/SPARK-8992 ?

This might have to live in an algo module as pivot requires all (?) / some (?) data in memory at once.


allow non-dict arguments to region definition

Currently outlines, numbers, names, abbrevs, centroids has to be a dictionary. Would be nice to be able to pass any iterable.

closed via #44


Port to BlitzDB

Port to BlitzDB so we have the option of saving worlds to flat file or MongoDB.

Change of plan: Abandon MongoDB entirely in favor of BlitzDB.  The authentication mechanism also shifts around frequently, making different versions incompatible with each other. BlitzDB should be one-and-done. There will also need to be a migration tool.

Makes sense as MongoDB was designed for a very different type of application anyway. Also, just playing devil's advocate, but what about SQLite?

If we use plain SQLite, we have to write queries for everything. I did that with the original Dennis and it was extremely unwieldy and didn't feel natural to write. BlitzDB allows just dumping Python data structures into the database which is very transparent and simple to work with. It's kind of like pickling but without the cross-compatibility issues.
If you can find an abstraction layer that lets us just drop Python data structures directly into SQLite, I'm open to suggestions.

If it’s already been looked at, then I don’t have any further arguments for it. :)

I'm setting up to use a similar Python database solution called TinyDB because it's a more mature project. This is started with bd2440dcc3c2c4e6ef17d66d154ec483eae899c6 which will allow me to convert the database of the testing world.


calicoctl profile rules example is invalid

The help message for profile rule  manipulation outputs an invalid ruleset
  $ calicoctl profile add only-local-pings
  $ calicoctl profile only-local-pings rule add inbound deny icmp
  $ calicoctl profile only-local-pings rule add inbound --at=0 allow from <IP_ADDRESS>/16

The last command is invalid and only produces the help message
$ calicoctl version
0.22.0

Should be ... from cidr <IP_ADDRESS>/16 IIRC

Fixed in #1234


Incorrect XML report generated when registering more than one fixture declared in a non-namespaced unit (fix included in the issue)

https://github.com/VSoftTechnologies/DUnitX/blob/12c4c6f0305b2a5065438a5ccb0d9c8f79c3fb39/Source/DUnitX.FixtureProviderPlugin.pas#L177
This
if not tmpFixtures.TryGetValue(fixtureNamespace,parentFixture) then

should fix it (the parentFixture will be filled with an existing fixture after the first iteration for a fixture declared in a non-namespaced unit).

Possibly also replace the following check: https://github.com/VSoftTechnologies/DUnitX/blob/12c4c6f0305b2a5065438a5ccb0d9c8f79c3fb39/Source/DUnitX.FixtureProviderPlugin.pas#L207
By an assertion like
System.Assert(Assigned(parentFixture));

since I believe it would always be assigned.

Thanks, fixed.


Replace find-remove library

The library works synchronously, which can be a problem in production. Either write an own asynchronous implementation or find a library that is doing the work asynchronously.

Resolved with 75133ebd25a022fd77fd65039e63c52b9169d24d


Samples do not function for multiple reasons

Version is required on Maven <dependency> snippets

@EnableOAuth2Sso is missing (now requires new Maven dependency):
			<groupId>org.springframework.security.oauth.boot</groupId>
			<artifactId>spring-security-oauth2-autoconfigure</artifactId>
			<version>2.0.0.RELEASE</version>
		</dependency>

Attempting to access a protected URL results in exception org.springframework.security.oauth2.client.resource.UserRedirectRequiredException: A redirect is required to get the users approval
Debugging info:
...
2018-04-03 12:16:05.274  INFO 41754 --- [           main] o.s.s.web.DefaultSecurityFilterChain     : Creating filter chain: org.springframework.security.web.util.matcher.AnyRequestMatcher@1, [org.springframework.security.web.context.request.async.WebAsyncManagerIntegrationFilter@1706a5c9, org.springframework.security.web.context.SecurityContextPersistenceFilter@31da3d60, org.springframework.security.web.header.HeaderWriterFilter@468be356, org.springframework.security.web.csrf.CsrfFilter@6a0cbc6f, org.springframework.security.web.authentication.logout.LogoutFilter@4b869331, org.springframework.security.oauth2.client.filter.OAuth2ClientAuthenticationProcessingFilter@64412d34, com.microsoft.azure.spring.autoconfigure.aad.AADAuthenticationFilter@221ca7c, org.springframework.security.web.savedrequest.RequestCacheAwareFilter@65ec8b24, org.springframework.security.web.servletapi.SecurityContextHolderAwareRequestFilter@5e8c34a0, org.springframework.security.web.authentication.AnonymousAuthenticationFilter@4d178d55, org.springframework.security.web.session.SessionManagementFilter@4df39a88, org.springframework.security.web.access.ExceptionTranslationFilter@58e85c6f, org.springframework.security.web.access.intercept.FilterSecurityInterceptor@35178483]
...
org.springframework.security.oauth2.client.resource.UserRedirectRequiredException: A redirect is required to get the users approval
	at org.springframework.security.oauth2.client.token.grant.code.AuthorizationCodeAccessTokenProvider.getRedirectForAuthorization(AuthorizationCodeAccessTokenProvider.java:359) ~[spring-security-oauth2-2.2.1.RELEASE.jar:na]
	at org.springframework.security.oauth2.client.token.grant.code.AuthorizationCodeAccessTokenProvider.obtainAccessToken(AuthorizationCodeAccessTokenProvider.java:205) ~[spring-security-oauth2-2.2.1.RELEASE.jar:na]
	at org.springframework.security.oauth2.client.OAuth2RestTemplate.acquireAccessToken(OAuth2RestTemplate.java:221) ~[spring-security-oauth2-2.2.1.RELEASE.jar:na]
	at org.springframework.security.oauth2.client.OAuth2RestTemplate.getAccessToken(OAuth2RestTemplate.java:173) ~[spring-security-oauth2-2.2.1.RELEASE.jar:na]
	at org.springframework.security.oauth2.client.filter.OAuth2ClientAuthenticationProcessingFilter.attemptAuthentication(OAuth2ClientAuthenticationProcessingFilter.java:105) ~[spring-security-oauth2-2.2.1.RELEASE.jar:na]
	at org.springframework.security.web.authentication.AbstractAuthenticationProcessingFilter.doFilter(AbstractAuthenticationProcessingFilter.java:212) ~[spring-security-web-5.0.3.RELEASE.jar:5.0.3.RELEASE]
	at org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:334) ~[spring-security-web-5.0.3.RELEASE.jar:5.0.3.RELEASE]
	at org.springframework.security.web.authentication.logout.LogoutFilter.doFilter(LogoutFilter.java:116) ~[spring-security-web-5.0.3.RELEASE.jar:5.0.3.RELEASE]
...

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

ID: 2cf5423d-6c4c-26a8-89bb-7686794716d0
Version Independent ID: 734cd358-3dfb-14cd-ee3d-5ce54be8d613
Content: How to use the Spring Boot Starter for Azure Active Directory
Content Source: docs-ref-conceptual/spring-framework/configure-spring-boot-starter-java-app-with-azure-active-directory.md
Service: active-directory
GitHub Login: @rmcmurray
Microsoft Alias: robmcm

This example is broken. Please see item #3 and post solution. stewartadams hints, while useful,  do not solve the problem.  Nor do the 'See also:' items above. These issues have been opened for an entire month with no resolution. You need to keep up!

any idea on when this example will be fixed?

@tschn - we are working on that article now; we should have an update soon.

FWIW, if you're looking for something that works with Spring 5, I published my own sample here:
https://github.com/stewartadam/azuread-springsecurity5-oauth2-kickstart
It uses the built-in OAuth 2 support for Spring Security for Spring 5 instead of the add-in module, which is no longer compatible with Spring 5.

@brunoborges any updating for the issue? Still get the "A redirect is required to get the users approval" exception. Is there anyone can help to fix it?
Thanks!!

@jason279 - I am experiencing the same issue that you are seeing, and we are currently looking into that.
However, for now you can use the example azure-active-directory-spring-boot-sample application as a starting point for building an application; that sample has been tested and it works as expected.

OK. Thanks @rmcmurray .

@stewartadam, @jason279, and @tschn - we've made changes the the Spring starter which resolve the issues that you were seeing, and we've updated the article accordingly:
https://docs.microsoft.com/java/azure/spring-framework/configure-spring-boot-starter-java-app-with-azure-active-directory
Thanks!

Error in import (HelloController.java):
The import org.springframework.security.access.prepost.PreAuthorize cannot be resolved
And
(WebSecurityConfig.java)
The type org.springframework.security.authentication.AuthenticationManager cannot be resolved. It is indirectly referenced from required .class files
These files are present under
Maven Dependencies > spring-security-core-5.0.6.RELEASE.jar
Please help to resolve these errors. Thanks.


How to use parallelism and numPartitions, to speed up the query process and use multicluster for hnsw search.

Hey Thankyou for this project.
I am going though this project but I am not able to understand how to better use hyperparameters used in HnswSimilarity.
I am using like below:
HnswSimilarity(identifierCol='product_id',
                        queryIdentifierCol='user_id',
                        featuresCol='Embedding_items',
                        distanceFunction='cosine',
                        numPartitions = 500,
                        excludeSelf=True,
                        k = 10,
                        parallelism = 100)

I have 144 cores total CPU, 1008 GB RAM, User and product vector size is 300, and user embeddings is total 38 million and product embeddings to be search on is 14 thousands.
How can I use the parameters effectively so that it can run the search in parallel computation.
I tested in my initial run but its making 500 task in spark UI but only 4 core seems to be working.

You should not use a large number of partitions like that. while it will speed up indexing it will slow down querying.
Every partition is a partial index and you will need to query all partitions to resolve the best match (unless you pre-partition your data)
Instead use a small number of partitions on big machines (loads of memory and cores) leave parallelism undefined and use the suggested settings

closing for inactivity

hi, jelmerk, i hava some user vector in million level, and product vector in 10 million level, how to compute the topk product for user? Do I need to join the user and product vectors in advance?


[New Feature]: add adv installer as another default installer

Description of the new feature/enhancement
advanced installer as another default installer  in yaml files
Proposed technical implementation details (optional)
No response

[Policy] Area-Manifest

Would it be good to create another PR to (re)address this issue again?

Yup, thanks


typo error

[Enter feedback here]
the link: https://docs.microsoft.com/en-us/microsoft-365/security/defender-endpoint/onboard-configure?view=o365-worldwide
Micofoft instead microsoft
Configure capabilities of the service
Onboarding devices effectively enables the endpoint detection and response capability of Micorosft Defender for Endpoint.

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

ID: 1e4b5c5a-10ba-3e0c-af47-c51fa5a7ccc3
Version Independent ID: 1704fddc-5cda-e55e-3190-74a748ae2e3d
Content: Onboard devices and configure Microsoft Defender for Endpoint capabilities
Content Source: microsoft-365/security/defender-endpoint/onboard-configure.md
Product: m365-security
Technology: mde
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@tiagodasilva1 Perfect finding typo error

@mjcaparas , Please verify and merge this  PR #8219

@RAJU2529 Thank you very much for the contribution and updating the documentation with PR. @tiagodasilva1 Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Update devices.js to support FB56+ZSW1HKJ2.5 2 gang Nue smart switch

This has the same model number HGZB-042 as FB56+ZSW1HKJ1.7

Thanks!


WatchFile not working

the on change event doesn't work, however I have traced the issue back to the fact that watchFile does not detect changes in the gpio files.  I managed to rig up a ghetto watcher by setInterval and comparing the values then calling a function, but that is bad.
Is there a fix/replacement for this?

I am trying to find a solution. I love this module and without change event working this module is useless. As far as I know hardware interruption doesn't change the mtime of file and fs.watchfile will not work. onoff uses epoll module in which requires python libraries which i don't like. Let's try to find a perfect solution and give you pull request.

@amalraj-nodejs neither onoff nor epoll require python libraries, at least not directly. npm requires node-gyp for building the epoll addon and node-gyp in turn requires python.

fs.watchFile doesn't get fired because the file never gets 'accessed' when setting header via hardware manually watching value changes.

This means that the fileWatcher within rpi-gpio.js will not fire an event after the initial read.
A fix for this is to watch for Interrupts generated by the GPIO chip. If you would like me to build this feature I will be happy to submit a pull request.

Resolved by #23


Code Cleanup

For v1.5

Coverage remained the same at 91.594% when pulling ba8b737000764c20d7fdaffc026d408aa92838f5 on ZY-Ang:Cleanup into b8e55ff12b93e4a830dd1c268a946cc867ed990d on CS2103AUG2017-W11-B2:master.

Coverage remained the same at 91.594% when pulling ba8b737000764c20d7fdaffc026d408aa92838f5 on ZY-Ang:Cleanup into b8e55ff12b93e4a830dd1c268a946cc867ed990d on CS2103AUG2017-W11-B2:master.

Coverage increased (+0.003%) to 91.597% when pulling ac268fd5cd63e597ce8826cee814b96b244c3edc on ZY-Ang:Cleanup into b8e55ff12b93e4a830dd1c268a946cc867ed990d on CS2103AUG2017-W11-B2:master.

Coverage increased (+0.04%) to 91.63% when pulling ad4e3cbe27f78ee99ec53710f83c67d02f1c5a7c on ZY-Ang:Cleanup into b8e55ff12b93e4a830dd1c268a946cc867ed990d on CS2103AUG2017-W11-B2:master.

Coverage increased (+0.003%) to 91.63% when pulling 33a5b2bd71b6a28a2aa2aac0e4eed9e4db0ab3d2 on ZY-Ang:Cleanup into c932d0d347aa890455189cb5ec631227e0483c07 on CS2103AUG2017-W11-B2:master.

Coverage increased (+0.003%) to 91.63% when pulling 33a5b2bd71b6a28a2aa2aac0e4eed9e4db0ab3d2 on ZY-Ang:Cleanup into c932d0d347aa890455189cb5ec631227e0483c07 on CS2103AUG2017-W11-B2:master.

Coverage increased (+0.006%) to 91.633% when pulling b7384c095ef68da580620612450b2ebd1dbfb61c on ZY-Ang:Cleanup into c932d0d347aa890455189cb5ec631227e0483c07 on CS2103AUG2017-W11-B2:master.

Coverage increased (+0.006%) to 91.633% when pulling b7384c095ef68da580620612450b2ebd1dbfb61c on ZY-Ang:Cleanup into c932d0d347aa890455189cb5ec631227e0483c07 on CS2103AUG2017-W11-B2:master.

Coverage increased (+0.08%) to 91.712% when pulling 15cbcc222626f39c317586621277532c1fbd4fdb on ZY-Ang:Cleanup into c932d0d347aa890455189cb5ec631227e0483c07 on CS2103AUG2017-W11-B2:master.

Coverage increased (+0.08%) to 91.712% when pulling 601533d13247d7a1033cf0acb5e3d1d222e148ec on ZY-Ang:Cleanup into c932d0d347aa890455189cb5ec631227e0483c07 on CS2103AUG2017-W11-B2:master.


Documentations site seems to be down / misconfigured

The site https://doc.crablang.org found in README.md seems to be not working:

this needs to be addressed still. there are a lot of links that are 404s right now. in general we need to figure what we're going to do about the documentation. we haven't forked it yet.

You can't cover up homophobic behavior in your team!

hi, I'd love to look into the documentation and why its broken,

@elidakirigo the documentation repo has been forked, so we need to get it deployed under a subdomain of crablang.org


Enable secret store testing

Doc: https://github.com/paritytech/parity/wiki/Secret-Store
Example scripts: https://github.com/svyatonik/sstore_test
--secret-store and defaults to all nodes being key servers?

Closing.


MAINT: Changed default rack height to 0.

Simple change after discussing with others about physical rack space in R89 and R26. Rack height should be 0 as some "devices" such as PDU's don't take up any rack space. Whereas a HV does.

We're currently processing your upload.  This comment will be updated when the results are available.


[WIP] auto launch setting

Adding autolaunch setting
Has a strange bug where reloading the page always enables autolaunch, but disabling and closing then starting the app allows it to stay disabled.

@jessopb How difficult is it to auto launch in a minimized state? (not saying we should necessarily, just curious)

Closing in favor of #3210


'Go to definition' doesn't work for component type name in C#

Version used:
VS 2022 v17.3 Preview 1
To reproduce:

Create blazor server-side project
Go to App.razor
'Go to definition' on MainLayout

Expected behavior:
I appear in MainLayout.cs
Actual behavior:
Nothing happens
Demo:

Additional context:
Very similar to https://github.com/dotnet/razor-tooling/issues/6399, but in https://github.com/dotnet/razor-tooling/issues/6399 the defenition is .razor file, while here - .cs file

@chsienki Can we up the priority of this issue? I run into this issue frequently when I'm trying to walk users through the execution of a Blazor Web App and I want to navigate to the definition of the App component from the app.MapRazorComponents<App>() call in Program.cs.

@jjonescz Can you take a look at this?

A quick note, this might need coordination/design with tooling. The issue is the class definition in Goo.razor.g.cs doesn't map to anything in Goo.razor, but thats because there is nothing for it to map to. If something hacky works, like adding a zero length mapping pointing to the start of the .razor file, that would be great, but otherwise we might need to come up with something special that tooling has explicit support for.

If something hacky works, like adding a zero length mapping pointing to the start of the .razor file, that would be great

Let me try that and see what happens :)

FYI, I have already tried that and there were problems with renaming IIRC. I think some changes in Roslyn might be necessary.


wrong template for Cognito pre-signup triggers

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v14.1.0
Amplify CLI Version
4.51.0
What operating system are you using?
Windows
Amplify Categories
auth, storage, function, api
Amplify Commands
push
Describe the bug
I created a lambda function that runs in cognito triggers (pre-signup) then I ran amplify push and everything worked fine.
Then I created another function that has nothing to do with the previous scenario and when running amplify push I got an error that has to do with the auth module:
UPDATE_FAILED               authanynameauth              AWS::CloudFormation::Stack Fri May
(hora estándar del Atlántico) Parameters: [functionanynameauthPreSignupName, functionanyname ist in the template

Expected behavior
run amplify push and save the new function without getting any errors
Reproduction steps
amplify update auth
select: Walkthrough all the auth configurations
add a trigger in pre-signup
edit the code of the function created
upload changes with: amplify push
add a new function with the Hello Word template: amplify add function
edit the code of the function created
upload changes with: amplify push Error!
GraphQL schema(s)

# Put schemas below this line

Log output

UPDATE_FAILED               authanynameauth              AWS::CloudFormation::Stack Fri May
(hora estándar del Atlántico) Parameters: [functionanynameauthPreSignupName, functionanyname ist in the template

Additional information
No response

I also see that the error seems in the end is not correct but I could understand it.
I went to the .yml template of the auth module and found:
UserPoolPreSignupLambdaInvokePermission:
    Type: "AWS::Lambda::Permission"
    DependsOn: UserPool
    Properties:
      Action: "lambda:invokeFunction"
      Principal: "cognito-idp.amazonaws.com"
      FunctionName: !Ref functionanynameauthPreSignupName
      SourceArn: !GetAtt UserPool.Arn

so I deduced that the error was referring to not finding the value in !Ref functionanynameauthPreSignupName
so I went to the parameters.json file of the auth module and added a new property with the name of the function to be executed in the pre-signup:
{
  "functionanynameauthPreSignupName": "anynameauthPreSignup"
}

Then I ran amplify push and it worked, I'm not sure of the quality of this solution and if amplify really needed to add that information in the parameters, that's why I'll leave the problem open

@DeniferSantiago Was the amplify project created with an earlier version?

Yes, but I don't know which one exactly

as I no longer used cognito-triggers I removed all reference to them in the .yml and .json file

Can you try it with the latest version?

@ammarkarachi  The project was created in a previous version but is currently updated to the latest version of amplify

@DeniferSantiago Can you send us your amplify folder to<EMAIL_ADDRESS>Also specify the issue number in the subject.

I am no longer using lambda triggers due to: https://github.com/aws-amplify/amplify-js/issues/5522


Few URLs scraped through the splash instances returns "Your Browser Is No Longer Supported" error.

Hi All,
Few/List of URLs scraped through the Splash instances (SplashRequest) returns the image with "Your Browser Is No Longer Supported" (Refer the screenshot attached for more info)
Python version - 3.7
Python packages used:

scrapy-splash==0.7.99
scrapy==1.5.1

List of sample URLs:
https://www.ashmoreavenida.com/impact/
https://www.bfycap.com/
https://www.lightship.capital/portfolio
LUA script used to reproduce the error:
function main(splash, args)
assert(splash:go(args.url))
assert(splash:wait(0.5))
return {
html = splash:html(),
png = splash:png(),
har = splash:har(),
}
end
Screenshot:

Please let me know if you need more information on the issue or how to reproduce the error

Did you find a solution?

Using engine=chromium solves this problem for all three of your examples
note this only works for the endpoint render.html
eg
http://<IP_ADDRESS>:8050/render.html?engine=chromium&wait=0.5&images=1&expand=1&timeout=90.0&url=https%3A%2F%2Fwww.bfycap.com%2F&lua_source=function+main%28splash%2C+args%29%0D%0A++assert%28splash%3Ago%28args.url%29%29%0D%0A++assert%28splash%3Await%280.5%29%29%0D%0A++return+%7B%0D%0A++++html+%3D+splash%3Ahtml%28%29%2C%0D%0A++++png+%3D+splash%3Apng%28%29%2C%0D%0A++++har+%3D+splash%3Ahar%28%29%2C%0D%0A++%7D%0D%0Aend

I am also encountering a website that returns the image "Your Browser is No Longer Supported". Setting engine=chromium is an interesting approach, but is limited to endpoint  render.html. Does any one have a solution when dealing with the other endpoints (e.g. render.json execute)?

following this

i am also encountering a website that name https://www.mech.hku.hk/people

i got the same problem - I dont know why it choose automatically safari even my default browser is set to chrome + scrapy splash is running on a vps linux


ci: fixing tests

wow... amazingly, just importing pymmcore before importing any Qt backend ended up fixing stuff on here. 😱
opened an issue at pymmcore: https://github.com/micro-manager/pymmcore/issues/119
but tests are passing here again as long as we ensure that we import pymmcore first


plutus-tx: remove special case handling of typeclass methods

This is misleading and suggests that we have more support for
typeclasses than we do. In addition, it complicates doing things
generically with GHC names since some of them are special.
Clients who want to avoid TH splices can still use the names in
Language.PlutusTx.Builtins, those who wish to have a consistent
interface can use the quotes in Language.PlutusTx.Prelude.

It's already ugly in the typical case where people are dealing with Value, Slot etc. I don't think there's actually all that many integer operations going on. People are free to rebind them to shorter names, of course.
Fixities would be good, indeed.

@j-mueller I made some changes to the tutorials, can you have another look.


Support line filtering in tests

Line filtering was not working when running tests with bin/test.
e.g. bin/test test/attributes_test.rb:82 would run all tests, not only the one defined on line 82.
This is because rails/plugin/test does not support line filtering. We should probably fix that upstream.
I opened https://github.com/rails/rails/issues/48696 to track that.

It works as expected.
$ bin/test test/attributes_test.rb:82
Run options: --seed 43010

# Running:

.

Finished in 0.011638s, 85.9265 runs/s, 85.9265 assertions/s.
1 runs, 1 assertions, 0 failures, 0 errors, 0 skips
$


Investigate the best way to batch send navtech transactions.

Use the anonsend feature of the RPC wallet
Create the transactions in JS and use the RawTx send
Do we split the transactions or not? Can we specify inputs and split the transactions?
Do we do the querying, encyrption and tx creation in Javascript or not.
Size building this function into the RPC wallet
Only use 1 navtech server per transaction cycle

Elegantt data. What's this?

Why would be batch transactions? Seems like something we should only worry about once we have enough transaction going through to actually batch.
I think this should be closed, and remade if/when we decide batch transaction.

@craigmacgregor @matt-auckland

navtech transactions can not be batched, it's only possible to have one anon destination per tx


Updating to work with v12.1 and above

Desired Outcome
Working only with 2nd Gen API (Safes 1st Gen API are soon to be deprecated)
Implemented Changes

Changed all functions to use only 2nd Gen API (Safes and Login)

Documentation

[X] Docs (e.g. READMEs) were updated in this PR

Behavior

[X] This PR changes product behavior and has been reviewed by a PM

@bab29 I would appreciate your review on this
I also would love your suggestion on how to manage this - should we keep the old script as well (assuming we should for customers still on older versions) - how should we do it?
Duplicate the script? different folder?

I don't see an issue with keeping a copy, but I believe the new safe management Rest API has been in place for all currently supported versions.

Agree
But you should be aware that there is a plan to deprecate the 1st
generation API
On Tue, Nov 30, 2021, 20:54 bab29 @.***> wrote:

I don't see an issue with keeping a copy, but I believe the new safe
management Rest API has been in place for all currently supported versions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cyberark/epv-api-scripts/pull/217#issuecomment-982922699,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC3PN3JSI3XAVF7WTUWEUMDUOUMWBANCNFSM5IGSJALQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


V0.4 smart input parsing

Implement escape character \ so that users can use \by to add by in adding or editing name to their entries.
no escape letters for tags yet, i  don't see much point in it
remove logic manager smart input parsing, too many overlapping possibilities to devise a good algorithm.

What was the smart input parsing's job previously?

to automatically decide where the 'by the bay' should go to without user needing to write the escape key .
but its pretty difficult to do add that feature to the edit command, probably possible with some regex but im lazy to do that

Coverage increased (+0.006%) to 77.071% when pulling 93657a8e93e766e0428b609025deb95361cfff8f on V0.4_smart_input_parsing into 23aa45f02b459f8b324639d682406a4717f6269e on V0.4.


DBeaver Project settings are not saved in Eclipse

Project settings consist of locations for scripts, diagrams and nav bookmarks. After I change one of them and press OK and reopen settings values remain default. Version 4.0.4, Eclipse Neon3

Thanks for report


Notify users when probes are not meeting their deadlines

See https://github.com/weaveworks/scope/issues/897#issuecomment-214456825

We should probably also provide feedback when probes are intentionally dropping reports, but that's trickier.

Dupe of #551?  In that ticket we suggest that we keep the latest report from every probe we have ever seen, and if the report is older than the collection window we display the nodes from that report as greyed out (to indicated failed).

Releated https://github.com/weaveworks/scope/issues/943


Add generic "building" as result in searching for not-yet-specified buildings

For example currently searching "shop building" (#7392) has multiple values, but not generic "building".
I tested also with "wat building" search term that can be expected to never return specific building type. Its search results also have no entry for generic building.
I think that it is preferable to have escape hatch with unspecific building=yes rather than forcing user to select one of incorrect building types.
This way someone searching for an overly specific building type or one with currently missing synonyms will have automatically presented downgrade to a generic building.

This sounds okay, we want preset search to be as assistive as possible in iD v3.


M3-3201 Domains - Analytics

Description
Adding domain creation analytics in a similar pattern to what John implemented here: https://github.com/linode/manager/pull/5190
Type of Change

Non breaking change ('update')

Note to Reviewers
Currently we have 2 ways to create domains:
• From the 'Add New' menu dropdown
• From the domains landing page
Please test both these methods of Domain creation. Test that they work as expected, and that the appropriate GA events are being sent.

@johnwcallahan thanks for reviewing! I addressed your feedback.


Update creating-pages.md

Motivation
Typo / wording fix.
Have you read the Contributing Guidelines on pull requests?
yes

(Grammar note: another way to fix this is "Pages do not have sidebars, only docs do."

(Grammar note: another way to fix this is "Pages do not have sidebars, only docs do.")

It's a small difference between the two, but I like this suggestion. Updated it.

@Christopher-Hsieh thanks!


backport: bitcoin#16766, #17164, #17283, #17290, #17388, #17437, #17439, #17470, #17568, #17587, #17599

Issue being fixed or feature implemented
It's next batch of backports from v20
What was done?

bitcoin#17164
bitcoin#16766
bitcoin#17388
bitcoin#17437
bitcoin#17470
bitcoin#17439
bitcoin#17599
bitcoin#17283
bitcoin#17290
bitcoin#17568
bitcoin#17587

How Has This Been Tested?
Run unit/functional tests
Breaking Changes
No breaking changes. Despite that there are several changes related to RPC (backward compatible).
Checklist:

[x] I have performed a self-review of my own code
[x] I have assigned this pull request to a milestone

Looks good but... ❌ Check Merge Fast-Forward Only 👀

Looks good but... ❌ Check Merge Fast-Forward Only 👀

May just need to rebase on develop?


test(css/parser): quotes in value

Just tests for edge cases, all works fine here

We have bug :smile:

@alexander-akait Current css parser is quite naive, and there are really lots to do.
I hope I have more time.

@kdy1 don't worry, I will help you

Thank you a lot!

We really need good CSS parser, currently parser is not good, to be honestly same for HTML, but I think we will do it late

@kdy1 fixed and completed
Notes:

in future we should covert fn read_str(&mut self) -> LexResult<JsWord> -> fn read_str(&mut self) -> LexResult<Token> and add BadStringToken, it allow do recover mode

logic is some mixed, for example read_escape - https://github.com/swc-project/swc/blob/master/css/parser/src/lexer/mod.rs#L313

For spec:

4.3.7. Consume an escaped code point
This section describes how to consume an escaped code point. It assumes that the U+005C REVERSE SOLIDUS () has already been consumed and that the next input code point has already been verified to be part of a valid escape. It will return a code point.

But we use self.input.eat_byte(b'\\'), \ should be already eaten, so we need do check/etc before run read_escape, it allow to have more pure logic

we need more helpers like is_newline/is_whitespace and etc, to have logic in one place

I will improve this in other PRs, here just a fix, better to add more tests with edge cases and do refactor latest, it allow to prevent regressions

What do you think about codegen? How to better fix it?
Codegen output:
content: 'ab';

Source:
    content: "a\
b";

It happens due \\n logic, CSS ignore newline in this case, so newlines just stripped from output.

@kdy1 if we want to keep string as is, we need introduce String { value, rawValue } and keep original string in rawValue and using this for codegen, do you agree with this?

Maintaining two String for values causes lots of problems in my experience.
So I think the answer depends on if a\nb is identical to ab in CSS. I'll search more about it.
If it's identical, I think it should be simply ab and if it's different, storing as ab is breaking input.

Original:
a {
  // These are same
  content: "a\
b";
  content: "ab";
}

Spec - https://www.w3.org/TR/css-syntax-3/#consume-string-token

U+005C REVERSE SOLIDUS ()
If the next input code point is EOF, do nothing.

Our case, just consume

Otherwise, if the next input code point is a newline, consume it.

Otherwise, (the stream starts with a valid escape) consume an escaped code point and append the returned code point to the ’s value.

Ah, I see. Then I think creating a type with two fields is better.
(I want to allow using the parser for various tools)

@kdy1 thanks for feedback, I will do it in near future

I am working on fixes, there are a lot of bugs with lexer/parser, hope I will fix it today

@kdy1 Found bugs/features:

Token::Ident should have raw value too, for example - [attr=\2c] {}, \2c
Token::Url should be raw value too, for example - url(./\2c.css)
Wrong ast for @import (./style.css)

Token::Ident should have raw value too, for example - [attr=\2c] {}, \2c

I see. I think it's not avoidable, so I think ast change is the right way to go.

Token::Url should be raw value too, for example - url(./\2c.css)

Same here.

Wrong ast for @import (./style.css)

Seems like we need to change it as enum? I didn't know that it's allowed to use import like it.

I see. I think it's not avoidable, so I think ast change is the right way to go.
Same here.

Yes, I will do it in other PRs, when rebase it

Seems like we need to change it as enum? I didn't know that it's allowed to use import like it.

Yes, possible values is Str - @import './style.css', Url - @import url(./style.css), Fn -> @import url("./style.css")
Maybe you can give advice how implement this, I am very new in rust and here a lot of magic for me, especial for attributes and macros

@kdy1 ready for merge


add simple description about get slot value

Get the value of a slot using .get_slot() with simple example
Proposed changes:

...

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

Thanks for submitting a pull request 🚀 The assigned reviewer will take a look at it as soon as possible✨


update codecov action

Update Codecov action

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 89.21%. Comparing base (231b038) to head (b35a5fe).
Report is 5 commits behind head on dev.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##              dev      #51   +/-   ##
=======================================
  Coverage   89.21%   89.21%
=======================================
  Files           8        8
  Lines         853      853
=======================================
  Hits          761      761
  Misses         92       92

Flag
Coverage Δ

unittests
89.21% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Expo Update setup in React Native cli init project doesn't reflect changes on Android Emulator

Build/Submit details page URL
https://expo.dev/accounts/ujjwal.manandhar/projects/OTAExample
Summary
I've setup expo updates to react native cli project and followed all the steps mentioned here and used command eas update --channel production to see changes in android emulator but no changes is reflected.
Managed or bare?
Bare workflow
Environment
npx expo-env-info
expo-env-info 1.0.5 environment info:
System:
OS: macOS 13.3.1
Shell: 5.9 - /bin/zsh
Binaries:
Node: 19.9.0 - /usr/local/bin/node
Yarn: 1.22.19 - /usr/local/bin/yarn
npm: 9.6.3 - /usr/local/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
Managers:
CocoaPods: 1.12.1 - /Users/fmi-pc-lt-49/.rvm/gems/ruby-2.7.5/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.4, iOS 16.4, macOS 13.3, tvOS 16.4, watchOS 9.4
IDEs:
Android Studio: 2021.3 AI-213.7172.25.2113.9123335
Xcode: 14.3/14E222b - /usr/bin/xcodebuild
npmPackages:
expo: ^48.0.0 => 48.0.17
react: 18.2.0 => 18.2.0
react-native: 0.71.3 => 0.71.3
npmGlobalPackages:
eas-cli: 3.13.2
Expo Workflow: bare
npx expo-doctor
✔ Validating global prerequisites versions passed
✔ Checking for incompatible packages passed
✔ Checking for conflicting global packages in project passed
✔ Verifying prebuild support package versions are compatible passed
✔ Checking dependency versions for compatibility with the installed Expo SDK passed
✔ Validating Expo Config passed
✔ Checking package.json for common issues passed
Didn't find any issues with the project!
 Learn more: https://expo.fyi/javascript-bundle-sizes
[expo-cli] Finished saving JS Bundles
[expo-cli] Saving assets
[expo-cli] saving /node_modules/react-native/Libraries/NewAppScreen/components/logo.png
[expo-cli] Files successfully saved.
[expo-cli] Dumping asset map
[expo-cli] Dumping source maps
[expo-cli] Preparing additional debugging files
[expo-cli] Export was successful. Your exported files can be found in dist
✔ Exported bundle(s)
✔ Uploaded 2 app bundles
✔ Uploading assets skipped - no new assets found
✔ Channel: main pointed at branch: main
✔ Published!
Branch             main
Runtime version    1.0.3
Platform           android, ios
Update group ID    fb4db8b2-8965-4971-a144-67d50b3583b2
Android update ID  1cc2c3c8-4a42-4ddc-8783-63577237d897
iOS update ID      0dbb505a-5bdb-4e4e-8970-7fc908564773
Message            testing update 6
Commit             98fa93e48a84887ef352581cc77d05e46ab4b89d*
Website link       https://expo.dev/accounts/ujjwal.manandhar/projects/OTAExample/updates/fb4db8b2-8965-4971-a144-67d50b3583b2
fmi-pc-lt-49@fmi-pc-lt-49 OTAExample % npx expo-doctor
✔ Validating global prerequisites versions passed
✔ Checking for incompatible packages passed
✔ Checking for conflicting global packages in project passed
✔ Verifying prebuild support package versions are compatible passed
✔ Checking dependency versions for compatibility with the installed Expo SDK passed
✔ Validating Expo Config passed
✔ Checking package.json for common issues passed
Didn't find any issues with the project!
fmi-pc-lt-49@fmi-pc-lt-49 OTAExample % EXPO_DEBUG=true eas update --channel production --message "testing update 6"

Exporting...
[expo-cli]
Exporting...
[expo-cli] Expo Metro config:
Exporting...
[expo-cli] - Version: 0.7.1
Exporting...
[expo-cli] - Extensions: ts, tsx, js, jsx, json
Exporting...
[expo-cli] - React Native: /Users/fmi-pc-lt-49/Documents/OTAExample/node_modules/react-native
Exporting...
[expo-cli] - Babel config: /Users/fmi-pc-lt-49/Documents/OTAExample/babel.config.js
Exporting...
[expo-cli] - Resolver Fields: react-native, browser, main
Exporting...
[expo-cli] - Watch Folders:
Exporting...
[expo-cli] - Node Module Paths:
Exporting...
[expo-cli] - Exotic: false
Exporting...
[expo-cli] 2023-05-31T03:29:18.256Z expo:start:server:metro:router routerEntry /Users/fmi-pc-lt-49/Documents/OTAExample/node_modules/expo-router/entry /Users/fmi-pc-lt-49/Documents/OTAExample/app ../../app
Exporting...
[expo-cli] Starting Metro Bundler
Exporting...
[expo-cli] warning: Watchman watch-project returned a warning: Recrawled this watch 1 times, most recently because:
Exporting...
[expo-cli] MustScanSubDirs UserDroppedTo resolve, please review the information on
Exporting...
[expo-cli] https://facebook.github.io/watchman/docs/troubleshooting.html#recrawl
Exporting...
[expo-cli] To clear this warning, run:
Exporting...
[expo-cli] watchman watch-del '/Users/fmi-pc-lt-49/Documents/OTAExample' ; watchman watch-project '/Users/fmi-pc-lt-49/Documents/OTAExample'
Exporting...
[expo-cli] warning: Watchman query returned a warning: Recrawled this watch 1 times, most recently because:
Exporting...
[expo-cli] MustScanSubDirs UserDroppedTo resolve, please review the information on
Exporting...
[expo-cli] https://facebook.github.io/watchman/docs/troubleshooting.html#recrawl
Exporting...
[expo-cli] To clear this warning, run:
Exporting...
[expo-cli] watchman watch-del '/Users/fmi-pc-lt-49/Documents/OTAExample' ; watchman watch-project '/Users/fmi-pc-lt-49/Documents/OTAExample'
Exporting...
[expo-cli] Android Bundling complete 2375ms
Exporting...
[expo-cli]
Exporting...
[expo-cli]
Exporting...
[expo-cli] Bundle                     Size
Exporting...
[expo-cli] ┌ index.ios.js           838 kB
Exporting...
[expo-cli] ├ index.android.js       841 kB
Exporting...
[expo-cli] ├ index.ios.js.map      3.55 MB
Exporting...
[expo-cli] └ index.android.js.map  3.56 MB
Exporting...
[expo-cli] 💡 JavaScript bundle sizes affect startup time. Learn more: https://expo.fyi/javascript-bundle-sizes
Exporting...
[expo-cli] iOS Bundling complete 2377ms
Exporting...
[expo-cli] Finished saving JS Bundles
Exporting...
[expo-cli] Saving assets
Exporting...
[expo-cli] 2023-05-31T03:29:20.699Z expo:export:saveAssets saving /Users/fmi-pc-lt-49/Documents/OTAExample/node_modules/react-native/Libraries/NewAppScreen/components/logo.png
Exporting...
[expo-cli] saving /node_modules/react-native/Libraries/NewAppScreen/components/logo.png
Exporting...
[expo-cli] Files successfully saved.
Exporting...
[expo-cli] Dumping asset map
Exporting...
[expo-cli] Dumping source maps
Exporting...
[expo-cli] Preparing additional debugging files
Exporting...
[expo-cli] 2023-05-31T03:29:20.767Z expo:export:write Writing metadata.json to /Users/fmi-pc-lt-49/Documents/OTAExample/dist/metadata.json
Exporting...
[expo-cli] Export was successful. Your exported files can be found in dist
Exporting...
✔ Exported bundle(s)
Uploading...
Loaded 2 platform(s): ios, android
Uploading...
✔ Uploaded 2 app bundles
✔ Uploading assets skipped - no new assets found
✔ Channel: main pointed at branch: main
Publishing...
✔ Published!

Branch             main
Runtime version    1.0.3
Platform           android, ios
Update group ID    c2a3ae21-d4df-407b-a188-bf6f123915fd
Android update ID  f601c03f-2899-4032-aad0-4219a63bbe5a
iOS update ID      121fdbf5-3884-4d3e-8af1-5ba87d5384ae
Message            testing update 6
Commit             98fa93e48a84887ef352581cc77d05e46ab4b89d*
Website link       https://expo.dev/accounts/ujjwal.manandhar/projects/OTAExample/updates/c2a3ae21-d4df-407b-a188-bf6f123915fd
Reproducible demo or steps to reproduce from a blank project
Here's the repo link ->https://github.com/fmujjwalmanandhar/OTAExample
app.json file looks like this
{
"name": "OTAExample",
"displayName": "OTAExample",
"expo": {
"name": "OTAExample",
"slug": "OTAExample",
"ios": {
"jsEngine": "jsc",
"buildNumber": "2",
"bundleIdentifier": "com.otaexample"
},
"android": {
"versionCode": 3,
"package": "com.otaexample"
},
"runtimeVersion": "1.0.3",
"version": "1.0.3",
"updates": {
"enabled": true,
"fallbackToCacheTimeout": 1000,
"requestHeaders": {
"expo-channel-name": "production"
},
"url": "https://u.expo.dev/b3d3e81f-1249-471e-96f4-0ea66b2d7196"
},
"jsEngine": "jsc",
"extra": {
"eas": {
"projectId": "b3d3e81f-1249-471e-96f4-0ea66b2d7196"
}
},
"owner": "ujjwal.manandhar"
}
}
and eas.json
{
"cli": {
"version": ">= 3.13.2"
},
"build": {
"development": {
"distribution": "internal",
"developmentClient": true,
"android": {
"gradleCommand": ":app:assembleDebug"
},
"ios": {
"buildConfiguration": "Debug"
},
"channel": "development"
},
"preview": {
"distribution": "internal",
"channel": "preview"
},
"production": {
"channel": "production"
}
},
"submit": {
"production": {}
}
}
Any help or direction to how updates can be seen in android emulator would be helpful :)

expo updates doesnt woek on native updates ...or assets

hey @fmujjwalmanandhar from the repro and links you provided, it looks like your app is configured for runtimeVersion 1.0.5 but you published an update to runtimeVersion exposdk:48.0.0.  See https://expo.dev/accounts/ujjwal.manandhar/projects/OTAExample/deployments

Closing due to inactivity. We've rewritten the debugging guide to help fix issues like yours more easily: https://docs.expo.dev/eas-update/debug/
If you've gone through this new guide and are still having trouble, feel free to refile another issue.


Bug Fix: Secrets object and key separated by ":"

Make sure you have checked all steps below.
Jira

[ ] My PR addresses the following Airflow Jira issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-XXX
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a Jira issue.

Description

[ ] Here are some details about my PR, including screenshots of any UI changes:

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference Jira issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[ ] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[ ] Passes flake8

Was trying to pass kubernetes secrets to a worker pod and realized the config example was wrong. I changed it in the the airflow project rather than config documentation because I felt it made sense to follow the structure
 = <secrets_object>:<secrets_key>

@uesenthi Travis is failing. Can't we supply a dict instead of exploding strings?

Need to handle backward compatibility as it is already launched in 1.10


kf.show()

how could we show connectivity checks lyrdb in klayout?
    c = mzi_with_bend(radius=50)
    ly = c.connectivity_check()
    ly.save('/home/archie/Downloads/checks.lyrdb')
    # c = gf.routing.add_fiber_single(c)
    kf.show('/home/archie/Downloads/checks.lyrdb')

klive already supports this (and l2n for that matter):
https://github.com/gdsfactory/klive/blob/main/klayout/python/klive_server.py#L126
So, all that's left to do is to make show accept rdb/lyrdb file paths and pass them on to klive as lyrdb/l2n in the data json.

fixed by https://github.com/gdsfactory/kfactory/pull/296


Dialog - "Error - Repository is read only"

In some situations this extension results in showing a dialog "Error: Repository is read only" if one wants to create a new "Untitled.ipynb" notebook by clicking in the launchpad on "Python 3".
I assume it comes from this lines of code:
https://github.com/jupyterlab-contrib/jupyterlab-gitlab/blob/main/src/contents.ts#L321
  newUntitled(options: Contents.ICreateOptions = {}): Promise<Contents.IModel> {
    return Promise.reject('Repository is read only');
  }

As soon as one has interacted with the filebrowser in some way, like creating a new file for example, this problem does not occur anymore. So I assume that the function "newUntitled" of the file-browser-plugin is called when a "Python 3"-notebook is newly created. And somehow the implementation of JupyterLab-GitLab is first in the row - which should not be the case.
I have seen this in JupyterLab 3.6 and now also in 4.2.1.

This might be a general issue with Contents.IDrive so I link this issue as reference: https://github.com/jupyterlab/jupyterlab/issues/15143

The current work-around:
If you want to create a new notebook directly after restarting JupyterLab and do not want to open an existing one, you have to use the context menu of the file browser and not the buttons in the launchpad:

Thanks @jhgoebbert for reporting :+1:
This is likely the same issue as https://github.com/jupyterlab/jupyterlab/issues/15629, as this also happens with the jupyterlab-github extension.


keyed node on validation

Closing this since it has conflicts. We can figure out a better way to do it.


Edit Run Pulsar Locally for clarity

Add explanations for pulsar-daemon start standalone.

Apply the Title Case rule for all headings.

Change gerunds to base verbs in headings.
The gerund is commonly used as a heading since it is suggestive and softer, while the heading plays a significant role in the search result, and using the imperative (base verb) means the heading of the topic match closely to the questions entered by users. For example, users type search keywords like "run pulsar manually" instead of "running pulsar manually".

@sijie @Jennifer88huang Could you please help review? Thanks.

rerun c++ tests

rerun C++ tests

Run C++ / Python Tests

Run CPP Tests

@Anonymitaet Good point.

Concerning the heading capitalization, we have two styles: sentence-style capitalization or headline-style capitalization. Both are Ok. Currently, IBM, Google, Microsoft docs apply sentence-style capitalization, so we can also adopt this style, as long as we keep it consistent.
Base verbs are much better than gerunds in heading. It's great you point this out.


Pin events

Adds functionality for pinned events. This is to ensure that some events don't get deleted, and always stay at the top of the UI. I've wanted this kind of functionality when building a Probot App that requires a specific payload, and I've seen important payloads get lost in a big list.
Here's how it looks (where installation is pinned):

And the button to make it happen:

Todo:

[x] Persist pinnedDeliveries to localStorage
[x] Pass existing tests
[x] Write new tests

Closes #53

One minor nit would be maybe clarifying in the UI that the top event is 'pinned'

@hiimbex yeah I agree with this. Might add a blue stroke to the .Box for the pinned logs? Will update here with some screenshots.

Blue border:

With header:

Let me know which one y'all prefer!

Let me know which one y'all prefer!

I like the option without the blue box, but maybe we just move the heading for "Recent Deliveries" down and change the title to just "Webhook Deliveries"?

I'm a little conflicted, but I think I like the blue box best, but maybe combined with the additional headers @tcbyrd suggested? Although I'm a little thrown off by the additional bold text in that last screenshot.

Although I'm a little thrown off by the additional bold text in that last screenshot.

Yeh, ignore that. I was just editing it in the browser, so it had no styles applied. Used more to illustrate the idea.

Now I'm seeing the word Deliveries ~four~ five times 🙈 There must be a better way. I'll explore it some more.

Here's how it looks as of f78b5b6 - will 🚢 with one last ✅! (or if you have other thoughts we can change it, or iterate in future PRs)

✅My one last tid bit would be change Filter Deliveries [Uses the get-value syntax] to => Filter by [get-value syntax] with the brackets representing the link.

Nice @hiimbex I dig it. Here's how it looks now as of c2519c4:

I'm going to go ahead an merge once CI passes - if there are any changes y'all want to see we can implement in follow up PRs. Thanks for the feedback folks ❤


store,compact,sidecar: replace all usage of json.Decoder with json.Unmarshal

fixes issue #1160
Changes
replaced usages of json.NewDecoder Decode, with ioutil.ReadAll/File and json.Unmarshal
Verification
been running in production for a few days, memory graphs appear marginally better fixes

On my cluster the change is inconsequential:

The difference is only ~50MiB which you can attribute to the different queries which come into my cluster since they are load-balanced. My cluster has ~2TiB data so it's not a small one :(
What's the difference on your cluster with this change? Do you have any graphs?

It is definitely answering to queries hence the sharp-tooth pattern that you see (: that is Thanos Store, by the way. Either way, technically it is a good change.


Issue with eslint 9: .eslintrc-json.json?mtime=1712586949000" needs an import attribute of "type: json"

Since eslint v9, we have the following issue in MegaLinter
[eslint-plugin-jsonc] command: ['eslint', '--no-ignore', '--fix', '-c', '/action/lib/.automation/.eslintrc-json.json', '--no-config-lookup', '--format', '@microsoft/eslint-formatter-sarif', '-o', '/tmp/257cdc5b-5fd5-4117-9ea3-9a25b1c03ecb/sarif/JSON_ESLINT_PLUGIN_JSONC.sarif', 'json_fix_1.json']
[eslint-plugin-jsonc] CWD: /tmp/lint/.automation/test/json
[Sarif] ERROR: there is no SARIF output file found, and stdout doesn't contain SARIF
[Sarif] stdout:
Oops! Something went wrong! :(

ESLint: 9.0.0

TypeError [ERR_IMPORT_ATTRIBUTE_MISSING]: Module "file:///action/lib/.automation/.eslintrc-json.json?mtime=1712586949000" needs an import attribute of "type: json"
    at validateAttributes (node:internal/modules/esm/assert:88:15)
    at defaultLoad (node:internal/modules/esm/load:153:3)
    at async ModuleLoader.load (node:internal/modules/esm/loader:403:7)
    at async ModuleLoader.moduleProvider (node:internal/modules/esm/loader:285:45)
    at async link (node:internal/modules/esm/module_job:78:21)

[eslint-plugin-jsonc] result: 2
Oops! Something went wrong! :(

ESLint: 9.0.0

.eslintrc-json.json contains the following:
{
    "overrides": [
        {
            "files": ["*.json"],
            "extends": [
                "plugin:jsonc/recommended-with-jsonc"
            ],
            "parser": "jsonc-eslint-parser",
                "parserOptions": {
                    "jsonSyntax": "JSONC"
            }
        },
        {
            "files": ["*.jsonc"],
            "extends": [
                "plugin:jsonc/recommended-with-jsonc"
            ],
            "parser": "jsonc-eslint-parser",
                "parserOptions": {
                    "jsonSyntax": "JSONC"
            }
        },
        {
            "files": ["*.json5"],
            "extends": [
                "plugin:jsonc/recommended-with-json5"
            ],
            "parser": "jsonc-eslint-parser",
                "parserOptions": {
                    "jsonSyntax": "JSON5"
            }
        }
    ]
}

Is there some options /configs to change to make eslint-plugin-json work with eslint v9 ?
Many thanks :)

I can't reproduce your problem. This package is not provided by ESM in the first place, so I suspect it is not a plugin issue.

Oh, you're using a JSON for config. JSON config is no longer supported by default in ESLint 9.0.0. You should refer to the migration guide.
https://eslint.org/docs/latest/use/migrate-to-9.0.0

@ota-meshi would you have a ready to use eslint.config.js for eslint-plugin-jsonc ?


Making sense of what works and what doesn't

I want to believe that the scenarios below don't reveal a bug in Cosmosdb 3 with .Net 6, but at the moment I fail to find the consistency in those behaviours.
Could you please read this StackOverflow question , and explain what is wrong there?
I would normally never have the cheek of posting here for such a trivial problem, but there are many questions on the web related to the proper use of the partition key, so I'm not the only one struggling with this.

@mathieubergouniouxcab can you please share a full repro code of the issue? Including any Type definitions and how the client is created, with custom Serializer? We need to be able to repro to investigate.
@kundadebdatta could you take a look once information is available?

@mathieubergouniouxcab

Related to the 2nd SO example: Currently, SDK doesn't support custom serializers in the GetItemLinqQueryable. If you invoke container.GetItemLinqQueryable<MyType>(true).Where(m => m.Id == id).Expression you can see translated to SQL query which will be: SELECT VALUE root FROM root WHERE (root["Id"] = <some id>) that uses original property name ('Id'), not a name from JsonPropertyName attribute ('id' in lowercase). It's a known issue and the SDK team working on this.
See related #2685 for more information.
Related to the 5th SO example: This part of code: .Where(m => m.Id == input.Id).FirstOrDefault(); raises Microsoft.Azure.Cosmos.Linq.DocumentQueryException: 'Method 'FirstOrDefault' is not supported. Currently, SDK does not directly support the FirstOrDefault() method on the GetItemLinqQueryable query.
See LINQ to SQL translation - Azure Cosmos DB for NoSQL | Microsoft Learn for more information.

Very clear answer, thanks.
I thought I was being a good little dev by trying to use System.Text.Json in my CosmosDb code but it's bringing nothing but issues.
Your answer solves my issue.


MIOpen abort when there's no /tmp folder

I see the following crash reported when there's no "/tmp" folder in the docker container.
2020-06-15 01:59:24.750820: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library librocblas.so
2020-06-15 01:59:25.070385: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libMIOpen.so
MIOpen(HIP): Error [LockFilePath] File system operation error in LockFile. Error code: system:20. Description: 'boost::filesystem::temp_directory_path: Not a directory: "/tmp"'
MIOpen(HIP): Error [LockFilePath] File system operation error in LockFile. Error code: system:20. Description: 'boost::filesystem::temp_directory_path: Not a directory: "/tmp"'
MIOpen Error: boost::filesystem::temp_directory_path: Not a directory: "/tmp"
MIOpen Error: boost::filesystem::temp_directory_path: Not a directory: "/tmp"
2020-06-15 01:59:25.802308: F tensorflow/stream_executor/rocm/rocm_dnn.cc:3564] call to miopenFindConvolutionForwardAlgorithm failed: miopenStatusUnknownError
2020-06-15 01:59:25.802327: F tensorflow/stream_executor/rocm/rocm_dnn.cc:3564] call to miopenFindConvolutionForwardAlgorithm failed: miopenStatusUnknownError
Fatal Python error: Aborted

@sunway513, would you be able to assign a priority to this issue?

@DrizztDoUrden please take a look. We should have a environment variable to be able to specify the /tmp directory to be used.
@sunway513 how are other components handling no /tmp directory? Or is it only MIOpen that uses it?

@daniellowell This is just a corner case, by default the OS will have /tmp folders available.
I manually delete that folder (60GB+) after hours of re-tuning to free some space. I've only seen complains from MIOpen end when the /tmp folder is missing.

We should have a environment variable to be able to specify the /tmp directory to be used.

We already have it:
ISO/IEC 9945: The path supplied by the first environment variable
found in the list TMPDIR, TMP, TEMP, TEMPDIR. If none of these
are found, "/tmp", or, if macro __ANDROID__ is defined, "/data/local/tmp".

See https://www.boost.org/doc/libs/1_73_0/libs/filesystem/doc/reference.html#temp_directory_path

comgr uses llvm::sys::path::system_temp_directory(), see
https://github.com/llvm/llvm-project/blob/dab859d1bf250c4d0299ac505e2a6773c56b6503/llvm/lib/Suppor/Unix/Path.inc#L1205
It tries the following temp vars, in order: TMPDIR, TMP, TEMP, TEMPDIR. If none variables defined, /tmp is returned. This matches the logic of boost::temp_directory(), under linux at least.

Quick test shows that both comgr and MIOpen follow the TMPDIR env var.

@sunway513 is this still an issue?

@daniellowell , should I try the tip of develop to test again?

@sunway513 The environment that does not have nether one of {TMPDIR, TMP, TEMP, TEMPDIR} variables set nor /tmp directory does not comply the POSIX base specification.
In other words, any, er, "normal" system should have temp directory created and it should provide apps with information where it resides.

@atamazov , thanks for the info, that's fair enough. I'll close this issue then.


manual decorate dial with fbopt

Codecov Report

Merging #391 (1f0a0ed) into fbopt (4201e02) will increase coverage by 0.41%.
Report is 25 commits behind head on fbopt.
The diff coverage is 46.55%.

@@            Coverage Diff             @@
##            fbopt     #391      +/-   ##
==========================================
+ Coverage   90.43%   90.84%   +0.40%
==========================================
  Files         139      140       +1
  Lines        5240     5289      +49
==========================================
+ Hits         4739     4805      +66
+ Misses        501      484      -17

Flag
Coverage Δ

unittests
90.84% <46.55%> (+0.40%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

domainlab/algos/observers/b_obvisitor.py
84.21% <100.00%> (ø)

domainlab/algos/zoo_algos.py
100.00% <100.00%> (ø)

domainlab/models/a_model.py
93.10% <100.00%> (+0.51%)
:arrow_up:

domainlab/algos/trainers/fbopt.py
16.48% <0.00%> (+16.48%)
:arrow_up:

domainlab/algos/trainers/train_fbopt.py
21.00% <0.00%> (+21.00%)
:arrow_up:

domainlab/algos/trainers/train_dial.py
86.66% <25.00%> (-13.34%)
:arrow_down:

domainlab/algos/trainers/a_trainer.py
88.54% <50.00%> (-6.64%)
:arrow_down:

domainlab/algos/builder_fbopt_dial.py
48.00% <48.00%> (ø)

... and 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

RuntimeError: element 0 of tensors does not require grad and does not have a grad_fn


SQL injection possible

Since you are inserting raw text in raw statements, a user can exploit your bot with an SQL injection.
Sanitize your SQL statements - use prepared statements and the ? placeholder to insert user input - and additionally validate the input before inserting it.

Hi,
Which SQL Statement in particular are you talking about?
All of my SQL Statements that I wrote here are using placeholders.

Hey, you are right - I was pretty sure I saw a statement where you were building a raw sql statement via %s replacement before calling the cursor.execute() method. But I can't find it anymore, so I guess I just misread.
This was probably in one of those lines:
https://github.com/xlanor/CryptoKitties/blob/master/src/commands.py#L225-L228
because of the strange formatting I lost track of where the string belongs to. But of course these placeholders are being used by the cursor.execute() method to insert the parameters in the prepared statement.
Sorry for the false positive 🙈.
PS: The bot isn't online anymore?

Nope, I took it offline since I stopped playing cryptokitties - I initially wrote the bot for my colleagues because they were all caught up in that game!


Will using a single version of fmt tool be a good idea?

I noticed that the workflow will use all fmt tools from stable, beta and nightly version. But cargo fmt tool will change from version to version, and the result may be variable. So will it be a good idea to just use a single version of fmt tool?

@leavelet good point. We can fix the fmt tool to the stable version.

@qianxi0410 can you work on this?

EMM. This error is not caused by the FMT tool, in fact, you did not format:
let mut user_now = user!(name="tmp".to_owned());

it should be:
let mut user_now = user!(name = "tmp".to_owned());

/cc @hsluoyz @leavelet

Maybe some setting went wrong on my own machine. let me take a look.

My settings:
  "rust-analyzer.checkOnSave.command": "clippy",
  "[rust]": {
    "editor.defaultFormatter": "rust-lang.rust-analyzer",
    "editor.formatOnSave": true
  },

FYI @leavelet

After applying @qianxi0410 's suggestion, the PR has pass all CI checks. But I think it's still beneficial to reduce some unnecessary checks, currently 9 combinations: (win, mac, linux) x (stable, beta, nightly). it is not only slow in PR validation but also waste the CI minutes of the org limited by github. Any suggestions? @leavelet @qianxi0410 I think we can remove nightly at least?

I think maybe stable version on Linux is enough. So far all the code is platform independent, and if we don't use unstable feature, Rust will provide strong compatibility

OK. @qianxi0410 I think we can keep 4: (windows, ubuntu) x (stable, beta)

I'll do this later.

And the tests may fail on GitHub as they can't run without a valid casdoor deployment.
Maybe we can use the casdoor demo for test, and tests should be reconfigured.

@leavelet the demo site: https://door.casdoor.com/ could be used. It will be always online. It's a read-only site.


Now that we have Changed, optimize all the things

We should be using Changed<T> everywhere we can to avoid doing redundant work:

[X] Transforms
[ ] Parents/Children
[X] Labels
[ ] Render Nodes

[ ] lights
[ ] camera
[X] render_resources (potentially a huge win here)

[X] Cameras
[ ] ShaderDefs
[ ] Sprites
[ ] UI update system (only update "dirty" parts of the tree)
[X] Pipeline Compiler (only run when RenderPipelines component has changed)

I was looking into doing this for the transforms systems. For transform_translation_rotation_scale_system, the current query is:
Query<Without<LocalTransform, Without<NonUniformScale, (&mut Transform, &Translation, &Rotation, &Scale)>>>

ideally we could do something like
Query<Without<LocalTransform, Without<NonUniformScale, (&mut Transform, &Translation, &Rotation, &Scale, Any<Changed<Translation>, Changed<Rotation>, Changed<Scale>>)>>>

the idea being we want to run this system on anything that has Transform, Rotation and Scale, and at least one of those was changed.
I asked in Discord and it seems like this is not currently possible (tldr queries are and only, no or right now)

@cart it may be worth changing your list into checkboxes so we can track status

What's left for this task?

What's left for this task?

Camera seems undone (from my cursory grepping through the codebase) but it's honestly kind of hard to figure out what has been done.

Just updated the description with checkboxes indicating whats been done.

I'm currently working on a rework of parenting/children. And #1046  is covering the ShaderDefs case.

This should be looked at again once #68 is solved and we add explicit dependencies to our default systems for 0.5.


RSA Interop

Various changes to registration object
Reorganizes RSA modes to be their own ciphers, similar to how the KDFs are implemented
Adds files for sig and keygen kat handlers
Updates app code to handle processing of the vectors (only for the modes we register for)

Hi all,
So I've been trying to update to the PKCS #11 driver from the old libacvp to the new one and I've ran into the issue that the new libacvp now has a dependency on the openssl BigNum library, which means libacvp must link with openssl even though your library under test isn't openssl. I think it should have some library independent way of passing the RSA components, either as straight hex (as it comes from the server) or as a simple byte array. All the libraries know how turn byte arrays into their own individual BigNums. Would you be willing to entertain a patch that does this?
Thanks,
bob

Hey Bob!
I actually just added this to my todo list earlier today... we are on the same page! I was thinking the easiest way to do this would be to leave the values as character arrays since that is what they are when we parse the JSON, then the crypto handler can do what it needs to do. Does that seem reasonable?
Thanks!
Ellie

Cool. Yes leaving them as hex char arrays seems reasonable. The avoids the double conversion hex->byte array->native BN.


Vyper scaffolding

Motivation

Added support for Vyper
Solution

no need to copy over the entire brownie config, a usual config should be pretty minimal, something like:
networks:
    default: hardhat

project_structure:
    build: vyper/build
    contracts: vyper/contracts/
    interfaces: vyper/interfaces
    reports: vyper/reports
    scripts: vyper/scripts
    tests: vyper/test

autofetch_sources: true

if you already use hardhat, there is no need introduce another dependency of ganache, brownie supports hardhat properly.
for pytest+foundry, we'd likely need to migrate to ape later, but hold off for now, it doesn't support everything we need yet.

config
no need to copy over the entire brownie config, a usual config should be pretty minimal, something like:
networks:
    default: hardhat

project_structure:
    build: vyper/build
    contracts: vyper/contracts/
    interfaces: vyper/interfaces
    reports: vyper/reports
    scripts: vyper/scripts
    tests: vyper/test

autofetch_sources: true

hardhat
if you already use hardhat, there is no need introduce another dependency of ganache, brownie supports hardhat properly.
for pytest+foundry, we'd likely need to migrate to ape later, but hold off for now, it doesn't support everything we need yet.
isolation
also add a vyper/test/conftest.py with this to get the test chain reverted to a state after all fixtures are set up after each test:
import pytest

@pytest.fixture(autouse=True)
def isolate(fn_isolation):
    pass

read more here
dev revert comments
also note that brownie supports dev revert strings, e.g.
assert water > 0, "no water in seaport"

can be written as a comment
assert water > 0  # dev: no water in seaport

then it can be caught in tests like this:
with brownie.reverts(dev_revert_msg="no water in seaport"):
    seaport.check_water_levels()

gotcha, on it ser

Changes LGTM, as part of this scope do you want to setup CI github actions?
this is one example we have, altho it may have some steps that arent needed at this point.
https://github.com/yearn/yearn-vaults/blob/main/.github/workflows/test.yaml

Let’s get this in on the next PR so that we unblock on setting up scaffolding!


[Not for merge] test

This PR is to test initial prow set-up is working as expected from https://github.com/openshift/release/pull/39231

good that basic set-up is working
Now, need to see what tweaking is needed for containerlint test to pass

/test containerlint

/test containerlint

Not needed. Should be fixed with https://github.com/openshift/rhcos-image-layering-examples/pull/16


Add MyData Taiwan organization: https://www.mydata.org.tw/

The purpose of MyData Global is to empower individuals by improving their right to self-determination regarding their personal data. It is based on the MyData Declaration.
MyData Global has nearly 90 organisation members and over 600 individual members from over 40 countries, on six continents. We facilitate the 2000+ strong MyData Global community working on the ethical use of personal data.
MyData Taiwan is one of the local branches of MyData Global.
Signed-off-by: Bofu Chen (bafu<EMAIL_ADDRESS>

請問您的目標是？Types of changes

[ ] 新增追蹤團隊(或個人) Add new org/user the tracking list
[ ] 移除現有的追蹤團隊(或個人) Remove new org/user the tracking list
[ ] New feature
[ ] Bug fix

請問您的理由是？ Description

@bafu
Thanks~ would you mind update your PR with the new template? I just paste under your comment.

接受 PR 規則討論中 Slack #infra

已加mylog14


Allow hotkeys to be turned off

Allow consumers to opt out of the hotkeys.

This is solved in #51 since hotkeys are now handled by the consumer and they can now choose to not put the hotkeys on the buttons in the toolbar.


Code Coverage

[WIP] Trying to get codeclimate.com setup properly.

Any word on this? I remember there being some odd changes we would need to make to get our code climate score up. I'd rather ignore that arbitrary rating if it will require mangling our source.

Closing since we're running of the babel boilerplate now


Send an exit code when the script terminates

Proposed Changes
This fixes the issue identified in https://github.com/Dealerdirect/phpcodesniffer-composer-installer/pull/80#issuecomment-531586417 where even when the setting of installed_paths failed, the plugin would exit with exit code 0.
A 0 exit code will be returned if successful, 1 - or the exit code returned by PHPCS itself - if not.
This will allow wrapper scripts and/or process scripts which take exit codes into account to fail correctly and/or notify users of failure.
Testing this change

On this repo, throw away the vendor directory and composer.lock file.
Make sure you are on the master branch.
Run composer install.
Now for the sake of testing this change, copy the vendor/squizlabs/php_codesniffer/CodeSniffer.conf.dist file to vendor/squizlabs/php_codesniffer/CodeSniffer.conf and make it read only.
Now run composer install-codestandards --verbose.
The output will look something like:Composer version 1.9.1 2019-11-01 17:20:17

> install-codestandards: Dealerdirect\Composer\Plugin\Installers\PHPCodeSniffer\Plugin::run
Running PHPCodeSniffer Composer Installer
Failed to set PHP CodeSniffer installed_paths Config
ERROR: Config file I:\path\to\phpcodesniffer-composer-installer\vendor\squizlabs\php_codesniffer\CodeSniffer.conf is not writable

Run echo $? and take note of the exit code being 0 (= success).
Now switch to this branch and run composer install-codestandards --verbose again.Composer version 1.9.1 2019-11-01 17:20:17

> install-codestandards: Dealerdirect\Composer\Plugin\Installers\PHPCodeSniffer\Plugin::run
Running PHPCodeSniffer Composer Installer
Failed to set PHP CodeSniffer installed_paths Config
ERROR: Config file I:\path\to\phpcodesniffer-composer-installer\vendor\squizlabs\php_codesniffer\CodeSniffer.conf is not writable

Run echo $? and take note of the exit code now being 1 (= failure).

Another test which tests another part of this change:

Follow steps 1 to 3 from above.
Now for the sake of testing this change, delete the vendor/squizlabs/php_codesniffer directory.
On master, run composer install-codestandards --verbose.
The output will look something like:Composer version 1.9.1 2019-11-01 17:20:17

> install-codestandards: Dealerdirect\Composer\Plugin\Installers\PHPCodeSniffer\Plugin::run
Running PHPCodeSniffer Composer Installer
PHPCodeSniffer is not installed

Run echo $? and take note of the exit code being 0 (= success).
Now switch to this branch and run composer install-codestandards --verbose again.Composer version 1.9.1 2019-11-01 17:20:17

> install-codestandards: Dealerdirect\Composer\Plugin\Installers\PHPCodeSniffer\Plugin::run
Running PHPCodeSniffer Composer Installer
PHPCodeSniffer is not installed

Run echo $? and take note of the exit code now being 1 (= failure).

@Potherca @mjrider Anything I can do to move this PR (and the others) forward ?


Added support for description in options

The description in options is currently not shown. This pull request shows the description in a format which matches the new assistant preview.

@andrewyuau thank you for you contribution. Could you please remove the html tags around title

@stevenpkg Yes I can remove the tags though the title will look like the description. How do you propose we differentiate the title from the description? See screenshot below:

The preview link bolds the title as shown below:

@stevenpkg I have removed the tags around the title. Thanks.

I think the title and description distinction was a good idea. Why did you want to remove it @stevenpkg?
Thanks again for the changes @andrewyuau

@germanattanasio The title is part of the conversation, not the title of a page. Perhaps make the description italic if you need to differentiate. Although why do we need to differentiate?

@andrewyuau Have you tried using the Welcome message in a separate Dialog node (welcome or start_conversation) with the list/buttons as a child node and the title as "I can help with directions to...." ? This would provide greater flexibility in terms of traffic control as the watson assistant develops.

@germanattanasio @andrewyuau lets make the discription italic. Please add this to app.css
.description-area {
  font-style: italic;
}

and add
var description = '
' + gen.description + '
';

@germanattanasio @andrewyuau @stevenpkg Perhaps I'm missing something here but users can set the format on the node itself rather than hard-coding a format at the code level. Shouldn't we leave the system open to individual requirements?

@germanattanasio @andrewyuau @stevenpkg Perhaps I'm missing something here but users can set the format on the node itself rather than hard-coding a format at the code level. Shouldn't we leave the system open to individual requirements?

Most of our users wouldn't know that's possible (at least I don't!). As this is a sample application, I would say the less the user needs to do something, the better it is. If they have individual requirements, they should be advanced users who can fork this repo and make their customisations. Thoughts?

@germanattanasio @andrewyuau lets make the discription italic. Please add this to app.css
.description-area {
  font-style: italic;
}

and add
var description = '
' + gen.description + '
';

Ok done.

@germanattanasio The title is part of the conversation, not the title of a page. Perhaps make the description italic if you need to differentiate. Although why do we need to differentiate?

Since we allow the capability to input title and description separately, I assume users would expect to see the differentiation in this sample app.

@germanattanasio @andrewyuau @stevenpkg Perhaps I'm missing something here but users can set the format on the node itself rather than hard-coding a format at the code level. Shouldn't we leave the system open to individual requirements?

Most of our users wouldn't know that's possible (at least I don't!). As this is a sample application, I would say the less the user needs to do something, the better it is. If they have individual requirements, they should be advanced users who can fork this repo and make their customisations. Thoughts?

The same can be said for the inverse. By hard-coding the text format we're moving the work required from a simple HTML format in Watson Assistant to more complex code changes, it's not reducing the amount of work required, simply changing the nature of the work required.
Changing the JSON in Watson assistant is a simpler process than changing the code in the UI and allows for users of all levels to work with the Sample App.
As you said, if users want to deviate from the generic code they can fork the repo and make their customisations.
@stevenpkg As you're responsible for the code I'll leave this with you.

@Oliver-I you brought a perfect point. @andrewyuau, @stevenpkg what do you guys think? in this case we would do the change in the training json I guess

@germanattanasio I think its okay for now, to hard code description italic. Perhaps in the readme we can write or point a link about formatting title and description etc in the Assistant workspace. I am happy either way. @Oliver-I what do you think?


Overall cost does not match up

So the cost shown in my cli is a bunch of

metagpt.provider.openai_api:update_cost:89 - Total running cost: $0.196 | Max budget: $3.000 | Current cost: $0.016, prompt_tokens: 4660, completion_tokens: 491
metagpt.provider.openai_api:update_cost:89 - Total running cost: $0.214 | Max budget: $3.000 | Current cost: $0.018, prompt_tokens: 5165, completion_tokens: 615
metagpt.provider.openai_api:update_cost:89 - Total running cost: $0.180 | Max budget: $3.000 | Current cost: $0.015, prompt_tokens: 4684, completion_tokens: 326
...

A bunch of such lines show the cost. but they add up to more than the cost shown in my openAI account. I am happy but wanted to know the calculation and what do each cost show above mean.

This is a cost estimation based on the number of tokens exchanged between MetaGPT and the Language Model (LLM), and it's provided solely as a reference metric. The actual expenses will be determined by the official settlement. You can find this code snippet in the _calc_usage function in metagpt/provider/openai_api.py.


bindfs: Let mount find bindfs mount helper

When mount is used with mount-type "fuse.bindfs", it cannot find the mount helper.
If mount can't find mount.fuse.bindfs, it executes the mount.fuse mount helper and passes fuse.bindfs as argument. Then mount.fuse tries to execute bindfs on its own, but it is not found in the PATH.
By creating a mount.fuse.bindfs link to the bindfs executable, this problem is avoided because mount will just execute the mount.fuse.bindfs mount helper without mount.fuse in the middle.
Things done:

[ ] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

More
cc @lovek323, Sorry for the duplicate PR.

Please note, that points are not mandatory, but rather desired.

Built locally, thanks.

Hey cool, glad I could help :)


PacketEvents packet adapter is broken

NPCs don't show up when using the PacketEvents packet adapter in 1.19.3. If I add ProtocolLib then it works fine.

I'm aware of that, I will try to fix that tomorrow.

Fixed in b088d466623825c04c663f0a5e866dfdd3c7b6d4


Allow Developers to change DOM parsing method

As a user of Stagehand, I might want to change the method of DOM parsing:
from Travis:

Have some modularity (backstage) that allows developers to choose which method they want to extract data from (DOM cleaning vs marking up DOM vs screenshotting)

Related to #149 and #184


juju-resources fails verification when files have the same name

We have hadoop-2.4.1.tgz for x86_64 and aarch64.  When fetching --all resources from a resources.yaml, juju-resources gives an "invalid" error for aarch64 because the x86_64 tarball has the same base name.
juju-resources should handle the case when multiple resources have the same base filename.

If you have the SHA doesn't it make sense to use that name?

I went with using the resource name because we apparently support URLs for remote hashes, though I'm not sure it's being used at the moment.  It also matches a bit better with how Python resources are handled.
TBH, it would be great to scrap the whole mirror thing in favor of something more robust using existing mirror packages, as Adam had been discussing.


pass encoding=utf8 when opening files

I think this should solve #1284
https://dev.to/methane/python-use-utf-8-mode-on-windows-212i

May also solve #1276

Do you mind resolving the conflicts? I think it's just a case of running the new nbdev_clean AFAICT.

@jph00 , done.
Would you also like to merge https://github.com/fastai/nbdev-template/pull/9 ?

Would you also like to merge fastai/nbdev-template#9 ?

Yes I would! :D (BTW, if you at-mention me on a PR then it goes to my inbox and won't be missed or forgotten.)


Dict feature is non-nullable while nested dict feature is

When i try to create a Dataset object with None values inside a dict column, like this:
from datasets import Dataset, Features, Value

Dataset.from_dict(
    {
        "dict": [{"a": 0, "b": 0}, None],
    }, features=Features(
        {"dict": {"a": Value("int16"), "b": Value("int16")}}
    )
)

i get ValueError: Got None but expected a dictionary instead.
At the same time, having None in nested dict feature works, for example, this doesn't throw any errors:
from datasets import Dataset, Features, Value, Sequence

dataset = Dataset.from_dict(
    {
        "list_dict": [[{"a": 0, "b": 0}], None],
        "sequence_dict": [[{"a": 0, "b": 0}], None],
    }, features=Features({
        "list_dict": [{"a": Value("int16"), "b": Value("int16")}],
        "sequence_dict": Sequence({"a": Value("int16"), "b": Value("int16")}),
    })
)

Other types of features also seem to be nullable (but I haven't checked all of them).
Version of datasets is the latest atm (2.18.0)
Is this an expected behavior or a bug?

It looks like a bug, by default every feature should be nullable.

I've linked a PR with a fix :)

@mariosasko awesome thank you!


Empty horizontal spacing creates empty line

Description
Consider the case
#set page(width: auto, height: auto, margin: 10pt)
#h(0pt) #rect()

where a block elements follows a paragraph containing only empty horizontal spacing (weak or not).
The behavior of this case changed with 0.12:
Previously, the empty line before the rectangle was fully discarded, but now, the line remains, shifting the rectangle down.

≤ 0.11.1
0.12.0

Reproduction URL
No response
Operating system
Web app, Windows
Typst version

[X] I am using the latest version of Typst

I think the 0.12 behaviour is pretty reasonable. The horizontal spacing becomes part of a paragraph and that paragraph is spaced apart from the rectangle. I'm not sure why 0.11 even behaved the way it did. If the spacing is non-empty 0.11 also adds vertical spacing, which could be considered inconsistent.

Wouldn't it make sense though to collapse the whole paragraph when nothing is in it? Maybe a strong 0pt spacing can be considered as something, but a weak spacing only really exists when there is something else that keeps it alive.

The weak spacing case indeed seems like a bug.


Extension not available anymore on the Chrome Web Store

Sadly, the url to the Chrome Web Store leads to an error 404.
It seems to be not available for some time now.
What is the reason for this? Could you reupload the extension?

The extension has been removed from Chrome Web Store. To use it on Chrome, download the compressed archive like so.
Then, uncompress that archive somewhere on your PC.
Then go to chrome://extensions on your Chrome browser.

Enable Developer mode
Click Load unpacked
Select the folder where the extension resides