Identifiers validation failed for dataset Los mamíferos del estado de Durango, México

Identifier validation failed for the dataset Los mamíferos del estado de Durango, México:

Crawler attempt: 307
Publishing organization: Comisión nacional para el conocimiento y uso de la biodiversidad
Publishing country: MEXICO
Installation: IPTCONABIO
Number of occurrences indexed for this dataset at the time of the issue: 6419
Cause: GBIF ID problems exceed 50% threshold: 100% duplicates; 6328 total records; 6328 absent records
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3740394/step

New IDs sample:

Old IDs sample:

5cd8886d985beba7b1586cae3b8f4abe
7a3eba3c19a17e0feaafe408cb54f933
0d5e71fe452de378c2c59319bcee4d5a
3be65dc75aa599714e2af304c8277a73
b8e8e8193bb5da18d9e6c25f8a37e112
ee8780af86b7424e613e846d74928209
08ba94cf0832ac4c66fcd5349512f269
0e823726932c9d091768364d0d5f1a01
1437f56923e20a611617422a3006bf35
157a79d53f8f485c29c28f9178a6a552

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Los mamíferos del estado de Durango, México : https://doi.org/10.15468/jt3nwh.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.

Identifier validation still  failing for the dataset Los mamíferos del estado de Durango, México:

Crawler attempt: 308
Publishing organization: Comisión nacional para el conocimiento y uso de la biodiversidad
Publishing country: MEXICO
Installation: IPTCONABIO
Number of occurrences indexed for this dataset at the time of the issue: 6419
Cause: GBIF ID problems exceed 50% threshold: 100% duplicates; 6328 total records; 6328 absent records
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3741211/step

New IDs sample:

Old IDs sample:

5cd8886d985beba7b1586cae3b8f4abe
7a3eba3c19a17e0feaafe408cb54f933
0d5e71fe452de378c2c59319bcee4d5a
3be65dc75aa599714e2af304c8277a73
b8e8e8193bb5da18d9e6c25f8a37e112
ee8780af86b7424e613e846d74928209
08ba94cf0832ac4c66fcd5349512f269
0e823726932c9d091768364d0d5f1a01
1437f56923e20a611617422a3006bf35
157a79d53f8f485c29c28f9178a6a552

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Los mamíferos del estado de Durango, México : https://doi.org/10.15468/jt3nwh.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.


Reactive form  are working on local projects but not on playground its the issue of untouched form

playground is setup for this on demand

Could you please send us a saved session which is reproducing the issue?

ok will send you the link of playground as i get some time


[Feature Request] Personal website link in footer

I would like to add a link to my personal website in the footer, however it is not possible right now.

you can try like this

copyright = "Nishant Srivastava - hosted in AWS. ©{year}, All Rights Reserved"

I just made the footer text markdonwify.
So, you can do this
...
copyright = "[Nishant Srivastava](https://www.example.com) - c2015~"

I think we can close this issue.
Feel free to reopen if something wrong.

This works, thanks for this!
Just a small thing though, the link opens in the same page instead of opening a new page. Can this be done?

I can't control the markdown syntax link right now.
It could be possible using rendering hooks.
But it will make the whole link to open external.
Hugo doc says

currently only possible to have one set of render hook templates, e.g. not per Type or Section. We may consider that in a future version. ↩

If it's ok in your case(by default open links external) you can make a hook.
I just copy-pasted the official Hugo site document
layouts/_default/_markup/render-link.html
{{ .Text }}

I tried what @talz26 suggested and that worked too :thinking:
copyright = "Nishant Srivastava - ©2015 - {year}, All content is licensed under "

Oh, it also good
I didn't notice it.
Markdown makes a tag anyway.
That is the easest solution


[devsite] add clarifications to multicast addressing primer

Resolves #5153.

LGTM

Thanks for sorting these out, everyone 👍


Add backend logic for bounding box plots

main <- #5227 <- #5241 <- this <- #5305
This PR updates our comparison dvc output fixture with bounding box info and updates the backend accordingly. It also adds logic for a user toggling boxes on/off.

Going to go ahead and merge this and #5305 into our first PR since they've been sitting for a while and are approved.


Create bubble.py

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Any PRs should be done to the update branch only
Type of change
Please delete options that are not relevant.

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

Browsers Tested on?

[ ] Chrome
[ ] Firefox
[ ] Edge

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have checked the responsiveness of the component I built
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] My changes generate no new warnings

Spam.


s/Ini/INI/

According to Go conventions of initialism:

Words in names that are initialisms or acronyms (e.g. "URL" or "NATO") have a consistent case. For example, "URL" should appear as "URL" or "url" (as in "urlPony", or "URLPony"), never as "Url"

So the struct Ini should ideally be ini or INI. Since it's an exported struct the latter is the only option. I got the code changes successfully built and tested by doing the following:
gofmt -s -w -r 'Ini -> INI' -d *.go
Since the struct Ini doesn't actually appear in user code (as illustrated in the examples), renaming it shouldn't require anything more than a recompilation of the user code.

Oh indeed you are right.
Thank you.


Mac: Update xattr names, DNS bundle identifier, and magic number in header

We need to drop the "gvfs" part of the xattr names anyway, might as well do it before we get any real users.
And I'm not sure I have a strong opinion on what the magic number in the header ought to be. Is the current value good enough?

There's also the reverse DNS bundle identifier for the kext and the corresponding class names which still use io.gvfs and io_gvfs_, respectively. (Doing a find-all in Xcode on those strings should bring up all instances.)

I've noticed that some identifiers use com.vfsforgit. while others use org.vfsforgit. Which one are we standardising on?
(I'm having a go at implementing this fix as I've got a bit of time while waiting on PR feedback for all my other open tasks.)

Thanks to PR #751, the only remains of the io.gvfs. prefix are in legacy-supporting/upgrade code which is on track to be removed somewhere down the line (#891).


Correction and bug note for Ink Man

100 defeats instead of 125. Level 30-40 Eagle, Serpent, and Tiger Blue Ink Men count, and all level 50-54 Blue Ink Men count. Level 30-40 Dragon Blue Ink Men don't count, due to a likely oversight in badges.bin.

Thanks. I'll add your comment in as a note as well, in case anyone wants to know the reason why certain ink men don't count.


Optimize algorithms for size ?

Description

Type: Question
Priority: Blocker

Question
I'm integrating Bluetooth Low Energy secure connections in mbed OS BLE. This feature requires the generation of Diffie-Hellman keys and the computation of a shared secret. Mbed TLS being integrated into mbed OS; it seems like the right candidate to achieve the task.
Unfortunately the overhead in term of stack memory is too important for us (more than 3K); code size overhead is non negligible either.
Thanks to ARMCC static stack analysis capability I was able to locate the functions consuming too much stack:

mbedtls_mpi_fill_random allocates on the stack an array of MBEDTLS_MPI_MAX_SIZE bytes regardless of the number of random bytes requested. The default value of MBEDTLS_MPI_MAX_SIZE is set to 1024 bytes. That may be fine for desktop or embedded linux platform but it is very large for mbed OS application; the size of mbed OS stacks being by default 4K large.
mbedtls_internal_sha512_process allocates on the stack an array W of 80 uint64_t (640 byte) to realize the computation. This array is used to pre-compute values that will be used later by the algorithm .

I've solved 1. using a variable length array instead of a fixed length array (N/A in mbed tls); As the code exit if the size in input is larger than MBEDTLS_MPI_MAX_SIZE it is not more vulnerable than the existing code. Other solutions like dynamic memory allocation of the temporary buffer or iterating in loop the cycle random number generation + copy  can be considered.
I've solved number 2. by removing W pre-computation and delaying computation to the point of use (my solution here). I've also removed loop unrolling; in the end that's 3.6 to 3.9X less code (1.8K to 2.5K) generated and 512K of bytes of stack saved but at the expense of a 30% to 80% performance degradation for the function. Benchmarked as a whole in my application the performance degradation observed is less than a percent.
After these two changes more than 1.5K of bytes are saved on the stack and I'm able to use mbed TLS.
Now I'm coming to my questions:

What is the recommended strategy to fix large stack allocation ?
Have you considered a compile time flag that would favor size over speed optimizations ? With such flag it would be possible to have algorithms implementation that are crafted specifically for their footprint regardless of their performance. In my case I cannot see myself submitting a PR that yield 80% performance degradation for all users even if the footprint gains are large.

Hi @pan-
To answer your questions:-

What is the recommended strategy to fix large stack allocation ?

We don't really have one. This is more a known problem that we're working on.

Have you considered a compile time flag that would favor size over speed optimizations ? With such flag it would be possible to have algorithms implementation that are crafted specifically for their footprint regardless of their performance. In my case I cannot see myself submitting a PR that yield 80% performance degradation for all users even if the footprint gains are large.

That's not really how we work today. We have a lot of fine-grained configurations held in config.h which allows a sophisticated user to tweak the library for code-size over performance. We're working on making those configurations work at a higher level, and in a more easily configurable way, but that work is still in the pipeline.
Does that satisfy your questions?

Hi @sbutcher-arm
Thanks for the quick reply. It perfectly answers my questions. Now I know what I have to do to submit my enhancements.


Node layering and interaction with iOS VoiceOver

Related to #852 - There is a layering issue with iOS VoiceOver. This was discovered in phetsims/gravity-force-lab#210 with @zepumph.
The screen reader virtual cursor can be on a Node that is behind another in the z-order. If a user gesture is made to initiate dragging of the Node that is in the back, the node in front will receive the input events. This is because the assistive device sends a pointer event at the location of the DOM element which is under the virtual cursor. But the the event target is the Display div. Scenery catches this event and dispatches it normally and so the top most interactive Node receives the event.
We should investigate if there is any way around this. Perhaps we can get information from the DOM event such as the DOM element under the virtual cursor which initiated the event. From here we can look up the trail to the Node the user intended to interact with.

@jessegreenberg and I aren't going to work on this again until it because an issue with a sim. Removing from meeting.


WatchKit

Thanks for this great framework!
It would be nice to add support for the WatchKit UI elements...

