python code generator version and grpc python dependency for protobuf are not aligned

We hit the following issue recently:

the Python gRPC 0.13 stubs are generated with protobuf 3.0.0-beta2
(which is also the version you use as submodule)

the gRPC python 0.13 source install command requires python protobuf >= 3.0.0-alpha3. However, 3.0.0a3 is not compatible with the Python code generated by protobuf 3.0.0-beta2.

When we try to run the 3.0.0b2 generated code with the 3.0.0a3 library, we see this error:
TypeError: __init__() got an unexpected keyword argument 'syntax'

Usually, this is not a problem because pip install . installs the most recent version 3.0.0b2 by default. But we've hit certain cases where it installs 3.0.0a3 and the code fails to run.
Tagging @soltanmm to get his attention on this.

This doesn't sound like a protobuf 3.0.0b2 vs. 3.0.0a3 issue, it sounds like a protobuf 2.x vs protobuf 3.x issue. The stubs are generated from C++ code that has remained (w.r.t. the salient bits of this discussion) unchanged as we've upgraded protobuf versions AFAIK.
 Can you provide, for both your runtime and your build environments, the output of pip show protobuf?

You're right. It seems to be a protobuf 2.x vs 3.x issue.
Initially, we thought the "syntax" parameter was not part of protobuf 3.x. But that's not the case.
The only thing we know so far is that:
from google import protobuf
print protobuf.__version__

prints:
3.0.0-alpha-1

but:
from google.protobuf import descriptor as _descriptor
DESCRIPTOR = _descriptor.FileDescriptor(
  name='topodata.proto',
  package='topodata',
  syntax='proto3',
  serialized_pb=_b('\n\x...')
)

