Fix for CI - Keep using older natural_earth version for testing

Natural Earth recently released a new version that changed some of the marine regions (e.g. #208), leading to incompatibility issues.
Here I am using some new regionmask features to make sure that the older version is used for testing.
This is an attempt to quickly fix the CI (which has failed for the last few weeks), but I am wondering if it would be worth in the long run, to upgrade to the latest NE version.
Related question: @mathause, is there a way to identify the NE version from a regions object?
Something similar to this:
basins = regionmask.defined_regions.natural_earth_v4_1_0.ocean_basins_50
assert basins.version == 'something'

?
That way I could check that in my module and give a more useful error message
TODO:

[ ] Mention this in the docs. Link to here

Ill merge this and add an issue to remind me to add this to the docs

Related question: @mathause, is there a way to identify the NE version from a regions object?
Something similar to this:
basins = regionmask.defined_regions.natural_earth_v4_1_0.ocean_basins_50
assert basins.version == 'something'

No unfortunately not - I didn't think of this but it's a good suggestion. version would only be relevant for natural earth regions at the moment, but I guess I can just default it to None.


--service-account not working - 403 when fetching each individual message id

The issue tracker is for reporting product deficiencies. How do I questions should be posted to the discussion forum at https://groups.google.com/group/got-your-back. When in doubt, start at the discussion forum and return here only when instructed to do so.
Please confirm the following:

I have upgraded to the latest GYB release from https://github.com/jay0lee/got-your-back/releases and I still have this issue.
I am typing the command as described in the GAM Wiki at https://github.com/jay0lee/got-your-back/wiki

Full steps to reproduce the issue:

Run gyb --email<EMAIL_ADDRESS>--service-account

Expected outcome (what are you trying to do?):
Backup a google user's email account
Actual outcome (what errors or bad behavior do you see instead?):
Successfully requests a list of message Ids but fails when requesting each message id.
<HttpError 403 when requesting https://www.googleapis.com/gmail/v1/users/me/messages/1*>

I am having the same problem I have created a new project and I still have the same issue..
It did work 2 weeks ago,

Ok, it works only when the account you are trying to backup is not suspended.

 Though it worked on suspended accounts last week. Did something change?

I just opened a ticket with Google as transferreing data from a Gsuite account when you delete it is also not working, you will have to enable the account and use the transfer ownership tool on the drive admin for it to work. So looks like an issue on Google's side.

Facing the same issue -- error when attempting to backup suspended gsuite user accounts.
Seems like google changed something on their end in the past month (I ran into this issue issue early last week).

checking with Google to see if there was a change here but I have no timeline on a response. For now I suggest resetting user's password and locking down the account rather than suspending while GYB backup is occurring.

Hi all, this seems to be working again, just did a backup on a suspended account and it didn't come up with any errors.

Hi all, this seems to be working again, just did a backup on a suspended account and it didn't come up with any errors.

Just tested on a suspended account as well -- I can confirm backup on a suspended account now works correctly.


Feature/new way to setup

Mackup Logic added

Sorry, I made this PR on the wrong repo. Please reject and ignore it.
Thanks

You can close your own PR with the Close button below


Initial draft of the app

Hi guys, this is an initial draft of the app.
The first two tabs are the working ones at the moment.
Please review it and let me know what you think of it.
It took ~10 hours to build this.

To address your points

Although it is built as a package, this app is not meant to function like that unless absolutely necessary.
This is due to the fact that setting it up as a package will require some extra work which is not in out interest at the moment.
So I'd say it's best to treat it as an 'ordinary' shiny app instead of an R package as of now.
I'll add a file that will let the user run the app by clicking a 'Run App' icon on top of the file, just like a standard shiny app. And it will also let us deploy on shinyapps.io like other shiny apps.
I'll take of the rest of the points Inn Shaa Allah.

Great, thankyou - let me know when done and I'll merge changes & deploy.

App deployed to shinyapps.io
https://shahreyar-abeer.shinyapps.io/livinghtademo/

Awesome, I can now deploy the same app on shinyapps.
However both my version (https://rasmithbresmed.shinyapps.io/app_golem/) and your version breaks when attempting to run the model :

Any ideas on why this may be?

Okay, problem fixed.
Please give it a try now (pulling the latest changes).

Awesome, thanks that works.


optimize CodeStream.iter

a couple fixes:

CodeStream.program_counter was a hotspot but not in __slots__
avoid write-back for local pc variable

What was wrong?
How was it fixed?
Todo:

[ ] Clean up commit history

[ ] Add entry to the release notes

Cute Animal Picture

(a bit unclear to me that avoiding the pc writeback is better than having a local variable.)


Ex 3: How to add footnotes

To add this to the project for Exercise 3, go to that project, and then click add cards and then drag this issue to the todo column


Launcher crash on tray startup

log.txt

This is also happening to me on 17.04. Here's my log file:
subutai-launcher-ubu1704.txt

Fixed


Script should exit with a non zero status on error

To use this script with CI tools it would be useful if it could exit with a non zero status in case it detects errors.

It does exit with a non-zero code in case of errors. Could you specify which version are you using?

I confirm this, for me it exits with 0 when errors are present. I'm using version 0.3.0 installed from pip, (shell is zsh, but I don't think it is related).


Query about GIFs

Question about an existing feature
What are you trying to achieve?
I'm trying to achieve an image which is 'pasted' on a gif for every frames using Sharp Library
const sharp = require('sharp')

const baseGIF =
"test.gif";

const baseImage = "unknown.png"

sharp(baseImage)
.resize(10,10)
.toBuffer().then(buffer => {
    const overlay = [
        {
          input:
          buffer,
          tile: true, gravity: 'northwest'
        },
      ];

      sharp(baseGIF, { animated: true })
      .composite(overlay)
      .resize(100, 100)
      .toFile("output.gif", (err) => {
        if (err) {
          console.log("error: ", err);
        }
      });

})

I get returned this, with many of the same images.
I'm trying to make a image card with a photo of someone in the left, and the background is a GIF.
How can I also position the image?

Does removing the following line produce the expected output (albeit at the wrong size)?
-       .resize(100, 100)

Unfortunately, it does not. I did the resize on the GIF so it looks more appropriately like a GIF card.

These are my assets:

(the gif)

(image)
I just want one of the image in a specific area, like the middle or left. Only one, not multiple. How can I achieve this?

@lovell
Pretty much solved this problem.
However, is there a way you can like composite/dissolve a gif over an image?
For example :
A flag of america 'merged' or dissolved into an image making the image look like an american

figured out


Connecting a signal to a method with default arguments with binds to a subclass with the Connect dialog fails

Godot version
4.2b1
System information
Linux/Manjaro
Issue description
Given three classes: Class A (extends Node, has a signal with one argument), Class B (extends Node, has a method with two arguments one of which is optional), and Class C (extends Class B),  attempting to connect Class A's signal to Class C's inherited method with binds using the Connect dialog will fail with the message Cannot connect to '<signal>': the provided callable is not valid: Node(<script.gd>)::<method> printed to the output. This does not occur for connecting Class A's signal with binds to Class B's method.
Given how specific this is, I'm not sure if this is a regression, but #66313 might have caused this.
Steps to reproduce

Create a node class with a signal that contains one argument (Class A)
Create a node class with a method that takes two arguments, one optional (Class B)
Extend Class B (Class C)
Put instances of Class A and Class C in a scene
Try to connect Class A's signal to Class C's method, binding one extra argument
The output will print Cannot connect to 'signal': the provided callable is not valid: Node(ClassB.gd)::method

Minimal reproduction project
bind-bug.zip

Open main.tscn
Select NodeWithSignal
Try to connect its foo signal to SubNodeWithMethod's method baz, binding an extra argument
Observe the error in the output

Also a method with additional default argument(s) is not listed as a compatibile one (even for the base class in the MRP):


Improve Format trait docs

Closes https://github.com/knurling-rs/defmt/issues/330

bors r+

bors r+

bors retry

bors retry


vault and external secrets operator in differenet k8s cluster can access forbiden.

Describe the solution you'd like
Describe the end goal of this proposal. What is this new functionality or the new behaviour (or what problem does it fix)?
vault and external secrets operator in differenet k8s cluster can access forbiden.
What is the added value?
Explain the value that it adds. e.g. "Secret refreshing will make internal secrets up to date with external secrets".
Give us examples of the outcome
Provide templates if you are proposing changes in the CRD. Provide example workflows or code snippets if they make sense to present.
Observations (Constraints, Context, etc):
Give here all extra information that could be interesting. Such as Golang version and Kubernetes version if you are reporting a bug/problem. You can also foresee technical constrains like "this could only be implementing using this specific technology or approach, because of this and that".

Hey @13567436138 could you please elaborate a bit on the issue you're facing? e.g. what are you trying to do? Can you provide yaml manifests and share the error that you have observed?

secret/ap-project-clusterResourceWhitelist.yaml
kubectl apply -f ap-project-clusterResourceWhitelist.yaml -n argocd
apiVersion: argoproj.io/v1alpha1
kind: AppProject
metadata:
  name: project
spec:
  clusterResourceWhitelist:
  - group: "*"
    kind: "*"
  sourceRepos:
    - https://gitee.com/hxpgit/argocd.git
  destinations:
    - server: https://<IP_ADDRESS>:6443
      namespace: demo
    - server: https://<IP_ADDRESS>:6443
      namespace: demo

secret/app-external-secret.yaml
kubectl apply -f app-external-secret.yaml -n argocd
apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: app
spec:
  destination:
    server: https://<IP_ADDRESS>:6443
    namespace: demo
  source:
    repoURL: https://gitee.com/hxpgit/argocd.git
    path: demo1/secretManager/external-secret
    targetRevision: HEAD
  project: project
  syncPolicy:
    automated:
      prune: true
      selfHeal: true

apiVersion: external-secrets.io/v1beta1
kind: ClusterSecretStore
metadata:
  name: secretstore
  namespace: demo
spec:
  provider:
    vault:
      server: "http://<IP_ADDRESS>:31758"
      path: "kv"
      version: "v2"
      auth:
        kubernetes:
          # Path where the Kubernetes authentication backend is mounted in Vault
          mountPath: "kubernetes"
          # A required field containing the Vault Role to assume.
          role: "myapp"
          # Optional service account field containing the name
          # of a kubernetes ServiceAccount
          serviceAccountRef:
            name: external-secrets
            namespace: external-secrets

[root@member1-153 ~]# kubectl get clustersecretstores.external-secrets.io -n demo secretstore -o yaml
apiVersion: external-secrets.io/v1beta1
kind: ClusterSecretStore
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"external-secrets.io/v1beta1","kind":"ClusterSecretStore","metadata":{"annotations":{},"labels":{"app.kubernetes.io/instance":"app"},"name":"secretstore"},"spec":{"provider":{"vault":{"auth":{"kubernetes":{"mountPath":"kubernetes","role":"myapp","serviceAccountRef":{"name":"external-secrets","namespace":"external-secrets"}}},"path":"kv","server":"http://<IP_ADDRESS>:31758","version":"v2"}}}}
  creationTimestamp: "2023-04-17T14:43:04Z"
  generation: 1
  labels:
    app.kubernetes.io/instance: app
  name: secretstore
  resourceVersion: "3811711"
  uid: 17b3ab64-db1f-412a-a017-0378398301b0
