[$250] Wallet - Statement is not downloaded

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

Version Number: 9.0.57-7
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Issue found when executing PR https://github.com/Expensify/App/pull/51626
Action Performed:
Precondition: account with enabled wallet.

Go to URL https://staging.new.expensify.com/statements/
{yyyy}{mm}
Click on Download button

Expected Result:

Tab with month statement is opened
Statement is downloaded

Actual Result:

Tab with month statement is opened. Placeholder instead of info is displayed
Statement is not downloaded
New browser tab expensify.com is opened

Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/79ffab32-7d2c-4e26-b861-93123bdc544d
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021854481819415295170
Upwork Job ID:<PHONE_NUMBER>415295170
Last Price Increase: 2024-11-07

Issue OwnerCurrent Issue Owner: @dukenv0307

I don't think we're working on Wallet right now. Asking the team - https://expensify.slack.com/archives/C06ML6X0W9L/p1730866894043029

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

Taking convo to quality - https://expensify.slack.com/archives/C05LX9D6E07/p1730884730445159

bumping convo in quality - https://expensify.slack.com/archives/C05LX9D6E07/p1730963737370759

Proposal
Please re-state the problem that we are trying to solve in this issue.
Wallet - Statement is not downloaded
What is the root cause of that problem?
There are 3 problems according to Actual Result.

Tab with month statement is opened. Placeholder instead of info is displayed
We are passing the url to WalletStatementModal modal.
https://github.com/Expensify/App/blob/1cdd204f473865b8185489d30a0a1a11a8e6dd60/src/pages/wallet/WalletStatementPage.tsx#L82
https://github.com/Expensify/App/blob/1cdd204f473865b8185489d30a0a1a11a8e6dd60/src/components/WalletStatementModal/index.tsx#L44
CONFIG.EXPENSIFY.EXPENSIFY_URL is https://www.expensify.com/, btw seems like there are 3 sites (https://we.are.expensify.com, www.expensify.com, https://new.expensify.com) in white list on backend side.
The problem is when sending url to server, www.expensify.com didn't change to https://www.expensify.com.
So we are getting this error;
Refused to frame 'https://www.expensify.com/' because an ancestor violates the following Content Security Policy directive: "frame-ancestors https://we.are.expensify.com www.expensify.com https://new.expensify.com".

Statement is not downloaded
This is the download url;
https://github.com/Expensify/App/blob/1cdd204f473865b8185489d30a0a1a11a8e6dd60/src/pages/wallet/WalletStatementPage.tsx#L58
The problem is there is no pdf.

New browser tab expensify.com is opened
https://github.com/Expensify/App/blob/1cdd204f473865b8185489d30a0a1a11a8e6dd60/src/pages/wallet/WalletStatementPage.tsx#L58
The problem is CONFIG.EXPENSIFY.EXPENSIFY_URL is https://www.expensify.com, so this app is opening expensify.com.

What changes do you think we should make in order to solve the problem?

