add new custom step: NLP - Extract Entities

Please include answers to these questions as part of your pull request
In the GitHub webUI, use the Write tab to modify the Markdown text that is part of the pull request. For each question simply place an X inside the square brackets, [X], that represents your answer. Make sure there are no blanks inside the brackets, otherwise MarkDown doesn't render properly. Using the Preview tab while editing this form, you can see the formatted/rendered version of the message.

Q1: Confirm that you have the right to submit the code that is being contributed. Please consider the origin of your code and confirm you have the appropriate rights to make the submission subject to the Apache 2.0 license that applies to everything in this repository of custom steps. If so, follow the instructions for the Contributor Agreement (which is based on the industry-standard Developer Certificate of Origin (DCO)).

[X] Yes, I have the right to submit the contributed code on behalf of myself, my company, or any other owner of the code. I have also attached my signed copy of the DCO to this message.
[ ] No

Q2: Confirm that your contribution does not include any personally identifiable information (PII), for example, in any examples used in your README file.

[X] My contribution does NOT include PII data
[ ] My contribution includes PII data

Q3: Confirm your contribution does not include any encryption or other export-controlled technology.

[X] My contribution does NOT contain encryption or other export-controlled technology
[ ] My contribution includes encryption or other export-controlled technology
Toporowski Github Contributor Agreement.txt

Looking at your SAS code, I wondered why you move the data between CAS and Compute? The kupcase() function is available in CAS, so I think you could do all post-processing directly in CAS.
So instead of this:
data work.POS_ENTITY_EXTRACT (drop= _ComplexTag_ _Attribute_ _Parent_ _Sentence_ _Paragraph_ );
   set casuser.POS_ENTITY_EXTRACT;
run;
data &outputtable1;
   set work.POS_ENTITY_EXTRACT;
   where kupcase(_Role_) contains "NLP"
         &exclude_NounGroup          ;
run;

couldn't you do the following?
data &outputtable1;
   /* Use data set drop= option to avoid reading unused columns */
   set casuser.POS_ENTITY_EXTRACT(drop= _ComplexTag_ _Attribute_ _Parent_ _Sentence_ _Paragraph_ );
   where kupcase(_Role_) contains "NLP"
         &exclude_NounGroup          ;
run;

Not sure how much CAS automatically optimizes, but you might even want to consider putting the where-clause on the set statement as well using another data set option. Here is link to the current SAS Viya doc for the where= data set option This should result in unsed records being skipped while reading data from disk into memory.

Yes, that works!  I couldn't get that to work a year ago, and I have had this custom step published in the internal GitLab repository with the following notes:
I could not figure out how to get the where clause to work in the first data step without getting an "Out of Memory" error (something to do with using a where clause on a caslib table?). So did it in two datasteps. If you can find a more efficient way to do this please update the step code. Also, maybe this can all be done in the tpParse action set, but I couldn't figure out how to do that either. Please update the step code if that is possible.
Maybe I just had a coding error.  Or maybe the more recent Viya release works better?
Thanks, it much more efficient this way!
I was also able to implement your suggestion for the "data set where = " option.  The code looks like this now.
data &outputtable1;
/* Use data set drop= option to avoid reading unused columns /
set casuser.POS_ENTITY_EXTRACT( drop= ComplexTag Attribute Parent Sentence Paragraph
/ use data set where= option to avoid reading rows not needed */
where = (kupcase(Role) contains "NLP" &exclude_NounGroup)      ) ;
run;
I notice that in a flow the output &outputtable1 in the data step always defaults to the WORK library.  Is there any way to get it to go to a caslib by default?
Arn
From: Wilbram Hazejager @.>
Sent: Tuesday, December 05, 2023 11:40 AM
To: sassoftware/sas-studio-custom-steps @.>
Cc: Arnold Toporowski @.>; Mention @.>
Subject: Re: [sassoftware/sas-studio-custom-steps] add new csutom step: NLP - Extract Entities (PR #99)
You don't often get email from<EMAIL_ADDRESS>Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
EXTERNAL
Looking at your SAS code, I wondered why you move the data between CAS and Compute? The kupcase() function is available in CAS, so I think you could do all post-processing directly in CAS.
So instead of this:
data work.POS_ENTITY_EXTRACT (drop= ComplexTag Attribute Parent Sentence Paragraph );
set casuser.POS_ENTITY_EXTRACT;
run;
data &outputtable1;
set work.POS_ENTITY_EXTRACT;
where kupcase(Role) contains "NLP"
     &exclude_NounGroup          ;

run;
couldn't you do the following?
data &outputtable1;
/* Use data set drop= option to avoid reading unused columns */
set casuser.POS_ENTITY_EXTRACT(drop= ComplexTag Attribute Parent Sentence Paragraph );
where kupcase(Role) contains "NLP"
     &exclude_NounGroup          ;

run;
Not sure how much CAS automatically optimizes, but you might even want to consider putting the where-clause on the set statement as well using another data set option. Here is link to the current SAS Viya doc for the where= data set optionhttps://go.documentation.sas.com/doc/en/pgmsascdc/default/ledsoptsref/p0ny9o8t8hc5zen1qn3ft9dhtsxx.htm This should result in unsed records being skipped while reading data from disk into memory.

Reply to this email directly, view it on GitHubhttps://github.com/sassoftware/sas-studio-custom-steps/pull/99#issuecomment-1841176260, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOPEP2EIOFBSPJK4FON643TYH5E7RAVCNFSM6AAAAABADE6FV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBRGE3TMMRWGA.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>
Si vous pr?f?rez ne plus recevoir des messages commerciaux ?lectroniques par SAS, veuillez-vous d?sabonner icihttp://www.sas.com/emailpreferences/optout.html Merci.
If you do not wish to receive commercial electronic messages from SAS in the future, please click herehttp://www.sas.com/emailpreferences/optout.html to unsubscribe. Thank you.

To answer your question about temporary tables always going to work at the moment: A new feature is in the works to allow the user to specify a target library on the output port of a step. Not sure when that feature will be implemented though.

Here is a code snippet to determine the Active CAS library of the current session, which might come in handy.
%put Active caslib: %sysfunc(GETSESSOPT(&_SESSREF_, CASLIB));

And a snippet to drop the intermediate table POS_ENTITY_EXTRACT that is created by textParse.tpParse. When you do not specify a caslib in the table.dropTable call, it will use the active CAS library of the current session, so no need to figure out what that caslib is.
proc cas;
   table.dropTable / name="POS_ENTITY_EXTRACT"  quiet=true;
run; quit;

Could you add this? Also noticed that readme and About tab have typos in release dates. Seems you have mixed up v1.0 and v1.1. Could you fix that as well?

Re: Also noticed that readme and About tab have typos in release dates. Seems you have mixed up v1.0 and v1.1. Could you fix that as well?
The v1.0 has a 2022 date (published last year to the internal GitLab).  V1.1 is the first version to be submitted to github.  Should I just remove the v1.0 line since it was never in github?
From: Wilbram Hazejager @.>
Sent: Friday, December 08, 2023 4:02 AM
To: sassoftware/sas-studio-custom-steps @.>
Cc: Arnold Toporowski @.>; Mention @.>
Subject: Re: [sassoftware/sas-studio-custom-steps] add new custom step: NLP - Extract Entities (PR #99)
You don't often get email from<EMAIL_ADDRESS>Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
EXTERNAL
Here is a code snippet to determine the Active CAS library of the current session, which might come in handy.
%put Active caslib: %sysfunc(GETSESSOPT(&SESSREF, CASLIB));
And a snippet to drop the intermediate table POS_ENTITY_EXTRACT that is created by textParse.tpParse. When you do not specify a caslib in the table.dropTable call, it will use the active CAS library of the current session, so no need to figure out what that caslib is.
proc cas;
table.dropTable / name="POS_ENTITY_EXTRACT"  quiet=true;
run; quit;
Could you add this? Also noticed that readme and About tab have typos in release dates. Seems you have mixed up v1.0 and v1.1. Could you fix that as well?

Reply to this email directly, view it on GitHubhttps://github.com/sassoftware/sas-studio-custom-steps/pull/99#issuecomment-1846799303, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOPEP2BZY4N5N7JAP34E3NLYILJQZAVCNFSM6AAAAABADE6FV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBWG44TSMZQGM.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>
Si vous pr?f?rez ne plus recevoir des messages commerciaux ?lectroniques par SAS, veuillez-vous d?sabonner icihttp://www.sas.com/emailpreferences/optout.html Merci.
If you do not wish to receive commercial electronic messages from SAS in the future, please click herehttp://www.sas.com/emailpreferences/optout.html to unsubscribe. Thank you.

To summarize: The step now  has been enhanced to do the following:

doesn't touch the data unnecessary anymore
issues an error with a self-explanatory message when the input table is not a CAS table, and stops further processing
dynamically retrieves the caslib associated with a libname pointing to CAS, so does not assume that libref is same as caslib name
dynamically retrieves active caslib and writes intermediate results table to active caslib
performs preprocessing on that intermediate results table and writes it to the "table on the output port". So when no table is connected, it writes it to SAS work. When a CAS table is connected it writes it to that table and the data stays in CAS and is not transferred to compute.

SAS Internal Due Diligence Review - COMPLETED on 15DEC2023

@ArnoldToporowski , am I correct that the sample data you provided is fake data, and you have the rights to distribute it?

Yes, the sample data is fabricated data.  Thanks for asking.


Question about tokenization

❓ Questions & Help

I have a question about the "tokenization_albert.py" provided by huggingface. It was built on the basis of sentencepiece, but i have no idea how to use it with the squard problem. For instance ,how to add special tokens to the pretrained vocab_list, since it was provided with a model.
I found that the original bert tokenization still work well on your project! So are there any potiential problems of using the pretrained albert model with bert tokenization ?  Thank you so much !

Of cause, you can use it, but Albert tokenizer and Bert tokenizer is
different by whole word and sub word. I think if u use Bert tokenizer, your
length of tensor with the same sentence will be longer than Albert.
On Fri, Dec 6, 2019 at 13:46 gyc913<EMAIL_ADDRESS>wrote:

❓ Questions & Help
I have a question about the "tokenization_albert.py" provided by
huggingface. It was built on the basis of sentencepiece, but i have no idea
how to use it with the squard problem. For instance ,how to add special
tokens to the pretrained vocab_list, since it was provided with a model.
I found that the original bert tokenization still work well on your
project! So are there any potiential problems of using the pretrained
albert model with bert tokenization ? Thank you so much !
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pohanchi/huggingface_albert/issues/3?email_source=notifications&email_token=AIEAE4FB4FSHML4JWMRUDVLQXHRKDA5CNFSM4JWMMJ22YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4H6QQ2JA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIEAE4F5MRUTYH5L7XLZNZLQXHRKDANCNFSM4JWMMJ2Q
.

Oh I make wrong. You can’t use Bert tokenizer. Because the token ID with
Albert and Bert is different. So u can’t use Bert tokenizer
On Fri, Dec 6, 2019 at 13:48<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:

Of cause, you can use it, but Albert tokenizer and Bert tokenizer is
different by whole word and sub word. I think if u use Bert tokenizer, your
length of tensor with the same sentence will be longer than Albert.
On Fri, Dec 6, 2019 at 13:46 gyc913<EMAIL_ADDRESS>wrote:

❓ Questions & Help
I have a question about the "tokenization_albert.py" provided by
huggingface. It was built on the basis of sentencepiece, but i have no idea
how to use it with the squard problem. For instance ,how to add special
tokens to the pretrained vocab_list, since it was provided with a model.
I found that the original bert tokenization still work well on your
project! So are there any potiential problems of using the pretrained
albert model with bert tokenization ? Thank you so much !
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pohanchi/huggingface_albert/issues/3?email_source=notifications&email_token=AIEAE4FB4FSHML4JWMRUDVLQXHRKDA5CNFSM4JWMMJ22YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4H6QQ2JA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIEAE4F5MRUTYH5L7XLZNZLQXHRKDANCNFSM4JWMMJ2Q
.

If u want to add special token, I think I will add that feature in the
future.
On Fri, Dec 6, 2019 at 13:49<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:

Oh I make wrong. You can’t use Bert tokenizer. Because the token ID with
Albert and Bert is different. So u can’t use Bert tokenizer
On Fri, Dec 6, 2019 at 13:48<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:

Of cause, you can use it, but Albert tokenizer and Bert tokenizer is
different by whole word and sub word. I think if u use Bert tokenizer, your
length of tensor with the same sentence will be longer than Albert.
On Fri, Dec 6, 2019 at 13:46 gyc913<EMAIL_ADDRESS>wrote:

❓ Questions & Help
I have a question about the "tokenization_albert.py" provided by
huggingface. It was built on the basis of sentencepiece, but i have no idea
how to use it with the squard problem. For instance ,how to add special
tokens to the pretrained vocab_list, since it was provided with a model.
I found that the original bert tokenization still work well on your
project! So are there any potiential problems of using the pretrained
albert model with bert tokenization ? Thank you so much !
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pohanchi/huggingface_albert/issues/3?email_source=notifications&email_token=AIEAE4FB4FSHML4JWMRUDVLQXHRKDA5CNFSM4JWMMJ22YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4H6QQ2JA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIEAE4F5MRUTYH5L7XLZNZLQXHRKDANCNFSM4JWMMJ2Q
.

I think add special token that it is same with GPT-2 tokenizer . I can reference by that python code and generate it

Thank you ! I will learn your suggestions

By the way, my squad task, you just need to follow up the readme, download
the squad data and run the command. You will start to train Squad 1.1 and
2.0
On Fri, Dec 6, 2019 at 13:58 gyc913<EMAIL_ADDRESS>wrote:

Thank you ! I will learn your suggestions
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pohanchi/huggingface_albert/issues/3?email_source=notifications&email_token=AIEAE4FJPTYHSROWNE3AT2LQXHSZHA5CNFSM4JWMMJ22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGDDNJA#issuecomment-562443940,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIEAE4G5MFKN6QSBYIDLOPTQXHSZHANCNFSM4JWMMJ2Q
.


Support internationalization of interface components

Thanks for the library. I found it from a comment you left on a Reddit thread. It was easy to incorporate into my SvelteKit app. I like the scope of what you've chosen to take on here, and the UX is pretty nice.
I'm offering an internationalized app with multiple translations, but using the tabbed interface, I see that the tab names "Write" and "Preview" are hard coded. Could you support passing in an optional dictionary of strings for these and any other strings that appear in the interface?
If you like the idea, I'd be happy to contribute a PR. Might take me until after the December holidays to get to it though.

Hi! I created a simple PR that add the option to customize the labels you mentioned. You can check it out and tell me if that's ok for you.

I'm gonna merge that for now and mark the issue as completed. Feel free to reopen it/open another for other issues.

Thanks for adding the feature while I was out on vacation. Works great. Here's how it looks in my app, when the french translation is active:

I may put in a PR to add the ARIA labels to the list of customizable labels, but this is great. I don't often use third party UI components because of the need to do precise customizations, but this editor worked out well, and you're on top of issues. Thanks again!

Glad you liked it! Thanks for the kind works :)


Add section about the removal of networks

Add a small section that documents uninstall.

Thanks @fabaff!


[ja] Translate content/en/blog/_posts/2024-08-20-websocket-transition.md

This is a Feature Request
What would you like to be added
Japanese translation for the following blog post:

path: content/en/blog/_posts/2024-08-20-websocket-transition.md
URL: https://kubernetes.io/blog/2024/08/20/websockets-transition/

Why is this needed
There is no Japanese translation for this page
Comments
/language ja
/assign
/area localization

/triage accepted


Improve PrintOperation to allow for parallel page drawing

The PrintingOperation object has the methods set_defer_drawing() and draw_page_finish() which the docs mention to be used when printing the pages on other threads. However, because the PrintOperation Object is not Send + Sync, I can't call draw_page_finish() from another thread.
Maybe there could be a method called draw_page_par() which requires the closure to be Send or something similar?
An example using unsafe (and probably rightfully so) code is:
use gtk4::Align;
use gtk4::{
    glib, glib::clone, prelude::*, Application, ApplicationWindow, Button, PrintOperation,
    PrintOperationAction, Unit,
};

fn main() {
    let app = Application::new(Some("gtk4parprinting.gtk4parprinting"), Default::default());

    app.connect_activate(move |app| {
        let window = ApplicationWindow::builder()
            .title("gtk4parprinting")
            .default_width(200)
            .default_height(50)
            .application(app)
            .build();

        let print_button = Button::builder().valign(Align::Fill).halign(Align::Fill).vexpand(false).hexpand(false).label("print").build();

        print_button.connect_clicked(clone!(@weak window => move |_print_button| {
            let print_op = PrintOperation::builder().unit(Unit::Points).build();

            print_op.connect_begin_print(move |print_op, _print_cx| {
                print_op.set_n_pages(5);
            });

            print_op.connect_draw_page(move |print_op, print_cx, page_no| {

                print_op.set_defer_drawing();

                // Wraps print_op and print_cx in the type that is Send + Sync
                let print_op_ss = unsafe { force_send_sync::SendSync::new(print_op.clone()) };
                let print_cx_ss = unsafe { force_send_sync::SendSync::new(print_cx.clone()) };

                std::thread::spawn(move || {
                        let cairo_cx = print_cx_ss.cairo_context().expect("print_cx.cairo_context() failed");

                        cairo_cx.set_source_rgba(1.0, 1.0, 1.0, 1.0);
                        cairo_cx.paint().expect("cairo_cx.paint() failed");

                        cairo_cx.set_source_rgba(0.0, 0.0, 1.0, 1.0);
                        cairo_cx
                            .show_text(&format!("page: {}", page_no + 1))
                            .expect("cairo_cx.show_text() failed");
                        cairo_cx.stroke().expect("cairo_cx.stroke() failed");

                        print_op_ss.draw_page_finish();
                    });
            });

            print_op.run(PrintOperationAction::PrintDialog, Some(&window)).expect("print_op.run() failed");
        }));

        window.set_child(Some(&print_button));
        window.present();
    });
    app.run();
}

}

