[GOBBLIN-1348] Uses own logic for generating tags instead of the docker meta action

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-1348

Description

[x] Here are some details about my PR, including screenshots (if applicable):
Removes use of non-verified third party github action to generate tags, and instead generates tags explicitly in a previous job
Runs the action if the github action is modified or if docker files are modified, allowing earlier detection of bugs

Tests

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

Commits

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

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

Codecov Report

Merging #3187 (ac0dbb2) into master (d7fc475) will decrease coverage by 37.09%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #3187       +/-   ##
============================================
- Coverage     46.19%   9.10%   -37.10%
+ Complexity     9752    1723     -8029
============================================
  Files          2014    2014
  Lines         77099   77099
  Branches       8559    8559
============================================
- Hits          35615    7018    -28597
- Misses        38176   69396    +31220
+ Partials       3308     685     -2623

Impacted Files
Coverage Δ
Complexity Δ

...c/main/java/org/apache/gobblin/util/FileUtils.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-4.00%)

...n/java/org/apache/gobblin/fork/CopyableSchema.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-2.00%)

...java/org/apache/gobblin/stream/ControlMessage.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

...va/org/apache/gobblin/dataset/DatasetResolver.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-3.00%)

...va/org/apache/gobblin/converter/EmptyIterable.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

...org/apache/gobblin/ack/BasicAckableForTesting.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-3.00%)

...n/java/org/apache/gobblin/salesforce/SfConfig.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

.../org/apache/gobblin/yarn/HelixMessageSubTypes.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

...va/org/apache/gobblin/cluster/SingleHelixTask.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-5.00%)

.../apache/gobblin/records/ControlMessageHandler.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

... and 1058 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d7fc475...ac0dbb2. Read the comment docs.

Codecov Report

Merging #3187 (ac0dbb2) into master (d7fc475) will decrease coverage by 37.09%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #3187       +/-   ##
============================================
- Coverage     46.19%   9.10%   -37.10%
+ Complexity     9752    1723     -8029
============================================
  Files          2014    2014
  Lines         77099   77099
  Branches       8559    8559
============================================
- Hits          35615    7018    -28597
- Misses        38176   69396    +31220
+ Partials       3308     685     -2623

Impacted Files
Coverage Δ
Complexity Δ

...c/main/java/org/apache/gobblin/util/FileUtils.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-4.00%)

...n/java/org/apache/gobblin/fork/CopyableSchema.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-2.00%)

...java/org/apache/gobblin/stream/ControlMessage.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

...va/org/apache/gobblin/dataset/DatasetResolver.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-3.00%)

...va/org/apache/gobblin/converter/EmptyIterable.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

...org/apache/gobblin/ack/BasicAckableForTesting.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-3.00%)

...n/java/org/apache/gobblin/salesforce/SfConfig.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

.../org/apache/gobblin/yarn/HelixMessageSubTypes.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

...va/org/apache/gobblin/cluster/SingleHelixTask.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-5.00%)

.../apache/gobblin/records/ControlMessageHandler.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

... and 1058 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d7fc475...ac0dbb2. Read the comment docs.


No constant buffer's info in some shader disassemblies

Some shader disassemblies (like in issue #249) don't contain information about CB's values used by Unity (like UnityPerDraw, $Globals and so on) so i can't convert shader disassembly back into hlsl without this. Can you make UtinyRipper adding this information in shaders if it is possible?

There is this information inside ShaderSubProgram class
https://github.com/mafaca/UtinyRipper/blob/d78186190fac0d11dcaa6f696b168dbbb838de28/uTinyRipperCore/Parser/Classes/Shader/ShaderSubProgram.cs#L330-L341

OK, I'll test some other DX11 games, if their shaders will be disassembled with globals and buffers listed (as in inspector when clicking "compile and show code") then I will close the issue, maybe it just was a broken .vrca game file. If not then I will write.

I mean if you need this information you can get it from those fields :)

I will try setting debug breakpoints in code to view fields values when exporting file.

Alright I've managed to build solution and found this info using debugger from those fields. Quite strange and unusual way to get data but it works fine, thank you.


Bug: Transposed arguments in function call

There are two call sites to the get_transfer_fee function where epoch and pre_fee_amount_x are transposed (code snippet quoted below). The function definition specifies the epoch argument second, and the pre_fee_amount_x third.
        TransferFeeInfo {
            mint: token_mint_0,
            owner: mint0_account.owner,
            transfer_fee: get_transfer_fee(&mint0_state, pre_fee_amount_0, epoch),
        },
        TransferFeeInfo {
            mint: token_mint_1,
            owner: mint1_account.owner,
            transfer_fee: get_transfer_fee(&mint1_state, pre_fee_amount_1, epoch),
        },

OK， thanks for your feedback, we have update in commit 4faf9c0595907cbb399a00660d5efb9d7f721504


Add an easy way to mock responses

I am using the ImageAnnotatorClient. There doesn't seem to be an easy way to mock the response. Maybe there is already a way to do this but I haven't found one yet without mocking all the sub classes.

Hi @Livijn
Thanks for filling the issue.
There is a way to mock responses for every API call using the transportOptions.restOptions.handler configuration for REST, but the issue is that every response needs to be de-serialized into a relevant protobuf object.
Consider the batchAnnotateImages method of the ImageAnnotatorGapicClient.
This method returns an instance of the BatchAnnotateImagesResponse class, so it expects the relevant data to be present in the mocked response.
Although, I can't speak for the use of the following snippet in production by any means, but the following did work for me:
$client = new ImageAnnotatorClient();
$requests = [];

// $batchResponse is an instance of the BatchAnnotateImagesResponse class
$batchResponse = $client->batchAnnotateImages($requests, [
    'transportOptions' => [
        'restOptions' => [
            'handler' => function () {
                // I am using the serializer here to easily convert the array to an instance of BatchAnnotateImagesResponse
                $serializer = new Serializer();
                $response = $serializer->decodeMessage(
                    new BatchAnnotateImagesResponse,
                    ['responses' => [
                        ['context' => ['uri' => 'https://google.com']]
                    ]]);

                return new Response(200, ['content-type' => 'application/json'], $obj->serializeToJsonString());
            }
        ]
    ]
]);

foreach($batchResponse->getResponses() as $row) {
    // $row is an instance of the Google\Cloud\Vision\V1\AnnotateImageResponse class
    echo $row->getContext()->getUri();  // https://google.com
}

You can mock an empty array by just using $obj = new BatchAnnotateImagesResponse([]). I don't know if that works for your testing, but this is the information I have.
Closing this issue for now, but please reopen this for more discussion.


vendor libnetwork @1861587

Fixes #14041
With this change if  ip_forward is enabled by docker daemon then the iptables FORWARD policy will be set to DROP.  On a docker upgrade if ip_forward is already enabled this change will not take effect, because in that case the user might have intended to have the default policy ACCEPT. Changing it to DROP will break such setups. Current users of docker can avoid this issue by either

setting the FORWARD policy to DROP manually
reload the host before upgrading to 1.13. If docker daemon enabled ip_forward its not persistent. So after a reload the fix will take effect and the policy will be set to DROP.

Signed-off-by: Santhosh Manohar<EMAIL_ADDRESS>

LGTM

this seems to have broken the following fix #490
whilst you could configure --ip-forward inside the daemon config file,
there is no way to manage this via the command line...

It appears this change of policy broke my use case (PHP debugging with XDebug, which involves opening TCP connections from the container back to the host).
How can I restore the old policy (short of pinning the 1.12.6 version on my development machine)?

Solved. Note that if you accidentally upgraded to 1.13 its not enough to downgrade back to 1.12, the iptables default policy has to be manually reverted.

@1ma what if I remove docker 1.13 and install 1.12, should I fix iptables policy manually?

Yes, that's what I had to do.

Is Ip-forwarding required to run docker. If i disable ipforwarding will it impact any functionality of docker application.

@bharathvishwanathan please don't use old PR's (this pull request was merged three years ago) to start new discussions.
Please keep in mind that the GitHub issue tracker is not intended as a general support forum,
but for reporting bugs and feature requests. For other type of questions, consider using one of;

the Docker community Slack channel
the Docker Support Forums - https://forums.docker.com
StackOverflow


Unable to get the queue value returns from external metrics

Describe the bug
I have followed the example of how to scale using Service Bus Queue as an external metric. But I was unable to get the queue value by running the following command:
kubectl  get --raw "/apis/external.metrics.k8s.io/v1beta1/namespaces/default/queuemessages"
Error from server (BadRequest): metricName is required

The external metric API seems to work well.
kubectl  get --raw "/apis/external.metrics.k8s.io/v1beta1/"
{"kind":"APIResourceList","apiVersion":"v1","groupVersion":"external.metrics.k8s.io/v1beta1","resources":[]}

I appreciate for your kind assistance. Thank you so much!
To Reproduce

Create a AKS cluster through Azure Portal

Kubernetes version: 1.13.7
Enable RBAC: Yes
Network configuration: Basic
Enable container monitoring: Yes

Validate the Metric Server by running kubectl get --raw "/apis/metrics.k8s.io/v1beta1/nodes" command.

Install Helm on AKS

Create a service bus in Azure

Create a namespace
Create a queue
Create an auth rules for queue:

Enable Access to Azure Resources by configuring a service principal

Create a service principal scoped to the resource group the resource you monitoring and assign Monitoring Reader to it:

az ad sp create-for-rbac -n "adapter-sp" --role "Monitoring Reader" --scopes /subscriptions/{SubID}/resourceGroups/{ResourceGroup1}

Deploy the environment variables via secret:

kubectl create secret generic adapter-service-principal -n custom-metrics --from-literal=azure-tenant-id=<tenantid> --from-literal=azure-client-id=<clientid>  --from-literal=azure-client-secret=<secret>

Since I don't have the go language environment, I used the Service Bus Explorer to create messages as producer.

Configure Secret for consumer pod

Deploy Consumer

Deploy the consumer:

kubectl apply -f deploy/consumer-deployment.yaml

Check that the consumer was able to receive messages:

 kubectl logs consumer-547467bf87-bhdnt
 connecting to queue:  externalq
 setting up listener
 received message:  <?xml version="1.0" encoding="utf-8"?>
 <message>Hi mate, how are you?</message>

Deploy the adapter

Create a namespace

kubectl create namespace custom-metrics

Create a Service Principle and Secret:

az ad sp create-for-rbac -n "azure-k8s-metric-adapter-sp" --role "Monitoring Reader" --scopes /subscriptions/{SubID}/resourceGroups/{ResourceGroup1}