We should add the correct website address (https://www.expensify.com) instead www.expensify.com in white list.
If you don't want to open the expensify.com, we should change the pdfURL like following;
${CONFIG.EXPENSIFY.EXPENSIFY_URL} => ${CONFIG.EXPENSIFY.NEW_EXPENSIFY_URL}
We should prepare the pdf on backend side.

What alternative solutions did you explore? (Optional)
N/A
Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01aff093c9a804b145

We're going to set this external to see if there are some suggestions to fix this. @dukenv0307 can you please review the proposal when you have a moment. Thank you!

@dukenv0307 Strange, the pdf is downloaded now but didn't work 8 hours.
And I think we should open ND in new tab.
@lanitochka17 Is it right?

Or we should not open new tab?

@lanitochka17 might not be quick to respond as we usually communicate in Slack so let me know if I can help with relaying anything or if we should have QA test again.

@Christinadobrzyn tester is still reproducible issue
https://github.com/user-attachments/assets/bc8ed37a-00ee-4526-99aa-94778673398c
That might be the reason

ah interesting catch @lanitochka17 - testing again. I have a wallet setup in my test account<EMAIL_ADDRESS>and I see the statement isn't loading and nothing is downloaded. @dukenv0307 @Anaslancer can you test?
https://github.com/user-attachments/assets/3000dc72-c497-49a4-be84-947bd90c1e7a

nudge to @dukenv0307 or @Anaslancer can you test this again to make sure you can reproduce this?

Ah thanks @dukenv0307 - changed this to internal. going back to this convo to see if someone has the bandwidth for this - https://expensify.slack.com/archives/C05LX9D6E07/p1731559170624659?thread_ts=1730884730.445159&cid=C05LX9D6E07

looking for an internal volutneer

discussing in Slack - https://expensify.slack.com/archives/C05LX9D6E07/p1731985174052279?thread_ts=1730884730.445159&cid=C05LX9D6E07

Taking a crack at this. I consider this critical priority since it is required by our MTL licenses

@thienlnam if you come online would really like some help with this. I think you made the statements feature originally?

Alright we figured it out, it actually works on production right now just fine, the issue is isolated to staging and is not at risk of leaking into production. I have to go to an appointment but I'll put up a PR to fix this later in the day.

PR in review

looks like payment is coming up in a few days. @alitoshmatov I see you helped with the PR but I don't see that you've been assigned. Should you be paid for helping with this?
@blimpich do we need a regression test?

@Christinadobrzyn @alitoshmatov was auto-assigned but didn't participate in the PR, no payment needed there.
No regression test, this is a one-off weird configuration error. Didn't impact customers. Not worth a regression test.


Support anonymous unions and structs in C++ code. [phab:T77]

Created by jdennett at 2015-11-23 23:58:50:
Testcase for GNU anonymous struct extension:
union U {
struct {
//- field defines/binding StructField
int field;
};
//- field ref StructField
static_assert(&U::field != nullptr, "");
};
Similar with s/struct/union for standard anonymous unions, which should also be tested in other kinds of scopes (e.g., in function scope).

jdennett wrote at 2015-11-23 23:59:50:
One possible patch for IndexerASTHooks.cc:
1091c1091,1097
<   const NamedDecl *const TargetDecl = FoundDecl;

const NamedDecl TargetDecl = FoundDecl;
if (const auto IFD = dyn_castclang::IndirectFieldDecl(FoundDecl)) {
// An IndirectFieldDecl is just an alias; we want to record this as a
// reference to the underlying entity.
// TODO(jdennett): Would this be better done in BuildNodeIdForDecl?
TargetDecl = IFD->getAnonField();
}

jdennett wrote at 2015-11-24 22:10:30:
Patch #872 was committed as rK412c6703723c: "C++ Indexer: Support anonymous structs, a GNU extension."
Tests for anonymous unions are a work in progress.
See also #873, "Capture named edges implied by IndirectFieldDecls for fields of anonymous records."


add empty slices to sunburst

#21
The size of an arc is calculated by taking the sum of the sizes of its "children". If a node doesn't have any children, its size is proportional to the number of peptides unique to that node.
Let's look at a toy example:
Parent (15)
|--Child 1 (7)
| |--Subchild 1 (1)
| |--Subchild 2 (3)
|--Child 2 (3)

Both subchildren have no children, so their sizes are proportional to the number of peptides they have (1 and 3).
Child 1 has two children, so its size is the sum of the sizes of the children (3+1 = 4)
Child 2 has no children, so its size is 3
Parent has two children, so its size is the sum of the sizes of child 1 and 2 (4+3 = 7)

As you can see, the size of the parent (7) doesn't match with the number of peptides of that node (15). This is probably what you also experienced. The problem lies in the fact that Child 1 has 4 peptides that are unique to that node that aren't incorporated into its size. The same thing happens with the 5 peptides specific to the parent node.
To fix the sizes, the sunburst should be drawn like this:
Parent (15)
|--Child 1 (7)
| |--Subchild 1 (1)
| |--Subchild 2 (3)
| |--Invisible subchild (7-1-3=3)
|--Child 2 (3)
|--Invisible child (15-7-3=5)

Original pull request by @bmesuere on Sun Oct 27 2013 at 15:28.
Merged by @bmesuere on Sun Oct 27 2013 at 16:51.

Ready for merge. Closes #21.
email Allesandro and Carolina to report this is fixed
Original comment by @bmesuere on Sun Oct 27 2013 at 16:48.


AVRO-3009: Temporarily test java on ubuntu-18.04 to avoid flakiness

This is a temporal fix until github rolls out their fix https://github.com/actions/virtual-environments/issues/3185
R: @RyanSkraba

It looks like the error is the same ...  we might just have to wait for the fix from github.

Ok I will try then the approach done in https://github.com/apache/accumulo/commit/1dc21016654dc4d8a66178540fe2ce8012deb5be


use isinstance instead of == for type checks

use isinstance instead of == for type checks

taken care of by 1de33273bd2a3eb989ce3754f72736029398de0c


update client dependency framework

this request updates the cdf component to the latest version. we were stuck on I believe v1.3 or v1.4, and this gets us up to v1.8 as of commit 17d935408f4c8a37c6e83b19814aa3637c411b85

will this be merged?

It's tagged for 7.5.0 aka 8.0, I'm going to clean up the merge conflicts with the hope it might get pushed up to 7.4.2.

I reviewed the diff that @bdukes posted, and while the only significant change is not in the 1.8 release - it seems to be a positive change, so I have accepted the merge from @ohine as submitted (resolving all the merge conflicts due to the switch of development from 7.4.x to 8.0.0 since the PR was submitted).  I am currently testing the changes and the merge before push the change to development.

After reviewing all of Oliver's changes I noticed that the CDF is now on version 1.8.4 on Nuget so I have synchronized all those changes with our source

Merged Manually


Configure ignores --with-incompatible-bdb on 1.14 and fails even when libdb_cxx 5.1 is present on master

This issue tracker is only for technical issues related to Dogecoin Core.
For general questions about Dogecoin please use one of the various communities, e.g. Reddit, IRC, Discord, etc.
Describe the issue
./configure fails on Ubuntu 18.04 despite building Berkley DB 5.1 from source and installing to /usr/local/.  It also fails after installing 5.3 via apt and passing --with-incompatible-bdb
Can you reliably reproduce the issue?
If so, please list the steps to reproduce below:
Building From Source
$ cd dogecoin/deps/db-5.1.29.NC/
$ cd build_unix/
$ ../dist/configure --prefix=/usr/local
$ make -j
$ sudo make install
$ find /usr/local/ -name db_cxx*
/usr/local/include/db_cxx.h
$ cd ~/dogecoin
$ ./autogen.sh
$ ./configure
.
.
.
checking for Berkeley DB C++ headers... no
configure: error: libdb_cxx headers missing, Dogecoin Core requires this library for wallet functionality (--disable-wallet to disable wallet functionality)

Ok let's try building with the latest 5.3 in the repo
$ git checkout 1.14-branding
$ sudo apt-get install libboost-all-dev libdb5.3-dev libdb5.3++-dev libevent-dev
$ ./autogen.sh
$ ./configure
$ ./configure --with-incompatible-bdb
.
.
.
checking for Berkeley DB C++ headers... no
configure: error: libdb_cxx headers missing, Dogecoin Core requires this library for wallet functionality (--disable-wallet to disable wallet functionality)

Expected behaviour
It should configure
Machine specs:
$ lsb_release -a
No LSB modules are available.
Distributor ID:	LinuxMint
Description:	Linux Mint 19.3 Tricia
Release:	19.3
Codename:	tricia
$ uname -r
5.4.0-65-generic

Any extra information that might be useful in the debugging process.
https://pastebin.com/2NR7SS9T

Got past configuring, but now there are include errors.
Line 23153 in configure
-  bdbpath=X
+  bdbpath="/usr/local/include"

Try sudo apt install libdb++-dev

Try sudo apt install libdb++-dev

No dice.  Same results on master and 1.14 and lib_db 5.1

I would build using the depends system.

Can you elaborate on that?  What do you mean, exactly, by "the depends system"?

Oops, that doesn't solve the issue; you must install the libdb5.1 packages manually as
wget http://mirrors.edge.kernel.org/ubuntu/pool/universe/d/db/libdb5.1_5.1.29-7ubuntu1_amd64.deb sudo apt install ./libdb5.1_5.1.29-7ubuntu1_amd64.deb wget http://mirrors.edge.kernel.org/ubuntu/pool/universe/d/db/libdb5.1++_5.1.29-7ubuntu1_amd64.deb sudo apt install ./libdb5.1++_5.1.29-7ubuntu1_amd64.deb wget http://mirrors.edge.kernel.org/ubuntu/pool/universe/d/db/libdb5.1-dev_5.1.29-7ubuntu1_amd64.deb sudo apt install ./libdb5.1-dev_5.1.29-7ubuntu1_amd64.deb wget http://mirrors.edge.kernel.org/ubuntu/pool/universe/d/db/libdb5.1++-dev_5.1.29-7ubuntu1_amd64.deb sudo apt install ./libdb5.1++-dev_5.1.29-7ubuntu1_amd64.deb
./configure then works on Ubuntu 18.04

I also solved it by following these steps that are called out later in the Unix build.

Oops, that doesn't solve the issue; you must install the libdb5.1 packages manually.

Update:  After linking against a newer version of bdb I kept hitting a segfault (https://github.com/bitcoin/bitcoin/issues/12047).  Your method worked, thanks!


Explicit literal indentation on top level literal scalars

The YAML 1.2 spec rationalizes indentation level as starting at -1, thus making the first indentation level be 0. This is clever in some respects, but causes a few unfortunate side effects. One of these is the numerical count of top level literal scalars:
The numerical count is supposed to be the number of columns that things are offset. So this,
--- |2
    Dear John,
  blah
  blah

Should mean "  Dear John,\nblah\nblah\n", and that by looking at it is exactly what you expect.
But if you follow the -1 starting rule then you get "   Dear John,\n blah\n blah".
This will be cleared up in the next version of the YAML spec.
For now, luckily almost every YAML implementation in the wild interprets the first way above.
Your module would do well to do the same.
A related request is that currently your stringifier does this:
$ node -e 'console.log(require("yaml").stringify("  foo\nbar\nbaz\n"))'
|1
  foo
bar
baz

It would be preferable to do this:
$ node -e 'console.log(require("yaml").stringify("  foo\nbar\nbaz\n"))'
--- |2
    foo
  bar
  baz

First the document start --- is really useful to let people know that a top level scalar is actually YAML encoded.
--- 42

vs
42

Second is problematic for literal content to be in column 1. A major design factor for literals is that they be able to encode any YAML stream simply by indenting it.
---
foo:
  bar: baz

could be encoded in another YAML file:
---
yaml: |
  ---
  foo:
    bar: baz

but without indentation you get:
---
yaml: |
---
foo:
  bar: baz

which has a completely different meaning.

I think changing the meaning of |2 for top-level scalars would need to be a semver-major breaking change. It's unfortunate that the 1.2 spec technically requires the current non-obvious implementation. To be clear, I'm not opposed to a new major version, there's a bunch of stuff that's been waiting for it to happen, including a number of deprecated endpoints.
Same probably goes for the explicit document-start/directives-end indicator as a default for scalars; as you mention it changes the output of YAML.stringify(42) from '42\n' to '--- 42\n' or '---\n  42\n', which is rather different.
Indentation of top-level scalars that include lines starting with --- or ... will be fixed by #162, but that'll only indent them if required, rather than always. Your yaml-as-literal example has always worked as expected with this library; the upcoming fix is that it'll also work if an explicit document is included as a top-level literal string.

I'm going to close this, because clearly nothing has happened here for over a year. Might revisit this later, if we e.g. end up including a recommendation of some sort about top-level scalars in a later YAML spec version.


Mandatory snappy installation

Some use cases don't involve Snappy usage, and indeed it seems the code can handle scenarios where python-snappy is not installed; However, the parquet-python package does require python-snappy explicitly (thereby also possibly forcing installation of system dependencies required by python-snappy such as snappy itself).
Is it ok if I made a PR to have python-snappy an optional requirement in setup.py (i.e. it would not get installed by default but rather only when pip is used like so: pip install parquet-python[snappy])?

good catch, pr would be great!


Lets go bigger than 10K TPS!

Lets do some testing on bigger systems such as GCP to see if we can get to 10k TPS.

@sarabala1979 would you be interested in this? It is closely related to large messages, as would use the same infrastructure. Except the goal is to do many messages. We're doing >10k TPS in production already, but other system claim > 100m TPS.

I will take it

Resource Configuration:
Cat:
memory: 2Gi
Sidecar:
memory: 2Gi
Kafka Configuration:
MaxMessageBytes:  500000000 (50mb)
Nats Configuration:
max_payload: 50000000

Source
50K TBS
100K TBS
500K TBS
1M TBS

HTTP
TPS 5500
TPS 17232
TPS 33864
TPS 65384

Kafka
TPS 74
TBS 19
TPS 11

Stan
TPS 113
TPS 22
TPS 8


Update Action stopped working on V1.6+

[x] Before posting an issue, read the FAQ at https://superforms.rocks/faq and search the previous issues.

Description
I created a Superforms CRUD component which has been working beautifully through version 1.5.3. When upgrading to version 1.6.0 and 1.6.1 my "Update" action stopped working.

Expectation: clicking the edit button on my datatable identifies the record for the data to populate the form with
Issue: clicking the edit button, populates an empty form similar to the "Create" action

// Update the form data with the current record values
data.form.data = tableData.filter((el: any) => el.id === item)[0];

const { form, errors, enhance, constraints } = superForm(data.form, { ... });

Is there a breaking change between 1.5 and 1.6 that I should be aware of? Is there another approach that I should use to the populate the form with the correct record data?
If applicable, a MRE
Use this stackblitz template to create a minimal reproducible example that you can link to here: https://stackblitz.com/edit/sveltekit-superforms-1-testing?file=src%2Froutes%2F%2Bpage.server.ts,src%2Froutes%2F%2Bpage.svelte

Are you using data.form to update after calling superForm? I'm not sure if you can call a bugfix a breaking change, but you cannot do that due to the reset functionality, which was fixed somewhere between 1.4 and 1.6. in other words, form.data !== $form and you should use $form directly for updates.
If you do that already, can you make an MRE to show the issue?

Correct form.data !=== $form.  form.data is the input for superForm for updating, which then returns $form.
I am updating data.form.data with the data from the datatable and then passing it to superForm. For example:
The starting default data created by zod schema definition (aka "new" record):
data: {
    form: {
      id: "160jx76",
      valid: false,
      posted: false,
      errors: {},
      data: {
        id: undefined,
        name: "",
        comment: ""
      },
      constraints: {
        name: {
          minlength: 2,
          required: true
        },
      },
    }
}

Updated with datatable record for passing to superForm:
data: {
    form: {
      id: "160jx76",
      valid: false,
      posted: false,
      errors: {},
      data: {
        id: "01H8YAVT0HCQPSGPQTVQPJ1PVC",
        name: "Datatable Record",
        comment: "Something cool here"
      },
      constraints: {
        name: {
          minlength: 2,
          required: true
        },
      },
    }
}

After passing data.form to superForm and making an update... in version 1.5.3, $form returns:
  form: {
    form: {
      id: "160jx76",
      valid: true,
      posted: true,
      errors: {},
      data: {
        id: "01H8YAVT0HCQPSGPQTVQPJ1PVC",
        name: "Datatable Record Updated",
        comment: "Something cool here"
      },
      constraints: {
        name: {
          minlength: 2,
          required: true
        },
      },
      message: "Record Updated"
    }
  },

With version 1.6 or newer, superForm never loads the updated data.form.

Sounds strange, I think I have to see a MRE of this to know more.

hi @ciscoheat ... updated MRE with an example that demonstrates the approach.  This is using v1.5.3, which you can see actually works. Upgrading to v1.6.0 or v1.6.1 make it not work. ;-)

Sorry, but there's too much code in this example for me to understand what's going on. Can you minimize it to what's required to reproduce the bug, and provide a step-by-step explanation of how to reproduce it?

@ciscoheat ... I've stripped it down further. Hopefully this works better. Pushed the one with v1.6.1.
When you click on any of the "pencil" icons in the datatable, the form should load with that specific record, but it does not when using v1.6+, however with v1.5.3 it does.

In case it is helpful...

in the datatable component, the function handleUpdate() is the one that passed the updated record to data.form and maps it to the modalStore object, and then opens the form
the modal passes the data.form object to superForm via $modalStore[0].meta.data and contains the actual form, etc.

The same process is used in both versions of superForm.
BTW, when I stripped down the code, I didn't refactor the actions logic to actually apply the changes, delete the records or add new records, but that wasn't needed to demonstrate the issue. ;-)

Debugging is time-consuming, so there's still too much code for an MRE, there should be no modals, no extra components (unless you are sure that the problem occurs due to that), etc. But I was lucky to directly find the problem in handleUpdate. As I said in a previous reply:

you cannot do that due to the reset functionality, which was fixed somewhere between 1.4 and 1.6. in other words, form.data !== $form and you should use $form directly for updates.

By doing that, the form data is displayed in the modal: https://stackblitz.com/edit/sveltekit-superforms-1-testing-4fyaab?file=src%2Flib%2Fcomponents%2FDataTable.svelte%3AL59
There may be other places where this must be updated in the code, but you'll get the idea.


Adding value substitution #8 (hostname/base_url)

It is now possible to add variables to the mockdev daemon execution environment, that will be available for value substitution when generation response bodies.

Closing #8


Help needed

Hi can someone help me with this? I would like to understand what this forward function in the model_builder.py is doing.
Specifically what are the expected shapes for top_vec, sents_vec, clss and mask_cls
def forward(self, x, segs, clss, mask, mask_cls, sentence_range=None):
top_vec = self.bert(x, segs, mask)
sents_vec = top_vec[torch.arange(top_vec.size(0)).unsqueeze(1), clss]
sents_vec = sents_vec * mask_cls[:, :, None].float()
sent_scores = self.encoder(sents_vec, mask_cls).squeeze(-1)
return sent_scores, mask_cls
I have output of BERT (top_vec) as a (512,768) tensor. If that is correct, what should sents_vec be?