instead of _par I think we should follow the convention of using _local for the variant that does not require Send.


Auto-rerun if script is taking longer than 1min

If script has run for >1min without producing anything, decrease the number of words being fit by 1, and try again.

Added retry logic that can be set at any time limit. Rather than removing words, it reshuffles the words. Surprisingly this doesn't help much, implying that the trick may be related to letter frequencies


add hugetlbfs and hugepages handling for containers

Signed-off-by: Christian Brauner<EMAIL_ADDRESS>

On Mon, Mar 02, 2020 at 08:41:10AM -0800, TomP wrote:

tomponline commented on this pull request.

@@ -926,3 +926,9 @@ Introduces the ability to create a storage pool from an existing non-empty volum
This option should be used with care, as LXD can then not guarantee that volume name conflicts won't occur
with non-LXD created volumes in the same volume group.
This could also potentially lead to LXD deleting a non-LXD volume should name conflicts occur.

+## container_syscall_intercept_hugetlbfs
+When mount syscall interception is enabled and hugetlbfs is specified as an
+allowed filesystem type LXD will mount a separate hugetlbfs instance for the
+container with the uid and gid mount options set to the container's root uid
+and gid. This ensure that processes in the container can use hugepages.
@brauner just a nit "ensures",

Woops, thanks!

On March 2, 2020 6:40:32 PM GMT+01:00, "Stéphane Graber"<EMAIL_ADDRESS>wrote:

stgraber commented on this pull request.

if args.fstype == "hugetlbfs" && args.data == "" {

 args.data = fmt.Sprintf("uid=%d,gid=%d", nsuid, nsgid)

 args.shift = false

}

This only handles the case where the user didn't specify a uid/gid.
If uid or gid are specified by the user, we should parse them and
shift them using the instance map.

Yeah I wasn't sure about that. Or whether we should not guess. But sure.

On March 2, 2020 6:42:22 PM GMT+01:00, "Stéphane Graber"<EMAIL_ADDRESS>wrote:

stgraber commented on this pull request.

@@ -926,3 +926,16 @@ Introduces the ability to create a storage pool
from an existing non-empty volum
This option should be used with care, as LXD can then not guarantee
that volume name conflicts won't occur
with non-LXD created volumes in the same volume group.
This could also potentially lead to LXD deleting a non-LXD volume
should name conflicts occur.

+## container_syscall_intercept_hugetlbfs
+When mount syscall interception is enabled and hugetlbfs is specified
as an
+allowed filesystem type LXD will mount a separate hugetlbfs instance
for the
+container with the uid and gid mount options set to the container's
root uid
+and gid. This ensure that processes in the container can use
hugepages.
+
+##limits_hugepages.*
Missing space and invalid name (*) for an extension.

Ohz I wasn't too hasty it seems.

On March 2, 2020 6:43:22 PM GMT+01:00, "Stéphane Graber"<EMAIL_ADDRESS>wrote:

stgraber commented on this pull request.

+limits.hugepages.64KB                       | string    | -
| yes           | container         | Fixed value in bytes (various
suffixes supported, see below) to limit number of 64 KB hugepages
+limits.hugepages.1MB                        | string    | -
| yes           | container         | Fixed value in bytes (various
suffixes supported, see below) to limit number of 1 MB hugepages
+limits.hugepages.2MB                        | string    | -
| yes           | container         | Fixed value in bytes (various
suffixes supported, see below) to limit number of 2 MB hugepages
+limits.hugepages.1GB                        | string    | -
| yes           | container         | Fixed value in bytes (various
suffixes supported, see below) to limit number of 1 GB hugepages

We still need a note somewhere that the availability of the various
sizes differs based on architecture.

Probably good to do it in api_extension.md?