fails:
$ ./test/resharding.py
Traceback (most recent call last):
  File "./test/resharding.py", line 14, in <module>
    from vtproto import topodata_pb2
  File "/home/travis/gopath/py-vtdb/vtproto/topodata_pb2.py", line 23, in <module>
    ount\x18\x05 \x01(\x05\x1ar\n\x11KeyspacePartition\x12)\n\x0bserved_type\x18\x01 \x01(\x0e\x32\x14.topodata.TabletType\x12\x32\n\x10shard_references\x18\x02 \x03(\x0b\x32\x18.topodata.ShardReference\x1aI\n\nServedFrom\x12)\n\x0btablet_type\x18\x01 \x01(\x0e\x32\x14.topodata.TabletType\x12\x10\n\x08keyspace\x18\x02 \x01(\t*2\n\x0eKeyspaceIdType\x12\t\n\x05UNSET\x10\x00\x12\n\n\x06UINT64\x10\x01\x12\t\n\x05\x42YTES\x10\x02*\x8f\x01\n\nTabletType\x12\x0b\n\x07UNKNOWN\x10\x00\x12\n\n\x06MASTER\x10\x01\x12\x0b\n\x07REPLICA\x10\x02\x12\n\n\x06RDONLY\x10\x03\x12\t\n\x05\x42\x41TCH\x10\x03\x12\t\n\x05SPARE\x10\x04\x12\x10\n\x0c\x45XPERIMENTAL\x10\x05\x12\n\n\x06\x42\x41\x43KUP\x10\x06\x12\x0b\n\x07RESTORE\x10\x07\x12\n\n\x06WORKER\x10\x08\x1a\x02\x10\x01\x42\x1a\n\x18\x63om.youtube.vitess.protob\x06proto3')
TypeError: __init__() got an unexpected keyword argument 'syntax'

See the full log here: https://travis-ci.org/youtube/vitess/builds/113076391
I also cannot reproduce the issue locally when I explicitly install protobuf=3.0.0a3. I'll do more debugging on Travis :(

When y'all are in your runtime environment, do you have multiple installations of protobuf across the system, a virtualenv, and/or any other configuration that results in multiple versions of protobuf existing on the same system? Even if you think they're not supposed to interact, are they there? This includes things like *.egg files that are installed when gathering dependencies via setup.py scripts et al.

@soltanmm I found the issue now.
The clash is between protobuf 3.0.0 -alpha-1 and a3.
grpc python install triggers install of protobuf 3.0.0a3:
Downloading/unpacking protobuf>=3.0.0a3 (from grpcio==0.13.0)
  Downloading protobuf-3.0.0-alpha-1.tar.gz (193Kb): 193Kb downloaded
  Running setup.py egg_info for package protobuf

[...]

  Running setup.py install for protobuf
    Skipping installation of /home/travis/gopath/dist/grpc/usr/local/lib/python2.7/site-packages/google/__init__.py (namespace package)

    Installing /home/travis/gopath/dist/grpc/usr/local/lib/python2.7/site-packages/protobuf-3.0.0_alpha_1-py2.7-nspkg.pth

But as you can see it's installing the wrong version, namely 3.0.0-alpha-1.
I can reproduce the issue when I install alpha-1 locally.
The work-around is to manually install version a3 or higher:
$ pip install -U protobuf==3.0.0a3

The problem is probably that "-alpha-1" always wins over "a3" when pip compares the versions?

Issue was already reported on the protobuf project: https://github.com/google/protobuf/issues/855
Nathaniel already ran into it last year :(
Given that Travis still has the package available, you should consider to pin gRPC Python to a specific version until protobuf 3.0.1 or higher is released ;-)

I'm gonna push back on that, please bear with me:
Have you tried clearing the pip cache on the target machine (or skipping it w/ --no-cache-dir)? It looks like Travis CI doesn't run its own PyPI mirror, so the package has to be coming from somewhere else (hopefully somewhere relatively ephemeral).

I feel like Travis CI is caching the bad package somewhere on their side.
I am not familiar with Python/easy_install but this looks suspicious:

Installed /tmp/easy_install-ov8mg_/protobuf*-3.0.0-alpha-1/google_apputils-0.4.2-py2.7.egg

It looks like /tmp/easy_install-ov8mg_/protobuf-3.0.0-alpha-1/ is a cache which contains multiple packages.
I'll file a bug with them.
For now, I'll probably just locally patch the >=3.0.0a3 to a == during our bootstrap phase as long as we are in this transition period.
Thanks for looking into it :)


Add Nix flake

i'm sorry but i'm not looking to add packaging stuff. you're welcome to do so in your own repo.

Got it. Thank you!


Nydus sub-blob garbage collection support

In order to support container OS upgrade, each increased blob stores the data difference of previous container OS images.
However, the number of such blobs can finally be very huge which impacts performance and even cannot work properly anymore, so we need to decrease the number of total blobs.

Why there is an impact on performance if layers is getting increased.?

Why there is an impact on performance if layers is getting increased.?

since we need to mount (or open all blobs in advance.)
The description is still incomplete for now(just to give a link to who is needed), I will fill the full request later.


Unable to update ingresses

Describe the bug
The operator is unable to update ingresses after changing the ingressHost
ingresses.networking.k8s.io "cluster-artemis-wconsj-0-svc-ing" is forbidden:
User "system:serviceaccount:artemis:activemq-artemis-controller-manager" cannot update
resource "ingresses" in API group "networking.k8s.io" in the namespace "artemis"

[!TIP]
Vote this issue reacting with :+1: or :-1:

This issue will be fixed in 1.1.0


unable to create installer-provisioned 4.6 cluster on AWS

I think this is related to https://github.com/openshift/okd/issues/430 but for AWS. It started failing for me on 17 Dec when the first of those issues was raised and I see the same error as https://github.com/openshift/okd/issues/435#issuecomment-747989107
ryan9.seldondev.com:6443/api/v1/namespaces/kube-system/configmaps?allowWatchBookmarks=true&fieldSelector=metadata.name%3Dbootstrap&resourceVersion=16143&timeoutSeconds=442&watch=true": dial tcp <IP_ADDRESS>:6443: connect: connection refused
ERROR Cluster operator authentication Degraded is True with IngressStateEndpoints_MissingSubsets::OAuthServiceCheckEndpointAccessibleController_SyncError::OAuthServiceEndpointsCheckEndpointAccessibleController_SyncError: OAuthServiceEndpointsCheckEndpointAccessibleControllerDegraded: oauth service endpoints are not ready
OAuthServiceCheckEndpointAccessibleControllerDegraded: Get "https://<IP_ADDRESS>:443/healthz": context deadline exceeded (Client.Timeout exceeded while awaiting headers)

Log bundle attached
log-bundle-20210104104832.tar.gz
I've only tried with the default for networkType so based on that linked issue I guess I should try ovn
It is not a one-off or intermittent connectivity issue. I've tried 8 times with different variations on cluster size.
The end of the install log shows
INFO Bootstrap gather logs captured here "/home/ryan/Downloads/openshift-install-linux/ryan9/log-bundle-20210104104832.tar.gz"
FATAL Bootstrap failed to complete: failed to wait for bootstrapping to complete: timed out waiting for the condition

Namespaces for the cluster are created and an external loadbalancer is also created. But two pods in openshift-kube-apiserver are in CrashLoopBackoff.
Their logs show repeated connection errors
I0104 11:19:05.818497      17 apf_controller.go:170] NewTestableController with serverConcurrencyLimit=4000, requestWaitLimit=15s
I0104 11:19:05.819253      17 client.go:360] parsed scheme: "endpoint"
I0104 11:19:05.819303      17 endpoint.go:68] ccResolverWrapper: sending new addresses to cc: [{https://<IP_ADDRESS>:2379  <nil> 0 <nil>} {https://<IP_ADDRESS>:2379  <nil> 0 <nil>} {https://localhost:2379  <nil> 0 <nil>}]
W0104 11:19:05.819809      17 clientconn.go:1223] grpc: addrConn.createTransport failed to connect to {https://<IP_ADDRESS>:2379  <nil> 0 <nil>}. Err :connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:2379: connect: connection refused". Reconnecting...

Also noticed I can't get routes
oc get route
error: the server doesn't have a resource type "route"

Now tried with networkType: OVNKubernetes. Still getting errors:
E0104 12:41:36.597466   39261 reflector.go:307] k8s.io/client-go/tools/watch/informerwatcher.go:146: Failed to watch *v1.ConfigMap: Get "https://api.ryan10.seldondev.com:6443/api/v1/namespaces/kube-system/configmaps?allowWatchBookmarks=true&fieldSelector=metadata.name%3Dbootstrap&resourceVersion=16647&timeoutSeconds=366&watch=true": dial tcp <IP_ADDRESS>:6443: connect: connection refused
ERROR Cluster operator authentication Degraded is True with IngressStateEndpoints_MissingSubsets::OAuthServiceCheckEndpointAccessibleController_SyncError::OAuthServiceEndpointsCheckEndpointAccessibleController_SyncError: OAuthServiceEndpointsCheckEndpointAccessibleControllerDegraded: oauth service endpoints are not ready
OAuthServiceCheckEndpointAccessibleControllerDegraded: Get "https://<IP_ADDRESS>:443/healthz": dial tcp <IP_ADDRESS>:443: connect: connection refused
IngressStateEndpointsDegraded: No subsets found for the endpoints of oauth-server
INFO Cluster operator authentication Progressing is Unknown with NoData:
INFO Cluster operator authentication Available is False with OAuthServiceCheckEndpointAccessibleController_EndpointUnavailable::OAuthServiceEndpointsCheckEndpointAccessibleController_EndpointUnavailable: OAuthServiceEndpointsCheckEndpointAccessibleControllerAvailable: Failed to get oauth-openshift enpoints
OAuthServiceCheckEndpointAccessibleControllerAvailable: Get "https://<IP_ADDRESS>:443/healthz": dial tcp <IP_ADDRESS>:443: connect: connection refused
ERROR Cluster operator etcd Degraded is True with EnvVarController_Error::InstallerController_Error::RevisionController_ContentCreationError::ScriptController_Error::StaticPods_Error: StaticPodsDegraded: pods "etcd-ip-10-0-175-88.eu-west-2.compute.internal" not found
StaticPodsDegraded: pods "etcd-ip-10-0-143-96.eu-west-2.compute.internal" not found
ScriptControllerDegraded: "configmap/etcd-pod": missing env var values
EnvVarControllerDegraded: at least three nodes are required to have a valid configuration
InstallerControllerDegraded: missing required resources: [configmaps: etcd-scripts,restore-etcd-pod, configmaps: config-1,etcd-metrics-proxy-client-ca-1,etcd-metrics-proxy-serving-ca-1,etcd-peer-client-ca-1,etcd-pod-1,etcd-serving-ca-1, secrets: etcd-all-peer-1,etcd-all-serving-1,etcd-all-serving-metrics-1]
RevisionControllerDegraded: configmaps "etcd-pod" not found
INFO Cluster operator etcd Progressing is True with NodeInstaller: NodeInstallerProgressing: 2 nodes are at revision 0; 0 nodes have achieved new revision 1
INFO Cluster operator etcd Available is False with StaticPods_ZeroNodesActive: StaticPodsAvailable: 0 nodes are active; 2 nodes are at revision 0; 0 nodes have achieved new revision 1
INFO Cluster operator ingress Available is False with IngressUnavailable: Not all ingress controllers are available.
INFO Cluster operator ingress Progressing is True with Reconciling: Not all ingress controllers are available.
ERROR Cluster operator ingress Degraded is True with IngressControllersDegraded: Some ingresscontrollers are degraded: ingresscontroller "default" is degraded: DegradedConditions: One or more other status conditions indicate a degraded state: DeploymentAvailable=False (DeploymentUnavailable: The deployment has Available status condition set to False (reason: MinimumReplicasUnavailable) with message: Deployment does not have minimum availability.), DeploymentReplicasMinAvailable=False (DeploymentMinimumReplicasNotMet: 0/2 of replicas are available, max unavailable is 1)
ERROR Cluster operator insights Degraded is True with PeriodicGatherFailed: Source config could not be retrieved: certificatesigningrequests.certificates.k8s.io is forbidden: User "system:serviceaccount:openshift-insights:gather" cannot list resource "certificatesigningrequests" in API group "certificates.k8s.io" at the cluster scope: RBAC: clusterrole.rbac.authorization.k8s.io "cluster-reader" not found, configmaps is forbidden: User "system:serviceaccount:openshift-insights:gather" cannot list resource "configmaps" in API group "" in the namespace "openshift-config": RBAC: clusterrole.rbac.authorization.k8s.io "cluster-reader" not found, customresourcedefinitions.apiextensions.k8s.io "volumesnapshots.snapshot.storage.k8s.io" is forbidden: User "system:serviceaccount:openshift-insights:gather" cannot get resource "customresourcedefinitions" in API group "apiextensions.k8s.io" at the cluster scope: RBAC: clusterrole.rbac.authorization.k8s.io "cluster-reader" not found, hostsubnets.network.openshift.io is forbidden: User "system:serviceaccount:openshift-insights:gather" cannot list resource "hostsubnets" in API group "network.openshift.io" at the cluster scope: RBAC: clusterrole.rbac.authorization.k8s.io "cluster-reader" not found, pods is forbidden: User "system:serviceaccount:openshift-insights:gather" cannot list resource "pods" in API group "" at the cluster scope: RBAC: clusterrole.rbac.authorization.k8s.io "cluster-reader" not found

Log bundle attached.
log-bundle-20210104125220.tar.gz
attached

Now tried with networkType: OVNKubernetes. Still getting errors:
E0104 12:41:36.597466   39261 reflector.go:307] k8s.io/client-go/tools/watch/informerwatcher.go:146: Failed to watch *v1.ConfigMap: Get "https://api.ryan10.seldondev.com:6443/api/v1/namespaces/kube-system/configmaps?allowWatchBookmarks=true&fieldSelector=metadata.name%3Dbootstrap&resourceVersion=16647&timeoutSeconds=366&watch=true": dial tcp <IP_ADDRESS>:6443: connect: connection refused
ERROR Cluster operator authentication Degraded is True with IngressStateEndpoints_MissingSubsets::OAuthServiceCheckEndpointAccessibleController_SyncError::OAuthServiceEndpointsCheckEndpointAccessibleController_SyncError: OAuthServiceEndpointsCheckEndpointAccessibleControllerDegraded: oauth service endpoints are not ready
OAuthServiceCheckEndpointAccessibleControllerDegraded: Get "https://<IP_ADDRESS>:443/healthz": dial tcp <IP_ADDRESS>:443: connect: connection refused
IngressStateEndpointsDegraded: No subsets found for the endpoints of oauth-server
INFO Cluster operator authentication Progressing is Unknown with NoData:
INFO Cluster operator authentication Available is False with OAuthServiceCheckEndpointAccessibleController_EndpointUnavailable::OAuthServiceEndpointsCheckEndpointAccessibleController_EndpointUnavailable: OAuthServiceEndpointsCheckEndpointAccessibleControllerAvailable: Failed to get oauth-openshift enpoints
OAuthServiceCheckEndpointAccessibleControllerAvailable: Get "https://<IP_ADDRESS>:443/healthz": dial tcp <IP_ADDRESS>:443: connect: connection refused
ERROR Cluster operator etcd Degraded is True with EnvVarController_Error::InstallerController_Error::RevisionController_ContentCreationError::ScriptController_Error::StaticPods_Error: StaticPodsDegraded: pods "etcd-ip-10-0-175-88.eu-west-2.compute.internal" not found
StaticPodsDegraded: pods "etcd-ip-10-0-143-96.eu-west-2.compute.internal" not found
ScriptControllerDegraded: "configmap/etcd-pod": missing env var values
EnvVarControllerDegraded: at least three nodes are required to have a valid configuration
InstallerControllerDegraded: missing required resources: [configmaps: etcd-scripts,restore-etcd-pod, configmaps: config-1,etcd-metrics-proxy-client-ca-1,etcd-metrics-proxy-serving-ca-1,etcd-peer-client-ca-1,etcd-pod-1,etcd-serving-ca-1, secrets: etcd-all-peer-1,etcd-all-serving-1,etcd-all-serving-metrics-1]
RevisionControllerDegraded: configmaps "etcd-pod" not found
INFO Cluster operator etcd Progressing is True with NodeInstaller: NodeInstallerProgressing: 2 nodes are at revision 0; 0 nodes have achieved new revision 1
INFO Cluster operator etcd Available is False with StaticPods_ZeroNodesActive: StaticPodsAvailable: 0 nodes are active; 2 nodes are at revision 0; 0 nodes have achieved new revision 1
INFO Cluster operator ingress Available is False with IngressUnavailable: Not all ingress controllers are available.
INFO Cluster operator ingress Progressing is True with Reconciling: Not all ingress controllers are available.
ERROR Cluster operator ingress Degraded is True with IngressControllersDegraded: Some ingresscontrollers are degraded: ingresscontroller "default" is degraded: DegradedConditions: One or more other status conditions indicate a degraded state: DeploymentAvailable=False (DeploymentUnavailable: The deployment has Available status condition set to False (reason: MinimumReplicasUnavailable) with message: Deployment does not have minimum availability.), DeploymentReplicasMinAvailable=False (DeploymentMinimumReplicasNotMet: 0/2 of replicas are available, max unavailable is 1)
ERROR Cluster operator insights Degraded is True with PeriodicGatherFailed: Source config could not be retrieved: certificatesigningrequests.certificates.k8s.io is forbidden: User "system:serviceaccount:openshift-insights:gather" cannot list resource "certificatesigningrequests" in API group "certificates.k8s.io" at the cluster scope: RBAC: clusterrole.rbac.authorization.k8s.io "cluster-reader" not found, configmaps is forbidden: User "system:serviceaccount:openshift-insights:gather" cannot list resource "configmaps" in API group "" in the namespace "openshift-config": RBAC: clusterrole.rbac.authorization.k8s.io "cluster-reader" not found, customresourcedefinitions.apiextensions.k8s.io "volumesnapshots.snapshot.storage.k8s.io" is forbidden: User "system:serviceaccount:openshift-insights:gather" cannot get resource "customresourcedefinitions" in API group "apiextensions.k8s.io" at the cluster scope: RBAC: clusterrole.rbac.authorization.k8s.io "cluster-reader" not found, hostsubnets.network.openshift.io is forbidden: User "system:serviceaccount:openshift-insights:gather" cannot list resource "hostsubnets" in API group "network.openshift.io" at the cluster scope: RBAC: clusterrole.rbac.authorization.k8s.io "cluster-reader" not found, pods is forbidden: User "system:serviceaccount:openshift-insights:gather" cannot list resource "pods" in API group "" at the cluster scope: RBAC: clusterrole.rbac.authorization.k8s.io "cluster-reader" not found

Log bundle attached.
log-bundle-20210104125220.tar.gz
attached

This is different, etcd didn't come up

This is different, etcd didn't come up

I've now tried one using just the defaults (no changes to install-config so openshfitsdn) and that actually did work today. Wasn't working before. I need a larger cluster though so will try that.

I've now tried one using just the defaults (no changes to install-config so openshfitsdn) and that actually did work today. Wasn't working before. I need a larger cluster though so will try that.

That is working now too. Can't see what has changed as I'm doing the same thing that failed multiple times before. I guess I'll close this issue.

That is working now too. Can't see what has changed as I'm doing the same thing that failed multiple times before. I guess I'll close this issue.


facepunch.libsdf API changes

LayerTextures is now ReferencedTextures in Sdf2DLayer
BoxSdf is now RectSdf (for 2D stuff)
Slight shader parameter tweak

Thank you, very cool!


Exception when trying to provision from a cloud image

