Vivus will not initialize on DOM elements added by Ajax or HTML interpolation (Vue.js)

Please define your problem or issue as clear as possible.
I am using Vue.js with Vivus in my case. Vue has a handy way of passing a direct DOM node reference of a component around for use with libraries like this through this.$el.
In my mounted() hook (the equivalent of element.onload or jQuery.ready()), I am initializing Vivus after requesting an SVG template through fetch and assigning the response string to a data variable on the Vue component. Using v-html in my component template, I am assigning that component's innerHTML to be the resulting SVG string. The SVG displays just fine.
I'm also making sure to make the call to Vivus after the template is fetched and added to the DOM, so this isn't a race condition issue.
Vivus fails to see that the DOM node is an element, even though I'm logging the node prior to making the Vivus call in the CodePen.
console.log(this.$el) // -->
<svg>...</svg>

new Vivus(this.$el, { duration: 60 })

Results in:
Error: Vivus [constructor]: "element" parameter is not valid (or miss the "file" attribute)

Vivus version:
0.4.2
Browser:
Chrome 60.0.3112.90 (Official Build) (64-bit)
Steps to reproduce it:

Import (webpack) or fetch an SVG and inline its string result into an HTML element through innerHTML or with v-html in Vue.js.
After this is done, call Vivus on the parent element.
The element passed in is not recognized.

Other considerations

I tried passing an ID to Vivus of the parent component's root element, which wasn't recognized as a node either, despite already being mounted and ready on the DOM. It's a mystery. :(

JSFiddle link (or similar platform):
https://codepen.io/andymerskin/pen/00eca5c1836b5ff43c2db2aa1981b5db/
Any help would be appreciated. I've tried every approach I could think of at this point. :/ Vivus is amazing and I'd love to use it!

Hi Andy,
Before all, thank you very much for taking the time to fill an issue correctly. I really like the way you introduced your approach with the different alternatives. It's so useful and pleasant, and I wish everybody was like you. 👍
I played with your codepen and managed to make work. Vivus constructor only takes the ID of a DOM element or the DOM element itself. Not the parent of the SVG.
As you mentioned the this.$el is
<svg></svg>
 which is not valid for Vivus. But you can pass the first child and the magic will happen : this.$el.childNodes[0].
new Vivus(this.$el.childNodes[0], { duration: 60 })
I hope this will be helpful
Max
PS: You found a nice workaround to bypass CORS ;-)

@maxwellito
Thanks for the swift reply. I'm glad I could make dissecting this a little easier for you!
I thought for sure I tried a similar approach in Vue, but I must have missed grabbing the child element. (DOM operations are discouraged in Vue wherever possible, but in this case it's unavoidable).
Thanks for the simple fix. I'll give that shot in my larger project next week and see how it goes.
I appreciate your help!
On CORS: Yeah, it's weird. CodePen allows CORS for including assets in your project, so I was surprised the file option didn't take, though that's just passing the URL to an <object>'s data attribute I'm assuming, so that's probably why.
It would be kind of nice to see Vivus support making an AJAX request for instances where using an object element isn't ideal. But fetch and other libs make it painless anyway, so ¯_(ツ)_/¯

:-)
Its pretty cool you sent this issue at this moment because Im rebuilding
triangulart in vue.js, just to discover it. It would be interesting to see
if the framework provide an option to avoid DOM checking on this component
to improve performances.
I will properly consider doing an AJAX request to load the SVG but I will
have security concerns to check. But I would love to implement it only with
'fetch' and not all these old APIs (actually it shouldn't be a problem).
Thanks again for your issue. You have no idea how pleasant it was :-) Do
not hesitate to ask for help!


Update Permutation class and kernels.

This PR updates the permutation class and the kernels.

The constructor is updated to take in the type of permutation with the sizes.
Similar to the strategies, the permutation masks can be get and set to allow for some flexibility.
The CUDA permutation kernels for row, column and their inverses are added.

TODO

[ ] Add HIP kernels and tests.

Could you also add kernel invocations and tests for HIP?

@thoasm

on a side note: Why did you skip the 2 (0b0010) in your mask options? (You have row_permute = 1, column_permute = 4 and inverse_permute = 8)

Initially, I had both inverse_row_permute and inverse_column_permute and then combined those into inverse_permute. I think I forgot to change the values.


[WIP] Net plugin multithread

Change Description

Testing a multithreaded read in net_plugin.

Consensus Changes
None
API Changes
None
Documentation Additions
None

I disabled the AWS 2 build and test steps in buildkite because last night at ~2 AM Amazon pushed an update to their filesystem package that causes "yum update" to fail, breaking our image. Automation will fix this Monday morning and I can revert these changes for you, but please be sure not to merge any modified files in the /.buildkite/ folder to develop.


Remove unused isNull utility function

/botio test

/botio-windows test

Not only is this not used, it doesn't appear to ever have been in use :-) Thanks for the patch!


fix: missing build of local helm-charts for public cloud CI

Fix #1105

Note from the slack:

Yes that's the idea but this part of code has to be changed too
https://github.com/epinio/epinio/blob/main/acceptance/helpers/epinio/epinio.go#L25-L27

The Install() function currently receives only a set of strings, to become the args of the helm install command.
This actually includes the domain information, in a domain=... argument.
I am loath to parse the domain out of that.
As a first attempt I will see what breaks if I change the func to take the domain information as an explicit argument as well.

RKE failure - Domain looks ok, chartmuseum startup seems to fail, or is not properly recognized as started.
GKE failure - The domain looks broken.

Giving for today.

Looking into the GKE flow I see that it is possible to have a secret value in the domain. I now believe that the domain is correct, and the weirdness in the log is just GH hiding the secret to prevent its leakage through the log.
With that said, GKE then has the same issue as RKE. I.e. the museum is either not coming up, or the code waiting for it is simply not managing to reach it.

With the curl output visible GKE reports

0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0curl: (6) Could not resolve host: chartmuseum.id17997-***

