'Copy to Markdown' improvements

Related to #486 and #487

[x] Images with a width or height attribute set needs to be converted back to an <img> as Markdown images can't have a width/height
[ ] Mentions should not be converted to links
[x] Copying a rendered emoji (🔥🔥🔥) in Chrome on macOS results in: <g-emoji alias="fire" fallback-src="https://assets-cdn.github.com/images/icons/emoji/unicode/1f525.png" ios-version="6.0" title=":fire:">🔥</g-emoji><g-emoji alias="fire" fallback-src="https://assets-cdn.github.com/images/icons/emoji/unicode/1f525.png" ios-version="6.0" title=":fire:">🔥</g-emoji><g-emoji alias="fire" fallback-src="https://assets-cdn.github.com/images/icons/emoji/unicode/1f525.png" ios-version="6.0" title=":fire:">🔥</g-emoji> We should strip out all the junk and just copy the character.

[ ] Copying the a whole code block (and nothing else) should not include markdown. e.g. triple click this on a Mac:chicken.chicken = chicken(chicken).then(chicken)

Mentions should not be converted to links

Links are a bit complicated: #130 should be copied as:
https://github.com/sindresorhus/refined-github/issues/130 instead of
[#130](https://github.com/sindresorhus/refined-github/issues/130)
but perhaps it's not worth the trouble.

[ ] Exclude elements outside .markdown-body. The copy event is triggered on the element that contains the selection start, but if your selection ends outside, it will also include a bunch of other unrelated elements.

Mentions should not be converted to links

Are you sure about that? If I mention @sindresorhus and then someone copies this text, I'm not sure if the intent would be to notify you again.

[ ] No line breaks when selecting any list:

Juan

to

tree

hey
bee
see

one

multiline
text

three

[ ]  <ol> not supported?

one
two
three

Are you sure about that? If I mention @sindresorhus and then someone copies this text, I'm not sure if the intent would be to notify you again.

Agreed. Didn't think about that.

Hi, I'm facing some issues with this feature, often I want to copy sample code from readmes etc, and now it is coping with the whole markdown syntax. Can we make this feature optional or at least detect if the text we want to copy is code block?

@sindresorhus we could leave the standard copy alone and instead show a "copy as markdown" button next to selections.

@bfred-it Yeah maybe. Or we could use a different keyboard shortcut like Cmd+Option+C.

I was just searching through the issue tracker to try to find a way to disable this selectively, and was indeed thinking a keyboard shortcut would be the way to go.
My use case is copying commands from READMEs; if I Ctrl+C them now, I get backticks, which of course confuse my shell.

Yeah, I've had this extension removed for a week or so now and my github happiness has increased significantly. It's a PITA to have enabled with this, especially, especially, the amount of times I copy simple things like yarn add <package> or just a package name from the npm to convert myself into yarn syntax and I get markdown syntax with linebreaks and all.

I've disabled this functionality temporarily now (will be published automatically within a day) until we can implement an alternative way to activate it. I agree now that it should not be the default copy behavior.
I've also been experiencing it making copy not work at all randomly. @bfred-it Seems like https://github.com/sindresorhus/refined-github/pull/636 didn't totally fix it after all.


Not an issue but a hope

Type of issue
Other (describe below)
Description
We have seen many improvements in C# across different objectives. Improving readability is also a goal. I would like to suggest that in the next versions, the old with construct (from the Paleolithic era of VB6 ....) allowed writing comfortably:
With objectExpression
[ statements ]
End With

With A
.prop1 = 1
.prop2 = 2
.prop3 = 3
End With

Nothing extraordinary, but very 'readable' and not 'annoying' with the millions of objects we have to deal with today. I don't know if you agree, but I would love to see it in future versions of C#.
Thank you anyway.
Page URL
https://learn.microsoft.com/it-it/dotnet/csharp/fundamentals/types/interfaces
Content source URL
https://github.com/dotnet/docs/blob/main/docs/csharp/fundamentals/types/interfaces.md
Document Version Independent Id
1f0a9a2b-6495-370d-620f-a6dabec7df3b
Article author
@BillWagner
Metadata

ID: d9fd340a-0f74-8602-28a1-9569296a5951
Service: dotnet-csharp
Sub-service: fundamentals

Related Issues

Hi @FlauzerOriginal
You should file this as an issue in the dotnet/csharplang repository. That's where the C# LDM handles new feature requests.  The README has information on how to raise a new feature for discussion.
As the docs do represent the current design, I'll close this.


Netbox DNS does not show up in Web UI

Hi. I just installed netbox-dns plugin.
After installing I run collectstatic and migrate commands on manage.py, than restarted services.
Everything looks fine but there is nothing on plugins menu on Web UI.
Inside Django Admin there is a Netbox DNS section, but not in main web UI.
Netbox: 3.0.3 (fresh install)
Python 3.9.2
CentOS 8
I also have installed secretstore, Qr-code and topology-views plugins. Just to be noted.

Just solved.
I find out the the secretstore plugin must be placed in the en of plugins at configuration.py.


Add c-ares as Termux library dependency

Termux's libcurl was just changed to add a dependency on c-ares in libcurl 8.10.1-1 (https://github.com/termux/termux-packages/pull/21686), so we need to add that to the libraries downloaded and processed by get-packages-and-swift-source.swift in order for FoundationNetworking to work.

Ugh … libcurl loads fine with this addition, but it is no longer working with FoundationNetworking. Every http and https URL is now returning NSFileReadUnknownError (256). It was working fine with the last libcurl 8.9.1, but has broken since libcurl 8.10.1-1 with the c-ares dependency.

This does indeed seem to be related to name resolution, which makes me suspicious of this c-ares dependency. My URL download tester program outputs the following with curl debugging enabled:
zap swift-projects/demo % ANDROID_SERIAL=19091FDF600BAY skip android run --env URLSessionDebugLibcurl=YES --env URLSessionDebug=YES demo "http://www.example.com/"
Building for debugging...
[0/3] Write swift-version--5366198ACE037756.txt
Build complete! (0.18s)
[✓] Check Swift Package (0.19s)
[✓] Connecting to Android (0.05s)
[✓] Copying executable files (1.05s)
[1]                   Could not resolve host: www.example.com␊
[1]                   Could not resolve host: www.example.com␊
[1]                   closing connection #0␊
Connecting to: http://www.example.com/
Error: Error Domain=NSCocoaErrorDomain Code=256 "(null)"

I'm fine with this pull, if you need this newly added dependency.

This does indeed seem to be related to name resolution, which makes me suspicious of this c-ares dependency. My URL download tester program outputs the following with curl debugging enabled

Tough to debug without a link to the actual failing code 😉

Rebuilt the SDK with this and ran it through the CI, so going ahead and merging. Let us know if you still see those testing failures and if so, what tests you're actually running.


Weird folder with Biocontainers JSON file

@hmenager @osallou
Hi guys,
There is a folder:
https://github.com/bio-tools/content/tree/master/data/CCMetagen
with a file called
https://github.com/bio-tools/content/blob/master/data/CCMetagen/biocontainers.biotools.json
which is not a bio.tools file, I assume this CCMetagen folder, which is different from
https://github.com/bio-tools/content/tree/master/data/ccmetagen
is just some folder that got left behind. Do we need this? I'm also asking because it contains the file with the name biocontainers.biotools.json which is the naming convention for bio.tools files.
Thanks,
Hans

@hmenager seems you renamed this file
Biocontainers, via auto PR, adds a biocontainers.json file. Should we modify our CI to inject biocontainers.biotools.json file instead?

Please don't add files that end in .biotools.json
Those should only be files from bio.tools annotations.
The convention we have is that we have the folder names the same as the tool ids and the files have the same name + the specific extension, e.g.:
For tool id blast
Folder is data/blast
Files are:

data/blast/blast.biotools.json
data/blast/blast.debian.yaml

See more files like this at:
https://github.com/bio-tools/content/tree/master/data/blast
I think a biocontainers.json file should be fine, but not biocontainers.biotools.json. The problem came up because the file was called biocontainers.biotools.json

I was the one who made that mistake, ironically enough this was while I was trying to enforce the rules you mention @hansioan.

this is, as far as I understand, the only "erroneous" file at this stage in the repository:

CCMetagen/biocontainers.biotools.json
I suggest I rename it to ccmetagen/ccmetagen.biocontainers.json.
Is that fine with you @hansioan ? @osallou ?

all our files are biocontainers.json , so best if possible is to keep same naming

ok, it sounds reasonable to keep the names this way then.
Out of curiosity @osallou, where are these files stored on the biocontainers side? Is this a repo somewhere? And how/at which frequency are they updated?

They are not on our side, in biocontainers CI we create a PR with new/updated tools based on our metadata


Fix Benchmark

Description
As suggested by @MoisesHer , use static_alloc and static_shape.
In addition, try to enable batch.
Checklist
Essentials

[ ] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage
[ ] Code is well-documented

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

cc @dmlc/gluon-nlp-team

Thanks, it looks good to me

Thanks, it looks good to me


iPhone 12 fallback to bad format

We tried to use this library for a web project and discovered that on all iPhone devices there is a fallback with wrong dimensions. Whatever settings we provide for the image format, it is squeezed and presented Landscape instead of Portrait.
Any suggestions would be highly

appreciated.

Hi @StoyanONLIME,
Sorry for the late reply, I am on vacations until monday. Just to let you know that I fixed the docs website: feel free to check again. The issue was related to a fixed ratio applied to the camera viewport.

As a side note, when you specify a size on initialization, you are "requesting" a size for WebRTC, meaning that the hardware can support the requested size... or not: in most time, WebRTC will try to be as closer as possible to your requested size, but can fallback to another one if the hardware doesn't support it.


Auto updating SAMUpdater seems to be broken

http://pastebin.com/KYj8ZdzB

New version system should fix the auto update issues.  Unfortunately this also means breaking all backward compatibility for versions prior to v<IP_ADDRESS> of SAMUpdater.


README better examples

"You could amend your readme to explain that one needs to set up rounded corners in picom.conf, set borders to 0 in the i3 config, that the picom corner radius should match `border-radius and give a short example json-file like
{
  "border-radius": 10,
  "border-width": 5,
  "border-rgba": "FFFF00FF"
}

might seem obvious, but still...."
u/r_wraith on reddit

@arnavgr
./xborders --border-width=1


CASSANDRA-16384 test case AuditLoggerTest fail on aarch64 platform

CASSANDRA-16384 test case AuditLoggerTest fail on aarch64 platform
CASSANDRA-16392 Unable to load the library netty_tcnative_linux_aarch_64
Upgrade chroniclejar and netty-tcnativejar to fix above 2 issues on aarch64

The license files under lib/licenses/ also need to be renamed (and checked for updates)

The license files under lib/licenses/ also need to be renamed (and checked for updates)

it's looking much better now.
can we please (even though these faults existed before, it would be nice to fix them up to standard)…

rename lib/licenses/chronicle-core-2.20.126-SNAPSHOT.txt to lib/licenses/chronicle-core-2.20.126.txt
rename lib/licenses/netty-4.1.58.txt to lib/licenses/netty-all-4.1.58.txt
rename lib/licenses/netty-tcnative-2.0.36.txt to lib/licenses/netty-tcnative-boringssl-static-2.0.36.txt

it's looking much better now.
can we please (even though these faults existed before, it would be nice to fix them up to standard)…

rename lib/licenses/chronicle-core-2.20.126-SNAPSHOT.txt to lib/licenses/chronicle-core-2.20.126.txt
rename lib/licenses/netty-4.1.58.txt to lib/licenses/netty-all-4.1.58.txt
rename lib/licenses/netty-tcnative-2.0.36.txt to lib/licenses/netty-tcnative-boringssl-static-2.0.36.txt

Hello, mck, now files have been renamed as your request.

Hello, mck, now files have been renamed as your request.

Hello, mck, if no issue, could you help to merge this?

Hello, mck, if no issue, could you help to merge this?


Support for different, 'swappable' RF/timing models

This adds support for simulation with different RF/timing models for RF frame propagation. The models can be defined as Go files and can be 'swapped' at runtime using a CLI command.
To enable this, a different event message format (V2) is defined while keeping also the V1 (default) format to communicate with OT nodes that use the default format. (TODO: consider if better to just add new event types.)
Two distinct radio models are included:

Ideal - every packet has 1 us transmit duration and an infinite amount of packets can be transmitted at the same time by nodes, even by the same node.
MutualInterference - more realistic frame transmit duration, and RF channel is busy if a node sends, causing other nodes to experience CCA failure.  This takes into account the node's CCA ED thresholds for determining 'busy' / 'not busy'.

To use the new radiomodel features of this PR (TODO: consider to support also 'classic' OT firmwares), use this OT-NS with a firmware built with PR https://github.com/openthread/openthread/pull/7897 .

Medium is busy for all nodes when 1 node sends

Does it mean that the entire medium for all nodes is considered busy whenever one single node started transmitting? Did you consider the channel used? (I am still reading your code, but answering here can help future discussions).

Medium is busy for all nodes when 1 node sends

Does it mean that the entire medium for all nodes is considered busy whenever one single node started transmitting? Did you consider the channel used? (I am still reading your code, but answering here can help future discussions).

Initially all nodes were seeing "channel busy", but currently the power of the signal and CCA ED threshold per node is used to calculate this. Good that you mention channel, because that still needs to be added! Currently a single channel is assumed and that was also the way OT-NS was practically used by me, so far.

Medium is busy for all nodes when 1 node sends

Does it mean that the entire medium for all nodes is considered busy whenever one single node started transmitting? Did you consider the channel used? (I am still reading your code, but answering here can help future discussions).

Initially all nodes were seeing "channel busy", but currently the power of the signal and CCA ED threshold per node is used to calculate this. Good that you mention channel, because that still needs to be added! Currently a single channel is assumed and that was also the way OT-NS was practically used by me, so far.

Thanks for clarifying. In PR #235, I made a way to verify interference per channel. I believe we could use your model system in something like the functions in sendChannelActivity and ReceivePacket. What do you think?

For the present PR, CHANNEL_ACTIVITY would not be strictly needed to do CCA prior to a frame transmit because the CCA is already done by OT-NS as part of the procedure for handling a "Frame Tx" event that the OT node sends. But it could be introduced as a separate event but I don't see yet if this brings any benefit.

Ok, nice. IMO, CCA should still be handled by OT-NS and with the single event of "Frame Tx", as the channel change request happens in the package struct that arrives to the radio functions. However, treating channels separately should be very necessary, since the default protocol is definitely using this property to avoid collisions.

Do you see a benefit of letting the OT node (C code) handle the CCA explicitly?

I believe that RSSI and CCA values should be a model of the simulator, as you seem to have done, but the decision to say it is busy or not is definitely for the protocol in charge of it. Therefore, independently of the language (in this case C), the algorithm to handle it should be in the simulated device, imo.

As next steps I'll fix the multi-channel issue in my radiomodel, and try to incorporate the new state events so the energy reporting should work. If that works out then we would have a more or less clean separation of the "energy reporting" feature from the rest of improvements.

Since we are willing to unify our codes to do essentially the same feature, I'd ask you to avoid doing a duplicated work on that, instead, we should study each other's code.

Codecov Report

Merging #316 (ba44861) into main (c616148) will decrease coverage by 1.06%.
The diff coverage is 49.63%.

@@            Coverage Diff             @@
##             main     #316      +/-   ##
==========================================
- Coverage   49.97%   48.90%   -1.07%
==========================================
  Files          38       43       +5
  Lines        4608     4950     +342
==========================================
+ Hits         2303     2421     +118
- Misses       2123     2334     +211
- Partials      182      195      +13

Impacted Files
Coverage Δ

cli/ast.go
66.66% <ø> (ø)

dissectpkt/dissectpkt.go
57.14% <0.00%> (-42.86%)
:arrow_down:

radiomodel/radiomodelMutualInterference.go
0.00% <0.00%> (ø)

types/node_config.go
100.00% <ø> (ø)

web/site/bindata.go
99.00% <ø> (-0.01%)
:arrow_down:

cli/CmdRunner.go
21.93% <4.34%> (-1.21%)
:arrow_down:

radiomodel/radiomodel.go
20.45% <20.45%> (ø)

simulation/node.go
40.97% <25.00%> (-0.44%)
:arrow_down:

simulation/simulation.go
55.75% <36.36%> (-2.51%)
:arrow_down:

dispatcher/send_queue.go
85.18% <55.55%> (-14.82%)
:arrow_down:

... and 9 more

For information,

there is now a "radiomodel kbps" command to view/change the current PHY model data rate in kbps. To avoid more complexity in this PR it is in a separate branch to later make a PR from: https://github.com/EskoDijk/ot-ns/tree/pr-kbps-setting
multiple RF channels are now also supported in the MutualInterference radio model. In the "ideal" models, this was already supported (due to no interference modeled there anyhow).

Based on the feedback/suggestions on this PR and in https://github.com/openthread/openthread/pull/7897, I've made a new design for radiomodel simulation, so probably this PR can be closed and archived!
Thanks mostly to @simonlingoogle and @Vinggui for your comments and discussions that shaped this.
In the new design, the details of the radio operation (and state machine for it) are included in the OT node C code. This then allows multiple types of nodes to run in the same simulation in principle, e.g. nodes with different radio implementations alongside eachother. (Like Wi-Fi interference nodes, Thread nodes with Aloha-CCA, sub-Ghz Thread nodes, etc.)
The new design also integrates the features of the RTEA (Road To Energy Analysis) PRs.


Documentation is in a socialect. What does Productization mean?

Consider
https://openui5beta.hana.ondemand.com/#docs/api/symbols/sap.ui.table.AnalyticalTable.html

Experimental API:
Since version 1.21. The AnalyticalTable will be [i]productized[/i] soon.

What does Productization mean in the context of UI5?

Hi Andreas,
this is supposed to mean something like "it will be declared to be stable and to be official part of the product". I have opened the internal ticket<PHONE_NUMBER> to get the documentation fixed and possibly the whole thing declared non-experimental.
Thanks
Andreas


fix: webhook operation parsing

Problem
Currently, OpenAPI 3.1 webhooks do not render correctly. See issue https://github.com/scalar/scalar/issues/1811
Explanation
This happens because when transforming webhooks in the parse function from @scalar/api-reference, the originalWebhook variable is assigned to the PathItem object instead of the Operation, as implied by this cast.
Solution
With this PR, the originalWebhook variable is correctly assigned to the Operation.
Note: the PathItem also has summary and description fields. It might be useful to fall back to them if they are missing in the operation.

Yesss, great contribution!

@marclave like this?

perfect!!


Gist 9.0.0 fails load from URL into Protege

Gist 9.0.0 does not load properly from web URL into Protege.
Steps to reproduce:

open protege (e.g. version
select File > Open from URL > ...
put in URL:  https://ontologies.semanticarts.com/o/gistCore9.0.0
Observe this failure message:
sun.security.validator.ValidatorException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
Try the http version of the URL (technically incorrect):  http://ontologies.semanticarts.com/o/gistCore9.0.0
Observe that it cannot locate the next imported ontology automatically on the web (as gist 8.0.0 does), it prompts for it.

Desired behavior:
Behave like gist8.0.0, which properly loads all imports with no additional dialogs, but using the proper HTTPS URLs.

@marksem what version of Protege are you using?
If not the latest version could you update and try again?

Works in Protege 5.5.0 (latest).
(Problem shows itself in older versions of Protege, such as 5.2.0)


CSS Desired outcomes

Because

This PR

Issue

Closes #XXXXX
Additional Information

Pull Request Requirements

[x] I have thoroughly read and understand The Odin Project Contributing Guide
[x] The title of this PR follows the location of change: brief description of change format, e.g. 01-flex-center: Update self check
[x] The Because section summarizes the reason for this PR
[x] The This PR section has a bullet point list describing the changes in this PR
[x] If this PR addresses an open issue, it is linked in the Issue section
[x] If applicable, I have ensured that the TOP solution files match the Desired Outcome image

we don't accept personal solutions to this repo

I mistook this for a css exercise repo update. Could you please fill out the PR fields @katyalmohit to add context to this PR?

Ah now I see. This is indeed a css exercise PR. We don't accept PR's for your personal solutions :)

Thank you for reviewing my pull request. I understand now that this particular exercise is not suitable for submission as a pull request. I appreciate your clarification and will keep it in mind for future contributions.


set-output deprecation changed how multilines are handled

Q
A

Documentation
no

Bugfix
yes

BC Break
no

New Feature
no

RFC
no

QA
no

Description

Several tags were being created that were invalid e.g. v1%0A1.21.0. It seems that when set-output was deprecated, github changed how multiline strings were handled. This PR should fix that issue according to: https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#multiline-strings

Rebased using https://github.com/laminas/laminas-ci-matrix-action/pull/164 to allow CI to pass

Nasty one: good catch!


GigaGrid should be to handle initialSubtotalBys and initialSortBys as list of colTags

Would make props simpler.  I will create a PR for this.

PR #63 opened


Use a transcribe tool like otter.ai for our meetings

Following SIG-Security steps: https://github.com/cncf/sig-security/issues/314. It would be nice to have a tool like https//otter.ai for our meetings like @timfong888 mentioned.

Zoom will give us an automatic transcript, it just won't be live.

@amye it would be great to post the Zoom transcripts somewhere. Thx!

That is actually the question - where would be these be posted? in this repo or on YouTube?

I'm not sure if YouTube has a place to post CC transcripts from Zoom(?). They do show up when you click on CC while watching the video. The GitHub repo is alternative but I don't think there's a Zoom transcript integration with GitHub, so that would mean more work creating that integration.
Using something like https://otter.ai is a bit different because a tool like that provides more like actual meeting notes, which would be more comparable to a meeting scribe.
My initial thought behind this was to add a link for meeting notes under the meeting agenda for every meeting, but I suppose if we can figure how to put a link to the CC content under the agenda is good too (I see that Zoom does provide a link to the transcript of every recorded meeting)

Has anyone tried scribbl.co?  It's supposed to be real-time.

@yeelinglam I haven't tried scribbl.co but it also seems like a viable alternative.

Closing this as zoom already has transcription.


Overlay Animation Causes Overflow

Describe the bug
All components built with Overlay can cause their parent container to overflow and show scrollbars for a second while the Overlay performs a slide-up animation after it is opened.  This happens when the overlay is placed such that it can fit between its anchor and the bottom of the view port, but the slide-up animation cause it to transition from a placement that is outside the dimensions of the viewport.
Proposed Solution
We could account for the distance of the slide-up animation when calculating the placement of the overlay.  If it won't fit below the anchor if you consider overlay height + transition height, then it needs to be placed elsewhere.
Alternatively, we could still place the overlay below the anchor, but shorten or skip the slide-up if there isn't room for it.
Open to other suggestions @vdepizzol
To Reproduce
Steps to reproduce the behavior:

Go to the DropdownMenu story in storybook
Adjust the Actions panel height so that there is just enough room for the DropdownMenu to open below the button
Open the DropdownMenu
Note the scroll bar flicker in and out on the right

Expected behavior
Overlay animation should not cause temporary overflow
Screenshots

@dgreif thanks for writing this! ✨
I wonder if we can mitigate this entirely by implementing the full animation plan we had where the sliding transition is rendered depending on which outside-* anchor side is used. This means we can have a visual connection between the source and the overlay. Similar to how select menu handles it in production:

In your example above, outside-bottom would animate from top to bottom, while something like outside-right would follow the same animation style, being from left to right and always coming from the source. I guess using it that way the overlay wouldn't overflow?

outside-bottom example
outside-right example

Yes, this looks like a great solution!  I didn't realize that was part of the original design, but it makes perfect sense and would fix the overflow issue.  Thanks @vdepizzol!

💖 💖 💖


failed to open source object: open file failed: cannotDownloadAbusiveFile

What is the problem you are having with rclone?
When trying to download some APK or EXE files that I have on GDrive, I receive this error message:
file: Failed to copy: failed to open source object: open file failed: googleapi: Error 403: This file has been identified as malware or spam and cannot be downloaded., cannotDownloadAbusiveFile
What is your rclone version (eg output from rclone -V)
rclone v1.41

os/arch: darwin/amd64
go version: go1.10

Which OS you are using and how many bits (eg Windows 7, 64 bit)
macOS High Sierra 10.13.4
Which cloud storage system are you using? (eg Google Drive)
Google Drive
The command you were trying to run (eg rclone copy /tmp remote:tmp)
rclone sync remote:folder local/Folder
Possible solution
If we add the flag acknowledgeAbuse=true we can bypass this error: https://github.com/prasmussen/gdrive/issues/182

Looks easy enough.. Fancy sending a PR?

I have no idea about Go but I'll take a look at it :)

I would very much appreciate this. I have a ton of legitimate files getting flagged as abusive.

OK I've implemented this.
I can't test it though as I don't have any abusive files!
So can you

Attempt to download an abusive file.  You should get an error which includes the text "Use the --drive-acknowledge-abuse flag to download this file"
Try downloading again with the  --drive-acknowledge-abuse flag and check it does download!

Thanks
https://beta.rclone.org/v1.41-090-g6be5a989-fix-2317-drive-abuse/ (uploaded in 15-30 mins)

without --drive-ackowledge-abuse:
2018/06/08 10:53:25 INFO  : Local file system at /tmp/A.Robot.Named.Fight.v<IP_ADDRESS>-SiMPLEX: Waiting for checks to finish
2018/06/08 10:53:25 INFO  : Local file system at /tmp/A.Robot.Named.Fight.v<IP_ADDRESS>-SiMPLEX: Waiting for transfers to finish
2018/06/08 10:53:25 ERROR : SiMPLEX.exe: Failed to copy: failed to open source object: open file failed: Use the --drive-acknowledge-abuse flag to download this file: googleapi: Error 403: This file has been identified as malware or spam and cannot be downloaded., cannotDownloadAbusiveFile
2018/06/08 10:53:25 INFO  : SiMPLEX.ini: Copied (new)
2018/06/08 10:53:25 INFO  : bassmod.dll: Copied (new)
2018/06/08 10:53:25 INFO  : simplex.nfo: Copied (new)
2018/06/08 10:53:26 INFO  : unarc.exe: Copied (new)
2018/06/08 10:53:31 INFO  : SiMPLEX.bin: Copied (new)
2018/06/08 10:53:31 ERROR : Attempt 1/3 failed with 1 errors and: failed to open source object: open file failed: Use the --drive-acknowledge-abuse flag to download this file: googleapi: Error 403: This file has been identified as malware or spam and cannot be downloaded., cannotDownloadAbusiveFile
2018/06/08 10:53:31 INFO  : Local file system at /tmp/A.Robot.Named.Fight.v<IP_ADDRESS>-SiMPLEX: Waiting for checks to finish
2018/06/08 10:53:31 INFO  : Local file system at /tmp/A.Robot.Named.Fight.v<IP_ADDRESS>-SiMPLEX: Waiting for transfers to finish
2018/06/08 10:53:31 ERROR : SiMPLEX.exe: Failed to copy: failed to open source object: open file failed: Use the --drive-acknowledge-abuse flag to download this file: googleapi: Error 403: This file has been identified as malware or spam and cannot be downloaded., cannotDownloadAbusiveFile
2018/06/08 10:53:31 ERROR : Attempt 2/3 failed with 1 errors and: failed to open source object: open file failed: Use the --drive-acknowledge-abuse flag to download this file: googleapi: Error 403: This file has been identified as malware or spam and cannot be downloaded., cannotDownloadAbusiveFile
2018/06/08 10:53:31 INFO  : Local file system at /tmp/A.Robot.Named.Fight.v<IP_ADDRESS>-SiMPLEX: Waiting for checks to finish
2018/06/08 10:53:31 INFO  : Local file system at /tmp/A.Robot.Named.Fight.v<IP_ADDRESS>-SiMPLEX: Waiting for transfers to finish
2018/06/08 10:53:32 ERROR : SiMPLEX.exe: Failed to copy: failed to open source object: open file failed: Use the --drive-acknowledge-abuse flag to download this file: googleapi: Error 403: This file has been identified as malware or spam and cannot be downloaded., cannotDownloadAbusiveFile
2018/06/08 10:53:32 ERROR : Attempt 3/3 failed with 1 errors and: failed to open source object: open file failed: Use the --drive-acknowledge-abuse flag to download this file: googleapi: Error 403: This file has been identified as malware or spam and cannot be downloaded., cannotDownloadAbusiveFile
2018/06/08 10:53:32 Failed to copy: failed to open source object: open file failed: Use the --drive-acknowledge-abuse flag to download this file: googleapi: Error 403: This file has been identified as malware or spam and cannot be downloaded., cannotDownloadAbusiveFile
with flag:
2018/06/08 10:54:56 ERROR : bassmod.dll: Failed to copy: failed to open source object: open file failed: googleapi: Error 403: This file has not been flagged for abuse., invalidAbuseAcknowledgment 2018/06/08 10:54:56 ERROR : SiMPLEX.ini: Failed to copy: failed to open source object: open file failed: googleapi: Error 403: This file has not been flagged for abuse., invalidAbuseAcknowledgment 2018/06/08 10:54:56 ERROR : simplex.nfo: Failed to copy: failed to open source object: open file failed: googleapi: Error 403: This file has not been flagged for abuse., invalidAbuseAcknowledgment 2018/06/08 10:54:56 ERROR : unarc.exe: Failed to copy: failed to open source object: open file failed: googleapi: Error 403: This file has not been flagged for abuse., invalidAbuseAcknowledgment 2018/06/08 10:54:58 ERROR : SiMPLEX.bin: Failed to copy: failed to open source object: open file failed: googleapi: Error 403: This file has not been flagged for abuse., invalidAbuseAcknowledgment 2018/06/08 10:54:58 ERROR : Attempt 1/3 failed with 5 errors and: failed to open source object: open file failed: googleapi: Error 403: This file has not been flagged for abuse., invalidAbuseAcknowledgment 2018/06/08 10:54:59 ERROR : SiMPLEX.ini: Failed to copy: failed to open source object: open file failed: googleapi: Error 403: This file has not been flagged for abuse., invalidAbuseAcknowledgment 2018/06/08 10:54:59 ERROR : simplex.nfo: Failed to copy: failed to open source object: open file failed: googleapi: Error 403: This file has not been flagged for abuse., invalidAbuseAcknowledgment 2018/06/08 10:54:59 ERROR : unarc.exe: Failed to copy: failed to open source object: open file failed: googleapi: Error 403: This file has not been flagged for abuse., invalidAbuseAcknowledgment 2018/06/08 10:54:59 ERROR : bassmod.dll: Failed to copy: failed to open source object: open file failed: googleapi: Error 403: This file has not been flagged for abuse., invalidAbuseAcknowledgment 2018/06/08 10:55:08 ERROR : SiMPLEX.bin: Failed to copy: failed to open source object: open file failed: googleapi: Error 403: This file has not been flagged for abuse., invalidAbuseAcknowledgment 2018/06/08 10:55:08 ERROR : Attempt 2/3 failed with 5 errors and: failed to open source object: open file failed: googleapi: Error 403: This file has not been flagged for abuse., invalidAbuseAcknowledgment 2018/06/08 10:55:12 ERROR : bassmod.dll: Failed to copy: failed to open source object: open file failed: googleapi: Error 403: This file has not been flagged for abuse., invalidAbuseAcknowledgment 2018/06/08 10:55:12 ERROR : simplex.nfo: Failed to copy: failed to open source object: open file failed: googleapi: Error 403: This file has not been flagged for abuse., invalidAbuseAcknowledgment 2018/06/08 10:55:12 ERROR : SiMPLEX.ini: Failed to copy: failed to open source object: open file failed: googleapi: Error 403: This file has not been flagged for abuse., invalidAbuseAcknowledgment 2018/06/08 10:55:12 ERROR : unarc.exe: Failed to copy: failed to open source object: open file failed: googleapi: Error 403: This file has not been flagged for abuse., invalidAbuseAcknowledgment 2018/06/08 10:55:33 ERROR : SiMPLEX.bin: Failed to copy: failed to open source object: open file failed: googleapi: Error 403: This file has not been flagged for abuse., invalidAbuseAcknowledgment 2018/06/08 10:55:33 ERROR : Attempt 3/3 failed with 5 errors and: failed to open source object: open file failed: googleapi: Error 403: This file has not been flagged for abuse., invalidAbuseAcknowledgment 2018/06/08 10:55:33 Failed to copy: failed to open source object: open file failed: googleapi: Error 403: This file has not been flagged for abuse., invalidAbuseAcknowledgment

So it looks like when we receive the error cannotDownloadAbusiveFile  we then should retry but adding the --drive-acknowledge-abuse flag...

FYI v2 API does no have this issue if the file is abusive, it downloads it regardless.

So, if you use the flag, abusive files do download but non-abusive files will not. I have lots of mixed files so this doesn't help much :/

Thanks for testing.  I will revise the patch!

If you do keep it as a flag, I'd like it if the drive remote had an option
to download all abusive files. Otherwise every copy/sync I do I'll have to
add the flag because I don't know which folders have "abusive" files.
There are no instances in any of my rclone+drive use-cases where I want to
be "protected" from "abusive" files.
On Fri, Jun 8, 2018, 11:29 AM Nick Craig-Wood<EMAIL_ADDRESS>wrote:

Thanks for testing. I will revise the patch!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ncw/rclone/issues/2317#issuecomment-395815082, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKOQXbAN5VM7o6yE3ER8nvZMsZt2fUBpks5t6qZigaJpZM4UL_IB
.

OK here is a revised patch which tries the download without the query parameter and if it fails with that error, retries it with the query parameter if you supply the --drive-acknowledge-abuse flag.
Can you do the two tests again please?

Attempt to download an abusive file.  You should get an error which includes the text "Use the --drive-acknowledge-abuse flag to download this file"
Try downloading again with the  --drive-acknowledge-abuse flag and check it does download!

Here is a beta to try
https://beta.rclone.org/v1.41-092-g3bf9c26a-fix-2317-drive-abuse/ (uploaded in 15-30 mins)
@jahands wrote:

If you do keep it as a flag, I'd like it if the drive remote had an option
to download all abusive files

My reading of the API docs makes me thing that I shouldn't be doing this all the time - the user will have to do something to download the file.
The --drive-acknowledge-abuse flag if supplied will allow all abusive files to be downloaded.  You can set this in an environment variable RCLONE_DRIVE_ACKNOWLEDGE_ABUSE=true.  When I've finished the config restructure (after 1.42 is released) you'll be able to put it in the config file too.

Files identified as abusive (malware, etc.) are only downloadable by the owner. Additionally, the query parameter acknowledgeAbuse=true must be included to indicate that the user has acknowledged the risk of downloading potential malware. Your application should interactively warn the user before using this query parameter.

I would consider a user setting an environment variable or setting it in
the config file to be user consent.
On Fri, Jun 8, 2018, 12:02 PM Nick Craig-Wood<EMAIL_ADDRESS>wrote:

OK here is a revised patch which tries the download without the query
parameter and if it fails with that error, retries it with the query
parameter if you supply the --drive-acknowledge-abuse flag.
Can you do the two tests again please?

Attempt to download an abusive file. You should get an error which
includes the text "Use the --drive-acknowledge-abuse flag to download this
file"
Try downloading again with the --drive-acknowledge-abuse flag and
check it does download!

Here is a beta to try
https://beta.rclone.org/v1.41-092-g3bf9c26a-fix-2317-drive-abuse/
(uploaded in 15-30 mins)
@jahands https://github.com/jahands wrote:
If you do keep it as a flag, I'd like it if the drive remote had an option
to download all abusive files
My reading of the API docs makes me thing that I shouldn't be doing this
all the time - the user will have to do something to download the file.
The --drive-acknowledge-abuse flag if supplied will allow all abusive
files to be downloaded. You can set this in an environment variable
RCLONE_DRIVE_ACKNOWLEDGE_ABUSE=true. When I've finished the config
restructure (after 1.42 is released) you'll be able to put it in the config
file too.
Files identified as abusive (malware, etc.) are only downloadable by the
owner. Additionally, the query parameter acknowledgeAbuse=true must be
included to indicate that the user has acknowledged the risk of downloading
potential malware. Your application should interactively warn the user
before using this query parameter.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ncw/rclone/issues/2317#issuecomment-395823768, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKOQXVs4xZDTl_gtqt6XPyaVjG1wJGnfks5t6q4agaJpZM4UL_IB
.

I think that the approach taken is the perfect one. Great job @ncw, too bad that I cannot test it now.

I agree I like the latest revision. I'll test it later today.
On Fri, Jun 8, 2018, 12:36 PM KrLx1994roller<EMAIL_ADDRESS>wrote:

I think that the approach taken is the perfect one. Great job @ncw
https://github.com/ncw, too bad that I cannot test it now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ncw/rclone/issues/2317#issuecomment-395833558, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKOQXaW41fJYuRgS1d20pE-ZW0Ytk3cbks5t6rYlgaJpZM4UL_IB
.

@ncw Tested the new beta and it is indeed working!

@jahands how did your testing go?  Shall I merge this fix?

Works great!
Without flag:
rclone-v1.41-092-g3bf9c26a-fix-2317-drive-abuse-windows-amd64\rclone.exe -v copy gd:<REDACTED>/igfxrcht.lrc .
2018/06/10 08:16:20 ERROR : igfxrcht.lrc: Failed to copy: failed to open source object: open file failed: Use the --drive-acknowledge-abuse flag to download this file: googleapi: Error 403: This file has been identified as malware or spam and cannot be downloaded., cannotDownloadAbusiveFile
2018/06/10 08:16:20 ERROR : Attempt 1/3 failed with 1 errors and: failed to open source object: open file failed: Use the --drive-acknowledge-abuse flag to download this file: googleapi: Error 403: This file has been identified as malware or spam and cannot be downloaded., cannotDownloadAbusiveFile
2018/06/10 08:16:21 ERROR : igfxrcht.lrc: Failed to copy: failed to open source object: open file failed: Use the --drive-acknowledge-abuse flag to download this file: googleapi: Error 403: This file has been identified as malware or spam and cannot be downloaded., cannotDownloadAbusiveFile
2018/06/10 08:16:21 ERROR : Attempt 2/3 failed with 1 errors and: failed to open source object: open file failed: Use the --drive-acknowledge-abuse flag to download this file: googleapi: Error 403: This file has been identified as malware or spam and cannot be downloaded., cannotDownloadAbusiveFile
2018/06/10 08:16:21 ERROR : igfxrcht.lrc: Failed to copy: failed to open source object: open file failed: Use the --drive-acknowledge-abuse flag to download this file: googleapi: Error 403: This file has been identified as malware or spam and cannot be downloaded., cannotDownloadAbusiveFile
2018/06/10 08:16:21 ERROR : Attempt 3/3 failed with 1 errors and: failed to open source object: open file failed: Use the --drive-acknowledge-abuse flag to download this file: googleapi: Error 403: This file has been identified as malware or spam and cannot be downloaded., cannotDownloadAbusiveFile
2018/06/10 08:16:21 Failed to copy: failed to open source object: open file failed: Use the --drive-acknowledge-abuse flag to download this file: googleapi: Error 403: This file has been identified as malware or spam and cannot be downloaded., cannotDownloadAbusiveFile

With flag:
rclone-v1.41-092-g3bf9c26a-fix-2317-drive-abuse-windows-amd64\rclone.exe -v copy gd:<REDACTED>/igfxrcht.lrc . --drive-acknowledge-abuse
2018/06/10 08:16:38 INFO  : igfxrcht.lrc: Copied (new)
2018/06/10 08:16:38 INFO  :
Transferred:   421 kBytes (32.407 kBytes/s)
Errors:                 0
Checks:                 0
Transferred:            1
Elapsed time:       12.9s

I also tried copying a non abusive file with the flag and that worked also.
LGTM

Thanks to you all for testing :-)
I've merged this here:
https://beta.rclone.org/v1.41-098-g9289aead/ (uploaded in 15-30 mins)
and it will be in the latest beta from now on and v1.42.