Got this while testing #10561.  Am also seeing this on master branch when provisioning from a cloud image on the Provider screen.
[----] I, [2016-08-19T14:23:43.209894 #29770:3c94020]  INFO -- : Started POST "/vm_cloud/x_button?pressed=image_miq_request_new" for ::1 at 2016-08-19 14:23:43 -0700
[----] I, [2016-08-19T14:23:43.226772 #29770:3c94020]  INFO -- : Processing by VmCloudController#x_button as JS
[----] I, [2016-08-19T14:23:43.226863 #29770:3c94020]  INFO -- :   Parameters: {"miq_grid_checks"=>"10r55", "pressed"=>"image_miq_request_new"}
[----] F, [2016-08-19T14:23:43.328866 #29770:3c94020] FATAL -- : Error caught: [NoMethodError] undefined method `get_timezones' for #<ManageIQ::Providers::Amazon::CloudManager::ProvisionWorkflow:0x007f91ceea1de0>
Did you mean?  get_buttons
/home/dclarizio/dev/manageiq/app/models/miq_request_workflow.rb:318:in `get_field'
/home/dclarizio/dev/manageiq/app/models/miq_request_workflow.rb:305:in `block in get_all_fields'
/home/dclarizio/dev/manageiq/app/models/miq_request_workflow.rb:305:in `each_key'
/home/dclarizio/dev/manageiq/app/models/miq_request_workflow.rb:305:in `get_all_fields'
/home/dclarizio/dev/manageiq/app/models/miq_request_workflow.rb:145:in `block in init_from_dialog'
/home/dclarizio/dev/manageiq/app/models/miq_request_workflow.rb:144:in `each'
/home/dclarizio/dev/manageiq/app/models/miq_request_workflow.rb:144:in `init_from_dialog'
/home/dclarizio/dev/manageiq/app/controllers/application_controller/miq_request_methods.rb:809:in `prov_set_form_vars'
/home/dclarizio/dev/manageiq/app/controllers/application_controller/miq_request_methods.rb:210:in `prov_edit'
/home/dclarizio/dev/manageiq/app/controllers/application_controller/miq_request_methods.rb:110:in `pre_prov'
/home/dclarizio/dev/manageiq/app/controllers/application_controller.rb:1869:in `prov_redirect'
/home/dclarizio/dev/manageiq/app/controllers/application_controller/explorer.rb:94:in `x_button'

Any update on this?  I'm actually trying to learn more about programmatic provisioning and having a heck of a time with the documentation.  Is there a simple test you can provide, such as:
ruby my-test.rb
?

@b-long You might want to take a look at this tool that ships with the project to help rebuild existing provision requests: https://github.com/ManageIQ/manageiq/pull/8425

Cool, thanks @gmcculloug 😄  Will do!


Responsive viewer size

Hi,
we are currently using RCSB 1D viewer in the PrankWeb project (GitHub, live demo). I would like to know if it is possible to add an option to make the 1D component responsive (i.e. the 1D viewer will fit into a fixed-size <div>).
Currently, as far as I know, we have to somehow compute the track sizes based on the actual window size, which kind-of works, but the final design does not look that nice, as the viewer contains some fixed-size components such as labels or the side menu. It would be great to have the opportunity not to do the sizing manually.
Thanks in advance, by the way the library is great.

Hi @luk27official, thank you for using rcsb-saguaro library and feedback. If I understand correctly, you would like the 1D viewer to adapt its width to the parent container automatically and dynamically change when the parent component width changes. Is that correct?

Hello,
exactly, the adaptation is the main thing that would be nice to have implemented, setting the sizes manually is a bit tedious.
The dynamic change would be great as well, I have looked through the documentation and it seems like that maybe could be done via the updateBoardConfig method that would get called on any window resize? (but that is just my wild guess)
Anyway, we would appreciate both of these. Thank you.

Hi,
I am working on a react component to mitigate this issue (and I see I am not alone with this problem).

You can check it and feel free to use it or review and give feedback.
https://gist.github.com/cycle20/25dcd7db9e99462bbb8a93df0eebd8f5
Final usage after registering the custom element:
<tm-saguaro-1d  id="tms1d" elementId="xy" autoResize="true" rowTitleWidth="150" trackWidth="700" url="./zdh15_saguaro_7khm_A.json" side1="Inside" />

(registering: customElements.define("tm-saguaro-1d", TmFv1DElement);)
About autoResize. I need to handle Fenix (Android Firefox) case, when the autoresize does not work at all,
since during the component rendering the parent element does not get its final width and it causes a shorter
axis track. Only in Fenix; desktop firefox and chrome works well.
So I have to disable autoResize on Fenix and provide fix widths.
But I hope there will be an official solution soon ;)

Hello again after almost a year ... It took me some time to focus on this issue. I have published a new version v3.1.0. If RcsbFvBoardConfigInterface does not include trackWidth the track width will be defined based on the root container. If the root container width is modified the sequence annotation viewer will be rendered. Let me know if this solves your issue.

Hi,
thanks for the update, regarding the width - this indeed works!
Maybe it could be useful to have similar functionality for the track heights - i.e. if I have 4 tracks that should fit into a 200px <div>, then the track heights would be automatically assigned the heights of 50px (let's not count margins/paddings etc.), again this could be dynamic. On the other side, I understand that this would be more complicated and this approach would not allow the user to assign different heights for the tracks if this setting was used. I'm leaving the comment here just as an inspiration for potential improvement.


[QA] Knights of Antrom (144,-7) scene missing

Explorer Alpha build versions:

v0.21.0

Issue Description:
The scene "Knights of Antrom" at coordinates (144,-7) is missing. It does not load or appear as expected. In the production environment (Prod), the scene is available and loads correctly.
❗ This is an SDK7 scene.
STR:

Launch the explorer in EA.
Teleport to coordinates (144,-7).
Observe that the "Knights of Antrom" scene does not appear.

Expected behavior:
The "Knights of Antrom" scene should be present and load correctly at coordinates (144,-7) in EA, as it does in Prod.

Current behavior:
The "Knights of Antrom" scene is missing and does not load at coordinates (144,-7) in EA.

Fixed with latest conversion after the recent AB server changes


Preventing the className calls on SVG elements

Instead of a standard dom node, the className method on a SVG element will return an SVGAnimatedString instead of a string, wich doesnt implements the match method.

Thanks! Could you also add a test case?


How to stop VMenu from closing

When I type text in a pop-up window, it automatically closes when the mouse is over the input method. How do I fix this

?

Hi, I just happen to see this while I was searching for an answer myself.
You can provide the property autoHide to the VMenu component and set it to false.


Release needed

Hi, the plugins needs a release :) as reported in https://github.com/getgrav/grav/issues/935 there is an issue with Admin/Grav 1.1 which I fixed in https://github.com/ellioseven/grav-plugin-view/pull/1, if you make a new release the GPM will pick it up and users won't have issues with it. Thanks!

All done thank you!


PE-429- declaration summary box changes

PE-429- declaration summary box changes

PE-442,443,444 commits
442 - PM: Documents uploaded of second owner is displayed as zero in summary and preview  page
443 - PM: Validation not shown when date of allotment is greater than date of possession
444 - PM: Preview displays same document name multiple times for owner document.


new UI

Change UI :

add main color
fix font-family
fix tabs
change font-size (22px -> 15px)
remove Bootstrap
and small fixes

Cool, thank you!
Already merged it into develop branch. But we should probably set the header's left padding/margin to same as the main content's, right?

@andywer Yes 👍  And i soon add new features for UI

Awesome! I merged, but did not yet re-release.

PS: Pull once from this repo to get two minor fixes. Just changed the footer link styling slightly (text-decoration) & matched the header margin/padding.

@andywer Thx 👍


[monitoring] Expose public metrics to a different port

Motivation
Expose public_metrics to a different port. Using a separate port so that it will be easier for partners to config on their side.
Temporarily put network_gauge as the public_metrics we expose right now, will gather a list of metrics we want to publish and replace it later.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan

cargo run -p libra-swarm -- -s -l -n 4
get the port number from node.config.toml file
curl --silent "http://localhost:63470/metrics"
curl --silent "http://localhost:63472/metrics" // public port
make sure metrics are collected correctly.

Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

:hourglass: Testing commit ba9486c367a918f0f25a671ec3c8e6696efd2cab with merge b03e524b924be5c6600ee37a677af598913e6d71...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: mgorven
Pushing b03e524b924be5c6600ee37a677af598913e6d71 to master...


Unit tests for package internal/gem

Need to write unit tests for the internal/gem package

[x] #2
[x] #3
[x] #10
[x] #11
[x] #15

all subtasks complete, marking as complete


small images doesn't work (?)

Hi, I updated to the new code but when I put an small image (more small than a container preview) this image doesn't load. I try to set the new property smallImage to allow but nothing. By the way, with the old code all runs fine.
Thanks :-)

It should work. Do you mind posting your code snippet?

If you are using the v0.4.0, the property smallImage: 'allow' should be work.


Change c-style for loop to for-in loop based on proposal SE-0007

Based on @erica's proposal: SE-0007, remove C-style for loop in Prespecialized.swift to prevent future warnings and removal in Swift 2.2 and 3.0 respectively

Thanks!
This code never runs, so it can not affect performance.  Merging.


General notes on usability issues revealed by documentation writing

I just wanted to publicly track the things I came across somewhere as I find them, so I don't forget.

[ ] We need to write these magic declarations for things like __nondet that really should be implicitly available already. The prelude isn't easily includable, and shouldn't need explicit including anyway.
[ ] cargo-rmc doesn't really work well (e.g. doesn't handle dependencies), and running rmc and cargo on a crate is currently a magic incantation.
[ ] "Assertion failed: blah: SUCCESS" is confusing. Failed...Success?
[ ] MIR gives us a lot of generated variables that are mostly just copies and obfuscate traces. (e.g. var_10 = 0; then x = var_10;)
[ ] Needing the harness to be unmangled main is bothersome, I really think we want something like #[proof] #464

Re: point 3, previously filed this similar one: https://github.com/model-checking/rmc/issues/189

I think this one remaining item still has value:

There should be a "simpler" version of traces for the common case of "just tell me the nondet values in my proof harness"

But otherwise this tracking issue is largely complete and no longer useful. Closing.


Bump lucene codec to 99

Description
This PR includes the following changes:

Add new kNN codec that wraps Lucene new codec 9.9 and make it default
Fix import statements for Lucene Codec 95
Fix the arguments mismatch issue with SegmentInfo Constructor
Temporarily Ignore the old lucene codec tests (KNN910Codec, KNN920Codec, KNN940Codec, KNN950Codec). Due to some recent changes in Lucene for SegmentInfo causing these tests to fail which will be fixed later with a separate PR.

Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

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

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

@naveentatikonda CI is failing for mac.

@naveentatikonda CI is failing for mac.

Yes, CI is failing on macOS with below error. Not sure what was the exact reason. Guessing if something was wrong while publishing the changes by core to maven because it is a valid class and working fine for other OS.
java.util.ServiceConfigurationError: org.apache.lucene.codecs.Codec: Provider org.opensearch.knn.index.codec.KNN910Codec.KNN910Codec could not be instantiated
»  	at java.base/java.util.ServiceLoader.fail(ServiceLoader.java:586)
»  	at java.base/java.util.ServiceLoader$ProviderImpl.newInstance(ServiceLoader.java:813)
»  	at java.base/java.util.ServiceLoader$ProviderImpl.get(ServiceLoader.java:729)
»  	at java.base/java.util.ServiceLoader$3.next(ServiceLoader.java:1403)
»  	at org.apache.lucene.util.NamedSPILoader.reload(NamedSPILoader.java:68)
»  	at org.apache.lucene.codecs.Codec.reloadCodecs(Codec.java:136)
»  	at org.opensearch.plugins.PluginsService.reloadLuceneSPI(PluginsService.java:753)
»  	at org.opensearch.plugins.PluginsService.loadBundle(PluginsService.java:706)
»  	at org.opensearch.plugins.PluginsService.loadBundles(PluginsService.java:533)
»  	at org.opensearch.plugins.PluginsService.<init>(PluginsService.java:195)
»  	at org.opensearch.node.Node.<init>(Node.java:484)
»  	at org.opensearch.node.Node.<init>(Node.java:411)
»  	at org.opensearch.bootstrap.Bootstrap$5.<init>(Bootstrap.java:242)
»  	at org.opensearch.bootstrap.Bootstrap.setup(Bootstrap.java:242)
»  	at org.opensearch.bootstrap.Bootstrap.init(Bootstrap.java:404)
»  	at org.opensearch.bootstrap.OpenSearch.init(OpenSearch.java:180)
»  	at org.opensearch.bootstrap.OpenSearch.execute(OpenSearch.java:171)
»  	at org.opensearch.cli.EnvironmentAwareCommand.execute(EnvironmentAwareCommand.java:104)
»  	at org.opensearch.cli.Command.mainWithoutErrorHandling(Command.java:138)
»  	at org.opensearch.cli.Command.main(Command.java:101)
»  	at org.opensearch.bootstrap.OpenSearch.main(OpenSearch.java:137)
»  	at org.opensearch.bootstrap.OpenSearch.main(OpenSearch.java:103)
»  Caused by: java.lang.NoClassDefFoundError: org/apache/lucene/backward_codecs/lucene95/Lucene95Codec
»  	at org.opensearch.knn.index.codec.KNN910Codec.KNN910Codec.<clinit>(KNN910Codec.java:20)
»  	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
»  	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:77)
»  	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
»  	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:499)
»  	at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:480)
»  	at java.base/java.util.ServiceLoader$ProviderImpl$2.run(ServiceLoader.java:797)
»  	at java.base/java.security.AccessController.doPrivileged(AccessController.java:712)
»  	at java.base/java.util.ServiceLoader$ProviderImpl.newInstance(ServiceLoader.java:802)
»  	... 20 more
»  Caused by: java.lang.ClassNotFoundException: org.apache.lucene.backward_codecs.lucene95.Lucene95Codec
»  	at java.base/java.net.URLClassLoader.findClass(URLClassLoader.java:445)
»  	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:592)
»  	at java.base/java.net.FactoryURLClassLoader.loadClass(URLClassLoader.java:872)
»  	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:525)
»  	... 29 more

