Ability set holiday year start date

This change allows users on start to set the companys holiday year start (for example April or defaulted to January) with the holiday year running from then (for example 1st April-> 31st March)
We had this problem within our Company and looking though the issues it appears several users had similar request. See #218 , #114 and #354
I has set this to WP as I am never new to this project and believe there will likely be some feedback or changes to make.

@gamereat have you noticed this PR breaks the functions for manual adjustment of leave? I've been trying to work out where it goes wrong, but have failed to find the cause so far.

@gamereat have you noticed this PR breaks the functions for manual adjustment of leave? I've been trying to work out where it goes wrong, but have failed to find the cause so far.

Hey @portableant, I hadn't noticed. I will take a look into that soon. As i said this is a large change so was likely to introduce elements that were not expected.

@gamereat have you noticed this PR breaks the functions for manual adjustment of leave? I've been trying to work out where it goes wrong, but have failed to find the cause so far.

Hey @portableant, I hadn't noticed. I will take a look into that soon. As i said this is a large change so was likely to introduce elements that were not expected.

Much needed change, so well done for attempting to make this work!

@portableant that should the manual adjustment fix now

@gamereat I will give it a go! Thank you for having a look.

Thanks for putting the work into this!
Just had a test of this and looks like it (mostly) works, a few issues noticed with it:

The periods are still displayed as calendar years in the view (e.g. 2022 runs from August 2022 to July 2023 if year start is set to August)
Holidays which overlap the boundary take from the previous year only (requesting a 5 day holiday where 2 are in July and one is in August will result in 5 taken from the current year's allowance)
New employees that start halfway through a period do not have a scaled allowance (they get a full year of allowance, tested with August start date by setting a user to start in the middle of November, they still had the default 20 days)

Is there any likelihood of seeing this integrated into main / officially supported?

Thanks for putting the work into this!
Just had a test of this and looks like it (mostly) works, a few issues noticed with it:

The periods are still displayed as calendar years in the view (e.g. 2022 runs from August 2022 to July 2023 if year start is set to August)
Holidays which overlap the boundary take from the previous year only (requesting a 5 day holiday where 2 are in July and one is in August will result in 5 taken from the current year's allowance)
New employees that start halfway through a period do not have a scaled allowance (they get a full year of allowance, tested with August start date by setting a user to start in the middle of November, they still had the default 20 days)

Is there any likelihood of seeing this integrated into main / officially supported?

Yeah,
I'm unlikely to be able to support this PR much further. I had hoped to get some progress getting it offically supported when I first set work on this but over the past year my company has moved away from timeoff.
Regarding those issues, The second one was a core issue with Timeoff outside this branch and likely need to be addressed independently.

Thanks for getting back to me so quick!
We're currently looking at some options for this stuff, I know it's a bit off topic for this thread then but did you find a suitable alternative that did have this feature? Any recommendations would be helpful!

Thanks for getting back to me so quick!
We're currently looking at some options for this stuff, I know it's a bit off topic for this thread then but did you find a suitable alternative that did have this feature? Any recommendations would be helpful!

We moved to BreathHR. It does cost money but compared to everything else it cheap and has a lot of other features that once you grow as a business is super useful.
We found Time off was fine when it was a small team and I had time to maintain this branch and other issues but as we grew my time was more elsewhere


Method not found (code -32601)

Hi i am trying to setup arcticcoin master node but when i tryt o pass this command
Code:
goldmine genkey
form this link
https://arcticcoin.org/en-EN/guides-goldmine
i am getting this error  " Method not found (code -32601) "
Do you any one what is issue and how to solve this iisue

Try "goldminenode" instead of "goldmine".  The instructions are incorrect.


Dem insertion: clear and add

Description of the problem

LPBF spreading simulations requires a lots of particles. Most of them are useless during most of the simulation time, as they are deep the powder dispenser. It would be more efficient to only simulate the particles required for the layer N and not for the following layers.

Description of the solution

A new insertion method called "clear and add" has been implemented.
This method is very similar to file insertion in that it reads particle position, diameter and velocity from input files.
First, we define a clearing box, where particles will be deleted. Then, we insert new particles from an insertion file. Each insertion, a different insertion file can be use.

How Has This Been Tested?

Custom parameter file. (See slack)

Comments

I still need to do a test, and example and the documentation for this method.
I have change the order for the parsing of the parameters. It is now in alphabetical order. It is cleaner IMO.

Doc is done

Will review it today :)

@OGaboriault You  will need to reindent :)

I have removed the indent short cut on my IDE last week by accident... iI will put it back.


Prevent scrollbar on text fields

If too much text is added to a text field a scrollbar appears which looks weird and is obstructive for users, especially when inserting longer texts. This can be fixed by automatically adding more height to the text field.

Will be fixed inPR: https://github.com/UST-QuAntiL/qc-atlas-ui/pull/109


NNAPI Reference does not output the same result as CPU for style transfer app on android for tensorflow lite

I have altered this app significantly such that "StyleTransferModelExecutor" could configure the interpreter with more fine granularity:
For example:
StyleExecutor(appContext, quant = true, device = Device.NNAPI, NNAcc = "nnapi-reference")
I am more than willing to share all that code, however the main point is:
running inference with:
style_predict_quantized_256.tflite"
style_transfer_quantized_384.tflite
On CPU is different than NNAPI reference.
This is how I set the options for nnapi reference:
val tfliteOptions = Interpreter.Options()
var opts = NnApiDelegate.Options()
opts.setAcceleratorName("nnapi-reference")
tfliteOptions.addDelegate(NnApiDelegate(opts))
Interpreter(loadModelFile(context, modelName), tfliteOptions)

I have an instrumented test that essentially runs noise through the models, and compares the results, and the results are significantly different.
I believe this is a bigger issue than just transfer style, and really relates to quantized models on nnapi not agreeing with cpu. I just use this app as a basis for re reproducibility
Feeding in noise:

CPU and GPU inference agree:

NNAPI Reference differs:

(Further information)
The models are taken from the tflite example project "style transfer" and looking at this particular gradle file, you can see the links where the models are stored:
https://github.com/tensorflow/examples/blob/master/lite/examples/style_transfer/android/app/download_model.gradle
or:
https://www.tensorflow.org/lite/models/style_transfer/overview
at the bottom of this page, the models are there (though I used the gradle links)
And I will include them in this message as well
quantized_models.zip

Can confirm that the output is different for NNAPI vs CPU using the inference tool:
$ blaze build -c opt   --config=android_arm64   //third_party/tensorflow/lite/tools/evaluation/tasks/inference_diff:run_eval && adb push blaze-bin/third_party/tensorflow/lite/tools/evaluation/tasks/inference_diff/run_eval /data/local/tmp
...
$ adb shell /data/local/tmp/run_eval \
  --model_file=/data/local/tmp/style_predict_quantized_256.tflite \
  --delegate=nnapi
...
OutputDiff[0]: avg_error=2.25797, std_dev=0.0116284

This shows a raw error of 2.2 averaged across all elements of the output tensor. For comparison, delegate=hexagon gives an error of ~0.008. I tried delegating only half the graph by modifying the delegate, and the error is still ~0.99.
Adding Miao from NNAPI.

The issue has been found in NNAPI delegate and the fix has been submitted internally.
You should be able to sync later today and verify the fix.
Thanks for reporting the issue!


the goquery string doesn't work.

The book suggests the following
s := "html body div#b_content ol#b_results li.b_algo div.b_title h2"

but in reality, it doesn't work. I have attempted to make small changes to debug but I didn't get any luck.
I assume the logic is to loop through every one of these tag from s, then attempt to find if a with attr href exist, if it does then save it, if it doesn't, then return. However, the loop dies when it reaches div#b_content, although the html elements of that page from bing does have a div tag with id b_content. Please give some advice how to fix this. thanks.

@JizongL see this PR https://github.com/blackhat-go/bhg/pull/13


iterblock: remove duplicate line of code

Signed-off-by: Russell Bryant<EMAIL_ADDRESS>

:+1:

👍

thanks!


ETABS add v19 to installer

Source: https://speckle.community/t/etabs-connector-multiple-versons/2507

The installer still works installing it, but I've encountered cases where it won't install it properly depending if ETABS is open. Just innosetup not working well with editing config files.
Not a big concern since CSI products allow people to add plugins quite easily by just browsing for the file path.


Items can be seen by members of groups which have the permission to see this item

User Stories
As the owner of an item I want my item to only be displayed to members of groups I allowed to see this item
Dependencies

#21
#11
#8

UI Mockup
not applicable
Acceptance Criteria

[ ] when editing/creating an item I can choose which groups are allowed to see that item
[ ] only groups i allowed to see an item can see that item

We want to split this and the other permission stories into the following Stories:

Implement data model for permissions on items (21, maybe a little bit smaller?)
Add UI for editing permissions (tbd)
Add UI that hides/shows items for each user depending on permissions (tbd)
Update permissions in lending process (tbd)
Only allow owners to edit their items (tbd)


Update realm_summary_table.html

Comment for tell about How to work??

@keangkai Can you join chat.zulip.org and run your ideas on the chat before posting a PR which only adds/changes comments? Thanks!


onChanged method is called with the reference of the internal json

The json (_data) returned in the onChanged method is the same object that is manipulated in the JsonEditor. Therefore, if you assign the json returned in the onChanged method to some variable somewhere and continue with more edits in the JsonEditor, the variable outside would also be changed as it's assigned by reference.
To fix this issue, you should not send the (_data) to onChanged method immediately.
Instead, copy it with _data is Map ? {...(_data as Map)} : [..._data] or with a better way and then send it to onChanged.

@ArinFaraj, Thank you for reporting this!
I will fix this issue in this next version.


Fix incomplete calculation of Darcy-Weisbach head loss derivative

As pointed out by Simpson and Elhay (J. Hyd. Engrg, 137(6), 2011), EPANET does not consider the dependence of the Darcy-Weisbach friction factor on flow rate when computing the derivative of head loss with respect to flow for the aformentioned head loss formula. This needs to be corrected.

I think this is the paper @LRossman refers to:
Simpson, Elhay - 2011 - Jacobian Matrix for Solving Water Distribution System Equations with the Darcy-Weisbach Head-Loss Model

Lew,
You and I had actually worked on this in 1991.  It's a fairly complicated expression to start with, and then you've got absolute values of flows, with flows changing sign. At least it was hard for me to get it right the first time, so maybe this helps to avoid so much pain in the next go-round.
dhe.txt