No, the doc about hugepages should probably be a separate paragraph in instances.md.
api_extensions.md is really just meant for developers.

No, the doc about hugepages should probably be a separate paragraph in instances.md.
api_extensions.md is really just meant for developers.

I added a note both to the table entry for each key but added a whole section on hugepage limits too.

So we're still missing the container startup logic. The code in place will handle a live change to the limit just fine but there's no code to apply the limit on initial startup.
Also, can you split the changes to lxd/cgroup to their own commit?

So we're still missing the container startup logic. The code in place will handle a live change to the limit just fine but there's no code to apply the limit on initial startup.
Also, can you split the changes to lxd/cgroup to their own commit?

Yeah, I've missed to finish that part yesterday. Though you might not have caught my message aobut it.

Testsuite passed


props.history.goBack() not work in ios WeChat browser after first history.push('xxx')

i test many times for this behavior and find the bug only in ios WeChat after first history.push()
for example :   in the Main page i click a link (use history.push) and jump into A page, but the back button ( use props.history.goBack()) is not work , but when i go on click a link(history.push) in A page jump into B page, the back button in B page can go back to A page and now the back button in the A page is working fine(can go back to Main page) !!!
No such strange behavior in Android WeChat！
In ios WeChat, you must jump 2 times to be normal, all the first jump operation cannot go back
both HashBrowser and Browser router are all find this bug！
below is my relation package
"react": "^16.11.0",
"react-dom": "^16.11.0",
"react-router-dom": "^5.2.0",

@wowtrxqn
I have the same problem，have you solved it?

@wowtrxqn  I have the same problem，have you solved it?

@wowtrxqn  I have the same problem，have you solved it?

@wowtrxqn I have the same problem，have you solved it?

@wowtrxqn I have the same problem，have you solved it?


Allow new excluded_exceptions config

Adds new config option from https://github.com/getsentry/sentry-php/pull/583
Shouldn't be merged until a new release is made.
I'd really appreciate if this was picked into the 0.8.x branch as well, as I'm currently having to use that version for old php support... its horrible but true...

Ehi @mcfedr, the Sentry SDK released a 1.9! Can you update your PRs?

Done, here is a new version with change to composer.json

Thanks! Can you also open the same PR targeting the 1.x branch? I would like to backport it there too.

@Jean85 Maybe its time for a new version of the bundle as this isn't in 2.0.3? Its been quite a while now...

Ouch, I totally forgot about this, thanks for reminding me!
I've added this to the changelog: https://github.com/getsentry/sentry-symfony/pull/159/commits/d39d086f3d4b7a86d7a1d23fef71c549e73a3649
I hope to merge it ASAP.

sentry-symfony started relying on the SDK's excluded_exceptions implementation, which is great but removes that the SDK is aware of Symfony exceptions that did get caught by default before.
We did not have any exceptions configured in excluded_exceptions yet, meaning that since skip_capture was removed in v3 the defaults the implementation used to rely on are gone.
When upgrade to v3, Sentry starts capturing HttpExceptionInterface instances. Did create PR #269 to restore the old behavior.


Support Metrics Report November 2019

Support Metrics Report November 2019
This report looks at the trends around support to help us better understand areas we need to improve and highlight key topics for the month.
Tickets received

The number of tickets received has dropped by 37% from 295 in October to 187 in November. If we take into account the tickets falling under the recruitment tag and blank tickets received, the decrease is more modest - 19% - with 118 tickets raised in November compared to 146 in October. This will be the more accurate representation of real support issues raised by our users.
Due to our decision of reviewing pending tickets older than 7 days, we can see a trade-off in November between Unresolved and Resolved tickets as we closed quite a few pending tickets. This also explains why the number of resolved tickets for November is higher than the number of tickets raised.

Expenses
The most obvious trend to note from this month is the reduction in the issues raised around the expenses topic - dropping by over 50% from 20 tickets in October to 9 in November. With the expense flow is currently being redesigned, we hope to see a more significant reduction in issues going forward.
Blank tickets
During November, 61 tickets were closed without assigning them a tag because they were received without any content (which is in line with the 67 tickets during October). We have opened a Github issue to investigate this and suggest they be stopped if possible as it uses up support capacity to deal with them.
Recruitment
We used Freshdesk to track recruitment inquiries, but those tickets have been excluded from this report as they don’t have relevance to support analysis. These numbers are also much less significant in comparison with the previous month (8 tickets vs 82 in October).
Trends and Recommendations
After removing irrelevant tickets, we had 118 that required support attention.

We have been receiving complaints from a few Collectives about spam and this is something the team is currently looking into.
This month we received more enquiries about alternative methods of payment. One particular topic we are currently looking into is payments to Nigeria, where Paypal isn’t an option. We’re working with Samson on setting up a local Collective to manage payments for bounties using local mobile banking.
We can see a drop in the questions related to gift cards and this seems to be a direct outcome from the upgrades done in this space, linking the user to the card directly rather than the old email notification method.
Project support enquiries tend to cover a wide variety of topics around getting better use of our system. Nothing of significance is worth noting this month that has contributed to the increase in the number of tickets raised.
We have had a handful of occurrences where there were problems in processing the payments to users, whether due to incorrect email details provided for PayPal profile or an error on our behalf. In all cases, these issues were promptly addressed and resolved and we record our learnings in the process.

Any feedback on the above is welcomed - @alanna @piamancini @Betree @znarf @xdamman

Thanks @alinamanko, that will be helpful to help prioritize!
I'm going to write specs for the SPAM / Moderation issue during this week.

Thank you @alinamanko - looks good!


Example Routes for Demo 11_1

Hi,
I'm new to Laravel and think these components look great!
I was hoping alongside the demo code (specifically demo 11_1) you could provide me an example set of routes as well? Currently my knowledge on routing is basic at best and if I could see how you handle routing to do things like sorting and searching keywords on filtering that would help out very much.
Thanks!

Hi,
Route::get('my-grid', 'MyController@showGrid');
That's all,
everything related to filtering / sorting are handled by Grids automatically

Did I answered your question?

Good morning,
I just got in for work. I am going to try this out again today. From what I remember I basically had my route setup so that the View was rendered with the grid when you hit the "'MyController@showGrid'" function in the routes.
After I would click sort, filter, etc. the page would blow up with a stack trace.
I will try and get it back to that state and post it for you to examine.
Thanks for your reply by the way! Great job on this.

Hi!
I'm closing this issue. If you still have a questions, submit new issues please.


what should happen next?

hello I trying to run ETS2 multiplayer but cant get launcher to run. here is the terminal output:
http://imgur.com/a/H1tTG
And then nothing happens
wine version 2.0.0
ubuntu 16.04
please help!

Actually it looks like the launcher ran correctly according the logs you provided. The game even seems to start correctly looking at the last 4 lines of the log.
Have you tried running  the game from steam ? And steam obviously needs to be launched using wine. It won't work if you use the native linux version of steam.
Do you get an error message or does it just exit without doing anything ?

steam is running and exiting without doing anything.

EDIT:
I run this command  and I get nothing:
WINEPREFIX=~/wineprefix64 ./truckersmp-cli /home/swipis/wineprefix64/drive_c/Program\ Files\ (x86)/steam/steamapps/common/Euro\ Truck\ Simulator\ 2/
but if I run without WINEPREFIX=~/wineprefix64 MP is running!

I did a tests, and looks like you started steam in a different WINEPREFIX than truckersmp-cli.
If I had to guess, I would say you started steam with the default WINEPREFIX (.wine) and then tried to start truckersmp-cli with WINEPREFIX=~/wineprefix64.
I reckon your fix of using .wine works because both .wine and .wineprefix64 are 64bits installations.
To confirm that that's indeed what is happening, try to start steam using WINEPREFIX=~/wineprefix64
If that doesn't work then I'm completly wrong ^^

lhark yes you are right I was running steam with default prefix and now I did what you said and working fine! thank you!


propagate Extension and ExtraExtension from CSR to Certificate

PR makes the Extensions and ExtraExtensions fields to propagate from CSR into the signed Certificate.
Fixes #282.

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

done in #292


fea-ext-ignore_.gitignore_paths_in_header_test

Modulus Pull Request
Description

Changing the header test to ignore items in .gitignore. This is needed for pre-commit
Addresses this issue: https://github.com/NVIDIA/modulus-launch/issues/52
Checklist

[x] I am familiar with the Contributing Guidelines.
[x] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.
[x] The CHANGELOG.md is
up to date with these changes.

Dependencies

/blossom-ci

/blossom-ci

@mnabian Can you resolve the conflicts on this PR?

/blossom-ci


Make container struct members public and stop allocating base containers

In the interests of "privacy" the containers have been defined in source and internals are inaccessible outside. This requires a double allocation, one for the container itself and then for the data. In the next version of the project I will reverse this decision to allow containers to be stack allocated. Container members such as length and dimensions will be public but their access will still be encouraged to be done through the API.

Updated to just refer to array class. Will create separate issues for each container type.


Error when setState empty string

I got error if the state is empty string:
Error: NoPayloadError: mutate requires a payload.

Thanks @aldiand! This was released in 4.0.0


all_content_dump.html (template) routing incorrectly

When a visitor lands on https://localhost:8000/article/all_content_dump/, I am expecting Django to serve /templates/essays/all_content_dump.html as specified at line 13 in /essays/urls.py. However right now, Django returns a 404. This needs to be corrected.
The other three lines inside that same urls.py need to route correctly as well:

'article/objections/'
'article/content_changelog/',
'article/bibliography/'

Resolved in PR #97

Resolved in PR #98


Do not require vendoring packages in git (i.e., use a global cache)

Problem
CUE only looks for packages in cue.mod/{pkg,gen,usr} (merging these directories). This has required us to commit packages to git.
Solution
We do have our own dagger.mod and dagger.sum files, like Go. We can make them good enough to be depended on to reinstall packages in new git clones like in CI (verifying checksums). This makes committing cue.mod/pkg less necessary.
But I want to propose using a global cache here.
Global cache
We can take a hint from cuemod, which downloads modules into a global cache (GOMODCACHE in that case), and then symlinking them into cue.mod/pkg. @morlay took inspiration from pnpm (homebrew does this as well), although it uses hard links instead of symlinks (there's a whole discussion on why that's better).
We could do the same. Use the global cache to avoid re-downloading repos and also for verifying the checksums. At which point, link packages into cue.mod/pkg so that CUE can read them. CI can also be configured to cache this directory.
Ask upstream to add setting
It would be much better if upstream supported looking for packages in a $CUEMODCACHE environment variable, besides cue.mod/pkg, even if they don't use/manage it themselves yet. It would avoid the need to symlink and I think it's a very easy change on their part. Planned even.
UPDATE: not an easy change to ask because of lack of version concept.