@naveentatikonda CI is failing for mac.

Yes, CI is failing on macOS with below error. Not sure what was the exact reason. Guessing if something was wrong while publishing the changes by core to maven because it is a valid class and working fine for other OS.
Block (41 lines)
java.util.ServiceConfigurationError: org.apache.lucene.codecs.Codec: Provider org.opensearch.knn.index.codec.KNN910Codec.KNN910Codec could not be instantiated
»  	at java.base/java.util.ServiceLoader.fail(ServiceLoader.java:586)
»  	at java.base/java.util.ServiceLoader$ProviderImpl.newInstance(ServiceLoader.java:813)
»  	at java.base/java.util.ServiceLoader$ProviderImpl.get(ServiceLoader.java:729)
»  	at java.base/java.util.ServiceLoader$3.next(ServiceLoader.java:1403)
»  	at org.apache.lucene.util.NamedSPILoader.reload(NamedSPILoader.java:68)
»  	at org.apache.lucene.codecs.Codec.reloadCodecs(Codec.java:136)
»  	at org.opensearch.plugins.PluginsService.reloadLuceneSPI(PluginsService.java:753)
»  	at org.opensearch.plugins.PluginsService.loadBundle(PluginsService.java:706)
»  	at org.opensearch.plugins.PluginsService.loadBundles(PluginsService.java:533)
»  	at org.opensearch.plugins.PluginsService.<init>(PluginsService.java:195)
»  	at org.opensearch.node.Node.<init>(Node.java:484)
»  	at org.opensearch.node.Node.<init>(Node.java:411)
»  	at org.opensearch.bootstrap.Bootstrap$5.<init>(Bootstrap.java:242)
»  	at org.opensearch.bootstrap.Bootstrap.setup(Bootstrap.java:242)
»  	at org.opensearch.bootstrap.Bootstrap.init(Bootstrap.java:404)
»  	at org.opensearch.bootstrap.OpenSearch.init(OpenSearch.java:180)
»  	at org.opensearch.bootstrap.OpenSearch.execute(OpenSearch.java:171)
»  	at org.opensearch.cli.EnvironmentAwareCommand.execute(EnvironmentAwareCommand.java:104)
»  	at org.opensearch.cli.Command.mainWithoutErrorHandling(Command.java:138)
»  	at org.opensearch.cli.Command.main(Command.java:101)
»  	at org.opensearch.bootstrap.OpenSearch.main(OpenSearch.java:137)
»  	at org.opensearch.bootstrap.OpenSearch.main(OpenSearch.java:103)
»  Caused by: java.lang.NoClassDefFoundError: org/apache/lucene/backward_codecs/lucene95/Lucene95Codec
»  	at org.opensearch.knn.index.codec.KNN910Codec.KNN910Codec.<clinit>(KNN910Codec.java:20)
»  	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
»  	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:77)
»  	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
»  	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:499)
»  	at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:480)
»  	at java.base/java.util.ServiceLoader$ProviderImpl$2.run(ServiceLoader.java:797)
»  	at java.base/java.security.AccessController.doPrivileged(AccessController.java:712)
»  	at java.base/java.util.ServiceLoader$ProviderImpl.newInstance(ServiceLoader.java:802)
»  	... 20 more
»  Caused by: java.lang.ClassNotFoundException: org.apache.lucene.backward_codecs.lucene95.Lucene95Codec
»  	at java.base/java.net.URLClassLoader.findClass(URLClassLoader.java:445)
»  	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:592)
»  	at java.base/java.net.FactoryURLClassLoader.loadClass(URLClassLoader.java:872)
»  	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:525)
»  	... 29 more

can we confirm with core team on this?

I think we need to add into this PR capability of providing segmentInfo format. Tests are not critical and can be fixed later, but adding a format should be part of the codec version bump.

I don't think so we need that. The older codecs are just used for reading the segment files and not for doing any kind of writes(we can see that BWC are successful here). What the issue happening in unit tests is we are using unit test to create a proper segment with a codec that doesn't support writing to segment files.
I think we should just remove these unit tests as they are not required and keep the unit test for the latest codec.

I think we need to add into this PR capability of providing segmentInfo format. Tests are not critical and can be fixed later, but adding a format should be part of the codec version bump.

I don't think so we need that. The older codecs are just used for reading the segment files and not for doing any kind of writes on the segment info file(we can see that BWC are successful here). What the issue happening in unit tests is we are using unit test to create a proper segment with a codec that doesn't support writing to segment files.
I think we should just remove these unit tests as they are not required and keep the unit test for the latest codec.
Nevertheless I synced with @naveentatikonda and asked him to verify this understanding again by doing some kind of manual tests or adding a BWC tests where after upgrade we delete few documents and then do a query.

Agree for tests for older codecs, we can keep write tests only for latest codec version. I still think we need to add a segment info format to the latest knn codec that is based on lucene 99 codec, similarly as we did for docValues and compound formats in KNNFormatFacade

Why? Is something failing? 🤔

I think we need to add into this PR capability of providing segmentInfo format. Tests are not critical and can be fixed later, but adding a format should be part of the codec version bump.

I don't think so we need that. The older codecs are just used for reading the segment files and not for doing any kind of writes on the segment info file(we can see that BWC are successful here). What the issue happening in unit tests is we are using unit test to create a proper segment with a codec that doesn't support writing to segment files.
I think we should just remove these unit tests as they are not required and keep the unit test for the latest codec.
Nevertheless I synced with @naveentatikonda and asked him to verify this understanding again by doing some kind of manual tests or adding a BWC tests where after upgrade we delete few documents and then do a query.

Agree for tests for older codecs, we can keep write tests only for latest codec version. I still think we need to add a segment info format to the latest knn codec that is based on lucene 99 codec, similarly as we did for docValues and compound formats in KNNFormatFacade

Why? Is something failing? 🤔

I think we need that extra layer of abstraction to allow usage of delegate codec in classes that may extend for our knn codec class. It was the idea why the format facade class is needed at the first place, maybe things changed now. Otherwise it will use format from default delegate as per parent class implementation in lucene, that's why it's not failing for us.

Here to just try understanding the changes added in the PR

knn codec class

Do we have such classes? I have not seen such classes.
@martin-gaievski

Agree for tests for older codecs, we can keep write tests only for latest codec version. I still think we need to add a segment info format to the latest knn codec that is based on lucene 99 codec, similarly as we did for docValues and compound formats in KNNFormatFacade

For docValues and compound format, we have our custom logic which require us to override get methods for those. For segment info, I don't understand why we need same pattern. Or what does add a segment info format to the latest knn codec mean?

@martin-gaievski if a any other plugin is using the KNNCodec then they should also start depending on the latest codec of K-NN. Just like k-NN relies on the latest codec from lucene and we upgrade every time there is an upgrade.
One this which we are not doing is moving the codec to a package which has backward.codec in path which is done by lucene. I think we can take that as a next step.

knn codec class

Do we have such classes? I have not seen such classes.
@martin-gaievski

It was an assumption at the time that facade was added, that users can extend our knn codec or use their own delegate. I haven't seen such classes either.
I think we use the segmentinfo format from default delegate and add it to facade later if needed.

@martin-gaievski if a any other plugin is using the KNNCodec then they should also start depending on the latest codec of K-NN. Just like k-NN relies on the latest codec from lucene and we upgrade every time there is an upgrade.
One this which we are not doing is moving the codec to a package which has backward.codec in path which is done by lucene. I think we can take that as a next step.

While this statement makes sense, especially regarding moving old codecs to a separate package, my point was different. I was referring to an option of passing custom delegate using constructor with args. It's not necessarily lucene codec implementation. Currently if user uses this constructor and passes their own codec we will take doc value and compound formats from that delegate.


Electro Gloves in syndicate uplink

About the PR
https://user-images.githubusercontent.com/63245550/208825706-790661dd-cc0c-4bb1-9e23-cf92ca0ed54b.mp4

Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

add: Added electrogloves in syndicate uplink, with a slot for powercell, cost 6 TC and work as a stun baton.

who hurt you

Imho the gloves should be more expensive, disguised stuns and all, 8 TC is probably the go-to.
Ninja gloves could also have the stun capability, as SS13 Ninjas (at least on /tg/) could do zappy things with their gloves.

@metalgearsloth you meant file organization like this?

im ngl i think this is a horrible idea

im not super convinced this will be good for the game, is there more justification for it?


