problems

sorry to disturb, I don't have a clue to open and run it.....could you tell me by which way? pycharm?
I am working on an article on machine translate, needing the dodrio to provide me a visual access to the mechanism during the process of transformer model and I want to change the input into the sentences chosen by myself, but the online demo version seems not able to.
thank you greatly.

Thank you so much for trying out Dodrio!
Dodrio is a web-based tool. You can first download this repository to your local machine (or git clone). After installing npm, you can run the following commands in the terminal:
npm install
npm run dev

Use a web browser to open localhost:5000, and you will see Dodrio running there :)
To see how we trained the Transformer or customize the visualization with a different model or dataset, visit the ./data-generation/ directory. You can also see https://github.com/poloclub/dodrio/issues/4 for more information on using Dodrio for customized model/dataset.

Firstly, I have successfully run your demo on my device, I appreciate your favor greatly.  But I have encountered another one.
I have had all the packages install on my device, however, it still warns. I want to know what is the "modeling_bert" for? isn't it included in the transformers? And, the package "datasets" ,how can I get it?
Thank you!

Hello. The "modeling_bert" comes from the transformer package. You should be able to import it if you run pip install transformers (https://github.com/huggingface/transformers).
The "datasets" is also a huggingface package. You can install it by pip install datasets (https://github.com/huggingface/datasets)
On Mar 30, 2022, 3:29 AM -0400, Danny-C-Auditore @.***>, wrote:

Firstly, I have successfully run your demo on my device, I appreciate your favor greatly. But I have encountered another one.
I have had all the packages install on my device, however, it still warns. I want to know what is the "modeling_bert" for? isn't it included in the transformers? And, the package "datasets" ,how can I get it?
Thank you!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you modified the open/close state.Message ID: @.***>

hi, @xiaohk ,actually I had the "transformers" installed before, and tested well in a test.py file,  while it still didn't work out in
dodrio-data-gen.py.
I wonder if there is any requirements for the version of the transformers, since I didn't find any subfile named "modeling_bert", and my version of transformers is 4.17.0
apology for disturbance, again. Thanks for your patience.

thanks, I have solved the problems by finding the right path " transformers.models.bert.modeling_bert "
@xiaohk

No problem. Good luck with your project. Let me know if you have more questions.


Add colorFormat option

Changes
Adds colorFormat option to the CSS and Sass plugins, allowing users to transform colors to any colorspace on-the-fly.
Though I debated having this at the top-level core config, I think that’s actually not recommended for the same reason there are disagreements on color formats in the spec—there’s not a universal color type that works for all platforms (web and native). Other than hex.
How to Review

Tests should pass

🤔 Hm this has actually uncovered some hidden complexity in the way transformations currently work in plugins. Not the API so much as the way the current plugins are written. Which is a good thing.
But will take a step back and rethink how this is handled first.


Volume 1 Security Considerations gather the AuditEvent profiles

For ease of finding the AuditEvent profiles, the Volume 1 Security Considerations section could itemize each of the AuditEvent profiles and point to their transaction specific security considerations definition.

note if this is agreed, then I will add this issue to be addressed in the future in MHD. and include it in the supplement template.

I support this recommendation (cannot think of a better approach).

no strong reaction on today call. will continue to discuss

add link from volume 1 SC to the volume 2 SC sections
bring forward to volume 1 'hints' at authn/authz/audit requirements to be found in volume 2 (e.g. where volume 2 mentions IUA requirements, this should be mentioned in volume 1, where volume 2 defines audit messages, this should be mentioned in volume 1). Mentioning may be by http links.


Avoid deleting output buffer to fix 14 tests that PHPUnit have considered risky

When my projects updated from 3.8.1 to 3.9.0, many of my phpunit tests started resulting into risky tests. It turned out that reverting back to 3.8.1 caused phpunit to stop classifying the tests as risky. Running phpunit on Slim, I also saw that 3.8.1 had only one risky test while 3.9.0 have 18 risky tests.
The 17 new risky tests have the message: "Test code or tested code did not (only) close its own output buffers".
This commit uses ob_get_contents and ob_clean instead of ob_get_clean to avoid destroying the current output buffer.
All the tests are still passing but now, only 3 of the 17 tests are classified as Risky due to  "Test code or tested code did not (only) close its own output buffers".
Here is the lisf of risky tests on 3.9.0
dmelo@merov2:~/proj2/Slim$ git checkout 3.9.0
Previous HEAD position was 53853027... Uppdate App:VERSION to 3.8.1
HEAD is now at 575a8b53... Version 3.9.0
dmelo@merov2:~/proj2/Slim$ ./vendor/bin/phpunit --stderr --verbose
PHPUnit 4.8.36 by Sebastian Bergmann and contributors.

Runtime:	PHP 7.1.9 with Xdebug 2.5.1
Configuration:	/home/dmelo/proj2/Slim/phpunit.xml.dist

...............................................................  63 / 588 ( 10%)
..................R.........R.............SR................... 126 / 588 ( 21%)
.............................RRRRRRRRRRRR.R.................... 189 / 588 ( 32%)
.........SSSSSSSSSSSSS......................................... 252 / 588 ( 42%)
............................................................... 315 / 588 ( 53%)
............................................................... 378 / 588 ( 64%)
............................................................... 441 / 588 ( 75%)
............................R.................................. 504 / 588 ( 85%)
........................................................R...... 567 / 588 ( 96%)
.....................

Time: 973 ms, Memory: 16.00MB

There were 18 risky tests:

1) Slim\Tests\AppTest::testExceptionErrorHandlerDoesNotDisplayErrorDetails
Test code or tested code did not (only) close its own output buffers

/home/dmelo/proj2/Slim/vendor/phpunit/phpunit/phpunit:52

2) Slim\Tests\AppTest::testExceptionErrorHandlerDisplaysErrorDetails
Test code or tested code did not (only) close its own output buffers

/home/dmelo/proj2/Slim/vendor/phpunit/phpunit/phpunit:52

3) Slim\Tests\AppTest::testExceptionOutputBufferingOff
Test code or tested code did not (only) close its own output buffers

/home/dmelo/proj2/Slim/vendor/phpunit/phpunit/phpunit:52

4) Slim\Tests\Handlers\ErrorTest::testError with data set #0 ('application/json', 'application/json', '{')
Test code or tested code did not (only) close its own output buffers

/home/dmelo/proj2/Slim/vendor/phpunit/phpunit/phpunit:52

5) Slim\Tests\Handlers\ErrorTest::testError with data set #1 ('application/vnd.api+json', 'application/json', '{')
Test code or tested code did not (only) close its own output buffers

/home/dmelo/proj2/Slim/vendor/phpunit/phpunit/phpunit:52

6) Slim\Tests\Handlers\ErrorTest::testError with data set #2 ('application/xml', 'application/xml', '<error>')
Test code or tested code did not (only) close its own output buffers

/home/dmelo/proj2/Slim/vendor/phpunit/phpunit/phpunit:52

7) Slim\Tests\Handlers\ErrorTest::testError with data set #3 ('application/hal+xml', 'application/xml', '<error>')
Test code or tested code did not (only) close its own output buffers

/home/dmelo/proj2/Slim/vendor/phpunit/phpunit/phpunit:52

8) Slim\Tests\Handlers\ErrorTest::testError with data set #4 ('text/xml', 'text/xml', '<error>')
Test code or tested code did not (only) close its own output buffers

/home/dmelo/proj2/Slim/vendor/phpunit/phpunit/phpunit:52

9) Slim\Tests\Handlers\ErrorTest::testError with data set #5 ('text/html', 'text/html', '<html>')
Test code or tested code did not (only) close its own output buffers

/home/dmelo/proj2/Slim/vendor/phpunit/phpunit/phpunit:52

10) Slim\Tests\Handlers\ErrorTest::testErrorDisplayDetails with data set #0 ('application/json', 'application/json', '{')
Test code or tested code did not (only) close its own output buffers

/home/dmelo/proj2/Slim/vendor/phpunit/phpunit/phpunit:52

11) Slim\Tests\Handlers\ErrorTest::testErrorDisplayDetails with data set #1 ('application/vnd.api+json', 'application/json', '{')
Test code or tested code did not (only) close its own output buffers

/home/dmelo/proj2/Slim/vendor/phpunit/phpunit/phpunit:52

12) Slim\Tests\Handlers\ErrorTest::testErrorDisplayDetails with data set #2 ('application/xml', 'application/xml', '<error>')
Test code or tested code did not (only) close its own output buffers

/home/dmelo/proj2/Slim/vendor/phpunit/phpunit/phpunit:52

13) Slim\Tests\Handlers\ErrorTest::testErrorDisplayDetails with data set #3 ('application/hal+xml', 'application/xml', '<error>')
Test code or tested code did not (only) close its own output buffers

/home/dmelo/proj2/Slim/vendor/phpunit/phpunit/phpunit:52

14) Slim\Tests\Handlers\ErrorTest::testErrorDisplayDetails with data set #4 ('text/xml', 'text/xml', '<error>')
Test code or tested code did not (only) close its own output buffers

/home/dmelo/proj2/Slim/vendor/phpunit/phpunit/phpunit:52

15) Slim\Tests\Handlers\ErrorTest::testErrorDisplayDetails with data set #5 ('text/html', 'text/html', '<html>')
Test code or tested code did not (only) close its own output buffers

/home/dmelo/proj2/Slim/vendor/phpunit/phpunit/phpunit:52

16) Slim\Tests\Handlers\ErrorTest::testPreviousException
Test code or tested code did not (only) close its own output buffers

/home/dmelo/proj2/Slim/vendor/phpunit/phpunit/phpunit:52

17) Slim\Tests\Http\UploadedFilesTest::testMoveToStream
This test did not perform any assertions

/home/dmelo/proj2/Slim/vendor/phpunit/phpunit/phpunit:52

18) Slim\Tests\RouteTest::testInvokeWithException
Test code or tested code did not (only) close its own output buffers

/home/dmelo/proj2/Slim/vendor/phpunit/phpunit/phpunit:52

dmelo@merov2:~/proj2/Slim$ php -v
PHP 7.1.9 (cli) (built: Sep 23 2017 12:09:20) ( NTS DEBUG )
Copyright (c) 1997-2017 The PHP Group
Zend Engine v3.1.0, Copyright (c) 1998-2017 Zend Technologies
    with Xdebug v2.5.1, Copyright (c) 2002-2017, by Derick Rethans