Use values from service principle created above to create secret

kubectl create secret generic azure-k8s-metrics-adapter -n custom-metrics --from-literal=azure-tenant-id=<tenantid> --from-literal=azure-client-id=<clientid>  --from-literal=azure-client-secret=<secret>

Deploy the adapter with the service principle:

helm install --name sample-release ../../charts/azure-k8s-metrics-adapter --namespace custom-metrics --set azureAuthentication.method=clientSecret --set azureAuthentication.tenantID=<your tenantid> --set azureAuthentication.clientID=<your clientID> --set azureAuthentication.clientSecret=<your clientSecret> --set azureAuthentication.createSecret=true`

I can hit the external metric endpoint by verifying with the following command

kubectl get --raw "/apis/external.metrics.k8s.io/v1beta1
{
  "kind": "APIResourceList",
  "apiVersion": "v1",
  "groupVersion": "external.metrics.k8s.io/v1beta1",
  "resources": []
}

Configure Metric Adapter with metrics

Replace the resourceGroup and resourceName in externalmetric.yaml
Create ExternalMetric resource with following command

kubectl apply -f deploy/externalmetric.yaml

List external metrics via following command

kubectl get aem
NAME            AGE
queuemessages   40s

Deploy the HPA

Deploy the HPA:

kubectl apply -f deploy/hpa.yaml

Validate that the HPA is configured

kubectl get hpa consumer-scaler
NAME              REFERENCE                    TARGETS        MINPODS   MAXPODS   REPLICAS   AGE
consumer-scaler   Deployment/consumer-scaler   <unknown>/30   1         10        1          3d2h

You can also check the queue value returns manually by running:

kubectl  get --raw "/apis/external.metrics.k8s.io/v1beta1/namespaces/default/queuemessages"
Error from server (BadRequest): metricName is required

Expected behavior
I can get the queue value by running the following command:
kubectl  get --raw "/apis/external.metrics.k8s.io/v1beta1/namespaces/default/queuemessages"

Kubernetes version (kubectl version):

[1.13.7] Running on AKS

Logs (kubectl logs <metric adapter pod id>)
kubectl logs sample-release-azure-k8s-metrics-adapter-69556544c9-c7pt6
E0728 12:53:52.320936       1 reflector.go:205] github.com/Azure/azure-k8s-metrics-adapter/pkg/client/informers/externalversions/factory.go:117: Failed to list *v1alpha1.ExternalMetric: the server could not find the requested resource (get external
metrics.azure.com)
E0728 12:53:52.320936       1 reflector.go:205] github.com/Azure/azure-k8s-metrics-adapter/pkg/client/informers/externalversions/factory.go:117: Failed to list *v1alpha1.CustomMetric: the server could not find the requested resource (get custommetr
ics.azure.com)
E0728 12:53:53.322735       1 reflector.go:205] github.com/Azure/azure-k8s-metrics-adapter/pkg/client/informers/externalversions/factory.go:117: Failed to list *v1alpha1.ExternalMetric: the server could not find the requested resource (get external
metrics.azure.com)
E0728 12:53:53.323570       1 reflector.go:205] github.com/Azure/azure-k8s-metrics-adapter/pkg/client/informers/externalversions/factory.go:117: Failed to list *v1alpha1.CustomMetric: the server could not find the requested resource (get custommetr
ics.azure.com)
E0728 12:53:54.324518       1 reflector.go:205] github.com/Azure/azure-k8s-metrics-adapter/pkg/client/informers/externalversions/factory.go:117: Failed to list *v1alpha1.ExternalMetric: the server could not find the requested resource (get external
metrics.azure.com)
E0728 12:53:54.325311       1 reflector.go:205] github.com/Azure/azure-k8s-metrics-adapter/pkg/client/informers/externalversions/factory.go:117: Failed to list *v1alpha1.CustomMetric: the server could not find the requested resource (get custommetr
ics.azure.com)
E0728 12:53:55.326181       1 reflector.go:205] github.com/Azure/azure-k8s-metrics-adapter/pkg/client/informers/externalversions/factory.go:117: Failed to list *v1alpha1.ExternalMetric: the server could not find the requested resource (get external
metrics.azure.com)
E0728 12:53:55.327096       1 reflector.go:205] github.com/Azure/azure-k8s-metrics-adapter/pkg/client/informers/externalversions/factory.go:117: Failed to list *v1alpha1.CustomMetric: the server could not find the requested resource (get custommetr
ics.azure.com)
E0728 12:53:56.335038       1 reflector.go:205] github.com/Azure/azure-k8s-metrics-adapter/pkg/client/informers/externalversions/factory.go:117: Failed to list *v1alpha1.ExternalMetric: the server could not find the requested resource (get external
metrics.azure.com)
E0728 12:53:56.335947       1 reflector.go:205] github.com/Azure/azure-k8s-metrics-adapter/pkg/client/informers/externalversions/factory.go:117: Failed to list *v1alpha1.CustomMetric: the server could not find the requested resource (get custommetr
ics.azure.com)

Additional context
kubectl describe hpa consumer-scaler
kubectl de
scribe hpa consumer-scaler
Name:                              consumer-scaler
Namespace:                         default
Labels:                            <none>
Annotations:                       kubectl.kubernetes.io/last-applied-configuration:
                                     {"apiVersion":"autoscaling/v2beta1","kind":"HorizontalPodAutoscaler","metadata":{"annotations":{},"name":"consumer-scaler","namespace":"de...
CreationTimestamp:                 Sun, 28 Jul 2019 22:34:45 +0800
Reference:                         Deployment/consumer
Metrics:                           ( current / target )
  "queuemessages" (target value):  <unknown> / 30
Min replicas:                      1
Max replicas:                      10
Deployment pods:                   1 current / 0 desired
Conditions:
  Type           Status  Reason                   Message
  ----           ------  ------                   -------
  AbleToScale    True    SucceededGetScale        the HPA controller was able to get the target's current scale
  ScalingActive  False   FailedGetExternalMetric  the HPA was unable to compute the replica count: unable to get external metric default/queuemessages/nil: unable to fetch metrics from external metrics API: metricName is required
Events:
  Type     Reason                        Age                     From                       Message
  ----     ------                        ----                    ----                       -------
  Warning  FailedComputeMetricsReplicas  6m33s (x12 over 9m19s)  horizontal-pod-autoscaler  failed to get external metric queuemessages: unable to get external metric default/queuemessages/nil: unable to fetch metrics from external metrics API: metricName is required
  Warning  FailedGetExternalMetric       4m16s (x21 over 9m19s)  horizontal-pod-autoscaler  unable to get external metric default/queuemessages/nil: unable to fetch metrics from external metrics API: metricName is required

Based on the logs from the pod of the adapter, it seems like it still queries the ExternalMetric and CustomMetric API with the old version (v1alpha1). I found both of that has already moved to v1alpha2 from v1alpha1 around 5 months ago, but the docker image seems like still leave the old version, but the docker image is still on the v1alpha1 version. Should we need to update the docker image? The last image was created at the end of last year.

Thanks for the detailed report.
You are correct it is the wrong image.
Something went wrong with the release automation.  I will look into it. In the mean time to try it out you can build it manually.

@jsturtevant Got it. Thank you so much!

Hi I am facing the same issue, looking at the logs I can see that it is still using v1alpha1 to query custommetrics.azure.com. Please can you give suggest a workaround for this. I've posted the logs below.
E0828 03:14:37.034658       1 reflector.go:205] github.com/Azure/azure-k8s-metrics-adapter/pkg/client/informers/externalversions/factory.go:117: Failed to list *v1alpha1.CustomMetric: the server could not find the requested resource (get custommetrics.azure.com) E0828 03:14:37.235328       1 reflector.go:205] github.com/Azure/azure-k8s-metrics-adapter/pkg/client/informers/externalversions/factory.go:117: Failed to list *v1alpha1.ExternalMetric: the server could not find the requested resource (get externalmetrics.azure.com) E0828 03:14:38.036096       1 reflector.go:205] github.com/Azure/azure-k8s-metrics-adapter/pkg/client/informers/externalversions/factory.go:117: Failed to list *v1alpha1.CustomMetric: the server could not find the requested resource (get custommetrics.azure.com) E0828 03:14:38.237448       1 reflector.go:205] github.com/Azure/azure-k8s-metrics-adapter/pkg/client/informers/externalversions/factory.go:117: Failed to list *v1alpha1.ExternalMetric: the server could not find the requested resource (get externalmetrics.azure.com) E0828 03:14:39.038207       1 reflector.go:205] github.com/Azure/azure-k8s-metrics-adapter/pkg/client/informers/externalversions/factory.go:117: Failed to list *v1alpha1.CustomMetric: the server could not find the requested resource (get custommetrics.azure.com) E0828 03:14:39.239036       1 reflector.go:205] github.com/Azure/azure-k8s-metrics-adapter/pkg/client/informers/externalversions/factory.go:117: Failed to list *v1alpha1.ExternalMetric: the server could not find the requested resource (get externalmetrics.azure.com) E0828 03:14:40.040313       1 reflector.go:205] github.com/Azure/azure-k8s-metrics-adapter/pkg/client/informers/externalversions/factory.go:117: Failed to list *v1alpha1.CustomMetric: the server could not find the requested resource (get custommetrics.azure.com)

Built the image locally to pick up the latest changes. Working now 👍

Hi @jsturtevant,
Would you help to update the image for this case?

Will anyone from Microsoft looking into this issue? Leaving a latest tag in a deploy is always a cause of problems... There is more than a month this was reported and we are facing the same issue here...

Sorry for the delay.  The latest image with the support for v1alpha2 schemas:
docker pull mcr.microsoft.com/k8s/metrics/adapter:v0.7.3-296
You can find all the images version at https://mcr.microsoft.com/v2/k8s/metrics/adapter/tags/list and latest is tagged with the highest version.


[Bug] Mobile OTP not sent

[REQUIRED] Please fill in the following fields:

Pre-built SDK from the website or open-source from this repo: from Website
Firebase C++ SDK version: 11.3.0
Problematic Firebase Component: Auth
Other Firebase Components in use: Auth
Platform you are using the C++ SDK on: Android
Platform you are targeting: Android, iOS

[REQUIRED] Please describe the issue here:
I am using QT framework along with Firebase SDK version (11.3) to get PhoneNumber Authentication OTP. I am giving correct format of 10 digit Phone number with "+countrycode". I follow the same procedure mentioned in the Firebase documenation for C++.
Steps to reproduce:
When i called the below VerifyPhoneNumber function, the OnCodeSent() function being called and it is confirming that the phone_listener is working fine. However, I am not able to receive the OTP to the mobile Number. I tried with nearly 3 phone numbers and same issue exists. Also, i do not get any error from Firebase as a Notification in the console.
So, i am clueless where it is going wrong. I have already done the project settings in Firebase and enable phoneNumber authentication.
The code is given below
Relevant Code:
// TODO(you): code here to reproduce the problem

localmobile = "+918939608135";
PhoneListener phone_listener;
firebase::auth::PhoneAuthOptions options;
options.timeout_milliseconds = 30;
options.phone_number = localmobile.toUtf8().constData();
qDebug()<< "options.phone_number"<< options.phone_number;
firebase::auth::PhoneAuthProvider& phone_provider = firebase::auth::PhoneAuthProvider::GetInstance(m_auth);
phone_provider.VerifyPhoneNumber(options, &phone_listener);

Hi @vinothkumarc2805
Trying it on my end, I'm not having the problems you are seeing.  One thing does stand out, that the timeout_milliseconds you are using is 30ms, which might be causing a problem?  That should definitely be higher, probably around a minute (60 * 1000).
If that doesn't work, you might need to reach out to Firebase support, https://firebase.google.com/support, so that they can verify that your project on the backend is set up correctly.

Still the Issue exists even after changing to 1 minute  (60 * 1000)
On Wed, Aug 16, 2023 at 6:51 AM a-maurice @.***> wrote:

Hi @vinothkumarc2805 https://github.com/vinothkumarc2805
Trying it on my end, I'm not having the problems you are seeing. One thing
does stand out, that the timeout_milliseconds you are using is 30ms, which
might be causing a problem? That should definitely be higher, probably
around a minute (60 * 1000).
If that doesn't work, you might need to reach out to Firebase support,
https://firebase.google.com/support, so that they can verify that your
project on the backend is set up correctly.
—
Reply to this email directly, view it on GitHub
https://github.com/firebase/firebase-cpp-sdk/issues/1424#issuecomment-1679831297,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQJSVZIQO4KH3SL5YEQUEHDXVQOBJANCNFSM6AAAAAA3PUIBP4
.
You are receiving this because you were mentioned.Message ID:
@.***>


mgmt servicebus, update api-version to 2021-11-01

Description
fix https://github.com/Azure/azure-sdk-for-java/issues/30267#issuecomment-1204727825
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


chore: add "version" file to go dist directory

jsii-release requires a file called "version" so it will know the go-runtime version.

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

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject89A8053A-6Jw05QLuWWwe
Commit ID: a0f61b1cb88551b665170dd1e8be2f24e62c4a5c
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Issue with Windows 11 (Windows 10 is fine)

First of all, a big thanks for this program.
Srapbook Helper runs fine under Windows 10.
Login via SSO (S&F account), as well as via the normal login (character name + password), works.
Manual and automatic attacks also work without any problems.

Unfortunately, I can't get Scrapbook Helper working under Windows 11.
Even with deactivated real-time protection of the Defender and with various compatibility modes, the program would not start.

The window remains empty and shortly afterwards the program hangs.
Update:
I was able to start the program on another Windows 11 PC without any problems.

I've now tried to save the .exe in different locations, which always resulted in the same error.
However, as soon as I copy the .exe to the desktop or create a shortcut there, I can start the program without any problems, regardless of where I have saved the .exe.
Very strange. :D

Hm... very strange indeed. If the location of the executable matters, this sounds more like an anti-virus, or privilege settings issue. I tried to replicate it with the same windows build version, but for me it just always works, regardles of the location, or security settings. Even the typically roblematic Programfiles folder and network drives work flawlessly...
Regardless, I tried to just change a few things, that might have an influence on the issue. If you want, you can try to run these and tell me, what you see:
test.zip
Otherwise I would just add a disclaimer in case anyone runs into the same issue, if you can otherwise just the program fine from the desktop/shortcut.
Oh, and thanks for following up on the issue: 😄

I tested all 4 .exe files from the test.zip on my system.
In my case, the sf-scrapbook-helper-db.exe was the only one that I could start without any problems, regardless of where I saved it.
I am assuming that it is simply due to my user profile, although I reinstalled the entire system about 6 weeks ago.
Even the shortcut on the desktop doesn't start right away from time to time.
On my wife's Windwos 11 system (same build), I can start the program from anywhere without any restrictions.
Only Microsoft Defender is active on both systems. No other third-party programs.
A disclaimer would certainly be an advantage.
No problem. I want to help as much as I can. Besides, I also benefit from the program ;)

If only the "-db" version works, that means it is an issue with the GUI library I used and I can not fully fix it myself. For some reason, hiding the terminal window, that opens at launch, just does not seem to work on your machine.
Guess I am going to add a disclaimer


Reformat files after Black updated to 24.3.0

After merging #584, checks failed and needed to run  pip install "black[jupyter]" and reformat these notebooks.

Yeah, it wasn't there last time. I noticed all single quote became double quote and less indentation.


Create interface to md_tian2 potentials

Here is your new PR @clbox. I will make a couple of modifications before we merge it.

I've added a test now that I was able to run locally but obviously won't run in the CI because the library is missing. Ideally I would like this to work using BinaryBuilder to provide the cross platform library automatically so we could run this on Github. Unfortunately BinaryBuilder has to use the gfortran compiler which doesn't work with md_tian2 because they use a few ifort specific features, specifically the directory keyword for the inquire intrinsic. So we would have to fork the repository and patch anything that doesn't work with gfortran. Ultimately this would be a lot of hassle for minimal gain.
Another option is to set up a new test job that will download and compile md_tian2 before running the tests. This might be easier but again I'm not sure about the availability of ifort in Github actions. If we do want to do either of these probably we should move this code to a separate package as we've done previously with other extension models. This would make it easier to set up alternative testing infrastructure or additional binary dependencies. As it is I don't feel too great about including an interface that cannot be included in the automated tests in the main package.

I can see that you are valiantly attempting to convert the positions to the correct units. Though I believe I did this already using the set_coordinates! function at the bottom of the file. The only thing that needed to be further converted was the unit cell before initalisation.

I can see that you are valiantly attempting to convert the positions to the correct units. Though I believe I did this already using the set_coordinates! function at the bottom of the file. The only thing that needed to be further converted was the unit cell before initalisation.

ah ok thanks, so positions were right but I don't think the cell was converted originally.
Just trying stuff because I ran a benchmark and the enegy was factor ~1.7 out from running md_tian2 without the interface

So now for a test structure:
NQCD + md_tian2:    0.5226285929371041   Ha.
MD_tian2 direct:        0.5226123679033052 Ha.
maybe close enough for now


Idea - allow users to customise their display name via chat

How about allowing users in Twitch chat to customise their channel name in Fauna via a chat command?

The more I think about this I'm not sold on it.  Let's discuss.  Here are a few issues I've been thinking of:

We update the user profile regularly so we'd need to have an additional field we use to hold this so it doesn't get overwritten.
My biggest issue would be confusion. If your Twitch chat handle is foo but you set your name to bar, what do I call that person? Do I say "foo you're right" or "bar you're right"? I think it would be confusing for viewers (especially new viewers) to see a different name on screen than in chat.

Thoughts?

Closing as inactive


Add Building documentation

This fixes or at least gives basic details on :

https://github.com/flood-io/ruby-jmeter/issues/59

There's no need to build and install the gem locally. Just use bundle exec. Please see the updated readme and ping me with any questions ..


LEAR bug: exogeneous used at D-2, should be D-7

In file _lear.py, line 240, function build_and_split_XY():
There is
for past_day in [1,2]:
Should be:
for past_day in [1,7]:

@gmarcjasz What do you think of this?

It should be [1, 7] as Michal suggests - I tested both versions and the difference (depending on the dataset) of MAE and RMSE metrics is between 0 and 1% - but in all cases the corrected version is not worse. The code will be updated to reflect that change.


Fix docker credentials so they use a paid account

We were getting rate limit problems in these two tests in https://github.com/openshift/origin/pull/26989:

test_pull_request_origin_cmd
test_pull_request_origin_extended_clusterup-release-3.11

It turns out the tests were not using the credentials for our paid account; this PR gets those credentials onto the test VM.
I rolled out the changes for this PR for just those two tests and now the tests are  passing:

https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/26989/test_pull_request_origin_cmd/25552
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/26989/test_pull_request_origin_cmd/25553
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/26989/test_pull_request_origin_extended_clusterup-release-3.11/2677
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/26989/test_pull_request_origin_extended_clusterup-release-3.11/2678

I ran the tests twice to show it's not a fluke.

/assign @jupierce
Justin, if this fix seems sane to you, I can roll out the change to the rest of the tests.  No big rush since that PR this fix is helping now has passing tests that didn't require me doing something manually.

This has been rolled out to Jenkins for the rest of the tests.  Just looking for a merge at this point.


Add a link to GOV.UK's Terms and Conditions to the footer

It was advised that we include this link prominently in the (rare)
event of legal action against us for something on the site.

Trello:
https://trello.com/c/cZrYVw6E/258-gov-uk-terms-and-conditions-need-to-appear-in-the-footer

:+1:


fix: fixed ssp assemble for OSCAL 1.0.0 with single dummy component

Signed-off-by: FrankSuits<EMAIL_ADDRESS>Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] All commits are signed-off.

fixed ssp assemble and changed the md output so that it uses true lists with labels wrapped in []

When using:
Profile: NIST_SP-800-53_rev5_MODERATE-baseline_profile.json
Catalog: NIST_SP-800-53_rev5_catalog.json
The following error occurs:
2021-07-01 15:31:29,575 trestle.core.base_model:296 DEBUG: oscal_read content type FileContentType.JSON and alias catalog from /Users/chris/Desktop/trestle_test/catalogs/NIST_SP-800-53_rev5_catalog/catalog.json
Traceback (most recent call last):
  File "/Users/chris/opt/anaconda3/bin/trestle", line 33, in <module>
    sys.exit(load_entry_point('compliance-trestle', 'console_scripts', 'trestle')())
  File "/Users/chris/go/src/github.com/IBM/compliance-trestle/trestle/cli.py", line 65, in run
    exit(Trestle().run())
  File "/Users/chris/opt/anaconda3/lib/python3.8/site-packages/ilcli/command.py", line 147, in run
    return parsed_args.func(parsed_args, extra_args=extra_args)
  File "/Users/chris/opt/anaconda3/lib/python3.8/site-packages/ilcli/command.py", line 174, in _validate_and_run
    return self._run(parsed_args)
  File "/Users/chris/go/src/github.com/IBM/compliance-trestle/trestle/core/commands/author/ssp.py", line 108, in _run
    return ssp_manager.generate_ssp(catalog, profile, markdown_path, sections, yaml_header)
  File "/Users/chris/go/src/github.com/IBM/compliance-trestle/trestle/core/commands/author/ssp.py", line 337, in generate_ssp
    param_list = profile.modify.set_parameters
AttributeError: 'NoneType' object has no attribute 'set_parameters'


fixed vector label inistialise

fixes #1009
there is error in this URL.
http://localhost:5173/?style=67#4.99/-14.11/28.34
the following one is working
http://localhost:5173/?style=70#9.44/-2.0185/30.5457

this PR's approach is not good. I will close it


material-ui-next.com not compatible with Internet Explorer 11

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior
Navigate the Material-UI 1.0 docs using Internet Explorer 11 (useful to check component demos, etc).
Current Behavior
An error is thrown in the console and unable to click on the drawer navigation.

Steps to Reproduce (for bugs)
Go to https://www.material-ui-next.com using Internet Explorer 11 and attempt to click on a link on the drawer.
Context
Check component compatibility with Internet Explorer 11.
Your Environment

Tech
Version

Material-UI
v1.0.0-beta.33

browser
Internet Explorer 11

It should be working. We had had numerous regressions in the past.

Who is using startswith? Could it be a Next.js 5 regression?

I can confirm the behavior. Best of luck with it :)

I can still see the problem

@mnemanja The fix was only just merged. The docs site won't be updated until the next release of Material-UI, otherwise the docs would be out of sync with the release version.

@mbrookes keep up with to good work and thanks for the update! :)

Once a week on average, typically at the weekend, so... soon 😄


commands are not in the same order as per order of selection

That is true, I was also experiencing this issue. I am working on functionality to reorder the commands and also visualise the order in the selection window!

Hi @kzshantonu. In the new release this is fixed. It is now possible to see the order of the commands and also reorder them


Remix doesn't use browser cached versions of route JS/manifest

What version of Remix are you using?
1.7.2
Steps to Reproduce
Production version deployed of any application with correctly configured CDN permanently caching assets.
Chrome version 105.0.5195.125 if relevant.
Expected Behavior
Requests to any assets that have been previously retrieved by the browser should hit the browser cache first.
Actual Behavior
Assets retrieved in the root of the application hit the browser cache first but other routes will always be requested from the CDN.
If you look at the below screenshot you'll see that all the top-level assets are returned from memory/disk cache but the $interactionld-6GDCZIFD.js and manifest-888341CD.js files hit the CDN and get a 304 even though these are available locally (hence the 304).

@waynerobinson What are the other response headers on those two files? (Especially cache-control) I assume this is using the built in remix server? (We have no issues with this when using Express)

Closing since no feedback was provided.


Type issue in atomWithMutation

When using atomWithMutation, type of mutateAsync resolves to any:
live example

I found that in /dist/src/atomWithMutation.d.ts it tries to import that type from the file that doesn't exist:

fixed in #66


Add support similar to srcset to overcome resolution switching problem

I currently have this HTML to cater for retina devices:
<img src="assets/images/continuous-improvement.png" srcset="assets/images/continuous-improvement.png 1x<EMAIL_ADDRESS>2x" alt="" />
Is there any way to achieve the same result with progressively?

Yes, you can use the data-progressive-sm attribute for loading lower resolution/quality image on mobile devices.

@thinker3197 It's not about the viewport width, it's about the device pixel ratio. With
<img src="assets/images/continuous-improvement.png" srcset="assets/images/continuous-improvement.png 1x<EMAIL_ADDRESS>2x" alt="" />
It uses extra high res version of the image at high DPI and normal res at normal DPI.
if (window.devicePixelRatio >= 2) {
// retina display
} else {
// standard display
}

Retina displays can be on large screens so a viewport check is not appropriate.

Agreed, my bad I didn't understood it first. At the moment progressively doesn't support a similar functionality as srcset for higher resolution devices.

No worries. I'll have a go at a feature enhancement PR next week.

Actualmente tengo este HTML para atender dispositivos de retina: <img src="assets/images/continuous-improvement.png" srcset="assets/images/continuous-improvement.png 1x<EMAIL_ADDRESS>2x" alt="" />
¿Hay alguna forma de lograr el mismo resultado progresivamente?


MKL Oversubscription in Cython if Numba njitted function is run first

Reporting a bug

[x] I have tried using the latest released version of Numba (the most recent is
visible in the change log (https://github.com/numba/numba/blob/master/CHANGE_LOG).
[x] I have included a self-contained code sample to reproduce the problem.
i.e. it's possible to run as 'python bug.py'. (files to reproduce)

When running MKL accelerated NumPy functions such as np.dot inside numba njitted's prange block it leads to oversubscription ( #2861 ).
But when using those njitted numba functions together with cython extension functions which also has MKL functions (like np.dot) inside their prange (cython) block it cause them to oversubscribe also.
Usually, by default MKL run single-threaded when running inside OpenMP parallel block (cython's prange) but loading/running numba njitted functions first seems to break that
# numba_mkl.py
@njit(parallel=True)
def numba_mkl(A, B):
    for i in prange(128):
        np.dot(A, B)

# cython_mkl.py
def cython_mkl(A, B):
    cdef Py_ssize_t i
    for i in prange(128, nogil=True):
        with gil:
            np.dot(A, B)

Running numba first
A = np.random.random((10**3, 10**3))
B = np.random.random((10**3, 10**3))

numba_mkl(A, B)                  # ------> 32 threads
cython_mkl(A, B)                 # ------> 36 threads

Running cython first
cython_mkl(A,B)                  # ------> 8 threads
numba_mkl(A, B)                 # ------> 32 threads

Happens in both workqueue and tbb except omp  threading layer.

Thanks for the report. How many cores does your machine have? 8?

Thanks for the report. How many cores does your machine have? 8?

I am using my laptop with Intel i5-9300H, 4 cores ( 8 with hyper threading )

I am using my laptop with Intel i5-9300H, 4 cores ( 8 with hyper threading )

I am using my laptop with Intel i5-9300H, 4 cores ( 8 with hyper threading )

Thanks! Helps to make sense of the above.
I can replicate the over-subscription situation in the Numba code, I suspect the threadpools simply carry over into the cython call.
Solely for the Numba issue... I note that setting the environment variable MKL_NUM_THREADS=1 seems to have the desired effect of keeping the thread count to what might be expected. It's also possible to get MKL to cooperate with Numba's tbb backend by setting  MKL_THREADING_LAYER=tbb to achieve a reasonable subscription docs on all this. I think those two environment variables should be sufficient as workarounds for the problems arising in the workqueue and tbb cases.
Hope this helps?

I am using my laptop with Intel i5-9300H, 4 cores ( 8 with hyper threading )

Thanks! Helps to make sense of the above.
I can replicate the over-subscription situation in the Numba code, I suspect the threadpools simply carry over into the cython call.
Solely for the Numba issue... I note that setting the environment variable MKL_NUM_THREADS=1 seems to have the desired effect of keeping the thread count to what might be expected. It's also possible to get MKL to cooperate with Numba's tbb backend by setting  MKL_THREADING_LAYER=tbb to achieve a reasonable subscription docs on all this. I think those two environment variables should be sufficient as workarounds for the problems arising in the workqueue and tbb cases.
Hope this helps?

You are right threadpools are indeed carrying over into cython call.
I remember trying both MKL_NUM_THREADS=1  and MKL_THREADING_LAYER=tbb before also, but they were only able to prevent oversubscription in the first function call only.

MKL_NUM_THREADS=1

numba_mkl(A, B)                  # ------> 8 threads
cython_mkl(A, B)                 # ------> 16 threads

cython_mkl(A, B)                 # ------> 8 threads
numba_mkl(A, B)                  # ------> 16 threads

Same results with MKL_THREADING_LAYER=tbb, this reaffirms that threadpool is carrying over.
I will be sticking with omp threading layer, having common threading runtime between cython and numba seems to fix this.
Thnaks for you help!

You are right threadpools are indeed carrying over into cython call.
I remember trying both MKL_NUM_THREADS=1  and MKL_THREADING_LAYER=tbb before also, but they were only able to prevent oversubscription in the first function call only.

MKL_NUM_THREADS=1

numba_mkl(A, B)                  # ------> 8 threads
cython_mkl(A, B)                 # ------> 16 threads

cython_mkl(A, B)                 # ------> 8 threads
numba_mkl(A, B)                  # ------> 16 threads

Same results with MKL_THREADING_LAYER=tbb, this reaffirms that threadpool is carrying over.
I will be sticking with omp threading layer, having common threading runtime between cython and numba seems to fix this.
Thnaks for you help!

No problem. Are the threads actually doing work or are they just in existence? I wonder if it's a situation where threads from the user defined code are just delegating to a new thread in MKL, in which case, there's a bit of effort for threads handing off work and waiting but no over-subscription of cores.

No problem. Are the threads actually doing work or are they just in existence? I wonder if it's a situation where threads from the user defined code are just delegating to a new thread in MKL, in which case, there's a bit of effort for threads handing off work and waiting but no over-subscription of cores.

After checking again using htop and restricting the numba and cython threads to 2, it's clear that threads are only in existence and not doing any work.
OMP_NUM_THREADS=2 NUMBA_NUM_THREADS=2 MKL_NUM_THREADS=1 ./check_thread.sh ipython test.py

(Thread count in the photo is actually 4, the extra one is spawned by ipython for saving command history)
There is no substantial regression in performance also.
❯ ipython test.py cython
only cython_mkl
cython : 2.3212240000025486

❯ ipython test.py cython-first
cython then numba
cython : 2.3246966999940923
numba : 2.7127484000011464

❯ ipython test.py
numba then cython
numba : 2.7117838000049233
cython : 2.4403693000000203

Looks like I was unnecessarily worrying over thread count  :relieved:

After checking again using htop and restricting the numba and cython threads to 2, it's clear that threads are only in existence and not doing any work.
OMP_NUM_THREADS=2 NUMBA_NUM_THREADS=2 MKL_NUM_THREADS=1 ./check_thread.sh ipython test.py

(Thread count in the photo is actually 4, the extra one is spawned by ipython for saving command history)
There is no substantial regression in performance also.
❯ ipython test.py cython
only cython_mkl
cython : 2.3212240000025486

❯ ipython test.py cython-first
cython then numba
cython : 2.3246966999940923
numba : 2.7127484000011464

❯ ipython test.py
numba then cython
numba : 2.7117838000049233
cython : 2.4403693000000203

Looks like I was unnecessarily worrying over thread count  :relieved:

@ashutoshvarma thanks for checking and reporting back. I think that it is not uncommon for thread pools to keep threads alive but give them no work as there's no benefit at a particular time/it's not the requested threading behaviour so they do nothing (particularly TBB). Looks like this is resolved so am going to close this ticket. Please do reopen if there are any further issues, or feel free to open another ticket if a different problem is observed. Thanks again!

@ashutoshvarma thanks for checking and reporting back. I think that it is not uncommon for thread pools to keep threads alive but give them no work as there's no benefit at a particular time/it's not the requested threading behaviour so they do nothing (particularly TBB). Looks like this is resolved so am going to close this ticket. Please do reopen if there are any further issues, or feel free to open another ticket if a different problem is observed. Thanks again!

No problem !
Thanks for your help

No problem !
Thanks for your help


SSH config hostnames should be accepted as "address"

For example, if you run vagrant boxes, you can do:
$ vagrant ssh-config core-02
Host core-02
  HostName <IP_ADDRESS>
  User core
  Port 2200
  UserKnownHostsFile /dev/null
  StrictHostKeyChecking no
  PasswordAuthentication no
  IdentityFile /Users/kimmo/Projects/go/src/github.com/k0sproject/k0sctl/.vagrant/machines/core-02/virtualbox/private_key
  IdentitiesOnly yes
  LogLevel FATAL

If you put this into your ~/.ssh/config, it would be nice if you could do something like this in rig-using apps:
hosts:
  - ssh:
      address: core-02

As an alternative, something like this could be nice too:
hosts:
  - ssh:
      config: |
        Host core-02
        HostName <IP_ADDRESS>
        User core
        Port 2200

Solved in #145


Fix analytics_spec.rb test to pass when run from M1

On M1 running from a rosetta terminal - the code puts a ", Rosetta" in the analytics string.
Can repro by running from a rosetta emulated terminal (eg alacritty). This fixes it!

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

Thanks for the PR!
I think a better way to handle this is to just control for whether or not we're running under Rosetta and test all cases. That way, we're still testing the output to Utils::Analytics::os_arch_prefix_ci and not just testing the modifications that we made to the output. Does that make sense?
I made some changes to reflect what I think is best and pushed them as a second commit to this PR. Do they fix your problem? If so, I think something along those lines (even if it's not exactly my changes) is the way we want to go.
If you disagree with the changes, feel free to revert and we can discuss.

Yep this looks good to me. Pushed another commit to fix the test in CI (CI includes a ", CI" at the end of the analytics string). That's why the old test used to use "include" instead of "eq" - but this seems even better.

yep looks good - works on my machine.
Appreciated!

Awesome, thanks so much for your first contribution to Homebrew, @nipunn1313!


HIVE-27569: Backport of HIVE-22405 to branch-3

What changes were proposed in this pull request?
Backport of HIVE-22405: Add ColumnVector support for ProlepticCalendar to branch-3
Jira link - https://issues.apache.org/jira/browse/HIVE-27569

@vihangk1 @sankarh Can you please review and merge this?


[JS Game] : Sea Horse Game

Describe -
Hello @TusharKesarwani @MohitGupta121 ,
I'm a one of my best game which I build in 15 days a SeaHorse game which we can play and it's design for the best user experience.
Tech Stack :
HTML, CSS, JS
Soon raise a PR assign me this please.

@Killer2-O But you're not a part of GSSoC 23 and I did this earlier by youtube course and I have canceled the all the PR which I have made so whats the issue now.
@gurjeetsinghvirdee you can report this guy.
He is targeting me and just stalking me with a new ID now.
Previously his ID was Killer2O and now he came up with Killer2-O
#help


newestOnTop global option seems to  be ignored when adding a Toast

It seems that newestOnTop is being ignored with the latest updates for angular 2.2.0.  Just wanted to let you know so you can look into it.  I wish I had the time to investigate further but I am slammed right now.  I have tried your demo (https://scttcper.github.io/toastr-ng2/) and what I am seeing in my app is the same as the demo.
Thank you,
Robert

I am using toastr-ng v.2.0.1

BTW, thanks for all your work on this.  This is a great component and service.

Hey thanks!
I believe its fixed in 2.0.2

Sure thing.  Thank you Scott.  I will give it a whirl.
Also, not sure if this is an issue, so I will ask and if so I can open another bug:
When extended timeout is 0 and the progress bar is enabled, if there is a bunch of auto dismiss toasts and you quickly run the mouse across them, some toasts can just disappear instead of what I think should happen, which is just to stop the auto dismiss (stick around).
Let me know if it is working as designed or I need to create a new bug.
Thank you,
Robert

Okay.  As far as I can tell it is fixed with 2.0.2.  Thanks a bunch.

I will create a new bug for the other possible issue and close this one since it is resolved.  If the other thing I am seeing is working as designed or not then we can deal with it on the other issue. Thanks Again.


Add option to reset DPS counter after you leave combat

Describe the bug
DPS Counter doesn't reset after a monster dies (even with auto-pause enabled) and keeps ticking down even when you are not in combat with anything.
Also, although not a bug, there should be a way to reset the counter without restarting Runelite, there currently isn't any way to do this (that i know of).
To Reproduce
Steps to reproduce the behavior:

Enable DPS Counter
DPS Counter>Settings>Auto Pause (can be on or off, doesn't seem to do anything)
Attack and kill a monster
Wait a while
Attack another monster, the DPS meter will have gone down in the time where you weren't attacking anything, even with auto-pause enabled.
???
Profit!

Expected behavior
After you kill something the DPS meter should pause, not keep tracking.
Environment (please complete the following information):

OS: Windows 10
RuneLite version: <IP_ADDRESS>

In future, we do have a template for feature requests that should be used for stuff like this.

I submitted it as a bug, thought it was an intended feature.

Also, although not a bug, there should be a way to reset the counter without restarting Runelite, there currently isn't any way to do this (that i know of).

You should be able to shift right-click the overlay and select the Reset option to accomplish this. You can also toggle the plugin off then back on to reset it.

@TheStonedTurtle Nice, thanks. both ways seem to work now. I did try the disable/enable plugin earlier (before i wrote the bug report) and it didn't reset that time, it worked now though. I did restart Runelite between then and now.

You don't need to restart RuneLite just go into the plugin tab and turn damage counter off and back on again. The overlay will disappear up until you attack a new target.


Use variable names for arguments of attr

I would like to do something like this:
{{ form.name|attr:"placeholder:"some_variable }}
So that value of some_variable appears as placeholder of name field.
This obviously throws a Template Syntax Error .
Is there any other way to do this ?

I am experiencing the same issue.

You can use {% render_field %} as an alternative.
{% render_field form.name placeholder=some_variable %}

@razh Thanks


Why are Azure entries in to sub categories?

https://docs.microsoft.com/en-us/power-bi/connect-data/desktop-data-sources
.
I'll prefix this by saying the only thing I know about Azure is that it is a cloud Database.
Azure has a category, with 16 entries.
.
Then the next category, "Online Services Data Sources" lists 2 more "Azure" entries
74.	Azure DevOps (Boards only)
75.	Azure DevOps Server (Boards only)
.
I'm curious why they are not part of the Azure category

Hi @ronssssss, thanks for your question. Forwarding it along to the doc owner.

Thanks for your question, we've passed it on to the engineering team.


qutebrowser: 1.3.1 -> 1.3.2

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/qutebrowser/versions.
These checks were done:

built on NixOS
/nix/store/d5f7w3hcgxzhk1sgk1gjnl36nrq30wlm-qutebrowser-1.3.2/bin/qutebrowser passed the binary check.
/nix/store/d5f7w3hcgxzhk1sgk1gjnl36nrq30wlm-qutebrowser-1.3.2/bin/..qutebrowser-wrapped-wrapped passed the binary check.
/nix/store/d5f7w3hcgxzhk1sgk1gjnl36nrq30wlm-qutebrowser-1.3.2/bin/.qutebrowser-wrapped passed the binary check.
3 of 3 passed binary check by having a zero exit code.
0 of 3 passed binary check by having the new version present in output.
found 1.3.2 with grep in /nix/store/d5f7w3hcgxzhk1sgk1gjnl36nrq30wlm-qutebrowser-1.3.2
directory tree listing: https://gist.github.com/86db26ab52e4c4aaabb2949ceba69142
du listing: https://gist.github.com/47c80976cbfff66061ccbffa47d02669

meta.description for qutebrowser is: '"Keyboard-focused browser with a minimal GUI"'.
cc @jagajaga for testing.
Experimental: this build is cached with Cachix. To use the cache follow these experimental instructions:
One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout 6ab9687000d63035459dde56f5abdff658a68b13 && nix-shell --pure -I nixpkgs=. -p qutebrowser

Success on x86_64-linux (full log)
Attempted: qutebrowser
Partial log (click to expand)
/nix/store/d5f7w3hcgxzhk1sgk1gjnl36nrq30wlm-qutebrowser-1.3.2/share/qutebrowser/userscripts/cast: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/qckzjk3406va7h6s40cy9s75z2w715rq-bash-4.4-p19/bin/bash"
/nix/store/d5f7w3hcgxzhk1sgk1gjnl36nrq30wlm-qutebrowser-1.3.2/share/qutebrowser/userscripts/readability: interpreter directive changed from "/usr/bin/env python" to "/nix/store/rsa1rsd6c6szy0dmi61599b33aqiy269-python3-3.6.5/bin/python"
/nix/store/d5f7w3hcgxzhk1sgk1gjnl36nrq30wlm-qutebrowser-1.3.2/share/qutebrowser/scripts/dictcli.py: interpreter directive changed from "/usr/bin/env python3" to "/nix/store/rsa1rsd6c6szy0dmi61599b33aqiy269-python3-3.6.5/bin/python3"
/nix/store/d5f7w3hcgxzhk1sgk1gjnl36nrq30wlm-qutebrowser-1.3.2/share/qutebrowser/scripts/keytester.py: interpreter directive changed from "/usr/bin/env python3" to "/nix/store/rsa1rsd6c6szy0dmi61599b33aqiy269-python3-3.6.5/bin/python3"
/nix/store/d5f7w3hcgxzhk1sgk1gjnl36nrq30wlm-qutebrowser-1.3.2/share/qutebrowser/scripts/open_url_in_instance.sh: interpreter directive changed from "/bin/bash" to "/nix/store/qckzjk3406va7h6s40cy9s75z2w715rq-bash-4.4-p19/bin/bash"
/nix/store/d5f7w3hcgxzhk1sgk1gjnl36nrq30wlm-qutebrowser-1.3.2/share/qutebrowser/scripts/importer.py: interpreter directive changed from "/usr/bin/env python3" to "/nix/store/rsa1rsd6c6szy0dmi61599b33aqiy269-python3-3.6.5/bin/python3"
checking for references to /build in /nix/store/d5f7w3hcgxzhk1sgk1gjnl36nrq30wlm-qutebrowser-1.3.2...
wrapping `/nix/store/d5f7w3hcgxzhk1sgk1gjnl36nrq30wlm-qutebrowser-1.3.2/bin/.qutebrowser-wrapped'...
postPatchMkspecs
/nix/store/d5f7w3hcgxzhk1sgk1gjnl36nrq30wlm-qutebrowser-1.3.2

