[lit-html] styleMap is not compatible with a strict style-src content security policy

Which package(s) are affected?
Lit Core (lit / lit-html / lit-element / reactive-element)
Description
With a strict content security policy, like:
  <meta http-equiv="Content-Security-Policy" content="style-src 'self'">

the style map directive does not work.
Reproduction
See this playground example:
https://lit.dev/playground/#gist=5b8305f3b56083774104ad858faed869
Workaround
We could potentially write to the style object on the element, something like this: https://lit.dev/playground/#gist=0985f5699785c208bcc837e14e89664a
This isn't a drop in replacement for styleMap because:

doesn't work in SSR, though it could be made to pretty easily
doesn't work properly when re-rendering with a removed property
property names are the DOM names not the CSS names

Is this a regression?
No or unsure. This never worked, or I haven't tried before.
Affected versions
3.1.4
Browser/OS/Node environment
Confirmed the bug repros in Chrome, Safari, and Firefox.

We can fix this pretty easily by not having update() delegate to render() here: https://github.com/lit/lit/blob/ce81de9e8adf2c5c7eece635a54d76decb64ce1d/packages/lit-html/src/directives/style-map.ts#L77
We did this to ensure that render() and update() behave the same, and so that render() then update() works, but we can do that with tests.

Has there been anything new regarding this issue?


Survey link

Prototype testing questionnaire: https://docs.google.com/forms/d/1rrPNT0GtthACdNwFUTbW_QSIb37MXd94SpPcNurK7Rs/edit

will be making another one of these to embed at a later date

duplicate of #48


a_sh_rd_delta_o

constexpr int a_sh_rd_delta_o = 2 * ((threads / 32) / (thread_n_blocks / 4));

Does the 32 here refer to a warp?
What does 4 here mean?
What does 2 here mean?

same question, I can't figure out why a_sh_rd_delta_o has something to do with thread_n_blocks. @efrantar Could you pls help explain that when you are not busy? Many thanks!


UnboundLocalError when trying to view Twitter list

To reproduce do,
twitter mylist list