Thanks! That's great idea. Putting it in TODO list. I don't have much time at the moment though.

Sure!
I just tried to do it myself starting with WKInterfaceLabel, however I was blocked by the fact that WatchKit properties are write only - so there is a setText but you can’t read back the current value of the text property!
For this we might need write-only Dynamic ?

On 21 May 2015, at 13:15, Srđan Rašić<EMAIL_ADDRESS>wrote:
Thanks! That's great idea. Putting it in TODO list. I don't have much time at the moment though.
—
Reply to this email directly or view it on GitHub https://github.com/SwiftBond/Bond/issues/66#issuecomment-104213197.

Not planned at the moment.


[NEW CONTENT TYPE REQUEST] RPG Maker VX rgss1a file

What type of file would you like magika to detect?
RPG Maker VX RGSS encrypted archive (.rgss1a)
What software can create/open these files?
RPG Maker RPG Maker VX, RPG Maker Decrypter, RGSS Decryptor.
Where can these files be found?
Many RPG Maker RPG Maker VX games. Use RPG Maker RPG Maker VX to compress your own.
If possible, please provide a specification for this file type.
Additional context

Closing, see #152 for context. Thanks!


GD-262: After installing gdUnit3 v2.2.0, Tests do not appear and "generate test" tooling does not create new tests.

$ dotnet --version
6.0.300
Visual Studio Code Version: 1.67.2
The used Godot version:
godot v3.4.4 stable mono official version
OS including version:
Edition	Windows 10 Pro
Version	21H2
Installed on	3/17/2021
OS build	19044.1706
Experience	Windows Feature Experience Pack 120.2212.4170.0
Processor	AMD Ryzen 9 3900X 12-Core Processor               3.79 GHz
Installed RAM	32.0 GB
System type	64-bit operating system, x64-based processor
NVidia GeForce RTX 2080 SUPER
GeForce Game Ready Driver
Version: 512.95
Describe the bug
After installing gdUnit3, Tests do not appear and "generate test" tooling does not create new tests.
Steps to Reproduce
Test not showing
Steps to reproduce the behavior:

Perform the installation via these steps (using 2.2.0 .Net 6 and Lang version 10) .
https://mikeschulze.github.io/gdUnit3/first_steps/install/

Generate Test fails
Steps to reproduce the behavior:

Open vscode to cs file to be tested.
Using cursor select method to test in cs file.
Right click
Select "Create GdUnit Test"

Expected : New file to be generated
Actual Output:
Generate Test Case ...
Can't generate test case! 100 null

Minimal reproduction project:
GDTest_Example.zip

Thanks for reporting this bug.
I'm able to reproduce it with the attached example.
I will take a look deeper tomorrow and provide a fix soon as possible

Ok here my findings.
There are two problems in your project

You have configured a custom test folder Tests which is not taken into account when running the test, I will file an extra bug for that
Your test-suite has no namespace, which is the cause that the test-suite is not found.

Therefore currently valid until these bugs are fixed:

Always span a namespace around the test-suite.
Use the default settings for test folders.


Welcome -> Recent -> More... no longer works

Type: Bug
After updating to the latest version, the "More..." link in the "Recent" section on the Welcome page no longer works - nothing happens when clicking on it. It is supposed to open a dropdown/search box with more recent projects that you have worked on.
VS Code version: Code 1.82.0 (8b617bd08fd9e3fc94d14adb8d358b56e3f72314, 2023-09-06T22:09:41.364Z)
OS version: Darwin x64 22.6.0
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz (12 x 2200)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
3, 3, 2

