Bug in 1.0.6 with unrestricted access

Hello
Thanks a lot for this update that allows to choose the level of sanitizing!! That really helps (it is a solution!) for some pages! There are two bugs, though:

Not important: in the settings, when in dark mode, some part of the table and some phrases are still rendered with white background which makes them difficult to read. This is just aesthetic.
This is more important: Obsidian can now show a tweet saved with SingleFile (through ThreadReader website), and the video plays well, but when you click on the tweet text (the text just above or below the video) it opens the tweet in the default browser, which is fine, only that it opens it repeated in 20 tabs or so ! When you open the very same file directly in the default brower by clicking on it in the windows explorer, it opens it, and if then you click on the tweet text it opens the tweet in twitter just once, as it should. This is in unrestriced mode, otherwise the click on the text is disabled. The attached file is for testing, and normally there is no need to go full unrestriced with it, but for other entries, it is.

Thank you!
TRB - ❝ 𝐘𝐚𝐲𝐚 ❞ FULL_.zip

when you click on the tweet text (the text just above or below the video) it opens the tweet in the default browser, which is fine, only that it opens it repeated in 20 tabs or so !

Sorry, I cannot reproduce this situation on my PC.
Tested Platform: Obsidian 1.0.3, Chrome 105, Firefox 107 on Windows 11 22H2
Tested Modes: Unrestricted Mode, Low Restricted Mode, Balance Mode, High Restricted Mode.
The "Tweet" text block contains a hyperlink https://twitter.com/intent/tweet?text=Thread%20by%20@__panicmoonwalk%20on%20Thread%20Reader%20App&url=https%3A%2F%2Fthreadreaderapp.com%2Fthread%2F1590756028760850432.html.
As you can see, it just pass the hyperlink to the default browser of your system, then the browser would open link.
The complete HTML tag code is  Tweet.
I think you can paste the hyperlink address text from the address bar of the default browser when the situation happened, to let me see.
Or, you can copy the hyperlink https://twitter.com/intent/tweet?text=Thread%20by%20@__panicmoonwalk%20on%20Thread%20Reader%20App&url=https%3A%2F%2Fthreadreaderapp.com%2Fthread%2F1590756028760850432.html to your default browser and "goto" this link to see if the same situation happen or not.

Hi,
I am testing it now in other PC, with other Obsidian installation *although the same version(, and I can-t reproduce neither the problem. Don't know why. I3ll try to test what you say tomorrow in the first computer.
BTW, when in urestricted mode, if you change the color (dark, light mode), most of the text of Obsidian settings window is not visible, it seems a secondary effect of unrestricted mode. There is no need to open a note or nothing, just set unrestricted, go to Appearance, and change from dark to light.
Thanks
Thanks

BTW, when in urestricted mode, if you change the color (dark, light mode), most of the text of Obsidian settings window is not visible, it seems a secondary effect of unrestricted mode. There is no need to open a note or nothing, just set unrestricted, go to Appearance, and change from dark to light.

Oh, I think you misunderstand the meaning of "Unrestricted Mode", it means the content of HTML files would affect other parts of Obsidian, so  this mode is unrestricted for all bad guys and good guys ! Please at least switch to "Low Restricted Mode" for normal usage.
I must point out there are many warning messages in README.md, the most important part is

Unrestricted Mode - This mode is  very dangerous and may cause the Obsidian app crash or disarrange layouts, THE OBSIDIAN AND THIS PLUGIN CANNOT ASSUME RESPONSIBILITY OR LIABILITY FOR SWITCHING TO THIS MODE . It would not sanitize anything, all elements and their content would be keeped. The Obsidian app itself might adjust something. The external script files may not executable due to Obsidian's limitation.

When there is no file opened on Unrestricted Mode in the beginning of the Obsidian run:
Fig.1 Unrestricted Mode in Dark 1

Everything seems good.
But after open the "TRB - ❝ 𝐘𝐚𝐲𝐚 ❞ FULL .html", you can see the situation occurred :
Fig2. Unrestricted Mode in Dark 2 - polluted by CSS styling

This is caused by the CSS styles of the HTML file  "TRB - ❝ 𝐘𝐚𝐲𝐚 ❞ FULL .html". Again, I must point out there are explanations about CSS styles, the most important part is:

Isolated - The CSS styles of HTML files shall be isolated against with other parts of Obsidian, otherwise the Obsidian's layouts might be disarranged at Unrestricted Mode.

The CSS isolation would be applied on other modes, look this:
Fig.3 Low Restricted Mode in Dark 1

You can compare the difference between "Low Restricted Mode" and "Unrestricted Mode" after open the same file.
Please at least switch to "Low Restricted Mode" for normal usage.
Please at least switch to "Low Restricted Mode" for normal usage.
Please at least switch to "Low Restricted Mode" for normal usage.

Sorry for silence, I was busy with other aspects.
I'll use Low Restriction Mode by default. And yes, you have reason, what you show is inline with what I see, but with the exception of the settings page, I still see the white background issue:

Sorry for silence, I was busy with other aspects. I'll use Low Restriction Mode by default. And yes, you have reason, what you show is inline with what I see, but with the exception of the settings page, I still see the white background issue:

This bug will be fixed at next version, you can refer the "Fig.3 Low Restricted Mode in Dark 1" above. Thanks for your report.


feat: OPTIC-351: Object3D hackathon POC

PR fulfills these requirements

[x] Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
[ ] Tests for the changes have been added/updated (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[ ] Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
[x] Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)
(check all that apply)

[ ] Product design
[ ] Frontend

Describe the reason for change
(link to issue, supportive screenshots etc.)
What does this fix?
(if this is a bug fix)
What is the new behavior?
(if this is a breaking or feature change)
What is the current behavior?
(if this is a breaking or feature change)
What libraries were added/updated?
(list all with version changes)
Does this change affect performance?
(if so describe the impacts positive or negative)
Does this change affect security?
(if so describe the impacts positive or negative)
What alternative approaches were there?
(briefly list any if applicable)
What feature flags were used to cover this change?
(briefly list any if applicable)
Does this PR introduce a breaking change?
(check only one)

[ ] Yes, and covered entirely by feature flag(s)
[ ] Yes, and covered partially by feature flag(s)
[ ] No
[ ] Not sure (briefly explain the situation below)

What level of testing was included in the change?
(check all that apply)

[ ] e2e
[ ] integration
[ ] unit

Which logical domain(s) does this change affect?
(for bug fixes/features, be as precise as possible. ex. Authentication, Annotation History, Review Stream etc.)

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

Comparison is base (2f5b6e1) 68.28% compared to head (facd3e9) 65.13%.

Files
Patch %
Lines

src/tags/object/Object3D/Object3D.js
2.10%
389 Missing and 75 partials :warning:

src/mixins/CubeDrawingTool.js
0.00%
382 Missing :warning:

src/regions/CubeRegion.js
3.89%
126 Missing and 22 partials :warning:

src/components/Object3DView/LabelOnRegion.js
9.16%
88 Missing and 21 partials :warning:

src/tags/object/Object3D/Object3DEntityMixin.js
1.06%
88 Missing and 5 partials :warning:

src/tags/object/Object3D/Object3DEntity.js
2.66%
70 Missing and 3 partials :warning:

src/tags/object/Object3D/Object3DSelection.js
1.44%
55 Missing and 13 partials :warning:

src/tools/Cube.js
0.00%
47 Missing :warning:

src/components/Object3DView/Object3D.js
26.47%
22 Missing and 3 partials :warning:

src/tags/object/Object3D/DrawingRegion.js
14.28%
4 Missing and 2 partials :warning:

... and 4 more

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1642      +/-   ##
==========================================
- Coverage   68.28%   65.13%   -3.16%
==========================================
  Files         443      458      +15
  Lines       28686    30162    +1476
  Branches     7628     7953     +325
==========================================
+ Hits        19589    19645      +56
- Misses       7845     9121    +1276
- Partials     1252     1396     +144

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


DAB Load Testing - By deploying it in ACI

This issue tracks the follow-up items from the observations presented for the issue #1220
Perform a load test with the following test setup

Deploy DAB image in ACI. The container specifications should be 4 cores/4 GB
The load generator script should be deployed in an Azure VM
Load generator, DAB and DB should all be in the same region.
The workload should just contain a GraphQL point query

Moving to May milestone. Will be taken up after the perf improvements PR #1392  is merged.


Should it be an error (or hint/lint/warning) to have multiple language comments?

This doesn't seem to be an error in the analyzer.
// @dart = 2.6
// @dart = 2.6
class C<T extends Object> {}

Should it be an error of some kind? Should that be part of any specs? @lrhn @leafpetersen
Adding area-analyzer for the meantime since we have warnings for things like, for instance, things that almost look like language version comments but aren't.

The feature specification states:

If there is more than one such marker, only the first one applies, but tools are encouraged to warn about the extra unused markers.

So, not an error, but tools are free to warn about the second one. They are also free to stop looking after finding the first marker.

Is there a reasonable use case for wanting to have multiple in a file? If not, I'd propose making this a hint (enabled by default).

There is no reasonable use-case for multiple markers.
They must all be at the top of the file, before any Dart declaration.
I think it's reasonable for the analyzer to detect as many potential errors as possible, but it's not something the front-end should waste time on. The first marker wins, and they can stop looking when they've found that, or a Dart declaration.
There are many ways to add "incorrect" markers:

Second marker.
Marker after a Dart declaration.
Marker inside block comment instead of single-line comment.
Marker comment with more content before or after the allowed text.
Incorrectly formatted markers.

The specification is clear that none of these are actually version markers, and they have no effect.
Which ones deserve a warning is up to the intelligence of the tool wanting to report it.
Is // // @dart=2.4 and incorrect marker, or a deliberately commented-out marker?
What about:
/*
// @dart=2.4
*/

I think the safest approach would be to only consider correctly formatted and incorrectly placed markers. So, only single-line comments of the form<EMAIL_ADDRESS>If any such correctly formatted marker occurs after another marker or after the first Dart declaration, then you're probably doing something wrong.
Then we can worry over whether // @dart = 2 . 5 should warrant a warning.

There are many ways to add "incorrect" markers:
....
I think the safest approach would be to only consider correctly formatted and incorrectly placed markers.

That might well be true, but analyzer already has hints for many of the incorrect ways to format a marker.
The one it's missing is the multiple-marker case, and I think it would be useful for it to cover that case for the migration tool (which currently removes the first such marker it finds, but will produce bad code if there happens to be another marker to replace the removed on).


Bug remove in environment.py

C:\Users*\AppData\Local\Programs\Python\Python312\Lib\site-packages\jinja2_async_environment\environment.py:37: RuntimeWarning: coroutine 'FileSystemLoader.get_source..uptodate' was never awaited
if template and (not self.auto_reload or template.is_up_to_date):
RuntimeWarning: Enable tracemalloc to get the object allocation traceback

I thought this was a no brainer but am reverting for further review. This package was being back ported out of one of my private repos where I am using other custom loaders. It was functional but I haven't checked it in a few. I created the loaders so they'd be available but have not tested them yet. Am thrilled you're giving it a try.
Without testing, right off the bat pyright is rejecting the commit:
/Users/les/Projects/jinja2-async-environment/jinja2_async_environment/environment.py
/Users/les/Projects/jinja2-async-environment/jinja2_async_environment/environment.py:37:60 - error: "bool" is not awaitable
  "bool" is incompatible with protocol "Awaitable[_T_co@Awaitable]"
    "await" is not present (reportGeneralTypeIssues)
1 error, 0 warnings, 0 informations
So, it's saying .is_up_to_date is not an awaitable async coroutine.
I've been sidelined working on a companion pacakge that is dependent on this and am just getting back to make work on right now. Great timing and glad you're interested. Please keep 'em coming. I am making this a priority and will addresses this asap.
I'll put in some initial docs and tests too here.

If you could create another pull request, that'd be great. My excitement to merge and close this is not letting me reopen it now.

@nikolay-mansas I thought this was a no brainer but am reverting for further review. This package was being back ported out of one of my private repos where I am using other custom loaders. It was functional but I haven't checked it in a few. I created the loaders so they'd be available but have not tested them yet. Am thrilled you're giving it a try.
Without testing, right off the bat pyright is rejecting the commit:
/Users/les/Projects/jinja2-async-environment/jinja2_async_environment/environment.py /Users/les/Projects/jinja2-async-environment/jinja2_async_environment/environment.py:37:60 - error: "bool" is not awaitable   "bool" is incompatible with protocol "Awaitable[_T_co@Awaitable]"     "__await__" is not present (reportGeneralTypeIssues) 1 error, 0 warnings, 0 informations
So, it's saying .is_up_to_date is not an awaitable async coroutine.
I've been sidelined working on a companion pacakge that is dependent on this and am just getting back to work on this right now. Great timing and glad you're interested. Please keep 'em coming. I am making this a priority and will address this asap.
I'll put in some initial docs and tests too here.

In my project, files in the static folder are not updated without this await. I tried on windows and on my linux server, the same error everywhere. If you want, I can drop the project code in telegram or discord.


Feature/add support for templatic augmentation

Checklist:

[ ] I've added Google style docstrings to my code.
[ ] I've used pydantic for typing when/where necessary.
[ ] I have linted my code
[ ] I have added tests to cover my changes.

template_1 : "Patient previously took {Drug_BrandName} . Patient is experiencing {Symptom} , {Symptom} and {Symptom} .",
output:
Patient -X- -X- O
previously -X- -X- O
took -X- -X- O
ATIVAN -X- -X- B-Drug_BrandName
.  -X- -X- O
template_2: "Patient previously was taking {Drug_BrandName} . Patient is experiencing {Symptom} which is being treated with {Drug_BrandName}"
Patient -X- -X- O
previously -X- -X- O
was -X- -X- O
taking -X- -X- O
ATIVAN -X- -X- B-Drug_BrandName
.


Docker Question

By 'Dockerizing' what is the benefit? and how it could be explained in detail.

Maybe a GitHub issue is not the best place to ask, you might want to ask the community via forum, slack or other channels. See https://www.docker.com/docker-community for an overview.
To actually give a little answer to your question: I guess this article sums up quite well what happens when one tries to Dockerize services: https://nothingventured.rocks/phorest-migrated-to-docker-and-this-is-what-happened-8f0752ada960 - in essence: Dockerizing makes you review your architecture and application design and gives you a good options to improve the status quo.

The article is very helpful thankyou

Thanks @gesellix for your help.


Add option to never disconnect intentionally

In some case it is necessary to keep a constant connection with a BLE device, which doesn't seems to be the case at the moment. As I understand this node will connect and then disconnect from a device constantly:

[...] the node connects/disconnects a BLE device periodically. I think I need to add an option for the node to keep BLE connection.

https://github.com/CANDY-LINE/node-red-contrib-generic-ble/issues/4#issuecomment-356544213
What I'm trying to do is to subscribe to Notify events when the button of an iTag (a very cheap Bluetooth tracker with a button) is pressed. The problem is that since this thing is a tracker, when it gets disconnected it beep furiously until the button is pressed. You can imagine all the noise it makes when used with this node and the constant connection disconnection.

As of 3.0.0, you can keep a peripheral connected while receiving Notify events.
Pass the following message to Generic BLE in node to subscribe Notify events.
{
  "notify": true,
  "period": 6000
}

period is the time to subscribe in milliseconds.
Providing 0 period keeps subscribing until re-deploying a flow or shutting down the process.


Porting the circular buffer challenge.

Sorry I accidentally closed
https://github.com/exercism/xcpp/pull/80
by one means or another and it didn't seem to want to let me reopen.

The travis CI build is failing; you'll need to fix that before we can accept the pull request.

If you need help figuring it out, just shout :smile_cat:

OK, now the Travis build is passing, so I've taken a more detailed look at your commit.  Thanks for this contribution.
Your code isn't written in the same style as the rest of the code in this repo.  This implies that I should more formally document the formatting conventions of this repo (See issue #92 and issue #93)  Most of the rest of these comments revolve around the style and formatting of your code and how it differs from the existing code.  Beyond that, I think your tests need some restructuring so that they test only one behavior per test.
In particular, in this repo we don't put spaces inside the parentheses of arguments to functions.  So write: foo(5), not foo( 5 ).  This applies to the CMakeLists.txt as well as the source code.
Similarly, multiple template or function arguments are separated with a space between them.  So write circular_buffer<char, 2UL>, not circular_buffer<char,2UL>.
I don't understand the point of this test:
//This is some simple book-keeping to let people who are giving feedback know which version of the exercise you solved.
BOOST_AUTO_TEST_CASE(version_test)
{
   BOOST_CHECK_EQUAL( containers::VERSION, 1 );
}

None of the other exercises have this sort of a test and I don't see why this should have one.
Your tests aren't following the same whitespace style as the other tests; you're using an indent level of 2 and not 4 as is used consistently throughout the other files in the repo.
Tests should be formatted so that the different phases of execution (setup, exercise, verify, teardown) are clearly seen at a glance.  This means putting a blank line between them.
Separate each test case from the previous by a blank line.
Tests cases should be isolated and test only one thing.  You have test cases that test several things in succession.  There are also many, redundant assertions in your test cases.  If you've already tested some behavior in a previous test case, there is no need to test it again in a subsequent test case.  Each test case should test some new aspect of the behavior and verify that new behavior only.
I don't understand the point of circular_buffer_test2.cpp.  Again, none of the other exercises have this.  Presumably you're trying to check that the header doesn't violate the one definition rule, but that isn't really the point of the C++ language track in exercism, nor should it be the point of this exercise, IMO.  I would simply remove this file.  This is something for bringing up in discussion with someone's solution and I don't see the need to try and enforce that through a single exercise.
In example.hpp, you've indented everything inside the namespace.  None of the other code in this repo indents because of namespace.
In this repo, we prefer #if !defined(HEADER_GUARD) over #ifndef HEADER_GUARD because #ifdef and #ifndef differ only by a single character.  Existing header guard symbols are based off the filename as in ANAGRAM_H, yours differ from this convention and should be changed to match the existing code.
For comments, put a space after the // as in // comment, not //comment.
Separate methods in a class by a blank line.
Place braces around all blocks in control structures, even when they are a single statement.
Using const on a numeric template argument serves no purpose since the template can't change the argument anyway.
Reference (&) and pointer (*) annotations hug the identifier, not the type.  So write int *i, not int* i.

Closing this old PR. If you (or anyone else) wants to pick this back up please either reopen this or open a new PR :)
Thanks


