An error occurred while loading view: github:createPullRequestWebview

Testing #6102
I followed the instructions in my fork of vscode, which github had said was synced with main, clicked Revert and saw this:

I wonder if we could surface error info in the view?
The issue was I didn't have a main branch in my fork, so feel free to close.
mainThreadExtensionService.ts:78 [vscode.git]No such branch: main.
$onExtensionRuntimeError @ mainThreadExtensionService.ts:78
mainThreadExtensionService.ts:79 Error: No such branch: main.
    at N.getBranch (vscode-file://vscode-app/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/git/dist/main.js:2:936524)
    at T.retryRun (vscode-file://vscode-app/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/git/dist/main.js:2:1031903)
    at T.run (vscode-file://vscode-app/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/git/dist/main.js:2:1031507)
    at T.getBranch (vscode-file://vscode-app/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/git/dist/main.js:2:1019856)
    at ml.getCreateParams (vscode-file://vscode-app/Applications/code-insiders-portable-data/extensions/github.vscode-pull-request-github-0.93.2024072304/dist/extension.js:2198:5496)
    at ml.getCreateParams (vscode-file://vscode-app/Applications/code-insiders-portable-data/extensions/github.vscode-pull-request-github-0.93.2024072304/dist/extension.js:2216:3294)
    at ml.doInitializeParams (vscode-file://vscode-app/Applications/code-insiders-portable-data/extensions/github.vscode-pull-request-github-0.93.2024072304/dist/extension.js:2198:7883)

There are probably other things that won't work right if you don't have your repo's default branch, but this is easy enough to improve.


FIX: Wildcard domain cannot be given due to AWS tag rule violation

Module would be able to handle wildcard domains but AWS tag rules block terraform apply.
I have added a placeholder instead of '*'. Please feel free to suggest a better one or modify it on your own.

Thanks for the contribution. Waiting for another approval from the team and then I'll merge the PR.


Change references of old service name MSI to MI (Managed Identity)

Description
Upon reviewing the Azure docs, Managed Identity 'is the new name for the service formerly known as Managed Service Identity (MSI)'. There are references in code (specifically the Key Vault client Initialization for each helium service) and documentation for this old term that will need to be modified to avoid confusion.

I've changed all the references of MSI to MI, and have submitted Pull Requests for the same. I'm new to this so my contributing message might not be 100%  accurate to the guidelines. Sorry about that. @sivamu

No worries, thanks for submitting the changes! @swapniljha001


Added support for multiple organizations for a user - currently in BETA

Would be nice to be able to use this Zendesk client with the upcoming support for multiple organization that is currently in BETA.
I could host the code myself but it would be nice to be able to get it from a released version :)
https://support.zendesk.com/hc/communities/public/posts/203464106-Managing-Organization-Memberships-via-the-Zendesk-API

You'll need to edit the pull request to exclude 25d9f18 and 6ae39a2 also I'd rather not merge in BETA features...

Fully understand that you don't want to merge BETA features, however they've been BETA for about 2 years I think and I would be surprised if they didn't release it publicly soon :)
I'll fix my release commits, sorry about those :)

Replaced with #38


Configuration file support

It would be nice to have a configuration file, which can be used to en-/disable different checks on a broader scale than the tfsec:ignore comment.
This could also be used to exclude whole directories, configure tfvars, output format etc.

this has been done now


Windows Authentication support

If you remove username and password from https://github.com/storebrand/target-mssql/blob/main/target_mssql/connector.py#L65-L66 and you run on Windows this will connect to the SQL server just fine!

We've moved on from Meltano and are no longer maintaining this repository. Therefore I'm closing this issue.


don't leak what Go version built the garbled binary

Right now, we strip module build info, so go version -m binary doesn't say anything about a module's path or version. However, go version binary will still tell you exactly what Go version built the binary.
This is being leaked for no real reason. Let's get rid of it.
/cc @nick-jones

@mvdan interestingly after your commit the version still ends up in the binary (at least with my local env)
$ go version
go version go1.14.4 darwin/amd64
$ garble build
$ rg -N -o -a 'go1\.\d+\.\d+' test
go1.14.4

I have a feeling this might be because -X is ineffective on constants, so sys.TheVersion is untouched.
$ cat main.go
package main

var x = "foo"

func main() {
        println(x)
}
$ go build -ldflags="-X main.x=bar" .
$ ./test
bar
$ cat main.go
package main

const x = "foo"

func main() {
        println(x)
}
$ go build -ldflags="-X main.x=bar" .
$ ./test
foo

If that is the reason I'm not sure there is easy fix? In any case, since runtime.buildVersion is modified, go version <exe> no longer spits out the version, so it's certainly better than nothing :)

@nick-jones correct again :) I keep forgetting that constants don't work with -X, and the flag will silently do nothing instead of erroring.
I'm not sure what the path forward is. We could modify the source for the runtime/internal packages, but I want to avoid that if possible, as it would make cached builds harder in the future - right now, we don't modify any standard library packages, let alone the runtime package, which is the base for everything.
The answer might be to mess with the final binary to replace the strings. I assume a simple search-and-replace won't work well, as it could mess up other strings from other packages. I wonder if there would be any good way to locate strings in a binary and modify them. We likely need to do that to obfuscate import paths, too: https://github.com/mvdan/garble/issues/13

The answer might be to mess with the final binary to replace the strings. I assume a simple search-and-replace won't work well, as it could mess up other strings from other packages. I wonder if there would be any good way to locate strings in a binary and modify them. We likely need to do that to obfuscate import paths, too: #13

How strings manifest in Go built binaries is something I've taken a recent casual interest in. I'm by no means super knowledgable in this space, but with my current understanding I'd suggest this would be tough to achieve (at least for this particular case, I've not explored where import paths end up).
A naive replace would of course be possible but as you suggest, rather risky. If we wanted to avoid that risk and work out where the contents of exactly sys.TheVersion is located, I think this would be difficult. Even in a perfect setting (e.g. with symbols available), it's seemingly non-trivial to locate strings and understand where they came from. (I've recently written down some of my findings in relation to constant strings here - strong caveat being that I've never studied the Go compiler in any great depth, so this is pretty much entirely based on analysing binaries.)
Still, worth researching some more. And perhaps another angle to explore is modifying the object files produced during the compile step?

modifying the object files produced during the compile step?

Hmm. Modifying the compiler's output carries the same cost as modifying the compiler's input Go files, though. If we modify the compilation of the runtime packages, we have to recompile everything. That already happens today, but it's only because the build cache doesn't support -toolexec at all. In the future I hope it will.
But if there isn't a nicer way to do this, then I agree it's probably the best course of action.

Still, worth researching some more.

Thanks for the info! That's certainly very interesting. I think it would come in handy for #13 for sure, because we can't mess with import paths during the compilation steps, unless we heavily control the resolution of dependencies, which I would rather not do. So that should happen either before or after link time, that is, with object files or with the final binary.
Do you think object files would be significantly easier? For example, say if you wanted to replace a string in one package. I imagine the very tricky part of final binaries is that they're in different formats (ELF, Mach-O, PE/EXE, ...), and we've already stripped useful bits of info like the symbol table and DWARF.

modifying the object files produced during the compile step?

Hmm. Modifying the compiler's output carries the same cost as modifying the compiler's input Go files, though. If we modify the compilation of the runtime packages, we have to recompile everything. That already happens today, but it's only because the build cache doesn't support -toolexec at all. In the future I hope it will.
But if there isn't a nicer way to do this, then I agree it's probably the best course of action.

If it has the same cost, then indeed it doesn't seem like a worthwhile exercise, given modifying the input should be plenty simpler (I have to admit, I know essentially nothing about how the compiler toolchain ends up caching :)

Still, worth researching some more.

Thanks for the info! That's certainly very interesting. I think it would come in handy for #13 for sure, because we can't mess with import paths during the compilation steps, unless we heavily control the resolution of dependencies, which I would rather not do. So that should happen either before or after link time, that is, with object files or with the final binary.
Do you think object files would be significantly easier? For example, say if you wanted to replace a string in one package. I imagine the very tricky part of final binaries is that they're in different formats (ELF, Mach-O, PE/EXE, ...), and we've already stripped useful bits of info like the symbol table and DWARF.

I sort of get the impression either route could get quite hairy. You're right though, targeting the binary does feel less preferable for the reasons mentioned. I don't know much about the object files unfortunately, something I might go research a bit more (by the way, this was part of the reason why this project piqued my interest - there is a lot of opportunity to learn more about Go internals)


CodePen

Hi!
Created a codepen that makes it easy to get up and running, it also uses an iframe so you can overlay it over your favorite website/image,
http://codepen.io/joshnuss/pen/qdZdjw

Thank you! This is exactly what i was looking for on how to get some things properly going! Have a wonderful day!


Gradient color instead of Single color?

Is it possible to use gradient color instead of a single solid color?

Hey, it's been a long time - I guess you solved the gradient issue by yourself.
Anyway - here, I forked the repo and made some changes so it'll accept two colors for Radial Gradients:
https://github.com/BMarton/android-ripple-background


Is there a way to define a max number of concurrent downloads?

Say I want to download 100 files.
Do I start the 100 downloads and everything else is automatically managed?

As of now, it will start all the downloads.
There is no way to define a max number of concurrent downloads.
But I need to refactor this module, so I will maybe add this functionality at the same time.
Thanks for the suggestion!

That's a great feature request.  +1!

this would help me out too! +1


Cog notebook feature broken

Hi all, great project!
Playing around this, hoping to get a notebook server running via cog. Looking at the docs, I should just be able to run cog run jupyter notebook but I immediately run into trouble. It seems like jupyter isn't in the PATH of the docker container?
cog.yaml

build:
  gpu: true
  python_version: "3.9"
  system_packages:
    - "git"
  python_packages:
    - "torch==2.0.0"
    - "jupyterlab==3.3.4"
    - "transformers"

predict: "predict.py:Predictor"

Output:
Running 'jupyter lab --allow-root --ip=<IP_ADDRESS>' in Docker with the current directory mounted as a volume...
[FATAL tini (7)] exec jupyter failed: No such file or directory

Running 'jupyter notebook --allow-root --ip=<IP_ADDRESS>' in Docker with the current directory mounted as a volume...
[FATAL tini (7)] exec jupyter failed: No such file or directory

Looking at the generated Dockerfile via cog debug > Dockerfile, I'd guess it's because pyenv isn't setup correctly?
...
ENV PATH="/root/.pyenv/shims:/root/.pyenv/bin:$PATH"
...
RUN curl -s -S -L https://raw.githubusercontent.com/pyenv/pyenv-installer/master/bin/pyenv-installer | bash && \
	git clone https://github.com/momo-lab/pyenv-install-latest.git "$(pyenv root)"/plugins/pyenv-install-latest && \
	pyenv install-latest "3.9" && \
	pyenv global $(pyenv install-latest --print "3.9") && \
	pip install "wheel<1"
RUN --mount=type=cache,target=/var/cache/apt apt-get update -qq && apt-get install -qqy git && rm -rf /var/lib/apt/lists/*
RUN --mount=type=bind,from=deps,source=/dep,target=/dep cp -rf /dep/* $(pyenv prefix)/lib/python*/site-packages || true
WORKDIR /src
EXPOSE 5000
CMD ["python", "-m", "cog.server.http"]
COPY . /src

Hi @li-dennis. As you alluded to, the directory where Python libraries are installed isn't included in the PATH. If you want to do cog run jupyter, you'll need to install it another way, like in a run step:
build:
  run:
    - pip install jupyterlab


vm clone depends on deprecated em-winrm gem

https://github.com/chef-partners/knife-vsphere/blob/master/lib/chef/knife/vsphere_vm_clone.rb#L812
The Gem em-winrm has been deprecated, and conflicts with the current winrm gem used by Test Kitchen.
Customization Complete
ERROR: Gem::ConflictError: Unable to activate em-winrm-0.7.0, because winrm-1.7.3 conflicts with winrm (~> 1.3.0)

Thanks, there's some discussion in #247, I need to grab a few minutes to test out the fix.

Since I'm here anyway :) Do you want to try #251 to see if it fixes the problem? I'll try to make sure it can clone a Windows box with no problem tomorrow before I merge.

I can confirm removing the line referenced works to fix the issue. I did the same thing to test a fix before I saw your comments. Nevertheless, I will also try out PR #251 just to make sure there isn't any other issues and get back to you shortly. Thanks!

Fixed in v1.2.9, please update your gem. Thanks for your help in reporting the bug and verifying the fix.


RB3200-ESP32 drill file not rendering within board

Hey, not sure what's going on with this one. Gerbers are from RoboticsBrno/RB3200-ESP32universal. It doesn't seem to be the normal Eagle trouble with zero supression (I tried adding INCH, TZ and METRIC, TZ but neither look right).
Eagle version is 7.7 I believe and CAM job is http://technika.tasemnice.eu/trac/browser/eagle_libraries/Seeed_Gerber_Generater_2-layer.cam
Gerbv and KiCAD Gerber Viewer seem ok with this one.
@yaqwsx can probably give more info if needed.
manufacturing files
drill file
outline gerber
svgs

Don't put a space after the comma. I'm able to get it to render by adding:
METRIC,TZ
(Leading zero suppression, 3 integer places, 3 decimal places)

I would like to just point out that none of the fab houses we used so far had any trouble with the gerbers and I don't like the idea of manually post-processing gerbers in order to supported gerber-to-svg. I think this is a feature/a bug, which should be implemented/fixed in gerber-to-svg.
Eagle export to the Excellon format seems to produce two files - one with hole dimensions (and unit specification) and the second one only with the hole positions.

It's always been just a work around for sure and my plan has been to implement some kind of trailing zero detection on the drill files (at least in pcb-stackup). It's worth pointing out that the fabs normally will have someone step in and make adjustments with errors like this.
The surprising thing for me on this is that KiCAD and Gerbv actually deal with this fine so I will look into how they do that. If they do something more interesting than just assume leading zero suppression then it might be worth copying.
Assuming trailing zero suppression might still be the better option as well, trying to find the cases where Gerbers break then and compare would be worth it even just for my peace of mind (I've only ever seen it break one way, not the other because it's been this way quite a while).

@yaqwsx it's for sure an annoying/unreasonable step to manually edit the drill file. However, the drill file output from Eagle is consistently lacking. Excellon has a mechanism to specify zero suppression (,TZ and ,LZ), and it also "specifies" (in quotes because it's not a real spec) a default of trailing zero suppression. Accordingly, gerber-to-svg will look for the zero command (which does not exist in this file) and then fall back to Excellon default. Could you attach that other information drill file you mentioned so I can see if there's anything there?
A few versions back, the default was swapped, so this file would've worked, but it broke other drill files, so it wasn't really a win.
Zero suppression detection would need to be implemented in gerber-parser. Totally doable with this file, as there are trailing zeros present in a few coordinates that could be detected.
The other option would be, as @kasbah said, to implement something in an orchestration layer like pcb-stackup. gerber-to-svg works on files in isolation, so there's only so much information it can bring to the game.

Tracking zero-suppression detection in mcous/gerber-parser#15

These are the files produces by Eagle using standard CAM file (by SeedStudio), which all the manufactures I met accept with no problems:
The DRI and the TXT file contain the drill information.
RB3200-ESP32universal.TXT

Interesting. There's definitely good information in that DRI file about the coordinate format (standard Excellon in inches is 2:4, this file is called out as 2:5 in the DRI). Still nothing about zero suppression. I have no doubt the fabs are able to handle these files (because they've been dealing with Eagle's problems forever). I, however, don't use (nor like...) Eagle so gerber-to-svg has never been the best at handling its idiosyncrasies.
For reference, here is the drill file header of this Eagle file (// comments added by me). Notice the lack of any format or zero suppression information.
%
M48  // specify header start
M72  // specify inches (implies coordinate format 2:4)
T01C0.01181
T02C0.02362
T03C0.03937
T04C0.04000
T05C0.04331
T06C0.05118
T07C0.08661
%

Now, contrast that with a drill file from KiCad:
M48  // specify header start
;DRILL file {Pcbnew (2013-07-07 BZR 4022)-stable} date 11/9/2013 6:13:08 PM
;FORMAT={2:4/ absolute / inch / keep zeros} // detailed formatting info
FMAT,2 // specify excellon version
INCH,TZ // specify inches and keep trailing zeros (suppress leading zeros)
T1C0.015
T2C0.020
T3C0.035
T4C0.098
T5C0.142
%

@kasbah do you think perhaps an Eagle preset in gerber-parser/gerber-to-svg could help? It'd still require a little bit of user intervention, but less so than putting in the exact formatting options

do you think perhaps an Eagle preset in gerber-parser/gerber-to-svg could help? It'd still require a little bit of user intervention, but less so than putting in the exact formatting options

Wouldn't it be exactly the same as the --zero=L option?

Haha yes... but it would be easier to remember! (I didn't think through it very hard before suggesting it)

Closed via mcous/gerber-parser#15


Remove unneeded class variable

What this PR does / why we need it:
Removal of the unneeded class variable.
Change type

[ ] New feature
[ ] Bug fix
[ ] Unit tests
[ ] Integration tests
[ ] CI
[ ] Documentation
[ ] Auto-generated SDK code

Impacted services

[ ] Workflow Service
[ ] Notification Service

Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.

/lgtm

/approve


Sends the right dimensions to the advanced export dialog

Fixes a problem that prevented sending the right dimensions to the advanced dialog (the problem was that this.options.width and this.options.height was being overwritten in the extend)

cc @xavijam

:+1:


How to access Claims when using Ws-Fed?

Is there any guidance available on how to load/access the Claims once Ws-Federation is configured without Identity?
Seems the options.CallbackPath needs to be configured to an endpoint where the claims will be posted. But there is no guidance on how to implement such an endpoint and how to read user's claims based on the (SAML?) claims received from Idp.

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

ID: 59d6d442-8d46-3f9e-e75a-ad9d900fc117
Version Independent ID: d07c4058-452f-cdac-c59d-8f09e70a43f3
Content: Authenticate users with WS-Federation in ASP.NET Core
Content Source: aspnetcore/security/authentication/ws-federation.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @chlowell
Microsoft Alias: scaddie

@Rick-Anderson
Please share what more info is needed and I'll try to provide.

@Rick-Anderson any update on this request is appreciated.

@halter73 can you please review this? Thanks!


CC is the name of a compiler...

{
  "bin": { "cc": "..." }
}

You realize that cc is the standard link name for the system's C compiler, right? You're going to cause a ton of conflicts when installing this package.

Hello, if you didn't see it there's logic to spawn the underlying cc directly when invoked with arguments:
https://github.com/lovell/cc/blob/07c15eff755f9f4513b05d371c240c293986f672/bin/cc.js#L10-L13
Was this question about real problem you've experienced? If so, are you able to provide steps to reproduce?

I'm not aware of any tooling that is broken by this approach.
I do agree that the naming could cause confusion so let's go with something like cclint for the .bin entry instead, perhaps initially as an additional entry, then we can remove cc in a v2 major increment.

The bin entry is renamed cpplint in v2 to avoid confusion - thanks for reporting this.


can we replace "reachtools convert2" with "samtools fastq" for sam2fastq function?

can we replace "reachtools convert2" with "samtools fastq" for sam2fastq function?

No. samtools fastq does not natively support extracting Paired-Tag cellular barcodes.


added an anchor

There was a broken link on the top of the article linking to an undefined macos anchor. The only reason why I haven't use the automatically generated one is because GitHub and OPS are generating different ones in this case, so I'll open a bug to get this fixed.

LGTM


FIX for #7: Address nested slot issues and more

🚨 Warning: This is a work in progress, intended as a proof of concept for now only 🚨
Primarily addresses #7 by doing the following:

Ensure properly selecting own slots and not that of descendants and
Ensure we're setting up and tearing down state on connect/disconnect, necessary when nested elements render from inside out.
Consistent line endings with semicolons (right now it is mixed, probably due to copy/paste).
This depends on changes from #15

This may necessitate further refactor later. Right now it's still using my own IDE's code style so -- sorry about that. Please consider adopting this but if you'd prefer, I can still reformat to match repo style if desired.  See whitespace ignored version here for cleaner diff: https://github.com/crisward/svelte-tag/pull/18/files?diff=split&w=1

Closing this out in favor of https://github.com/patricknelson/svelte-retag/pull/5 on separate svelte-retag package.


Fix broken build mypy errors

Updates AzureBlobClient._get_metadata return type per discussion in #172
Explicitly silences mypy errors from azure.storage.blob.*. I decided we should implement this workaround because neither of these two following things seem like they're on track to getting fixed.

The Azure SDK typing errors are out of our control, but the upstream issue hasn't had any news in 9 days. https://github.com/Azure/azure-sdk-for-python/issues/20771
There is some kind of bug in mypy, as site-packages errors are supposed to be ignored but aren't. https://github.com/python/mypy/issues/11239#issuecomment-934479522

Closes #172

Thanks @jayqi!


Added suppor to store dat to the DynamoDB database

@matllubos Travis has failed.

@matllubos Travis has failed.


Love Marzipano but...

Not sure if its ok to post this here. But i love Marzipanos as its easy to use. I have older co-workers and they love using Marzipano and creating Spheres. My question is, will there be a software version? I would love to be able to use Marzipano offline without the need of coding.

We don't have plans to implement a native (non-web) version of the Marzipano Tool, sorry. Note, however, that all processing done by the tool occurs locally on the browser. The panoramas are never sent to a remote server.


Merge branch master into wasm-typescript

Sync changes on master with wasm-typescript, resolve merge conflicts. Most changes are unrelated to WasmThemis, but I want to pull dependency bumps that fix npm audit advisories. And speedy Ninja builds for BoringSSL.
TODO:

[x] #846 is merged
[x] This merge is redone, pulling in newer master

Checklist

[x] Change is covered by automated tests
[x] The coding guidelines are followed

[ ] This is a premade merge commit. Do not use GitHub PR UI for merging, just git push once approved.

 So I guess I'll put this PR to draft for the time being, then redo the merge.

I've redone the merge. Now the branch includes newly added examples and tools for WasmThemis.
WasmThemis examples are expected to fail due to backwards-incompatible changes on wasm-typescript branch. These failures will be resolved after this merge. The merge discards changes to tools previously made by @maxammann and pulls in the master version of example code. This will show where the breakage is.
WasmThemis unit tests are expected to pass.

Changed my mind once again and redone the merge. Now all tests are expect to pass, examples are expected to work.
Conflicts in wasm/package-lock.json have been resolved by merging changes from both branches.
There were also conflicts in tools/js/wasm-themis as 'wasm-typescript' branch has introduced some breaking changes in API. I've reconciled the changes between branches, favoring the "broken" API here. I've also updates other WasmThemis examples to use this API.
The plan is to get the CI running, helping with this, and then work on removing backwards-incompatible changes in later commits.

Now all tests are expect to pass

Well, tests are passing..

@vixentael, I don't have a good git fu to teach you on how to review merge commits between branches that both have loads of new commits on them.
“Trust me, I'm an engineer!”
The answer probably is don't let the situation deteriorate to this state 🤷

don't let the situation deteriorate to this state 🤷

in next life probably :)

Well, at least you can check that I've merged the commit I intended to merge, not pushed something completely different.


Update documentation link from HTTP to HTTPS

@hiroyuki-sato Hi, thank you for your pointed out.
Nice catch 😊 I overlooked that... I updated footer link as well 7bd2f9d


practice 7

If I run the code using command: roslaunch practice_7 practice_7.launch tfl_detector:=camera
then after entering destination I will get the following printouts and error message:

[[5000051, 2000585, 1255, 1372, 879, 1024], [5000051, 2000225, 1668, 1890, 932, 1210], [5000051, 2001115, 837, 953, 1090, 1234]]
[[5000051, 2000585, 262, 340, 923, 1020], [5000051, 2000225, 521, 670, 953, 1140]]
[ERROR] [1718657673.127233,<PHONE_NUMBER>.186276]: bad callback: <bound method SimpleLocalPlanner.detected_objects_callback of <__main__.SimpleLocalPlanner object at 0x7f4ba7144250>>
Traceback (most recent call last):
  File "/opt/ros/noetic/lib/python3/dist-packages/rospy/topics.py", line 750, in _invoke_callback
    cb(msg)
  File "/home/edgar/aw_mini_pr_github/src/practice_7/nodes/planning/local/simple_local_planner.py", line 238, in detected_objects_callback
    target_velocity=targetVelocity)
UnboundLocalError: local variable 'targetVelocity' referenced before assignment

[[5000051, 2000585, 1250, 1369, 870, 1019], [5000051, 2000225, 1679, 1909, 916, 1207], [5000051, 2001115, 823, 941, 1085, 1232]]
[[5000051, 2000585, 258, 338, 917, 1016], [5000051, 2000225, 526, 681, 943, 1137]]
[[5000051, 2000585, 1245, 1367, 861, 1013], [5000051, 2000225, 1689, 1929, 901, 1203], [5000051, 2001115, 811, 931, 1081, 1231]]
[ERROR] [1718657673.248394,<PHONE_NUMBER>.287088]: bad callback: <bound method SimpleLocalPlanner.detected_objects_callback of <__main__.SimpleLocalPlanner object at 0x7f4ba7144250>>
Traceback (most recent call last):
  File "/opt/ros/noetic/lib/python3/dist-packages/rospy/topics.py", line 750, in _invoke_callback
    cb(msg)
  File "/home/edgar/aw_mini_pr_github/src/practice_7/nodes/planning/local/simple_local_planner.py", line 238, in detected_objects_callback
    target_velocity=targetVelocity)
UnboundLocalError: local variable 'targetVelocity' referenced before assignment

These need to be fixed.

I think it was because I sent you wrong zip file. Please let me know if there is still problem.

OK, I guess I did mix up something here.


VSC doesn't support git in MSYS2 ?

VSCode Version: Code 1.8.0 (38746938a4ab94f2f57d9e1309c51fd6fb37553d, 2016-12-13T17:45:32.595Z)
OS Version: Windows_NT ia32 10.0.14986
Extensions:

Extension
Author
Version

clojure
avli
0.4.2

toml
be5invis
0.0.3

vscode-eslint
dbaeumer
1.2.2

python
donjayamanne
0.5.5

EditorConfig
EditorConfig
0.3.4

tslint
eg2
0.7.1

vscode-npm
fknop
3.3.0

Go
lukehoban
0.6.51

partial-diff
ryu1kn
0.2.0

RustyCode
saviorisdead
0.19.1

Align
steve8708
0.2.0

language-stylus
sysoev
1.6.0

debug
webfreak
0.14.0

ReactSnippets
xabikos
1.2.0

Steps to Reproduce:

install MSYS2 and pacman -S git.
add %MSYS@_HOME%/usr/bin to Path.
install VSCode and open a project versioned by git.
open the git panel.

Maybe https://github.com/Microsoft/vscode/issues/4651?

Yeah, that's it. Sorry for didn't notice it.


Auto create Text Index

Issue Name
AutoCreateTextIndex
Summary
For datasets a text index should be created automatically or at least be described as part of the deployment documentation

Now documented at
https://scicatproject.github.io/documentation/Devops/MongoDB/

Text index creation now added to boot process (see PR #89 )


Hangs on building sameboy-sys on Linux Mint 21

Building from latest commit on Linux Mint 21

I was able to build it


[CrossPlatformLocation] demo crashes in Delphi DX12.1

The CrossPlatformLocation works in DX11. We are testing the upgrade to DX12.1 using the CPLAndroidGroupD11.groupproj demo.
Using the 3.0.0.jar files (e.g. dw-kastri-base-3.0.0.jar.) we compiled the project as found in Demos/CrossPlatformLocation.
As soon as the app starts on Android it crashes. First time it starts it does request for location permissions, which were granted. We tested on Android v12 and Android v14. Both the 32 bit as the 64 bit versions of the application seem to crash.
Would it be possible to fix this and perhaps add a D12 version?

I expect you did not change the reference for dw-fusedlocation.jar to dw-fusedlocation-3.0.0.jar, however I have added group and project files for Delphi 12.x with all the necessary configuration, and updated the readme to detail what is required to configure your own (i.e. not the demo) project


About offload stage3 source code learning problems

Hello everyone, I want to learn the stage3 part of zerooffload when learning the source code of deepspeed, but I can't find the scheduling process code of the gradient between cpu and gpu, please help me if you know

The gradients are not offloaded in ZeRO. The only parts that can be offloaded are the optimizer states and the parameters.

@lzy-edu, see
https://github.com/microsoft/DeepSpeed/blob/fc4e73370d84af5242996a90b32b3ffce8e6b922/deepspeed/runtime/zero/stage3.py#L1463

@lzy-edu, see
DeepSpeed/deepspeed/runtime/zero/stage3.py
Line 1463 in fc4e733
def partition_grads(self, params_to_release: List[Parameter], grad_partitions: List[Tensor]) -> None:

Thank you for your answer. I would also like to ask you a question about the initial parameter partitioning. When I enabled zerooffload3 during initialization, will all parameters be unloaded to the cpu first? Where is this part


Spike modular structure

@LaughingOutLoud this branch is likely abandoned. Please don't open pull requests for branches containing code not written by you.


Add fee_allowance table

height int
Granter str
Grantee str
Allowance str
expiration timestamp
is_active bool

index on grant, grantee

https://github.com/bro-n-bro/spacebox-crawler/pull/52

Can we decode base64 bytes str to str in the allowance field?


React Native Maps plugin

Create a plugin for react-native-maps with a generator for creating map components.

Done.

Ref #789 too.


compose with opengl

I recently experimented with a desktop application for compose and opengl. A window contains basic controls such as button and image rendering area. Is there any control that can implement opengl image rendering?

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Implement an admission webhook for Catalog resources

If we continue with the approach introduced by https://github.com/operator-framework/catalogd/pull/65 , we should introduce an admission webhook for Catalog resources as mentioned in https://github.com/operator-framework/catalogd/pull/65#discussion_r1198286085 to ensure Catalog resources are meeting the proper requirements for reconciliation.

We should first look to see if we can do this with CEL before starting down the webhook path


Display of the preview image of the EmailPass in the OIDC4VCI flow

need before https://github.com/TalaoDAO/AltMe/issues/2342

The description of the card comes from the issuer metadata


Create ID's for Header elements so they can be referenced in anchor tags

This allows rendered markdown to be compatible when viewed via parsed markdown or GitHub
Closes #710

I was reading how league/commonmark does the slugs, and it turns out to be stripping a lot characters as well, so I sent a PR https://github.com/thephpleague/commonmark/pull/467

I was reading how php-league/commonmark does the slugs, and it turns out to be stripping a lot characters as well, so I sent a PR thephpleague/commonmark#467

Didn't even know about that one 👍

so the tests are failing now because the leading # is no longer applicable.  The question is, should trim the leading hyphen?

Hi @netniV and @Ayesh thanks for the great work you've done towards these changes to arrive at a very sensible slug function :)
The topic of slugs (or header ID's) have come up a few times in the past (e.g.), and the position has previously been that there wasn't a universal choice that everyone using the library would want, so to avoid making the choice for everyone we'd leave it alone and up to extensions.
I've built upon the changes you've worked on here in PR #767, which modifies these changes for the new 2.0.x branch, as well as adding the option for a user-defined slug function (so that anyone who doesn't like the default can specify their own).
@netniV I've listed you as a co-author in the commit to that PR (which I hope is okay?) because most of the work has been done here in arriving at a sensible slug function to use.

Looks good. I added one comment as per our discussions here over str_replace. More involved is your change but I was thinking it should be an optional addition as it could affect other elements due to a duplication of ID
The only other thought is with trimming a trailing and leading hyphen as that seems wrong to keep those to me.

Does this mean this PR should now be closed as it will only be applied to the 2.0 version?

@aidantwoods - is there a timeframe for when 2.x will be released? I'd rather have my users use the auto IDs for headers than enable ParsedownExtra.

@aidantwoods - is there a timeframe for when 2.x will be released? I'd rather have my users use the auto IDs for headers than enable ParsedownExtra.

It would be nice if this PR could be merged into the 1.8.x branch and a 1.8.1 released.  Either that, or have 2.0 released.  At the moment, I can't get these changes through composer without publishing my own version of the package.

I don't think drastic changes like this would be realistically merged to the 1.x branch. What I do is simply extend the Parsedown class, and override the necessary parts. It turned out a good approach long term because I wanted to add some extra markup on my own too.

Let me see if I can manage that.  I did something for images so it may be possible.
Personally, I wouldn't call it a drastic change, but I can see ID conflicts could break things so I see where you are coming from.  I would submit a forked version with it in, but that seems a waste for such a small feature.

Firstly, I think it would be good to have this 1.8.0-beta7 as a full release given there has been no other changes in the past year that I can see (Aug 2020).  Requiring a beta version in a composer.json looks ... well bad to me.  The reason that I need to use the beta version is for handler array usage.
Now that I have fixed on the latest versions, I am still getting a problem which is due to changes in this beta version causing an issue with your parsedown-extra class.
https://github.com/netniV/ParsedownID/issues/1

@Ayesh or @aidantwoods do you have the ability to sort out the current mess of Parsedown vs Parsedown-Extra ?  Due to the beta changes that haven't been published as a version, parsedown-extra relies on older versions.  But to use the newer array-based handler definitions, you need this beta version and parsedown-extra apparently hasn't been updated in two years so can't handle it.

It's been a while, but now very close to a finished product for 2.0 in Parsedown (https://github.com/erusev/parsedown/pull/708) and ParsedownExtra (https://github.com/erusev/parsedown-extra/pull/172). I'd appreciate any feedback you have :)


Easy GroupManager migration path

This needs

Take a look at GM's command structure -> http://wiki.ess3.net/wiki/Group_Manager/Commands -- we need to simplify commands a bit

A good format converter from the GM bukkit format

Be able to handle world inheritance/world-specific stuff in a way compatible to what GM had

For second part: http://convert2gm.chrisgward.com has a converter from PEX to GM, could it be reverse engineered?

well, the 2.0 format is fairly different. I'm probably going to do the conversion mostly by looking at existing files

The GM compatibility interface is being worked on in the extrabackends repository
Useful reference is http://wiki.ess3.net/wiki/Group_Manager/Config
GM appears to handle worlds entirely differently from how PEX both does now and used to do, so that should be taken into account.

With latest commits to PEX and ExtraBackends, GM import is supported. However, due to the fact that GM structures its data quite differently from PEX, users will still have to do some minor changes to the structure of the permissions file to make everything work best with PEX.
There is the possibility of possibly improving the conversion to at least support the default world, which would help a lot of cases.


Add OnDialogEnded trigger

Salem from PVA wants this.

@cwhitten we have OnEndOfActions trigger. Would it suffice for the scenarios you need this for?

@garypretty , can you give us an indicator of the priority of this item? Thanks.

Does OnEndOfActions work for this?  (as vishwacsena mentioned)

@mrivera-ms let's confirm if @vishwacsena's suggestion works.  Either way, let's ensure this get's addressed as part of R11 given this is a requirement for PVA.

@cwhitten , could you please let us know if OnEndOfActions would work for the customer?

@sbacha What do you think?

Just to understand the scope of OnEndOfActions ?  Is it global on the bot or per dialog ?
I don't see it exposed in composer yet ...

@cwhitten , can we expose OnEndOfActions in Composer first and see if this satisfies the requirements?

@mrivera-ms The SDK team should still be able to answer Salem’s question, right?

Just to understand the scope of OnEndOfActions ? Is it global on the bot or per dialog ?
I don't see it exposed in composer yet ...

Hi, @sbacha, this OnEndOfActions is trigger and it's scope is per dialog. Actually in AdaptiveDialog, all triggers must belong to a certain dialog and thus all trigger can be perceived as a per dialog level thing.

@sbacha
Here is an example of using OnEndOfActions to do something at the end of per dialog. OnEndOfActions In Dialog
If you have any question or concern, feel free to leave comments.

@sbacha and @cwhitten , is this issue still needed or the proposed solution works? Please let me know so we can close this issue.


Sing! feature doesn't work

I installed Spotify Web Player for Linux (Ubuntu 16.04 LTS). Didn't receive an error. Tried the Sing! feature in a song and received the following error:

Went to the specified directory and discovered that LyricCache folder wasn't auto generated. After I created the folder manually, everything worked fine.

Yes I noticed this a day or two ago, I must have accidentally forgot to put the generating folder function in whilst adding the Sing! Sync functionality in. I think there's also a problem with new synchronisations where they don't get uploaded now due to moving some web scripts around. I'll come back to this sometime today/tomorrow

Whilst the synchronisations upload ability has been fixed it might be a bit of time before I get the chance to re-upload another release


Istio 1.1 compatibility with APIGee private cloud

After following steps in https://github.com/apigee/istio-mixer-adapter/issues/249 I took binary from 1.0.6 release and yaml files from https://github.com/apigee/istio-mixer-adapter/tree/master/samples/apigee/grpc. Though things seem to go fine but after I try creating an API Product I don't see Istio Services there as in screenshot below. Am I missing something?

I had to update as mentioned in https://community.apigee.com/articles/65504/configuring-istio-mixer-adapter-with-opdk-418xx-an.html where I had to change hardcoded HTTP Target.

Closing the issue as this was due to older version of apigee


createDir method doesn't comply to the docs/intent behavior

the method https://flysystem.thephpleague.com/v1/docs/usage/filesystem-api/#create-directories
as described recursively creates a path.
So the result of calling createDir on any adapter i.e. $filesystem->createDir('foo/bar/baz') should be identical of mkdir -p foo/bar/baz (in the bash) while the google drive adapter ends with an error:
>>> $filesystem->createDir('foo/bar')
Google\Service\Exception with message '{
 "error": {
  "errors": [
   {
    "domain": "global",
    "reason": "notFound",
    "message": "File not found: foo.",
    "locationType": "parameter",
    "location": "fileId"
   }
  ],
  "code": 404,
  "message": "File not found: foo."
 }
}
'

Did you try masbug/flysystem-google-drive-ext?
The configs are the same and do the expected behavior

Did you try masbug/flysystem-google-drive-ext?
The configs are the same and do the expected behavior

Indeed I did! Although my first choice was nao-pon package (more :star: I guess :)). masbug package works as expected in this regard. Thanks for sharing and reply!