Success on x86_64-darwin (full log)
Attempted: qutebrowser
Partial log (click to expand)
/nix/store/r6y48nsvn85y1dwkic95jl7nkzgkvg2v-qutebrowser-1.3.2/share/qutebrowser/userscripts/format_json: interpreter directive changed from "/bin/sh" to "/nix/store/9gghkkqhn6v7dvpf2319qvrxkkw03vwr-bash-4.4-p19/bin/sh"
/nix/store/r6y48nsvn85y1dwkic95jl7nkzgkvg2v-qutebrowser-1.3.2/share/qutebrowser/userscripts/qute-pass: interpreter directive changed from "/usr/bin/env python3" to "/nix/store/l1xvmpqqb3mchsjnjki18siij7yw2w2s-python3-3.6.5/bin/python3"
/nix/store/r6y48nsvn85y1dwkic95jl7nkzgkvg2v-qutebrowser-1.3.2/share/qutebrowser/userscripts/qute-keepass: interpreter directive changed from "/usr/bin/env python3" to "/nix/store/l1xvmpqqb3mchsjnjki18siij7yw2w2s-python3-3.6.5/bin/python3"
/nix/store/r6y48nsvn85y1dwkic95jl7nkzgkvg2v-qutebrowser-1.3.2/share/qutebrowser/userscripts/password_fill: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/9gghkkqhn6v7dvpf2319qvrxkkw03vwr-bash-4.4-p19/bin/bash"
/nix/store/r6y48nsvn85y1dwkic95jl7nkzgkvg2v-qutebrowser-1.3.2/share/qutebrowser/userscripts/taskadd: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/9gghkkqhn6v7dvpf2319qvrxkkw03vwr-bash-4.4-p19/bin/bash"
/nix/store/r6y48nsvn85y1dwkic95jl7nkzgkvg2v-qutebrowser-1.3.2/share/qutebrowser/userscripts/readability: interpreter directive changed from "/usr/bin/env python" to "/nix/store/l1xvmpqqb3mchsjnjki18siij7yw2w2s-python3-3.6.5/bin/python"
/nix/store/r6y48nsvn85y1dwkic95jl7nkzgkvg2v-qutebrowser-1.3.2/share/qutebrowser/userscripts/qutedmenu: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/9gghkkqhn6v7dvpf2319qvrxkkw03vwr-bash-4.4-p19/bin/bash"
wrapping `/nix/store/r6y48nsvn85y1dwkic95jl7nkzgkvg2v-qutebrowser-1.3.2/bin/.qutebrowser-wrapped'...
postPatchMkspecs
/nix/store/r6y48nsvn85y1dwkic95jl7nkzgkvg2v-qutebrowser-1.3.2

Failure on aarch64-linux (full log)
Attempted: qutebrowser
Partial log (click to expand)
[4875/17529] CC obj/third_party/libvpx/bundled_libvpx/alloccommon.o
[4876/17529] CC obj/third_party/libvpx/bundled_libvpx/swapyv12buffer.o
[4877/17529] CXX obj/third_party/icu/bundled_icuuc/uinit.o
[4878/17529] CXX obj/third_party/icu/bundled_icuuc/stringtriebuilder.o
[4879/17529] CC obj/third_party/libvpx/bundled_libvpx/reconinter.o
[4880/17529] CC obj/third_party/libvpx/bundled_libvpx/loopfiltersimplehorizontaledge_neon.o
building of '/nix/store/lssdkibmydaff326bfilmdkgzmhbdz7k-qtwebengine-5.10.1.drv' timed out after 3600 seconds
cannot build derivation '/nix/store/cdkfra7xdjkzw6gnvhyj8jbsxsv7m2ib-python3.6-PyQt-5.10.1.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/6b9hqh8vqwfyznp8z0dxbclwak2hx56i-qutebrowser-1.3.2.drv': 1 dependencies couldn't be built
error: build of '/nix/store/6b9hqh8vqwfyznp8z0dxbclwak2hx56i-qutebrowser-1.3.2.drv' failed


spring for horizontal ticker (no acceleration)

Hi, I am trying to find the correct spring values for a horizontal ticker.. The behavior I want is:

No acceleration or deceleration, just constant smooth movement (I think damping: 0 is helping but, it still seems slow at the beginning and then speeds up)

No reverse / loop back. I am animating marginLeft from '2000px' to '-2000px' to get the scrolling from right to left but I don't want it to then loop back left to right.

Is this behavior possible with react-motion / spring?

This is currently impossible with react-motion to do any other animation axcept spring.

I see.. in that case, I might be able to hack my way past the reverse-direction part at the halfway mark.. I wonder if there is a way to remove the animation / deceleration part and have just smooth motion?

Just don't use react-motion, use https://github.com/tweenjs/tween.js =)

And please contribute somehow to make this a reality:
https://github.com/tweenjs/tween.js/issues/329

There was an animation I was trying to do and realized a spring wouldn't do it easily...  might have been a simple ease-in curved animation. A spring is a cool thing, but it would be great to have extensible animation modules for this tool if it is maintained long-term.


Rename master to aggregator

What this PR does / why we need it:
Slightly more intuitive and useful name. Kept the old name
as an alias which we can decide to remove later if we want.
Did not update a few of the config values that are named with
Master in order to avoid potentially breaking changes.
Also took this as an opportunity to slightly tweak the logic
of how the master command is run (utilizing RunE to return errors
via the cobra command flow). This is how we'd like to see the
commands structured in the future since it gets away from a global
value being used.
Note, this will be backward compatible but if someone with the
newer Sonobuoy CLI tries to target an older Sonobuoy image, then
we will fail since we provide the newer command. This is not
terribly blocking since the user just has to use an older version
of the Sonobuoy CLI.
Which issue(s) this PR fixes
Fixes #238
Special notes for your reviewer:
The although this is backward compatible (Older clients can call this image) we actually run into a problem where this version couldn't call older versions. To avoid that we'd have to add logic about the if version < x then use a different command. I'd really prefer not to do that. Maybe we just phase this in and in this release just do dual support but default to the master command and then next release we can switch over? That way we don't have such a sharp, and short, break in compatibility?
Release note:
Sonobuoy `master` command renamed to `aggregator`.

Codecov Report

Merging #754 into master will decrease coverage by 0.03%.
The diff coverage is 36%.

@@            Coverage Diff             @@
##           master     #754      +/-   ##
==========================================
- Coverage   42.59%   42.55%   -0.04%
==========================================
  Files          70       70
  Lines        4029     4028       -1
==========================================
- Hits         1716     1714       -2
  Misses       2208     2208
- Partials      105      106       +1

Impacted Files
Coverage Δ

pkg/config/config.go
58.62% <ø> (ø)
:arrow_up:

pkg/plugin/driver/daemonset/daemonset.go
48.73% <ø> (ø)
:arrow_up:

pkg/worker/request.go
53.33% <0%> (ø)
:arrow_up:

pkg/client/retrieve.go
0% <0%> (ø)
:arrow_up:

cmd/sonobuoy/app/worker.go
10% <100%> (ø)
:arrow_up:

cmd/sonobuoy/app/root.go
93.1% <100%> (ø)
:arrow_up:

cmd/sonobuoy/app/master.go
44.44% <43.75%> (-5.56%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4901e55...ad4545e. Read the comment docs.

@zubron but what did you think about this section of the special notes:

although this is backward compatible (Older clients can call this image) we actually run into a problem where this version couldn't call older versions. To avoid that we'd have to add logic about the if version < x then use a different command. I'd really prefer not to do that. Maybe we just phase this in and in this release just do dual support but default to the master command and then next release we can switch over? That way we don't have such a sharp, and short, break in compatibility?

@johnSchnake I forgot to come back to that comment. I'm getting a bit mixed up in the flow here, sorry. In what situation would we have a newer client calling older versions? The newer client would generate a new config (which would include the newer image) but I guess it could use a config from a previous version which could reference an old image?

Will avoid changing things like top-level script names/etc which could affect backward compatibility. Will update comments/variable names etc to be consistent with our current, more descriptive naming.


Don't Merge! PR real time test for webhook case

Docs Build status updates of commit b8a12d4:
:clock10: Pending: waiting for processors (19 builds ahead of you)
:warning: Docs Build is busy, currently there are 19 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit b8a12d4:
:clock10: Full build: the average full build time is 7 min(s) 45 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit b8a12d4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/testing.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


should it retry for 404 responses?

By default when retry option is enabled (it is by default) react-query will retry to fetch whenever it gets an failed response from server, even when the status code is 404, which it's a status code normally used when no data is found for the requested resource.
is this a normally behavior?

Yes it is. If you wish, you can use the retry option with a function to determine whether the error should result in a retry:
https://github.com/tannerlinsley/react-query#retries


vm and template CDROM info is not imported

Description
When you import a vcenter template or a wild vm with a cdrom drive, this information is not imported into opennebula.
To Reproduce
Import a vcenter template having a cdrom drive or create a vm with a cdrom drive into vcenter, after that import it into opennebula.
"onetemplate show <template_id>" and "onevm show <vm_id>" will not show info about the existing cdrom drive.
Expected behavior
"onetemplate show <template_id>" and "onevm show <vm_id>" should show information about the cdrom drives.
Details

Affected Component: Sunstone, Scheduler
Hypervisor: ESXi
Version: All versions

Additional context
Add any other context about the problem here.

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

This is already fixed in master and one-5.10.
We are closing this issue until any one report any problem about it.


Remove deprecated from extension package

Kind of change

[ ] Bug fix
[ ] New non-breaking feature
[x] New breaking feature
[ ] Documentation update
[ ] Build improvement

Description
What is the goal of this pull request?
Remove deprecated methods from extension package.
How does it achieve that?
Simply removing the marked code.
Are there any alternative ways to implement this?
no
Are there any implications of this pull request? Anything a user must know?
Users will need to update their code if they use some of the removed methods.
The PR includes a migration guide.
Issue
If this PR fixes an open issue, please add a line of the form:
Fixes #1203
Release notes
Please add a corresponding entry to the file CHANGELOG.adoc

Thanks!


Is there any way to EASILY limit the number of polygons per object?

Is there any way to EASILY limit the number of polygons per object?
For example the snake has over 5mil verts.
Is there a script parameter/etc which will limit the vert count?  I assume not?

If you set target_face_size.global_multiplier to 4 instead of 2 in base.gin it will signal to the system that faces should be an avg. of 4px across instead of 2px, and should in theory reduce the polycount to somewhere 50% or 25% depending on the asset.
In practice, not all assets will respond perfectly to this multiplier at the moment - we are still working on standardizing the API and optimizing everything.
Snakes, unfortunately, are a bit of a special case - the scales are implemented by realizing a large number of instanced individual scales, and the poly-count of each scale nor the overall number of scales can really be adjusted without ruining the effect. So if you wanted a truly lower poly snake, you'd likely need turn off this instanced scales strategy altogether (by modifying the code), or just generate the snake then decimate to your desired resolution. I hope to make the API for making changes like this (disabling sub-features, etc) more standardized and convenient in the future also.


能否添加支持连续帧标注？

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Thanks for your feedback. It will be considered.


docs: XMTP Kit update

What changed? Why?
Updated the introduction to XMTP Kit with the metadata needed for it to work

Also make sure to use yarn format at the root of the repo.

Great work @fabriguespe, feel free to keep polishing any docs in OnchainKit.


use release cpu model got error:unknown Torch class <torch.CudaTensor>

my runtime env: torch7, lua5.2
release model script is:
th tools/release_model.lua -force -model   model_epoch14_8.51.t7  -gpuid 1
when I run translate script:
th translate.lua  -model  model_epoch14_8.51_release.t7 -src s.txt   -output result.txt
I got error informations:
[02/06/17 16:38:18 INFO] Loading 'model_epoch14_8.51_release.t7'...
work/openNMT/torch/install/share/lua/5.2/torch/File.lua:343: unknown Torch class <torch.CudaTensor>
stack traceback:
        [C]: in function 'error'
        .../work/openNMT/torch/install/share/lua/5.2/torch/File.lua:343: in function 'readObject'
        .../work/openNMT/torch/install/share/lua/5.2/torch/File.lua:369: in function 'readObject'
        .../work/openNMT/torch/install/share/lua/5.2/torch/File.lua:369: in function 'readObject'
        .../work/openNMT/torch/install/share/lua/5.2/torch/File.lua:369: in function 'readObject'
        .../work/openNMT/torch/install/share/lua/5.2/torch/File.lua:369: in function 'readObject'
        .../work/openNMT/torch/install/share/lua/5.2/torch/File.lua:409: in function 'load'
        ./onmt/translate/Translator.lua:35: in function '__init'
        .../work/openNMT/torch/install/share/lua/5.2/torch/init.lua:91: in function 'new'
        translate.lua:78: in function 'main'
        translate.lua:197: in main chunk
        [C]: in function 'dofile'
        ...nNMT/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:150: in main chunk
        [C]: in ?

use gpu mode is OK.
please help!

What training options did you use?

my training script as below:
th train.lua -gpuid 1 -layers 2  -max_batch_size 64 -rnn_size 500 -optim adam -learning_rate 0.0002  -end_epoch 13  -save_every 3000 -data data/save-train.t7 -save_model model

Can you make sure that you are using an up-to-date release_model.lua script? I suspect that this fix is missing.

I use the latest version of release_model.lua  script,  I also tried using multiple older versions, but the problem persists.
The training process is as follows:
When training  to epoch5, interrupted it, and then created release model: model_epoch5_9.47_release.t7
model_epoch5_9.47_release.t7 can work fine in cpu mode
and then continue to train, my resuming training script as below:
th train.lua -train_from  model_epoch5_9.47.t7 -start_epoch 6 -continue   -gpuid 1 -layers 2  -max_batch_size 64 -rnn_size 500 -optim adam -learning_rate 0.0002  -end_epoch 14  -save_every 5000 -data data/save-train.t7 -save_model model
When the training is over, and then run:
 th tools/release_model.lua -force -model model_epoch14_8.51.t7 -gpuid 1
run translate script:
th translate.lua -model model_epoch14_8.51_release.t7 -src s.txt -output result.txt
I got error informations:
work/openNMT/torch/install/share/lua/5.2/torch/File.lua:343: unknown Torch class <torch.CudaTensor>
stack traceback:

I also tried to use lua51, lua52, but the problem remains.

Thank you. I was able to reproduce the issue with the useful information you shared.
I fixed the root of the issue so that new trainings will not be impacted by it. On you side, you could:

Update the code with the fix and start another retraining with the latest model. The next saved model should work on the CPU after a release.
OR
Add the following line in release_model.lua:

checkpoint.options.optim_states = nil

just after the line 61:
checkpoint.info = nil

thanks for your great help!


pwconf workflow:help opens webpage everytime Alfred is open without pressing Enter

I found out about this just now.
How to reproduce

Trigger Alfred
Type pwconf
Select "Open Help" then Enter
Github page is opened as intended
Trigger Alfred again. Now it will open Github page without me pressing Enter.

That can only be bypassed by quickly hit any key, or delete key upon Alfred pops out to clear the query.

This happens because you have the following option turned on:
Alfred Preferences > Advanced > History > Show last query if within 5 minutes
To stop it happening, turn off the above option.
"Active" queries are a fundamental part of the way the underlying workflow library works. It's very unlikely that I will change that behaviour at this point: you're the first person to raise it as an issue in ~4 years, and it  is generally a useful feature for workflow authors and users.

Thanks for looking at this @deanishe.
Even though I'm not really getting the "active" query concept here as I'm confused why it required pressing Enter the first time, and not subsequent times when I trigger Alfred with the query showing. If it's your decision, then that's it. I guess I will not be using the query that much to be bothered with.
Thank you for making this awesome workflow!!!

Even though I'm not really getting the "active" query concept here as I'm confused why it required pressing Enter the first time

You don't have to press Enter. The way it works is the library the workflow is based on allows you to register "magic" queries that cause the library to carry out certain actions automatically (opening a URL in this case) as soon as the user query matches. It is not necessary (nor possible) to action the item (i.e. pressing Enter on it doesn't pass control back to Alfred and onto the next workflow element).


ipmicfg: 1.30.0 -> 1.32.0

Motivation for this change
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109817 run on x86_64-linux 1

1 package built:

ipmicfg

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109817 run on x86_64-linux 1

1 package built:

ipmicfg

LGTM. Maybe add yourself to maintainers as well since I don't really need this package anymore.

I'll consider it with the next update.

LGTM. Maybe add yourself to maintainers as well since I don't really need this package anymore.

I'll consider it with the next update.


Update machine config - 1892

[Auto generated]
Number: [#1892]

openshift-python-wrapper(9f1119f0-3f01-11ef-9ded-b24cefa3f5c5)[PR 1892]: Closing issue for PR: Update machine config.
PR was closed.


Bootloading Python script cannot find attached watch

The wristwatch.py script function 'find_device' starting at line 64 cannot connect to an attached watch.  It is looping over all the detected HID devices that match the VID/PID for the watch and then checking for the correct Manufacturer/Product pair.  However, this information is not present in the HID objects being generated by the hid.enumerate() function.  If you print one of these objects you get this ::
{'path': b'3-2:1.0', 'vendor_id': 5824, 'product_id': 1500, 'serial_number': '', 'release_number': 17, 'manufacturer_string': '', 'product_string': '', 'usage_page': 0, 'usage': 0, 'interface_number': 0}
...so you can see that the Manufacturer/Product entries are null values.  This makes sense since the MANUFACTURER='kevincuzner.com', PRODUCT='LED Wristwatch' values that are being searched for don't appear in the STM32 project code.  The USB descriptor defined for the watch in the usb_desc.c file is ::
/**

Device descriptor
*/
static const USB_DATA_ALIGN uint8_t dev_descriptor[] = {
18, //bLength
1, //bDescriptorType
0x00, 0x02, //bcdUSB
0x00, //bDeviceClass (defined by interfaces)
0x00, //bDeviceSubClass
0x00, //bDeviceProtocl
USB_CONTROL_ENDPOINT_SIZE, //bMaxPacketSize0
0xc0, 0x16, //idVendor
0xdc, 0x05, //idProduct
0x11, 0x00, //bcdDevice
1, //iManufacturer
2, //iProduct
0, //iSerialNumber,
1, //bNumConfigurations
};
...so as can be seen here, iManufacturer and iProduct don't point to a string.  To get around this issue I am simply disabling the checks for Manufacturer/Product and just returning the first object found matching the correct VID/PID which should already be unique enough as it is.  But I'm left curious though as to how this code could have ever worked initially.

This is a permissions issue. You need to see if the handle to the device in /dev is readable and writable by your user. If it isn't you'll not be able to actually query the device. Similar behavior can be observed by running lsusb -v and then sudo lsusb -v. You'll find that the sudo version outputs much more information as it opens the device handles and queries for more extensive information.
To remedy this you'll need to either run the python script with sudo or add a udev rule like this to /etc/udev/rules.d/. I typically name mine something like 99-hidraw.rules or 10-voti.rules. The number at the beginning is used for sorting to ensure a certain order when processing the rules.
# Permissions for libusb raw acccess
SUBSYSTEM=="usb", ATTRS{idVendor}=="16c0", ATTRS{idProduct}=="05dc", MODE="0666"

# Permissions for higher-level hidraw access (this is what Python uses)
KERNEL=="hidraw*", ATTRS{busnum}=="1", ATTRS{idVendor}=="16c0", ATTRS{idProduct}=="05dc", MODE="0666"

Thank you for the response.  I'm not clear on how this comes down to a permissions issue since as I mentioned the 'kevincuzner.com' and 'LED Wristwatch' strings don't appear anywhere in the C code, so how could they possibly be included in the USB descriptor?

Sorry for the late reply. Please see this file: https://github.com/kcuzner/led-watch/blob/master/firmware/src/usb_desc.c
In my later projects, such as the midi-fader, I used an xml-based descriptorgen.py which scans comments for XML and assembles the descriptors from them, including string descriptors like these.


Simple whitespace cleanup

This makes the spacing in the file more consistent

Made the change myself in the online editor. Github magic.

Aaaaaand I forgot a spot. Forget about that magic.
Pushed a new change.

Oh alright! I will go ahead and change the other instance.


Can`t make my window title bar custom