Memory (System)
32.00GB (1.25GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (22)

Extension
Author (truncated)
Version

emojisense
bie
0.10.0

markdown-emoji
bie
0.3.0

vscode-markdownlint
Dav
0.51.0

vscode-eslint
dba
2.4.2

vscode-axe-linter
deq
4.7.2

gitlens
eam
14.3.0

EditorConfig
Edi
0.16.4

prettier-vscode
esb
10.1.0

vscode-github-actions
git
0.26.2

vscode-pull-request-github
Git
0.70.0

rainbow-csv
mec
3.7.0

dotenv
mik
1.0.1

vscode-docker
ms-
1.26.0

vscode-kubernetes-tools
ms-
1.3.13

vsliveshare
ms-
1.0.5883

vscode-yaml
red
1.14.0

code-spell-checker
str
3.0.0

intellicode-api-usage-examples
Vis
0.2.8

vscodeintellicode
Vis
1.2.30

vscode-todo-highlight
way
1.0.5

vscode-import-cost
wix
3.3.0

vscode-caddyfile-syntax
zam
1.0.4

It works for me, though I'm on Windows.
If you try it after running the Developer: Toggle Developer Tools command in the palette do you get any clues?

Might also be worth testing after running the command to disable all installed extensions temporarily.

Thanks @gjsjohnmurray, I have tried both of your suggestions but unfortunately the result is the same, nothing happens. No output in the developer console either.

I am seeing the same issue on Windows.
Version: 1.82.2 (user setup)
Commit: abd2f3db4bdb28f9e95536dfa84d8479f1eb312d
Date: 2023-09-14T05:55:25.390Z
Electron: 25.8.1
ElectronBuildId: 23779380
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621

The functionality itself seems to be working fine, as I can access the complete list without any issues through the File > Open Recent > More... menu

Also been noticing this issue over the last few weeks. Disabling extensions has no effect.
Version: 1.82.2 (user setup)
Commit: abd2f3db4bdb28f9e95536dfa84d8479f1eb312d
Date: 2023-09-14T05:55:25.390Z
Electron: 25.8.1
ElectronBuildId: 23779380
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19045

Same issue here for at least a month (probably more).
Version: 1.83.1 (Universal)
Commit: f1b07bd25dfad64b0167beb15359ae573aecd2cc
Date: 2023-10-10T23:46:55.789Z (5 days ago)
Electron: 25.8.4
ElectronBuildId: 24154031
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 22.6.0 (macOS Ventura 13.5.2)

Sharing an X thread about this issue.
It looks like people who experience the issue might the ones who see the GitHub Copilot ad on the right side of the welcome page.

Commented about this on X, but would be helpful if someone could run a Bisect here and see if that isolates the issue.

I ran a Bisect and issue is still present.

I'm unable to repro this scenario also but investigating recent changes in this area to see if there are any side effects.

Can you open the developer tools and search for showMoreRecents. This should be found in gettingStarted.ts. Slap a break point on that method (~line 366) and see what happens when you click "More".

Just did this but it looks like it's not reaching the breakpoint. In fact, it's not even reaching the parent method runDispatchCommand.
Should there the be a third click handler on the More button?

Thanks for checking. I'll dig deeper to see what could be happening. I'm still no repro for the scenario though.

We've disabled featured extensions. This issue should also be fixed in the upcoming release.


Internal audit of governance inc tooling+hotfixes

Work in pairs, or divide-and-conquer.
Goal is to produce a brief document we can walk-through as a group that covers the following:

[ ] How does this work?
[ ] What are the dependencies?
[ ] What design docs or specs cover this feature? Any gaps? Is the spec wrong, or the implementation?
[ ] Are there areas missing unit tests?
[ ] What e2e tests cover this? What could be added?
[ ] Are inputs validated?
[ ] Are parameters and constants set sensibly?
[ ] If we discover a serious issue, how many hrs to a full fix inc data recovery?
[ ] What is the path for upgrading or changing this feature (e.g governance, backwards-compatible validator upgrade, hard fork, ..)?  Does this difficulty match the quality and completeness?
[ ] How could this be attacked?
[ ] Are the user docs correct and complete? Links to code?

Share document, then comment on the doc, then let's have the walkthrough meeting before end of Sprint.

In the governance process is written that contract upgrades can be proposed by every actor in the system known or unknown.
Any user may submit a “Proposal” to the Governance smart contract, along with a small deposit of Celo Gold. This deposit is required to avoid spam proposals, and is refunded to the proposer if the community decides the proposal is worthy of voting on.
How to detect malicious actors / malicious code in this process? Once the code has been proposed it will be stored as bytecode. How to make sure (i.e. the code changes when upgrading smart contracts) being voted on is not harmful?
There is a spam protection in place (with Celo gold at stake). Potential other mechanism to rule out bad code could be some sort of attestation or testimony by the community or other voters. To enable this the voters need details about the changes, like actual code changes or a delta of the code before and after.


fix: new regex for short & long de description

Description

Updated REGEX to allow all special characters in DE & EN for service onboarding short and long descriptions

Why

Input validation REGEX was blocking entering text in DE description boxes of Service Release Process when using some special characters. It also make sense to do the same for EN incase their are scenarios where the user would like to mention their company name which might also include umlauts and special characters.

Issue
#965
Checklist
Please delete options that are not relevant.

[x] I have followed the contributing guidelines
[ ] I have performed IP checks for added or updated 3rd party libraries
[ ] I have created and linked IP issues or requested their creation by a committer
[x] I have performed a self-review of my own code
[x] I have successfully tested my changes locally
[ ] I have added tests that prove my changes work
[ ] I have checked that new and existing tests pass locally with my changes
[ ] I have commented my code, particularly in hard-to-understand areas

change should also be reviewed by @MaximilianHauer once it's ready for review


Clipboard imports now get named as "(clipboard)" instead of "clipboard"

Describe the bug
One side effect of #2720 is that clipboard imports now get named "(clipboard)" by default instead of "clipboard". Was this intended?
Since this is not something I was expecting I would propose to only include this in 3.5, not 3.4, since there is a risk that there are other unforeseen consequences. Not that I think this particular change is necessarily bad (we might actually want to keep the brackets), just that this is not something I was expecting.
To Reproduce
Steps to reproduce the behavior:

Type in anything in the clipboard import area
Validate
See the default project name in the top right text input

Current Results
The project name is "(clipboard)" by default.
Expected behavior
So far "clipboard" was the default.
Additional context
This new name seems to be generated in the backend, so I was about to propose to localize it but unless we manage to generate it in the frontend instead, it might not be doable at the moment.

That's weird. It was not intended and I can't think of what would cause it. Are you sure it's related to #2720?

I tested before and after the commit and observed a difference… I have not investigated further to find the root cause though.

I see what's going on. It was always "(clipboard)":
https://github.com/OpenRefine/OpenRefine/blob/749704518c330c6f96ec3e784c45b8c055d5c6cd/main/src/com/google/refine/importing/ImportingUtilities.java#L249
but the more stringent filter was getting rid of the parentheses. Now that we're relaxed the filter, they make it through.

I think changing this on the backend introduces more risk than it removes. In particular, it will require adjusting tests.
I'm not 100% convinced that it needs fixing at all, but in the interests of getting the release out the door, here's the line that needs adjusting:
https://github.com/tfmorris/OpenRefine/blob/55171a85ebf0b5145d295b5f18e6805070e0b989/main/webapp/modules/core/scripts/index/default-importing-controller/parsing-panel.js#L63
[not sure why the permalink isn't rendering code]
We could add parentheses "(" ")" to the list of characters which aren't legal in project names or special case the string "(clipboard)".
Preferences?

Since this seems to only affect new projects made? And old projects would still load just fine with a name like "clipboard" (like the 100's I have), then I think leaving it as new updated "(clipboard)" name is fine for users.  It definitely is fine for me if it pops up when I next create a project from a clipboard. As a user, I personally don't see the parentheses ( ) as a problem.

We could add parentheses "(" ")" to the list of characters which aren't legal in project names or special case the string "(clipboard)".
I'm definitely against this

Adding the parentheses seems slightly odd, but I don't think it matters a huge amount. I'm happy to see them left in for now

"For now"? Surely if we want to revert that back at some point, it should be done now, to avoid releasing things that we roll back later, no?

I really don't like the solution to trim the parentheses - that seems like a hammer to crack a nut
I think we should either not include the parentheses when we name the project, or not worry about it
I'm happy to not worry about it (for now or forever)

Ok, I agree. For me the safest way forward would be to remove the commit which created this change from 3.4, leave it in 3.5, and so 3.5 will have "(clipboard)" instead of "clipboard" and that will be fine. That does not require any messing around with either the frontend or the backend, and follows the principle of only including safe fixes with no unintended consequences in a release branch.

It's hard to believe that a purely cosmetic minor change to a field default value generates such a volume of discussion. The user can change the default to anything that they want.
Reverting the change means that only ASCII alphanumerics will be included from filenames. No multinational characters. No punctuation, such as the " (2)" that Mac OS X adds to disambiguate duplicate filenames. It will undo the fix to the 3 year old bug report about multinational character project names #1352.
The PR #2812 is misleadingly named because it strips parentheses from all filenames (including the above mentioned ones generated by OS X), not just "(clipboard)".
Yes, risk management for minor releases is important, but I'm having a hard time viewing this as a significant risk. If we're going to change it, though, the change should be an improvement. Since the value needs to be translatable anyway, I've special cased it to replace the internal "(clipboard)" with the translated value for "Clipboard," so French users will see their default clipboard project name as "Presse-papier" rather than "clipboard." #2776

add parentheses "(" ")" to the list of characters which aren't legal in project names

This was poorly worded. It's actually the list of characters which are stripped from filenames when creating the default project name. I'm not actually a fan of stripping any characters, but rather than remove it altogether, the list has been trimmed to a minimal set.
In any case, we still need to internationalize hardcoded English values.

Sorry, but it is my role to be a bit stubborn on this:

this patch introduced an unexpected change that has high visibility to the user
it generates a high volume of discussion indeed
it does not address a bug that was introduced recently
so it should not be introduced between a beta release and a final one.

It is not the end of the world if this only gets released in 3.5 - we have lived with the previous behaviour for years.
Also, now that I think of it, it is not clear to me why we should strip any characters from project names at all.

Sorry if it feels a bit feels a bit rigid, but I have sore memories of previous failures to argue for the stability of releases, most importantly the change in the list facet that we released in 3.1 which was an epic fail. I understand you want your patches released as soon as possible, and as you can see in other PRs I am not systematically that stubborn!


Bug Report: Character Editor being laggy and refusing to work.

Describe your bug here.
Sometimes, the character editor just does not let me type and lags. This is a problem with normal Psych 0.5.2h as well.
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
I didn't edit anything to do with the character editor, and this also happens on the pre-compiled version as well.

the typing is if you zoom out the camera or move it the text box wont update size/pos idk why

You need to press R to fix the text input
Might not work every time


Return py tree

This patch is demonstrated to work in chia-blockchain here: https://github.com/Chia-Network/chia-blockchain/pull/6435


Use a verb singular form at the beginning of one sentence

Storage Explorer Version: 1.10.1
Build: 20191015.4
Branch: master
Platform/OS: Windows 10/ Linux Ubuntu 19.04/macOS High Sierra
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Launch Storage Explorer.
Open Settings panel -> Transfers.
Check the descriptions under Maximum transfer rate & Network Concurrency & File Concurrency parts.

Expect Experience:
Use a verb prototype at the beginning of one sentence.
Actual Experience:
Use a verb singular form at the beginning of one sentence.

Won't fix. We're taking these strings directly from AzCopy, so we're going to defer to their grammar.


context menus not getting hooked up to commands in Insiders

i found this because my extension https://github.com/chrisdias/vscode-opennewinstance stopped working when you right click on a folder.  Same problem happens with Erich's npm extension.

VSCode Version: insiders 1.11.0 c995ecf
OS Version: MacOS

Steps to Reproduce:

clone https://github.com/chrisdias/vscode-opennewinstance, run npm install, open code-insiders
but a breakpoint in extension.ts on the line 16
press F5
in development instance, open some workspace that has folders in it
right click on a folder and choose "Open New Workbench Here"

Result: Nothing happens, breakpoint is not hit, the command is not executed.

Open command palette
execute command "Open New Workbench Here"

Result: breakpoint is hit.
However, notice that e is undefined.  in stable, it comes in as null.

Duplicate of #23162.


Es

https://metamask.zendesk.com/hc/en-us/articles/360015489591-Basic-Safety-and-Security-Tips-for-MetaMask

Ok


VHI get end of day DATE item ISO 8601 standard

Describe the bug
End of day in ISO: "An amendment was published in October 2022 featuring minor technical clarifications and attempts to remove ambiguities in definitions. The most significant change, however, was the reintroduction of the "24:00:00" format to refer to the instant at the end of a calendar day"
https://en.wikipedia.org/wiki/ISO_8601
To Reproduce
Steps to reproduce the behavior:

Go to https://sys-data.int.bgdi.ch/#/collections/ch.swisstopo.swisseo_vhi_v100/items/2023-08-30t235959
item name is: 2023-08-30t235959

Expected behavior
2. item name is: 2023-08-30t240000
Additional context
we do not add 2. item name is: 2023-08-30t235959z zulu since a) it is end of day and b) we would need to fix as well s2 sr wcih would break BGDI integration

easy fix: just edit https://github.com/swisstopo/topo-satromo/blob/012c0d6948cafb8dd1207660954acc424c0efc61/step1_processors/step1_processor_vhi.py#L356 @Tschoun


Argument is not assignable to parameter (SPHttpClientConfiguration)

Category

[ ] Question
[ ] Typo
[ x] Bug
[ ] Additional article idea

Observed Behavior
error TS2345: Argument of type 'SPHttpClientConfiguration' is not assignable to parameter of type 'SPHttpClientConfiguration'.
Steps to Reproduce
I use the following code:
WebPart.ts
import { SPHttpClient, SPHttpClientResponse } from '@microsoft/sp-http';
this.context.spHttpClient.get(this.context.pageContext.web.absoluteUrl + '/_api/web/lists?$filter=Hidden eq false', SPHttpClient.configurations.v1)    .then((response: SPHttpClientResponse) => {      return response.json();    });
Package.json
{   "name": "robeco-list-web-part",   "version": "0.0.1",   "private": true,   "engines": {     "node": ">=0.10.0"   },   "scripts": {     "build": "gulp bundle",     "clean": "gulp clean",     "test": "gulp test"   },   "dependencies": {     "@microsoft/sp-core-library": "1.5.1",     "@microsoft/sp-webpart-base": "1.5.1",     "@microsoft/sp-lodash-subset": "1.5.1",     "@microsoft/sp-office-ui-fabric-core": "1.5.1",     "@types/webpack-env": "1.13.1",     "@types/es6-promise": "0.0.33"   },   "devDependencies": {     "@microsoft/sp-build-web": "1.5.1",     "@microsoft/sp-module-interfaces": "1.5.1",     "@microsoft/sp-webpart-workbench": "1.5.1",     "gulp": "~3.9.1",     "@types/chai": "3.4.34",     "@types/mocha": "2.2.38",     "ajv": "~5.2.2"   } }
Offcourse I started a new SPFx Project.
Ran NPM outdate and updated the SharePoint elements to 1.6.
How can I solve this error?
Read these previous issue but got no clue what to do:
https://github.com/SharePoint/sp-dev-docs/issues/365
https://github.com/SharePoint/sp-dev-docs/issues/911

You need to change Response to SPHttpClientResponse. This is the correct way as the response will be in the type SPHttpClientResponse. JSON data will then be extracted from the response using the methods available.
Demo method:
`private _getListData(): Promise {
var queryUrl = this.context.pageContext.web.absoluteUrl + "/_api/web/lists/GetByTitle('${this.sharepointLists[this.properties.SharepointList]/items";

return this.context.spHttpClient.get(queryUrl, SPHttpClient.configurations.v1)
           .then((response: SPHttpClientResponse) => {
                    return response.json();
});

}`

Answered, so closing this. Thx @Seiyasu.


Crash on trying to enter a math calculation

Microsoft PowerToys version
0.68.1
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce
when I try to calculate a sum and I enter the first digits it hangs and then I can do nothing with the software I have to end task it to get it go away
so I press alt+space for the runner to come up then type in lets say 1234 and pause a little to type lets say + 1234 but it freezes up before i can type the plus
✔ Expected Behavior
I expect it not to freeze up
❌ Actual Behavior
Froze the application had to end task to close it
Other Software
no other software

/reportbug

PowerToysReport_2023-04-06-23-22-58.zip
hope this helps

Not able to repro this, going to close. Let us know if you see this again!


Issues with scheduler applying to RDS

Hello,
I am having issues with the scheduler working with a RDS MySQL instance.
I have deployed version v1.3.1. The configuration is setup for Both EC2 and RDS services. I have a schedule that works with EC2 instances correctly, but the same schedule applied to RDS (no multi-AZ, just a single MySQL master instance) does not work. I tried adding the default running period but no luck.
The State DynamoDB table shows the RDS as stopped and the stop tag has been applied. However, the RDS is still Active.
Thank you!
John

Hello. Thanks for reporting the issue @jwholme2. We tried testing the RDS scenarios and RDS instances stopped as expected when we tagged the relevant RDS instances. So we are not able to validate/reproduce issue from our end
Can you share the schedule argument details from DDB (config table)?


Remove the tmp-app from the deploy script

We've deprecated this in favour of the actual test environment in Azure, which gives us more flexibility.

 Remove the --tmp-app from the deploy script


Change installation instructions

While the installation instructions work well, they are hard for a node.js novice like me to follow.
I was attempting to install this bot on a Pi 3, and encountered some warnings and errors.
After some research, I have learned that the 'default' options could lead to issues down the road.
An error relating to the node-icu-charset (which appears to come from a missing library from discord-irc), can be fixed by installing certain packages.
https://github.com/mooz/node-icu-charset-detector#installing-icu
The installer shouldn't be run as root/administrator. This allows for permissions to be used properly (the user owns their local libs instead of root)
https://docs.npmjs.com/getting-started/fixing-npm-permissions#option-2-change-npms-default-directory-to-another-directory

This should be fixed with https://github.com/Throne3d/discord-irc/commit/bb2d4e6307317b5e593344676ad04b39ef07c542 - happy to merge more PRs that make the instructions more understandable though!


Added Blazorous and BlazorDB

When creating a new PR, please delete this template text and reference the issue number.
To automatically close the issue when the PR is merged, cite the issue number in the first line with ...
Fixes #Issue_Number
To only address an issue with a PR and not automatically close the issue when the PR merges, cite the issue number in the first line with ...
Addresses #Issue_Number

Moved BlazorDB to "Libraries" section


Fix race that would allow GetDateHeaderValues to return null

Fixes #866

@cesarbs ping

:ship:

Merged: cb284b9


./. GTs not being converted to 0/0

Original spec converted unknown genotypes to refhom (./. -> 0/0). This is "wrong" but necessary due to limitations of the Canrisk tool itself.
This conversion was removed during refactoring but is resulting in generation of useless VCFs (if unknown GTs occur).
Should we instead make the conversion but report which variant positions are affected?

Alternatively, is this considered a QC fail instead?


fix matplotlib imports, pd.util

CI was failing due to matplotlib changes.  This fixes those issues.

Thanks @Dr-Irv !

Create the matplotlib issue at https://github.com/matplotlib/matplotlib/issues/26812


Docs error and suggestion

On https://cogeotiff.github.io/rio-cogeo/API/ under 1. you have:
src_transform = from_bounds(*bounds, 1024, 1024)

src_profile = dict(
    driver="GTiff",
    dtype="float32",
    count=3,
    height=1024,
    width=1024,
    crs="epsg:4326",
    transform=dst_transform,
)

But this should read:
src_transform = from_bounds(*bounds, 1024, 1024)

src_profile = dict(
    driver="GTiff",
    dtype="float32",
    count=3,
    height=1024,
    width=1024,
    crs="epsg:4326",
    transform=src_transform,
)

i.e. transform=src_transform,
Also a suggested improvement is to show in from_bounds the height/width order
from_bounds(*bounds, width=1024, height=1024)

Thanks. Would you be able to make a PR with the docs change?

Thanks. Would you be able to make a PR with the docs change?

Another error spotted:
dst_profile = cog_profiles.get("deflate")

with MemoryFile() as mem_dst:
    # Important, we pass `mem_dst.name` as output dataset path
    cog_translate("my-input-file.tif", mem_dst.name, profile, in_memory=True)

should be
dst_profile = cog_profiles.get("deflate")

with MemoryFile() as mem_dst:
    # Important, we pass `mem_dst.name` as output dataset path
    cog_translate("my-input-file.tif", mem_dst.name, dst_profile, in_memory=True)

i.e. profile -> dst_profile
@kylebarron sure I will create a PR, doing some work with rio-cog so might find some more changes to include

Another error spotted:
dst_profile = cog_profiles.get("deflate")

with MemoryFile() as mem_dst:
    # Important, we pass `mem_dst.name` as output dataset path
    cog_translate("my-input-file.tif", mem_dst.name, profile, in_memory=True)

should be
dst_profile = cog_profiles.get("deflate")

with MemoryFile() as mem_dst:
    # Important, we pass `mem_dst.name` as output dataset path
    cog_translate("my-input-file.tif", mem_dst.name, dst_profile, in_memory=True)

i.e. profile -> dst_profile
@kylebarron sure I will create a PR, doing some work with rio-cog so might find some more changes to include

i.e. profile -> dst_profile.

👍

However I assume we can write straight to S3 without needing to write a local file and using cog_translate?

I'm not sure to understand, you mean trying to use Rasterio to write directly to S3? I never tried and as far as I remember it wasn't possible 🤷♂
just re-tried, and I don't think it's possible (at least with goal 2.4)
import rasterio
from rasterio.shutils import copy

with rasterio.open("file.tif") as c:
    copy(c, "s3://my-bucket/test.tif", **c.profile)

>>> CPLE_OpenFailedError: Attempt to create new tiff file 's3://my-bucket/test.tif' failed: No such file or directory

i.e. profile -> dst_profile.

👍

However I assume we can write straight to S3 without needing to write a local file and using cog_translate?

I'm not sure to understand, you mean trying to use Rasterio to write directly to S3? I never tried and as far as I remember it wasn't possible 🤷♂
just re-tried, and I don't think it's possible (at least with goal 2.4)
import rasterio
from rasterio.shutils import copy

with rasterio.open("file.tif") as c:
    copy(c, "s3://my-bucket/test.tif", **c.profile)

>>> CPLE_OpenFailedError: Attempt to create new tiff file 's3://my-bucket/test.tif' failed: No such file or directory

@vincentsarago the following approach is valid
with MemoryFile() as memfile:
    with memfile.open(**source_meta) as mem:
        # Populate the input file with numpy array
        mem.write(source_data, 1) # single band

    s3_client.upload_fileobj(memfile, destination_bucket, destination_key)

@robmarkcole it's already documented in https://cogeotiff.github.io/rio-cogeo/API/#using-the-api-with-in-memoryfile

aha so it is :-)