Can you run it again with -vv and post the output here?
On Wed, Jun 13, 2018, 1:34 PM Fabian Röling<EMAIL_ADDRESS>wrote:

I just tried it and it did not synchronise the file. It didn't give out an
error message (it had absolutely no output) and it synchronised everything
else, but it didn't synchronise the two blocked files.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ncw/rclone/issues/2317#issuecomment-397041086, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKOQXUdmaIglL88F9o0NP_hZVqithlKdks5t8VsugaJpZM4UL_IB
.

Oh wait, the file wasn't even on Google Drive. Nevermind.
Tested again and it works. Thank you!

By having it be a command line argument, scripting programs like RCloneSync
still don't work. It would be better as an option in rclone config or
something else that's permanent.
Should this be reopened or should I make a new report?
On 20:48, Wed, 13 Jun 2018 Jacob Hands<EMAIL_ADDRESS>wrote:

Can you run it again with -vv and post the output here?
On Wed, Jun 13, 2018, 1:34 PM Fabian Röling<EMAIL_ADDRESS>wrote:

I just tried it and it did not synchronise the file. It didn't give out
an
error message (it had absolutely no output) and it synchronised
everything
else, but it didn't synchronise the two blocked files.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ncw/rclone/issues/2317#issuecomment-397041086, or
mute
the thread
<
https://github.com/notifications/unsubscribe-auth/AKOQXUdmaIglL88F9o0NP_hZVqithlKdks5t8VsugaJpZM4UL_IB
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ncw/rclone/issues/2317#issuecomment-397045580, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJ_vYmY2SHyaQgtQgqpr32vSi7d0zMRDks5t8V5ygaJpZM4UL_IB
.