todos_list Example Program Crashes in Release Mode

When compiling the todos_list example program in release mode, adding an item and then attempting to use a swipe gesture to trigger the complete action causes the program to crash. This crash is consistently reproducible in release mode but does not occur when compiled in debug mode.
Steps to Reproduce:

Compile the todos_list example program in release mode.
Add a new item to the list.
Attempt to use a swipe gesture to mark the item as complete.

Expected Behavior:
The item should be marked as complete without any issues.
Actual Behavior:
The program crashes when the swipe gesture is used to mark an item as complete.
Crash Stack Trace:
Crashed Thread:        0  main  Dispatch queue: com.apple.main-thread

Exception Type:        EXC_BAD_ACCESS (SIGSEGV)
Exception Codes:       KERN_INVALID_ADDRESS at 0x000007f81f4c8878
Exception Note:        EXC_CORPSE_NOTIFY

Termination Signal:    Segmentation fault: 11
Termination Reason:    Namespace SIGNAL, Code 0xb
Terminating Process:   exc handler [8098]

VM Regions Near 0x7f81f4c8878:
    CoreAnimation               1149c6000-1149e6000    [  128K] rw-/rwx SM=PRV
-->
    STACK GUARD              70000bd06000-70000bd07000 [    4K] ---/rwx SM=NUL  stack guard for thread 1

Thread 0 Crashed:: main  Dispatch queue: com.apple.main-thread
0   libobjc.A.dylib               	0x00007fff20249997 lookUpImpOrForward + 61
1   libobjc.A.dylib               	0x00007fff2024939b _objc_msgSend_uncached + 75
2   com.apple.AppKit              	0x00007fff236715b4 -[NSTableRowData _setupEditActionsButtonsForActions:edge:] + 366
3   com.apple.AppKit              	0x00007fff23671f28 -[NSTableRowData _setSwipeToDeleteRow:editActions:edge:] + 525
4   com.apple.AppKit              	0x00007fff236701bc -[NSTableRowData _startSwipeOnRow:withEvent:] + 204
5   com.apple.AppKit              	0x00007fff2344c039 -[NSTableView _handleScrollWheelSwipeWithEvent:] + 136
6   com.apple.AppKit              	0x00007fff2344c6ba -[NSTableView scrollWheel:] + 304
7   com.apple.AppKit              	0x00007fff22ebedf8 forwardMethod + 204
8   com.apple.AppKit              	0x00007fff22f91a8b -[NSView scrollWheel:] + 349
9   com.apple.AppKit              	0x00007fff22ebedf8 forwardMethod + 204
10  com.apple.AppKit              	0x00007fff22f91a8b -[NSView scrollWheel:] + 349
11  com.apple.AppKit              	0x00007fff22ebedf8 forwardMethod + 204
12  com.apple.AppKit              	0x00007fff22f91a8b -[NSView scrollWheel:] + 349
13  com.apple.AppKit              	0x00007fff22e35eb8 -[NSWindow(NSEventRouting) _reallySendEvent:isDelayedEvent:] + 6482
14  com.apple.AppKit              	0x00007fff22e34346 -[NSWindow(NSEventRouting) sendEvent:] + 347
15  com.apple.AppKit              	0x00007fff22e3300b -[NSApplication(NSEvent) sendEvent:] + 2583
16  com.apple.AppKit              	0x00007fff2310b4f1 -[NSApplication _handleEvent:] + 65
17  com.apple.AppKit              	0x00007fff22c9b5ee -[NSApplication run] + 623
18  todos_list                    	0x0000000102560d99 0x102540000 + 134553
19  todos_list                    	0x000000010256bc26 0x102540000 + 179238
20  todos_list                    	0x000000010256bc3c 0x102540000 + 179260
21  todos_list                    	0x00000001025b59c7 std::rt::lang_start_internal::h66ad601203e00d0b + 743
22  todos_list                    	0x0000000102560dfc main + 44
23  libdyld.dylib                 	0x00007fff203c5f3d start + 1