Fuzzer action: redundant control flow wrapping

This is a catch-all issue for any fuzzer actions that take a statement S and wrap it in a control flow that is guaranteed to execute once and only once.
This could be:

an if statement with an always-true guard;
a do... while loop with an always-false guard;
a while or for loop engineered to break after one iteration (eg, while true or for (;;) with a break, though this'll need care to avoid optimisation

Done (in if and do…while flavours).


arm64 docker - unexpected error

When the container boots up I'm able to step through all the welcome screens and even authenticate to UI.com but on the final step I receive:
UNVR-4 Setup Error
An unexpected error occured during setup.

errors.log is filled with:
2021-04-13T17:18:07.574Z - error: UUM does not appear to be running: request to http://<IP_ADDRESS>:9080/api/ucore/manifest failed, reason: connect ECONNREFUSED <IP_ADDRESS>:9080
system.log shows:
2021-04-13T16:52:50.294Z - error:  Error: ENOENT: no such file or directory, open '/data/unifi-core/5fcf3809-f016-4b18-865b-550012ca4c2d'
may be related to postgresql-cluster:
systemctl status shows:
<EMAIL_ADDRESS>- PostgreSQL initial setup service
   Loaded: loaded<EMAIL_ADDRESS>disabled; vendor preset: enabled)
  Drop-In<EMAIL_ADDRESS>           └─cluster-migrate.conf
   Active: active (exited) since Tue 2021-04-13 17:15:05 UTC; 6min ago
  Process: 75 ExecStartPost=/usr/bin/unifi-protect-db-cluster-cleanup (code=exited, status=0/SUCCESS)
 Main PID: 56 (code=exited, status=0/SUCCESS)

Apr 13 17:14:15 docker-desktop systemd[1]: Starting PostgreSQL initial setup service...
Apr 13 17:15:05 docker-desktop unifi-protect-db-cluster-cleanup[75]: Error: Cluster 9.6 protect does not exist
Apr 13 17:15:05 docker-desktop systemd[1]: Started PostgreSQL initial setup service.

The no such file or directory error is always there I guess.
The postgress error is the real problem. It's looking for a cluster which does not seem to exist. Which version of the Docker container are you using?
Maybe clear out all the storage directories and restart the container.


Why do I need curly braced in my stylus?

Hello,
I'm working on a new project, and I'm trying gulp instead of grunt. I have a stylus file:
.logo
  height 60px

And a gulpfile:
gulp.task('styles', function() {
  return gulp.src(paths.stylus + '/*.styl')
    .pipe(stylus())
    .pipe(gulp.dest(paths.css));
});

Now, my css file is empty, but when I change my stylus file to:
.logo {
  height 60px
}

Then I get a css file with
.logo {
  height: 60px;
}

Which is strange, because you shouldn't need curly braces in stylus. Is this a bug in the plugin? Am I using it wrong?
Thanks!

This is a pass through plugin to the stylus project, you do not need braces for stylus.

@stevelacy so you're suggesting that the stylus package is causing this odd behavior, not gulp-stylus. Could this have to do with a versioning issue, i.e. which version of stylus gulp-stylus points to? If not, where should I look for where this is coming from?

It seems like the last time the tests were run via Travis was last year (9 months ago). I'm not sure how to run them myself, but it might be worth just running them again for a sanity check.

not needed - stylus has not updated in over a year https://www.npmjs.com/package/stylus

@stevelacy I think I've solved this. I was using 2-space indents. I switched to 4-space indents and now it works. I'm confused about it but oh well..


Continue Searching the same term

I would like a feature allowing the user to continue searching for the same term they were searching for previously. Similar to the way pressing "n" works in vim's normal mode.

For me on Windows, Ctrl+F3 does this when focus is in the file that I'm using the Ctrl+F find-widget on.

We closed this issue because it is a question about using VS Code rather than an issue or feature request. Please search for help on StackOverflow, where the community has already answered thousands of similar questions. You may find their guide on asking a new question helpful if your question has not already been asked. See also our issue reporting guidelines.
Happy Coding!


        # PYSIDE-1398: sys._getframe(1) may not exist when embedding.
        calling_frame = _cf = sys._getframe().f_back
        importing_module = _cf.f_globals.get("__name__", "__main__") if _cf else "__main__"
        existing = pyside_feature_dict.get(importing_module, 0)
        if name == "__feature__" and args[2]:
            __init__()
            # This is an `import from` statement that corresponds to `IMPORT_NAME`.
            # The following `IMPORT_FROM` will handle errors. (Confusing, ofc.)
            flag = 0
            for feature in args[2]:
                if feature in _really_all_feature_names:
                    flag |= globals()[feature]
                else:
                    raise SyntaxError("PySide feature {} is not defined".format(feature))
            flag |= existing & 255 if isinstance(existing, int) and existing >= 0 else 0
            pyside_feature_dict[importing_module] = flag
            if importing_module == "__main__":
                # We need to add all modules here which should see __feature__.
                pyside_feature_dict["rlcompleter"] = flag
            # Initialize feature (multiple times allowed) and clear cache.
            sys.modules["PySide2.QtCore"].__init_feature__()
            return sys.modules["__feature__"]
        if name.split(".")[0] == "PySide2":
            # This is a module that imports PySide2.
            flag = existing if isinstance(existing, int) else 0
        else:
            # This is some other module. Ignore it in switching.
            flag = -1
        pyside_feature_dict[importing_module] = flag
>       return original_import(name, *args, **kwargs)
E       ModuleNotFoundError: No module named 'appnope'

11 hours ago there was yanked release of appnope 0.1.1
5 hour ago there was release 0.1.2
https://pypi.org/project/appnope/#history
So this reason of faliture is rather solved


Add gw3583 to reviewers list.

This change is

Through an unfortunate sequence of events, I'm currently unable to log in to my normal @glennw account (for at least the next two weeks). So, I'm adding a new account to use in the interim.
r? @edunham

@bors-servo r+

:pushpin: Commit f8e6165 has been approved by edunham

:hourglass: Testing commit f8e6165d79b716ca8f020e663326cff03d63d5d7 with merge 4b845dbb8694032958a46107b5d79fc3831ff224...

I'm verbally confirming with the real-world instance of Glenn that this change is correct before Salting it out :)


how to add Css and template dynamically from string with vue.comonent

Version
2.6.10
Reproduction link
https://jsfiddle.net/nbkhqvf9/
Steps to reproduce
let cssCode =My css which is coming from backend api as a string
var MyComponent = Vue.extend({
template:
My HTML CODE
,
      })

      Vue.component('my-component', MyComponent);

What is expected?
My style should get appended to mycomponent
What is actually happening?
style doesn't get applied

How do i add


New pyth mocks

Grabbed from here:
https://github.com/pyth-network/pyth-crosschain/blob/main/target_chains/ethereum/sdk/solidity/AbstractPyth.sol

This probably should've been a draft, some issues to fix still as tests expect the old mock. The real contracts should be backwards compatible so I was hoping (gambling) the mock would just work.

Hmm yea I think we should be using the pyth mock from cannon instead. So I'm leaning towards closing this. But it would be good if perps and spots starts using the new method

Note this branch was merged into https://github.com/Synthetixio/synthetix-v3/tree/feat/pyth-erc7412-wrapper


2673 upgrade js libraries (redux)

Still dependent on npm v12
For mercycorps/TolaActivity#2673. See that issue for a play-by-play. Reinstates work in PRs #621, #622, #623, including replacing uglify with terser for production environments
Removes all critical vulnerabilities and will resolve all js-related Dependabot PRs
I have verified dev and prod builds on my local, but not on a server

This breaks on dev2. The villain seems to be that render_bundle doesn’t correct append STATIC_URL.
This may be because webpack configs are incorrectly specifying, or failing to specify, publicPath ??
See https://github.com/django-webpack/django-webpack-loader/issues/160

This is obviated by #634


Bump timeout window to 150ms

Tests show that average coldstart
report building & HTTP communications
take 83ms. Providing some room for outliers,
suggesting 150ms for the default timeout.
Signed-off-by: Erica Windisch<EMAIL_ADDRESS>

Marking WIP until tested and performance improving patches land.

Tested and working. I have concerns that invocations are running a bit closer to the max duration than I expected but it's still plenty inside the 150ms window we're setting.

Nice! #149 has this included, cool if we add it there?

I don't mind.


Allow javac Argument Files passing to compilerArgs in JavaCompile task

Expected Behavior
compilerArgs.add("@javacArgs")
or
compilerArgs.add("@\"C:/path/to/file/javacArgs\"")
or
<EMAIL_ADDRESS>or any other suitable variant (with quotes or not, etc.) which DOES work with direct java invocation
works in gradle
Current Behavior
execution failed for task ':compileJava'.
> invalid flag: @"C:\path\to\file\javacArgs"

Your Environment
WITH or WITHOUT IDE does not matter. Still does not work

Gradle 4.6
Build time:   2018-02-28 13:36:36 UTC
Revision:     8fa6ce7945b640e6168488e4417f9bb96e4ab46c
Groovy:       2.4.12
Ant:          Apache Ant(TM) version 1.9.9 compiled on February 2 2017
JVM:          9.0.4 (Oracle Corporation 9.0.4+11) OR 1.8 (Oracle Corporation 1.8+162)
OS:           Windows 10 10.0 amd64
IntelliJ IDEA 2018.1 EAP (Ultimate Edition)
Build #IU-181.4096.5, built on March 6, 2018
IntelliJ IDEA EAP User
Expiration date: April 5, 2018
JRE: 1.8.0_152-release-1136-b16 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Windows 10 10.0

What would be the benefit of this feature? Why not use the DSL to define arguments?

I use java8Args separately from java9Args for example.
I understand that could be done in many ways which are workarounds.
This is a matter of choice and preferences. Maybe some one, like me, prefer to separate "concerns" in different configuration files as much as possible e.g. javac args in @javacArg file
Arg files are very native Java feature. I consider so much native and quite simple feature could be supported by any jvm tool like build tool.
So, the answer is - separation and maintenance of different environments (instead adding more code in build logic, like if (..isJava9Compatible) else if (..isJava8Compatible))

Such files would be more compact and easier to write and maintain.  compilerArgs.addAll(['putting', 'some'] ) compilerArgs << "delimiters" << "between" << "arguments" make s them tedious to write and change.
Moreover, it currently appears that the JavaCompile task does sanity checking on arguments before passing them to javac - I am for example trying to change the garbage collector used by javac using the option '-XX:+UseConcMarkSweepGC', which results in an error diagnostic that it is an invalid flag.
This mechanism could be used as a way to say 'just trust me' when using nonstandard options.

We can't "just trust" such a file, we'd read it and handle the arguments inside it just as if you had added them through the DSL. Otherwise we would potentially miss important flags that influence the tasks's output and thus break up-to-date checking and caching. The -XX flag should not be a problem. If that doesn't work, please open a separate issue.

@oehme
Kotlin 1.2.50 is also adding support for arg files
https://youtrack.jetbrains.com/issue/KT-24472