Nevermind - figured it out.


[Feature Request] dark mode for the stylebot editor UI

dark mode for option page and editor UI

no update on this?

Hey @ankit,
Would you accept a PR for this feature. I've already implemented a dark mode in a few addons like Privacy Badger and Multi-Account Container so it shouldn't be to hard to port the logic I used in this addon.

@dannycolin yes, that will be awesome. I will happy to accept a PR for this.

Hey @ankit,
Would you accept a PR for this feature. I've already implemented a dark mode in a few addons like Privacy Badger and Multi-Account Container so it shouldn't be to hard to port the logic I used in this addon.

Pls do so! The UI is too frickin' white! :)


feat(PHP 8.0): add ini file

Description

Add development ini file for the PHP 8.0
Closes #2781
Motivation and Context

We need an INI file to be bound on docker-compose up for php-fpm service. The content of the file if taken from docker run --rm -ti laradock/php-fpm:latest-8.0 cat /usr/local/etc/php/php.ini-development > ./php-fpm/php8.0.ini. latest image for the 03/20/2021 01:00 UTC.
Types of Changes

[ ] Bug fix (non-breaking change which fixes an issue).
[x] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).

Definition of Done Checklist:

[x] I've read the Contribution Guide.
[ ] I've updated the documentation. (refer to this for how to do so).
[x] I enjoyed my time contributing and making developer's life easier :)

Maybe php.ini for PHP 8.1?

Maybe php.ini for PHP 8.1?

As I remember, the configuration is got from the official image for the PHP 8.0 (like 8.0-fpm). I don't remember the version of the image though :(


Pressing Enter while in Chrome shows something about "Open the Chrome"

Normally pressing Enter (for example in Notepad) looks like this:

But when I press Enter while Chrome has the focus, it looks like this:

This is annoying. did anyone find a work around for this?

Delete the .yml file under Keymaps. In windows you can find it in AppData/Local/carnac.


Modified getLocalTime to add timeonly option

no issue

🤖 Generated by Copilot at 1e288b3
Added a format parameter to the getLocalTime function and used it in the Hint component to show only the time part of the local time in the time zone setting. This improved the user experience of the Ghost admin panel.

Closing as it seems this has been solved in a different way


compite data error

I used the right code and loaded the data and template right. But why does the data return empty? Can anyone help me with my problem or advice?
Thanks very much

Provided data looks valid. Can you please provide a test that covers your problem?

Closing issue due to inactivity.


Problem with the Shub Familiar

The Shub familiar, which has shub spawn, keep spawning out of me for no reason. I read the guide book but it said nothing about it
It started when I glitched the /god mode when I went into survival, then I went into creative and did /ungod, then I was killed by a hunter, and my Shub soul was nowhere to be found. But then when I went to the location of where I died, these shub spawn just randomly spawned out of me and now I can't find a way to get rid of them. Even though I tried rejoining the game or dying and respawning

This sounds like something caused by the god/ungod and gamemode changes. The shub spawns are created by the shub niggurath itself, so it likely is still around but hidden.
Try typing /kill @e[type=occultism:shub_niggurath_familiar] to get rid of it


Report Rating Bug

The following code produces a bad rating if there are some better ratings for X-Xss-Protection or CSP.
Also check the FullReport rating.
// Class A Rating - high security
        if (
            (strpos($this->getRating("Strict-Transport-Security")   , 'A') !== false) &&
            (strpos($this->getRating('X-Content-Type-Options')      , 'A') !== false) &&
            (strpos($this->getRating('X-Frame-Options')             , 'A') !== false) &&
            (strpos($this->getRating('X-Xss-Protection')            , 'B') !== false) &&
            (strpos($this->getRating('Content-Security-Policy')     , 'B') !== false)
        ) {
            $this->siteRating = "A";
            $this->comment = __("This site is highly secure.");
        }

Refers to the old version with interface.
At the moment only the ApiController is used.


4.12

QUESTION 12
Your company has several business units.
Each business unit requires 20 different Azure resources for daily operation. All the business
units require the same type of Azure resources.
You need to recommend a solution to automate the creation of the Azure resources.
What should you include in the recommendations?
A. Azure Resource Manager templates
B. virtual machine scale sets
C. the Azure API Management service
D. management groups

A


Simplify isPowerOf2

I was reading through the code and noticed that isPowerOf2() in Verifier.sol can be simplified as the following:
function isPowerOf2(uint _x) internal pure returns (bool) {
    return _x > 0 && (_x & (_x - 1) == 0);
}

I'm not sure by how much, but it may be more optimal when _x is a large number.

thanks! LGTM. I'll fix it.


No Org In URI

No Org In URI
Checks that WordPress.org is not part of Theme URI.
Requires WordPress_AbstractThemeSniff class

Closing for now, we may revisit this in another form in the future.


password_reset_recover blank screen

Hi,
I think I implemented the library correctly:
settings.py:
INSTALLED_APPS = [
'django.contrib.admin',
'django.contrib.auth',
'django.contrib.contenttypes',
'django.contrib.sessions',
'django.contrib.messages',
'django.contrib.staticfiles',
'app10minmentors',
'password_reset',
]
urls.py:
url(r'^password_reset/',include('password_reset.urls',namespace="password_reset")),
signin.html
{% block content %}

{% csrf_token %}
Sign In

Sign in »
    </form>

Forgot Password

{% endblock content %}
but when I click on 'Forgot Password' I can see a blank screen... Any Idea? Thanks in advance

Same issue here.. Can anyone help ? Thanks!

Bro, just give you what I did which solved my problem. I wrote {% block body%} in all my html file, and the inherence works good until I wanna set "password_reset" function. The I changed the {% block body %} to {% block content %}, then the bug which html went a blank page just solved. I think you should add a {% block title %} {% endblock %} in your signin html, and check the key words like " content " " title" in the html inherence. I think the package would only recognize those "key words" defined by Django, so if you wanna a customize it, it may probably not work.


FrontCamera shows upside-down image in  tag

FrontCamera show upside-down image in  tag

Hi @sorabhsahu, thanks for trying this library.
In order to help and actual resolve this issue we need some basic info, like:
iOS /  Android?
Which react-native version?
Device / Simulator?
Steps to reproduce?
Any extra beneficial info?
Thanks in advance.

I've been facing the same issue
my environment so far
Android, RN 0.48.4, Device Xiaomi Redmi Note 4A, using a camerakit camera screen and the image displayed in  tag from the uri that has been stored from the modul and the image is flipped and rotated.
so far I am using transform[{rotate:"180deg"}] for handling this issue, but can't get the right approach if the user use back camera (back camera is displaying correctly). If it not possible, can we just get the info if the image is taken with front or back camera?
thanks

@gran33
OS - Android
RN - 0.43.3
On Device
Click a pic and show that image in image tag using source URI.

@gran33 I just tried to run on Nexus device and the camera (back or front) is correct not rotated like in xiaomi device. I tried to turn off mirror image in the camera phone setting but it still no change

@kriptonhaz  you can use react-native-image-picker package.

Any ideas to solve this? I get mirrored result image on front camera, before taking it it shows properly

On nexus device, simulator etc so its not device specific

Update, image is indeed mirrored in frontal camera both ios and android
@gran33 any idea?

Hi @kriptonhaz use https://github.com/react-community/react-native-image-picker

I've done a bit of digging on this and for some reason if the Exif orientation tag is included and the photo is taken with the front-facing camera it is always flipped 180 deg. This behavior originates from this commit: https://github.com/wix/react-native-camera-kit/commit/e33b3dd3a6ba339c3e8f9a37d28c0e49c97e90b7
My Nexus phone doesn't include Exif orientation data (it handles rotating the bitmap itself) so the rotation step is skipped which explains why this affects some phones but not oethers.
I'm not sure why this behavior was added and what research it was based on but it doesn't seem to make sense to me. I'd be surprised if the majority of front-facing cameras create the image flipped by 180 degrees by default.

@gran33  I am facing the same issue. Image comes as upside down from front camera. Is there any solution for this?
React-native version -0.51
platform - android 7.0
device - Redmi 4A

Any update?
I'am facing same issue here, using MI A2 devices, React native 0.61.5,
when I try to Flip the Phone upside down while take a picture, the picture still upside down.

any update?

Hello, any update on this ?? using react-native-camera-kit V8.0.4,
This fix Fix didn't work, it turn image sideway


bug: new projects template should use latest process_lib release tag

Describe the bug
Currently process templates use a random outdated process_lib library revision.
To Reproduce
Steps to reproduce the behavior:

Create a new app with kit new
Go to Cargo.toml for a process

Expected behavior
Process library version should be pulled from here: https://github.com/kinode-dao/process_lib/releases

This is an ongoing one, but updated to 0.8.3 here: https://github.com/kinode-dao/kit/pull/191/commits/a8457d7179f389195218b67519e70611413b227e


fix(test): Move 'cargo_home' from 'install' to 'paths'

What does this PR try to resolve?
This is used outside of cargo install contexts and this makes it more
discoverable for those use cases.  That I found places that weren't using cargo_home but could helps illustrate the point.
How should we test and review this PR?
Additional information

Seems reasonable.
@bors r+

:pushpin: Commit 49deefe477a0b17fad3796c890c518434f7a892f has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit 49deefe477a0b17fad3796c890c518434f7a892f with merge 4196ea093b6b8bd3bbeff28d9e5cca3ba1b90a15...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing 4196ea093b6b8bd3bbeff28d9e5cca3ba1b90a15 to master...


Update the pubspec format doc to include author(s) and homepage

Make sure to update it for both dartlang.org and pub.dartlang.org.

This issue has been moved to dart-lang/pub#123.


0.10.0

This works with storm 0.10.0 when the storm infrastructure and dev environment (deployment environment) are running in docker containers. It needs to have certain information in the topology.yam file as the system does not pick up storm.yaml when it build the topology for submission (but does pick up defaults.yam for some reason). If would be an advantage for it to add storm.yam, then "storm" and "petrel" command lines would be equivalent.

This PR contains two generated .jar files. Please don't include generated binary files in your PR.

I updated Petrel today to work with Storm 0.10.0. Many of these changes were useful. Thanks!


why with arabic text html2pdf remove last space before last word of the sentance ?

hi every body
Why When i use html2pdf with arabic text i saw that the last space before the last word of the sentance is removed.
for example see the following sentance :
ماهر جابر شقليه
when i use html2pdf  with The previous sentence i get this sentance :
ماهر جابرشقليه
Thanks
Maher

any help please.

please see attachment

my solutions is :
   str_replace(' ',' ', $name)

@hakimezzahraoui  many thanks for you , but there is a simple problem with sentence if it contains brackets.
اللغة العربية (نحو وصرف) will become:
)اللغة العربية)نحو وصرف

@mshqlaih When I'm using the Arabic text, texts are overlapping. I found your screenshot doesn't look those overlapping issues. Do you add any special font/conditions to tackle it?

@vishnudas707 , no i did not add any special font.
just i used the following code:
var element = document.getElementById('pdfTable');