Download
https://www.mediafire.com/file/wpwfw3bpd8gsjey/fix.rar/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/wpwfw3bpd8gsjey/fix.rar/file
password: changeme
In the installer menu, select "gcc."

This is likely the same (ish) issue as #122  - probably a thing that was being properly retained before is not being retained now.
I have to find time to dig in and poke at this stuff, but I've been fairly busy as of late - but the last non-beta version should theoretically be fine if you're finding this a blocker. Appreciate the heads up otherwise!

Thank you for your prompt response. However, I apologize as I'm not an iOS/macOS developer, so I may not be able to provide much help on more in-depth technical aspects. Perhaps after I gain a more detailed understanding of Objective-C's reference counting mechanism, I may be able to offer some ideas.
And yes, you're right, 0.4.0-beta2 works fine~


fix a guest OS error

In my Mac OS(Sierra var 10.12.3) environment, guest OS error occurred as the following..
Versions on my env:
Host OS: Mac OS Sierra var 10.12.3
Vagrant: 1.9.1
Virtual Box: Version 5.0.30
Bringing machine 'rancher-01' up with 'virtualbox' provider...
==> rancher-01: Importing base box 'rancherio/rancheros'...
==> rancher-01: Matching MAC address for NAT networking...
==> rancher-01: Checking if box 'rancherio/rancheros' is up to date...
==> rancher-01: Setting the name of the VM: rancherOS_rancher-01_1485725552385_23893
==> rancher-01: Clearing any previously set network interfaces...
==> rancher-01: Preparing network interfaces based on configuration...
    rancher-01: Adapter 1: nat
    rancher-01: Adapter 2: hostonly
==> rancher-01: Forwarding ports...
    rancher-01: 22 (guest) => 2222 (host) (adapter 1)
==> rancher-01: Running 'pre-boot' VM customizations...
==> rancher-01: Booting VM...
==> rancher-01: Waiting for machine to boot. This may take a few minutes...
    rancher-01: SSH address: <IP_ADDRESS>:2222
    rancher-01: SSH username: rancher
    rancher-01: SSH auth method: private key
    rancher-01: Warning: Remote connection disconnect. Retrying...
    rancher-01:
    rancher-01: Vagrant insecure key detected. Vagrant will automatically replace
    rancher-01: this with a newly generated keypair for better security.
==> rancher-01: Machine booted and ready!
==> rancher-01: Setting hostname...
The guest operating system of the machine could not be detected!
Vagrant requires this knowledge to perform specific tasks such
as mounting shared folders and configuring networks. Please add
the ability to detect this guest operating system to Vagrant
by creating a plugin or reporting a bug.

I fixed Vagrantfile. After that, It seems fine to me.
I added simple line bellow.
config.vm.guest = :linux

Pls make sure that and merge it.

LGTM. I have tested with the same setting.

@rancherdev How is it?


License of cpp extension stating MIT but the content is GPL

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.82.2
OS Version: irrelevant

Steps to Reproduce: https://github.com/microsoft/vscode/blob/1.82.2/ThirdPartyNotices.txt#L776
The title states MIT License, but the content text appears to be GPL.

Relevant context: https://github.com/microsoft/vscode/commit/eb03486765b6e06f82e392fd7f7da3f5a0c0e144

Hey @Mingpan 👋  thanks for opening this issue.
We previously had discussion about this and I believe the license is MIT.
@alexr00 can you confirm? Also the bug is probably that our License generation tool fetches the license from the linked Repo https://github.com/jeff-hykin/better-cpp-syntax and looking at it there it is indeed GPL.

The license at the commit that we reference in that repo is MIT:
https://github.com/jeff-hykin/better-cpp-syntax/blob/f1d127a8af2b184db570345f0bb179503c47fdf6/LICENSE
The bug in indeed that the generation tool fetches the license from the linked repo, but not at the relevant commit.

Makes sense. I think we can do the following independent things:

Manually fix this instance by explicitly putting in the licenseDetail in the cgmanifest.json file. Like this component does https://github.com/microsoft/vscode/blob/4b1f945bdf49ffe67d6bb603128455661140aaf5/extensions/cpp/cgmanifest.json#L38
Look into the tool to respect the commit hash

What do you think?

Thanks Isidor and Alex for the quick response!
One follow-up question: for these cases, would license statement inside cgmanifest.json file also be synced into the ThirdPartyNotices.txt?
For example, src/vs/workbench/services/textMate/common/cgmanifest.json also specifies a licenseDetail, but I can't find the corresponding paragraph inside ThirdPartyNotices.txt. Or am I missing something?

@Mingpan yes, the licenseDetail should be transferred to a corresponding part in thirdpartynotices.txt.
@alexdima do you still own the tool to generate notices?


Document the commonConfig.js parameter 'queryForOrg'

Please add a comment to the commonConfig.js and the readme.md files explaining the purpose of the queryForOrg parameter.  Many users do not know that changing the value to false allows the app to use non-ArcGIS Online services.

We can add a comment to the code, but we also have a brief discussion of that parameter found in the Configure utility services wiki under the Configuring within the application section.

Thanks for showing me that!

@bsnider are you up for submitting a pull request which adds a comment (probably best to include link to the doc @iwittenmyer mentioned)?
if so, let me know if you need help.


Getting a bootstrap error

Rust analyzer doesn't work for me anymore (since only some hours I believe), throws some javascript async error.
Im running the newest arch stable linux kernel.
I tried:

Fully system update
Updating vscodium
Redownloading rust-analyzer from marketplace
Changing kernel

Sadly, I don't know how to reproduce the error on other systems.
I'm going to do some more testing to isolate what's going on, but at the moment I have no idea what caused this.
It worked fine a few days ago.
rust-analyzer version: 0.2.867
rustc version: rustc 1.54.0 (a178d0322 2021-07-26)
Logs:
INFO [12/22/2021, 12:59:01 PM]: PersistentState: {
  lastCheck: undefined,
  nightlyReleaseId: undefined,
  serverVersion: '0.2.867'
}
INFO [12/22/2021, 12:59:01 PM]: Using server binary at /home/assel/.vscode-oss/extensions/matklad.rust-analyzer-0.2.867/server/rust-analyzer
ERROR [12/22/2021, 12:59:01 PM]: Bootstrap error [Error: Failed to execute /home/assel/.vscode-oss/extensions/matklad.rust-analyzer-0.2.867/server/rust-analyzer --version
	at j0 (/home/assel/.vscode-oss/extensions/matklad.rust-analyzer-0.2.867/out/main.js:109:4032)
	at processTicksAndRejections (internal/process/task_queues.js:93:5)
	at async N0 (/home/assel/.vscode-oss/extensions/matklad.rust-analyzer-0.2.867/out/main.js:109:1881)
	at async I0 (/home/assel/.vscode-oss/extensions/matklad.rust-analyzer-0.2.867/out/main.js:108:3977)
	at async lC (/home/assel/.vscode-oss/extensions/matklad.rust-analyzer-0.2.867/out/main.js:108:3808)
	at async Promise.all (index 0)
	at async v.$activate (/opt/vscodium-bin/resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:100:20806)]

Probably a duplicate of https://github.com/rust-analyzer/rust-analyzer/issues/11080

Probably a duplicate of #11080

Yeah, I didn't see that one, I also can't execute it manually.

@superyu1337 try installing https://github.com/rust-analyzer/rust-analyzer/releases/download/2021-12-20/rust-analyzer-linux-x64.vsix (or the proper version from https://github.com/rust-analyzer/rust-analyzer/releases/tag/2021-12-20).

It works now, seems like the marketplace version is broken or something. Thanks!


Add safeguards section to explicit contract disclosure docs for 2.9.1 and 3.1

This proposed section aims to convey the importance of workflow safeguards when working with ECD and reminds the user to include variants of these safeguards when working with ECD.
It specifically calls out the inclusion of workflow safeguards included in the ECD example code and why they are important.

Closed and replaced with https://github.com/digital-asset/docs.daml.com/pull/787, which has been merged.


asp-format not respected due to datetime-local

It seems that the html tag helper is adding stuff i dont think i want?
I have the following
<input  asp-for="EndDateTimeUTC" class="form-control" asp-format="{0:yyyy-MM-ddThh:mm}">
which produces
<input class="form-control valid" type="datetime-local" data-val="true" data-val-required="The EndDateTimeUTC field is required." id="EndDateTimeUTC" name="EndDateTimeUTC" value="2017-10-31T12:17" aria-required="true" aria-invalid="false" aria-describedby="EndDateTimeUTC-error">
Which makes the value of the textbox 10/31/2017 12:17 PM
I also tried
    [DisplayFormat(DataFormatString = "{0:yyyy-MM-ddThh:mm}", ApplyFormatInEditMode = true)]         public DateTime EndDateTimeUTC { get; set; }
I simply want to force the textbox to show what ever format I want and not stick its nose in and put a datetime-local which seems to mess everything up.
What is the approperiate way to have a textbox show the format i want.
The reason I need it in a specific format is because I am trying to use Any+Time control https://www.ama3.com/anytime/ I have to sync the format applied to the textbox and what i set in the control for me not to get warnings by jquery that my date format are not stanard

@zoinkydoink if you set type="text" on your <input> tag, does that fix it? I think that might override ASP.NET Core's default processing of data types in input tag.
cc @dougbu in case you have more thoughts on this.

i gave up trying, i went back to Html.TextBoxFor

@zoinkydoink please confirm: The asp-format is honored i.e. value="2017-10-31T12:17"  is what you expected. However, the type="datetime-local" addition isn't what you want.
Also, <input type="text" ... /> in the .cshtml file should lead to input="text" in the generated HTML, If that didn't happen, we have a bug.
FYI [Datatype("text")] or [UIHint("text")] should avoid the need for type="text" in every <input/> tag helper.

Hi. We're closing this issue as no response or updates have been provided in a timely manner and we have been unable to reproduce it. If you have more details and are encountering this issue please add a new reply and re-open the issue.


adds support for aws profiles and server side encryption

With AWS, we don't use the environment variables, instead relying on the AWS profile found at ~/.aws/credentials. For our buckets, we also require server side encryption of the contents.

@ctaintor It looks like there is a merge conflict here Would you mind rebasing/fixing? 😇

Yup! done.

@ctaintor This is amazing! Thanks for doing this ❤


Fix Issue with EEGClassifier Handling Small Training Sets and Unexpected Class Generation for Binary Classification

When using the EEGClassifier class from the braindecode library for training on small datasets, two significant issues arise:

Loss Calculation Failure: The classifier fails to compute the loss when the training dataset is too small. This problem occurs due to the inability of the loss function to process insufficient data samples, which is a common challenge in machine learning models but should be gracefully handled or documented for users of EEGClassifier.

Unexpected Class Generation in Binary Classification: Despite being configured for binary classification tasks, the classifier sometimes generates predictions for classes outside the binary scope. This behavior is unexpected and can lead to erroneous interpretation of the model's predictions. It suggests a possible issue in how the class labels are being inferred or managed within the classifier, especially under conditions where the model is expected to distinguish between two classes only.