@Fabian42 thanks for testing.  You can set it as an environment variable for now - you'll be able to set it in the config file in about a week after v1.41 is released (I'll merge my config unification changes once v1.41 is released).


RFC: Allowing Non-Determinism in ProcessProposal

Related to #1174
Rendered
This PR contains an extension to the ABCI specification, in the form of an RFC, that offers application designers a weaker alternative to the application requirements on PrepareProposal and ProcessProposal as they appear in the ABCI specification.
This has previously been reviewed in our knowledge base repo.

PR checklist

[ ] Tests written/updated
[ ] Changelog entry added in .changelog (we use unclog to manage our changelog)
[ ] Updated relevant documentation (docs/ or spec/) and code comments

Thanks @cason for your review. I think your comment coincides with this one when we were discussing on the algorithm's changes.
We decided to stop at the changes currently described in this draft for pragmatical reasons (how much improvement vs. complexity introduced by the changes).
But your comment has convinced me to add a paragraph mentioning this. Bear with me.

Thank you for continuing down this path @sergio-mena !!


rtol=1 seems dangerous

https://github.com/PsuAstro528/project-elizabethg60/blob/5c78e85793f46d7facc0eda92a67eb8fafac6217/src/test/runtests.jl#L37
When comparing serial and parallel, I'd hope that they agree to much better than rtol=1!
(For the previous tests, I didn't follow enough to know what was reasonable.)

should have been 0.1 - corrected!


Move conscrypt NONE checks internal to Handshake

Based on https://github.com/square/okhttp/issues/3973, looks like this can still happen

@JakeWharton @swankjesse @dave-r12 If this is happening in 8.1.0, it would be good to get the right fix in for 3.11.0 as adoption increases.  I'm not sure if this is it, or there is something else we should be doing to address #3973

Here I guess - https://github.com/google/conscrypt/blob/82dc6605fb3d06bf51be2e2f6097f3eee1f32d22/common/src/main/java/org/conscrypt/SSLNullSession.java
I don't think this part is really documented at all.  And since it's now baked into 8.1.0, there is no fix that makes our workaround not needed.

I’d prefer to remove RealConnection.isValid() completely and moving both the cipher suite check and the version check into Handshake.get(). That way we only check once!

This is a nice improvement.

Hi, i found this issue: when got this issue, the host is null. that mean the SSLSession is error ?..?
@Aspect
public class OkhttpHandshakeTLSAspect {
    @Pointcut("execution(* okhttp3.Handshake.get(..))")
    public void handshakePoint() {
    }

    @Around("handshakePoint()")
    public Object onHandshakePointExc(final ProceedingJoinPoint joinPoint) throws Throwable {
        try {
//            throw new IllegalArgumentException("Unexpected TLS version: NONE");
            return joinPoint.proceed();
        } catch (IllegalArgumentException throwable) {
            if (TextUtils.equals(throwable.getMessage(), "Unexpected TLS version: NONE")){
                submitBIEvent(joinPoint);
                throw new IOException("Unexpected TLS version: NONE");
            } else {
                throw throwable;
            }
        }
    }

    private static void submitBIEvent(ProceedingJoinPoint joinPoint) {
        try {
            SSLSession sslSession = (SSLSession) joinPoint.getArgs()[0];
            Log.e("HandshakeTLSAspect", "PeerHost: " + sslSession.getPeerHost());
            BIReportHammer.create().buildEvent(RunTimeManager.instance().getApplicationContext(), "OKHTTP_TLS_ERROR")
                    .addExtData("host", sslSession.getPeerHost())  // host: null
                    .addExtData("port", sslSession.getPeerPort()) //port: -1
                    .commit();
        } catch (Exception e) {
            e.printStackTrace();
        }
    }
}


Move the ColorPicker preview under the picker area

This moves it to be closer to the sliders, which in turn makes it easier for the user to preview the color.
This also makes it clearer that the Pick button can only pick colors in the editor window, not outside.
See https://github.com/Orama-Interactive/Pixelorama/issues/53 for more information.
Preview

Thanks!


make reboot more compatible with the rhts-reboot command (kill $PPID)

Some tests are using following approach to make sure the test does not continue past the rhts-reboot call.
rhts-reboot
sleep 3600

The rhts-reboot actually kills the $PPID to stop the invoking test.
Just add the kill $PPID into the tmt-reboot script to get closer to the original implementation. There should not be any issues with adding that.

Good catch, also thanks for describing the solution, I wasn't aware of the $PPID variable.


current date and time

current date and time

current date and time

current date and time

You are now blessed with Open Source Super Powers!
Congrats!


Bugs/typed tuple on zod

Status
WIP
Description
Support for typed tuples as they are produced by pydantic v2.0+
Reproduction for issue https://github.com/orval-labs/orval/issues/1592

in generateZodValidationSchemaDefinition, the array switch case has to be altered to adhere prefixItems constraints. If prefixItems exists,

[x] a tuple has to be pushed into functions instead of array,
[x]  each prefixItem has to be parsed and nested into the tuple
[x] if items exists, a rest function has to be pushed after the tuple with the schema in items

in parseZodValidationSchemaDefinition:

[x] logic to add positional parameters to tuple
[x] logic for the rest functions has to be added

somewhere:

[ ] prevent .min().max() on tuple.

[x] Tests added

Run npm run format to format the code.  Also for min() and max() i assume jus tlook at where min and max are being createed in the code and you porbbaly need to ignore that if its a tuple?

More issues that I would like to know if there is a better solution:

I have to cast to oas31.SchemaObject and back to oas30.SchemaObject because prefixItems is a oas 3.1.0 thing. Is there a better way? Without casting?

I think this is done now. Please review and comment (and merge)

tests are still failing?

Ok. I see. Fakerjs got triggered because I accidentially re-used a existing test oas file. I reverted the hijacked file and added an explicit one. Hope thats ok this way.

Running build now

Did huskies job and formatted the code


go vet

LGTM. Thanks.

@ccding Can you update the commit message to *: fix go vet reported issues


bug: Crypto not defined in envVars cli installer

Provide environment information
  System:
    OS: macOS 14.1
    CPU: (16) arm64 Apple M3 Max
    Memory: 4.21 GB / 64.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.18.2 - ~/.nvm/versions/node/v18.18.2/bin/node
    Yarn: 1.22.21 - /opt/homebrew/bin/yarn
    npm: 9.8.1 - ~/.nvm/versions/node/v18.18.2/bin/npm
    pnpm: 9.12.1 - /opt/homebrew/bin/pnpm

Describe the bug
The envVariables cli runner fails to install because of an undefined package:
Adding boilerplate...
✔ Successfully setup boilerplate for drizzle
✔ Successfully setup boilerplate for tailwind
⠋ Boilerplating envVariables...Aborting installation...
ReferenceError: crypto is not defined

We should see this:

Adding boilerplate...
✔ Successfully setup boilerplate for drizzle
✔ Successfully setup boilerplate for tailwind
✔ Successfully setup boilerplate for envVariables
✔ Successfully setup boilerplate for eslint

Reproduction repo
https://github.com/t3-oss/create-t3-app
To reproduce
Scaffold with:

TypeScript
Tailwind: Yes
tRPC: No
Auth: None
ORM: Drizzle
App Router: Yes
Run 'pnpm install': Yes
Default Imports: use default

Additional information
No response

I also got the same problem

i got this problem too

i'll soon be releasing the new version where we use node:crypto instead of global crypto
ooc, what your node version? globalThis.crypto has been around for a bit now

18.20.4

nvm im using node v20 and its work

18.20 seems like it should work too

I just tested it out as per the latest update. Node v18 doesn't work, even though v20 does work.


[Staff & Volunteers] Add headshot, bios

George Everts, School Outreach Specialist
David Lim, Development Officer

Scholarship Committee

Nell Shamrell-Harrington

Closing this out due to lack of activity. Please feel free to open a new issue if this is still needed.


Roulette Wheel to Spin constantly

Is there a way to trigger the roulette wheel to spin automatically and to never stop?
Im guessing this function needs to be called but struggling to figure it all out...
spin(speed) {

    const newSpeed = this.rotationSpeed + util.getRandomInt(speed * 0.85, speed * 0.15);

    this.setRotationSpeed(newSpeed);

    this.onSpin({
      event: 'spin',
      direction: this.rotationDirection,
      rotationSpeed: this.rotationSpeed,
    });

}

Hey @ayush-lal,
this.rotationSpeed controls how fast the wheel is spinning, so you could just comment out this line:
// Decrease rotation (simulate drag):
this.rotationSpeed += (this.rotationResistance * delta) * this.rotationDirection;

Thanks heaps! Do you know how I could go about triggering the Roulette Wheel to spin automatically as soon as the page has loaded?

Try this:
wheel.init({
  settings,
  rotationResistance: 0, // 1. Ensure the spin won't slow down over time.
});
wheel.setRotationSpeed(30); // 2. Spin the wheel, speed 30.


Initial draft for Sales Engineering

Just had a chance to read this and I think it looks great. There's a lot of stuff in there about being a good problem solver and teammate and everything which I love. I do think though that there could be more about being good advocates for our customers. Overall though 👍


ValueError: operands could not be broadcast together with shapes (4,224,224) (3,)

How is normalize in image_utils.py suppose to work on images with 4 dimensions?
File ~/tensorflow-metal/lib/python3.8/site-packages/transformers/image_utils.py:143, in ImageFeatureExtractionMixin.normalize(self, image, mean, std)
    141     return (image - mean[:, None, None]) / std[:, None, None]
    142 else:
--> 143     return (image - mean) / std

Here's an example from:
https://github.com/MaartenGr/Concept
https://github.com/MaartenGr/Concept/issues/12
Several images in this collection have shape
pix = numpy.array(image)
pix.shape
(960, 640, 4)

To download 25k images:
import os
import glob
import zipfile
from tqdm import tqdm
from sentence_transformers import util

# 25k images from Unsplash
img_folder = 'photos/'
if not os.path.exists(img_folder) or len(os.listdir(img_folder)) == 0:
    os.makedirs(img_folder, exist_ok=True)

    photo_filename = 'unsplash-25k-photos.zip'
    if not os.path.exists(photo_filename):   #Download dataset if does not exist
        util.http_get('http://sbert.net/datasets/'+photo_filename, photo_filename)

    #Extract all images
    with zipfile.ZipFile(photo_filename, 'r') as zf:
        for member in tqdm(zf.infolist(), desc='Extracting'):
            zf.extract(member, img_folder)
img_names = list(glob.glob('photos/*.jpg'))

Here's the code:
from sentence_transformers import SentenceTransformer, util
from PIL import Image

#Load CLIP model
model = SentenceTransformer('clip-ViT-B-32')

#Encode an image:
img_emb = model.encode(Image.open('.notebooks/photos/7Y0ZVBWCfNw.jpg'))

#Encode text descriptions
text_emb = model.encode(['Two dogs in the snow', 'A cat on a table', 'A picture of London at night'])

#Compute cosine similarities
cos_scores = util.cos_sim(img_emb, text_emb)
print(cos_scores)

---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Input In [6], in <cell line: 2>()
      1 #Encode an image:
----> 2 img_emb = model.encode(Image.open('/Users/davidlaxer/Concept/notebooks/photos/kYobqI1URDg.jpg'))
      4 #Encode text descriptions
      5 text_emb = model.encode(['Two dogs in the snow', 'A cat on a table', 'A picture of London at night'])

File ~/tensorflow-metal/lib/python3.8/site-packages/sentence_transformers/SentenceTransformer.py:153, in SentenceTransformer.encode(self, sentences, batch_size, show_progress_bar, output_value, convert_to_numpy, convert_to_tensor, device, normalize_embeddings)
    151 for start_index in trange(0, len(sentences), batch_size, desc="Batches", disable=not show_progress_bar):
    152     sentences_batch = sentences_sorted[start_index:start_index+batch_size]
--> 153     features = self.tokenize(sentences_batch)
    154     features = batch_to_device(features, device)
    156     with torch.no_grad():

File ~/tensorflow-metal/lib/python3.8/site-packages/sentence_transformers/SentenceTransformer.py:311, in SentenceTransformer.tokenize(self, texts)
    307 def tokenize(self, texts: Union[List[str], List[Dict], List[Tuple[str, str]]]):
    308     """
    309     Tokenizes the texts
    310     """
--> 311     return self._first_module().tokenize(texts)

File ~/tensorflow-metal/lib/python3.8/site-packages/sentence_transformers/models/CLIPModel.py:71, in CLIPModel.tokenize(self, texts)
     68 if len(images) == 0:
     69     images = None
---> 71 inputs = self.processor(text=texts_values, images=images, return_tensors="pt", padding=True)
     72 inputs['image_text_info'] = image_text_info
     73 return inputs

File ~/tensorflow-metal/lib/python3.8/site-packages/transformers/models/clip/processing_clip.py:148, in CLIPProcessor.__call__(self, text, images, return_tensors, **kwargs)
    145     encoding = self.tokenizer(text, return_tensors=return_tensors, **kwargs)
    147 if images is not None:
--> 148     image_features = self.feature_extractor(images, return_tensors=return_tensors, **kwargs)
    150 if text is not None and images is not None:
    151     encoding["pixel_values"] = image_features.pixel_values

File ~/tensorflow-metal/lib/python3.8/site-packages/transformers/models/clip/feature_extraction_clip.py:150, in CLIPFeatureExtractor.__call__(self, images, return_tensors, **kwargs)
    148     images = [self.center_crop(image, self.crop_size) for image in images]
    149 if self.do_normalize:
--> 150     images = [self.normalize(image=image, mean=self.image_mean, std=self.image_std) for image in images]
    152 # return as BatchFeature
    153 data = {"pixel_values": images}

File ~/tensorflow-metal/lib/python3.8/site-packages/transformers/models/clip/feature_extraction_clip.py:150, in <listcomp>(.0)
    148     images = [self.center_crop(image, self.crop_size) for image in images]
    149 if self.do_normalize:
--> 150     images = [self.normalize(image=image, mean=self.image_mean, std=self.image_std) for image in images]
    152 # return as BatchFeature
    153 data = {"pixel_values": images}

File ~/tensorflow-metal/lib/python3.8/site-packages/transformers/image_utils.py:143, in ImageFeatureExtractionMixin.normalize(self, image, mean, std)
    141     return (image - mean[:, None, None]) / std[:, None, None]
    142 else:
--> 143     return (image - mean) / std

ValueError: operands could not be broadcast together with shapes (4,224,224) (3,)

Packages:
print(transformers.__version__)
import transformers
print(transformers.__version__)
4.11.3
__version__
import sentence_transformers
print(sentence_transformers.__version__)
2.1.0
import numpy
print(numpy.__version__)

1.20.3

pix = numpy.array(image)
pix.mean()
71.51163167317708
pix.std()
106.86995132579301
pix.shape
pix.shape
(960, 640, 4)

I also encountered the same problem. Pictures with QR codes will inevitably have such problems.

You could convert your image to RGB with PIL before encoding it. In your case, it would be:
#Encode an image:
img_emb = model.encode(Image.open('.notebooks/photos/7Y0ZVBWCfNw.jpg').convert("RGB"))


RETURNING docs

Thanks! This needs to go somewhere else in the doc but I'm not sure where. I will fold it in next time I work on docs.


Can`t download the 1.0.18-SNAPSHOT@aar version from mavenCenter

So I have to use the 1.0.17 version

sorry, I just deployed it, it should be there now
https://oss.sonatype.org/#nexus-search;quick~tourguide


Research defining a NPSP REST API

With the payments integration Manelsky said he wanted to work on during the DC sprint.

During the NY Engineering Summit, we discussed implementing the REST API defined by OSDI (http://opensupporter.org/). It seems like both NPSP and NGOC teams agree adhering to an existing standard already used by other companies would be advantageous. Then every third party company that wishes to integrate with either of our applications only needs to target one API.
The specification can be found at https://github.com/opensupporter/osdi-docs.

OSDI notes (to list all possible concerns with the spec in one comment):

Only given_name and family_name available for person name. It doesn't support foreign last names, that might be composed of multiple last name parts.
The options for postal_addresses.address_type are only "Home", "Work", or "Mailing". Would "Mailing" be enough to support vacation addresses?

**lurch: attach W-007297

Tracking W-007297

Lots of global classes ending in _API are available in our codebase.


feat(changelog): Introduce property to skip merge commits in the changelog. Resolves #858

Fixes #858
Context
The idea is to add a property "skipMergeCommits" and if set to true, merge commits will be skipped.
According to git reference we can rely on the number of parents to identify merge commits.
Checklist

[x] Review Contribution Guidelines.
[x] Make sure all contributed code can be distributed under the terms of the
Apache License 2.0, e.g. the code was written by
you or the original code is licensed under a license compatible to Apache License 2.0.
[ ] Update documentation when applicable.

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

I tested locally the new property in this repo

Thank you @margarita-nedzelska-sonarsource. Please remember to read the CLA and sign it if you agree with it. You'll find the link in this PR.


Add "Incubating" to nested class names?

wdyt of renaming DbSystemValues to DbIncubatingSystemValues to make usage clearer when importing the nested class directly (and also possibly also since individual enum attribute values can be declared stable / experimental and so it may be more common to need both DbSystemValues to DbIncubatingSystemValues)
cc @lmolkova @jack-berg

Seems like a good idea!


[Feature Request] add background-repeat declaration to custom reset styles

While we may have recently added Normalize CSS  as a formal dependency, I still think it is worth evaluating the merits of best practices evolving outside of it. Something to consider is the motivation behind the new CSS Remedy project. It is seeking to create a base stylesheet that:

sets CSS properties or values to what they would be if the CSSWG were creating the CSS today, from scratch, and didn't have to worry about backwards compatibility

I think the historical default value of the background-repeat property is a perfect example of this. While repeating patterns may have been common in the early web, they are now the exception, not the norm. As such I think we should consider changing preflight's custom reset styles to include the universal removal of background repetition.
*,
*::before,
*::after {
  background-repeat: no-repeat;
  border-width: 0;
  border-style: solid;
  border-color: config('borderColors.default', currentColor);
}

Ended up releasing v1 without really digging in to this and don't want to add it since it would be a breaking change, so for now best bet is to add this to your own base styles if you want this behavior by default.
Reasonable reasons not to make this change:

It's still fairly common to use repeating backgrounds, I just used a repeating background for the Tailwind docs site actually
It could have surprising unwanted effects on third-party libraries that expect things to repeat by default. Our existing border reset messes up some stuff in Google Maps for example which is already a bummer.

Agree in principal that it's nicer to add bg-repeat to make something repeat than it is to add bg-no-repeat to undo default browser behavior, just don't want to make the breaking change.


[core]Incomplete search results and white screen for decompiled results

link:https://drive.google.com/file/d/1yEzUlBNPa8Jm7H2QozTKfgHM6q47VOuV/view?usp=sharing
A search for tv_value produced only these results:

However, in com.yuanrenxue.match2022.adapter.RefreshHeadViewAdapter.OooO0Oo(@NonNull final RecyclerViewHolder recyclerViewHolder, int i, OooO0OO oooO0OO) also exists tv_value, the search results are incomplete.

Also, the last result I got from the previous global search for tv_value, when I clicked in, was a white screen.

It's strange, because in my case, the bug has always existed.
I use is the latest jadx, source code version: c27f2ba, download the source code and compile it yourself to use.
I'll upload my jadx zip file here:https://drive.google.com/file/d/1vjURdNV01g-dgB-PxoefVcY0Couw0XDd/view?usp=sharing
jadx decompiled this apk to get the info level logs:https://drive.google.com/file/d/16asu2GRE33gjg9Iom9nPSgGpyODI3LFH/view?usp=sharing
My jadx configuration:https://drive.google.com/file/d/1UNTfeNu50qaCrahEhyb9cCqJGLcH_Wlv/view?usp=sharing
windows11
java -version(Use java8 to compile and use jadx)
openjdk version "1.8.0_322"
OpenJDK Runtime Environment (Temurin)(build 1.8.0_322-b06)
OpenJDK 64-Bit Server VM (Temurin)(build 25.322-b06, mixed mode)

@bhwxha thanks for sharing jadx config! Turns out this issue can be reproduced only with Inline anonymous classes option.
I found the cause and committed a fix :+1:

@skylot Thank you for the fix, the above bug has been solved.
However, I have found suspected new bugs:
Lookup reference to method com.yuanrenxue.match2022.adapter.RefreshHeadViewAdapter.OooO0Oo(@NonNull final RecyclerViewHolder recyclerViewHolder, int i, OooO0OO oooO0OO), the result is 0.

jeb:

@bhwxha I committed a fix, but the actual issue is slightly different.
The method name here is different, because it was inlined/merged with a bridge method, so jadx usage should show usage of that bridge method (I fixed that, also override annotation was missing). So if you want to compare with JEB, I suggest disabling all inline options in preferences (Inline methods for this case), this will show both methods, and it will be easier to compare usages.

@bhwxha I committed a fix, but the actual issue is slightly different. The method name here is different, because it was inlined/merged with a bridge method, so jadx usage should show usage of that bridge method (I fixed that, also override annotation was missing). So if you want to compare with JEB, I suggest disabling all inline options in preferences (Inline methods for this case), this will show both methods, and it will be easier to compare usages.

Ok, thanks for the fix.


updated index page

/approve

/approve

/approve

/approve

/hold

/hold

/hold cancel

/hold cancel

/lgtm

/hold cancel

/hold cancel

/lgtm cancel

/hold cancel

/lgtm

/approve

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

Needs approval from an approver in each of these files:

OWNERS [jstrachan]

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

/lgtm cancel

/approve cancel

[APPROVALNOTIFIER] This PR is APPROVED
Approval requirements bypassed by manually added approval.
This pull-request has been approved by: rawlingsj
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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

/approve cancel

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

Needs approval from an approver in each of these files:

OWNERS

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

/meow

/woof

/woof

/woof

/meow

/woof

/woof

/meow

/woof

/meow

@jstrachan:

In response to this:

/meow

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository.

/woof

/meow

@jstrachan:

In response to this:

/meow

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository.

/woof

/test all

/test tekton

/test this


Async move blocks not resolving properly

In the following code:
use async_std::task::block_on;
use std::future::Future;

struct A<T> {
    #[allow(dead_code)]
    val: T,
}

impl<T> A<T> {
    fn new(fut: impl Future<Output = T>) -> Self
    {
        A {
            val: block_on(fut),
        }
    }
}

impl A<()> {
    fn extra_fn(&self) {}
}

async fn returns_unit() {}

fn main() {
    let a = A::new(async move { });
    a.extra_fn();

    let b = A::new(returns_unit());
    b.extra_fn();
}

extra_fn in a.extra_fn(); is not resolved because the type of a is A<Future::Output<()>>, but it is resollved in b.extra_fn(); because the type of b is A<()>.
The type of a should resolve to A<()>.

Duplicate of #4018.


what is the timeline for support to airflow 2.8

Checks

[X] I have checked for existing issues.
[X] This report is about the User-Community Airflow Helm Chart.

Motivation
hi
we wonder what is the timeline for support to airflow 2.8 ?
as for now we use airflow 2.7.3 python 3.9 and our security team discover that there is more then 290 vulnerabilities.
The severities of the CVEs are as follows:
Critical : 10
High : 64
Medium : 58
Low : 160
The following 92 dependencies have vulnerabilities<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>zlib1g@1:1.2.11.dfsg-2+deb11u2, openssh-client@1:8.4p1-5+deb11u2<EMAIL_ADDRESS><EMAIL_ADDRESS>bind9-host@1:9.16.44-1~deb11u1, bind9-libs@1:9.16.44-1~deb11u1<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>72 more`
and 80% of the vulnerabilities are fixed in 2.8 version
Implementation
No response
Are you willing & able to help?

[ ] I am able to submit a PR!
[x] I can help test the feature!

+1 for a chart update to support Airflow 2.8 🙏

+1 for a chart update to support Airflow 2.8 🙏

+1 for airflow 2.8 support

Hey everyone, I am also eager to "officially" support Airflow 2.8 on the chart!
There is a chance that no additional changes are required to use it with the version 8.8.0 of the chart, (in which case, we can just update the documentation to say that).
Has anyone encountered any problems so far when using the 2.8.X container images with version 8.8.0 of the chart?
If people are willing to test (preferably in a test environment, NOT production) and you find anything, please reply to this issue so we can see if there are any big problems.
@Olaktal @yossisht9876 @Jarzamendia @vizeit

Also, please be aware that (in my experience) upstream airflow takes a while to stabilize any new release version, so there might be non-chart related issues or regressions.
(This is why the chart doesn't use the latest airflow version by default).

Hey everyone, I am also eager to "officially" support Airflow 2.8 on the chart!
There is a chance that no additional changes are required to use it with the version 8.8.0 of the chart, (in which case, we can just update the documentation to say that).
Has anyone encountered any problems so far when using the 2.8.X container images with version 8.8.0 of the chart?
If people are willing to test (preferably in a test environment, NOT production) and you find anything, please reply to this issue so we can see if there are any big problems.
@Olaktal @yossisht9876 @Jarzamendia @vizeit
Also, please be aware that (in my experience) upstream airflow takes a while to stabilize any new release version, so there might be non-chart related issues or regressions.
(This is why the chart doesn't use the latest airflow version by default).

I have installed 2.8.0 & 2.8.1 on multiple test environments(GKE clusters) using chart version 8.8.0 and have not noticed any issues so far

Hey everyone, we have released version 8.9.0 of the Helm Chart with official support for Airflow 2.8 and 2.9!


different output from io.FlowReader and io.FlowWrite

Goal:
Reduce size of mitmdump file, by removing large response entries.
Code:
from mitmproxy import io

with open(file,'rb') as file_in:
    fr = io.FlowReader(file_in)
    with open(f'{file}.out', 'ab') as file_out:
        fw = io.FlowWriter(file_out)
            for n,f in enumerate(fr.stream()):
                fw.add(f)

Got a new file, but with different sha256, so i used diff:

1,4c1
< 9112:4:type;
< 4:http;
< 7:version;
< 2:18#9:websocket;
---
> 9052:9:websocket;
35c32,33
< 8:HTTP/2.0,}17:timestamp_created;
---
> 8:HTTP/2.0,}6:backup;
> 0:~17:timestamp_created;
44,46c42,54
< 6313:4:via2;
< 0:~11:cipher_list;
< 0:]11:cipher_name;
---
> 6277:3:via;
> 0:~19:timestamp_tcp_setup;
> 17:1692359990.453262^7:address;
> 47:37:firefox.settings.services.mozilla.com;
> 3:443#]19:timestamp_tls_setup;
> 18:1692359990.4875603^13:timestamp_end;
> 0:~15:timestamp_start;
> 18:1692359990.4108424^3:sni;
> 37:firefox.settings.services.mozilla.com;
> 11:tls_version;
> 7:TLSv1.3;
> 11:cipher_list;
> 0:]6:cipher;
49c57,58
< 16:2:h2,8:http/1.1,]16:certificate_list;
---
> 16:2:h2,8:http/1.1,]4:alpn;
> 2:h2,16:certificate_list;