docs(docsWeb): readme full documentation link update to non broken link

PR Checklist
Please check if your PR fulfills the following requirements:

[x ] The commit message follows our guidelines: https://github.com/scullyio/scully/blob/main/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Other... Please describe:

What is the current behavior?
Readme full documentation link goes to a "Sorry, Could Not Load Static Page Content" page in the docs.
Issue Number: N/A
What is the new behavior?
Readme full documentation link goes to the Introduction page explaining what Scully is.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

 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.Matthew Magee seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


reduce log pollution (zerolog issues)

Description
Reducing a lot of log pollution.
once again, we're seeing these errors in production:
zerolog: could not write event: InvalidParameterException: Log events in a single PutLogEvents request must be in chronological order.

Fixes # (issue)
Type of change
Please delete options that are not relevant.

Refactor (refactoring code, removing useless files)

Testing steps
n/a
Checklist

[ ] make before_commit passes
[ ] updated documentation wherever necessary
[ ] added or modified tests if necessary
[ ] updated schemas and validators in insights-data-schemas in case of input/output change

/retest


AutoHorseshoe Not working

at first, it worked in the 1st or 2nd time. After that, it doesn't work anymore

Not maintained currently


ImagePickerWidget doesn't take "enable" property

ImagePickerWidget doesn't take "enable" property from ReactiveImagePicker

Hi @AlexSmirnov9107
Will take a look in 10 days after vacation

@AlexSmirnov9107 check v0.5.0
https://pub.dev/packages/reactive_image_picker


:white_check_mark: built version

Coverage remained the same at 98.419% when pulling fcbf1df3c85b87356ff2a37dd49b6f4492d429ae on tests/built into 36cdb97925efd4ad6eced8295f7cf1ce7b573987 on master.


Exercises command refactoring

Coverage increased (+0.07%) to 73.295% when pulling b7cc787d40d7c73ee2506539d386e2eaf2d1e264 on refactor-exercises-mikko into a9578fafe5eb5d8ad2613b3e41193ab353e344f2 on master.

Coverage increased (+0.05%) to 73.279% when pulling a80f634566fbc576a2302a5bc4297d73c5a2c175 on refactor-exercises-mikko into a9578fafe5eb5d8ad2613b3e41193ab353e344f2 on master.

:+1:

Coverage increased (+0.05%) to 73.312% when pulling 22301bfadea5b7c6c724ebffb943e29e2fb8e46c on refactor-exercises-mikko into adb5c4269446dd07b46ac177dcd3b4b68d2e5f51 on master.


Invalid ip format on connected in neo-cli

Invalid ip format on connected in neo-cli. Sometimes there will display an additional number on the end of ip.

Strange ... Any idea about how we can reproduce it? testnet, mainnet?

@shargon Run a node, mainet or testnet, show state after running for several hours.

We find the actual ip for this is <IP_ADDRESS>, but I'm confused why it sometimes random an additional number at the end if run for some hours.

This is worst xD

Is something related to print, i will check it

@superboyiii take a look to https://github.com/neo-project/neo-cli/pull/474

Let me test it.


Recursively check whole repository

It would be great if the bot could recursively check a whole repository.  Is that possible?

Closing as duplicate of #72.


ci/hubble: Fully remove Cilium installation

We are generating the Cilium YAML which contains more pods than just the
Cilium DaemonSet (the hubble-cli DaemonSet in this case). Therefore, we
should also remove everything we generate.

