Sum of column in Detail Dialog - Shown/calculated immediately in a field of Parent Dialog

Hi community,
Does anyone have a better solution regarding sum field in a form, exactly like issue #1021?
A solution where you do not have to save the master dialog before getting the sum-amount into the field in the master dialog? (Not write to SQL table before, preferably)
I have the exact same field, a sum of a column in detail dialog which I want to be calculated and shown immediately in the master dialog after the detail dialog has been closed. But if you take the steps in #1021, then you need to save the master-form before to be able to query directly from the saved table.

Is it possible to call List action in xyzDetailEndpoint to sum up the cached/in-memory column data?
OR could someone point me in the right direction on how to Save a Parent-Dialog automatically upon closing it's Child-Detail-Dialog.
Thanks

Why don't you use the grouping to show the total? like this -

Hi Ramveer,
I want the sum to appear in a specific field on the master form/dialog, not in the detail grid which is inside the form.
The grouping example doesn't cover that part.
I would be happy if I could just let the form save (apply changes) when the details dialog closes. With that, the sum field would calculate and show the value...  as per my picture of the form.
Regards, Biggi
-------- Original message --------From: Ramveer Singh<EMAIL_ADDRESS>Date: 8/16/16  07:47  (GMT+01:00) To: volkanceylan/Serenity<EMAIL_ADDRESS>Cc: biggikalli<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [volkanceylan/Serenity] Sum of column in Detail Dialog - Shown/calculated immediately in a field of Parent Dialog (#1068)
Why don't you use the grouping to show the total? like this -
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/volkanceylan/Serenity","title":"volkanceylan/Serenity","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/volkanceylan/Serenity"}},"updates":{"snippets":[{"icon":"PERSON","message":"@ramveersgh in #1068: Why don't you use the grouping to show the total? like this -\r\n\r\n"}],"action":{"name":"View Issue","url":"https://github.com/volkanceylan/Serenity/issues/1068#issuecomment-240007050"}}}

Here is a sample for you for OrderDialog constructor:
    @Serenity.Decorators.registerClass()
    export class OrderDialog extends Serenity.EntityDialog<OrderRow, any> {
        constructor() {
            super();

            (this.form.DetailList.view as any).onDataChanged.subscribe(() => {
                var total = 0;
                for (var k of this.form.DetailList.getItems()) {
                    total += k.LineTotal || 0;
                }
                Q.notifySuccess(total.toString());
            });
        }

Replace Q.notifySuccess with form.MyField.value = total;

@volkanceylan, thank you.
This code worked like a charm, just like you yourself always do :)

Hi! I'm tying this snipet in my Form and I get the Q.notifySuccess message upon openning the Master form but not after adding new records on the Detail Grid.
Any clues on this?

I just found out that I add not inherited the Dialog from GridEditorDialog. After changing the grid updated fine and the subscribe() function also worked!

Here is a sample for you for OrderDialog constructor:
    @Serenity.Decorators.registerClass()
    export class OrderDialog extends Serenity.EntityDialog<OrderRow, any> {
        constructor() {
            super();

            (this.form.DetailList.view as any).onDataChanged.subscribe(() => {
                var total = 0;
                for (var k of this.form.DetailList.getItems()) {
                    total += k.LineTotal || 0;
                }
                Q.notifySuccess(total.toString());
            });
        }

Replace Q.notifySuccess with form.MyField.value = total;

I've been dealing with this issue for about a week now. The code from @volkanceylan works amazing. The only problem is when deleting. I did some research and it seems that instead of using onDataChanged you can use onRowCountChanged. For my project it works amazing and as it should.

    @Serenity.Decorators.registerClass()
    export class OrderDialog extends Serenity.EntityDialog<OrderRow, any> {
        constructor() {
            super();

            (this.form.DetailList.view as any).onRowCountChanged.subscribe(() => {
                var total = 0;
                for (var k of this.form.DetailList.getItems()) {
                    total += k.LineTotal || 0;
                }
                Q.notifySuccess(total.toString());
            });
        }

This should help #3818 / #3769 and so on with delete issue. Many thanks to @volkanceylan for an amazing product and the help provided.

@ckrassas but how do u use onRowCountChange?
this:
(this.form.MailingList.view as any).onRowCountChange.subscribe(() => {
                this.UpdateEmailCount();
            });

fails, says there is not onRowCountChange

@ckrassas but how do u use onRowCountChange?
this:
(this.form.MailingList.view as any).onRowCountChange.subscribe(() => {
                this.UpdateEmailCount();
            });

fails, says there is no onRowCountChange:

Message: TypeError: _this.form.MailingList.view.onSubmit.subscribe is not a function

Sorry but it is OnRowCountChanged. I misspelled it when I was writing my reply. Hope that helps. I updated my response also.

@ckrassas
same with changed :(

Message: TypeError: _this.form.MailingList.view.OnRowCountChanged is undefined

My Editor  is:

export class MailingListEditor extends Common.GridEditorBase

I ont understand what is the diference here :(

@ga5tan This is not to placed in the editor. This is to be used within your Dialog. It is there to see changes made to the grid.

@ckrassas - yeah, I do it in EntityDialog and it's working:

(this.form.MailingList.view as any).onDataChanged.subscribe(() => {
this.UpdateEmailCount();
});

but your :

        (this.form.MailingList.view as any).OnRowCountChanged.subscribe(() => {
            this.UpdateEmailCount();
        });

wont work :(

@ga5tan Please check the version that you are using. I am using 3.7.7 and Also make sure that your grid is updating once a change is made to reflect the changes on the grid. Another this is to make sure this is placed inside the constructor(). Not sure why it is not working if you editor works correctly.

@ckrassas somehow I ended up with spelling mistake OnDataChanged instead of onDataChanged (leading O should be lowercase)
now it works! thx

Message: Uncaught TypeError: Cannot read property 'view' of undefined
Please Help Me . I geeting Error
(this.form.DetailList.view as any).onDataChanged.subscribe(() => {
var totaldr = 0;
for (var k of this.form.DetailList.getItems()) {
totaldr += k.Debit || 0;
}
Q.notifySuccess(totaldr.toString());
});

@ckrassas
Thanks for posting this answer - it is an extremely important detail - the intellisense does not show this as a function but it does work!
I have a button in my dialog that is dependent on an item in the DetailsList - Items present button visible, no items and button must be disabled.
I also have in the detailseditor a need to calculate the totals for that grid as items are added and removed.
With your direction I used the following code to achieve both requirements.
Dialog
            (this.form.ItemsList.view as any).onRowCountChanged.subscribe(() => {
                this.submitHidden = (this.form.ItemsList.rowCount() < 1);
                this.toggleSubmitVisibility(this.submitHidden);
            });

DetailEditor
            (this.view as any).onRowCountChanged.subscribe(() => {
                var total = 0;
                for (var k of this.getItems()) {
                    total += k.LineTotal || 0;
                }
	});

@ramveersgh       > Why don't you use the grouping to show the total? like this -

Hi Ramveer,     I'm trying this, but i didn't get it .please post clearly


Invariant violation on React Native

Awesome project! Does this work with React Native? I tried installing it but get this error when calling Avatar:

Invariant Violation: View config getter callback for component rect must be a function (received undefined). Make sure to start component names with a capital letter.

I'm using the example code.
import Avatar from "boring-avatars";

<Avatar
  size={40}
  name="Maria Mitchell"
  variant="marble"
  colors={["#92A1C6", "#146A7C", "#F0AB3D", "#C271B4", "#C20D90"]}
/>;

"react": "17.0.1",
"react-native": "0.64.1",

It does not. I'll try to propose some implementation tomorrow 😄

@kacgrzes , hi! Any news?

Hello, I'm sorry, but I didn't have time to implement it. 😞 but I'll try to propose something this week 😄

Hey, any update on this? Is react-native support part of the road map? Thanks.

Hello! Please, let me know if I can propose an implementation for the same. Thanks

@joebernard @kacgrzes @RohovDmytro @VirenMohindra @mayank-budhiraja
Hi everyone, I've made a react native library for boring-avatars. You can check it out here. Enjoy😄

@luhart This is great! Thanks for putting in the effort. I am wondering what your thoughts are on migrating your changes into this repo so there is only one to maintain. Is that possible or do you plan to keep the fork?

Hi everyone 👋 , theres is no plan to make the library work on react-native. We recommend using other implementations, like the one @luhart made 👍 .


[Bug]: Obsidian Crashing due to db folder

Contact Details
No response
What happened?
A bug happened!
The database folder plugin doesn't seem to handle concurrent rendering of multiple databases when i believe it is called simultaneously.
hat i did was i opened one database on one pane and other on other pane, whole obsidian freezes as a result. I tried initial debugging steps and they are : -

I tried launching obsidian from cli to view any warnings errors etc. (nothing was found, everything was normal)
I tried using dev tools, everything was fine there as well.

using dev tags gave me indication that when i open other database my whole dev panel give a simple msg, the dev tools disconnected and my  obsidian is pure white, nothing can be seen except a freezed dev panel with that msg.

What platform were you using?
Desktop
Version of the plugin
latest
Relevant log output
-

Are you using the latest version of the plugin in your Obsidian vault?

[X] I have verified that I am on the latest version

Are you check if there is a similar issue?

[X] I have verified that there are not similar issues

My memory and cpu usage is not peaked the entire time as well.

My memory and cpu usage didn't peaked and was stable the whole time.

I just came here looking to report the same thing, and its very strange that two people would get exactly the same issue within 15 minutes of each other...

I will try this use case with a workspace of multiple dbs. Thanks for the report.
Once its reproduced I will give it priority

My memory and cpu usage didn't peaked and was stable the whole time.

I thought my outdated hardware might be the issue, but good to know that my hardware alone is not the issue.

I just came here looking to report the same thing, and its very strange that two people would get exactly the same issue within 15 minutes of each other...

I thought my outdated hardware might be the issue, but good to know that my hardware alone is not the issue.

Glad to know that, do let me know if you need further assistance in replicating what we just faced.

I will try this use case with a workspace of multiple dbs. Thanks for the report.
Once its reproduced I will give it priority

Glad to know that, do let me know if you need further assistance in replicating what we just faced.

I cannot reproduce the problem with multiple databases on split views (5)
@rohanbatraio could you share a mock vault?

I have figured out what the problem is, the problem occurs when the Sliding
Panes (Andy Matuschak Mode) Obsidian Plugin is activated.
El jue, 6 oct 2022 a las 13:20, Rafael Gómez Bermejo (<
@.***>) escribió:

I cannot reproduce the problem with multiple databases on split views (5)
@rohanbatraio https://github.com/rohanbatraio could you share a mock
vault?
—
Reply to this email directly, view it on GitHub
https://github.com/RafaelGB/obsidian-db-folder/issues/469#issuecomment-1269845673,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOZFSFLH66QKY5KBKMY63LWB2YZDANCNFSM6AAAAAAQ4WCM74
.
You are receiving this because you commented.Message ID:
@.***>

--
Es mi trabajo no estar nunca satisfecho
Twitter: ajcastillo1 http://twitter.com/ajcastillo1
Flickr: http://www.flickr.com/photos/ajcastillo1/

there is a problem with stack tabs. A new feature of Obsidian 0.16.3. I am Sure it's related

there is a problem with stack tabs. A new feature of Obsidian 0.16.3. I am Sure it's related

You mean the db folder code base is updated keeping Obsidian 0.16.3 in mind right? and that's what causing the issue, Because i am still on non catalyst v15.9 of Obsidian and still facing the issue.

I have figured out what the problem is, the problem occurs when the Sliding Panes (Andy Matuschak Mode) Obsidian Plugin is activated. El jue, 6 oct 2022 a las 13:20, Rafael Gómez Bermejo (< @.>) escribió:
…
I cannot reproduce the problem with multiple databases on split views (5) @rohanbatraio https://github.com/rohanbatraio could you share a mock vault? — Reply to this email directly, view it on GitHub <#469 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAOZFSFLH66QKY5KBKMY63LWB2YZDANCNFSM6AAAAAAQ4WCM74 . You are receiving this because you commented.Message ID: @.>
-- Es mi trabajo no estar nunca satisfecho Twitter: ajcastillo1 http://twitter.com/ajcastillo1 Flickr: http://www.flickr.com/photos/ajcastillo1/

Seems just right, because that's what common in all the people facing the freezing issue.

I have figured out what the problem is, the problem occurs when the Sliding Panes (Andy Matuschak Mode) Obsidian Plugin is activated. El jue, 6 oct 2022 a las 13:20, Rafael Gómez Bermejo (< @.>) escribió:
…
I cannot reproduce the problem with multiple databases on split views (5) @rohanbatraio https://github.com/rohanbatraio could you share a mock vault? — Reply to this email directly, view it on GitHub <#469 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAOZFSFLH66QKY5KBKMY63LWB2YZDANCNFSM6AAAAAAQ4WCM74 . You are receiving this because you commented.Message ID<EMAIL_ADDRESS>-- Es mi trabajo no estar nunca satisfecho Twitter: ajcastillo1 http://twitter.com/ajcastillo1 Flickr: http://www.flickr.com/photos/ajcastillo1/

Seems just right, because that's what common in all the people facing the freezing issue.

I have verified the issue is resolved when Sliding panes plugin is disabled, that's the root cause of all conflicting. @RafaelGB You should check how sliding panes is interfering with the rendering of database tables. I would update the title of this bug report in the meantime.

rolling back some changes....

I have figured out what the problem is, the problem occurs when the Sliding Panes (Andy Matuschak Mode) Obsidian Plugin is activated. El jue, 6 oct 2022 a las 13:20, Rafael Gómez Bermejo (< @.>) escribió:
…
I cannot reproduce the problem with multiple databases on split views (5) @rohanbatraio https://github.com/rohanbatraio could you share a mock vault? — Reply to this email directly, view it on GitHub <#469 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAOZFSFLH66QKY5KBKMY63LWB2YZDANCNFSM6AAAAAAQ4WCM74 . You are receiving this because you commented.Message ID: @.>
-- Es mi trabajo no estar nunca satisfecho Twitter: ajcastillo1 http://twitter.com/ajcastillo1 Flickr: http://www.flickr.com/photos/ajcastillo1/

Hello there @ajcastillo , I just tried the same on a mock vault, and Sliding panes are working just fine in my case, can you please share some more insights?

This task will be done here = https://github.com/RafaelGB/obsidian-db-folder/issues/435


Add as_json/to_json methods

also removes some code duplication
todo:

add base_collection to remove code duplication in collection
add a method to get hash with qualtrics keys (not ruby style keys)
reorganize files

Added some small changes so it works with active-support. Tested thanks to @MatUrbanski!


Customize an automation generated through SAP UI5 Recorder

Tutorials: https://developers.sap.com/tutorials/irpa-customize-s4hana-recorder-automation.html
Recording login doesn't work in current SAP S/4HANA Cloud trial environment.
Because, when open the S/4HANA Cloud Trial (URL I set in step 7 Creating Automation (Trial Account) - sub step 5), browser popups dialog to select certificate first but not the ID/PWD login page.
In step 9 Test the automation, after login successfully, the "Manage Purchase Order" is opening in another tab but not the tab that we login. Then I received "Timeout waiting for: application a_managePurchaseOrders" error but the other time it work. Not sure how to deal with this.
Please see screen recording in below.

Dear Nick Yang,
In your case, in order to make the recording login work in your SAP S/4HANA Cloud trial environment, please do the following:

capture in UI automation the page with the popup dialog that asks you to select a certificate. Once the page is declared, declare an element (I see you click on the cancel button so declare that button)
in your automation add a screen switch activity with in the first branch the page with the popup dialog (where you will add a click activity on the cancel button for example).
the second branch will automatically set to open the S/4HANA Cloud Trial with the ID/PWD login page.

Please let me know if this works.
All the best,
Céline


[request] commitizen config files

Hello again!
Please add commitizen config files

cz.toml
cz.json
.cz.json
cz.yaml
.cz.yaml

Looks like they maintain a logo at their github org: https://github.com/commitizen-tools
Thanks as always for maintaining this delightful package.

Thanks mate. 😀
I've added it, but it'll be a while until you see it land in a release. Basically, I've started work on a huge-ass migration away from IcoMoon to a FontForge-powered pipeline (which I should really resume work on…). Assuming it turns out, I'll move the config.cson from the Atom package repo to this one, and use release assets and an NPM package to host the generated font-files.
That way, I can serve the same thing to both Atom and VS Code automatically, and eliminate the stress of preparing a release manually (which involves so many steps, I have to triple-check everything before publishing anything).


added feedback section and form

see #51 #50

@rwblair no worries :-)
--> here's another one for you to review


POSTing invalid JSON causes the request to hang

Consider the following service.
import ballerina/http;

map<json> ordersMap = {};

@http:ServiceConfig { basePath: "/ordermgt" }
service orderMgt on new http:Listener(9090) {

    @http:ResourceConfig {
        methods: ["POST"],
        consumes: ["application/json"],
        produces: ["application/json"],
        path: "/order"
    }
    resource function addOrder(http:Caller caller, http:Request req) returns error? {
        json orderReq = check req.getJsonPayload();
        string orderId = orderReq.Order.ID.toString();
        ordersMap[orderId] = orderReq;

        json payload = { status: "Order Created.", orderId: orderId };

        http:Response response = new;

        response.setJsonPayload(untaint payload);
        response.statusCode = 201;
        response.setHeader("Location",
            "http://localhost:9090/ordermgt/order/" + orderId);

        _ = caller->respond(response);
        return;
    }
}

Now send an invalid JSON to this service as follows:
curl -d '{name:\"Pubudu\"}' -X POST http://localhost:9090/ordermgt/order -v -H "Content-Type: application/json"

For the moment, handle the error rather than using check operation. The implementation of resource returning functionality is scheduled for coming release.
Affected version
0.990.0

This is fixed with https://github.com/ballerina-platform/ballerina-lang/pull/12614 and https://github.com/ballerina-platform/ballerina-lang/pull/12626


fix: add support for mssql_managed_instance

addressing issue: #2063
infracost currently supports the "deprecated" sql_managed_instance -> add support for new mssql_managed_instance
any feedback appreciated

@tim775 I think this is ready for review - I would appreciate some feedback / some guidance on what else is left to do here

Thanks for this @tim775 , I will make these changes including the example usage file :)

Hey @tim775 , I'm still pretty new to this so I'd appreciate some help here - running make test_azure with the above commands is failing and the golden test file produced is showing all $0.00's.
I've followed the steps in CONTRIBUTING.md including the azure-credentials section but still no luck. I'm unsure whether it's a problem with my code changes or my local setup at this point - maybe you can help?
Apologies, still getting into this :laughing:

@grixxie, no worries I actually ended up merging this with the intention of updating the golde file test after and ran into the same issue you're seeing now.  It looks like the filters got messed up somehow, but it was a quick fix.


Vista support of x86 Windows build

Originally reported by @Vangelis66 here
There is an error in the "supported windows versions" given in https://www.python.org/downloads/windows
See: https://bugs.python.org/issue38704
The correct versions are:
WinXP SP3 => EOLed by 3.4.x
Vista SP2 => EOLed by 3.7.x
Win7 SP1 => EOLed by 3.8.x
3.9.x and higher require at least Win8.1
So we should build windows x86 using py3.7

Originally reported by @Vangelis66 here
(snipped)
The correct versions are:

Being pedantic, my original report of CPython version support among Windows OSes
was first posted in this very repo last February 18th:
https://github.com/yt-dlp/yt-dlp/pull/93#issuecomment-780914024 😉  😆
If there's any interest, third party (non-PSF) installers/packages for 3.7.10 can be found here 👍 ...
Thanks, kindest regards 😄 .

Being pedantic, my original report of CPython version support among Windows OSes
was first posted in this very repo last February 18th:

Yes, I do remember that. However, it is only normal that when a comment and the official sources differ, I go by the official one. Hence why it is important for me to note down that it is in-fact the official source that is incorrect in this case

it is only normal that when a comment and the official sources differ, I go by the official one.

... Understood 😄 ; with me, I tend to stick to the "if it's coming straight from the horse's mouth" adage 😉 ; ie. actual Vista user vs the PSF, who never tested their binaries on Vista... 👎
Additionally, you'll probably find that several other "official" documentations remain un-updated/flawed/misleading, case-in-point a recent "upstream" issue I had to comment on:
https://github.com/ytdl-org/youtube-dl/issues/29222
Once again, many thanks for your coding efforts! 👍


[DOC] Data Prepper OpenSearch sink add aws serverless options field

What do you want to do?

[ ] Request a change to existing documentation
[X] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request. Provide a summary of the request and all versions that are affected.
Adding a new serverless options field in the aws section of the opensearch sink. See below for example
  1 version: "2"
  2 log-pipeline:
  3   source:
  4     http:
  5       path: "/test/path"
  6   processor:
  7     - grok:
  8         match:
  9           log: [ "%{COMMONAPACHELOG}" ]
 10     - date:
 11         from_time_received: true
 12         destination: "@timestamp"
 13   sink:
 14     - opensearch:
 15         hosts: [ "https://cw4c***************.us-east-1.aoss.amazonaws.com" ]
 16         index: "test_index"
 17         aws:
 18           sts_role_arn: "arn:aws:iam::*************:role/****************Role"
 19           region: "us-east-1"
 20           serverless: true
 21           serverless_options:
 22             network_policy_name: "new-adi-test-collection"
 23             collection_name: "adi-test-collection"
 24             vpce_id: "vpce-**********"

What other resources are available? Provide links to related issues, POCs, steps for testing, etc.
See https://github.com/opensearch-project/data-prepper/pull/3510 for the pull request with the code.

@Naarcha-AWS - Does this PR close this doc issue? And I assume this is for version 2.6?

Yes, we can close this one. It was addressed by #5663


Added the Metal framework; rewrote CluProcessor with Metal

@kwalcock : can you please take a look at why this PR fails?

This test fails with "sbt test":
[info] ConstEmbeddingsGlove
[info] - should not be empty *** FAILED ***
[info]   300 was not equal to 5 (TestConstEmbeddingsGlove.scala:14)
But it passes when run individually...
@kwalcock: can you please take a look?

Yes, when this new test run is complete.

Unless something can be done about it, tests are run as if they are part of one big program.  TestConstEmbeddingsGlove fails because by the time it tries to make its singleton, TestLiblinearClassifier has already arranged for the singleton and that's why the dimension is 300 instead of 5.  There are lots of options for parallelism in testing, but I haven't yet found a way to tell sbt to test one file separately.  One workaround is to allow for unsetting the singleton so something else running serially can later set it again.

I see. Very nice catch! Thanks!
Given how dumb this test is, maybe the simplest option is to simply disable it?

We could add functionality for unsetting it. But then another test that may run in parallel and that expects the big file would crash...

@kwalcock : i disabled that test. Thanks!

@kwalcock : I merged your branch in here, and re-enabled that test. Let's see.


Updated dev RELEASE and KUBE_GIT_VERSION_FILE for 1.19-6

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/hold
until after 1.19-5 release

Never mind. We're going forth with the original plan and PRs for 1.19-5, which include these changes


Handle nullable join-condition in non-grouping join aggregations

Codecov Report

Merging #5277 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #5277      +/-   ##
============================================
+ Coverage     82.63%   82.63%   +<.01%
+ Complexity    11025    11020       -5
============================================
  Files          1327     1327
  Lines         44820    44824       +4
  Branches       5620     5621       +1
============================================
+ Hits          37035    37042       +7
+ Misses         5177     5169       -8
- Partials       2608     2613       +5

Impacted Files
Coverage Δ
Complexity Δ

...anner/consumer/MultiSourceAggregationConsumer.java
96.29% <100%> (+0.29%)
12 <0> (+1)
:arrow_up:

...n/java/io/crate/exceptions/JobKilledException.java
0% <0%> (-66.67%)
0% <0%> (-1%)

...in/java/io/crate/data/CollectingBatchConsumer.java
69.23% <0%> (-15.39%)
4% <0%> (-1%)

...crate/executor/transport/TransportShardAction.java
77.5% <0%> (-12.5%)
8% <0%> (-1%)

...a/io/crate/operation/join/NestedLoopOperation.java
82.85% <0%> (-11.43%)
14% <0%> (-1%)

.../java/io/crate/analyze/symbol/ParameterSymbol.java
87.5% <0%> (-6.25%)
6% <0%> (-1%)

...src/main/java/io/crate/jobs/NestedLoopContext.java
66.66% <0%> (-5.56%)
7% <0%> (-1%)

...io/crate/protocols/postgres/types/VarCharType.java
91.66% <0%> (-4.17%)
10% <0%> (ø)

...or/transport/distributed/DistributingConsumer.java
76.04% <0%> (-3.13%)
20% <0%> (ø)

...c/main/java/io/crate/exceptions/SQLExceptions.java
67.01% <0%> (-3.1%)
33% <0%> (-2%)

... and 14 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 bfd7737...8ac5601. Read the comment docs.


Switching activities with the Harmony Hub component is very slow

I'm running the docker version of the current release of Home Assistant, and I'm experiencing very slow switching of activities with the Harmony Hub component - on the order of 5-10 seconds.
I'd expect the problem to lie with the underlying pyharmony library, which I see has a number of forks under heavy development. It'd be great to know what others' experiences are and if anyone has already narrowed down where the problem is.

I'm sick and I couldn't sleep so I investigated this a bit.
I had thought that the delays were happening with any device, but more experimentation showed that the really long delay that motivated me to file this bug only happened when turning an Apple TV 4 on. (Not when turning it off, oddly enough.) I experimented a bit with harmonyhub-cli, and my best guess is that the Harmony Hub is sending the unpair command at some point while starting the activity, because that command seems to be really slow compared to all of the other commands.
The upshot is that it's possible to speed up the process of starting an activity with an Apple TV 4 in it pretty easily. Just start the activity as normal, but do it as part of a sequence that also sends a device command to set the TV to the correct input. The input switch happens much faster, so you'll be able to start interacting with your Apple TV almost immediately. Here's the code I used:
script:
  start_appletv_activity:
    sequence:
      - service: remote.turn_on
        entity_id: remote.harmony_tv
        data:
          activity: (Watch Apple TV activity goes here.)
      - service: remote.send_command
        entity_id: remote.harmony_tv
        data:
          device: (The device ID for your TV goes here.)
          command: Input1

I'll close this, because it doesn't seem to be a Home Assistant problem at all, but hopefully this will show up in Google and help out anyone else who runs into this problem.


Support Metadata-Version in package.metadata.pyo3-pack

First off, I have to use artifactory which does not support Metadata-Version higher than 2.0, so this strange feature request stems from that.
In the Python metadata section the docs state "You can use other fields from the python core metadata in the [package.metadata.pyo3-pack] section (...)". I was somehow expecting I could override/set the Metadata-Version in this way by declaring something like:
[package.metadata.pyo3-pack]
metadata-version = "2.0"

Does this make sense to support? If you point me to the right direction I can try to do a PR myself.

Are you sure that artifactory supports Metadata-Version up to 2.0? The 2.0 was draft was discarded and 2.1 is essentially 1.2 with some small additions. The specification of Metadata-Version says :

Version of the file format; legal values are “1.0”, “1.1”, “1.2” and “2.1”.

From what I could find with google, there's also this artifactory bug for metadata version 2.1 which seems to have been resolved.

First of all, a big thank you for looking into this issue. I have to confess I am not sure of the 2.0 support, I have assumed it to be the case after downloading some wheels from our repo and wheel unpacking them. They all had metadata 2.0.
Further, unfortunately, our company size somehow means that updates are slow and I have limited leverage in speeding up updating artifactory - that's why I am interested in working around the problem although I know it's probably not the optimal way. I currently unpack the pyo3-built wheel, replace the string 2.1 with 2.0 (yes, I feel dirty), pack it and upload it. My selfish thought was that by making the version user-configurable I would be able to work around the problem more easily...

I'm sorry, but I don't want to support bugs in old software that require violating the metadata PEPs. IMHO dirty patching and waiting for a software update is the most reasonable way to deal with a case like that.

Absolutely reasonable. Thanks for looking into that anyways!


Add ToS to the connext bridge sites

@alexblanq here is terms of service that we need to get up at the bottom of our bridge pages.
Connext_-_Terms_of_Service_06.2.2021_1.docx

example: bottom of each page https://synapseprotocol.com
links to https://explorer.synapseprotocol.com/terms


Fix copy-paste errors in generated docs

I happened across this when trying to implement my own middleware that looked a lot like AwsMiddleware. It seems some IDE type hints were copied over along with the code itself, so I removes those and fixed a s/,/;/, and the example code works again.
The reason these weren't caused by CI/tests btw is that this code block is marked as ignored, presumably because it'd require a bunch of dependencies to compile.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

It actually looks like the default stack has changed a bit since this was last written. Should it be updated to something more like
https://github.com/awslabs/smithy-rs/blob/9708aa830730956a5a5a1500c37a6f95fffc9bfb/aws/rust-runtime/aws-inlineable/src/middleware.rs#L21-L33

Yeah, it should match https://github.com/awslabs/smithy-rs/blob/9708aa830730956a5a5a1500c37a6f95fffc9bfb/aws/rust-runtime/aws-inlineable/src/middleware.rs#L57-L73

Thinking some more about this, is there a reason the AWS middleware definition is in aws-inlineable rather than being a public type in one of the smithy crates? That would certainly make it easier to re-use.

I think there are two reasons the default middleware is currently in aws-inlineable:
1. Both `aws-http` and `aws-sig-auth` declare middleware layers for the stack, and the generated service crate is the only thing that depends on both of these and can consume all the middleware layers. To have default middleware in the runtime crates would require introducing another crate since `aws-config` is an optional dependency for customers.

Hmm, yeah, I suppose it would need to be its own crate. That said, there are also a fair number of aws-* crates, so is having one more actually a problem? It's rare that users will actually take dependencies on these manually, but it would give them the option. There may also be other things from aws-inlineable that could be moved to such a crate, I'm not sure?

I'm not opposed to creating a new crate, but I think it probably deserves a RFC to lay out what it should be named and what should go into it. @rcoh probably has some strong opinions in this area.

it isn't public because it's really a property of an individual service—having a public stack that we claim is "the stack" is a little misleading. There might be a "probably right" stack, but ultimately I expect that as things evolve different services may have different requirements

that said, I'm not opposed to a service builder we expose in aws-http or something like that that does wire up the "normal" stuff


In addition, refactored course data in a couple student views to be in vuex.

Coverage remained the same at 61.883% when pulling<PHONE_NUMBER>36014b51c59177c6938ee39f1cb227 on cumulative-ui-fixes into 79b23d81db27e246be60f11654ee768cd3cb15d9 on master.


current page forgotten when refreshing page

Describe the bug
When refreshing the page (F5) it is not remembered what page you are on, but it defaults to http://localhost:9880/files
All pages seem to have this.
Expected behaviour
When refreshing the page, return/remain on the same page.
General Information
Running on the master-testnet  docker image.

Fixed by #424


Making public key error more descriptive and moving the errors themselves over to i18n

The error texts were accidentally left out in the initial transfer for i18n.

This change is

:+1:


Add built-in extensions

📝 Provide a description of the new feature
Like lyrics+ etc.
➕ Additional Information
No response

@dinckelman What do you mean?


simulation of filename template issue

$ ytube --version
2011.11.23
#pgm aliased as ytube
I was trying to check what the output template should produce but I'm not sure why it doesn't output the filename
$ytube -s -o '%(stitle)s_%(id)s.%(ext)s' 'http://www.youtube.com/user/classxteam#g/p'
[youtube] Setting language
[youtube] user classxteam: Downloading video ids from 1 to 51
[youtube] user classxteam: Downloading video ids from 51 to 101
[youtube] user classxteam: Collected 83 video ids (downloading 83 of them)
[youtube] KV4-2yDY7ZE: Downloading video webpage
[youtube] KV4-2yDY7ZE: Downloading video info webpage
[youtube] KV4-2yDY7ZE: Extracting video information
[youtube] Kul-oCw4hj8: Downloading video webpage
Isn't it supposed to do 'title_KV4-2yDY7ZE.mp4' of the playlist to the std output? title should be replaced with whatever title is returned by youtube.

-o just modifies the outputted filename, not the stdout. It couldn't possibly do so since at the moment when the message "Downloading video webpage" is generated, we haven't downloaded the video webpage containing the title yet.
You want the --get-filename option:
$ youtube-dl  -s -o '%(stitle)s_%(id)s.%(ext)s' 'http://www.youtube.com/user/classxteam#g/p' --get-filename
00017_KV4-2yDY7ZE.mp4
ClassX_Mobile_Kul-oCw4hj8.mp4
Information_Theory_A_Lecture_15_Part_6_X38RctVDofM.mp4
Information_Theory_A_Lecture_15_Part_5_Z-WMe7eiyDo.mp4
Information_Theory_A_Lecture_15_Part_4_ZtuQD2HFWNI.mp4
Information_Theory_A_Lecture_15_Part_3_fqwBlUFqE4U.mp4
...

got it. Thanks.


AVIF images lacking file extension in URL are not found

The following image will give an error when passed through the proxy.
https://vcdn.polarismedia.no/77ad7bce-c115-407d-b82d-d42dd0cf3707?fit=crop&w=400&h=200&q=80&tight=false
As such:
https://next-image-transform.fotballfeber.com/image/https://vcdn.polarismedia.no/77ad7bce-c115-407d-b82d-d42dd0cf3707?fit=crop&w=400&h=200&q=80&tight=false
Source image is unreachable
Logs:
Completed in 63.353636ms /pr:sharp/resize:fill:0:0/q:75/plain/https://vcdn.polarismedia.no/77ad7bce-c115-407d-b82d-d42dd0cf3707@webp  request_id=YX4uHc_LnohczlpnjQnvg method=GET status=404 client_ip=<IP_ADDRESS> error="Can't download source image: Status: 400; {\"code\":400,\"message\":\"Invalid signature\"}"

I tried setting the env var IMGPROXY_ENABLE_AVIF_DETECTION=true as per the imgproxy docs, but it does not work.
I believe the issue is in this line: https://github.com/coollabsio/next-image-transformation/blob/main/index.js#L36
When @webp is set in the url env vars such as IMGPROXY_ENABLE_WEBP_DETECTION and IMGPROXY_ENABLE_AVIF_DETECTION will not work.

It seems that only browsers can access this image. When I try with curl, I encounter the same error.
What type of CDN are you using to serve that image? Is this your image or just a random one?

It's just a random image, unfortunately, so I don't know which CDN they are using.

I will investigate that exact image.
However, I fixed the avif detection and removed the webp forced extension.


config: let the cert-allowed-cn same with other components

Signed-off-by: nolouch<EMAIL_ADDRESS>
What problem does this PR solve?
TiDB, TiKV Supports multiple CNs, but we depend on etcd, let the config be consisted firstly.
What is changed and how it works?
changed config items.
Check List
Tests

Manual test

Codecov Report

Merging #2251 into master will decrease coverage by 0.03%.
The diff coverage is 57.89%.

@@            Coverage Diff             @@
##           master    #2251      +/-   ##
==========================================
- Coverage   76.78%   76.75%   -0.04%
==========================================
  Files         197      197
  Lines       21292    21308      +16
==========================================
+ Hits        16349    16354       +5
- Misses       3744     3750       +6
- Partials     1199     1204       +5

Impacted Files
Coverage Δ

pkg/grpcutil/grpcutil.go
73.68% <57.14%> (-10.32%)
:arrow_down:

server/config/config.go
79.91% <60.00%> (-0.32%)
:arrow_down:

pkg/etcdutil/etcdutil.go
82.60% <0.00%> (-4.35%)
:arrow_down:

server/schedulers/random_merge.go
61.19% <0.00%> (-2.99%)
:arrow_down:

pkg/mock/mockhbstream/mockhbstream.go
89.23% <0.00%> (-1.54%)
:arrow_down:

server/tso/tso.go
81.02% <0.00%> (-1.46%)
:arrow_down:

server/server.go
80.56% <0.00%> (-1.08%)
:arrow_down:

server/core/storage.go
73.73% <0.00%> (-1.02%)
:arrow_down:

server/config/option.go
90.04% <0.00%> (-0.87%)
:arrow_down:

server/member/leader.go
73.94% <0.00%> (-0.85%)
:arrow_down:

... and 9 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 fc0953f...93b0db9. Read the comment docs.

/merge

/merge

will break the compatibility between https://github.com/pingcap/pd/pull/2241 and  https://github.com/pingcap/pd/pull/2251.


setAttribute is not Function

lyric_selected.setAttribute is not a function at HTMLAudioElement.
var lrc = new Lyrics(LRC_text);
document.getElementById('audio_player').addEventListener('timeupdate', function(){
	//Unhighlight all the lyrics
	var lyric_selected = lyrics_container.querySelectorAll('[selected]');
	for (var i = 0; i < lyric_selected.length; i++) {
		lyric_selected[i].removeAttribute('selected');
	}

	//Get the lyric to highlight
	var lyric_selected = lrc.select(this.currentTime);

	//Highlight the lyric
	if (lyric_selected != undefined && lyric_selected >= 0) {
		lyric_selected.setAttribute('selected', 'selected');
	}
});

i got error when lyric_selected.setAttribute('selected','selected'); i dont know why, but the error is setAttribute is not function

Thank you for your suggestion.
I've updated the documentation with the sample code updated.

Thank you for your response Mr Frank, can you show me the html form  lyrics_container
because i have changes for that
ex :

and script like this document.getElementById("subtitles").querySelectorAll('p')[lyric_selected].setAttribute('selected', 'selected');``
and the result still same Uncaught TypeError: Cannot read property 'setAttribute' of undefined
Thank you Mr Frank

The documentation has been updated just now, and added the code for adding
 elements to the lyrics container DOM, each
 element contains the text of each lyric (While it's better to use vue.js for better DOM management).


passport authenticate with support for XHR

I just wrote some code for authenticate that handles xhr
https://gist.github.com/cultofmetatron/5349630
in this day and age, I feel it should be default behavior to have parameters for xhr requests in express. If enough people agree or I have the blessings of the upstream guys, I can go in and try to add such functionality myself.
This is the idea of what Im going for
passport.authenticate('local', {
successRedirect: '/',
failureRedirect: '/login',
successXhr: JSONToSendBack
failureXhr: JSONToSendifFailure
failureFlash: true

Why isn't this documented?

This should be documented. Client-side needed to retrieve errors in .json format, so flashes would not work. Fail with error fixed my problem though. Thanks.

+1 I'd love to see this documented too
Someone wrote up an issue for that here: https://github.com/jaredhanson/passport/issues/458

This is still useful in 2021. The API support in passport needs to be updated imo. This soldved my issue with client receiving errors in JSON format.


Implement threshold and refund feature for WCA Contract

We finished the basic framework of milestone. Now we are moving on to implement threshold and refund feature.
By design, creator can specific a milestone by INDEX (i.e. count from 0) as a threshold. Before this threshold milestone is finished or expired, buyers can regret and make a refund. The method I preferred is that depends on when a buyer makes a refund. If a buyer makes a refund before the threshold milestone is finished or expired, then the token will transfer to his account immediately, and the purchase record will be deleted from buyer info. If a buyer makes a refund after the threshold milestone, then he has to wait for WCA finished to get his token. This will mark his buyer info with the milestone index he quit at.
Note: If we need to support buyers to make a partial refund, then the on payment method should be able to find and accumulate the amount when a buyer make multiple purchase with the same WCA identifier.

Done by #10


DX: show big help message when baseurl problems

💥 Proposal
Not configuring the baseurl properly is the most common Docusaurus error, and lead to site deployments such as:

It's very often asked on issues or Discord why a site does not work after deployment, and the answer is always the same:  baseurl.

I suggest that:

We inline (HTML + JS + CSS) a big error message at the top of the homepage of a docusaurus site
We recommend to use baseUrl = document.location.pathname (can be computed from the inlined JS script)
We hide this message in the first remotely-loaded CSS file, as it proves that baseurl setting is correct

This way, the user can figure it out himself.
This should rather be made theme-agnostic, and be in core only.
To test this, run in the monorepo: yarn build:v2 and then yarn serve website and open http://localhost:5000/build/ => should show a broken site.
The idea is to get something like:

But without any flashing or whatever: if the site works fine, then the user should never see this.
Drawback: if for some weird reasons the baseurl is correct, the HTML loads, but not the CSS (network error for example), the user would still see this error message. That does not seem a big deal to me, as it should be quite rare, and he'll see a broken site anyway...
Edit: here's the final error message merged and a tweet about it: https://twitter.com/sebastienlorber/status/1329360348349083650

@slorber I would love to take a shot at this
Can you please guide me with a starting point?

Thank you! I will try this :+1:

This feature is causing CSP rules to fail in production when unsafe-inline is not set. Although this is a nice to have, should we keep this feature only for dev mode and remove it for the production build?
Let me know, if you guys agree with this, I am happy to work on it.

I see
@mpsq most issues happen when users try to deploy to their prod env imho, so I'd rather give you an option to disable this option, but keep it enabled by default.
What should we do to pass that unsafe-inline rule? just remove the banner with inline style?

Thanks for you super quick answer.
Inline styles are an issue yes, but more importantly, the following snippet:
<script type="text/javascript">
            {`
document.addEventListener('DOMContentLoaded', function () {
  var baseUrlSuggestion = document.getElementById(
    '${BaseUrlSuggestionContainerId}',
  );
  if (baseUrlSuggestion) {
    var actualHomePagePath = window.location.pathname;
    var suggestedBaseUrl = actualHomePagePath.substr(-1) === '/'
        ? actualHomePagePath
        : actualHomePagePath + '/';
    baseUrlSuggestion.innerHTML = suggestedBaseUrl;
  }
});
`}
</script>

Source
Is causing "even more" trouble since it means enabling unsafe-inline for script-src. Although inline styles can be fixed easily, do you believe this script can also be somehow reworked to not be inlined? A flag to disable this feature would also work :).

We'll add a flag to disable then.
This inlining is required, because the banner has to be displayed when the js/css assets can't load, so we can't un-inline them 😅

will be fixed by https://github.com/facebook/docusaurus/pull/3802

https://github.com/facebook/docusaurus/releases/tag/v2.0.0-alpha.69
let me know if this works for you

Yes, it is all good now, no more CSP issues :tada:
Thanks Seb!

great :) 👍


Enchantment applicator destroys enchanted book stack

Industrial Foregoing Version: 1.12.13
TeslaCoreLib Version: 1.0.17
Crashlog (If Any):
Inside the enchantment applicator, you can put a stack of enchanted books... but when you put an item to be enchanted, it destroys all the enchanted books in the book slot to enchant 1 item.

Confirming this bug.

:wave: Sorry but Industrial Foregoing has ended support for version 1.12. I'm closing this issue as it will no longer be updated for that version. Thanks for playing with the mod.


Search dv only IP masks

Description
it fixes an edge case when docvalues-only (index:false) IP field is queried via IP mask (<IP_ADDRESS>/22 syntax). Now such query is ignored, this change makes it work.
Related Issues
relate #11508
Check List

[v] Functionality includes testing.
[ ] API changes companion pull request created, if applicable.
[ ] Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

TODO

sign off comit
Improve both true case (indexed&dv)

@mkhludnev  Could you please add entry to CHANGELOG.md for this change? thank you

❌ Gradle check result for bf572a8: FAILURE

Please rebase against the latest main

@msfroh LGTY? thanks!


Is it possible to provide a prop to control a scroll while dragging?

Is your feature request related to a problem? Please describe.
For example, if I have 100+ items inside a container but am able to see only the first 20(for example) I must reach the bottom of the last item while dragging to make the scroll work
Describe the solution you'd like
I want that scroll would de configurable and would not depend on the root component. For example, if I'm starting to drag an item from the center(visible center) a tree should understand that it should be scrolled up or bottom. Instead of waiting until I reach the border of the top or bottom of the root component. The root element bounds may not always be visible and this creates problems :(

@aslanalyiev
Would the following examples be helpful?
(Open Folder1 and try it.)
https://codesandbox.io/s/scroll-control-413jp0?file=/src/App.tsx

Hey, thank you a lot

@aslanalyiev Would the following examples be helpful? (Open Folder1 and try it.)
https://codesandbox.io/s/scroll-control-413jp0?file=/src/App.tsx

@aslanalyiev hi, It looks like this link is no longer valid. Can you please provide a new link for demo that is available?

@clarkzsd Sorry, I deleted the old link to the sandbox.
I have created a new similar sandbox, please refer to this one.
https://codesandbox.io/p/sandbox/scroll-control-ts-4s4pq4
In this sandbox src/Treeview.tsx will be helpful.


[gaia.com] AttributeError: 'NoneType' object has no attribute 'items'

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2<IP_ADDRESS> (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
anywhere
Description
Test link:
https://www.gaia.com/video/connecting-universal-consciousness?fullplayer=feature
Verbose log
./yt-dlp -v -F "https://www.gaia.com/video/connecting-universal-consciousness?fullplayer=feature"
[debug] Command-line config: ['-v', '-F', 'https://www.gaia.com/video/connecting-universal-consciousness?fullplayer=feature']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2<IP_ADDRESS> [a86e01e74] (zip)
[debug] Plugins: ['SamplePluginIE', 'SamplePluginPP']
[debug] Python version 3.9.13 (CPython 64bit) - macOS-12.4-arm64-arm-64bit
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: avconv -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] Checking exe version: avprobe -bsfs
[debug] exe versions: none
[debug] Optional libraries: sqlite3-2.6.0
[debug] Proxy map: {}
[debug] [Gaia] Extracting URL: https://www.gaia.com/video/connecting-universal-consciousness?fullplayer=feature
[Gaia] connecting-universal-consciousness: Downloading JSON metadata
[Gaia] 113091: Downloading JSON metadata
[Gaia] 89356: Downloading JSON metadata
ERROR: 'NoneType' object has no attribute 'items'
Traceback (most recent call last):
  File "./yt-dlp/yt_dlp/YoutubeDL.py", line 1427, in wrapper
    return func(self, *args, **kwargs)
  File "./yt-dlp/yt_dlp/YoutubeDL.py", line 1497, in __extract_info
    ie_result = ie.extract(url)
  File "./yt-dlp/yt_dlp/extractor/common.py", line 647, in extract
    ie_result = self._real_extract(url)
  File "./yt-dlp/yt_dlp/extractor/gaia.py", line 86, in _real_extract
    media = self._download_json(
  File "./yt-dlp/yt_dlp/extractor/common.py", line 1009, in download_content
    res = getattr(self, download_handle.__name__)(url_or_request, video_id, **kwargs)
  File "./yt-dlp/yt_dlp/extractor/common.py", line 973, in download_handle
    res = self._download_webpage_handle(
  File "./yt-dlp/yt_dlp/extractor/common.py", line 841, in _download_webpage_handle
    urlh = self._request_webpage(url_or_request, video_id, note, errnote, fatal, data=data, headers=headers, query=query, expected_status=expected_status)
  File "./yt-dlp/yt_dlp/extractor/common.py", line 780, in _request_webpage
    return self._downloader.urlopen(self._create_request(url_or_request, data, headers, query))
  File "./yt-dlp/yt_dlp/extractor/common.py", line 746, in _create_request
    return sanitized_Request(url_or_request, data, headers)
  File "./yt-dlp/yt_dlp/utils.py", line 770, in sanitized_Request
    return compat_urllib_request.Request(url, *args, **kwargs)
  File<EMAIL_ADDRESS>line 326, in __init__
    for key, value in headers.items():
AttributeError: 'NoneType' object has no attribute 'items'

Working in yt-dl master:
$ python -m youtube_dl -v -F 'https://www.gaia.com/video/connecting-universal-consciousness?fullplayer=feature'
[debug] System config: [u'--prefer-ffmpeg']
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'-F', u'https://www.gaia.com/video/connecting-universal-consciousness?fullplayer=feature']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Git HEAD: e0ec049d5
[debug] Python version 2.7.17 (CPython) - Linux-4.4.0-210-generic-i686-with-Ubuntu-16.04-xenial
[debug] exe versions: avconv 4.3, avprobe 4.3, ffmpeg 4.3, ffprobe 4.3
[debug] Proxy map: {}
[Gaia] connecting-universal-consciousness: Downloading JSON metadata
[Gaia] 113091: Downloading JSON metadata
[Gaia] 89356: Downloading JSON metadata
[Gaia] 89356: Downloading m3u8 information
[info] Available formats for 89356:
format code        extension  resolution note
audio-0-en__Main_  mp4        audio only [en]
audio-1-en__Main_  mp4        audio only [en]
audio-2-en__Main_  mp4        audio only [en]
audio-3-en__Main_  mp4        audio only [en]
563                mp4        480x270     563k , avc1.420015, video only
872                mp4        640x360     872k , avc1.42001e, video only
1126               mp4        640x360    1126k , avc1.4d001e, video only
1525               mp4        960x540    1525k , avc1.4d001f, video only
2072               mp4        960x540    2072k , avc1.4d001f, video only
2401               mp4        1280x720   2401k , avc1.4d001f, video only
4050               mp4        1920x1080  4050k , avc1.640028, video only (best)
$

Same with Py 3.9.
On a quick diff, some code was moved from _real_initialize() to _perform_login() and I believe is now only run if credentials have been provided. OP could probably revert to youtube_dl/extractor/gaia.py as a work-around.

This bug actually exists in ytdl as well, but is triggered only when data is not None and headers is None. See https://github.com/ytdl-org/youtube-dl/blob/8a158a936c8b002ef536e9e2b778ded02c09c0fa/youtube_dl/extractor/common.py#L628-L629

My thought was that urllib2 ought to be handling the headers=None case.

I've got a quick follow up for this case.
Test links:
https://www.gaia.com/video/highest-pass?fullplayer=feature
https://www.gaia.com/video/a-treacherous-ride-home?fullplayer=feature
These fail with:
ERROR: [Gaia] highest-pass: Unable to download JSON metadata: HTTP Error 403: Forbidden (caused by <HTTPError 403: 'Forbidden'>); please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
but if I use https://www.gaia.com/video/highest-pass cutting off the ?fullplayer=feature junk, it downloads just a trailer.
I assume to get full video I need to login and pass cookies but extractor does not detect this case(though I see there's is some code for this at a quick glance).
at the same time this link works fine even with "fullplayer junk": https://www.gaia.com/video/connecting-universal-consciousness?fullplayer=feature
So, it's a bit inconsistent.


Menu Items should be linked to order now section !!

all menu items can be picked from add to cart while adding items to to cart !! RN it is having dummy data !!

@c4coderandcreator Could you assign this to me? I can work on it.
Could you please elaborate little bit more on the issue?
Thanks!

These are menu Items

This should also be included in this section

Got it. Could you assign this issue to me?

Hey @vishnukaushik  it is assigned to you !!

@c4coderandcreator I see that the menu items are all hard-coded. Should we hard-code all items in the "add to card" section?

open the menu in small screen you will find the menu items in a list , i like to add the same functionality within the order now section with pricing included !! no matter what logic you apply just implement it

@c4coderandcreator Created #177. Please review when you get a chance.
Thanks!

@c4coderandcreator Created #177. Please review when you get a chance. Thanks!

create the PR with UI-Improvement branch

@c4coderandcreator created #179 with UI-Improvement branch


Make :center work on visual selections

Is your feature request about something that is currently impossible or hard to do? Please describe the problem.
Yes it's very hard to center text selections in vim at the moment
Describe the solution you'd like
                       w  h sa sb sl sr ....
             width |  |  1  0  0  0 -1 -1  0  0 |
            height |  |  0  1 -1 -1  0  0  0  0 |
       space_above |  |  0  0  0  0  0  0  0  0 |
       space_below |  |  0  0  0  0  0  0  0  0 |
        space_left |  |  0  0  0  0  0  0  0  0 |
       space_right |  |  0  0  0  0  0  0  0  0 |
               ... |  |  0  0  0  0  0  0  0  0 |

CTRL-V selecting the first column and entering the command :'<,'>center should center the column not the lines
Describe alternatives you've considered

Writing my own centering function in Vim
writing my own script to center lines and putting it in my path

Neither of these solutions are trivial sadly. I think basic text manipulation like this should be part of Vim in the first place.
Vim provides :center, it just doesn't center anything except lines, even if you've made a visual selection.

ex commands always work linewise. There is already an issue for, to make it work column wise using e.g.  `< marker value. But so far, this does not exist. It's part of the todo list and there is an issue about it already (sorry don't find it currently), so I am closing this as duplicate

Here it is: #3292


[Request] Add "updated" timestamp to FB2 Bridge output

FB2 Bridge is not parsing the timestamp for each post, so when opening an RSS reader it stamps each entry with the time the RSS reader first reads it rather than the actual time of the Facebook post.
Particularly bad when adding a feed for the first time, when there are 100 RSS entries with an identical date in the RSS reader.

I believe I've implemented this in PR #849

Closing this since my PR #849 has now been merged.


UI router ng2 not working for ng4 - which is backward compatible?!

"dependencies": {
    "@angular/animations": "^4.0.2",
    "@angular/common": "^4.0.2",
    "@angular/compiler": "^4.0.2",
    "@angular/compiler-cli": "^4.0.2",
    "@angular/core": "^4.0.2",
    "@angular/forms": "^4.0.2",
    "@angular/http": "^4.0.2",
    "@angular/material": "^2.0.0-beta.3",
    "@angular/platform-browser": "^4.0.2",
    "@angular/platform-browser-dynamic": "^4.0.2",
    "@angular/platform-server": "^4.0.2",
    "@angular/router": "^4.0.2",
    "typescript": "^2.2.2",
    "ui-router-ng2": "^1.0.0-beta.4",
    "zone.js": "^0.8.5"

npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\ch
okidar\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for fsevents@
1.1.1: wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"}
)
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none
was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of @angular/platform-browser@^2.
0.0 but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none
was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was ins
talled.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>.10 but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>bu
t none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but
none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>bu
t none was installed.
.....................................
ERROR in Error encountered resolving symbol values statically. Only initialized variables
and constants can be referenced because the value of this variable is needed by the templa
te compiler (position 15:22 in the original .ts file), resolving symbol ROUTES in<EMAIL_ADDRESS>resolving sy
mbol makeRootProviders in D:/..............s/node_modules/ui-router-ng2/lib/uiRoute
rNgModule.d.ts, resolving symbol UIRouterModule.forRoot in D:/...................s/node_
modules/ui-router-ng2/lib/uiRouterNgModule.d.ts, resolving symbol AppModule in D:/................s/src/app/app.module.ts, resolving symbol AppModule in D:/..............s/src/app/app.module.ts
ERROR in ./~/ui-router-ng2/lib/uiRouterNgModule.js
Module not found: Error: Can't resolve '@angular/router/src/router_config_loader' in 'D:,,,,,,s\node_modules\ui-router-ng2\lib'
@ ./~/ui-router-ng2/lib/uiRouterNgModule.js 6:0-66
@ ./~/ui-router-ng2/lib/index.js
@ ./src/app/app.module.ts
@ ./src/app/index.ts
@ ./src/main.ts

It work up to the latests minor versions of angular 2. angular 4 is still in development.
Try switching angulars deps to ^2.4.0.
see #31

Hi @Asone, any news on when there will be a ui-router for angular 4?

@efistasija : I have no idea, i'm not part of the original team. I just suggested a PR a few months ago to update a bit the demo, that's mainly all.
Apparently there's chances that this demo will be abandoned according to the comments on this page : #26

https://github.com/ui-router/angular/releases/tag/1.0.0-beta.6 works with ng 4.1.3

latest beta.6 release is compatible with Angular 4
Note: The package ui-router-ng2 has been renamed to @uirouter/angular
This demo will likely be replaced by something involving the Angular-CLI


Figure out what contract-based account address would be using CREATE2

Did you mean CREATE instead of CREATE2? We are using a pre-deployed contract from tasit-contracts to tests. We should move to add the deployment process to the tasit-contract-based-account test suite and use that.

Hmm, I mean CREATE2.
This would be for a user of an app powered by the SDK. The use case I'm picturing here is mostly for if we wanted to deploy a contract-based-account for the user. Or rather, if we wanted to know the address of the contract before having the eth to deploy it, but still sending something to it ahead of time. So counterfactual deployment, to use the technical term.
When talking about pre-deployed contracts, do you mean singleton contracts like the Decentraland Marketplace or the Gnosis Safe one?
For our tests, we'd still use our pre-deployed contracts of other 3rd party deps for all singleton contracts. I agree that deploying a separate Safe contract for each test run could be interesting to explore in the future (although that's a separate topic). And having that deployment happen as part of tasit-contract-based-account could make sense. I'll have to ponder that more.


Msatg inv3

Adds teacher guide content for Investigation 3

Forgot to add Inv 3.4

Did you mean to close this? If it's simply that it's not ready you could mark it a work-in-progress, i.e. not ready for review instead.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/31 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-yml-archive-non-versioning-archive.git

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

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/index.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

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


Proof is not working anymore

What is the bug?
Proof is not being created , trapped in error
How can it be reproduced? (optional)
Simple try logging in with Aadhar
Additional context (optional)
This is the error Uncaught (in promise) Error: Not all inputs have been set. Only 96 out of 97
and i think its become zkey json was changed

@Meyanis95  can you check , what issue it could be ?

It's not working in the example website too

Hey @dipanshuhappy
Yes, we just released a new version of the protocol today and updated the artifacts.
Can you tell me what packages are you using and their version?
Also, can you update the packages to version 0.1.8 and test again?

Hey @dipanshuhappy
Yes, we just released a new version of the protocol today and updated the artifacts.
Can you tell me what packages are you using and their version?
Also, can you update the packages to version 0.1.8 and test again?

I updated to 0.1.8 and got the error  witness.app_id is not defined

@Meyanis95  you can check the example website , it too doesn't seem to work

Are you using the anon-aadhaar-react package or just the anon-aadhaar-pcd package?
The proving changed since the v0.1.8, now you need to instantiate an application_id that will be needed as input to compute the witness.
Let me know which package you're using and I can help you solve this.
Also, yes we are aware of the application app, a fix should be pushed today.

Are you using the anon-aadhaar-react package or just the anon-aadhaar-pcd package?
The proving changed since the v0.1.8, now you need to instantiate an application_id that will be needed as input to compute the witness.
Let me know which package you're using and I can help you solve this.
Also, yes we are aware of the application app, a fix should be pushed today.

I am using both in my application with version 0.1.8
How do I pass the application I'd? In the react one ?

I just redeployed the example app and it's should be working.
I suggest you to clear your browser and cached data et test again.
Let me know if it's working.

@Meyanis95
Hey i did all you said , but still witness app_id is undefined , i am not sure where i am supposed to define that , i cleared my cache too
can you once check this repo , about it https://github.com/dipanshuhappy/anon-aadhaar-icp

I see that you forked the example-app, have you tried to pull the example app?

I see that you forked the example-app, have you tried to pull the example app?

Got it I just recloned and typed ...my code ...thanks for the help... by the way I am implementing anon aadhar on the ICP with integration to Internet identity

Great to hear!
I'm closing this ticket as it's resolved.


Device Activity not working for some Alexa devices

Hi, I really do love your node and have integrated it into my home control environment.
The "ON Device Activty" does work fine for my ECHO DOT and ECHO SHOW devices. Unfortunately, I cannot see any activty when giving same comands on

my android phone using the Alexa app
my ECHO CAR
my FireTV

Any idea on how I can get this working ?
thanks for your help

I have had the same issue as well, specifically with the device activity coming from my phone/tablet, the node to get echo devices does show them. It used to work at one point though.

Hi, I have the same issue, "ON Device Activity" works fine with the ECHO DOT devices but not with the android app. Did you find how I can get this working?
Thanks for your help


What about Roboto Mono

What do you think about Roboto Mono?
It is my current favorite font for programming / source code in Linux.
https://fonts.google.com/?category=Monospace&selection.family=Roboto+Mono

@vikivivi Thank you for your advice,I will add it later.


Moto G 4 broken layout

Hi, when i visit the site from my device it seems to be broken:

Any way to fix it ?

@erikcunha  Can you please let know what you was expecting? When I tested with same device it is working smooth.

I guess it was fixed already but the white bar on the bottom, it shouldn't be there.


Loads Pages Without In-Place Replacement

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

In my environment -- and it's unfortunately restricted to it, I can't reproduce it in an example -- I have the problem that all my redirects are done correctly and the server acts like expected with custom routing, but, the pages are loaded normally, not via in-place replacement (AJAX). So no XHR will be done on click on a link, instead the entire page gets reloaded.
How can I debug such an issue? Are there any common pitfalls? Do you have tips to find the issue? Any help is highly appreciated.
Just to show some code in case it's easy to find the issue, here's my server code:
const express = require('express');
const next = require('next');
const {parse} = require('url');
const i18nextMiddleware = require('i18next-express-middleware');
const Backend = require('i18next-node-fs-backend');
const i18n = require('../hoc/i18n');

const port = parseInt(process.env.APP_PORT, 10) || 3000;
const dev = process.env.NODE_ENV !== 'production';
const app = next({dev});
const handle = app.getRequestHandler();

// init i18next with server-side settings
// using i18next-express-middleware
i18n.use(Backend).use(i18nextMiddleware.LanguageDetector).init({
  preload: ['en', 'de'],
  ns: ['common', 'home', 'content'],
  backend: {
    loadPath: __dirname + '/../locales/{{lng}}/{{ns}}.json',
    addPath: __dirname + '/../locales/{{lng}}/{{ns}}-missing.json',
    jsonIndent: 2
  }
}, () => {
  app.prepare().then(() => {
    const server = express();

    // Translation routing
    server.use(i18nextMiddleware.handle(i18n));
    server.use('/locales', express.static(__dirname + '/../locales'));
    server.post('/locales/add/:lng/:ns', i18nextMiddleware.missingKeyHandler(i18n));

    // Application Routing
    server.get('*', (req, res) => {
      // Be sure to pass `true` as the second argument to `url.parse`.
      // This tells it to parse the query portion of the URL.
      const parsedUrl = parse(req.url, true);
      const {pathname, query} = parsedUrl;

      if (pathname.startsWith('/_next')) {
        return handle(req, res, parsedUrl);
      } else if (pathname === '/') {
        return app.render(req, res, '/', query);
      } else {
        return app.render(req, res, '/content', query);
      }
    });

    server.listen(port, err => {
      if (err) {
        throw err;
      }
      console.log(`> Application server ready on http://localhost:${port}`);
    });
  })
});

Context

Navigating pages via AJAX.
Your Environment

Tech
Version

next
5.0.0

node
6.11.3

OS
Win 10

browser
Chrome & FF latest

etc

Even though the server maps the dynamic URL correctly, the client-side still has to use the following link syntax to make in-place replacements work (especially important is the "as" attribute):
<Link href={'/content'} as={'/the-real-url'}>
  Test Link
</Link>

I only found it out after playing aroung with it several hours. The reason it wasn't clear is that this isn't documented. There's no hint about that in the section Custom server and routing, nor is there an example with the JSX syntax at all (like in my example above).
Would be good to improve the docs at this place.

With this solution I'm experiencing the same issue like this - Next.js uses replaceState instead of pushState, which means that you can't use browser back.

Closing due to follow-up ticket #4871


Downgrade common to 0.17.0

@JustinBeckwith Mayday! Our stuff isn't running with @google-cloud/common 0.18.3.
$ npm run system-test
run: Executing tests in: C:\Users\sawch\dev\nodejs-storage
run: Running: mocha system-test/*.js --no-timeouts

  storage
run: Success! Test finished in 3.34s

👎   Taking a look now.


No .d.ts file for typescript configuration

I may be doing something wrong but apparently even if on npm it says that this plugin is typecript compatible, when installing it on my project it can't find any .d.ts file and in fact in the node_modules no d.ts files for this plugin.
Maybe someone already did one for this plugin and could make a pr ?

Hello. Thanks for bringing this to my attention. This issue has been fixed in the new release. Details can be found here: https://github.com/AbreezaSaleem/react-voice-recorder-player/releases/tag/v1.2.0


Explicitly use python3

The Issue
The Node backend makes system calls to python, but this symlink is not available on e.g. a fresh install of Ubuntu/WSL2.
On such a system it is still possible to install lmql and run lmql playground, but as soon as you hit 'Run', you'll get
Compiled successfully!

You can now view web in the browser.

  Local:            http://localhost:3000
  On Your Network:  http://<IP_ADDRESS>:3000

Note that the development build is not optimized.
To create a production build, use yarn build.

webpack compiled successfully
Error: spawn python ENOENT
    at ChildProcess._handle.onexit (node:internal/child_process:286:19)
    at onErrorNT (node:internal/child_process:484:16)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  errno: -2,
  code: 'ENOENT',
  syscall: 'spawn python',
  path: 'python',
  spawnargs: [ 'live.py', 'endpoints', 'lmql' ],
  cmd: 'python live.py endpoints lmql'
}

I suspect the reason this wasn't caught earlier is that installing conda creates a python symlink. So very few developers would encounter this issue.
The Solution
Simply replace all OS calls to python with python3.

Thanks for this. One question I have regarding this: Under conda, is python3 always the conda version? If not, this could cause issues in some cases, no?

I cannot find documentation to cite here, but the python and python3 symlinks are both in Conda's bin directory, which gets added to PATH during the installation process.
They point to the same location
$ which python
/home/f/miniconda3/bin/python
$ file /home/f/miniconda3/bin/python
/home/f/miniconda3/bin/python: symbolic link to python3.11
$
$ which python3
/home/f/miniconda3/bin/python3
$ file /home/f/miniconda3/bin/python3
/home/f/miniconda3/bin/python3: symbolic link to python3.11
$
$ which python3.11
/home/f/miniconda3/bin/python3.11

So if you have Conda installed, python3 should work just like python. I really don't think there is any benefit to using python.

Sorry for the delay. I just tested this under conda, and it should not cause any issues. Thank you for this contribution.


Problem about creating logger

Hi, thanks for your great work. I tried to rerun your code, but I encounter a problem as follows. I wonder whether it is related to multiprocessing module.
Traceback (most recent call last):
Traceback (most recent call last):
Traceback (most recent call last):
Traceback (most recent call last):
Traceback (most recent call last):
Traceback (most recent call last):
  File "run/train_3d.py", line 329, in <module>
  File "run/train_3d.py", line 329, in <module>
  File "run/train_3d.py", line 329, in <module>
  File "run/train_3d.py", line 329, in <module>
  File "run/train_3d.py", line 329, in <module>
  File "run/train_3d.py", line 329, in <module>
Traceback (most recent call last):
  File "run/train_3d.py", line 329, in <module>
    main()
  File "run/train_3d.py", line 138, in main
            main()main()main()

    main()  File "run/train_3d.py", line 138, in main
main()main()  File "run/train_3d.py", line 138, in main

  File "run/train_3d.py", line 138, in main

  File "run/train_3d.py", line 138, in main
  File "run/train_3d.py", line 138, in main
  File "run/train_3d.py", line 138, in main
    config, args.cfg, 'train')
  File "/mnt/lustre/liqikai.vendor/open_mmlab/pose3d/mvp/lib/utils/utils.py", line 43, in create_logger
            config, args.cfg, 'train')config, args.cfg, 'train')config, args.cfg, 'train')

  File "/mnt/lustre/liqikai.vendor/open_mmlab/pose3d/mvp/lib/utils/utils.py", line 43, in create_logger
  File "/mnt/lustre/liqikai.vendor/open_mmlab/pose3d/mvp/lib/utils/utils.py", line 43, in create_logger
  File "/mnt/lustre/liqikai.vendor/open_mmlab/pose3d/mvp/lib/utils/utils.py", line 43, in create_logger
    config, args.cfg, 'train')
    config, args.cfg, 'train')  File "/mnt/lustre/liqikai.vendor/open_mmlab/pose3d/mvp/lib/utils/utils.py", line 43, in create_logger

  File "/mnt/lustre/liqikai.vendor/open_mmlab/pose3d/mvp/lib/utils/utils.py", line 43, in create_logger
    config, args.cfg, 'train')
  File "/mnt/lustre/liqikai.vendor/open_mmlab/pose3d/mvp/lib/utils/utils.py", line 43, in create_logger
    root_output_dir.mkdir()
  File "/mnt/lustre/liqikai.vendor/anaconda3/envs/pt180cu111py37mmcv1317/lib/python3.7/pathlib.py", line 1273, in mkdir
    root_output_dir.mkdir()
    root_output_dir.mkdir()  File "/mnt/lustre/liqikai.vendor/anaconda3/envs/pt180cu111py37mmcv1317/lib/python3.7/pathlib.py", line 1273, in mkdir

  File "/mnt/lustre/liqikai.vendor/anaconda3/envs/pt180cu111py37mmcv1317/lib/python3.7/pathlib.py", line 1273, in mkdir
    root_output_dir.mkdir()
  File "/mnt/lustre/liqikai.vendor/anaconda3/envs/pt180cu111py37mmcv1317/lib/python3.7/pathlib.py", line 1273, in mkdir
    root_output_dir.mkdir()
    root_output_dir.mkdir()
  File "/mnt/lustre/liqikai.vendor/anaconda3/envs/pt180cu111py37mmcv1317/lib/python3.7/pathlib.py", line 1273, in mkdir
            self._accessor.mkdir(self, mode)    self._accessor.mkdir(self, mode)
self._accessor.mkdir(self, mode)self._accessor.mkdir(self, mode)

self._accessor.mkdir(self, mode)
self._accessor.mkdir(self, mode)
FileExistsErrorFileExistsErrorFileExistsErrorFileExistsErrorFileExistsErrorFileExistsError    self._accessor.mkdir(self, mode)
FileExistsError: : : : [Errno 17] File exists: '/mnt/lustre/liqikai.vendor/open_mmlab/pose3d/mvp/output': [Errno 17] File exists: '/mnt/lustre/liqikai.vendor/open_mmlab/pose3d/mvp/output': [Errno 17] File exist
s: '/mnt/lustre/liqikai.vendor/open_mmlab/pose3d/mvp/output'[Errno 17] File exists: '/mnt/lustre/liqikai.vendor/open_mmlab/pose3d/mvp/output': [Errno 17] File exists: '/mnt/lustre/liqikai.vendor/open_mmlab/pose
3d/mvp/output'[Errno 17] File exists: '/mnt/lustre/liqikai.vendor/open_mmlab/pose3d/mvp/output'[Errno 17] File exists: '/mnt/lustre/liqikai.vendor/open_mmlab/pose3d/mvp/output'

could you try delete the output directory

I have tried deleting the output directory. But it didn't work. So I just hard-coded the final_output_dir in the train_3d.py and I also set the logger right in the train_3d.py, thus without calling the create_logger() function and it worked.


Error "connection reset by peer" after docker load

Issue description
Having issues following the hello-world-python guide, at the "Install the image step", we run the command:
docker save $APP_NAME | docker --host tcp://$DEVICE_IP:$DOCKER_PORT load

And we get the following error:
error during connect: Post "[http://DEVICE_IP:DOCKER_PORT/v1.41/images/load?quiet=0":](http://DEVICE_IP:DOCKER_PORT/v1.41/images/load?quiet=0%22:) write tcp <IP_ADDRESS>:43498->DEVICE_IP:DOCKER_PORT: write: connection reset by peer

I've correctly set the DEVICE_IP and DOCKER_PORT with those of my device.
The connection works fine running the command of the previous step:
docker --host tcp://$DEVICE_IP:$DOCKER_PORT system prune --all --force

As we get the following output:
Total reclaimed space: 0B

Any clue on how to solve this?
Thanks
Environment

Axis device model: Q1656-LE camera
Axis device firmware version: 11.4.63

Hello @lucapatarnello
Do you have an SD card installed in the device? Is the docker acap app using the SD card?
Even the hello-world example might be too large without an SD card.
Check the system log and the docker acap log to get more information about what's going on. (you can do it from the web gui).

Hi @Corallo
As you can see from the screenshot below, the SD card is mounted, used by docker and with plenty of space available

The logs do not report any information regarding the error that we are getting.

Strange, are you checking the log from the app log? (pressing the three dots next to the application -> app log)
Pulling an image directly to the camera works?
docker -H <camera_ip>:<docker_port> pull ubuntu

We cannot perform the image pull as the camera is not connected to the internet, all the files are being provided by a VM, which has access to the camera.
We also tried to copy the saved .tar of the image and to transfer it to the camera using scp as following:
scp -P $SSH_PORT image.tar root@$CAMERA_IP:/root/image.tar

But we get the following error:
image.tar: Permission denied

Could it be that we do not have writing permissions on the camera?

Can you try coping it in /tmp/image.tar

The permission denied error persists

Your camera must be in a bad state.
I think the best solution might be a factory default.
Try to install the latest firmware (11.5) and in doing so select factory default
From the GUI
Maintenance -> Firmware upgrade -> Upgrade -> Factory default

We updated the firmware to Q1656-LE_11_5_64.bin and docker to Docker_Daemon_1_4_1_aarch64_signed.eap.
We cannot perform a FACTORY DEFAULT as we would lose important ip settings configuration.
We opted for a RESTORE.
We noticed that by running
shell docker -H <camera_ip>:<docker_port> pull ubuntu
the event that the camera cannot pull the image as it is not connected to the internet is registered in the camera logs (docker daemon app log).
Still, the command
shell docker save $APP_NAME | docker --host tcp://$DEVICE_IP:$DOCKER_PORT load
returns the error reported at the beginning of this issue. Since the request does not reach the camera, nothing is registered in its logs this time.
One thing we did not mention is that, the network to which the camera is connected is protected and only reachable by white-listed IPs. The VM's IP is white-listed and working as we've tested running telnet commands.

After the restore, do you still have the permission issue when running?
scp -P $SSH_PORT image.tar root@$CAMERA_IP:/tmp/image.tar
Are you sure the Permission denied is referring to the destination /tmp/image.tar on the camera and not the source?
Might it be that there's a size limit of the file that you can upload to the camera? (imposed by your network settings)
Let's try to tunnel the docker communication using ssh:
In one shell, run sudo ssh -L 8080:<camera_ip>:<docker_port> root@<camera_ip>
And try again with:
docker save $APP_NAME | docker --host tcp://$DEVICE_IP:8080 load

scp worked and we were able to copy the .tar image in the tmp folder, but that is not useful as when connecting to the camera via ssh docker is not available.
Indeed, running any docker command from the camera when connected via ssh to the ssh port results in docker: not found .
We also tried ssh tunnelling but results in a connection timeout.

Some additional info, just to double check we run the command
docker --host tcp://$CAMERA_IP:$DOCKER_PORT info

We obtained an output, confirming that there is a connection:
Client:
Context:    default
Debug Mode: false
Plugins:
  buildx: Build with BuildKit (Docker Inc., 0.6.3+azure)

Server:
Containers: 0
  Running: 0
  Paused: 0
  Stopped: 0
Images: 0
Server Version: dev
Storage Driver: overlay2
  Backing Filesystem: extfs
  Supports d_type: true
  Native Overlay Diff: true
  userxattr: false
Logging Driver: json-file
Cgroup Driver: cgroupfs
Cgroup Version: 1
Plugins:
  Volume: local
  Network: bridge host ipvlan macvlan null overlay
  Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
Swarm: inactive
Runtimes: io.containerd.runc.v2 io.containerd.runtime.v1.linux runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 579a6380ec93ab92a6e7f26167fe4f18dfcf2a4b.m
runc version: v1.1.3-2-g1e7bb5b7-dirty
init version: de40ad0
Security Options:
  apparmor
  seccomp
   Profile: default
Kernel Version: 5.15.13-axis9
Operating System: AXIS OS (based on Poky from the Yocto Project) 11.5.64 (langdale)
OSType: linux
Architecture: aarch64
CPUs: 4
Total Memory: 959.5MiB
Name: axis-b8a44f32d713
ID: BLOW:BXPF:ZW54:AKBD:HHTK:YYUW:YRMC:DXJV:QP62:EGMY:IGU6:LO52
Docker Root Dir: /var/spool/storage/SD_DISK/dockerd/data
Debug Mode: false
Registry: https://index.docker.io/v1/
Labels:
Experimental: false
Insecure Registries:
  <IP_ADDRESS>/8
Live Restore Enabled: false

WARNING: API is accessible on http://<IP_ADDRESS>:2375/ without encryption.
         Access to the remote API is equivalent to root access on the host. Refer
         to the 'Docker daemon attack surface' section in the documentation for
         more information: https://docs.docker.com/go/attack-surface/

So the connection to docker daemon works, the info command works, but the load command doesn't.
Could it be some firewall rule on the docker port interfering with the load command?

Yes, you are probably facing a network issue that is blocking the file transfer in that port.
You can try with the docker compose acap
 Differently from the docker acap it should allow you to run docker command directly from the device.

We did as you suggested and we finally made it work, thank you.
The question we have now, is there a way, after transfering the image via scp, to launch the load command from the VM instead of having to connect via ssh to the camera and then launching it from the device itself?
We tried running the following command (the path you'll see is where we placed the image):
sudo docker -H $CAMERA_IP:$DOCKER_PORT load -i /var/spool/storage/SD_DISK/images_tar/image.tar

But it cannot find the file.
Any suggestion on how we could do it?

Thank you for helping us figure it out.
I'm closing the issue.


How to manage long running processes

Hi @santiq, your approach is interesting, and the whole architecture is nice and clean. I'm working on an EDD approach in my project which I used Redis streams under the hood as my event stream. on EDD there is a concept of projection that defines a read model and it's a long run process as my process listens to events and take an action when new events pushed to stream (CQRS).

each projection is a long-running dispatcher function that handles the events and updates db.
I would like to manage these types of projections under the forked process. would you mind explaining how you do that in your code structure?

Have you given a thought of using agenda and make them long running process into background processes
He already has examples of using agenda


introduce latest cf

Although I updated the RFC to write latest CF first, and then move keys between CFs, I suggest to always double write in POC as it has less compatibility issues.


RealtimeIndexTask and AppenderatorDriverRealtimeIndexTask have considerable amount of duplicated code

@leventov could you please link to an example from each file?

@vogievetsky One example is the stopGracefully method


Support torch.where(condition)

🚀 Feature
Motivation
Mixtral uses it: https://github.com/mistralai/mistral-src/blob/8598cf582091a596671be31990448e0620017851/moe_one_file_ref.py#L215
Pitch
Support

from https://pytorch.org/docs/stable/generated/torch.where.html
Additional context
We already support torch.where(condition, input, other): https://github.com/search?q=repo%3ALightning-AI%2Flightning-thunder+"def+where"&type=code
cc @apaz-cli

As of now, we cannot support data-dependent ops, alas...

@carmocca , looking at the code I think the solution would be to modify litgpt model. The result of topk can be sorted, and then we do not need to apply where at all. This will also eliminate the device sync caused by such a usage.

@nikitaved Faster and better code very welcome in LitGPT. I benchmarked a few different implementations when this was added and this came out to be the best in general (see description and discussion in https://github.com/Lightning-AI/litgpt/pull/823). It would be useful to see them compared to whatever you propose.

triage review:

can the call to torch.where(condition) in mixtral use the hypothetical shape parameter to nonzero to make the output shape known at compile-time?
we should implement nonzero(..., shape=...)`

nonzero doesn't have a shape= argument. Did you mean as_tuple=?

nonzero doesn't have a shape= argument. Did you mean as_tuple=?

We were referring to a parameter that would be analogous to jax.lax.nonzero's size parameter:
https://jax.readthedocs.io/en/latest/_autosummary/jax.numpy.nonzero.html

This prototype adds torch.where(boolean_tensor) https://github.com/Lightning-AI/lightning-thunder/pull/303

@IvanYashchuk @kshitij12345 Is this still unsupported when using ThunderFX? If torch.where(condition) is supported when using ThunderFX (because the operator is sent to PyTorch for execution?), then maybe we can close or amend this issue to refer more specifically to using torch.where(condition) with the Thunder interpreter as the entrypoint?

torch.where(condition) works with ThunderFX path by sending it to PyTorch. We also have a test for the same.
https://github.com/Lightning-AI/lightning-thunder/blob/b28d5b3536e60fb0b30896bdd4df6e288cf6a5c8/thunder/tests/test_dynamo.py#L346-L349
Will update the issue title to reflect the request for torch.where(condition) not being supported by thunder.jit entrypoint.


Unexpected Token

I'm running 4.2.1 on ubuntu server 16.04 with the command below against the attached tweets.json file (I just added the .txt file to be able to upload it).
json2csv -i tweets.json -f user,text,fullname,id,timestamp -o out.csv
tweets.json.txt
The error I get is below and the json file is valid according to http://jsonlint.com. Do you know what the problem might be? I run json2csv on 2 other machines without any problems, so I'm a bit puzzled.
/usr/local/lib/node_modules/json2csv/bin/utils/TablePrinter.js:86
      ...Array(vertPadTop).fill(emptyLine),
      ^^^

SyntaxError: Unexpected token ...
    at exports.runInThisContext (vm.js:53:16)
    at Module._compile (module.js:374:25)
    at Object.Module._extensions..js (module.js:417:10)
    at Module.load (module.js:344:32)
    at Function.Module._load (module.js:301:12)
    at Module.require (module.js:354:17)
    at require (internal/module.js:12:17)
    at Object.<anonymous> (/usr/local/lib/node_modules/json2csv/bin/json2csv.js:11:22)
    at Module._compile (module.js:410:26)
    at Object.Module._extensions..js (module.js:417:10)

Sounds like you are running an old version of Node.js. Give the latest version a try (or 6+).

I'm running version 10 and a quick test has worked... if I run into any other problems, I'll let you know... Thanks for your help!


RazorViewEngine.cs throw MissingMethodException beta8-15793

var page = _pageFactory.CreateInstance(applicationRelativePath);

Creating an empty beta 7 project by visual studio and using the mvc b8-15793
when return view(), this exception will be thrown.

Don't mix beta7 and beta8

I did not mix them.
This is my project.json
  "dependencies": {
    "Microsoft.AspNet.Server.IIS": "1.0.0-beta8-*",
    "Microsoft.AspNet.Server.WebListener": "1.0.0-beta8-*",
    "Microsoft.AspNet.Mvc": "6.0.0-beta8-*",
    "Microsoft.AspNet.StaticFiles": "1.0.0-beta8-*"
  },

What's in your global.json

{
  "projects": [ "src", "test" ],
  "sdk": {
    "version": "1.0.0-beta7"
  }
}

I don't know how to install beta 7.
dnvm upgrade only can update to beta7

So first, you're mixing beta7 and beta8. Secondly, you need to remove the sdk version if you want VS to use the default. Third, if you do dnvm upgrade -u you'll get the latest unstable build.

Let me have a try

I have upgrade and dnu restored.

You're still running beta7, it says it right there in the error message.

Thanks. The problem has been solved.


Locating the correct vstest.console.exe (VS2017)

What You Are Seeing?
there is a closed issue https://github.com/cake-build/cake/issues/1522 for locating the correct VS2017 mstest.exe. But that should be applied when locating vstest.console.exe.
What is Expected?
If i am right the same modification should be applied to find the correct vstest.console.exe here.

Microsoft have a package:
https://www.nuget.org/packages/Microsoft.TestPlatform/
For deploying the console for VSTest. Should be not be using this as the #tool directive for it? I'd rather download a pinned, isolated tool that hope for some console runner on the PC somewhere.

Yes!

Until Cake's VSTest converts to using the new, officially-recommended package https://www.nuget.org/packages/Microsoft.TestPlatform/, this is what I'm doing:
VSTest(asssemblyPath, new VSTestSettings
{
    // https://github.com/cake-build/cake/issues/2077
    #tool Microsoft.TestPlatform
    ToolPath = Context.Tools.Resolve("vstest.console.exe")
});

Ah, see @paulomorgado's https://github.com/cake-build/cake/issues/2152!

#2152 is now merged and functionality for resolving VSTest through Microsoft.TestPlatform will be available in 0.29.0.

Was this issue meant to be closed?

Fixed by #2754


Nuget version does not include Controls namespace

Hi, I found WpfMath available as a nuget package in VS2015. Although it claims to be v0.2 the whole namespace WpfMath.Controls is missing there, thus making it useless for usage as a WPF control.
Would be great to just reference the nuget package except the need to build my own version from the sources.

@torfranz we haven't yet published the version with the Controls; it'll be v0.3. Sorry for inconvenience! I think that the new version will be published today or tomorrow.

That would be great :-)

Here it is! https://www.nuget.org/packages/WpfMath/0.3.0


After uploading an image the returned path ignores a port that is customized

When a file got uploaded using strapi the port, configured in the env variables is used to upload the file but not passed back. This results a link that by default uses port 80.

@schlak Thanks for your contribution, version 1.1.1 has been released


Sonarqube chart maintenance

The purpose of this PR is to not provide fixes or improvements for user experience, but these changes will be valuable for chart maintainers.

helper templates for labels - defined labels and label selectors in helpers to get them in sync over all resource templates
helper template for pod - removes duplicated pod spec in Deployment and StatefulSet templates and defines one place for future updates
fixed outlinings - with correct outlining its much simpler to expand/collapse needed part of template and simplifies navigation over template

Multiple minor changes that it brings to rendered templates:

removes install-plugins configMap volume if there no plugins to install.
removes rendering of service.labels in change-admin-password-hook job template

PS: Will create the same for dce chart after merge.

Hey @bonddim!
Thank you for your fantastic contribution! Sorry, it took too long to approve it.
Due to some limitations on our CI, I'm continuing this in a separate PR (#490).
For future reference, I'm also updating Chart.yaml and CHANGELOG.


Validation chain .in() method

The new validation chain reduces 50% of my validation code. However, there is something that I want to further enhance.
Let's take this situation, I have the exact same validation chain for the same field BUT it in different places (e.g. body vs params).
I can do this with 2 options with some cons:

2 validation chains: this will lead to duplicate code
User check rather thang checkBody or checkParams: this will leads to some unexpected side effect. E.g. foo field is optional required in the body, but if the client send foo field in the params which, if not pass the test, will throw an validation error.

My proposal:
Add a new method .in(locations) which override the location of this validation chain whether which check function is used.
It is somewhat like optional() function and the old v3 in field in schema validation.

For example:
const checkFoo = check('foo').isInt().withMessage('Invalid');

And in the router I can do something like this:
router.get('/',
  [
    checkFoo.optional().in(['query'])
  ], (req, res, next) => {
  // ...
});

router.post('/',
  [
    checkFoo.in(['body'])
  ], (req, res, next) => {
  // ...
})

Ah, yes. That's a good idea 👍

Closing. Let me know of further motivation for this.