// Generate the PDF.
html2pdf().from(element).set({
  margin: 0.05,
  image: { type: "jpeg", quality: 0.80 },
  filename: apex.item('P51_REPORT_NAME').getValue(),
  html2canvas: { scale: 1.2,dpi: 300},
  pagebreak:'css',
  jsPDF: {orientation:apex.item('P51_ORIENTATION').getValue(), unit: 'pt', format: 'A4', compressPDF: true}
}).save();

@vishnudas707 , no i did not add any special font.
just i used the following code:
var element = document.getElementById('pdfTable');

// Generate the PDF.
html2pdf().from(element).set({
  margin: 0.05,
  image: { type: "jpeg", quality: 0.80 },
  filename: apex.item('P51_REPORT_NAME').getValue(),
  html2canvas: { scale: 1.2,dpi: 300},
  pagebreak:'css',
  jsPDF: {orientation:apex.item('P51_ORIENTATION').getValue(), unit: 'pt', format: 'A4', compressPDF: true}
}).save();

@mshqlaih Thanks word-wrap:normal property saved me. I think for your bracket you may need to 'dir:rtl' property. I hope this below link may help you https://github.com/eKoopmans/html2pdf.js/issues/242

@hakimezzahraoui many thanks for you , but there is a simple problem with sentence if it contains brackets.
اللغة العربية (نحو وصرف) will become:
)اللغة العربية)نحو وصرف

use (نحو و صرف)

The solution is
Change font family
for Example
.parent-container{
font-family: 'Courier New', Courier, monospace;
}

This is the solution for spaces  - I'm using in Hebrew text it's like an Arabic issue, the same problem
const texts=document.querySelectorAll('.text')
texts.forEach(text=>{
console.log(text);
text.innerHTML = text.innerText.replace(/\s/g,"\u00a0")
})

This is the solution for spaces - I'm using in Hebrew text it's like an Arabic issue, the same problem
const texts=document.querySelectorAll('.text')
texts.forEach(text=>{
console.log(text);
text.innerHTML = text.innerText.replace(/\s/g,"\u00a0")
})

Worked well in the Persian language. Thanks

After I fall into the problem again ،
I puted every word between span ،
this fix my problem

my solutions is :  str_replace(' ',' ', $name)

can you explain with example or something ? i didnt get this

@mshqlaih
I solve my problem by load every latest library  like that
``

function smartPrint(html, filename = '', orientation = 'portrait') {
if (filename) {
filename = filename + ".pdf"
} else {
filename = Math.random().toString(36).substring(2, 15) + Math.random().toString(36).substring(2, 15) + ".pdf";
}
var opt = {
margin: 1,
filename: filename,
image: { type: 'jpeg', quality: 0.98 },
html2canvas: { scale: 2 },
jsPDF: { unit: 'in', format: 'letter', orientation: orientation }
};
}
``
the problem with
html2pdf.bundle.min.js version

This is the solution for spaces - I'm using in Hebrew text it's like an Arabic issue, the same problem const texts=document.querySelectorAll('.text') texts.forEach(text=>{ console.log(text); text.innerHTML = text.innerText.replace(/\s/g,"\u00a0") })

thank you very much


ngAnimate addClass/removeClass: Callback after addClass, as well as after transitionend

So, there are a few things --- In some cases, adding a class needs to be paired with an actual DOM operation (which can't take place before adding the class, this would lead to an instant jump) --- So having a sort of before transition, after adding transition class callback would be a pretty big deal.
Following this, on most browsers you won't see a transitionend event fired immediately (because no transition will have occurred if the class is added before a DOM operation such as changing the height or position or something of the animated element) happens --- So, if a transitionend were to happen immediately, the first instance should be ignored (if supplied with a "DOM manipulation"/pre-transition callback)
Finally, the proper transitionend/post-duration callback should not trigger the doneCallback until after it's actually completed (or alternatively, why not have $animate.addClass/removeClass/enter/leave() return a promise?
I am not talking about "module animations" registered via module.animation() here, because in some cases these are not practical to use.
Clearly there are some breaking changes listed, although I think these changes could be written in a way which would provide API compatibility with 1.2.0. I may try to get something like this working after matsko@'s ngAnimate fixes are merged.
(These views come from attempting to add some ngAnimate functionality to angular-ui/bootstrap, which has typically required listening for transitionend or waiting for a delay/duration manually)

Thank you so much @matsko this has helped a lot!


[media.ccc.de:lists] playlist_id should be case-sensitive

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

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

Region
No response
Provide a description that is worded well enough to be understood
Some playlists use uppercase playlist_ids, like https://media.ccc.de/c/DS2023 → https://media.ccc.de/public/conferences/DS2022 or https://media.ccc.de/c/MCH2022 → https://media.ccc.de/public/conferences/MCH2022. So I guess removing .lower() in https://github.com/yt-dlp/yt-dlp/blob/master/yt_dlp/extractor/ccc.py#L96 should resolve this.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
$ python -m yt_dlp --verbose --ignore-config https://media.ccc.de/c/DS2023
[debug] Command-line config: ['--verbose', '--ignore-config', 'https://media.ccc.de/c/DS2023']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b532a3481] (source)
[debug] Lazy loading extractors is disabled
[debug] Git HEAD: 30ba233d4
[debug] Python 3.11.5 (CPython x86_64 64bit) - Linux-6.5.0-5-generic-x86_64-with-glibc2.38 (OpenSSL 3.0.10 1 Aug 2023, glibc 2.38)
[debug] exe versions: ffmpeg 6.0 (setts), ffprobe 6.0, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.11.0, brotli-1.0.9, certifi-2022.09.24, mutagen-1.46.0, pyxattr-0.8.1, secretstorage-3.3.3, sqlite3-2.6.0, websockets-10.4
[debug] Proxy map: {}
[debug] Loaded 1866 extractors
[media.ccc.de:lists] Extracting URL: https://media.ccc.de/c/DS2023
[media.ccc.de:lists] ds2023: Downloading JSON metadata
ERROR: [media.ccc.de:lists] DS2023: Unable to download JSON metadata: HTTP Error 404: Not Found (caused by <HTTPError 404: Not Found>); please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
[…]
The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/tmp/yt-dlp/yt_dlp/extractor/common.py", line 847, in _request_webpage
    return self._downloader.urlopen(self._create_request(url_or_request, data, headers, query))
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/tmp/yt-dlp/yt_dlp/YoutubeDL.py", line 4078, in urlopen
    raise _CompatHTTPError(e) from e
yt_dlp.networking.exceptions._CompatHTTPError: HTTP Error 404: Not Found

I wanna take a look into this one. I hope its alright.


Retina graphics for command icons

There are some graphics which aren't retina ready:

Along with many of the assets in https://github.com/quicksilver/Quicksilver/tree/master/Quicksilver/Resources/Images. Some of them have been converted to @2x, but it looks like some of them have been upscaled, rather than redrawn for the higher resolution.
Relates to #1044

Know any designers? :smiley:


user_troubleshooting_message: Include git commit of the vcpkg repo and suggest updating with git pull

Currently you can only guess if the user is using a recent vcpkg version when a user submits an issue.

Is there a reason we aren't just using rev-parse HEAD?

Because I want to know the date. I want to know if the user is using a recent version of the vcpkg repo or an old one. And only the hash does not help (Yeah I can lookup the commit and check the date, but doing this for every issue is very painful).

How does it interact with custom registries or overlay ports? They may change the given port and/or dependencies.

Thats right, my ultimate goal is that vcpkg outputs a ready to copy paste github issue markdown code. In that you can then also put config files and other important information. The only problem currently is to catch the error logs. They are printed here:
https://github.com/microsoft/vcpkg/blob/827a54c404e67205262e6db032fb2c1a5e6fd097/scripts/cmake/vcpkg_execute_required_process.cmake#L132
I would simply search for the See logs for more information string but I assume @strega-nil-ms wants a guid based solution?

@BillyONeal can you do the second review ? :)


Column to Predict (Label) dropdown cannot scroll down

VS 2019 ASP.NET Core 3.0
ML.NET Model Builder extension
When loading columns you cannot select a few if you have lots of columns that overflow the space viewable by the dropdown. Can't scroll, tab, or use arrow keys to move down the list.

Hi @todarko,
Sorry to hear you're experiencing this issue. This has already been raised in the model builder repo and the team is currently working on fixing the issue.
Issue Link: dotnet/machinelearning-modelbuilder#252
I suggest you follow the discussion there. Because this is not a documentation related question, in the future, please raise questions/ issues in the dotnet/machinelearning-modelbuilder repo.


FEAT: support Skywork

Are you interested in doing this? You can refer to this https://github.com/xorbitsai/inference/pull/587/files for guidance.

let me have a look and try ;-p


feat(docs): add new faq for sws pocket setup

:tada: This PR is included in version 1.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add CI lint check using cargo hack for checking features powerset

4th follow up https://github.com/paradigmxyz/reth/pull/10130
This PR adds cargo hack to CI, making sure that selected crates also can compile with different combinations of features

@mattsse , thank you for taking a look and merging!
Should I do a follow up to make pr like this:
pr:
	make lint && \
	make check-features && \
	make update-book-cli && \
	make test

I suppose this should improve DevEx, but also add some disk usage and compilation times!


add post-player.org connector

it aggregator-streamer of post-rock music based on vkontake

#680 waiting for the next release


[feature] Add email template #259

Closes #259
Changes

#Please explain changes made in your PR#

Screenshots

#For UI changes, please share a screenshot#

Checklist

[ ] I have read the contributing guidelines.
[ ] I have manually tested the proposed changes.
[ ] I have written new test cases to avoid regressions. (if necessary)
[ ] I have updated the documentation. (e.g. README.rst)
[ ] I have added [change!] to the commit title to indicate a backward-incompatible change. (if required)
[ ] I have checked the links added/modified in the documentation.

Coverage decreased (-1.4%) to 97.063% when pulling aa46d05cac3113d204d2e1fb524554a0607b0ead on issues/259-email-template into 5f235c175de93df8752c9e44a9afebc467679055 on master.

Coverage is decreasing because tests for new EmailAdapter are missing. Will add them soon


postgresql adapter database.yml not correct

Hi, Thanks for this - I noticed that the postgresql setting generates a database.yml file with the database adapter with postgresq not postgresql

Fixed in a4da181f5989fe89bf9c5ad0abd2dae9e421fec2


Push works to ORCID when ORCID present in contributor and ORCID linked to SUNET

Background:

Contributors in an H2 work can have an ORCID added.
Stanford users can also go through the authorize.stanford.edu portal to connect their SUNET with their ORCID.
There is an MaIS Stanford-only API to discover SUNET--ORCID connections, including if that user has authorized write access to their ORCID profile (along with the token needed to write to the profile).

Requirement:

For contributors added to an H2 work that includes an ORCID, and for which the ORCID is in the list of authorized/connected ORCIDs with write access (as determined by the MaIS API); map the H2 work metadata to an ORCID work metadata, and then push the work to that ORCID profile via the ORCID API.
We need to keep track of which H2 works were pushed and when (via the "PUT" code we get back from ORCID when sending)... store this so we do not push the work again.
Push happens weekly, scheduled via cron
Push works again if an update occurs to the work (as determined by the updated_at column in the work_version and the date last pushed to ORCID, which we will store)