@mpdx maybe you can give a ⭐ for the future


document steps involved in doing a release

This should go in CONTRIBUTING.
Off of the top of my head:

update the changelog; give the current [unreleased] section a version, update the 'last release' note, add a new [unreleased] section
update readme to point to most recent version (before publishing crate, so that the readme on docs.rs is up to date)
bump versions of druid, druid-shell and druid-derive crates
update our own dependencies to their most recent versions, where feasible
publish to crates.io; for druid-derive we need to do --no-verify because it depends on druid which won't be released yet
update docs to use new version in examples: grep for the current version. This will be in the book, but also in the readme and in various docstrings.
update screenshots in readme: this means taking new screenshots, and pushing them directly to the screenshots branch, and then linking to that from the readme.
push a new tag and do a release on github

... that's all I have off the top of my head.

The current CONTRIBUTING.md has most of this.
Currently missing:

Publishing to crates.io (the --no-verify trick in particular)
Updating version numbers.
Updating screenshots.
Add a tag and release on github.


Add bit-shuffle encoding page

Parent Issue #1305
Bitshuffle is efficient encoding method for highly correlated data. It has relatively high compression ratio and high throughput.
I want to add a bit-shuffle encoding page for new file format.
reference: https://github.com/kiyo-masui/bitshuffle.git