spec:
  provider:
    vault:
      auth:
        kubernetes:
          mountPath: kubernetes
          role: myapp
          serviceAccountRef:
            name: external-secrets
            namespace: external-secrets
      path: kv
      server: http://<IP_ADDRESS>:31758
      version: v2
status:
  conditions:
  - lastTransitionTime: "2023-04-17T14:43:04Z"
    message: unable to create client
    reason: InvalidProviderConfig
    status: "False"
    type: Ready

{"level":"error","ts":1681742666.7798572,"msg":"Reconciler error","controller":"clustersecretstore","controllerGroup":"external-secrets.io","controllerKind":"ClusterSecretStore","ClusterSecretStore":{"name":"secretstore"},"namespace":"","name":"secretstore","reconcileID":"d50e801b-e327-48a8-a55e-0eef32de87e3","error":"could not get provider client: unable to log in to auth method: unable to log in with Kubernetes auth: Error making API request.\n\nURL: PUT http://<IP_ADDRESS>:31758/v1/auth/kubernetes/login\nCode: 403.  The issue is that you configured your ClusterSecretStore to use external-secrets/external-secrets service account, where your Vault role is actually configured for demo/default. If you fix the permissions your ClusterSecretStore will stop providing any errors.
You can see that is the case with the following message here:
"could not get provider client: unable to log in to auth method: unable to log in with Kubernetes auth: Error making API request.\n\nURL: PUT http://<IP_ADDRESS>:31758/v1/auth/kubernetes/login\nCode: 403. Errors:\n\n* permission denied"

I can not access even I changed to
vault write auth/kubernetes/role/myapp \
   bound_service_account_names=external-secrets \
   bound_service_account_namespaces=external-secrets \
   policies=app \
   ttl=1024h

because my external-secret and vault is in different k8s cluster

When vault lives in a different k8s cluster you need to ensure that your reviewer_jwt has the permission in the other cluster (that jwt probably needs to come from that cluster, not the vault k8s one) and that the k8s host and ca are correct.
You can check the vault logs, that should point you to the exact issue.
vault write auth/kubernetes/config \ token_reviewer_jwt="$(cat /var/run/secrets/kubernetes.io/serviceaccount/token)" \ kubernetes_host=https://${KUBERNETES_PORT_443_TCP_ADDR}:443 \<EMAIL_ADDRESS>

I tryed k8s host and ca from external-secrets cluster，but failed。token_reviewer_jwt should come from which token from external-secret cluster？

Looking at the docs i would:

leave token reviewer jwt empty
set disable_local_ca_jwt=true
configure kubernetes ca crt and host

That way you can use the jwt presented during authentication to verify the token itself. Though be sure to attach the necessary RBAC permissions to that service account.

solved


Causal Analysis

What is it?
Generally speaking, we need a better way to think about how to show that funding OSO metrics has the intended effect, with retrofunding being the largest experiment to date.
To start with, we should have time-series rolling windows on all metrics so that we can show change pre/post funding.
We should also be monitoring it over time, whether there is any stickiness.
In the future, we might consider controlled experiments to factor out other causes.
Also related:

Relentless monetization

https://kermankohli.substack.com/p/arbitrums-85m-growth-campaign?r=30w0j0&utm_campaign=post&utm_medium=web&triedRedirect=true

There's lots of materials on causal inference:

https://matheusfacure.github.io/python-causality-handbook/landing-page.html
https://www.hsph.harvard.edu/miguel-hernan/causal-inference-book/
https://aayushmnit.com/posts/2022-09-19-SyntheticControl/2022-09-19_SyntheticControl.html

Which is related to the broader field of econometrics

https://www.aeaweb.org/conference/cont-ed/2017-webcasts
https://www.aeaweb.org/conference/cont-ed/2020-webcasts
https://www.mostlyharmlesseconometrics.com/
https://www.masteringmetrics.com/

Starting some thoughts here
https://docs.google.com/document/d/1S7jqzB_JkAyygZ7z3z6m5TVmBZWza7ngMlrrWASs0wA/edit

As per convo with @ccerv1 , he's going to cover this in his talk and in the blog post. I'm going to move this back to the previous sprint and close it out as the initial exploration

https://www.openblocklabs.com/research/arbitrum-ltipp-efficacy-analysis


Add support for extension downloads

It is now possible to download extension data as-is (not indexed) through the API: https://techdocs.gbif.org/en/data-use/api-downloads#verbatimextensions. This should be documented and supported in pygbif.

closing as duplicate


regen lockfile

regenerates the lockfile, maybe this will make nix happy

@cameron1024 Should we close this PR?


Mypy static typing

Get mypy static typing set up
appropriate lists of exclude and include files
(optional) incorporate into CI pipeline
Get mypy compiling on a single file

Looked into static typing alternatives. Not great for this kind of project. None of the libraries I'm using have application stubs for MyPy (Numpy, OpenCV, etc.)


Wrong results using checkpoints for Bart Program baseline

Hi (@shijx12 @ShulinCao @teacherpeterpan @Flitternie @stellarkey),
thank you for your work.
I have some issue using the code base, can you kindly help me?
I don't know if the training works, but when I try to restore the checkpoint the predict.txt is full of "no".
My requirements.txt is the following:
transformers==4.0.0
torch==1.6.0
sentencepiece==0.1.96
nltk==3.7
numpy==1.19.2

The steps that I perform (from the KQAPro_Baseline/ folder [the root one]) are the following:
python3 -m Bart_Program.preprocess --input_dir data/ --output_dir preproc_data/ --model_name_or_path KQAPro_ckpt/program_ckpt/
cp data/kb.json preproc_data/
python3 -m Bart_Program.predict --input_dir preproc_data/ --save_dir log_folder --ckpt preproc_data/

Thank you in advance.
Kind regards,
Andrea

How is you training process?
I recall there is a validation step while train. Are those statistical measures normal?
When I am running Bart_Program I also had a accuracy problem (the acc is very low). It turns out to be kb.json's problem.
Take a look of https://github.com/shijx12/KQAPro_Baselines/blob/master/utils/load_kb.py#L9:
"""
knowledge json format:
    'concepts':
    {
        'id':
        {
            'name': '',
            'instanceOf': ['<concept_id>'],
        }
    },
    'entities': # exclude concepts
    {
        'id':
        {
            'name': '<entity_name>',
            'instanceOf': ['<concept_id>'],
            'attributes':
            [
                {
                    'key': '<key>',
                    'value':
                    {
                        'type': 'string'/'quantity'/'date'/'year'
                        'value':  # float or int for quantity, int for year, 'yyyy/mm/dd' for date
                        'unit':   # for quantity
                    },
                    'qualifiers':
                    {
                        '<qk>':
                        [
                            <qv>, # each qv is a dictionary like value, including keys type,value,unit
                        ]
                    }
                }
            ]
            'relations':
            [
                {
                    'predicate': '<predicate>',
                    'object': '<object_id>', # NOTE: it may be a concept id
                    'direction': 'forward' or 'backward',
                    'qualifiers':
                    {
                        '<qk>':
                        [
                            <qv>, # each qv is a dictionary like value
                        ]
                    }
                }
            ]
        }
    }
"""

Be sure your kb.json follows the exact statements above. (concept instanceOf, relation predicate, ...)

Hi @stellarkey,
Thank you for your quick response.
The kb.json file I am using is the one that can be downloaded from here (http://thukeg.gitee.io/kqa-pro/index.html).
This is a portion of that file (the file is too long to copy and paste, sorry for that ahah)
{"concepts": {"Q7270": {"name": "republic", "instanceOf": ["Q7174"]}, .... "entities": {"Q786": {"name": ..., "instanceOf": ..., "attributes": ...
I have not modified the file in any way.
I cloned the repository, downloaded data.zip and checkpoints, and tried the commands I mentioned above.
Can you share the right kb.json?
Thank you for your support.
Kind regards,
Andrea

Sorry for the delay.
I uploaded it to google drive.

The newly updated version fixed the problem. Please check the new checkpoint file and the change log.

Thank you for your help.
Please can you also update the bart-base checkpoint? (The non fine tuned version)?

We did not change the bart-base model, which is downloaded from huggingface.


Add documentation for creating large kubernetes clusters

Closes #10779 (adds guidance for addon container scaling)
CC @dchen1107 @satnam6502

Perhaps show a few kubectl commands which diagnose a pod being repeatedly killed because it execeeds its resource limits? Just to be concrete?

Thanks for the feedback @satnam6502
PTAL

Apart from the mis-aligned headers looks great to me and still LGTM.

Apart from the mis-aligned headers...

Sorry I'm probably being dense here, but could you clarify

Events:
  FirstSeen             LastSeen            Count   From                    SubobjectPath               Reason      Message
  Tue, 07 Jul 2015 12:53:51 -0700   Tue, 07 Jul 2015 12:53:51 -0700 1   {scheduler }                        scheduled   Successfully assigned simmemleak-hra99 to kubernetes-minion-tf0f
  Tue, 07 Jul 2015 12:53:51 -0700   Tue, 07 Jul 2015 12:53:51 -0700 1   {kubelet kubernetes-minion-tf0f}    implicitly required container POD   pulled      Pod container image "gcr.io/google_containers/pause:0.8.0" already present on machine

Of course, I suspect this is just what kubectl prints out.

Got it. Yes, that's what kubectl prints. But aligned for clarity.

@k8s-bot ok to test

The "detecting resource starved containers" section partially fixes #10108, specifically the request for a section about Setting CPU and Memory Limits

@erictune Split the `Detecting Resource Starved Containers" section out into a separate doc.
PTAL

@satnam6502 @erictune Can you PTAL when you get a chance
The 1 failed E2E test looks like a flake

LGTM. Thanks.

try to merge your troubleshooting with the section in compute_resources.md that talks about "OOM Killed" Reason.

@erictune PTAL

See cluster admin doc reorg in #11013.
Will need to be rebased once all of the new munging passes have been merged.

Ok, I'll rebase if #11013 gets merged first.

LGTM


setOldInputProvider for Check and Radio Inputs

Did the setOldInputProvider method not prefill values on radio and checkbox inputs?

Hmm it should, can you post an example? These tests are all passing: https://github.com/adamwathan/form/blob/master/tests/FormBuilderTest.php#L150-L187

I have posted a message related to that topic here:
https://github.com/adamwathan/form/pull/27


In Flutter Gallery, list items wiggle as they are tapped

Looks like a rounding error, maybe related to moving content onto layer?

Open "material" list (On Nexus5x running O)
Long press on any list item

Expected: Ripple highlight, nothing more
Observed: Ripple highlight, and the list item title offsets vertically by a single pixel

cc @cbracken

Closing as no longer relevant.


Font sizes on iOS seem off

I created some screenshots of a wonder screen with the iOS font size setting screen pasted in on top. It looks like the font size in the app is generally too small compared to the setting?

cc @jmagman to get her input

Looks pretty close? I don't have a preference, seems like a design choice with the custom font and not a bug since it did scale larger.

Here's a better comparison, where I also used a light background from the setting screen.
I can comfortable read the text from the settings screen, but it's very straining for me to read the wonderous text.

Thanks for the report! It does seem like the body text here is noticeably smaller than what is shown as an example in the device settings.
We do downscale our font sizes a bit on smaller devices to help the UI fit better. Maybe we should exempt the body text to better align with user expectations?
Any thoughts @jaredbell?

@esDotDev  Took a look into this across devices, platforms, etc. — I think the best option is to remove the concept of scaling down.
This was a safeguard for older, smaller devices but ultimately affects common/large sized ones as well as it's only looking at device pixel width.
Removing this brings the dynamic type scale back in line which is great for accessibility and user expectations.
We could retain the slight upscaling in place (1.15x-1.25x) as it tends to help fill out the screen a bit more on tablets and desktop/web with larger text/titles and more generous margins.

This has been addressed in v2.2.2 which should be available for download in test-flight.
@mit-mit The font size should be more like you expect now! Please let us know if this is not the case.

Font size looks great now, thanks!


Why not use Build API of esbuild instead of Transform API?

sometimes people may need to have a programmatic usage, such as reading the ts file on the runtime,and in that time they will need to use some feature only the Build API support, like Inject. and it can easily run a way like transform by stdin and other opions.Just what i say in my PR of esbuild-register ,evanw said he may remove the transform api in the future and have people use build api instand(in this esbuild issue),so why we don't choose the powerful one?
if you agree i would contribute this feature.

tsx is designed to enhance Node.js, instead of overriding behavior and delegating it to another program.
Using the esbuild build API would prevent Node.js features such as it's resolution algorithm, and being able to toggle it from working.
Besides, what you're describing sounds very simple to do via Bash pipe: esbuild ... | node
Not sure if I see the merit for it in a loader but if you want to abstract it into a Node.js module, you should consider a separate project for it.


skhd functionality missing for some time?

Will skhd functionality return for yabai? I found it very easy to config in the same way as setting rules in yabai. It was handy to have, but for the past several versions of yabai, it seems to have stopped working.
Was something affected in an upgrade that I should be changing to have this kind of functionality restored? Or are there plans to enable this in the future? Thanks in advance.

Nothing has changed; the architecture is the same as it has been since the very beginning. skhd is probably just not running correctly on your system for some reason. You should take a look here: https://github.com/koekeishiya/skhd/issues/48

The issue was an outdated payload addition.
My mistake, I tend to leave my monitor on my laptop turned off to avoid further "burning" of the monitor, and since the notification telling me the payload was outdated was on the screen that was turned off, I kept missing it. Will keep this in mind for future updates. Thanks!


Frequently error on generate songs

I always got error whenever i generated 5 requests or some time fews . i need to go to suno.com to recreate a song. so i can generate it again. is this a hCapcha problem or else ?

yes, I have the same issue as well. It's also noted in the readme that you have to do so


GestureHandler already initialized for root view

Hi there,
I'm basically trying to replicate native driver example from docs:
const PanAndRotate = () => (
  <PanGestureHandler onGestureEvent={Animated.event({ ... }, { useNativeDriver: true })}>
    <Animated.View>
      <RotationGestureHandler onGestureEvent={Animated.event({ ... }, { useNativeDriver: true })}>
        <Animated.View style={animatedStyles}/>
      </RotationGestureHandler>
    </Animated.View>
  </PanGestureHandler>
);

and get this error. Maybe I'm missing something, is there any working samples with native driver and Animated.Views? Without native driver seems it's working for me, but I wish to get more animation performance.

Would you mind give us a wider context of this issue? Where do insert PanAndRotate component?
I have that admit that this issue provided me to notice a bug in Example App (https://github.com/kmagiera/react-native-gesture-handler/pull/176)
If you want to see how it should work, you may checkout on this branch (or wait, it gonna be merged soon) and see "Scale, rotate & tilt"
Thank you for this issue.
Please let me know  if you still face the problem

Yes, thank you for you suggestion. I think you're right. Link to example I mentioned above is related to already fixed example.

@zinoviev,
Thank you :)


Support selection text in Vivaldi

I just deployed new version for supporting Chromium based browsers.
If you want to add your favorite browser, follow this steps.


Template - Outro

Feature
In addition to preparing and styling the templates used by the CLI, we need to provide information about the next steps for the developer.
Notably, we should detect if no satellite has been configured yet and prompt the developer to either start a local development environment or configure an existing satellite.
We should also detect or note that the CLI is required to deploy a project.

Resolved in 770afe8112259df608ef4ea20fdc4b6d32f5e6a9


Brightness adjustment missing for GU10 Led bulbs

Hi, I'm not really sure what happened, but when I first setup these sets of 5 gu10 wifi bulbs, I was able to adjust brightness
(on the master bedroom lights card there was percentage for brightness level)
But recently the brightness option is no longer available, and I can only toggle on/off only
I'm not quite sure the logs  output I should place here, anyone could guide me on that?

Show this devices in sonoff.json file

Closed because no response


support skipping reports or checks via ignore files

It could be handy to be able to ignore certain report variants and/or skip checks at all levels in a target repo.
While this could be done for individual versions via comments in ebuilds similar to programming language related linters, that precludes performing this at repo, category, package, and other levels. Instead it would be most configurable to support a defined config format and name, for example .pkgcruft-ignore, for a file that could be contextually placed in various locations.
The main downside of not supporting line-based ignoring is not being able to ignore reports from specific lines in files; however, this is not as prevalent in ebuild linters since most reports are contextually dependent at a higher level (for example, they depend on an ebuild's version, the package set, or even some profile settings in the repo). Also, if ebuilds are copied to overlays and proper pkgcruft reports are wanted, any related ignore files would have to be copied along with them.
Beyond the contextual upsides for a separate file approach, it would probably be more performant and support higher ignore visibility in an easier fashion. For example, the files could be read from the repo in parallel, constructing a restriction to setting mapping that would be used when determining checks to run and/or reports to output.

It should also be noted, if pkgcraft ever completes work on its own bash parser, then supporting line-based QA directives would be valid for the related subset of report variants.


Tween.evaluate breaks Law of Demeter

Tween.evaluate should take a double and not an Animation.

I believe you're looking for Tween.lerp(t)

Yeah I ended up with that in my case but I think my case still stands: Tween.evaluate only takes an Animation to get the value out of it.  It thus should take a double or maybe even not exist at all o_O

Yeah, we should audit the callers to see if how it is used.  I was probably influenced too much by the early examples I looked at.

I tried doing this but it's a lost cause. There's over 60 call sites just in the framework, if that's even slightly indicative of usage in the wild, it would break tons of customers.

@InMatrix While looking at this I also looked at the related issue of having a way to do something like:
   Animation<double> animation = new AnimationController(); // or some other Animation<double>
   Tween<Color> colorTween = new Tween<Color>(begin: Colors.red, end: Colors.blue);
   Animation<Color> result = animation.chain(colorTween);

The problem is that the Animation<T> class doesn't know if T is double, and if it isn't, this "chain" method wouldn't be type-correct. It's the same reason Tween in general chains backwards rather than forwards -- it needs to be able to guarantee the type of the parent in the chain, and itself cannot guarantee its own type.
Still looking at what can be done with this, but nothing obvious comes to mind.

@Hixie Thanks for looking into this. I took a look at the source code, and it looks like AnimationController is always an Animation. Can we give AnimationController a method called makeAnimation, which turns a given Animatable object into an Animation?
AnimationController controller = new AnimationController();
Animation<Color> colorAnimation = controller.makeAnimation(
    new Tween<Color> (begin: Colors.red, end: Colors.blue)
);

This is obviously a less general solution, but hopefully it works?

If we're willing to limit this to AnimationController, rather than any Animation<double>, then that's more doable, yes.

(BTW do we have a bug for this issue? I couldn't find one.)

No bug for this yet. It's on my todo list. I'll file one soon and move subsequent discussions there.

Filed: https://github.com/flutter/flutter/issues/14482
And I think it's sufficient to limit this to AnimationController. It's mainly for pointing users unfamiliar with the animation system to an easier starting point centered on the AnimationController.


Fixed node input

Fixes #1

Thanks @UWUplus! I appreciate it since I haven't had much time to work on this project among the myriad of others. For anyone else looking at this though, this may break the nodes for older blender versions, as I'm assuming this broke because the added a node to the principled shader in the newer versions. If you find a way to specify the nodes so that they won't break if their index changes, please fix and submit a pull request.

I'm not too much into Blender development but I'll dig trough their docs.


Webpack Code Splitting -> Remove Chunk Vendors

remove vendor chunk, make everything depend on main chunk, stuff like chart.js should only be used in its own file
http://<IP_ADDRESS>:8888/ to debug
see configureWebpack.optimization.splitChunks.cacheGroups for info

#314


spudguid

getting Ignoring runtime actor Test, missing or blank SpudGuid property
what is best way to set this when spawning through blueprints at runtime?

ok nevermind, it needed to be a  GUID struct type. thanks!

Yep, for future ref here's the doc on that: https://github.com/sinbad/SPUD#runtime-spawned-actors


devonfw/my-thai-star#DEVC-348: My Thai Star electron integration simplified

Added module ngx-electron

Hi @jambulud, please check this PR again as we have merged others and there are some pending conflicts.
Thanks!

Conflicto resuelto


Non-ASCII Parameters Not Working

#!/usr/bin/env python
# -*- coding: utf-8 -*-
import json, vertica_python

cnxinfo = {'host': 'verticadb01', 'port': 5433, 'user': 'user1', 'password': 'blahblah', 'database': 'mydb'}

cnx = vertica_python.connect(**cnxinfo)
crsr = cnx.cursor()

## This doesn't work
#crsr.execute(u'SELECT * FROM my_schema.my_table WHERE lang = %s ORDER BY id', (u'Español',))

# This works
crsr.execute(u'SELECT * FROM my_schema.my_table WHERE lang = \'Español\' ORDER BY id')

d = crsr.fetchall()
cnx.close()
print(json.dumps(d))

Perhaps it's related to the absence of apostrophes in the first (not working) example?
I would suggest to test whether surrounding the %s placeholder with apostrophes ('%s') fixes the broken example.
On Oct 4, 2016 6:20 PM, Chun-Kwong Wong<EMAIL_ADDRESS>wrote:
#!/usr/bin/env python
-- coding: utf-8 --
import json, vertica_python
cnxinfo = {'host': 'verticadb01', 'port': 5433, 'user': 'user1', 'password': 'blahblah', 'database': 'mydb'}
cnx = vertica_python.connect(**cnxinfo)
crsr = cnx.cursor()
This doesn't work
#crsr.execute(u'SELECT * FROM my_schema.my_table WHERE lang = %s ORDER BY id', (u'Espa?ol',))
This works
crsr.execute(u'SELECT * FROM my_schema.my_table WHERE lang = 'Espa?ol' ORDER BY id')
d = crsr.fetchall()
cnx.close()
print(json.dumps(d))

You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/uber/vertica-python/issues/130, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATyyDOJKSpt22zcxFPxqfYt_35cva0sDks5qwm6rgaJpZM4KNzki.

It works if we replace the 2nd parameter, which is a tuple, with (u'English',).

thanks for reporting this.
should be fixed as of 0.6.8.


SendPrimitiveAsync throws InvalidOperationException for enums

I've an endpoint which returns enum (C#):
return Ok(MyEnum.Value1);

The generated code for GetAsync looks like this:
return await RequestAdapter.SendPrimitiveAsync<MyEnum?>(requestInfo, errorMapping, cancellationToken).ConfigureAwait(false);

But it throws InvalidOperationException with message error handling the response, unexpected type
I can't find any workaround for it right now

@baywet Out of the scope of this issue, but if the server returns collection of enums, the generated client code is
return await RequestAdapter.SendPrimitiveCollectionAsync<MyEnum?>(requestInfo, errorMapping, cancellationToken).ConfigureAwait(false);
SendPrimitiveCollectionAsync internally calls GetCollectionOfPrimitiveValues https://github.com/microsoft/kiota-serialization-json-dotnet/blob/main/src/JsonParseNode.cs#L310 and it doesn't handle enums.
IParseNode exposes GetCollectionOfEnumValues and GetEnumValue, but they have constraint for T limited to enum. ModelType doesn't have any constraint.

Transferring issue as part of https://github.com/microsoft/kiota-abstractions-dotnet/issues/238


Career Path Tree

becoming a full stack developer path
Why?
Students who are new to programming don't know what to learn and in which order. It all seems overwhelming. This page will show them the path and if they follow it they'll be ready for interviews and jobs in a systematic manner.
Implementation
Something like this: https://www.w3schools.com/howto/howto_css_timeline.asp
Few features or changes:

Needs to be in React
Can read data from a JSON using gatsby plugin
Instead of years it will be technology/course name. Clicking on each leaf should take to a specific URL.

entries in JSON would like this:
{
    header: "html/css basics",
    intro: "learn basic tags and style before jumping into anything advanced",
    url: "/create-profile-course",
    icon: "someimage.png",
    status: "released"
},

{
    header: "data structures",
    intro: "now when you know all of this, start learning DS & Algo to prepare for interviews",
    url: "/ds-for-interviews",
    icon: "some-other-image.png",
    status: "upcoming"
}

We can make use of this: https://github.com/stephane-monnot/react-vertical-timeline
I can work on this.
I guess then we can have file structure like this:
- data
   - careerpaths
     - fullstackdev.json
   - careerpathleafs
     - fullstackdev
        - htmlcss.md
        - datastructure.md

or
- data
   - careerpaths
     - fullstackdev
        - index.json
        - htmlcss.md
        - datastructure.md

@imchetanyadav we need only one career path for now. So, file structure can be
data
   careerpaths
      full-stack-dev.json

Start with this. This should suffice. We need to put only one/two lines for each leaf, don't need a separate node for it.
If you start working on it then send a WIP(Work In Progress) PR referencing this issue and I'll assign this issue to you.

md files I listed in file structure were referencing to content of pages that open when leaf node is clicked not leaf node content itself.

@imchetanyadav update on this?

Done with the basic implementation and raised PR. See if it seems fine.

I'll check it out. Apologies for the delay in reply.
On Sun, 9 Jun, 2019, 10:18 PM Chetan Yadav<EMAIL_ADDRESS>wrote:

Done with the basic implementation and raised PR. See if it seems fine.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/learncodingfree/learnCodingFree.org/issues/10?email_source=notifications&email_token=ACN6LH4KJFA25QTNJJGAEYLPZUX43A5CNFSM4HTYJYF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXIN5EI#issuecomment-500227729,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACN6LH4HAK7E4W7UK65VT3DPZUX43ANCNFSM4HTYJYFQ
.


[android] support checkbox tinting

Motivation
RN offers checkbox component on android: https://facebook.github.io/react-native/docs/checkbox.html
The Checkbox colors for checked and unchecked states cannot be controlled from JS at the moment; this PR adds support for that.
The essence of changing colors for the states is this:
ColorStateList cls = new ColorStateList(
        new int[][] {
                new int[] { -android.R.attr.state_checked }, // unchecked
                new int[] {  android.R.attr.state_checked }  // checked
        },
        new int[] {
                uncheckedColor,
                checkedColor
        }
);
checkBox.setSupportButtonTintList(cls);

Because of this, I did it so that both colors have to provided together in an object. This is similar to switch
Test Plan
I have added this to the RN tester.
Related PRs
https://github.com/facebook/react-native/pull/11940 Support tintColor and thumbTintColor for Switch on Android
Release Notes
[ANDROID]  [FEATURE]  [Checkbox] - support custom colors of checked and unchecked states

@janicduplessis I believe you mentioned in https://github.com/facebook/react-native/pull/18318 that you were going to review this PR, may I ask you for the review? Thanks.

I wonder if we could simplify the logic a bit by moving this to ReactCheckBox.java and saving the colors of the checked / unchecked status as nullable int ivars (null means the theme color).

yes, that would certainly lead to simpler code. The approach I took was using CompoundButtonCompat.getButtonTintList because I wanted to keep a single source of truth - one maintained by the checkbox itself rather than having two more instance variables that would essentially hold a duplicate of what can be obtained via CompoundButtonCompat.getButtonTintList and also eat extra memory (although that probably isn't a big deal in this case).
@janicduplessis please let me know if I should keep the logic the way it is or rewrite it as you suggested. Thanks.

@janicduplessis can you please answer to my comment above? Thanks.

@janicduplessis would you care to do a final review of this?


support of environment variables in httpheaders and replacement patterns

This PR introduces a special replacement pattern {{env.<env_var_name>}} for environment variables, that can be used in httpHeaders and in pattern replacement strings. From my point of view it fixes the problem mentioned in #103.
Once merged, credentials can be passed to markdown-link-check via env vars.
{
  ...
  "httpHeaders": [
    {
      "urls": ["https://example.com"],
      "headers": {
        "Authorization": "Bearer {{env.EXAMPLE_COM_TOKEN}}",
        "Foo": "Bar"
      }
    }
  ],
  ...
}

@NicolasMassart Could you please do me a favour and release a new version containing the changes made with this PR?

@volkc done https://github.com/tcort/markdown-link-check/releases/tag/v3.8.6
Sorry for the delay...

@volkc done https://github.com/tcort/markdown-link-check/releases/tag/v3.8.6
Sorry for the delay...

@tcort the release did not trigger the package creation on NPM, could you have a look why? Did you do it by hand on the previous versions?

@tcort the release did not trigger the package creation on NPM, could you have a look why? Did you do it by hand on the previous versions?

@tcort the release did not trigger the package creation on NPM, could you have a look why? Did you do it by hand on the previous versions?

I don't think that behaviour is configured. I've always just used npm to do releases...
npm version patch && git push && git push --tags && npm publish

I've published v3.8.6

@tcort the release did not trigger the package creation on NPM, could you have a look why? Did you do it by hand on the previous versions?

I don't think that behaviour is configured. I've always just used npm to do releases...
npm version patch && git push && git push --tags && npm publish

I've published v3.8.6

Thanks a lot!
We need to create an action for that them... I don't think we can push on our own without your credentials for npm login

Thanks a lot!
We need to create an action for that them... I don't think we can push on our own without your credentials for npm login


Compare against concatenated file?

Is there a way to compare the file against a file of a different name?
For instance if the file has no destination because it is being concatenated with a number of other files. It would be nice to be able compare the file against the file that is the result of the concatenation.
Maybe allow destination to be a directory or a file?
ex.
partial1.js and partial2.js are concatenated into main.js
__ assets
  |__ partials
  |  |__ partial1.js
  |  |__ partial2.js
  |__ main.js

Alright, I figured this out using a custom hasChanged function. Basically I just modified some of the source code and stuck it in a function in my gulpfile.
Here's what I used in case anyone else needs this functionality.
var fs = require('fs');
var gulp = require('gulp');
var gutil = require('gulp-util');
var changed = require('gulp-changed');

var compareAgainstFile = function (stream, cb, sourceFile, destPath, destFileName) {
	var destPathTo		= destPath.match(/((\\|\/)?(.+(\\|\/))*)((.+)\.(.+))$/)[1];
	var modDestPath 	= destPathTo+destFileName;

	fs.stat(modDestPath, function (err, targetStat) {
		if (err) {
			if (err.code !== 'ENOENT') {
				stream.emit('error', new gutil.PluginError('gulp-changed', err, {
					fileName: sourceFile.path
				}));
			}

			stream.push(sourceFile);
		} else if (sourceFile.stat.mtime > targetStat.mtime) {
			stream.push(sourceFile);
		}
		cb();
	});
};

gulp.task('myTask', function () {
	return gulp.src('dev/scripts/**/*.js')
	.pipe(changed('app/scripts/', {hasChanged: function (stream, cb, sourceFile, destPath) {

		var destFileName = 'main.js'; /* You can do more complicated logic here to determine your destination file's name. */

		compareAgainstFile(stream, cb, sourceFile, destPath, destFileName);

	}})
	.pipe(/* run processes */);
});


Migrate to Prometheus Community?

Is there any interest in maintaining this exporter? Maybe we could move it to the Prometheus Community repos?
https://github.com/prometheus-community

Hi @SuperQ thanks for your interest.
I'm not sure sure how much that organization is open to accept repos that has not been well maintained by someone, so I think we need an active maintainer for this project anyway.
Let me try to do some cleanup for this project within a week first. In any case are you interested in helping to maintain this project?

Yes, as one of the Prometheus maintainers, I would be willing to sponsor getting maintenance started.
One of the things I was looking at was your prometheus-exporter-harness code. I think that something like that would be really useful for https://github.com/prometheus/common.

Yes, as one of the Prometheus maintainers, I would be willing to sponsor getting maintenance started.

oh that's nice :) Let me prepare to transfer the ownership.

One of the things I was looking at was your prometheus-exporter-harness code. I think that something like that would be really useful for https://github.com/prometheus/common.

Oh it's interesting to hear heat. I thought most exporter authors are just relying on standard prom modules because its just enough.
so when it's ready I'll just ping you and move the ownership, does it works?

I filed an issue in the community tracker for this.
There are two ways to do the move.

I can grant you an admin role to move the exporter. You can continue to be a contributor to the exporter.
You can grant me admin access to this project and I can do the move for you.

Either way, you're welcome to continue to be a maintainer/contributor of the project after the move.

@SuperQ ok that works. Let me add you as an admin.
Before that one quick question; what's the difference between github.com/prometheus-community and github.com/prometheus ? Apparently both contains various repositories of exporter implementations.

FYI, It turns out it's not possible to make others "admin" on "personal" repos. I will create a group on prometheus-community so it can be moved.

github.com/prometheus is for "official" projects maintained by the core team.
github.com/prometheus-community is where we're putting community maintained projects. If a community project is popular enough, and there are enough core team members available, we may build a process to graduate them to the "official" org.

@SuperQ I see, thanks for the clarification!

I will create a group on prometheus-community so it can be moved.

I assume it's not ready yet? I've just tried to transfer the ownership but failed by lack of permission on prometheus-community.

I sent an invite to join the json_exporter team, but it looks like it expired a couple days ago. Once you accept the invite, you can transfer the project.

@SuperQ Ugh, sorry it seems that I overlooked the previous invitation.
I've just transferred the ownership and now this repo is at prometheus-community.
Thanks for your help :)


Replace Apache HttpClient with JDK 11 HttpClient

The Apache HttpClient was added to get around SSL errors when calling shields.io with micronaut's HttpClient. Replacing it with JDK 11's HttpClient would have the project depend only on micronaut apart from the JDK, thereby resulting in a hopefully simpler native image creation. The change needs to be tested to make sure the SSL errors don't come back

Turned out the SSL errors were due to a JDK 11.0.1 bug. The upgrade to 11.0.8 in https://github.com/devatherock/artifactory-badge/commit/ab1cb2c26356b013c8d47e83d0c7cbea5148e374 fixed it, so we can just remove Apache HttpClient - no need to replace it with JDK HttpClient


AM-2688:ET_WA - Add mapping rules in ORM for STAFF users

ET_WA - Add mapping rules in ORM for STAFF users
https://tools.hmcts.net/jira/browse/AM-2688
JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[X] No

Reviewed alongside #1434 and current documented mappings in config; see comments against #1434.


Update to SPDX version 3.6 2019-07-10

Could this package be updated to use the licenses of SPDX version 3.6 2019-07-10?
From the code it would seem that you can regenerate and push the updates.

Thanks for updating, I immediately updated to the new version.


Optimize the Expression field iteration between the number of Class properties and DbDataReader columns.

[Scenarios]

If the class has more properties than the dbdatareader number of columns, then iterate the dbdatareader columns and extract the result back to the model.
If the dbdatareader number of columns is much more greater than the number of class properties, then iterate the class properties and retrieve the value from the dbdatareader.

This has been checked-in to master branch, and will be available at version >= v1.9.6.


[cattle] way to identify network driver service and CNI driver service

Earlier, the network/cni config was under the primary service of the networking stack, but after it got moved to cni-driver, there is no easy way to identify the network driver service. Need some kind of flag in metadata/cattle.

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


chore: clean up after #17615

This PR goes through the diff of #17615, which automatically replaced erw with rw, to clean up as follows:

delete obsolote comments
merge adjacent rw calls
restore old code that was commented out when it broke at some point but now works again

[ ] depends on: #17615

Thanks!
maintainer merge

Thanks!
bors

(I assume you mean bors merge?)

I'm going to go ahead and assume that was meant as a:
bors merge

Ah whoops! Yes, somehow I managed to drop the r+...


[TOPIC-BLE-LLCP]: Bluetooth: controller: llcp: split out control procs

The control procedures are splitted out in their own file, making it
easier to merge in the new LLCP
This work is far from finished, but is intended to give us a basic for discussion on our thursday meeting

Closing this PR. The conclusion is that we do not want to refactor the existing ull_conn due to the amount of work and risks


Check out pytest-spark for spark tests

https://github.com/malexer/pytest-spark
Could get spark tests up and running on Travis then maybe?

@fdosani I saw you mention this elsewhere - wouldn't mind picking your brain sometime if you dig into it

Sound good, it looks neat and can help simplify testing spark apps which might be a bit cumbersome to begin with.

yep - I've had some weird bugs with spark environments and testing, so would be nice if it simplifies :)

As it turns out, my team is creating a Spark Python library right now and we spent a fair amount of time digging into how pyspark runs their own tests. :)
I'm curious as to what the difficulty is running Spark tests on Travis since I've never tried it. If the issue is just finding pyspark to import it, I don't think a plugin is needed for it. The Spark tests here also already create a SparkSession fixture to be used once for all tests.

@theianrobertson We've gotten pytest working for our project without any special stuff - just pip install pyspark pytest basically. I think it'll come down to what the Travis environment is like and what kind of restrictions it puts on you. Another approach we're moving towards is building a Docker image with the test environment in place so that it's always consistent wherever you run it.
Let me know if you'd like me to look into this for this project. Not sure what sort of access/permissions I'd need to fool around with it.

I think for testing things in datacompy we would want to go the docker route if that makes sense, will need to dig into Travis a bit more to see what support they offer.

As per #40 I've got the spark tests running on Travis, and did swap over to pytest spark (not that it's totally required, but swapped anyway).  Will be doing a major version bump when that gets merged to master.


Allow to bump the amount of compute with a parameter

This patch, with #1181, should allow to easily scale-out the amount of
computes, without having to edit the validated-architecture-1.yml nor
the 3-nodes.yml.
It's taking the same parameter as the libvirt_manager role,
"cifmw_libvirt_manager_compute_amount".
The VA-1 scenario will pick the maximum amount, meaning we'll get at
least 3 computes; the 3-nodes will do the same, meaning we'll get at
least 1 node.
As a pull request owner and reviewers, I checked that:

[X] Appropriate testing is done and actually running

/approve


Opera versions with wrong driver versions

Re-run the Opera dataset and fix those versions with wrong driver

Fixed.


[Bug]: Jest hangs for 5 minutes before even loading test files

Version
28.1.0
Steps to reproduce
Steps to reproduce:

Own a 16" 2019 MacBook Pro
Install jest or install a repository containing the jest binary
Run jest

Expected behavior
Expect jest to immediately start and run tests.
Actual behavior
Jest takes around 5 minutes to even "load in the files" or to show any progress in the CLI.
Additional context
It seems to occur on this specific model of MBP, as I have been able to recreate it on two of these devices, but no others. I have tried multiple versions of Jest, Node, etc.
It is not repo-specific - this behavior is demonstrable on any repo with a valid jest config/binary path.
I have created a bountied SO thread to no avail: https://stackoverflow.com/questions/72478765/jest-takes-a-long-time-to-even-begin-to-execute-tests
Environment
System:

> OS: MacOS Monterey 12.3.1
> CPU: Intel Core i7 (Coffee Lake), 6-core, 2.6 GHz
> Memory: 32 GB 2667 MHz DDR4

Binaries:

> Node: 16.15.0 (have tried several Node versions)
> Jest: 28.1.0 (have tired several Jest versions)

I apologize, after further validation I believe the issue to be with ts-jest not jest itself.


Updates action versions, change how notebooks are tested

This does two things:

Updates action versions to get rid of the warning during github actions
Changes how notebooks are tested: we were previously parallelizing across all of them, which I think was actually costing us time (and making it output to navigate), so now we run them all in one action.

Turns out that this now looks like it will 22 to 28 minutes for the notebooks, whereas before, if things ran successfully, they would take ~20 minutes. I think the small increase in time is worth it, given that our previous setup would occasionally just hang during one of the notebooks, and how difficult it was to navigate the output.


Added visible modifier

Fixes #1607
Tested with "from visible chuck every instance air" and folded regions will be untouched.
Checklist

[x] I have added tests
[-] I have updated the docs and cheatsheet
[-] I have not broken the cheatsheet

fixed


remove json as a dependency

There is no need to keep json as a dependency as ruby 1.9.3+ now includes the compiled gem.

Could you please provide a PR?

https://github.com/mcanevet/rspec-puppet-facts/pull/118 was merged so this can be closed.


Starting cli.py gives error: ModuleNotFoundError: No module named 'stravavis'

Thanks for the app, it looks quite nice. But as I'm a bit new to Python, I'm having trouble to get it started.
I cloned the repo and tried from its root directory: jmizv@MacBook strava_py % python3 src/stravavis/cli.py /Users/jmizv/Downloads/gpx
But I just got that error message:
Traceback (most recent call last):
  File "/Users/jmizv/IdeaProjects/strava_py/src/stravavis/cli.py", line 91, in <module>
    main()
  File "/Users/jmizv/IdeaProjects/strava_py/src/stravavis/cli.py", line 41, in main
    from stravavis.plot_calendar import plot_calendar
ModuleNotFoundError: No module named 'stravavis'
jmizv@MacBook strava_py %

I'm probably missing something obvious but I don't see it. Could you, @marcusvolz, please give an advice?

You need to install it first. For example:
git clone https://github.com/marcusvolz/strava_py
cd strava_py
pip install -e .

And then you can run stravavis directly, also from another directory:
stravavis --help
stravavis tests/gpx

Something we could next do is publish this to PyPI, then you only need to do pip install stravavis
@marcusvolz Shall I go ahead and do this?

@hugovk that would be great!

Done!
https://pypi.org/project/stravavis/
You can install with pip install stravavis
I followed https://github.com/marcusvolz/strava_py/blob/main/RELEASING.md
@marcusvolz If you create an account on https://pypi.org/ (and optionally https://test.pypi.org/) I'll also give you admin access to those pages.
I'll check why images aren't showing at https://pypi.org/project/stravavis/
I'll also update the README to include the pip install command (and we should really say python -m pip installinstead ofpip install`: https://snarky.ca/why-you-should-use-python-m-pip/).

Instructions and fixed images: PR https://github.com/marcusvolz/strava_py/pull/17.

Merged and released as v0.0.2 with working images: https://pypi.org/project/stravavis/
Thanks @jmizv for the report, I think we can close this now.

Thanks @hugovk - P.S. username on pypi.org is marcusvolz


Add --ignore-version-conflict for old version support purpose

we have a use case that after the new version 23.05.0 is released, we using branch support-23.04.x for cherry-pick commits and making releases to the old version.
However, running bumpver update --set-version v23.04.2 to bump the version from old version, errors raise because of VCS version in the remote is higher:
╰─± bumpver update --set-version v23.04.4 --dry
INFO    - fetching tags from remote (to turn off use: -n / --no-fetch)
INFO    - Working dir version        : v23.04.2
INFO    - Latest version from VCS tag: v23.10.0a0
ERROR   - Invariant violated: New version must be greater than old version
ERROR   -   Failed Invariant: 'v23.04.4' > 'v23.10.0a0'
ERROR   - Invalid argument --set-version='v23.04.4'

I add --ignore-version-conflicts option to CLI to skip this check, test for my package and all works as expected.

If you have the time, there is some housekeeping still left:

Is there a better/shorter name you can think of?
Can you write a test case? I think you can base it on test/test_cli.py#L930.
Run scripts/update_readme_examples.py.
Update the CHANGELOG.md


weights path

Hello, I am just wondering where are the weights path for resnet 50, it seems that it is deleted.

No, it's not just follow this link :
https://github.com/fchollet/deep-learning-models/releases/download/v0.2/resnet50_weights_tf_dim_ordering_tf_kernels.h5


Ability to match on multiple/absence of class

This comes from the discussion on this issue:
https://github.com/cheezy/page-object/issues/429
Theoretically we could implement something like this with code like
//*[contains(concat(' ', @class, ' '), ' yes ') and not(contains(concat(' ', @class, ' '), ' no '))]

But this could cause issues with backward compatibility because right now Watir lets you select with more than one class (which Selenium's :class selector does not) based on XPath's string comparison.
Right now this:

Match me!

gets matched by:
browser.div(class: 'a')
browser.div(class: 'b')
browser.div(class: 'c')
browser.div(class: 'a b')
browser.div(class: 'b c')
browser.div(class: 'a b c')

but does not get matched by:
browser.div(class: 'c b a')
browser.div(class: 'c b')
browser.div(class: 'c a')
browser.div(class: 'b a')
browser.div(class: 'a c')

I mostly think this is bad, but I also think it would break a lot of people's stuff if we changed it to raise exception if spaces are found. I guess this is the difference between treating "class" in html as separate objects vs an ordered string.
but what if we implemented a :classes locator (raise exception if both class & classes used at the same time)
# 1
browser.element(classes: {true => ['a', 'b'], false => ['d']})
# 2
browser.element(classes: ['a', 'b', 'd' => false])
# 3
browser.element(classes: ['a', 'b', '!d'])
# 4
browser.element(classes: "a b !d")

Anyone have opinions? @chrismcmahon?

I like the :classes locator idea. I find myself working with elements sharing a mix of classes quite often.

I was under the impression, in terms of applying CSS, that the order of class names in a class attribute value is irrelevant (https://stackoverflow.com/a/29695391/1200545). Personally, I would prefer to make the :class locator order insensitive. To me, all of the current not matching examples should match. It's always strange to have to explain that
browser.div(class: 'a b') will be different than browser.div(class: 'b a') when applying the CSS doesn't care.
While there could be backwards incompatibility, I would hope it does not affect many people. At least, I would hope that people are not relying on an order that a developer could change at any time. For the scenarios that are impacted, wouldn't they be able to switch to a regular expression and get the same behaviour?
I would also suggest that we stick to just a :class locator rather than :class/:classes. It is easier to remember.
In terms of the absence of a class, if we are okay with the backwards incompatibility, would it be more concise if we prefix absent classes with an exclamation mark:
browser.element(class: 'a b !d')

In terms of applying CSS, class order definitely doesn't matter. In terms of the XPath engine matching strings in the DOM, it does. :)
Heh, I deleted the paragraph where I discussed an implementation of browser.element(class: 'a b !d') because I hate breaking backward compatibility (even if it is actually a widely used bug), without being able to throw a warning or a deprecation notice. The WebDriver W3C Spec no longer includes class as an option, but the JSON Wire Protocol specified "compound class names are not permitted." Selenium will continue to support that functionality with the same limitation.
As such, I think we should implement :classes and throw a deprecation notice for anyone using :class with spaces in the String. That would keep the behavior more consistent between Selenium & Watir (for whatever difference that actually makes), and be more clear and intentional about what is actually getting matched. I updated the original post with the 4 different options I see for syntax... I like the idea of the value being an Array instead of a String, especially since you can't just copy/paste the String from class to classes and expect to get the desired result.

One

Two

# Now
browser.div(class: "a c").text       # => "Two"
# Proposed
browser.div(classes: "a c").text     # => "One"

Merged with #579


Make localStorage access safer

Now this covers the case where accessing the property throws (I think this happens when a security policy restricts local storage use in Chromium) and it covers the case where set and remove don't work (I don't know of a case where this happens, but it makes sense that it could happen for "not allowing trackers to set stuff" reasons.)

@sipec my guess is this should fail more gracefully in the case we were looking at on Monday where local storage was being blocked in incognito cross-origin iframes.


FR: Mack Trucks Configurator Embed to website

Apply lessons learned from Volvo ticket to then deliver the same experience to Mack Trucks.
Dependance on: https://github.com/Netcentric/vg-volvotrucks-us-rd/issues/199

@BeckyMedlin I've transferred the issue to the Mack Trucks repo and added the MT Configurator milestone.

@shirin27 Below are the updated staging URLs from @WendyKruger:
we're still in a staging environment, and we don't have the main landing page done yet so you have to go to each truck on a separate url. if it helps, here are the links to the three trucks we have in staging as of now:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

2/21 update - We should be ready with 7 main truck models (Anthem, Granite, Pinnacle, MD, MD Electric, LR, LR Electric, excluding TerraPro) goal to deploy together for public launch by end of Q1.
Each truck model URL will start with for example - macktrucks.com/trucks./anthem/build, then become: macktrucks.com/trucks/build/#/anthem for sharing or deep linking.
Up to date staging link from Rapid Images: https://truckbuilder-mack-100031.onecx.prod.rapidimages.net/
auth_user = 'mack'
auth_pass = 'jkjK8z8LXz21'

Kamino cloned this issue to hlxsites/vg-macktrucks-com


Fix | Trying to fix the issue with SqlBulkcopy Async pattern -draft design

Related Issue(s): Issue #716
This is a draft pull request and still has room to improve.
Current Design
Async and sync calls in SqlBulkcopy are distinguished by two factors in Current design:

By a boolean variable, _isAsyncBulkCopy,  which could be set to true or false based on the call from the user.
For example, if user calls WriteToServer(IDataReader reader), this will set the mentioned variable to false, sets the statistics and calls  WriteRowSourceToServerAsync(reader.FieldCount, CancellationToken.None).
When user calls WriteToServerAsync(DbDataReader reader), this will call an overload of the same function with Cancelation token added to it and from there it calls the same function as sync call, WriteRowSourceToServerAsync.

From here the only difference between sync and async calls are based on return value of each task. If a return value of a task is null, that implies that sync pattern is called and if it returns a a task that means an async call is in progress.

Note
Same pattern has been used in different places of the M.D.S. For example reconnect task in SqlConnection follows the same pattern.

What has been done so far

SqlBulkCopy is separated to two partial classes, SqlBulkCopy and SqlBulkCopy.Async, in a new folder called BulkCopy. I have been planning to do this for a long time and I found a good occasion. All SqlBulkCopy related is moved to this folder.
All related Async functions are moved to SqlBulkcopy.Async
The  _isAsyncBulkCopy is removed as it is not needed any more.
Async/await pattern does not accept out/reference/in values. changed them accordingly.
I have tried to remove TaskCompletionSource as much as possible in async/await pattern, but not much luck. Needs more thinking and time. I may move back to this after GA.
I have removed the if/else validation for task result. The one validates if an actual task is returned (async) or it is null (sync).
Changed the new design to use async/await pattern, which is not completed yet. (We probably need to redesign SqlBulkCopy.Async).

Is there a particular goal for this set of changes other than making the code easier to work with?
It's worth noting that while going to language based async makes the code a lot easier to work with it can have detrimental performance characteristics if you're capturing a lot of state. C#9 static lambdas would help with that if we could bump the langauge version.

It's worth noting that while going to language based async makes the code a lot easier to work with it can have detrimental performance characteristics if you're capturing a lot of state.

That certainly used to be true, and one must still be careful with what gets capture with async/await. However, nowadays async/await is generally optimized and can be considerably more efficient than more traditional callback-based approaches. I benchmarked async/await vs. ContinueWith a while ago and the former was considerably faster.
It's always good to work with benchmarks when doing this kind of code change, but I'd say that any change from traditional callback-based async programming to async/await is in principle a positive direction :)

Is there a particular goal for this set of changes other than making the code easier to work with?

I believe this could be the first step of changing Async pattern in the driver. Nothing could/would be changed drastically, without designing the core of the driver for async pattern, what we need is RunAsync in TdsParser. Separating sync and Async will give us the opportunity to work solely on Async part without touching the fine working sync part. If there is any plan to re-design the code this change makes it easier and less harmful. Even with the current pattern, SqlBulkCopy.Async could be re-designed to perform better. I can address/direct you to one of the parts that makes async slower in this design. CopyRowsAsync method! the entire for loop is designed for sync pattern and it works perfect there, but it always exits the loop with the first try/run in async and the entire process starts from the beginning..... if you take the null part of the task out, VS will show unreachable code for variable i++ ........

It's worth noting that while going to language based async makes the code a lot easier to work with it can have detrimental performance characteristics if you're capturing a lot of state. C#9 static lambdas would help with that if we could bump the langauge version.

I agree, but we cannot jump to C#9 while we are supporting netcore 3.1 and netfx 4.6. In near future, support for .NET 5 will be added to the driver and in my personal opinion that is where we will need all this changes to happen.

I believe this could be the first step of changing Async pattern in the driver. Nothing could/would be changed drastically, without designing the core of the driver for async pattern, what we need is RunAsync in TdsParser. Separating sync and Async will give us the opportunity to work solely on Async part without touching the fine working sync part

I really like the idea but have found that this can lead to multiple implementations all the way down and I'm not sure having two implementations of Run is good for maintenance. @Roji probably has some good guidance here from the postgres driver, do you end up parallel or have a single implementation that both worlds use? and if so is it sync or async?

it always exits the loop with the first try/run in async and the entire process starts from the beginning..... if you take the null part of the task out, VS will show unreachable code for variable i++ ........

Yes I've seen that pattern. Parts of SqlCommand do the same and it's part of the cause of the string perf issues we've noted in the past. Essentially it tries fails and then tries again from the beginning. Easier on state management and simpler from an initial implementation perspective (because you always have full state to debug) but sort of sulky for perf.
I have ideas on how to address it in a general way but I was trying to merge as much of the codebases from netfx and netcore before I started on major internal reworks. Run and all the methods above and below it in the parser should be changed to return OperationStatus instead of bool, with that done you can now identify the cause of a parse failure and possibly suspend state cleanly for a resumption on the next async packet.

I agree, but we cannot jump to C#9 while we are supporting netcore 3.1 and netfx 4.6. In near future, support for .NET 5 will be added to the driver and in my personal opinion that is where we will need all this changes to happen.

Well... you sort of can. It's not a supported scenario but it is a working scenario. 😁 I checked with the langauge team on discord and as long we we don't use any language features that require library or runtime support it'll still work on downlevel frameworks. I understand your point though and the example of static lambdas to prevent accidental closure allocations is unlikely to be worth it, it would emit correct il working though.

I really like the idea but have found that this can lead to multiple implementations all the way down and I'm not sure having two implementations of Run is good for maintenance. @roji probably has some good guidance here from the postgres driver, do you end up parallel or have a single implementation that both worlds use? and if so is it sync or async?

I don't know the internals at all, but doesn't SqlClient already have separate implementations for sync and async? If so, then at minimum, rewriting the callback-based async implementation to use async/await wouldn't be making the situation worse...
More in detail... Npgsql does have a unified code path for sync and async for almost all the logic. This mean that you typically have a single method which accepts an async flag and returns Task/ValueTask. The flag determines whether, if and when I/O occurs, it will be done via sync or async; in other words, at the very bottom of the stack, at some point you have a condition on the flag, and call Read or ReadAsync. Intermediate methods - those not performing I/O themselves but only calling other methods - simply pass this flag down and don't have to care about it. This all means we don't have duplicate code paths to maintain - everything goes through the same methods.
Now, if all your methods have the async keyword on them, that means that for user sync APIs, you're potentially paying the overhead of the compiler-generated async state machine although it's not needed (since the method will never actually do any async operations. In fact, even for user async APIs, many calls won't necessarily need to do any I/O - the driver buffers internally, and so calls like DbDataReader.Read won't need to do any I/O (because the next row is already in memory).
The overhead of async state machines has been greatly optimized in recent versions of .NET Core. However, for true pedal-to-the-metal perf, it's possible to provide a fast-path sync implementation which only calls into an async one when necessary:
public ValueTask<SomeData> GetData(bool async, CancellationToken cancellationToken = default)
{
    return dataAlreadyInMemory
        ? theData
        : GetDataAsync();

    async ValueTask<SomeData> GetDataAsync(bool async CancellationToken cancellationToken = default)
    {
        // Perform actual I/O and return the data
    }
}

With this kind of trick, you avoid going through state machines as long as data is already buffered; and if you have to do I/O, then the state machine overhead is likely negligible anyway. It's also sometimes worth merging two async methods into one, again to cut down on the number of async state machines you have in your stack - the overhead does add up.
But I'd recommend not going into this kind of optimization right away - that can always happen later. My personal recommendation in terms of next steps would be something like:

Incrementally convert callback-based async code to async/await based code. This will likely not regress perf (though always measure), would probably make the code much saner and expose/fix quite a few bugs.
Optionally look into merging sync and async paths - since after conversion to async/await the sync and async paths should look quite similar.
Look into optimizing the code with fast-paths to avoid state machine overhead, and cutting down on the number of async state machines.

Hope this helps... Am definitely available for any further questions/discussions on this.

Well... you sort of can. It's not a supported scenario but it is a working scenario

You're right here - static lambdas (and some other C# 9 features) are compiler-only features which should always work. Other features (e.g. covariant return types) require the new runtime and therefore will not work. Static lambdas are generally a good idea to reduce allocations - I agree that SqlClient is probably facing more important perf problems, but it sure wouldn't hurt to use static lambdas when writing new code.

There is some interesting code that handles sync and async callers in sync mode using api-async. An example of this is TdsParserStateObject.WriteBytes , it's fairly central to low level operations of writing bytes to the packets. Have a glance at the code. It's a fully sync method that signals whether it's working in async mode by returning a task to callers, if it returns a task that usually gets passed up the stack so whoever is doing the language async part gets to wait on it. The library itself does not await anything that I've ever seen it just sets up tasks and continuations.
The problem I have with it is that because of all the looping and conditionals that exist here and levels above it it takes a lot of mental space to make sure your handle all routes through the code cleanly. I don't know if others have the same experience but I'd like to have some of the more important code in the library be significantly more straightforward to understand.
There's a big Run method in the parser which is the main state machine implementation for incoming packets. State switches get long quickly and there are lots of quirks to account for because of old protocol versions and bug fixes. I'd prefer not to have two large state driver methods but having one which works in both sync and async modes either means a lot of conditionals on async or a completely separating io from parser driving and making all state transitions completely resumable which is complicated.

@Wraith2 I very much agree with everything you say, and I'd say that the various async-related issues found in SqlClient are a good additional argument to try to make things saner by incrementally doing async/await (if possible). Callback-based async (like the WriteBytes method you linked to) is notoriously hard to follow and grasp, is inherently bug-prone, and those are the very reasons why async/await was invented - it allows writing and reasoning about async code in almost the same way as simple sync code.
BTW the WriteBytes method you linked to is odd, in that it both returns a Task and accepts a TaskCompletionSource as a continuation (which, by the way means a lot of allocations as TCS cannot be recycled). I'd expect it to do either one or the either, but not both.

@Wraith2 , @roji Thanks for all the efforts. We are currently busy with the GA activities. The moment we are done with it I'll come back to this PR and go through all the comments and recommendations. I really appreciate all the support.


update minimist dependency to require a minimum version of 1.2.3

This addresses https://npmjs.com/advisories/1179, specifically  https://github.com/karma-runner/karma-mocha/issues/214

@googlebot I signed it!

I guess you need to rebase on master to trigger tests on node 10 + 12.

@johnjbarton I'm not too familiar yet with the processes. Can you elaborate on what I need to do?

Those instructions are ok but overkill.
I don't know what happens if you use a master branch on a fork.

Then you need to rebase. I use the name upstream:
# add a repo named upstream
$ git remote add upstream git@github.com:karma-runner/karma-mocha.git
$ git pull -f upstream master

Now your master matches upstream master
# move to change branch
$ git checkout minimist-update
$ git rebase master
$ git push origin minimist-update

Now you have a new PR to create on github.
You might be able to do this without the branch but this is the workflow I know.

Will try to do the rebasing thingy @johnjbarton suggested.

@johnjbarton I've succesfully completed a rebaseed version of the PR: https://github.com/karma-runner/karma-mocha/pull/217
Thanks for your help.

Sorry for commenting to a closed pull request. But can somebody create a release including the latest changed, please?


Install Helm chart stored locally

I am not sure if I have missed it in the documentation, but is it possible to install Helm charts from a local folder instead of a repo?

Indeed you are right it is currently not mentioned anywhere that it supports path based and oci based charts as well

Hi @tuunit - Thank you for the reply. And apologies for going with a "look before you leap" approach. I was researching what tools might fulfill my need and was just looking at the docs.
Will give it a try and get back if I encounter any problems :)

No worries :)


Get pushes continuously

Hello,
I had a new problem using pushbullet.py : I want to get push continuously but i'm limited by query limits.
How can i get push everytime like in all pushbullet clients ?
Thanks
(sorry for bad english)

Hi,
try websocket. You can also check in  my pushbullet-rpc
pushbulletrpc.py -> recv_and_process()

Ok, I take a look at this.
But i've got an exception : InvalidKeyError when I start my programm but my key is valid...

Ohh, let's debug your key. :)

Very strange, it work 10 seconds and after ... nothing.

Hello,
i have really the same problem.
I guess the problem is the Ratelimiting:
Ratelimiting
When you do a request to the API you will receive headers like the following on the response:
X-Ratelimit-Limit: 32768
X-Ratelimit-Remaining: 32765
X-Ratelimit-Reset:<PHONE_NUMBER>
These tell you what the ratelimit is, how much you have remaining and when it resets (integer seconds in Unix Time). The units are a sort of generic 'cost' number. A request costs 1 and a database operation costs 4. So reading 500 pushes costs about 500 database operations + 1 request = 500*4 + 1 = 2001. You can see how much a request cost by the change in X-Ratelimit-Remaining between two requests.
maybe it's possible that you can store this X-Ratelimit-Remaining and give it back over a function?
Thanks

It would seem fairly easy to implement the logic in _get_data() function to store the ratelimit information. Then a helper function added to pushbullet.py to return the remaining limit at any given time.
However, as @kovacsbalu  said, the best way to continuously receive pushes is via websockets using the pushbullet real time event stream. I wouldn't mind having a go at implementing that but the websockets client library I'd use is for python 3.3+ only. I guess I could try websocket-client but it doesn't look as nice :)


Fixes for tasks which can fail during the update operation

I was trying to use this for a new charm and found on repeat update commands task fail this seems to be because copier makes a temporary folder clones into that, and then compare to the final destination. The temporary clone still runs the actions, which fail b/c they can't find the expected files. This avoid the issue by exiting if their is no file to operate on.

The failing tests appear to be b/c the branch of the plugin I was using has been merged (and deleted in the process). I'll get that updated and remove the draft when it's passing tests again.


Samoan Translation v1

Haven't been across the translation stuff too much but how about we order the list of languages by how commonly they are used in New Zealand?
IMO we have a fairly large Samoan community so it would nice to put our pacific languages + Chinese at the top before less used languages like Russian and Spanish. Any thoughts @olafwrieden ?

@wseagar I swear you read my mind... I am literally doing that right now :D

Great point @wseagar!

@olafwrieden I believe @tausani-ah-chong will review and make the final sign off on this translation.

@tausani-ah-chong could you review the changes when you get a chance please. There are a few glottal stops that I've kept in as they're pretty much required otherwise the word would look out of sorts.


Improve looping over blocks

https://github.com/TexZK/bytesparse/blob/58bded0fcbd0c547671538c84629ad827d6f87cc/src/bytesparse/inplace.py#L676
See if the final block can be determined via ImmitableMemory.block_index_*().

Using Memory._block_index_endex() requires even more checks to be added, despite the logarithmic nature of the search.
Leave as-is for now, while keeping the implementation simple.


createEdit: preview shows "Not provided!" for What's New when actually provided

The fix (db76913) for #256 and #284 introduced a regression where the preview for What's New is not updated.

Verified working (testUser1) in both Create/Preview and Listing Mgmt/Preview for all browsers (IE9, IE11, FF24, CH36)

Verified that the What's New text appears and is removed on the Details tab of the Detail View on the Create/Edit listing page before and after saving as well as on the Detail View on Listing Management. Tested on Staging with IE9, FF, Chrome


Value not setted on autocomplete's textinput

Hi, it seems that value property of autocomplete doesn't work, can you verify this?
example: I'm filling a form for edit but the initial value of autocomplete is not displayed.
Thank you.

Hi,
This problem delay, but I want to announce it has been solved.


OpenstackInfra ironic extra data from swift

OpenstackInfra get ironic extra data from swift. The extra data
were moved to swift because of limiting size in ironic
attributes. Now there is serialized json stored in swift
objects.

@blomquisg Please review

@Ladas Please rebase after Openstack namespacing PR https://github.com/ManageIQ/manageiq/pull/3664 was merged

@Ladas, looks like a legit test failure.

@blomquisg hm, weird, grizzly and havana have a different way of saving things?

@Ladas @blomquisg What is the status of this PR?

@chessbyte @blomquisg should be ready


CenteredRenderer Padding

The padding functionality doesn't properly preserve the correct number of background characters.

Solved with https://github.com/ethanlchristensen/bruhanimate/commit/196c2cc80fcd1e8e30e61b2e204b5f970badea0e


fix: incorrect iterator access in base64 decoding

I would not have noticed this, but MSVC (with the right warnings) did.
More fixes to get the code to compile on MSVC will be coming in future
PRs, but this is useful by itself.

PTAL


mWeb - chat - On starting conversation with phone contact, +91 is added as seperate contact

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.23
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4879523
Issue reported by: Applause - Internal Team
Action Performed:

Go to https://staging.new.expensify.com/home
Login with new account
Tap fab -- start chat
Scroll and check +91 contact not in contact list
Enter a photo number eg:+919878675687
Start chat with that user and send a message
Navigate to LHN
Tap fab - start chat
Now note in recents you can see number eg:<PHONE_NUMBER>7 and scroll the user list and note +91 is added as seperate contact.

Expected Result:
On starting conversation with phone contact, +91 must not be added as seperate contact
Actual Result:
On starting conversation with phone contact, +91 is added as seperate contact.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/user-attachments/assets/d69a60c8-abea-4941-b90a-d45efabe5053
View all open jobs on GitHub

@miljakljajic  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #vip-vsp

Proposal
Please re-state the problem that we are trying to solve in this issue.
On starting conversation with phone contact, +91 is added as seperate contact
What is the root cause of that problem?
When we type +91 we request search to the BE and the BE response and return a contact personal detail info with +91 only

And the response from the BE will apply the Onyx changes which will create a new +91 contact number
What changes do you think we should make in order to solve the problem?
in the contact section we can exclude if the contact response is from BE, we can use the isFromServerSearch property to determine if the personal detail is from BE
And we will show only if we click the contact meaning we start conversation with the contact, because the new personal detail contact info is not from server search
What alternative solutions did you explore? (Optional)

Bug is fixed.
https://github.com/user-attachments/assets/20ca3138-bc31-4de5-b6f7-61056df04d2c

Closing!


Refactor gtest-parallel.

What I tried to do here is to create some classes to store the information about the tests and test executions so they're not saved in multiple times, and move things from the global scope to functions to improve readability.
I also introduced the capability to retry failed tests.

@pbos @kwiberg
Please take a look.

Is it possible for you to break this up into smaller changes? The linediff is huge, and you're making the file about 20% larger.

I'll try.
If it's not too complex, can you take an overall view of the changes and tell me what you think?


Children created in OrderedCrossover are identical

The children created in OrderedCrossover are identical because the randomization occurs only once, but there are two children created from it.
See PR: https://github.com/giacomelli/GeneticSharp/pull/20

We have an unit test (OrderedCrossoverTest.Cross_ParentsWith10Genes_Cross) that validate the children generated by OrderedCrossover, in that test we have the chromosomes below:
Parent one: 8 4 7 3 6 2 5 1 9 0
Parent two: 0 1 2 3 4 5 6 7 8 9
Child one: 0 4 7 3 6 2 5 1 8 9
Child two: 8 2 1 3 4 5 6 7 9 0
As you can see, there is no identical children.
Please, can you provide us the use case where you found the identical children?


VS2022 Setup Error: The package 'Component.GitHub.VisualStudio' is not recognized.

Unable to create an offline cache for VS2022 Enterprise Setup. The Setup was downloaded from https://visualstudio.microsoft.com/thank-you-downloading-visual-studio/?sku=enterprise&rel=17
The command I ran was
.\vs_enterprise__408428872.1625117389.exe --layout C:\VS2022Enterprise\ --add Microsoft.VisualStudio.Workload.ManagedDesktop --add Microsoft.VisualStudio.Workload.NetWeb --add Component.GitHub.VisualStudio --includeOptional --lang en-US
The result :
Visual Studio Setup Layout Tool, version 3.0.4492.23473
c Microsoft Corporation. All rights reserved.

Download of 'https://aka.ms/vs/17/release/channel' succeeded using engine 'WebClient'
Download of 'https://download.visualstudio.microsoft.com/download/pr/7aa16be3-9952-4bd2-8ecf-eae91faa0a06/6a7baa54d8758a4ae95028b3addb78cce39534cbfb49a23aee4c25766ff0c154/VisualStudio.vsman' succeeded using engine 'WebClient'
Catalog load operation correlation: {"id":"f44b5fdb-84e6-48ec-bf89-41785f522830"}
Layout operation correlation: {"id":"e4a78386-9519-4ea5-8831-6981c38a1186"}
Planning 'Layout' operation
Error: The package 'Component.GitHub.VisualStudio' is not recognized.
Error: Failed to plan for Layout.
Error: Setup completed with errors.
Error: The package 'Component.GitHub.VisualStudio' is not recognized.
Press any key to continue...

@kvsrkrishnan -- Sivarama, thank you for your feedback.  Are you following a particular docs.microsoft.com article?  Meanwhile, please consider these resources:

Visual Studio on Microsoft Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit


Unable to resize drawer to fit the content

Code in initialize():
val drawerPane = drawStackPaneController.drawerStackPane
drawer.defaultDrawerSize = 100.0
drawer.setContent(this.drawerStack)
drawer.setSidePane(drawerPane)

<JFXDrawer fx:id="drawer" defaultDrawerSize="0.0" direction="RIGHT" layoutX="549.0" layoutY="9.0"
 overLayVisible="false" resizableOnDrag="true" resizeContent="true" styleClass="jfx-drawer"
<EMAIL_ADDRESS>

I can only resize the drawer to fit it after dragging it mannually. So, what exactly should I do to make it resizeable? Thanks in advance

can you provide SSCCE for this issue?

Here's the demo here. Part of codes are written in Kotlin. But I think that's acceptable
JFeonixDrawerDemo.zip

Hello,
Well you are kind of misusing the JFXDrawer, you can check JFoenix main demo for a good example on JFXDrawer. Will close this issue as it's not related to JFoeinx development.
Regards,

I'm misusing？Why did you say that？
Yes, I've already checked your demo/components/DrawerDemo. My current code is based on your demo. Can you point out why I'm misusing please? That would be really helpful


Show all lecture units in Lecture overview

Describe the solution you'd like
Currently, the lecture overview only shows the amount of attachments.

However, with the introduction of lecture modularization, it would be helpful to also show all other lecture units. For example, as lecture slides are not attached as attachments anymore, but as lecture units, the attachment count is not as helpful as before.
Visualization:

Describe alternatives you've considered
Removing the attachment count as it is not helpful anymore for lectures with modularized lecture units.

I am closing the PR, since the amount of attachment count has been removed in this PR due to performance issues (reference) and that is the alternative solution described.


When installing a stateless system, it prompts that there is insufficient remaining space.

When installing a stateless system, it prompts that there is insufficient remaining space.

There is not much to go off here. Please provide more information. What do you mean by a "stateless system"? What does this prompt look like?

When I install the system on a Mac m2 air, I will get an error that lack of space when I mount /mnt on tmpfs and /mnt/nix on /dev/nvme0n1p5 to implement a stateless system. Today it worked. But this problem is not known for some reason.


How to turn off after initialized?

I have this flow in my app:

When my web app loads I run: TagManager.initialize({ myConfig });
But if my user clicks on Decline Cookies I need to turn it off

Is there any function like: TagManager.turnOff that will remove all cookies associated with Google Tag Manager?

Same here any solution.

@LucasAndrad  I think ONLY if the user consents with your cookies usage, you should initialize TagManager?

No @dcorb, that's not the flow the company wants. That first track if TagManager is anonymous, so no need any consent for that (at least following the company country laws)

How about add a persistent condition-trigger like:
if (userAcceptCookies) {TagManager.initialize({ myConfig })}

Anyone found a solution?

I'm no expert but

That first track of TagManager is anonymous,

So that's fine
Then later, push the userId and other private data into the dataLayer only if they consented. If anonymous data is good prior to consent it should be good after.
Otherwise, convince the company to start it post consent for consistency?

Having the same issue.
The user could accept all cookies and later on update his preferences to deny tracking for example. That could be a use case to turn off the tag manager. A TagManager.turnOff() method would be handy.
Another option would be to use the API for consent settings via Google Tag manager. https://developers.google.com/tag-manager/templates/consent-apis
However, I didn't manage to make it work with this library.

Is there any update here?
I need to turn off the GTM after the user logs in. A util function to disable the GTM would be quite handy


Update jackson-databind to 2.16.1 and remove suppression. Also logging

CI: No security warnings found


Review 2(MidTerm): pypi package detecting all metadata from catimages

Make a pypi package that is capable of creating all metadata that the catimages could.
With tests preferably.

We should have an overview of the state. A table like we have for the deps that lists capabilities and categories as well as the state (done, in progress, open).