test-me-please

Coverage increased (+0.01%) to 44.762% when pulling d108766db910489711a83b7840431cf3db61e971 on pr/gandro/ci-hubble-delete-all into 89b622cf4e0a960e27e5b1bf9f139abee25dfea0 on master.

test-gke

test-gke


[WIP] MGMT-17001: add support for spec rollback

This PR is a sketch of adding support for rolling back device spec. This is still very much a draft and not yet intended for review.

this is being reworked will raise a new PR for review.


++cue jet crashes the runtime when the sample is a cell

Calling ++cue with a cell sample causes the runtime to crash non-recoverably:
(slum cue *^)

This is due to the fact that u3s_cue_atom is called without checking if its argument is an atom.

Good catch.

Another similar bug:
(slum max 0 0 0)
(slum min 0 0 0)


build: switch to tsup, un-bundle dependencies, and support cjs

This PR updates our build tool from rollup to tsup to remove dependencies from the output build. It also adds support for CommonJS when running validation programmatically.

Superseded by #39.


pyopengl: use secure pillow

Motivation for this change
Fixes https://github.com/NixOS/nixpkgs/issues/124590
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Added a release notes entry if the change is major or breaking
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 125352 at d072f0a8 run on aarch64-linux 1

3 packages marked as broken and skipped:

grass
impressive
metamorphose2

5 packages failed to build:

cq-editor
python39Packages.binwalk-full
python39Packages.enamlx
python39Packages.pyqtgraph
python39Packages.spyder_3

9 packages skipped due to time constraints:

gnuradio3_7
gnuradio3_7Packages.ais
gnuradio3_7Packages.gnuradio
gnuradio3_7Packages.gsm
gnuradio3_7Packages.limesdr
gnuradio3_7Packages.nacl
gnuradio3_7Packages.osmosdr
gnuradio3_7Packages.rds
python39Packages.rl-coach

7 packages built successfully:

cura_stable
displaycal
loxodo
plover.stable
python39Packages.pyopengl
torchat
winpdb

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.

Result of nixpkgs-review pr 125352 run on x86_64-darwin 1

11 packages marked as broken and skipped:

cq-editor
gnuradio3_7
gnuradio3_7Packages.ais
gnuradio3_7Packages.gnuradio
gnuradio3_7Packages.nacl
gnuradio3_7Packages.osmosdr
gnuradio3_7Packages.rds
grass
impressive
python39Packages.enamlx
python39Packages.rl-coach

3 packages failed to build:

plover.stable
python39Packages.binwalk-full
python39Packages.pyqtgraph

3 packages built:

python39Packages.pyopengl
torchat
winpdb


[FEATURE REQUEST] Searchengine

Is it possible to add a sitewide search into the static html-page? We got a pretty big archive and finding old posts will be much easier with a searchengine included.

I haven't thought about this, but I guess you could include some sort of a JS search plugin in your site template.

Suggestion for a static local search engine: Lunr
There's a Python implementation which produces files readable by the JavaScript version: https://pypi.org/project/lunr/

Will experiment with Lunr sometime, @joschi. The size of the index might get out of hand for big channels.

Why not search via MySQL. This would of course require that the database runs in a Mysql database.

@f0xcb MySQL is quite a heavy dependency and you'd need an active backend service talking to the MySQL database (which doesn't exist yet) in order to run searches.

MySQL is not necessary. tg-archive stores all data in an sqlite DB anyway, which has good full text support. It's possible to build a separate frontend which simply reads this DB file and presents a searchable site. That's not in the scope of tg-archive though, as it only aims to sync data and generate static sites.

I would like to have this feature too. My telegram group is deleted and I'm struggling to search for old but valuable content in chat history.

Relevant to this discussion: https://github.com/tinysearch/tinysearch

Would love to have server search as well.


option name showPolePoint is not same with YawPitchControl's

Description
YawPitchControl still have showPolePoint option name as old name showPole.
So, it makes weird behavior.
Steps to check or reproduce
var panoViewer = new PanoViewer(el, {
  fovRange: [30, 65],
  fov: 65
});

// then
panoViewer.getFov() === 65; // false. but it should me true.

fixed with this commit
https://github.com/naver/egjs-view360/commit/ed9d3e5d89f50444ab142353a77d2eadcd480bea


Support to react-native windows

Although Windows is a non-first-class citizen on react-native ecosystem, do you have plans to add support to it?
In case you don't, is there any concerns or red flags regarding adding support to it? Would you accept a PR adding support to it (considering that it follows the repository's standards)?
Thanks!

It would be nice to support windows and mac :)

@enverbisevac, there is already support for mac!

I opened a ticket - react-native-windows


styled TouchableOpacity does not forward the onPress prop

Current Behavior
Hi team! I found something when I was working with styled in a native only expo app.
Given this situation
import {  styled } from 'tamagui'
import { TouchableOpacity } from 'react-native'

const CustomTouchable = styled(TouchableOpacity, { bg: 'green' })

...

<CustomTouchable onPress={() => console.log('LOG!') />

When I press on this touchable, it won't print anything. It's working when I remove the second argument in the styled declaration! For some reason, if we provide a style object, onPress is not forwarded.
Expected Behavior
I expect that, if I wrap a TouchableOpacity using styled and providing a default style, it should still forward the onPress property!
Tamagui Version
1.27.3

Reproduction
https://github.com/Christopher2K/touchable-tamagui-bug/commit/dedf0eeebeeccf4accc7bbe8abcf7540a48c6bc3

System Info
System:
    OS: macOS 13.3.1
    CPU: (8) arm64 Apple M1 Pro
    Memory: 67.55 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 16.14.2 - ~/.asdf/installs/nodejs/16.14.2/bin/node
    Yarn: 1.22.19 - ~/.asdf/installs/nodejs/16.14.2/bin/yarn
    npm: 8.5.0 - ~/.asdf/plugins/nodejs/shims/npm
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Browsers:
    Safari: 16.4

This is actually just about intended, those are utility components that overlap with built-in behavior, I'm adding a docs not about this in a PR.

Thanks for the clarification!

hello, i am stuck in this trouble.
any docs for here?

I also faced this. So where are the documents?

 Because I would probably wrap a Touchable or Pressable class on the outside of the ButtonFrame. Thanks for this great @tamagui/core


Free Download Manager 5.1

You need to fill the PR template. You haven’t been doing so.

up & rebase

up & rebase


improve the performance Ink

Hello @ Someone Amazing on the Team Excalidraw  !
Is it possible to improve the performance of the tool freedraw ? I've seen #4726 you have already reduced the number of dots in the ink, can this be continued (if that's the reason for the performance to many dots) as now with a large number of lines starts hangs navigation, new lines appear with a delay, there is no such a thing in google keep. I suppose it could be because:

Too many dots in the line

[ ] Create a condition, the further, the less to reduce the load, take the beauty out of the strings, making it simple like google keep or one note.

It's as if smoothing line doesn't keep up with my hand - the characters become incomprehensible and have to be redrawn.

[ ] Add on/off "line smoothing" function

The peculiarities of the engine?

[ ] Add "pen and selection only" economy mode if that's the reason for performance - too much functionality.

Thanks for the attention!

Hello!

I've seen #4726 you have already reduced the number of dots in the ink, can this be continued (if that's the reason for the performance to many dots) as now with a large number of lines starts hangs navigation, new lines appear with a delay, there is no such a thing in google keep. I suppose it could be because:

In your video, the lag you're seeing comes from the fact there are already "too many" lines drawn on canvas which get re-rendered on each pen event as you draw. Simplifying the shape wouldn't help much as we'd still be rendering the scene on each pen movement (it would help only in that it would allow for more elements on scene before you started to notice slowdown) — we'd have to further throttle the pen events (which is where the initial big performance boost came from https://github.com/excalidraw/excalidraw/pull/4727).
For further performance improvement we need new rendering strategies and optimizations.

@dwelle, Do You think it is possible to achieve this kind of drawing performance in Excalidraw (ED) as in Google Keep (GK) ? I'm crazy about drawing in GK: I don't experience delays when changing tools, selecting colors, creating lines - my thought process goes on without delays. I understand ED recently turned 1 year old, GK is 9 years old, in Your opinion (it looks like you've been in the software creation environment since 2013, if not sooner, I'm asking you, because I haven't had that kind of experience yet), how soon would it be possible to achieve a similar result for ED ?
Thank you for your time and attention !

@dwelle I would also support better pen support. I once did some trials to make my own drawing app with an infinite canvas. I used fabric.js as a base library to do drawing.
Personally, I think that the performance in drawing with this approach is better than with your rendering engine perfect-freehand. Of course a lot of features are not present.

https://github.com/thfrei/infinite-drawing-canvas
Demo: https://thfrei.github.io/infinite-drawing-canvas/


Add typings for sql.base, sql.compiler, and sql.ddl

Blocked by #17. Once it lands, I'll probably have to rebase this PR on top of it before review.

This ticks the boxes for sql/base.pyi, sql/compiler.pyi, and sql/ddl.pyi in #3


Add a note about recommended hook ordering

The Ruff documentation remarks that the Ruff pre-commit hook should run before code formatting tools (see https://beta.ruff.rs/docs/usage/). I copied the sentence from the Ruff documentation to the README here to increase its visibility.

Good idea, thanks so much.


Request to merge

Migrated the project to null safety

@nickpatrick please merge this request

Thanks @Yczar ! I just merged your work directly into master. Closing this PR.


structural break test for SARIMAX and statespace models

Do you have any plan to implement a structural break test for SARIMAX and statespace models?
Thanks!

There have been discussions about this, but nothing formalized yet. You might be interested in the comments in #3497 as well as the example notebook I posted there.


FancyZones

Description of the new feature / enhancement
Some of my programs always open new windows in fullscreen. I would like to force a given program to always open in a specified window.
Scenario when this would be used?
Using FancyZones
Supporting information
No response

/dup https://github.com/microsoft/PowerToys/issues/11833


But I think this could be a nice little addition. I think of small pages witch dont have to be a static site with content. Maybe even just a info popup or overlay thing, witch goes over an existing content - but triggered by the router. I was into a case like that recently.
I made a little demo / test case (with a simple dynamic route). And it looks promising. The only two tings are (running the build sources):

A config addition, maybe in the static.config.js, like dynamicRoutes() ore even in the getRoutes config like ignoreExport: true. Then we would not get a 404 on the /dynamic/routeData.json file lookup.
Once we refresh / reload the page, the CSS gets not assigned. That happens as the css classes are not anymore defined in the HTML. Maybe only a CSS Modules issue.

In many ways, react-static is a nice starter-kit with much power, and can easily compete with many of the top rated projects on this site: www.javascriptstuff.com/react-starter-projects

So if I understand what you're saying, you want to use React-Static as a replacement for a non-static react app, eg create-react-app?

react-static looks like it could serve as a replacement for create-react-app for non-static sites with a few tweaks. I would want to use it in that way if possible.
I think pre-rendering as much as possible on a non-static react app is still be beneficial. Maybe certain components depend on dynamic content, but the majority of the site doesn't. Or maybe just a shell is static. Maybe some routes should be static and some not.
You can prerender an app made w/ create-react-app w/ react-snapshot or react-snap, but I have had trouble using this out of the box.
An option to ignoreExport at the route level. Or maybe a component/HOC that could be used to tell react-static that a certain portion of the site shouldn't be prerendered (I haven't looked at react-static internals, but maybe that means wrapping something like: https://github.com/kadirahq/react-no-ssr). Or both.
I think it would be awesome if react-static supported these use cases.

I'll start working on this then. Great input!
On Tue, Oct 31, 2017 at 5:45 PM Matthew Sisley<EMAIL_ADDRESS>wrote:

react-static looks like it could serve as a replacement for
create-react-app for non-static sites with a few tweaks. I would want to
use it in that way if possible.
I think pre-rendering as much as possible on a non-static react app is
still be beneficial. Maybe certain components depend on dynamic content,
but the majority of the site doesn't. Or maybe just a shell is static.
Maybe some routes should be static and some not.
You can prerender an app made w/ create-react-app w/ react-snapshot or
react-snap, but I have had trouble using this out of the box.
An option to ignoreExport at the route level. Or maybe a component/HOC
that could be used to tell react-static that a certain portion of the
site shouldn't be prerendered (I haven't looked at react-static internals,
but maybe that means wrapping something like:
https://github.com/kadirahq/react-no-ssr). Or both.
I think it would be awesome if react-static supported these use cases.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/nozzle/react-static/issues/111#issuecomment-340939404,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFUmCSXJh3858coINbICvT24EGBnHEOFks5sx7EYgaJpZM4QMfwe
.

I would also like to point to the discussion at https://github.com/zeit/next.js/pull/1576.
NextJS ended up not taking the PR and went in a different direction. But there are ideas here worth noting (ie, having some props available at build time on a route (markdown files, json data, maybe data from a CMS like contentful) and then other props that should come dynamically).

@mjsisley Thanks for clarifying and great summary!

So this is actually quite easy to handle.  But there are a few things to handle first.  Currently when you navigate to an endpoint that is not a static exported route, you see a flash of 404 before react mounts. How should we fix this?

My thoughts on possible solutions:
Instead of flashing 404, maybe a placeholder statically rendered page could be flashed (probably best to leave config at the route level, maybe something like: { ..., ignoreExport: true, staticPlaceholder: StaticPlaceholderComponent }. Could also maybe have a default for all pages available.
If using an HOC (or maybe just a wrapping component) to block prerendering of any arbitrary portion of the tree, the route would still be set statically. But maybe having a staticPlaceholder prop on that HOC/wrapping component would be helpful... so a placeholder component can be statically prerendered... and then replaced when React mounts with the components that should only be used dynamically.
staticPlaceholder may not be the best name, that may require some more thought.

Let me take it to the next level. Imagine there is no route registered at
all for this endpoint. I think this could be a server level requirement.
Normally with a react app you want every uel to redirect the index.html
file so react can handle it. In this case, index.html is not blank. I think
a possible solution here would be to render the root 404 page as a loading
spinner of sorts. This would ensure that when a route is not recognized at
the server level, it is sent to the 404 page, which loads, then uses
internal routing to determine what to show.
On Thu, Nov 2, 2017 at 11:06 AM Matthew Sisley<EMAIL_ADDRESS>wrote:

My thoughts on possible solutions:
Instead of flashing 404, maybe a placeholder statically rendered page
could be flashed (probably best to leave config at the route level, maybe
something like: { ..., ignoreExport: true, staticPlaceholder:
StaticPlaceholderComponent }. Could also maybe have a default for all
pages available.
If using an HOC (or maybe just a wrapping component) to block prerendering
of any arbitrary portion of the tree, the route would still be set
statically. But maybe having a staticPlaceholder prop on that
HOC/wrapping component would be helpful... so a placeholder component can
be statically prerendered... and then replaced when React mounts with the
components that should only be used dynamically.
staticPlaceholder may not be the best name, that may require some more
thought.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/nozzle/react-static/issues/111#issuecomment-341491017,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFUmCfPEys32mjYerQrKiSiLwVP28U_Sks5syfaxgaJpZM4QMfwe
.

I think this would be great, as long as it doesn't impact the ease of use on the static side. I think all the defaults should be based on the needs of a purely static marketing site, including the 404, with dynamic setups requiring config overrides.

I think the great thing about React is, that it is a library, and this forces so many to make frameworks.
Zeit.js gains a huge popularity. But the question is why? I think, react-static has no many benefits unlike SSR, that it's just a question of time that frameworks like react-static will gain a huge popularity. Developers will make all kinds of stuff with tools like this one 🙀 - I think.

Alright, I just released a patch that should get rid of any 404's you encounter when loading non-statically-exported routes. I think this is a good time and place to just see what happens.  Given that error, there really isn't anything stopping you from using react-static as is as a replacement for create-react-app.  I am already doing that myself on a few projects. Closing this issue for now, but we can keep discussing here.

@tannerlinsley So you are suggesting using the first load 404 page as a loading screen, and only serve a true 404 page from react? This was actually my first thought too, but didn't suggest because it feels a little "hacky" to hijack the server 404 page. Maybe that is fine though... every case should be handled (as long as javascript is enabled). Having the option to have each route to have their own placeholder adds some flexibility though, as each dynamic route could have its own shell. But both could be included.
@derekperkins raises another issue with this method, that going this route would alter 404 behavior for purely static sites (though it could be a loading screen that times out to a 404). Maybe just adding something like enableDynamicRoutes: true to the config to swap in this behavior would work though.
@tujoworker Agreed! React is great. A flexible high level prerendering framework like this one.

@tannerlinsley Sounds great! Will take a look soon and bring any issues (if there are any) up here. Thanks!

That may be true, but for now, I'm really trying to avoid any changes to the api to satisfy this use-case. I think we should try our hardest to make things work with the existing api first, then as we discover the full repercussions of the use case, we can determine if more needs to be done. :)

I updated my test case with the latest 4.0.1 version and tested out the static.config.js is404: true setting. Things did not work as expected. I can't se the effect on setting the is404: true component anywhere anytime. But the <Route component={NotFound} /> in App.js gets exported.
So, once I disable JavaScript, I se the 404 page once we visit /dynamic.
I expected that the is404 component will show up there, right?


Add ability to get project build logs

Commands
$ modulus deployLogs
$ modulus project deployLogs
$ modulus project deployLog (follows convention of modulus project log(s))

Closes #74

👍

Looks ok to me. 👍 , if it is behaving correctly.

Checked project w/ no logs again, looks good now 👍


Focus on Outline View command doesn't work when Explorer is not visible

Type: Bug
For example, when the Search view is visible, when running the command, it only shows the Explorer view, so you need to run the command again to focus on the Outline view.
 This causes the focus command to be run before the tree has been created. So we need to make sure to set focus after _handleEditorControlChanged has finished.
This would be one approach of fixing this: https://github.com/microsoft/vscode/pull/234905


Clean up the KeepAlive semantics

Addresses https://github.com/rsocket/rsocket-cpp/issues/772

I would suggest to make FollyKeepaliveTimer an internal class as there is no other implementation of this we ever needed. It would be created by the RSocketClient with the correct event base and passed into the RSocketStateMachine. We won't need the keepalive interface either.
We can have a new parameter for the RSocketClient ctor std::milliseconds keepaliveInterval = 5

Would totally love to that.  I remember at some point there was a conversation about allowing application to pass in a KeepAlive implementation, so left it there assuming it is still the case.
Will update the PR.

Two relevant issues for cleanup
https://github.com/rsocket/rsocket-cpp/issues/603
https://github.com/rsocket/rsocket-cpp/issues/706
Close when the cleanup happens


Update README.md

Added a link to a similar extension.

@manastungare, thank you for your work and for supporting my initiative. It is much easier to work when you feel the support of the author.
Manas, tell me, maybe you have any ideas where else I can post a link so that as many people as possible know about my extension?


Request to move weights on huggingface

Do you have any plans to upload the weights to huggingface, its convenient to use it from there.
If you are busy, I would be happy to help you.

Thanks for your advice! I'd appreciate your help. Is there anything specific I should do?

It is quite a bit of work. But it really helps everyone else and gets your paper into the millions.
https://huggingface.co/docs/transformers/add_new_model
If you are busy, I can also help out if needed.

OK, I will check it out. If anyone else also thinks this approach is better, please comment here.

A good suggestion! It will be convenience for people to experience your work on huggingface! Thank you!


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_bf5bfd8a-d0cb-4d26-b577-1093219ae632/pull/1 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target_bf5bfd8a-d0cb-4d26-b577-1093219ae632.git

Docs Build status updates of commit 261b1a7:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-UserAccountBuildPermissionInsufficientError] Owner v-alji no longer has Read permission of the repository https://github.com/opstest2/docs-archive-test-source. It might be due to recent GitHub changes on SSO. Please ask the owner v-alji to logon to Docs Portal https://ops.microsoft.com once again. If it is not the case, please ask anyone of Admin permission of the repo to assign a new owner via Docs Portal https://ops.microsoft.com

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Preview/Download files: 'Could not send file' error in public record

Package version (if known):
invenio-cli 0.23.0
InvenioRDM v2.0
Instance: https://inveniordm.web.cern.ch/ 2021-03-29
Describe the bug

We created records in the invenioRDM-demo with different accounts (both on firefox) and all of our records with several versions and published them completely public.
I wasn't able to preview or download the record-files from my workmate. He wasn't able to preview or download the record-files from the record I created. We also checked a record that was created by someone else that we don't know. I was able to download it my workmate wasn't able.
Later I checked again and tried different browsers (firefox and chromium).
The error message in chromium was for me 'zipfile is not previewable' and "status 500" "errors: Could not send file: Resource not found: b'/data'".
In firefox I get a 'broken preview-picture' and the same error message for a png-file.
Interestingly, files that I cannot see and download in firefox I can in chromium, and the other way round.
I tried to see the reproduce the same error with different accounts on a local instance, but so far I wasn't able to reproduce it.

My firefox version: 78.8.0esr (64-bit) (debian 10)
My chromium version: Version 88.0.4324.182 (Developer Build) built on Debian 10.8, running on Debian 10.8 (64-bit)
Steps to Reproduce
Unfortunately, we have no idea what is going on.
Expected behavior
Public uploaded files can be previewed and downloaded from everybody without error.
Screenshots (if applicable)

Record from (for us) unknown person - on the left firefox - on the right chromium

Record from (for us) unknown person - clicking on 'Download'

Record from my workmate - on the left firefox - on the right chromium

Record my workmate - clicking on 'Download'

Local fresh setup - instance using different accounts.

Uploaded files
My files:

Workmate files:

Files from random record (I never checked what is in there):
cleaned.zip
userstudy.zip

Yeap, I wasn't able to reproduce it with my own instance neither. @lnielsen asked me on discord to put it here. If anyone ever find out what is/was the issue (even if it is related to the setup of deployment, ...) we would happy to know - so that we do not get into that 'trap' as well when we deploy.
Thanks for taking a look. :)


feat: add i18n support

This feature includes the core functions added to /internal, English and Hebrew json files, removal of text strings from the html files, updates to the routes and model updates. Also included are tests. Please note, that this feature is waiting on user registration for final hooks.
Issue: #23

Codecov Report

Merging #115 (65db418) into main (2055db7) will decrease coverage by 0.85%.
The diff coverage is 92.98%.

@@            Coverage Diff             @@
##             main     #115      +/-   ##
==========================================
- Coverage   98.72%   97.87%   -0.86%
==========================================
  Files          10       11       +1
  Lines         236      282      +46
==========================================
+ Hits          233      276      +43
- Misses          3        6       +3

Impacted Files
Coverage Δ

app/main.py
85.00% <50.00%> (-9.12%)
:arrow_down:

app/internal/languages.py
96.42% <96.42%> (ø)

app/database/models.py
100.00% <100.00%> (ø)

app/routers/agenda.py
100.00% <100.00%> (ø)

app/routers/event.py
100.00% <100.00%> (ø)

app/routers/profile.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Codecov Report

Merging #115 (65db418) into main (2055db7) will decrease coverage by 0.85%.
The diff coverage is 92.98%.

@@            Coverage Diff             @@
##             main     #115      +/-   ##
==========================================
- Coverage   98.72%   97.87%   -0.86%
==========================================
  Files          10       11       +1
  Lines         236      282      +46
==========================================
+ Hits          233      276      +43
- Misses          3        6       +3

Impacted Files
Coverage Δ

app/main.py
85.00% <50.00%> (-9.12%)
:arrow_down:

app/internal/languages.py
96.42% <96.42%> (ø)

app/database/models.py
100.00% <100.00%> (ø)

app/routers/agenda.py
100.00% <100.00%> (ø)

app/routers/event.py
100.00% <100.00%> (ø)

app/routers/profile.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

@yammesicka any idea on how to handle the missing code coverage where I do a test to check if I'm running from tests or live?
And here are the steps to add new translations:

Create base.pot file: pybabel extract --mapping-file=app/babel_mapping.ini app tests -o app/locales/base.pot
cd app
3.1 Create he base.po file: pybabel init -l he -i locales/base.pot -d locales -D base
3.2 Create en base.po file: pybabel init -l en -i locales/base.pot -d locales -D base
Translate strings
Create .mo files: pybabel compile -d locales -D base
Update translations.
Update .mo files: pybabel update -i locales/base.pot -d locales

@yammesicka any idea on how to handle the missing code coverage where I do a test to check if I'm running from tests or live?
And here are the steps to add new translations:

Create base.pot file: pybabel extract --mapping-file=app/babel_mapping.ini app tests -o app/locales/base.pot
cd app
3.1 Create he base.po file: pybabel init -l he -i locales/base.pot -d locales -D base
3.2 Create en base.po file: pybabel init -l en -i locales/base.pot -d locales -D base
Translate strings
Create .mo files: pybabel compile -d locales -D base
Update translations.
Update .mo files: pybabel update -i locales/base.pot -d locales

3. And here are the steps to add new translations:
3.1. Create base.pot file: pybabel extract --mapping-file=app/babel_mapping.ini app tests -o app/locales/base.pot
3.2. cd app
3.3.1. Create he base.po file: pybabel init -l he -i locales/base.pot -d locales -D base
3.3.2. Create en base.po file: pybabel init -l en -i locales/base.pot -d locales -D base
3.4. Translate strings
3.5. Create .mo files: pybabel compile -d locales -D base
3.6. Update translations.
3.7. Update .mo files: pybabel update -i locales/base.pot -d locales

Awesome work! Can you please add this to the workflow? We can do it automagically using GitHub Actions.
It can be considered as another ticket :)

3. And here are the steps to add new translations:
3.1. Create base.pot file: pybabel extract --mapping-file=app/babel_mapping.ini app tests -o app/locales/base.pot
3.2. cd app
3.3.1. Create he base.po file: pybabel init -l he -i locales/base.pot -d locales -D base
3.3.2. Create en base.po file: pybabel init -l en -i locales/base.pot -d locales -D base
3.4. Translate strings
3.5. Create .mo files: pybabel compile -d locales -D base
3.6. Update translations.
3.7. Update .mo files: pybabel update -i locales/base.pot -d locales

Awesome work! Can you please add this to the workflow? We can do it automagically using GitHub Actions.
It can be considered as another ticket :)

Hi,
I've merged it by mistake, but this actually should go into the develop branch.
I've reverted the changes to avoid poor repo structure.
Can you please re-create the PR to the develop branch and not to the main branch?

Hi,
I've merged it by mistake, but this actually should go into the develop branch.
I've reverted the changes to avoid poor repo structure.
Can you please re-create the PR to the develop branch and not to the main branch?

Sure, could you explain what to do? I tried git pull upstream develop  but
it wants to me commit the files that are now there. Is that how I do it?
Don't want to create a new mess here.
On Sat, Jan 30, 2021 at 9:08 PM Yam Mesicka<EMAIL_ADDRESS>wrote:

Hi,
I've merged it by mistake, but this actually should go into the develop
branch.
I've reverted the changes to avoid poor repo structure.
Can you please re-create the PR to the develop branch and not to the main
branch?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PythonFreeCourse/calendar/pull/115#issuecomment-770265478,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD5O6JHJ4J45L3UUNEHR66DS4RKJFANCNFSM4WOICH6Q
.

Sure, could you explain what to do? I tried git pull upstream develop  but
it wants to me commit the files that are now there. Is that how I do it?
Don't want to create a new mess here.
On Sat, Jan 30, 2021 at 9:08 PM Yam Mesicka<EMAIL_ADDRESS>wrote:

Hi,
I've merged it by mistake, but this actually should go into the develop
branch.
I've reverted the changes to avoid poor repo structure.
Can you please re-create the PR to the develop branch and not to the main
branch?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PythonFreeCourse/calendar/pull/115#issuecomment-770265478,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD5O6JHJ4J45L3UUNEHR66DS4RKJFANCNFSM4WOICH6Q
.


Ability to hide/disable "blocked attampted request" banner

I'm getting so much popups of "blocked attempted request" that it's starting to annoy me. I constantly have to try to hit a tiny "x" with the mouse pointer to close the banner.
Would it be possible to add some configuration option, for the popup bar to automatically disappear after x seconds, or disable it entirely?
Thanks :)