Issue Type: Bug

Im choosing Title Bar Style - custom by adding
"window.titleBarStyle": "custom" to User level settings.json
My title bar is still native to OS X Big Sur
What am I doing wrong?

 Do you have this conflict? Can you share a gif or screen shot?

@sbatten I checked in settings. And it seems that window.nativeTabs turned off.
Here is my VS Code screenshot:

@fedorinoGore thanks for the added info. Can you try loading vs code with a fresh user directory and no extensions installed. You can do this without modifying your existing settings by specifying empty directories to the CLI like so:
code --user-data-dir ~/Desktop/emptyuserdatadir --extensions-dir ~/Desktop/emptyextdir

@sbatten Thanks! That worked out. The VS Code has started and now it has custom titlebar.
Please advice - how to track the setting or extension that causes the problem?
Or I need to reset VS Code and customise it from scratch?


Make first row and first column sticky on tables.

This makes things easier to read on large tables.

Duplicate of #2255


BLE Encryption

I have an Xamarin application and I have been using your plugin to communicate and it works flawlessly.
However now with one customer we have a requirement of encryption and the need to pair.
I can't find anything on Plugin BLE supporting this?
Is that scenario possible with Plugin BLE?

Hi @sisaacks,
I'm the friendly issue checker.
It seems like (100.00 %) you haven't used our issue template :cry: I think it is very frustrating for the repository owners, if you ignore them.
If you think it's fine to make an exception, just ignore this message.
But if you think it was a mistake to delete the template, please close the issue and create a new one.
Thanks!