This is the stacktrace I get,
Traceback (most recent call last):
  File "/usr/bin/twitter", line 11, in <module>
    load_entry_point('twitter==1.18.0', 'console_scripts', 'twitter')()
  File "/usr/lib/python3.6/site-packages/twitter/cmdline.py", line 793, in main
    Action()(twitter, options)
  File "/usr/lib/python3.6/site-packages/twitter/cmdline.py", line 445, in __call__
    doAction()
  File "/usr/lib/python3.6/site-packages/twitter/cmdline.py", line 438, in <lambda>
    doAction = lambda: action(twitter, options)
  File "/usr/lib/python3.6/site-packages/twitter/cmdline.py", line 462, in __call__
    statuses = self.getStatuses(twitter, options)
  File "/usr/lib/python3.6/site-packages/twitter/cmdline.py", line 535, in getStatuses
    return ListsAction.getStatuses(self, twitter, options)
  File "/usr/lib/python3.6/site-packages/twitter/cmdline.py", line 525, in getStatuses
    return list(reversed(twitter.lists.statuses(
UnboundLocalError: local variable 'list' referenced before assignment

I was not able to see what is causing this error as  list is a proper python keyword and it has not been redefined in that scope.

Nice catch
Can you try to change in /usr/lib/python3.6/site-packages/twitter/cmdline.py the variable "list" into "lst" on lines 520 and 522 and see if that solves it ?

I got a different error this time probably due to change in the API. If I just specify the name of the list I get,
details: {'errors': [{'code': 112, 'message': 'You must specify either a list ID or a slug and owner.'}]}

I follow a list for which I am not the owner so it might be triggering this error.

I don't know whether the cli tools have a way to access those. You can easily write python code to do so following the main readme and looking at the api documentation https://developer.twitter.com/en/docs/accounts-and-users/create-manage-lists/api-reference/get-lists-statuses


Fix a source comment

PR description
Correct a source code comment: TRUE_RESPONSE is defined as 1 padded out with zeros to take up 32 bytes.
(This is probably a copy-paste error from the Node controller, which uses 256 bits of 1...)
Signed-off-by: chaals<EMAIL_ADDRESS>

Sigh. DCO checks seem to have an additional requirement, perhaps it is a line of comment stating that I fixed a typo?
Also, it would be good to document somewhere in the workflow that using the button on github.com to merge updates will fail, so the PR needs to be brought up to date with a sign-off locally.
Should I close this PR and make another one, or can you over-ride the DCO fail?

The merge commit is fine, it's the fix typo commit that has an incorrect sign-off (it's missing the <> around the email address: https://github.com/hyperledger/besu/pull/669/commits/169d9ad497d5f088669d3af3698bc966a1289527
The "Details" link for the DCO check always indicates the problematic commit.

OK, let's see if I can get this right next time.


chore: upgrade to godot 4.3

#192
#191  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


Prepare objectives for Aerilynn Nguyen internship

Identify 1-mo, 3-mo and 6-mo objectives.
Validate the objectives with Jake.

$ nx g @angular/material:ng-add --project=test-app

>  NX  Generating @angular/material:ng-add

✔ Choose a prebuilt theme name, or "custom" for a custom theme: · indigo-pink
✔ Set up global Angular Material typography styles? (y/N) · false
✔ Include the Angular animations module? · enabled
UPDATE package.json
UPDATE apps/test-app/src/app/app.module.ts
UPDATE workspace.json
UPDATE apps/test-app/src/index.html
UPDATE apps/test-app/src/styles.scss

Here is what the above command did:

Added this content to the app style styles.scss.
html, body { height: 100%; }
body { margin: 0; font-family: Roboto, "Helvetica Neue", sans-serif; }

Added this content to index.html:
<link rel="preconnect" href="https://fonts.gstatic.com">
<link href="https://fonts.googleapis.com/css2?family=Roboto:wght@300;400;500&display=swap" rel="stylesheet">
<link href="https://fonts.googleapis.com/icon?family=Material+Icons" rel="stylesheet">

Added the BrowserAnimationsModule module to app.module.scss.

Added this material theme css files to project.json (build target).
<EMAIL_ADDRESS>

Aerilynn received an offer from her current internship for a full-time position. She has withdrew her previous acceptance of our offer.


Baseline MAINTAINERS, CODEOWNERS, and external collaborator permissions

Follow https://github.com/opensearch-project/.github/issues/125 to baseline MAINTAINERS, CODEOWNERS, and external collaborator permissions.
Close this issue when:

MAINTAINERS.md has the correct list of project maintainers.
CODEOWNERS exists and has the correct list of aliases.
Repo permissions only contain individual aliases as collaborators with maintain rights, admin, and triage teams.
All other teams are removed from repo permissions.

If this repo's permissions was already baselined, please confirm the above when closing this issue.

@jordarlu

@jordarlu

Closing this issue as part of https://github.com/opensearch-project/opensearch-benchmark/pull/219

Verified permissions, looks good!


Hello,  I am trying to create a new entry to my keypass db. Used the following commnd   $masterKey = ConvertTo-SecureString -AsPlainText "********" -Force  $psw = ConvertTo-SecureString -AsPlainText "****" -Force  New-KeePassEntry -DatabaseProfileName "vCenter-Admin" -KeePassEntryGroupPath "D:\Script\VC-AdminPasswordReset\KeyPass\vCenter-Admin.kdbx"  -Title "VC1" -UserName "Admin1" -KeePassPassword $psw -Notes "VC" -MasterKey $masterKey

            $PassThruResult = New-KeePassEntry -KeePassEntryGroupPath 'PSKeePassTestDatabase' -Title 'testPassThru' -UserName 'testuser' -Notes 'testnotes' -URL 'http://url.test.com' -DatabaseProfileName 'SampleProfile' -PassThru

            $PassThruResult.KPEntry | Should BeOfType KeePassLib.PwEntry
            $PassThruResult.KPEntry.ParentGroup.Name | Should BeLike 'PSKeePassTestDatabase'
            $PassThruResult.KPEntry.Strings.ReadSafe('Title') | Should Be 'testPassThru'
            $PassThruResult.KPEntry.Strings.ReadSafe('UserName') | Should Be 'testuser'
            $PassThruResult.KPEntry.Strings.ReadSafe('Notes') | Should Be 'testnotes'
            $PassThruResult.KPEntry.Strings.ReadSafe('URL') | Should be 'http://url.test.com'
            $PassThruResult.Expires | Should be $false

KPScript -c:AddEntry "C:\Work\KeePass\New folder\vCenter-Admin.kdbx" -pw:"C+e=31%gefGY;2m" -Title:"123" -UserName:"LocalAdmin" -Password:"pao5j3eg" -URL:https://example.com/
Hello,
I am trying to create a new entry to my keypass db. Used the following commnd
$masterKey = ConvertTo-SecureString -AsPlainText "********" -Force
$psw = ConvertTo-SecureString -AsPlainText "****" -Force
New-KeePassEntry -DatabaseProfileName "vCenter-Admin" -KeePassEntryGroupPath "D:\Script\VC-AdminPasswordReset\KeyPass\vCenter-Admin.kdbx"  -Title "VC1" -UserName "Admin1" -KeePassPassword $psw -Notes "VC" -MasterKey $masterKey
I am getting the following error:
InvalidKeePassConfiguration : No KeePass Configuration has been created.
At C:\Windows\system32\WindowsPowerShell\v1.0\Modules\PoShKeePass\<IP_ADDRESS>\PoShKeePass.psm1:207 char:17

...             throw 'InvalidKeePassConfiguration : No KeePass Configura ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : OperationStopped: (InvalidKeePassC...s been created.:String) [], RuntimeException
FullyQualifiedErrorId : InvalidKeePassConfiguration : No KeePass Configuration has been created.

Not sure if I am missing something. Could you please help me on this?
Thanks and Regards,
Dijo Jose.

Hello,
I am trying to create a new entry to my keypass db. Used the following commnd
$masterKey = ConvertTo-SecureString -AsPlainText "********" -Force
$psw = ConvertTo-SecureString -AsPlainText "****" -Force
New-KeePassEntry -DatabaseProfileName "vCenter-Admin" -KeePassEntryGroupPath "D:\Script\VC-AdminPasswordReset\KeyPass\vCenter-Admin.kdbx"  -Title "VC1" -UserName "Admin1" -KeePassPassword $psw -Notes "VC" -MasterKey $masterKey
I am getting the following error:
InvalidKeePassConfiguration : No KeePass Configuration has been created.
At C:\Windows\system32\WindowsPowerShell\v1.0\Modules\PoShKeePass\<IP_ADDRESS>\PoShKeePass.psm1:207 char:17

...             throw 'InvalidKeePassConfiguration : No KeePass Configura ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : OperationStopped: (InvalidKeePassC...s been created.:String) [], RuntimeException
FullyQualifiedErrorId : InvalidKeePassConfiguration : No KeePass Configuration has been created.

Not sure if I am missing something. Could you please help me on this?
Thanks and Regards,
Dijo Jose.


Source code issue while running qmake -spec macx-clang -Wall ../src/tableau-log-viewer.pro

While running qmake -spec macx-clang -Wall ../src/tableau-log-viewer.pro on mac terminal getting 29 warnings and 12 errors generated
Untitled document.docx
I have uploaded the errors I am receiving,. Can you please help me on this ?

It's not safe to open a .docx file. If you want someone to look at this, please replace the the .docx file with a text file.


Which network strucetures are supported by  lottery_ticket_pruner?

Can  the project prune neural network with any structure?
Or it only supports specific structure network, such as CNN.

This project supports pruning of arbitrary neural networks that have any of these layers:
'Conv1D',
'Conv2D',
'Conv2DTranspose',
'Conv3D',
'Conv3DTranspose',
'Convolution1D',
'Convolution2D',
'Convolution2DTranspose',
'Convolution3D',
'Convolution3DTranspose',
'Dense',
'DepthwiseConv2D',
'SeparableConv1D',
'SeparableConv2D',
'SeparableConvolution1D',
'SeparableConvolution2D'

Thanks for your prompt response！


Abandoned?

@alinz Would be nice to know if you're just busy, or if you're done maintaining this -- for the sake of knowing whether we should invest time building on top of this.

@alinz I second @ivanakimov.

@ivanakimov very busy at the moment, I have done some major refactor but it's not nearly ready for production. My plan was to put it into the core but things keep coming up. Please bear with me or fork it for now until I finish all my stuff.

@alinz Fair enough. Bearing in progress ;)

Is there an estimate for the baring time ^^;

Closing, since @hmedney is kicking ass and taking names :muscle: :smirk:

@ivanakimov could you explain what @hmedney is doing?


Disable Metadata

Disable metadata embedding on generated images.
This will allow users to protect their workflows.

Oh nice, I didn't know this was a thing.


Module not found: Error: Can't resolve 'ngx-take-until-destroy'

After running npm i @jdrks/ngx-deep-linking on an Angular 12 project, I get the following error:
./node_modules/@jdrks/ngx-deep-linking/fesm2015/jdrks-ngx-deep-linking.js:6:0-56 - Error: Module not found: Error: Can't resolve 'ngx-take-until-destroy' in '{path}/node_modules/@jdrks/ngx-deep-linking/fesm2015'

It seems that take-until-destroy is deprecated and should be replaced with until-destroy

Hi @shezadkhan137 ,
thanks for trying out my library.
ngx-take-until-destroy is a "peer-dependency" of this library. That means you have to install it in your project yourself. Npm should also have pointed this out to you in the install logs.
Regarding the deprecation of ngx-take-until-destroy: I wanted to ensure downward compatibility for Angular9 with this, but as I build the whole library on Angular 12 this was a failed attempt.
In future I will use Angular 9 in the library so it can easily be used with older version of Angular too. Also I will most likely remove ngx-take-until-destroy completely to have no further dependencies.


Do not acces network for building

On NixOS when running cmake there is no network access.  I did not find a way to download the files before, and then give cmake a path to the downloaded files, to make it work under NixOS.

On NixOS when running cmake there is no network access.

This restriction exists at least for Fedora / Centos too. You can't access internet while building pacakages.

FWIW Arch has the same rule, although it isn't enforced for now because of projects like this


please pass request param to django.contrib.auth.authenticate

for some reason i want to use "request" in AUTHENTICATION_BACKENDS
but i got request as None
i need request because:

shopping cart is in cookie before login
shoping cart is in server after login
i want to decorate anywhere when login ( logon 、login 、oauth2)
in my decoration : i get user from request
i write a new AUTHENTICATION_BACKENDS give User to request as user property
no request no user

just change you serializers line 50
from
user = authenticate(**credentials)
to
user = authenticate(self.context["request"] , **credentials)
thanks

@jpadilla If i create a PR, will this be merged?

No, most likely not. Project hasn't been updated for 2 years. Use https://github.com/davesque/django-rest-framework-simplejwt/ instead


Shortcut doesn't work after reinstalling GW2 Addon Manager

The default shortcut doesn't work for me (shift+alt+m).
It used to work perfectly and so did the plugin but ever since GW2 DX11 rolled out, I had to remove my Plugin Manager and uninstall all addons.
Now when updates have been done, I want to use it again. For some reason it doesn't work but the DX12proxy plugin does so not sure what the issue could be.
Any pointers on how to make it work again? Addon has been installed in Addon folder where GW2 is.

Please provide the information the bug report template indicates, namely which version of the addon you're using, how you've installed it (and versions for any dependencies).

Please provide the information the bug report template indicates, namely which version of the addon you're using, how you've installed it (and versions for any dependencies).

No idea which version it is, installed it via Unofficial Addon Manager (v1.4) and updated there to latest version they offered.
Any clue how I can find the version when installing through addon manager?
FYI: I got issues with Addon Manager too, can't open it (403 remote server forbidden) so I assume I opened it too many times while trying to troubleshoot or something...

GW2Radial is still in pre-release, you will need to download the latest version manually.

Hey,
I have checked for you and I got GW2Radial 1.2.4al installed
Got the d3d9.dll in my bin64 directory as well but shortcut key combination is not calling it in game.
So I was wondering if I have changed the key combination to call the interface ingame? Anyway I can trace it back in of the files of GW2Radial? Maybe that's the issue?
It worked perfectly well until GW2 dropped the DX11 beta and had to remove every single add on to make the game work again. So now after 1 week I want to install it again but so far GW2Radial is the only one I cannot call ingame

GW2Radial is still in pre-release, you will need to download the latest version manually.

Thanks for coming back to me but Im confused. You say I need to download the newest version manually but on that same page it also states and I quote:
Addon loader support. It is recommended to use the GW2 Addon Manager to handle this easily. This is now the recommended method to install GW2Radial.
Thats exactly what I did and said.
Anyway I downloaded the latest version and copied it over to the addons folder and my game simply won't run anymore..

To add: the readme also states installation via GW2 Add On Manager.
It says to copy over the .dll file to bin64 but that file is not in .zip folder  you provided the download for so perhaps this might be the issue? I still have a d3d9.dll file in there but perhaps its from previous version?

Ye despite removing GW2radial, my game doesnt run anymore completely. The client refuses to run after clicking Start Game..
Help is needed

@Oliviervdv1990 If you're still having issues launching the game, i would suggest repairing your client (https://help.guildwars2.com/hc/en-us/articles/201863008-Repairing-the-Game-Client)
I am having the same issue. I have tried installing it manually, by placing the dll (renamed to d3d9.dll) into bin64, but this way as through the manager, there is no evidence that the addon is loaded at all.
I have tried by completely uninstalling all addons and only installing that one through the new manager (1.4.2), and tried manually. Neither work.

Please double/triple check that you are not running the DX11 beta.

Attempted one last time, making sure to go into the game, disabling DX11, exiting, rerunning to make sure it was in DX9, quit again, removed all addons, reinstalled through manager, and it now works!
Thank you!

@Oliviervdv1990 If you're still having issues launching the game, i would suggest repairing your client (https://help.guildwars2.com/hc/en-us/articles/201863008-Repairing-the-Game-Client)
I am having the same issue. I have tried installing it manually, by placing the dll (renamed to d3d9.dll) into bin64, but this way as through the manager, there is no evidence that the addon is loaded at all. I have tried by completely uninstalling all addons and only installing that one through the new manager (1.4.2), and tried manually. Neither work.

I fixed my client by removing all plugins and Add On Manager.
I reinstalled everything but GW2 radial only one not working. Not sure if its because keyboard shortcut doesn’t work or the add on.  Oh well.


🐛 Biome not detected in dependencies

VS Code version
1.91.1
Extension version
2.3.0
Biome version
1.8.3
Operating system

[ ] Windows
[X] macOS
[ ] Linux

Description
I see this popup, and don't get any linting from Biome, even though I have Biome in my project's devDependencies in package.json

Steps to reproduce

Add "@biomejs/biome": "1.8.3", to devDependencies
Install the Biome extension
Reload the window

Expected behavior
Biome should detect that it's installed from package.json
Does this issue occur when using the CLI directly?
Yes
Logs
No response

Thank you @tylerlaprade for opening the issue. Would you mind sharing a reproduction?

@ematipico, once I clicked Download Biome, I'm not getting it anymore, but that is just a band-aid fix. I tried deleting all "Biome" files on my computer in order to repro, but it's still globally installed somewhere. (Not on the command line, so I can't which biome).
To repro, any node project should suffice with Biome in devDependencies.

Sorry @tylerlaprade, but I use Biome myself in other Node.js. projects, and I don't experience this issue, it's possible that it's something related to your project or your machine.

Sure, makes sense. In that case, what's the best way for me to uninstall Biome from wherever that button put it? I can't find it in file search.

@tylerlaprade  There's a command palette for that Biome: clear versions cache

@ematipico, I still have this "bundled" thing showing after clearing versions cache. My teammate is also getting the constant popup warning as well.

Ah, this might have been poor communication on my part. This command does not actually clear the downloaded versions, but we cache the request that retrieves the list of available versions from GitHub to avoid hitting the rate limit. This command is used to reset this cache.
To clear this "bundled" version, you need to delete the following folder manually:
~/Library/Application Support/Code/User/globalStorage/biomejs.biome/server

We'll make sure to add a command that does this in the next version.

I'm closing this as I believe the latest nightly fixes this issue. Let me know if it's not the case.

Thanks @nhedger. I assume you meant pre-release, as "nightly" is deprecated.

After switching to pre-release, I have a different popup, but I'm still missing Biome. I expect to have it because it's in my package.json and I've run npm i. Could the problem be that it's a monorepo and package.json isn't in the root folder?

@nhedger, hmm, that's still not working, both with and without a slash at the end of the path. I'm in a monorepo, but with only one relevant project - the back-end is in Python. So I have biome.jsonc in my workspace root, but package.json and node_modules/ are in the subfolder. I confirmed that node_modules/ contains @biomejs.

Would you mind providing a minimal reproduction repo? I'll have a look!

Sure! I was able to repro without a monorepo.
https://github.com/tylerlaprade/biome_bug
Steps:

Open this project in a fresh VSCode install
Run npm i to install Biome
Developer: Reload Window to see the error trigger

Sure, how's this: https://www.loom.com/share/708aae2faab4444b8a31caf6e0931ba3?sid=833839d4-f0d1-49f3-a8e7-6667ac0782f4

I think I found the issue.
When the extension boots, it creates a global LSP session and a project-specific LSP session. I believe the popup you see is triggered by the global LSP session starting because a global Biome cannot be found on your system. When you install Biome in your node_modules and restart VS Code, it again attempts to start the global session. When you dismiss the notification, it actually boots the project-specific LSP session (you can see Biome 1.8.3) in your status bar.
I initially intended for users to download Biome on the fly using the popup so that they could start the global LSP session. However, I'm realizing now that the intent may not be obvious.
In the meantime, clicking Yes in the popup to download Biome should solve the issue. This will download a biome binary used for the global LSP session and shouldn't prompt you anymore.
I'll have to think of a better way to signal users why the see this popup.

@nhedger, even though I see "Biome 1.8.3" in the status bar, the extension doesn't seem to be able to make use of it. I don't get formatting on save or any errors/warning squiggles in my code
Regarding clicking Yes for the global download, I don't want my local environment to ever use a different version that what I have in CI. That would cause me to not realize my PR will fail the build until I submit it.

To be clear, the downloaded version would only be used with the global session which handles files that are not part of your project such as Untitled (unsaved) files.

Ah okay - in that case it seems like there's still a bug? The project one is not working on files that are part of my project.

@nhedger just for clarity sake, can this be re-opened as it was closed due to belief it was fixed in the pre-release, which doesn't appear to be true?

@nhedger just for clarity sake, can this be re-opened as it was closed due to belief it was fixed in the pre-release, which doesn't appear to be true?

Would you mind creating a new issue with your own reproduction side, please?

FWIW @321ckatz123 works with me so we both have the same repro :)

@nhedger, I added the following line to my settings.json and now I get the original (but still buggy) behavior I first reported.
"biome.projects": [{"path": "paqarina"}]
I now have the correct Biome version displayed in the bottom right, but I am still unable to format or lint any files in VSCode, regardless of whether they're in the project or not. I also get this popup for every single file I open (even though those files are in the paqarina/ folder), which is quite noisy.

Happy to try and reproduce it if you can provide a minimal reproduction repo with these settings.

Sure thing - here it is @nhedger
https://github.com/tylerlaprade/biome_bug


reutrn initially planned "back" url/icon for test info view

expand any test run
click onto the eye icon to open new test info view
-> need easy way to return from test info into the current test run

fixed


Move wire fuselage::projection into common functions

The fuselage contains the following code
https://github.com/DLR-SC/tigl/blob/5bbb3da215cb35a1d472c86ae78653f06a11a0e6/src/fuselage/CCPACSFuselage.h#L125-L126
that should be moved to a more generic location

The functions moved to ITiglFuselageStructureInterface in https://github.com/DLR-SC/tigl/commit/8a5931e84f16df4a254e860f349974aca1de6b09


Firefox 107: COLRv1

https://hg.mozilla.org/integration/autoland/rev/b3b91aca9d8b
https://bugzilla.mozilla.org/show_bug.cgi?id=1791558#c4

Closed: 4 hours ago
status-firefox107: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → 107 Branch

Not in nightly yet, but surely the next one :)
https://tests.caniuse.com/colr-v1:

Thanks!


Python 3.9 image has symlinks point at Python 3.11

I've recently noticed that the newest changes introduced on python:3.9 are breaking the /usr/bin/python3 symlink pointing it to python3.11 instead of python3.9
A python:3.9 image freshly pulled from DockerHub:

A python:3.9 image I had on another machine pulled some time ago:

Looking at DockerHub it seems that this layer is introducing Python 3.11:

Tell me if there's anything else I would need to add to this issue.

This is the same as: https://github.com/docker-library/python/issues/174 https://github.com/docker-library/python/issues/345 https://github.com/docker-library/python/issues/486, but in this case the one from Debian is newer than the one we add to /usr/local/
See also the blurb on the Docker Hub docs: https://github.com/docker-library/docs/tree/883d8e242e5cdb1a95c6d530d49b76631a6af030/python#multiple-python-versions-in-the-image

To put that another way, if you've been using /usr/bin/python3, you might as well switch to using raw buildpack-deps or even FROM debian because you haven't been running the Python build provided by this image. :sweat_smile:

I see 🤔 Using python:3.9-slim I'm still getting Python 3.11, is this intentional? Per https://github.com/docker-library/docs/tree/883d8e242e5cdb1a95c6d530d49b76631a6af030/python#multiple-python-versions-in-the-image I assumed it shouldn't happen.

Using python:3.9-slim (or any of the python images), you need to use /usr/local/bin/python3, /usr/bin/env python3 or just plain python3 if you want the Python we compile in the image. If you use the full path to the Debian python, then you will get the Debian python version. We don't replace/override that because it could break system packages.
If /usr/bin/python3 (Debian python) exists in the slim image, then it means you installed a Debian package that required it.
$ docker run -it --rm python:3.9-slim bash
Unable to find image 'python:3.9-slim' locally
3.9-slim: Pulling from library/python
5b5fe70539cd: Already exists
f4b0e4004dc0: Pull complete
ec1650096fae: Pull complete
2ee3c5a347ae: Pull complete
d854e82593a7: Pull complete
Digest: sha256:0074c6241f2ff175532c72fb0fb37264e8a1ac68f9790f9ee6da7e9fdfb67a0e
Status: Downloaded newer image for python:3.9-slim
root@dbad579b3cc2:/# /usr/local/bin/python3 --version
Python 3.9.17
root@dbad579b3cc2:/# /usr/bin/env python3 --version
Python 3.9.17
root@dbad579b3cc2:/# python3 --version
Python 3.9.17
root@dbad579b3cc2:/# /usr/bin/python3
bash: /usr/bin/python3: No such file or directory

That makes sense to me. Appreciate the time you took to explain it! I'll then close this as it doesn't make sense to keep it open.

While re-reading our description with the fresh eyes you've provided (especially the Debian Python now in the rare case of being newer than many of ours), I've put up https://github.com/docker-library/docs/pull/2344 to hopefully clarify this better. :bow: :heart:

That's more clear now! Thank you for the time you took to explain it and for the attention! 🙇🏽♂


[FEATURE] auto generated code coverage reports

Is your feature request related to a problem? Please describe.

Add a CI to calculate code coverage and generate coverage reports, so we can find which part of the runtime is not test enough.
Describe the solution you'd like

Maybe we can try
https://about.codecov.io/

@yunwei37 almost all Projects rely on Codecov, I have some experience with CI, I would like to work on this. I will try to raise draft PR soon.

Thanks! That's great!

Hey @yunwei37 , for this someone from the organization can link bpftime with codecov, I was trying but since I am not part of the organization, I can't do it. I can pick it up if you include me in the organization.

Hi @hp77-creator , would you like to be part of the organization? We can add you there

Hi @hp77-creator , would you like to be part of the organization? We can add you there

Sure!

Hi @hp77-creator , any progress on this issue?

Hi @yunwei37
Please check your dm. There is one access token that needs to be added in project's settings. I do not have access to do that.


Adding transforms does not update Bounds property

Maybe I am confused about how the library works, but I would assume that after a transform has been applied to an element, the Bounds property would be updated? In the following code the Bounds property does not get updated.
var circle = SvgDocument.FromSvg<SvgDocument>("<svg height=\"100\" width=\"100\">  <circle cx=\"50\" cy=\"50\" r=\"40\" stroke=\"black\" stroke-width=\"3\" fill=\"red\" /></svg>");
// Bounds: X=8.5 Y=8.5 Width=83 Height=83
circle.Transforms.Add(new SvgTranslate(50, 50));
// Bounds still: X=8.5 Y=8.5 Width=83 Height=83

Am I totally missing something here or is this intended functionality?

i think you need to add the transform to the actual circle, not the whole svg document...

@jcvandan - is this still an issue, or can this be closed?

@jcvandan - is this still an issue, or can this be closed?

Could still be an issue, but I have no idea. T'was a long time ago I created this.

Yeah, cleaning up old issues - thanks!


⚠ Bofrost.at has degraded performance

In ae474f1, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2040 ms

Resolved: Bofrost.at performance has improved in 829d5e5.


Remove isolate support from flutter_deps recipe module

The flutter_deps recipe module includes a flutter_engine call which downloads a local_engine build from storage.
I have added support for rbe-cas storage (cas, content-addressed-storage), which is replacing the LUCI isolate servers.
The support for downloading from the LUCI isolate servers, starting at line 29 of
[recipes repo]/recipe_modules/flutter_deps/api.py should be removed, leaving only the cas support, once the usage of the isolate servers for download by the Web Engine builders is migrated to cas.

This has already been done.


VanDam to ChildRecords tutorial

https://github.com/LAAC-LSCP/ChildRecordsData/blob/docs-improvements/docs/TUTORIAL_VANDAM.md

I was looking over the daylong vandam and realized that, actually, there are three types of annotation: .its, .rttm (generated by vtc over the course of corpus adaptation), and human, inside .cha file.
.cha is a very standard file in language acquisition, I think there could be packages we can rely on:

https://github.com/SeedlingsBabylab/pyclan
https://github.com/SeedlingsBabylab/parse_clan2
https://jacksonllee.com/papers/lee-etal-2016-pylangacq.pdf

This will probably increase interest, and would make this dataset also a good example for ChildRecordsR @alsonicr !


Rework redirect logic in QS UI

Remove all the inline logic related to IsPkceClient and simplify it.

done (for now)


Fix have_enqueued_mail for Rails 7 and argument matching

This is a combo of #2516 and #2537, supposed to fix #2351 and #2531 with a slight fix for argument matching and a small fix for our system spec spec on Rails 7.

@JonRowe Rails 7 specs are green. This is a good first step to support it.

Thanks @pirj for your work.
Is there an ETA for this PR to be merged and a release version?

Is there an ETA for this PR to be merged and a release version?

RSpec is a community driven project (e.g. volunteers in there free time only) and doesn't commit to ETAs

I intend to merge this on green. In any case, RSpec is unusable with Rails 7 without this.
Checked using my current project (Rails 6.0, previously rspec-rails 4.0.2) - everything seems to be intact.
I'll keep the branch not removed not to break it for those who gem 'rspec-rails', github: 'rspec/rspec-rails', branch: 'combo-have_enqueued_mail-fixes'.

Thank you! Will there be a tagged release to go with this or should we just use main in the meantime?

We're working on an undetermined number of failures now. The release will come when our test suite will pass with Rails 7. Using main is the only safe option now.

I was holding off avoiding merging these @pirj, but sorry I didn't communicate that with you, because it makes it easier to do our release steps.

Merge any outstanding 6.x patches we can, release 5.0.3/
Release 5.1.0 with the few tweaks for rails 6.1
Release 6.0.0 supporting Rails 7.

This is because major version changes are required for new rails versions for us under our release policy.

I was suspecting this. My apologies for rushing.
Can I help with the release somehow, @JonRowe ?

See #2560 @pirj

If you happen to track main in your Gemfile and the support for Rails 7 breaks while we're working on properly releasing a version of rspec-rails for it in #2560, please use combo-have_enqueued_mail-fixes branch in your Gemfile instead:
gem 'rspec-rails`, github: 'rspec/rspec-rails', branch: 'combo-have_enqueued_mail-fixes'

This temp patch fixes issue, im on rspec-rails (5.0.2) and works great:
https://github.com/rspec/rspec-rails/issues/2531#issuecomment-972836152

You also have to specify the rest of RSpec gems to be fetched from GitHub like here https://github.com/rspec/rspec-rails/blob/main/Gemfile-rspec-dependencies#L7


error message: "error creating project - plot shp file failed to load."

Hi, i tried creating a project and it is not working, it says "error creating project - plot shp file failed to load.".

tried on different computers
different accounts (other collaborators tested it)
different sets of points which should be working - tested beforehand

We created a project earlier this week and it was fine, i suppose its a server issue ? anyway posting the info below.

Durango_Mexico_n490_stratifiedMap.zip
Durango_Mexico_n490_stratifiedMap_SubPlots.zip

Hi Ricardo,
I am currently tackling this issue, as soon as I have a solution in place I'll let you know.
Thank you


Color using assembly graph connection

Just an idea. It would be nice if scaffolds/contigs could be annotated by if they are connected or not in the assembly graph.
If the assembler outputs a .gfa, gfatools can be used to pull out subgraphs (in example below, effectively the whole subgraph attached to the given ${EDGEID} would be pulled out into an independent file):
./gfatools view -l ${EDGEID} -r<PHONE_NUMBER>99 test/MT.gfa > sub.gfa

Some assemblers (SPAdes for example), use the scaffold/contig ID as the edge IDs in the GFA, so this type of integration wouldn't require too much of the user (just submit a .gfa that corresponds to the genome fasta file)

Interesting idea. I can see that this would be useful and should be relatively straightforward to implement.
Thanks for the suggestion.

You probably realized this, but just to be clear since I was recently working with a SPAdes assembly graph & realized I was wrong: Actually, the nodes in the graph have different IDs from the scaffold.fasta file, but the scaffold.paths file makes the link between node IDs in the GFA & record IDs in the FASTA.


Segmentation fault when rendering

Hi,
I encountered a segmentation fault when trying to render the bunny.xml in src/optixRenderer/cbox. My environment is ubuntu 20.04, CUDA 10.2, optix 5.11.0. The error message is listed as follows:
Input file name: /home/zod/OptixRenderer/src/optixRenderer/cbox/bunny.xml
Output file name: /home/zod/OptixRenderer/src/optixRenderer/cbox/bunny.rgbe
Material num: 5
Shape num: 8
Vertex num: 72
Camera Parameters:
Camera Field of View: 60
Width: 512
Height: 512
FovAxis: x
Target: 278 273 -799
Origin: 278 273 -800
Up: 0 1 0
Sample Count: 8192
Sampler: independent
Sampler Count: 8192
No Environmental maps.
No point light source.
GPU Num: 1
The length of diagonal of bouncding box: 960.74
Max X: 556.000 Max Y: 548.800 Max Z: 559.200
Min X: 0.000 Min Y: 0.000 Min Z: 0.000
Total number of materials: 5
段错误 (核心已转储)
No output file is generated. I tried all examples in cbox, all encountering the same problem. Could you please help me with it?

That's very weird. I suggest trying different rendering modes, to render diffuse albedo and normal and see what's going on.

I have tried all other modes, all reporting the same segment fault problem.

hello, what's about your GPU volumn?

I encountered a similar problem, changing CMAKE_BUILD_TYPE from Release to Debug can solve this. I have no idea of how this happens. I  guess it is caused by the mismatch of cudadriver and deprecated OptiX6 library. I once had a  Release build working fine, but got Segmentation fault error after I upgraded my nvidia driver from 470 to 510 and linux headers from 5.4 to 5.10. The error persists even though I downgraded them.

Hi, i'm benefit from you r talking. and i have a problem, Are you using CUDA 10.2, optix 5.11.0 on RTX3090 ? I can't use the GPU properly when using the provided docker

RTX3090 do not support cuda10

thank you very much [jiangjiechu],it takes well


Error in android

I have tried to implement Google sign in with this module. But in android, it doesn't work.
Same on emulator and real device.
Error is as follows:
There was a problem communicating with google servers.
I have added sha1 key(debug and release key) in firebase.
Anyone else who can figure out this?

hi there! this issue tracker is reserved for bugs only. your post seems to be a usage question which is better suited for stackoverflow or discord. see https://reactnavigation.org/help.html for more information on how to get help. thanks!


Docker networking and accessories

When having an accessory (like redis or mysql), I'd like to access that from within my app using its docker name (like "my-app-redis"), similar to how it works with docker-compose.
This doesn't work right now as the service and the accessory are both using the default docker network and this kind of hostname resolution/dns only seem to work for user-defined networks. Creating a docker network and using that for services and accessories works, but then traefik won't be able to route traffic to the service without further configuration or connecting traefik to the same network.
Is the idea of creating a shared default docker network for traefik, the service and accessories something we want to pursue in the gem or is this something the user should take care of? A simple network: <network> deploy.yml option for service, traefik and accessories would suffice to generate the required --network docker args I guess.

@tbuehlmann I think I had a similar issue to yours. I try to run mrsk on Synology NAS that has docker installed. Next to web service, I have docker with postgres and redis on the same host. In order to connect web service with the database and Redis one has to add --add-host=host.docker.internal:host-gateway to docker run command to let the container access the host.
If you look at the current source code you will notice guys take *role.option_args to add to docker run command. So configuration like this:
servers:
  web:
    hosts:
      - <IP_ADDRESS>
    options:
      "add-host": host.docker.internal:host-gateway

does the trick. If you run docker with --add-host the host.docker.internal will resolve to the container's host IP. From there you can already access the database, Redis, and any other service that is run on the host.
One remark! Currently, mrsk version 0.9.1 is available in RubyGems (gem install mrsk gives you that version). However, this version doesn't have *web.option_args added to the healthcheck's docker run command. It is only available in master branch, so make sure you use the master version. Otherwise, the mrsk deploy will still fail.

Right, I thought a networking was a thing with mrsk, but it wasn't. Right now I just use the host's IP address from inside the container and that works well.

Another option here is to create the docker network yourself by sshing into the machine, and then specify the network name in your deploy.yml. For example:
servers:
  web:
    hosts:
      - <IP_ADDRESS>
    options:
      "network": "my-network"

traefik:
  options:
    network: "my-network"

accessories:
  db:
    image: postgres:14
    host: <IP_ADDRESS>
    ... etc
    options:
      network: "my-network"

@tbuehlmann I think I had a similar issue to yours. I try to run mrsk on Synology NAS that has docker installed. Next to web service, I have docker with postgres and redis on the same host. In order to connect web service with the database and Redis one has to add --add-host=host.docker.internal:host-gateway to docker run command to let the container access the host. If you look at the current source code you will notice guys take *role.option_args to add to docker run command. So configuration like this:
servers:
  web:
    hosts:
      - <IP_ADDRESS>
    options:
      "add-host": host.docker.internal:host-gateway

does the trick. If you run docker with --add-host the host.docker.internal will resolve to the container's host IP. From there you can already access the database, Redis, and any other service that is run on the host.
One remark! Currently, mrsk version 0.9.1 is available in RubyGems (gem install mrsk gives you that version). However, this version doesn't have *web.option_args added to the healthcheck's docker run command. It is only available in master branch, so make sure you use the master version. Otherwise, the mrsk deploy will still fail.

This is a very old thread at this point but did you need to do anything special to deploy to your Synology NAS? I've been trying over and over and in spite of absolutely having docker installed and accessible both for root and an admin user, it keeps failing to detect docker (docker -v fails so it tries to reinstall it).


LinkedQL Schema

This PR implements OWL schema generation for LinkedQL to be used for code generation in client libraries. The only missing part is including comments for docs and testing properly.

This change is

I've created a repository demonstrating how the schema will be used in client libraries: https://github.com/cayleygraph/python-client

Currently missing: unify properties (use owl:unionOf for domain and assuring range)

Also: the ability to easily consume the produced schema file from other repositories.

Implemented owl:unionOf

The vision is: creating a JSON-LD OWL document that can be fetched by client libraries at build time and be used to generate the objects where the steps structure are translated to methods on Path object. The python client example demonstrates that by creating a Path class. This approach will work for JavaScript as well. For Go, it will be similar but with methods for Path struct.
The document specifies the structures, their properties (which are translated to arguments for the methods), and the types those properties accept (e.g. single/multiple values, and the specific type of values).

Should use Description for adding a comment property to the schema


Playtime UIs in lobby should reflect role bans as well

Right now it just silently doesn't let you pick the role if you're role banned but ideally it would also give a message alongside playtime tracking.

iirc, there was a pr for that


Use InMemoryColumnChunkReader (~20% faster)

Which issue does this PR close?
Part of #1163
Rationale for this change
https://github.com/apache/arrow-rs/pull/1810 added the InMemoryColumnChunkReader, but I stupidly neglected to actually hook it up :facepalm:
What changes are included in this PR?
Alters ParquetRecordBatchStream to use the InMemoryColumnChunkReader to avoid an unnecessary copy. For large column chunks, this represents a significant saving.
Are there any user-facing changes?
No

Codecov Report

Merging #1956 (9edb297) into master (464e8d1) will increase coverage by 0.00%.
The diff coverage is 0.00%.

:exclamation: Current head 9edb297 differs from pull request most recent head 7e308a4.

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

@tustvold does this means we not support SerializedPageReader any more?

SerializedPageReader is still used by the sync APIs, i.e. SerializedFileReader, etc...


test: explicitly specify encoding for subprocess

Ensure that we explicitly use the UTF-8 encoding for the subprocess
rather than ASCII which would be unable to encode extended characters.
This trips up incrparse/Simple.swift.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

CC: @tbkka @drexin @tachoknight

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - 1dc23b8957f1d11593e5a09ff27c5a88173c182b

Build failed
Swift Test Linux Platform
Git Sha - 1dc23b8957f1d11593e5a09ff27c5a88173c182b


Fix bug that generated toc links weirdly

Hi, I came across your site emile.space and really liked it and wanted to create a similar site of my own, so I dug deep and found your site generate vokobe and tried generating your website locally, But it had a weird bug, the links that were being created from toc were also appending new line character in the text like this

So i looked in the rust file and made some changes now it works like the way it is supposed to

idk if this was a bug or it was just behaving differently on my machine only.

Hey, thanks for the PR! I'm probably going to rewrite vokobe some time this year, as I'm not happy with the overall codebase (which was more a prototype tbh.), but I'm happy that people are starting to use it.


Build failed: Push English strings to extension localization repository

Build: https://dev.azure.com/monacotools/a6d41577-0fa3-498e-af22-257312ff0545/_build/results?buildId=302954
Changes: https://github.com/microsoft/vscode-copilot/compare/fa99c8f...fa0bfb3

Duplicate of https://github.com/microsoft/vscode-extensions-loc/issues/1168


Add support to iOS

Add the same operations available on Android to iOS

any plan for iOS @Perdiga @sonikro ?

@nhancv,  we don't have access to a Macbook machine right now, so we are searching for someone willing to contribute on our project.

@sonikro I have sent a PR for this. Could you please review and merge it.

@sonikro I have sent a PR for this. Could you please review and merge it.

@chaitanya-bhagavan , great ! Thanks for contributing. I'll borrow my girlfriends's Macbook to test it out

@chaitanya-bhagavan , great ! Thanks for contributing. I'll borrow my girlfriends's Macbook to test it out

@sonikro Did you get a chance to review the code ?

@sonikro Did you get a chance to review the code ?

Hello @chaitanya-bhagavan,. I'll approve the PR. Could you please update the README file to include the iOS part ? I'll also add you as a contributor to this project, so you can help us with the iOS part.

Hello @chaitanya-bhagavan,. I'll approve the PR. Could you please update the README file to include the iOS part ? I'll also add you as a contributor to this project, so you can help us with the iOS part.

@sonikro I have updated the README. I have added a new property issuer in BaseRequest.

@sonikro I have updated the README. I have added a new property issuer in BaseRequest.


Birthday partial commit.

Added Remarks to userguide.
Partially Done Birthday Command.
Added command and modified add and edit command.
Created Birthday object

Changes Unknown when pulling 5751dbf8d4c534c8fd25fe7011916b20105dbcff on chilipadiboy:master into ** on CS2103AUG2017-W10-B3:master**.

Changes Unknown when pulling 5751dbf8d4c534c8fd25fe7011916b20105dbcff on chilipadiboy:master into ** on CS2103AUG2017-W10-B3:master**.


This cmdlet outputs nothing

The cmdlet does not return any object upon successful printerport creation.
According to the doc the output of this cmdlet should be the newly created PrinterPort CimInstance object.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 23f3ea76-3e25-ff08-7ba8-a6308424c11a
Version Independent ID: 23ff5a20-6da6-e2fc-22b0-e772c8af494a
Content: Add-PrinterPort (PrintManagement)
Content Source: docset/winserver2012r2-ps/printmanagement/Add-PrinterPort.md
Product: powershell
Technology: ****
GitHub Login: @Kateyanne
Microsoft Alias: v-kaunu

Hello, @sukkerfri This question can best be answered via PowerShell Community.
Thanks again.
The issues section of this repository is intended for documentation problems.
Feel free to reply if there is anything else that could be done to improve the document itself.
Thanks.

Hi.
I didn't mean it to be a question more a statement. The documentation states that the cmdlet will output the newly created object. But this appears to be wrong. The cmdlet does not give any output upon creating a new printer port.


SEPARATE DIV FOR REVIEWS

Description

This PR does the following:

Updates REVIEW SECTION

Related Issues
Fixes #568
Changes

Fixed STYLE

SCRENNSHORT

Checklist
Make sure to check off all the items before submitting. Mark with [x] if done.

[X ] I have performed a self-review of my code
[X ] I have commented my code, particularly in hard-to-understand areas
[X ] My changes generate no new warnings
[X ] I am working on this issue under GSSOC

@Soujanya2004 Please Review PR..

@Soujanya2004 make sure that you accepth it with labed that you give in issue..


Skipped printing empty check lists

#5  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @CatChen and the rest of your teammates on  Graphite


BUGFIX requiring of autoload.php in Google/Client.php

Hi Tim,
working on youtube subscription in PHP using code [here] (https://developers.google.com/youtube/v3/code_samples/php#add_a_channel_subscription) I found a bug in Google PHP API library, in src/Google/Client.php:
if (!class_exists('Google_Client')) {
    require_once dirname(__FILE__) . '/autoload.php';
}

class Google_Client {...}

The require_once dirname(__FILE__) . '/autoload.php'; is never executed as class Google_Client is defined in this file. I have just removed the condition.
BR
Mojo

I signed it!)
2015-09-10 20:05 GMT+02:00 googlebot<EMAIL_ADDRESS>
Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project, in which case you'll need to
sign a Contributor License Agreement (CLA).
[image: :memo:] Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll
verify. Thanks.

If you've already signed a CLA, it's possible we don't have your
GitHub username or you're using a different email address. Check your
existing CLA data https://cla.developers.google.com/clas and verify
that your email is set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
If you signed the CLA as a corporation, please let us know the
company's name.

—
Reply to this email directly or view it on GitHub
https://github.com/google/google-api-php-client/pull/668#issuecomment-139329738
.

This is an innocuous change, but also it's not going to really fix anything either (If someone included Client.php manually then this change would prevent a fatal error, but we don't want to encourage that behavior anyway)