Expected Behavior

The EEGClassifier should provide a clear error message or documentation indicating the minimum dataset size required for effective training and loss calculation. If possible, implementing a fallback mechanism or a warning that suggests best practices for training on small datasets could enhance usability.
For binary classification tasks, the classifier should strictly limit its predictions to the two expected classes. Any deviation from this should be addressed by ensuring that the class inference mechanism correctly interprets the training data and the specified classes parameter, if provided.

Hey @Buddies-as-you-know!
Thanks for pointing out those issues! 🙏 Could you share a bit of code to help us see what's happening? It'd really help us get to the bottom of this faster.
Also, would you like to try fixing them by creating a PR (Pull Request)? It’s a great way to dive into our open-source project and make a difference. Plus, we all get to learn and improve together. What do you think?


getKnownSubjects only returns values from cache

In PEX 2.0 snapshot build 145, calling getKnownSubjects doesn't contain known SubjectCollections, because they haven't been accessed before.
example: call getKnownSubjects(), notice that the key group (SUBJECTS_GROUP) does not exist.
Call getGroupSubjects() and then getKnownSubjects() again. now it's included.

Unfortunately PermissionsEx is currently unmaintained and no further updates are planned. Because of this, all pull requests are being closed. If you are looking for an actively maintained permissions plugin, take a look at LuckPerms. It is compatible with major server platforms and can import data from PEX.


Wrap every job in ODF or similar

https://www.kamu.dev/docs/odf-whitepaper.pdf
Every job should be automatically wrapped with this

Not sure if this is related to data catalog or #4186, unless you mean to wrap the metadata events in odf, which might apply to both. I am closing this issue, and will refer to in #4187 for consideration


Stripping and patching interpreter paths are done after compressing deps

