Nested fragments produce the wrong payload query

Originally here (only for Vendr folk): https://github.com/Blissfully/ironzion/pull/11352/commits/7a9dfff05db1559bc128225e256e86d60fbe4036
Tried to improve a query to generate better names by using fragments, like specified in LifeOfAQuery.md, which resulted in the following query:
query MyPortal_OnboardingPlans() {
  onboardingPlans {
    ... OnboardingPlan
  }
}

fragment OnboardingPlan on OnboardingPlan {
  icon
  title
  description
  tasks {
    ... OnboardingTask
  }
}

fragment OnboardingTask on OnboardingTask {
  icon
  image
  previewTitle
  fullTitle
  description
  primaryAction {
    __typename
    ... on OnboardingGoToRoute {
      label
      route
    }
  }
  secondaryAction {
    __typename
    ... on OnboardingAppCue {
      label
      id
    }
    ... on OnboardingExternalLink {
      label
      url
    }
  }
  completed
}

This query fails when run at runtime with the error:
ERROR: Unknown fragment "OnboardingTask".

GraphQL request:98:11
97 | description
98 | tasks {...OnboardingTask} }
   |           ^
99 | fragment OnboardingTask_batch_1 on OnboardingTask {icon

ERROR: Fragment "OnboardingTask_batch_1" is never used.

GraphQL request:99:1
 98 | tasks {...OnboardingTask} }
 99 | fragment OnboardingTask_batch_1 on OnboardingTask {icon
    | ^
100 | image

The generated code for the fragments in the query is:
toFragments_ : Int -> String
toFragments_ version_ =
    String.join
        """
"""
        [ ("fragment " ++ GraphQL.Engine.versionedName version_ "OnboardingPlan"
          )
            ++ """ on OnboardingPlan {icon
title
description
tasks {...OnboardingTask} }"""
        , ("fragment " ++ GraphQL.Engine.versionedName version_ "OnboardingTask"
          )
            ++ """ on OnboardingTask {icon
image
previewTitle
fullTitle
description
primaryAction{__typename
... on OnboardingGoToRoute {label
route}}
secondaryAction{__typename
... on OnboardingAppCue {label
id}
... on OnboardingExternalLink {label
url}}
completed }"""
        ]

Which uses the versioned fragment name when defining the OnboardingTask fragment, but it doesn't use it in the OnboardingPlan fragment itself, resulting in the above mentioned error.

This should be fixed with 0.7.0!


fix(token): correct 'WWW-Authenticate' header in response

This PR is to fix https://github.com/honojs/hono/issues/291
FYI: https://www.rfc-editor.org/rfc/rfc2617#page-8

Hi @metrue
Thank you for fixing it!

Thanks for making the change @metrue.
Looking at it though, I'd argue that the return code should be 401 Unauthorized not 400 Bad request, whenever there is missing or invalid authentication headers.
https://www.rfc-editor.org/rfc/rfc2617#section-3.2.1

3.2.1 The WWW-Authenticate Response Header
If a server receives a request for an access-protected object, and an
acceptable Authorization header is not sent, the server responds with
a "401 Unauthorized" status code, and a WWW-Authenticate header...

Also, and this is much less critical, I notice that the realm in the WWW-Authenticate header for the basic auth middleware can be specified. I suspect all auth middleware providers should do something similar.
The additional error info in your JWT change seems a good improvement.
See
https://github.com/honojs/hono/blob/master/src/middleware/basic-auth/index.ts#L44
https://github.com/honojs/hono/blob/master/src/middleware/basic-auth/index.ts#L70

@sdarnell

Looking at it though, I'd argue that the return code should be 401 Unauthorized not 400 Bad request, whenever there is missing or invalid authentication headers.

Yeah, it's a little hesitation between 400 and 401,  and I agree it totally a 401 here. PR: https://github.com/honojs/hono/pull/296

I notice that the realm in the WWW-Authenticate header for the basic auth middleware can be specified. I suspect all auth middleware providers should do something similar.
The additional error info in your JWT change seems a good improvement.
I think it's good idea, let me think about and can do the improvement.


Sync eng/common directory with azure-sdk-tools for PR 1109

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/1109
See eng/common workflow

/check-enforcer override


[GitLab] Fix type of SourceProjectID in MergeRequest struct

It should be an int64, but was a string.
https://docs.gitlab.com/ce/user/project/integrations/webhooks.html#comment-on-merge-request
The incorrect type caused the MergeRequest field in comment event payloads to be empty.

Thanks @tulir will merge today


Basic auth dosn't work if value for user,password from secret

Hello!
I find a very strange bug with basic authentification in scope.
I use:
docker version: 18.06.1-ce
scope version: try on 1.10.1, 1.10.2
kubectl version: v1.12.3

I try to get value for username and password from kubernetes secret, for app and agent, something like this:
env:
  - name: ENABLE_BASIC_AUTH
    value: 'true'
  - name: BASIC_AUTH_USERNAME
     valueFrom:
       secretKeyRef:
         name: weave-secret
         key: username
  - name: BASIC_AUTH_PASSWORD
     valueFrom:
       secretKeyRef:
         name: weave-secret
         key: password

When attach to the pod, I see env variables
$ kubectl -n weave exec -ti weave-scope-app-5b487f6fcd-m8j5k env
ENABLE_BASIC_AUTH=true
BASIC_AUTH_USERNAME=AdmIn
BASIC_AUTH_PASSWORD=12345678

For app and agent the env values are the same. But authentication in the browser, I don't pass.
The same effect, if I use command line argument in spec.containers.args for app and agent, like this
- '--app.basicAuth=true'
- '--app.basicAuth.username=$(WEAVE_SCOPE_USERNAME)'
- '--app.basicAuth.password=$(WEAVE_SCOPE_PASSWORD)'

And again, env variables in the pod exist, but authentification failed.
If I use value for username and password without kubernetes secret, all work is fine.
- '--app.basicAuth.username=AdmIn'
- '--app.basicAuth.password=12345678'

or if env:

env:
  - name: ENABLE_BASIC_AUTH
     value: 'true'
- name: BASIC_AUTH_USERNAME
  value: 'AdmIn'
- name: BASIC_AUTH_PASSWORD
  value: '12345678'

I'm not sure I understand your whole issue, but I can see one problem:
- '--app.basicAuth.username=$(WEAVE_SCOPE_USERNAME)'

environment variable expansion like $(XXX) is done by a shell, and in that context there is no shell, just the Scope process.

I don't think this is a problem, because settings below work perfectly.
In deployment.yaml for app:
args:
- '--app.basicAuth=true'
- '--app.basicAuth.username=$(WEAVE_SCOPE_USERNAME)'
- '--app.basicAuth.password=$(WEAVE_SCOPE_PASSWORD)'
env:
- name: WEAVE_SCOPE_USERNAME
  value: 'AdmIn'
- name: WEAVE_SCOPE_PASSWORD
  value: '12345678'

And in log app we see:
root@master1:# kubectl -n weave logs -f  weave-scope-app-6b7f8b7498-k85rx
<app> INFO: 2019/04/02 11:43:26.514466 app starting, version 1.10.2, ID 1cd568e14de96959
<app> INFO: 2019/04/02 11:43:26.514598 command line args: --app.basicAuth=true --app.basicAuth.password=12345678 --app.basicAuth.username=AdmIn --mode=app
<app> INFO: 2019/04/02 11:43:26.525909 Basic authentication enabled
<app> INFO: 2019/04/02 11:43:26.527100 listening on :4040
...

And in daemonset.yaml for agent:
args:
- '--probe.basicAuth=true'
- '--probe.basicAuth.username=$(WEAVE_SCOPE_USERNAME)'
- '--probe.basicAuth.password=$(WEAVE_SCOPE_PASSWORD)'
env:
env:
- name: WEAVE_SCOPE_USERNAME
  value: 'AdmIn'
- name: WEAVE_SCOPE_PASSWORD
  value: '12345678'

And log for agent:
root@master1:# kubectl -n weave logs -f  weave-scope-agent-szsgq
time="2019-04-02T11:49:31Z" level=info msg="publishing to: weave-scope-app:80"
<probe> INFO: 2019/04/02 11:49:31.133555 Basic authentication enabled
<probe> INFO: 2019/04/02 11:49:31.169102 command line args: --mode=probe --probe-only=true --probe.basicAuth=true --probe.basicAuth.password=12345678--probe.basicAuth.username=AdmIn --probe.docker=true --probe.docker.bridge=docker0 --probe.kubernetes=true --probe.no-controls=true weave-scope-app:80
<probe> INFO: 2019/04/02 11:49:31.169241 probe starting, version 1.10.2, ID fc2bd4ee1f5c863
...

Hi @qurname2
This works for me, please have a look.
Secret:
Name:         weave-secret
Namespace:    weave
Labels:       <none>
Annotations:
Type:         Opaque

Data
====
password:  12 bytes
username:  6 bytes

Weave-Scope-App deployment
    spec:
      containers:
      - args:
        - --no-probe
        - --app.basicAuth=true
        - --app.basicAuth.username=$(WEAVE_SCOPE_USERNAME)
        - --app.basicAuth.password=$(WEAVE_SCOPE_PASSWORD)
        env:
        - name: WEAVE_SCOPE_USERNAME
          valueFrom:
            secretKeyRef:
              key: username
              name: weave-secret
        - name: WEAVE_SCOPE_PASSWORD
          valueFrom:
            secretKeyRef:
              key: password
              name: weave-secret

Hi @qurname2,
Did the solution that @qiell posted above work for you or are you still experiencing problems?

@qurname2 I'll close this issue since we haven't heard from you in a while, but feel free to re-open if you're still experiencing problems and could give us more details :)


SAS output for quantiles

https://github.com/PSIAIMS/CAMIS/blob/5e19d3d0ac003e46dca043f203b3c844bb8ecca4/SAS/summary-stats.qmd#L8
Change title to be Calculating Quantiles (percentiles) in SAS.  to highlight they are the same thing
Add some text to say that the Q1= 25%,  Q2=50%, Q3 = 75%,
Update to have the output coming out using same dataset as in R page

@DrLynTaylor From what i can see the example in the r page is the same as the sas one? so not sure what needs updating?

Thanks @alanahjonas95    That's odd !! But I agree, example looks the same so I think we are done with this issue and it can be closed.  Thanks so much :)


Add key service check for public key fetch

This PR is to keep the public key fetch inline with the java codebase here: https://github.com/yahoo/athenz/blob/master/libs/java/auth_core/src/main/java/com/yahoo/athenz/auth/impl/PrincipalAuthority.java#L229-L240. This will allow project tokens to fetch the public key from zts from the sys.auth domain.

Thanks @mcieplak


Mount a crypted mounted fuse

Hi i have the Problem that i want to Mount a crypted fuse from another program.
But if i Mount the Folder is empty. i guess it trys to Mount the underlying Folder which is of couse empty.
I am using the latest rclone. Mounting the crypted files directly works great but i Need this Feature for another fuse System because rclone don't Support it.

This is probably question best for the forum, but try the --allow-other flag on the rclone mount - that should help.
Also make sure your fuse config has user_allow_other in it.
$ cat /etc/fuse.conf
# /etc/fuse.conf - Configuration file for Filesystem in Userspace (FUSE)

# Set the maximum number of FUSE mounts allowed to non-root users.
# The default is 1000.
#mount_max = 1000

# Allow non-root users to specify the allow_other or allow_root mount options.
user_allow_other


Bug in coordinated worker that causes processes to hang on signal errors

The offending code from coordinated_worker.go with my comments inline with RW:
	cancel := make(chan error)  // RW: Note this is unbuffered
	stopHeartbeating := make(chan struct{})

	go c.heartbeat(activityTask, stopHeartbeating, cancel)
	defer close(stopHeartbeating)

	ticks := time.NewTicker(c.tickMinInterval)
	defer ticks.Stop()
	for {
		select {
		case cause := <-cancel:
			c.cancel(activityTask, input)
			return nil, cause
		case <-ticks.C:
			cont, res, err := c.handler.Tick(activityTask, input)
			if !cont {
				return res, err
			}
			if res != nil {
				//send an activity update when the result is not null, but we are continuing
				if err := c.worker.signalUpdate(activityTask, res); err != nil {
					Log.Printf("workflow-id=%s activity-id=%s activity-id=%s at=signal-update-error error=%q", LS(activityTask.WorkflowExecution.WorkflowId), LS(activityTask.ActivityType.Name), LS(activityTask.ActivityId), err)
					cancel <- err // RW: We block here because sends are synchronous on unbuffered channels.
					continue // go pick up the cancel message
				}
				Log.Printf("workflow-id=%s activity-id=%s activity-id=%s at=signal-update", LS(activityTask.WorkflowExecution.WorkflowId), LS(activityTask.ActivityType.Name), LS(activityTask.ActivityId))
			}
		}
	}

I can send a PR to fix this one, just depends on what approach is preferred.  I could either make the cancel channel buffered, or could run cancel <- err in a newly spawned goroutine.
Thoughts?

ah, crap, that should totally be buffered. thanks!


Getting error from autodoc when running Sphinx to create documentation

Getting following error and traceback:
WARNING: autodoc: failed to import module 'views' from module 'fpiweb'; the following exception was raised:

Traceback (most recent call last):
File "/Volumes/MBPC/Dvl/Python/PythonProjects/Food-Pantry-Inventory/venv/lib/python3.7/site-packages/sphinx/ext/autodoc/importer.py", line 232, in import_module
import(modname)
File "/Volumes/MBPC/Dvl/Python/PythonProjects/Food-Pantry-Inventory/fpiweb/views.py", line 6, in
from django.contrib.auth.mixins import LoginRequiredMixin
File "/Volumes/MBPC/Dvl/Python/PythonProjects/Food-Pantry-Inventory/venv/lib/python3.7/site-packages/django/contrib/auth/mixins.py", line 3, in
from django.contrib.auth.views import redirect_to_login
File "/Volumes/MBPC/Dvl/Python/PythonProjects/Food-Pantry-Inventory/venv/lib/python3.7/site-packages/django/contrib/auth/views.py", line 10, in
from django.contrib.auth.forms import (
File "/Volumes/MBPC/Dvl/Python/PythonProjects/Food-Pantry-Inventory/venv/lib/python3.7/site-packages/django/contrib/auth/forms.py", line 10, in
from django.contrib.auth.models import User
File "/Volumes/MBPC/Dvl/Python/PythonProjects/Food-Pantry-Inventory/venv/lib/python3.7/site-packages/django/contrib/auth/models.py", line 3, in
from django.contrib.contenttypes.models import ContentType
File "/Volumes/MBPC/Dvl/Python/PythonProjects/Food-Pantry-Inventory/venv/lib/python3.7/site-packages/django/contrib/contenttypes/models.py", line 133, in
class ContentType(models.Model):
File "/Volumes/MBPC/Dvl/Python/PythonProjects/Food-Pantry-Inventory/venv/lib/python3.7/site-packages/django/db/models/base.py", line 111, in new
"INSTALLED_APPS." % (module, name)

RuntimeError: Model class django.contrib.contenttypes.models.ContentType doesn't declare an explicit app_label and isn't in an application in INSTALLED_APPS.

Please add bug label

I'm taking a look and this, and am making some progress.  Want to assign it to me?

I've gotten rid of the RuntimeErrors and reduced the warnings by removing duplicate references.
I'm now down to these two warnings:
/docs/source/JavaScriptAndJQuery.rst: WARNING: document isn't included in any toctree
/docs/source/modules.rst: WARNING: document isn't included in any toctree
And several cases of:
WARNING: plantuml command 'plantuml' cannot be run
I'm assuming this one is because I don't have plantuml installed locally.
Finally there's this warning, which is still baffling me at the moment:
/StandaloneTools/QRCodePrinter.py:docstring of StandaloneTools.QRCodePrinter:13: WARNING: Option list ends without a blank line; unexpected unindent.

@deeppunster found a couple missing commas that resolved the unexpected indent warning.
After installing plantuml those errors went away


Reduce @log_level to error for fluentd.

The goal is to avoid fluentd logging to stdout to have those logs
collected by itself again and streamed to Elasticsearch, eventually
exhausting Elasticsearch storage capacity.

See my comment at https://github.com/bitnami/kube-prod-runtime/issues/193#issuecomment-436266408,
adding that configfile with fluent.** filtering-out should solve the issue, and let's please
have this PR only addressing this one.

Thanks for the help @jjo !


adding determinism settings for the pytorch models

This PR passes the seed from the config file to pytorch models to allow deterministic training. Unlike with TF, the torch models need to have the seed passed to both the model function and the train function. I added this functionality to all 4 torch models, but I only tested it with RGCN_v1.

This page indicates that a CUDA environment variable may need to be set for deterministic results. I'm posting it here for reference. I tried it in one test run but I didn't include it in the PR (and I don't think it made a permanent change to my environment variable). If we have issues with determinism in the future this could be something to check.
https://pytorch.org/docs/stable/generated/torch.nn.LSTM.html


[bug] ReferenceError: Headers is not defined

if you meet this bug,Don't warry about it.
this bug caused by a new programer feature "new Headers()",
that means you can upgrade your node version to higher.
i meet this bug on Node v16.19.0. when i solve this bug on Node v18.17.1

By the way,
If possible, please update the minimum node dependency version number of this npm package, that would be great

https://developer.mozilla.org/en-US/docs/Web/API/Headers/Headers#browser_compatibility
Headers is supported from Node v18.0.0 (Released 2022-04-19)

is there a way to avoid this issue using node 16? still want the ability to add an unsubscribe link

Unfortunately, Node.js 16 has reached End-of-Life [1] and only Node 18 is supported by this package [2].
[1] https://nodejs.org/en/blog/announcements/nodejs16-eol
[2] https://github.com/resendlabs/resend-node/blob/main/package.json#L10-L12


Fix non-default autoshrink print_mode

Commit aee6a2854d ("Cleanup. Replace theft_autoshrink_wrap with tagged
union.") failed to hook up theft_autoshrink_print as part of refactoring.
This broke setting print_mode to something non-default.

Thanks. I likely won't get to reviewing this for a bit because of the holidays.
When I do, I'm going to retarget the PR to the develop branch to fit my versioned release process (as noted in CONTRIBUTING).


xmlpnpschemaversion not uptodate

Hello,
I'm using PowerShell to extract PnP Schema and when I'm trying to:
Get-PnPProvisioningTemplate -Schema LATEST -Out tmp.xml
The Schema version is
<pnp:Provisioning xmlns:pnp="http://schemas.dev.office.com/PnP/2016/05/ProvisioningSchema">...</pnp:Provisioning>
Instead of the latest schema:
<pnp:Provisioning xmlns:pnp="http://schemas.dev.office.com/PnP/2017/05/ProvisioningSchema">...</pnp:Provisioning>
Indeed xmlpnpschemaversion  is not containing the V201705
The last schema is not working too with Apply-PnPProvisioningTemplate
There is a mistake?
Thank you for your help!

We closed this issue as it had not been any activity within last 180 days. This is a generic process we have decided to perform for issues, which have not been explicitly marked still to be "work in progress" based on tags. We are performing this cleaning to make sure that old issues that have already been solved (but not closed) or are no longer relevant are cleaned out and make the issues more manageable. If this issue still valid, we would ask you to open a new issue and follow the guidance in the issue template related to the recommended location. We do apologize for any inconveniences this might cause. Please do remember that issues on the issue lists are also messages for others in the community, so you can also check if you can assist on any of them. “Sharing is caring!”


DECIMAL_PRECISION whereas it's only 2

Hello,
I have an issue but I don't understand why, the value are less than 2 decimals...
{
		"application_context": {
			"shipping_preference": "SET_PROVIDED_ADDRESS"
		},
		"intent": "CAPTURE",
		"purchase_units": [
			{
				"amount": {
					"breakdown": {
						"discount": {
							"currency_code": "xxxxxx",
							"value": "xxxxxx"
						},
						"item_total": {
							"currency_code": "EUR",
							"value": 9.71
						},
						"shipping": {
							"currency_code": "EUR",
							"value": 6.5
						},
						"tax_total": {
							"currency_code": "EUR",
							"value": 0
						}
					},
					"currency_code": "EUR",
					"value": 16.21
				},
				"invoice_number": "xxxxxx",
				"items": [
					{
						"name": "Pipe longue coudée REG",
						"quantity": 1,
						"tax": {
							"currency_code": "EUR",
							"value": 0
						},
						"unit_amount": {
							"currency_code": "EUR",
							"value": 9.71
						}
					}
				],
				"payee": {
					"merchant_id": "7KH6PMA7FYN6G"
				},
				"reference_id": "ea7b5eb4-3f51-43fd-b24b-7360ab9951c7",
				"shipping": {
					"address": {
						"address_line_1": "*****",
						"admin_area_2": "Aubeterre-sur-Dronne",
						"country_code": "FR",
						"postal_code": "16390"
					},
					"name": {
						"full_name": "*****"
					}
				},
				"soft_descriptor": "Sylius PayPal Payment"
			}
		]
	}

And the error :
"body": {
		"debug_id": "2ec0be278095d",
		"details": [
			{
				"description": "If the currency supports decimals, only two decimal place precision is supported.",
				"field": "/purchase_units/@reference_id=='2e03277d-110a-4d59-a103-1a38eef64506'/amount/value",
				"issue": "DECIMAL_PRECISION",
				"value": "xxxxxx"
			}
		],
		"links": [
			{
				"href": "https://developer.paypal.com/docs/api/orders/v2/#error-DECIMAL_PRECISION",
				"method": "GET",
				"rel": "information_link"
			}
		],
		"message": "The requested action could not be performed, semantically incorrect, or failed business validation.",
		"name": "UNPROCESSABLE_ENTITY"
	}

SO DECIMAL_PRECISION respect the "If the currency supports decimals, only two decimal places are supported." definition with "16.21"...
Can you help me?
Sylius 1.10.2 and Paypal 1.2.2

I did lot of tests! The issue with  "value": 16.21 is weird.
For example, if the value is 16.2, it will work, but it will work with the string "16.21"...
SO the idea should be to set all value as string in src/Api/CreateOrderApi.php
for example :
'amount' => [
                        'currency_code' => $order->getCurrencyCode(),
                        'value' =>   number_format( (int) $payment->getAmount() / 100, 2),

I am surprised to be the only "one" with this issue...
The more surprised thing is the same request (rejected) with Postman will work.... It is so crazy!

I contacted the Paypal Support, I hope they will help me.

Hi, there is already an open issue about this and a PR

I don't know if it's the same issue because even if I changed the file like the PR, I still have the issue in my side...

Hey @Snowbaha i just created pr where all amounts were changed to floats, to keep decimal precision, could you try this out? https://github.com/Sylius/PayPalPlugin/pull/213

I have the same issue with your PR,
the value looks good but I have error with the API response :(
{
	"additional_properties": "xxxxxx",
	"body": {
		"application_context": {
			"shipping_preference": "SET_PROVIDED_ADDRESS"
		},
		"intent": "CAPTURE",
		"purchase_units": [
			{
				"amount": {
					"breakdown": {
						"discount": {
							"currency_code": "xxxxxx",
							"value": "xxxxxx"
						},
						"item_total": {
							"currency_code": "EUR",
							"value": 38.02
						},
						"shipping": {
							"currency_code": "EUR",
							"value": 15.4
						},
						"tax_total": {
							"currency_code": "EUR",
							"value": 0
						}
					},
					"currency_code": "EUR",
					"value": 53.42
				},
				"invoice_number": "xxxxxx",
				"items": [
					{
						"name": "Adhésif haute performance",
						"quantity": 1,
						"tax": {
							"currency_code": "EUR",
							"value": 0
						},
						"unit_amount": {
							"currency_code": "EUR",
							"value": 38.02
						}
					}
				],
				"payee": {
					"merchant_id": "7KH6PMA7FYN6G"
				},
				"reference_id": "194b64fb-af52-4ca8-9676-6398b535df95",
				"shipping": {
					"address": {
						"address_line_1": "*****",
						"admin_area_2": "Auxerre",
						"country_code": "FR",
						"postal_code": "89000"
					},
					"name": {
						"full_name": "*****"
					}
				},
				"soft_descriptor": "Sylius PayPal Payment"
			}
		]
	},
	"header": {
		"accept": "application/json",
		"akamai-x-forwarded-for-7": "xxxxxx",
		"authorization": "xxxxx",
		"client-auth": "xxxxx",
		"client_info": "xxxxxx",
		"content-length": "951",
		"content-type": "application/json",
		"correlation-id": "2abe8f70186cf",
		"edge-locator": "xxxxxx",
		"edgeid": "xxxxxx",
		"host": "api.sandbox.paypal.com",
		"paypal-partner-attribution-id": "Sylius_MP_PPCP",
		"paypal-request-id": "27952ec9-68f8-4292-ba3a-48033e3635a3",
		"paypal-routing-metadata": "xxxxxx",
		"pp_client_ssl_cipher": "ECDHE-RSA-AES128-GCM-SHA256",
		"pp_client_ssl_protocol": "TLSv1.2",
		"pp_geo_loc": "AZ",
		"pp_remote_addr": "<IP_ADDRESS>",
		"pp_vip": "<IP_ADDRESS>",
		"user-agent": "GuzzleHttp/6.5.5 curl/7.29.0 PHP/7.4.16",
		"x-ads-si-context": "{}",
		"x-forwarded-for": "<IP_ADDRESS>",
		"x-forwarded-proto": "xxxxxx",
		"x-paypal-correlation-id": "2abe8f70186cf",
		"x-pp-ads-performed": "true",
		"x-pp-ads-request-id": "2abe8f70186cf_02fd95d217c0a60a81e6701bff4c97f3",
		"x-pp-ads-usecase-id": "default_ip_operation_feed",
		"x-pp-anomaly-detection-mechanism": "ForwardRequest",
		"x-pp-corrid": "2abe8f70186cf",
		"x-pp-idempotencyid": "2abe8f70186cf_1632137745",
		"x-pp-slingshot-targetapp": "apiplatformproxyserv",
		"x-sigsci-agentresponse": "xxxxxx",
		"x-sigsci-mac": "xxxxxx",
		"x-sigsci-requestid": "xxxxxx",
		"x-slr-orig-script_uri": "https://api.sandbox.paypal.com/v2/checkout/orders"
	},
	"method": "POST"
}

And the Answer failed:
{
	"additional_properties": "xxxxxx",
	"body": {
		"debug_id": "2abe8f70186cf",
		"details": [
			{
				"description": "If the currency supports decimals, only two decimal place precision is supported.",
				"field": "/purchase_units/@reference_id=='194b64fb-af52-4ca8-9676-6398b535df95'/amount/value",
				"issue": "DECIMAL_PRECISION",
				"value": "xxxxxx"
			}
		],
		"links": [
			{
				"href": "https://developer.paypal.com/docs/api/orders/v2/#error-DECIMAL_PRECISION",
				"method": "GET",
				"rel": "information_link"
			}
		],
		"message": "The requested action could not be performed, semantically incorrect, or failed business validation.",
		"name": "UNPROCESSABLE_ENTITY"
	},
	"duration_time": 101,
	"header": {
		"APPLICATION_ID": "APP-80W284485P519543T",
		"BORDER-IP": "xxxxxx",
		"CALLER_ACCT_NUM": "7KH6PMA7FYN6G",
		"Date": "Mon, 20 Sep 2021 11:35:45 GMT",
		"Keep-Alive": "xxxxxx",
		"PROCESSING-IP": "xxxxxx",
		"SERVER_INFO": "xxxxxx",
		"paypal-debug-id": "2abe8f70186cf"
	},
	"status": 422
}

I created a commit with a fix working for me :
https://github.com/Snowbaha/PayPalPlugin/commit/a2bd4d99f14fb2d1663c6f8ad28bcfbe4925f378
What do you think?

@Snowbaha I really support the use of "number_format" in this integration with Paypal. BUT, you MUST specify the fourth parameter as an empty string otherwise "number_format" will use "," as thousands separator and Paypal will throw an error.
It should be like this:
number_format($amount, 2, '.', '')

@LucaGallinari you are right!
I added the change: https://github.com/Snowbaha/PayPalPlugin/commit/493f855a525ab4f49986d82435e9e93b1ce7a3e3

Okay i see the problem now, in connected PR i added full spec coverage for it and changed to strings can you check if that solves the problem? Unfortunatelly i wasnt able to reproduce same issue. With this PR i was able to create order and pay it.

@SirDomin Thank you, it is working now with your PR too :)


cocos2d extension are broken

i forgot what the error was but http stuff dont work

i just cant import cocos2d at all


Add rules_pkg 0.7.0

I'm not sure why the CI is failing. I copied the presubmit.yml from modules/rules_pkg/0.5.1<EMAIL_ADDRESS>declared in thepresubmit.yml, so I suspect there was a similar error on 0.5.1`.
I'm not sure what I should do to fix this. I could exclude the :make_rpm target which is triggering the issue, but that feels unsatisfactory.

@moyamo Could you take a look at the CI failures? Something is referencing @rules_sh_shim_exe, but the module doesn't use_repo it.

@moyamo The failing tests seem to require additional setup (see https://github.com/bazelbuild/rules_pkg/blob/c74ef150b11633763b127174c897205b729db832/docs/latest.md#pkg_rpm:~:text=pkg_filegroup and friends.-,pkg_rpm() depends on the existence of an rpmbuild toolchain,-. Many users will). Feel free to disable them.
@aiuto Would you be available to help make rules_pkg's pkg_rpm work with Bzlmod? Looks like all it would take would be a module extension calling find_system_rpmbuild(name="rules_pkg_rpmbuild").

I think the extension is the way to go. People need the ability to use rules_pkg without rpm by default, and the people who want rpm need to depend on an extra thing to register the toolchain.
But... starting in bcr is the wrong way. Someone first has to make sure rules_pkg builds from modules. If you want to send me a PR that also gets CI running with and without bzlmod, that would be great.

Oh, I just saw this, I merged rules_pkg 0.7.0 by disabling the rpm target in https://github.com/bazelbuild/bazel-central-registry/pull/167. But would love to see the toolchain issue get fixed.

Do you have a pattern available for how we are supposed to add modules that have multiple versions and some optional toolchains?  For rules_pkg, we want most people to use it and only specify the module.  If you want rpm, you have to also register the rpm toolchain. We'll have other optional features in the future.
Also, it's not clear what the value of the presubmit is for modules that have distinct source and distribution releases. For instance, rules_pkg has no tests available in the distribution that people use. If you want the tests, you need to import the full sources. Those have additional dependencies not recorded the bzlmod file.

Thanks @meteorcloudy. I'm going to close this since #167 replaces this PR.
@aiuto

Do you have a pattern available for how we are supposed to add modules that have multiple versions and some optional toolchains? For rules_pkg, we want most people to use it and only specify the module. If you want rpm, you have to also register the rpm toolchain. We'll have other optional features in the future.

rules_nixpkgs solved the problem of optional features by using multiple MODULE.bzl files. See https://github.com/tweag/rules_nixpkgs/issues/182 for more details.

If you want rpm, you have to also register the rpm toolchain. We'll have other optional features in the future.

We currently don't have optional dependencies support. For this case, shouldn't rules_pkg automatically register the rpm toolchain for the users?

Also, it's not clear what the value of the presubmit is for modules that have distinct source and distribution releases.

This is really a sanity check to ensure the most basic stuff work so that you don't ship broken things to user.

If you want the tests, you need to import the full sources. Those have additional dependencies not recorded the bzlmod file.

That would not be an issue as the test module can specify its own set of dependencies.

rpmbuild only exists on some linux versions so the auto-detection in the toolchcain only works for some users.  If we had full optional toolchain support, then we could work something out, but then we have to make the rules bazel 6.x specific, and have a different version for 5.x and below. Up until now, we have been able to avoid having a rule set track for each LTS branch.


feat(lint): add eslint

/claim #15
closes #15
https://github.com/user-attachments/assets/89299d51-5511-415a-9fdd-5f8a3c2af04e
@gemanor - attached video recording as requested in the issue #15 to demonstrate all features are working
Implemented the rules and refactored the code to clear the current errors and warning
integrated prettier into eslint so one command will fix formatting and linting

Closing, as we got another contribution from a previous contributor.


⚠ GitLab Server has degraded performance

In ee553e5, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 14232 ms

Resolved: GitLab Server performance has improved in 1d62418.


Watching value and star value

Both values match, watching copying stars value

will sort issue starting January 2025


fix: fix use headers that are set via the onPreDev of a plugin

🎉 Thanks for submitting a pull request! 🎉
Summary
Depends on: https://github.com/netlify/build/pull/4932
Fixes https://github.com/netlify/cli/issues/5544
This PR only parses the headers but we might want to do the same for redirects in a follow up or other config mutations not sure what frameworks needs here?
cc @kitop @ascorbic

For us to review and ship your PR efficiently, please perform the following steps:

[ ] Open a bug/issue before writing your code 🧑💻. This ensures we can discuss the changes and get feedback from everyone that should be involved.
[ ] Read the contribution guidelines 📖. This ensures your code follows our style guide and
passes our tests.
[ ] Update or add tests (if any source code was changed or added) 🧪
[ ] Update or add documentation (if features were changed or added) 📝
[ ] Make sure the status checks below are successful ✅

A picture of a cute animal (not mandatory, but encouraged)

How does this handle live updates to the config/headers/redirects files? For example, let's say that I have a plugin with a onPreDev event that reads redirect rules and mutates them somehow. On the first run, the plugin will read the input, do the mutations, and produce the right output. But then I make a change to the redirects file and the plugin is not running again, so will we get the right output?

How does this handle live updates to the config/headers/redirects files? For example, let's say that I have a plugin with a onPreDev event that reads redirect rules and mutates them somehow. On the first run, the plugin will read the input, do the mutations, and produce the right output. But then I make a change to the redirects file and the plugin is not running again, so will we get the right output?

So if you have headers in a _headers or netlify.toml file they get picked up and merged with the plugin config:
/** @type {import('@netlify/build').Om} */
export const onPreDev = async ({ netlifyConfig }) => {
  console.log('inside onPreDev');
  const headers = netlifyConfig.headers;
  headers.push({
    for: '/*',
    values: {
      'x-test': 'value',
    },
  });

  netlifyConfig.headers = headers;
};

If you change the toml file and update the header the header from the plugin is still in there:

The onPreDev hook i only run before the command and has no watch mode so changes to that would need a restart of ntl dev
Does this answer your question?

The use case I'm thinking is something like this:
export const onPreDev = async ({ netlifyConfig }) => {
  const { headers } = netlifyConfig;

  headers[0].values['x-test'] += " modified"

  netlifyConfig.headers = headers;
};

So the plugin appends something to the value of a header. If I re-save the file and change foo to bar, will x-test change from foo modified to bar modified?
(I tried checking out the branch and trying this myself, but I'm getting the same error with @netlify/config that we're seeing in the CI.)

@eduardoboucas should work now, the newer version of @netlify/config was not in. Now it's merged into it: https://github.com/netlify/cli/commit/bd9823e5ae9994cadc39e1c9571047b340515811
THat was the dependent PR from the description (had it linked locally when developing)

  headers[0].values['x-test'] += " modified"

@eduardoboucas you are right this behaviour is not working as the onPreDev is only executed before starting the devServer and NOT during reloads.
TBH I would see this as "expected" behaviour as no build hook is run on file updates.

I'm not sure I agree. It's true that no hooks currently run on file updates, but that's because all hooks used to be tailored for the build process, and they didn't run in Netlify Dev at all. That changed with onPreDev and onDev.
If we say that these plugin hooks will run in Netlify Dev, and if we say that Netlify Dev has live reloading for any platform configuration files, I think we need to make these two concepts co-exist, otherwise we're creating the precedent that the state of your application may differ based on whether you've updated any files. I think that somewhat defeats the purpose of supporting live reloading in the first place.
The way I was imagining we'd approach this is by running the *Dev hooks on file reloads. Netlify Build already has the plumbing for running just the dev pipeline, we'd just need to accommodate the fact that on subsequent runs we'll leave the initial dev command alone and run just the hooks.

I agree with Eduardo. It would be unexpected if saving a file meant the headers suddenly changed

Chiming in from the future...
for onPreDev.. I expect them to run once - I see it as on pre dev command. For onDev, I could see them running multiple times based on throttled file changes.
Can we get this shipped and sort out onDev later? Even if we want to sort out the watching + invoking of onPreDev (I'd disagree with the need) but the initial pass should be simply allowing them to work at all

bumping this issue as we are having the same experience with pushing dynamic headers
module.exports = {
  onPreDev({ netlifyConfig }) {
    netlifyConfig.redirects.push({
      from: '/*',
      to: `${netlifyConfig.build.environment.APP_BASE_URL}/:splat`,
      status: 200,
    });
  },
};

due to almost a year, creating a new PR was easier than sorting the merge conflicts... so, let's carry the convo over here: https://github.com/netlify/cli/pull/6365


3.4.3 release

Updated the version to 3.4.3
Updated UBUNTU_OPTS to 14.04

:metal:
-----> Kitchen is finished. (20m55.46s)
15:24:22 rabbitmq > (3.4.3) bundle exec kitchen list
Instance                  Driver        Provisioner  Last Action
default-centos-6-5-x64    Digitalocean  ChefZero     <Not Created>
default-debian-7-0-x64    Digitalocean  ChefZero     <Not Created>
default-ubuntu-12-04-x64  Digitalocean  ChefZero     <Not Created>
default-ubuntu-14-04-x64  Digitalocean  ChefZero     <Not Created>
cluster-centos-6-5-x64    Digitalocean  ChefZero     <Not Created>
cluster-debian-7-0-x64    Digitalocean  ChefZero     <Not Created>
cluster-ubuntu-12-04-x64  Digitalocean  ChefZero     <Not Created>
cluster-ubuntu-14-04-x64  Digitalocean  ChefZero     <Not Created>
lwrps-centos-6-5-x64      Digitalocean  ChefZero     <Not Created>
lwrps-debian-7-0-x64      Digitalocean  ChefZero     <Not Created>
lwrps-ubuntu-12-04-x64    Digitalocean  ChefZero     <Not Created>
lwrps-ubuntu-14-04-x64    Digitalocean  ChefZero     <Not Created>
15:24:43 rabbitmq > (3.4.3)

:+1:


Changing status of multiple tasks does not work

When I use the checkmark to select all tasks in the current list and use the dropdown to change their status to 'deleted' nothing happens.

That is actually for selecting tasks that are already in a specific status (Deleted in this case) or have a specific priority. The only bulk status change we currently support is resetting back to Created status, which is done with a separate control. We could certainly enhance that to allow to setting to additional statuses.

We now support bulk changes to nearly every status.


Old fork of the sphere with features

I have an old fork of the sphere, even before it became free.
and there are a couple of interesting features there.
I can even share the source code with you if it makes your task easier.
#item1
"plocal" like "ref1" but with "nametag"
plocal.blabla =
plocal.name = House
plocal.bounce
#item2
I don't use a LIST, but when working with tags
Instead of;
if
for i 0 <eval( - 1)>
if strmatch("template_*","<tagat.<local.i>.key>")
	endif
endfor

endif
we have this;
fortags template_*
say <local._var_name> <local._var_value>
endfor
#item3
trigger @SaveStart @SaveEnd
on item and chars
#item4
serv.addtimerd 0 f_function
like timerf but work on serv. with timer 1/10 sec
[FUNCTION f_function]
ref1 =
ref1.try
#item4
[ADIALOG
dialogtype
DTYPE_NEW  01 // do not interfere with targets and dclick
DTYPE_SINGLE 02 // allows only one such dialog on the screen
DTYPE_SINGLE_OBJ 04 // allows only one such dialog on the object
DTYPE_NOUPDATE 08 // with 02 and 04 - forbid updating the dialog on repeated call
#item5
ADIALOGCLOSE type, arg1, arg2, buttonID
0x1 - arg1 = resource id of dialog (example "ad_dialog") - close the dialog with this ID
0x2 - arg2 = object uid - close the dialog on this object
0x4 - close all dialogs with these flags, otherwise the first one
if type = 0 - arg1 = dialog uid (return like ) - close the dialog with this UID
buttonID - The ID of the button that the dialog will close with (-1 - system close)
	On=@Equip
		src.tag.xz = <adialog ad_xz>

	On=@Unequip
		src.adialogclose 0,<src.tag.xz>,0,-1  - close the dialog with this UID

	On=@Equip
		adialog ad_xz

	On=@Unequip
		src.adialogclose 07,ad_xz,<uid>,-1 - close all dialog on this OBJ

I don't see any reason to be added except FORTAGS, FORTAGS can be added but for others we already have them,
REF1 is a CUID reference so you can use .NAME .TYPE .BLABLA, so no need to add PLOCAL for it.
@SaveStart and @SaveEnd triggers for every item can be really expensive, I am not sure but it can be added after made some tests.
We don't need ADDTIMERD as we already have TIMERFMS for it.
For dialog close, we already have dialogclose command for any dialog but it may be improved.

Can you explain better PLOCAL? It looks like REFx.
I fear this will encourage using a lot of tags, polluting them, but per se it could be a good idea.
100% sure it will be really too much expensive.
4, 5) I second what xwerswoodx said.

I'll finish you off
we have also PTAG like tag+ref.
ptag.item =
ptag.item.name = blablba
ptag.item.cont =

first of all, it is clear and readable
plocal.sys
plocal.plr
plocal.item
plocal.npc
on the third page, I already forget what happened at the beginning.
well, for example, there is also comfort
for 1 2
local.i = "<eval(<local.for>)>"
doswitch <local.i> - 1
begin
plocal.npc<local.i> = <f_newnpc c_enc_angel>
plocal.npc_<local.i>.speechcolor 050
end
begin
plocal.npc_<local.i> = <f_newnpc c_enc_daemon>
plocal.npc_<local.i>.speechcolor 112
end
enddo
plocal.npc_<local.i>.p =
doswitch <local.i> - 1
plocal.npc_<local.i>.move 1 <eval {-1 1}>
plocal.npc_<local.i>.move <eval {-1 1}> -1
enddo
endfor

Well, these are actually my seconds, why should you count them?
I use BACKGROUND SAVE  and it's almost unnoticeable

or you can do it so that it can be enabled and disabled from scripts. in principle, is this enough for statistics once a day?

So, i don't really see a use for @SaveStart and @SaveEnd, but i can change my mind if there are some interesting examples.
FORTAGS could be useful, but i think it encourages bad scripting practices.
DTYPE_SINGLE is solved by using SDIALOG, though the other flags aren't covered by the current features. Could you suggest some real-world uses of those flags with ADIALOG?

@SaveStart @SaveEnd
I create temporary quest or auction items that can be deleted after a month or a year without having to use a timer.


glitchtech-nightly failing to build

These build failures are related to the current lack of know how on the new appDir flow in Next.js in this project. This will be either fixed within the coming week or the nightly build will be removed to keep working on the Pages based version till further notice.

Moved to sepperate repo for now.


Using fast_float to do local independent parsing

Using fast_float as suggest by @bubnikv to resolve the issue addressed by #322.
PR is already based on #324 to make the ci pass.

Thanks Jan.
I have asked Lukas Hejl from Prusa3D to do the code review.
Vojtech
On Fri, Nov 3, 2023 at 2:08 PM Jan Orend @.***> wrote:

@3dJan https://github.com/3dJan requested your review on: #325
https://github.com/3MFConsortium/lib3mf/pull/325 Using fast_float to do
local independent parsing.
—
Reply to this email directly, view it on GitHub
https://github.com/3MFConsortium/lib3mf/pull/325#event-10855842696, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABMPSI4V7NG4WMZTDAMBQZDYCTUE7AVCNFSM6AAAAAA64NCR4KVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJQHA2TKOBUGI3DSNQ
.
You are receiving this because your review was requested.Message ID:
@.***>

But I don't have permission to approve pull requests.


Alter basefolder definition

This matches the intention expressed in the example code.

Thanks for your help! Sorry for not fixing this right away.
Merging and closing.


Websocket errors

I had some initial issues that I thought was entirely down to me and my code so I tried the example in the Readme just to have a working starting point. However, that throws errors as well. I've tried it with both Python 2.7 and 3.7 and both gives the same errors.

[ERROR:websocket] 2018-08-17 11:58:04,009 - error from callback <bound method CertStreamClient._on_open of <certstream.core.CertStreamClient object at 0x7facdeefb3d0>>: _on_open() takes exactly 2 arguments (1 given)
[ERROR:websocket] 2018-08-17 11:58:04,068 - error from callback <bound method CertStreamClient._on_message of <certstream.core.CertStreamClient object at 0x7facdeefb3d0>>: _on_message() takes exactly 3 arguments (2 given)
[ERROR:websocket] 2018-08-17 11:58:05,445 - error from callback <bound method CertStreamClient._on_message of <certstream.core.CertStreamClient object at 0x7facdeefb3d0>>: _on_message() takes exactly 3 arguments (2 given)
[ERROR:websocket] 2018-08-17 11:58:05,482 - error from callback <bound method CertStreamClient._on_message of <certstream.core.CertStreamClient object at 0x7facdeefb3d0>>: _on_message() takes exactly 3 arguments (2 given)
[ERROR:websocket] 2018-08-17 11:58:05,528 - error from callback <bound method CertStreamClient._on_message of <certstream.core.CertStreamClient object at 0x7facdeefb3d0>>: _on_message() takes exactly 3 arguments (2 given)
[ERROR:websocket] 2018-08-17 11:58:05,537 - error from callback <bound method CertStreamClient._on_message of <certstream.core.CertStreamClient object at 0x7facdeefb3d0>>: _on_message() takes exactly 3 arguments (2 given)

I've tried both version 1.8 and 1.9 - just to check - and both gives the same errors under both versions of Python.
I'm still very open to it being me - but the sample code should in theory work - could there have been any changes to websocket that might need looking at from this projects code?

What version of websocket-client do you use?

print(websocket.version)
0.49.0

@wagner-certat Thanks for giving me the idea. I tested and installed websocket-client 0.48.0 and it works. It seems that version 0.49.0 that was released a couple of days ago must have changed something that breaks this.

You could apply the change to your local installation and check if the problem's gone?

I uninstalled 0.48.0, downloaded 0.49.0 from the repo, changed according to the pull request and re-installed using python setup.py install. No dice - still the same issue. I believe my method of testing is fairly correct?
I've not seen any changelog that indicates an enforced changed behavior from the websocket-client end so I assume the issue would be there and I should close this ticket?
Now that I got the code I might try and see - low hope - if I can spot what they've changed to break this.

If anyone runs into the same problem just use the following version :
websocket-client==0.48.0
certstream==1.9

i had the same issue and the above post worked for me. created a venv and installed specifically those two versions, and things seem to be working well

Hey folks! Glad to see that worked. I'm going to pin the dependency then as part of the certstream-python installation so people shouldn't have to pin it themselves. Once I cut 2.0 do you folks mind helping me test this is working as expected?

We're live with release 1.10, please upgrade and let me know if you all have any further issues!


[android] White screen before camera on 🐛

What were you trying to do?
I get a white screen in 3s before camera on.
            {device != null && hasPermission && (
                <Camera
                  ref={cameraRef}
                  onInitialized={showPopupInstruction}
                  style={StyleSheet.absoluteFill}
                  device={device}
                  isActive
                  orientation="portrait"
                  focusable
                  enableDepthData
                  photo
                />
             )}

Reproduceable Code
No response
What happened instead?
I get a white screen in 3s before camera on.
Relevant log output
No response
Device
android real device
VisionCamera Version
2.15.2
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar issues in this repository and found none.

Having the same issue here, Any update please

@ChauVV @shakuniyamihir8400
Hey guys, I'm facing the same issue on some devices. Were you able to find a workaround for this?

Hey! I've rewritten the entire Android codebase of VisionCamera from CameraX to Camera2 in the efforts of ✨ VisionCamera V3.
I just now completed the Camera2 rewrite and I believe the core structure is running, but there might be some edge cases to iron out. Can you try and test the PR #1674 for me to see if you can still reproduce this issue here?
Here's an instruction on how you can test that: https://github.com/mrousavy/react-native-vision-camera/pull/1674#issuecomment-1684104217
If the issue cannot be reproduced with that version/PR anymore, then hoorayy, I fixed it! 🎉
Otherwise please let me know and I'll keep this issue open to keep track of it.
Thank you!

The white screen might be because your View is white. Add backgroundColor: black to your view


Fix eject command updating scripts output

Hi :) I've a spotted a small bug in eject command output.
Steps to reproduce:
Given the package.json file:
"start": "react-scripts start",
"build": "react-scripts build",
"test": "react-scripts test --env=jsdom",
"eject": "react-scripts eject",
"stylelint": "node ./node_modules/stylelint/bin/stylelint.js src/**/*.scss",
"eslint": "node ./node_modules/eslint/bin/eslint.js src/**/*.js"

As you can see there are two custom scripts that were added after the project was created.
The output of npm run eject is:
Replacing "react-scripts start" with "node scripts/start.js"
Replacing "react-scripts build" with "node scripts/build.js"
Replacing "react-scripts test" with "node scripts/test.js"
Replacing "react-scripts stylelint" with "node scripts/stylelint.js"
Replacing "react-scripts eslint" with "node scripts/eslint.js"

While the updated package.json looks like:
"start": "node scripts/start.js",
"build": "node scripts/build.js",
"test": "node scripts/test.js --env=jsdom",
"stylelint": "node ./node_modules/stylelint/bin/stylelint.js src/**/*.scss",
"eslint": "node ./node_modules/eslint/bin/eslint.js src/**/*.js"

so those two custom scripts were untouched and it's ok, because they are not related to the Create React App.
I have added a simple check to fix the misleading message. After the fix the output looks like:
Replacing "react-scripts start" with "node scripts/start.js"
Replacing "react-scripts build" with "node scripts/build.js"
Replacing "react-scripts test" with "node scripts/test.js"

Cheers,

This looks good to me. Thanks!


release-2024.5.3-hotfix

Short description
Gold cards can not be activated since expirationDate = 0 isBefore today
Proposed changes

return false if expirationDate = 0 for isExpired()

Side effects

none

Resolved issues

Fixes: #1517
Testing

Create a gold card
Try to activate the gold card
card should not be expired anymore and can be activated

wrong release branch was used

As I mentioned in the other pr, this is not the root of the cause.
ExpirationDay should be null for the golden card.
Please fix it in the card mutation service, and then add a migration script that sets those entries to null that have the value 0.


Add haddock option to override googleapis font URL

originally brought up here: https://gitlab.haskell.org/ghc/ghc/-/issues/18392 (not by me but I support the proposal. I would  have deleted "in Debian" from the second sentence.)
Haddock produces documents using fonts from the hard-coded URL https://fonts.googleapis.com/css?family=PT+Sans:400,400i,700. Having a package downloading something from an external resource every time a user opens the documentation is considered bad practice  in Debian. Lintian (an error reporting tool for Debian packages) produces the following warning:
W: privacy-breach-generic [<link rel="stylesheet" type="text/css" href="https://fonts.googleapis.com/css?family=pt+sans:400,400i,700" />] (https://fonts.googleapis.com/css?family=pt+sans:400,400i,700)

It would be great to add an option to haddock (just like --mathjax=URL) to allow users to override the hard coded fontUrl to something available locally. Debian already packages this font family (fonts-paratype package) so we could use that flag to point to them instead of Google.

Are these fonts needed at all? I have googleapis.com blocked by umatrix (naturally) and can still read and navigate haddock output. What features am I missing?

We should ship the used fonts ourselves, if they are used at all.
Thank you for raising this.

We should ship the used fonts ourselves, if they are used at all.
Thank you for raising this.

Any update on this issue? (since it's been a few years)


Where are the binary packages?

I need binary packages.

There has been some delay in actually publishing a binary package ecosystem.
The package manager (developed at https://github.com/brechtsanders/winlibs_tools) works well, but I need to make sure licenses and conditions are respected when publishing binaries. This has proven to be a lot more work than I thought.


JAMES-3107 Desactivate metrics p99 log

Using metrics to capture p99 lead to over-snapshoting, and incurs a 33% throughtput penalty on JMAP draft.
Other SLOW logging implementations do rely on manual time
measurements (eg Datastax Cassandra driver).
As such, due to the low benefits and high costs I propose
to deprecate the logP99 API and migrate away from it.
Tools like Glowroot are able to capture slow traces at a
lighter cost and should rather be used.
Performance
I did run 2 benchmarks to evaluate the changes in terms of performance:

a. Run 1000 users doing a representative JMAP polling request with a pause of 2-4 seconds in between. This represents the regular usage.
-b. Run 2000 users doing a representative JMAP polling request with a pause of 2-4 seconds in between. This adapts to system maximum throughtput (as high latencies would add up to the delays and acts as a feedback loop).

For each of these scenari we include:

Gatling overview
Glowroot average, that includes time spent executing Cassanra queries
Glowroot percentiles
Glowroot averages (1 min)
And finally Glowroot Cassandra queries breakdown

Testing infrastructure:

3 James, 3 CPU, 6GB memory
3 Cassandra (SSD, 8 CPU, 32GB ram) - OVH B2-30)
OVH S3 object store

The instances are pre-provisionned with representative email corpus. Each account have a representative mailbox distribution 50 -> 1000 mailboxes (hence Mailbox/get dispersion)
Before
a. LOG p99 and Regular scenario

b. LOG p99 and maximum throughtput

After
a. LOG p99 and Regular scenario

b. LOG p99 and maximum throughtput

Conclusion

Dramatic improvments of all KPIs at all levels of load. We can reach workloads with +50% higher
Surprisingly enough, Cassandra queries executed faster. I think because of CPU shortage Cassandra query execution lead to delays.

Side note: I tried many variations of the metrics, and oddly enough the final one behave even better than a "logging" metric implementation.

Rebase please

Rebased, I added one more commit which should be non-controversial...


Continue respecting privacy, but allow a code context function.

I really respect the privacy first mentality. But to satisfy those who need context, using a descriptive function like :Neural.getcontext "prompt" could be very useful while maintaining your design pattern.

I want a feature that sends information from the  editor, but only for local models. There's no issue with sending passwords, trade secrets, and so on to a process running on your local machine, or maybe also to a private organisation server over an encrypted connection.
I would like to set up connections for OpenAssistant and Llama.

I'm going to say this is a duplicate of #16. The intentation for that issue is to track privacy-respecting analysis of code.


fix: update using deno in workers

v1.22 enabled the Deno namespace in workers by default. This PR updates the manual examples that make use of it.

Ah, looks like the  Deno,emit need an update as well

Updated Deno.emit part (Removed the page for it)
@bartlomieju PTAL


Deploy CodeQL Scans for Kubernetes Repos

Motivation
Currently, only a subset of repositories under in Kubernetes org are onboarded for codeQL scanning (example repo running scans via Github Runners).  This issue is opened with the goal of discussing adding/enabling codeQL scanning for the remaining repos, in particular k/k.
Open Questions

[ ] Running scans in Prow vs. Github Runners

[ ] Base Github Runners may not have the required hardware resources to scan all relevant repos

[ ] Running scans as cron vs. on PRs/other events of interest

[ ] Would likely need to run as cron at least initially

/assign

/assign


perf(client): stop exporting dmmf.schema

Closes https://github.com/prisma/client-planning/issues/17

TODO: move Pick<DMMF.Document, 'datamodel' | 'mappings'> type definition to runtime bundle and reference by name, or use raw { [K in 'datamodel' | 'mappings']: DMMF.Document[K] }.
Pick<T, K> conflicts with potential model name Pick.

TODO: investigate missing-engine tests failing due to Invalid `prisma.user.findMany()` invocation in ... not being present in the expected error message:
    - Snapshot  - 9
    + Received  + 1

    @@ -1,14 +1,6 @@
    -
    - Invalid `prisma.user.findMany()` invocation in
    - /client/src/__tests__/integration/errors/missing-engine-native-binaryTarget/library.test.ts:0:0
    -
    -   31 })
    -   32
    -   33 await expect(async () => {
    - → 34   await prisma.user.findMany(
    -   Query engine library for current platform "TEST_PLATFORM" could not be found.
    + Query engine library for current platform "TEST_PLATFORM" could not be found.
      You incorrectly pinned it to TEST_PLATFORM

      This probably happens, because you built Prisma Client on a different platform.
      (Prisma Client looked in<EMAIL_ADDRESS>

Posted in #prisma-ecosystem-tests
:warning: From what I can see, this PR landed in 3.16.0-dev.82 and since then many tests are failing
see https://github.com/prisma/ecosystem-tests/actions/runs/2541422355
Failures seem to relate to this only
Error: node_modules/@prisma/client/runtime/index.d.ts(4,19): error TS2307: Cannot find module 'ts-toolbelt' or its corresponding type declarations.

Also in optional-test it broke the community-generators (typegraphql-prisma) test
https://github.com/prisma/ecosystem-tests/runs/7001616435?check_suite_focus=true#step:6:106
TypeError: Cannot read property 'enumTypes' of undefined
    at new DmmfDocument (/home/runner/work/ecosystem-tests/ecosystem-tests/community-generators/typegraphql-prisma/node_modules/typegraphql-prisma/lib/generator/dmmf/dmmf-document.js:10:30)
    at generateCode (/home/runner/work/ecosystem-tests/ecosystem-tests/community-generators/typegraphql-prisma/node_modules/typegraphql-prisma/lib/generator/generate-code.js:50:26)
    at Object.generate [as onGenerate] (/home/runner/work/ecosystem-tests/ecosystem-tests/community-generators/typegraphql-prisma/node_modules/typegraphql-prisma/lib/cli/prisma-generator.js:47:39)
    at async LineStream.<anonymous> (/home/runner/work/ecosystem-tests/ecosystem-tests/community-generators/typegraphql-prisma/node_modules/@prisma/generator-helper/dist/generatorHandler.js:33:24)


frog: init at v0.13.7

Motivation for this change
frog is a natural language parser for dutch.
It depends on some other packages in the LanguageMachines organisation.
I have included an update script.
Things done
Please check what applies. Note that these are not hard requirements but merely serve as information for reviewers.

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Added recurseIntoAttrs to also build test and any other languagemachines projects that may be added in the future
Rebased onto master

Thanks @Mic92 and @FRidh for the review and help!

There seems to be a derivation failing:
$ nix-shell --keep-going -p languageMachines.test
bash: /nix/store/6jaydhdipi87z9i4p8czq46nj2ga04ni-frog-test: line 1: syntax error near unexpected token `('

I would expect Travis CI to catch that. I will look into it.

Alright, what happens is that nix-shell wants to run the output of the test. I have no idea why that happens, because for example
nix-shell --keep-going -p languageMachines.frog

does not run the executable. I did't expect nix-shell -p to try to run text files as scripts. (Did you?)
The following works and seems sufficient:
nix-build -A languageMachines.frog '<nixpkgs>'

I think the nix-shell behaviour might be a bug.

I guess it is ok then.


Dependency blocks installation

Describe the bug
The dependency ring-client-api seems not compatible with nodejs 18 (maybe?).
Installation on ioBroker running with nodejs 18 fails.
Installation of any adapter fails if the ring adapter was installed before updating nodejs to 18.
To Reproduce
Steps to reproduce the behavior:

Install ioBroker with nodejs 16
Install Ring adapter
Upgrade nodejs to 18
Try to install any other adapter

Expected behavior
Not blocking any other installation.
Screenshots & Logfiles
npm ERR! code EBADENGINE
npm ERR! engine Unsupported engine
npm ERR! engine Not compatible with your version of node/npm<EMAIL_ADDRESS>npm ERR! notsup Not compatible with your version of node/npm<EMAIL_ADDRESS>npm ERR! notsup Required: {"homebridge":">=1.4.0","node":"^16"}
npm ERR! notsup Actual:   {"npm":"8.18.0","node":"v18.8.0"}

Versions:

Adapter version: 3.1.1
JS-Controller version: 4.0.23
Node version: 18.8.0
Operating system: Raspbian

Additional context
Some weird thing: After removing the engine homebridge from the package.json on ring-client-api, installation was working again.

resolved with 3.1.3


test: rewrite models to use snapshots and tables

This should make it easier to maintain since we won't need to edit raw JSON - I wouldn't say this is strictly better as some of the tests were small enough that I'd argue the initial overhead of using snapshots makes it less of a win, but ultimately these don't change often so I think it's still better to be consistent with the rest of the codebase
(I've also got a fix or two coming up in this section which'll be made easier by this)

This will conflict with #1381 - that pull request would probably be the better one to land first

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

Project coverage is 68.33%. Comparing base (8509b99) to head (3d38626).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1382      +/-   ##
==========================================
- Coverage   68.52%   68.33%   -0.19%
==========================================
  Files         183      183
  Lines       17574    17574
==========================================
- Hits        12043    12010      -33
- Misses       4873     4900      +27
- Partials      658      664       +6

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


Allow schedule_after(0, ...) to work

Right now, if a user puts schedule_after(0, ...), the call will fail because there is a check that the scheduled block must be greater than the current block:
if when <= now {
	return Err(Error::<T>::TargetBlockNumberInPast.into())
}

This PR simply adds one additional block to any schedule_after call. So schedule_after(0, ...) will be the block after the current block, schedule_after(1, ...) will be the block after that, etc... So after this PR, all schedule_after will happen one block later. This allows schedule_after to always pass this check with any after input.
This is just a quality of life improvement which should capture the intention of the caller.
cc @xlc

@shawntabrizi you need to merge master

bot merge


getting a blank csv

Not sure what's going wrong, but when I run this, I get a blank .csv file. No tweets.

This repository is 3 years old, not sure what went wrong on your side but i tested it on my side and it is working fine for me.


Redundant Namespace in XML

Describe the bug
There is an extra namespace declaration in the child view.
Component
SDC gallery app
To Reproduce
Steps to reproduce the behavior:

Go to datacapure module
Click on res
Scroll down to 'layout'
click on questionnaire_item_radio_group_view.xml

Expected behavior
The parent view already has a namespace, so there is no need to declare again in child view.
Screenshots
N/A
Smartphone (please complete the following information):

Device: xiaomi mi a3
Android version: 11
Build number:  qssi-user 11 RKQ1.200903.002 V<IP_ADDRESS>.RFQMIXM
Version [e.g. 22]

Additional context
N/A

since #344 was merged this can be closed.


Fix counter notifications

This refactors and improves on the badge notification setup. I've centralized and streamlined the logic for badge behavior all to a single if/else block in createMainWindow, so that it is easier to reason about as a whole. I believe the intended behavior is as follows:

When using the counter option, a numbered badge should appear on the app icon corresponding to the window title, whether focused or not. If no number can be parsed, then no badge should appear.
When not using the counter option, the • badge should appear when a notification is sent, and should clear when the app is focused. (I made a slight tweak to use the smaller • instead of ●, which looks a bit more native to my eye.)

Along the way, I fixed the page-title-updated listener to use the title argument instead of mainWindow.getTitle(), which incorrectly was retrieving the title before the change happened. It seems like the electron event should be named page-title-will-update instead. /shrug

Sorry for the delay, thank you so much for the fix 😄


[Web] Implement dummy IP and NetSocket

In this PR:

Replace IPUnix with dummy IPWeb.

Note: This commit ties the IPUnix to the UNIX_SOCKET_UNAVAILABLE define, disabling it when set. It is maybe not semantically correct (getifaddrs) is not part of the "socket" API, but it's reasonable to expect that a platform not supporting Unix-style sockets, would also not support other Unix network functions.

Implement dummy NetSocketWeb to prevent error spams when using UDP/TCP classes.

This makes the class available to avoid spam from classes using it, even if without any actual implementation (since raw sockets are not available on the web).

Thanks!


Integrate usage data to the compare item grid

this is the next feature I've been working on while cleaning the code.

Added initial functionality on 0.4 release.


DatePicker Inside Modal Not Working Bootstrap 4

I'm using Laravel 5.8 and Bootstrap 4. When using this date picker in a modal, if the month or next / previous buttons are clicked, the calendar just closes. Have you come across this before and know how to fix it? Thanks.
Here is my modal html:

                    Date
                    <input type='text' class="input-sm form-control" id='startingDate' name="starting_date" value="" />

Here is my js:
     $.fn.modal.Constructor.prototype.enforceFocus = function() {};

     $('#modalTaskForm').on('show.bs.modal', function (evt) {

          window.myDatePicker = new DatePicker('#startingDate', {
             datePickerClass: 'date-picker ',
             closeOnSelect: false,
          });
      });

I tried all the suggestions made earlier but they do not work because I am using the date picker inside a modal on a ASP.NET page. If I move the date picker input field to anywhere on the page outside the modal then it works. Really crazy!

Greetings,
After reading the code again and the suggestions made by Mary Haak, I was able to get the date picker to work inside a modal. I changed the options as she suggested:
var options = {
body: document.querySelector("#my-modal-dialog"),
//instead of:
//body: document.body,
...
};
Clean and simple! Thank you so much for your help. I spent 2 hours trying to figure this out!
Cheers,
Everardo


Is there a description of the fields?

I'm trying create some dashboards based on the data collected by pypowerwall and my greatest problem is a understanding of the fields and their values
i.E:

what is site and load
what are the 3 power fields and their difference?

instant_power: 62
instant_reactive_power: -253
instant_apparent_power:<PHONE_NUMBER>8128064

what are the energy_imported and energy_exported fields?

Thanks for clarification
Dakky

HI @dakky
Unfortunately, Tesla hasn't provide any details as to the official descriptions of these.  Here is what we have generally condluded:

Site = utility grid
Load = home (think of it as the "load" that the battery or grid powers)
instant_power = current power (instant) - also called "true power" in wattage (W)
instant_reactive_power = The dissipated power resulting from inductive and capacitive loads measured in volt-amperes reactive (VAR)
instant_apparent_power = The combination of reactive and true power measure in volt-amperes (VA)
energy_imported  = kWh pulled from grid over a duration of time (since Powerwall commissioning it seems)
energy_exported = kWh pushed to grid

If you are building a dashboard, you might check out https://github.com/jasonacox/Powerwall-Dashboard#powerwall-dashboard either to use or help you build yours.

Thanks 4 the feedback @jasonacox
It might be a good idea to collect these infos in the documentation? I can imagine that more people struggle with that issue.
Concerning your dashboard: Yes thats my target. But I have a slightly different setup:

I already have a influxdb2 up and running
influx2 unfortunetly/luckily it uses  a new scripting language for quering.

So basically I am rewriting the queries queries in the dashboard for influxv2, which is quite a hassle because I have to read the json and try to understand whats happening there. The rest of the setup is working fine: pypowerwall runs out of the box, telegraf just needs a new output and now the visualization is the real struggle :-D

Thanks for the suggestion!  I add a glossary: https://github.com/jasonacox/pypowerwall/blob/main/README.md#glossary
Is it possible to run a instance of influxdb 1.8 as well as 2.x?  That would save the effort.  Getting the QL syntax updated to Flux and porting the continuous queries to tasks doesn't seem easy.  Of course, if you do, I'm sure others will appreciate your work and want to know how you did it.

after fiddling around with more frustration than success i added an influxdb1.8 to my setup. Then everything works out of the box

Thanks for the update @dakky  - and great job getting it to work!
I anticipate there will come day we have to migrate to 2.x.  Unfortunately, the migration path (data and CQs) doesn't seem easy or clean, at least from my research.  Hopefully someone in this community will discover an easy path. :)


Problems installing scs in Catalina

Trying to install cvxpy in a project and having issues installing the scs dependency on OS X 10.15.4 (Catalina). I can install scs from source but not scs-python. I've confirmed that I can install on Ubuntu 18.04.4.
I'm aware this isn't strictly an issue with cvxpython but hoping the maintainers may have come upon the issue at some point.
The issue is something related to the compilers in OS X. I've tried the suggestions in #947 to no avail, and both gcc-8 and gcc-9.

This issue belongs in the SCS repository (https://github.com/cvxgrp/scs), though I do understand it's also a CVXPY issue, since SCS is a hard requirement.
@chase-dwelle , do you mind creating this issue in the SCS repository? That way, the SCS developers can also chime in. When you do so, please do include the entire output from installation. You can do so by pasting it into a github gist, for example. (https://gist.github.com/)

Issue was very simple. in the end, needed to delete the OS 10.14 command line tools SDK. Something to do with how the compiler looked for the header files.
sudo rm -rf /Library/Developer/CommandLineTools/MacOSX10.14.sdk

Not quite sure why this is an issue, scs is the only time I've encountered this problem, but glad it's been resolved.


Updated READMEs for the examples Detect and Track Objects, Dicom MRI, Face Tracking, Gesture Detection, Human Pose Tracking, LiDAR, Live Camera Edge Detection, Live Depth Sensor

Updated READMEs for the examples:
Detect and Track Objects
Dicom MRI
Face Tracking
Gesture Detection
Human Pose Tracking
LiDAR
Live Camera Edge Detection
Live Depth Sensor

What
Checklist

[x] I have read and agree to Contributor Guide and the Code of Conduct
[x] I've included a screenshot or gif (if applicable)
[x] I have tested the web demo (if applicable):

Using newly built examples: app.rerun.io
Using examples from latest main build: app.rerun.io
Using full set of examples from nightly build: app.rerun.io

[x] The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG
[x] If applicable, add a new check to the release checklist!

PR Build Summary
Docs preview
Examples preview
Recent benchmark results
Wasm size tracking

Tip: configure your editor to trim trailing newlines. See https://github.com/rerun-io/rerun/blob/main/CONTRIBUTING.md#tools for more :)


Collect ingest metadata within ingest

Rather than expecting the constructor of ingest definitions to specify various pieces of metadata necessary to ingest tiles, the ingest should be written so as to grab this metadata from the headers of to-be-ingested files and (when useful) persist this information on attribute store for the layer being ingested.

Resolved by #1029


Button colors: black or charcoal?

Your documentation page yesterday showed black buttons (now they appear blue) and described how to use the class .black to get them. The code, however, used the class .charcoal. Those buttons appear blue today and are using .charcoal in code; maybe you changed this back to .black?
Here's the relevant documentation page:
http://foundation.zurb.com/docs/buttons.php

Good catch - we had been notified that our code examples reference 'black' but the code was 'charcoal' so we fixed that....and broke this. Its been corrected and committed.


User Themes button in Settings does nothing on MacOS HS

Installed on MacOS High Sierra.
I go into settings, and click appearance, then User themes button with the folder icon, expecting to be able to browse the directory and add some iTerm2 themes I have on hand.
It does nothing. Doesn't open in Mac Finder, or show anything else.

I should open that folder in the finder for you. Clearly a bug somewhere.

This will be in 0.41.


Forwarder doesnt appear to like intermediary devices

Scenario:
User1 has personal meshtastic device
User2 has personal meshtastic device
Router1 is another meshtastic device
All are on the same firmware and channel settings
Router1 is deployed halfway between User1 and User2 who are just barely out of range by themselves
Router1 is within range of both User1 and user2
Expected result: Router1 should relay messages between User1 and User2
Actual result: User1 and User2 cannot communicate

Can you validate that Meshtastic chat messages go through with this configuration? The plugin should be setting the hopLimit on messages to 3.

Sorry, I should have updated this issue. It appears the problem is with Meshtastic itself and likely not with the Forwarder at all: https://github.com/paulmandal/atak-forwarder/issues/160

I will close this issue, and if the aforementioned issue is resolved but the functionality is still not working I will open a new issue here.


only output all associated files on --verbose

Ref. lib/reel/bundle.rb:54
          # TODO: only output all associated files on --verbose
          # shell.list(repo_path)
          puts "\n"
        end
…

Ref. lib/reel/bundle.rb:53
          # TODO: only output all associated files on --verbose
          # shell.list(repo_path)
          puts "\n"
        end
…

Ref. lib/reel/bundle.rb:61
        # TODO: only output all associated files on --verbose
        # shell.list(repo_path)
        puts "\n"
      end
…

Need to implement printing out the repo's files... also see #46


Turn on libgit2sharp by default, and update it to latest

We'd had them side by side for ages, and have been running tests in both mode. We should now be able to use it by default.

:shipit:   any hidden goal (eg. try to work around git issue etc.).

A few goals:

I'm hoping that this will reduce the odd of getting that ugly "VirtualAlloc pointer is null" error.
Our msysgit is ancient, and it's hard to upgrade it. Would like to move away from it
Generally cleaner to call a library than to spawn exe's left and right


Thread crash on receiving SMS

I just had the library crash the thread when it received an SMS. here is the log:
2021-10-17 09:03:56,265 - gsmHat.gsmHat - DEBUG - Received Data: +SAPBR: 1,3,"<IP_ADDRESS>"
2021-10-17 09:03:56,295 - gsmHat.gsmHat - DEBUG - Received Data: OK
2021-10-17 09:03:56,307 - gsmHat.gsmHat - DEBUG - Lock Off
2021-10-17 09:03:56,313 - gsmHat.gsmHat - ERROR - Timeout during data reception
2021-10-17 09:03:56,318 - gsmHat.gsmHat - INFO - Command sent: AT+SAPBR=2,1
2021-10-17 09:03:56,324 - gsmHat.gsmHat - INFO - Actual state of programme: 61
2021-10-17 09:03:56,326 - gsmHat.gsmHat - CRITICAL - Exception: Unhandled timeout during data reception

looking at the script after sending the critical warning it basically causes the script to end using the raise command. Why does it do that? As soon as that happens the library stops working within my script and I don't get any sms messages being sent out or received. Is there a more graceful way to handle the error? Maybe making it wait a few seconds and then try again later?

@flykarlos I faced same issue, can you confirm that the used of electrolytic capacitor prevent this error ?


Fix errors from rubocop

Rubocop currently returns 79 errors. We should fix these. and create a .rubocop.yml file to calibrate the settings. The file from curation_concerns might be a good model.

Oops. I was looking at some older code. I see that rubocop.yml already exists. We should wait until curation_concerns is upgraded to 0.3.0 to fix rubocop errors.

Closed by #44.


Implementing Errorhandling for src/players.rs

Description
I've implemented error handling such that the .unwrap isn't called, instead it's logging the errors and sends if needed an appropriate response back
Please include a summary of the changes.
New internal trait called Logging. Makes it easy to log Results. Changed a few write locks to readlocks.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
(If this is not realvent to your changes delete this field)
Built the project and ran it locally to verify that my error handling did not cause a failure.
Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Checklist:

[x ] My code follows the style guidelines of this project (I just added on my parts type hinting, I like to be as deliberate as possible)
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[ ] I have added tests that prove my chnages are stable
[ ] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Seems like it works pretty well. Nicely done!
We are moving these check boxes from the requirement so don't worry about it this time, but for next time make sure all checklist items are complete before you submit the pr as a non-draft.

You can submit it as a draft PR whenever you like even before the code is finished. We encourage this because it helps others see what is in progress.


Math rendering in Markdown cell not working at all

A very simple problem. I start a notebook, put inline or display math in a cell, and "execute" with ctrl+alt+enter. The raw code is displayed instead of rendered math.
$ f(x) = 42$
$$ e^x $$
Environment data

VS Code version: 1.53.0-insider
Jupyter Extension version (available under the Extensions sidebar): 2020.12.414227025
Python Extension version (available under the Extensions sidebar): XXX
OS (Windows | Mac | Linux distro) and version: Windows_NT x64 10.0.18363
Python and/or Anaconda version: 3.8.5
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Jupyter server running: Local

Expected behaviour
To see rendered math in the notebook editor after executing markdown cell.
Actual behaviour
I see un-rendered math.
Logs
Nothing.

Sorry but LaTeX is not currently supported in the insiders version. See this bug for more information:
#1216

Sorry but LaTeX is not currently supported in the insiders version. See this bug for more information:
#1216


Missing collection/object_collection.twig for array of objects

Hello.
I have just noticed there is missing object_collection.twig template under collection directory.

Hi sorry for the late answer.
Memio currently doesn't support the generation of multiple classes per file, this was done to follow the PSR standard.
However I'm aware that some valid use cases can require this feature (e.g. optimize autoloading but joining classes into a single file), so any Pull Requests are welcomed :) . You would need to work in the sub project memio/twig-template-engine for this.


Reworded docs for encoder::Writer::write_chunk()

Reworded a section from the write_chunk() method.
I felt as though the previous doc comment's wording was somewhat awkward. I added the size limit as well based on both the code from the function as well as from the png spec.

I don't see where it panics. It reports Err if the length can't be encoded.

I don't see where it panics. It reports Err if the length can't be encoded.

Oh I missed that, I'll edit it.


Impossible to render on canvas.

It's literally impossible to render on canvas instead of video.
this._canvas is not setted and after the procedures on video it is checked
this._canvasParent is only defined if video is also defined, the documentation clearly states one is options when the other is defined (and for video it is actually true).
if (this._video) {
  this._canvasParent = document.createElement('div')
  this._canvasParent.className = 'JASSUB'
  this._canvasParent.style.position = 'relative'

  if (this._video.nextSibling) {
    this._video.parentNode.insertBefore(this._canvasParent, this._video.nextSibling)
  } else {
    this._video.parentNode.appendChild(this._canvasParent)
  }
} else if (!this._canvas) { // Checking thos this._canvas before it's even set..
  this.destroy('Don\'t know where to render: you should give video or canvas in options.')
}

The condition can be replaced by something like this:
if (options.canvas || options.video) {
  this._canvasParent = document.createElement('div')
  this._canvasParent.className = 'JASSUB'
  this._canvasParent.style.position = 'relative'
} else {
  this.destroy('Don\'t know where to render: you should give video or canvas in options.')
}

if (this._video) {
  if (this._video.nextSibling) {
    this._video.parentNode.insertBefore(this._canvasParent, this._video.nextSibling)
  } else {
    this._video.parentNode.appendChild(this._canvasParent)
  }
}

later, setVideo is called with or without a video element
this._boundResize = this.resize.bind(this)
this._boundTimeUpdate = this._timeupdate.bind(this)
this._boundSetRate = this.setRate.bind(this)
this.setVideo(options.video)

Erroring with this message:
Caused by: "Video element invalid!"
which can be solved by adding a check

ah yeah, I never actually bothered with rendering canvas only, never really tested it, so didn't even go down that chain, I'll look at it another time, I'm currently having a fist fight with Electron, as it really doesn't seem to like requestVideoFrameCallback or more like the data it reports isn't vsynced which is pain

I have made a pull request that solves this issue. [#13]

I'm well aware

see latest version, it's not on NPM yet, so you're gonna need to get the dist files from gh

Appear to fix the issue, sadly i'm unable to test right now.
About your electron issue... give Tauri a try.

https://github.com/ThaUnknown/jassub/commit/46fabb90578322b0df848edbb7b4db2931517a57


Proxy not forwarding messages

Not sure if this is a bug or not, but would appreciate feedback on the following Stack Overflow thread:
http://stackoverflow.com/questions/43278998/python-3-smtp-aiosmtpd-proxy-relay
Thanks.

There seems to be several bugs in the Proxy handler.
handle_DATA seems to assume that envelope is a (unicode) string, but it's actually a bytes object.
_deliver contains log.exception('got', e.class) as if you could use the same parameters as in print(), instead of log.exception('got %s', e.class)
My quick fix was as below:
20d19
< EMPTYSTRING = ''
101c100
<             if NLCRE.match(line):
---
>             if NLCRE.match(line.decode('ascii')):
105,106c104,105
<         lines.insert(i, 'X-Peer: %s%s' % (session.peer[0], ending))
<         data = EMPTYSTRING.join(lines)
---
>         lines.insert(i, b'X-Peer: %s%s' % (session.peer[0].encode('ascii'), ending))
>         data = b''.join(lines)

I've answered your SO post and opened PR #74.

Your solution does seem to sort out the issue.
Thanks!


PR Template Change

Description
In PR Template,
### Issues Solved

[//]: # (List Issue Numbers)

needs to be changed to
### Issues Solved

Fixes #{Issue Number}

[//]: # For eg. `Fixes #5`

Required Information

[x] Provide a minimal code snippet / example that reproduces the bug.
[ ] Provide screenshots (if required)
[x] Provide github revision

The contributor prediction failed because there are not enough contributors. Three contributors must have authored comments on at least three issues each.


feat: Validate config in extendConfig (closes #32)

This adds validations of config after they've been defaulted in the extendConfig function.
This is a prerequisite to finishing #25

@ItsNickBarry any thoughts on this approach?

I apologize for the delay, haven't had time to think about this in depth.  My first thought is that it doesn't address #32  because that issue refers to moving the existing instances of throw HardhatPluginError to the extendConfig function (still not completely convinced that I want to do that, however).  Seems like a good change on its own, though.

Sounds good. I'll only reference the issue instead of closing it.


Feature: Add PWA functionality to the Default template

Purpose:
Adding PWA functionality would make it installable. The minimum requirements for a PWA force it to also add a service worker, which could set a foundation for eventually offline support and more dynamic pages with JavaScript.
Solution:

[x] Add manifest file
[x] Add the required images
[x] Add service-worker

Alternatives considered:
N/A
Additional context:
The site is currently using full round-trips for each interaction which render the service-worker kind of useless, but that's OK for an initial step.

Done


TypeError: init() missing 21 required positional arguments: [When running test.py and read.py]

I am getting the following errors when I am trying to reproduce the code, and run test.py and read.py
Additional keyword arguments: {'charset_test': '0123456789abcdefghijklmnopqrstuvwxyz'}
Traceback (most recent call last):
  File "test.py", line 133, in <module>
    main()
  File "/home/ashishpapanai/miniconda3/envs/parseq/lib/python3.7/site-packages/torch/autograd/grad_mode.py", line 28, in decorate_context
    return func(*args, **kwargs)
  File "test.py", line 87, in main
    model = load_from_checkpoint(args.checkpoint, **kwargs).eval().to(args.device)
  File "/home/ashishpapanai/Desktop/parseq/strhub/models/utils.py", line 85, in load_from_checkpoint
    model = ModelClass.load_from_checkpoint(checkpoint_path, **kwargs)
  File "/home/ashishpapanai/miniconda3/envs/parseq/lib/python3.7/site-packages/pytorch_lightning/core/saving.py", line 161, in load_from_checkpoint
    model = cls._load_model_state(checkpoint, strict=strict, **kwargs)
  File "/home/ashishpapanai/miniconda3/envs/parseq/lib/python3.7/site-packages/pytorch_lightning/core/saving.py", line 203, in _load_model_state
    model = cls(**_cls_kwargs)
TypeError: __init__() missing 21 required positional arguments: 'charset_train', 'max_label_length', 'batch_size', 'lr', 'warmup_pct', 'weight_decay', 'img_size', 'patch_size', 'embed_dim', 'enc_num_heads', 'enc_mlp_ratio', 'enc_depth', 'dec_num_heads', 'dec_mlp_ratio', 'dec_depth', 'perm_num', 'perm_forward', 'perm_mirrored', 'decode_ar', 'refine_iters', and 'dropout'

Using the following command helps:
./read.py pretrained=parseq refine_iters:int=2 decode_ar:bool=false --images demo_images/*
Is there any way to specify the weights path? Instead of using the weights provided by the author?

The pretained parameter is provided for easy fine-tuning of the released weights. If you want to load your own checkpoint, use the ckpt_path parameter. This expects a Lightning checkpoint (weights + training Paramus). The released .pt files contain the model weights only.
For any other use-case, you can just manually edit the code. See the create_model() function for more info.

Thanks @baudm,
How to use ckpt_path, is there any documentation for it?
How to obtain the lightning parameters?

I think you're using a wrong checkpoint. You can check by manually loading the checkpoint using torch.load and checking the values stored in the hyperparameters key.

This works, Thanks @baudm


feat(layer-cache): writes are gossiped

This implements the read and write side of the network cache update protocol. As things are written to disk, we also publish them to a nats jetstream topic, which all instances of LayerDB are listening to. When they get a write, they automatically place it in their memory and disk caches, making sure that data is hot in memory when possible.
As a side effect, we have changed the lower level interface of the caches so that their keys are always strings. This helps with the internal implementation, but will also help with debugging in the future (it wouldn't be super fun to look at a database full of raw bytes for primary keys). It simplified the code a bunch, and doesn't seem to have had any meaningful impact on performance.
Test coverage for the gossiping of writes is included in the integration tests for the casdb implementation.

bors merge

bors merge


Add autocomplete for file, class, and function names

Using ctags (#66 ) to get relevant names, autocomplete them as the user types. Right now we have autocomplete for prompt history, but that's just for the entire request the user types, not on the word level. I'm not sure what switching that will look like.
This will probably have some conflicts with #59 , since that also changes user input. Might be good to merge that first.

Made a draft PR for this here #79! Will be waiting for #59 to be merged since the auto completions for commands and for code syntax share the same python object for prompt_toolkit sessions.


Change Responsive Things.

Update apps user interface, login form, signup form, main activity data show form and navigation drawer form etc.
you can review my code and marge it.
Thank you
Md. Zobayer Hasan Nayem
CEO of Trodev.

Oky I will check it 😊


Plugin should not spawn VMs when all VM are "Scheduled to stop"

We noticed a behavior where

if we limit to e.g. 3 VM
when all 3 VM are "scheduled to stop"
if demands picks up (of if we click "start")

The plugin would deploy a 4th VM
It would be nice if the plugin would wait for those that are "scheduled to stop" to completely stop and start them instead of deploying extra VMs beyond the limit set (in this example 3)
Thanks!

@btdevopsteam, it's a common for all cloud integrations in TeamCity and I suspect that not everyone could agree to wait until all build agents will be stopped which could cause delays for queued builds.
So could you please report it in the TeamCity issue tracker and we'll try to evaluate it?

Please allow me to give a better example

5 agents max
5 running
1 was idling and the plugin tries to stop it so it ends up as "scheduled to stop", this is expected and perfectly fine
at this time the build queue grows and the plugin then deploys a 6th VM :'(

IMHO, if we wait for that VM to stop then start it, we could avoid getting a newly deployed VM when existing VMs are "scheduled to stop"
The total number of stopped VM could increase well beyond 5 if such scenarios repeat

@btdevopsteam, ok, thanks for clarification. You could try to update to the latest plugin build where it was fixed.

Thanks! It works "Scheduled to stop" is considered as active and would not allow additional agents to be spawned


_cascade=delete isn't working

Describe the bug
Patrick Werner: Just tried to use ?_cascade=delete with hapi (jpastarter) 5.3.0, which isn't working.
I tried it with DiagnosticReports and Observations {{baseurl}}/DiagnosticReport/24591/?_cascade=delete
Patrick Werner: This only deletes the DR, not the Observation.
HAPI returns: "succesful deleted 1 resource(s)"
To Reproduce
Steps to reproduce the behavior:

create Observations
create a DiagnosticReport linking to these Observations
DELETE [baseURL]/DiagnosticReport/[ID]?_cascade=delete isn't working

Expected behavior
Diagnostic Report AND Observations are deleted
Environment (please complete the following information):

HAPI FHIR Version 5.3.0
JPA Starter Project

zulip thread:
https://chat.fhir.org/#narrow/stream/179167-hapi/topic/_cascade.3Ddelete.20isn't.20working

It seems, this also applies for the cascade/expunge-combination:
https://chat.fhir.org/#narrow/stream/179167-hapi/topic/JPA.20Server.20-.20Cascade.20Delete.20with.20Expunge/near/230875113

@patrick-werner, I've observed something similar: the response only identifies the one resource (e.g. DR in your case) but does not identify the resources to which the delete was cascaded (e.g. OBS in your case). However, the logs did show the delete being cascaded, and all of the expected resources were deleted.
Is this what you are observing or are the Observations truly not deleted?
Attn. @KevinDougan-SmileCDR as I understand he is working on the issue I've described.

Also, are you able to replicate this on hapi.fhir.org?
I tested cascade deletes for a demo about a week ago and it was working for me..

@patrick-werner, I've observed something similar: the response only identifies the one resource (e.g. DR in your case) but does not identify the resources to which the delete was cascaded (e.g. OBS in your case). However, the logs did show the delete being cascaded, and all of the expected resources were deleted.
Is this what you are observing or are the Observations truly not deleted?
Attn. @KevinDougan-SmileCDR as I understand he is working on the issue I've described.

no in our case the Observations were still on the server.

Also, are you able to replicate this on hapi.fhir.org?
I tested cascade deletes for a demo about a week ago and it was working for me..

Tested it on the public server, still failing. I also created a test case for you @jamesagnew @dmuylwyk :
http://hapi.fhir.org/baseR4/DiagnosticReport/2004297

Also, are you able to replicate this on hapi.fhir.org?
I tested cascade deletes for a demo about a week ago and it was working for me..
@jamesagnew @dmuylwyk

Yes. And i am still able to reproduce it. Steps to reproduce:
DELETE: http://hapi.fhir.org/baseR4/DiagnosticReport/2004297?_cascade=delete
GET: http://hapi.fhir.org/baseR4/Observation/2004286
The observations aren't deleted.

@jamesagnew @dmuylwyk
i created a test case for this issue, which reproduced the issue:
https://github.com/hapifhir/hapi-fhir/commit/6eb23162f3122b618e3d7bf6e4fedc8ebdb2133f#diff-230ae6f5a7f424b1eb888c50086db45dc9ed4d2b7a5bc20fe0a8b6c887f59431

@patrick-werner  is this solve or it is still open?

@MCoderIndia I observed this recently.


Why no HAL_PCDEx_PMAConfig for H7xx?

I'm trying to port some (MIDI) USB code written to make a STM32f1xx as USB device. I need the code to run on a STM32h743. The first board invokes HAL_PCDEx_PMAConfig during initialization, but of course there isn't such a thing for h7xx.
Why is it so, and what to do instead?
As a first try, I just avoided the call and the H743 USB device is recognized as configured, but no data can be transferred. Invocations to USBD_LL_Transmit on the board return USBD_OK but no data is ever received by the host.
If relevant, this is with a Nucleo-H743ZI2

It sounds like you're using ST's HAL. Can you explain how you're using that with this project?

Thanks for getting back to me.
I am trying to port this project: https://github.com/Hypnotriod/midi-box-stm32 @Hypnotriod
After some trying, and without calling HAL_PCDEx_PMAConfig, I eventually got it to work. So to answer my own question, that function is not needed for h7xx and no need to do anything instead (not sure if this is true in general, but it seems to be the case for what I am doing).
I guess my main gripe is how much different are the APIs for doing the same thing (in this case simply initializing USB) between "very similar" boards (or maybe the gripe is that these boards should not be treated as "very similar", e.g. by marketing and sales)

Thanks for the discussion. This question seems to be resolved so I will close the issue.


Add Stackdriver

         "project_id": "string",
        "metric_prefix": "string",
        "default_labels": { "key": "value" }
}

project_id is the gce project ID
metric_prefix is the prefix to append to the metrics
default_labels are key value pairs that are added to the metrics (so you can filter)


[Feature Request] Floating Labels

Existing Component
Yes
Component Name
Input
Description
In the Sketch files it shows inputs with "floating labels", however I can't find any documentation on this.  Is this implemented, if not it would be a great feature.
I'm attaching a screenshot of the input, it was listed under "Special Usuage"

Been looking for a floating-label for element. I've been toying around with this;
https://github.com/bkzl/vue-float-label
which works fine for Input, not at all for el-select and date...

select and date are wrapped in <div>, you may need to modify vue-float-label to get is working with them.

Sorry, we have no plan to support this.