Pairing (bonding) is triggered in the os if you are writing a secured characteristic.

@midoragh meaning the exterior device has a secured characteristic?  From my understanding Plugin BLE does not support this mainly because of iOS.  I cannot find a specific example.


a structured curriculum with some modularity?

Common questions from instructors
As an instructor trainer and mentor for Data Carpentry, some of the most common questions I hear are:

how do I chose what material to cover/omit to fit my workshops needs and constraints?
what order should I teach things and what sequences have had more success in the past?
can I veer off script?
can I contribute my own motivational story rather than the story provided in the lesson materials?

This makes me wonder if there is a way we can provided structured modularity to the lesson materials.
Modularity idea borrowed from LaTex
I'm using of Latex and Overleaf to write and compile my manuscripts and phd thesis. I've got a system where I keep all the subsections save in individual files and I call them in the appropriate order.
This makes me wonder if we could create a curriculum that has structure and modularity so that instructors can tailer their teaching script to the workshop's needs and contraints. The added benefit is that these reproducbly-created workshop documents could serve as a record of what material was or was not used in each workshop, which could help us better assess how things are working.
Here is a hypothetical way that an instructor could simultaneously outline the workshop's lesson contents and/or create the Jupyter notebook file for use in workshops A and B:
%%% Workshop A
\input{learningobjectives_A}
\input{introduction_forbiologists}
\input{installation}
\input{programming}
	\input{exercise_01}
	\input{exercise_03}