Cue not  resolve sub vendor (cue.mod/pkg) like go vendor did.
Once import path resolved, all sub module will use the first one
cue.mod
├── module.cue
└── pkg
    └── x.io
        ├── a
        │   ├── a.cue
        │   └── cue.mod  // cue will never resolve this folder
        │       ├── module.cue
        │       └── pkg
        │           └── x.io
        │               └── b
        │                   └── b.cue
        └── b
            └── b.cue

If cue support sue vendor resolve,
we could create store like pnpm did, to support multi versions (just for semver conflicts)
https://github.com/morlay/cue-mod-multiple-versions


NIFI-2887 NumberFormatException in HL7ExtractAttributes for repeating segments with order control codes

Fixes NIFI-2887 and adds more test cases

+1 LGTM, ran the unit tests and on a full NiFi, verified the NFE is not presented and the flow file is parsed successfully. Merging to master, thanks!

+1 LGTM, ran the unit tests and with a full NiFi, verified the NFE is not presented and messages are parsed successfully. Merging to master, thanks!


About demo-data

Thanks for your tutorial!
However, where can I download the demo-data, there are no 'cellranger-res/Pre-B/outs/filtered_feature_bc_matrix/'  and 'cellranger-count/' paths on your github.

Hi！Due to the project is unpublished, so the raw count is not available at present.  However, I uploaded the processed data (most .rds object) to Baidu Netdisk. The download link is updated in the README.md. I'm not sure whether you can download the demo-data from Baidu Netdisk. If you meet any question, please let me know.

Hi! Thank you for your sharing, now all the scripts can run smoothly. I'll close this issue.


Update index.adoc(Include Requirements)

Is there any issue with the code ?

It's not related to your PR. Look at the log.

Oh okay


能不能将pixi原生对象放入fairygui 容器里面？

fairygui是不是只能作为hud使用， 不能将pixi显示对象放入任何fairygui容器内部？
@jcyuan

it does.
    let myObj = new PIXI.Graphics();     ....     (fgui.GRoot.inst.getChildByName("test") as fgui.GGraph).setNativeObject(myObj);
create an empty graph in the editor, and call setNativeObjuect to insert any native object into it.
but don't forget to remove/destory your own object while destroying the fgui object it was related to.


Fix IRI serialization

In calamus 0.4.0, IRI are serialized as xsd:string when using add_value_types=True. They should instead be serialized as xsd:anyURI. This PR addresses the issue.
Example:
from dataclasses import dataclass
from typing import Optional

from calamus.schema import JsonLDSchema
from calamus import fields
from rdflib.namespace import Namespace

SDO = Namespace("http://schema.org/")

@dataclass
class Organization:
    """See http//schema.org/Organization"""

    _id: str
    logo: Optional[str] = None

class OrganizationSchema(JsonLDSchema):
    _id = fields.Id()
    logo = fields.IRI(SDO.logo)

    class Meta:
        rdf_type = SDO.Organization
        model = Organization
        add_value_types = True

sdsc = Organization('https://datascience.ch', logo='https://datascience.ch/wp-content/uploads/2019/04/logo_SDSC-300x82.png')

print(OrganizationSchema().dumps(sdsc, indent=2))

Output before the PR:
{
  "@id": "https://datascience.ch",
  "http://schema.org/logo": {
    "@id": {
      "@value": "https://datascience.ch/wp-content/uploads/2019/04/logo_SDSC-300x82.png",
      "@type": "http://www.w3.org/2001/XMLSchema#string"
    }
  },
  "@type": [
    "http://schema.org/Organization"
  ]
}

Output after the PR:
{
  "@id": "https://datascience.ch",
  "http://schema.org/logo": {
    "@value": "https://datascience.ch/wp-content/uploads/2019/04/logo_SDSC-300x82.png",
    "@type": "http://www.w3.org/2001/XMLSchema#anyURI"
  },
  "@type": [
    "http://schema.org/Organization"
  ]
}

Hi @cmdoret thank you so much for opening a PR.
The way calamus currently handles it is clearly wrong. However, I'm not sure if the proposed solution is correct, either.
With the xsd datatype and without @id, it's not an IRI reference (reference to a node) anymore, it becomes a string property with type coercion.
I.e. in ttl <https://datascience.ch> <http://schema.org/logo> "https://datascience.ch/wp-content/uploads/2019/04/logo_SDSC-300x82.png"^^<http://www.w3.org/2001/XMLSchema#anyURI> . vs. <https://datascience.ch> <http://schema.org/logo> <https://datascience.ch/wp-content/uploads/2019/04/logo_SDSC-300x82.png> ., so it loses the <...>.
I think @id is already a sort of native type in JSON-LD. So not adding an XSD type in IRI fields might be more correct? Alternatively having two field types, one for IRI node references, one for IRI properties (which adds the xsd type) might fit better?
I couldn't really find any detail on if you could keep the @idness of the property, so to speak, while also adding an XSD type, maybe we could ask on the JSON-LD repo on what approach is correct?
It's certainly beyond me to judge if your proposed solution is identical to the output produced without add_value_types or if having it as a string, without <...> is meaningfully different (from a JSON-LD processor/RDF standpoint).

Thanks @Panaetius, I just had a chat with @rmfranken about this and there's no obvious way to represent xsd:anyURI in json-ld. However json-ld.org uses "@type": "@id" in the context of type coercion:
{
  "@context":
  {
    ...
    "homepage":
    {
      "@id": "http://schema.org/homepage",
      "@type": "@id"
    }
    ...
  }
...
  "homepage": "http://manu.sporny.org/",
...
}

This approach seems to work in our case as well:
{
  "@id": "https://datascience.ch",
  "http://schema.org/logo": {
    "@value": "https://datascience.ch/wp-content/uploads/2019/04/logo_SDSC-300x82.png",
    "@type": "@id"
  },
  "@type": [
    "http://schema.org/Organization"
  ]
}

Converted to the following ttl:
@prefix schema: <http://schema.org/>.

<https://datascience.ch>
    a schema:Organization;
    schema:logo
    <https://datascience.ch/wp-content/uploads/2018/04/logo_SDSC-300x82.png>.

Which is effectively the same as just omitting "@type". I see two potential solutions:

adding "@type": "@id" to IRIs with add_value_types=True
Never add "@type" for IRIs

I would lean towards the latter because with the conversion sequence json-ld > ttl > json-ld, it keeps the same data. By contrast, the first option loses the @type. What do you think?

Below are all the turtle serializations we considered for an IRI:

"myURI.com"^^xsd:anyURI :x: Not what we want (string that looks like a URI)
<myURI.com>  :heavy_check_mark:
<myURI.com>^^xsd:anyURI :x: Invalid turtle syntax
<myURI.com> rdf:type  xsd:anyURI :x: Incorrect: rdf:type is not appropriate for xsd datatypes

Which can just be serialized as { "@id": "myURI.com" }. This is what the last commit does:
{
  "@id": "https://datascience.ch",
  "http://schema.org/logo": {
    "@id": "https://datascience.ch/wp-content/uploads/2019/04/logo_SDSC-300x82.png"
  },
  "@type": [
    "http://schema.org/Organization"
  ]
}

If you think add_value_types=True should explicitely add "@type": "@id", I can change it.

I think add_value_types not doing anything on IRI fields makes sense :+1:

I had to change some settings to run tests on PRs created from forks. You can merge now :slightly_smiling_face:


[FLINK-1472] Fixed Web frontend config overview with wrong value.

I like the idea - that means that the webfrontend now shows what values are actually used, even if they are not in the config.
To make it even nicer, it would be cool to:

Show which values come from the config (top) and show for which ones what default is used (in a second section below)
add a Unit test that checks that all keys from the ConfigConstants are converted. I think it is easy to check this via enumerating all fields in the class using reflection.

The configuration view is separated into user and default now:

Nice, looks good to me.
+1 to merge

According to travis, the code doesn't seem to build.

The default key value pairs are added manually. It seems that some of the keys in ConfigConstant is deleted recently.

Hi,
sorry for the long wait on this. I really like the feature but the implementation is not scalable: If new config values are added this needs to be updated in several places now.
Could you change ConfigConstants and add a static initializer block that builds the hash maps that you manually build in DefaultConfigKeyValues using reflection. The code would just need to loop through all fields that have _KEY at the end, and then find the matching default value without the _KEY at the end. From the default value field the type of the value can be determined and it can be added to the appropriate hash map. This way, the defaults will always stay up to date with the actual config constants.

That would be a nice way, but

key/value pairs are not always distinguished by _KEY suffix, we have to format all the fields in ConfigConstants
some values are not in the ConfigConstants. e.g task manager slot = 1, akka_startup_time=aka_ask_timeout

In summary, we need a formatted one-to-one map of the key/value pairs in ConfigConstants

But then I think the solution is to normalise the constants in ConfigConstants.

I agree to enforce normalization in the ConfigConstants, by adding a test case that reflectively gets all fields and checks whether they start with "DEFAULT_" or end in "_KEY".
The mapping between keys/values will probably have to be done manually, unless we add annotations above the key fields declaring what the default value is.
The type of the default value should help to figure out what "get" method to use. (getInteger, getDouble) in order to get the correct conversion method.

Yes, but then we should change this now and not build more code on top of this that can fail in the future if someone forgets to add the names to the correct hash set in some other class.

I added a Jira for this: https://issues.apache.org/jira/browse/FLINK-1936

@qmlmoon We merged a new web dashboard a few days ago which makes this PR obsolete. Sorry for not merging it earlier. Could you close this PR please?
Thank you

@fhueske This issue is present in the new frontend also. See the attached screenshot:

We should probably continue in another PR though.

Yes, this should be addressed through a new PR.
The JIRA issue FLINK-1472 is still open.


ColumnDataSource's columns must be of the same length.

Hi,
I try to use panel 3d plotly surface when I give x,y axes parameter in 3d surface as surface =go.Surface(x=xutm , y=yutm ,z=z,colorscale='Earth') pn.panel(plotly_layout).servable() gives this error :
ColumnDataSource's columns must be of the same length. Current lengths: ('x', 78), ('y', 49), ('z', 65) when I omit x,y axes, it works.
Regards

Now fixed on master.


Option to add notes in editor

@amaldinesh7 _a

@jagannathBhat _a Please research on this.

@karthiknmenon Please take a look https://vimeo.com/695692608/8be1ea79a9

@karthiknmenon is this issue still relevant? As far as I understand, the notes feature is only used in neetoDesk. Since it is a single requirement, would it be better not to bring it to neetoEditor?

After some thought, this issue should be taken up in respective repositories.
Also, kb would have notes.
Closing this.


Data Dashboard for Expedited EDA

There’s a lot of available data, but determining its level of usefulness (via exploratory data analysis) can be a very time-consuming process. I have been working on a dashboard to visualize distribution information and samples of datasets efficiently and with no coding required. We have a pre-release hosted for use during Image Datapalooza. It is currently a bit Imageomics-focused, though images aren’t required to gather distribution statistics. I think it would be great to expand out the functionality for a more general audience as a way to quickly generate visuals for data or explore whether new data would be suitable for experiments without having to invest large amounts of time into EDA.