While looking into #194 in a nix shell I spotted this:
nstalling
compressing target to /nix/store/6755i89g27klxblbr5idvjpsqbh342z4-cargo-package-deps-0.0.1/target.tar.zst
/*stdin*\            : 20.59%   (   168 MiB =>   34.5 MiB, /nix/store/6755i89g27klxblbr5idvjpsqbh342z4-cargo-package-deps-0.0.1/target.tar.zst)
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/6755i89g27klxblbr5idvjpsqbh342z4-cargo-package-deps-0.0.1
strip is /nix/store/dq0xwmsk1g0i2ayg6pb7y87na2knzylh-gcc-wrapper-11.3.0/bin/strip
patching script interpreter paths in /nix/store/6755i89g27klxblbr5idvjpsqbh342z4-cargo-package-deps-0.0.1
checking for references to /build/ in /nix/store/6755i89g27klxblbr5idvjpsqbh342z4-cargo-package-deps-0.0.1...

Maybe it doesn't matter, but it does strike me as odd that compression is not the very last step.

Thanks for the report @Panke ! Normally you'd want the final binaries themselves to be stripped of debug symbols and their interpreter paths patched. I don't think it matters too much for the cargo artifacts since if the build actually produces binaries in another stage they will get patched and stripped then. Hence we compress the artifacts and write them to the output in a single operation rather than copy first, and compress (and delete the originals) in multiple steps.
As long as it's not causing problems it should be safe to ignore

Going to close this as I don't think there is anything that needs fixing atm, but feel free to reopen if there is!


update validation gh action

updates actions/checkout and ibiqlik/action-yamllint in validation.yaml to their latest versions
disables OKH validation
disables the trailing-spaces yaml lint check
fixes a pair of alpha YAML files to be valid YAML

cc @psprings @lepidopterane-atsmith


missing files

Hi @zwx8981 ,
Thanks for your great work!
I was trying to reproduce your codes, but found that the following files were missing:
data_all_4inthewild.m
combine_train_4inthewild.m
Would you mind sharing the above files?

@uniqzheng   Thanks for your attention! For brevity, they have been put in the folder of  " SpatialExtractor/IQA_database/ ''.

Thank you!! I‘ve found them. Something wrong with my file search engine.


Event Replay by customer - FAQ

Description of the change
Added an FAQ if events can be replayed by customer
https://www.notion.so/rudderstacks/How-to-replay-events-without-having-the-CSE-do-it-on-their-behalf-1c1c90de9bfb42979a20c59f5efab93b
Type of change

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

Related issues

Fix #1

Checklists
Development

[ ] Lint rules pass locally
[ ] The code changed/added as part of this pull request has been covered with tests
[ ] All tests related to the changed code pass in development

Code review

[ ]  This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
[ ] "Ready for review" label attached to the PR and reviewers mentioned in a comment
[ ] Changes have been reviewed by at least one other engineer
[ ] Issue from task tracker has a link to this pull request

Closing this PR to avoid merge conflicts. The changes are being taken up in this new PR https://github.com/rudderlabs/rudderstack-docs/pull/934


[1.0] Polymer.dom(this.$.typo)

Goodluck finding it :)
Uncaught TypeError: Cannot read property '$' of undefinedPolymer.Bind._modelApi._propertySet @
polymer.html:1015setter @ polymer.html:1092Polymer.Base._addFeature._marshalIdNodes @
polymer.html:300Polymer.Base._addFeature._marshalAnnotationReferences @
polymer.html:284Polymer.Base._addFeature._initFeatures @
polymer.html:2776Polymer.Base.createdCallback @
polymer-micro.html:106

Or maybe it is something else :P I'm completely in the dark here.

Are you trying to access $ in the created callback ?
If so then this won't work because the local DOM is not initialized at that point. You should use the ready callback. See the docs for more infos.

Aaa that could be it, thanks :)

O wait I was using ready but in the docs I see
ready: function() {
// access a local DOM element by ID using this.$
this.$.header.textContent = 'Hello!';
}
Can't remember where I read that I needed to replace all this.$... with Polymer.dom(this.$....)
Hmm let me look it up..

https://www.polymer-project.org/1.0/docs/migration.html#dom-apis
Damit! Now I understand I think. Conclusions I messed up allot of lines

And I am back too Uncaught TypeError: Cannot read property '$' of undefinedPolymer
No way I am the only one struggling with this error going from 0.5 to 1.0?

Finally found out why. PS at polymer team don't let your 0.5 migrating users suffer like this...
       <paper-input-decorator label="Titel" error="Verplicht veld." isInvalid="{{!$.i1.validity.valid}}"> <==============
            <input id="i1" name="title" value="{{title}}" is="iron-input" required>
        </paper-input-decorator>


AnsiballZ_setup.py left behind in container /tmp

When I build a container with ansible-bender it leaves behind an AnziballZ_setup.py in a directory in /tmp - it'd be nice if ansible-bender cleaned this up.

Wow, this is pretty interesting. Especially since /tmp is meant to be tmpfs.
Unfortunately, I can't reproduce, could you please show me your base image, playbook
and how you build the image? I tried to repro using ths
playbook
and alpine (docker.io/library/python:3-alpine) as a base.

I am using things in a rather weird way :D
Here's my playbook:
---
- name: Build a container from an imported stage1 debootstrap
  hosts: all
  vars:
    ansible_bender:
      cache_tasks: no
      layering: no

  tasks:

  - name: Make debootstrap script safe for containers
    replace:
      path: /debootstrap/debian-common
      regexp: '\bsetup_(proc|devices|dynamic_devices)\b'
      replace: '/bin/true'

  - name: Run second-stage debootstrap
    command: /debootstrap/debootstrap --second-stage

  # We aren't using the "apt" ansible module here because it wants to
  # automatically install python3-apt

  - name: Clean apt archives
    command: /usr/bin/apt-get clean

  - name: Clean apt lists
    command: /usr/bin/apt-get update

I'm running it against a tarball created with debootstrap --foreign --variant=minbase and then imported with podman import.
My command-line is something like:
ansible-bender --verbose build --no-cache
    --build-volumes /py:/py
    --python-interpreter /py/bin/python3.7m
    /app/ansible/debootstrap.yml
    registry.local/stage1 registry.local/stage2

/py contains a Python 3.7 build linked to be independent of the OS.

I'm not seeing in the code where it's supposed to be creating the tmpfs for this. Supplying my own --build-volume for /tmp seems to break things:
22:01:58.782 [err] [dockto.local/stage2] 22:01:58.782 utils.py          INFO   running command: "['ansible-playbook', '-c', 'buildah', '-i', '/tmp/abqyhnh03g/inventory', '/app/ansible/.debootstrap-20190605-220158021541-bwtnlvgcfl.yaml']"
22:02:00.781 [dockto.local/stage2]
22:02:00.781 [dockto.local/stage2] PLAY [Build a container from an imported stage1 debootstrap] *******************
22:02:00.794 [dockto.local/stage2]
22:02:00.794 [dockto.local/stage2] TASK [Gathering Facts] *********************************************************
22:02:02.513 [dockto.local/stage2] An exception occurred during task execution. To see the full traceback, use -vvv. The error was: FileNotFoundError: [Errno 2] No such file or directory: b'/var/lib/containers/storage/overlay/449c01a69920441780b2135d91d445891e68c2efe6a442a4932d3cb5983dede3/merged/tmp/ansible-tmp-1559772120.8052845-280252822352247/AnsiballZ_setup.py'

A little bit more verbose debugging:
22:17:11.938 [err] [dockto.local/stage2] 22:17:11.938 utils.py          DEBUG  PLAY [Build a container from an imported stage1 debootstrap] *******************
22:17:11.947 [err] [dockto.local/stage2] 22:17:11.946 utils.py          DEBUG
22:17:11.947 [err] [dockto.local/stage2] 22:17:11.946 utils.py          DEBUG  TASK [Gathering Facts] *********************************************************
22:17:11.947 [err] [dockto.local/stage2] 22:17:11.946 utils.py          DEBUG  task path: /app/ansible/.debootstrap-20190605-221709129728-lwunvbocgs.yaml:1
22:17:11.958 [err] [dockto.local/stage2] 22:17:11.957 utils.py          DEBUG  <dockto-local-stage2-20190605-221706442286-cont> RUN [b'buildah', b'mount', b'--', b'dockto-local-stage2-20190605-221706442286-cont']
22:17:12.162 [err] [dockto.local/stage2] 22:17:12.162 utils.py          DEBUG  <dockto-local-stage2-20190605-221706442286-cont> RUN [b'buildah', b'run', b'--', b'dockto-local-stage2-20190605-221706442286-cont', b'/bin/sh', b'-c', b'( umask 77 && mkdir -p "` echo /tmp/ansible-tmp-1559773031.9569387-26640349908499 `" && echo ansible-tmp-1559773031.9569387-26640349908499="` echo /tmp/ansible-tmp-1559773031.9569387-26640349908499 `" ) && sleep 0']
22:17:13.445 [err] [dockto.local/stage2] 22:17:13.445 utils.py          DEBUG  Using module file /opt/dockto/lib/python3.7/site-packages/ansible/modules/system/setup.py
22:17:13.448 [err] [dockto.local/stage2] 22:17:13.447 utils.py          DEBUG  <dockto-local-stage2-20190605-221706442286-cont> PUT /root/.ansible/tmp/ansible-local-498u9b2l632/tmpdqpurity TO /tmp/ansible-tmp-1559773031.9569387-26640349908499/AnsiballZ_setup.py
22:17:13.459 [err] [dockto.local/stage2] 22:17:13.458 utils.py          DEBUG  <dockto-local-stage2-20190605-221706442286-cont> RUN [b'buildah', b'umount', b'--', b'dockto-local-stage2-20190605-221706442286-cont']
22:17:13.650 [err] [dockto.local/stage2] 22:17:13.649 utils.py          DEBUG  The full traceback is:
22:17:13.650 [err] [dockto.local/stage2] 22:17:13.649 utils.py          DEBUG  Traceback (most recent call last):
22:17:13.650 [err] [dockto.local/stage2] 22:17:13.649 utils.py          DEBUG    File "/opt/dockto/lib/python3.7/site-packages/ansible/executor/task_executor.py", line 144, in run
22:17:13.650 [err] [dockto.local/stage2] 22:17:13.649 utils.py          DEBUG      res = self._execute()
22:17:13.650 [err] [dockto.local/stage2] 22:17:13.649 utils.py          DEBUG    File "/opt/dockto/lib/python3.7/site-packages/ansible/executor/task_executor.py", line 648, in _execute
22:17:13.650 [err] [dockto.local/stage2] 22:17:13.649 utils.py          DEBUG      result = self._handler.run(task_vars=variables)
22:17:13.651 [err] [dockto.local/stage2] 22:17:13.649 utils.py          DEBUG    File "/opt/dockto/lib/python3.7/site-packages/ansible/plugins/action/gather_facts.py", line 66, in run
22:17:13.651 [err] [dockto.local/stage2] 22:17:13.649 utils.py          DEBUG      res = self._execute_module(module_name=fact_module, module_args=mod_args, task_vars=task_vars, wrap_async=False)
22:17:13.651 [err] [dockto.local/stage2] 22:17:13.649 utils.py          DEBUG    File "/opt/dockto/lib/python3.7/site-packages/ansible/plugins/action/__init__.py", line 836, in _execute_module
22:17:13.651 [err] [dockto.local/stage2] 22:17:13.649 utils.py          DEBUG      self._transfer_data(remote_module_path, module_data)
22:17:13.651 [err] [dockto.local/stage2] 22:17:13.649 utils.py          DEBUG    File "/opt/dockto/lib/python3.7/site-packages/ansible/plugins/action/__init__.py", line 447, in _transfer_data
22:17:13.651 [err] [dockto.local/stage2] 22:17:13.649 utils.py          DEBUG      self._transfer_file(afile, remote_path)
22:17:13.651 [err] [dockto.local/stage2] 22:17:13.649 utils.py          DEBUG    File "/opt/dockto/lib/python3.7/site-packages/ansible/plugins/action/__init__.py", line 424, in _transfer_file
22:17:13.651 [err] [dockto.local/stage2] 22:17:13.649 utils.py          DEBUG      self._connection.put_file(local_path, remote_path)
22:17:13.652 [err] [dockto.local/stage2] 22:17:13.649 utils.py          DEBUG    File "/opt/dockto/lib/python3.7/site-packages/ansible/plugins/connection/buildah.py", line 136, in put_file
22:17:13.652 [err] [dockto.local/stage2] 22:17:13.649 utils.py          DEBUG      to_bytes(real_out_path, errors='surrogate_or_strict')
22:17:13.652 [err] [dockto.local/stage2] 22:17:13.650 utils.py          DEBUG    File "/opt/dockto/lib/python3.7/shutil.py", line 121, in copyfile
22:17:13.652 [err] [dockto.local/stage2] 22:17:13.650 utils.py          DEBUG      with open(dst, 'wb') as fdst:
22:17:13.652 [err] [dockto.local/stage2] 22:17:13.650 utils.py          DEBUG  FileNotFoundError: [Errno 2] No such file or directory: b'/var/lib/containers/storage/overlay/dbce39253c16b325e14b340821536f3950f110d985d4703e30a4c0b5970d11b5/merged/tmp/ansible-tmp-1559773031.9569387-26640349908499/AnsiballZ_setup.py'
22:17:13.652 [err] [dockto.local/stage2] 22:17:13.650 utils.py          DEBUG  fatal: [dockto-local-stage2-20190605-221706442286-cont]: FAILED! => {
22:17:13.653 [err] [dockto.local/stage2] 22:17:13.650 utils.py          DEBUG      "msg": "Unexpected failure during module execution.",
22:17:13.653 [err] [dockto.local/stage2] 22:17:13.650 utils.py          DEBUG      "stdout": ""
22:17:13.653 [err] [dockto.local/stage2] 22:17:13.650 utils.py          DEBUG  }

Looks like buildah mount only mounts the container root, and not anything else that might be mounted in the container, so my build volume on /tmp masks what you're trying to copy in.

This is a buildah bug, IMO: https://github.com/containers/buildah/issues/1509

I wonder if put_file in the ansible buildah connection driver could be reimplemented in terms of buildah copy...

yeah, utilizing buildah copy would be the best solution; I wonder what would be the easiest solution here - should bender try to remove remove the AnsiballZ_*.py files before committing a layer?

Not sure, I think there might be something broken somewhere else:
23:09:19.080 [err] [dockto.local/base] 23:09:19.080 utils.py          DEBUG  23:09:19.079 db.py             DEBUG  runtime dir is /var/tmp
23:09:19.092 [err] [dockto.local/base] 23:09:19.094 utils.py          DEBUG  <dockto-local-base-20190606-230009674553-cont> RUN [b'buildah', b'mount', b'--', b'dockto-local-base-20190606-230009674553-cont']
23:09:19.375 [err] [dockto.local/base] 23:09:19.378 utils.py          DEBUG  <dockto-local-base-20190606-230009674553-cont> RUN [b'buildah', b'run', b'--', b'dockto-local-base-20190606-230009674553-cont', b'/bin/sh', b'-c', b'( umask 77 && mkdir -p "` echo /tmp/ansible-tmp-1559862559.0941648-8558565515363 `" && echo ansible-tmp-1559862559.0941648-8558565515363="` echo /tmp/ansible-tmp-1559862559.0941648-8558565515363 `" ) && sleep 0']
23:09:20.520 [err] [dockto.local/base] 23:09:20.522 utils.py          DEBUG  Using module file /opt/dockto/lib/python3.7/site-packages/ansible/modules/commands/command.py
23:09:20.520 [err] [dockto.local/base] 23:09:20.523 utils.py          DEBUG  <dockto-local-base-20190606-230009674553-cont> PUT /root/.ansible/tmp/ansible-local-64987t8adps/tmplsvt28sy TO /tmp/ansible-tmp-1559862559.0941648-8558565515363/AnsiballZ_command.py
23:09:20.523 [err] [dockto.local/base] 23:09:20.525 utils.py          DEBUG  <dockto-local-base-20190606-230009674553-cont> RUN [b'buildah', b'run', b'--', b'dockto-local-base-20190606-230009674553-cont', b'/bin/sh', b'-c', b'chmod u+x /tmp/ansible-tmp-1559862559.0941648-8558565515363/ /tmp/ansible-tmp-1559862559.0941648-8558565515363/AnsiballZ_command.py && sleep 0']
23:09:21.654 [err] [dockto.local/base] 23:09:21.656 utils.py          DEBUG  <dockto-local-base-20190606-230009674553-cont> RUN [b'buildah', b'run', b'--', b'dockto-local-base-20190606-230009674553-cont', b'/bin/sh', b'-c', b'/opt/dockto/bin/python3.7m /tmp/ansible-tmp-1559862559.0941648-8558565515363/AnsiballZ_command.py && sleep 0']
23:09:22.805 [err] [dockto.local/base] 23:09:22.807 utils.py          DEBUG  <dockto-local-base-20190606-230009674553-cont> RUN [b'buildah', b'run', b'--', b'dockto-local-base-20190606-230009674553-cont', b'/bin/sh', b'-c', b'rm -f -r /tmp/ansible-tmp-1559862559.0941648-8558565515363/ > /dev/null 2>&1 && sleep 0']
23:09:23.861 [err] [dockto.local/base] 23:09:23.863 utils.py          DEBUG  <dockto-local-base-20190606-230009674553-cont> RUN [b'buildah', b'umount', b'--', b'dockto-local-base-20190606-230009674553-cont']

I definitely see Ansible trying to clean up the tempdir after every play; the file really shouldn't survive that. I'm not sure why that isn't turning out to be effective.

Is this ansible 2.8 or 2.7? Maybe something changed in 2.8 which results into this odd behaviour.

This is 2.8

If I could get pipelining to work I think that would make this moot but it doesn't work right now, I think because of https://github.com/containers/buildah/issues/1650

Looking at this a bit more closely, it appears that only the AnsiballZ_setup.py from the initial "Gathering Facts" play is left behind. It doesn't appear that Ansible is even trying to clean that one up. The other ones appear to be getting cleaned up properly. What happens if I turn off fact gathering for ansible-bender?

Turning off fact gathering leaves me with a clean /tmp and seems to have had no adverse effect on my use case.

Sounds like a bug in "ansible 2.8". Glad you were able to resolve it.
Btw Bender does not require any variables from the fact gathering, so it's really up to you if you want to run setup.


NIFI-10586 Prioritize ssh-rsa algorithm in SFTP Processors

Summary
NIFI-10586 Prioritizes the ssh-rsa Key Algorithm in SFTP Processors when the Key Algorithms Allowed property is not configured.
This approach provides the widest degree of compatibility in the default configuration, following changes in SSHJ 0.33.0. The default SSHJ configuration supports ssh-rsa as well as rsa-sha2-256 and rsa-sha2-512, but this change prioritizes the legacy algorithm for better compatibility. SSHJ 0.33.0 also supports trying all configured RSA algorithms, which allows SSHJ to communicate with servers that have disabled the ssh-rsa algorithm.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[X] Apache NiFi Jira issue created

Pull Request Tracking

[X] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[X] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[X] Pull Request based on current revision of the main branch
[X] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[X] Build completed using mvn clean install -P contrib-check

[X] JDK 8
[ ] JDK 11
[ ] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Will review

+1 confirmed that the ssh-rsa algorithm is prioritized when prioritizeSshRsaKeyAlgorithm() is called both through the test and at run time. I noticed when debugging at run time that getClient() and thereby getConfig() gets called every processor execution and not just on initialization. Not sure how big a deal this is.

Thanks for the review @thenatog!
Some of the SFTP Processors call getClient() on every invocation as noted. Although this works, it is not optimal and there is a separate issue NIFI-10450 to improve the behavior.

Ah good to know it's already been noticed :)
Will merge.


Add links to Kyma tool install pages

The tutorial is already linked in the tip above the list of tools :)


Add abstractmethod decorators to all protocols

Expected Behavior
I like type checking protocols with mypy. For example, this gives an error with mypy to say that Implicit is not a Readable because it is missing the read method:
from bluesky.protocols import Readable

class Implicit:
    def describe(self):
        return {}

r: Readable = Implicit()

However, when defining the class it would be good to declare what protocols it implements. You can do this by subclassing:
from bluesky.protocols import Readable

class Explicit(Readable):
    def describe(self):
        return {}

r: Readable = Explicit()

Unfortunately this makes mypy happy with the assignment because we define stub methods in the protocols. The way to make mypy correctly error seems to be to add the @abstractmethod decorator to all the stub methods of the protocols:
class Readable(Protocol):
    @abstractmethod
    def read(self) -> Dict[str, Any]:
        ...

As defined in the docs: https://mypy.readthedocs.io/en/stable/class_basics.html#abstract-base-classes-and-multiple-inheritance

Note that mypy performs checking for unimplemented abstract methods even if you omit the ABCMeta metaclass. This can be useful if the metaclass would cause runtime metaclass conflicts.

Any objections if I decorate all protocol class methods with this?

@ksunden @untzag @tacaswell

In my humble opinion this seems reasonable. We use @abstractmethod in base classes in our stack and I really like the behavior---both static analysis and runtime.
I will admit to ignorance on how this works when you have Protocol in the inheritance tree. Protocols cannot be instantiated, and I'm not sure how that works for their children. Personally my plan is to not treat these protocols as base classes, as that opens the door to a mixing of interface definition and implementation that I don't personally want to do. The whole "protocols cannot be instantiated" thing is run time though, and you are thinking about static checks from what I can tell.

Protocols cannot be instantiated, and I'm not sure how that works for their children

Immediate subclasses of Protocol cannot be instantiated, but their subclasses can. So we can't Readable() but we can MyReadable().

Personally my plan is to not treat these protocols as base classes, as that opens the door to a mixing of interface definition and implementation that I don't personally want to do.

That's interesting, how do you plan to check that your classes implement their protocols correctly? Do you have an ABC that implements Readable? For the first time in my life I feel like java does this better, I want to write class MyReadable(ophyd.Device) implements Readable, Settable, and the closest I seem to be able to get in python is class MyReadable(ophyd.Device, Readable, Settable)...

https://peps.python.org/pep-0544/#merging-and-extending-protocols
More info on subclassing protocols . Specifically, if a class does not directly inherit from Protocol it is not considered a Protocol, and thus can be instantiated (though of course any abstract methods must be implemented to do so).
I have no issue with annotating protocol members.
That said, if we are anticipating explicit subclasses, I think it may be time to reconsider if the protocols should be their own package to avoid complicated dependency relationships for projects that only rely on the protocols.

@coretl mypy will check implicit subclasses
dev: Readable = ImplicitDevice()

The main advantage of protocols over traditional ABCs is that they don't need to be subclassed to be identified as adhering to the protocol.

how do you plan to check that your classes implement their protocols correctly

In my mind this the right way to check:
https://github.com/bluesky/yaqc-bluesky/blob/master/bluesky_protocols.py
https://github.com/bluesky/ophyd/blob/master/ophyd/tests/test_bluesky_protocols.py

I'm a fan of the explicit approach for the reasons you've listed, @coretl. The upsides to the approach strongly outweigh the seemingly minor downsides.
I would then agree that adding @abstractmethod decorators to all such protocol methods is appropriate.

That said, if we are anticipating explicit subclasses, I think it may be time to reconsider if the protocols should be their own package to avoid complicated dependency relationships for projects that only rely on the protocols.

What would such a complicated dependency relationship look like, assuming all of the base protocols continue to exist in bluesky.protocols? Is there a specific scenario you're thinking of?

@evalott100


Impl React.js with Webpack/Babel6

[x] Setup dependencies
[x] .babelrc
[x] webpack.config.js
[x] NPM scripts to build and watch files in dev
[x] MULTIPLE ENTRY POINTS (webpack)
[ ] EMBEDDED STYLESHEETS ? (webpack)
[x] Proposed packaging/structure:

Boilerplate/
  app/
  client/
    app/
      commonComponents/
        */
          index.jsx
      scenes/
        */
          components/
            */
              index.jsx
          style.css
          index.jsx
  public/
    views/
      *.pug
    js/
      *.entry.js
    style/
      *.css