\input{publishing}
\input{git}
	\input{exercise_05}
	\input{exercise_07}

%%% Workshop B
\input{learningobjectives_B}
\input{introduction_forlibraians}
\input{installation}
\input{programming}
	\input{exercise_02}
	\input{exercise_03}
\input{git}
	\input{exercise_05}
	\input{exercise_06}
\input{publishing}

Does that make sense? Does anyone think it would be useful or problematic?

I think this is a great idea. As was discussed in the "what will be the motivating dataset(s)" thread, I think participants will get the most use out of this workshop if it is catered to their field of study. Developing a curriculum with the same basic backbones, but interchangeable lessons that are directed to the workshop audience, will provide enough structure and adaptability to maximize impact.

This relates to the data set selection idea too - that is, a generic data set or subject specific datasets? Depending on how a curriculum is framed, it may be possible to use a generic example as a toy world to demonstrate a principle using a simple teaching example, and then allow the exercise to be repeated / extended using domain specific data? There are risks in using two datasets in a workshop though, as there are risks in allowing participants to use their own data.
Does the abstraction / plugin idea also work for the actual tool selection too? eg Jupyter notebook vs RStudio? To what extent might "scaffolding" slides be reusable across workshops?
And to what extent does modularity work for different languages - eg R vs Python vs (Python)pandas ?
For example, should a modular set of workshop slides include generic slides/scaffolding that could cover the following?