Thanks @jamesuber for the "blast from the past". Actually the current EPANET 2.x code does compute df/dq (where f = friction factor and q = flow) but it's commented out. When I first wrote it sometime in the 1990's (I can't remember exactly when) I realized it wasn't quite right and decided to ignore it. The corrrect way to do it, for both the Colebrook formula and the transition zone polynomial approximation for f(q) is described in the Simpson and Elhay paper. The resulting formulas are long winded but easy to implement. I have already done so in the code that's been posted to epanet-dev. Now it's just a mechanical matter of including them in the 2.2 code. This will break the automated testing since slightly different numerical results will be found (compared to  2.00.12) for test files that use D-W. I see this being an ongoing issue to get commits approved for any improvements or bug fixes to the code's numerical engine.

I was going to suggest refactoring in Fortran, but thought better about it.
@samhatchett had just commented to me exactly the issue you raised regarding passing the tests.
Maybe we need to enable a super secret "but this is really the correct result, I promise" key and base our testing on that.
It's a difficult issue that deserves discussion.

Just to be clear, when there is justification and it is clearly understood why its being done, it is fine to update the benchmark. Like when you are fixing a bug.


Add support for Symfony5

We finally used https://github.com/lightsaml2/SpBundle instead of this repo


Not able to spawn Airbyte connectors on Jitsu running on Kubernetes using DinD

Environment

Deploy method: jitsucom/jitsu docker
Jitsu version:  latest
OS: Linux
Cloud Provider: AWS EKS
Docker Params (if deployed with docker): Mounting all required file paths including airbyte & docker.sock

Description
Trying to deploy Jitsu on kubernetes with docker as DinD mode mentioned in the issue https://github.com/jitsucom/jitsu/issues/545.
The log says ❌ Airbyte integration is disabled: failed to inspect current docker container by containerID [jitsu]: Error: No such container: jitsu
Steps to reproduce
Steps to reproduce the behavior:
This is the configuration i have used to create a run Jitsu on K8s.
apiVersion: apps/v1
kind: Deployment
metadata:
    name: jitsu
spec:
    selector:
        matchLabels:
            app: jitsu
    template:
        metadata:
            labels:
                app: jitsu
        spec:
            hostname: jitsu
            volumes:
                - name: jitsu
                  persistentVolumeClaim:
                      claimName: jitsu-claim
                - name: docker-graph-storage
                  emptyDir: {}
            containers:
                - name: jitsu
                  image: jitsucom/jitsu:latest
                  imagePullPolicy: IfNotPresent
                  env:
                      - name: CONFIGURATOR_ADMIN_TOKEN
                        value: kwaTnRNzdgMZ5gqDUAf3vNupvKw8Q9yY
                      - name: SERVER_ADMIN_TOKEN
                        value: TKNu4iPuJcvhsdfX9ukFqF6VhXmtpc4j
                      - name: UI_AUTH_ACCESS_SECRET
                        value: 8ydWLwpRWnHnKPMFq24a2iBZcMoyMfYL
                      - name: UI_AUTH_REFRESH_SECRET
                        value: EtahBGriN99tGt5EXQtYDBtoiMVnyWmA
                      - name: REDIS_URL
                        value: redis://redis:15601
                      - name: DOCKER_HOST
                        value: tcp://localhost:2375
                  ports:
                      - containerPort: 8000
                - name: dind-daemon
                  image: docker:dind
                  command: ["dockerd", "--host", "tcp://<IP_ADDRESS>:2375"]
                  securityContext:
                      privileged: true
                  volumeMounts:
                    - mountPath: /home/configurator/data/logs
                      name: docker-graph-storage
                    - mountPath: /home/eventnative/data/logs/events
                      name: docker-graph-storage
                    - mountPath: /home/eventnative/data/logs
                      name: docker-graph-storage
                    - mountPath: /home/eventnative/data/airbyte
                      name: docker-graph-storage
                    - mountPath: "/var/run/docker.sock"
                      name: docker-graph-storage

            initContainers:
                - name: init-jitsu
                  image: jitsucom/jitsu:latest
                  command: ['sh', '-c', 'set -x', 'chmod -R 777 /home/eventnative', 'chmod -R 777 /home/configurator']
                  volumeMounts:
                      - mountPath: /home/configurator/data/logs
                        name: docker-graph-storage
                      - mountPath: /home/eventnative/data/logs/events
                        name: docker-graph-storage
                      - mountPath: /home/eventnative/data/logs
                        name: docker-graph-storage
                      - mountPath: /home/eventnative/data/airbyte
                        name: docker-graph-storage
                      - mountPath: "/var/run/docker.sock"
                        name: docker-graph-storage

Expected behavior
Expected behaviour is to successfully run Airbyte connectors on kubernetes in DinD or DooD modes.
Actual behavior
Currently when Jitsu starts and i create a simple file (airbyte) source. It throws following error in logs :
2022-03-16 20:33:06 [ERROR]: source-file-0.2.9-6b1b8385-2f21-400c-8c9a-1909cc2b4136 airbyte runner closing failed. Killing. Closing error: exit status 1

And when i press next after filling details of File connector, it throws following error :
2022-03-16 20:33:49 [ERROR]: Error executing airbyte check:
	{"type": "LOG", "log": {"level": "FATAL", "message": "[Errno 2] No such file or directory: '/tmp/airbyte/7cda4ceffc75441495f5e826c23ccb89/8fda8740c9374561b9f9fbf486229556.json'\nTraceback (most recent call last):\n  File \"/airbyte/integration_code/main.py\", line 13, in <module>\n    launch(source, sys.argv[1:])\n  File \"/usr/local/lib/python3.7/site-packages/airbyte_cdk/entrypoint.py\", line 127, in launch\n    for message in source_entrypoint.run(parsed_args):\n  File \"/usr/local/lib/python3.7/site-packages/airbyte_cdk/entrypoint.py\", line 80, in run\n    self.configure_sentry(source_spec.connectionSpecification, parsed_args)\n  File \"/usr/local/lib/python3.7/site-packages/airbyte_cdk/entrypoint.py\", line 67, in configure_sentry\n    config = self.source.read_config(parsed_args.config)\n  File \"/usr/local/lib/python3.7/site-packages/airbyte_cdk/connector.py\", line 42, in read_config\n    with open(config_path, \"r\") as file:\nFileNotFoundError: [Errno 2] No such file or directory: '/tmp/airbyte/7cda4ceffc75441495f5e826c23ccb89/8fda8740c9374561b9f9fbf486229556.json'"}}

@shubham19may
Don't kill the messenger, It's only my personal opinion.
I think that DnD and k8s are not things you should run in prod, for a myriad of reasons.
We installed Airbyte on k8s as is.

I think that DnD on k8s is not something you should run in prod, for a myriad of reasons.
We installed Airbyte on k8s as is. It's a much cleaner approach IMHO.

@viggin543 so basically you are only using Airbyte on K8s rather than Jitsu?
And you are suggesting we should only use Airbyte because its more clean?

@shubham19may clean === safe, monitorable, scalble etc..

The fix was committed to master. We are going to give it day or two of extra testing before releasing new hotfix image.
Also i've noticed few mistakes in config example you have provided. E.g. all volumes was mounted to dind-daemon container instead of jitsu. init-jitsu command wouldn't  run anything except of set -x
I suggest config example with fixes and simplifications, and explanation how jitsu - DinD - airbyte schema works:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: jitsu
  labels:
    app: jitsu
spec:
  selector:
    matchLabels:
      app: jitsu
  template:
    metadata:
      name: jitsu
      labels:
        app: jitsu
    spec:
      hostname: jitsu
      volumes:
        - name: server-logs
          hostPath:
            path: /data/jitsu/server-logs
            type: DirectoryOrCreate
        - name: configutator-logs
          hostPath:
            path: /data/jitsu/configutator-logs
            type: DirectoryOrCreate
        - name: jitsu-workspace
          emptyDir: {}
        - name: docker-graph-storage
          emptyDir: {}
      containers:
        - name: jitsu
          image: jitsucom/jitsu:latest
          imagePullPolicy: IfNotPresent
          env:
            - name: CONFIGURATOR_ADMIN_TOKEN
              value: admin_token
            - name: SERVER_ADMIN_TOKEN
              value: admin_token
            - name: REDIS_URL
              value: redis://host.minikube.internal:6379
            - name: DOCKER_HOST
              value: tcp://localhost:2375
            - name: SERVER_VOLUMES_WORKSPACE
              value: /tmp/airbyte/
          ports:
            - containerPort: 8000
          volumeMounts:
            - mountPath: /home/configurator/data/logs
              name: configutator-logs
            - mountPath: /home/eventnative/data/logs
              name: server-logs
            - mountPath: /home/eventnative/data/airbyte
              name: jitsu-workspace

        - name: dind-daemon
          image: docker:dind
          command: [ "dockerd", "--host", "tcp://<IP_ADDRESS>:2375" ]
          securityContext:
            privileged: true
          volumeMounts:
            - name: docker-graph-storage
              mountPath: /var/lib/docker
            - name: jitsu-workspace
              mountPath: /tmp/airbyte/

      initContainers:
        - name: init-jitsu
          image: jitsucom/jitsu:latest
          command: ["/bin/bash"]
          args: ['-c', 'sudo chmod -R 777 /home/eventnative; sudo chmod -R 777 /home/configurator']
          volumeMounts:
            - mountPath: /home/configurator/data/logs
              name: configutator-logs
            - mountPath: /home/eventnative/data/logs
              name: server-logs

jitsu -> airbyte bridge through Docker-in-Docker

Jitsu configured with SERVER_VOLUMES_WORKSPACE=/tmp/airbyte/
Before Jitsu runs Airbyte source it generates config files and put them into /home/eventnative/data/airbyte directory
/home/eventnative/data/airbyte directory mapped to jitsu-workspace named volume
When Jitsu runs Airbyte source it uses command like docker run -v SERVER_VOLUMES_WORKSPACE:/tmp/airbyte/ ... --config /tmp/airbyte/config.json which translate to docker run -v /tmp/airbyte/:/tmp/airbyte/ ... --config /tmp/airbyte/config.json
Since jitsu container have DOCKER_HOST=tcp://localhost:2375 airbyte source image actually runs inside dind-daemon container
dind-daemon container have /tmp/airbyte/ path mapped as jitsu-workspace named volume
that is how through  jitsu-workspace named volume jitsu and airbyte source manage to interchange files

We will update docs with that information soon.

@absorbb thank you for the fix. I appreciate the prompt response. Will try it out.
One more thing though, DinD mode is not recommended to run things in prod :
http://jpetazzo.github.io/2015/09/03/do-not-use-docker-in-docker-for-ci/
Do you think DinD approach is good solution for concurrent accesses (Using multiple airbyte sources or same source multiple times) ?

@shubham19may Thank you for the article. It is very interesting.
I think concerns explained in this article dont apply to Jitsu case.
Our use case is quite simple. We dont run multiple docker instances and don't share anything critical between them.
At max we have docker from kubernetes contatiner runtime
and one docker instance from dind-daemon container. And nothing that may affect stability of docker shared between them - we don't share images, caches,etc - only source config jsons.
I don't see reasons why running multiple airbyte sources may be a problem here.
Regarding using in prod. Right now we don't test Jitsu on any k8s setups. But we have plans to start doing that in the future. I guess you need to make a decision based on your testing results.

@absorbb
Making jitsu cloud-native ( Jitsu + k8s ) will greatly increase adoption, and can really explode in popularity.
A high-quality ( bitnami style ) helm chart will make running jitsu simple and safe.
It will allow one-click production-ready installation.  that can be configured and fine-tuned to every ORG needs.
There will always be people that would like to use a managed Jitsu regardless of how easy it is to run on your personal cloud.  So don't worry about that.
And cloud-native means that you don't use DnD.   You deploy stuff horizontally.
Airbyte has the same problems. Maybe they want it to be a bit hard to run it yourself...
Running Jitsu k8s on prod should be as easy as running a MongoDB, redis, or Postgres with regards to DevOps resources required to operate and install it.

Fixed in 1.41.3
@viggin543
Thank you for sharing this. We started moving to this direction. We don't forget about your helm chart work either 👍


Show placeholder in place of a deleted or moved event

It’s implemented in the view layer, but currently doesn’t load data for it.

I don't get it. This is the placeholder:

We should choose another icon for that, but that's minor, it's working.
And data should be loaded, it's implemented here: https://github.com/cvut/fittable/blob/master/src/app.js#L261
I don't know what's wrong with that.

Well, it apparently doesn’t work; for example, look at BI-AAG on week 44 (Wednesday) and 47 (Tuesday).


set ec2-user password to a known password

This module has a function to set the password for the ec2-user to a random password. To help with instance automation, would it be possible to allow users to set the password for the ec2-user to a predefined value?
This will then enable interaction with the keychain etc during the process of building AMI's for use within our organisation.

Hey @mieliespoor, the short answer to your question is that ec2-macos-init does not support setting pre-defined user passwords.
That being said, you would be able to have ec2-macos-init run a shell command to change the password. This can be done either in userdata or by creating a new module in the config.
The command that ec2-macos-init uses to set the random password is the following:
dscl . -passwd /Users/ec2-user <random password>

However, I’m not comfortable with this type of solution as it would mean storing the password in plain-text. Please refer to the first warning in the EC2 metadata and userdata docs with regards to storing sensitive data.

Although you can only access instance metadata and user data from within the instance itself, the data is not protected by authentication or cryptographic methods. Anyone who has direct access to the instance, and potentially any software running on the instance, can view its metadata. Therefore, you should not store sensitive data, such as passwords or long-lived encryption keys, as user data.

What if you could script this to retrieve a pre-defined encrypted password from Secrets Manager? This assuming you have access to the key and Secrets Manager of course. Could you assign that value to a variable and use that to set the password? It wouldn't be plan text then right?

The command to do so would be
/opt/homebrew/bin/aws --region <name_of_your_region> secretsmanager get-secret-value --secret-id <name_of_your_secret> --query SecretString --output text

The EC2 instance must have permission to make this call.  This is done by attaching a role to the EC2 instance. The minimum set of permission to authorize the above is
{
    "Version": "2012-10-17",
    "Statement": [
        {
            "Sid": "SecretsManagerAccess",
            "Effect": "Allow",
            "Action": "secretsmanager:GetSecretValue",
            "Resource": [
                "arn:aws:secretsmanager:YOUR_REGION:YOUR_ACCOUNT_ID:secret:name_of_your_secret"
            ]
        }
    ]
}

So this would presumably be a valid script:
#!/bin/bash

Given the two commands above. This could set the password.
#!/bin/bash
PASSWORD=$(aws --region us-east-1 secretsmanager get-secret-value \
    --secret-id test/ec2-user/password | jq --raw-output .SecureString \
    | jq -r ."password")

dscl . -passwd ?users/ec2-user $PASSWORD

Hi guys,
Yes thanks. I forgot to update this thread, but I opted to do that because the risks with having a password known because of an instance build is too big - as Matt also alluded. Matt's response then helped us consider secrets manager, so we will be using the aws cli to pull a password from secrets manager and set that during the instance provisioning. We will then also make use of the rotation capabilities of secrets manager to rotate the password regularly.
What I should however add is that we are also making changes to our cicd pipelines to not rely on the default login keychain anymore but rather create a keychain specifically for the cicd pipeline. This will greatly reduce the need to know the user password during instance provisioning.


Update comments, fix Doxygen problems and sync with comments in C++

Thank you @magol !
Thats a good and clean PR ! :smiley:

What did you wanted to say by:
Todo:
Update the second paragraph in this documentation.

Otherwise, I converted doxygen most of the commands (like \see) to the xml version to try to keep it clean.
https://github.com/dokan-dev/dokan-dotnet/commit/8bf441387096764f3fdeaa4db567019c287ca153
I know that some commands does not exist in xml but at least we do our best :smile:
I also found that link to NativeMethods does not work because it is an internal class and doxygen do not want to document them :smile:
Keep up the good work !

Ups, it was supposed to fix the \todo before PR.
I did write the \todo because that I don't understand the second paragraph in the comment, I do not really understand the meaning of:
/// In case <see cref="FileMode.OpenOrCreate"/> and
/// <see cref="FileMode.Create"/> are opening successfully a already
/// existing file, you have to <c>SetLastError(ERROR_ALREADY_EXISTS)</c>.

Can you help me with that?

What parameter is it that can have the FileModes
Instead of SetLastError(ERROR_ALREADY_EXISTS), do I return that flag?

For references to private structs, maybe we can use \if in some way, so that it don't be in the documentation, but still be in the code for reference.  And if we for some reason want to generate a more detailed documentation with the privates, it is possible.

This paragraph should be edite for ZwCreateFileProxy since there is no FileMode but rawDesiredAccess (that has C flags).
For IDokanOperations.CreateFile the paragraph is correct but should probably say it is the modeparam.
SetLastError(ERROR_ALREADY_EXISTS) it is just before returning STATUS_SUCCESS. (the last error is retrieved later by dokan library).
I never used \if have you already used it on doxygen ?
Just to know, do you use doxywizard for testing the documentation ?

If I understand you right:
If parameter mode==FileMode.OpenOrCreate | FileMode.Create and the file specified in fileName already exist, you have to use the code below and then return NtStatus.Success
public const int ERROR_ALREADY_EXISTS = 183;
[DllImport("kernel32.dll", SetLastError = true)]
static extern void SetLastError(uint dwErrorCode);
...
SetLastError(ERROR_ALREADY_EXISTS);

I hope I have misunderstood it. In C #, it feels like a strange solution.

It is exactly :+1:
The goal is to notify that the file has not be created by the open but already was there.
Do you have another idea how this could be handle ?

Maybe we can add a new parameter to CreateFile, and then let ZwCreateFile use SetLastError:
NtStatus CreateFile(string fileName, FileAccess access, FileShare share, FileMode mode,
            FileOptions options, FileAttributes attributes, out bool alreadyExists, DokanFileInfo info);

This should probably be done yes.
At first I tought creating a variable just for this would be too much but compared to make the user dllimport it is much better :smile:

Great 😊
Then I think all this out parameters is bad practice in c#.
Beter to return a structures with the returned data.   It becomes some new structures,  but I think that c# programmers feels more comfortable.
But it is a major change.

which other  out parameters are you pointing ?

It is a lot of function in this interface that have out parameters. But I don't know if it is a good idea to return structs instead of NtStatus.  ;)

Better to keep the same logic on each callback rather than have sometimes a struct because there is a out parameter.

You are right. There are more disadvantages than advantages with my proposal.


Onboarding broken after patch

Description:
Related issue (if applicable): fixes #21777
Onboarding isn't working after #21777 . This fixes the issue so that onboarding now works, although I'm not sure it is consistent with the intent of the original patch. Possibly the front end needs to have username be renamed client_id ? I haven't done any front end stuff as yet so can't confirm.
To reproduce:
hass -c new_folder

Attempt to do the onboarding login, you get a data validation error. This patches the issue.
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

Make sure you run the latest frontend.
The PR that you linked, linked to a PR in the frontend. That PR diff shows we are adding client_id: https://github.com/home-assistant/home-assistant-polymer/pull/2894/files#diff-f756c17d3b2e50b6d92ecc0f523e0934R13


Add missing break on RemoveDisallowedJobs

Wait, are jobs removed from the list completely if you don't meet the time requirements?

Wait, are jobs removed from the list completely if you don't meet the time requirements?

Yes. This is used for gameticker when spawning a player to know which jobs they aren't allowed.

Ah, sure.

Feel free to just merg, tbh


Clickup integration not working

OS version: 10.15.5
Browser version: Chrome 85
Extension version: 1.66.1

Relevant integration (if any):
ClickUp
🐛 Describe the bug
TB is not showing in Clickup

Expected behaviour
TB should show
Other details or context
Slack
case

case reference

I don't know what the issue is for the user, maybe they have something cached.
For me, the clickup looks a bit different (note the dark theme) and I can clearly see the Toggl Button.
It also correctly starts/stops the time entry.

@nidarasheed can you check with the user whether the problem still appears for them?

@kornelijepetak This is from my Clickup account

In the extension  settings

Version 1.66.3
I also cleared cache and tried again, no change. :(

Since this integration (for now) depends on them providing a UI element in which we inject the button, I have contacted Clickup about this and am awaiting their response.

I got back from the ClickApp support.
This is a non-issue.
The user simply needs to have two things turned on:

Time Tracking in their ClickApps settings (see screenshot below)
The permission for ClickApp enabled in Toggl Track browser extension integration settings

The button will then normally appear.


Run tests against Apache Traffic Server 9.1

This PR closes #7044 by making t3c integration tests, the tc-health-client integration tests, and CDN in a Box use ATS 9.1.x instead of what they currently use, ATS 8.1.x.

Which Traffic Control components are affected by this PR?

Traffic Control Cache Config (t3c, formerly ORT) - tests
Traffic Control Health Client (tc-health-client) - tests
CDN in a Box - ATS version used by Edge and Mid caches
Automation - GitHub Actions

What is the best way to verify this PR?

Verify that the GitHub Actions for CDN in a Box, the t3c integration tests, and the tc-health-client integration tests pass
PR submission checklist

[x] This PR affects tests
[x] This PR has documentation
[x] This PR has a CHANGELOG.md entry
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Rebased to fix a changelog merge conflict

Rebased to fix another merge conflict


Support kernel detection (pull) in addRemoteJupyterServer API

With the movement to multiple remote kernel finder and making the kernel detection push model (other than pull), we might want to change how the extension api addRemoteJupyterServer API works to better support third party extensions.
Firstly the assumptions I took are the remote kernel finders work in such fashion:

When a jupyter server uri is added to the storage, we will create a RemoteKernelFinder for it, and register the RemoteKernelFinder to the kernel finder registry.
A kernel spec/session will be kicked off asynchronously. When kernel specs/sessions are found, it will broadcast the event and the Controller component will create controllers for it based on the strategy users choose (Traditional or the new workflow @IanMatthewHuff is working on).

With above in mind, we also want to support third party extensions to set a remote jupyter server uri, and wait for NotebookControllers to be created for them. Thus the addRemoteJupyterServer will need to

[x] Register a remote jupyter server uri to the storage
[x] A new RemoteKernelFinder is created for the server uri and registered to the kernel finder registry
[ ] We can then get the RemoteKernelFinder instance for the server uri
[ ] We can then wait for the RemoteKernelFinder to find the kernel specs/sessions lookup.
[ ] We can then ask the ControllerLoader to create NotebookControllers for the newly found kernels.
[ ] Now the API method can resolve

I like it. Lines up with how I'm thinking about the kernel sources.
One question. Since we were discussing this a bit on the other item. Where exactly is the ownership between points 1 and 2 on your list? Right now I kinda had the UniversalRemoteKernelFinderController doing this (watching the server list, then creating individual kernel finders) but I'm not fully sure about that I did design. Feels a bit awkward to me.
Is the storage doing it? That also seems like an odd responsibility for a storage class.
Also I'm not sure if this is just a wording difference, but this here:

We can then ask the ControllerLoader to create NotebookControllers for the newly found kernels.

I thought that everything from the ControllerLoader perspective was running through the single main kernel finder and the message to see when kernels were updated. Do you just mean signaling that kernels were updated at this point (which is what I would assume)? Or did you mean actually some other channel to the ControllerLoader?

One question. Since we were discussing this a bit on the other item. Where exactly is the ownership between points 1 and 2 on your list

Since we need to store the remote uri list, we always need a storage (which is ServerUriStorage now) and a manager which can create kernel finders from the entries in the storage. I'm wondering if we can do something similar to the Editor Types in VS Code core, the storage will contain both Type and additional info:
interface IKernelFinderInstance<T> {
  type: KernelFinderType;
  data: T;
}

then we can store a list instances in momento as
[
  { type: LocalKernelFinder }
  { type: UniversalRemoteKernelFinder, data: { uri } }
]

When we deserialize the cache from momento, we can look up the types and know how to create a kernel finder instance from it.
At the end, the ServerStorageUri and kernel finder manager would be the same thing.

This is archived through MRU kernel picker and will be tested there.


VM and Compiler options

fixes #130

On second thought, I'm gonna refactor this some more, so please hold on reviewing for now.

I'll also update the docs while I'm at it.


Create an intro

Add an intro about yourself: who are you, what is your strength, why are you here.

Created intro with no problems


[Soundcloud] 64k opus higher quality than 128k mp3

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm requesting a site-specific feature
[X] I've verified that I'm running yt-dlp version 2023.03.04 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
No response
Example URLs
can be any, heres just the one i used to test: https://soundcloud.com/hykeemcarter/killstreaks-feat-don-toliver
Provide a description that is worded well enough to be understood
when downloading audio from soundcloud, yt-dlp prefers the 128k opus format to the 64k opus format that soundcloud provides for best audio quality

when using spek (https://www.spek.cc/) to analyze the spectrums, the 128k mp3 outputs a file peaking at 16khz (128kbps) while the 64k opus outputs a file peaking at 20khz (320kbps)

should yt-dlp prefer opus as it outputs a higher quality file despite being a lower bitrate?
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
C:\Users\Techno\Downloads\fs-new>yt-dlp -FvU "https://soundcloud.com/hykeemcarter/killstreaks-feat-don-toliver"
[debug] Command-line config: ['-FvU', 'https://soundcloud.com/hykeemcarter/killstreaks-feat-don-toliver']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[392389b7d] (win_exe)
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.19045-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: ffmpeg 5.1-essentials_build-www.gyan.dev (setts), ffprobe 5.1-essentials_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.17, brotli-1.0.9, certifi-2022.12.07, mutagen-1.46.0, sqlite3-2.6.0, websockets-10.4
[debug] Proxy map: {}
[debug] Loaded 1786 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>Current Build Hash: 5590c57bd0433ed239a2deaaf92e2ad6f37fe50f53664c821575cafe106a9421
yt-dlp is up to date<EMAIL_ADDRESS>[soundcloud] Extracting URL: https://soundcloud.com/hykeemcarter/killstreaks-feat-don-toliver
[soundcloud] hykeemcarter/killstreaks-feat-don-toliver: Downloading info JSON
[soundcloud]<PHONE_NUMBER>: Downloading JSON metadata
[soundcloud]<PHONE_NUMBER>: Downloading JSON metadata
[soundcloud]<PHONE_NUMBER>: Downloading JSON metadata
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
[info] Available formats for<PHONE_NUMBER>:
ID           EXT  RESOLUTION │ PROTO │ VCODEC     ACODEC   ABR
──────────────────────────────────────────────────────────────
hls_opus_64  opus audio only │ m3u8  │ audio only unknown  64k
hls_mp3_128  mp3  audio only │ m3u8  │ audio only unknown 128k
http_mp3_128 mp3  audio only │ http  │ audio only unknown 128k

C:\Users\Techno\Downloads\fs-new>yt-dlp -vU "https://soundcloud.com/hykeemcarter/killstreaks-feat-don-toliver"
[debug] Command-line config: ['-vU', 'https://soundcloud.com/hykeemcarter/killstreaks-feat-don-toliver']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[392389b7d] (win_exe)
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.19045-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: ffmpeg 5.1-essentials_build-www.gyan.dev (setts), ffprobe 5.1-essentials_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.17, brotli-1.0.9, certifi-2022.12.07, mutagen-1.46.0, sqlite3-2.6.0, websockets-10.4
[debug] Proxy map: {}
[debug] Loaded 1786 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>Current Build Hash: 5590c57bd0433ed239a2deaaf92e2ad6f37fe50f53664c821575cafe106a9421
yt-dlp is up to date<EMAIL_ADDRESS>[soundcloud] Extracting URL: https://soundcloud.com/hykeemcarter/killstreaks-feat-don-toliver
[soundcloud] hykeemcarter/killstreaks-feat-don-toliver: Downloading info JSON
[soundcloud]<PHONE_NUMBER>: Downloading JSON metadata
[soundcloud]<PHONE_NUMBER>: Downloading JSON metadata
[soundcloud]<PHONE_NUMBER>: Downloading JSON metadata
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
[debug] Default format spec: bestvideo*+bestaudio/best
[info]<PHONE_NUMBER>: Downloading 1 format(s): http_mp3_128
[debug] Invoking http downloader on "https://cf-media.sndcdn.com/UUVklxNtpKAj.128.mp3?Policy=eyJTdGF0ZW1lbnQiOlt7IlJlc291cmNlIjoiKjovL2NmLW1lZGlhLnNuZGNkbi5jb20vVVVWa2x4TnRwS0FqLjEyOC5tcDMqIiwiQ29uZGl0aW9uIjp7IkRhdGVMZXNzVGhhbiI6eyJBV1M6RXBvY2hUaW1lIjoxNjgwMDUzMDk0fX19XX0_&Signature=cYqRz7Uzto2EWotGjfDtRJ0zLXlSiNKdOJictOvWO0jAlVv7U7r840EvplpQ6sVBXcTIDehHlUkaCdOZQG6WaBFwUrAMV2o4E8LutrIKeDnWKvnuFscwVGbVEfZYnbykgQU4HlQri54MS0qcnFptxhfNaoOaXOuCFBllmS2NM9dJaFPdhBXSwE6Aw0~-vwoYEAEHIcaFG43crVTSRWWFQCbD8vpsC6EjumXRBBTc-~8qeQcqMyX18Ue1ENG8DPso1riElvuVYfkG0F9EWVcNOshtoiw8QUeEgurqHPJL4D6SXfjOZTSLCYuLjs1or0EBLc6EW6YE4RFUyYkErjHuFw__&Key-Pair-Id=APKAI6TU7MMXM5DG6EPQ"
[debug] File locking is not supported. Proceeding without locking
[download] Destination: killstreaks (feat. Don Toliver & PinkPantheress) [1371553090].mp3
[download] 100% of    3.29MiB in 00:00:00 at 4.11MiB/s

C:\Users\Techno\Downloads\fs-new>

Issue is that acodec is not being extracted. If extractor implements it, sorting will fix itself. Alternately, it may be reasonable to have some core code to guess the codec from ext

Closed by 86a972033e05fea80e5fe7f2aff6723dbe2f3952 and a2d0840739cddd585d24e0ce4796394fc8a4fa2e


Make delete command more regular

Right now, the command to delete a connector is kcctl delete <connector name>. This is rather irregular, as for all the other command, the resource to act on is to be given as a sub-resource, e.g. kcctl pause connector <connector name>. Arguably, this is a bit more to type for those cases where there only is one kind of matching resource, but I still believe this is the better approach for the sake of regularity (e.g. kcctl get ... supports multiple resource types). I therefor suggest to apply the same reasoning to delete and change it into kcctl delete connector <connector name>.

@C0urante, curious about your take.

Totally agree--it threw me the first time I tried to delete a connector that I didn't have to specify the resource type 👍

Done.


Get-MpPreference fails on initial call if current location path contains single quotation mark

Steps to reproduce
Start pwsh.exe and navigate to a path that includes a single quotation mark, then run the Get-MpPreference cmdlet twice.
Alternatively, copy and run:
New-Item "$HOME\Desktop\quotation'error" -ItemType Directory | Out-Null
Set-Location "$HOME\Desktop\quotation'error"
Get-MpPreference
Get-MpPreference
Set-Location ..
Remove-Item "$HOME\Desktop\quotation'error" -Confirm

Note: The error will only occur on the initial call of the Get-MpPreference cmdlet, to reproduce it again you have to restart pwsh.exe.
Expected behavior
Get-MpPreference runs twice.
Actual behavior
Get-MpPreference throws an error on the initial call.

The 'Get-MpPreference' command was found in the module 'ConfigDefender', but the module could not be loaded. For more information, run 'Import-Module ConfigDefender'.

From the second call on, the cmdlet runs fine.
Environment data

Name                           Value
----                           -----
PSVersion                      7.1.3
PSEdition                      Core
GitCommitId                    7.1.3
OS                             Microsoft Windows 10.0.19042
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Additional info

Works fine in PS5.1
Works fine when location path doesn't contain a single quotation mark
Error already present in 7.0.3
Tested on my system and in a VM (both Win 10)
Possible workaround:Get-MpPreference 2>&1>$null
Get-MpPreference

Running Import-Module ConfigDefender will also throw an error on initial call if PWD contains a single quotation mark (and run fine afterwards):
WARNING: Module ConfigDefender is loaded in Windows PowerShell using WinPSCompatSession remoting session; please note that all input and output of commands from this module will be deserialized objects. If you want to load this module into PowerShell please use 'Import-Module -SkipEditionCheck' syntax.
Import-Module:
Line |
   1 |  Import-Module ConfigDefender
     |  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | The string is missing the terminator: '.

Note, that both commands run fine after one of them threw an error.


⚠ SpeakUpConference.com has degraded performance

In 8b05b70, SpeakUpConference.com (https://www.speakupconference.com) experienced degraded performance:

HTTP code: 200
Response time: 8826 ms

Resolved: SpeakUpConference.com performance has improved in 46f5180 after 11 minutes.


Hotfix the large number of connections

Re-purposes discovery_only_if_under_num into allow_out_only_if_under_num, which blocks all new outgoing connection attempts after the limit has been reached.
The limit right now is set to value-of---out-peers + 15. Also note that it is possible that we go slightly higher than the limit, as we don't cancel pending connections. Only new connections are forbidden when above the limit.
Also enables the incoming_limit feature of the SwarmBuilder to be set to the value of --in-peers.
This is obviously not great, and will possibly make discovery work way less greatly, but it should hotfix the issue with the large number of connections which we've been seeing.

I reverted the addition of incoming_limit, as I realized that this was limiting only the number of simultaneous non-negotiated connections, which is irrelevant for this issue.

So, for my understanding, in contrast to the existing limit which affects only when a new DHT random lookup is started, this change is supposed to even cut short such a query via returning early from addresses_of_peer? Note though that I think this affects all Kademlia queries, not just these random lookups, i.e. the actual DHT put/get operations that substrate uses will also be affected.

I think this affects all Kademlia queries, not just these random lookups, i.e. the actual DHT put/get operations that substrate uses will also be affected.

Normally it shouldn't, as the response is fetched only from the k-buckets:
https://github.com/libp2p/rust-libp2p/blob/b6a93b3c5e6242683d745e067f55f5702692a769/protocols/kad/src/behaviour.rs#L551-L562

I think this affects all Kademlia queries, not just these random lookups, i.e. the actual DHT put/get operations that substrate uses will also be affected.

Normally it shouldn't, as the response is fetched only from the k-buckets:
https://github.com/libp2p/rust-libp2p/blob/b6a93b3c5e6242683d745e067f55f5702692a769/protocols/kad/src/behaviour.rs#L551-L562

That is how a request is answered. If a node performs a get or put operation on a Kademlia DHT, the same iterative query towards the k closest nodes to the key happens as during a get_closest_peers with a random ID. These queries will then also be cut short. But maybe that is what you want here and why this change seems "effective", I don't know.

But maybe that is what you want here and why this change seems "effective"

Ah, yes that's what I want here.

(note: this PR is being put on ice while we try another fix)

Let's close this, since the origin issue has been fixed by #5086


Update configure-read-scale-availability-groups.md

add high level instructions on creating listener when cluster manager not present

@suresh-kandoth : Thanks for your contribution! The author(s) have been notified to review your proposed change.

thanks @suresh-kandoth! #sign-off


Marketplace: Automated tests

What automated tests would we like to add for vetting the marketplace offering?

We have test automation around the CNAB bundle verification (helm chart and ARM template linting basically).
A reasonable next test would be a helm install smoke test and deploying a sample application.


Direct readers to SSMS page after introduction

Direct readers to more information about SSMS and how to download it. This is also inline with the previous section where links to relevant topics are given after each introduction.

@oswellchan : Thanks for your contribution! The author, @stevestein, has been notified to review your proposed change.

Thank you @oswellchan merging.


mouse position to cam position

hi everybody... i have a question, i want to map current mouse position to a position a camera view... the main goal is to let people drag things from a 2D canvas to the 3D space...
i would really thank you if you can give me any help...
@jdf  and thanks for your wonderful library...

This question should probably go to the Processing forum instead.

Also, thanks for the kind word. :)

@jdf, thanks for your response... i'm deep into the problem and learned a lot in the process :-) thanks again


Improve modularity of Bus code

the ozpIwc namespace for the ozpIwc-bus library is flat and disorganized. For developer's who dig through running code to gain an understanding this is nightmarish.
Looking to break the library's heiarchy down into a more walkable structure,ex:
ozpIwc.apis
ozpIwc.apis.error
ozpIwc.apis.services
ozpIwc.apis.nodes

To prevent lengthy function calls like:
ozpIwc.apis.services.DataApi.prototype.someFunc = function(){...};

Implement a modular/submodular pattern to improve readability, and have the added benefit of easier component swapping in development:
ozpIwc.apis.services = (function(base, services){

    var DataApi = ozpIwc.util.extend(base,function(){...});
    DataApi.prototype.someFunc = function(){...};

    services.DataApi = DataApi;
    return services;

})(ozpIwc.apis.ApiBase, ozpIwc.apis.services || {});

Note: Dependency orders still apply when using the modular pattern, so ordering in grunt-concat still applies

merged with PR #332


pin binary dependency versions

This lets us track binary dependencies through dependabot without having to maintain a bin/ folder of magic binaries.

I'm going to leave converting the tools under /operator to use these for a separate PR once we start working on stuff there.


Upgrade to latest

I had to fix some build issues after upgrading to the latest of Fundamentals and SDK
┆Issue is synchronized with this Asana task

I approve of these changes


Adjust codeql.yml and badge

ssia

Coverage remained the same at 100.0% when pulling<PHONE_NUMBER>3d7c2b0a3e11ff5fb94c6a7484ab95 on codeql into fcc20a369b7ac9cb282740b4b42dff946ae8c380 on main.


Devious Minds Quest Helper does not detect orb in pouch

On the "gift for entrana" step, quest helper does not move on to the next step after filling the large rune pouch with the orb even after checking the pouch and using the orb directly on the pouch.

fixed with https://github.com/Zoinkwiz/quest-helper/commit/8cb3b02ae294a44c959746e78b384d490d6640f2


Doc/install/all/ddns.md: Add DDNS configuration doc

Add a new doc for using custom DNS server rather than cloud
provider DNS. Add new install/all directory for generic items
applicable to two or more providers.
Ref https://jira.prod.coreos.systems/browse/DOCS-61

@joshix Just as a reference point, the docs for the Terraform DNS provider can be found here.

@justaugustus Thanks for the notes. Tuned up, please take a look for a thumbs up.

@joshix LGTM! Thanks for the fixes. :)


Is this repository still maintained?

I wonder, because there has been months since we don't hear from the author.

Not yet now. because I'm busy. But I will be back to maintain this library.

That's very nice to hear... thanks!!

Any plans?
After some time, I am afraid Forks will be unavoidable, and that would be a real pity.

I am writing my own support library and using this repo for initial reference. Here's the initial work: https://gist.github.com/macu/3baefec350a4af056773


Add converter as an alias of openapi attribute

Closes #5
Adds an alias of the openapi attribute to converter for apispec >= 3.0.0.

Thanks, released version 2.1.0.


name conflict between imports and underscore prefixed function-scoped properties

Describe the bug
This is a regression - works fine for me in 1.2.74
During transpilation imports are renamed to file name prefixed with underscore. If you define a class property with the same name, it will instead overshadow the import.
This works fine if you define simple variable with the same name - transpiler resolves the conflict by using number as a suffix.
Input code
bar.ts
export const bar = () => console.log("foo");

foo.ts
import { bar } from "./bar";

const makeX = () => {
  const _bar = () => bar();
  return { _bar };
}

makeX()._bar();

Throws:
        this._bar = ()=>(0, _bar).bar()
                                  ^

TypeError: (0 , _bar).bar is not a function

Config
{
  "env": {
    "targets": {
      "node": "16.8"
    }
  },
  "jsc": {
    "target": "es2019",
    "parser": {
      "syntax": "typescript",
      "tsx": true
    }
  },
  "module": {
    "type": "commonjs"
  },
  "sourceMaps": "inline"
}

Expected behavior
Generated import names should not conflict with class properties after transpilation.
Version
The version of @swc/core: 1.2.84

Where does this._bar = ()=>(0, _bar).bar() come from?
It's not reproducible with the input you provided.

Sorry - this error gets throw when I run node_modules/.bin/swc-node foo.ts:
> node_modules/.bin/swc-node foo.ts
/private/tmp/swc-test/foo.ts:4
    const _bar = ()=>(0, _bar).bar()
                               ^

TypeError: (0 , _bar).bar is not a function
    at Object._bar (/private/tmp/swc-test/foo.ts:4:32)
    at Object.<anonymous> (/private/tmp/swc-test/foo.ts:10:9)

Still a problem if a function is defined within scope, but no exported. It will generate name conflict if foo.ts is changed to below:
import { bar } from "./bar";

const makeX = () => {
  const _bar = () => bar();

  const alfa = () => _bar();

  return { alfa };
}

Should I submit as a separate issue?


Unable to Validate OTP

#199  OTP..
Unable to Validate OTP
Response: {"message": "Endpoint request timed out"}
Retry with ? (y/n Default y):
@pallupz @vishalv26

getting same error can anyone help?

it should be fixed in the latest code


ISPN-7324 DDAsyncInterceptor indirection slows down replicated reads

https://issues.jboss.org/browse/ISPN-7324

+0.1
I've heard instanceof checks could be tricky, because the type from the class hierarchy is cached. Do you know about some good read on this topic?

I'm using https://shipilev.net/blog/2015/black-magic-method-dispatch/ as a reference. One of the experiments he tries is to "peel off the hottest type with manual checks" to improve performance for megamorphic calls.

That's about megamorphic calls, but manual instanceof improves perf by caching the hierarchy, and that's where my knowledge is below freezing point. See https://issues.apache.org/jira/secure/attachment/12833228/reduce_instanceof_typechecking.pdf

@rvansa Yeah, that link doesn't sound very encouraging... the theory is that C2 will have better information on the else branch, e.g. when you have 66% reads, 33% writes, 1% "other".
And it actually worked beautifully for reads without transactions and with implicit transactions. But it actually made things much worse with explicit transactions, because there we have 33.3% read, 33.3% prepare, 33.3% commit, so I'm closing the PR.


Slack Invite Link gone

https://snapsvg.slack.com/shared_invite/MTM2NTE4MTk3MDYwLTE0ODYwODgzNzUtYjQ0YmM1N2U0Mg

It's a bummer this link doesn't work (in the readme or on http://snapsvg.io/support/). I'd love an invite to the slack room!

If you leave your email, I will add you.

@ibrierley I think most people use Slackin

https://github.com/rauchg/slackin

There's "Slack Invite Automation" too

https://github.com/outsideris/slack-invite-automation

Hey is the snap svg slack group still active? I am hoping to get an invite.

Hi, we just need your email address somehow. There's a thread in the Snap google group, if you want to reply to that, it should be able to be picked up.


Unable to build Platform benchmarks when uploading local .dll

I'm trying to measure the performance of a local build of the runtime, but am getting the following build error:
acovingt@DESKTOP-DOTNET-T2:~/projects/dotnet-benchmarking$ crank --config https://raw.githubusercontent.com/aspnet/Benchmarks/master/scenarios/platform.benchmarks.yml --config amd.benchmarks.yml --scenario plaintext --profile local --application.framework netcoreapp5.0 --application.options.outputFiles ~/projects/runtime/artifacts/bin/testhost/net6.0-Linux-Release-x64/shared/Microsoft.NETCore.App/6.0.0/System.Private.CoreLib.dll
[03:49:20.520] Running session 'd7c18df9ea6d4187aead247f460bd79e' with description ''
[03:49:20.563] Starting job 'application' ...
[03:49:20.566] Submitted job: http://localhost:5010/jobs/6
[03:49:21.582] Job has been selected by the server ...
[03:49:21.585] Uploading /home/acovingt/projects/runtime/artifacts/bin/testhost/net6.0-Linux-Release-x64/shared/Microsoft.NETCore.App/6.0.0/System.Private.CoreLib.dll
[03:49:21.665] Job is now building ... http://localhost:5010/jobs/6/buildlog
[03:49:31.676] Job failed on benchmark server, stopping...
Job failed at runtime:
Kestrel Platform-Level Application
/plaintext
/json
Args:

[03:49:31.682] Job has failed, interrupting benchmarks ...
[03:49:31.684] Stopping job 'application' ...
[03:49:32.692] Deleting job 'application' ...

| application         |        |
| ------------------- | ------ |
| Build Time (ms)     |  5,253 |
| Published Size (KB) | 94,305 |

If I drop the --application.options.outputFiles flag, then I get a normal run:
acovingt@DESKTOP-DOTNET-T2:~/projects/dotnet-benchmarking$ crank --config https://raw.githubusercontent.com/aspnet/Benchmarks/master/scenarios/platform.benchmarks.yml --config amd.benchmarks.yml --scenario plaintext --profile local --application.framework netcoreapp5.0
[03:50:54.670] Running session '8e9a5ede235943bb84dd17770b9e7681' with description ''
[03:50:54.714] Starting job 'application' ...
[03:50:54.717] Submitted job: http://localhost:5010/jobs/7
[03:50:55.729] Job has been selected by the server ...
[03:50:55.730] Job is now building ... http://localhost:5010/jobs/7/buildlog
[03:51:04.739] Job is running ... http://localhost:5010/jobs/7/output
[03:51:04.743] Starting job 'load' ...
[03:51:04.744] Submitted job: http://localhost:5010/jobs/8
[03:51:06.747] Job has been selected by the server ...
[03:51:06.748] Job is now building ... http://localhost:5010/jobs/8/buildlog
[03:51:13.755] Job is running ... http://localhost:5010/jobs/8/output
[03:51:32.770] Stopping job 'load' ...
[03:51:33.779] Deleting job 'load' ...
[03:51:33.780] Stopping job 'application' ...
[03:51:35.788] Deleting job 'application' ...

| application          |                                |
| -------------------- | ------------------------------ |
| CPU Usage (%)        |                             50 |
| Raw CPU Usage (%)    |                         604.39 |
| Working Set (MB)     |                             85 |
| Build Time (ms)      |                          4,883 |
| Start Time (ms)      |                            132 |
| Published Size (KB)  |                         94,305 |
| .NET Runtime Version | 5.0.0-rc.2.20475.5+c5a3f49c... |
| ASP.NET Core Version | 5.0.0-rc.2.20475.17+25bd01f... |

| load                   |            |
| ---------------------- | ---------- |
| CPU Usage (%)          |         51 |
| Raw CPU Usage (%)      |     609.60 |
| Working Set (MB)       |         50 |
| Build Time (ms)        |      3,914 |
| Start Time (ms)        |          0 |
| Published Size (KB)    |     76,404 |
| Requests/sec           |  5,799,441 |
| Requests               | 87,515,872 |
| Mean latency (ms)      |       6.52 |
| Max latency (ms)       |     246.88 |
| Bad responses          |          0 |
| Socket errors          |          0 |
| Read throughput (MB/s) |     696.88 |
| Latency 50th (ms)      |       3.32 |
| Latency 75th (ms)      |       8.52 |
| Latency 90th (ms)      |      15.42 |
| Latency 99th (ms)      |      43.58 |

Am I misunderstanding how to upload a local build of the runtime? Any help would be appreciated!

System.Private.CoreLib.dll might not be the only file necessary. But the most probably reason is that you are building a 6.0 version of the runtime while targeting a net5.0 environment.
On the command line you can change that with --application.framework net6.0 --application.channel edge and you will probably need a custom sdk version since right now it's pulling a 5.0 sdk: --application.sdkVersion 6.0.0-alpha.1.20525.3
Soon only the framework will need to be specified.

I see, I rebased my local build off of the release/5.0-rc2 branch instead of master which has all of the 5.0 version numbers instead of the 6.0 version numbers. Using that, the following worked for me:
 crank --config https://raw.githubusercontent.com/aspnet/Benchmarks/master/scenarios/platform.benchmarks.yml --config amd.benchmarks.yml --scenario plaintext --profile local --application.framework netcoreapp5.0 --application.options.outputFiles '/home/acovingt/projects/runtime/artifacts/bin/testhost/net5.0-Linux-Release-x64/shared/Microsoft.NETCore.App/5.0.0/*'

Probably overkill to send the entire testhost over, but I believe this is working now. Thanks for the timely response!

Yes, overkill, you can try a few combinations, even just with System.Private.CoreLib.dll now to see if that works. But if it's not too slow to send it, then you are fine.


command: aad o365group report activityfilecounts

implements #1285 and restructured the o365group reports.
Implements the suggestion of @garrytrinder in #1264 by changing the parameter --outputFile, using the new naming convention in all existing o365group reports.
Once this is merged in, I am going to implement #1286.

Coverage remained the same at 100.0% when pulling 6055e532822d9465c613143253e74b34562409cb on plamber:report-groupactivityfilecounts into<PHONE_NUMBER>8c5e5d70cb505e8ad390921a7c474e on pnp:dev.

Thanks for the heads up @VelinGeorgiev 👍

Merged manually. Thanks! 💪


[BD-35] PAR-285 feat: change breadcrumb component according to new design

@abutterworth
Ticket on Jira: https://openedx.atlassian.net/browse/PAR-285
Update breadcrumb component according to new design

Thanks for the pull request, @amalbas! I've created BLENDED-781 to keep track of it in Jira.
When this pull request is ready, tag your edX technical lead.

Coverage remained the same at 89.134% when pulling e176435e972d2a86ddb0028fd6bd96c0ddff7d32 on eduNEXT:ama/PAR-285 into 1f38b510acfa5c2fcfb22f9985cd02dda877cc4d on edx:master.

:tada: This PR is included in version 13.14.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Covid Tracker Android App

Title
Covid Tracker Android App
Project link
[Click here](Paste link of project here)
Domain
Add 'x' inside [ ] to cross check box.

[ ] App with Flutter
[x] App with Java
[ ] App with Kotlin
[ ] Web with Html,css javascript
[ ] Web with React

Please describe your feature.
Android Application to track the covid cases in a state or any of its district.
Describe the solution you'd like
The app provides a precise tracking facility to the users about the number of active cases, deaths , recovered cases in a state and districts.
Additional context
NA.

Sorry to say , please don't contribute as it is excluded from Hacktoberfest 2022.

Hello @DevilsAutumn Please assign me this project under hacktoberfest22.
I have done this project already.

Hello @DevilsAutumn Please assign me this project under hacktoberfest22.
I have done this project already.

This repo has been excluded from hacktoberfest.

Thanks for confirming.


Third Transform does not remove second Mobject [BUG-General]

Describe the bug
If we have a scene with three Transform animations one after the other, the last one does not replace the second object and a very weird artifact appears.
I scouted around to see if this is a known problem but I failed to find some documentation on it or a related issue here on GitHub.
To Reproduce
Save this code in a script called "test.py" and run the command "python -m manim test.py SquareToCircle -pl -i"
from manim import *

class SquareToCircle(Scene):
    def construct(self):

        text = Tex('Hello world')

        self.play(Write(text))
        self.wait()

        square = Square()
        square.flip(RIGHT)
        square.rotate(- 3 * TAU / 8)

        self.play(Transform(text, square))
        self.wait()

        circle = Circle()

        self.play(Transform(square, circle))

Expected behavior
In the above example, the square should be entirely replaced by the circle, but it's not.
Output Media Files

System Specifications
System Details

OS : Windows 10 Home v2004
RAM : 8 GB
Python version : Python 3.8.6 (tags/v3.8.6:db45529, Sep 23 2020, 15:52:53) [MSC v.1927 64 bit (AMD64)]
Installed modules :

Package           Version
----------------- -------
-umpy             1.19.4
cairocffi         1.2.0
CairoSVG          2.5.0
cffi              1.14.3
colorama          0.4.4
colour            0.1.5
commonmark        0.9.1
cssselect2        0.4.1
defusedxml        0.6.0
grpcio            1.33.2
grpcio-tools      1.33.2
html5lib          1.1
manimce           0.1.0
numpy             1.19.3
pangocairocffi    0.3.2
pangocffi         0.6.0
pathtools         0.1.2
Pillow            8.0.1
pip               20.2.4
progressbar       2.5
protobuf          3.13.0
pycairo           1.20.0
pycparser         2.20
pydub             0.24.1
Pygments          2.7.2
Pyphen            0.10.0
rich              9.2.0
scipy             1.5.4
setuptools        50.3.2
six               1.15.0
tinycss2          1.1.0
tqdm              4.51.0
typing-extensions <IP_ADDRESS>
watchdog          0.10.3
WeasyPrint        52.1
webencodings      0.5.1

Latex details

Distribution :
TeX Live version 2020
Installed packages :

i amsfonts: TeX fonts from the American Mathematical Society
i amsmath: AMS mathematical facilities for LaTeX
i atbegshi: Execute stuff at \shipout time
i atveryend: Hooks at the very end of a document
i auxhook: Hooks for auxiliary files
i babel: Multilingual support for Plain TeX or LaTeX
i babel-english: Babel support for English
i bibtex: Process bibliographies for LaTeX, etc
i bibtex.win32: win32 files of bibtex
i bigintcalc: Integer calculations on very large numbers
i bitset: Handle bit-vector datatype
i booktabs: Publication quality tables in LaTeX
i cm: Computer Modern fonts
i cm-super: CM-Super family of fonts
i dehyph: German hyphenation patterns for traditional orthography
i doublestroke: Typeset mathematical double stroke symbols
i dvipdfmx: An extended version of dvipdfm
i dvipdfmx.win32: win32 files of dvipdfmx
i dvips: A DVI to PostScript driver
i dvips.win32: win32 files of dvips
i dvisvgm: Convert DVI, EPS, and PDF files to Scalable Vector Graphics format (SVG)
i dvisvgm.win32: win32 files of dvisvgm
i ec: Computer modern fonts in T1 and TS1 encodings
i epstopdf-pkg: Call epstopdf "on the fly"
i etex: An extended version of TeX, from the NTS project
i etexcmds: Avoid name clashes with e-TeX commands
i etoolbox: e-TeX tools for LaTeX
i euenc: Unicode font encoding definitions for XeTeX
i fancyvrb: Sophisticated verbatim text
i filehook: Hooks for input files
i firstaid: First aid for external LaTeX files and packages that need updating
i float: Improved interface for floating objects
i fontspec: Advanced font selection in XeLaTeX and LuaLaTeX
i framed: Framed or shaded regions that can break across pages
i fundus-calligra: Support for the calligra font in LaTeX documents
i geometry: Flexible and complete interface to document dimensions
i gettitlestring: Clean up title references
i glyphlist: Adobe Glyph List and TeX extensions
i graphics: The LaTeX standard graphics bundle
i graphics-cfg: Sample configuration files for LaTeX color and graphics
i graphics-def: Colour and graphics option files
i grffile: Extended file name support for graphics (legacy package)
i helvetic: URW "Base 35" font pack for LaTeX
i hycolor: Implements colour for packages hyperref and bookmark
i hyperref: Extensive support for hypertext in LaTeX
i hyph-utf8: Hyphenation patterns expressed in UTF-8
i hyphen-base: core hyphenation support files
i iftex: Am I running under pdfTeX, XeTeX or LuaTeX?
i inconsolata: A monospaced font, with support files for use with TeX
i infwarerr: Complete set of information/warning/error message macros
i intcalc: Expandable arithmetic operations with integers
i jknapltx: Miscellaneous packages by Joerg Knappen
i knuth-lib: A small library of Metafont sources
i kpathsea: Path searching library for TeX-related files
i kpathsea.win32: win32 files of kpathsea
i kvdefinekeys: Define keys for use in the kvsetkeys package
i kvoptions: Key value format for package options
i kvsetkeys: Key value parser with default handler support
i l3backend: LaTeX3 backend drivers
i l3kernel: LaTeX3 programming conventions
i l3packages: High-level LaTeX3 concepts
i latex: A TeX macro package that defines LaTeX
i latex-amsmath-dev: Development pre-release of the LaTeX amsmath bundle
i latex-bin: LaTeX executables and man pages
i latex-bin.win32: win32 files of latex-bin
i latex-fonts: A collection of fonts used in LaTeX distributions
i latex-tools-dev: Development pre-release of the LaTeX tools bundle
i latexconfig: configuration files for LaTeX-related formats
i latexmk: Fully automated LaTeX document generation
i latexmk.win32: win32 files of latexmk
i letltxmacro: Let assignment for LaTeX macros
i lm: Latin modern fonts in outline formats
i lm-math: OpenType maths fonts for Latin Modern
i ltxcmds: Some LaTeX kernel commands for general use
i lua-alt-getopt: Process application arguments the same way as getopt_long
i luahbtex: LuaTeX with HarfBuzz library for glyph shaping
i luahbtex.win32: win32 files of luahbtex
i lualatex-math: Fixes for mathematics-related LuaLaTeX issues
i lualibs: Additional Lua functions for LuaTeX macro programmers
i luaotfload: OpenType 'loader' for Plain TeX and LaTeX
i luaotfload.win32: win32 files of luaotfload
i luatex: The LuaTeX engine
i luatex.win32: win32 files of luatex
i mdwtools: Miscellaneous tools by Mark Wooding
i metafont: A system for specifying fonts
i metafont.win32: win32 files of metafont
i mfware: Supporting tools for use with Metafont
i mfware.win32: win32 files of mfware
i microtype: Subliminal refinements towards typographical perfection
i modes: A collection of Metafont mode_def's
i ms: Various LaTeX packages by Martin Schroder
i natbib: Flexible bibliography support
i pdfescape: Implements pdfTeX's escape features using TeX or e-TeX
i pdftex: A TeX extension for direct creation of PDF
i pdftex.win32: win32 files of pdftex
i pdftexcmds: LuaTeX support for pdfTeX utility functions
i physics: Macros supporting the Mathematics of Physics
i plain: The Plain TeX format
i preview: Extract bits of a LaTeX source for output
i ragged2e: Alternative versions of "ragged"-type commands
i refcount: Counter operations with label references
i relsize: Set the font size relative to the current font size
i rerunfilecheck: Checksum based rerun checks on auxiliary files
i rsfs: Ralph Smith's Formal Script font
i scheme-infraonly: infrastructure-only scheme (no TeX at all)
i setspace: Set space between lines
i standalone: Compile TeX pictures stand-alone or as part of a document
i stringenc: Converting a string between different encodings
i tex: A sophisticated typesetting engine
i tex-ini-files: Model TeX format creation files
i tex.win32: win32 files of tex
i texlive-scripts: TeX Live infrastructure programs
i texlive-scripts.win32: win32 files of texlive-scripts
i texlive.infra: basic TeX Live infrastructure
i texlive.infra.win32: win32 files of texlive.infra
i times: URW "Base 35" font pack for LaTeX
i tipa: Fonts and macros for IPA phonetics characters
i tlgs.win32: Freely available PostScript interpreter
i tlperl.win32: internal Perl for Windows
i tlpsv.win32: cross-platform PostScript and PDF previewer (PS_view, psview, psv)
i tools: The LaTeX standard tools bundle
i unicode-data: Unicode data and loaders for TeX
i unicode-math: Unicode mathematics support for XeTeX and LuaTeX
i uniquecounter: Provides unlimited unique counter
i url: Verbatim with URL-sensitive line breaks
i wasy: The wasy fonts (Waldi symbol fonts)
i wasysym: LaTeX support for the wasy fonts
i xcolor: Driver-independent color extensions for LaTeX and pdfLaTeX
i xetex: An extended variant of TeX for use with Unicode sources
i xetex.win32: win32 files of xetex
i xetexconfig: crop.cfg for XeLaTeX
i xkeyval: Extension of the keyval package
i xunicode: Generate Unicode characters from accented glyphs
i zapfding: URW "Base 35" font pack for LaTeX

FFMPEG
ffmpeg version 4.3.1 Copyright (c) 2000-2020 the FFmpeg developers
built with gcc 10.2.1 (GCC) 20200726
configuration: --enable-gpl --enable-version3 --enable-sdl2 --enable-fontconfig --enable-gnutls --enable-iconv --enable-libass --enable-libdav1d --enable-libbluray --enable-libfreetype --enable-libmp3lame --enable-libopencore-amrnb --enable-libopencore-amrwb --enable-libopenjpeg --enable-libopus --enable-libshine --enable-libsnappy --enable-libsoxr --enable-libsrt --enable-libtheora --enable-libtwolame --enable-libvpx --enable-libwavpack --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxml2 --enable-libzimg --enable-lzma --enable-zlib --enable-gmp --enable-libvidstab --enable-libvmaf --enable-libvorbis --enable-libvo-amrwbenc --enable-libmysofa --enable-libspeex --enable-libxvid --enable-libaom --enable-libgsm --enable-librav1e --disable-w32threads --enable-libmfx --enable-ffnvcodec --enable-cuda-llvm --enable-cuvid --enable-d3d11va --enable-nvenc --enable-nvdec --enable-dxva2 --enable-avisynth --enable-libopenmpt --enable-amf
libavutil      56. 51.100 / 56. 51.100
libavcodec     58. 91.100 / 58. 91.100
libavformat    58. 45.100 / 58. 45.100
libavdevice    58. 10.100 / 58. 10.100
libavfilter     7. 85.100 /  7. 85.100
libswscale      5.  7.100 /  5.  7.100
libswresample   3.  7.100 /  3.  7.100
libpostproc    55.  7.100 / 55.  7.100

Thank you for reporting.

Does this require three Transforms? Does it not happen when there are only two in a row?

I believe this is what ReplacementTransform is for. Can you please try that? (If that works, then it is absolutely our fault for not documenting it, but I'm just trying to see what works and what doesn't for now..)

First of all, I have to state a correction: the problem arises with 3 or more Mobjects with 2 or more Transforms. As I stated the bug the title should be: "Second Transform does not remove second Mobject". Sorry, My bad :)
Anyway, I did a bunch of experiments with the following setup:

4 Mobjects (Tex, Square, Circle, Triangle) to be transformed in sequence,
There are 4 Mobjects, so 3 animations are needed,
Each animation can be either a Transform (indicated by T) or a ReplacementTransform (indicated with R), giving us 8 possibilities.

Here are the resulting animations for each experiment.
TTT

TTR

TRT

TRR

RTT

RTR

RRT

RRR

At this point I have a question: what is the purpose of the Transform animation if it does not replace the initial Mobject?

Thank you for the thorough report. I think the way to go here is to encourage and document the use of ReplacementTransform, but also try to find any bugs in Transform.
I can see some use cases where Transform may be useful, even if it does not replace the mobject.
If you think the use of ReplacementTransform solves your original problem, please close this issue. If you would like to contribute to the project by providing documentation for these animations, that is always welcome and feel free to open a PR for it :)

Yes, the ReplacementTransform do actually solve my problem. Thank you for your time!

I think that the fact that Transform does not actually transform a given object, but rather a copy of it, is weird and not really intuitive.
Maybe this is not a popular opinion, but I think that ReplacementTransform should be Transform, and Transform should be TransformCopy (or whatever it is that it actually does).

Yeah keeping them both but changing the names sounds like the way to go.

Yes, I ended up here facing the same issue by using Transform myself. I think this issue is not popular because most people don't use Transform more than once. It's keeping the copy of the old mobject on the screen from the second transform. Yes, ReplacementTranform is solving the problem.
I'm using the latest version 0.3.0 and still the changes that @behackl proposed to change names are not implemented. I just want to remind you, developers, that you people can consider this change to be implemented in the next version.


CakeContrib.Guidelines should be marked developmentDependency

Since CakeContrib.Guidelines is a utility to help/guide with development it is only needed during build-time of the project it is included in.
Therefore the nuspec should contain <developmentDependency>true</developmentDependency> to  prevent the package from being included as a dependency in other packages.
Also, this will lead to PrivateAssets being set (automatically) in consuming packages using PackageReference.
For consistency with #6 having <PrivateAssets>all</PrivateAssets> on a reference to CakeContrib.Guidelines should also be checked/validated.

@nils-a this is a great suggestion.
Can we also have the same check on Cake.Addin.Analyzer package, which we are also recommending that folks use?

@gep13 @nils-a just FYI, the Cake.Addin.Analyzer will automatically install itself as a private asset (if installed through Visual Studio, or through the .NET Core utility).
Still a :+1: from me though, in case people add it manually.

@AdmiringWorm yes, CakeContrib.Guidelines will now install that way, too :-)