Wait really? What platform and version are you running? What do you do that triggers this?

OS X 10.12.1, 53.0.2785.116 (64-bit).
I don't know, i just opened Chromium for example, and immediately got a popup. Closed it, then browsed to some random website and within 15 seconds got a second popup. Another website, no popup. Went to Reddit.com clicked around a bit and another popup within 20 seconds.... etc.
http://i.imgur.com/3bsvWTS.png

Are you logged into Google?

Nope (i don't even have a Google account..)

So it's basically the same as #104 since the URL is the same, but it's a different platform and slightly different version. I have a feeling that they are triggered by the same common code, but I'm a bit puzzled why you have it but @9Morello and I don't have it.
I'll mark this as a bug for now as I can't say for sure this is a duplicate of #104.

Regarding the intent of the original post, it might be a good idea to implement a way to disable the notifications of blocked connection attempts. While a power user might prefer to have them on, I can see this being a problem if someone setups ungoogled-chromium for another person.

@9Morello You have a good point.

Due to a situation I encountered (more details in #104), I've changed #104 to capture all instances of the bug reported here, and reverted this issue back to the enhancement proposed originally.

I just realized this is a duplicate of #112. Going to close this as such.


Add the ability for some enums to still deserialize properly

This is just a small enhancement to the fuzzy enum deserializer so that it works when some enums (but not all) use toString()

Thanks for the pull request, but I have a couple of questions:

I think we already handle the @JsonCreator annotation on factory method in enums. If the annotation is present, we use the specific deserializer based on this method. Could you provide a test case, where it doesn't work as expected?
I don't think we should try to  deserialize an enum based on the toString method, unless the user didn't explicitly configured the mapper to do so. It's not obvious and could break backward compatibility for users, who use toString just for logging purposes. If enabling this feature in the mapper is not an option, the user can create a factory method with @JsonCreator or register a custom deserializer.

@arteam Please see the commit.  I already provided a unit test that causes a failure with the current FuzzyEnumModule used in dropwizard.  The patch I provided fixes this issue.  If there's a different/better way to fix this issue then I'm fine with that as well, but this current approach should maintain backwards compatibility and allow for more flexible enums.
Even when using tools like jsonschema2pojo, you get enums generated that can look like this:
@Generated("org.jsonschema2pojo")
public enum DeployedEnvironmentType {
DEV("development"),
ALPHA("ALPHA"),
LOAD("LOAD"),
STAGING("STAGING"),
PRODUCTION("PRODUCTION");
private final String value;
private static Map<String, DeployedEnvironmentType> constants = new HashMap<String, DeployedEnvironmentType>();

static {
    for (DeployedEnvironmentType c: values()) {
        constants.put(c.value, c);
    }
}

private DeployedEnvironmentType(String value) {
    this.value = value;
}

@JsonValue
@Override
public String toString() {
    return this.value;
}

@JsonCreator
public static DeployedEnvironmentType fromValue(String value) {
    DeployedEnvironmentType constant = constants.get(value);
    if (constant == null) {
        throw new IllegalArgumentException(value);
    } else {
        return constant;
    }
}

}
Without my change then if you passed in "development" it wouldn't deserialize properly.

Okay, I am convinced now that the problem is worth to fix and the patch would not break anything. I only have a one question, please see an inline comment.
Also, it would be great if you squashed your commits into a single change.

I agree.  I'll make the adjustments above and squash the commits down.

@arteam do you want to get this into 0.9 as soon as @brentryan makes the requested changes?

Probably not. I think we should add only bugfixes for 0.9. Will happily merge the PR to 1.0, though.

I'll re-create a new PR with the squash.

See https://github.com/dropwizard/dropwizard/pull/1316 for details on the new PR


scannerModule.BarcodeType is undefined

Hey, I just started using react native and was trying to implement your barcode scanner on a small project and I keep getting an error that says "undefined is not an object (scannerModule.BarcodeType is undefined)". I'm just using your example code and haven't added anything to it, any idea why this might be happening?
thanks.

Hey @jshtx,
Not quite sure. I just did a clean test following my instructions, and it worked flawlessly.
Some ideas:

Note that this will only work on Android! You will for sure run into problems if you try to use this module for any iOS device/emulator. Use eg. react-native-camera on iOS.
Did your linking work well? When you run react-native link react-native-barcode-scanner-google (basically, this command will add some Java/Android code within your app to link this module to the JS side) your output should look somewhat like this:

Scanning 558 folders for symlinks in /Users/<User>/Path/To/BarcodeApp/node_modules (5ms)
rnpm-install info Linking react-native-barcode-scanner-google android dependency
rnpm-install info Android module react-native-barcode-scanner-google has been successfully linked

Additionally:

Make sure that your app builds well when you run react-native run-android.
I'm assuming you're on Mac/Linux... 😄

Hopefully there's something helpful in there.

same here can you please provide manual steps to link, some times it not linked

@ShaileshPrajapati-BTC maybe it late, but when i delete ios build, clean xcode, remove derived data, reset cache... and re-run on xcode, it work. (i use this library with RN camera)

there is a workaround if this is breaking your ios build.
`import React, { Component } from 'react';
import Camera from 'react-native-camera';
import PropTypes from 'prop-types';
import { Platform } from 'react-native';
const ScannerComponent = Platform.select({
android: () => require('react-native-barcode-scanner-google').default,
ios: () => require('react-native-camera').default
})();
export default class BarCodeScannerComponent extends Component {
constructor(props) {
super(props);
}
static propTypes = {
//style: RNCamera.propTypes.style,
onBarcodeRead: PropTypes.func,
onPhoto: PropTypes.func,
permissionDialogMessage: PropTypes.string,
permissionDialogTitle: PropTypes.string,
cameraType: PropTypes.string
};
render() {
const {
style,
cameraType = Camera.constants.Type.back,
permissionDialogTitle = 'Permission to use camera',
permissionDialogMessage = 'We need your permission to use your camera',
onBarcodeRead,
onPhoto,
} = this.props;
return (
  Platform.OS === 'iso' ? <ScannerComponent
      ref={cam => this.cameraComponent = cam}
      style={style}
      type={cameraType}
      permissionDialogTitle={permissionDialogTitle}
      permissionDialogMessage={permissionDialogMessage}
      onBarCodeRead={onBarcodeRead}
      onPhoto={onPhoto}
      barCodeTypes={[Camera.constants.BarCodeType.code128]}>
      {this.props.children}
    </ScannerComponent>
    :
    <ScannerComponent
      style={style}
      onBarcodeRead={onBarcodeRead}
    />
);

}
}
`

@ognjetina your solution worked for me if I only used the component, but when I tried to use other attributes from the library, e.g. FocusMode, the iOS build was still broken as all the other stuff was required (imported) on iOS as well. I ended up doing something like this, which worked for me:
const Scanner = Platform.select({
  android: () => ({
    Component: require('react-native-barcode-scanner-google').default,
    attributes: {
      CameraFillMode: require('react-native-barcode-scanner-google').CameraFillMode,
      FocusMode: require('react-native-barcode-scanner-google').FocusMode,
      Exception: require('react-native-barcode-scanner-google').Exception
    }
  }),
  ios: () => ({
    Component: require('react-native-camera').default
  })
})()

and then in my render:
	if (IS_ANDROID) {
		return (
			<Scanner.Component
				focusMode={Scanner.attributes.FocusMode.TAP}
				{...otherProps}
			/>
		)
	}
	return (
		<Scanner.Component
			aspect={Camera.constants.Aspect.fill}
			{...otherProps}
		/>
	)

I am getting still the same error in IOS, in Android it works fine
scannerModule.BarcodeType is undefined

I have same problem try to run this module on Android, get
null is not an object (evaluating 'scannerModule.BarcodeType')


[HUDI-3525] Introduce JsonkafkaSourcePostProcessor to support data preprocess before it is transformed to DataSet

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
Currently, we have Transform to transform source to target dataset before writing, but it is based on DataSet.
In some scenarios, our kafka data is not in the right format we need, such as binlog json format.
We need a way to extract/prepare the data we need from the original data before converting it into a DataSet.
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Can you check the CI failure please

@pratyakshsharma : Can you assist in reviewing this patch.

Can we add a test case by adding a test JsonKafkaPostProcessor and ensure it works. also add a test where you set some invalid class for the new config added. and assert for exception.

done

@hudi-bot run azure

@hudi-bot run azure

@nsivabalan  please take another look when free :)

@wangxianghu The changes look good to me. I have a high level query. You have mentioned facing issues with binlog json format for example. Do you mean to say transformation is not possible with data in binlog json format? Data coming from binlogs also has a structure/schema assigned to it as far as I remember. Can you post a sample event where you feel this new PostProcessor you introduced might be useful?
Basically I want to understand the motivation behind introducing this PR.

@wangxianghu The changes look good to me. I have a high level query. You have mentioned facing issues with binlog json format for example. Do you mean to say transformation is not possible with data in binlog json format? Data coming from binlogs also has a structure/schema assigned to it as far as I remember. Can you post a sample event where you feel this new PostProcessor you introduced might be useful? Basically I want to understand the motivation behind introducing this PR.

@pratyakshsharma thanks for your review!
{
"database": "test",
"table": "maxwell",
"type": "update",
"ts":<PHONE_NUMBER>,
"xid": 940786,
"commit": true,
"data": {"id":1, "daemon": "Firebus!  Firebus!","update_time" : "2022-02-03 12:22:42"},
"old":  {"daemon": "Stanislaw Lem"}
}

@wangxianghu The changes look good to me. I have a high level query. You have mentioned facing issues with binlog json format for example. Do you mean to say transformation is not possible with data in binlog json format? Data coming from binlogs also has a structure/schema assigned to it as far as I remember. Can you post a sample event where you feel this new PostProcessor you introduced might be useful? Basically I want to understand the motivation behind introducing this PR.

It is possible to deal with data in binlog json format, but not very convenient.

For maxwell(our company use it to capture changed data)

{
    "database": "test",
    "table": "maxwell",
    "type": "update",
    "ts":<PHONE_NUMBER>,
    "xid": 940786,
    "commit": true,
    "data": {"id":1, "daemon": "Firebus!  Firebus!","update_time" : "2022-02-03 12:22:42"},
    "old":  {"daemon": "Stanislaw Lem"}
  }

all we want is just :
{
    "id": 1,
    "daemon": "Firebus!  Firebus!",
    "update_time": "2022-02-03 12:22:42"
}

we can add write a processor to extract the data from the entire json and maybe do some custom define process, without configuring a huge schema file(including all the fields in the binlog json, no matter if we need them or not)

in some scenes, we need to encode some fileds for safety purpose, the processor can help us

sometimes our data quality is not very well, some key field let's say precombine field have null value, we can use processor to fix it

when our schema is read from jdbc or hive, we can adjust our kafka data compatible to it.

All in all, with custom processor we can do anything we want on the incoming json data before they are converted into DataSet
Of course Transformer is a very useful feature too, but it is based on Spark DataSet, and have certain requirements for data quality.

Right. With Debezium, this extraction of data is already supported. But with maxwell and any other similar service, this processor will be helpful.

hi @nsivabalan  any orther concern ?


Using pyscenedetect in lambda function?

Just came across pyscenedetect and it is awesome, I tested it out on my machine with a video and it worked great. I was wondering if it was possible to use this package in a lambda function by passing a video to it and then returning a csv or json with the scenes? Many thanks and great work!

Hi @jmsp;
See the example code here in the manual (link) for an example of using the Python API to detect scenes and print a list of them.
Thank you for your interest.

Hi @jmsp;
Due to inactivity on this issue, I will close it assuming it to be resolved.  If this is not the case, please feel free to re-open it or create a new issue.
Thank you!