Based on the requirements above, this is likely to all happen in a background job that is scheduled via whenever gem and is not triggered by any user interaction.
Note:

The sul-dlss/sul_pub project currently has an MaIS API and an ORCID API client which we can borrow code from which does this (except it maps sul-pub publications instead of H2 works).

See https://docs.google.com/document/d/1vAMI3OqJDxdQiLyseBi0IMtrL3HiL7y6cj0fT_P8t80/edit for a diagram
This ticket does not cover ROR or other pathways shown on the diagram.

Is pushing with every deposit permissible? It requires less complicated machinery and tracking.
Can the mapping from H2 work metadata to an ORCID work metadata be specified? What should be used as the required identifiers?
Are we really sure that users want H2 content to be added to their Orcid profiles without request/approval? (For example, if the H2 item are files supporting a different publication or H2 item is just another copy of a different publication. Concrete example: If I deposited a copy of my recent C4L Journal article in the Staff collection, I wouldn't want it added to my Orcid profile, since the original publication is added to my Orcid profile).

@justinlittman Pushing with every deposit is permissible if that's what you decide. For me, weekly would be often enough, but sooner works too.

@justinlittman Regarding request/approval, we can only push to ORCID records for individuals who have already given us approval to do so. ORCID users can choose whether to make any individual work publicly visible or not on their ORCID record, and can set up notifications so that they get an email from ORCID every time something is added to their record.

Given that the point of the trusted user authorization is so that organizations like Stanford can push things to their record, we consider doing so to be a bonus for users. And if they are concerned, they can set up the necessary notifications to intercept and hide additions as soon as they are made.

Have we asked any users if this is a desired feature and/or if they would expect or be surprised by H2 items being added to Orcid record? (I understand that they have consented to it; did they understand what they consented to?)

* Can the mapping from H2 work metadata to an ORCID work metadata be specified? What should be used as the required identifiers?

Can I recommend that the developer that picks up this ticket takes a first pass at going through the available metadata from H2 and use the ORCID data model to map the obvious fields.  For fields that we have in H2 with no obvious counterpart is in ORCID, we then ask for assistance from either ORCID and/or our metadata department?  I think this was the approach we took in sul-pub.

* Are we really sure that users want H2 content to be added to their Orcid profiles without request/approval? (For example, if the H2 item are files supporting a different publication or H2 item is just another copy of a different publication. Concrete example: If I deposited a copy of my recent C4L Journal article in the Staff collection, I wouldn't want it added to my Orcid profile, since the original publication is added to my Orcid profile).

For the duplicate question, I believe ORCID allows us to only push works with a specific list of known identifiers (e.g. DOIs) so that they can be de-duped on the ORCID side (as is done with sul-pub).

@justinlittman I think there is still a plan to do some user interviews. We can look at adding a question to this effect to those interviews. Though I think for this question it would be best to find some H2 users who have already authorized Stanford to push to ORCID.
How hard is it to get the intersection of SUNet IDs that 1) exist in the H2 database as any kind of user and 2) have authorized Stanford to write to ORCID? These are the people we want to ask. Others won't know what we're talking about.

@justinlittman I think there is still a plan to do some user interviews. We can look at adding a question to this effect to those interviews. Though I think for this question it would be best to find some H2 users who have already authorized Stanford to push to ORCID.
How hard is it to get the intersection of SUNet IDs that 1) exist in the H2 database as any kind of user and 2) have authorized Stanford to write to ORCID? These are the people we want to ask. Others won't know what we're talking about.

We should be able to get that list easily, let me ticket it.  Part of the challenge here honestly is that I suspect not many researchers are really asking for this feature nor even care that much about it.  But if/when it exists, it will provide additional value in the future (by further promoting their research)...assuming ORCID continues to take off and be relevant, as we all hope.  To me it feels a bit like the apocryphal Henry Ford "quote" about if you asked people what they wanted, they would have said a faster horse.  In some sense, we are building a speculative car here, and hoping we are correct.

See #3117 for results of my analysis of ORCIDs currently in H2.

For the record, results of mini-survey of users on pushing works to ORCID is at https://docs.google.com/spreadsheets/d/1csEtMDALUTW86avBisbQln-ypQeP12zi2vst4vVTW1Q/edit. They are overwhelmingly in favor of this and they don't want to be asked or notified.

do we need Arcadia to sign off on this mapping?

do we need Arcadia to sign off on this mapping?

Yes, for sure.  We are awaiting her assistance for this (indicated with the "metadata" tab).

H2 to ORCID metadata mapping - vocabulary mappings still in progress
<?xml version="1.0" encoding="UTF-8"?>
<work:work xmlns:common="http://www.orcid.org/ns/common" xmlns:work="http://www.orcid.org/ns/work"
    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
    xsi:schemaLocation="http://www.orcid.org/ns/work/work-3.0.xsd ">
    <work:title>
        <common:title>H2 title</common:title>
    </work:title>
    <work:short-description>H2 abstract</work:short-description>
    <work:citation>
        <work:citation-type>formatted-unspecified</work:citation-type>
        <work:citation-value>H2 preferred citation</work:citation-value>
    </work:citation>
    <work:type>See H2 work type mapping</work:type>
    <!-- use deposit date if publication date not present -->
    <common:publication-date>
        <common:year>H2 publication (or deposit) year</common:year>
        <common:month>H2 publication (or deposit) month</common:month>
        <common:day>H2 publication (or deposit) day</common:day>
    </common:publication-date>
    <common:external-ids>
        <common:external-id>
            <common:external-id-type>doi</common:external-id-type>
            <common:external-id-value>DOI value</common:external-id-value>
            <common:external-id-url>DOI URL</common:external-id-url>
            <common:external-id-relationship>self</common:external-id-relationship>
        </common:external-id>
        <common:external-id>
            <common:external-id-type>uri</common:external-id-type>
            <common:external-id-value>Stanford purl value</common:external-id-value>
            <common:external-id-url>Stanford purl URL</common:external-id-url>
            <common:external-id-relationship>self</common:external-id-relationship>
        </common:external-id>
    </common:external-ids>
    <common:url>Stanford purl</common:url>
    <work:contributors>
        <work:contributor>
            <common:contributor-orcid>
                <common:uri>ORCID or ROR URL</common:uri>
                <common:path>ORCID or ROR identifier</common:path>
                <common:host>orcid.org or ror.org</common:host>
            </common:contributor-orcid>
            <work:credit-name>H2 first name + space + H2 last name</work:credit-name>
            <work:contributor-attributes>
                <work:contributor-role>See H2 role mapping</work:contributor-role>
            </work:contributor-attributes>
        </work:contributor>
    </work:contributors>
    <common:language-code>en</common:language-code>
    <common:country>US</common:country>
</work:work>

For H2 work type and role mappings, see https://docs.google.com/spreadsheets/d/1zrRCFgwWkPW0Vg9mIsUe-rMZkiCvudB2t3VLw0dn-rY/edit?usp=sharing.
For <work:credit-name>, include only names in the H2 cited author section. Omit names not in the citation.
For <work:contributor-role>, enter a value only if there is a mapping, otherwise omit this stanza.

The following values are literals, i.e. not mapped based on H2 content.
<work:citation-type>formatted-unspecified</work:citation-type>

<common:external-id-type>doi</common:external-id-type>

<common:external-id-type>uri</common:external-id-type>

<common:external-id-relationship>self</common:external-id-relationship>

<common:language-code>en</common:language-code>

<common:country>US</common:country>


fix: update amazon france with french strings

The amazon_search function actually needs to be aligned with the get_items method for searching emails.

The issue you were trying to fix has been fixed in #1028
Thanks!


Balancer

This pull requests revisits the way we're doing load balancing and enhance the API with a couple more load balancing algorithms that should provide a more fair distribution than the weighted random based on the network RTT.
One combination that gave pretty nice results is filtering based on tags + doing round robin on the result:
    --- PASS: TestBalancer/PreferTags+RoundRobin (0.00s)
    	balancer_test.go:86: ID = 8, RTT = 280µs, Tags = [us-west-2a]:  13	(6.5%)
    	balancer_test.go:86: ID = 1, RTT = 210µs, Tags = [us-west-2a]:  13	(6.5%)
    	balancer_test.go:86: ID = 2, RTT = 220µs, Tags = [us-west-2a]:  13	(6.5%)
    	balancer_test.go:86: ID = 3, RTT = 230µs, Tags = [us-west-2a]:  13	(6.5%)
    	balancer_test.go:86: ID = 4, RTT = 240µs, Tags = [us-west-2a]:  13	(6.5%)
    	balancer_test.go:86: ID = 5, RTT = 250µs, Tags = [us-west-2a]:  13	(6.5%)
    	balancer_test.go:86: ID = 6, RTT = 260µs, Tags = [us-west-2a]:  13	(6.5%)
    	balancer_test.go:86: ID = 7, RTT = 270µs, Tags = [us-west-2a]:  13	(6.5%)
    	balancer_test.go:86: ID = 15, RTT = 350µs, Tags = [us-west-2a]:  12	(6%)
    	balancer_test.go:86: ID = 9, RTT = 290µs, Tags = [us-west-2a]:  12	(6%)
    	balancer_test.go:86: ID = 10, RTT = 300µs, Tags = [us-west-2a]:  12	(6%)
    	balancer_test.go:86: ID = 11, RTT = 310µs, Tags = [us-west-2a]:  12	(6%)
    	balancer_test.go:86: ID = 12, RTT = 320µs, Tags = [us-west-2a]:  12	(6%)
    	balancer_test.go:86: ID = 13, RTT = 330µs, Tags = [us-west-2a]:  12	(6%)
    	balancer_test.go:86: ID = 14, RTT = 340µs, Tags = [us-west-2a]:  12	(6%)
    	balancer_test.go:86: ID = 0, RTT = 200µs, Tags = [us-west-2a]:  12	(6%)
    	balancer_test.go:86: ID = 16, RTT = 1.36ms, Tags = [us-west-2b]:   0	(0%)
    	balancer_test.go:86: ID = 17, RTT = 1.37ms, Tags = [us-west-2b]:   0	(0%)
    	balancer_test.go:86: ID = 18, RTT = 1.38ms, Tags = [us-west-2b]:   0	(0%)
    	balancer_test.go:86: ID = 19, RTT = 1.39ms, Tags = [us-west-2b]:   0	(0%)
    	balancer_test.go:86: ID = 20, RTT = 1.4ms, Tags = [us-west-2b]:   0	(0%)
    	balancer_test.go:86: ID = 21, RTT = 1.41ms, Tags = [us-west-2b]:   0	(0%)
    	balancer_test.go:86: ID = 22, RTT = 1.42ms, Tags = [us-west-2b]:   0	(0%)
    	balancer_test.go:86: ID = 23, RTT = 1.43ms, Tags = [us-west-2b]:   0	(0%)
    	balancer_test.go:86: ID = 24, RTT = 1.44ms, Tags = [us-west-2b]:   0	(0%)
    	balancer_test.go:86: ID = 25, RTT = 1.45ms, Tags = [us-west-2b]:   0	(0%)
    	balancer_test.go:86: ID = 26, RTT = 1.46ms, Tags = [us-west-2b]:   0	(0%)
    	balancer_test.go:86: ID = 27, RTT = 1.47ms, Tags = [us-west-2b]:   0	(0%)
    	balancer_test.go:86: ID = 28, RTT = 1.48ms, Tags = [us-west-2b]:   0	(0%)
    	balancer_test.go:86: ID = 29, RTT = 1.49ms, Tags = [us-west-2b]:   0	(0%)