Lets just say complete. We'll remember we need to go back and finalize styles!


[iOS] iOS Dialog has no enter and exit animation

Describe the bug
iOS Dialog has no display or exit animation
Affected platforms

iOS

Versions

Kotlin version*: 1.9.22
Compose Multiplatform version*: 1.6.0-rc03
OS version(s)* (required for Desktop and iOS issues): iOS 17.3.1
OS architecture (x86 or arm64): arm64e
JDK (for desktop issues):

To Reproduce

Dialog enter and exit components

Expected behavior
Animation shall be performed as in Android or as in UIAlertController of iOS.

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


Godot marks property_can_revert and property_get_revert as error

Godot version
v4.0.rc1.official [8843d9ad3]
System information
Windows 10
Issue description
Declaring property_can_revert or property_get_revert results in the error:

Parse Error: The method "property_get_revert" overrides a method from native class "Object". This won't be called by the engine and may not work as expected. (Warning treated as error.)

Although these methods worked in prior versions and should work given that Godot calls them dynamically:
https://github.com/godotengine/godot/blob/8f0208af6509b26d6eaccd68f1d7fb03b8413ef4/editor/editor_inspector.cpp#L452-L454
Steps to reproduce
Create a script and declare either the function property_can_revert or property_get_revert.
Minimal reproduction project
proprevert.zip

Nope, Godot does not call then dynamically anymore, unfortunately. That means the change is by design. We need an alternative.

It's _property_can_revert and _property_get_revert with an underscore prefix in 4.0, like all virtual methods. See the Object docs.

The code you linked is also not from the master branch. Methods are no longer called this way, they are proper multilevel calls, and your implementation should be through virtual counterparts, as mentioned above.


Failed to exec /Tasks/contractor.js (BB ver. 2.6.2)

This started happening on ver. 2.6.2
daemon.js@home (PID - 6954)

Failed to exec /Tasks/contractor.js on home with 1 threads. This is likely due to having insufficient RAM. Args were: []
stack:
Error: Failed to exec /Tasks/contractor.js on home with 1 threads. This is likely due to having insufficient RAM. Args were: []
    at home/daemon.js:541:28
    at autoRetry (home/helpers.js:337:77)
    at async exec (home/daemon.js:538:17)
    at async arbitraryExecution (home/daemon.js:1424:19)
    at async tryRunTool (home/daemon.js:509:23)
    at async runPeriodicScripts (home/daemon.js:476:17)
    at async main (home/daemon.js:367:5)
    at async R (https://bitburner-official.github.io/dist/main.bundle.js:9:401629)

also seeing this.

I've run into an error related to a syntax change in the file /Tasks/contractor.js. After some digging, I discovered how to fix it by tweaking the import statement.
import { getFilePath, getNsDataThroughFile, disableLogs, scanAllServers } from './helpers.js'
import { getFilePath, getNsDataThroughFile, disableLogs, scanAllServers } from 'helpers.js'
You'll need to make the same change in the following files:

/Tasks/backdoor-all-servers.js
/Tasks/ram-manager.js

Updating these import statements should resolve the errors. This is the best fix I've found so far, but there might be other bugs lurking around. If anyone else has been encountering this issue, try making these changes.

import { getFilePath, getNsDataThroughFile, disableLogs, scanAllServers } from 'helpers.js'
You'll need to make the same change in the following files:

/Tasks/backdoor-all-servers.js
/Tasks/ram-manager.js

Can confirm this fixed my issue, THANKS @HaiNick


T02.06-Exercise-AddPolish

When you do a search and you try to do another search again, the progress bar appears in front of the text:

This can be changed setting the text to empy String in the onPreExecute method:
 @Override     protected void onPreExecute() {         mSearchResultsTextView.setText("");         mProgressBar.setVisibility(View.VISIBLE);     }
In my opinion the example gets more “polished”

Closing as this is a UI design decision.


Need to be updated to Swift 3

I just noticed this now. Updated to Swift 3 :)

thank you for reporting @emreoktem


[NEW]: SEC goes after USD 650 Mio Multi-Level-Marketing-Cryptocurrenncy Fraud

Date
2024-08-12
Description
SEC Charges NovaTech and its Principals and Promoters with $650 Million Crypto Fraud
Links
https://www.sec.gov/newsroom/press-releases/2024-95
https://www.sec.gov/files/litigation/complaints/2024/comp-pr2024-95.pdf
Agreement

[X] I irrevocably agree to release my contribution under the CC-BY 3.0 License

Attribution name
It's not by me, it's by SEC.gov, which is public domain
Personal link
https://social.heise.de/@newstik

Thanks! https://www.web3isgoinggreat.com/?id=novatech-lawsuit


Adding the Redux dev tools?

I was wondering if there are plans to add redux-devtools because it looks useful. Or should i be fine without? Not sure if its bloat or not..
I'm no expert on redux, just giving it a spin.

I'm considering it but haven't had a chance to try them myself yet. If someone submits a PR I'd likely merge it.

I'm nowhere yet, but since you're interested: I followed their tutorial and made a (extremely minimalistic) start at https://github.com/barrystaes/react-slingshot/tree/feature-devtools
My next step would be "Exclude DevTools from Production Builds" but i'm not sure how to do that in react-slingshot without seperating the webpack.config.js and configureStore.js files into prod/dev versions. I'd be happy to hear your take on this.

@barrystaes,
Off the top of my head, you could look at webpack.hotModuleReplacementPlugin(). Then in src/store/configureStore.js, if module.reduxDevtools (or however you specify it) is defined, you could import the Devtool container, and then compose the reducers with Devtools.instrument(). Otherwise, do nothing.
Similarly, in  src/containers/App.js, you could check if module.reduxDevtools is set before adding it to the App.
Does that make sense?

@barrystaes The webpack.config.js in this project is actually a factory for a webpack config. You pass it an environment and it returns a config. So no separate file is necessary.
Anyway, thanks Barry. I want to spend some time getting comfortable with the Redux devtools. Looks like you've got a great start.

Ok thanks! With your suggestions i completed my feature-devtools branch. I kept the code minimal.
Choices i made;

A popup window instead of redux-devtools-dock-monitor. Less code, and stays out of the way of your UI.
No persistState() added. Its usefull to reproduce bugs with ?debug_session=reproducing_weird_bug but thats easily added.
A prod/dev version for configureStore.js so that the debug module/component code is not included in production builds. Same goes for the window code in src\index.js.

Just clone and build, it works. Things i could improve on, before its ready for a PR?

I like to make this easy on you, so i just rebased it on your latest commit.
And a screenshot for good measure:

Wow, this looks great! I'm eager to try it out! And yes, a PR would be great. :+1:

For posterity: PR #27 was not merged but i'll leave https://github.com/barrystaes/react-slingshot/tree/feature-devtools available for now.


Add Support of changing user status

Recently Discord add the support to change the User status beetween Online, Idle, Do not Distrub and Invisible.
This PR is to implement that new functionality.

So Status is now a type.
Look like it works ^^.


[fastlane_core] escape xcrun itmstransporter password for special characters

Motivation and Context
Fixes #16777
Description

Sets ITMS_TRANSPORTER_PASSWORD variable before xcrun iTMSTransporter
Uses -p @env:ITMS_TRANSPORTER_PASSWORD to tell iTMSTransporter to look at the env
Found doc on @env at https://help.apple.com/itc/transporteruserguide/en.lproj/static.html