pls refer to this PR #1304


Allow wordpress.org favorites username to be configured

https://developer.wordpress.org/cli/commands/user/meta/

Added in 7bd8b9df87b5a9108eda2df1c20d102fdda380bb


Fix playground

Fix the broken playground
as per https://github.com/strapi/strapi/issues/13126

@bidoubiwa
I added empty folders in /playground : public/uploads. This is required by Strapi when launching the development server.
I also updated the steps to launch the playground in the README.md and CONTRIBUTING.md

bors merge


ability to change "SERVICE_" prefix?

What do you think about providing runtime option for changing SERVICE_ env or label prefix?
SERVICE_NAME sometimes feels too generic and thus there is a possibility that it may collide with other tools.

This would be interesting for me as well.  The reason is that I would like to use labels instead of environment variables.  According to the docker documentation, an underscore is not a legal character for a docker label (though it does seem to work because I don't think they enforce it).  The problem is that I wouldn't want to rely on that behavior, and there are some tools (AWS) that already make it difficult to do that because of the checking they do.
It would be nice to be able to change it to something that might make more sense for us.

I have the same problem. I use service name for others configurations and SERVICE_NAME causes a lot of confusion.

I would like to deploy registrator with the ability to register some ports using the external IP and other ports using the internal IP.  With -service-prefix, two copies of registrator could be deployed with different -service-prefix values.  Applications could then just set the appropriate environment variables to get registered with the appropriate IP.


matcha-gtk-theme: 2021-01-12 -> 2021-02-04

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/vinceliuice/matcha-gtk-theme/releases.
meta.description for matcha-gtk-theme is: "A stylish flat Design theme for GTK based desktop environments"
meta.homepage for matcha-gtk-theme is: "https://vinceliuice.github.io/theme-matcha"
meta.changelog for matcha-gtk-theme is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

directory tree listing: https://gist.github.com/71bdc3f0f6f12be7f9afa22a9527f25e

du listing: https://gist.github.com/bb56baa5a2380253af597578d564d4ff

Rebuild report (if merged into master) (click to expand)

994 total rebuild path(s)

342 package rebuild(s)

341 x86_64-linux rebuild(s)
299 i686-linux rebuild(s)
54 x86_64-darwin rebuild(s)
300 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
_9pfs
acd-cli
adapta-gtk-theme
adbfs-rootless
aefs
afflib
afpfs-ng
afuse
almanah
android-file-transfer
apfs-fuse
appimagekit
apt-cacher-ng
archivemount
avfs
backintime
backintime-common
backintime-qt
bareos
bat-extras.batdiff
bindfs
blobfuse
borgbackup
boxfs
btfs
bubblemail
buildah
buku
bup
calls
cantata
casync
catfs
ceph
ceph-client
ceph-dev
chrome-gnome-shell
chromium
chunkfs
cinnamon.cinnamon-common
cinnamon.cinnamon-control-center
cinnamon.cinnamon-gsettings-overrides
cinnamon.cinnamon-screensaver
ciopfs
clementine
clementineUnfree
cni-plugins
cromfs
cryfs
curlftpfs

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8v085bl8a4hfv3lnfhf1h5xpyr5kxbv4-matcha-gtk-theme-2021-02-04 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A matcha-gtk-theme https://github.com/r-ryantm/nixpkgs/archive/44bcd6773ad14437e5dce458cc9a52f23634e580.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8v085bl8a4hfv3lnfhf1h5xpyr5kxbv4-matcha-gtk-theme-2021-02-04
ls -la /nix/store/8v085bl8a4hfv3lnfhf1h5xpyr5kxbv4-matcha-gtk-theme-2021-02-04/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @romildo for testing.

Tested & working on xfce


Error: Can't subset columns that don't exist. x Column link doesn't exist.

Hi,
Thank you for an interesting and useful package.
I am trying to replicate the tutorial http://coornet.org/index.html on the dataset I collected from Crowdtangle.  successfully run the code and got the results till Top URLs section but I am getting below error when I am trying Top Shares section.
When I run this line top_shares_per_components <- get_top_coord_shares(output, order_by = "engagement", component = TRUE, top = 10). I got the error

Error: Can't subset columns that don't exist. x Column link doesn't exist.

I don't understand why this section is having problem/error when Top URLs section is running smoothly?
Can you please help solving this issue?
Thank you in advance!

Hi :) thanks for pointing out this problem. We have identified and correct the error. Please update CooRnet to the latest version, restart r and let me know if it works.
Best, Fabio