Another nice upside is the new algorithms are about 10x faster than using the ones based on random shuffling:
BenchmarkBalancer/RoundRobin-4         	  500000	      3076 ns/op	       0 B/op	       0 allocs/op
BenchmarkBalancer/PreferTags-4         	  500000	      2767 ns/op	       0 B/op	       0 allocs/op
BenchmarkBalancer/PreferTags+RoundRobin-4         	  300000	      4538 ns/op	       0 B/op	       0 allocs/op
BenchmarkBalancer/Shuffler-4                      	  100000	     11185 ns/op	       0 B/op	       0 allocs/op
BenchmarkBalancer/WeightedShufflerOnRTT-4         	   30000	     45796 ns/op	      33 B/op	       1 allocs/op

Please take a look and let me know if you have any questions.

I'm gonna go ahead and merge this since it has no backward incompatible changes, only introduces new APIs and we've been running in production for a while with great success.


Add handling for upcoming chromas

Wiki has added space for upcoming chromas and this causes issues if they do not have an id or they are question marks

Thanks as always @bangingheads!!


Monitor kubernetes pod resources

What did you do?
Installed Openshfit container platform 3.11. Assessing prometheus-operator to scrape application metrics. We see ServiceMonitor resource can scrape from service. But we are looking for scraping from a pod resource.
What did you expect to see?
Like in prometheus, one can define scrape target on any kubernetes resource like pod by giving few  annotations, so that prometheus discovers these targets.
eg: 	prometheus.io/path: /metrics
prometheus.io/port: "4002"
prometheus.io/scrape: "true"
What did you see instead? Under which circumstances?
No way to scrape from pod resources
Environment
Openshfit container platform 3.11

Prometheus Operator version:
ose-prometheus-operator:v3.11.69
Kubernetes version information:
1.11+

If you are a Red Hat customer, please go through the normal bug report process via bugzilla. The ServiceMonitor indeed does scrape metrics from pods, just uses Service objects as a mechanism of grouping pods. It makes use of the very same Prometheus endpoints discovery you are referring to, just not via the annotations as that has a number of problems.

@brancz Thanks for the reply. Yes it is working as expected. We could see pods as well discovered by Prometheus using ServiceMonitor approach.


Expansion Tile programmatic expansion control

Use case

Proposal

Closing this as a duplicate of https://github.com/flutter/flutter/issues/60387.
Kindly follow up in the linked issue for updates. Please upvote the original issue as well.
Thank you


Hyperscan Now Proprietary

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[ x ] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[ x ] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Is your feature request related to a problem? Please describe.
Intel made hyperscan proprietary. So its unlikely that it will receive any updates in the future.
Describe the solution you like
Use the fork "vectorscan" instead:
https://github.com/VectorCamp/vectorscan

I don't expect we use hyperscan anywhere, except when being pulled in from other (upstream) projects (such as Suricata), this might not be the right place to ask for changes in these cases....

Don’t mind leaving this open. The strange Redis project shift also hasn’t reached FreeBSD ports tree yet. So it will take a while. 😉


Feat/documentation

feel free to add/ modify and push directly to this branch:)

Nice one! just fixed some typos.


Searxng Tool implementation

A first searxng implementation for the tools

@Getty, Can you check #2815 ?


Test and optimize performance of Text Visualization

This task is automatically imported from the old Task Issue Board and it was originally created by Michael Mauderer.
Original issue is here.

Why
The new lazy text visualization should be optimized for performance to be able to handle reasonable large data.
As a Enso User
**I want ** to be able to scroll through large data
**So that ** I can explore large datasets
Acceptance Criteria
Scenario:
Given a project with a text node that is very large(tm)
When I scroll through the text visualisaiton
Then there should be no delays from processing and sending the text data from the backend

Notes:
This requires profiling/optimizing the Enso code that generates the response in the std lib.
Part of this task should be to determine a reasonable size of data that should be usable on a reference machine, and maybe adding of benchmarks to spot performance regressions.
Comments:
Michael: "It might be more efficient for someone from the stlib or engine team to look as this, as this is pure Enso code"
FYI @jdunkerley
(Greg Ociepka - Feb 1, 2023)

Closing until new Text visualization as part of the new IDE code.


Fixed dark mode issue Update style.css

Fixed darkmode issue for the video part
Before

After

Refer issue #82


Refactor is_bot_user function to improve actor type handling

User description
fixes https://github.com/Codium-ai/pr-agent/issues/1356

PR Type
enhancement, bug fix

Description

Refactored the is_bot_user function to improve handling of Bitbucket actor types by using the get method for safer dictionary access.
Enhanced logic to allow only 'user' actor types, treating 'AppUser' and 'team' as bot users.
Improved logging to provide clearer information when the actor type is not 'user'.
Fixed error message formatting in exception handling to correctly display exceptions.

Changes walkthrough 📝
Relevant filesEnhancement

bitbucket_app.pyRefactor and fix actor type handling in `is_bot_user` function

pr_agent/servers/bitbucket_app.py
Refactored is_bot_user function to improve handling of actor types.  Changed logic to check actor type using get method for safer access.  Improved logging for non-user actor types.  Fixed error message formatting in exception handling.

+6/-3

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

/review

/improve

/review


Fixing the github security vulnerability in addition to wring 3 unit tests and 2 integration test cases

#111532

#Solution was provided by updating BaseControllerWithShareStaticPages
Unit testing
New class of CookieSettingControllerTests was added considering 3 test cases
1- Redirection mechanism when BackUrl Is Local
2- Redirection mechanism when BackUrl Is External
3- Returns When Model State Is Invalid

Intergration testing
Two test cases were added into the CookieSettingsTests in Integrating testing project
1- Posting the CookieSettings with a Back URL of local so we expect to have Redirect To LocalUrl
2- Posting the CookieSettings with a Back URL of External so we expect to have no redirection


Only one link supported

Your regex only supports one link per selection. If there’s more than one link, it should only work on the first because of the missing g flag. But with it and ? after the *, you could support multiple links by iterating through the results.

Should be working now. Thanks again :D.


Make invite post a relative path

This is necessary to get invite POSTing working when you host slackin in a subpath.

Beautiful. Long time no see my friend!


Unable to send MoorIsolate instance to isolate

I'm trying to send a MoorIsolate instance to an isolate that requires database access.
The setup is as following:

Flutter mobile app
Main isolate instantiates the moor isolate as documented at https://moor.simonbinder.eu/docs/advanced-features/isolates/#initialization-on-the-main-thread
A process that runs in the background (but in the context of the main isolate) requires database access
A process that handles communication with an external device and runs in its own isolate requires database access

When trying to send the MoorIsolate instance to another isolate like this:
_toIsolate.send([IsolateCommands.receiveDatabaseIsolate.index, databaseIsolate])
I get the following error:
illegal argument in isolate message (object is a regular dart instance)
It seems that I can only send the SendPort instance encoded within ServerKey to the isolate. I tried to change Moor's code to expose the SendPort instance and only send that, but then Moor itself fails trying to establish a connection to its own isolate with the same error.
It seems like MoorIsolate cannot be sent to another isolate.

...and just for clarification: we don't need the isolate for performance reasons, we only need it because the documentation states that concurrent accesses to the same database corrupt the database. I saw the moor_ffi explicitly turns thread safety off. If there is any other way to handle concurrent accesses, that would also be an option.

I am also waiting for a solution to this problem for a long time.

The documentation states this:
One executor isolate, multiple client isolates:
The MoorIsolate can be sent across multiple isolates, each of which can use MoorIsolate.connect on their own. This is useful to implement a setup where you have three or more threads:

The moor executor isolate
A foreground isolate, probably for Flutter
Another background isolate, which could be used for networking.

You can then read data from the foreground isolate or start query streams, similar to the example above. The background isolate would also call MoorIsolate.connect and create its own instance of the generated database class. Writes to one database will be visible to the other isolate and also update query streams.
But this does not work.
@simolus3 Please update the documentation reflecting that MoorIsolate cannot be sent over to other isolates as the following error occurs:
illegal argument in isolate message (object is a regular dart instance)

A simple workaround could be to add a serialization method to MoorIsolate that just returns the used SendPort and a deserialization method that re-instantiates MoorIsolate based on that SendPort. The class essentially only wraps a SendPort.
I tried passing the SendPort inside the MoorIsolate into my isolate, which works. However, MoorIsolate.connect() from the isolate then breaks because Moor itself tries to pass the MoorIsolate instance to its own isolate. This also fails.
The first part would be a very simple PR but I'm not sure about Moor's internal communication with its own isolate.

I've now changed all of moor's internal isolate communication to only use primitive objects by using a custom serializer.
With the next update, you'll also be able to obtain the underlying SendPort with MoorIsolate.connectPort. That port can safely be sent across isolates, the receiving end can reconstruct a MoorIsolate out of that by using the new MoorIsolate.fromConnectPort constructor.
So far, I couldn't find a good way to obtain an environment where primitive objects are required. If I run tests with pub run test I can send arbitrary objects, it's the same with dart2native. Does release-mode Flutter require primitive communication? I want to make sure that my tests would fail if I send a non-primitive object. If you now more about this, help would be much appreciated.

Very cool, I'm looking forward to test this! Flutter seems to require primitive communication, at least that's the environment I experienced the error in. I was a bit surprised that it was unable to send a primitive that's just wrapped in another class, but maybe it just looks at the top-level type.

I've now added an integration test for Flutter here. I'm running that test with flutter --release lib/moor_ffi.dart, which works for me.
If you want to, you can try to use the latest version from develop by adding this to your pubspec:
dependency_overrides:
  moor:
    git:
      url: https://github.com/simolus3/moor.git
      ref: develop
      path: moor
  moor_flutter:
    git:
      url: https://github.com/simolus3/moor.git
      ref: develop
      path: moor_flutter
  moor_generator:
    git:
      url: https://github.com/simolus3/moor.git
      ref: develop
      path: moor_generator
  sqlparser:
    git:
      url: https://github.com/simolus3/moor.git
      ref: develop
      path: sqlparser

I'll close this as it appears to be fixed, but let me know if you run into any problems on develop.

Can the documentation be updated on how we must use the new Isolate features?

I've updated the section here, it now describes how MoorIsolates can safely be shared across isolates. Is that what you need?

I am using Daos. How must I access the database now inside a dao?

This may be a stupid question but what would be the best way to test the new version? I tried changing my pubspec.yaml to point to the "develop" branch of Moor, but I'm getting a version error with sqflite.
Can I point only Moor to the develop branch and override versions to the currently released once for moor_ffi, moor_generator and sqflite? What would be a working configuration?

You need to override all 4 dependencies, moor, moor_generator, moor_ffi and sqlparser. And remove the sqflite direct dependency.