pandas
R
...

Jupyter notebook

Rstudio

...

As a side note, I've started to notice "tidy data" themes from the R world appearing in py contexts eg the pandas cheatsheet

I was working with colleague teaching a course in data science for their field, the way they structured each week was basically having a list of topics they wanted to cover for that week. These topics would be drawn from one of "city planning", "coding", and "data analysis".
In this way they made it easier to make sure the course was covering new territory in each of those domains, and I think it also helped connect the course with more broad data science-y topics.
I thought it was a good way to achieve some degree of modularity and I think it might be worth exploring.

We have very good outline for the workshop now. Each group can create instructor notes about the suggested order of the lesson. Will keep these ideas in the back of my mind. Thanks everyone for you contributions to the discussion!


Optimisation: Don't pencode() singletons as back references

This commit reduces the encoded size of repeated None values (from 5
bytes to 1 byte) and any bool value(from 5 bytes to 1 byte). To achieve
this

None, False & True are encoded before backref checking is performed
the wire format is modified, to encode bool values in the type code.

$ python -c"import chopsticks.pencode; print len(chopsticks.pencode.pencode([None]*100))"
501
$ python -c"import chopsticks.pencode; print len(chopsticks.pencode.pencode([True]*100))"
502

$ python -c"import chopsticks.pencode; print len(chopsticks.pencode.pencode([None]*100))"
105
$ python -c"import chopsticks.pencode; print len(chopsticks.pencode.pencode([True]*100))"
105

