e2e-test: make e2e-test codes can be reused by other package

make some functions be public

fixes https://github.com/confidential-containers/cloud-api-adaptor/issues/1641

run e2e-test with the changed codes, test passed on providers:

libvirt
ibmcloud

provider aws and azure doesn't be tested as no related env.

I'm seeing the overlay fail now:
Wait for the cc-operator-controller-manager deployment be available
INFO[0015] Customize the overlay yaml file
INFO[0018] Install the cloud-api-adaptor
Wait for the cc-operator-daemon-install DaemonSet be available
FATA[0320] timed out waiting for the condition

I'll have a play with it manually, but thought it was mentioning as it's an area that has had some refactoring

I'm seeing the overlay fail now:
Wait for the cc-operator-controller-manager deployment be available
INFO[0015] Customize the overlay yaml file
INFO[0018] Install the cloud-api-adaptor
Wait for the cc-operator-daemon-install DaemonSet be available
FATA[0320] timed out waiting for the condition

I'll have a play with it manually, but thought it was mentioning as it's an area that has had some refactoring

Ok, there is something else going on here that I don't fully understand. It looks like we don't get a caa pod as the ds has desired number of 0?:
# kubectl get ds -A
NAMESPACE                        NAME                           DESIRED   CURRENT   READY   UP-TO-DATE   AVAILABLE   NODE SELECTOR                            AGE
confidential-containers-system   cc-operator-daemon-uninstall   0         0         0       0            0           katacontainers.io/kata-runtime=cleanup   93m
confidential-containers-system   cloud-api-adaptor-daemonset    0         0         0       0            0           node.kubernetes.io/worker=               93m
kube-flannel                     kube-flannel-ds                2         2         2       2            2           <none>                                   7h56m
kube-system                      kube-proxy                     2         2         2       2            2           kubernetes.io/os=linux                   7h56m

Ok, there is something else going on here that I don't fully understand. It looks like we don't get a caa pod as the ds has desired number of 0?:
# kubectl get ds -A
NAMESPACE                        NAME                           DESIRED   CURRENT   READY   UP-TO-DATE   AVAILABLE   NODE SELECTOR                            AGE
confidential-containers-system   cc-operator-daemon-uninstall   0         0         0       0            0           katacontainers.io/kata-runtime=cleanup   93m
confidential-containers-system   cloud-api-adaptor-daemonset    0         0         0       0            0           node.kubernetes.io/worker=               93m
kube-flannel                     kube-flannel-ds                2         2         2       2            2           <none>                                   7h56m
kube-system                      kube-proxy                     2         2         2       2            2           kubernetes.io/os=linux                   7h56m

Maybe the latest operator isn't compatible with the 0.8 peer pods config?

I'm getting the same result on the main branch, so I guess it's either a problem with my environment, or something widerspread. I'll try it on libvirt tomorrow to see if that helps

Ok, there is something else going on here that I don't fully understand. It looks like we don't get a caa pod as the ds has desired number of 0?:
# kubectl get ds -A
NAMESPACE                        NAME                           DESIRED   CURRENT   READY   UP-TO-DATE   AVAILABLE   NODE SELECTOR                            AGE
confidential-containers-system   cc-operator-daemon-uninstall   0         0         0       0            0           katacontainers.io/kata-runtime=cleanup   93m
confidential-containers-system   cloud-api-adaptor-daemonset    0         0         0       0            0           node.kubernetes.io/worker=               93m
kube-flannel                     kube-flannel-ds                2         2         2       2            2           <none>                                   7h56m
kube-system                      kube-proxy                     2         2         2       2            2           kubernetes.io/os=linux                   7h56m

Maybe the latest operator isn't compatible with the 0.8 peer pods config?

I'm getting the same result on the main branch, so I guess it's either a problem with my environment, or something widerspread. I'll try it on libvirt tomorrow to see if that helps

@stevenhorsman thanks for try this big pr in your local. Here is passed internal job for this PR with ibmcloud provider, https://sys-zaas-k8s-jenkins.swg-devops.com/job/cloud-api-adaptor-e2e-tests-opensource/1121/console you can do a reference.

e2e provisioning for AWS itself doesn't work. In the first scenario when using CIDR "<IP_ADDRESS>/16" it failed to create the private network because it uses the default CIDR and not the input CIDR as the basis
Second, when I used the <IP_ADDRESS>/24 as the CIDR it failed with the following error:
time="2024-01-08T13:47:32+05:30" level=info msg="CNI add-on installed. Waiting to be activated (timeout=5m0s)..."
F0108 13:48:33.729214 1350686 env.go:369] Setup failure: envconfig: client failed: Get "https://B6B78244D9897CA8283E46154094DF66.gr7.us-east-2.eks.amazonaws.com/api?timeout=32s": getting credentials: decoding stdout: couldn't get version/kind; json parse error: json: cannot unmarshal string into Go value of type struct { APIVersion string "json:\"apiVersion,omitempty\""; Kind string "json:\"kind,omitempty\"" }

None of these errors are due to this PR though. But unfortunately I can't test without figuring out what's wrong with the AWS e2e provisioning. This is going to take time. So considering this situation, please don't hold the PR.
cc @wainersm for awareness

e2e provisioning for AWS itself doesn't work. In the first scenario when using CIDR "<IP_ADDRESS>/16" it failed to create the private network because it uses the default CIDR and not the input CIDR as the basis Second, when I used the <IP_ADDRESS>/24 as the CIDR it failed with the following error:
time="2024-01-08T13:47:32+05:30" level=info msg="CNI add-on installed. Waiting to be activated (timeout=5m0s)..."
F0108 13:48:33.729214 1350686 env.go:369] Setup failure: envconfig: client failed: Get "https://B6B78244D9897CA8283E46154094DF66.gr7.us-east-2.eks.amazonaws.com/api?timeout=32s": getting credentials: decoding stdout: couldn't get version/kind; json parse error: json: cannot unmarshal string into Go value of type struct { APIVersion string "json:\"apiVersion,omitempty\""; Kind string "json:\"kind,omitempty\"" }

None of these errors are due to this PR though. But unfortunately I can't test without figuring out what's wrong with the AWS e2e provisioning. This is going to take time. So considering this situation, please don't hold the PR.
cc @wainersm for awareness

Thanks for the update @bpradipt so that I will merge this pr now.
Please feel free leave comments when you try the new e2e test cases after the AWS e2e provisioning issue is fixed.


"Undefined" cell preview freezes

First of all, this is awesome :heart_eyes: It's super useful for stuff like making logs legible, thank you!
I managed to break this on accident with poorly formatted data. To reproduce it, you can use this sample set:
Date, Name, Count
2015-09-29, test/wrong
2015-09-29, test/wrong
2015-09-29, test/ok, 40
2015-09-28, test/ok, 78
Note the missing third column in the first 2 rows of data.

Paste that in the CSV tab
Click the Form tab (looks a little wonky with two missing cells
Click the Preview tab (two missing cells now say undefined)
Try clicking CSV, Markdown, or Form.

Doesn't do anything when you click on those three tabs from preview. The only way to get out of it is to click HTML, which somehow fixes the problem ¯\_(ツ)_/¯

Thank you @cmrberry
I think I've fixed this by adding empty cells to make up for any extra long rows. It messes up the headers a bit but I think that's preferable to smooshing the extra cells into each other.
I've popped the change onto the live version, does that seem correct to you? :smiley_cat:

:sparkles:! Thanks!


Have recent bans, mutes, and warns show past bans, mutes, and warns

Would be nice instead of having an empty section on the page.

Huh, that should actually work already:
http://bans.yeahwh.at/index.php?action=viewplayer&player=Ghetto&server=0
http://bans.yeahwh.at/index.php?action=viewplayer&player=blockhead23&server=0

I mean in this area: http://i.imgur.com/lEdBzUr.png

Aaaah, I see :)
Good idea, I like this feature request.

Closing in favour of #168


Add comments in a recipe

Hello,
I think it would be a great feature to have the possibility to comment a recipe directly in the script.
This would allow several collaborators to have a better view of how works the recipe and facilitate collaborative working.
Thank you!

I support Herve proposition. Preparing data is like telling a story as there is never one way of doing something.
Quickly a data preparation process can go through hundreds of steps and understanding the logic afterwards can be complex. Like developers comment their code to with their logic, we should be able to do the same in Refine.
This will help with collaboration and reproducibility of the code along with improving auditing.