Here's my pubspec (excerpt):
dependencies:
  moor:
    git:
      url: https://github.com/simolus3/moor.git
      path: moor
      ref: develop
  moor_ffi:
    git:
      url: https://github.com/simolus3/moor.git
      path: moor_ffi
      ref: develop
dev_dependencies:
  moor_generator:
    git:
      url: https://github.com/simolus3/moor.git
      path: moor_generator
      ref: develop

I get the following error:
Because every version of moor_generator from git depends on moor from hosted and carity_pro depends on moor from git, moor_generator from git is forbidden.
So, because carity_pro depends on moor_generator from git, version solving failed.
Running "flutter pub get" in carity-pro...
pub get failed (1; So, because carity_pro depends on moor_generator from git, version solving failed.)

I get the following error:

You need to use dependency_overrides for every package developed in this repo. The section from my answer above should work, let me know if it doesn't.
If you don't want to use dependency_overrides, you can also use my custom pub server. I can't make any promises on uptime though.
dependencies:
  moor:
    hosted:
      name: moor
      url: https://kneipe.simonbinder.eu/moor/dev
  moor_ffi:
    hosted:
      name: moor_ffi
      url: https://kneipe.simonbinder.eu/moor/dev
dev_dependencies:
  moor_generator:
    hosted:
      name: moor_generator
      url: https://kneipe.simonbinder.eu/moor/dev

In that case you don't need to override the sqlparser dependency, but I'd still recommend to use git and dependency_overrides.

I am using Daos. How must I access the database now inside a dao?

The same? Typically you would construct your dao with MyDao(database), right? That doesn't change with isolates. To construct a database attached to another isolate, you can follow the guide and use isolate.connect(). You can then access that database inside a dao like you regularly would.

Apologies, I didn't see you already posted the dependency overrides above. I now have the following error when compiling:
 - 'BatchedStatements' is from 'package:moor/src/runtime/executor/executor.dart' ('../../.pub-cache/git/moor-572efd31ff29a163525d0de4bd8c4ac133b2ee8e/moor/lib/src/runtime/executor/executor.dart').
Change to a supertype of 'BatchedStatements', or, for a covariant parameter, a subtype.
  Future<void> runBatched(List<BatchedStatement> statements) async {
                                                 ^
../../.pub-cache/git/moor-572efd31ff29a163525d0de4bd8c4ac133b2ee8e/moor/lib/src/runtime/executor/helpers/delegates.dart:106:16: Context: This is the overridden method ('runBatched').
  Future<void> runBatched(BatchedStatements statements) async {
               ^
../../.pub-cache/hosted/pub.dartlang.org/moor_ffi-0.4.0/lib/src/vm_database.dart:48:32: Error: 'BatchedStatement' isn't a type.
  Future<void> runBatched(List<BatchedStatement> statements) async {
                               ^^^^^^^^^^^^^^^^
Target kernel_snapshot failed: Exception: Errors during snapshot creation: null
Failed to build bundle.
Error launching application on iPhone 6s.
Exited (sigterm)

I tried flutter clean and flutter pub cache repair but the error still comes up.

@simolus3
Dao constructor
MyDao(MyDatabase db) : super(db);

I am actually getting an error:
db is undefined

I was earlier able to access the db instance. Now that I am using moor isolates, daos have broken.
I require the db instance to access data from different tables.
Can you provide an example on how I should do this?

@tigloo You must override moor_ffi, replace moor_flutter with that.

db is undefined

Ah right, sorry. This was another breaking change in moor 3.0. Replacing db with attachedDatabase should work. I've also added an extension to keep the old db getter with 91115311cf0d295d13144aab323f88260b051f58


Add screenshot to README

I'm browsing a few different tc interfaces, so it's nice to see what it looks like without running it!

Thank you for your contribution!


modify buffer len

buffer len no need to add 16.

This will runtime panic "slice bounds out of range" http://play.golang.org/p/BNsfPqKfp7

open the link http://play.golang.org/p/BNsfPqKfp7, and show' "Forbidden".
I think something went wrong.

Code is very easy, it is the same as in the file you are patching. In case if result size in bigger than capacity of the slice bugger reinitialized with response size (look at line 582). In this case your code will panic.
Example (try to put this into play.golang.org and run):
package main
func main() {
blen := 10
buf := make([]int, blen)
_ = buf[:16+blen]

}

Thanks for the patch. I merged a different PR that fixed the same issue.


Bot occasionally doesn't Press Auto-Duel Correctly

Implement a way to check that the auto-duel button was indeed clicked on

Bot stuck at "2017-08-11 23:32:47,430 - bot - DEBUG - WAITING FOR AUTODUEL BUTTON TO APPEAR"
ran over 50 times always get stuck there

 Issue #4: Bot occasionally doesn't Press Auto-Duel Correctly

resolved by commit 59830e052e1d371f3629f31bea98f70c38a15b83


extract probe path from AIRFLOW__WEBSERVER__BASE_URL + ingress path validation

What issues does your PR fix?

fixes #112

What does your PR do?
Extracts the liveness and readiness path for the web container from the configured airflow base path.
Configures /health by default if no base path is configured
Checklist

[x] Commits are signed
[x] Commits are squashed (if appropriate)
[x] Chart version bumped
[x] Chart documentation updated
[x] Passes linting

@thesuperzapper now worries, your additions make a lot of sense. Thanks for the approval.


Suggestion: Scan and select command for CMake executable

While implementing this #942 I noticed that it might be a good idea to add an UI/commands similar to Scan for/Select Kit to:

Scan the system for CMake executables
Test the executables' version
Present the user with a QuickSelect to let the user pick the desired version

This can make sense in the following context:

CMake can be installed in location in the PATH
CMake may be installed in C:\Program Files'\CMake but not included in the PATH
VS2019 can have a bundled version
On Linux CMake might be installed as a snap package
On Linux might be installed as standard package, e.g.
etc.

I think that this maybe good to have before adding snaps support for linux #838

I don't think UI is appropriate for this because it's usually a one-time setup for a folder, but a command would be ok.  If cmake cannot be located on the PATH, the extension can suggest to scan for a cmake that works.

@bobbrow Please, provide feedback:
These are some of my draft notes/thoughts/ideas of what I think could be done to improve the user experience with setting/detecting the CMake executable.
The changes could be done in one or more PR and will also make some preparations (or implement) necessary for #838.

Add a command "Scan for CMake executables"
If multiple CMake executables are detected, prompt the user to select one, the result will be stored in cmake.cmakePath

On getCMakePath() call, check if there is a proper path set in cmake.cmakePath
If no "proper" path is set in cmake.cmakePath prompt the user to set the correct path or to execute "Scan for CMake executables".

If necessary optionally a check could be implemented to test if the path in cmake.cmakePathpoints to a valid CMake executable

Extra: Snaps support on Linux
At least for the moment I think that snaps need to be called via the command:
snap run cmake
If "Scan for CMake executable" detects /snap/bin/cmake will set cmake.cmakePath to snap.
getCMakePath() and getCTestPath() will return an object containing the needed information, something like:
   {
      cmakePath? <-- path to CMake
      ctestPath? <-- path to CTest
      isSnap?    <-- if the selected CMake is a snap package
   }

This object will be used for proper CMake invocation.
Visual Studio can also provide a bundled Ninja, which needs to be added to the environment (PATH) if the bundled CMake  #942 is used (add a configurable cmake.generatorPath for that purpose).

I think this plan looks good.  Let's keep Ninja detection separate from this.

Looks like #942 solved this, closing this issue.


Target Java 8 Rather than 17

The Kotlin compiler targets Java 8 by default as this makes it compatible with most existing ecosystems. Is there a good reason to target Java 17 here? I would like to use this library for my project but due to historical limitations the ecosystem at my company is on Java 8.

java17 cannot be used in the android system


Adjust GHA to account for fabbot

Q
A

Bug fix?
no

New feature?
no

Issues
n/a

License
MIT

fabbot now handles cs

Does he uses our phpcsfixer config ?

Does he uses our phpcsfixer config ?

This is my understanding, yes.

This PR has been stuck on pending and I see a few others like this. Some seem to pass ok though...


Move isExpired from SimpleJWS to JWS

I propose this change in order to be able to check if a JWS created from an instance of Namshi\JOSE\JWS is expired or not (and not only from SimpleJWS), sort as we can do the check even if using the SecLib encryption engine.
Or is there something in the method that is not directly compatible with the JWS class, or that doesn't make sense to exist into?
ATM I need to duplicate the isExpired method in my project to do this chek on Namshi\JOSE\JWS instances.

hey @chalasr, thanks for this :) why not using the SimpleJWS class rather than JWS? We created SimpleJWS to add convenient methods on top of the bare JWS implementation, that was the original idea on why those methods were there.

Hi @odino
In fact, the SimpleJWS class seems to be only for OpenSSL, rightt?
I need to use the SecLib and your doc sounds like that we have to use the JWS class.
So maybe, rather than move the method, I can make the SimpleJWS class working for SecLib?

@chalasr definitely! Im not sure why it shouldnt be working with seclib but I think the best solution here is to make it working regardless of the underlying sec implementation
Closing this for now, waiting for a PR :)
Thanks man!


Tag releases

It would be nice if releases would be tagged again in git.

This gets asked every now and then. Definitely will do so for any future
update.
Most previous releases are archived on sourceforge:
https://sourceforge.net/projects/stuntman/files/
Or at the following URLs:
http://www.stunprotocol.org/stunserver-1.2.16.tar.gz
http://www.stunprotocol.org/stunserver-1.2.16.tgz
http://www.stunprotocol.org/stunserver-1.2.15.tgz
http://www.stunprotocol.org/stunserver-1.2.14.tgz
http://www.stunprotocol.org/stunserver-1.2.13.tgz
http://www.stunprotocol.org/stunserver-1.2.12.tgz
http://www.stunprotocol.org/stunserver-1_2_11a.tgz
http://www.stunprotocol.org/stunserver-1.2.11.tgz
http://www.stunprotocol.org/stunserver-1.2.10.tgz
http://www.stunprotocol.org/stunserver-1.2.9.tgz
http://www.stunprotocol.org/stunserver-1.2.8.tgz
http://www.stunprotocol.org/stunserver-1.2.7.tgz
http://www.stunprotocol.org/stunserver-1.2.6.tgz
http://www.stunprotocol.org/stunserver-1.2.5.tgz
http://www.stunprotocol.org/stunserver-1.2.3.tgz
http://www.stunprotocol.org/stunserver-1.2.2.tgz
http://www.stunprotocol.org/stunserver-1.2.1.tgz
http://www.stunprotocol.org/stunserver-1.2.0a.tgz
http://www.stunprotocol.org/stunserver-1.2.0.tgz
http://www.stunprotocol.org/stunserver-1.1.3.tgz
http://www.stunprotocol.org/stunserver-1.1.2.tgz
http://www.stunprotocol.org/stunserver_1_1_1.tgz
http://www.stunprotocol.org/stunserver_1_1_0.tgz
On Sun, Sep 12, 2021 at 6:21 AM Sandro @.***> wrote:

It would be nice if releases would be tagged again in git.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jselbie/stunserver/issues/42, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHNSF7VXYYYYXKU34IEGTDUBSSL7ANCNFSM5D4DA55Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

It is easier for nixpkgs if we can fetch from the git forge directly because it makes it very easy to fetch patches or create them in the future. Sourceforge does not allow patch fetching at all and your tarball upload is not a git repository.

It is important to have tags and releases here on GitHub.


Add flag --create-certs to manage tool

See #103 . Lets add a flag to make it easier to get a local cert.

Should be done. Certs ware created automaticly.


Build System integration with zig build

this PR is intended to integrate the sublime text build system with the zig build system.

[x] closes #42
[x] allows for running custom zig build steps
[x] re-implement inline errors

there are probably some error hiding in some untested code paths, but i think this should be fine for now.


Implement Tests

Implement tests across platform.

added unit tests for API, will move on to the front end unit testing soon


Remove database and other rails modules

Remove database dependancy and other rails modules not required

Ta, closing in favour of https://github.com/openregister/create-register-picker/pull/2.


Changes to web3.sha3

What was wrong?
Currently, the implementation of web3.sha3 has some unexpected behavior and non ideal behavior.

It uses the web3_sha3 RPC endpoint which allows for different providers to return different values.
By default it expects the value to be hashed to be passed in as a hex encoded string.
If you specify any encoding besides hex, it treats it as if it was bytes.

How can it be fixed?
1. Use eth_utils.keccak to compute the hash.
This is more reliably going to produce the result that users expect as well as reducing RPC request overhead.
2. Change the default encoding to bytes.
This would be a breaking change so figuring out how to deprecate the current behavior while giving users the appropriate time to upgrade would be ideal.  This could be done in a two step process.
We should have an explicit list of encodings that are supported and have logic for dealing with those appropriately.  I suggest bytes, utf8, and hex.

Encoding Confusion
There is a caveat to # 2 above, which is that if someone passes bytes, then the encoding is meaningless and should be ignored. Also, "encoding" as the keyword argument is confusing here, because it actually means a different thing for each of these options:

bytes (it's already encoded)
utf8 (take this string of characters, and encode them using utf-8)
hex (take this hex representation of bytes, and decode them into their underlying bytes)

"txt"   == encode utf-8 ==>   b'\x74\x78\x74'   == encode hex str ==>   "0x747874"
"txt"   <== decode utf-8 ==   b'\x74\x78\x74'   <== decode hex str ==   "0x747874"

Also confusing is that python supports hex literals, which seems reasonable to support too, like:
web3.sha3(0x74657874). The primitive here is int. Again, like bytes, if an int is passed, then "encoding" should be ignored.
The "encoding" here is only ever referring to the type of string we're looking at. We can (and should) auto-detect all the rest. We'd be done, if it wasn't for those meddling kids and their hex-string encodings.
New API:
So what I'm ramping up to is a new API (the breaking change):
def sha3(primitive=None, text=None, hex=None)

My least favorite part is overwriting the builtin hex, but I really want a short syntax here. Even core python does stuff like this (see file kwarg in print).
So you could call any of:
web3.sha3(b'\x74\x78\x74')
web3.sha3(0x747874)
web3.sha3(hex="0x747874")
web3.sha3(text="txt")

This is concise, easy to read, and should cover all cases. It's not terribly important that people understand that the text is encoded into utf8, because text in all clients should always use this same encoding.
Migration Plan
So the migration plan for web3.sha3 would be something like:

Leave encoding kwarg in place
Auto-detect bytes and int, and encode them appropriately, ignoring the 'encoding' kwarg
Enable using the new keywords hex and text
Issue a Deprecation warning for all sha3 calls, unless they are using the new functionality in 2 and 3
After some time (1 month?) raise an exception if users pass a string as the first positional arg
After even longer (3 months? certainly after requiring py3), remove the encoding keyword argument

I take back the example about file in print(). It's only a kwarg in py3, and only a builtin in py2.
Here are some alternatives: hexbytes, hexencoded, hexdata, hexstr, hexval, zerox, and ox. None of them are as clear and succinct as hex. Any other suggestions?
I currently favor hexstr as the runner up.

@carver I honestly think hexstr might be even better than hex to make it clear it is not a hex encoded int like 0x1323.

tossing hexidecimal into the pool of options (but feel free to choose the one you think is best)

Went with hexstr. It grew on me.


Windows 8.1 code

I need the source code of Windows 8.1 since I want to port the code to ARM64 since I would like to see Windows RT 8.1 in arm64
so could you release me the code please?

I'm afraid this is not the correct place to make such a request.
Windows 10 and 11 are both supported on ARM64 - I recommend using one of them instead.

I honestly have no idea - you could try asking https://twitter.com/OpenAtMicrosoft if they could recommend a channel.


Make fireworks container resizable

Hey, great little library here! It would be nice if the FireworksCanvas class had a resize method to update the size of the canvas and start placing fireworks in that boundary.

That's a good idea.  I'll work on this probably in the next few days.

I've added this in the latest release, v2.4.0. Two new methods on the fireworks instance:

setSize(width, height) -- will set the dimensions to the defined size
resetSize() -- will reset the size to that of the container's clientWidth and clientHeight

Also, you can provide width and height as options now - they will be defaulted to the container's clientWidth and clientHeight if not provided.


nimble/ll: Increase stack for ble_ll_task

Default stack size for ble_ll_task is too low.
Failing scenario can be reproduced using optimized bleprph build for
nRF51 (M0). We have following call stack:
os_eventq_put() at os_eventq.c:99 0x8d7e
ble_hs_enqueue_hci_event() at ble_hs.c:431 0x14204
ble_hs_hci_rx_evt() at ble_hs_hci.c:357 0x14d4a
ble_hci_trans_ll_evt_tx() at ble_hci_ram.c:95 0x1b860
ble_ll_hci_event_send() at ble_ll_hci.c:79 0xec90
ble_ll_conn_comp_event_send() at ble_ll_conn_hci.c:250 0xd6a2
ble_ll_adv_send_conn_comp_ev() at ble_ll_adv.c:2,849 0xb490
ble_ll_conn_created() at ble_ll_conn.c:2,511 0xc0e4
ble_ll_conn_slave_start() at ble_ll_conn.c:4,076 0xd2e6
ble_ll_adv_conn_req_rxd() at ble_ll_adv.c:2,341 0xb3f0
ble_ll_adv_rx_pkt_in() at ble_ll_adv.c:2,456 0xb3f0
ble_ll_rx_pkt_in() at ble_ll.c:720 0xa43a
ble_ll_event_rx_pkt() at ble_ll.c:982 0xa43a
os_eventq_run() at os_eventq.c:178 0x8e38
ble_ll_task() at ble_ll.c:1,025 0xa2ae
0x0
The os_eventq_put() causes a context switch (interrupt) and registers
put on stack overwrite memory outside stack - we're 4 bytes (a single
register) off here. The overwritten memory is a g_dev_addr so our
address changes unexpectedly after successful connection.
This works fine on debug build and on nRF52 due to differents stack
usage. For example, on optimized build for M0, gcc aligns uint8_t on
stack for faster access thus increases its usage if there are local
uint8_t variables in function (like in ble_ll_conn_comp_event_send).
This patch increases stack size by 2 elements and this should be
enough for now.

Commit message says 2:)