Could this integrate with the practice of documenting datasets via "dataset cards"? (see e.g. https://huggingface.co/docs/hub/datasets-cards):

Definitely. I have actually used it to generate a static display for one of the Institute's datasets (currently private) and plan to do so for more. I would be interested in integrating the interactive dashboard into a HF Space as well.

Turns out deployment in a HF Space is relatively straightforward with a Docker container. We have v1.0.0 on HF, and set up a development version for testing off our dev branch.


add action for importing sequences from NCBI

note: this could be accomplished using RESCRIPt... the only issue would be if there are specific metadata you want to import as well.

note: this could be accomplished using RESCRIPt... the only issue would be if there are specific metadata you want to import as well.


Add rvertnet

Insert URLs to the project here:
https://github.com/ropensci/rvertnet

[x] The projects is active, documented, open source licensed, shows usage from external parties and is directly targeting environmental sustainability. Find more details in the Contribution Guide.

All issues labeled as 'Good First Issue' of the project listed on OpenSustain.tech will be visible on ClimateTriage.com. This is a great way to welcome new community members to your project.

@slager


Use CDS in start.sh

Probably if we include the script like specified in the manpage of Java in the start.sh, we could benefit from a faster startup in a loop environment.
Some testing would be required to see if there are some performance improvements.
https://docs.oracle.com/en/java/javase/17/docs/specs/man/java.html#creating-a-shared-archive-file-and-using-it-to-run-an-application

Go for it!

Hi @Thihup just wondering, did you ever look at this?
Might be interesting too to base a blog article on.

Hi @arjantijms!
I have done some tests a few months ago, but a new test would be appropriate and document the results.
Also, I want to have a look using the -Xshareclasses from OpenJ9 (IBM Semeru), which is similar to CDS. Maybe creating a blog post about both would be awesome.
I'm very busy at the moment, but as soon as I get some free time I want to have a better look at this.


Repair equipment_model generator to handle ordering

Resolves #1662

set ordering to number of existing models in equipment_model generation
add validations for generating multiple objects
resolves error that can occur when randomly generating reservations for
equipment_model with max_checkout_length = 1

merging!

@zeffman can you add this to the changelog?


Implement EIP 4788 (Beacon block root)

closes #1272

Ah my bad, I had assumed this would be one of those things were we'd just provide it as is but not any check. I'll update it!

I think that if we don't change the storage of the beacon roots contract, we won't be able to verify the new state trie after the block, so I don't think there's a way to avoid it.


Gradle Sync Issue

ERROR: Unable to resolve dependency for ':app@brdDebug/compileClasspath': Could not resolve project :CoreLibrary.
Show Details
Affected Modules: app
ERROR: Unable to resolve dependency for ':app@brdDebugAndroidTest/compileClasspath': Could not resolve project :CoreLibrary.
Show Details
Affected Modules: app
ERROR: Unable to resolve dependency for ':app@brdDebugUnitTest/compileClasspath': Could not resolve project :CoreLibrary.
Show Details
Affected Modules: app
ERROR: Unable to resolve dependency for ':app@brdRelease/compileClasspath': Could not resolve project :CoreLibrary.
Show Details
Affected Modules: app
ERROR: Unable to resolve dependency for ':app@brdReleaseUnitTest/compileClasspath': Could not resolve project :CoreLibrary.
Show Details
Affected Modules: app
ERROR: Unable to resolve dependency for ':app@brdTestnetDebug/compileClasspath': Could not resolve project :CoreLibrary.
Show Details
Affected Modules: app
ERROR: Unable to resolve dependency for ':app@brdTestnetDebugAndroidTest/compileClasspath': Could not resolve project :CoreLibrary.
Show Details
Affected Modules: app
ERROR: Unable to resolve dependency for ':app@brdTestnetDebugUnitTest/compileClasspath': Could not resolve project :CoreLibrary.
Show Details
Affected Modules: app
ERROR: Unable to resolve dependency for ':app@brdTestnetRelease/compileClasspath': Could not resolve project :CoreLibrary.
Show Details
Affected Modules: app
ERROR: Unable to resolve dependency for ':app@brdTestnetReleaseUnitTest/compileClasspath': Could not resolve project :CoreLibrary.
Show Details
Affected Modules: app

This is a duplicate of #99.
Closing.


UI: Distinguish accepted vs not accepted followings

ie, on followings pages like https://fed.brid.gy/web/snarfed.org/following, indicate which users have Accepted our Follow and which haven't.

Related to #710


Cannot set property 'theme' of null

Issue Id: f40c44ff-2959-779d-f3ef-83f96ed97c10Versions - 1.15.0-  2c3e2c291c515619f64e7c9cddf14c565b7f3d92Stack TypeError: Cannot set property 'theme' of null/vs/workbench/common/theme.ts#L405:15 (theme)/vs/base/common/callbackList.ts#L63:26 (apply)/vs/base/common/event.ts#L123:26 (call)/vs/workbench/services/themes/electron-browser/workbenchThemeService.ts#L551:29 (fire)/vs/workbench/services/themes/electron-browser/workbenchThemeService.ts#L377:8 (updateColorCustomizations)/vs/base/common/callbackList.ts#L63:26 (apply)/vs/base/common/event.ts#L123:26 (call)/vs/workbench/services/configuration/node/configuration.ts#L324:34 (fire)/vs/workbench/services/configuration/node/configuration.ts#L305:82 (onBaseConfigurationChanged)/vs/base/common/callbackList.ts#L63:26 (apply)

duplicate of #32385


Could not find Mac using Client ID

Authentication with username works fine, but if using Client ID is it impossibile find the Mac.
Error in a attachment:
"Computer Record
Could not find this computer, please check the serial number"
If I set a wrong Client Secret I get a different error: "Authentication Error".

Hi there
At the moment I cannot reproduce the error you are seeing.
Could you check the permissions you are using for the role?
I'm using these.
Read Computers, Update Local Admin Password Settings, View Local Admin Password Audit History, Send Local Admin Password Command, View Local Admin Password

Hi there
Was wondering if you managed to check the permissions for the API Role?

Hello,
I have exactly the same problem, I can't use the login and password because I activated SSO on jamf pro so I have to use the client ID and the API key. However, I'm able to connect with the Jamf LAPS tool, but it can't find the serial number of the mac I entered.

Thanks


Update tour guide

Tour updates:
Info box, interactive filtering, layer visualization, import/export/ user feedback

@NEDJIMAbelgacem this the PR for this Task https://github.com/worldbank/WB-rezoning-explorer/pull/24

@akshay2408 Step6 needs to be re-written, as we have moved many of the contextual layers out of the legend.
Please update to:
"Toggle contextual layers to visualize the resulting output on the map and additional relevant spatial data"
Step7 - does not download 3 CSVs, but 1 CSV per window (Spatial/ Economic/ Weights)
Therefore please update text to:
"Export Tour
7 / 10
Click here to download the input parameters and associated thresholds in CSV format"
Step 8
Import Tour
8 / 10
You can import previously saved input parameters in CSV format"
Step 10 should come before Step 9 -> Step 9 (to re-open the guide when needed) should be last.
Step 10-update text to: " You can use the Feedback Form to report any bugs or give suggestions for improvements"
Step 10 is not fully visible, the box does not adjust to screen size:

Hi @ClaraIV
you mentioned this Step 10 is not fully visible, the box does not adjust to screen size:
So because zone analysis window is closed initially so that is why tour step is not fully visible.
Please suggest, do I only need to adjust the tour box size OR I need to show zone analysis window as well.

my comment is solely for the box that is not visible
On Wed, 10 May 2023 at 11:46, akshay2408 @.***> wrote:

Hi @ClaraIV https://github.com/ClaraIV
you mentioned this Step 10 is not fully visible, the box does not adjust
to screen size:
So because zone analysis window is closed initially so that is why tour
step is not fully visible.
Please suggest, do I only need to adjust the tour box size OR I need to
show zone analysis window as well.
—
Reply to this email directly, view it on GitHub
https://github.com/kartoza/rezoning-2-project/issues/61#issuecomment-1541597348,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACHU3HIQ334JFG462ZLTC63XFNIU7ANCNFSM6AAAAAAWY7R5II
.
You are receiving this because you were mentioned.Message ID:
@.***>

@ClaraIV for feedback it goes inside the zone analysis panel, but it is hidden. So user cannot see the feedback form step.

oh, I see what you mean!
Is it possible that the Results Window opens with this step and then closes back up?

Hi @ClaraIV
I debugged this issue and found that it is little difficult because tour box overlaps with zone analysis window.
So I have a solution for it and I implemented it for testing.

We can open zone analysis window before tour guide reaches to it (open it one step before it reaches to zone window)
And when user clicks on Finish it will be closed.

Hi @ClaraIV
could you please tell me what should be the correct order of steps.
This is not clear to me It might be good to re-arrange the steps so that both the window and the feedback form are presented at the same time; if not, perhaps let's add it?


A way to force placeholder in input field in multi select using "force-placeholder" attribute

What will be a use case for this? Also can you remove /dist files?

Closing due to history issues (should be no revert commit) & lack of activity - feel free to open a new PR on top of current master if desired.


⚖ Register Validator: NODEJUMPER

Network Choice
dentrite-1
Who are you?
Nodejumper is an experienced POS validator and staking provider. Contributing to Cosmos Ecosystem open-source projects, and offering valuable data analytics, scripting tools & services.
Delegator address
axone13dgy7rtjjvvlkzu2upgtntdyzy4hnr6e8820fx
gentx
{"body":{"messages":[{"@type":"/cosmos.staking.v1beta1.MsgCreateValidator","description":{"moniker":"NODEJUMPER","identity":"FFB0AA51A2DF5954","website":"https://nodejumper.io","security_contact":"validator@nodejumper.io","details":"Professional POS Validator services, Cosmos contributors and supporters, Restake.app compatible. Making the blockchain world a better place."},"commission":{"rate":"0.050000000000000000","max_rate":"0.200000000000000000","max_change_rate":"0.100000000000000000"},"min_self_delegation":"1","delegator_address":"","validator_address":"axonevaloper13dgy7rtjjvvlkzu2upgtntdyzy4hnr6ehjxany","pubkey":{"@type":"/cosmos.crypto.ed25519.PubKey","key":"3nLuQL2EYw8PhhG1gbOukZjc07j+9pgYB6h1JpWy7pk="},"value":{"denom":"uaxone","amount":"1000000000000"}}],"memo":"e5abfa9c78479a1ea35f54d7bfb1a25921b6f387@<IP_ADDRESS>:26656","timeout_height":"0","extension_options":[],"non_critical_extension_options":[]},"auth_info":{"signer_infos":[{"public_key":{"@type":"/cosmos.crypto.secp256k1.PubKey","key":"AkGn7anW86q+i8QgSlDptoa4XgpNB7qSUa7ocXX+1ov6"},"mode_info":{"single":{"mode":"SIGN_MODE_DIRECT"}},"sequence":"0"}],"fee":{"amount":[],"gas_limit":"200000","payer":"","granter":""},"tip":null},"signatures":["ULKr+qYVIOqauX/4lEZP20j2HpYvKVBFTwHzYmtUSOx+iMM+9rslWrMEXqicmVZrAzKqMOunc1vtp2KmEY42qw=="]}

😉 Here is the corresponding PR: https://github.com/axone-protocol/networks/pull/790


[Bug]: No directive automodule from sphinx-apidoc

To Dos

[X] I tested with the latest released version.
[X] I have checked the issues and think that this is not a duplicate.
[X] I added a very descriptive title to this issue.

Example Code (rst)
scripts package
===============

Submodules
----------

scripts.foo module
-------------------

.. automodule:: scripts.foo
   :members:
   :undoc-members:
   :show-inheritance:

Relevant log output
scripts.rst:10: (INFO/1) No directive entry for "automodule" in module "docutils.parsers.rst.languages.en".
scripts.rst:10: (ERROR/3) Unknown directive type "automodule".

Description
sphinx-apidoc fields don't appear to be recognized, how can this be solved?
Operating System
macOS
Operating System Details
No response
Python Version
3.11.4
rstcheck Version
These come from pip install rstcheck[sphinx]

rstcheck CLI Version: 6.1.2
rstcheck-core Version: 1.0.3

Additional Context
No response

Currently rstcheck only supports standard rst and default sphinx directives. Therefore you would need to put automodule on the ignorelist for directives.
Example .rstcheck.cfg file:
[rstcheck]
ignore_directives = automodule

There is currently a feature request which aims to change this: https://github.com/rstcheck/rstcheck-core/issues/68


🐛 trim space characters in events string

Trims whitespace from the events string passed to hears function in corebot. Passing a string separated by commas and spaces would previously trigger only for the first event. This PR allows comma plus space separated strings of event names to register the events for each.
Previously, given 'ambient, direct_message', it would split the string into an array that looked like this: ['ambient', ' direct_message'] notice the extra space in direct_message string. It would then register a listener to that event, with the space included, which could never be triggered by botkit.
I'm afraid this might be breaking, or introduce unintended results for people. However, it still will only trigger events they have explicitly stated, but developers might get surprised by their inert code running where it was not before.
Closes #807

Thanks @jonchurch and @anonrig!


Bot crass after typing !close

[NOTE] Creating new thread channel sniffieftw-0595
Closing thread dd33bc93-fe22-4916-aea7-a343737da5f1
Deleting channel<PHONE_NUMBER>19959812
TypeError: getLogChannel(...).createMessage is not a function
at Object.postLog (/home/ubuntu/ModMail/src/utils.js:65:19)
at Command.bot.registerCommand [as execute] (/home/ubuntu/ModMail/src/modules/close.js:108:11)
--> CRASH
this is what i get when i type it , also i have already done the URL ip thing :)