I have been using comments for  a while now, by adding column
transformations of the form
jython:return value # comment
(which do nothing) and then using an overloaded renderer of the
HistoryPanel class in a custom plugin:
// labels in history panel
HistoryPanel.prototype._renderOrig=HistoryPanel.prototype._render;
HistoryPanel.prototype._render= function() {
this._renderOrig();
var self=this;
var elmts = DOM.bind(this._div);
//console.log(elmts);
elmts.bodyDiv.find(".history-entry").each(function() {
var text = this.childNodes[1].firstChild.nodeValue;
var mtch=text.match(/Text transform on 0 cells in column [^:]+: jython:
*return +value #(.)/);
if (mtch) {
this.childNodes[1].firstChild.nodeValue=mtch[1];
$(this.childNodes[1]).addClass("history-comment");
}
});
}
together with some css for the history:
.history-past a.history-entry .history-comment {
color: #00f
}
.history-now a.history-entry .history-comment {
color: #ff4
}
.history-future a.history-entry .history-comment {
color: #88f
}
Cheers,
Herwig
Herwig Van Marck, PhD.
Senior Expert Research Informatics
Rijvisschestraat 126 3/R, 9052 Zwijnaarde
Tel: +32 (0)9 248.16.01
Bio Informatics Training and Service Facility (BITS
http://www.bits.vib.be/)
On 2 September 2015 at 15:04, magdmartin<EMAIL_ADDRESS>wrote:

I support Herve proposition. Preparing data is like telling a story as
there is never one way of doing something.
Quickly a data preparation process can go through hundreds of steps and
understanding the logic afterwards can be complex. Like developers comment
their code to with their logic, we should be able to do the same in Refine.
This will help with collaboration and reproducibility of the code along
with improving auditing.
—
Reply to this email directly or view it on GitHub
https://github.com/OpenRefine/OpenRefine/issues/1057#issuecomment-137067159
.

Herwig
Thank you for sharing this! Is it currently available in a standalone
extension ? Available on public repository? I'd love to see how we can
integrate this to OpenRefine.
Martin.
On Sep 2, 2015 9:23 AM, "Herwig Van Marck"<EMAIL_ADDRESS>wrote:

I have been using comments for a while now, by adding column
transformations of the form
jython:return value # comment
(which do nothing) and then using an overloaded renderer of the
HistoryPanel class in a custom plugin:
// labels in history panel
HistoryPanel.prototype._renderOrig=HistoryPanel.prototype._render;
HistoryPanel.prototype._render= function() {
this._renderOrig();
var self=this;
var elmts = DOM.bind(this._div);
//console.log(elmts);
elmts.bodyDiv.find(".history-entry").each(function() {
var text = this.childNodes[1].firstChild.nodeValue;
var mtch=text.match(/Text transform on 0 cells in column [^:]+: jython:
*return +value #(.)/);
if (mtch) {
this.childNodes[1].firstChild.nodeValue=mtch[1];
$(this.childNodes[1]).addClass("history-comment");
}
});
}
together with some css for the history:
.history-past a.history-entry .history-comment {
color: #00f
}
.history-now a.history-entry .history-comment {
color: #ff4
}
.history-future a.history-entry .history-comment {
color: #88f
}
Cheers,
Herwig
Herwig Van Marck, PhD.
Senior Expert Research Informatics
Rijvisschestraat 126 3/R, 9052 Zwijnaarde
Tel: +32 (0)9 248.16.01
Bio Informatics Training and Service Facility (BITS
http://www.bits.vib.be/)
On 2 September 2015 at 15:04, magdmartin<EMAIL_ADDRESS>wrote:

I support Herve proposition. Preparing data is like telling a story as
there is never one way of doing something.
Quickly a data preparation process can go through hundreds of steps and
understanding the logic afterwards can be complex. Like developers
comment
their code to with their logic, we should be able to do the same in
Refine.
This will help with collaboration and reproducibility of the code along
with improving auditing.
—
Reply to this email directly or view it on GitHub
<
https://github.com/OpenRefine/OpenRefine/issues/1057#issuecomment-137067159
.

—
Reply to this email directly or view it on GitHub
https://github.com/OpenRefine/OpenRefine/issues/1057#issuecomment-137076887
.

Hey Martin,
It is part of an internal plugin (not public) because I see this as a
temporary solution/hack. It would be good if comments could be implemented
properly in OpenRefine!
Cheers,
Herwig
Herwig Van Marck, PhD.
Senior Expert Research Informatics
Rijvisschestraat 126 3/R, 9052 Zwijnaarde
Tel: +32 (0)9 248.16.01
Bio Informatics Training and Service Facility (BITS
http://www.bits.vib.be/)
On 3 September 2015 at 04:40, magdmartin<EMAIL_ADDRESS>wrote:

Herwig
Thank you for sharing this! Is it currently available in a standalone
extension ? Available on public repository? I'd love to see how we can
integrate this to OpenRefine.
Martin.
On Sep 2, 2015 9:23 AM, "Herwig Van Marck"<EMAIL_ADDRESS>wrote:

I have been using comments for a while now, by adding column
transformations of the form
jython:return value # comment
(which do nothing) and then using an overloaded renderer of the
HistoryPanel class in a custom plugin:
// labels in history panel
HistoryPanel.prototype._renderOrig=HistoryPanel.prototype._render;
HistoryPanel.prototype._render= function() {
this._renderOrig();
var self=this;
var elmts = DOM.bind(this._div);
//console.log(elmts);
elmts.bodyDiv.find(".history-entry").each(function() {
var text = this.childNodes[1].firstChild.nodeValue;
var mtch=text.match(/Text transform on 0 cells in column [^:]+: jython:
*return +value #(.)/);
if (mtch) {
this.childNodes[1].firstChild.nodeValue=mtch[1];
$(this.childNodes[1]).addClass("history-comment");
}
});
}
together with some css for the history:
.history-past a.history-entry .history-comment {
color: #00f
}
.history-now a.history-entry .history-comment {
color: #ff4
}
.history-future a.history-entry .history-comment {
color: #88f
}
Cheers,
Herwig
Herwig Van Marck, PhD.
Senior Expert Research Informatics
Rijvisschestraat 126 3/R, 9052 Zwijnaarde
Tel: +32 (0)9 248.16.01
Bio Informatics Training and Service Facility (BITS
http://www.bits.vib.be/)
On 2 September 2015 at 15:04, magdmartin<EMAIL_ADDRESS>wrote:

I support Herve proposition. Preparing data is like telling a story as
there is never one way of doing something.
Quickly a data preparation process can go through hundreds of steps and
understanding the logic afterwards can be complex. Like developers
comment
their code to with their logic, we should be able to do the same in
Refine.
This will help with collaboration and reproducibility of the code along
with improving auditing.
—
Reply to this email directly or view it on GitHub
<

https://github.com/OpenRefine/OpenRefine/issues/1057#issuecomment-137067159

.

—
Reply to this email directly or view it on GitHub
<
https://github.com/OpenRefine/OpenRefine/issues/1057#issuecomment-137076887
.

—
Reply to this email directly or view it on GitHub
https://github.com/OpenRefine/OpenRefine/issues/1057#issuecomment-137305308
.

@hvmarck That's a interesting hack. what's the proper way to handle the comment according to your experience? I think it's a standalone plugin which you can apply the comment to any selected step chosen from the history panel.
Just curiosity if you don't mind. what's the other functionalities of your internal plugin?

The other stuff that is in the internal plugin:

project locking support (to run OpenRefine on a server accessed by
multiple people), making sure only one person can open a project at a time
project ownership (keeps track who owns a project, defaulting to
whoever created it) in conjunction with the previous locking support
adding an extra export button in the 'Open Project' page (between
'delete' and 'rename')

error handling to ProcessPanel.update to avoid process update error
(automatically restarting when an error occurs)
a visualizer that treats columns that start with the string 'diff ' as
a special column in which diff like annotations (created with a diff tool)
are visualized with red strikethrough and green highlights
a function that compares a project to another project using the Coopy
highlighter diff library http://dataprotocols.org/tabular-diff-format/
a function that compares the history of a project to the history of
another project

a 'GoTo-record' to pagingControls, that allows you to click on the
page control numbers and select a row number to start the display of the
table (does not need to be a multiple of the range (e.g. 50) choosen)
Wrangler http://vis.stanford.edu/wrangler/ type bars in the headers
to show how many non-blanks a column has (can be switched on and off)

So all in all a nice collection. Some of these could probably be put in a
pulic plugin, but one reason I did not do that yet is that they rely on
overloading the OpenRefine code, which must not change too much (after an
update).
Cheers,
Herwig
Herwig Van Marck, PhD.
Senior Expert Research Informatics
Rijvisschestraat 126 3/R, 9052 Zwijnaarde
Tel: +32 (0)9 248.16.01
Bio Informatics Training and Service Facility (BITS
http://www.bits.vib.be/)
On 4 September 2015 at 00:58, QI<EMAIL_ADDRESS>wrote:

@hvmarck https://github.com/hvmarck That's a interesting hack. what's
the proper way to handle the comment according to your experience? I think
it's a standalone plugin which you can apply the comment to any selected
step chosen from the history panel.
Just curiosity if you don't mind. what's the other functionalities of your
internal plugin?
—
Reply to this email directly or view it on GitHub
https://github.com/OpenRefine/OpenRefine/issues/1057#issuecomment-137595820
.

The recommended way to do this is use Javascript style comments and run things through JSmin before doing the JSON decoding.
The one reservation that I have to doing this is that it encourages people to consider the JSON editable which is something that we haven't thought through the implications of.

Has there been any further movement on this?

No movement, but someone could just add jsmin (perhaps from jawr-core https://github.com/j-a-w-r/jawr-main-repo/tree/master/jawr/jawr-core) to where it would strip out the comments prior to where we apply operation history here: https://github.com/OpenRefine/OpenRefine/blob/master/main/webapp/modules/core/scripts/project/history-panel.js#L292
@cameronstewart You can add a bounty for that work and someone can hack on it: https://www.bountysource.com/issues/26383867-add-comments-in-a-recipe

@cameronstewart Comments are now supported via the VIB-BITS plugin and its History Tools that support comments in Undo/Redo history.  Read their manual. https://www.bits.vib.be/software-overview/openrefine
@hvmarck Thanks for finally adding support for comments in Undo/Redo history and making it a part of your public plugin !
Closing this issue, since feature is now handled via a public plugin that is available.

If this is useful functionality, which it would appear to be, it should be in the core product, not an extension (plus the VIB extension appears to be gone.
My reservation (https://github.com/OpenRefine/OpenRefine/issues/1057#issuecomment-148820539) still holds, but a better suggestion for a way to support comments, assuming we don't switch to an entirely new DSL, would be to support YAML and use YAML's comment facility. JSON readers typically support YAML as well since the two syntaxes are equivalent/compatible.

@tfmorris sounds reasonable.

Hi,
Is there something new about this issue?
Thanks in advance,
Best,
Aude

Couldn’t you also support transformations in a dummy comment language and just ignore those transformations?  They should end up in the recipe looking like comment:here are my comments.  Also, I’m not sure how the suggestion to use YAML improves things.  Don’t the same concerns exist there too?

Another option is to add a checksum to the serialized format to ensure integrity.  So that if people edit the serialization without updating the checksum you get an error.  This would probably catch the more naive programmers.  You could even encrypt with a secret key if you want better assurances.  This could all be done after stripping comments to make editing comments safe.

@audealexandre This issue contains the complete history and status. It will be updated when there is a change.
@gitonthescene Thanks for the implementation suggestions. The advantage of YAML is that it has a native comment syntax, whereas JSON doesn't.

Ah, JSON, not Javascript.  Now I get it.  Thanks.


Adding Support for error handling management outsite the infrastruct…

Adding Suppoort for error handling management outsite the infrastrure this improvement should enable us to hanble exception and take ownnership about how to manage it and how we take actions about it.

@jofallas Do you know that in version 1.0+ the errors are being serialized and being sent to Node?
If an exception occurs in a handler in .NET:
connection.send('request', (err, response) => {     //err will be the serialized .net exception });
Just want to make sure you know this because the stderr stream that you are exposing is receiving no "treatment", i.e. you can send a message from .NET by doing Console.Error.Write(....) and get only half of it in two calls to the onError callback.
Also, regaring onExit, the onDisconnect that Connection already exposes should serve the same purpose, or is there a specific use case for onExit that you had in mind that onDisconnect doesn't cover?

@ruidfigueiredo, Yes we checked the available changes in the new version, but it only handles the errors in .NET inside the handlers excluding any unhandled exception throw by the application outside it.
That's why we are adding the onError callback and leaving the responsibility of the "treatment" on the user customization.
Regarding onExited, we found a specific use case where the stream is not being closed when we are starting an external console app .NET process which keeps alive after the main .NET process ends, keeping open the stream until the new process is terminated. Also we think that should be a good practice to have access of the exit code when the .NET process ends.
Thanks

@jofallas If there's an error outside a .NET handler but inside ElectronCgi.DotNet then please let me know because that's a bug that needs to be fixed.
If there's an error outside the .NET handler and outside ElectronCgi.DotNet then that's an error in your application and I don't see how using the connection's stderr might be useful in that scenario (maybe you can provide an example?)
The onExit case is more interesting. You are using ElectronCgi to connect to a .NET console app that launches another .NET console app?

@ruidfigueiredo  Well, there is not a bug in any side. I resolve a bug in our application which throws an unhandled exception in our code that is executed outside the .NET handlers, and that could be normal in any application so I was missing the details about the exception details because the stderr is only being written in the process standard output.
That's why I believe that's is a good enhancement to provide a mechanism in the electron-cgi infrastructure to provide the corresponding treatment to the errors data in the stderr.
Related with the onExit, yes I am using ElectronCgi as you said with a .NET console app that launches another .NET console app that keeps alive the stream connection when the first .NET process ends.

@ruidfigueiredo I quite understand, don't worry about it, this pandemic is causing a lot of impact in the entire world.
Thanks a lot for your support this library it's have a very good future, I hope that we can pass this current global situation.
Regards and thanks again

thanks @jofallas
I am trying out the resulting merge, can I just suggest an additional change, instead of onExited, lets name it onExit ?
Also, can I make it so that it can run like this:
const connection = new ConnectionBuilder()
    .connectTo(...)
    .onExit(exitCode => console.log(exitCode))
    .onStderr(error => console.log(error))
    .build();

?

hi @jofallas I did not hear back from you, is it ok if I push this with the onExit change?


Map stays in background behind the view

Hey,
I have been using cordova-plugin-googlemaps with ionic framework.
in the version 1.3.7, If I go to the map view, then go back to the previous view, It seems like the map is still in background. All interactions are disabled in the area where the map was displayed.
Plus, when I reload, the app, my current view disappears (that's normal), and I can see the map (it is supposed to be closed).
The probleme is not there in the previous versions (at least it works great using 1.3.4, but this version does not support crosswalk ...).

Also having this problems as well.

Once the map is created it wil last there till you remove it. It's not html content, it is a native view behind the webview. If you change your view, hide the map or call .remove()  and reinit. If you just want to make it invisible, call setVisible() and setClickable(false).

@Hirbod thanks for the info - however in prior versions, when the view wasn't visible (like an ionic) that had the map on it it would not impact the other views. I assume this changed in 1.3.7?

Yeah there is a bug in the npm version. You could try GitHub master. I will incorporate some new fixes in the new year which will also fix some other crosswalk / white view relates problems. But I'm pretty sure that the master should fix your problem

@Hirbod sounds good - do you know if there are still issues removing plugins using cordova plugin rm?

@Hirbod this issue is not fixed in 1.3.9 - shall we re-open?

No, as it is not a plugin bug. You are doing something wrong

I guess something changed in ionic which will interfere here. On view change, hide map, setClickable false and on return, make it visible again and set setClickeable to true. I handle this the same way

I've tried this and it doesn't appear to work correctly. Is there a way to see if the view is behind the WKWebview?

I am facing the same issue.
My div map is showing behind the view and i am not sure how to fix this.
I've tried set the z-index of the div, but doesn't work, the same with the setClickable and setVisible.
Could someone help me fix this?

@Hirbod You should take care if you reopen this.

I am building an app with Ionic2 and managed to make the map show up. The problem was what @Hirbod explained:
Once the map is created it wil last there till you remove it. It's not html content, it is a native view behind the webview.
So what I did was use the framework's lifecycle to hide my webview content when I was navigation to the view with the map and then display it again when navigating back:
export class SearchPage {

    public ionViewWillEnter(): void {
        // Entering in SearchPage, assuring the content will show up
        document.getElementById('search-view').style.display = 'initial';
    }

    public ionViewWillLeave(): void {
        // Going to map view
        document.getElementById('search-view').style.display = 'none';
    }
}

#search-view is an <ion-content> in this case.
<ion-content class="search" id="search-view">
...
</ion-content>

And my map view has a fullscreen native map inside my <ion-content>. And it worked very well in my case. The code is in: https://github.com/fmsouza/ionic


Question about sage_conv in version 1.6.0

❓ Questions & Help

Sage_conv in master branch is quite different from original paper that have concatenation.
Could you explain more about   # propagate_type: (x: OptPairTensor)
if isinstance(x, Tensor):
x: OptPairTensor = (x, x)
propagate_type: (x: OptPairTensor)
out = self.propagate(edge_index, x=x, size=size)
Thank You

Hi,
it is true that we do not use concatenation, but instead use summation for the addition of central node information. Note that both formulations are equivalent in case features are first transformed by a weight matrix.
Regarding your second point: You can use SAGEConv both for homogeneous and bipartite graphs. In case of the latter input features are given as a tuple of node features, where information from the source nodes x[0] is send to the destination nodes x[1]. Therefore, in case of homogeneous graphs, we can simply set x to (x, x).

@rusty1s any tips on what we might have to change to make it a concatenation? Also what does it mean to transform features by a weight matrix?

For concatenation, you can do the following:
h = self.propagate(...)
h = torch.cat([x, h], dim=-1)
out = self.lin(h)

but it's actually the same as doing
h = self.propagate(...)
out = self.lin1(x) + self.lin2(h)

The latter has the advantage of not explicitly creating a tensor of shape [num_nodes, 2 * num_features].


Watch Resource - support for Property and Path Filters

I'd like to use this provider for the watch support, but we need the include/exclude filters.
I've tested #36  and also found that:

Property filters are not supported
Resource documentation is missing

I'd be happy to contribute, but it's unclear as to how the schema should be defined to support the filters.
One potential option would be to name specific filter types - property_filter and path_filter, but this goes against the generic filters block.
resource "xray_watch" "test" {
	name  = "%s"
	description = "%s"
	resources {
		type = "repository"
		name = "%s"
		bin_mgr_id = "%s"
		filters {
			type = "package-type"
			value = "%s"
		}
                property_filter { # 1 or more of these can be defined
                  key = "someKey"  # required field
                  value = "someValue" # required field
                }
                path_filter { # 0 or 1 of these can be defined.
                  exclude_patterns = ["some/path"] # default value of []
                  include_patterns = ["some/path"] # default value of []
                }
	}
	assigned_policies {
		name = xray_policy.test.name
		type = "security"
	}
}

Personally, I would consider changing the interface and have a specific filter parameters for each filter type, i.e. package_type, etc, as there are quite a few filter options because:

the filter type string isn't clear and obvious. For example, the path filter for builds is ant-patterns but for repositories it is path-ant-patterns.
There are different filters that apply to a repository or a build. I'm not sure how obvious/helpful the error message that artifactory/xray returns.

You can go to https://github.com/jfrog/terraform-provider-artifactory/issues/6 to view some fuller examples.

@chb0github i appreciate the terraform provider efforts.
Can you comment on whether or when this enhancement can / will be picked up ?

I can honestly tell you that enhancements and bug fixes for this code are on hold until at least end of May. We might actually consider a major version bump as several non-backward compatible changes have been requested.

Hi @chb0github, what's the latest on the updates to the watch resource?
I'm happy to implement, but am unable as it's unclear on the design jfrog will support.
Cheers

Please find me on gitter tomorrow to discuss.

@josh-barker-coles Are we going to be able to move this forward?


feat: add eu countries const

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add a tweak to activate the Build Order tab on build completion

A small tweak to automatically switch the output window to the "Build Order" tab when a build completes.
Useful for finding the source of errors after a parallel build of multiple projects.
Very similar code to the Build Stats tweak.

I love it. Thank you!


Gas metering tracking issue

Below is a draft on how the overall setup could look like. Let me know your thoughts!
https://gist.github.com/vlopes11/9fd12e61903b3fa52542d14677bfb982


Some questions regarding the data

Hi,
Thanks for your amazing work!
When I was going through your paper and code, I found that the wrench values are used for calculating the per-point motions for rigid objects and for the mpm&deformable objects, the difference between target pointcloud and the initial pointcloud is regarded as per-point motions. The related code snippets can be found:
rigid objects
deformable objects
mpm objects
I'm wondering whether this calculation process is the "flow model" illustrated in the Figure. 2 and Figure. 3.
Moreover, I noticed that in the rigid object dataset script, there is a function for generating random wrench for an object, and this function is turned on in the training script. Why do you add such a random function given the wrench value in the training data. cause from my understanding, the wrench value should be related with the contact heatmap and the force heatmap. If my understanding is not correct, please feel free to point it out. I would sincerely appreciate your correction :p
btw, I'm also wondering how you set the wrench values for rigid objects in real robot experiments.
Thanks in avdance for your patient and kind help!
Cheers,
Yucheng

Yucheng, thanks for your attention. Since we have discussed it, I'm closing the issue now. :)


How do you use this feature?

The readme says "It'll work with the default format keyboard binding" but a lot of people will not know what that is, or even what to search for.
Please consider adding explicit instructions for how to use it. Both default key combination, and via the VS Code menus if that's applicable.
Also, it does not seem to work in a new, unnamed file. If I create a new file, do not save it, type in some bash scripting, and set the language mode to 'Shell Script' then neither of the following works:

format document (shift+option+F), does nothing
format selection (cmd+K, cmd+F), results in 'Sorry, but there is no formatter for 'shellscript'-files installed."

Thanks.

Unfortunately, I can no longer maintain this project, I changed platforms long time ago

OK, thanks. Please consider archiving the project and update your README and description to make it clear to visitors that it’s no longer maintained and it's no longer possible to contribute.

I will, thank you


NodeJS mysql module transforms localhost to <IP_ADDRESS> and fails connection

I am trying to connect to ProxySQL that is set-up to connect to a MySQL database. From command line, the following works. The port number 6033 is for ProxySQL.
mysql -u demo1 -P6033 --database=database -p

However, the connection to ProxySQL with this package does not work.
var connection = mysql.createConnection({
        "host" : "localhost",
        "port": 6033,
        "user": "demo1",
        "password" : "password",
        "database": "database"
    })
 connection.connect( (err) => { console.log(err.stack) } )

This is the error message.
Error: ER_ACCESS_DENIED_ERROR: ProxySQL Error: Access denied for user<EMAIL_ADDRESS>(using password: YES)

The user was created as
CREATE USER 'demo1'@'localhost' IDENTIFIED BY 'password';

I tried adding another user as<EMAIL_ADDRESS>and granted privileges for the same; didn't work.
How do I retain the definition above and get mysql to return a successful connection?

Hi! And thanks for the report. I'm not familiar with any code here that specifically modifies the host at all. Would you be willing to make a pull request with a fix or perhaps some step by step instructions for how to set everything up, the versions of software involved, etc. so I can reproduce and see what is going on?

Hello @dougwilson
Please find instructions to reproduce. However, let me set-up some context.
The aim is to use this package to connect to ProxySQL which is set-up to connect to MySQL. The issue here is, while I can connect to ProxySQL from mysql command prompt, the same connection parameters do not work with this package.
To be fair, the issue could be with either of the products - mysql package, ProxySQL or MySQL.
Steps to reproduce
These steps were executed on Ubuntu 18.04 LTS on AWS.
Product installation
MySQL installation
cd /tmp
curl -OL https://dev.mysql.com/get/mysql-apt-config_0.8.13-1_all.deb
#  2. Install the .deb file
#     - Scroll down to the last option (ok) and hit Enter
sudo dpkg -i mysql-apt-config_0.8.13-1_all.deb
#  3. Refresh apt repository
sudo apt update
#  4. Install MySQL server
#     - Set root password and save it somewhere secure.
#     - Use strong authentication option.
sudo apt install -y mysql-server

ProxySQL installation
# 1. Download latest .deb from here https://github.com/sysown/proxysql/releases/tag/v2.0.5
cd /tmp
curl -OL https://github.com/sysown/proxysql/releases/download/v2.0.5/proxysql_2.0.5-clickhouse-ubuntu18_amd64.deb
# 2. Install the package
sudo dpkg -i proxysql_2.0.5-clickhouse-ubuntu18_amd64.deb
# 3. Start service
sudo service proxysql start

Package installation
npm i mysql --save

Configuration
MySQL
The following DDL were saved in a file install.sql and run as
mysql -u root -p < install.sql

-- Create schema
CREATE SCHEMA MYDB DEFAULT CHARACTER SET utf8 ;

-- *******************
-- The following were the original CREATE and GRANT statements.
-- I tried adding another pair of CREATE and GRANT for<EMAIL_ADDRESS>
-- That didn't work.
-- *******************

-- Create users for our demo
CREATE USER 'demo1'@'localhost' IDENTIFIED BY 'password';
-- Grant privileges
GRANT ALL PRIVILEGES ON MYDB.* TO 'demo1'@'localhost' WITH GRANT OPTION;
FLUSH PRIVILEGES;

ProxySQL
The following configuration was copy-replaced into /etc/proxysql.conf.
admin_interfaces = {
	admin_ifaces="<IP_ADDRESS>:7031"
}

mysql_servers =
(
	{
		address="localhost"
		port=3306
		hostgroup=0
		max_connections=100
	}
)

mysql_users:
(
	{
		username = "demo1"
		password = "password"
		default_hostgroup = 0
		max_connections=500
		default_schema="MYDB"
		active = 1
	}
)

The ProxySQL service was reloaded thus:
sudo proxysql service restart

NodeJS
The options object passed to connect() is as shown below.
{
    "host" : "localhost",
    "port" : 6033,
    "user" : "demo1",
    "password" : "password",
    "database" : "BROKERS"
}

And, this is how I am using mysql package.
const mysql = require('mysql')
//
function connect(options, callback) {
    var connection = mysql.createConnection(options)
    connection.connect( (err) => {
        if (err) {
            return callback(err, null)
        }
        return callback(null, connection)
    })
}
//
module.exports = {
    connect
}

The err object is populated with sqlMessage as Access denied for user<EMAIL_ADDRESS>(using password: YES).

Thank you so much! I will find some time to get it all set up. And you are 100% sure the connection with the command line that worked actually went though proxysql server and did not just connect over unix socket to the mysql server instance?
I'm asking because the standard "mysql" command line program thinks the host of "localhost" means to connect over unix socket and not over tcp as a convenience, and the port you pass in (-P) is ignored.

You can add --protocol=TCP to your command line to ensure that is is using a TCP connection and will then connect to the proxysql port.

Basically, let me know what the result of running the following command is on your setup when you have a chance:
mysql -u demo1 --protocol=TCP -P6033 --database=database -p

Finished my tests; and, you are right! It wasn't using a TCP connection.
ubuntu@ip-172-31-23-36:/tmp$ mysql -u demo1 --protocol=TCP -p -P6033 --database=MYDB
Enter password:
ERROR 1045 (28000): ProxySQL Error: Access denied for user<EMAIL_ADDRESS>(using password: YES)

Can you please help me understand, how to enable TCP connection in ProxySQL configuration? Alternatively, can socketPath be used without changing the current ProxySQL configuration?

Before you mentioned it, I hadn't heard of ProxySQL before, so really don't know how to really assist with ProxySQL configuration. I'm happy to help with anything relayed to this Node.js module, though. You can use a UNIX socket with this module if that is what you're asking?

Just pass in the path to your UNIX socket to the socketPath option of this module, if so, and it will use that UNIX socket instead of TCP to connect to the server.

From mysql> show variables like 'socket';, I got the path as /var/run/mysqld/mysqld.sock. I used it as so:
//
var connection = mysql.createConnection({
"socketPath" : "/var/run/mysqld/mysqld.sock",
"user" : "demo1",
"password": "password",
"database":"MYDB"
})
//
    connection.connect( (err) => {
        if (err) {
            console.log('ERROR ' + err.sqlMessage)
        } else {
        console.log('INFO ' + err.sqlMessage)
}
    })

I get this error message.
ERROR Client does not support authentication protocol requested by server; consider upgrading MySQL client
Thoughts?

Interesting. What are the grants on that user? The mysql server usually gives that error if it is not set to enable password auth over the unix socket.

You can validate if the password is being used for auth with the command line client by removing the -p and seeing if the auth still works. This would indicate that user is jot set up to use password auth over the unix socket (which is the only auth method this module currently supports).

You can validate if the password is being used for auth with the command line client by removing the -p and seeing if the auth still works.

I tried both options as shown below - is this what you meant?
Via ProxySQL
ubuntu@ip-172-31-23-36:~$ mysql -u demo1 -P6033
ERROR 1045 (28000): Access denied for user 'demo1'@'localhost' (using password: NO)

Direct connection
ubuntu@ip-172-31-23-36:~$ mysql -u demo1
ERROR 1045 (28000): Access denied for user 'demo1'@'localhost' (using password: NO)

What are the grants on that user?

 TO 'demo1'@'localhost' WITH GRANT OPTION;

Hm, then I'm not sure why the server is rejecting it. Would you mind updating your original instructions to get to reproduce this issue so I can take a look? Also I see they are based on AWS, is that a free service?

I have updated the steps to reproduce this error with instructions on connecting via Unix socket.
I am using AWS only to set-up the Ubuntu server. It uses the opens source versions of MySQL and ProxySQL.

Ah, sorry, I missed the Ubuntu note. I should be able to run that locally, as Ubuntu is available in the Windows store 👍 . I will try to get everything set up this weekend and look into the issue for you.

👍 Thanks for your support so far!

SOLVED! Thanks to this SO question - https://stackoverflow.com/a/50131831/919480
Two things.

MySQL has to be told to use mysql_native_password. I think, this is related to the choice of option 'Strong Password' when installing MySQL.
The host parameter should not be localhost because on Unix, it is treated differently. That is, the connection is via a socket. Therefore, set host parameter to <IP_ADDRESS>. Or, remove host parameter and use socketPath.

Hi @chainhead , great to know! Do you still need me to look into this, or is it OK to close the issue?

@dougwilson Yes, ok to close; closing now. 👍


Expose external code loading with std::execution::run_external

Description
Fixes #1846
Fixes #5691
Checklist

[x] I have linked to any relevant issues.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have updated the documentation where relevant (API docs, the reference, and the Sway book).
[x] I have added tests that prove my fix is effective or that my feature works.
[x] I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
[x] I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
[x] I have requested a review from the relevant team or maintainers.

Unfortunately, type mismatch in loaded code arguments is undefined behavior, and there isn't really a lot we can do about that.
I would want us to introduce some form of static checking later if we can, but the best practice here is going to be for the programmers to implement the same ABI (defined by a separate library ideally) in their proxy and target.


Take the project context into account when generating code

Currently, the openapi command seems to generate code without taking the context in to account (e.g., is it a single file or is this within a project). The issue with this is, when I'm not within a project, I still get multiple files generated and I have to manually organize these files in a project or copy them in to a single file to run the generated service.
It would be great if I can have control over how the files are generated. For example if I'm not in a project, being able to specify whether I want everything (schema + service) generated in the same file or separate files.
If I'm within a project, a better way to pick the contract to generate code for. For example, if there are multiple modules, being able to specify for which modules I want to generate code for (the contract will have to be placed in the resources directory of the module). If I want to generate code for a specific file in the module, then an additional option to specify the file name. The idea is to have a properly structured way of using OpenAPI files with a project which would simplify the use of the command as well.

Had a chat with @lnash94 regarding this. To be clear, the concern here is that we can generate source files from anywhere, without giving an output directory. So this means I can generate the code from a non-Ballerina project directory. But when you do that, you can't run the resulting code as-is. You either have to copy the code into 1 file or copy the files to a Ballerina project. So instead of being able to generate from anywhere, how about restricting the generation if I'm generating the code from a non-Ballerina project directory and haven't specified an output directory for the resulting generated code?


Request for depth engine release on debian stretch

Hi Everyone,
as the title tells, I would like to request Debian packages for genuine debian distros as well, preferably for stretch, but also for buster. Is there hope for that in the near future?
BR.

Please head over to  https://aka.ms/azurekinectfeedback to ask for this feature.

https://github.com/microsoft/Azure-Kinect-Sensor-SDK/blob/develop/docs/usage.md#debian-package
"We currently have debian packages available for Ubuntu 18.04. If you have need for a different debian distribution, please file an issue."
Guess where the hyperlink leads:
https://github.com/microsoft/Azure-Kinect-Sensor-SDK/issues/new?assignees=&labels=Enhancement&template=feature-request--enhancement-.md&title=

Ahh, thank you for pointing that out. I will update the link


Fix null temp

Overview
Similar to #59. I encountered the error when trying to rechunk from a zarr store to zarr store on the cloud using fsspec mapper. This PR works in the same exact way as PR #64, but with direct zarr store to zarr store rechunking rather than from dataset. It seems like current test doesn't capture this case.

Fantastic, thanks so much for this fix!


feat: add support for embed video files

Type of change

[x] New feature (non-breaking change which adds functionality)

Description
I want to embed a video file hosted on my own blog, CDN, or publicly available without a video platform using the HTML 5 <video> element.
Implementation
Here are examples on how you can include video file with respect to img_cdn and img_path variables.
{% include embed/video.html
src='https://upload.wikimedia.org/wikipedia/commons/transcoded/a/a7/How_to_make_video.webm/How_to_make_video.webm.720p.vp9.webm'
autoplay=true
loop=true
muted=true
title='How to make video' %}

{% include embed/video.html src='demo.mp4' autoplay=true loop=true title='Demo video' %}

Arguments:

src - required, path to a file
poster='/path/to/poster.png' - optional, poster image for a video that is shown while video is downloading
title='Text' - optional, title for a video that appears below the video and looks same as for images
autoplay=true - optional, video automatically begins to play back as soon as it can
loop=true - optional, automatically seek back to the start upon reaching the end of the video
muted=true - optional, audio will be initially silenced

Demo

Note: 2nd and 3rd videos are added using new functionality.

@kungfux when you've added on the blank lines suggested above, try merging into the master branch on your own :)

@kungfux when you've added on the blank lines suggested above, try merging into the master branch on your own :)

Am I correct that Codacy Static Code Analysis' errors can be ignored and 2019-08-08-write-a-new-post.md' file is subject to inspection only because it was touched and so will not cause similar problems for future PRs?

Am I correct that Codacy Static Code Analysis errors can be ignored and 2019-08-08-write-a-new-post.md file is subject to inspection only because it was touched and so will not cause similar problems for future PRs?

That's the markdown line length limit detection, in reality most markdown lines will be more than 80 characters, so yes, you can ignore these warnings.

@kungfux when you've added on the blank lines suggested above, try merging into the master branch on your own :)

Roger that :)


chore(deps): update dependency @types/node to v16.11.54

This PR contains the following updates:

Package
Type
Update
Change

@types/node
devDependencies
patch
16.11.51 -> 16.11.54

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: package-lock.json
You don't have permission to create /tmp/renovate/cache/buildpack.
curl: (23) Failed writing received data to disk/application
tar: /tmp/renovate/cache/buildpack/7576b42decc67f70b61585f41c5074bea75c260886346e567578ea0e0dc2d571/node-v16.17.0-linux-x64.tar.xz: Cannot open: No such file or directory
tar: Error is not recoverable: exiting now
bash: npm: command not found


Indices separated by commas is not a thing in Python

Page 3 of the handout details separating indices by commas to get non-consecutive items in a list. This is not a thing...

No longer relevant, not present in course book


[URGENT//SEVERE] - Severe ANR due to update to android branch 5.1.4

We are receiving tons of ANR on all mobiles after updating to the latest android dependencies through the android dependency resolver for unity android.
Input dispatching timed out (802bba2 com.Proxy42.WorldLeagueLiveSoccer/io.branch.unity.BranchUnityActivity (server) is not responding. Waited 10001ms for MotionEvent)
this is happening only when the unity library use io.branch.sdk.android.library-5.1.4 - it doesn't happen on io.branch.sdk.android.library-5.1.3

@popisdead do you happen to have  Branch logs that will help us better understand the issue? You can enable logging via .enableLogging()


Is there a way to parse group keys (tags) in Bosun to get a particular tag value?

I have a bosun server setup which queries OpenTSDB. We have some alerts setup which are grouped using certain tags.
Sample Query:
$query= sum:metrics.get{key1=*,key2=*}{outcome=ERROR}

Sample tags obtained are:
"Tags": "key1=value1,key2=value2"

This value is currently a string. I want to extract the value of a particular key from the Tags string. This would be something like treating this string as a map. Is there a way to do this in Bosun expression language?

What is the thing you are trying to achieve by extracting the value of the tag ?

Mark

I have a bosun server setup which queries OpenTSDB. We have some alerts setup which are grouped using certain tags.
Sample Query:
$query= sum:metrics.get{key1=*,key2=*}{outcome=ERROR}

Sample tags obtained are:
"Tags": "key1=value1,key2=value2"

This value is currently a string. I want to extract the value of a particular key from the Tags string. This would be something like treating this string as a map. Is there a way to do this in Bosun expression language?

Have you figured out a way ?

I have a similar problem, I want to get the top 10 results after sort for the avg response time of the partner.
I used the filter function to implement it,
I want to leave it here just in case someone have the same problem.

filter(variantSet, numberSet) (seriesSet|numberSet)
Returns all results in variantSet that are a subset of numberSet and have a non-zero value. Useful with the limit and sort functions to return the top X results of a query.

filter(q("avg:$sample-avg-zero:avg_overall_ms{partner=h_}", "$start", ""),limit(sort(avg(q("avg:$sample-avg-zero:avg_overall_ms{partner=h_}", "$start", "")),"desc"),10))


2024 governance update

This PR follows discussions on the forum. The goal is to document how the project currently operates.
What changed
As discussed I

Added Apache meritocratic model back
Clarified the process for voting new members into the different role
Clarified how funds are managed.
Introduced the Core Developer Group
Remove the Steering Committee

I also took the opportunity to introduce the following role

Clarify language in the Code of Conduct
Code of conduct committee
Designer

What will be updated in a second phase

I left the Project Manager description as-is. The new scope is currently being discussed with the Advisory Committee.
I have documented the current process for renewing the Advisory Committee. Once we finalize the process, I propose we update the Governance to make it more transparent.
Mention the mission, vision, and values.  We need to add them first on the website and reference them in the Governance and Code of Conduct document.

the goal is to make minimal changes to accurately document how OpenRefine operates today, even if it is flawed

@magdmartin both my comments regarding the "people buckets", the CoC, and the AC are about how OpenRefine operates today. If you want to criticize one of my points please do it in the threads instead of trying to bulldoze my concerns.

@Abbe98 I believe your comments are outside the scope of this pull request, and I will not address them here. In my previous message, I welcomed your input and suggested it might be more suitable for a forum discussion. Additionally, some of your messages come across as quite harsh. I encourage you to adopt a less accusatory tone if you want your arguments to be more convincing.

I think my comments deserves more than a plain refusal to comment on the subjects at hand, especially given the broader subject of governance. However, given your reservation I will reach out to you in private so that we can resolve any issue that concerns you.

Following recent conversation (Dec 12 Advisory Committee call and forum conversation)  I will update the governance to

Extract Policy into a new section (within the same file). This aims to separate governance and how different committee operates.
The governance will focus on the different Roles and responsibilities. It will link to the relevant policy that applies to the group
The Policy will include

Member election
Conflict of interest
Managing funding


Feature/dgnss updates

Overall: Adds RTK GNSS capabilities via ublox apckage and visualization via mapviz
Details:

Remove most references to the old GPS/IMU node
Adds Ublox and Mapviz packages
Adds new node written in Python that maps Ublox GPS messages to CoordinateMsg
Created launch files for Ublox/ArduSimple node, mapper node, and mapviz

The submodules have been accidentally deleted in this PR, please revert this change. Also will review the code later.

Comparison between the src folder in this branch and master. Submodules are confirmed missing.


dep init failed

What version of dep are you using (dep version)?
dep:
version     : devel
build date  :
git hash    :
go version  : go1.11
go compiler : gc
platform    : linux/amd64
features    : ImportDuringSolve=false
What dep command did you run?
dep init
What did you expect to see?
What did you see instead?
init failed: unable to determine direct dependencies: gopkg.in/natefinch/lumberjack is not a valid path for a source on gopkg.in

Dep was officially deprecated earlier this year, and the proposal to archive this repository was accepted. As such, I'm closing outstanding issues before archiving the repository. For any further comments, please use the proposal thread on the Go issue tracker. Thanks!


feat(pixels): add my new pixel (4, 1)

Checklist

[x] I ran npm test locally and it passed without errors.

[x] I only edited the _data/pixels.json file.

[x] I entered the username in the pixels.json that I'm also using to create this pull request.
[x] I acknowledge that all my contributions will be made under the project's license.

Code Review / build (pull_request) results seem to indicate that <UNCLAIMED> should not be altered.
Danger: ⅹ Failing the build, there are 2 fails.
8
## Failures
It seems like you are accidentally deleting or editing some contributions of others. Please make sure you have pulled the latest changes from the master branch and resolved any merge conflicts. https://help.github.com/en/articles/syncing-a-fork

Make sure that the following usernames are indeed included and unchanged: <UNCLAIMED>

As such is it the case that there is no space left for my contribution? Thank you.


Graphics Card is not recognized

I am using a stock Dell G7 Gaming Laptop with a Nvidia GeForce GTX 1060 Max-Q graphics card inside.  The miner does not see my graphics card for whatever reason.  Any advice/help would be very welcomed and appreciated.

Closing. Feel free to reopen if this is still relevant.


Create token validation for the request when exposing the route publicly

https://github.com/BirthdayyBot/BirthdayyBotSapphire/blob/1d885340828ee1069363529f1f890863a948c19c/src/routes/main.ts#L6

Done in 9b6ee83d0336d0e1189d0afe181cef34a9f72cfa


Generated client has wrong namespace for ComparisonHelper

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
When using version 13.0.0-preview.93 of StrawberryShake and generating a client from graphql scheme, the generated client has some methods that depend on "ComparisonHelper". These methods has the wrong namespace in the generated code, they refer to global::StrawberryShake.Helper.ComparisonHelper.SequenceEqual(Items, other.Items) when the correct namespace seems to be  "Internal", so this seems correct: global::StrawberryShake.Internal.ComparisonHelper.SequenceEqual(Items, other.Items).
Steps to reproduce

dotnet graphql generate
Open generated client code, check errors.

Relevant log output
Severity	Code	Description	Project	File	Line	Suppression State
Error	CS0234	The type or namespace name 'Helper' does not exist in the namespace 'StrawberryShake' (are you missing an assembly reference?)	Application	".."\Generated\GraphApiClient.Client.cs	1683	Active

Additional Context?
No response
Product
Strawberry Shake
Version
13.0.0-preview.93

Following the Get started with Strawberry Shake in a Console application I see the same issue using .NET 7, SDK 7.0.102.

I am also having this issue and I think its due to the nuget packages for StrawberryShake.CodeGeneration.CSharp.Analyzers still being on version 12.17.0. Inspecting the generated client code it still has references to using 12.17.0.

Hi there
I also answerd in #5831 and this solved the issues @cornonthecobb  was having:
We added a bunch of new meta packages that avoid common mistakes that are often done.
Have a look at this:
https://chillicream.com/blog/2023/02/08/new-in-hot-chocolate-13#strawberry-shake

Thanks @PascalSenn Now it works. I read the release notes yesterday but when trying the getting started I could not remember that detail. It would be great to update  Get started with Strawberry Shake in a Console application


Overhaul: Principles of site design text is too big

...which makes text run onto two lines awkwardly.

this page is a work in progress, there's another open issue on it already.


Corrected Ambiguous use of 'SQLITE_DETERMINISTIC' #986

#986 Ambiguous use of 'SQLITE_DETERMINISTIC'

This project is dead or what?
If this one-liner is fixing issue why is it stuck since June?
Especially now when everyone is moving up to Xcode 12 and the issue is back on top.

@ypopovych Can you take care about it please?

@jberkel Wanna check please??

@TheNephilim88
@docpitz
66d4c76, this one works for me in Xcode 12.1 with Podfile like this...
pod 'SQLCipher', '~>4.2.0'
pod 'SQLite.swift/SQLCipher', '~>0.12.0'
Is this the final / official solution?

Is this change still required? The current head builds without errors.

no activity, closing this


Add shared job to abort GitHub Actions execution if a PR branch is not up to date

Overview
Instead of wasting CI time (since I'll end up rebasing and force-pushing), abort CI checks until the branch is up to date with the primary (e.g., "main" or "master") branch.
References

https://stackoverflow.com/questions/76151411/use-github-actions-to-check-if-branch-is-up-to-date-with-main
https://www.packtpub.com/product/go-for-devops/9781801818896

So far so good. Will reopen if the latest batch of changes doesn't prove to properly cover corner cases.

After closing this out I stumbled across this:

https://stackoverflow.com/questions/61393981/getting-base-branch-sha-on-pull-request-in-github-action-workflow
https://stackoverflow.com/a/73534917/903870

What worked for me was: github.event.workflow_run.head_sha.
The top-voted answer suggesting github.event.pull_request.head.sha didn't work for me.
I found this by examining all the possible data in the github.context object using the method suggested here - https://stackoverflow.com/a/70107953

It looks like github.event.workflow_run.head_sha will likely provide the HEAD sha for all event types. This is what I would have ended up using before the changes in #139.
I may return to this point if the changes in #139 prove confusing or don't cover enough workflow cases.


Login Page Demo

Tabbed Login Blog post demo

Login and register don't match fields.

Login requires a username

Register doesn't have username, only email

You should ditch the username and use only email.


WebGL Vector Layer: Clear existing buffers before allocating new ones

This should fix a longstanding memory leak reported with the WebGL Vector Layer Renderer.
Reported in #15910 #15850 #15132
With this change I don't see the WebGL Vector Layer example crash on iOS, and the memory footprint looks stable locally. Could someone please try out this branch and confirm that this indeed fixes it? Thanks in advance.

On now I was just bit by the flaky tests... will work on this next (as well as the flashing bug in the webgl vector example)


Ribbon pcb has no trimmer silkscreen descriptions

On the Ribbon board, there are two trimmers, but neither has any text on the silkscreen hinting at what they do.
See the other boards for the format used. I've been making little KibBuzzard text blocks near the trimmers.

New ribbon board has labels.


badges for communities

Changes:

Added fields to the contracts type to support user inputted and manually inputted data for profile elements and badges
Added dataloaders and queries for getting all contracts that are from tokens owned by a user
Added fields in the graphQL schema for querying all of a user's contracts as well as the new community and contract fields

Considerations:

As I made this PR, I noticed some repetition now between the Community and Contract types. The community type seems to be a super-type of the contract, but if we are planning on using contracts in a "gallery way" like we do tokens (not the blockchain way), is it worth consolidating these types?
The major difference right now between communities and contracts is that communities are tied to users that display NFTs from a given contract where the current getContractsForUser is all of the contracts they own tokens from regardless of display. This leads me to ask: Should the query for contracts of a user only correspond to contracts of NFTs they have displayed or ones they own?
Also, right now the get community query does some expensive querying for getting all of the holders in a community with preview images. If we are going to consolidate types, we could turn this into a separate resolver. We would also need to consider if the current description field on a community which is derived from the description field on one of the NFTs in the community should be a new field on the contracts table as well.

Also, thinking out loud: maybe we actually want a Badges resolver as a child of the GraphQL User type, instead of (or in addition to) the Contracts resolver?
Contracts would be "all of the contracts this user holds tokens for," but Badges would be "all of the badges this user has." At present, I think the sole distinction between SQL queries would be that user.Badges should only return contracts that have those badge profile URL fields filled out, instead of returning all contracts. In the future, Badges would also return manually-added badges, but the frontend wouldn't need to care about that detail, because the backend would just include them as part of the Badges resolver.
And maybe a GraphQL Badge type? Given that some future badges may not be contracts, maybe a Badge object is just the set of badge-specific URLs, and an optional contract_id pointing to the contract if there is one? Maybe a name, too, if badges have names?


Add the command package to support executing a series of commands by a config file that defines some instructions

Codecov Report

Merging #58 (bb0be66) into main (6e8ab6c) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              main       #58    +/-   ##
==========================================
  Coverage   100.00%   100.00%
==========================================
  Files           41        55    +14
  Lines         1328      1520   +192
==========================================
+ Hits          1328      1520   +192

Impacted Files
Coverage Δ

command/command.go
100.00% <100.00%> (ø)

command/cp.go
100.00% <100.00%> (ø)

command/echo.go
100.00% <100.00%> (ø)

command/hash.go
100.00% <100.00%> (ø)

command/is_dir.go
100.00% <100.00%> (ø)

command/is_empty.go
100.00% <100.00%> (ø)

command/is_equal.go
100.00% <100.00%> (ø)

command/is_exist.go
100.00% <100.00%> (ø)

command/mkdir.go
100.00% <100.00%> (ø)

command/mv.go
100.00% <100.00%> (ø)

... and 4 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[BEEG CHANGES] HoYoPlay API Adjustments

Main Goal

Adjust Collapse for the new API provided from HoYoPlay (at least for the Hi3 Mainland China as of May 16th).
Use sophon installer for Genshin

PR Status :

Overall Status : In Progress
Commits : In Progress
Synced to base (Collapse:main) : Yes
Build status : OK
Crashing : No
Bug found caused by PR : unk

Templates

Changelog Prefixes
  **[New]**
  **[Imp]**
  **[Fix]**
  **[Loc]**
  **[Doc]**

Please note all of the changes here will be SQUASHED upon merge

Also please ignore Qodana errors
It doesnt like when you add new submodule, lol.


[Feature] Support impersonate service-account for BigQuery

PR checklist

[x] Ensure you have added or ran the appropriate tests for your PR.
[x] DCO signed

What type of PR is this?
feature added
What this PR does / why we need it:
add GCP service-account impersonation support
Which issue(s) this PR fixes:
sc-32423
Special notes for your reviewer:

We use monkey-patch to wrap credentials to impersonated credentials
The implementation is temporary until https://github.com/googleapis/python-bigquery-sqlalchemy/pull/906/files add official support

Configuration example
Piperider will use impersonate_service_account to profile data
jaffle_shop_bq:
  outputs:
    dev:
      dataset: my_dataset
      job_execution_timeout_seconds: 300
      job_retries: 1
      location: US
      method: service-account
      priority: interactive
      project: dev
      threads: 1
      type: bigquery
      keyfile: service-account.json
      impersonate_service_account<EMAIL_ADDRESS>  target: dev

Fix #899


Improve accuracy

I'm using this library in another library for running benchmarks, and I need 2 features out of pretty-ms:

Precision: I need the formatted time to be reasonably accurate.
Cleaness: At the same time I don't need a verbose formatted time, e.g. if something takes 200ms I don't care how many extra μs it also takes.

I can get a clean output by setting compact: true, but then I can't get reasonable precision, as for example the following happens:

501ms gets rounded up to 1s, which gives an error of ~100%.
1.49s gets rounded up to 1s, which gives an error of ~33%.

I think a possible solution to this would be to avoid overriding secondsDecimalDigits and millisecondsDecimalDigits when setting compact: true.
this may not solve the issue completely though, as for instance I imagine minutes would be rounded up too and I see no minutesDecimalDigits option, so perhaps a more general decimalDigits option should be used instead?

Also displaying 50ms rather than 0.5s would be cleaner IMHO.

On a second thought I think a better and more general solution would be to add a maxRoundingDelta option, which will tell how many percentage points the rounded number can deviate from the original number, so for instance if it's set to 10 rounding will only occur if the rounded number will be at most 10% higher/lower than the original number, so for instance 950ms will be rounded to 1s, but 900ms will remain 900ms. By default it should be set to Infinity for backwards compatibility.

Also displaying 500ms rather than 0.5s would be cleaner IMHO.

It already does that: https://github.com/sindresorhus/pretty-ms/blob/37cf0a9b8a458a416642fd4756e65d74a7ca2f51/test.js#L7

I don't think maxRoundingDelta will be useful for other use-cases than yours, so I would try to think of some different solutions. Maybe {prioritizeAccuracy: true} or something.

It already does that:

Right, I don't remember why I wrote that then 🤔

Maybe {prioritizeAccuracy: true} or something.

But accuracy to what degree? Like probably I want 990ms to be rounded up to 1s but I don't want 501ms to be rounded up to 1s too, or maybe I want 990.43ms to be rounded up to 990ms rather than 1s.

I don't think maxRoundingDelta will be useful for other use-cases than yours

I see it as a more powerful/less friendly version of prioritizeAccuracy, it solve the issue that numbers may get rounded up so much that they become kind of useless when accuracy is important. I'm not sure many people would use it, but does it matter? Like would it be better to fork this library just for this use case?

@fabiospampinato in the current version seconds are never rounded up - if precision is 0 decimals and 9.999 seconds has elapsed, pretty-ms will return 9s. This is to make sure we don't have any truncated seconds: if we were rounding 0.5s to 1s, the 0th second would only display 0s between 0-0.5s. See the tests here:
https://github.com/sindresorhus/pretty-ms/blob/eda21362097d47ab309dca8cf07dc79b25fb0efa/test.js#L250-L254
As the original issue no longer applies, I suggest closing this issue, unless there is a need to introduce a roundUp flag to restore the old behavior.


Can't install Twitch Leecher

I uninstalled Twitch Leecher a long time ago and recently tried to reinstall it, but every time I run the .exe nothing happens. Really confused. Have already tried to run as administrator.

You can run the installer (the exe) in the command line with the parameter -l "[logfile]".
Example: TwitchLeecher_1.3.7_x64.exe -l "log.txt"
The logfile should be created in the same folder as the installer. if the logfile gets created please send it over to me. if the logfile does not get created, you have a problem with your PC, preferrably the .NET Framework. Please make sure .NET Framework 4.5 or higher is installed on your system.
And please consider ploviding more information when opening an issue:
https://github.com/Franiac/TwitchLeecher#support--issues

Can you explain what I do? I am computer illiterate first computer I ever owned

Press windows key
type "cmd" and hit enter
navigate to the directory where you saved the installer
type TwitchLeecher_1.3.7_x64.exe -l "log.txt" and hit enter

http://i.imgur.com/4ItUqFq.png
I get this and nothing happens. What am I doing wrong?

yea, and now? did the log file get created or not?

No nothing happened

then you definitely have an issue not related to TL I'm afraid. If the logfile does not get created, not even a single line of code of the installer is actually run. so i cannot help you on this one. have you installed recent updates or something related to the .NET framework or something? I cannot really remote-debug this, i'm sorry.


bug Add retry for reSetupBdbEnvironment() restore.execute()

In reSetupBdbEnvironment() restore.execute() may throw NullPointerException, add retry for restore.execute()

Proposed changes
Issue Number: close #18766
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 34.21 seconds
stream load tsv:          425 seconds loaded<PHONE_NUMBER>9 Bytes, about 167 MB/s
stream load json:         21 seconds loaded<PHONE_NUMBER> Bytes, about 107 MB/s
stream load orc:          59 seconds loaded<PHONE_NUMBER> Bytes, about 17 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230419123329_clickbench_pr_131098.html


Feature: add new backend, Tektronix seqx files

Add the ability to export broadbean Sequence objects as .seqx files as used by Tektronix' AWG70000A series.
Still pending:

[x] Finish outputForSEQXFile function
[ ] Add example usage in the documentation
[ ] Add a test

Note that this PR hinges on https://github.com/QCoDeS/Qcodes/pull/911.
Also fixes #46.

Looks good to me but havn't tested


FIX: proper handling of objects without Object in prototype chain

Seems while using Apollo some objects constructed as input types were not inheriting from Object. Seems this can happen when using Object.create(null), etc.
isNaN fails with that kind of Object, and param.hasOwnProperty is also not available
cheers

Thanks @allain,
A character "f" has been introduced breaking the execution on the utils.js merging and fixing


feature request: ability to login to hauk

I would like to use this with a few friends. Some of us are using hauk directly while others are only using owntracks.
I would like to password protect the hauk instance so it does not use empty password.
Would it be possible to add password or username with password logging to hauk please?

Good point, I'll see what I can do.

This can be close, thanks @tuffnerdstuff 👍🏻


[release/1.6] fix the task setting the runtime path

cherry-pick: https://github.com/containerd/containerd/pull/8208
Added a new commit https://github.com/containerd/containerd/commit/e8840f688ae402bef461963321d63cf9cf6fbb34 that skips TestContainerStartWithAbsRuntimePath if the runtime is v1

/retest

Hi @Iceber could you take a look at the test failure please? Hoping to have your fix into the next containerd 1.6 tag that I have currently have a PR out for - https://github.com/containerd/containerd/pull/8463


Additional checking for uploaded file

Why is_uploaded_file missing in ->moveTo? but specified in https://github.com/zendframework/zend-diactoros/blob/master/src/UploadedFile.php#L135

This repository has been closed and moved to laminas/laminas-diactoros; a new issue has been opened at https://github.com/laminas/laminas-diactoros/issues/18.


fix(k8s-infra): include otel-auth security context

More context: https://signoz-community.slack.com/archives/C01HWQ1R0BC/p1701124166307409?thread_ts=1700184882.205159&cid=C01HWQ1R0BC
Signed-off-by: Prashant Shahi<EMAIL_ADDRESS>

Thanks @prashant-shahi 🙏


Avoid attribute protocol

First batch of my big change (breaking it up a bit for review friendliness).

All this use of AttributeProtocol is using the internal interface rather than the proper vm attribute path.

Codecov Report

Merging #709 into master will decrease coverage by 0.31%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master     #709      +/-   ##
==========================================
- Coverage   50.72%   50.41%   -0.32%
==========================================
  Files          81       81
  Lines       15031    15120      +89
  Branches     3628     3658      +30
==========================================
- Hits         7625     7623       -2
- Misses       5592     5700     +108
+ Partials     1814     1797      -17

Impacted Files
Coverage Δ

src/main.rs
14.1% <0%> (-10.23%)
:arrow_down:

vm/src/frame.rs
49.84% <0%> (-0.11%)
:arrow_down:

vm/src/stdlib/re.rs
27.1% <0%> (-18.09%)
:arrow_down:

vm/src/stdlib/io.rs
30.19% <0%> (-0.65%)
:arrow_down:

vm/src/vm.rs
55.1% <100%> (+4.14%)
:arrow_up:

vm/src/sysmodule.rs
61.66% <100%> (-0.41%)
:arrow_down:

vm/src/obj/objstaticmethod.rs
46.87% <50%> (-7.3%)
:arrow_down:

vm/src/import.rs
48.27% <50%> (-14.99%)
:arrow_down:

vm/src/obj/objclassmethod.rs
50% <55.17%> (-9.26%)
:arrow_down:

... and 7 more

Continue to review full report at Codecov.

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

Review comments addressed. Merging.


random backoff to better avoid collision

To avoid collisions (HTTP code 429) when 2 - 3 runners running tests at the same time.

@guihao-liang - let hold off on merging this until we know how much more reliable the s3 unit tests become with the internal configuration changes.

related to https://github.com/aws/aws-sdk-cpp/issues/1359

reopen when needed.


Is it possible to manage multiple swarm cluster in different datacenter with one portainer

Is your feature request related to a problem? Please describe.
We have lots of vms in different datacenters, we want to know is that possible to use one portainer to manage multiple swarm cluster.
But we couldn't find the answer in https://portainer.readthedocs.io/en/stable/

Hi @yunfandev
Yes, it is possible to manage multiple Docker environments (standalone Docker hosts or Swarm clusters) via a single Portainer instance.
Spin up a Portainer instance, go to Endpoints and add your environments here. You can also assign access policies to different users and teams.


so idk what to do i'm new to this and someone please help me

this

You don't have the python library "customtkinter" installed in your system. Just install the package using package using pip install customtkinter and run the code again.

This is a problem with Python, please uninstall it from control panel and reinstall (make sure to enable "Add to PATH option").

@ghubrakesh There is a problem going on with Python where PIP is not properly installed upon Python installation, this usually gets fixed when reinstalling Python.
FYI, the builder.bat already attempts to install required libraries.

@ghubrakesh There is a problem going on with Python where PIP is not properly installed upon Python installation, this usually gets fixed when reinstalling Python.
FYI, the builder.bat already attempts to install required libraries.

Oh, I am sorry. I thought the customtkinter is not installed beforehand.

You don't have the python library "customtkinter" installed in your system. Just install the package using package using pip install customtkinter and run the code again.

where do i get customkinter

This is a problem with Python, please uninstall it from control panel and reinstall (make sure to enable "Add to PATH option").

it still doesnt work
i deleted it from control panel and when i reinstalled it i downloaded blank grabber and it still dont work-_-

Did it show the same error? Which python version are you using? Also try to type python -m pip install customtkinter in command prompt.

This is a problem with Python, please uninstall it from control panel and reinstall (make sure to enable "Add to PATH option").

it still doesnt work i deleted it from control panel and when i reinstalled it i downloaded blank grabber and it still dont work-_-

Open cmd and write this: pip install customtkinter

You don't have the python library "customtkinter" installed in your system. Just install the package using package using pip install customtkinter and run the code again.

where do i get customkinter

Just run the pip install command I provided and `customtkinter' will get installed in your machine.


runtime-evaluated-decorators not working for the annotations of decorated functions.

Hello :)
First, thank you for the great work on ruff !
I'm trying to enforce TCH003 with quote-annotations = true and I need to exclude some decorated functions.
But it seems that runtime-evaluated-decorators does not work with functions but only with classes. Or I'm doing something wrong...
A minimal reproducing code snippet:
# filename: TCH003/__init__.py

from pathlib import Path
from uuid import UUID

def runtime_evaluated(obj):
    return obj

@runtime_evaluated
class Toto:
    id: UUID

@runtime_evaluated
def toto(
    id: UUID,
    path: Path,
) -> None:
    pass

The corresponding ruff config:
select = ["TCH003"]

[lint.flake8-type-checking]
quote-annotations = true
runtime-evaluated-decorators = ["TCH003.__init__.runtime_evaluated"]

Now, ruff outputs:
$ ruff --version
ruff 0.1.9
$ ruff --config .ruff.toml
TCH003/__init__.py:3:21: TCH003 Move standard library import `pathlib.Path` into a type-checking block
Found 1 error.
No fixes available (1 hidden fix can be enabled with the `--unsafe-fixes` option).

The setting runtime-evaluated-decorators is working for classes since only pathlib.Path is subject to TCH003. But it does not work for the function.
The end goal of this is to make TCH00X rules work with FastAPI routers .
I found this issue but I failed to make it work : https://github.com/astral-sh/ruff/issues/6137.
Thank you for your help :)

I've messed up with same issue around @validate_call from pydantic module
It would be nice if Ruff would take such decorators into account while discovering which types should be quoted
from app import User

from pydantic import validate_call

@validate_call(config={'arbitrary_types_allowed': True})
def test(user: User):
    ...

Converts to
from typing import TYPE_CHECKING

from pydantic import validate_call

if TYPE_CHECKING:
    from app import User

@validate_call(config={'arbitrary_types_allowed': True})
def test(user: 'User'):
     ...

Which cause a NameError from pydantic

It looks like we don't apply these to functions at all right now (but we probably should?).

Hm the documentation for runtime-evaluated-decorators says

Exempt classes decorated with any of the enumerated decorators from needing to be moved into type-checking blocks.

We can definitely consider expanding the scope of the setting, I do not know why it was originally limited.
It's only used at https://github.com/astral-sh/ruff/blob/2895e7d1266c661b36bbeeea31c293bdf5fdf0e7/crates/ruff_linter/src/rules/flake8_type_checking/helpers.rs#L74-L92 right now

PR is up :)

this seems to still be broken in ruff 0.2.2. i've raised a new issue #10089


Logitech g305 reports led that it can't control

Information

ratbagd version (ratbagd --version): 0.13
Distribution: Ubuntu 20.04
Kernel version (ex. uname -srmo): Linux 5.4.0-21-generic x86_64 GNU/Linux

Describe the bug
Piper shows a controllable led that it cannot control, (there is actually a led on the mouse, below the dpi-switch button, but it only stays on shortly after pushing the button)
To Reproduce
Steps to reproduce the behavior:

open piper, go to led tab
config the led
apply
see that it doesn't do anything

Logs
Please start the daemon with verbosity and reproduce the issue.

That led is the DPI indicator. On some devices that led is configurable but I just tested and it does not seem to be the case of the G305. The firmware reports a led, which seems to be incorrect. We could overwrite that as it seems like a firmware bug.


[MAS1.4.11] [Visual Requirements-Welcome] Luminosity Ratio is less than 3:1 for border outline with respective background color for create a new bot configuration control.

Actual Result:
Luminosity Ratio is less than 3:1 for border outline with respective background color for create a new bot configuration control.
Expected Result:
Luminosity Ratio should be greater than or equal to 3:1 for border outline with respective background color for create a new bot configuration control.
Note: This issue exits for throughout application for all interactive controls.
Note 1.: Same issue repro for Emulator,Web chat,channels,View analytics ,continues deployment,Dispatch,Qna maker...etc
Same issue repro for below path
Navigate to the Open bot ->Select the Open bot control->Open bot dialog
Welcome bot Framework emulator ->Navigate to the Sign in with Azure account -> Sign in with Azure account dialog
Welcome Bot Framework Emulator ->Create new bot configuration...etc
Repro Steps:
1.Open the Bot Framework V4 Emulator.
2.Welcome tab screen gets opened
3. Navigate to the create a new bot configuration control
4. Open 'Color Contrast Analyser' application.
5.Observe the issue
User Impact:
Users with low vision or color blindness needs that difference between contrast of text is always high.
Test Environment:
OS Build: Version 1903(OS Build 18362.239)
Browser: Edge
Application: Bot Framework V4 Emulator
Tool: Color contrast Analyzer
MAS Reference Link:
https://microsoft.sharepoint.com/:w:/r/teams/msenable/_layouts/15/Doc.aspx?sourcedoc=%7B6CA3BF14-2635-434A-832C-28DD9A18B9FC%7D&file=MAS%201.4.11%20%E2%80%93%20Non-text%20Contrast.docx&action=default&mobileredirect=true&cid=c3837dea-3bb1-4c5a-9374-19ec384f14c6

Addressed in #1783


net/http: 301 redirects to site with https to  after a 303 redirect from a POST request don't work

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.7.6 darwin/amd64
go version go1.6.4 darwin/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/zhuqiuzhi/work"
GORACE=""
GOROOT="/Users/zhuqiuzhi/.gvm/gos/go1.7.6"
GOTOOLDIR="/Users/zhuqiuzhi/.gvm/gos/go1.7.6/pkg/tool/darwin_amd64"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/dc/q_v8vj2957s1k0krncnl26q40000gn/T/go-build403359302=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/zhuqiuzhi/work"
GORACE=""
GOROOT="/Users/zhuqiuzhi/.gvm/gos/go1.6.4"
GOTOOLDIR="/Users/zhuqiuzhi/.gvm/gos/go1.6.4/pkg/tool/darwin_amd64"
GO15VENDOREXPERIMENT="1"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fno-common"
CXX="clang++"
CGO_ENABLED="1"
What did you do?
package main

import (
	"net/http"
	"net/http/httptest"
	"fmt"
)

func main() {
	ts := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
		w.Header().Set("Location", "http://www.qiniu.com/")
		w.WriteHeader(http.StatusSeeOther)
	}))
	defer ts.Close()

	resp, err := http.Post(ts.URL,"multipart/form-data", nil)
	if err != nil {
		fmt.Printf("Post fail, ", err)
	}
	fmt.Println(resp.StatusCode)
	if resp.StatusCode != http.StatusOK {
		fmt.Printf("Fail follow 301, https\n")
	} else {
		fmt.Println("OK")
	}
}

What did you expect to see?
200
OK

What did you see instead?
301
map[Connection:[keep-alive] Date:[Sun, 13 Aug 2017 12:33:51 GMT] X-Swift-Savetime:[Sun, 13 Aug 2017 12:33:51 GMT] Timing-Allow-Origin:[*] Eagleid:[7529f0cc15026276318052012e] Server:[Tengine] Content-Type:[text/html] Content-Length:[178] Location:[https://www.qiniu.com/] Via:[cache17.l2em21-1[31,301-0,M], cache8.l2em21-1[32,0], cache2.cn26[46,301-0,M], cache4.cn26[47,0]] X-Cache:[MISS TCP_MISS dirn:-2:-2] X-Swift-Cachetime:[0]]
Fail follow 301, https

Hello there @zhuqiuzhi, this bug is fixed on Go1.8 and above, please update. It successfully redirects as you'll see in my screenshot below

We worked on redirects with CLs https://go-review.googlesource.com/c/29852, https://go-review.googlesource.com/c/35633, https://go-review.googlesource.com/c/34657 and numerous other ones during the 1.8 cycle.
I'll close this issue, but please feel free to reopen if the problem persists. It'd also be nice to update since there are lots of bug fixes and great features that have been added since Go1.8; we'll soon be releasing Go1.9, which will mean that supporting Go1.6, Go1.7 is going to be even harder.


refine def: glycine import into mitochondrion

GO:1904983
glycine import into mitochondrion
Definition
The directed movement of glycine across a membrane from cytosol to mitochondrion. PMID:26821380
refine to
GO:1904983    glycine import into mitochondrial matrix
redefine
The process in which a solute is transported from the mitochondrial intermembrane space to the mitochondrial matrix.
See @asangrador  comment from
https://github.com/geneontology/go-annotation/issues/1750#issuecomment-358339508
According to PMID: 28404662:
"Mitochondria possess both outer and inner membranes with differential permeabilities. The outer mitochondrial membrane is permeable to solutes up to ∼5 kDa, while the inner membrane is comparatively impermeable to enable efficient oxidative phosphorylation. Various transporters reside within the inner mitochondrial membrane to overcome its permeability barrier, including transporters required to facilitate the synthesis of heme."
And SLC25 is supposed to be a family of inner mitochondrial membrane transporters. The SLC25A38 member from S. cerevisiae Hem25/YDL119C was detected among these [10930523]. I think that paper is where our annotation came from for MIM.

This is what we currently have :
x import into the mitochondrion:

'folic acid import into mitochondrion'
*'glycine import into mitochondrion'
*'RNA import into mitochondrion'
*'tRNA import into mitochondrion'
*'rRNA import into mitochondrion'
'pyrimidine nucleotide import into mitochondrion'

all except 'pyrimidine nucleotide import into mitochondrion' already have 'target end location' in the logical definition; I dont think these can really end up anywhere else.
Surprisingly, 'pyrimidine nucleotide import into mitochondrion' has target end location 'mitochondrial intermembrane space'. However the paper cited mentions transport to the matrix: PMID: 16194150
So I will
[] adjust the textual definitions for the above terms
[] fix the logical definition for 'pyrimidine nucleotide import into mitochondrion' to make the end location = matrix
@ValWood @deustp01
Let me know if you see a problem with this plan.
Thanks, Pascale


querystrings are not ignored in the router

localhost:8080?foo=bar will break the current router. Oops

Encountering this too :( Checkin it out


Wallet Manager

Issues in #75

Still can not see the Balances

@kimisan ah, i forgot to deploy. try again :) http://tokenmint.io/#/addr/0xe9a7e26bf5c05fe3bae272d4c940bd7158611ce9

@elaineo
Still not see the Token Balances.
Only "Token" text showed but no Balances number.

The only tokens in the system right now are BEC, DEX, and ALMS. Do you have token balances for those three?

@kimisan You can see the contract address for tokens created at the very bottom, right? I can add token balance for user-created tokens, sure :)

@elaineo Yap, i can see the total supply and address of the created token.
But if it can check the token balance will be good.


Flame - create publishable clips

Brief description
Flame converting timeline segments to publishable instances
Description
First step towards publishing workflow in Flame. This feature will allow user to convert selected timeline segments into publishable instances for farther processing. Segments are sequentially renamed on demand and visually tagged.
Testing notes:

Open Flame project from openpype launcher
open your testing sequence
select any clip on timeline
right button click and go to Openpype/Create
Hit accept on Create publishable clips
creator gui will popup
configure hierarchy and name convention and hit accept
clips should be marked in middle with openpypedata marker with json text in comment

Task linked: OP-1536 Flame - Create publishable clips


Add support for bucket name format in GCP provider.

This change fixes #381 by applying NameFormat standard.
Typically, when comes down to resources crossplane paradigm is(was) to default to some "hardcoded" constant string prefix followed by kubernetes object UID value (with the exception of cloud managed resource with special naming limitation).
Proposed change is aimed to add flexibility by using NameFormat value, which allows user to define resource name in addition of supporting existing use cases (prefix/suffix + UID).

NameFormat
Object UID
Resource Name
Description

"" (empty)
"test-uid"
"test-uid"
NameFormat is not provided, default to UID value

"%s"
"test-uid"
"test-uid"
NameFormat is set to replacement value, identical to use-case above

"sql-%s
"test-uid"
"sql-test-uid"
NameFormat is provided as prefix value

"%s-bucket"
"test-uid"
"test-uid-bucket"
NameFormat is provided as suffix value

"foo-bar"
"test-uid"
"foo-bar"
NameFormat is provided without substitution character

Resolves #381
Checklist:

[ ] Documentation has been updated, if necessary.
[ ] Pending release notes updated with breaking and/or notable changes, if necessary.
[ ] Upgrade from previous release is tested and upgrade user guide is updated, if necessary.
[ ] Code generation (make generate) has been run to update object specifications, if necessary.
[ ] CRD manifests generation (make manifests) has been run to update CRD manifests yaml file specifications, if necessary.
[ ] Update dependencies (make vendor) has been run to update Gopkg.lock file, if necessary
[ ] All related commits have been squashed to improve readability.
[ ] Comments have been added or updated based on the standards set in CONTRIBUTING.md

Do you think we'll port this pattern to other providers?


Facebook video download fail

youtube-dl.exe -o llll.mp4 https://www.facebook.com/1387904501238313 --verbose
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['-o', 'llll.mp4', 'https://www.facebook.com/1387904501238313', '--verbose']
[debug] Encodings: locale cp949, fs mbcs, out cp949, pref cp949
[debug] youtube-dl version 2016.10.02
[debug] Python version 3.4.4 - Windows-10-10.0.10586
[debug] exe versions: none
[debug] Proxy map: {}
[generic]<PHONE_NUMBER>238313: Requesting header
WARNING: Could not send HEAD request to https://www.facebook.com/1387904501238313: HTTP Error 404: Not Found
[generic]<PHONE_NUMBER>238313: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 404: Not Found (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpmbxrebnz\build\youtube_dl\extractor\common.py", line 402, in _request_webpage
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpmbxrebnz\build\youtube_dl\YoutubeDL.py", line 2001, in urlopen
File "C:\Python\Python34\lib\urllib\request.py", line 470, in open
File "C:\Python\Python34\lib\urllib\request.py", line 580, in http_response
File "C:\Python\Python34\lib\urllib\request.py", line 508, in error
File "C:\Python\Python34\lib\urllib\request.py", line 442, in _call_chain
File "C:\Python\Python34\lib\urllib\request.py", line 588, in http_error_default

I guess this video requires login?

@yan12125 it appears so.  Browsing that URL.

Thanks you @wamatt.