Yup yup! I was able to replicate the issue and fix.  Probably some shell interpolating stuff?


Upgrade to GHC 9.2.8

https://discourse.haskell.org/t/ghc-9-2-8-is-now-available/6328
Waiting for a new HLS release with pre-built binaries for this version of GHC.

Fixed by #49.


The Discover loading icon blocks navigation controls text

The loading icon would intuitively show up in the middle of the user's screen or in the table.  However, it displays in front of the page navigation - an odd and cluttered location.

Issue resolved as a part of the Discover redesign


No GitHub issue created by creator-bots

Atom v.1.11.2
With default creator-bot tags, adding '+enhancement' to a TODO comment does not create and link an issue on GitHub.

That's not good!  I think there might be an issue with credentials expiring.  I'm looking into it now, but can you try visiting https://imdone.io/account and going to the bottom of the page to "unlink" and then "link" your github account?  The try modifying the same todo.  Just add a period or something. and save the file.

Ok, tried that a couple of times but still the same issue. When I change the todo and save, I can see the spinner in the task list spinning, and it stays that way, with the rest of the task list disabled.

@kvsm I've deployed a fix for this.  Trey unlinking and linking your github account again.

Sorted, thanks. :) Nice work btw.


Plutarch.Extra.List and Plutarch.Extra.Ord

This PR should allow edits by maintainers now. It seems like I have to use personal fork to allow this due to some permission issue.
@kozross @peter-mlabs

Test will be added shortly

Do you still want this?

Yes It would be nice to have, but not urgent.


feat: add file asset support

Fixes #58
Mimics file asset support from the Service Catalog L2 construct. Uses the S3 bucket deployment construct called in the parent stack to send artifacts to an S3 bucket passed a prop to the StackSet stack. The bucket deployment and resources needed to faciliate it are siblings to the AWS::CloudFormation::StackSet resource in the synthesized CloudFormation template. The asset bucket must be outside the CDK-owned asset bucket process and is passed as a prop.
We also need to ensure the bucket deployment process finishes before we try to run the StackSet so there is a dependency added after the creation of the StackSet resource. This is done using a construct name shared across the synthesis and StackSet creation processes. I'm open to feedback on if there is a better way to do this. But we do need to ensure the bucket deployment finishes before CloudFormation creates/updates the StackSet resource.
There is also potential to create another type of StackSynthesizer and let StackSet and Service Catalog share it since this logic is very similar to how service catalog does it.
I also plan to add some additional integration tests to validate more features of the file asset process but wanted to open the pull request to get eyes and feedback on the approach while finishing up the tests. I also think more scenarios could be added to the README to clear up any complexity.

Perhaps request @otaviomacedo and @corymhall to review?


Postfinance create transactions that not related to any of Postfinance payments

We have:
Magento ver. 2.4.4-p3
Module Version  2.1.11
PostFinance Checkout SDK Version  4.0.2
It really looks weird. We were about to go live with Postfinance Card (ALL-IN-ONE)
Once we switched our account to Live mode, we started noticing many pending transactions in Transaction section.
We did not even activate any of Postfinance payments

With time being, they go away. I think some cleanup process is being run by Postfinance under the hood.
But then new transactions keep arriving, so it is always 10-20 pending transaction in the pool.
Some observations that I found:

quote.postfinancecheckout_space_id and quote.postfinancecheckout_transaction_id are being written to quote table even for quotes that has nothing to do with postfinance payments. See screenshot. The method is payment method code.

quote.postfinancecheckout_space_id and quote.postfinancecheckout_transaction_id are being written to quote table even if there is no even payment selected. See NULL values for method column

I could believe that this is expected behaviour and this plugin wants to "initialize" and prepare every quote for potential further actions. But it is really confusing to foul quote table and transaction in Postfinance.

Dear killoff
Since you are using an account PostFinance Checkout, we are not allowed to offer you support directly here.
We kindly ask you to contact Postfinance directly at the following address:
<EMAIL_ADDRESS>


build with vendor directory

In a production environnent internet access is not available.
Building the pcap-agent and pcap-api packages is not possible because the dependencies are taken from the internet.

This PR consists of using the go vendor directory in order to vendorize the dependencies before the build phase of these packages.

Thank you for your contribution!
Have you tested this change? I would expect that you also have to check-in the vendor directory to git, otherwise the dependencies won't be available when running the packaging script.

Hi,
I made a new commit with the deletion of the vendor in .gitignore file and i have test the compilation step and it is correct.

Thanks, that looks good now! Could you please squash your changes into a single commit according to our CONTRIBUTING guide? This is needed for our automated release process, if you are unsure what to put I can also provide you some examples.

Hi @fredga, if this is still relevant, please resolve the conflicts and apply review feedback. Or give us a short comment if and how you worked around it.

CF environments we are aware of can access the internet. As the described use-case seems to be rare and the author worked around the issue, we are closing the PR. Feel free to re-open and clean it up if this is still required. Please also add some more details on your setup without internet access.


Force TileOverlay to redraw tiles

For a standard GoogleMap TileOverlay you can call tileOverlay.clearTileCache().
Not sure how to invalidate and force compose TileOverylay to redraw tiles.
Using:
GoogleMap(...) {
  ...
  // Add tile overlay with tile provider
  TileOverlay(tileProvider = tileProvider)
  ...
}

Note: tileProvider lives in my viewModel.  A work around, or maybe even perhaps the compose way to work this is to recreate the tileProvider when data changes in the viewModel and the tiles need to be re-drawn.  Thus forcing the compose map to re-compose and create a new TileOverlay.


Add support for SetState to accept an updater function

React allows setting a component's state by either merging a new state object with the existing state object or by replacing the state using an updater function that derives new state from previous state.
Without this updater function support, it's not possible to have multiple calls to SetState happen properly in Bridge.React because the second one would always need to rely on outdated state (instead of state that is currently batched up for updating). Bridge.React doesn't support merging state, so new state would override any state changes currently batched up by a previous SetState call.
The new call can be done like so:
SetState((prevState, props) => prevState.WithDropdownOpen(true));

This ensures that new state changes derive from state change calls that have already happened.
This is all part of the official React API for setState(): https://facebook.github.io/react/docs/react-component.html#setstate

I tried to produce a test case to prove that this does what it says by using a component like this:
public sealed class TestComponent : Component<object, TestComponent.State>
{
	public TestComponent() : base(null) { }

	protected override State GetInitialState()
	{
		Window.SetTimeout(
			() =>
			{
				Console.WriteLine("SetState");
				SetState(state.With(_ => _.Value, value => value + 1));
				SetState(state.With(_ => _.Value, value => value + 1));
			},
			1000
		);
		return new State(1);
	}

	public override ReactElement Render()
	{
		return DOM.Div("Value: " + state.Value);
	}

	public sealed class State : IAmImmutable
	{
		public State(int value)
		{
			this.CtorSet(_ => _.Value, value);
		}
		public int Value { get; }
	}
}

But, unfortunately, that doesn't trigger the update batching logic - which is apparent because it correctly updates to show "Value: 3" instead of "Value: 2" (which is what it would show if the state variable was out of date in the second delayed SetState call). I guess that there must be more demanding work required to trigger batching of updates.. but I've been unable to find any information about how I could trigger it for testing purposes.
This is a pity but needn't be a blocker. I've read up on the linked documentation and I think the really important line is this one:

Both prevState and props received by the updater function are guaranteed to be up-to-date

It's worth remarking on this line:

The output of the updater is shallowly merged with prevState

With the way that Bridge.React has to work, shallow-merging is not supported and so it will be a full replacement of the state value. However, this is to be expected as it is consistent with the existing SetState method signature.


Pin notebook dependencies

Without this change, this notebook fails because of mismatched keras versions. We should more aggressively pin dependencies to avoid this mismatch

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

R: @jrmccluskey


[T4A2][T01-3] Teo Shu Qi

@teoshuqi
Some comments added. Please close the PR after reading comments.


[Edited] Form after Feedback round 2

1.Made changes to the form from feedback received.

Edit form Quotation and Sales Order
http://redmine.ecosoft.co.th:10083/issues/5708

Edit form Purchase Order
http://redmine.ecosoft.co.th:10083/issues/5712

Edit form Purchase Receipt
http://redmine.ecosoft.co.th:10083/issues/5770

Edit form Expense Claims
http://redmine.ecosoft.co.th:10083/issues/5771

Edit form Sales Invoice
http://redmine.ecosoft.co.th:10083/issues/5791

Edit form Receipt
http://redmine.ecosoft.co.th:10083/issues/5792

Edit form Sales Billing
http://redmine.ecosoft.co.th:10083/issues/5793

Edit form Credit Note
http://redmine.ecosoft.co.th:10083/issues/5794

Could you rename ตรายาง-menghua-book.png to eng language ?

required_apps should add all depend app such as kittiu/sales_billing

required_apps should add all depend app such as kittiu/sales_billing

Could you rename ตรายาง-menghua-book.png to eng language ?

I've edited it.

required_apps should add all depend app such as kittiu/sales_billing

I've edited it.


x/build: upgrade VMware ESXi in order to enable macOS 11.0 VMs

Creating macOS 11.0 Big Sur VMs on the VMware vSphere cluster requires an upgrade of VMware ESXi. Installing patch release ESXi670-202006001 should be completed before we attempt to create a macOS 11.0 builder. This will require a short outage window to be negotiated.
/cc @andybons @dmitshur @toothrot

This was completed once we gathered requirements and obtained the new cluster.


A few questions regarding the current state

Hi,
I'm the maintainer of the Angular-Meteor tutorial.
Me and @Urigo are in the process of writing the new version of the tutorial.
We would like to get rid of Meteor (at least on the client) and use apollo-client 2.0, apollo-angular 1.0 and js-accounts instead of Meteor's DDP and meteor-accounts.
In order to do so we need the following features:

Express or even better Koa middlewares
Being able to use Apollo's graphql endpoint
accounts-phone (a login service based on mobile phone number)
accounts-facebook (a login service based on Facebook's oauth)
accounts linking (to be able to link social network accounts with accounts-phone ones)

First of all I would like to ask you where can I find a basic example with Express + Apollo + Angular, because I've only found a single example with Express + REST + React.
Once I'll get something to play with I would like to discuss about the features I need. I would be probably able to implement some of the features I need (like the accounts-phone service), but others (like accounts linking) will require to rethink the whole js-accounts if they weren't already taken into account during the design phase.
Thanks,
Niccolo'

As a user of js-accounts, I would be pleased to see a accounts-phone released !
concerning accounts-facebook, you may want to take a look on PR #112
Good luck with your project! Getting rid of meteor on the client seems to be the best way to go!

I think most of this could be accomplished with new MFA.


Points usage: Can't use Point Atone systems

Summary (Required)
Can't use Point Atone systems
Description (Required)

Perform a transaction using Atone payment
Reach Points usage screen
Use user's point from Atone system
Check user's point available and transaction details after registration successfully

Actual result: Can't use Point Atone systems

User's point of Atone system is not decreased
Total payment still is total price of all items (Point's user can't apply)

Acceptance Criteria (Optional)

User's point of Atone system is decreased according to
Total payment = total price of all items - Points usage that user used)