Coverage increased (+3.6%) to 96.906% when pulling c0e118c663a9490949e65033de4869c1fcc663b9 on dmelo:phpunit-risky-output-buffer into f6884a7425e4ccf031ff8dda30c14e196bb67156 on slimphp:3.x.

#2337 addresses this problem.


[FLOC-3617] remove obsolete _run_docker_build_ubuntu_vivid_fpm

Removes the now obsolete:
http://ci-live.clusterhq.com:8080/job/ClusterHQ-flocker/job/master/job/_run_docker_build_ubuntu_vivid_fpm/
as we no longer support ubuntu vivid

Lovely, thanks. Please merge when you've made your peace with the tests.

: - )
green
:peace:


Unify the "Log out" spelling

Hi,
while translating the project, I found out that there are 2 versions of log out string: "Log out" and "Log Out". This commit replaces the "Log Out" one with "Log out" to avoid having 2 translations.

Thanks @switowski!


under navigation bar

how i can show it under the navigation bar , like facecbook
on top the navigation it may block the UX,
facebook show it under the navigation for definitely a reason :)

not possible atm, sry, But you should be able to build this quite fast looking at my source.


[VS Code] Starting collaboration session: Already connected to a workspace.

Error:Starting collaboration session: Already connected to a workspace.Steps to Reproduce:1.2.||Version Data||-:|:-||**extensionName**|VSLS||**extensionVersion**|1.0.470||**protocolVersion**|2.2||**applicationName**|VSCode||**applicationVersion**|1.31.1||**platformName**|MacOS||**platformVersion**|18.6.0|

We are aware of the issue and working on a fix. Closing as a duplicate of #2359. Thanks for using LiveShare!

We published a new release of the extension to the marketplace which contains a fix for this issue. The version number is 1.0.471. Please update your extension in Visual Studio Code. Thanks everyone for the feedback!


VS Code extension has a dependency that isn't mentioned in the README

VS Code is displaying an error when I attempt to use the LO extension (installed from .vsix).

The error references, A shell for VS Code Web, a different extension than the one currently mentioned in the README.
Clicking "Install and Reload" fails with another error.

Steps to reproduce

Install the LO extension from the .vsix file.
Create an empty directory and open it in VS Code.
Run from the command palette: LO: Initialize project in the current workspace

Expected result
A LO project is initialized.
Actual result
VS Code displays an error (screenshot above).

Yeah, I tried to document this in extension README.
If the Web Shell or WASM ext. won't install automatically you need to search for them manually in the marketplace and install the pre-release version. But I think I saw some kind of "Install Pre-Release" popup when installing.
Hopefully it will get a release version soon, maybe this should get more +1s: https://github.com/microsoft/vscode-wasm/issues/152

Will close this as it looks like vscode-wasm extensions might get a release version soon.


Why the POOLING_MODE = 'crop' ?

When training the network in batches all input images need to be of the same size.
As you can see, the default (in this implementation) is to set the shortest axis to a length of 600 and scale the longer axis accordingly. (e.g. input 280x420 would become 600x900)
Though as input images tend to have different sizes, depending on the dataset, you will still receive different size images.
So the next step is to set a max. axis length (1000 by default in this implementation) and either pad the images that are too small or crop the images that are too big (i.e. the longest axis exceeds 1000).
I believe literature states that using (random) cropping also works as a form of data augmentation, which in this case is good for training your model.