1.3 is out and here is a link to issue tracker to track progress
https://youtrack.jetbrains.com/issue/KT-24613
This will really clean up the code since now it is like
        freeCompilerArgs += arrayOf(
            "-Xuse-experimental=kotlin.ExperimentalUnsignedTypes",
            "-XXLanguage:+InlineClasses",
            "-Xjsr305=strict",
            "-java-parameters",
            "-Xenable-jvm-default"
        )

and
tasks {
        withType<JavaCompile> {
            options.apply {
                encoding = UTF_8.name()
                isIncremental = true
                // compiler deamon JVM options
//                isFork = true // FIXME: Reported a bug https://github.com/rzwitserloot/lombok/issues/1603
//                forkOptions.jvmArgs = project.extra["org.gradle.jvmargs"].toString().split(" ")
                // javac tasks args file, but does not work
                // tried replace special characters - no luck
                // turn out Gradle does not support this, see https://github.com/gradle/gradle/issues/4679
//                compilerArgs.add("@" + project.rootDir.absolutePath.replace(Regex("(?<=[(,])([^(),]* [^(),]*)(?=[),])"), " \"\$1\"").replace('\\', '/') + "/javac" + current().majorVersion + "Args")
                // Could be like this, but this is not lesser ugly
//                file(rootDir.path + "/javac" + current().majorVersion + "Args").useLines { it.forEach { compilerArgs.add(it) } }
                compilerArgs.addAll(arrayOf(
                    "-Xlint",
                    "-Xlint:-processing",
                    "-Xdiags:verbose",
                    "-Adebug=false",
                    "-AaddGenerationDate=true",
                    "-parameters",
                    "-Amapstruct.defaultComponentModel=$componentModel"
                ))
            }
        }
    }


Update ndm to 1.1.0

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Hi the v1.1.0 has been deleted from the tags and releases.
Error log when trying to install:
==> Downloading https://github.com/720kb/ndm/releases/download/v1.1.0/ndm-1.1.0.

curl: (22) The requested URL returned error: 404 Not Found
Error: Download failed on Cask 'ndm' with message: Download failed: https://github.com/720kb/ndm/releases/download/v1.1.0/ndm-1.1.0.dmg
The incomplete download is cached at /Users/45kb/Library/Caches/Homebrew/Cask/ndm--1.1.0.dmg.incomplete
Error: nothing to install

@45kb it looks like v1.1.0 again on the place, it will remain so?

@artdevjs i think not because we are testing things, i'll tell you as it's fixed ok? :)

@45kb sure

is it automatic PR on this branch? Actually, i don't understand if you guys open them or everything is automatic reading github release. @artdevjs

No, I open PR's manually when new version is available @45kb

@artdevjs oh cool thanks a lot, however ... ok i'll let you know, thanks  👍

@artdevjs the new v1.1.0 is ok now thank you a lot 👍  👍  this can be installed now https://github.com/720kb/ndm/releases/tag/v1.1.0

https://github.com/caskroom/homebrew-cask/pull/30267


Cannot find 'SitesModule' in 'app/sites/sites.module'

When I try to access the "Manage Sites" menu the error below is reported on the console.
main.281198f362daffc2bea8.js:1 ERROR Error: Uncaught (in promise): Error: Cannot find 'SitesModule' in 'app/sites/sites.module'
Error: Cannot find 'SitesModule' in 'app/sites/sites.module'
    at tt (main.281198f362daffc2bea8.js:1)
    at main.281198f362daffc2bea8.js:1
    at t.invoke (polyfills.6fe99e701656a00f4585.js:1)
    at Object.onInvoke (main.281198f362daffc2bea8.js:1)
    at t.invoke (polyfills.6fe99e701656a00f4585.js:1)
    at e.run (polyfills.6fe99e701656a00f4585.js:1)
    at polyfills.6fe99e701656a00f4585.js:1
    at t.invokeTask (polyfills.6fe99e701656a00f4585.js:1)
    at Object.onInvokeTask (main.281198f362daffc2bea8.js:1)
    at t.invokeTask (polyfills.6fe99e701656a00f4585.js:1)
    at O (polyfills.6fe99e701656a00f4585.js:1)
    at O (polyfills.6fe99e701656a00f4585.js:1)
    at polyfills.6fe99e701656a00f4585.js:1
    at t.invokeTask (polyfills.6fe99e701656a00f4585.js:1)
    at Object.onInvokeTask (main.281198f362daffc2bea8.js:1)
    at t.invokeTask (polyfills.6fe99e701656a00f4585.js:1)
    at e.runTask (polyfills.6fe99e701656a00f4585.js:1)
    at v (polyfills.6fe99e701656a00f4585.js:1)
    at e.invokeTask [as invoke] (polyfills.6fe99e701656a00f4585.js:1)
    at y (polyfills.6fe99e701656a00f4585.js:1)

Thanks for reporting this.
The front-end UI for Respond is built in Angular, and stored in a different Github repo: https://github.com/madoublet/respond-ui
Whenever changes are made in that repo, the Angular project is rebuilt and the compiled .js files are integrated into this main repo.  My initial guess is that not all of the compiled .js files were updated the last time around.
I've done the last few updates and sort of had to muddle my way through the process manually as I wasn't clear on the steps used by the original author, @madoublet .  I wonder if it's possible to set up some sort of script that will automate pushing changes from the respond-ui repo into the main respond repo?
Maybe I'm getting ahead of myself.  I am not even sure that is the problem here.  I will try to look at this soon.

I will pull your changes and do a clean build and see if this goes away.  The build process is not great at this point.  I will see if I can write some scripts to improve it.

@madoublet an @axis80 when compiling the angled project and adding manually into the main project the problem was solved for me.

Would it make sense to include respond-ui as a dependency in packages.json?  Whenever there is a change in the UI, you could update your installation with git pull && npm install && gulp . There would be some tweaks needed to the gulpfile (and possibly angular.json as well) but I think it should work?

I think this would make sense.  I know I had it this way at one point b/c respond-ui is in NPM.  https://www.npmjs.com/package/respond-ui

I rebuilt respond-ui and updated the zip.  This should fix the issue.  I am not seeing it in my development build.

@madoublet thank you! Could you update the available angular application in the public directory? When doing my installation I do not extract the .zip file, as I saw that there was already an application in the public directory. I think the @axis80 proposal is valid, since respond-ui is a separate repository of respond.

I just double checked the timestamps and it does look like it is updated.  Are you still seeing that error? I will work on the @axis80 proposal.


I want “git checkout” command supports git submodule.

git checkout --recurse-submodules will be better when a project has git submodules, but vscode's git checkout to command doesn't have --recurse-submodules option, please support it.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

Good idea

非常棒

ok,very nice!

ok,very nice!

好需求，太需要了

Too much

support！！！

hold on

Danke!

support！！！

support！！！

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

Hi, so many users support this feature in  a short time, I think it can be moved to backlog.

Click this button to upvote.
The bot will move it to backlog after the number reaches 20.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

@lszomoru @VSCodeTriageBot Hi, 20 votes has been reached, what's the next plan? When can I see the implementation of the feature?

@DFVSQY, the issue will be moved to the backlog and it will be considered during the monthly planning conversations that happen at the beginning of each milestone. It is important to state that there is no commitment to when issues that are on our backlog would be prioritized so, please subscribe to this issue to be notified when progress is being made.
Until this feature is implemented you can use the following git config:
# Enable recursion for relevant commands, such that
# regular commands recurse into submodules by default
git config --global submodule.recurse true

@DFVSQY, the issue will be moved to the backlog and it will be considered during the monthly planning conversations that happen at the beginning of each milestone. It is important to state that there is no commitment to when issues that are on our backlog would be prioritized so, please subscribe to this issue to be notified when progress is being made.
Until this feature is implemented you can use the following git config:
# Enable recursion for relevant commands, such that
# regular commands recurse into submodules by default
git config --global submodule.recurse true

Thanks for the suggestion, I know this config, but I don't want to use global setting.  Hope see the feature soon!

:slightly_smiling_face: This feature request received a sufficient number of community upvotes and we moved it to our backlog. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

Hello
On Wed, Sep 14, 2022 at 18:25 qing0011 @.***> wrote:

非常棒
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/160897#issuecomment-1247462751,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT5KQVDWRJOHA3HDSPXNQY3V6J3G7ANCNFSM6AAAAAAQMMAWVU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Tina Silvia
My America inc


feat!: reimplement lua api 🎉

Closes #78

This time it's in a good shape.

API migration
require('Comment.api').{toggle => toggleln_linewise}()
require('Comment.api').{comment => commentln_linewise}()
require('Comment.api').{uncomment => uncommentln_linewise}()
require('Comment.api').{gcc => toggleln_linewise}()
require('Comment.api').{gbc => toggleln_blockwise}()
require('Comment.api').{gc => toggle_linewise_op}()
require('Comment.api').{gb => toggle_blockwise_op}()
require('Comment.api').{gco => insert_linewise_below}()
require('Comment.api').{gcO => insert_linewise_above}()
require('Comment.api').{gcA => insert_linewise_eol}()

Looks good, especially since it seems to remove duplicate API?
Only suggestion would be to drop the ln from the first five -- that looks redundant.

especially since it seems to remove duplicate API?

Quite the opposite ;)

Only suggestion would be to drop the ln from the first five -- that looks redundant.

I have three variant such as
toggleln_linewise() -- current line but no dot-repeat
toggleln_linewise_op() -- current line + dot-repeat
toggle_linewise_op() -- multiple line + dot-repeat

If I remove ln then
toggle_linewise() -- current line but no dot-repeat
toggle_linewise_op() -- multiple line + dot-repeat

Oh, I see. I thought the list above was complete.
May I then suggest toggle_single instead of toggleln? I think that's clearer.

May I then suggest toggle_single instead of toggleln? I think that's clearer.

Ahh, yes. I can change that. Thanks!
(those names are getting longer haha)

@clason I've changed ln to current. I think it's more clear than single ;)


improve style following MorrisJobke's request

@danimo sorry, one more...

wait a sec

Ok... Good to go.

:+1:

active on staging and live.

@danimo you rock!


Update ODK

@matentzn CL seems to already be on v1.2.30, but there are still a few changes - either up updated the ODK here

in src/ontology/uberon-odk.yaml, you need to set a new parameter:
git_main_branch: master

to make sure the docs are correct,
then regenerate.

refresh imports works well - will merge once CI checks pass


Add Nav2 configuration package

This PR is to enable the Nav2 stack on the C300 mobile base. See the new README for starting the robot with navigation.
It's still a WIP but getting close. Right now most of the nav stack starts up but I'm not getting a map to load so the tf frame from /map to /odom is failing.

Getting closer!!

I'm down to these two errors when mapping:


Weather Module

Sorry again about new pr, same reason - my last fork was a mess. No DS_store this time and fixed array error.

@swapagarwal Just been added to the testing list and the bot is better than I first expected. Super cool work man, and I'm so glad you've made it available to people like me!
I've just ran some local tests with my last commit and seems to be working good:)

Glad you like it!

Merged in 7fd41a12fd821bde70453b385061836785206ab2