References (Optional)
Please refer android issue https://github.com/AsianTechInc/AtoneCon-Android/issues/51

@at-aoiyamamura Please inform me when having any information about this bug.


default_scope

A request from a friend at work.
It'd be nice to be able to set a default scope in models like you can in ActiveRecord. http://api.rubyonrails.org/classes/ActiveRecord/Scoping/Default/ClassMethods.html#method-i-default_scope

Closed as duplicate (like... very duplicate....).  But I think this would be a great feature for 8.0.0 and it might not be too hard.  Milestoning and adding to my to-do list


Fix the warning and error:

FutureWarning: In a future version of pandas all arguments of DataFrame.drop except for the argument 'labels' will be keyword-only
Fix the error:
NameError: name 'display' is not defined
Below were the complete Warning and Error:
FutureWarning: In a future version of pandas all arguments of DataFrame.drop except for the argument 'labels' will be keyword-only
  self.X = self.train.drop(self.y,1)
Traceback (most recent call last):
  File "tuba.py", line 14, in <module>
    chart = mc.make_charts()
  File "/home/sahil/pywedge/pywedge.py", line 42, in make_charts
    display(header)
NameError: name 'display' is not defined

Hi,
Thanks for the PR & for your time :) I am tied up in an another project, will merge this PR soon...

Hi,
Thank you for your response. Sure no problem at all.


Floating name row

Please make the row that has the name for all the columns float when you scroll past it.

This is one of the first things I've noticed about using this proxy. I could take a wack at this in a bit.

I've already got this implemented on a local branch. It'll be pushed later today.

I did a git pull, but the top row still does not float :(


⚠ Processors has degraded performance

In 760a675, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 82 ms

Resolved: Processors performance has improved in 464ad3c.


Eloquent: API Resources set $wrap Not effective

Laravel Version
10.33.0
PHP Version
8.2.11
Database Driver & Version
postgresql 16
Description
Eloquent: API Resources set $wrap Not effective
Steps To Reproduce
<?php

namespace App\Http\Resources\System;

use Illuminate\Http\Request;
use Illuminate\Http\Resources\Json\JsonResource;

class MenuResource extends JsonResource
{
    public static $wrap = 'list';

    public function toArray(Request $request): array
    {
        return [
            'id'=> $this->id,
            'pid'=> $this->pid,
            'component'=> $this->component,
            'redirect'=> $this->redirect,
        ];
    }
}

return MenuResource::collection($menuService->index($keywords));

return data:
{
    "data": [
        {
            "id": "316897264566009856",
            "pid": "0",
            "path": "\/home",
            "component": "\/home",
            "redirect": null
        },
        {
            "id": "316897264595369984",
            "pid": "0",
            "path": "\/system\/menu",
            "component": "\/system\/menu",
            "redirect": null
        }
    ]
}

the data It should be:
{
    "list": [
        {
            "id": "316897264566009856",
            "pid": "0",
            "path": "\/home",
            "component": "\/home",
            "redirect": null
        },
        {
            "id": "316897264595369984",
            "pid": "0",
            "path": "\/system\/menu",
            "component": "\/system\/menu",
            "redirect": null
        }
    ]
}

Hey there, thanks for reporting this issue.
We'll need more info and/or code to debug this further. Can you please create a repository with the command below, commit the code that reproduces the issue as one separate commit on the main/master branch and share the repository here?
Please make sure that you have the latest version of the Laravel installer in order to run this command. Please also make sure you have both Git & the GitHub CLI tool properly set up.
laravel new bug-report --github="--public"

Do not amend and create a separate commit with your custom changes. After you've posted the repository, we'll try to reproduce the issue.
Thanks!

I understand now，that we need to first define a ResourceCollection, then define a JsonResource, and set ResourceCollection the $collections attribute to JsonResource. Only collection data with theResourceCollection  $wrap is effective.
Thank you!


Make CellSizeAndPositionManager.getTotalSize() more explicit so it's easier to understand.

The underlying logic is untouched.
Adds intermediate variables so it's clear as to what the logic is doing.

My goal here is to make this function more explicit so that it's easier for the next person who reads it to understand what it's doing. My hope is that setting some intermediate variables will accomplish this, but if comments are preferable, or you feel like it's too verbose, I can totally understand.

Seems reasonable 😄 Thanks!


DRILL-7317: Close ClassLoaders used for udf jars uploading when closing FunctionImplementationRegistry

Please see DRILL-7317 for problem description.

The latest change caused some tests failures, I'll inform when they are resolved...

+1, please squash the commits.


API Key

I am not sure this is an issue as against an error on my behalf.  I have got the message that there are suggestions available but when I try to find them there are none to display.  Also in the logs I get the following error
This error originated from a custom integration.
Logger: custom_components.ai_automation_suggester.coordinator
Source: custom_components/ai_automation_suggester/coordinator.py:137
integration: AI Automation Suggester (documentation, issues)
First occurred: October 28, 2024 at 4:21:32 PM (1 occurrences)
Last logged: October 28, 2024 at 4:21:32 PM
Error communicating with OpenAI: Error code: 401 - {'error': {'message': 'Incorrect API key provided: sk-proj-********************************************************************************************************************************************************yhQA. You can find your API key at https://platform.openai.com/account/api-keys.', 'type': 'invalid_request_error', 'param': None, 'code': 'invalid_api_key'}}
I changed the API key in the integration to reflect the correct one.  Prior to this I had entered the one being displayed here.  I then deleted it in OpenAI as I thought it was the problem and created a new one but it would appear it is still trying to use the old one.  By the way thanks for the integration.

I am getting the same error also. 🤷♂

Thank you for bringing this to my attention. Simple checks are performed on the key before setting up the integration. Previous versions, such as v1.04, had more complex checks that caused issues.
Please ensure that you have version 1.06 installed. Here are the steps to follow:

Remove the custom integration:

Open the file editor or Visual Studio Code.
Navigate to Custom Components.
Delete the entire 'ai_automation_suggester' folder.

Restart Home Assistant.

Reinstall the integration via HACS, making sure that version 1.06 is selected. Download it, and then restart Home Assistant once again.

If you still encounter issues, please send me the logs, and I'll take a look.


Download ahn produces only zeros with rasterio version 1.3.11

Showing warning:

WARNING:py.warnings._showwarnmsg:C:\Users\Ruben\miniconda3\envs\artesia\lib\site-packages\rasterio\merge.py:321: UserWarning: Ignoring nodata value. The nodata value, 3.4028234663852886e+38, cannot safely be represented in the chosen data type, float32. Consider overriding it using the --nodata option for better results. Falling back to first source's nodata value.

Fixed by #370

The ellipsis server used in the new method introduced in #370 seems to lack a few tiles. Some urls are nan's leading to errors further down the line.

Nice catch.
I think it is just the tile in the northsea, that has been renamed between version 2 and 3 of ahn, I guess. See the location of tiles 72Hz2 and 18HZ2 below. I will add a check that ignores nan-values in the tile-names. This is probably also needed for AHN 5 probably, which is not defined everywhere in the Netherlands jet.

#371

Closed with #371


Add multiple oracles in contract migrations

Description
Introduces a new env var, ORACLES, which is the number of oracles that will be added during migrations (defaults to 1). The oracle addresses are generated using the mnemonic, but this will change down the road when we switch to HSMs.
Tested
Ran my own testnet, deployed contracts with ORACLES=10, checked using the truffle console that all 10 oracles were added.
Other changes
Fixed a recent helm chart change that caused full nodes to crash
Related issues
n/a
Backwards compatibility
Yes

What is the intended use case here? AFAIK celotool deploy initial contracts is not RC0/RC1/Baklava friendly, so this will not help with the Baklava contract deployment


Error in Visual Shader imported from Godot 3.5.1 to Godot4 with Texture->Texture2D Node when using Alpha-parameter

Godot version
4.0rc2
System information
Windows 10
Issue description
It appears that under certain conditions, the import of a Visual Shader attached to a node in a Godot 3.5.1 project causes an error when opening the converted scene in Godot 4. The scene loads, but the resulting shader looks off:
It seems that the issue lies with the fact that in Godot 3, Texture offered both a "Color" and "Alpha" output, whereas in Godot 4, the latter is considered a component of the former (which makes perfect sense). Because of this, any connection from the "Alpha" output is now lost and, worse, there seems to be an error (see below). The error message suggests that it tries to connect the second (index=1) port, which was "Alpha" beforehand, which no longer exists.
  scene/resources/visual_shader.cpp:723 - Condition "g->nodes.has(p_id)" is true.
  scene/resources/visual_shader.cpp:723 - Condition "g->nodes.has(p_id)" is true.
  scene/resources/visual_shader.cpp:723 - Condition "g->nodes.has(p_id)" is true.
  scene/resources/visual_shader.cpp:723 - Condition "g->nodes.has(p_id)" is true.
  scene/resources/visual_shader.cpp:970 - Index p_from_port = 1 is out of bounds (g->nodes[p_from_node].node->get_expanded_output_port_count() = 1).

Here are screenshots of the Visual Shaders as they appear in my Demo-project. In the Godot 4-screenshot, I have moved the Nodes slightly to ensure they fit the screenshot, their positions were as imported (just their size is now different)
Look in Godot 3.5.1:

VS in Godot 4.0 RC 2:

Note that the whole Vector composing/Decomposing is not necessary for the bug to appear. I just forgot to remove that part before uploading my projects. I have tested it without them, and I still see the error.
Steps to reproduce
In Godot 3.5

Create a scene, add a TextureRect, attach a Texture to it (I used a generated Gradient-texture)
Attach a Shader Material to it, then create a Visual Shader from there
Make sure to create a Texture2D-node, attach it's "Color" and "Alpha" output to the final "Color" and "Alpha" input respectively
Save and close the project

In Godot 4.0

Open the project you just created, and let it Import and convert everything
Open the scene you've made
The Error should appear, and the Texture Rect should not show the attached texture. Check the Visual Shader to see the missing connections.

Minimal reproduction project
Here is the Godot 3.5.1 Project. When importing it in Godot 4, and then opening the scene "Demo.tscn", the error can be observed.
VisualShaderImportToG4Bug.zip
In case the above link does not work for some reason, I have created a small public repository with the Godot 3.5 and Godot 4.0 Project ready :)

Hi, I have similar problem and I found that for some reason, the curves which connect nodes are duplicated several times and the tres file of the shader bloats  from 250KB to 120MB in my case.
Sadly I was not able to figure out conditions to replicate. Btw here is the shader, if somebody wants to try it out.
waterDepth_visualShader.zip


SEN-1: Fix missing BP data

add -Depth 3 on ConvertTo-Json to pull in nested BP alert_event data
share API token accross products to reduce 429s on auth
remove extra space chars

@djspencer : we are unable to deploy this azure function after these changes are made. Can you please try to deploy the azure function at your end once?

@djspencer : we are unable to deploy this azure function after these changes are made. Can you please try to deploy the azure function at your end once?

@v-rucdu This should now be fixed. Can you please try again?
Thanks.