Fix travis 1.0.1 builds

And finally the fixes for 1.0.1 as well (still a work-in-progress).

Aaaaaaaaaaaand 1.0.1 is green too now, please review.

+1

merged, thanks!


WIP - SNMP Get Implementations

Not ready for review, but raising this for visibility of where I am.
Using https://github.com/geoffgarside/ber for encoding/decoding.
Get/GetNext/GetBulk implemented and happy-path tested.
Looking at Walk, next.
Also need to:

decide whether to implement a BulkWalk()
extend unit tests for exceptions
consider options for 'sugar' to facilitate conversion of SNMP variables
lint issues
implement API checks to volidate OIDs on entry
extend trace hooks, as necessary
implement Close

Pull Request Test Coverage Report for Build 99

105 of 125 (84.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.03%) to 94.437%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

v2/snmp/manager.go
93
113
82.3%

Totals

Change from base Build 98:
-1.03%

Covered Lines:
1375

Relevant Lines:
1456

💛  - Coveralls

Closing for the time being.
Will shortly be pushing updates that include changes to address review comments.
After that, still need to extend unit testing, add commentary and a general tidy-up.


rclone iterates through a large bucket to operate on a single object

I have a swift bucket at Hubic with many thousands of objects in it, without any folder hierarchy. Copying a single object from this bucket is quite slow. It looks like rclone first iterates through the entirety of a 'folder' before performing an operation on a single object.