Add link to Google Machine Learning Crash Course to playground index.html

as "back link", given that I've learnt about this (cool!) site from a link on
https://developers.google.com/machine-learning/crash-course/reducing-loss/playground-exercise

Hello, I'm doing a year end clean up of my personal https://github.com/pulls and wanted to ask if you would still consider merging this very old Pull Request, or if we should forget about and close this? (This is a bulk message I'm posting to many issues to gauge what is still revelant.)


Improve the layout of imports clauses

Analogously to assumes, imports should be indented and have the imported theories specified further indented on lines of their own. However, most if not all imports clauses currently don’t have this layout. We shall fix that.

Apparently, the only imports close currently in master is already in the required shape.

Apparently, the only imports close currently in master is already in the required shape.

That’s actually not true. This remarks was made because of an oversight.


Download size

It will be better if the size of the video is printed on the terminal before starting the download and also while the video is being downloaded in the format - [downloaded size] / [total download size]
With the simulate -s also, download size must be shown.

I could look into that, but I you want to have choice over the resolution (and the download size indirectly) you can take a look at the aria2c branch

Since all we do is fetch segments there's no way to know the total file size upfront. We could do some wild approximation based on video length but i'd rather not. Actially we kinda already do that for the progress bar with varying degreea of succes :)

@snobu we could do a very good estimate of the size with in really convoluted way: grab the playlist => grab 1 item for video and 1 for audio and find out they sizes => multiply that size by the number of segments => PROFIT?
But i't nothing important and it's a bit elaborate just for the filesize

Aren't all videos on MSStream encoded with variable bit rate? I'm just assuming since that's the norm and that would make the final size a gamble.


[1.10.x] [KOGITO-5639] NullPointerException when marshalling a process instance state inside an embedded process

Jira: composite nodes instance should no have instantiate the variable scope instances as some of them does not need to.
Jira: https://issues.redhat.com/browse/KOGITO-5639
backport of https://github.com/kiegroup/kogito-runtimes/pull/1498

The (build) Runtimes check has failed. Please check the logs.

The (build) Examples check has failed. Please check the logs.

jenkins rerun examples tests

jenkins rerun runtimes tests

The (build) Runtimes check has failed. Please check the logs.

The (build) Examples check has failed. Please check the logs.

The (build) Examples check has failed. Please check the logs.

Jenkins rerun examples tests


cloudctl update do is broken

cloudctl update do
Error: Get "http://localhost:8080/cloud/v1/version": dial tcp <IP_ADDRESS>:8080: connect: connection refused
That's not usefull :-P
Manually installing the latest version works of course, though that shows
cloudctl update check
checksum:56e809d92c962eaeb8f2267fd2ab956b
latest version:v0.12.17 from:2024-03-06T13:55:49Z
local  version:v0.12.18 from:2024-04-23T08:52:15Z
cloudctl is up to date
which is a little irritiating :-P

You updated to a newer version than strictly required, which should be no problem.
The error you get is irritating ? do you go through a proxy ?

Not sure if my Ticket was quite clear.
Point 1:
cloudctl update do
doesn't work, i get an error with it trying to connect to local host which makes no sense. curl etc. can download things without issue and it worked with older versions fine.
Point 2:
update check reports the latest version higher that mine. it doesn't say required version is lower :-D

Please run:
DEBUG=1 cloudctl update do

and remove potential authorization headers from your output. Still confused why your cloudctl will connect to localhost.
what does your ~/.cloudctl/config.yaml look like ?

Ah. My user doesn't have permissions to write in /usr/local/bin/ so i was running it with sudo. Apparently it doesn't use my ~/.cloudctl/config.yaml then any more and failes with that strange error.
Worked at some point ;-)

OK, so please close this issue if you feel confident

thx


[Bug]

Summary
A clear and concise description of what the bug is, and what you expected to happen.
How to reproduce

Go to '...'
Click on '...'
Scroll down to '...'
See error

Please include screenshots.
Additional context
Add any other context about the problem here.

Estimation of soil physical and mecanical properties using satellite information, soil spectral behavior  and topography via pedotransfer functions and geostatistics

Estimation of soil physical and mecanical properties using satellite information, soil spectral behavior  and topography via pedotransfer functions and geostatistics

Estimation of soil physical and mecanical properties using satellite information, soil spectral behavior  and topography via pedotransfer functions and geostatistics

Estimation of soil physical and mecanical properties using satellite information, soil spectral behavior  and topography via pedotransfer functions and geostatistics

www.google.com


constant properties

Currently properties are supposed to have a read/write accessor and a notifier to signal a change. Additional QFace supports the readonly modifier to allow properties to be read and notified about changes but the user is not able to modify the property value.
The const modifier would declare the property needs to be initialised immediately and can not be changed over time. This could be useful for data which will normally not change, e.g. OS, device type, CPUs or in embedded systems the display size or other variant coding.
It is useful when during HMI development a change could not be recovered from. For example a HMI which was only be tested with one screen dimension would probably not work correctly when on runtime the screen dimension changes. Also the HMI is required to know the display size to ensure a proper startup.
interface Display {
  const int screenWidth;
  const int screenHeight;
}

A const modifier should not generate any signals for that property

I think a constant property will be only really valuable if we have default values as part of the IDL.

Came with commit eb4c99c7a60e9aac3d0df04ac0e7c4618a5b1155


Adding an example on  Compact Convolutional Transformers

Colab Notebook is available here.

Hi,
I do want to add this example is CCT-2/3x2, which is our smallest model and should be at around 0.28M params, not 0.4M which is noted in the notebook (by taking a quick look I noticed the MHSA from Keras accounts for a lot of the increased parameters, so maybe there's something there worth exploring?). There are some details in the training that help achieve the best results including but not limited to cosine LR schedule and more augmentation policies (AutoAug or Mixup and CutMix). However at this level the results are more or less the same with what we ran ourselves so it shouldn’t matter much.

Thanks for sharing your comments. I have included the notes on augmentation already inside the notebook. Will add some more notes on the model being CCT-2/3x2 as well as comments on parameters.
For the slight increase in parameters, I'd not worry too much since 0.4 M and 0.28 M are not that drastic.

Thank you.
Our model is about 30% smaller. That difference will only grow as larger models are used, which are needed for better performance. Our best model on CIFAR-10 is 3.17M parameters. So if this scales it would be over 10M (almost as big as a ResNet18). A key aspect of our model is it’s compactness, or how it achieves high performance with a low number of parameters.
And I can confirm that when I rewrote the MHSA myself, model.summary() returned 275K params.

Yeah got it. But the purpose of Keras Examples is also to showcase the use of how we can use the factory layers to put an idea to use quickly with Keras best practices incorporated. I can note all your points down inside the example to provide a more holistic view to the readers. So, reimplementing MHSA in this example is a bit out of scope. I hope you'd understand.

A key aspect of our model is it’s compactness, or how it achieves high performance with a low number of parameters.

IMO, the current example does convey this idea (I noted the performance and params with respect to a vanilla ViT) but I understand your point on making it even more compact.

That makes sense. We appreciate you integrating our work!

@fchollet addressed all the feedback you provided. Thank you.

@fchollet done. Ready for you to take over.

Thank you for another great contribution! 👍


VDC: REST API support for VMs

part of https://github.com/threefoldtech/home/issues/1091

status update:

the rest API for the virtual machine is implemented (list, add, delete) and added to the swagger open API with examples.
there is a bug in the deployment code, https://github.com/threefoldtech/js-sdk/issues/3239 needs to be addressed.
there are some parts of the code that need to be refactored.

status update:

the PR is ready for review.

https://github.com/threefoldtech/js-sdk/pull/3251


Uncaught TypeError: undefined is not an object! When running the project.

I get the following error when running the project.
I use MVC5 with Webpack and server rendering.
The application has a simple component. Does anybody has an idea why this error could occur.

export default class IndexServer extends React.Component {
    render() {
        return (

               Test

        )
    }
}

This is server.js file:
import React from 'react';
import ReactDOM from 'react-dom';
import ReactDOMServer from 'react-dom/server';
import "@babel/polyfill";

global.IndexServer = IndexServer;
global.React = React;
global.ReactDOM = ReactDOM;
global.ReactDOMServer = ReactDOMServer;

This is the webpack config:
"use strict";

var path = require("path");
var WebpackNotifierPlugin = require("webpack-notifier");
var BrowserSyncPlugin = require("browser-sync-webpack-plugin");

module.exports = {

    entry: {
        main: ['babel-polyfill', './Content/components/server.js']
    },
    output: {
        filename: 'server.bundle.js'
    },
    resolve: {
        extensions: ['*', '.js', '.jsx']
    },
    module: {
        rules: [
            {
                test: /\.jsx$/,
                exclude: /node_modules/,
                use: {
                    loader: "babel-loader"
                }
            }
        ]
    },
    devtool: "inline-source-map",
    plugins: [new WebpackNotifierPlugin(), new BrowserSyncPlugin()]
};

@simonssspirit Try installing a version 2.4.2 of JavaScriptEngineSwitcher.V8.Native.win-x86 and JavaScriptEngineSwitcher.V8.Native.win-x64 packages:
Install-Package JavaScriptEngineSwitcher.V8.Native.win-x86 -Version 2.4.2
Install-Package JavaScriptEngineSwitcher.V8.Native.win-x64 -Version 2.4.2

But in the future I still recommend to update the ReactJS.NET to version 4.0.

Thank you, everyone, for the suggestions.
I was able to run the project an render it successfully using this:
            .SetLoadBabel(false)
            .AddScriptWithoutTransform("~/dist/server.bundle.js");

The issue is that now there is no scripts loaded, on the pre rendered HTML which leaves the page with uninteractive components.

Do you have a
    @Html.ReactInitJavaScript()
    @Html.React("IndexServer", new
{
    initialComments = "test"
})

@Html.ReactInitJavaScript() must be called after Html.React, does that work?

@dustinsoftware that was it, thank you so much for the assistance to all involved.


Fix README

The README suggests using a random string as a DSN, this will fail. Using an empty string will cause Sentry to just not load (or so it seems).
This PR updates the docs to match the test
https://github.com/wix/sentry-testkit/blob/b876bb80b832af8b1df6ff0cea42c028721b96cd/test/index.test.js#L9-L17
Also, the README uses testKit instead of testkit

why do you think that fake DSN fails Sentry?

thanks @dangreenisrael  🍺


Fix the typo issue for README.md that replaces interopability to interoperability.

Fix the typo issue for README.md that replaces interopability to interoperability.

Thanks Lam :)


Add get scheduled local notifications