ok :) Thank you for your feedback.
2015-10-09 19:45 GMT+02:00 Brent Shaffer<EMAIL_ADDRESS>
This is an innocuous change, but also it's not going to really fix
anything either (If someone included Client.php manually then this change
would prevent a fatal error, but we don't want to encourage that behavior
anyway)
—
Reply to this email directly or view it on GitHub
https://github.com/google/google-api-php-client/pull/668#issuecomment-146944425
.


Update rule severities table

Fixes #20433
cc: @mavasani

I have an internal PR to docs repo which adds the below section:
### 'Hidden' severity versus 'None' severity

`Hidden` severity rules that are enabled by default are different from disabled or `None` severity rules in couple of ways.

- If any code fix has been registered for a `Hidden` severity rule, then the fix is offered as a light bulb code refactoring action in Visual Studio, even though the hidden diagnostic is not visible to the user. This is not the case for disabled `None` severity rules.
- `Hidden` severity rules can be bulk configured by entries that [set rule severity of multiple analyzer rules at once in an EditorConfig file](#set-rule-severity-of-multiple-analyzer-rules-at-once-in-an-editorconfig-file). `None` severity rules cannot be configured this way. Instead, they must be configured through entries that [set rule severity in an EditorConfig file for each rule ID](#set-rule-severity-in-an-editorconfig-file).

I have an internal PR to docs repo which adds the below section:
### 'Hidden' severity versus 'None' severity

`Hidden` severity rules that are enabled by default are different from disabled or `None` severity rules in couple of ways.

- If any code fix has been registered for a `Hidden` severity rule, then the fix is offered as a light bulb code refactoring action in Visual Studio, even though the hidden diagnostic is not visible to the user. This is not the case for disabled `None` severity rules.
- `Hidden` severity rules can be bulk configured by entries that [set rule severity of multiple analyzer rules at once in an EditorConfig file](#set-rule-severity-of-multiple-analyzer-rules-at-once-in-an-editorconfig-file). `None` severity rules cannot be configured this way. Instead, they must be configured through entries that [set rule severity in an EditorConfig file for each rule ID](#set-rule-severity-in-an-editorconfig-file).

I believe the PR you meant is for VS docs, not dotnet docs.


cigien: Watch routerwebaid.com

cigien requests the watch of the watch_keyword routerwebaid\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

routerwebaid\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


Compiling code for WikiData_counterfact and Wikidata_recent

How can we compile the code for WikiData_counterfact and Wikidata_recent for Llama, gpt2-xl, gptj and other models like shown for zsre dataset. How do we do it for WikiData_counterfact and Wikidata_recent and from where do we download the data?

Hello, we public the data we used in huggingface, you can see it in our README.md and the example in https://github.com/zjunlp/EasyEdit/blob/main/examples/KnowEdit.md

Thank you for the reply.
When I am trying to run this code it is giving me Assertion error
python run_knowedit_llama2.py --editing_method=ROME --hparams_dir=./hparams/ROME/llama-7b.yaml --data_dir=./data/wiki_counterfact/test_cf.json --datatype='counterfact'
It is giving me Assertion error
2024-01-20 12:05:57,321 - easyeditor.editors.editor - INFO - Instantiating model
01/20/2024 12:05:57 - INFO - easyeditor.editors.editor -   Instantiating model
Loading checkpoint shards: 100%|████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 2/2 [00:02<00:00,  1.25s/it]
One Edit instance needs one locality input.....
Traceback (most recent call last):
File "/EasyEdit-main/examples/run_knowedit_llama2.py", line 235, in
metrics, edited_model, _ = editor.edit(
File "/EasyEdit-main/examples/../easyeditor/editors/editor.py", line 156, in edit
requests = self._prepare_requests(prompts, target_new, ground_truth, rephrase_prompts,
File "/EasyEdit-main/examples/../easyeditor/editors/editor.py", line 508, in _prepare_requests
assert len(locality_inputs[locality_key]['prompt']) == len(locality_inputs[locality_key]['ground_truth'])
AssertionError
I have data in this format
data-->wiki_counterfact --> test_cf.json
train_cf.json

please pull the newest code, we have fix this bug.

I am now working with the latest code and when running
python run_knowedit_llama2.py --editing_method=ROME --hparams_dir=/EasyEdit-main/hparams/ROME/llama-7b.yaml --data_dir=./data/wiki_counterfact/test_cf.json --datatype='counterfact'
This is generating the below error
Traceback (most recent call last):
File "/EasyEdit-main/examples/run_knowedit_llama2.py", line 73, in
an=pr["ground_truth"][0][0]
IndexError: list index out of range

I checked and found that the list at this location is empty. Could you manually add an empty string to the data with 'subject': 'Hong Chau'? We may update the dataset after our discussion.

We've just updated the datasets on Hugging Face. Please download the latest version to ensure compatibility.
Thank you for bringing this to our attention!

Hi, have you solved your issue?

We've just updated the datasets on Hugging Face. Please download the latest version to ensure compatibility. Thank you for bringing this to our attention!

 Thank you so much for the super quick fixes.
I have tried just with ROME for Llama2-7b. Yes the issue has been resolved. Thank you!


Weather station - SGP30 TVOC and eCO2

My SGP30 TVOC and eCO2 will be on the same board as my PM sensor so time to get this working too.

SGP30 Air Quality Sensor Breakout
ESPHome

Originally posted by @jcallaghan in https://github.com/jcallaghan/home-assistant-config/issues/13#issuecomment-623392099

Had issues with ESPHome finding a device when I was using the default i2c pins. In the log, I would get an error saying Found no i2c devices. When I change to using the second i2c pins it found my sensor.
i2c:

  - id: i2c_1
    sda: 21
    scl: 22
    scan: True

  - id: i2c_2
    sda: 33
    scl: 32
    scan: True

sensor:

  - platform: sgp30
    i2c_id: i2c_2
    eco2:
      name: "${system_name}_eCO2"
      accuracy_decimals: 1
    tvoc:
      name: "${system_name}_TVOC"
      accuracy_decimals: 1
    address: 0x58
    update_interval: 5s

A useful review of different VOC sensors.

This scale is from the German Federal Environmental Agency’s guidelines on VOCs. (According to the BME680 data sheet)

Now the temperature and humidity are available from BME680 I will use them to calculate compensation which the SGP30 sensor allows for the temperature and humidity is available.

Final wiring combined with BME680/


chore(cmake): allow user to override GIT_COMMIT from cmake.

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area build
Does this PR require a change in the driver versions?
What this PR does / why we need it:
Discovered during the development of https://github.com/falcosecurity/driverkit/pull/302; basically we don't allow the user to override GIT_COMMIT; but since we support DRIVER_VERSION explicitly set by user, i think we should let user also override the commit.
See the driverkit PR workaround: https://github.com/falcosecurity/driverkit/pull/302/files#diff-16a66a3bc20a8ec693f64ed821148f71efa386a7e942444444eb591512bf129bR51
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/cc @leogr @Andreagit97

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


storm-elasticsearch-examples: fixed all checkstyle warnings

I'd like to share my hands-on reflection on whether it makes sense to take over you checkstyle-config for some of my projects. The approx. 100 violations cost about 1,5h to fix which seems a realistic indicator for the time needed for similar fixes.

@krichter722 Thanks for the fixes. I'll review this as soon as I can. In the meantime, could you raise issues on https://issues.apache.org/jira to track your changes? It makes it easier for us to track which branches fixes have been applied to, and we use the JIRA issues to generate release notes. Please also rename the PR and commit to contain the JIRA issue number. Thanks :)

Thanks for the fixes. I'll review this as soon as I can. In the meantime, could you raise issues on https://issues.apache.org/jira to track your changes?

Nice. Done.

I agree with all your comments and removed the TridentEsTopology.get/setMaxBatchSize. The noisy comments are required (see comment above). Thanks for the review.

@krichter722 kindly reminder.

Last comments incorporated.

+1, thanks for being patient. @krichter722 could you add the issue number to the commit message, I'll merge after that. I'd add it myself, but then the PR won't show as merged.

@krichter722 Thanks, merged to master. I also added you to the contributors list in JIRA, so you should be able to assign tasks to yourself now.


126 avoid std and core mem

here it's said to not use format!, but here says that using format! is a convention. Do we need to put this detector?


Removed broken Google+ link.

Since Google+ is shut down, broken Google+ link is removed.

Indeed, thanks for removing the link!


Dependency manager didn't pull the latest releases

When I ran edm init main --workspace ~/checkout/everest-workspace, it didn't pull the latest release for everest-framework. The CMakeLists.txt file still showed a version # of 0.8.1 and I had to comment out some deprecated lines of code related node_api_version to compile properly.
Not sure if this version control is intended behaviour.

Thanks for reporting @A-thaVan!
I am not sure about your intention:

a) You want to checkout the latest release:
that would mean not checking out all the main branches but the latests tagged version of everest-core and the dependencies described in its dependencies.yaml:
https://github.com/EVerest/everest-core/blob/2023.10.0/dependencies.yaml (v0.8.0 for everest-framework in this case)
b) Checking out the current main branch to have a good starting point for development:
In this case your command doesn't result in such a good starting point as already adressed in https://github.com/EVerest/everest-dev-environment/issues/33
So the behaviour of edm is here to checkout everest-core's main branch and the versions defined in dependencies.yaml which is right now a non human readable git revision:

everest-framework:
  git: https://github.com/EVerest/everest-framework.git
  git_tag: a233ec8
  options: ["BUILD_TESTING OFF"]

This revsion is not the current main for everest-framework. It includes the version 0.8.1 in its CMakeLists.txt
Feel free to comment if you have any further questions on this matter.


Make "internals" configurable

I was looking through the README and noticed this internal folder thing that acts sort of like node_modules.
Can we make this configurable? If there's a folder internal in my project I'd like for it to be included but still have node_modules excluded.
Refs:
https://github.com/MailOnline/jest-tap-reporter/commit/02086ccca8ef6d4a8cc8e60f521ff891e41b45fe#diff-5e0b92ae1f4819f9023cb45116668ef6R5
https://github.com/MailOnline/jest-tap-reporter/commit/20018d5838a8516464a6af1365ba8f266d7df39e
PR?

The internals folder is excluded to hide stack traces from Node's /internal folder.
As I understand, the problem is that you have /internal folder in your project and the reporter hides stack traces from that?
In that case it is a bug, we should show stack traces from your folder but exclude only Node's /internal folder, if possible to discriminate between the two.


Show spinner on submit button and disable it during use

I want to work on this issue.

I want to work on this issue.


Icon Request

Icon Request For com.avast.android.mobilesecurity or advast mobile security

Hello! Weʼre switching to a new system for icon requests in order to be able to better manage them. From now on, icon requests can be submitted here: https://forms.gle/Fx8vZAiWdW1Tyjo57. If your icon request hasnʼt yet been fulfiled, it has been copied to our new icon request database, meaning no action is required. Consequently, this issue will be closed.


How to add a space before this line?

Hi, do you need to add empty line between code and Documentos lines? Am I right?

I actually wanted to add a space at the beginning of the line of code so that it would be proportional to that arrow!

Sorry, but that's not a prerogative of Spaceship. This theme affects only to PROMPT environment variable. It shouldn't touch the output of commands, so I can't help you.
This issue isn't related to Spaceship, so I'm going to close it. If you think, I didn't understand you or if you have any other question, feel free to open new issue or reopen current one.

No problem, thank you for your attention, and congratulations for the work.

Thank you


AttributeError in self.get_option('username', project).strip()  [Sentry 8.0.1]

After upgrading to Sentry 8.0.1 and sentry-slack 0.4.0 I get the following error while testing the Slack integration:
[ERROR] Plugin(slack) raised an error during test
Traceback (most recent call last):
  File "/home/beta/sentry_env/local/lib/python2.7/site-packages/sentry/plugins/base/configuration.py", line 41, in default_plugin_config
    test_results = plugin.test_configuration(project)
  File "/home/beta/sentry_env/local/lib/python2.7/site-packages/sentry/plugins/bases/notify.py", line 156, in test_configuration
    return self.notify(notification)
  File "/home/beta/sentry_env/local/lib/python2.7/site-packages/sentry/../sentry_slack/plugin.py",  line 129, in notify
    username = self.get_option('username', project).strip()
AttributeError: 'NoneType' object has no attribute 'strip'

Looks like it's related to #29, patching username and channel (same error) worked as a temporary fix.

This was fixed for 0.5.0.


Replace "#pragma once" macro with header guard

Is your feature request related to a problem? Please describe.
google c++ code style
header guard is preferred than "#pragma once"
Describe the solution you'd like
replace those "#pragma once" with header guard

I think it's not necessary to replace #pragma once with header guard.
#pragma once has been well supported across compilers since not actually part of the standard, and it's less prone to make mistakes and less code to type.

I actually think compiler dependent is part of the problem, although it's not so urgent.
In fact, you don't event have to type these guards, lots of IDEs, even editors such as "vs code" would do this things automatically.
If we were to do this, we could also enforce cpplint in the CI procedure.

使用头文件保护会导致以后修改文件名的时也需要一起修改宏的名称, 而使用 #pragma once 则没有这个顾虑.
在编译器兼容性方面, #pragma once 应该不再是一个问题了, 如果代码检查工具依然存在相关的规则, 可以考虑将相关规则禁用.


New query parameter validator component

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

17 of 27 (62.96%)  changed or added relevant lines in 2 files are covered.
85 unchanged lines in 11 files lost coverage.
Overall coverage decreased (-2.0%) to 35.248%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/QueryParameterValidator/QueryParameterValidator.php
17
20
85.0%

src/Api/QueryParameterValidator/Validator/ValidatorInterface.php
0
7
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/Exception/FilterValidationException.php
1
50.0%

src/GraphQl/Resolver/Stage/SerializeStage.php
1
0.0%

src/Symfony/EventListener/RespondListener.php
1
86.05%

src/Symfony/EventListener/AddLinkHeaderListener.php
2
87.5%

src/State/Util/RequestParser.php
7
52.38%

src/Symfony/EventListener/QueryParameterValidateListener.php
7
47.62%

src/HttpCache/EventListener/AddTagsListener.php
10
43.48%

src/Symfony/EventListener/DenyAccessListener.php
10
68.57%

src/Symfony/EventListener/ValidateListener.php
11
0.0%

src/HttpCache/EventListener/AddHeadersListener.php
15
43.33%

Totals

Change from base Build<PHONE_NUMBER>:
-2.0%

Covered Lines:
10099

Relevant Lines:
28651

💛  - Coveralls

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:

Merged upstream as https://github.com/api-platform/core/pull/6080


Update Dockerfile

Second release candidate for v0.2.15

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (e01999c) 3.99% compared to head (e94385c) 3.99%.

Additional details and impacted files
@@          Coverage Diff           @@
##           master    #129   +/-   ##
======================================
  Coverage    3.99%   3.99%
======================================
  Files          64      64
  Lines        4306    4306
======================================
  Hits          172     172
  Misses       4134    4134

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Started getting "Error: Can't wait without a fiber"

This error just started happening as of yesterday. Any ideas?
Error: Can't wait without a fiber
at Function.wait (/app/build/bundle/programs/server/node_modules/fibers/future.js:155:9)
at Object.Future.wait (/app/build/bundle/programs/server/node_modules/fibers/future.js:397:10)
at [object Object]._.extend.count (packages/mongo/mongo_driver.js:1059:1)
at [object Object].Cursor.(anonymous function) (packages/mongo/mongo_driver.js:869:1)
at [object Object].cursorProto.(anonymous function) (packages/meteorhacks_kadira/lib/hijack/db.js:126:1)
at [object Object].kadira_Cursor_count [as count] (packages/meteorhacks_kadira/lib/hijack/set_labels.js:81:1)
at updateRecords (packages/aldeed_tabular/server/tabular.js:118:1)
at updateRecords (packages/aldeed_tabular/server/tabular.js:118:1)
at [object Object].later [as _onTimeout] (packages/underscore/underscore.js:695:1)
at Timer.listOnTimeout [as ontimeout] (timers.js:121:15)

@maxko87 did you find wha was causing this? I'm getting a similar error.


Update Toga to use Python 3.5+

At pycon sprints @freakboy3742, @phildini, and I discussed no longer supporting Python 3.4. This will allow us to move forward with supporting the new asyncio declaration syntax and start testing with Python 3.7 as it is released soon.
I also replaced the sys.version Python checks using the python_requires syntax.
Finally, I updated the handler module with the newer syntax.
PR Checklist:

[ ] All new features have been tested
[ ] All new features have been documented
[X] I have read the CONTRIBUTING.md file
[X] I will abide by the code of conduct

@freakboy3742 I can do that, I volunteer to submit a similar PR to the other projects.

Build is currently locking up because of the missing gtk-py35 build image; I'm currently pushing an update...


Fix credentials validation when the external CCM is enabled

As per https://github.com/kubermatic/kubeone/pull/575#issuecomment-511318662, the validation is going to fail in some cases due to missing credentials when the external CCM is enabled.

/assign
/lifecycle active

I'm unable to reproduce this anymore.
/close


FOSSA workflow added

Fixes https://github.com/servicemeshinterface/smi-spec/issues/183
Signed-off-by: Ihor Dvoretskyi<EMAIL_ADDRESS>

It's unclear to me now what causes an issue - https://github.com/servicemeshinterface/smi-sdk-go/pull/86/checks?check_run_id=976150974#step:5:22; in touch with the FOSSA team on debugging.

@stefanprodan PTAL, looks good now.

The URL to the report returns a 404

@stefanprodan fixed, thanks for pointing out.

@idvoretskyi I can't access the report even if I'm a maintainer of the SDK and I did login with GitHub.

@stefanprodan if you'll scroll down, you'll see the full public report.
In addition, I'm trying to invite you and other project maintainers to the FOSSA UI to get the maintainers access, but this part of UI doesn't seem to be working (reported to FOSSA).
However, the full report is already r/o available so I think we can proceed with merging this now. I'll work on granting you the full r/w access separately.

@stefanprodan your access should be fixed now, please check. If all good, I'll proceed with adding other project maintainers to the org.

@stefanprodan anything else is needed to get this merged?

Ref - https://github.com/cncf/foundation/issues/109

Hey @idvoretskyi I was on PTO, I'll merge the FOSSA PRs now, look ok thank you!

@stefanprodan thanks!


macOS: Category Dropdown Pre-filled with first value

The Category dropdown is always pre-filled with "ADC/DAC/Data Conversion" when opening the "Assign LCSC Number" Dialog.

I believe I've found a patch for this by adding
self.category.SetSelection(wx.NOT_FOUND)

after line 111 in partselector.py - that seems to force wxwidgets to clear out the Category box.

@asinghani Nice find!
Do you want to submit a PR?


New updates?

Is this project with support?

Yes, it's a live project. I'm trying to find new places to improve.

@elhanan-mishraky great!... Congrats!


encapsia database backup fails when tmp and destination are on different devices

encapsia database --host adi-test.encapsia.com backup /backups/daily/adi-test/Friday/adi-test.encapsia.com-201219.sql
Backing up database....Done
Traceback (most recent call last):
File "/usr/local/bin/encapsia", line 11, in
sys.exit(main())
File "/usr/local/lib/python3.6/dist-packages/click/core.py", line 764, in call
return self.main(*args, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/click/core.py", line 717, in main
rv = self.invoke(ctx)
File "/usr/local/lib/python3.6/dist-packages/click/core.py", line 1137, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/usr/local/lib/python3.6/dist-packages/click/core.py", line 1137, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/usr/local/lib/python3.6/dist-packages/click/core.py", line 956, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/usr/local/lib/python3.6/dist-packages/click/core.py", line 555, in invoke
return callback(*args, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/click/decorators.py", line 27, in new_func
return f(get_current_context().obj, *args, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/encapsia_cli/database.py", line 28, in backup
temp_filename.rename(filename)
File "/usr/lib/python3.6/pathlib.py", line 1309, in rename
self._accessor.rename(self, target)
File "/usr/lib/python3.6/pathlib.py", line 393, in wrapped
return strfunc(str(pathobjA), str(pathobjB), *args)
OSError: [Errno 18] Invalid cross-device link: '/tmp/tmpncvu23yb' -> '/backups/daily/adi-test/Friday/adi-test.encapsia.com-201219.sql'

Created Pull request https://github.com/tcorbettclark/encapsia-cli/pull/15 to propose solution

PR #15 merged. Closing bug.


Add delay for "autoPlayTime"

When using the autoPlayTime attribute, it would be nice to have delayAutoPlayTime.
This feature allows a user to set each carousel to start at different times.
PS. Love this carousel Lib, you've done an amazing job 👌

@LukeSeers
Thank you for your feedback 👍
I'd like to confirm your needs are ?
[unit: ms]
carouselA {delayAutoPlayTime: 0, autoPlaytime: 2000}
carouselB {delayAutoPlayTime: 1000, autoPlaytime: 2000}

window.addEventListener('load', function () {
  [carouselA] -> (0s) initPlay -> (2s) play -> (2s) play
  [carouselB] -> (1s) initPlay -> (2s) play -> (2s) play
})

@LukeSeers Hi
Thank you for your feedback 👍
I'd like to confirm your needs are ?
[unit: ms]
carouselA {delayAutoPlayTime: 0, autoPlaytime: 2000}
carouselB {delayAutoPlayTime: 1000, autoPlaytime: 2000}

window.addEventListener('load', function () {
  [carouselA] -> (0s) initPlay -> (2s) play -> (2s) play
  [carouselB] -> (1s) initPlay -> (2s) play -> (2s) play
})

Perfect 👍

@LukeSeers Hi, support this feature in v4.0.10
https://react-carousel.bearests.com/docs/feature/auto-play
try it ！


POST wrong

hi  when i post  to dowload data
<IP_ADDRESS> - - [31/Mar/2016 23:05:30] code 400, message Bad HTTP/0.9 request type ('Content-Length:')
::ffff:<IP_ADDRESS> - - [31/Mar/2016 23:05:30] "Content-Length: 63" 400 -

It seems that you send wrong HTTP request line :confused:
It should be either of POST /foobar HTTP/1.0 or POST /foobar HTTP/1.1.
Check what you sent to proxy2 server.

POST http://www.bttiantang.com/download.php HTTP/1.1
Accept: /
Proxy-Connection: Keep-Alive
Content-Type: application/x-www-form-urlencoded
Accept-Encoding: gzip, deflate
User-Agent: Mozilla/4.0 (compatible; MSIE 7.0; Windows NT 6.0; WOW64; Avant Browser; SLCC1; .NET CLR 2.0.50727; Media Center PC 5.0; InfoPath.1; .NET CLR 3.5.30729; .NET CLR 3.0.30618)
Host: www.bttiantang.com
HTTP/1.1 200 OK
Server: nginx/1.2.9
Date: Tue, 05 Apr 2016 15:26:10 GMT
Content-Type: text/html; charset=utf-8
Vary: Accept-Encoding
X-Powered-By: PHP/5.2.17p1
Content-Encoding: gzip
::ffff:<IP_ADDRESS> - - [05/Apr/2016 23:23:24] code 400, message Bad HTTP/0.9 request type ('Content-Length:')
::ffff:<IP_ADDRESS> - - [05/Apr/2016 23:23:24] "Content-Length: 64" 400 -
I am confused，and want get the torrent file
http://www.bttiantang.com/download.php?temp=yes&id=27871&uhash=05043b3a7b498dd1ad68406a

The request is bad. It's not this project's issue. Close.

#16


Package.json | syntax for "test" and "push" pre-commit hooks fail on Windows machines

There was an effort made by @dawsonbooth to do this, but it is currently in a state that only works in mac/linux shells. A solution that works in both would be ideal, barring that some OS detection which runs a different script?

I believe I only updated the lint and prettier scripts to work in all platforms. This was by wrapping the glob patterns in single-quotes in order that the shell's own glob parsing does not interfere with that of eslint and prettier.
It looks like the test and push hooks may need something different since there do not seem to be any glob patterns exposed in the commands.

:tada: This issue has been resolved in version 1.16.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fixed Bug with Property IconDensityAware

When IconDensityAware is set then the Control doesn't render any more

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks a lot for your contribution. The icon density aware functionality insap/m/FeedInput was refactored heavily and the code lines effected by this PR were removed. Therefore closing this PR.


Cleanup of assertions page

Fixes #106

@ReedCopsey ah, braces 👍 That's what I get for being a non-native English speaker lol


Arabic pages paths with ":" is not working

Reproduction
https://stackblitz.com/edit/github-awuqxh
Steps to reproduce the bug
1- go to the Arabic page with :
2- the page is not found.
3-go to the English page with : and it works
Expected behavior
Any Arabic page with : should work.
Actual behavior
Any page with : and Arabic characters is not working.
Additional information
I am trying to add the below page to my project
/خاص:جديد
but the issue is that any page with a colon : and with Arabic characters is not working.
as you can see in the reproduction i have created multiple pages
/profile:new
/خاص:جديد
/جديد
and if you try to go the English page with a colon it is working fine, but when you try to go to the Arabic one its not working while going to the page without a colon is working fine.
Note that the issue at first appeared using Nuxt 3 but is seems it is also reproducible in Vue 3
https://github.com/nuxt/nuxt/issues/21582

Params must be alphanumerical. They can also contain _

Params must be alphanumerical. They can also contain _

Thank you for your reply, But what if i want just to have a colon : in my path with arabic characters? not as a param

You escape it with \

You escape it with \

I have tried to escape it with \ but its not working
i have tried it like this without encoding the colon
routes.push({   path: '/'+encodeURI('جديد')+'\:'+encodeURI('خاص'),   component: pageComponent, });
and like this with encoding the colon
routes.push({   path: encodeURI('/خاص\:جديد'),   component: pageComponent, });

You need to escape the \ to be a \ inside a string 🙃


Added Nordic SoftDevice license agreement

Adding basic Cortex-M4 support
Tidy up targets.py

If a person doesn't agree to the terms of the Nordic license should they remove the whole mbed git repo from their machine?  It is kind of misleading to say that the mbed SDK is Apache 2.0 licensed in the main README and then find this license buried down in the tree.
I often have GDB configured to disassemble a bit of code each time the debuggee is halted when I am debugging my own code that has been optimized.  When performing such debugging on my nRF51822 board the code will inevitably crash in the SoftDevice blob somewhere and GDB will disassemble a few instructions at the point of this crash.  Does this put my use of the mbed SDK on Nordic devices in violation of section 3 of that license?

@screamerbg Can you rebase this PR to the latest master please?


Increase the speed optimizations for computing direction on covidcast signals

One of the recent direction runs took 90 minutes to complete, which seems excessive.
I suspect we are doing something that was easy to implement but not terribly scalable, like updating the direction for all dates and all geo_ids when only a small number of them were invalidated, and perhaps also using a separate query for each time series with updates (there were 72546 of those in the 90-minute run).
If it turns out the database code isn’t the issue, we might have to think about moving the direction computation outside of epidata and into the individual indicators.

@AlSaeed proposed the following:

The primary id of each row should be also retrieved when retrieving the time series.
4 lists of row ids should be computed (across timeseries). These are the rows that will change their direction value to [-1,0,1,NULL].
4 batched queries to update the direction field only, based on the ids of the lists above.
When retrieving the stale timeseries we should store them in temporary table.
Updating timestamp2 can be done in 1 query, using the temporary table from [4.].
For the retrieval part, if we expect the data of all timeseries to fit in memory simultaneously, we can use the temporary table from [4.] to retrieve the entire data and use pandas to separate them into independent series.

This will work, with some modifications to support the upcoming shift to include issue dates. I've consulted with @jacobbien, and he suggests that since direction is relatively complex to compute (it's the slope of a line fit to all values of the previous 7 days, thresholded based on the variance of historical data), that it's really a computed product, not raw data. Therefore, updating the direction  column should be done in-place without creating a new issue for the affected timepoint.
As an example, consider the following input to the direction updater:

geo value
value
time value
issue
direction
direction timestamp

ca
4.1
20200601
20200601
0
stale

ca
4.0
20200602
20200602
0
stale

ca
4.1
20200603
20200603
0
stale

ca
4.2
20200604
20200604
0
stale

ca
5.0
20200605
20200605
null
stale

The proposed solution is the following:

geo value
value
time value
issue
direction
direction timestamp

ca
4.1
20200601
20200601
1
fresh

ca
4.0
20200602
20200602
1
fresh

ca
4.1
20200603
20200603
1
fresh

ca
4.2
20200604
20200604
1
fresh

ca
5.0
20200605
20200605
1
fresh

The alternative is the following:

geo value
value
time value
issue
direction
direction timestamp

ca
4.1
20200601
20200601
0
stale

ca
4.1
20200601
20200605
1
fresh

ca
4.0
20200602
20200602
0
stale

ca
4.0
20200602
20200605
1
fresh

ca
4.1
20200603
20200603
0
stale

ca
4.1
20200603
20200605
1
fresh

ca
4.2
20200604
20200604
0
stale

ca
4.2
20200604
20200605
1
fresh

ca
5.0
20200605
20200605
1
fresh

@melange396, barring unforeseen complications we're expecting a PR for query optimizations on this tomorrow -- if database calls turn out to be the top contributor in your profiling efforts, ignore them for now.

First pass of implementation is in #133


Atom helper high CPU on Mac

Using 0.180.0 on Yosemite 10.10.2, I consistently the Atom Helper process run at 80%-100% CPU.

Thanks for the report @roboen -- I'm closing this in favor of https://github.com/atom/atom/issues/3426. If you have any other details you'd like to add -- please add a comment to that issue.

Same problem here

replace /Applications/Atom.app/Contents/Resources/app/apm/bin/node with newer version, like 0.12.*
It works perfectly now

As I mentioned in my post on Discuss, replacing that copy of node doesn't change the version of node that Atom is running on.


Alva crashes when it is placed outside of the application folder.

Describe the bug
Alva cannot start when the app is placed in another folder than the application folder.
To Reproduce
Steps to reproduce the behavior:

Download 'Alva'
Move 'Alva' to any folder except of the application folder
Start 'Alva'
See error

Screenshots

Screenshot Alva Error
Device (please complete the following information):

OS & Version: [e.g. macOS 10.13.6]
Version of Alva [Version 0.7.0 (0.7.0)]

Hey Fabrice,
thanks for letting us know that you found a bug 👍
I tried to reproduce it with the same version of Alva and OS, sadly I couldn't.
Did you verified that the application is working from the Application folder?
If yes, could you try to move Alva to your Downloads or Desktop folder and try again?
I hope we can solve this issue together, thanks again for letting us know.
Have a great Friday
Julian

Hey @FabriceSpahn ,
di

Hey @FabriceSpahn ,
did you try to reproduce to issue with the new version of Alva? Does the issue still occur?

Hey @markusoelhafen and @Jumace,
I tried to reproduce it several times with the old and now with the new beta version, but it seems like the bug is gone. It works smoothly.


Change tests for event rules

Now all event tests extend EventSpec
Helper fixtures moved to EventSpec

@gcgbarbosa why doesn't your PR has a codecov report?
like https://github.com/lum-ai/odinson/pull/87#issuecomment-645858742

@marcovzla it does. The report was generated. The question is: why codecov bot sometimes come and sometimes does not come :(
You can find the report here:
https://codecov.io/gh/lum-ai/odinson/commit/8ae2ac35e8e20e50d0c95f6cce6e047c417ffe2c

I noticed that the report says Waiting for CI to complete. Please review report with caution, it may change.
May that be related to the bot issue?

Yeah, I just noticed the same thing. Whenever codecove detecs the CI status it sends the bot. It is not detecting our CI status for some reason.

I found a few people saying that installing the code-cov github plugin would solve the problem, so I went ahead and asked for permition to install the plugin
https://github.com/codecov/codecov-python/issues/157

I love the idea of a TAG svg of all the docs!!!!! love love love it (at
least all docs that are short enough to be feasible!)
I think an id could be something like "ninjaTurtles" for the ninja turtles
sentence, or "gummyBears", etc... it doesn't need to be exhaustive, but
something human-readable that we can use as a reference
On Mon, Jun 22, 2020 at 7:18 PM George C. G. Barbosa <
<EMAIL_ADDRESS>wrote:

External Email
@gcgbarbosa commented on this pull request.
In core/src/test/scala/ai/lum/odinson/events/EventSpec.scala
https://github.com/lum-ai/odinson/pull/88#discussion_r443923495:

   val matchedForThisRole = groupedMatched(desiredRole)

   desired should have size matchedForThisRole.size

   for (d <- desired) {

     matchedForThisRole should contain (d)

   }

   // there shouldn't be any found arguments that we didn't want

   val unwantedArgs = groupedMatched.keySet.diff(groupedDesired.keySet)

   unwantedArgs shouldBe empty

 }

}

def createArgument(name: String, start: Int, end: Int): Argument = Argument(name, start, end)

def getJsonDocument(id: String): String = {
val json = Map(

 "1" -> """{"id":"48fb577b-f5ba-4e16-864f-f8ba20ba9cfa","metadata":[],"sentences":[{"numTokens":8,"fields":[{"$type":"ai.lum.odinson.TokensField","name":"raw","tokens":["The","consumption","of","gummy","bears","and","donuts","."],"store":true},{"$type":"ai.lum.odinson.TokensField","name":"word","tokens":["The","consumption","of","gummy","bears","and","donuts","."]},{"$type":"ai.lum.odinson.TokensField","name":"tag","tokens":["DT","NN","IN","NN","NNS","CC","NNS","."]},{"$type":"ai.lum.odinson.TokensField","name":"lemma","tokens":["the","consumption","of","gummy","bear","and","donut","."]},{"$type":"ai.lum.odinson.TokensField","name":"entity","tokens":["O","O","O","B-dessert","I-dessert","O","B-dessert","O"]},{"$type":"ai.lum.odinson.TokensField","name":"chunk","tokens":["B-NP","I-NP","B-PP","B-NP","I-NP","O","B-NP","O"]},{"$type":"ai.lum.odinson.GraphField","name":"dependencies","edges":[[1,0,"det"],[1,4,"nmod_of"],[1,7,"punct"],[4,2,"case"],[4,3,"compound"],[4,5,"cc"],[4,6,"conj"]],"roots":[1]}]}]}""",

Do you think using the id is enough?
I was planning to have a TAG svg of all documents used for test somewhere
in a readme.
Meanwhile the only thing I could think would be informative would be the
document id.jk
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/lum-ai/odinson/pull/88#discussion_r443923495, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJCPCOZ4G43SK5AM6EJT5TRYAGIBANCNFSM4OC73DZQ
.

@gcgbarbosa There is a conflict. I just merged #94 so that could be it.
Can you please take care of it?
Thanks!

@marcovzla done.


[Bug]: Unable to write data when the total amount of mock data is lower than batchSize during data simulation

ODC version
odc423
OB version

What happened?

What did you expect to happen?
Data can be successfully written
How can we reproduce it (as minimally and precisely as possible)?
When executing simulated data, set the total amount of mock data to be lower than batchSize, and execute simulated data
Anything else we need to know?
No response
Cloud
No response

pass


Reading Forces & Accelerometer from PyBullet

I am writing this as a reference as I might forget about it by next week. I will fix it in the next few days, but here are some issues,

PinBulletWrapper.get_force()  seems to return contact forces in the negative direction (clear on the z axis not so sure about the other directions). Would it make sense to flip it? I will look into pybullet documentation and why is this happening.

The other issue  with PinBulletWrapper.get_force() is that it returns only the active contacts, so we get a varying size list, would it make sense to store this list in the fixed number of end effectors, and set all in-active contacts to zero? or will that break too much code ? @jviereck

@nrotella as I understand you added all the IMU functionality to the same wrapper? Just to confirm  PinBulletWrapper.get_base_imu_linacc() and PinBulletWrapper.get_base_imu_angvel() transform the IMU readings from IMU frame to base frame or is it the other way arround? but in order to do that I have to call PinBulletWrapper.compute_numerical_quantities(dt) first ?

let me know what you guys think and I'll work on fixing/changing stuff if need be :)

Sure
Happy to change this as well.
How about we call PinBulletWrapper.compute_numerical_quantities(dt) with the step function?

I can take care of the first two points, and it seems that PinBulletWrapper.compute_numerical_quantities(dt) gets computed in BulletEnv.step()  so no need to take action on this one. I will close this issue once I fix the first two points


Modified README.md with docs fix

After testing bdk-flutter, I noticed that there are several errors / typos in readme.md. The usage has been fixed.

For genSeed() and getBalance() method.

Error: : Error: Undefined name 'BdkRn'.
Fix: Instead of BdkRn used BdkWallet

For all the methods, response is declared as const which throws.

Error: : Error: Not a constant expression.
Fix: final response = await BdkWallet().methodName();

Also, static access to instance member is not allowed, like final response = await BdkWallet.genSeed();

Fix: final response = await BdkWallet().genSeed();

For confirmedTransactions() method.

Error:
The method 'confirmedTransactions' isn't defined for the type 'BdkWallet'. Try correcting the name to the name of an existing method, or defining a method named 'confirmedTransactions'.
Fix: final response = await BdkWallet().getConfirmedTransactions();

Amazing Project, Thanks !

@aniketambore Thanks for this, the readme has been updated a lot due to new methods added. So this PR may need to be updated. Also, I would suggest make the PR for the ffi branch if you can, as thats the branch which will be merged to main.
I am happy for you to update it and I will merge when its done. You can use this as a reference to save you some trouble: https://github.com/LtbLightning/bdk-rn/blob/main/README.md