What is your rclone version (eg output from rclone -V)
rclone v1.33
Which OS you are using and how many bits (eg Windows 7, 64 bit)
Debian Linux 64 bit
Which cloud storage system are you using? (eg Google Drive)
Hubic
The command you were trying to run (eg rclone copy /tmp remote:tmp)
rclone copy -v hubic:GA/afile .
A log from the command with the -v flag (eg output from rclone -v copy /tmp remote:tmp)
rclone copy -v hubic:GA/afile .
2016/10/05 16:24:19 rclone: Version "v1.33" starting with parameters ["rclone" "copy" "-v" "hubic:GA/afile" "."]
2016/10/05 16:24:20 Hubic: Got swift credentials (expiry 2016-10-07 00:52:33 +0200 +0200 in 23h28m12.913624349s)
2016/10/05 16:24:21 Local file system at /tmp/asd/9: Modify window is 1ns
2016/10/05 16:24:21 existingfile1: Excluded from sync (and deletion)
2016/10/05 16:24:21 existingfile2: Excluded from sync (and deletion)
2016/10/05 16:24:21 existingfile3: Excluded from sync (and deletion)
... and so-on ...

Deleting a single object is also quite slow:
rclone -v delete hubic:GA/thisisfile
2016/10/05 16:19:21 rclone: Version "v1.33" starting with parameters ["rclone" "-v" "delete" "hubic:GA/thisisfile"]
2016/10/05 16:19:22 Hubic: Got swift credentials (expiry 2016-10-07 00:52:33 +0200 +0200 in 23h33m10.930861748s)
2016/10/05 16:19:22 Hubic Swift container GA: Modify window is 1ns
2016/10/05 16:19:22 Waiting for deletions to finish
2016/10/05 16:20:22
Transferred:      0 Bytes (0 Bytes/s)
Errors:                 0
Checks:                 0
Transferred:            0
Elapsed time:        1m1s
2016/10/05 16:21:22
Transferred:      0 Bytes (0 Bytes/s)
Errors:                 0
Checks:                 0
Transferred:            0
Elapsed time:        2m1s
2016/10/05 16:22:22
Transferred:      0 Bytes (0 Bytes/s)
Errors:                 0
Checks:                 0
Transferred:            0
Elapsed time:        3m1s
2016/10/05 16:23:22
Transferred:      0 Bytes (0 Bytes/s)
Errors:                 0
Checks:                 0
Transferred:            0
Elapsed time:        4m1s
2016/10/05 16:24:22
Transferred:      0 Bytes (0 Bytes/s)
Errors:                 0
Checks:                 0
Transferred:            0
Elapsed time:        5m1s
2016/10/05 16:25:22
Transferred:      0 Bytes (0 Bytes/s)
Errors:                 0
Checks:                 0
Transferred:            0
Elapsed time:        6m1s
2016/10/05 16:26:22
Transferred:      0 Bytes (0 Bytes/s)
Errors:                 0
Checks:                 0
Transferred:            0
Elapsed time:        7m1s
2016/10/05 16:27:14 thisisfile: Deleted
2016/10/05 16:27:14
Transferred:      0 Bytes (0 Bytes/s)
Errors:                 0
Checks:                 1
Transferred:            0
Elapsed time:     7m52.6s
2016/10/05 16:27:14 Go routines at exit 2