(This is a remake of #6907, which I botched pretty good with a rebase.)
This returns an Array of Local Notifications that have been scheduled to be delivered.
Available attributes on return Notification object (if set in the local notification itself):
alertAction
alertBody
applicationIconBadgeNumber
category
fireDate
soundName
userInfo
More could be added to this but I just matched what was available in the Object passed to presentLocalNotification and scheduleLocalNotification.
Motivation
I needed to determine the number and other details about local notifications that were already scheduled. There is an API for this in iOS but one hadn't been built yet for React Native.
I created the Obj-C method and updated the documentation in PushNotificationIOS.js as well.
Usage:
PushNotificationIOS.getScheduledLocalNotifications( (notifications) => {
  console.log(notifications);
});

Sample Output:
[
  Object {
    alertBody: "It's about time you check in with Aaron, eh?",
    applicationIconBadgeNumber: 2,
    soundName: "UILocalNotificationDefaultSoundName",
    fireDate: "2016-04-10T14:43:45.755-06:00",
    userInfo: Object {
      id: 5
    }
  }
]

I think the name should be getScheduledLocalNotifications instead. Otherwise it'll be easy confuse it with scheduleLocalNotification as you said.

@satya164 It actually is getScheduledLocalNotifications. We made that change in the previous PR. I'll update the PR text to reflect that. Thanks.

I'm curious where RN is going with promises vs callbacks. Maybe this isn't the best place for this and if it is we can end it here, but I just implemented a promise based response to requesting permissions in PushNotificationIOS, couldn't this also be written to return a promise so that we can get the benefits of async/await?:
const notifications = await PushNotificationIOS.getScheduledLocalNotifications();

@JAStanton We're moving to promises. If you're writing a new API, you can just support the promise based implementation and not accept a callback at all. For existing callback APIs we will eventually move to promises and deprecate the callback based API, then remove it completely after few releases. Would be awesome if you could send a PR for that too :D

@joshuapinter updated the pull request.

Just a couple of nits, but otherwise looks good.

Can you also change the API to be promise based please :)

@javache Edits all updated.
Good call on the Static method and the unnecessary RCTNullIfNil around @[].
I removed all the white spaces and aligned ='s as well.
@satya164 Can we merge this in and I can I do the promise API at a later date?

@joshuapinter updated the pull request.

@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.

@joshuapinter Well, the problem with that is then we have to maintain both callback and promise based APIs before we deprecate the callback API.


Issue 483 input currents

Description
Added some new current profiles. In particular, data can be loaded from a csv file (see get_current_data.py). Interpolating at the top of the function worked (thanks @tinosulzer !) but I thought it would be easier for the user to just pass the filename to an object when creating a model, rather than explicitly changing get_current_data.py. This now means that some current functions are just functions, but that one is a class. I could change them all to being classes, or just put this one in a folder separate to standard_current_functions to make it clear it behaves differently. Thoughts @tinosulzer @Scottmar93  ? I think in general it is probably useful to be able to pass parameters to the user-defined input currents (e.g. see results/drive_cycles/user_sin_current_simulation.py, where you may want to pass some list of parameters to a current function and compare the results).
P.S. after you comments, I'll add more info so that it is very clear what data get_current_data.py expects (non-dimensional current amplitude vs time in seconds), and maybe a notebook about adding your own input current function.
Fixes #483
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Key checklist:

[ ] No style issues: $ flake8
[x] All tests pass: $ python run-tests.py --unit
[ ] The documentation builds: $ cd docs and then $ make clean; make html

You can run all three at once, using $ python run-tests.py --quick.
Further checks:

[x] Code is commented, particularly in hard-to-understand areas
[x] Tests added that prove fix is effective or that feature works
[x] Any dependent changes have been merged and published in downstream modules

Let me know when this is ready again for review :)

still need to merge master, fill out doc properly and (maybe) an example notebook about adding your own current function, but I've added the classes GetConstantCurrent which accepts one parameter (the value of the current, by default I_typ), GetUserCurrent which allows the user to enter their own function with however many parameters they want, and GetCurrentData which loads current data (dimensional or not) and optionally voltage data and creates an interpolant. See here for examples.

@tinosulzer @Scottmar93  I think this is ready for another look now


Hold persistent data in a webhook

I try to hold persistent data in a webhook. Is there a common solution, already ?
Actually I hold it in an array in the script header. But after every server restart the known states are gone.

Not yet, but @rodrigok is working on that, using MongoDB.

Ok, i will wait for it.
Thanks for your replay.

@rodrigok any updates?

Closing this due lack of updates


Updated docs with more functionality

Codecov Report

Merging #45 (957c177) into main (58aa3c5) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main      #45   +/-   ##
=======================================
  Coverage   95.79%   95.79%
=======================================
  Files           4        4
  Lines         357      357
  Branches       27       27
=======================================
  Hits          342      342
  Misses         10       10
  Partials        5        5

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


Add $attributes to Html methods input(), buttonInput(), submitInput() and resetInput()

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Fixed issues
#104

👍


Having trouble with vitest

I am having trouble making the loader work with vitest. Is there any documentation regarding setup of asset handling? I seem to be getting an import path rather than a render object.

It was my bad. Apparently there is a separate vite.config.js with a test config in each of the pacakges in my monorepo.


✅ Kill Agent RPC

Description:
will trigger the agent to restart usually depending on config and can be used to update agent

done in #227


It is not working, buggy

Even the code from this link shows bugs. http://www.typescriptlang.org/Playground

What is not working?

Simplest things did not work yesterday. Now it is working. I did not install or uninstall something else than this plugin.

Can we close this issue?


ESC to Close Panel

Closes #1347

Since AFAIK this doesn't work 100% maybe we should decline the PR? Or do you prefer to keep it open just as a reminder of looking this more in depth in the future?

This does work, but this is going the opposite direction of the way I prefer keys should be handled. I've pushed the commits just in case, but I don't mind closing this PR. If the other shortcut PR's are finished, the whole code written here would be deleted anyway.

Update:
This change is now also relevant for my other shortcut branch since https://github.com/wise0704/FlashDevelop/commit/5928a702d8de1b4ebbf61bd76e590a4d346ea86b.


database_pool_sz_ovfl option is ignored

Environment

OS / version:  openSUSE Tumblweed
Processor architecture: x86_64
TPM Manufacturer:  virtual
Keylime version: 7.3.0

Description
The option
database_pool_sz_ovfl

is silently ignored. Although its set I see following error
QueuePool limit of size 5 overflow 10 reached, connection timed out, timeout 30.00

with journctl -u keylime_verifier.
The option was set /etc/keylime/verifier.conf and for debugging purpose in /usr/etc/keylime/verifier.conf.

Hi @mslacken, which database have you used on this test? Can you provide the contents of database_url from your /etc/keylime/verifier.conf?

I just used the 'SQlite' backend as I did not want to configure that much. I don't have the paths right now, but can provide them later.

Ah, I see... sqlite backend does not have support for database_pool_sz_ovfl, it is explicitly ignored on this particular case.


How to enable social media login?

Hello,
I have found that you are using social lite but I am unable to find any easy way to enable social media login like google, facebook, etc.
Can you help me with the same.
Thanks

Heya, this is a bit out of scope for this project. If you need help to set up other social media logins please contact the library itself. Laravel.io only offers Github integration.


server side script

in this code have any server side script pls tell me

Please rephrase your question. It's not clear what "server side script" is referring to.
On the off chance you're asking about storing Javascript functions on the MongoDB server, that was recently answered in https://github.com/mongodb/mongo-php-library/issues/379.

Closing this due to inactivity and general lack of question marks.


chore: add scripts for ios-modelgen e2e test

Description of changes

Issue #, if available

Description of how you validated changes
Checklist

[x] PR description included
[x] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] Breaking changes to existing customers are released behind a feature flag or major version update
[ ] Changes are tested using sample applications for all relevant platforms (iOS/android/flutter/Javascript) that use the feature added/modified

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #629 (90fb62f) into main (ffd7cf7) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main     #629   +/-   ##
=======================================
  Coverage   84.44%   84.44%
=======================================
  Files         152      152
  Lines        6435     6435
  Branches     1383     1383
=======================================
  Hits         5434     5434
  Misses        901      901
  Partials      100      100

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

What is the empty file .codebuild/scripts/run-ios-modelgen-e2e-test.sh?

What is the empty file .codebuild/scripts/run-ios-modelgen-e2e-test.sh?

That's updating the permissions on that script to be executable.


Deal with curves and other shapes when they do not want to be overstroked multiple times

Fixed by https://github.com/pshihn/rough/pull/160


SQL parsing into an AST fails on INTERVAL and SAMPLE statements

What happened:
Clickhouse query which contains:

"INTERVAL 1 MINUTE" in where clause
or
"FROM table SAMPLE 1/10"
fails at SQL parsing into an AST when adhoc filter is applied

What you expected to happen:
Apply adhoc filter and perform query
How to reproduce it (as minimally and precisely as possible):
Apply any adhoc filter and run the query:
SELECT *
FROM data
SAMPLE 1/10
WHERE time > NOW() - INTERVAL 1 MINUTE

and watch javascript console log:
Unexpected int token: "1". Instead, I was expecting to see one of the following:

    - A "lparen" token
    - A "kw_cross" token
    - A "kw_left" token
    - A "kw_right" token
    - A "kw_full" token
    - A "comma" token
    - A "kw_inner" token
    - A "kw_where" token
    - A "kw_join" token
    - A "kw_group" token
    - A "kw_order" token
    - A "kw_offset" token
    - A "kw_for" token
    - A "kw_limit" token
    - A "kw_fetch" token
    - A "kw_union" token
    - A "semicolon" token

Environment:

Grafana version: 11.1.4 (2355de00c6)
Plugin version: 4.3.2
OS Grafana is installed on: Rocky Linux release 8.10 (Green Obsidian)
User OS & Browser: macOS 14.6.1 and Chrome 127.0.6533.120

The AST is notorious for having SQL syntax problems. I'll try to reproduce this locally and see if it can be patched.
@bossinc We might need to start exploring other options for the AST. I believe one of its main purposes is for extracting the table name (at least for adhoc filters). Maybe we can add a field to the SQL UI where people can manually write the table name. Or we can find a good Go module on the backend to handle it and use that to get an AST via HTTP request to the plugin.

Thank you.
For INTERVAL 1 MINUTE I can use workaround "toIntervalMinute(1)" already now.
But for SAMPLE I have no solution. I took a fast look on AST. If I'm correct it's AST plugin for Postgresql and it knows "TABLESAMPLE" with the very similar meaning. Maybe it could be just renamed.

The AST plugin is pretty flexible actually, maybe we can just add the keyword somewhere. Thanks for mentioning that

Any update about the issue?

No update yet, although I've been looking into alternatives for the AST parser since this is just one of many issues related to the current implementation


I so example  aria20 aria21

Hi Marc, please review this branch and merge it to master repository, <Thank you very much!>

Updates added via a separate branch.  Closing this PR due to merge conflicts.


Update base images used during docker build

Tests fail with an interesting error. I don't quite know if this is easy to fix. I will close this PR for now.


Wrong information autoClose props

autoClose
PRO+ accounts only. Time until the embedded typeform should automatically close after a respondent clicks the Submit button. Your typeform will automatically close after the time you specify, so respondents won’t have to manually close your typeform popup. In seconds.Popup mode option
N/A

Hello,
On the last version, README has been updated with a link to the official documentation regarding supported options (https://developer.typeform.com/embed/configuration/#available-options), so this info has been removed.
Cheers.