Thank you Fabio. Problem solved.
I am closing the issue.
Thanks all who worked on creating this wonderful package.


bug: IonSelect doesnt show selected placeholder sometimes

Bug Report
Ionic version:

[x] 5.x
Current behavior:

If IonSelect is preselected, it doesnt shows the content of IonSelectOption sometimes. This happens randomly. If I open and close IonSelect, it shows the placeholder again. In the DOM I can see that everything is fine.
Expected behavior:

It should show the Content of the Option Element which is selected.
Steps to reproduce:

Cannot reproduce. Just try reloading, and hope.

The Bug happens if an AJAX Call takes a little bit longer. I could simulate an AJAX Call which takes a second (with setTimeout) and there you can see the bug: https://codepen.io/BerkeAras/pen/vYLLqMp

Thanks for the issue. I am going to close this as a duplicate of https://github.com/ionic-team/ionic/issues/19324.


Directline API returning 500 internal server error in Azure

Bot Info

SDK Platform: Node.js
SDK Version: 3.14.1
Active Channels: Skype, Direct Line (Webchat)
Deployment Environment: Azure Bot Service

Issue Description
Getting different errors from directline API endpoint today. The bot is not performing and frequently stops responding to user's request.
Please find below one of the error log entry. We are running the bot in production for a few months now and are observing such errors for the first time. Please help.
POST to 'https://directline.botframework.com/v3/conversations/4****************/activities/4******************************8' failed: [500] Internal Server Error
at Request._callback (D:\home\site\wwwroot\node_modules\botbuilder\lib\bots\ChatConnector.js:647:46)