I think this is the same issue as #610 which I'm planning to fix for the next release.

Yes it does, sorry for the dupe, I did not notice that issue.

I've fixed this in http://beta.rclone.org/v1.33-55-g98804cb/ (it will take 15 mins to appear).  Have a go and re-open the ticket if there is a problem.
Thanks
Nick


Fabric Loader Problems

I installed it to play the game with Fabric 1.20.1, but I get a log saying
'Replace mod 'Fabric Loader' (fabricloader) 0.15.6 with version 0.15.7 or later'.
However, my Fabric Loader is currently the highest version of 1.20.1
And is not compatible with all the mods that can be done with Fabric at the moment (which are posted by the modders of this mod on Curseforge).
So, I want to run this mode on this 1.20.1 fabric.
(It may seem strange because you are using a translator. I'm sorry. :->)

upgrade to 1.20.2 is can't be to play my world..

The current version of this mod requires Fabric loader version 0.15.7. Please try to play on an instance with Fabric loader version 0.15.7 installed.
However, since there was little need to limit the loader to 0.15.7, the loader version restriction will be loosened in the next version.

Please try version 1.2.2 for fabric.


Razor duplicate registers the IVsTextViewFilter

Repro steps:
1.Open a Blazor project, and go to a .razor file
2. Get a Gray Text Completion, such as from Intellicode or GitHub Copilot completions
3. Press Alt + / to jump into an Inline Refinement
An exception is thrown (gold bar appears only in experimental instance):
An exception is thrown:
    <description>System.ArgumentException: Item has already been added. Key in dictionary: 'Microsoft.VisualStudio.TextManager.Interop.IVsTextViewFilter'  Key being added: 'Microsoft.VisualStudio.TextManager.Interop.IVsTextViewFilter'
   at System.Collections.Hashtable.Insert(Object key, Object nvalue, Boolean add)
   at Microsoft.VisualStudio.Utilities.PropertyCollection.AddProperty(Object key, Object property)
   at Microsoft.VisualStudio.Editor.Implementation.SimpleTextViewWindow.AddCommandFilter(IOleCommandTarget pNewCmdTarg, IOleCommandTarget& ppNextCmdTarg, Boolean projectionAware)
   at Microsoft.VisualStudio.Razor.RazorLSPTextViewConnectionListener.RazorLSPTextViewFilter.CreateAndRegister(IVsTextView textView)
   at Microsoft.VisualStudio.Razor.RazorLSPTextViewConnectionListener.SubjectBuffersConnected(ITextView textView, ConnectionReason reason, IReadOnlyCollection`1 subjectBuffers)
   at Microsoft.VisualStudio.Text.Utilities.GuardedOperations.CallExtensionPoint(Object errorSource, Action call)</description>

Root cause:
RazorLSPTextViewConnectionListener contains code that hooks up the IVsTextViewFilter for command handling, on buffer connection. This code probably works fine in all normal text documents, but it trips up in certain projection scenarios such as diff view.
In Diff view, there can be multiple text views connected to a single buffer, resulting in this code trying and failing to register the command filter multiple times.
In general, ITextViewConnectionListener.SubjectBuffersConnected() should not assume that it's called only one time.

Somehow Microsoft.WebTools.Languages.Html.VS.Commands.Html.HtmlTextViewFilter is added first, then Microsoft.VisualStudio.Razor.RazorLSPTextViewConnectionListener.RazorLSPTextViewFilter

https://devdiv.visualstudio.com/DevDiv/_git/VS-Platform/pullrequest/579299 fixes the exception, but the RazorLSPTextViewFilter is still not used in this case due to the HtmlTextViewFilter already being active.

AFAIK this never had any user facing impact, and with the exception fixed that would seem to be even less impact, so I'm going to close this out. Thanks all!


Can we combine the get-app-token into this as an option

Currently we often have to use this action together with the awesome app-token-action:
      # Setup private actions
      - name: Get Token
        id: token
        uses<EMAIL_ADDRESS>        with:
          app_id: <app_id>
          app_base64_private_key: ${{ secrets.APPLICATION_PRIVATE_KEY_BASE64 }}
          auth_type: 'installation'

      - uses<EMAIL_ADDRESS>        with:
          token: ${{ steps.token.outputs.token }}
          base_dir: .actions
          repos: org/repo1@version, org/repo2@version

Although this works fine, it is a little bit big.. Can we add an option in inner-source-checkout-action to do these two steps in one. Something like:
      - uses<EMAIL_ADDRESS>        with:
          app_id: <app_id>
          app_base64_private_key: ${{ secrets.APPLICATION_PRIVATE_KEY_BASE64 }}
          base_dir: .actions
          repos: org/repo1@version, org/repo2@version

If app_id and app_base64_private_key are given instead of token, it should get the token itself.

This is also done in https://github.com/daspn/private-actions-checkout
Maybe see if we can start using the other action.


Changeset hash is calculated using comments following the command.

Search first

[X] I searched and no similar issues were found

Description
I get the following error:
Caused by: liquibase.exception.ValidationFailedException: Validation Failed:
     1 change sets check sum
          db-changelogs/inventory.sql::1::tim was: 8:d44e2311b2da45086f580f028fb3378f but is now: 8:cc69d03bb1c01a76df70d58f59d30b32

when attempting to write a comment after a statement/before another statement in my changelog SQL.
Steps To Reproduce
Run this changelog:
--changeset tim:1
CREATE TABLE `inventory` (
  `id` int UNSIGNED NOT NULL AUTO_INCREMENT,
  `bytes` BLOB,
  PRIMARY KEY (`id`),
) ENGINE=InnoDB

then update the changelog to be:
--changeset tim:1
CREATE TABLE `inventory` (
  `id` int UNSIGNED NOT NULL AUTO_INCREMENT,
  `bytes` blob,
  PRIMARY KEY (`id`),
) ENGINE=InnoDB

-- Nifty comment explaining this changeset
--changeset tim:2
ALTER TABLE `inventory` MODIFY COLUMN `bytes` MEDIUMBLOB;

Outcome:
liquibase.exception.ValidationFailedException: Validation Failed:
     1 change sets check sum
          db-changelogs/inventory.sql::1::tim was: 8:d44e2311b2da45086f580f028fb3378f but is now: 8:cc69d03bb1c01a76df70d58f59d30b32

Expected/Desired Behavior
I expect comments to not influence the hash of the changeset
Liquibase Version
v4.28.0
Database Vendor & Version
MySQL 8
Liquibase Integration
No response
Liquibase Extensions
No response
OS and/or Infrastructure Type/Provider
Windows, AL2023 Linux both reproduce issue
Additional Context
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR (Thank you!)

@timendez thanks for filing this issue. I see two things going on:

In your example you changed BLOB to blob which will definitely affect the checksum. You can try re-running with those same values.

Even when you re-run, Liquibase does include comments in the checksum calculation, so I believe this is working as expected from the product side.

Sorry, didn't directly copy-paste due to removing sensitive information. I've updated the comment.
Perhaps it would make sense to ignore comments in the checksum since the behavior is the same?

@timendez we're not planning to update the checksum behavior to ignore comments. If more requests come in for that behavior, we will definitely consider it.
In light of that, I'm going to close this issue. If there is something broken or not working as described in our documentation, please reopen.

Ok, thank you!


apply: further reduce cmd clone of cmd observer

What problem does this PR solve?
Issue Number: close #xxx
Problem Summary:
Even after https://github.com/tikv/tikv/pull/10138, TiKV still needs to clone applied cmd 1 time if only resolevd-ts worker is observing and 2 times if the cdc worker is also observing, and the memory problem mentioned at https://github.com/tikv/tikv/pull/10138 is still unsolved.
What is changed and how it works?
What's Changed:
This PR further reduces the cmd cloning by share the cmd with post_apply observer, after this PR, TiKV does not need to clone cmd if only resolevd-ts worker is observing and clone once if the cdc worker is also observing. This PR also reduces the abovementioned memory problem by filter default_cf data when there is only resolevd-ts worker observing the region.
Check List
Tests

Integration test

Release note

No release note

PTAL @JmPotato

/rebuild

/test

/test

/rebuild

/rebuild

/rebuild

/merge


upd default

23  upd:{{[t;x] insert[t;x]}}               // default upd function used for replaying data
\Is this nested function as a default ok?

realupd:{[f;t;x]
228  	// increment the tablecounts
229          tablecounts[t]+::count first x;
230  	// run the supplied function in the error trap
231  	.[f;(t;x);{[t;x;e] errorcounts[t]+::count first x}[t;x]];
232  	}[.replay.upd]
The function needs unnested essentially here or am I misisng something?
[ Note Im confused as this defo used to work fine. ]

Nested function: probably ok. We could probably lose the nesting and just set to insert, I think the fact it's in a function adds in the region of 100-200 ns to each invocation e.g. comparing
q)\ts:1000000 10+20
246 608
q)\ts:1000000 {x+y}[10;20]
371 1120
125 ns per invocation of second version
And to be clear - assuming you are talking about this line:
https://github.com/AquaQAnalytics/TorQ/blob/master/code/processes/tickerlogreplay.q#L22
then the definition of upd will end up being
{[t;x] insert[t;x]}
r.e. second question - not sure.  What's happening?  That bit looks ok to me (in isolation at least)

This is giving me a rank error as it stands. Ill look to see if its soemthing silly Im doing.


Add confirmWechatPayPayment types and tests

Summary & motivation

Adds confirmWechatPayPayment types. confirmWechatPayPayment is currently in beta.
Adds wechat_pay_display_qr_code interface.

Testing & documentation
Wrote unit tests that pass.

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


Left align Radio Card

In the first example for the Checkbox Card and Radio Card in the docs, the text is left aligned because it has a Flex child that sets this.

https://www.radix-ui.com/themes/docs/components/checkbox-cards

https://www.radix-ui.com/themes/docs/components/radio-cards
By default the alignment of contents within a checkbox/radio card is centered.
If the intention is to support left alignment of text in a checkbox/radio card item, could it be given a prop to set this, rather than requiring nesting a Flex within it?
If the intention is to discourage left alignment, could the example at the top of the docs be changed to something centre aligned? 😄

The alignment choice is by design:

The checkbox card is left aligned by default because of the checkbox on the right.
The radio card was made center aligned by default because it's the most vanilla / good-looking option out of the box for simple one-liners.

That Flex example in the docs should probably be moved to the Examples section, and the base example should be simpler.
Using composition to achieve a different layout is encouraged. Not sure how we'd feel about a layout prop there as some common layouts are just too different and can't be covered with props reasonably well compared to composition (namely: a single center-aligned line of text (works out of the box), text + icon horizontally (works out of the box), 2+ lines of text left-aligned (needs composition)).