I was playing and stumbled across this. Feel free to reject it for any reason or no reason, I bow to your will.

Although the tests passed, this change was wrong. Reverting and retrying, with improved tests

Outstanding! Thanks!


Modular Page Issue

I created a base theme from the devtools plugin, and I'm trying to make a single, modular page. However, when I go to load the site in my browser I'm getting no content, aside from the default nav bar and footer. I'm 95% sure that my "page" directory structure is correct, because when I switch to a different theme (I've tried Antimatter and Bones) I see my page content as I'm supposed to.
The only things I've done so far after downloading the base theme are:

create "modular.html.twig" in the "template" directory with the following content:

{% extends 'partials/base.html.twig' %}

{% block content %}
    {{ page.content }}
{% endblock %}

Create a directory called "modular" in the "templates" directory with the 4 files that correspond to the Markdown files in the "pages" directory

Screenshot of my theme directory:

All of the files in the "modular" directory have the same content (as this is my first time making a modular page and wanted to test it):
``
 {{ page.content }}
``
EDIT:
I just tried making a base theme again using devtools, and now this is happening:

I'm not sure if it's related or not...

What's the content of your modular.html.twig page?
Also, what is your pages folder structure? is the home page a modular page?

The home page is indeed a modular page. The contents of my modular.html.twig page is in my OP, and my page folder structure is:

Ok the problem is modular.html.twig (missed it!) - copy it over from Antimatter. It must loop the child pages. Otherwise all it does is output the modular.md page markdown content.

@flaviocopes Fantastic! Thank you SO much.


Google’s built-in web translator does not work

There is not much to describe. When I want to translate a website and select “Translate to [language]” by right-clicking, nothing happens. This is not due to the settings, which are identical to an Iron Secure Browser installed in parallel.
v. 126.0.6478.256 in Win10 Pro

Oh dear, shame on me. I had “--ungoogled-supermium” as a start option. No wonder it didn’t work. Without this start option, Google’s built-in web translator works fine.

Small addendum: Basically, I would prefer to use Supermium with ”--ungoogled-supermium”. Is there any way (maybe a “flag”?) to use the ungoogled option and still be able to use Google’s built-in web translator?

Strange, it works fine for me.

If I can allow requests for translation while ensuring no other requests are made to Google through Supermium itself (as opposed to visiting Google web pages, as is already permitted), I'll consider making a flag to allow an exception for the built-in translator.

Hello Shane,
That sounds good, or rather it’s very kind of you to consider making a flag to allow an exception for the built-in translator.
In general, I’m very fond of Supermium, which “cures” just about everything annoying about Google Chrome updates.
In issue #474, candrapersada gave the tip to add “disable-sync” to the --ungoogled-supermium switch for another concern. Would there be a similar addition to the --ungoogled-supermium switch for Google Chrome’s built-in translator?


[HOLD] Web - Chat - Composed box is not sticked to the bottom when user is using collapse and expend options.

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

Action Performed:

Go to staging.new.expensify.com
Log in with any account
Go to any chat
Use Shift +Enter key a couple times to go to new line
Type any messages
Use Shift +Enter key a couple times to go to new line
Select Expand and Collapse option

Expected Result:
Composed box should not move when user using collapse and expend options.
Actual Result:
Composed box is not sticked to the bottom when user is using collapse and expend options.
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers): any
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

https://user-images.githubusercontent.com/93399543/196508422-44c046e6-295f-4d81-b7d3-6bfe177fe74f.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Thanks! Keeping it on HOLD then

Issue not reproducible during KI retests. (First week)

PR was merged. Keeping issue open for a while to ensure it is not reproducible

We should assign someone from Bug Zero to test this one now that #11437 is merged.

Haha, that's quite ironic. I will test this weekend.

I wasn't able to reproduce this issue, so I'm closing.


[release-1.21] Fix eds cluster loadassignment with invalid endpoint

Manual cherrypick required.
#50690 failed to apply on top of branch "release-1.21":
Applying: fix eds cluster has invalid endpoints
Using index info to reconstruct a base tree...
M	pilot/pkg/xds/endpoints/endpoint_builder.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/xds/endpoints/endpoint_builder.go
CONFLICT (content): Merge conflict in pilot/pkg/xds/endpoints/endpoint_builder.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 fix eds cluster has invalid endpoints
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

fixed