Make sure you set logChannelId in the config.

I have but I will double check and send an imgur of it
On Wed, Feb 27, 2019 at 10:40 PM Miikka<EMAIL_ADDRESS>wrote:

Make sure you set logChannelId in the config.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Dragory/modmailbot/issues/229#issuecomment-468134916,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AcMKFzXahshqtfj6ndwHb35lDDR23txXks5vR12pgaJpZM4bV-xV
.

--
-Brody Cole

The error in question appears when the bot can't find the specified log channel, so even if you've set it, verify the channel id is also valid and matches the current log channel (so e.g. if you've deleted and recreated the log channel, update the id as well).

does it have to say logchannel or can it be any channel?

Omg, I love you so much thank you, and can I ask another question?

Okay thanks that worked


57603d84-2caf-4e6d-91c8-adc0926d299a

this is for automated testing

9a7dbf41-151b-45d1-9c2c-990f8c1eb56d


Add catflix source and turbovid embed and fix nepu

This pull request resolves #XXX

[ ] I have read and agreed to the code of conduct.
[ ] I have read and complied with the contributing guidelines.
[ ] What I'm implementing was assigned to me and is an approved issue. For reference, please take a look at our GitHub projects.
[ ] I have tested all of my changes.


Using Ensembles With CloudKit

Hello
Is it possible to tell me how to use Core Data with CloudKit .
I did not find the two files
CDECloudKitFileSystem.h and CDECloudKitFileSystem.m in the Framework / Extensions directory
Do you have a Detailed tutorial
Thank you

CloudKit is only available in Ensembles v2, I'm afraid. The book explains how to use it, and the quick start chapter is a reasonable tutorial. See also the sample apps.


Identifier case conversion

Use case provided to me by email:
macro_rules! ld_r16_expr {
    ($($reg:ident)*) => {
        $(
            paste::item! {
                pub fn [<ld_ $reg _expr>] (val: Expr) -> Token {
                    token_for_opcode_two_args(
                        Mnemonic::Ld,
                        Register16::$reg.into(),
                        val.into(),
                    )
                }
            }
        )*
    }
}

ld_r16_expr!(Af Bc De Hl);

We would like some way to end up with functions ld_af_expr ld_bc_expr etc rather than ld_Af_expr ld_Bc_expr.
I would be be prepared to consider a PR implementing this as something like [<ld_ $reg:lower _expr>].

I just went to this repo to ask exactly about this feature! The paste macro is amazing. If it would allow for that, it would be just amazing * 100 <3

Paste 0.1.8 has added support for $var:lower. I'll keep this open and we may add support for other case modifiers as people need them.

@dtolnay, is it intentional that 0.1.8 is not published on crates.io?

Oops, published paste-impl but not paste. Fixed now.

I think we can close this issue at this point. We now support :lower, :upper, and :snake case as covered in https://github.com/dtolnay/paste/tree/0.1.10#case-conversion.


Error - Gen2 Camera Demo

This might have been referenced too many times, but I'm opening this over here for reference.
The gen2-camera-demo starts for a few seconds, and closes immidiately triggering an error -
StereoDepth config options:
    Left-Right check:   True
    Extended disparity: False
    Subpixel:           True
    Median filtering:   MedianFilter.MEDIAN_OFF
Creating Stereo Depth pipeline: MONO CAMS -> STEREO -> XLINK OUT
Creating DepthAI device
Starting pipeline
Traceback (most recent call last):
  File "main.py", line 306, in <module>
    test_pipeline()
  File "main.py", line 296, in test_pipeline
    image = q.get()
RuntimeError: Communication exception - possible device error/misconfiguration. Original message 'Couldn't read data from stream: 'depth' (X_LINK_ERROR)'

Sometimes with the error -
StereoDepth config options:
    Left-Right check:   True
    Extended disparity: False
    Subpixel:           True
    Median filtering:   MedianFilter.MEDIAN_OFF
Creating Stereo Depth pipeline: MONO CAMS -> STEREO -> XLINK OUT
Creating DepthAI device
Starting pipeline
Segmentation fault

It is using updated lib - depthai==<IP_ADDRESS>, and running the stereo demo, not point cloud, since it's running on armv7l - Rpi 4b 8gb
I checked the CPU load, and it was barely any, but still reciprocating an error.
Have kept this open for reference.

@csaba-luxonis - does this have anything to do with what you just addressed?

Thanks.  Asking.

No issues, had closed it bymistake.


No luck making this work with google json file

I am using a Google Service Account, and when I use Python it returns the access token fine.
https://developers.google.com/identity/protocols/OAuth2ServiceAccount#makingrequest
oauth_fields = {
        "type": "service_account",
        "project_id": "blah",
        "private_key_id": "blah",
        "private_key": "-----BEGIN PRIVATE KEY-----blah-----END PRIVATE KEY-----\n",
        "client_email": "blah@",
        "client_id": "blah",
        "auth_uri": "https://accounts.google.com/o/oauth2/auth",
        "token_uri": "https://accounts.google.com/o/oauth2/token",
        "auth_provider_x509_cert_url": "https://www.googleapis.com/oauth2/v1/certs",
        "client_x509_cert_url": "blah"
        }
        credentials = ServiceAccountCredentials.from_json_keyfile_dict(self.oauth_fields, scopes=['https://www.googleapis.com/auth/compute'])
        print(credentials.get_access_token())

However after spending about 12 hours on it I can't get this library to generate a token that Google will accept.
Has anyone managed to make this this library get an access token from Google?
If yes any examples that I can copy?
thanks!

Presumably you don't have google's private key and cannot sign tokens.  You probably need to make a request to google's authentication server using some client credentials and google will issue you an oauth token.
From the documentation:

Then, your application prepares to make authorized API calls by using the service account's credentials to request an access token from the OAuth 2.0 auth server.

I think this issue can be closed.


Migration file for Groups, Roles and User_Roles - Group Pink

Group Pink

Users, Organisations, Groups, Roles and User_Roles - Group Pink
In our group @daxiousm, @AkoAyukngat and @Pallabi-ghosh we all are working together.
Please provide a short summary explaining what this PR is about.
Fixes # (65)
How to test?
we tested in our local MySql database. This is working fine now.
Checklist
[ X ] I have performed a self-review of my own code
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation
This PR is ready to be merged and not breaking any other functionality

Conflict and will make another PR for this! Sorry for the inconvienience.


Possible to run something after php-fpm has started?

I'd like to run a script once after exec tini -- "${@}" in entrypoint-base but I'm coming up short on an easy way to do this other than tacking it into healthcheck-liveness..is there a better way to do this?

Container should run one and only one process, the exec here ensures that entrpoint will handover process control to tini which in turn will run the container CMD which defaults to php-fpm but technically can be any command like php <some-background-job-or-db-migration> and so -and by design- there is no way to run a command _once_ after the exec`.
Although I'd not recommend it but you can technically run a command in the background before the exec part and let it run the background (maybe with a sleep to make sure it runs after the entrypoint has executed).
What's your use-case ? What are you trying to achieve ?

Thanks for the insight. I'm trying to use laravel-opcache to compile application code when a container first starts. The issue is that to set data in opcache for php-fpm (not cli) it has to be done from an actual HTTP request.
So I need the web container to be ready (it is) and php-fpm to be running in order to make a successful call. It's also more ideal to make the call from the app container so I can check headers/app key from the same environment for security reasons.

@FoxxMD you can use postStart container, in Kubenretes you can use a PostStart hook but it's not guaranteed that it will run after Entrypoint is executed, but you can make a script that waits until php-fpm starts to execute.
But lets take a step back, AFAIK opcache will be generated once the application starts receiving requests, are you trying to warm it up ? or clear the cache?  If the latter why do you want to clear the cache in production? The way kubephp is deployed in production assumes that the code will be immutable and hence the defaults in php.ini

are you trying to warm it up ?

Yes and no-ish...I misinterpreted some behavior of opcache and thought I needed to explicitly cache some app files (that opcache would not ever cache) but I think I was wrong on this point.
My infra doesn't use kubernetes but this discussion has been helpful -- in the future if I need to run-once anything after the app container starts I'll just use a small scratch or alpine container to communicate with app. Thanks for the insight.


translated using typescript page

@srikanth-kandi Could you please review my PR? Thank you!

Thanks @shajahanshaik915 for your contribution. I look forward to seeing more contributions from you 🤝

I'm glad I could contribute, and I look forward to contribute more in the future 🤝.


🐛 [firebase_auth] Crash on FirebaseAuth.verifyPhoneNumber

Bug report
Describe the bug
Crash after a while on FirebaseAuth.verifyPhoneNumber function execution.
There is a crash on firebase_auth version 0.20.0+1.
There is no crash on firebase_auth version 0.18.4+1.
Steps to reproduce
Steps to reproduce the behavior:

FirebaseAuth.verifyPhoneNumber function

Expected behavior
No crash

Additional context
Add any other context about the problem here.

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
PASTE OUTPUT INSIDE HERE

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
PASTE OUTPUT INSIDE HERE

Hi @nashnub
I see there's an open issue addressing the case you described https://github.com/FirebaseExtended/flutterfire/issues/4651.
Please follow up on that issue, I'm closing the current one as a duplicate.
If you disagree, please write in the comments and I will reopen it.
Thank you


Change open portal hyperlinks

Description
Allow Connection Graph output resources name to be a link within the console output.
Type of change

This pull request fixes a bug in Radius and has an approved issue (issue link required).
This pull request is a minor refactor, code cleanup, test improvement, or other maintenance task and doesn't change the functionality of Radius (issue link optional).

Fixes: #7170

Hey @gpltaylor thanks for helping with this! It looks like the PR is on the right track. I left a suggestion about testing. I'll leave it up to you whether you want to refactor or not. Let me know when you're done and we'll re-review the PR.

In the mean time I've approved the functional tests.

@rynowak @radius-project/radius-core-team
I'm happy with the work done. Can you please review this PR and merge if you're happy to do so.

Can you rebase this? There are merge conflicts with main. If you help let me know.

@rynowak I think I managed to Rebase and get my code up-to-date.
Please let me know if this is ok.

I'll merge this after the functional tests pass.


[refactor-prompt] add import multisig_addr

What current issue(s) does this address, or what feature is it adding?
move #623 forward by adding multisig_addr
How did you solve this problem?
write code
How did you make sure your solution works?
write tests and make test
Are there any special changes in the code that we should be aware of?
no
Please check the following, if applicable:

[x] Did you add any tests?
[x] Did you run make lint?
[x] Did you run make test?
[x] Are you making a PR to a feature branch or development rather than master?
[ ] Did you add an entry to CHANGELOG.rst? (if not, please do)

Coverage increased (+0.09%) to 85.615% when pulling 072861a2623873f890ae24a18eeed000e0fcf06f on ixje:import_multisig into 2d72fad45d9b4ee2206ced4f9ef11652a65d239b on CityOfZion:refactor-prompt.

It did cross my mind to delete it, then I thought perhaps people might be using it. Then again, this whole PR is refactoring a lot anyway so people will just have to deal with breaking changes while we're not at v1.0. It's just needed to move forward.


allow specifying a base-url without appending hard coded values

Description

Currently the osie installer will always append /misc/osie to conf.MirrorBaseURL. This doesn't allow the runtime/user the ability to control the exactly URL/URI path. This restriction and hard coded values seems arbitrary and unneeded. As conf.MirrorBaseURL is used is multiple installers and there is custom logic around when to append to the URI path, introducing another CLI/env seemed required.
Why is this needed

Fixes: #
How Has This Been Tested?

How are existing users impacted? What migration steps/scripts do we need?

Checklist:
I have:

[ ] updated the documentation and/or roadmap (if required)
[ ] added unit or e2e tests
[ ] provided instructions on how to upgrade

Hey @mmlb , @nshalman. if/when you have some cycles would you mind checking this out, please? Thanks!

The PR description should cover the removal of --osie-path-override. I'm not aware of any users today, but this functionality is being removed in this PR and that is worth calling out.

will come back to this if needed.


Add 4 missing attributes to NSURLRequest

cachePolicy, timeoutInterval, networkServiceType, and allowsCellularAccess
Rework how attributes are stored.
Refactor header field code into existingHeaderField().

@salgernon - please take a look

I think we intentionally left these out, but hopefully @salgernon can comment.

I think it makes sense to have cachePolicy, timeoutInterval, and networkServiceType on all platforms.

If I recall correctly, the reason we left them out is that they are already properties on NSURLSession:
    /* default cache policy for requests */
    public var requestCachePolicy: NSURLRequestCachePolicy

    /* default timeout for requests.  This will cause a timeout if no data is transmitted for the given timeout value, and is reset whenever data is transmitted. */
    public var timeoutIntervalForRequest: NSTimeInterval

    /* default timeout for requests.  This will cause a timeout if a resource is not able to be retrieved within a given timeout. */
    public var timeoutIntervalForResource: NSTimeInterval

    /* type of service for requests. */
    public var networkServiceType: NSURLRequestNetworkServiceType

    /* allow request to route over cellular. */
    public var allowsCellularAccess: Bool

That's why NSURLSession has API that takes NSURL and API that takes NSURLRequest: The API that takes a request let's you override the values set by the NSURLSessionConfiguration. As the documentation states:

By creating a task based on a request object, you can tune various aspects of the task’s behavior, including the cache policy and timeout interval.

Yes, it was at the request of the owner of this particular API that we do not have this override capability, because they were concerned about the complexity it introduces.
However, since we're not really getting any response on this right now, we may as well add this for parity since it's confusing people right now. Let's leave the cellular one out.

I just noticed that HTTPBody and HTTPBodyStream are also missing. I’ll need these for https://github.com/apple/swift-corelibs-foundation/pull/299 in order to implement POST and PUT requests. Thoughts?

I've removed allowsCellularAccess, but added HTTPBody and HTTPBodyStream.

Only after adding HTTPBody and HTTPBodyStream did I realise that these should not have been added because the NSURLSession API has separate methods for uploads (e.g. POST / PUT). Will remove these again. Sorry for the noise.

@swift-ci Please test

I'll close this.

C.f. #306


Do we need pad audio when train hifigan on tacotron outputs?

Hello.
In the nvidia tacotron repository, a function that return mel spectrograms of audio first fills it with zeros on both sides. It means that the tacotron is trained to generate mel spectrograms not for the original audio, but for the filled with zeros on the right and left. When generating mels in teacher forcing mode for a hifi gan, we also get spectrograms that imply the presence of zeros in the audio at the beginning. If we try to find the correspondence between the original audio (without padding) and the generated mel spectrograms, then the correspondence between them will be shifted by the number of zeros that fill the original audio during the training of the tacotron. Does this mean that before training the vocoder, we must fill the audio with zeros (in particular at the beginning to avoid bias)?
In the case of the nvidia tacotron, this is:
https://github.com/NVIDIA/tacotron2/blame/185cd24e046cc1304b4f8e564734d2498c6e2e6f/stft.py#L85-L88

@CookiePPP I would be very grateful if you would help me in this not a simple matter.

https://github.com/jik876/hifi-gan/blob/master/meldataset.py#L61
    y = torch.nn.functional.pad(y.unsqueeze(1), (int((n_fft-hop_size)/2), int((n_fft-hop_size)/2)), mode='reflect')

This repo pads the audio by (n_fft-hop_size)//2 on both sides, tacotron2 pads by n_fft//2 on both sides.
This should be very easy for you to update, just remove -hop_size from this repo or add -self.hop_length to the tacotron2 repo.

https://pytorch.org/docs/stable/generated/torch.stft.html

If center is True (default), input will be padded on both sides so that the tt -th frame is centered at time t \times \text{hop_length}t×hop_length . Otherwise, the tt -th frame begins at time t \times \text{hop_length}t×hop_length .

Looks like torch.stft (used inside this repo) also performs it's own padding. I'm guessing that's why the implementation in this repo has -hop_size added.
You'll have to test what this does, I don't use the original spectrogram code from this repo in my stuff so I have no idea what it actually gives.

Thanks for the answers, but that's not exactly what I wanted to hear.
For clarity, I drew the original and padded audio that is used to generate mel spectrograms for the tacotron. As you can see, padding causes offset, as you can see in the image, audio segment1 is offset by the amount of padding.

Even though hifi gan uses padding for audio during fourier transform, it adds padding to an already shifted chunk. Therefore, the correct solution would be to add to the original audio padding, which is used when training the tacotron.


(LOW) Automatically wrap EVERY passed value into parameter

Right now i have a scalar values as fallback, i can remove it to make code cleaner.

This should not break anything but will simplify where statement building a lot.


Astrea-KG.ttl not up to date?

When using the OwlGenerator with the latest Astrea-KG.ttl file containing an RDF dataset with the Astrea queries, I get other shapes (less constraints) compared to when I use OwlGenerator without argument (and it gets the queries from your servers).
Could this be because the Astrea-KG.ttl on github is outdated compared to the one you use for the API and on the servers?
If so, would it be possible to update it to the latest version, or make this one available somehow?

I guess it could be that the .ttl is outdated. The server from where Astrea reads the shapes is this one. You can download the whole dataset from there, in any case I will update the file in the git as well with the name Astrea-KG-latest.ttl


"Transparent" cmdline popup

Is your feature request related to a problem? Please describe.
I would like to remove the background colors from all highlight groups for the cmdline popup, to only have a border and otherwise the backgroud color of whatever colorscheme I'm using. Kinda like in the screenshot, just with the icons in the popup also without background, I didn't figure out how to do that.

Right now it looks like this (with gruvbox theme):

I found this issue #126 about not being able to set FloatBorder, and I tried using your solution of overriding config.cmdline.opts. This works and I can set the highlight group to something without a background, but now all commandline popups also have the same foreground color too.
What I want is the foreground color to match with the mode (yellow for search, blue for command), but without background color.
Describe the solution you'd like
I would like to either have a no_background option under cmdline_popup that just turns off all background colors in the popup, OR
some way to set foreground and background colors (that can be nil) for each different type of cmdline popup (/ or :) OR
some way to just set highlight groups directly for the different kinds of popups
Describe alternatives you've considered
I tried just setting different values to the used highlight groups, but this didn't work because they are generated, and I wasn't able to figure out what the groups were called in the end and figured it would be useless anyway, because noice might just reset them.

I also want to have no_background option for mini view

This works and I can set the highlight group to something without a background

From here it seems you have indeed succeeded in "removing" the colour for FloatBorder (namely setting the corresponding highlight group to your background colour).

What I want is the foreground color to match with the mode (yellow for search, blue for command), but without background color.

Then you can just re-assign the highlight groups for the single commands: I do something along the lines of
local noice_hl = vim.api.nvim_create_augroup("NoiceHighlights", {})
local noice_cmd_types = {
	CmdLine = "Constant",
	Input = "Constant",
	Lua = "Constant",
	Filter = "Constant",
	Rename = "Constant",
	Substitute = "Define",
	Help = "helpVim",
}
vim.api.nvim_clear_autocmds({ group = noice_hl })
vim.api.nvim_create_autocmd("BufEnter", {
	group = noice_hl,
	callback = function()
		for type, hl in pairs(noice_cmd_types) do
			vim.api.nvim_set_hl(0, "NoiceCmdlinePopupBorder" .. type, {})
			vim.api.nvim_set_hl(0, "NoiceCmdlinePopupBorder" .. type, { link = hl })
		end
		vim.api.nvim_set_hl(0, "NoiceConfirmBorder", {})
		vim.api.nvim_set_hl(0, "NoiceConfirmBorder", { link = "Constant" })
	end,
})

(you can check my noice config here)
As you can see from the screenshot below, each single mode has its own colours: is this what you are after?
https://user-images.githubusercontent.com/15387611/200088998-a51b7cfa-c58c-47e4-876d-d9475e3085f8.mov

@gennaro-tedesco Yes, this is what I was looking for :) I added Search to your code and and also set the NoiceCmdlineIcon... group, because for me that had a wrong background as well. I also made 2 highlight groups based of stuff from my colorscheme, removed the background, and used those.
local search = vim.api.nvim_get_hl_by_name("Search", true)
vim.api.nvim_set_hl(0, 'TransparentSearch', { fg = search.foreground })

local help = vim.api.nvim_get_hl_by_name("IncSearch", true)
vim.api.nvim_set_hl(0, 'TransparentHelp', { fg = help.foreground })

local cmdGroup = 'DevIconLua'
local noice_cmd_types = {
  CmdLine    = cmdGroup,
  Input      = cmdGroup,
  Lua        = cmdGroup,
  Filter     = cmdGroup,
  Rename     = cmdGroup,
  Substitute = "Define",
  Help       = "TransparentHelp",
  Search     = "TransparentSearch"
}

local noice_hl = vim.api.nvim_create_augroup("NoiceHighlights", {})
vim.api.nvim_clear_autocmds({ group = noice_hl })
vim.api.nvim_create_autocmd("BufEnter", {
  group = noice_hl,
  callback = function()
    for type, hl in pairs(noice_cmd_types) do
      vim.api.nvim_set_hl(0, "NoiceCmdlinePopupBorder" .. type, { link = hl })
      vim.api.nvim_set_hl(0, "NoiceCmdlineIcon" .. type, { link = hl })
    end
    vim.api.nvim_set_hl(0, "NoiceConfirmBorder", { link = cmdGroup })
  end,
})

I still think it would be nice if we just could do something like this, then we wouldn't have to create our own autocommands, when Noice can just use the specific highlight groups in the first place.
local Cmdlines= {
  Normal = "ABC",
  FloatBorder = "XYZ"
}
local Searches= {
  Normal = "OtherStuff",
  FloatBorder = "OtherBorder"
}
...
require("noice").setup({
  cmdline = {
    format = {
      cmdline     = { pattern = "^:", winhighlight = Cmdlines, ... },
      search_down = { pattern = "^/", winhighlight = Searches, ... },
      search_up   = { pattern = "^%?", winhighlight = Searches, ... }
    },
    ...
  }
...

The autocommands are just to ensure that the groups are cleared up, should they have been assigned in other places of the configurations first. It should very well work without augroups if there is no other command/assignment taking precedence before up above (I just have a habit to assign highlight groups via autocommands because with old Vimscript it was unclear to me what was loaded before or after: I agree it's a bad habit, try and see if you get the same results by just assigning the highlight groups explicitly in the configs).

Hm, seems like just doing this (same thing, but without setting the highlight groups in an autocommand)
local search = vim.api.nvim_get_hl_by_name("Search", true)
vim.api.nvim_set_hl(0, 'TransparentSearch', { fg = search.foreground })

local help = vim.api.nvim_get_hl_by_name("IncSearch", true)
vim.api.nvim_set_hl(0, 'TransparentHelp', { fg = help.foreground })

local cmdGroup = 'DevIconLua'
local noice_cmd_types = {
  CmdLine    = cmdGroup,
  Input      = cmdGroup,
  Lua        = cmdGroup,
  Filter     = cmdGroup,
  Rename     = cmdGroup,
  Substitute = "Define",
  Help       = "TransparentHelp",
  Search     = "TransparentSearch"
}

for type, hl in pairs(noice_cmd_types) do
  vim.api.nvim_set_hl(0, "NoiceCmdlinePopupBorder" .. type, { link = hl })
  vim.api.nvim_set_hl(0, "NoiceCmdlineIcon" .. type,        { link = hl })
end
vim.api.nvim_set_hl(0, "NoiceConfirmBorder", { link = cmdGroup })

after require("noice").setup({...}) has been called is enough, Noice doesn't seem to overwrite the highlight groups after the setup

Yes, in fact, this is the cleanest solution.

Yep, that's exactly how you should do it. Noice sets default=true on the hl groups, so that's why they're not overriden by Noice.


No Longer Prompted To Create PR after Pushing Feature Branch

Fixes #4171

@alexr00 This fix appears to be working as expected now after downloading the update to VSCode - Insiders and creating a couple of topic branches across two different repositories. 👍🏾


Utility functions for self-affine surface model

I ended up creating a class that represents the parametrisation of a self-affine PSD, in order to make it more convenient to jump betweeen the 1D and the isotropic representation, and to compute the elastic energy for full contact analytically (and in order to have a tested implementation of these formulas somewhere).
I am looking for a place where I will find it again, would it make sense to put that into SurfaceTopography ?
I would also include the formulas  for hrms, h'rms etc.  from the STMP paper.
@pastewka  what do you think ?
class SelfAffinePSD():
    def __init__(self,
                 cr,
                 rolloff_wavelength,
                 hurst_exponent,
                 longcut_wavelength=np.infty,
                 shortcut_wavelength=0, ):
        self.cr = cr
        self.rolloff_wavelength = rolloff_wavelength
        self.longcut_wavelength = longcut_wavelength
        self.shortcut_wavelength = shortcut_wavelength
        self.hurst_exponent = hurst_exponent

        if self.shortcut_wavelength == 0:
            self.shortcut_wavevector = np.infty
        else:
            self.shortcut_wavevector = 2 * np.pi / self.shortcut_wavelength

        self.longcut_wavevector = 2 * np.pi / self.longcut_wavelength
        self.rolloff_wavevector = 2 * np.pi / self.rolloff_wavelength

    def isotropic(self, q):
        qs = self.shortcut_wavevector
        qL = self.longcut_wavevector
        qr = self.rolloff_wavevector

        return np.where(
            np.logical_and(q < qs, q > qL),
            np.where(q < qr,
                     self.cr,
                     self.cr * (q / qr) ** (-2 - 2 * self.hurst_exponent)),
            0)

    def profile(self, q):
        # TODO:  I think this is only the simplified formula.
        # I think I could implement the Fresnel integral

        qs = self.shortcut_wavevector
        qL = self.longcut_wavevector
        qr = self.rolloff_wavevector

        return np.where(
            np.logical_and(q < qs, q > qL),
            np.where(q < qr,
                     self.cr * qr / np.pi,
                     self.cr * (q / qr) ** (-2 - 2 * self.hurst_exponent)
                     * q / np.pi * np.sqrt(1 - (q / qs) ** 2)
                     ),
            0)

    def elastic_energy_analytic(self, shortcut_wavelength=None,
                                longcut_wavelength=None, Es=1):
        if shortcut_wavelength is None:
            shortcut_wavelength = self.shortcut_wavelength

        if longcut_wavelength is None:
            longcut_wavelength = self.longcut_wavelength

        return elastic_energy_analytic(
            self.cr,
            self.rolloff_wavelength,
            self.hurst_exponent,
            longcut_wavelength=longcut_wavelength,
            shortcut_wavelength=shortcut_wavelength,
            Es=1., )

My suggestion to create a new module "Models" and stick this in there as the file "SelfAffine.py". This would then mean from SurfaceTopography.Models.SelfAffine import SelfAffinePSD. We may want to have other models in there, e.g. for frozen capillary waves.

Can you also move the generators from SurfaceTopography.Generation there? This means removing Generation.py from the top level directory.

At some point we may want to abstract the generation and allow to refer to a model, like your SelfAffinePSD class. But I think this is further down the road and not yet necessary.

Agreed. The generation could simply be a method of the self-affine model, since all the parameters except discretization and seeds are already there.

Did you include this into the repo?

Uh probably not. I think I planned to include it the next time I would actually use it which didn't happen yet I think.

Would you like to have it soon ?

No it is not crucial


Unit tests are failing when wagtail >= 2.12

Some of the unit tests are failing when a wagtail version of >=2.12 is installed. The failing tests are test_custom_page_streamfield_data_complex, test_custom_page_streamfield_default_blocks, test_custom_page_streamfield and test_custom_page_streamfield_data. I've tried running the tests against Wagtail 2.9.3 and 2.11.7 and they pass.
The issue seems to be that an id attribute has been added to the block in Wagtail >= 2.12.
For example the data returned by the MyTestPageWithStreaFieldFactory call, with Wagtail >= 2.12 looks like this (page.body.streamdata):
[('char_array', ['foo', 'bar'], '35741861-c5ea-45a9-a8f5-8f5de95fe6d8'), ('struct', StructValue([('title', 'My Title'), ('item', StructValue([('label', 'my-label'), ('value', 100)])), ('items', []), ('image', None)]), 'a99daf3d-9055-45f3-a0f2-3b61fd45f58a'), ('int_array', [100], 'b7b60e57-5df2-4996-9916-d8f132ba57be'), ('image', <Image: Blub>, '4138701b-c8fe-45b9-91f9-6e0cdbcce520')]
Note the additional guids which are not expected in the test data.
I will try to fix this issue, but I'm looking for some advise on the best way to proceed. Can we simply drop support for older versions of wagtail and modify the unit tests to expect the additional guids? Or do we need to maintain support for older versions of Wagtail and modify the tests to take account of different versions (EG check the Wagtail version)? Or some other option which I have not thought of?

Hi @ColinBeeby-Developer and thank you for raising this issue!
This error is because of a larger streamfield refactoring that has been done in Wagtail 2.12 and 2.13, where the tests itself are relying on a deprecated property for comparison (stream_data), we should instead compare block_type and value directly on the streamfield property (page.body[0].block_type == "image" / page.body[0].value == [100]).
As to you questions, we need to keep supporting older Wagtail versions, but only those who are officially supported, those are 2.11 and 2.13 (according to https://github.com/wagtail/wagtail/wiki/Release-schedule).
I'm did not see a PR from you so I'm taking the initiative for a PR so we can get this solved as quick as possible.

Merged in https://github.com/wagtail/wagtail-factories/commit/a28a66eb08503a854ae5f91a36f71bdb01c14a55