I suspect RKE is similar, except it ran into a cluster creation error, i.e. never reached the place of interest :(

Lets see what I can get when running locally for comparison.

That works, as expected. It does not really give a clue towards the issues with PC ...
... Looking at the instructions for getting the CM url, they do not really look correct for PC ...
Having a look at chartmuseum/templates/NOTES.txt in the CM chart I notice that the CM entry service has a variety of options it can be configured with, i.e. NodePort, LoadBalancer, and ClusterIP.
The default is the last in that list, ClusterIP.
That does not seem to be good for PC.
Based on the text I suspect that I need LoadBalancer.
... On the other side, GKE does not even resolve the constructed cm url ...
Maybe we simply need an entry in some kind of DNS instead ?
I will have to retry the RKE, given that it uses an IP in a magic domain to construct the url. Which worked when construction was done in the go testsuite code.

RKE fails on getting micros os from the rancher repo ... That is something I cannot fix. Just wait until that repo is back ...

Back to GKE I see management by Route53, and adding the loadbalancer there as part of the setup ... Ok ... Lets try to get the CM configured for loadbalancer and see if that helps.

The service config looks mostly ok to me. The service exists, has a loadbalancer ip

"status": ***

    "loadBalancer": ***
        "ingress": [
            ***
                "ip": "<IP_ADDRESS>"
            ***
        ]

The

"externalTrafficPolicy": "Local",
might bite us.

As before the chartmuseum.id... host does not resolve.
I am tending to see it more on the DNS side, that our Route53 setup needs an entry for the chartmuseum host.
If so the service type might not be relevant.
@ldevulder @juadk May I ask you to look over my latest code and comments above ?
I fear that it may need debugging outside of CI, where changes can be trialed faster.
And also somebody with access to Route53.

@ldevulder @juadk May I ask you to look over my latest code and comments above ?
I fear that it may need debugging outside of CI, where changes can be trialed faster.
And also somebody with access to Route53.

I will check this yes and try to test it on my lab.

I just took a quick look at your code and the GKE issue.
I think you are right regarding the missing DNS entry for chartmuseum. However, I wonder if it could not be simpler to install chartmuseum in the github runner locally, this means a rework of the script.
What is the best between the 2 solutions...

Instead of reworking the script maybe a new script first ? Less conditionals to confuse ...
I cannot truly say which approach would be better. I do not have enough insight into either to make an estimate.

Actually, we have the script somewhere already (https://github.com/epinio/helm-charts/blob/main/scripts/build-charts-ci.sh).
It is used in the helm chart repo to build chart locally in the github runner.
Otherwise, if we choose to update DNS, we will have to deal with tedious things like cleaning and mainly waiting for DNS update in route53.

That sounds lovely ... Will have a look ...

Having thought about having the CM in the runner instead of the cluster ... That would mean that all the work on the system domain was superfluous. Mind, that is not bad, from first glance it keeps things simpler. Just sad about the lost time.
It also means that I believe that I should restart the work from scratch.
Although ... I should keep the removal of the code adding a helm repo for the chart, in the epinio Install() helper. That looks to be still needed.
Ok. I seem to have a plan again.

Ok. I seem to have a plan again.

I totally agree with your new plan, locally will be simpler to manage.

While GKE PCCI failed it looks to be "just" a mismatch in repository names.
The helm chart setup script creates a repo chartmuseum and the testsuite expects the repo epinio-charts instead.
That should be fixable.
To check viability first a hack which modifies the script in the action before I go for having a helm-charts PR adding customization features.

Ok. While still a failure this may now be an issue with the charts we are testing, instead of the setup.
My reading of the reported

Error: INSTALLATION FAILED: failed post-install: job failed: BackoffLimitExceeded

and log entries before it is that the testsuite is able to see the local chart museum, and apparently is able to pull the epinio-installer chart. And then fails to deploy the chart. This may be an issue in the current charts, or an issue caused by the patching of the epinio-installer chart done by the chart museum setup script. The host.k3d.internal magic.
Generally the new approach feels more viable tough, I am already much further along than with the first attempt.

@juadk I believe it is the host.k3d.internal magic which is the issue here.
This is magic for a k3d cluster to talk to the localhost.
The clusters made by the CI for public clouds are not based on k3d.
In the trial I am currently doing the cluster is made by GKE.
And to such a cluster host.k3d.internal is most certainly not a proper host name for the gh action runner.
Neither would be localhost.
Regular CI, being based on k3d cluster is of course fine with this.
Do we actually allow access from the public cloud cluster to our action runner general ?
If yes, what name would that be ?

It makes sense, I did not think about it but deployment in the GKE cluster can not reach the GH runner.... sounds obvious now...

Do we actually allow access from the public cloud cluster to our action runner general ?

Unfortunately, I don't think so...
I'm thinking.... but this time, I wonder if we really can avoid installing chartmuseum in the cluster hosted in public cloud...

Note: I have kept the latest of the previous attempt marked with a local branch. IOW I can rebase this PR and restore that.

Restored old approach, with compressed history.
Recap of its state:

CM configured for loadbalancer - possibly not needed
CM not reachable from GHA runner - Likely DNS issue, main problem

If we want to install chartmuseum in public cloud cluster, that means we have to deploy an ingress controller before.
Usually we use traefik deployed by Epinio but we need it before in that case, it also means we won't test the traefik installation by Epinio anymore.
Otherwise, we can use another type of service than loadbalancer and ingress for chartmuseum... sounds more and more complex...

@ldevulder Given that you and @juadk are now deep into the guts of this, should I assign this PR over to you ?

After more tests in Public Cloud I can confirm that it seems to be impossible to use the epionio-installer chart (aka. big helm chart). This is because epinio-installer also uses helm to install dependencies and in that case it sounds like these other helm installations are run inside some sandbox and in that case the port forwarding doesn't work...
So the solutions I see here:

let is as is, which can lead to some failure when the helm charts are changed with important modifications
use the Method 2 installation process: could be good as this one is not tested in CI
use a public chartmuseum/pre-production helm repository

@andreas-kupries you added some debugging code in your commits, do you think that some should be removed or not? My tests on RKE, GKE and EKS are OK, so I will re-read the code next week before putting the PR in the review column.

@ldevulder IMHO the debug code should be removed.


[Node] qnaThreshold parameter

Hi,
The qnaThreshold parameter specified in the IQnAMakerOptions interface does not seem to have any effect on the recognizer. See below for detailed explanation.
Example:
Here I have configured a QnAMakerRecognizer with a simple intent dialog and have specified the value of 0.6 in qnaThreshold parameter in the options.
var bot = new builder.UniversalBot(connector);
bot.set("storage", new builder.MemoryBotStorage());

var intentRecognizer = new builder.LuisRecognizer(constants.modelUrl);

var qnaRecognizer = new cognitiveservices.QnAMakerRecognizer({
  knowledgeBaseId: constants.knowledgeBaseId,
  endpointHostName: constants.qnaEndpoint,
  authKey: constants.qnaAuthKey,
  top: 3,
  qnaThreshold: 0.6
});

var intents = new builder.IntentDialog({
  recognizers: [intentRecognizer, qnaRecognizer]
})
  .matches("qna", (session, args) => {
    console.log(args);
    var answer = builder.EntityRecognizer.findEntity(args.entities, "answer");
    session.endDialog(answer.entity);
  })
  .onDefault(session => {
    session.endDialog("This is the default dialog.");
  });

bot.dialog("/", intents);

Actual Response:
QnA Maker is also returning answers which are of lower scores than the specified qnaThreshold
restify listening to http://[::]:3978
ChatConnector: message received.
UniversalBot("*") routing "windows" from "emulator"
Session.beginDialog(/)
/ - Session.sendBatch() sending 0 message(s)
/ - IntentDialog.matches(qna)
/ - waterfall() step 1 of 1
{ answers:
   [ { questions: [Array],
       answer: '... omitted ...',
       score: 0.5489,
       id: 26,
       source: 'ChatQnA.pdf',
       metadata: [] },
     { questions: [Array],
       answer: '... omitted ...',
       score: 0.5162,
       id: 13,
       source: 'ChatQnA.pdf',
       metadata: [] },
     { questions: [Array],
       answer: '... omitted ...',
       score: 0.4851,
       id: 14,
       source: 'ChatQnA.pdf',
       metadata: [] } ],
  score: 0.5489,
  entities:
   [ ... omitted ... ],
  intent: 'qna' }
/ - Session.endDialog()
Session.sendBatch() sending 1 message(s)

Expected Response:
QnA Maker should have ignored answers which scored below specified qnaThreshold. (Which is 0.6 in this example)
Possible Causes:
It seems to me that, the parameter qnaThreshold in the IQnAMakerOptions interface is not being used in the actual request or anywhere else aside from being declared as an option. So, it is obvious that it has no effect on the responses being returned from QnA Maker.
Solutions:
We should send the threshold parameter in the request to QnA Maker's generateAnswer endpoint if it is supported on the server side (it should be). Since it is basically the same way top works, and we'll avoid downloading answers that are not required, thus saving data and time.
For this, I tried adding "qnaThreshold" parameter to the raw request and sent it through Postman. It is still returning the same answers, below threshold. So, maybe filtering based on threshold is not supported in the QnA Maker API v4.0 GA.
See:

https://docs.microsoft.com/en-us/azure/cognitive-services/qnamaker/how-to/metadata-generateanswer-usage

https://docs.microsoft.com/en-us/azure/cognitive-services/qnamaker/quickstarts/nodejs#GetAnswers

Also, the strictFilters parameter in the API is only for using with metadata, so can't filter score through it.
Aside from few small descriptions at the above links, I can't seem to find any docs about the REST spec of the Endpoint (Runtime) API.
There are docs for v4 API Reference (https://westus.dev.cognitive.microsoft.com/docs/services/5a93fcf85b4ccd136866eb37/operations/5ac266295b4ccd1554da75ff), but it seems it's only for Management APIs and there is no information about the generateAnswer operation.
I would request that it should be documented clearly whether it is supported on the server side to filter answers based on threshold. And the client libraries should be updated to reflect that.
Possible Workarounds:
If due to any reason, it is not supported (yet) on the server side then, either the client libraries should filter the answers (for the time being, maybe based on a flag) on the client side after getting the response from QnA Maker (since the parameter is already specified in the IQnAMakerOptions interface).
Until we get the parameter working on the server side, we can use some kind of wrapper around the recognizer to filter out the answers that scored below the specified threshold before returning the response to the dialog.
Please look into this and update as this is an important feature and it is required in many scenarios.
EDIT: It appears that the .NET version of the SDK does actually filter the answers on the client side. (See: https://github.com/Microsoft/BotBuilder-CognitiveServices/blob/master/CSharp/Library/QnAMaker/QnAMaker/QnAMakerService/QnAMakerService.cs#L196). So I guess it would be fine to implement the same in the Node.js SDK as well. I would be happy to submit a PR once I get confirmation.
Thanks,
Afnan

Hi @afnan-ahmad,
'top' and 'qnaThreshold' are options to QnAMakerDialog, not QnAMakerRecognizer. Instead, if you were using QnAMakerDialog, you would apply them there. However, you are using IntentDialog which:

Does not offer a 'top' option
Takes 'intentTheshold' instead of 'qnaThreshold'

var intents = new builder.IntentDialog({
    recognizers: [intentRecognizer, qnaRecognizer],
    defaultMessage: 'No match! Try changing the query terms!',
    intentThreshold: 0.6

Please review the IntentDialog reference  for more information.

Hi @stevkan,
Thanks for your response. I would to like clarify here that if you look into QnAMakerRecognizer class, you would see that the options (including top and qnaThreshold) are part of the recognizer itself and not of the dialog. (There are other options for the dialog)

See: https://github.com/Microsoft/BotBuilder-CognitiveServices/blob/master/Node/src/QnAMakerRecognizer.ts#L56
The same options have also been defined in the TypeScript typings file as well.

And as you would have seen in my example that I have specified top as well, and it is working. Only qnaThreshold does not work as I have detailed in my original post.
You can also see my changes in PR #136
Thanks,
Afnan

You are right that IQnAMakerOptions is used in both QnAMakerRecognizer and QnAMakerDialog. Forgive my oversight on that. However, your code above shows you are trying to apply qnaRecognizer to IntentDialog which does not accept the qnaMaker recognizer options.
You can reference the IntentDialog class here and its associated options here.


Server side authentication

Is it possible to authenticate server-side as per https://ga-dev-tools.appspot.com/embed-api/server-side-authorization/, but still use the directives included in this script?

Ah, sorry the documentation is no complete.
the ngAnalyticsAuth-directive accepts a serviceAuthToken attribute, where you can pass your server token.
e.g.
<ng-analytics-auth label="Hallo: " service-auth-token="HERE_YOUR_TOKEN" hide-on-auth="true" auth-container="embed-api-auth-container"></ng-analytics-auth>

Perfect. Thanks!

Let me know if i can close the topic or close it on your own :)


AADLoginForLinux source code and customization + force chroot on ssh login

AADLoginForLinux extension: It could be useful to see the source code
Apparently this is a PAM implementation for a login with Active Directory but i can't find the source code in repository with a grep on source.
I try the search in this repo:
https://github.com/Azure/custom-script-extension-linux
May be this second question is not related to documentation: It could be possible to customize this extension or add an Extension to force user to login in a chroot environment (in any case preserving this AD login integration)?
Thank you a lot

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: bb8e2623-1f79-aee3-bd97-8b5f647b6980
Version Independent ID: 6346b54d-cc8d-0b79-a6b8-f9144cfa2459
Content: Log in to a Linux VM with Azure Active Directory credentials
Content Source: articles/virtual-machines/linux/login-using-aad.md
Service: virtual-machines-linux
GitHub Login: @cynthn
Microsoft Alias: cynthn

Thanks for the question. We are currently investigating and will update you shortly.

@antimoroma There is already a feedback post open here for source code availability request. I would recommend up-voting that post to get a reply from the product team there.
I will also share if I get more info on that.
Since this feature is in preview, I am not positive on whether the customization is possible. Please go ahead and share the feedback on [Azure AD feedback forum] so that product team are aware and can act on it.(https://feedback.azure.com/forums/169401-azure-active-directory?category_id=166032)
@cynthn @danielsollondon  Can either of you share if we have the option to customize the extension as asked in the second question?

@antimoroma There is nothing exposed through azure portal or any other client to setup a chroot environment beyond modifying ssh config on the VM.
This is not something that we are officially supporting at this time.
However, we tested the scenario and were able to login to a simple chroot environment by modifying ssdh config as shown below:
Including the following config in /etc/ssh/sshd_config will put the user “user1” into a chroot environment in /var/chroot.
Match user user1
                ChrootDirectory /var/chroot
                PasswordAuthentication no

@kaarins Thankyou a lot for your feedback and suggestion,and for the example, even if it's not supported is will be useful.


chore: update hero page title text

Issue

Closes #793

Description
I also took the liberty of removing the translated string concatenation.
Type of change

[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 not work as expected)
[ ] Prototype/POC (not to merge)
[ ] This change is a refactor/addresses technical debt
[ ] This change requires a documentation update
[ ] This change requires a SQL Script

How Can This Be Tested/Reviewed?
Navigate to the home page.
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have reviewed the changes in a desktop view
[x] I have reviewed the changes in a mobile view
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[x] I have assigned reviewers
[ ] I have updated the changelog to include a description of my changes
[ ] I have run yarn generate:client if I made backend changes
[ ] I have exported any new pieces in ui-components

This also closes #712!

This also closes #712!

Ah, wasn't sure if that bug was referenced anywhere else. I'll mark this as closing that issue as well, thanks!


Low framerate 3d models

The models for the paper doll and rotating chest icon for saving appear to be running at a noticeably lower framerate than the game normally does. Parts of the paper doll appear to move smoothly, like it rotating, but arm and leg movement are choppy. For reference, my client is at a locked 60fps.

Wow, well spotted.
For the chest icon, it's not a 3d model, just an animated texture. Therefor it's restricted by the amount of frames it's given. If it really bothers you you can disable the "Rotating Model" option in the config.
For the paper doll I already tried to make it run at more than 20 fps, but for some reason the model then starts to stutter a lot. Will have to look into this again, but it's not a priority.

Found the cause, fix will be released in the next version after v1.4.3.


Add ability to specify IP

This broke the integration tests. You'll need to update the callers to Start within `integration/integration_test.go.


Fix whisper-fetch.py --drop timestamps

Possible fix for #305 .
I didn't test this, not sure if this is correct.
Specifically I don't know how timestamps and offsets in Whisper files work. Is a simple t + step appropriate, or is taking into account an offset and wrap-around needed?

this should work, the wraparounds are handled by whisper.py

Sorry, forgot about this.
This is definitely an improvement over the current situation, but the json part is still incorrect as it removes values but does not even have a timestamp, there is no way to know which timestamps have been removed.
@cdeil did you have time to test this?

Sorry, forgot about this.
This is definitely an improvement over the current situation, but the json part is still incorrect as it removes values but does not even have a timestamp, there is no way to know which timestamps have been removed.
@cdeil did you have time to test this?


EXTERNAL-IP stays in  state

When I applied your .yaml to my two node cluster (one master, one worker) the external-ip stays in   Here's the service and pod:
NAME               TYPE           CLUSTER-IP      EXTERNAL-IP   PORT(S)        AGE
hello-kubernetes   LoadBalancer   <IP_ADDRESS>        80:30210/TCP   18m
$ kubectl get pods
NAME                                READY   STATUS    RESTARTS   AGE
hello-kubernetes-594f6f475f-4s9nw   1/1     Running   0          18m
hello-kubernetes-594f6f475f-7zb4h   1/1     Running   0          28s
hello-kubernetes-594f6f475f-jlrwb   1/1     Running   0          28s
Browser requests to port 30210 on either host is successful, but I should still be able to get an external-ip to connect on port 8080.
Do I need to change something in the .yaml to run on my two node cluster?

Able to replicate on stock k3s install:
$ sudo kubectl get pods
NAME                                READY   STATUS    RESTARTS   AGE
svclb-hello-kubernetes-npwb2        0/1     Pending   0          48s
hello-kubernetes-594f6f475f-gpfvs   1/1     Running   0          48s
hello-kubernetes-594f6f475f-tss77   1/1     Running   0          48s
hello-kubernetes-594f6f475f-kclvr   1/1     Running   0          48s

$ sudo kubectl get service
NAME               TYPE           CLUSTER-IP     EXTERNAL-IP   PORT(S)        AGE
kubernetes         ClusterIP      <IP_ADDRESS>      <none>        443/TCP        13m
hello-kubernetes   LoadBalancer   <IP_ADDRESS>   <pending>     80:32474/TCP   90s

Am able to connect to https://:32474

Hey all,
I also just ran into this issue on my vanilla k3s install.  From the docs:
If you try to create a Service LB that listens on port 80, the Service LB will try to find a free host in the cluster for port 80. If no host with that port is available, the LB will stay in Pending.
I switched the port for the load balancer in the yaml to 8080 and it works for me now.  FYI.

The recently added Helm Chart and Ingress options should provide options that work across more platforms.


patch-wise model could only reach 68%

I've tried to run the patch-wise model for multiple times, but could only reach a maximum accuracy of around 70% and average  accuracy around 60%, here's all the hyperparameters I've used:

The result has a huge gap towards the released result in the paper, did I do something wrong? or the released result is indeed a very rare result?

May I ask how accurate the PatchWiseNetwork you have trained yourself can be? My PatchWiseNetwork has trained 7 epochs, and the accuracy has been around 31. My parameters are consistent with the paper


CAN/CAN-FD API?

From working with the uavcan rust implementation I see the convenience of having a standardized CanFrame and CAN interface in Rust. Perhaps this is something this crate should aim to have?
I think it would make sense to both have a trait Can and a struct CanFrame.

+1 on this.  My one request would be to not differentiate between frames that have been received and frames that are intended to be transmitted (this seems to be what you're proposing, I'm just making it explicit).  ChibiOS does it this way and I find it creates a lot of duplicate code.  Is this feature only lacking a champion?  If so, I'd be glad to champion it.

Please mock up the interface you have in mind for discussion. A PoC implementation would be even better but is not necessarily required.

Here's my attempt https://docs.rs/embedded_types/0.3.2/embedded_types/can/index.html
i think a trait for sending these types is required as well.

Those types seem to be just the framing, those should probably live in their own crate anyway. What is needed for a hal interface are generic methods for setting up CAN transfers (and doing so) over the CAN peripherals of a typical MCU. I don't know particularly much about CAN transmissions but from reading the STM32F042 reference manual it pretty much seems to build down to:

CAN configuration
Switching between different modes
Stuffing data into a mailbox
Getting data out of a mailbox

Maybe there's more to it and/or other MCUs do it in a complete different way, don't know.

Yeah, CAN configuration is pretty straightforward.  I'm going to take my own stab at writing a proposal over the next day or two.  I do have a question before I dive in though,
In one of @japaric 's hal implementations (F3xx I want to say?) he has his own implementation of units, but the uom crate seems to already be for this purpose.  Would it be preferable to roll my own representation of a frequency type, or would it be better to depend on uom?

Quick preview for you guys:  https://github.com/dunmatt/embedded-hal/commit/24f492295d5856a21ed03542e77dc17f00d30575
It still needs documentation, obviously, but even without it I think it's pretty clear what's going on (for the most part).  One thing I wasn't sure of (again, very new to rust) was if I'm going the move correctly in CanInterface::receive... would it be better to use a mutable ref?
What do you all think?

I think CanFrame would be better as an enum like:
pub enum CanFrame {
    DataFrame(id: u16, data: &[u8]),
    ExtDataFrame(id: u32, data: &[u8]),
    FDDataFrame(id: u16, data: &[u8]),
    ExtFDDataFrame(id: u32, data: &[u8]),
    RemoteFrame(id: u16),
    ExtRemoteFrame(id: u16),
   ....
}

I'm not sure about splitting extended identifiers yet, but it might be a good idea to be explicit. Also with this approach we can support Can FD, which gives us access to faster speeds. That said we need to add mechanism to query capabilities of CAN controller.

@EugeneGonzalez For that struct what are you proposing as the signature of CanInterface::receive?

@robomancer-or there is a fixed number of messages that can be sent: data (only one that can be an FD frame), remote, error, and overload. I just got lazy with writing it out and expanding the enums fully. The current approach is pretty similar to SocketCan on Linux. That's not a bad thing, but I wanted to add some type safety to avoid shooting yourself in the foot. Like a remote frame doesn't have any data.
Ideas on how

Builder with read only CanFrame
Enums (would move the buffer into the enum)
Type Parameters (this would be messy)

More reference material [Uploading can_fd_spec.pdf…](Bosch CAN FD Document).

One of the reason i want to write Rust on embedded is to leverage the type system in good ways. That's why I don't think the single struct socketcan approach is a good idea.
This is the same exact reason why CanFD types should be left out of can frames. If i try to send a CAN-FD frame to a ordinary CAN device it's no reason it shouldn't be caught in compile time.
I think this can be a nice starting point. The CanFrame enum should probably be expanded one layer to look like this:
pub enum CanFrame {
    BaseDataFrame(BaseDataFrame),
    ExtendedDataFrame(ExtendedDataFrame),
    BaseRemoteFrame(BaseRemoteFrame),
    ExtendedRemoteFrame(ExtendedRemoteFrame),
}

Having types such as ExtendedDataFrame allows compile time checks and simpler implementations for protocols that will only use one kind of frames. (like uavcan)

My one request would be to not differentiate between frames that have been received and frames that are intended to be transmitted

agreed

One of the reason i want to write Rust on embedded is to leverage the type system in good ways. That's why I don't think the single struct socketcan approach is a good idea.
This is the same exact reason why CanFD types should be left out of can frames. If i try to send a CAN-FD frame to a ordinary CAN device it's no reason it shouldn't be caught in compile time.

Right, I agree with that whole heartedly.  I would add that it's important to consider the consumers of this API.  As a HAL, we're pretty much only going to be consumed by people writing drivers.  That is, it's not the job of this layer to idiot proof anything past the physical layer, it's the job of this layer to make idiot-proofing hardware portable and to not unduly restrict the options available at the implementer's and consumer's levels.
It's not in my previous preview, but your point about CAN-FD checking at compile time is spot on the money.  It's not up yet, but my next preview is going to have a type parameter on transmit and receive for exactly that reason.

Got another preview for you guys (in the PR above), what do you think?  I found myself needing a CAN parameters calculator, but writing it started turning into a rabbit hole, so I'm going to spin it into a separate can_utilities crate and then depend on that (or maybe not depend on it but leave a pointer to let HAL implementors know it exists and will save them headache... which option is better?).

@therealprof Ok, I've just updated #53 with a greatly cleaned up version.  No sample implementation as of yet, but I'll be working on that today.  In the mean time, I think it's ready to be seriously looked at.


New Rendering code mappings

Just making the PR so I can make sure people know I'm working on this. Gonna draft for a while.

@Waterpicker Are you going to maintain this? Please let me know if someone else can pick up your work in case you are no longer interested in maintaining this.

Sorry I've been gone. Been busy!


Couldn't save code（vue）

Version: 1.42.1 (user setup)
Commit: c47d83b293181d9be64f27ff093689e8e7aed054
Date: 2020-02-11T14:45:59.656Z
Electron: 6.1.6
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Windows_NT x64 6.1.7601
question:Every time I save, it appears as shown in the figure, which prevents me from saving the code：Running Save Participants for 'xxxx': Quick Fixes

Possible duplicate of #90889

I also encountered this problem. After opening vscode for a period of time, this problem will appear. I don't know what to do at all. Does anyone have any way??

Did you follow the advice in #90889? What were your results?

@13714273163 do you use the ESLint plugin by chance?

I assume this is ESLint's Auto Fix On Save feature: https://marketplace.visualstudio.com/items?itemName=dbaeumer.vscode-eslint
The dup issue is in microsoft/vscode-eslint, so please follow that one.

@13714273163 if you use ESLint can you please comment here: https://github.com/microsoft/vscode-eslint/issues/904
It would be great if you can provide a GitHub repository to clone which demos this.

i just reload Window From time to time


Handle empty values

There are some values such as service ssh disable-password-authentication that are empty maps. This patch handles that special case.

@foltik - fyi

@angelnu I didn't have time to look into this before now, I have synced up with upsteam from foltik since you made this request.
If it is still relevant, please modify it to remove merge conflicts and I will include it, but I recommend working directly against upsteam instead.

It is already merged upstream so closing.


Add badges

Changes Unknown when pulling c4c396f8087a121036e111d142cd90ffe3f45362 on badges into ** on master**.

Changes Unknown when pulling c4c396f8087a121036e111d142cd90ffe3f45362 on badges into ** on master**.


[DO-NOT-MERGE] Verify GitHub Actions test report

What changes were proposed in this pull request?
This PR is to trigger the test report at https://github.com/apache/spark/pull/29333.
Why are the changes needed?
N/A
Does this PR introduce any user-facing change?
N/A
How was this patch tested?
N/A

Test build #127433 has started for PR 29429 at commit c356076.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32054/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127433/
Test FAILed.

Test build #127437 has started for PR 29429 at commit ed2bb56.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32058/
Test PASSed.

Test build #127439 has started for PR 29429 at commit a9dc47e.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32060/
Test PASSed.

Test build #127437 has finished for PR 29429 at commit ed2bb56.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Test build #127439 has finished for PR 29429 at commit a9dc47e.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127439/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127437/
Test FAILed.


Adding dependency to files

I have been working on etcher locally for a bit, and I made it so it detects a specific image file on my machine and then when etcher is opened the file is selected already. However, if I run this version of etcher on a different machine the app will error endlessly. I think it has something to do with the fact that this image does not build correctly with the app or it is not detected somewhere. Is there a way I can change this, perhaps in the scripts it runs? Sorry if it's a bit too specific to me to be able to offer a solution, which is fine, but thanks for any help provided anyway!

Alright, I have a couple questions to help me understand the situation better:

Which operating system are you on?
What build steps are you running?
What are the errors it's spewing out?
Between what kind of machines are you moving it? Same OS, same architecture (i.e. both 64bit)?
Is the image supposed to be bundled with Etcher, or is it just in some location on those machines?

Whoops, sorry I was wrong about the error it spams. It actually spams there is no such file. I placed it in the assets folder in etcher, but it looks at the folder under C:/Users/USERNAME/AppData/Local/Temp/assets/

The assets directory is used for electron-builder assets, meaning those don't end up in any filesystem related space when building Etcher. You'll want to put that somewhere else, a good place would be lib/blobs/.

I think that takes care of it thanks!

Perhaps this indirectly links to #1386 ? i.e. being able to specify which image-file the Etcher GUI should open by default.

I have to unfortunately reopen this as I ran another build command causing the issues described above. If I run the portable installer (given through make electron-installer-portable) the build fails saying the image file I used cannot be found. I am guessing the file is not building correctly, so is there a NSIS file perhaps I need to change to include this file? Thanks again.

@lurch #1386 is not about this, but about making Etcher understand default OS command line arguments, so that it'll open & use an image file if you drag it onto it, or start it as a command with an argument

@aamm132 is it the build process itself that's failing, or the built Etcher, once you run it? Is the portable installer generate roughly the size of the image you're bundling?

Built etcher is failing once it is ran. It works fine doing the executable that is not the portable installed one. The portable installer is spamming there is no such file or directory, with regards to my image, when I run it.
My image file is about 1,440 KB while the portable installer generated is about 51,006 KB.

Oh, that's a really tiny image. How are you resolving the path to it?

@lurch #1386 is not about this, but about making Etcher understand default OS command line arguments, so that it'll open & use an image file if you drag it onto it, or start it as a command with an argument

Correct, I guess I should have clarified my thinking a bit better... I meant that if #1386 was implemented, which allowed you to start the Etcher GUI with an image-file on the command line, then @aamm132 would be able to solve his problem by distributing a zipfile containing Etcher, his image-file, and a batch-script, with the batch-script then doing something like
Etcher-Portable-1.3.1-x64.exe FILENAME.img

and he wouldn't need to try building a custom version of Etcher.

@aamm132 sorry for the belated response; the issue with the path is likely that the html tpl ends up in the webpack bundle, which sits in the generated/ directory, so the path will need adjusting to resolve from there I believe.


Return to top of page after selecting a new document tile

After scrolling through document tiles in the tile display side panel, and then selecting a document tile. The browser does not auto scroll to the top of the display window, forcing users to re-scroll to the top of the page to see their selected document.

Confirming this is still an issue.
How to reproduce the issue:
0. Set module to be in tiled mode.

Have a document with numerous related items.
Scroll down the edit panel and select a tile at the bottom of the page.
If that selected document, has numerous related items as well, the page won't auto scroll to the top, even though the selected document move's the top.


Implement AVL trees using Binary Tree.

Do the checklist before filing the issue:

[x] Do you have Node.js and all the relevant dev-dependencies installed?
[ ] Is this a bug fix?
[x] Is this an enhancement?
[x] Is this a feature request?

NOTE: Provide a clear and concise description of the feature that needs to be added! Or if its a bug, then provide the necessary steps to reproduce it along with screenshots.

Give your answer below:

A Binary Tree by itself can end up being skew, hence degrading it's efficiency. By balancing the Binary Tree using rotations we can ensure the height of the tree to be O(log n). AVL Tree is one way height-balance a Binary Tree.
For reference:
https://www.geeksforgeeks.org/avl-tree-set-1-insertion/

I would like to work on this.

Can I take another day for this one? I couldn't work on it in the morning and hardly have any time now. I can finish it by 7pm tomorrow evening though. Is that fine?

Sure @sateslayer

@sateslayer your time's up

Can I take this if my previous pr is merged?

can I work on this?

@devRD you're assigned and check the issue #158 for more details. Include all the public methods mentioned there.

@pbiswas101 Can you please assign this task to someone else? I had some emergency and was unable to work on it.

I would like to do that. gssoc19

@gourabk121 are you a gssoc19 participant?


OSError: [WinError 126] module could not be found

Hello,
I tried running
python Main.py --data-name flixster --epochs 40 --testing --ensemble
and got:

File "Main.py", line 11, in
from util_functions import *
File "C:\Users\PCOvice\Downloads\IGMC-master\IGMC-master\util_functions.py", line 13, in
from torch_geometric.data import Data, Dataset, InMemoryDataset
File "C:\Users\PCOvice\anaconda3\lib\site-packages\torch_geometric_init_.py", line 2, in
import torch_geometric.nn
File "C:\Users\PCOvice\anaconda3\lib\site-packages\torch_geometric\nn_init_.py", line 2, in
from .data_parallel import DataParallel
File "C:\Users\PCOvice\anaconda3\lib\site-packages\torch_geometric\nn\data_parallel.py", line 5, in
from torch_geometric.data import Batch
File "C:\Users\PCOvice\anaconda3\lib\site-packages\torch_geometric\data_init_.py", line 1, in
from .data import Data
File "C:\Users\PCOvice\anaconda3\lib\site-packages\torch_geometric\data\data.py", line 8, in
from torch_sparse import coalesce, SparseTensor
File "C:\Users\PCOvice\anaconda3\lib\site-packages\torch_sparse_init_.py", line 13, in
library, [osp.dirname(file)]).origin)
File "C:\Users\PCOvice\anaconda3\lib\site-packages\torch_ops.py", line 105, in load_library
ctypes.CDLL(path)
File "C:\Users\PCOvice\anaconda3\lib\ctypes_init_.py", line 364, in init
self._handle = _dlopen(self._name, mode)
OSError: [WinError 126] The specified module could not be found

Does someone know how to solve that?
Thank you in advance

Hi, from the message, it seems to be an issue about the installation of torch_sparse. Are you able to "import torch_sparse" in python directly? If not, you may try to reinstall torch_sparse and torch_geometric.

Hi, from the message, it seems to be an issue about the installation of torch_sparse. Are you able to "import torch_sparse" in python directly? If not, you may try to reinstall torch_sparse and torch_geometric.

Thank you.
Reinstallation worked.

Thank you.
Reinstallation worked.


Undocumented state drift blindness in network_policy_attachment

Provider Version
v0.25.19
Terraform Version
v1.0.5
Describe the bug
The code for network policy attachment states that it is unable to read the true state, which is not reflected in the documentation. As such, users may not realize that this resource is blind to drift.
https://github.com/chanzuckerberg/terraform-provider-snowflake/blob/main/pkg/resources/network_policy_attachment.go#L77
If this is still impossible to resolve, the documentation should have a very large warning that this resource is unable to observe drift, which is especially critical for a primary security control.
Currently no warning is present on https://registry.terraform.io/providers/chanzuckerberg/snowflake/latest/docs/resources/network_policy_attachment
Expected behavior
Changes made to network policies in Snowflake would be corrected by later terraform apply.
Code samples and commands
Make any change in Snowflake to network policy attachments. Terraform is blind to them.

Also seeing this. If we do a:
ALTER USER <user-name> UNSET network_policy;

After applying the network policy attachment, terraform says No Changes.
Thoughts on fixing this?

We are closing this issue as part of a cleanup described in announcement. If you believe that the issue is still valid in v0.89.0, please open a new ticket.


Keep migration/introspection engine test suite compile times in check

We observe a nearly 1 second compile time increase for a single PR in https://github.com/prisma/prisma-engines/pull/2858

Let's flesh out a proposal to — ideally — make most new tests have no impact on compile times, in the spirit of https://github.com/ziglang/zig/issues/11288.

Hi there,
As part of our effort to clean up our GitHub repo, we’re closing this issue to help focus on features and improvements that align with our focus areas and those of our community.
This issue was originally created for internal tracking purposes, but if you feel this is important to you, feel free to reopen the issue and get like-minded users to upvote as we will prioritize fixing issues based on community interest.
For more insight into our priorities and how we’re thinking about the future of Prisma, please see our blog post: https://www.prisma.io/blog/prisma-orm-manifesto.
Thank you for your understanding and support!


monitor: case insensitive filtering

Case insensitive filtering of frame names in monitor.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.677%

Totals

Change from base Build<PHONE_NUMBER>7:
0.0%

Covered Lines:
7259

Relevant Lines:
7749

💛  - Coveralls

thanks!


Create splash screen.

I thought this might be a good idea after seeing it done on fluttercrashcourse.com but after some thought I think it's unnecessary for our app.


Rename 6to5 project now that ES6 (name) is "dead".

Now that we now we will never see an official ES6, is 6to5 an accurate name ?
I read the 6to5 + esnext post.
Maybe we should keep esnext name don't you think ?
Source
https://esdiscuss.org/topic/javascript-2015
https://github.com/tc39/tc39-notes/blob/master/es6/2014-09/sept-25.md#conclusionresolution-1

See https://github.com/6to5/6to5/issues/568

Interesting.
(FYI, I do more CSS than JS, and I am working on cssnext, so esnext seems a good choice to me ^^)

Discussing in #568


add support for Dgraph schema file directly

I'd like to use graphql files to write my schema, can you add support for it ?
thanks so much! nice job


LAPACKException(22) for 32-bit storage and large matrices

For sufficiently large matrices and 32-bit storage, LAPACKException(22) arises when eigen is called.  eigen calls stegr! which throws the exception.

According to the dstegr documentation, 22 corresponds to error code -2 from dlarrv:

Problem in DLARRF when computing the RRR of a child. When a child is inside a tight cluster, it can be difficult to find an RRR. A partial remedy from the user's point of view is to make the parameter MINRGP smaller and recompile. However, as the orthogonality of the computed vectors is proportional to 1/MINRGP, the user should be aware that he might be trading in precision when he decreases MINRGP.

Can you convert the matrix to Float64 and find the eigenvalues, and find the minimum separation?

In particular, if you generate a random real skew-Hermitian matrix, the eigenvalues always come in ± pairs.  If, on top of that, you have a near degeneracy, this will lead to a near fourfold degeneracy of the eigenvalues, which LAPACK might not be built to deal with?

(If the problem is the near degeneracy, a simple trick would be to shift the tridiagonal matrix by e.g. the Frobenius norm to break up the ± pairs).

I tried to shift the SymTridiagonal matrix as you proposed but it didn't work, the error is still thrown by stegr!.  The following discussion also reports the problem https://bytemeta.vip/repo/Jutho/KrylovKit.jl/issues/57

However, scaling the matrix by its norm(thus scaling its eigenvalues) did work instead of shifting its eigenvalues.

Scaling the matrix doesn't make any sense to me.   The only thing that I can think of is that LAPACK has an erroneous absolute tolerance somewhere, which it really shouldn't — the algorithms to should be invariant under overall scaling.
I wouldn't do a multiplicative scaling, even if it happens to work for some mysterious reason, because it seems like just piling bugs on top of bugs.

(It seems like, since we know all of the eigenvalues come in pairs, there should be a way to reduce it to an eigenproblem of half the size somehow.)

Scaling the matrix doesn't make any sense to me. The only thing that I can think of is that LAPACK has an erroneous absolute tolerance somewhere, which it really shouldn't — the algorithms to should be invariant under overall scaling.
I wouldn't do a multiplicative scaling, even if it happens to work for some mysterious reason, because it seems like just piling bugs on top of bugs.

Scaling didn't make any sense to me either and I don't have any idea why it allowed stegr! to compile. I will have a further look at possibilities to get rid of this LAPACKException

The error only occurs for random SkewHermTridiagonal 32bit-storage matrices. Non-random matrices don't get the exception.
Moreover, the problem is created by the numbers inside the SkewHermTridiagonal(rand(Float32, n)) matrix. I mean by that that casting the matrix in Float64 when one calls stegr! doesn't fix the problem while the SkewHermTridiagonal(rand(Float64, n)) never creates errors.
This paper: https://netlib.org/lapack/lawnspdf/lawn163.pdf addresses the problem, but the solutions they bring ask to modify what stegr! does.

It seems like this will have to be fixed in LAPACK itself.

(Alternatively, you'd have to implement your own eigensolver, e.g. with a version of QR iterations, I guess.   In principle this might be able to outperform the generic symtridiagonal eigensolver because you can take advantage of the diagonals being zero and the eigenvalues coming in ±λ pairs, maybe ala algorithm 530.)

I think that trying to implement a fast eigensolver for tridiagonal skew-symmetric matrices could be a very nice challenge for my last week in Boston. As the package becomes quite complete, it is a very interesting problem to address, which is more on the "math side" of this package.

#95 new skew-symmetric eigensolver was added. It is faster for tridiagonal matrices than the symmetric solver. However it cannot always work where LAPACK failed. For dense matrices, it is faster for matrices of size less than 200(on my machine), then it becomes slightly slower due to the hessenberg reduction that is slower. The skew-symmetric mv product is thus the last stone to beat LAPACK in single threaded.

#95

Can this issue be closed if you are no longer using the LAPACK solver?


package.json: Lock @patternfly/react-styles version

It's part of patternfly-react, so we should update it in a controlled
fashion. Letting it auto-update freely often breaks the build.
Similar to https://github.com/cockpit-project/cockpit/commit/bbde4c01947

This avoids this kind of failures. That happened on the rhel8 branch (I'll send a backport in a moment), but could just as easily hit master. It recently did so for both cockpit and cockpit-podman, we did the same fix there.

Eh, what? That worked just fine locally...

Works now.


⚠ Bofrost.de has degraded performance

In 9b52ce9, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1715 ms

Resolved: Bofrost.de performance has improved in 1330c14 after 1 hour, 5 minutes.


Add guide for when popup doesn't show up

This also randomly occurs when 1password is locked and you try to invoke 1password mini via chrome or sudolikeaboss.

I've installed using brew on High Sierra and kept to the install and setup guide but then, when I use the shortcut nothing happens, also not when I invoke the binary sudolikeaboss from the command line in iterm2. What am I missing?


Timing of events relative to initial script load

Let's say I have the following worker:
var myCache;
caches.open('my-cache').then(function (cache) {
  myCache = cache;
});

self.oninstall = function(event) {
  event.waitUntil(myCache.add('index.html'));
};

Are there any guarantees about the order of operations? Specifically, will the promise created during script load, that sets the myCache variable, be resolved before the install event is triggered?
I'm guessing that, no, this is a race condition, and you can't rely on it. But it limits the useful setup work that can be done when a script is restarted. You can't defer boilerplate stuff to install or activate for scripts that are just being restarted.

There are no guarantees here, it's the same as events in a window.
You'd need to use:
self.oninstall = function(event) {
  event.waitUntil(
    caches.open('my-cache').then(function(myCache) {
      myCache.add('index.html')
    })
  );
};

ES7 async functions make it look more like sync code:
self.oninstall = function(event) {
  event.waitUntil(async function() {
    var myCache = await caches.open('my-cache');
    return myCache.add('index.html');
  });
};

You can use that today using either https://github.com/google/traceur-compiler or https://6to5.org/


NotImplementedError: Converter is not implemented (OperationDescription(domain='', operation_type='If', version=11))

NotImplementedError: Converter is not implemented (OperationDescription(domain='', operation_type='If', version=11))
The onnx file was generated at huggingface.co/spaces/stevengrove/YOLO-World

I met the same problem

Did you solved the problem now?

I also met the same problem


Laravel integration

How to integrate this with an laravel application?

You have installation section on the github main page..


Send a pull request to xtensor to support pkg-config

https://github.com/QuantStack/xtensor/pull/338


Documenting Mergify and other automations being done in the repository

Motivation

Automating helps the team with faster delivery, but it also makes onboarding harder if there's no documentation available

Designs

Improve actual documentation in the pipelines.
Explain queues and some options from Mergify.

Hey team! Please add your planning poker estimate with ZenHub @conradoplg @dconnolly @gustavovalverde @teor2345

Closed via https://github.com/ZcashFoundation/zebra/pull/4540


Add relayer module

Partially resolves https://github.com/informalsystems/cosmos.nix/issues/26

Notes for Adi's review:

To trigger hermes test run nix build .#hermes-module-test -L (-L adds additional logging output so that you can what is happening on your terminal screen)
First unexpected error message:

vm-test-run-hermes-module-test> relayer # [    7.178198] hermes[799]: Jan 12 02:32:12.996  WARN ThreadId(18)     Reason: health check failed for endpoint tendermint::ServiceClient on the gRPC interface of chain nixos:http://validator1:26557/; caused by: status: Unimplemented, message: "unknown service cosmos.base.tendermint.v1beta1.Service", details: [], metadata: MetadataMap { headers: {"content-type": "application/grpc"} }
vm-test-run-hermes-module-test> relayer # [    7.181653] hermes[799]: Jan 12 02:32:13.000  WARN ThreadId(18)     Some Hermes features may not work in this mode!
vm-test-run-hermes-module-test> relayer # [    7.185354] hermes[799]: Jan 12 02:32:13.003 ERROR ThreadId(01) [nixos] chain is unhealthy:
vm-test-run-hermes-module-test> relayer # [    7.187170] hermes[799]:    0: health check failed for endpoint tendermint::ServiceClient on the gRPC interface of chain nixos:http://validator1:26557/; caused by: status: Unimplemented, message: "unknown service cosmos.base.tendermint.v1beta1.Service", details: [], metadata: MetadataMap { headers: {"content-type": "application/grpc"} }

grep for hermes toml can be found here: to find path to generated hermes.toml in nix store. Current one is here /nix/store/0mnq7y5fg7n4zb14gxhsx0baqg1lfkfx-config.toml

The tests work on my machine locally.
I wonder if this kind of test can also be written to run the ibc-integration-test suite we have at ibc-rs. There we need more flexibility of spawning the gaiad services dynamically during the test, and the tests are currently handled by cargo test.
I somehow feel that it might be simpler to perform the tests externally from Nix. For example, the integration tests for ibc-rs still use Cosmos.nix on the CI. We could even write a CI that call dedicated NixOps scripts to test the deployment. I'm just not sure if the tests have to be bundled together with the distribution of Nix packages in Cosmos.nix.

Yeah, this makes sense to me. Putting these tests together was pretty much a research spike anyways. I think it would be fairly straightforward to map the tests over a list of gaia versions, and could have a different test setup for N instances. But obviously it should be whatever is most straightforward and ergonomic for the IBC-RS team. That said these tests are useful for testing the NixOS modules for the relayer and gaia node modules.


[Bug]: Unable to launch the software from mainland China (Doing this for a friend)

Debug Tips

[X] I'm sure I've read this project's Issues of README.

What happened?
The software loads to half point and then closes itself.
APP Version
latest as for 3 december 2024
Runtime Environment
Windows 10
Country/Region
China
Relevant log output
No log was present. The folder is empty.

Screenshots
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Dear @V1Htor
Due to the special nature of the network in China, it is difficult to verify here, so we need your help.
As far as the operation is concerned, it seems that the initialization process failed to start.
If it failed, there should be an error.log output in the folder where VRCT is stored.
Could you please share that file with us?

error.log
It tried to connect to github and couldn't do it (makes sense why it is like that). It is mandatory to have connection to github? Since VRCT uses local models, why does it need that? Either way, here you are.

Access to github is necessary to check the latest software version and to determine whether an update is required.
Therefore, it is not expected that proxies and other influences will prevent access to github.
If you want to fix this, you could do the following.
model.py
@staticmethod
def checkSoftwareUpdated():
	# check update
	update_flag = False
	try:
		response = requests_get(config.GITHUB_URL)
		json_data = response.json()

		version = json_data.get("name", None)
		if isinstance(version, str):
		new_version = parse(version)
		current_version = parse(config.VERSION)
		if new_version > current_version:
		update_flag = True
		print("software version", "now:", config.VERSION, "new:", version)
	except:
		pass
return update_flag

Access to github is necessary to check the latest software version and to determine whether an update is required. Therefore, it is not expected that proxies and other influences will prevent access to github.

Yes, but could you write a check that if connection to github fails, it launches the program without checking for updates?

You can have the code above implemented.
Unfortunately, we do not plan to make minor updates for this special environment in version 2.x.x.
We will implement it in version 3.0.0 if possible, but for now we ask each individual to implement it.

Daer @V1Htor
The above features were added in version 3.0.0.
Please check it.

I can't launch it... it's forever stuck in this

I think it's fixed now. try re-download the newest setup.exe.
Since the file may be downloaded twice, please uninstall the current version first, then download and install the latest version from Booth or Github. We apologize for the inconvenience.
*Note: You can still use the current version with administrator privileges if needed.


callums typing fix

Description of my changes to follow. All tests passed.
ActivationCache
Based on type signatures, the functions remove_batch_dim and to should return self.
Added some assert isinstance(layer, int) to signal to the typechecker that layer is an int, not None.
The args from get_neuron_results have type Union[Slice, SliceInput]. Even inside the loop if type(arg) is not Slice, the typechecker doesn't assume that the arg is a SliceInput, which gives some errors.
This is fixed by adding assert isinstance(neuron_slice, SliceInput) (which in turn requires that we define `SliceInput: Type = ... in utils (where Type is imported from typing)).
HookedTransformer
Python knows the output of forward is None or Tensor or Loss, but if you e.g. get logits from a forward pass then try to index into it, this raises an error because Python thinks logits might be None.
Solution - use typing.overload, and typing_extensions.Literal
Example of how this works - you put this code before the forward function:
@overload
def forward(self, input, return_type: Literal["logits"], **kwargs) -> TT["batch", "pos", "d_vocab"]:
    ...
@overload
def forward(self, input, return_type: Literal["loss"], **kwargs) -> Loss:
    ...

Another cool feature of doing this - the typechecker can see if return_type equals a string that ISN'T "logits" or "loss", and it throws a red line.
I did a similar thing with run_with_cache (which can either return a Dict or ActivationCache).
Lastly, I needed to change some instances of self.cfg.d_vocab to self.cfg._d_vocab, both here and in components.py.
HookedTransformerConfig
I implemented the -1 thing for configs. It was a little messy (required using properties, and replacing some instances of d_vocab, d_vocab_out with _d_vocab, _d_vocab_out) but seems to work fine.

Thanks, this looks great! Conditional on it passing all the tests... (Python 3.7 is a headache with types, I appreciate you using typing_extensions for Literal!)
The d_vocab stuff is mildly horrifying, but I can't see a better solution... @alan-cooney any takes?

Yeah I think that is pretty horrible, would be reasonable if you wanted to not include that and I could either add the model.d_vocab thing or just add appropriate # type: ignores for the MLAB exercises.

I think I prefer just having the = None default be -1? And cutting out all of the private variable nonsense.

As far as I can tell, the type checker won't be able to catch errors to do with d_vocab accidentally not being set either way?

Hmm, I've pushed some new commits to the main branch of your repo but it's not auto-updating the PR. Weird!

Agreed with your thoughts Neel, and nice work on the other overloaded type definitions @callummcdougall !
Have you run mypy on TransformerLens to check if this causes any issues? We have a broader issue open to fix typing everywhere ( https://github.com/neelnanda-io/TransformerLens/issues/130 ), but at least for now it's worth making sure we haven't broken anything else.

I did run all four tests in the tests directory, and it seemed to pass.
Agree the d_vocab thing is super bug prone, apologies for that!

I don't see any new mypy issues


Android: Crash with "markingMode: none" and "background-color" change (Angular)

From @jpierront on March 4, 2019 13:47
Environment

CLI: 5.2.0
Cross-platform modules: 5.2.2
Android Runtime: 5.2.1

Describe the bug

With "markingMode: none", errors "Attempt to use cleared object reference" occur when I change the background color of an element in my app with a css class name. ([class.element-active]="...")
It doesn't seems to happen when I change the color or the opacity, only the background-color.
It doesn't seems to happen when I use "[style.background-color]=..."
My app freeze too often without the "markingMode: none"

I don't know if it's related to NativeScript or NativeScript-Angular
To Reproduce

I did not succeed in reproducing it on a blank app, but the garbage collector is probably less often triggered with an empty app.

Do you have any tips to help test this kind of case?
Expected behavior
No crash ;-)
Sample project
export class MyComponent {
    public activeElement: Element | null = null;
    public elements: Element[] = [...];
}

.element {
    background-color: white;
}
.element-active {
    background-color: red;
}

Additional context
ERROR Error: com.tns.NativeScriptException: Attempt to use cleared object reference id=4370
    com.tns.Runtime.getJavaObjectByID(Runtime.java:1006)
    com.tns.Runtime.callJSMethodNative(Native Method)
    com.tns.Runtime.dispatchCallJSMethodNative(Runtime.java:1203)
    com.tns.Runtime.callJSMethodImpl(Runtime.java:1083)
    com.tns.Runtime.callJSMethod(Runtime.java:1070)
    com.tns.Runtime.callJSMethod(Runtime.java:1050)
    com.tns.Runtime.callJSMethod(Runtime.java:1042)
    com.tns.gen.java.lang.Object_vendor_177265_32_TouchListenerImpl.onTouch(Object_vendor_177265_32_TouchListenerImpl.java:18)
    android.view.View.dispatchTouchEvent(View.java:11772)
    android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2968)
    android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2657)
    android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2968)
    android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2657)
    android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.j...

ERROR CONTEXT {
  "view": {
    "def": {
      "nodeFlags": 9125889,
      "rootNodeFlags": 1,
      "nodeMatchedQueries": 0,
      "flags": 0,
      "nodes": [
        {
          "nodeIndex": 0,
          "parent": null,
          "renderParent": null,
          "bindingIndex": 0,
          "outputIndex": 0,
          "checkIndex": 0,
          "flags": 1,
          "childFlags": 9125888,
          "directChildFlags": 9125888,
          "childMatchedQueries": 0,
          "matchedQueries": {},
          "matchedQueryIds": 0,
          "references": {},
          "ngContentIndex": null,
          "childCount": 3,
          "bindings": [
            {
              "flags": 8,
              "ns": "",
              "name": "text",
              "nonMinifiedName": "text",
              "securityContext": 0
            },
            {
              "flags": 2,
              "ns": "",
              "name": "element-active",
              "nonMinifiedName": "element-active"
            }
          ],
          "b...

"element-active" is the added css class name containing a change of background-color.
Copied from original issue: NativeScript/NativeScript#6983

@jpierront I've used your snippets in this test app (with markingMode set to none) but I was also not being able to reproduce the issue with the blank app.
Please try to isolate the issue in a sample app and if you are still hitting this one, share the source code for further investigation.

From @jpierront on March 4, 2019 16:5
I succeeded to reproduce it.
MarkingModeNoneBackground.zip
The second part (getContentParts) is only to generate heavy content.
(<any>java).lang.System.gc(); is to try to force GC (I don't know if it's useful or not, the crash occurs even without it)
You can have to tap on links (0-9) a lot of time before crash. (sometimes 3, sometimes 50 ...)

From @jpierront on March 5, 2019 15:10
Android Virtual Device
3.2'' QVGA (ADP2) 320x480
Oreo Android 8.1 x86 (API 27)
Emulated Performance: Automatic
512 RAM
32 VM heap
800 Internal Storage

@jpierront using your demo I am still not able to reproduce the issue  - I am testing on Nexus 5, Nexus 5x, Pixel and Pixel XL. Are you experiencing this one on a specific device, API level?

From @jpierront on March 5, 2019 16:5
I use tns run android

From @jpierront on March 5, 2019 15:30
I "succeed" on Android 9 too with a similar config (API 28)

It crash too

From @jpierront on March 5, 2019 16:42

I succeeded to trigger the error on Android 8.1 and 9. (Demo App + My own App)
I succeeded to trigger the error on Android 7.1 and 6.0. (But only on my own App for now and it took a while before triggering the error)
I haven't succeeded yet to trigger the error on Android 4.4 and 5.1.
I succeeded to trigger the error on Nexus 5 (Emulator, Android 8.1, 1536 Ram, 128 VM heap). (Demo App + My own App) (But I have to click 15 or 20 times on links with changing background)

@jpierront thank you for the dedicated time to isolate the case! I\ve embedded the above snippet in this test app and can confirm that indeed there is a markingMode error manifestation on most Android devices (used Pixel and Nexus real devices for tests). Our Android developers will investigate the case further and we will post any related information as soon as we have more details to share.

From @jpierront on March 5, 2019 22:12
I have a new sample app that triggers the bug faster. (By changing a lot of background color every 100ms)
On Android 8.1, it take only 10 seconds on my computer. (Virtual Device: Nexus 5 or custom device)
On Android 9, it take less than 60 seconds on my computer.
On Android 4.4/5.1/6.0/7.1, no bug after 10 minutes of running (I can reproduce the bug only on my own app)
The delay will probably depend on the speed of your computer.
import {Component, OnInit} from '@angular/core';

@Component({
    selector: 'ns-app',
    moduleId: module.id,
    templateUrl: './app.component.html',
    styles: [`
        .mat-list-item {
            background-color: #ffffff;
        }

        .mat-list-item-active {
            background-color: #ffeeee;
        }
    `],
})
export class AppComponent implements OnInit {
    public static readonly MAX = 21;
    public elementsGrid: boolean[][] = [];
    public rows: string = '*,'.repeat(AppComponent.MAX);

    protected indexes = Array.from(Array(AppComponent.MAX).keys());

    public constructor() {
        let elementValue = true;
        for (let i in this.indexes) {
            const elementRow: boolean[] = [];
            for (let j in this.indexes) {
                elementRow.push(elementValue);
                elementValue = !elementValue;
            }
            this.elementsGrid.push(elementRow);
        }
    }

    public ngOnInit(): void {
        setInterval((): void => {
            this.update();
        }, 100);
    }

    protected update(): void {
        for (let i in this.indexes) {
            for (let j in this.indexes) {
                this.elementsGrid[i][j] = !this.elementsGrid[i][j];
            }
        }
    }
}

<StackLayout>
    <GridLayout [rows]="rows" [columns]="rows">
        <ng-container *ngFor="let elementsRow of elementsGrid; index as i">
            <ng-container *ngFor="let element of elementsRow; index as j">

            </ng-container>
        </ng-container>
    </GridLayout>
</StackLayout>

@jpierront after some additional investigation I've managed to simplify the app and it seems that the issue is not related to CSS and setting background-color but to the Angular structural directive ngFor.
[Here is my updated application](https://github.com/NickIliev/NS-Issues-2019-I/tree/master/test/marking-mode-bgcolor) which is hitting the very same issue without using any CSS but simply the ngFor directives. The same scenario is not reproducible if we substitute the ngFor directives with Repeater
Moving the issue (and updating the title) to the NativeScript Angular repository as it seems related to specific case where we are using binding with ngFor and markingMode

@NickIliev For me, it crash both with *ngFor and with Repeater.
You will find below a simplified version of the app.
import {Component, OnInit, ViewChild} from '@angular/core';

@Component({
    selector: 'ns-app',
    moduleId: module.id,
    templateUrl: './app.component.html',
})
export class AppComponent implements OnInit {
    @ViewChild('elementRepeater') public elementRepeater!: any; // Needed for Repeater version

    public elements: boolean[] = [];
    protected indexes = Array.from(Array(1000).keys());

    public constructor() {
        let elementValue = true;
        for (let i in this.indexes) {
            this.elements.push(elementValue);
            elementValue = !elementValue;
        }
    }

    public ngOnInit(): void {
        setInterval((): void => {
            this.update();

            if (this.elementRepeater) { // Needed for Repeater version
                this.elementRepeater.nativeElement.refresh();
            }
        }, 100);
    }

    protected update(): void {
        for (let i in this.indexes) {
            this.elements[i] = !this.elements[i];
        }
    }
}

<StackLayout>

</StackLayout>

Do you have any ideas on this problem? How can I help ?

I also have similar bug. I'm trying to apply class for the GridLayout based on property value.
<GridLayout  [class.selected]="isSelected"
Unhandled Promise rejection: com.tns.NativeScriptException: Attempt to use cleared object reference id=8849

zone-nativescript.js:676
com.tns.Runtime.getJavaObjectByID(Runtime.java:1006)
com.tns.Runtime.callJSMethodNative(Native Method)
com.tns.Runtime.dispatchCallJSMethodNative(Runtime.java:1203)
com.tns.Runtime.callJSMethodImpl(Runtime.java:1083)
com.tns.Runtime.callJSMethod(Runtime.java:1070)
com.tns.Runtime.callJSMethod(Runtime.java:1050)
com.tns.Runtime.callJSMethod(Runtime.java:1042)
com.tns.gen.java.lang.Runnable.run(Runnable.java:15)
android.os.Handler.handleCallback(Handler.java:790)
android.os.Handler.dispatchMessage(Handler.java:99)
android.os.Looper.loop(Looper.java:164)
android.app.ActivityThread.main(ActivityThread.java:6494)
java.lang.reflect.Method.invoke(Native Method)
com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:438)

@jpierront we are working on this but still haven't pinpointed the root of the problem

Thx :)

Thank you.

Do you have any news about this issue ?

My issue has gone after updating to NS 5.3.

Not for me :/

@jpierront sorry for the late reply -- we were finally able to locate the root of the problem and https://github.com/NativeScript/android-runtime/pull/1345 should address it.
You can test the fix by first removing the platforms/android folder from your app and installing the "next" version of the android runtime like this:
cd [your app folder]
rm -rf platforms/android
tns platform add android@next
tns run android

Apparently it works! Awesome!!!
My app was not viable without this fix. (Either I had a lot of freezes with markingMode or a lot of crash without markingMode)


Can I use fairseq without GPU?

I don't have access to GPU. However, I have access to powerful multicore machines with CPU.
Can I use fairseq with CPU? I need to train existing models with a new dataset?

You can specify the --cpu flag

@lematt1991 I think this should be reflected in the Readme.
Readme mentions GPU is a mandatory prereq.


update cppconf dependency

Consolidate publish proxy stack selection.
Add support for home region variable.

Source repo is out of sync, redoing this.


Form submission with Html Agility Pack

Hi,
I have been using HTML agility pack to write a custom web scrapper.
Few websites allow data scrapping and few requires user to login by using their account credentials to get somewhere in the website.
I have been trying to submit form through agility but I am not anywhere near.
I have to confirm if we can submit a form with the help of HAP or not?
If we can then please put some demo or direct me towards an example.
That would be a huge help.
Best Regards,
Sohaib

Hello @sohaibameen ,
I know that's possible to do such thing but I have never done it on my side and I don't have any example.
I recommend you to try https://stackoverflow.com/questions/tagged/html-agility-pack and hope someone that knows how to do it answer.
Best Regards,
Jonathan


python37Packages.azure-mgmt-recoveryservices: 0.5.0 -> 0.6.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python37Packages.azure-mgmt-recoveryservices is: "This is the Microsoft Azure Recovery Services Client Library"
meta.homepage for python37Packages.azure-mgmt-recoveryservices is: "https://github.com/Azure/azure-sdk-for-python"
meta.changelog for python37Packages.azure-mgmt-recoveryservices is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.6.0 with grep in /nix/store/i629wm3qn6n4wp3hgwpm5c0acmkvahd4-python3.7-azure-mgmt-recoveryservices-0.6.0

directory tree listing: https://gist.github.com/924800fe6a6f3592f3aa18b280d30c9a

du listing: https://gist.github.com/ad8faffa95460371dc062a8617660d13

Rebuild report (if merged into master) (click to expand)

13 total rebuild path(s)

4 package rebuild(s)

3 x86_64-linux rebuild(s)
3 i686-linux rebuild(s)
4 x86_64-darwin rebuild(s)
3 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
azure-cli
python37Packages.azure-mgmt-recoveryservices
python38Packages.azure-mgmt-recoveryservices
python39Packages.azure-mgmt-recoveryservices

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/i629wm3qn6n4wp3hgwpm5c0acmkvahd4-python3.7-azure-mgmt-recoveryservices-0.6.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python37Packages.azure-mgmt-recoveryservices https://github.com/r-ryantm/nixpkgs/archive/279834d8b5e981a81a529b620e232752c2802f46.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/i629wm3qn6n4wp3hgwpm5c0acmkvahd4-python3.7-azure-mgmt-recoveryservices-0.6.0
ls -la /nix/store/i629wm3qn6n4wp3hgwpm5c0acmkvahd4-python3.7-azure-mgmt-recoveryservices-0.6.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package failed to build:

azure-cli

2 packages built:

python37Packages.azure-mgmt-recoveryservices
python38Packages.azure-mgmt-recoveryservices

Maintainer pings
cc @mwilsoncoding for testing.

addressed by https://github.com/NixOS/nixpkgs/pull/107996

addressed by https://github.com/NixOS/nixpkgs/pull/107996


Landing page arrow overlaps content on smaller screens.

The vertical movement of the arrow at the bottom of the landing page overlaps the content.
Example:

currently non Clickable

make that arrow interactive
which will redirect to

the section next to it
ie. About

with transition


Policy Assignment Deploy-Private-DNS-Zones of archetype es_corp is missing permissions on Connectivity Management Group

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Versions
terraform: v1.0.11
azure provider: 2.86.0
module: v1.1.0
Description
We have defined a custom landing zone with archetype_id es_corp below the "${var.root_id}-landing-zones" ManagementGroup.
The es_corp archetype assigns the Deploy-Private-DNS-Zones policy set on it's scope.
This policy set contains among others the policy "Configure Container registries to use private DNS zones" (with id /providers/Microsoft.Authorization/policyDefinitions/e9585a95-5b8c-4d03-b193-dc7eb5ac4c32).
That's a DINE policy linking private endpoints in landingzones via privateDnsZoneGroup to the Privatelink Private DNS Zone deployed in the Connectivity Subscription.
Now when deploying a Private Endpoint for an ACR the DINE deployment fails with the following error message:
{"code":"DeploymentFailed","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details.","details":[{"code":"LinkedAuthorizationFailed","message":"The client '<some id>' with object id 'XXXXX' has permission to perform action 'Microsoft.Network/privateEndpoints/privateDnsZoneGroups/write' on scope '/subscriptions/<subscription id below landingzone management group>/resourcegroups/<some rg>/providers/Microsoft.Network/privateEndpoints/private-endpoint-<some name>/privateDnsZoneGroups/deployedByPolicy'; however, it does not have permission to perform action 'Microsoft.Network/privateDnsZones/join/action' on the linked scope(s) '/subscriptions/<connectivity subscripition>/resourceGroups/<root_id>-dns/providers/Microsoft.Network/privateDnsZones/privatelink.database.windows.net' or the linked scope(s) are invalid."}]}
(error message edited to redact IDs)
The XXXXX object_id matches to the SystemAssigned identity of the poliy set assignment in Azure AD (Enterprise application "Deploy-Private-DNS-Zones")
The policy mentioned above defines

    "roleDefinitionIds": [
            "/providers/Microsoft.Authorization/roleDefinitions/4d97b98b-1d4f-4787-a291-c67834d212e7"
          ]

(= Network Contributor)
The Network Contributor role is assigned at the management group level at the scope the policy set is assigned.
(with two more roles comming from the policy set assignment:

)
But the SystemAssigned Identity does NOT have any permissions at the Connectivity subscription scope.
Therefore the error.
When we've deployed custom policies to DINE privateDnsZoneGroups for private link subresources not covered by the builtin policies we encountered the exact same error.
We added an role assignment at the Connectivity subscription scope (and the landingzone MG scope of course) and the DINE deployments worked afterwards.
Probably the CAF module should add such an role assignment at the Connectivity Management Group scope?
Note: We don't have deployed the "Connectivity Resources" via this module. We have deployed the required Privatelink Private DNS Zones in the required resource group with our own terraform code. But I was not able to find the missing role assignment in this module's code to be created when the "Connectivity Resources" would be used.

@nikolai-fra this is a great ask, and very timely too as we were discussing this just last week.
To create the Role Assignments, we have to determine the Role Definition(s) required by the Policy Definition(s) being assigned. This is one of the steps already performed by the module.
We then need to identify the scope(s) at which the Role Assignment(s) need to be created. Currently we are only able to use the scope of the corresponding Policy Assignment, but we are looking to make use of the relatively new metadata values for parameters within a Policy Definition which allow us to identify parameters which contain a resource ID for something which the Role Assignment must cover.
You can see a little more about this by reviewing the metadata.assignPermissions attribute under Parameter properties for a Policy Definition.
This may take a while to reach the top of our backlog, but we are hoping to use this feature to drive support additional scopes when creating these Role Assignments, and you can see this in the following refactor of how we create Role Assignments to fix #266 (pending PR):
https://github.com/krowlandson/terraform-azurerm-caf-enterprise-scale/blob/23020c03c2e5d45a7e6675da53769046c5b674f4/resources.role_assignments.tf#L24-L50

Trigger ADO Sync

Going to close as tracking on #439

Trigger ADO Sync


Intro to git slides

Here are the Git Introduction slides. I added an additional slide about practicing git and encouraging new people to try with the chemistry repo. I feel if mistakes are made there it is not a big deal at all and can be used as a practice ground. Later when we talk about squashing I think it would be good to practice with that repo.

Thanks!


Write new test for nodejs api

Is your feature request related to a problem? Please describe.
Tests are not complete
Describe the solution you'd like
Write test for api's
Additional context
Test a Node RESTful API with Mocha and Chai - Tutorial
Testing A Node/Express Application With Mocha & Chai - Medium Article
How to correctly unit test Express server - Blog

@ArpitKotecha @vinitshahdeo @PragatiVerma18  I'd like to work on this issue. Please assign this to me.

@akt114 you're good to go. Try to finish the task as early as possible.

@akt114 what is the status for this issue. Please be active or else I may have to unassign you from this issue.

@PragatiVerma18 Can I work on this, if noone is working

@PragatiVerma18 Can I work on this, if noone is working

@darkmatter18  assigned

@ArpitKotecha Do I work on the branch feature/backend-node


Make service account credentials values settable indvidually

Right now, we rely on the googleauth library to fetch the appropriate oauth access token. That library relies on access to a json credentials file that contains service account information from FCM.
In order to make setting up an environment easier and more reproducible, it would be nice to be able to set the values in the json service account file individually, via env vars or some other mechanism.
It may be possible to do this with googleauth, but I think it may also be easier to use Google's authentication HTTP api directly, rather than their gem.
I imagine the api for this gem would like something like this:
service_account = FirebaseCloudMessenger::ServiceAccount.new
service_account.project_id = ENV['FCM_PROJECT_ID']
service_account.private_key = ENV['FCM_PRIVATE_KEY']
...
FirebaseCloudMessenger.service_account = service_account
FirebaseCloudMessenger.send(message: <message>)

This would involve a rewrite of AuthClient. I'd like also to do this in a way that is backward-compatible--If a credentials_path is supplied instead of service_account, the service account could be built by Client instead.
This would also allow us to drop our dependency on googleauth, which might make our maintenance a bit more difficult, but dropping a dependency is always nice.
cc. @bryanalves

After investigating the googleauth lib more, it allows setting the following env vars instead of supplying a path to the service account json:
GOOGLE_PRIVATE_KEY
GOOGLE_CLIENT_EMAIL
Instead of writing out googleauth, for right now, I'm going to change our code to be able to accept these vars, and put my proposed ServiceAccount api on hold.


Generate SSL Script Broken for Mac OSX

The mktemp command used in make-ssl.sh does not seem to work on a Mac deployment host. Seems that the Mac flags do not support the --tmpdir flag being passed. I believe this is a difference between the BSD and GNU versions of that tool. The offending line is here.
I was able to work around this by removing the --tmpdir portion, but that places the created directory in the current working directory and needs to be cleaned up after completion. That part isn't a huge deal, just wanted to make sure it was pointed out.
I'm happy to help fix this, but I'd like to hear a bit about what you folks view as an acceptable fix first!

Thanks @rsmitty, i don't know how if there's a command which would work on both OS (linux and mac). Please let me know if you have any idea

I think you can us a full path and drop the --tmpdir.  /tmp/asda.XXXX. It
will work for both platforms.
On Thu, Feb 18, 2016 at 11:18 AM, Smaine Kahlouch<EMAIL_ADDRESS>wrote:

Thanks @rsmitty https://github.com/rsmitty, i don't know how if there's
a command which would work on both OS (linux and mac). Please let me know
if you have any idea
—
Reply to this email directly or view it on GitHub
https://github.com/kubespray/setup-kubernetes/issues/151#issuecomment-185821761
.

I'll take a look at prototyping that out to see if it works as expected, will submit a pull if it does.


Nation Tab - Add Nation Modal

As a user
I want to be able to add a Nation
so that I can add them to a work
Acceptance Criteria
Scenario 1:
GIVEN The Nation Tab is open
WHEN I click on 'Add Nation' button
THEN 'Add Nation' Modal appears with the following fields

Nation with dropdown (mandatory)
Relationship holder with ghost text 'Relationship holder' (disabled)
PIP link with ghost text 'Link' (disabled)
PIN with dropdown ('Yes'/'No') and ghost text 'Select / Confirm PIN Status'
Toggle (active by default)

Screenshots:
Private Zenhub Image
Private Zenhub Image
Private Zenhub Image

Pin Status may be changed to "Pending", "Confirmed", etc.
not confirmed yet.

@salabh-aot
link to prototype
https://www.figma.com/file/j5onJ5RZGAxahxhYp68N8m/For-Development?type=design&node-id=3890-83994&mode=design

@maria-kuzmenko @dinesh-aot The Work -> IndigenousNation mapping table have a foreign key relation with IndigenousCategory, but the field for it not present in the design.


Save/Load Setups does not work at all

I have tried multiple times on multiple occasions to save and load various fashion set ups but the feature simply doesn't work.

What do you mean exactly? Do you have steps to reproduce? And is this happening for every slot, or only certain ones? I've used the feature myself on a few different accounts and devices and haven't seen anything like this

I reproduce it every single time I try to save/load a set up. I have tried saving full setups, partially full setups of all kinds. It will save the setup, but when i click load it loads a totally empty set up. This happens with every single setup I have ever tried to save.


question: pagination and search

Hi, @janczizikow
Thanks for your nice job, sleek is such a pretty theme that I love it so much 👍.
I wondered if sleek will support pagination and search in the future?
Append:

Is there any way to change the page's title?

And is there any way to add more social icons?

Hey @TonnyL,
Glad you liked the theme! To answer your first questions:

The title of the theme is controlled via jekyll seo plugin - you change the title of the index page, but modifying the title property in front matter of index.md:

---
#
# You don't need to edit this file, it's empty on purpose.
# Edit sleeks's default layout instead if you wanna make some changes
# See: https://jekyllrb.com/docs/themes/#overriding-theme-defaults
#
layout: default
title: Sleek Jekyll Theme # Change this to your desired title
---

Or you can remove the property and just control the title in _config.yml file by again changing the title property there.

Yes you can add more social icons. If you're using theme as a ruby gem it might be more complicated, but if you forked the repo, you can just modify the _includes/footer.html file. Just add a new link under <nav class="social". The icons are simply SVG code, so you can copy paste the SVG code of the icons you wish to add.

Regarding search and pagination:
Since this theme was made in mind for using with github pages, I don't plan to add pagination functionality. There're some jekyll plugins for adding that functionality, but github pages don't support those plugins. Search could be a nice feature to have, however, for that I would recommend to use algolia. But again to use algolia, you need an API key and usually it's good practice not to share this kind of key on a public repo. Perhaps I could add the functionality and allow users to simply add their key somewhere in the settings (_config.yml) or in a file. This way I wouldn't have to share my API key and people using the theme could still enjoy the search functionality. Let me know your thoughts or if you have any additional questions.
Cheers!

OK, that's the answer that I want. Thanks 😉!


269 improve test coverage

Trello card:
Partial timeboxed - https://trello.com/c/PCbcd3Ee/269-test-coverage-strategy
Description

Improve tests for all pages
Remove expose of unused init

@neontribe/sea-map

@Nikomus I had to fix a merge conflict .Please take another look.


Auto-update Readme with the latest version within the release flow

If you're using sbt-softwaremill and its release process, this is usually handled by updateDocs (e.g. updateDocs)