Is it normal, that generated output diffs original?
If yes, is it possible to keep old file format when rewriting file?

I can't really tell from the diff: is the flow the same with just reordered keys, or are we loosing information? The latter would be a bug we should fix. The former is less urgent, but something I'd be happy to review a PR for nonetheless, at least if it is possible to fix without adding complexity.


Address expire notification

we won't do it, it will produce too many notifications and save users' hassle only in limited number of cases


New release 1.2.8

== Break changes

N/A

== New features

Add support of filtering via NetStateFilter. (f128da4, c130bb6)
SR-IOV: Include SRIOV VF information per VF interface. (fd3c489)
New bond options: (77ede43)

BondInfo.lacp_active
BondInfo.missed_max
BondInfo.ns_ip6_target

== Bug fixes

Expose MptcpAddressFlag. (dd2a427)

Breaking nmstate for showing invalid bond options after switched to rtnetlink without filtering.


Text become fuzzy and unusable after scroll

VSCode Version: 1.12.0 Insider
OS Version: Windows 10 Version 1703

Steps to Reproduce:

Install it on a Hi-DPI screen device with 125% scale (May not require, it's my environment)
Paste some long code into editor
Take a screenshot without image compress
Scroll down mouse wheel by once, take another screenshot, or feel it with your eye

Screenshot

This is normal (clearly)

This is fuzzy

Screenshot with zoom

This is normal

This is fuzzy

Possible reason
I think it's an upstream issue with Chromium, if Code use CSS transform to scroll/move element will cause this issue.
I hope develop team can take a look with this, this make Code unusable at all
See also

Symbol & Text may fuzz and unclear (#7799) [upstream, electron]

@alexandrudima lets sync on this when we are both in the office, looks like a Chrome regression.

@ArianeLu I am on a Surface Book with 200% scale and regardless of the scroll top position the editor font is rendered with subpixel antialising:

Sometimes, when changing the scale, I need to sign out and sign back in to Windows for all the apps to react to the change. Does signing out help in any way?

Could you perhaps share your settings? (i.e. @bpasero @aeschli is it now possible users can define transparent backgrounds via themeing?)

@alexandrudima
Noting to share, It's fresh install, without custom settings.
If i changed screen scale, it should more fuzzy in all time, not just after scroll
My user settings:
{
    "workbench.iconTheme": "vscode-icons",
    "terminal.integrated.shell.windows": "C:\\WINDOWS\\Sysnative\\bash.exe"
}

https://github.com/Microsoft/vscode/issues/22336#issuecomment-293188857
This commend author may have same issue

I also did a quick test on my macBook pro both with my external monitor and the notebook screen and I am seeing subpixel antialising with any DPI scaling (100%, 125%, 150%). Would also like to understand what kind of setting could cause this.
Btw please only look at the editor and not anywhere else! We did loose subpixel antialising outside the editor but that is Electron unrelated (I can see this with 1.11 stable already).

@ArianeLu when you say in your screenshot "this is fuzzy", it seems to me from that screenshot that subpixel antialising is still being used, can you clarify that is different from the screenshots and what produces the fuzzyness?

@bpasero Look the char "L" on two screenshot, as you say, it looks like subpixel antialising, but it was difficult to read on my external monitor (DELL P2416D).
The fuzzy looks like have a shadow around the char, I don't think subpixel antialising will cause text fuzzy like this one. Something maybe wrong...
I can't reproduce this issue on VSCode 1.10.2.
Relative Code:
Element with class .monaco-editor-background have inline attribute transform: translate3d(0px, -455px, 0px)

translate3d(0px, 0px, 0px) - Good
translate3d(0px, -1px, 0px) - A bit fuzzy
translate3d(0px, -2px, 0px) - Unreadable
translate3d(0px, -3px, 0px) - Better than -2px, but still unreadable
translate3d(0px, -2px, 0px) - Good, text very clear

** I can't find this attribute on .monaco-editor-background with Code 1.10.2

And this one

@bpasero Can you understand my description? Or any idea with my guess about the transform attribute?

@ArianeLu
I think the root problem here is that Chromium decides to go to grayscale antialising on your machine/environment. Grayscale antialising is problematic exactly because, as you point out, it cannot correlate the font rendering with real screen pixels.
If you believe it is due to our usage of translate3d, then there is an easy way to test that: use the setting "editor.disableTranslate3d": true in your settings. You should then observe all the editor dom nodes being positioned with top or left and no longer with transform.

@alexandrudima here is an update from what I see after doing more testing with the editor.disableTranslate3d setting. I can confirm that there is a regression in subpixel-aa when starting to scroll.
Translate3d: ON, Scrolling: NO

Translate3d: ON, Scrolling: YES

I lost subpixel-aa when scrolling.
Things look a lot better with translate3d disabled:
Translate3d: OFF, Scrolling: NO

Translate3d: OFF, Scrolling: YES

I should add that I am only getting subpixel-aa in the editor with translate3d turned off ever since I fixed https://github.com/Microsoft/vscode/issues/24532. For some reason, the entire workbench was using grayscale-aa when the workbench container (.monaco-workbench) was not having a background-color associated. Puzzles me why that has an impact though...
Remains to understand why Chrome turns the editor into grayscale-aa once scrolling starts. Since its fixed after disabling translate3d, it is probably that. Checking for chrome issues that might be related I stumbled upon https://bugs.chromium.org/p/chromium/issues/detail?id=521364

@bpasero I don't think this is happening on all environments. e.g. it is not happening for me. What is the setup that triggers it? 125% scaling in the OS (mine is configured with 200% scaling)?

@alexandrudima I am running with 125% DPI but I also see this with 100% as well as 200%.

@bpasero Do you sign out of Windows and sign back in between changes to the scaling setting? There are numerous apps (I wouldn't be surprised if Chromium is one of them) that cannot react to changes to scaling unless you sign out of Windows. (I'm trying to find out what is the root cause of this)

@alexandrudima wow, you are right, subpixel-aa works in 200% after doing a windows restart. So maybe this is only an issue when the multiplier is an odd percentage?

According this tweet https://twitter.com/paulcbetts/status/852050882523000832 this only affects non-integral DPIs

Really said, for my device, 100% is too small and 150% is too large.
Hack it then work with 126%? LOL, That's impossible

@ArianeLu We will look into a solution. My favorite resolution would be to disable our usage of translate3d only in the cases where this issue is hit (i.e. not for folks running at 100% or 200%) due to the scrolling performance implications.

I could reproduce. Here are the values I get (with window.zoomLevel: 0) for:

Scaling factor
Zoom Level
Zoom Factor
Pixel Ratio

100%
0
1
1

125%
0
1
1.25

150%
0
1
1.5

I'm looking into disabling the usage of translate3d in the editor when the pixel ratio is not an integer.

@alexandrudima awesome, thanks!

@bpasero @alexandrudima Text become darken and tiny after open built-in terminal, I can't find out why

Look like this one

#26800


[.NET] Hindi DateTime Set enhancements

Support for additional cases in Set (3 in Extractor and 1 in Parser).
All new cases pass.

Just to confirm, these are just new specs to guarantee coverage, right? Do they differ in form from the previous ones? Or where they missed translations from the first round?

right, they were missed translations from the first round.


feat(grammar): capture comment directives

Adds additional capture group for comment directives. The purpose is to allow editors relying on tree-sitter for higlight to provide different higlight for directive comments that fullfil other than documenting purpose in the code.
The syntax of the directive comments is described in the syntax section of the comment documentation.
Checklist:

[ ] All tests pass in CI.
[ ] There are sufficient tests for the new fix/feature.
[x] Grammar rules have not been renamed unless absolutely necessary.
[x] The conflicts section hasn't grown too much.
[x] The parser size hasn't grown too much (check the value of STATE_COUNT in src/parser.c).

@aryx sorry, seems like my change didn't pass the pipeline: https://github.com/tree-sitter/tree-sitter-go/actions/runs/3628335707/jobs/6119224709. I didn't notice these tests. I can prepare revert if necessary (and will take a look at the changes one more time).

yes please send a revert PR

also @dcreager @maxbrunsfeld why those github actions are not run on the PR itself?

It used to work ... who changed it?


main.py is not running

File "src/main.py", line 43
logging.info(f"pong: {r.status_code} {r.content}" )
^
SyntaxError: invalid syntax

Which version of python are you using? This code is only compatible with Python 3.6+

(venv) RKMadugulas-MacBook-Pro:server yesbro$ python --version
Python 2.7.10
(venv) RKMadugulas-MacBook-Pro:server yesbro$ python3 --version
Python 3.8.2

@catchwayrk your virtual environment is using the 2.7 version by default . create a symlink to python3. so that you can get the 3.8 version

Or create the virtual environment like this:
virtualenv -p python3 venv
which will use python3 in the virtual environment

Thanks @UchihaSR @williamsiuhang ! Added to the doc


Upgrade htmlparser2

Ref https://github.com/fb55/htmlparser2/releases/tag/v5.0.0

@abea Is this good to go?

Sorry for the delay @TrySound. I'll review this early this week.

@TrySound Thanks for your patience with us on this. It's been a busy time.

That sounds reasonable. https://github.com/apostrophecms/sanitize-html/pull/429#discussion_r543485923


corrected the mistakes

Thank you @aaquib19 for your PR!
Since we want to keep the mistakes for other participants to work on, we wouldn't be merging this PR. Hope the practice was good nevertheless :)


Add profile to disable DocLint missing option

Summary
This PR add a pom profile for disabling DocLint missing option (sometimes this option is too strict) .
Background, Problem or Goal of the patch
N/A.
Design of the fix, or a new feature
N/A.
Related Issue, Pull Request or Code
N/A.
Wanted reviewer
N/A.

BUILD SUCCESS. <jenkins-prb-asakusafw-sdk-93>


Improve the way locale is set in the user account

When self-register regardless of the browser locale, 'en' is set as the local claim of the account.
local claim needs to be enabled by default to have above functionality, otherwise, the local claim is not set in the account.
Additionally, from the developer portal if we can capture the locale claim and give a dropdown to select the user's locale for admins that would be ideal.
From the user-portal also, I'd like to see a drop-down with available locales for the locale claim.

Yes, its really a must to have feature, getting user local from identity system is really very important when securing other application with identity server and we have to show right user local in those applications along with identity server.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Allowing complex var names;

Removing the else if block in order to allow complex var names on the left hand of the assignment.

Please fix the jshint error so that the build will pass. Also, jshint plugins/packages for your code editor are really helpful if you don't have one installed.
See https://travis-ci.org/raptorjs/marko/jobs/54955691

Funny, because I did the edits directly in Github. :-)
--
Sean Gates
Sent from my iPhone

On Mar 18, 2015, at 5:15 PM, Phillip Gates-Idem<EMAIL_ADDRESS>wrote:
Please fix the jshint error so that the build will pass. Also, jshint plugins/packages for your code editor are really helpful if you don't have one installed.
See https://travis-ci.org/raptorjs/marko/jobs/54955691
—
Reply to this email directly or view it on GitHub.

Published<EMAIL_ADDRESS>Thanks Sean!

You bet!
--
Sean Gates
Sent from my iPhone

On Mar 18, 2015, at 6:15 PM, Phillip Gates-Idem<EMAIL_ADDRESS>wrote:
Published<EMAIL_ADDRESS>Thanks Sean!
—
Reply to this email directly or view it on GitHub.


Server Error bug when path long

This fixes the SQL problem SQLSTATE[22001]: String data, right truncated: 1406 Data too long for column 'path' when user attempt to access very long URL path or accessed by some vulnerability scanner (e.g. Qualys PCI DSS scanner).
Steps to reproduce:

Go to your app URL.
Enter very long path that exceeds 255 characters.
An SQL error will appear on logs and page will return 5xx error instead of 404.

Example error log (executed by Qualys VAPT)

Thanks @woenel 👍  This is definitely a good fix.


Uuid -> Leafref at Notifications

All *Notification::objectIdRef attributes will become _objectIdRef attributes generated from shared associations to LTP class in UML. This applies on all notifications.
All *notification::object-id-ref leaf will become leafref to LTP.
Only object-deletion-notification::object-id-ref : leafref will be "require-instance false;"

All object-id-ref have been made require-instance false, because there was no way in Papyrus to make the UML2YANG tool to not add this statement.

All object-id-ref have been made require-instance false, because there was no way in Papyrus to make the UML2YANG tool to not add this statement.


Show all tasks does nothing when a webview is opened in the background

#93641
Version: 1.44.0-insider (user setup)
Commit: d7d1147d6b764a48b789000c03088ee9afabcd8a
Date: 2020-03-31T08:23:56.446Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18362

Works fine when a file is opened, not an extension (webview) though.

Is this a wider problem with multi-step quick picks?

Looks similar to #75209

This is an interaction between QuickInput and the WebView. When the first picker closes, it passes focus back to the editor which passes it to the webview element. The second picker then opens and takes the focus only to loose it immediately after.
I don't see 'focusin' nor 'focusout' events for the webview element, so that element seems to somehow be special. @mjbvz Do you have an idea of what might be going on here?


Getting local path of image or video

Now I can see in data array, that uri path is only remote (asset-library on ios), but how to get full local path?

can u send here the Data array response your getting from the plugin ?

Array [
Object {
"creationTime":<PHONE_NUMBER>000,
"duration": 0,
"filename": "IMG_6925.PNG",
"height": 1334,
"id": "7A70C176-78BC-43FA-B6D2-9032934306E0/L0/001",
"mediaSubtypes": Array [
"screenshot",
],
"mediaType": "photo",
"modificationTime":<PHONE_NUMBER>960,
"uri": "assets-library://asset/asset.PNG?id=7A70C176-78BC-43FA-B6D2-9032934306E0&ext=PNG",
"width": 750,
},
Object {
"creationTime":<PHONE_NUMBER>000,
"duration": 0,
"filename": "IMG_6929.PNG",
"height": 1334,
"id": "B5166E29-1E56-4630-8A0D-A71A8DCA9CF9/L0/001",
"mediaSubtypes": Array [
"screenshot",
],
"mediaType": "photo",
"modificationTime":<PHONE_NUMBER>698,
"uri": "assets-library://asset/asset.PNG?id=B5166E29-1E56-4630-8A0D-A71A8DCA9CF9&ext=PNG",
"width": 750,
},
]

i see this is a mediaSubtypes object , i didnt think about it because im an android user , this
"mediaSubtypes" means that the photo get shot with multi images saved , i will search for a solution for this issue
generally for now it will work only with "image" or "video" type , this "mediaSubtypes"
are Set of photos that iphone auto generate , you can disable it but yet ...
i will probably gonna check if image type is "mediaSubtypes" and then select the first image from the "set" as the asset,
OR
Ignore (dont let select this type of Roll images)
will see

@Ozzi48 can u share your package json ?

No, like for images everything is okey. I can use "assets-library://" uri, because when I push image to firebase it's works, but when I push video with remote uri, it is writing me that it is wrong path. And I have checked in forums that video should have local full path, like "file://".

If I use expo-image-picker I got his url from picked picture "file:///var/mobile/Containers/Data/Application/4BDCCE81-65BF-4834-A435-777E6388F199/Library/Caches/ExponentExperienceData/%2540ozzi38%252FVisionogy/ImagePicker/D474BD06-B3F7-4863-BCC0-9356FFCEBD2A.png" , but when I use this package I get the link like "assets-library://asset/asset.JPG?id=8360229E-BBF6-4143-91E6-4BC486E793A5&ext=JPG"
So, is it possible to get full path , like from first package?

still try to understand why its happen , because im dont have real iphone device but oneplus 8 i cant really find why
will keep investigate this

hi i checked it and only when using manipulate images actually created on other folder but you should be able to upload them too
can you try with this snack=> https://snack.expo.io/@natysoz/expo-images-picker-example
see what results ur getting ?

any update?

@natysoz, Hi im using your package and i have the same problem, on iOS, it returns an assets-library type. On Android it works flawlessly.
Object {
"creationTime":<PHONE_NUMBER>000,
"duration": 0,
"filename": "IMG_0027.PNG",
"height": 2208,
"id": "C3583236-E568-418C-BABB-8008525CD0E7/L0/001",
"mediaSubtypes": Array [
"screenshot",
],
"mediaType": "photo",
"modificationTime":<PHONE_NUMBER>433,
"uri": "assets-library://asset/asset.PNG?id=C3583236-E568-418C-BABB-8008525CD0E7&ext=PNG",
"width": 1242,
},
I tried to find something to convert it into a normal local file but i had not success

adding on the next version .

added in new v2.0.0


FEATURE: Replayable projections

WIP.
TODOS:

[ ] Merge #75
[ ] Implement ProjectionManager::catchup()

Resolves: #47

Thanks! This goes in a good direction.
I think the part adding the EventStreamFilter could be easily moved out and merged quickly, since we already pretty much agreed on that. That would keep this PR a bit more concise and easier to review.

@albe Thanks for the feedback! Very good point regarding the identify map. It didn't work at first, but that was a different issue apparently. I gladly removed the runtimeCache now ;)
Regarding the other parts: I added some comments. Keep in mind that this is just work in progress at this point.
One fundamental thing I'm not happy about yet is:
To replay/catchup a projection I currently use the eventTypes stream filter.. But those can be a lot of types in a large app. The common case would be the streamNamePrefix that replays all events for a certain aggregate type (also for example geteventstore.com won't allow to filter streams by event type).
IMO we need a mechanism that is a bit more clever/flexible here

Keep in mind that this is just work in progress at this point.

Yep ;) I just like to get concrete feedback ASAP myself, so if I can I try to do likewise on other's WIP PR

The common case would be the streamNamePrefix that replays all events for a certain aggregate type (also for example geteventstore.com won't allow to filter streams by event type).

Yeah. I had a similar insight on our project after implementing the "replay concrete projections by finding the event types". I'm currently debating with myself, if the use case really is:
'Replay Projection X' instead of 'Replay Aggregate X', or, more generically, 'Replay EventStream X'.
The latter two are problematic if a Projection listens to Events from different Aggregates/Streams...
From a Read-Side POV only the first (or at most the last) makes sense of course...
I think in getEventStore this would be solved by defining a new indexing Stream with linkTo() https://geteventstore.com/blog/20130218/projections-5-indexing/
So basically, every projection could have it's own indexing stream and to replay, it would just query that stream from the start.

Less than 5 minutes to import 125'000 events and around 30 seconds to replay a projection that need all the events ... nice performance by the way.

OK time to sleep, I think we can merge #75 and when catchup method is done this one is mergeable, or maybe we can even merge it and work on the catchup in an other PR
@bwaidelich Thanks a lots for this, really nice work

I merge this one, ProjectionManager::catchup() can be implemented in a separate PR


Implement Y movement for TouchpadWalking

The VRTK_TouchpadWalking.cs script does not allow us to traverse the Y axis, due to float fixY = transform.position.y. This means that players fall through the game level (or fly) while traversing hills, slopes, etc.
Relevant code here:
        public void Move()
        {
            var deviceDirector = VRTK_DeviceFinder.DeviceTransform(deviceForDirection);
            var movement = deviceDirector.forward * movementSpeed * Time.deltaTime;
            var strafe = deviceDirector.right * strafeSpeed * Time.deltaTime;
            float fixY = transform.position.y;
            transform.position += (movement + strafe);
            transform.position = new Vector3(transform.position.x, fixY, transform.position.z);
        }

If you apply the Player Presence script to your camera rig, it will add a collider and rigidbody that will allow the touchpad walking to traverse Y correctly.
Check out the example scene to see this working.

Wow that was super easy. Works perfectly. Thanks!


[BUG]

it wasnt running, i asked chat gpt why and it gave me this:
The script looks pretty solid at first glance, but I noticed a small typo that might be causing issues. In the following line:
echo Running mouse_yoke.py... >> "%LOGFILE%""

There's an extra double quote at the end. It should be:
echo Running mouse_yoke.py... >> "%LOGFILE%"

Remove that extra double quote, and it should be good to go.
Apart from that, if you're still encountering issues, it would be helpful to know what kind of problems or error messages you're getting. If there's a specific error or behavior you're trying to address, let me know, and we can troubleshoot it together! 🤙😄

Hi Pedro, although the typo described above is correct, it is irrelevant and shouldn't affect the behavior of the script.
Could you provide, if any, the latest log generated by the script?
What Python version are you running? Are you able to see it when typing > python -V in a terminal?

Closing this issue as no answer was given.