I think it's a bug in config.py.The default setting of pooling is roi align. But setting pooling_mode=crop doesn't affect the result. Because the training and testing code use config file(config/*.yml) as default. It will reset pooling mode=align.


serializedSize() fix

This commit fixes an incorrectly calculated size of a *FeatureVector in the serializedSize() function.
Deserialized messages read in via lnwire.ReadMessage are eventually passed to NewFeatureVectorFromReader in features.go.  In the loop on line 152, if a flag is invalid, it is not added to f.flags.
This is fine, except for the fact that when serializing the now deserialized message, the Encode function on line 192 allocates a []byte of length f.serializedSize().  If there are missing indices because of NewFeatureVectorFromReader not processing invalid flags, serializedSize() can more than likely calculate an invalid length because it uses len(f.flags).  For example, if we have a map that has indices in [0, 7764] and if indices (4, 5, 6, & 7) are found to have invalid flags and are thus not included in f.flags, the length calculated by serializedSize() will be 1941 instead of 1942.  This leads to a runtime error: index out of range when setFlag calls data[byteNumber] with data[1941] = ... which is out of bounds.

Should I include a small test / code snippet that demonstrates this?

The build now passes besides interface tests


[getmdl.io] ULs without LIs, non-a11y links

You have a footer on the getmdl.io with some problems in it:

Shouldn’t it be

 not
 according to spec (zero or more li elements)? If I were you I’d either add
 or remove
 at all. For educational reasons, at very least.
Empty  links are not accessible: it’s not clear where they lead you it you can’t see an icon. It would be good to put some text inside and hide it off-screen.

Haven’t found a special repository for getmdl.io, so posting it here.

You were right to open the issue here :)
This is something, #1113 addressed, but only for the templates, so thanks for paying attention. Will fix.


[feature] Remote storage

Is there any plans on releasing remote storage on waha-plus ?
I'm running my stack on Google Cloud Run, which are disk are ephemeral.
Would be nice if we could develop on our end, some common endpoints that WAHA would use to save and retrieve stored sessions

@allburov what is the path inside the container that should be persisted?
Thanks a lot for this NFS in GCP

@allburov thanks for the tutorial wasn't aware of it, but I will try to spawn a VM to run the project.
Last question, which path/folder inside the cointainer should be persisted when mounted a docker volume?

/app/.sessions
https://waha.devlike.pro/docs/how-to/sessions/


Xverse Wallet by device

Description of the bug
Xverse wallet is working well on laptop
but on mobile, it is very shit
i cant see wallet sign message
Working Environment (e.g. operating system, browser, device)
Windows
Please add screenshots (if applicable)
No response
Add any other context about the problem here
No response

that is concerned about device


Remove unused grip touch action from default OpenXR action map

The default OpenXR action map we create for Godot has a grip_touch action that is not mapped to any controller, because no controller supports it. It was added to the action map by accident (many other inputs do have touch versions, active when the user is touching, but not pressing down on, the button).
This PR remove the action.
This is not a breaking change as the default action map is only applied to new projects, or if the user resets their action map back to default. A users existing project is thus unaffected.
Also the action never did anything.

Thanks!

Cherry-picked for 4.2.1.


Expand ldap search filters to allow for any legal expression

Expand ldap search filters (user & group) to allow for any legal filter expression.
Older implementation assumed all ldap filters end with "={0}"
This newer implementation allows the user to craft any legal filter expression, including complex compound expressions, like
(&(|(member={0})(member={1}))(objectClass=group))
Introduces attribute: auth_provider.ldap_search_filter.group
obsoletes attribute: auth_provider.ldap_search_filter.member
Signed-off-by: Chuck Levesque<EMAIL_ADDRESS>

Note: does not do any validation assert that the string is legal, have seen an example regex, but this only checks characters/ordering.

Added unnecessary svcs (rhel8) nm-cloud-setup.timer & nm-cloud-setup
roles/prereqs/os/defaults/main.yml
per ENG

Don't create linux accounts on ecs hosts from  the list of Base Cluster accounts, we only need cloudera-scm on ecs hosts.
roles/prereqs/user_accounts/tasks/main.yml

On ECS hosts only create file acls for cloduera-scm and not other base accounts.
roles/security/tls_generate_csr/tasks/main.yml
roles/security/tls_generate_csr/tasks/acls_ecs.yml

@clevesque your branch here appears to have duplicated the commits from my Ranger fix, and also needs the DCO signing completed before we can merge please.


[stitcher] merge from youtube-dl

Description of your pull request and other information
merge stitcher extractor from youtube-dl
(fixes #21)

Is this an exact copy of the commits from upstream? or have you made any additional changes?

I cherry-picked the 2 commits from upstream without modificaton

hey @nixxo
I wanted to talk to you privately. Could you DM me on discord? Or can u give me some other means to contact you


Check for dust is incorrect, if the receiving address has an dust allowance

Issue description
The check for sending dust just verifies if amount is < 1000000i, it is possible to send a value less than that amount if a destination address has a dust allowance.
Wallet version
0.3.0
System specification
Windows
Is it a repeatable issue?
Y
Steps To reproduce the issue
Explain how the maintainer can reproduce the issue.

Try to send an amount < 1000000 to an address with dust allowance

Expected behaviour
Fails with error message "You cannot send less than 1 Mi."
Actual behaviour
Should be able to send dust to an address with dust allowance

I think the expected behavior and the actual behavior are mixed up.
Currently one can’t send < 1Mi to an address already having more than 1 Mi, but the expected behavior is that one can send.

The dust protection does not allow single UTXO < 1Mi, independent on what Mi's on the address already are.
The dust protection only allows UTXOs <1Mi to an address that has a special crafted "dust allowance" UTXO between 1 Mi and 10 Mi's (1 Mi allows up to 10 dust transactions, 10 Mi allows up to 100 dust transactions).
The resolution need to be:

allow send transactions <1Mi to addresses that have this special crafted "dust allowance" UTXO (and not already used up allowed dust transactions).
allow creation of dust allowed addresses with a deposit of 1 to 10 Mi's. Have a special list of such dust allowed addresses to copy / QR code these. Have a special function to collect dust transaction to a "collect address" (to allow more new dust transactions to the dust address).

I'm closing this down, since the issue is fixed.


Suggest substitution variables when editing launch.json ans tasks.json

It's rather awkward that any time I want to edit configuration files I need to look up on the internet the list of substitutions (${workspaceFolder}, ${file} and the rest). It would be nice to have them listed in comments or suggested by auto-completion.

This appears to be done. Type $ then trigger suggestions.


Completely broken

i have no idea why but in third person the animation is completely stuffed up and in first person it works but the shield isnt hidden

(edit i found out why i have a low shield texture pack but it still doesnt work in third person)

Fixed in 1.2.0


Need clarification

It seems brilliant, but it needs some clarification.
How are the weights in strategies determined?

phrase: "model"
weight: 100
contexts: ["as a language ", "as an AI language "]

what does the weight 100 represent?
and why in

phrase: "cannot"
weight: 20
contexts: ["I'm sorry, but I "]

it is 20...
How would I go with building bad and good strategies - how would I determine the weight?

Weight is a measure of importance, and serves as a way to balance the way the algorithm tries to meet its objective. Let's take the following situation;
Phrase 1
Probability in the model: 20%
Weight: 1
Phrase 2
Probability in the model: 2%
Weight: 1
If you yourself consider the reduction of phrase 2 just as important as phrase 1 you would apply a weight of 10 to phrase 2 so that its probability get inflated to match that of phrase 1. You could even put it higher then that.
When MergeMonster runs, each merge will be tested - If you had left phrase 2 at weight 1, a 1% reduction might not be sufficient to accept the merge, a matter which gets ridiculously complicated once you introduce a long list of phrases. However, with the new weight (10) the Monster now considers it a 10% reduction, and it might accept the merge.
This does come with the added risk of slightly inflating phrase 1's probability again - If phrase 1 increases by 2% but the (now inflated) phrase 2 has a 10% reduction the algorithm will still accept the merge. It becomes a matter of personal preference at that point.
I hope this somewhat explains the underlying concept!

I've started working on an optional mechanism that'll automatically adjust the weights of each phrase<>context pair so that they're considered equally important for the algorithm, potentially saving a lot of future headaches.


Adding path information to origins Endpoints

Also now support Netstorage and MSL origin types

@stuartio can you pull together an arrows.app schema diagram of the new schema layout which includes paths?  I seem to recall we did this at one point, but am not sure if we ever captured it.
I would like to compare that new schema layout to what we ingest from a test run using this version if that is ok.  It will also help us discuss it and document it for future reference.

Here is an arrows.app

@stuart-macleod-akamai are ready to get this merged?  I think we discussed that we are fine with this little tech debt in the pathing logic as long as it doesn't break the other functionality.
Have you run this against our account and show that the number of ingests pre-and post this change comes out the same.  Theoretically, if we ingest using the existing version, then again using this branch's code, the number of nodes should be indentical and the only diff should be the path relationship information.  Correct?


prysm forgets wallet password after update

Update from beta-5 to beta-6 somehow deleted/reset the password for the wallet and the user got error messages like
level=error msg="could not open wallet: password did not pass validation: password input cannot be empty" prefix=main

After just reimporting the error changes to
time="2021-06-05 13:50:13" level=info msg="Enabling web portal to manage the validator client" prefix=node
time="2021-06-05 13:50:13" level=info msg="Opened validator wallet" keymanager-kind=direct prefix=node wallet="/data/wallets/direct"
time="2021-06-05 13:50:13" level=error msg="could not read keymanager for wallet: could not initialize imported keymanager: failed to initialize account store: wrong password for wallet entered: invalid checksum" prefix=main

After doing

stop the services
delete ./data/prysm/validator/wallets
reimport again

Validator service resumes operation as expected.

Possible fix is to adapt update process:

copy ./data/prysm/validator/passwords/wallet-password to /etc/stereum/prysm-wallet-password.tmp before updating repos (git checkout <tag>)
copy /etc/stereum/prysm-wallet-password.tmp to ./data/prysm/validator/passwords/wallet-password after successful update

A similar thing happened to me.  I am running on mainnet, and after the update to Prysm v1.3.10, the web UI doesn't recognize my account.  The validator is running, I've run the list-accounts command and I can see the account.  The logs indicate that the validator is participating in the epochs successfully, but the web UI isn't seeing the accounts.  In my case, it might not be an issue with this repo, but a bug in Prysm.  I reported it in their Discord.  The above fix didn't work for me.

Hi @rhythnic thanks for dropping in! As I understand it your validators kept attesting?

Yes, that's right @stefa2k .  There is a confirmed bug(s) with the web UI in this release, but the validator seems to be working fine.

Here's the bug:
https://github.com/prysmaticlabs/prysm-web-ui/issues/181


fix: accept array of font face src

This PR adds support to the fontFace function accept an array of font face src
Implemented a font configuration using the fontFace function that includes multiple src paths.
This method appears to be a valid approach as it can be used without causing any type errors.
type FontFaceRule = {
    src: string | readonly string[];
    ...
}

export const Pretendard = fontFace([
  {
    src: [
      `url("/fonts/Pretendard-Regular.woff") format("woff")`,
      `url("/fonts/Pretendard-Regular.woff2") format("woff2")`,
      `local("Pretendard-Regular")`,
    ],
    fontStyle: "normal",
    fontWeight: "400",
    fontDisplay: "swap",
  },
  {
    src: [
      `url("/fonts/Pretendard-Medium.woff") format("woff")`,
      `url("/fonts/Pretendard-Medium.woff2") format("woff2")`,
      `local("Pretendard-Medium")`,
    ],
    fontStyle: "normal",
    fontWeight: "700",
    fontDisplay: "swap",
  },
]);

However, after converting the code to CSS, there was a problem that did not match the CSS standard.
The expected CSS format should show all source paths in a comma-separated line within the src property.
The currently generated CSS has each src separated into a separate declaration as follows,
This does not conform to the CSS standard, and as a result, there was a problem with the font not being applied correctly.
@font-face {
    src: url("/fonts/Pretendard-Regular.woff") format("woff");
    src: url("/fonts/Pretendard-Regular.woff2") format("woff2");
    src: local("Pretendard-Regular");
    font-style: normal;
    font-weight: 400;
    font-display: swap;
    font-family: "font_Pretendard__1obv0sp0";
}

The correct CSS format should include all paths in the src property separated by commas within one declaration as follows.

@font-face {
    src: url("/fonts/Pretendard-Regular.woff") format("woff"), url("/fonts/Pretendard-Regular.woff2") format("woff2"), local("Pretendard-Regular");
    font-style: normal;
    font-weight: 400;
    font-display: swap;
    font-family: "font_Pretendard__1obv0sp0";
}

To resolve this issue, I modified the renderCSS function and added a test code to validate the fix.

@365support Thanks for the PR.
This is actually intentional behaviour related to a Vanilla Extract feature that lets you write fallback styles for property values that may not exist in some browsers. This feature is available on any CSS property.
While I admit it would be nice to automatically concatenate an array when passed to src, this conflicts with the existing fallback feature, and is fairly trivial to implement as a consumer.


net.handle:handle:jar:9.3.0 is missing

Describe the bug
I pull master to local and use mvn compile but  report "[WARNING] The POM for net.handle:handle:jar:9.3.0 is missing, no dependency information available " ,
I can't find the net.handle:handle:jar:9.3.0 in center repo  but net.handle:handle-client:jar:9.3.0
where is bug , how to fix it

Closing, not reproducible. This should be moved to one of our Support options if the issue still exists: https://wiki.lyrasis.org/display/DSPACE/Support
The Handle.jar version 9.3.0 should be automatically downloaded via our Maven configuration (when you run mvn package or similar).  It's available here https://handle.net/maven/net/handle/handle/ , but the settings in our main pom.xml should autodownload it as we have that location listed as a Maven repository: https://github.com/DSpace/DSpace/blob/main/pom.xml#L1971-L1974
If you are still having issues, use one of the support options above to ask for help.


Fix #1065 : Wrong span name when request failing with a 401

This is more a proof of concept than an actual PR as I'm not convinced it's the proper way to fix this issue.
At the minimum the test case could be useful when providing a proper solution.

Too much change on TraceWebFilter to resolving conflict safely. I will close the PR for now.


[FireBullet] [Reviewed] Added bullet speed variance (useful for shotgun effect)

Bullet speed variance: Each bullet speed will be adjusted by a random value within this range (pixels/second)

Playable game
https://liluo.io/victrisgames/extension-fire-bullet
Project files
https://github.com/GDevelopApp/GDevelop-examples/pull/466
Video
https://user-images.githubusercontent.com/8879811/217987952-f81ba781-3cdb-4857-a780-304cd7a7a378.mp4

It seems that these 2 PR got removed:

https://github.com/GDevelopApp/GDevelop-extensions/pull/731
https://github.com/GDevelopApp/GDevelop-extensions/pull/727

You probably used an old version of the extension.

Good catch. I did use the previous FireBullet example without updating the extension first. I will fix this.


Revision of the agreement

Article 7:

Where processing is based on consent, the controller shall be able to demonstrate that the data subject has consented to processing of his or her personal data.

If the data subject's consent is given in the context of a written declaration which also concerns other matters, the request for consent shall be presented in a manner which is clearly distinguishable from the other matters, in an intelligible and easily accessible form, using clear and plain language. Any part of such a declaration which constitutes an infringement of this Regulation shall not be binding.

The data subject shall have the right to withdraw his or her consent at any time. The withdrawal of consent shall not affect the lawfulness of processing based on consent before its withdrawal. Prior to giving consent, the data subject shall be informed thereof. It shall be as easy to withdraw as to give consent.

I think this requires few actions:

[x] log the when the agreement is accepted (even at registration)
[x] give the admins a way to force members to re-accept the agreement
[x] visually separate the general agreement from the consent for data processing (maybe just a specific section in the agreement is enough?)
[x] the revocation is the tricky part: if "we" request approval to register, when a member requests to withdraw the consent, then he should consequently request an account deletion, this can be a special action, but the removal of the tick, IMO should lead to that.
[x] it may be necessary to actually log also the agreement they have consented to

Related to this, article 13:

Where personal data relating to a data subject are collected from the data subject, the controller shall, at the time when personal data are obtained, provide the data subject with all of the following information:

(a) the identity and the contact details of the controller and, where applicable, of the controller's representative;
(b) the contact details of the data protection officer, where applicable;
(c) the purposes of the processing for which the personal data are intended as well as the legal basis for the processing;
(d) where the processing is based on point (f) of Article 6(1), the legitimate interests pursued by the controller or by a third party;
(e) the recipients or categories of recipients of the personal data, if any;
(f) where applicable, the fact that the controller intends to transfer personal data to a third country or international organisation and the existence or absence of an adequacy decision by the Commission, or in the case of transfers referred to in Article 46 or 47, or the second subparagraph of Article 49(1), reference to the appropriate or suitable safeguards and the means by which to obtain a copy of them or where they have been made available.

In addition to the information referred to in paragraph 1, the controller shall, at the time when personal data are obtained, provide the data subject with the following further information necessary to ensure fair and transparent processing:

(a) the period for which the personal data will be stored, or if that is not possible, the criteria used to determine that period;
(b) the existence of the right to request from the controller access to and rectification or erasure of personal data or restriction of processing concerning the data subject or to object to processing as well as the right to data portability;
(c) where the processing is based on point (a) of Article 6(1) or point (a) of Article 9(2), the existence of the right to withdraw consent at any time, without affecting the lawfulness of processing based on consent before its withdrawal;
(d) the right to lodge a complaint with a supervisory authority;
(e) whether the provision of personal data is a statutory or contractual requirement, or a requirement necessary to enter into a contract, as well as whether the data subject is obliged to provide the personal data and of the possible consequences of failure to provide such data;
(f) the existence of automated decision-making, including profiling, referred to in Article 22(1) and (4) and, at least in those cases, meaningful information about the logic involved, as well as the significance and the envisaged consequences of such processing for the data subject.

Where the controller intends to further process the personal data for a purpose other than that for which the personal data were collected, the controller shall provide the data subject prior to that further processing with information on that other purpose and with any relevant further information as referred to in paragraph 2.

Paragraphs 1, 2 and 3 shall not apply where and insofar as the data subject already has the information.

I was looking into logging the acceptance of the agreement.
As a bonus I was thinking to also store in the db a log of the changes to the agreement (in order to have a unique and stable reference the accepted agreement).
The basic concept is easy, though there is something that makes it a little complex: multiple languages.
This opens up a rainbow of possibilities and I'm unsure what's the best.
The easiest may be just not to care about multiple languages at all, always store the English one and be done with it. Though this would work only in case English is actually updated, and if only the language specific agreement is updated it would fail completely.
The second easiest would be to store whatever is changed (and marked to be stored since I added a checkbox to tick in order to force the members to accept the agreement). The drawback is changing a language may force every user to have to accept again the agreement irrespective of their language.
Another alternative would be to go for the full-language-oriented, so only force the request when a a certain language agreement is changed. But that would make things a little more complex at saving/checking time and in case the member changes the language (because it could be forced to have to accept the agreement in that different language).
Another alternative could be to store a "dump" of all the agreements when a "force accept" is requested. That could be saved in the filesystem without having to disturb the database. The advantage would be the reference is simply the date the agreement was accepted (e.g. the agreements are store in a directory whose name is 2018-05-07 for example and in the log, this date is stored as reference to the accepted agreement). Drawback: usual issues with having to have a directory writable.
Opinions?

Just thinking how updated agreements are handled on other sites ... Maybe leave it up to the admin, if they update the agreement that alters how personal data is used/held then they can set a ACP flag to require a re-acceptance on next login.  It leaves it in the admins hands but ultimately they are responsible.

In general is done.
I think we can improve it a bit, but there is 2.0 for that.


Nnaja support

Added 3 functions to adapt to both nette.ajx.js and Naja.js.

Thanks a lot!

@filipcro Would you mind me taking some of your changes from your previous pull-request into future versions of datagrid? (mostly the npm part)

Sure, use it. I'll also suggest adding files property so npm won't include PHP files.

Nice, thanks!

@filipcro In this form, JavaScript console is throwing "naja is not defined" (https://github.com/contributte/datagrid/pull/837/files#diff-29df16b6d259eb43c865c3d6d2e8c4f7R3)
From JavaScript developer, is it OK to fix it likat that?
if (naja) {

->
if (typeof naja !== "undefined") {

(https://github.com/contributte/datagrid/commit/051b8932268db1fa4c661fe05a034be48cf0d366)

That will work.

@paveljanda, @filipcro
Are you sure about correct functionality of this merged pull request ?
Which version of Naja did you used ?
I got a bunch of errors:
TypeError: naja.addEventListener(...).bind is not a function datagrid.js:14 - 36 => maybe you should change "}).bind(this)"  to "}.bind(this))" like in Naja doc example.
Naja version 1.7.0

@MartinSzollos2016  it's because naja is loaded after datagrid. Try loading naja before.
https://github.com/contributte/datagrid/issues/859

@paveljanda maybe adding this somewhere to docs?

@dakorpar
Still not working properly, TypeError is not thrown but corresponding datagrid extensions are not loaded either.

which extension for example?
I'm using naja on one of projects and did not noticed problems?

Eg.datagrid.confirm don't show any message or auto submit text input causes redirect instead of ajax call.
Could you please send me your init script ? Maybe i have done something wrong. thx


One active runtime can support multiply session in the same time?

Actually,I wonder may I open multiply OpenXR app at the same time like the normal desktop app.

That depends on the runtime. It's not required by the spec, but in practice many runtimes will offer this.
You might want to try asking some of these questions in the forum https://community.khronos.org/c/openxr or Slack chat https://khr.io/slack/ to reach more people more quickly.

@rpavlik Thank you very much,that is a very helpful advice.


Getting No-Access-Control-Origin error on previously working/completed Camper News Project

Issue Description

I have noticed that when I return to view my completed Camper News project, I am failing to get access to the API via AJAX, as I'm now getting the "No-Access-Control-Origin" error in the console. This has been working from the moment I completed it (2015), but for reason, it does not work anymore, so no news info is populating the page as I designed. Not sure if FCC server still accepts CORS or if I'm doing something wrong?
I tried pulling up the project on codepen via Firefox Browser, as well as my iPhone's Safari browser. Same results.

My Browser Information

Chrome Version 49.0.2623.87 (64-bit)
Linux, Ubuntu Mate 15.10

My Code
My codepen
var req = new XMLHttpRequest();
req.open("GET", "http://www.freecodecamp.com/news/hot", true);
req.onload = function() {
  var fcc;
  fcc = JSON.parse(req.responseText);
  var amountToDisplay = 40;
  for (var i = 0; i < amountToDisplay; i++) {
    var storySection = document.querySelector(".story-section");
    var story = document.createElement("div");
    var headline = document.createElement("a");
    var img = document.createElement("img");[GitHub](http://github.com)
    var author = document.createElement("a");
    var likes = document.createElement("p");
    story.classList.add("story");
    headline.classList.add("headline");
    headline.setAttribute("href", fcc[i].link);
    headline.innerHTML = fcc[i].headline;
    headline.setAttribute("target", "_blank");
    img.setAttribute("src", fcc[i].author.picture);
    author.classList.add("author");
    author.setAttribute("href", "http://www.freecodecamp.com/" + fcc[i].author.username);
    author.innerHTML = "By " + fcc[i].author.username;
    likes.innerHTML = "♥ " + fcc[i].upVotes.length;
    likes.classList.add("likes");
    story.appendChild(headline);
    story.appendChild(img);
    story.appendChild(author);
    story.appendChild(likes);
    storySection.appendChild(story);
  };

};
req.send();

@Ramon-Carroll We have recently switched to SSL everywhere on the FreeCodeCamp website, the requests now need to be made over https.
Try changing your call to the http://www.freecodecamp.com/news/hot to https
This should fix your issue.
However on a side note we have retired Camper news, and perhaphs the API will be turned off soon, The challenge would be updated with a alternate data source/api when that happens.
Refer #7700 and #7082

Closing as duplicate of #7617

@raisedadead Yep, that was all to it. Thanks for that! :)
Also, I'm aware that we dropped Camper news, but I wasn't aware that the API would be dropped too. I have a fully built portfolio, and that basically means that one of the projects that I have on display will break, and I'll have to replace it. While I understand that there's not much of a way around that inconvenience, I'd hope that we are notified of the new project long before the API is turned off, so I have sufficient time to replace it on my website.
Either way, thank you for all that you guys do. It's highly appreciated! :)


Correcting rollingOnTheFloorLaughing and Instruction usage.

Corrected rollingOnTheFloorLaughing value.
Corrected usage of notAValidScalar value from increment to set.

Thanks for the fixes.


Using Reflector with Controller

I'm new to both custom controllers and kube-rs so apologies if this is a noob question. I'm trying to write a controller for a custom resource that will spawn pods similar to a Job. I'm using the controller-rs project as my starting point and have it set up to trigger reconcile on any changes to my custom resource or relevant pods. To decide what actions to take, reconcile ought to query the latest state of the custom resource and any associated pods. I think I'm meant to use a Reflector to maintain a cache of the latest known state of all relevant custom resources/pods to make these queries efficient.
The first part I'm struggling with is how to maintain the stores. I can create a reflector as in deployment_reflector.rs, but presumably to keep updating the cache I need to keep awaiting new events. My first instinct is to spawn a tokio task that will keep running the watcher loop, but this doesn't seem to work because the reflector stream is not Send.
The second part I'm not sure how to achieve is waiting for all the stores to catch up with the initial flurry of events before allowing the controller to start running and potentially acting on very incomplete state. Ideally, I would also like to tie the watcher loop into the controller somehow in a way that to ensures that reconcile will not process any events that have not yet been applied to the stores.

but this doesn't seem to work because the reflector stream is not Send.

It.. should be, that sounds like a bug.

I can create a reflector as in deployment_reflector.rs, but presumably to keep updating the cache I need to keep awaiting new events. My first instinct is to spawn a tokio task that will keep running the watcher loop

The correct thing to do here should be to use Controller::owns or Controller::watches, but neither of those actually support feeding a Reflector at the moment.. :/

The Send part is probably because the reflector returns a watch stream, which is not Send by default, unless it's put in a BoxedStream. But there's not actually a need require this, you could just put a for_each on the stream:
    let rf = reflector(store, watcher(myapi, lp));
    let drainer = try_flatten_touched(rf)
        .filter_map(|x| async move { std::result::Result::ok(x) })
        .for_each(|o| {
            debug!("Touched {:?}", Meta::name(&o));
            futures::future::ready(())
        });

and await the drainer future at the end in a select! with whatever other runtime futures you may have.
As for feeding the store back into the reconciliation loop, I would try to put the store on your Context struct.

Thanks for the detailed explanations, this makes sense now!

The correct thing to do here should be to use Controller::owns or Controller::watches, but neither of those actually support feeding a Reflector at the moment.. :/

I'm just gonna be prototyping things for a while so not concerned about getting it as correct as possible, but good to know this might be an option in the future.


Multi-tokens stop sequence

I would like to use mistral.rs as a backend on https://github.com/lucasavila00/LmScript
LmScript hopes to hide the token concept from users so it needs support for multi-token stop sequences.
I can try implementing it, if it is a desired feature.

@lucasavila00, thanks for offering: this is certainly a desired feature! Most of the code to support this should already be in place, and if you could try to implement it, that would be great.


Wrong MSI detection for S1 in the reference model leads to wrong OS driver quirk in the recent Linux kernel

In iommu_translate.c, the reference model is implemented in this way:
    // Count misses in TLB
    count_events(PV, PID, PSCV, PSCID, DID, GV, GSCID, IOATC_TLB_MISS);
    if ( two_stage_address_translation(req->tr.iova, check_access_perms, DID, is_read, is_write, is_exec,
                                        PV, PID, PSCV, PSCID, iosatp, priv, SUM, DC.tc.SADE,
                                        GV, GSCID, iohgatp, DC.tc.GADE, DC.tc.SXL,
                                        &cause, &iotval2, &gpa, &page_sz, &vs_pte) )
        goto stop_and_report_fault;

    // 18. If MSI address translations using MSI page tables is enabled
    //     (i.e., `DC.msiptp.MODE != Off`) then the MSI address translation process
    //     specified in <<MSI_TRANS>> is invoked. If the GPA `A` is not determined to be
    //     the address of a virtual interrupt file then the process continues at step 19.
    //     If a fault is detected by the MSI address translation process then stop and
    //     report the fault else the process continues at step 20.
    if ( msi_address_translation(gpa, is_exec, &DC, &is_msi, &is_mrif, &mrif_nid, &dest_mrif_addr,
                                 &cause, &iotval2, &pa, &gst_page_sz, &g_pte, check_access_perms) )
        goto stop_and_report_fault;
    if ( is_msi == 1 ) goto skip_gpa_trans;

That means for an S1 only translation, an address will be translated prior than to be detected as MSI.
While in the specification, MSI configurations are resident in the DC, which is independent of S1.
Also we can see a different model in qemu:
https://gitlab.com/danielhb/qemu/-/blob/riscv_iommu_v5_rc1/hw/riscv/riscv-iommu.c?ref_type=heads
Which detects MSI prior than performing S1:
static int riscv_iommu_spa_fetch(RISCVIOMMUState *s, RISCVIOMMUContext *ctx,
    IOMMUTLBEntry *iotlb)
{
    ....

    /* Early check for MSI address match when IOVA == GPA */
    if ((iotlb->perm & IOMMU_WO) &&
        riscv_iommu_msi_check(s, ctx, iotlb->iova)) {
        iotlb->target_as = &s->trap_as;
        iotlb->translated_addr = iotlb->iova;
        iotlb->addr_mask = ~TARGET_PAGE_MASK;
        return 0;
    }

The wrong reference model requires a special quirk to be introduced in the recent Linux kernel IOMMU driver, which requires a VA=PA mapping to be created for the MSI table. And now this is known to be the significant difference between Linux 6.6 and Linux 6.10.
	imsic_global = imsic_get_global_config();
	if (!imsic_global || !imsic_global->nr_ids)
		return 0;

	base = imsic_global->base_addr;
	stride = IMSIC_MMIO_PAGE_SZ << imsic_global->guest_index_bits;
	for (i = 0; i < BIT(imsic_global->hart_index_bits); i++) {
		if (riscv_iommu_map_pages(&domain->domain, base, base,
					  IMSIC_MMIO_PAGE_SZ, 1, prot,
					  GFP_KERNEL_ACCOUNT, &mapped)) {
			/* unroll mapping */
			do {
				riscv_iommu_unmap_pages(&domain->domain, base,
							IMSIC_MMIO_PAGE_SZ, 1,
							NULL);
				base -= stride;
			} while (i-- > 0);

			return -ENOMEM;
		}
		base += stride;
	}

Two stage address translation is always active in the IOMMU and there is no option to disable it. However, any stage can be effectively disabled by programming the mode for that stage as Bare. If VS-stage is not Bare for a transaction then the transaction carries an VA and not a GPA. The MSI detection is performed on a GPA and not an VA and the VS-stage address translation must be performed even if the VA is identity mapped (which the IOMMU cannot guess) in the VS-stage. So all of this is as per specification. Please take it up on LKML if there are questions about the Linux kernel.

The usage model is not related to the 2-stage address translation.
The problem is seen in a supervisor Linux kernel where a 6.6 based IOMMU driver patchset is applied. This version of IOMMU patchset is lack of the above mentioned MSI table identity mapping:

Same Linux kernel runs fine on Qemu.

This looks weird that in an S1 only environment, without MSI PT feature, only S1 PT translation should be required for MSI writes. While with MSI PT feature, not only S1 PT translation but also MSI PT translation is required for MSI writes.

This looks weird that in an S1 only environment, without MSI PT feature, only S1 PT translation should be required for MSI writes. While with MSI PT feature, not only S1 PT translation but also MSI PT translation is required for MSI writes.

If software has configured the IOMMU to do VS-stage address translations then the IOMMU does VS-stage address translations. If further software has configured MSI page tables to translate MSIs in Basic or MRIF mode then the IOMMU translates further using the MSI page table.

Yes, OSen like Linux only provides MSI specific operations in the vfiommu framework and the feature looks tightly related to the S2. But let me try to express the data flow, and please correct me if they are wrong.
My understanding is:

MSI PT unmapped with DMA remapped
When a guest OS (VS) is using the MSI feature, it need not to map the MSI address into a virtual address while still be able to perform the DMA remapping for the IO addresses used by the DMA devices, allowing such devices to send MSIs by treating the MSI address as a physical address. While the hypervisor OS (HS) detects the MSI writes from other remapped DMA transactions by matching them with MSI PT entries. NOTE that the physical address is only used by the device thus it won't be accessed by CPU, and not remapped into the virtual address space.
When the hypervisor OS is using the MSI feature, it can also not to map the MSI address into an virtual address while still be able to perform the DMA remapping for the IO addresses used by the DMA devices, allowing devices to send  MSIs by treating the MSI address as a physical address.

Similarly,
2. MSI PT remapped with DMA remapped
When a guest OS is using the MSI feature, it can map the MSI address into a virtual address while still be able to perform the S1 DMA remapping for the IO addresses used by the DMA devices, allowing devices to send MSIs by treating MSI address as a virtual address.  While the hypervisor OS detects the MSI writes from other remapped DMA transactions by matching them with MSI PT entries after performing the DMA translations. NOTE that the virtual address is not used by CPU, and there might not be a virtual address space region ready to identity map the physical MSI address.
When the hypervisor OS is using the MSI feature, it can also map the MSI address into a virtual address while still be able to perform the S1 DMA remapping for the IO addresses used by the DMA devices, allowing devices to send  MSIs by treating the MSI address as a virtual address.
 Among the above scenarios, which one is the better practice in OSen?
The QEMU model allows both usage models, RIVOS model only allows the 2nd usage model. Is there any concern to restrict MSI PT for being used only in vfiommu framework?

NOTE that in an SoC design, devices including MSI write addresses should be designed to be resident in the higher physical address space while DMA remapped zone should be resident in the lower virtual address space. Such detection should be safe without worrying about the conflict of DMA transactions and MSI writes.

Please see IOMMU specification section "Process to translate IOVA".

In step 17, the IOVA is first translated to a GPA.
Then in step 18, the GPA is used to determine if it is a MSI if MSI address translation is enable. The VS-stage cannot be skipped before doing MSI detection.

I briefly looked through the QEMU patches and see somewhere along the line it dropped en_s from the patch and this change makes it not compliant with the RISC-V IOMMU specification since when VS stage address translation is enabled whether the IOVA is identity mapped or not cannot be inferred i.e. IOVA may not be same as GPA when iosatp.MODE != Bare.

      /* Early check for MSI address match when IOVA == GPA */
-    if (!en_s && (iotlb->perm & IOMMU_WO) &&
+    if ((iotlb->perm & IOMMU_WO) &&

The GPA corresponding to the virtual IMSIC is mapped into the guest. It is also mapped into the virtual address space of the guest OS since that mapping is required for the OS to do IPIs.

The QEMU model allows both usage models, RIVOS model only allows the 2nd usage model.

What is RIVOS model?


Database factory injection

LGTM !
Only an entry arlas_database_factory_class is missing in docs/arlas-server-configuration.md file


Unable to use gRPC from external and REST from same machine at the same time

Background
When trying to use gRPC from outside and REST from the same machine at the same time REST calls does not work.
Your environment

lnd version 0.5.2-beta commit=v0.5.2-beta
Linux rv1025 3.13.0-108-generic #155-Ubuntu SMP Wed Jan 11 16:58:52 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
Ubuntu 16.04.6 LTS
Bitcoin Core Daemon version v0.17.1 (bitcoind)

Steps to reproduce

Configure lnd.conf

restlisten=localhost:19925
rpclisten=<IP_ADDRESS>:10009
tlsextraip=a.b.c.d
externalip=a.b.c.d

Delete certificate
Restart lnd to create new certificate
Use external application/wallet to connect to lnd via gRPC with admin.macaroon and tls.cert
Use internal (running on the same machine like lnd) application/wallet to connect to lnd via REST

external client can be a wallet like zap, pebble, shango, etc.
for internal client RTL was used (Ride The Lightning)

Expected behaviour
Step 4 and 5 should work.
Actual behaviour
Step 5 does not work.

All Subconns Are In Transientfailure, Latest Connection Error: Connection Error: Desc = "Transport: Authentication Handshake Failed: X509: Certificate Is Valid For <IP_ADDRESS>, ::1, a.b.c.d, <IP_ADDRESS>, xxx::xxxx:xxxx:xxxx:xxxx, Not <IP_ADDRESS>"

RTL is running on the same machine like lnd and it's configured to access it via https://localhost:19925/v1

This has been fixed in master (#2247) and will be included in the next release. See https://github.com/lightningnetwork/lnd/issues/1567#issuecomment-471178535 for a temporary workaround.


Failing to build Lorri when behing a proxy

I'm trying to build lorri at work. It's failing with the message below. We're behind a proxy my guess is that I need to somehow setup the HTTP(S)_PROXY environment variables (I'm using Darwin-nix with multi-user Nix and the nix-daemon is correctly setup with the proxy environment vars)
source root is lorri
patching sources
installing
    Updating crates.io index
warning: spurious network error (2 tries remaining): curl error: Could not resolve host: github.com
; class=Net (12)
warning: spurious network error (1 tries remaining): curl error: Could not resolve host: github.com
; class=Net (12)
error: failed to sync

Caused by:
  failed to load pkg lockfile

Caused by:
  failed to fetch `https://github.com/rust-lang/crates.io-index`

Caused by:
  curl error: Could not resolve host: github.com

I think the problem comes from rust (or some related utilities) trying to access the internet from the build sandbox.
For that, the appropriate environment variables (HTTP(S)_PROXY) need to be setup in the build sandbox environment.
I'm not sure how to do that.
Thank you

It looks to me like we are correctly setting the right proxy variables in fetchcargo (which is used by us to build lorri):
https://github.com/NixOS/nixpkgs/blob/master/pkgs/build-support/rust/fetchcargo.nix#L61
which is this list of proxy variables: https://github.com/NixOS/nixpkgs/blob/master/lib/fetchers.nix
Do you know how we could reproduce your environment to have a chance at reproduction?

Ah ah, thanks for the pointer!
The vars are defined with uppercase letters in my config, so maybe that's the problem.
I'll test adding the lowercase version tomorrow at work and let you know if it fixes the issue.

The different casing of the environment variables was indeed the problem.
I've added lower cased versions and now building lorri works like a cinch.
Thank you so much for your help Profpatsch!

Awesome, I think we can close this then.

Yes.  I’ll go ahead and close it.


Re-add the local_config.sh

local_config.sh should not be committed to project repos, so it's listed in the .gitignore inside the template. But, it should be included here in the template repo, to serve as a starter for projects, so I had to git add -f to add it over the objections of the .gitignore file inside the template subdirectory.
@bryanluby this will fix that problem you saw the other day.
@vokal/ios-developers quick one please

Sure


Cannot enable mouse support with views.Application

Correct me if I'm wrong, but it seems that you can't enable mouse support when using views.Application? If I make a simple program with my own screen and event loop, I can receive mouse events perfectly fine... However this example using views.Application will not give the expected output:
package main

import (
    "os"

    "github.com/gdamore/tcell"
    "github.com/gdamore/tcell/views"
)

type Widget struct {
    views.Spacer
}

func (widget *Widget) HandleEvent(event tcell.Event) bool {
    switch event.(type) {
    case *tcell.EventKey:
        panic("Key Event")
    case *tcell.EventMouse:
        panic("Mouse Event")
    }

    return false
}

func Run() int {
    screen, err := tcell.NewScreen()

    if err != nil {
        return 1
    }

    screen.EnableMouse()

    var app views.Application
    app.SetScreen(screen)
    app.SetRootWidget(&Widget{})

    if !screen.HasMouse() {
        panic("No mouse")
    }

    if err := app.Run(); err != nil {
        println(err)
        return 1
    }

    return 0
}

func main() {
    os.Exit(Run())
}

The "Key Event" panic can hit as expected when typing, but the "Mouse Event" cannot be hit by clicking/scrolling. Also screen.HasMouse() is still returning true anyway
Does screen.EnableMouse() need to be called after screen initialization? That's how I was using it in my other tests where I didn't use views.Application.
If that is the case, then it seems that there's the issue that initializing the screen before passing it to Application.SetScreen() seems to break the application, since it also attempts to initialize the screen:
https://github.com/gdamore/tcell/blob/761abf6821d766b234f301ec681ab63039d8d189/views/app.go#L128

Closed with https://github.com/gdamore/tcell/pull/494


Fix some type checker warnings

WIP

@hofaflo would you care to take a look, specifically if you are OK with the config change?

I think get_config_key() should be renamed to get_config_value() (thanks @hoechenberger!). I'll do that.

Yes, I think this is a very good idea, thanks for fixing that and the typing chaos! :sweat_smile:


(PDB-1632) Transfer old AMQP messages at startup

At startup, transfer any messages found in the old
com.puppetlabs.puppetdb.commands queue to puppetlabs.puppetdb.commands
so that pending commands won't be ignored after an upgrade.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1472/

Will this require some dashboard changes as well?

I think @ajroetker is right about that. Looks like the test failures may be caused by querying for inaccessible metrics:
https://github.com/puppetlabs/puppetdb/blob/master/acceptance/helper.rb#L562-562

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1478/

this doesn't seem to solve the root issue yet -- still can't see messages getting transferred on upgrade

We should also probably have some kind of test for this behavior as part of this PR

Reworked following @senior's suggestion to try JMX commands.

+1, verified it works

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1492/

Now with (at least some) tests!

looks good to me

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1495/

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1497/

@pljenkinsro retest this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1499/

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1501/


Add scigraph clique and candidate clique endpoints to biolink

Some time ago I added two functions to scigraph:
/dynamic/candidateClique/{id}
Returns ids that are equivalent and potentially equivalent (xrefs)
/dynamic/clique/{id}
Returns the id and all its equivalents
should we add this functionality to biolink?  this probably makes sense as a single function with params that specify how strict a user wants - equivalent, exactMatch, narrowMatch, broad, dbxref, proteins for genes, etc
cc @balhoff @mbrush

also related https://github.com/biolink/biolink-api/issues/346

exact same ticket from 2017 https://github.com/biolink/biolink-api/issues/55


fix: TASK-#15126 Airtable queries able to show the total count of rec…

Description
-This is the PR to show the total count of records for airtable queries.
fixes: #15126
changes in PR
-Updated the file in frontend file QueryDebuggerTabs.tsx, to make the data that is coming from api into array of objects.
-Updated the Frontend code , as a solution where the updated code will
- will check if the response is array of objects or not
- if not , then it will convert into an array of objects.
difference between PostgreSQL and Airtabls:
The main difference between relational and non-relational databases lies in how they represent data. Relational databases, like PostgreSQL, typically return data as an array of objects. This format is straightforward, with each object representing a row from a database table. On the other hand, non-relational databases, such as Airtable, often return data in a more nested structure. For instance, Airtable's data might be returned as an object that contains an array of records, where each record is itself an object. This difference in data representation can lead to inconsistencies in how data is processed and displayed, especially when integrating with systems expecting a specific format. Understanding and handling these differences is crucial for seamless data integration and manipulation across different types of databases

We can modify the SaasPlugin backend code to standardize API responses as arrays of objects. However, this is complex due to the need to handle various SaaS integrations like Google Sheets and AWS Lambda, requiring significant changes to ensure consistent data formatting across all services.

@ajinkyakulkarni

Summary by CodeRabbit

New Features

Enhanced the Query Debugger to better handle and display various response data structures, including arrays and objects with records.
Introduced improved error handling and parsing of response bodies during debugging.

I have reviewed and addressed the comments received from CodeRabbit. To resolve the issues, I added a separate method to handle the response parsing, thereby avoiding code duplication. After making these changes,
I cross-checked everything to ensure it works as expected. As a result, we can safely ignore the latest suggestion from CodeRabbit.ai.

Hii @ajinkyakulkarni and @chandannkumar, Please review this PR?

hii @rohan-arthur @Aishwarya-U-R @sumitsum , I hope you're well. I’m curious and excited to hear your thoughts on my pull request raised a while ago. Looking forward to your review!
Thank you.

hii @rohan-arthur @Aishwarya-U-R @sumitsum , I hope you're well. I’m curious and excited to hear your thoughts on my pull request raised a while ago. Looking forward to your review!
Thank you.

hii @rohan-arthur @Aishwarya-U-R @sumitsum , I excited to hear your thoughts on my pull request raised a while ago. Looking forward to your review!
Thank you.

hii @rohan-arthur @Aishwarya-U-R @sumitsum @Nikhil-Nandagopal  @NilanshBansal , can you assign reviewer to this PR.
Thank you.


Fix python call to spherical_project

This Commit fixes three issues:

tcsh does some weird argument splitting splits variables that should not be split, specifically: tcsh ... --py 'python3.10 -s' is split into "--py", "'python3.10", and "-s'". To fix this, the fs_time script was ported to bash, as it was impossible to accomplish this fix in tcsh.
spherical_project always failed, because lapy imports were outdated (lapy.diffGeo instead if lapy.diffgeo and lapy.triaMesh.TriaMesh instead of lapy.TriaMesh). This was a silent fail, as stderr was discarded in spherical_project_wrapper.py and so the fallback was always used.
spherical_project further "often" failed inside docker, when no user name could be found by getpass.getuser (in nibabel.io.freesurfer.write_geometry) -- raising an exception. (Happens if the user is defined for docker via -u, but there is no associated username inside the docker.) To fix this, we just add the environment variable "USERNAME" with value "UNKNOWN", if it is not set (this is a place getpass looks for the username last).

recon_surf/functions.sh

proper quoting of cmd recon_surf/spherical_project.py
fix lapy import recon_surf/spherical_project_wrapper.py
also print stderr output to the console
call spherical_project with USERNAME set recon_surf/fs_time
port to bash

thanks. do you know if this has been an issue already in the last release? Probably it also falls back to FreeSurfer in all cases then?

It should have been an issue since the lapy API was updated. I don't know when this was.
But theoretically, if you know what to search for, we can search the validation logs.

Correction 1 was introduced by my last PR... 2/3 would have caused failures before, but I don't know since when exactly


Add a signer: Text update

Task Description
Text updates for Add a Signer screen which is accessible via Manage Keys section on the home screen.

On the above page status of the Assisted Keys would be either of the following:

Pre-upgrade: "Upgrade to Hodler/ Diamond hands to use the key"
Post-upgrade: "The key is already added to your Manage Keys section"

https://github.com/bithyve/bitcoin-keeper/issues/4449 Ticket in 32 so closing this


Add token light to Flame Tongue and Flame Tongue (Greater). Add aura effect to Flame Tongue (Greater).

Note that the aura effect currently stacks due to https://github.com/foundryvtt/pf2e/issues/7654

Use a common slug if you don't want auras to stack.

Use a common slug if you don't want auras to stack.

The aura already has a common slug, so that part works. I guess I was unclear - I meant that the flaming rune granted by the aura stacks with the greater flaming on the sword itself.


bump bundler 2.2.17

What does this PR do?
Bump bundler
Change log

[ x ] The CHANGELOG has been updated, or
[ ] This PR does not include user-facing changes and doesn't require a CHANGELOG update

Test coverage

[ ] This PR includes new unit and integration tests to go with the code changes, or
[ x ] The changes in this PR do not require tests

Documentation

[ ] Docs (e.g. READMEs) were updated in this PR, and/or there is a follow-on issue to update docs, or
[ x ] This PR does not require updating any documentation

Bumping Bundler beyond 2.1.4 causes problems in the appliance build. I'll put a new PR in shortly that only goes to that version instead of all the way to 2.2.17


Update New-CsTeamsVoiceApplicationsPolicy.md

Updating the RealTimeXMetrics Important note.

@michbrown-png Thanks for pointing those out - the easiest way to repeat mistakes is copy and paste.  Spelling has been updated.

@CLYVR - Approved


Added russian translation

Hello, russian-speaking fellow here. Added russian translation(Русский) to Grid Garden, because I liked it so much and really understood CSS Grid by playing it. Manually checked for typos and all of that. Hope this will help this project to become more popular. Btw it's very popular in vk.com(russian facebook) right now, so I think people will like it.

@Zooll8, thanks for your help on this, always glad to make it accessible to more people.

@thomaspark, sorry for so late response. Glad I helped this project and made our small development world a little bit better. Thank you for making so useful and interesting way to learn CSS Grid by creating the Grid Garden.
🌱🌱🌱 Grid Garden 🌱🌱🌱


Add licensing information

This documents the licenses we expect, and the exact header which our
CI checks for.
Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>

hmm... why does it require a review from code owner @mkolesnik ... I'm a code owner (or should be) so that should be enough...

seems it likes me :)


Use it with Authboss

I'm trying to use authboss with Fiber.
I can't figure out how to use LoadClientStateMiddleware with adoptor like this:
app := fiber.New()
app.Use(adaptor.HTTPHandler(authboss.LoadClientStateMiddleware))

the error is:
Cannot use 'authb.LoadClientStateMiddleware' (type func(h http.Handler) http.Handler) as type http.Handler Type does not implement 'http.Handler' as some methods are missing: ServeHTTP(ResponseWriter, *Request)
Is there a way I can fix this?

Try HTTPHandlerFunc
app := fiber.New()
app.Use(adaptor.HTTPHandlerFunc(authboss.LoadClientStateMiddleware))

Name
Signature
Description

HTTPHandler
HTTPHandler(h http.Handler) fiber.Handler
http.Handler -> Fiber handler

HTTPHandlerFunc
HTTPHandlerFunc(h http.HandlerFunc) fiber.Handler
http.HandlerFunc -> Fiber handler

FiberHandler
FiberHandler(h fiber.Handler) http.Handler
Fiber handler -> http.Handler

FiberHandlerFunc
FiberHandlerFunc(h fiber.Handler) http.HandlerFunc
Fiber handler -> http.HandlerFunc

FiberApp
FiberApp(app *fiber.App) http.HandlerFunc
Fiber app -> http.HandlerFunc

It doesn't work either.

Hi @frederikhors
You can use new HTTPMiddleware wrapper to convert http middlewares to Fiber middleware
But LoadClientStateMiddleware sets context value which does not transfer to Fiber handler

Thanks @arsmn, I'll try it in a few minutes...
Meanwhile, I opened https://github.com/gofiber/fiber/issues/941because I have a doubt:

I cannot understand if using Authboss with LoadClientStateMiddleware before all routes can affect the speed I get from Fiber if I start processing all requests through that middleware.

Can you help me understand this point, please?
Thanks.

The http middleware works now!
Two remaining issues:

LoadClientStateMiddleware sets context value which does not transfer to Fiber handler, as you have noted; I'm trying to understand how to do

how to transform this code mux.Mount("/auth", http.StripPrefix("/auth", ab.Config.Core.Router)) for Fiber; this code comes from here: https://github.com/volatiletech/authboss-sample/blob/master/blog.go#L274.

I'm trying with this code which compiles:
func main() {
	ab := SetupAuthboss()

	app := fiber.New()

	app.Use(adaptor.HTTPMiddleware(ab.LoadClientStateMiddleware), remember.Middleware(ab))

	app.Group("/auth", adaptor.HTTPHandler(http.StripPrefix("/auth", ab.Config.Core.Router)))

	_ = app.Listen(":3000")
}

but crashes at runtime:
panic: use: invalid handler func(http.Handler) http.Handler

goroutine 1 [running]:
github.com/gofiber/fiber/v2.(*App).Use(0xc00011b200, 0xc00010ff58, 0x2, 0x2, 0x1, 0xbdcd94)
  C:/go/pkg/mod/github.com/gofiber/fiber/v2@v2.1.1/app.go:391 +0x2d7
main.main()
  C:/projects/go/fiberAndAuthboss/main.go:18 +0x129
exit status 2

FULL REPRODUCTION PROJECT HERE: https://github.com/frederikhors/fiber-and-authboss

Hi @frederikhors, thanks to @arsmn you now can use adaptor.HTTPMiddleware in v2.0.2
app := fiber.New()
app.Use(adaptor.HTTPMiddleware(authboss.LoadClientStateMiddleware))

Oh, thanks @Fenny.
I know that. I answered with another issue. Can you please re-open this, please?
Thanks.

of course using middleware affects on performance but I have not used Authboss and don't know how much is it.
I'm trying to figuring out a way to transfer context values
No idea about strip prefix right now, I guess you can handle it with a custom middleware
about your app crash, remember.Middleware(ab)  is not a valid Fiber middleware. You should use ```adaptor.HTTPMiddleware for this too

you're right. My mistake. I was incredibly distracted!

For future reference; there is an issue with transfering context values when using a middleware (lets say you  want to use auth0 jwt middleware). I'll try and find a solution and report here :)

@mrsufgi  did you find the solution?

@mrsufgi Wondering about that solution as well, without context values Auth0's go-jwt-middleware is essentially broken

He, Sorry for the late response, I ended up copying the HTTPMiddleware

// NOTE: we are copying the HTTPMiddleware adaptor because the request context doesn't get propegated to fiber context
func HTTPMiddleware(mw func(http.Handler) http.Handler) fiber.Handler {
	return func(c *fiber.Ctx) error {
		var next bool
		nextHandler := http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
			next = true
			// Convert again in case request may modify by middleware
			c.Request().Header.SetMethod(r.Method)
			c.Request().SetRequestURI(r.RequestURI)
			c.Request().SetHost(r.Host)
			for key, val := range r.Header {
				for _, v := range val {
					c.Request().Header.Set(key, v)
				}
			}

			reqCtx := r.Context().Value(jwtmiddleware.ContextKey{})
			c.SetUserContext(r.Context())
			c.Locals("user", reqCtx)
			// more code that relates to ad-hoc custom claims)
		})
		_ = adaptor.HTTPHandler(mw(nextHandler))(c)
		if next {
			return c.Next()
		}
		return nil
	}
}


UnicodeEncodeError: 'ascii' codec can't encode characters in position 15-17: ordinal not in range(128)

Spectrify version: 0.4.0
Python version: 2.7
Operating System: CentOS

Description
File "/usr/lib/python2.7/site-packages/spectrify/convert.py", line 70, in convert_redshift_manifest_to_parquet
convert_parallel(manifest, sa_table, out_dir, workers, compression_type)
File "/usr/lib/python2.7/site-packages/spectrify/convert.py", line 84, in convert_parallel
pool.map(_parallel_wrapper, convert_args, chunksize=1)
File "/usr/lib64/python2.7/multiprocessing/pool.py", line 250, in map
return self.map_async(func, iterable, chunksize).get()
File "/usr/lib64/python2.7/multiprocessing/pool.py", line 554, in get
raise self._value
UnicodeEncodeError: 'ascii' codec can't encode characters in position 15-17: ordinal not in range(128)
What I Did
Convert CSV to Parquet file

Hi @naveenarumugam, unicode support is limited to Python 3 right now.  This is because the python csv module in python 2 does not support unicode.  There are third-party solutions for this (e.g. unicodecsv), but I have not integrated one as yet.

Thanks @c-nichols; will try it out...do you've any thoughts on it?

I have limited experience with unicodecsv, but performance would be my main concern.  unicodecsv is written in Python, whereas the Python csv module is written largely in C.  As always, benchmarks would be required to say if it's significant, and I could see it being somewhat data-dependent as well.

@c-nichols  Agree.
About pandas, yes, there are some cases needs to be handled and did not want to re-invent chunk writing again ... where you've handled it already.  Now, the biggest blocker is only about data type mappings...  hopefully, we'll have some updates.  For Unicode, I'll take it case by case for now.


Web - LHN- The workspace name reverts to the default name when you come back online.

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

Action Performed:

Go to URL https://staging.new.expensify.com/
Login with expensifail account.
Create workspace and send at least 3 messages in the #announce room.
Navigate to the #announce room.
5.Disable the internet connection in the main testing device.
Send a message - Verify the message is grayed out.
Edit one of the messages sent before going offline - Verify the message is grayed out.
Send a image to the conversation - Verify there's a grayed out "Uploading attachment.."
Delete one of the previous messages - Verify the message display with a strikethrough.
In the secondary device, navigate to the workspace and delete it.
Enable the internet connection in the device

Expected Result:
The name of the workspace should be saved after coming back online.
Actual Result:
The workspace name reverts to the default name when you come back online.
Workaround:
Unknown
Platforms:

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

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/230515001-4e2f8822-94b7-45e8-aee6-90c9b6b5c9bc.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - internal team
Slack conversation:
View all open jobs on GitHub

Triggered auto assignment to @sonialiap (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

I think this is sort of a dupe of https://github.com/Expensify/App/issues/16090 if this is specifically a bug about the workspace name going back to the old name after you go online.
I'm pretty sure this, https://github.com/Expensify/App/issues/16090, and https://github.com/Expensify/App/issues/16776 all probably have the same root cause for the workspace name not properly updating so I think we should focus on https://github.com/Expensify/App/issues/16090 to fix that.
This and https://github.com/Expensify/App/issues/16776 also deal with deleting the workspace though so there may be another bug in these flows as well that has to deal with deleting the workspace so if we could isolate what that is exactly from the name change not working that'd be great!

@sonialiap Whoops! This issue is 2 days overdue. Let's get this updated quick!

@sonialiap Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!


use config file like in readme

make this part work:
// After that just use it like this:
$client = new PlentymarketsRestClient($configFilePath);
We could add some more error handling on saveConfigFile() if $config is null

Thanks!


It should be possible to send requests with no Accept-Language Header

from: ./src/qt/qtbase/src/network/access/qhttpnetworkconnection.cpp
// some websites mandate an accept-language header and fail
// if it is not sent. This is a problem with the website and
// not with us, but we work around this by setting
// one always.
value = request.headerField("accept-language");
if (value.isEmpty()) {
    QString systemLocale = QLocale::system().name().replace(QChar::fromLatin1('_'),QChar::fromLatin1('-'));
    QString acceptLanguage;
    if (systemLocale == QLatin1String("C"))
        acceptLanguage = QString::fromLatin1("en,*");
    else if (systemLocale.startsWith(QLatin1String("en-")))
        acceptLanguage = QString::fromLatin1("%1,*").arg(systemLocale);
    else
        acceptLanguage = QString::fromLatin1("%1,en,*").arg(systemLocale);
    request.setHeaderField("Accept-Language", acceptLanguage.toLatin1());
}

It seems like this code comes from qt, so maybe it's not your problem really, but at any rate it doesn't seem to be possible to send a request from phantomjs that does not include an Accept-Language header.  Personally I'd much rather see this thing send no header by default and force the user to specify it.  I tried overriding the headers with the empty string and null, and both just resulted in this code pulling the lang from the locale.  I didn't look any further than that (like at what isEmpty actually does, to see if i could trick it in a less obvious way.

Filed https://bugreports.qt.io/browse/QTBUG-45797 to see if Qt is receptive to adding a way to suppress the header.


Perf: Cache cert?

See: https://github.com/GlareDB/glaredb/pull/1767#discussion_r1334873698

I think that we shoud hold off on this until #1842 is in. Once we do that, we can remove some plumbing and pull the cert earlier.
I imagine it will be saved in memory in the first pass, though I'm open to suggestions on writing it to disk for a quick lookup.
Doing this means that we can build a reqwest client higher up the stack too.


Add "CRTSwitchRes" to "Per system advanced configuration"

When using a VGA-2-RGB/Component cable to connect a CRT TV or monitor, enabling CRTSwitchRes is said to be the preferred video output solution because it sets the CRT to the internal resolution of the used core.
Some cores offer the option to add a TV overscan to this internal output resolution (like Mednafen Beetle PSX, Genesis Plus GX and Stella), but most currently don´t. This results in the image to be streched too much horizontally on a real 4:3 CRT TV due to the missing overscan borders, and on some games and console emulations, the image is cut on the left and the right (with having not the "original" display aspect ratio at the same time).
A good example can be found at https://wiki.nesdev.com/w/index.php/Overscan: The real horizontal resolution of the original NES (NTSC) with overscan is 280px while only 256px are used (with 24px of borders added to the left and right).
None of the cores I tested allowed to add this overscan area of 24px or set it themselves with an appropriate option, and thus CRTSwitchRes just switched to the 256px on all cores, not 280px as it should be. This results in the image to be streched 24px wider when output to the CRT TV.
Since some cores allow adding those overscan borders (like mentioned above) while others don´t, I would prefer a per system advance configuration if I want to use CRTSwitchRes or my self-defined resolution instead (e.g. enabling it for PSX, but disabling it for NES where I would "manually" set it to a custom resolution of 280px x 240px for now).

This is not related to emulationstation, but to batocera.linux.
Can you delete this issue & create a new one in the batocera.linux repository.

Thanks for your feedback. I asked for an additional menu entry in the "Per system advanced configuration" in ES because I that it was harcoded.


[feature] Introduce flink jdbc catalog store plugin.

Contribution Checklist

Make sure that the pull request corresponds to a 3913.

What changes were proposed in this pull request
Issue Number: close #3913
Brief change log
Verifying this change

This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
jdbc factory load
*jdbc catalog crud test*
This change added tests and can be verified as follows:

*Added Test to verify the change.
Manually verified the change by testing locally. -->

Does this pull request potentially affect one of the following parts

Dependencies (does it add or upgrade a dependency): (yes)

hi Mrart:
HI, thanks for your contribution. There are a few questions to discuss:

Currently, this PR only implements catalog storage. What we need to do is catalog management. Of course, catalog management includes storage. Do you have a complete plan for the catalog management feature? Can it be divided into subtasks like this PR? In this way, we can better assess risks and know the development progress
The current PR only implements MySQL storage and has JSON type. How do we support databases, such as MySQL, PGSQL, and h2 on the streampark platform? Do you have any relevant plans?

hi Mrart:
HI, thanks for your contribution. There are a few questions to discuss:

Currently, this PR only implements catalog storage. What we need to do is catalog management. Of course, catalog management includes storage. Do you have a complete plan for the catalog management feature? Can it be divided into subtasks like this PR? In this way, we can better assess risks and know the development progress
The current PR only implements MySQL storage and has JSON type. How do we support databases, such as MySQL, PGSQL, and h2 on the streampark platform? Do you have any relevant plans?

1.In the next pr, I'll provide a full catalog backend management implementation. And help coordinate the front-end resources.
2. As mysql I insert json value only use java string，It can supported pg and h2， I have compatibility it by design.

hi:
Why delete .idea/icon.png and idea/vcs.xml?

hi: Why delete .idea/icon.png and idea/vcs.xml?

When moving the module of the project, the load fails, delete the idea directory again and pour it in. Restores.idea/icon.png and idea/vcs.xml


NuGet's restore incorrectly

Not sure which repo to file this, but this seems like a ship-stopper:
https://github.com/Reactive-Extensions/Rx.NET/tree/integration-tests/Rx.NET/Integration
Open that solution there, which I’m using to test the installation of the Rx packages into various platforms. The Win81 package looks wrong. It appears to be choosing the portable-win8+net45+wpa81 instead of the "closer" netstandard1.1 version.
That package has the following folder structure:

The portable- folders are there for backwards compatibility with VS 2013 and older project types. From everything I’ve understood,
netstandard TFM should be higher than any portable- TFM so that even here it should fall back to netstandard1.0 unless there’s a platform-specific TFM (win8, netcore45, etc) here.
This is going to break things badly where we’re trying to light-up as much as possible on newer platforms.
Two things: is this supposed to be correct behavior? Where would this bug/issue lie?
/cc @ericstj @harikm86
Also, separate issue but related:
https://github.com/NuGet/Home/issues/3000

Action: portable- should not be needed with NuGet 2.12 for VS 2012/2013: https://github.com/NuGet/Home/issues/2720


Upgrade to Scala 2.12.18 and 2.13.11

refs: https://github.com/apache/incubator-pekko/issues/344
Not for merge , helping test the new scala release.

refs: https://github.com/scala/scala/pull/10313

Converting to draft until Scala versions are properly released

@He-Pin Just to clarify, all of the additional code changes are necessary for the new versions of Scala 2.12/Scala 2.13?

@mdedetrich Yes, otherwise it won't compile, because the new lint.


fuse: automatically add ro option if mount with --read-only

Codecov Report

Merging #1661 (cef6bf4) into main (1e49f93) will increase coverage by 0.22%.
The diff coverage is 45.45%.

@@            Coverage Diff             @@
##             main    #1661      +/-   ##
==========================================
+ Coverage   61.53%   61.76%   +0.22%
==========================================
  Files         133      133
  Lines       22595    22607      +12
==========================================
+ Hits        13904    13963      +59
+ Misses       7126     7083      -43
+ Partials     1565     1561       -4

Impacted Files
Coverage Δ

cmd/main.go
50.29% <0.00%> (+1.42%)
:arrow_up:

pkg/fuse/fuse.go
77.59% <0.00%> (-0.51%)
:arrow_down:

pkg/utils/utils.go
79.24% <60.00%> (-2.01%)
:arrow_down:

cmd/gateway.go
87.20% <100.00%> (-0.11%)
:arrow_down:

cmd/mount.go
70.58% <100.00%> (+0.49%)
:arrow_up:

pkg/meta/sql.go
67.28% <0.00%> (-0.21%)
:arrow_down:

pkg/meta/tkv_tikv.go
62.23% <0.00%> (+0.19%)
:arrow_up:

cmd/format.go
62.58% <0.00%> (+0.20%)
:arrow_up:

pkg/vfs/backup.go
65.97% <0.00%> (+0.35%)
:arrow_up:

... and 6 more

Continue to review full report at Codecov.

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


NO-JIRA: Update Konflux 4.15 and perform migration

What this PR does / why we need it:
Replaces https://github.com/openshift/hypershift/pull/4480:

cherry picked the commit in https://github.com/openshift/hypershift/pull/4480
Removed BASE_IMAGES_DIGEST per the Konflux migration instructions - https://github.com/konflux-ci/build-definitions/blob/main/task/buildah/0.2/MIGRATION.md

Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/area ci-tooling

/retest

/lgtm


Librewolf + HSTS Cache partitioning

Librewolf shows HSTS cache fails state partitioning. This doesn't seem right
LW has network partitioning on by default. Looking at entries in SiteSecurityServiceState.txt they are (with some principals excepted) all eTLD or eTLD+1 with partitionKey= origin attributes. Where are 3rd party entries kept?
Same in Firefox, and in arkenfox with Firefox. Arkenfox is literally the same as LW (in v96 we move from FPI to dFPI), which is why I am interested in why LW fails
Reading your description and code which I can't quite wrap my head around
I wonder if HTTPS-Only Mode is affecting this result? LW also fails in this test in PB Mode and we know HoM also overrides HTTPS-First in PB mode. Could you run LW with dom.security.https_only_mode reset to false? And/or give any insight?
TIA

AFAIK any HSTS supercookies are rendered useless with HoM

in this example (scroll down to the third picture about half way down) it would return a=Y, b=Y, c=Y, d=Y, e=Y every single time regardless of HSTS

I doubt anyone is going to set HoM exceptions for numerous subresources, and HoM silently fails those anyway
So, if I understand this correctly, the HTTPS cache test should take Insecure website into account, reflect that in the result, and return "passed"?

in this example (scroll down to the third picture about half way down) it would return a=Y, b=Y, c=Y, d=Y, e=Y every single time regardless of HSTS

btw I don't think this is super accurate: the negotiation isn't taking place at all. arthur's test thinks it is, but instead it isn't and it's just being upgraded before even going out. maybe that's what you meant and I misread your comment tho.
rudimental proof after trying to connect to the http version of the url:

thanks @Thorin-Oakenpants for opening this issue which is relevant to me as well (and  sorry to both of you, I previously posted a - now deleted - comment that was meant for private research). I'm under the same impression that this could be a false positive as I tested and connections are always upgraded to https before leaving the browser.
I have a question for @arthuredelstein anyway: if you have the time, could you explain to me how you go about testing isolation of the HSTS cache? if it really turns out to be a false positive could it be worked around by having two subresources, one that asks for the secure connection and the other that doesn't?

Hi @Thorin-Oakenpants and @fxbrit -- you are right. This is a false positive for LibreWolf. I removed the X for now and I'm looking into a fix. Thank you both and sorry for the error.

Rather than create a new issue

Tor Browser has a Desktop + Desktop private mode. Isn't the first redundant?
same with TB Nightly and Nightly private mode
https://privacytests.org/news.html - link for 10 desktop is for current desktop issue
TB Tor test

issue 10: desktop private mode mode shows Tor as failed
issue 10.1: desktop shows Tor as failed

Change in issue 11 - font isolation is now a fail for gecko desktop. Font cache AFAIK is isolated by network partitioning

Rather than create a new issue

Please do create new issues! :) So I can close each issue one by one.

Rather than create a new issue

Please do create new issues! :) So I can close each issue one by one.

I split out #77.
The HSTS issue was fixed in Issue 11.

I split out #77.
The HSTS issue was fixed in Issue 11.