@vishmonne if you still require assistance, please share the requested information so we can investigate further.
Closing due to no response.


Fix for empty games list (Player::GetOwnedGames)

In case a player has no games, "games" key is not present and Exception is thrown.

Good catch.


renamed according to ocrd/constants.py

@kba @VolkerHartmann  Only a minor change, but prevents from continuing with ocr_kraken.

Fixes #61

I see. So a make call would fix my problem?

Not the right way!


Nocodb

@ScriptSlayer0 hey would you mind bumping issue 9344 over at nocodb?

I liked the idea, nocodb looks interesting.

Ifnyou can add a +1 in the comments yo that would be great to show them others are looking


[ADD] Suggestion

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Fix BG Masthead Image, on home-screen.
Describe alternatives you've considered
Uhh... keeping the picture that is currently on there..... but..🤢🤮(what I think about it)
Additional context
I think it would make the site more unique.
And I am currently working on a picture that could be used, so when I finish it, I’ll add it to here, and let me know what you think.

Alright.   ... thanks dsfas0976

Alright.   ... thanks dsfas0976


Deleted users continues to have access to resources.

Rancher versions: v2.0.0-alpha16
Steps to reproduce the problem:
As admin user , Create a cluster - c1.
Create a standard user1 - user1.
Add user1 as "cluster-owner" for cluster c1.
Using kube config generated for user1 , add to create namespace.
As part of deleting user1 , all the clusterRoleTemplateBindings for this user does not get deleted.
Cluster1 still references cluster role bindings for user1.
Using kube config generated for user1 , we are still able to get access to existing namespaces in this cluster c1 and also able to create new namespace.
Expected behavior:
As part of deleting user1 , all the clusterRoleTemplateBindings for this user should get deleted.
Using kube config generated for user1 should not provide access to any resource in cluster c1.

Tested with master from March 1st. Still see the issue above.
As admin user , Create a cluster - c1.
Create a standard user1 - user1.
Add user1 as "cluster-owner" for cluster c1.
Using kubeconfig generated for user1 ,  create namespace.
Delete user1 from the global level by logging in as admin
As part of deleting user1 , all the clusterRoleTemplateBindings for this user do not get deleted.
Cluster1 still references cluster role bindings for user1.
Through the user1's kubeconfig we can still create a new namespace and create more pods in the namespace

Tested with latest build from 2.0 master:
When trying to access a deleted user (who was a cluster owner previously), following error message is presented:
Error from server (InternalError): an error on the server ("{\"Cause\":null}") has prevented the request from succeeding```

Reopening this issue to see if better error message can be provided to the user.


Generated C# files, unsupported by Unity

Describe the bug
Feature 'not pattern' is not available in C# 8.0. Please use language version 9.0 or greater;
'NotNullAttribute' is inaccessible due to its protection level;
To Reproduce
Steps to reproduce the behavior:

Generate C# files from .bop
Go to 'Unity'
Use generated files
See compile errors

Expected behavior
Normal working code, without errors
Bebop info:

Version: 2.2.2
Runtime: .NET

Desktop (please complete the following information):

OS: Windows
Version: 10

Additional context
I am trying to use bebop on Unity along with a Node.JS server. On the server side, everything works, but unfortunately the generated C # files use syntax not supported by the Unity (Negated not patterns) and attributes (NotNull, DisallowNull, MaybeNull, AllowNull)

I can probably address this today with a language version flag on the compiler

That would be great

So I've fixed the codegen but there are still some manual steps to actually get code to successfully compile. I'm going to look into a writing a Unity plugin


Auto-Tunnel does not work on wired connections

Just ran into this on my Shield TV. This app sounds like it solves many problems if it works for Android boxes.

Just ran into this on my Shield TV. This app sounds like it solves many problems if it works for Android boxes.

Currently working on a fix for this. It should be available in the next release.


Get value with the enum key

In previous versions of simple_enum (e.g. v1.6.9), when I had some code like this:
class MyEntry
  include Mongoid::Document
  include SimpleEnum::Mongoid

  as_enum :status, %i{unresolved resolved}, prefix: true

It was possible to execute this kind of code:
if @my_entry.status == MyEntry.status_resolved

In newer versions ( => v2.0.0 ) this code raises an error
NoMethodError: undefined method `status_resolved' for MyEntry:Class

It was very useful to have this class methods to avoid the programmer to have typo errors.
In the version notes it says that now we must use MyEntry.statuses[:resolved], but this won't avoid typo errors of this kind:
if @my_entry.status == MyEntry.statuses[:resolve]

Note that I have introduced a type error (resolve instead of resolved) but the code won't raise any error. Also, if some day I change this status name, it will be very dangerous to repair all the code. With last version was easier, because after changing the name I will get a raise error in every mistake.

Yes, for this purpose we have added MyEntry.statuses.fetch(:resolve) which raises an exception (like Hash.fetch) when the key does not exist. Does that work for you?

yes, thanks!

You are welcome :+1:


OPCT-213: Update base image for plugin provider-cert

Fixes:

Replace plugin image to UBI, resolving bugs related of Go 1.20 binaries (glibc dependencies) and Alpine (details in https://issues.redhat.com/browse/OPCT-213 ) [1]
JQ new URL path

Updates:

oc client to 4.13-based

[1] Plugin error when running Go binaries built with Go 1.20 in OCP 4.14
2023-06-13_23:57:01 | [plugin] | ./global_fn.sh:207>  [extractor_start][openshift-tests] Extracting the utility
2023-06-13_23:57:06 | [plugin] | ./global_fn.sh:284>  [extractor_wait][openshift-tests] Check files exists=[/tmp/shared/openshift-tests.ready /tmp/shared/openshift-tests.failed]
2023-06-13_23:57:11 | [plugin] | ./global_fn.sh:284>  [extractor_wait][openshift-tests] Check files exists=[/tmp/shared/openshift-tests.ready /tmp/shared/openshift-tests.failed]
2023-06-13_23:57:11 | [plugin] | ./global_fn.sh:213>  [extractor_start][openshift-tests] check if it was downloaded
2023-06-13_23:57:11 | [plugin] | ./global_fn.sh:221>  [extractor_start][openshift-tests] move to /tmp/shared/openshift-tests
2023-06-13_23:57:12 | [plugin] | ./global_fn.sh:224>  [extractor_start][openshift-tests] set exec permissions for /tmp/shared/openshift-tests
2023-06-13_23:57:12 | [plugin] | ./global_fn.sh:231>  [extractor_start][openshift-tests] testing openshift-tests
Error loading shared library libresolv.so.2: No such file or directory (needed by /tmp/shared/openshift-tests)
2023-06-13_23:57:12 | [plugin] | ./global_fn.sh:180>  Creating failed JUnit result file [/tmp/sonobuoy/results/junit_e2e_failed_20230613-235712.xml]

Tests executed with success with plugin version dev20230613205136
$ ~/opct/bin/opct-linux-amd64-v0.4.0 run -w --plugins-image openshift-tests-provider-cert:dev20230613205136

...
Tue, 13 Jun 2023 22:49:23 -03> Global Status: running
JOB_NAME                           | STATUS     | RESULTS    | PROGRESS                  | MESSAGE
05-openshift-cluster-upgrade       | complete   |            | 0/0 (0 failures)          | waiting for post-processor...
10-openshift-kube-conformance      | complete   |            | 374/377 (3 failures)      | waiting for post-processor...
20-openshift-conformance-validated | running    |            | 652/3702 (8 failures)     | status=running=T/C/P/F/S=3702/652/609/8/35
99-openshift-artifacts-collector   | running    |            | 0/0 (0 failures)          | status=waiting-for=20-openshift-conformance-validated=(0/-3050/0)=[12/1080]
...

/assign @rvanderp3

Still validating the image, the collector plugin is broken as that image is missing dependencies (tar command)
/hold

/retest

/unhold


Proxy support?

Under the hood it seems is running httpx (i might be wrong?)
How can we set a proxy on client creation?

Yes it is httpx. I think you can implement your own request class with anything what you want using RequestBase. And then pass your request instance to the constructor of the client. Like request=myReq

Modifying the class directly did the job.
class AsyncRequest(RequestBase)
 def __init__(self, proxy_url: str = None) -> None:
        super().__init__()
        # Configure the AsyncClient with or without a proxy
        self._client = httpx.AsyncClient(
            follow_redirects=True,
            proxies={"all://": proxy_url} if proxy_url else None
        )

Worked this way with no issues:
import asyncio
from atproto import AsyncClient
from atproto_client.request import AsyncRequest
async def flow():
    proxy_url = "http://user:pass@host:port"

    custom_request = AsyncRequest(proxy_url=proxy_url)

    client = AsyncClient(request=custom_request)

    is_logged = await client.login(login="your_username", password="your_password", session_string="session")
    print("Login Response:", is_logged)

    post = await client.send_post(text="Hello World!")
    print("Post Response:", post)

asyncio.run(flow())

Question is, is there more efficient way to do it?


Added support for pre-release tags in $VersionRegex

The task should support pre-release tags as per NuGet versioning documentation.
Without this, all of the packages published with this task will appear in Latest stablein NuGet Package Manager.

Hi @codepic, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

I'm not sure if this is the best way to Support the pre-release tag. It's Kind of Special for NuGet but there could be other Tasks that cannot parse this new Version Format without being modified.
My Suggestion would be to let the Version-string as-it-is and add a new Checkbox and text-Input for the prefix-string. If the Checkbox is checked in the task-configuration, the prefix-string is added before the last Version-Digit and the last dot is replaced by a dash.
This would enable us to create pre-release NuGet packages without changing the Version-Information globally.

Probably not the best way. But I doubt it'd break anything as Build vNext defaults to YYYYMMDD.r and is very much open to how you want to lay out your versioning scheme. In fact, as far as I know, you are forced to add variables into the build definition in order to support Nuget Packages as Nuget follows the Semantic Versioning. For which reason I think the Nuget Packager Task should also follow the aforementioned scheme.
While the RegEx I provided doesn't fully conform to the SemVer spec, it at least allows packaging prereleases (alpha1, alpha2, beta1, rc1, rc2 and so forth) which the current master version does not. This is a huge shortcoming for companies like us who deliver development libraries to devs through MyGet and want to distinct between prerelease and release libraries. Especially now that VSO started supporting Release Management vNext.

@codepic , Pre-release tags do not have to end in a digit. For instance, from the NuGet docs, 1.0.1-alpha is a valid version, but your regex doesn't support it. Also, you can have a 4-segment numerical version in NuGet with a pre-release tag, such as <IP_ADDRESS>-alpha.

@jkeech yes, NuGet supports f.ex. <IP_ADDRESS>-alpha but it's not according to the Semantic versioning which NuGet suggests we'd be using. All of the following are valid according to the semantic versioning.
0.0.1
0.10.0
v1.0.0
0.0.0-foo
1.2.3-4
2.7.2+asdf
1.2.3-a+b.c.10.d.5
2.7.2-foo+bar
1.2.3-alpha.10.beta.0
1.2.3-alpha.10.beta.0+build.unicorn.rainbow

But I think we got a little side tracked here. What I suggested in my pull request was to add support for prerelease tags. Just to be able to make a prerelease in NuGet. Not the full spec of SemVer or full support of NuGet versioning which goes even beyond the semantic versioning spec.
With the current tasks it is not possible to make a prerelease in NuGet at all. All of the packages created are considered as `stable´.
Now, I don't think it would be even reasonable to implement a full spec. As @Anheledir pointed out, it could break other tasks. For example setting assembly version which won't support characters at all.
I deliberately only added optional dash and letters before the last digit in the original regex so that it will be still supporting the old versions and it'll be relatively easy to just ignore characters and replace a potential dash with a dot and come up with a full version number with major, minor, patch and revision.
Say, I have build number format as: $(Major).$(Minor).$(Patch)-ci$(rev:rr)
-> remove characters, replace dash with dot, you got $(Major).$(Minor).$(Patch).$(rev:rr) which is compatible with assemblyVersion and it'll keep incrementing on every build.
Personally I don't like the idea of having to specify the version number separately in the task itself as people tend to forget doing it and then you got to clean up. I like to specify major, minor and patch in build variables and let the build system increment the revision.

See PR #1253 for the approach I'd rather take. This PR is stale and out of date with the current versioning regex anyhow.

@vtbassmatt Thanks for taking this effort. I'll try to find some time to pull your changes and have a few tests with it. On the surface when looking at the diffs it looks good as we're not anyways trying to implement the full semver but in practice providing prerelease-capability. I'll have to run a couple of tests on our environment how it works out in practice. Hopefully prior to Friday but more probably prior to the end of next week.
But in any case. Your approach seems much more solid than my crude brute force approach.
We'll just have to see how it plays out in reality. And to do that, I need to chain up the whole pipeline from VSO to CI Builds to MyGet upstream.


Fail setup on no CUDA environment

After this PR is merged, Travis only checks if CuPy is built without errors (sdist) and passes style checking.

Note: it looks better to remove appveyor from CuPy because there is no CI test that makes differences between environments.


Parameters in cross-stitch units

Hi, it 's lucky to find your implementation for Cross-stitch units. Good job.
Well, in the original paper, I think the cross-stitch matrix is shared by each channel or each layer.  However, in your code implementation, I think your version is more like pixel-wise cross-stitch units, especially for convolution layer. Maybe it will introduce too many extra parameters.

Sorry for the late reply.
Thank you for your hint. I could be wrong.
You could update the code if you like.
Thanks

Hi Helloyide, hi TowardSun,
I also stumbled across your code while trying to implement Cross-Stitch units in tf.keras and agree with TowardSun's comment.
As far as I can tell, Misra et al intended the cross-stitch matrix to be of size [num_tasks, num_tasks], so for two tasks, it would be like alpha=[[alpha_aa, alpha_ab], [alpha_ba, alpha_bb]. Thus, in the cross-stitch unit, for every pair of feature "pixels" x at position ij, x_ij=[x1_ij, x2_ij], the unit calculates hat{x}_ij = alpha * x_ij, which is [alpha_aa * x1_ij + alpha_ab * x2_ij, alpha_ba * x1_ij + alpha_bb * x2_ij].
This has the effect that after the CS operation, every feature pixel hat{x}_ij in task t is a linear combination of the corresponding feature pixel in each task. So alpha_aa weights how much information from task a should be included in the output for tasks a, while alpha_ab weights how much info from task b to add.
What your code does is basically adding a fully-connected unit between the two task layers (without biases), which is of course a very valid (and very interesting, as I haven't come across any such publications!) approach, but gives very different results for the application at hand.
When I have finished my implementation in keras, I will share it here so you can take a look.

Hello,
as promised, I have shared my implementation at https://gist.github.com/LtSurgekopf/659231ce03eed4579f203577eed99c6c.
I don't have any results so far, but I am fairly confident that my implementation is valid.
Please note that I had to transform the matrix-vector multiplication into scalar multiplications and additions, as otherwise, my machine would not be able to carry out the computations required due to the number of hidden features I am working with. This implementation is seemingly easier on the hardware.


Multiple errors when copy pasting code

[x] This is a v1.x issue (v0.x is no longer maintained).
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior
No error is thrown when copy pasting code from the documentation.
Current Behavior
Three errors are thrown.
1.)
const { classes } = this.props;
Error:(47, 17) TS2459: Type 'Readonly<{ children?: ReactNode; }> & Readonly<{}>' has no property 'classes' and no string index signature.

2.)
SimpleModal.propTypes
Error:(74, 13) TS2339: Property 'propTypes' does not exist on type 'typeof SimpleModal'.

3.)
const SimpleModalWrapped = withStyles(styles)(SimpleModal);
Error:(79, 39) TS2345: Argument of type '(theme: any) => { paper: { position: string; width: number; backgroundColor: any; boxShadow: any;...' is not assignable to parameter of type 'Record<"paper", CSSProperties> | StyleRulesCallback<"paper">'.
  Type '(theme: any) => { paper: { position: string; width: number; backgroundColor: any; boxShadow: any;...' is not assignable to type 'StyleRulesCallback<"paper">'.
    Type '{ paper: { position: string; width: number; backgroundColor: any; boxShadow: any; padding: number...' is not assignable to type 'Record<"paper", CSSProperties>'.
      Types of property 'paper' are incompatible.
        Type '{ position: string; width: number; backgroundColor: any; boxShadow: any; padding: number; }' is not assignable to type 'CSSProperties'.
          Types of property 'position' are incompatible.
            Type 'string' is not assignable to type 'PositionProperty'.

Steps to Reproduce (for bugs)

Just copy-pase the contents of https://material-ui.com/utils/modals/ and it should instantly

Your Environment

Tech
Version

Material-UI
v1.0.0

React
16.0

Result of yarn list: https://paste.fedoraproject.org/paste/W-wNyIjgk8iuMm2hZk6z3g

Okay, it seems like all problems have been solved after I found this one:
https://material-ui.com/guides/typescript/
Sorry for bothering.

Okay, I found this documentation https://material-ui.com/guides/typescript/ which solves the first two problems!
However, I still get the 3rd error, saying that position has to be a PositionProperty.

No error is thrown when copy pasting code from the documentation.

@tzfrs This can't work. The demos are written in plain JavaScript. They will almost always require being typed in order to compile with TypeScript. I would encourage you to learn TypeScript or not to use TypeScript if you don't have time to spend into the learning curve. Both way, I don't think this issue can be leverage, I'm closing.


middleware: proxy

Fix proxy to work with all the features.
I'll probable leave adding EDNS0 "headers" out of the picture.

When proxying we need to copy any EDNS0 buffer size headers:
EDNS: version: 0, flags:; udp: 512

is wrong.

We also don't allow for port numbers other than 53

EDNS0 stuff should be fixed with #44


Fix URL Encoding for Upsert External ID

Adding URL encoding to the external ID used in a REST upsert call, as this can include "/" which will interpret as additional path without being encoded.

Hey, sorry this didn't get reviewed in time, we aren't doing feature work on master so I'll close this
ref: https://github.com/jsforce/jsforce/issues/1069


Update futures-rs to 0.3

https://github.com/rust-lang-nursery/futures-rs/blob/master/CHANGELOG.md#030---2019-11-5

https://github.com/rust-lang-nursery/futures-rs/blob/master/CHANGELOG.md#031---2019-11-7 is released


Is there a React / ES6 implementation?

Hi there,
I'm trying to set up KaTeX in react-16 using ES6. However I can't get it working. Also the react-katex repo is poorly maintained, so I'm currently kind of stuck. How to integrate katex rendering into react-16?

@sjorsvanheuveln does React 16 still support dangerouslySetInnerHTML?

@kevinbarabash Yes

There is a react component that is developed and used by Khan Academy in Khan/react-components/js/tex.jsx (Demo).

We should probably just add that to the README.md.

I didn't see anything particular wrong with https://github.com/talyssonoc/react-katex.  It hasn't been updated in a while, but it would only need updating if a person need access to recently added KaTeX options.  Although https://github.com/Khan/react-components/blob/master/js/tex.jsx has been updated recently it doesn't expose any of the new options either.

@kevinbarabash I think dangerouslySetInnerHTML it is still supported, but that react-katex is giving me all kinds of problems. Maybe I have some conflicting packages or something, but that would be weird. I'll check that tex component, maybe that will solve all my problems. Thanks.

How do I include this now? I have to use git submodules to install react-components? Can't I install through npm?

It looks like it's not handling the JSX.  You're babel config is probably excluding all files in node_modules.  You'll have to adjust the regex to all files in react-components to be transformed.

Hmm, when I compare all the other NPM packages to this package I see that most of this is in JSX and all these other packages are JS only. This package seems to be an exception. Is this built in another fashion or something? Also I cannot just simply do this after installation:
import { Tex } from 'react-components';
It says the path is unresolved but the folder clearly exists in node_modules.

@sjorsvanheuveln that package is poorly constructed.  It doesn't defined a main entry point.  It really should have an index.js and multiple packages for each individual component if we wanted to get fancy.  Sorry about all of the issue with that package.

No worries, is that on the planning though? Is it hard to make it like that?

@sjorsvanheuveln None of the authors of that package are developers of KaTeX. I'd suggest posting an issue there, or someone could send in a PR...

I did, but no response. Btw nice lectures on YT man! Learned some stuff from you.

@sjorsvanheuveln we have made react-latex and it works for us! Check it out https://github.com/zzish/react-latex

@Jonnygwi How can I change the setup so that the css is already included in the package and doesn't need to be loaded externally?

@sjorsvanheuveln That is something we were thinking of adding to the project. At the moment you can either download the css or link to the cdn and include it into your template. You should not need to touch your webpack.

If react-latex works well, we should probably link to it in the KaTeX README next to ng-katex.


Add support for plugin testing

Codecov Report
Attention: 8 lines in your changes are missing coverage. Please review.

Comparison is base (e6fd89c) 74.75% compared to head (5203adc) 75.20%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #152      +/-   ##
==========================================
+ Coverage   74.75%   75.20%   +0.44%
==========================================
  Files          34       34
  Lines        1616     1613       -3
==========================================
+ Hits         1208     1213       +5
+ Misses        408      400       -8

Files
Coverage Δ

src/ragger/conftest/configuration.py
100.00% <100.00%> (ø)

src/ragger/utils/__init__.py
100.00% <100.00%> (ø)

src/ragger/utils/misc.py
52.63% <85.71%> (+3.37%)
:arrow_up:

src/ragger/conftest/base_conftest.py
82.60% <40.00%> (+6.04%)
:arrow_up:

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


CI: initial rework of CI pipelines

Changes:

Moves previous Drone "pre-checks" pipeline to GHA.

Runs in parallel with Drone.

Removed misc. steps from pipelines that are no longer used or necessary for CI.
Combines NCTL compiling to a single wrapper for maintenance.
Brings in the use of cachepot which is paritys fork of sccache

Reviewer Notes:

The diff of .drone.yml might be tough to look at

bors try

bors try

bors try

bors try

bors r+


Fuzz FALCONN

See https://github.com/google/fuzzer-test-suite/blob/master/tutorial/libFuzzerTutorial.md

What is this for?

Make sure that FALCONN doesn't crash in bad ways when we get incorrect input.


Add files via upload

Adding all things open image, so we can use it for event landing page.

Looks like we got the image as part of #698 so I'm going to close this PR


Minor refactoring after https://github.com/istio/istio/pull/36639/

Address some followup comments
Please provide a description of this PR:
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[ ] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Please check any characteristics that apply to this pull request.

[ ] Does not have any user-facing changes.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all


Add my file

New file added

Created branch, added file and opened pull request

Re-opened pull request


My newly purchased 5,5mm Flat Driver no longer fits in this: Centerpieces/Tool_hooks/2x5 Wera Kraftform Plus 160i-6.stl

This was a set purchased in 2022 from Amazon US. Specifically, this set:
https://products.wera.de/en/tools_for_electricians_kraftform_plus__series_100_vde_160_i_6_rack.html
The 5.5mm flat driver no longer fits in the supplied location on this STL: Centerpieces/Tool_hooks/2x5 Wera Kraftform Plus 160i-6.stl
All other drivers fit exactly as expected.
I'm not able to find any dimensional specifications for that specific driver, but I measured this with my caliper:
shaft near top: 8.9mm
handle base (hexagon edge-to-edge): 23.0mm (a hair under)

Looking to either close or resolve this issue - the model as provided fits the measurements you indicate, which also match the driver.
Here's a look at that model, the hex cutout is 23.4mm dia and the shaft cutout is 9mm dia: https://i.imgur.com/4UqmA5k.png
On the off chance that you're still watching this, go ahead and respond if this is still an issue and I'll re-open.  Sorry for the late response!


Suggestions

-> reduced gc by caching arrays and temporary texture
-> reduced operations of calling vector normalized from ray constructors and assignments to ray direction
-> reduced vector operations

-> fixed realistic flow off by one error when addressing fluid cell index
-> reduced number of process when add force to boundary

-> enable the save flowmap texture feature


modify rest api test interface for unit test

add a chain param in restfultest for unit test
set 2 methods in router: httpRequest2Invocation and invocation2HTTPRequest public

Can one of the admins verify this patch?

Coverage decreased (-0.08%) to 66.063% when pulling 0ce1d5576fc5ea3ae11148ea6e95595540c190fb on liuqi04:master into 2aa87d1066dac428cd9f066a4c856261eb5e5ba6 on go-chassis:master.


Use HTML5 audio instead of Web Audio API

@villermen, can you test this on the production server?
If it works, it was an incredibly easy solution for https://github.com/villermen/soundboard/issues/52.

I've used the same approach, but with Content-Length. Let's see how this one will behave.

Also, while I'm at it. What's with the modal appearing on load, and then instantly fading out? Is it debug code?

The header seemed to fix it. You will receive all my internets for today. Too bad it will be over in 3 minutes...

I believe Content-Length header is automatically managed by Slim already.
Does the modal also appear and instantly fade out after a refresh? I have noticed it when LiveReload reloads the page, but when I pressed F5 it didn't.
All the internets for 3 1 no more minutes!

It happens on load and refresh for me. I'm not using LiveReload.

I'm making an issue for this now btw.


feat: define Aztec prelude

This PR breaks the definition of the prelude out from #4496 as the contents of the prelude is uncontroversial and changes to the imports are very prone to merge conflicts. I'd then like to merge this change to the imports separately as we can do this quickly (allowing aztec to iterate on what goes into the prelude) and we can improve how this is imported with a proper prelude syntax in a followup PR.
I've also removed the explicit imports of Option as this is part of the Noir prelude.

Agreed, it seems like aztec_macros was relying on NoteHeader being imported which meant that we got mystery compile failures if we didn't import it despite appearing to not use it at all.

Benchmark results
Metrics with a significant change:

note_successful_decrypting_time_in_ms (32): 547 (-37%)
note_trial_decrypting_time_in_ms (8): 114 (+933%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit ecfcb787 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
5,700
18,884
36,452

l1_rollup_calldata_gas
66,012
239,120
469,832

l1_rollup_execution_gas
193,960
500,426
909,090

l2_block_processing_time_in_ms
1,202
4,515 (+1%)
9,004 (+2%)

note_successful_decrypting_time_in_ms
198 (-3%)
:warning: 547 (-37%)
999 (+1%)

note_trial_decrypting_time_in_ms
:warning: 114 (+933%)
71.0 (-28%)
133 (+4%)

l2_block_building_time_in_ms
15,901 (-1%)
63,045
126,549

l2_block_rollup_simulation_time_in_ms
12,098 (-1%)
48,000
96,497

l2_block_public_tx_process_time_in_ms
3,773 (-1%)
14,973
29,924

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
14,955 (+4%)
27,461 (+2%)

note_history_successful_decrypting_time_in_ms
1,384 (+5%)
2,447 (+1%)

note_history_trial_decrypting_time_in_ms
114 (+38%)
139 (-4%)

node_database_size_in_bytes
18,800,720
35,536,976

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
247
44,736
28,001

private-kernel-ordering
177
52,625
14,627

base-rollup
1,282
177,932
933

root-rollup
70.0 (-1%)
4,192
825

private-kernel-inner
311 (-1%)
73,715
28,001

public-kernel-app-logic
191 (+1%)
32,254
25,379

merge-rollup
5.78 (+1%)
2,712
933

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
0 deployed contracts

tx_size_in_bytes
19,179

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes

tx_pxe_processing_time_ms
2,543
1,353

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
0.0312 (+10%)
467


feature request, concatenate keywords

I want something like join, prefix and bind keywords in ajv so I could first validate the json and second concatenate based on these keywords, for example:
var schema =
{
  type: 'array',
  prefix: 'all ',
  join: ', ',
  items:
  {
    type: 'string',
    bind: (item, i) => item + ` = %${i}`,
  }

}

var data = ['item1', 'item2', 'item3'];

console.log(result)
//all item1 = %0, item2 = %1, item3 = %2

I scripted somehting like this today in node but the performance was very poor so far, it took 100ms to concatenate 100k  schema and data,  while ajv took roughly 10ms to validate the same schema and data for the same amount.
I also wonder if it is possible to make my script as fast as ajv or the nature of the operation is culprit and so cpu-demanding?

Ajv implements JSON schema specification. Custom keywords can be added to ajv-keywords package - it already has “transform”, this idea seems related (see the discussion in the issue about applying transform to arrays).


fix(Portal): remove closeOn* disallow propTypes

Current coverage is 99.47% (diff: 100%)

Merging #700 into master will not change coverage

@@             master       #700   diff @@
==========================================
  Files           132        132
  Lines          2108       2108
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           2097       2097
  Misses           11         11
  Partials          0          0

Powered by Codecov. Last update 9673fc5...07fed6c


update PKGBUILDs

Hi ncaat,
does your issue with needing a development PKGBUILD persist after release 2.0.0?
If we are to accept your PR, would you be fine with release under MIT?
Do you have any name under which you'd want to be credited?
cheers
project-repo

I think that it's always nice to have a PKGBUILD that makes it easy to compile the latest development on github. I'm fine with release under MIT and just ncaat is fine.

Hi ncaat,
we have considered your PR and decided against accepting it for the
moment. However, in the future a PKGBUILD specifically suited for using
the development version with additional debugging and testing features
will be added.
This is simply a strategic decision on our part in view of our plans for
the upcoming releases and we are grateful and honoured that you want to
contribute.
(If you want to improve the ease of installing cagebreak in a different
way, you may consider a vote in the AUR?)
cheers
project-repo


Enums+PlaySound

I know strstr makes the "!" obsolete but I still think it's better to have the proper sound file


Scanning for service accounts from different org/folders

First use case:
There is one organisation, and two folders: eg. prod and dev. For both of them, there are different sets of rules and restrictions.
Unfortunately, a service account from dev project could have an access to project under prod folder.
This situation allows to unauthorised access from dev environment to production.
Second use case is that someone granted access to service account outside of our GCP organisation. We would like to scan the whole GCP org to find service accounts not created in our projects.
As service accounts looks very similar and there is no option to decide from which project service account comes from based on SA name, currently there is no option to look for that violations using IAM scanner.
It would be nice to have a tool for scanning against that violation.

It seems this issue can be solved with VPC Service Controls and/or Domain restricted sharing organization policy.


Document vim's encryption strengths/weaknesses

Hello,
Being a long time vim user (and very grateful for your hard work!), I was briefly considering using vim and it's encryption feature as a replacement to my password manager, and store an encrypted file on a public cloud.
My googling-fu suggests I'm not the only one having thought of this...
By sheer luck, I've stumbled upon those 2 threads:

https://github.com/vim/vim/issues/638
https://github.com/vim/vim/issues/639

Although vim's encryption feature could be applicable to some use cases, it seems mine is not one of them.
While those issues are pending, I feel adding a couple of sentences to the docs here could be useful and low burden for now.
And/or maybe adding some sort of warning regarding vim's crypto status that could be suppressed via an env var, or similar.
What do you think ?

While those issues are pending, I feel adding a couple of sentences to the docs here could be useful and low burden for now.
And/or maybe adding some sort of warning regarding vim's crypto status that could be suppressed via an env var, or similar.
What do you think ?

Any suggestions?

I didn't have the time to dig into the code base and figure out how to display a warning upon encryption usage.
And adding a couple of lines to the docs seems pretty pointless to me.
My main point was that some developers use vim as a password manager and make blog posts about it.
Although my memory is blurry now, I sumed it up to myself as:

that's a bad idea
don't ever use vim's encryption feature

Rather, use a dedicated external tool such as pgp or some derivative such as the beautiful passwordstore.
Vim is a great tool, but I feel it's a shame if some developers are led into thinking this feature is ok to use for highly sensitive data, considering the consequences.

@benjamin-thomas : About one page of scroll lower than what you link to, it is said that Vim uses three possible encryption methods at the user's choice:

a weak method
a method with flaws
a medium-strong method
I understand this as meaning that Vim is not capable of really strong encoding. IOW, it is already documented, albeit in a mealy-mouthed way, that as as you said in a later post, Vim is not the tool to use for really confidential text. And if you're working for a governmental agency, you should use a real typewriter and not leave your confidential text lying about on a computer, or anywhere not under lock and key when you aren't reading or writing them for that matter.

Best regards,
Tony.

it is said that Vim uses three possible encryption methods at the user's choice:
a weak method
a method with flaws
a medium-strong method

(this is slightly below :h :X).
So is this enough to be closed? Or should we more prominently mention this (or link to this section from :h 'cryptmethod'?

Closing this as I won't have time to dig into it...
Tbh, I don't understand why a weak encryption method would be useful to anyone, and it seems to me to be just a bad idea to let the editor handle encryption anyways.
It probably made sense at some point, but I think this feature should be marked deprecated.

Tbh, I don't understand why a weak encryption method would be useful to anyone,
and it seems to me to be just a bad idea to let the editor handle encryption anyways.

In defense of having encryption in the editor:  not only the file is encrypted
but also the swap file, the undo file, the backup file so it limits what information
can leak in clear when opening an encrypted file. There is also viminfo which is
not encrypted. One can also disable backup, undo and viminfo of course.
That said, it would obviously be preferable to have strong encryption.

side note to me: explore if libsodium could be plugged into the existing crypto backend.

In defense of having encryption in the editor: not only the file is encrypted
but also the swap file, the undo file, the backup file so it limits what information
can leak in clear when opening an encrypted file. There is also viminfo which is
not encrypted. One can also disable backup, undo and viminfo of course.

Is this leakage list complete?

@benjamin-thomas : The reason to have a weak method is that not so long ago it used to be the only method; also, it is the method used by encrypted zipfiles. If Vim wants to be able to read files written by, er, I think it was Vim 6, or to open encrypted zipfiles, it needs to have that encryption method available.
Similarly, to read files encrypted by slightly more recent versions of Vim, which only had the weak method and the flawed method at their diposal (the flaws not being all known by then), that method must also be available.
Another reason to have those less-than-perfect methods available is in order to be able to write encrypted files that those earlier versions of Vim can read.
But as I said earlier: if you want really strong safety, don't entrust your confidential files to a computer, keep them in a locked safe when you aren't perusing them, and if possible, deliver them in person to only the people who need to know them.
Best regards,
Tony.

@dpelle good points thanks for the reminders ;)
@tonymec I wouldn’t be surprised if I was using one of those old vim versions at the time, I don’t recall seeing attractive options encryption wise then.
As I said I ruled out vim being a useful tool for my use case, let’s say store passwords, API keys and so on. I’d rather delegate to a specialized tool that I’ll expect to receive timely security updates and so on.
So to sum up I think I’m still as confused about this feature as I was in 2018, but if it works for some then great ^^


Track feature lineage

As a user, I'd like to know the parent feature or features for a given feature computed by my pipeline.

@freddyaboulton Is this covered by what you added via feature provenances?


Make autosavePercentageStep a float

It's an integer at the moment, but for big board sizes like 25 or higher, the possibility to assign autosavePercentageStep a smaller value would be useful.

done in da88b56


Enhancing alphabet and moltype

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

21 of 22 (95.45%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.009%) to 90.338%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/cogent3/core/new_alphabet.py
13
14
92.86%

Totals

Change from base Build<PHONE_NUMBER>:
0.009%

Covered Lines:
31592

Relevant Lines:
34971

💛  - Coveralls


OCPBUGS-37758: Log node UUID and not entire node instance to avoid logging its driver_info field

Log node UUID and not entire node instance to avoid logging its driver_info field
Change-Id: I07eed090920c777046fe29618d5d8be42618a0f6

/retest

/jira-refresh

/jira refresh

/jira refresh

/test pep8

/lgtm

/jira refresh


Will u please add a start aria2 in AriaStatus?

I noticed that is a stop aria2 in it.

aria2 provides a shutdown method so AriaNg can close it. But AriaNg cannot control aria2 when it is not running.


How to visualize the norm of activation map ?

Hi @cuge1995 ,
We first calculate the L2-norm of the activation map and then use the Open3D library (https://github.com/intel-isl/Open3D) for visualization, the color map we use is jet.

Hi @cuge1995 ,
We first calculate the L2-norm of the activation map and then use the Open3D library (https://github.com/intel-isl/Open3D) for visualization, the color map we use is jet.

Which layer do you get the activation maps ? thanks


[Feature Request]: Implement MatrixOneCluster Controller

Is there an existing issue for the same feature request?

[X] I have checked the existing issues.

Is your feature request related to a problem?
No response
Describe the feature you'd like
Implement MatrixOneClusterCluster according to https://github.com/matrixorigin/matrixone-operator/blob/main/docs/rfc/2022-07-05-mo-cluster.md
depends on: #136, #137, #138
close https://github.com/matrixorigin/matrixone/issues/3728
Describe implementation you've considered
No response
Documentation, Adoption, Use Case, Migration Strategy
No response
Additional information
No response

closed by #181


Incorrect cli info

To enable or disable real-time-protection the cmd's are
mdatp config real-time-protection --value enabled
mdatp config real-time-protection --value disabled
please update as the --value is missing in the docs

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

ID: 07f72a75-ca85-0651-1512-8ffb62d0ab7b
Version Independent ID: 6e15dd30-1db2-4a18-a18c-c343e1d508cd
Content: Resources for Microsoft Defender ATP for Mac - Windows security
Content Source: windows/security/threat-protection/microsoft-defender-atp/mac-resources.md
Product: w10
GitHub Login: @Dansimp
Microsoft Alias: dansimp

(ref. #8721, #8797)

(ref. #8721, #8797)