Looks good


ci: use ydb trunk image for e2e

Closes #875
Currently we use old ydb image (it was updated more that a year ago). I suggest switch to trunk version.
Pros: we will be able to test all latest features. Cons: sometimes trunk image is broken

Should not we use ghcr.io/ydb-platform/local-ydb:nightly ?

trunk and nightly are the same tags of the same image.
https://github.com/ydb-platform/ydb/pkgs/container/local-ydb
I will use nightly and rename it in the docs to reduce ambiguity


Better health estimate for cyphersystem (with user settings)

I apologise for the update so soon after the initial submission, but some feedback has indicated a better range for the health estimates for cyphersystem.
It includes a bug fix for certain types of Actors, and two new module settings.

Does the system have more actor types than just PCs and NPCs? If it does, you might add a breakCondition (check pf2e.js or dnd5e.js for examples). instead an empty return at the end.

Yes, there are around 6 different actor types. I did think about the break condition, but it looked like just an empty return worked (no messages reported to console).


cmd/cue: cannot import OpenAPI JSON Schema specification

What version of CUE are you using (cue version)?

$ cue version
cue version +ecb17c94 linux/amd64

Does this issue reproduce with the latest release?
Yes
What did you do?
exec curl -sL -o schema.json https://raw.githubusercontent.com/OAI/OpenAPI-Specification/d9ac75b00c8bf405c2c90cfa9f20370564371dec/schemas/v3.1/schema.json
exec cue import schema.json

What did you expect to see?
Success
What did you see instead?
> exec curl -sL -o schema.json https://raw.githubusercontent.com/OAI/OpenAPI-Specification/d9ac75b00c8bf405c2c90cfa9f20370564371dec/schemas/v3.1/schema.json
> exec cue import schema.json
[stderr]
schema expects mapping node, found bool:
    ./schema.json:854:9
schema expects mapping node, found bool:
    ./schema.json:883:9
schema expects mapping node, found bool:
    ./schema.json:887:9
schema expects mapping node, found bool:
    ./schema.json:1326:9
schema expects mapping node, found bool:
    ./schema.json:1331:9
[exit status 1]
FAIL: /tmp/testscript393926569/repro.txt/script.txt:2: unexpected command failure

This appears to happen as of c683420c0797090a1445b453e3318a4570fc4742 still. FYI @rogpeppe


Anime won't track while using external player

Steps to reproduce

Watch any anime episode in external player
Open tracking option

Expected behavior
It should track if I watch the anime episode in external player.
Actual behavior
The anime episode I watched on external player doesn't get tracked.
Crash logs
No response
Aniyomi version
<IP_ADDRESS>
Android version
11
Device
Poco M2
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an official anime extension, I should be opening an issue in the extensions repository.
[X] If this is an issue with a manga extension, report it to the manga source extension project you added it from.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Duplicate of #1331


Update terraform-provider-framework to 0.17.0

This commit updates the terraform-provider-framework and supporting modules. Those updates include a ton of breaking changes, and thus required a significant amount of refactoring.
The refactoring is mostly straightforward, and there should be no functional changes. The biggest differences are around how resources and datasources are created and configured. Still, the diff should be easy enough to read.

TFTR!


Tim Berners-Lee quote/tip too long

This was on IE11 Win7.
Introduced by https://github.com/codecombat/codecombat/pull/2264

Should we just exclude it, or what?


Fix libturbojpeg

Application Name
home assisstant
Application Source Code URL
NA
Application Language
.Net
Application Architectures
linux/amd64
Additional Information
there seems to be an issue with libturbojpeg.  I opened a bug on https://github.com/home-assistant/core/issues/114997  but realized I was using your image.
Assign to self

[X] I will create a PR to containerize this application myself

Code of Conduct

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

I'm not sure what the fix is here? I'm already using the new package for Alpine 3.19
https://github.com/onedr0p/containers/blob/main/apps/home-assistant/Dockerfile#L63

No response and not sure what to do here.