echocon sample does not work

Environment
Windows build number: 18362.207
Windows Terminal version (if applicable):

Any other software? VS2019 with Windows 10 SDK 17763

Steps to reproduce
Launch the sample
Expected behavior
it should work
Actual behavior
The call to SetConsoleMode fails with ERROR_INVALID_PARAMETER. If I ignore this error, the sample works as written. No idea if it would still work with an actual VT100 client.
ETA: The ReadFile in PipeListener thread never returns while the child process is happily printing to the console window it was spawned with.
So, I don't really understand how this thing illustrates anything about ConPTY.

hi @DHowett-MSFT, I see you tagged it with "help-wanted", and I would be happy to fix the sample if there was a way.
As far as I can tell, the sample is using the relevant APIs as documented in MSDN and used in various Microsoft-authored articles and code snippets since ConPTY was announced.
However,  things simply do not work (the SetConsoleMode call is a distraction in the sample and can be removed with no consequences to anything it demonstrates).  There is a conhost.exe spawned with the child process, but the output is not redirected to the pipes.
So, I am really at a loss in understanding the correct way of using the ConPTY API.  I tried downloading the Terminal source but gave up on using it to troubleshoot EchoCon,  when I realized Terminal won't run out of Visual Studio but needs to be packaged and has other hoops.
Also, looking at the Terminal source where it creates ConPTYs , it is identical to the EchoCon sample, so I'm back to square one in that respect.
TL:DR; I think there is more here than just a sample issue.

Hey @amoldeshpande,
Thanks for the comprehensive report! I'm actually not able to reproduce this.
I've modified the code to replace the escape character with # only when it receives it on the output pipe
diff --git a/samples/ConPTY/EchoCon/EchoCon/EchoCon.cpp b/samples/ConPTY/EchoCon/EchoCon/EchoCon.cpp
index 428f1e3e..2cc12131 100644
--- a/samples/ConPTY/EchoCon/EchoCon/EchoCon.cpp
+++ b/samples/ConPTY/EchoCon/EchoCon/EchoCon.cpp
@@ -183,6 +183,8 @@ void __cdecl PipeListener(LPVOID pipe)
         // Write received text to the Console
         // Note: Write to the Console using WriteFile(hConsole...), not printf()/puts() to
         // prevent partially-read VT sequences from corrupting output
+       for(int i = 0; i < dwBytesRead; ++i)
+               if(szBuffer[i] == '\x1b') szBuffer[i] = '#';
         WriteFile(hConsole, szBuffer, dwBytesRead, &dwBytesWritten, NULL);

     } while (fRead && dwBytesRead >= 0);

This is what I'm getting (which illustrates that the output from ping is being received on the output pipe):
#[25l#[2J#[m#[H#]0;C:\WINDOWS\SYSTEM32\ping.exe#[?25h
Pinging DHOWETT-DEV [::1] with 32 bytes of data:
Reply#[70X from ::1: time<1ms
Reply from ::1: time<1ms
Reply from ::1: time<1ms
Reply from ::1: time<1ms
#[25X
Ping statistics for ::1:
    Packets: Sent = 4, Received = 4, Lost = 0 (0% loss),
Approximate round trip times in milli-seconds:#[10X
    Minimu#[46Xm = 0ms, Maximum = 0ms, Average = 0ms

Using ENABLE_VIRTUAL_TERMINAL_PROCESSING and passing the output pipe's contents directly to standard out makes the sample look exactly like running ping.exe in conhost, you're right.

hmm, that really makes no sense to me. Are you running 18362 or some internal build ?  I also updated my sdk to 18362 (though that should not matter)
I get error 87 from SetConsoleMode failing and never hit the breakpoint I put in your for loop above.
I see  separate windows for ping.exe and echocon.exe (and in the debugger I can see conhost.exe launched as well)
what else can I do to troubleshoot ?

Hold up, do you have the "Use Legacy Console" option turned on here? I think ENABLE_VIRTUAL_TERMINAL_PROCESSING won't work in the legacy console, and that's why you're seeing ping spawn its own console window.

Yes ! that was it ! I didn't turn it on, so it must default to that mode.
I feel appropriately silly now.  Though in my defense, the blog articles with code snippets do not mention this or have calls to SetConsoleMode :-)

Nah, you're totally right that this is poorly documented. We actually just discovered it ourselves! #1838 :smile:


TP2000-1001  Fix quota definitions list sort by filter

TP2000-1001  Fix quota definitions list sort by filter
Why
The sort by filter on quota definitions list view doesn't take effect, meaning users are unable to sort the list by quota definition sid or start date in ascending or descending order.
What

Adds ordering to queryset
Adds unit test

Codecov Report
Patch coverage: 95.91% and no project coverage change.

Comparison is base (f140fef) 92.46% compared to head (732aca8) 92.47%.
Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1001   +/-   ##
=======================================
  Coverage   92.46%   92.47%
=======================================
  Files         444      444
  Lines       31805    31837   +32
  Branches     2424     2429    +5
=======================================
+ Hits        29410    29441   +31
  Misses       1920     1920
- Partials      475      476    +1

Files Changed
Coverage Δ

quotas/views.py
98.36% <85.71%> (-0.38%)
:arrow_down:

importer/forms.py
91.66% <91.66%> (+0.51%)
:arrow_up:

conftest.py
93.72% <100.00%> (ø)

importer/models.py
83.52% <100.00%> (+0.19%)
:arrow_up:

importer/tests/test_forms.py
100.00% <100.00%> (ø)

importer/tests/test_importer_command.py
96.00% <100.00%> (ø)

importer/tests/test_views.py
100.00% <100.00%> (ø)

importer/views.py
79.09% <100.00%> (ø)

quotas/tests/test_views.py
100.00% <100.00%> (ø)

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


Can't train RTMDet on images with width or height greater than 1024

Describe the bug
Receive this error for any image size above 1024 (e.g. 1025) but none below.
Traceback (most recent call last):
  File "/data/ljh/anaconda3/envs/open-mmlab-new/lib/python3.10/contextlib.py", line 153, in __exit__
    self.gen.throw(typ, value, traceback)
  File "/data/ljh/anaconda3/envs/open-mmlab-new/lib/python3.10/site-packages/mmengine/optim/optimizer/optimizer_wrapper.py", line 351, in optim_context
    yield
  File "/data/ljh/anaconda3/envs/open-mmlab-new/lib/python3.10/site-packages/mmengine/model/base_model/base_model.py", line 114, in train_step
    losses = self._run_forward(data, mode='loss')  # type: ignore
  File "/data/ljh/anaconda3/envs/open-mmlab-new/lib/python3.10/site-packages/mmengine/model/base_model/base_model.py", line 326, in _run_forward
    results = self(**data, mode=mode)
  File "/data/ljh/anaconda3/envs/open-mmlab-new/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/data/ljh/mmdetection3/mmdet/models/detectors/base.py", line 92, in forward
    return self.loss(inputs, data_samples)
  File "/data/ljh/mmdetection3/mmdet/models/detectors/single_stage.py", line 77, in loss
    x = self.extract_feat(batch_inputs)
  File "/data/ljh/mmdetection3/mmdet/models/detectors/single_stage.py", line 148, in extract_feat
    x = self.neck(x)
  File "/data/ljh/anaconda3/envs/open-mmlab-new/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/data/ljh/mmdetection3/mmdet/models/necks/cspnext_pafpn.py", line 153, in forward
    torch.cat([upsample_feat, feat_low], 1))
RuntimeError: Sizes of tensors must match except in dimension 1. Expected size 104 but got size 103 for tensor number 1 in the list.

Looks like all of the model range are designed for 640x640 so I'm not sure what I could change in the neck to allow for larger images.

Hi, ddi you find a workaround for this issue?

Nope, spent a while trying to tweak different parts of the architecture but just hit other problems.
Will continue to try whenever I have a bit of spare time.

Think I'll try to use it with SAHI, not sure where that will leave me inference speed wise.

Bump

Excuse me. Have you solved the problem yet?

No I've not, tried a variety of sizes for different parts of the model. Couldn't work out the bottleneck, very curious to find out the resolution.

Hi @pd162 that's not much of a fix for me, because I definitely do want to keep the same ratio (and all of the transforms I'm trying would do so anyway).

@GeorgePearse did you find the solution? Encountering the same issue now


use docker healthcheck

This PR adds support for healthcheck in the docker-compose file.
ATM, your repo's tester has a built-in wait for health check in tests/es_acceptance.py in function wait_for_cluster_health. Using docker's healthcheck, other services can benefit from this without having to re-code a wait for health check function in each. On the down-side, this requires upgrading the yml version from 2.0 to 2.1, and hence upgrading docker-compose and docker to versions supporting it

docker info output:  (1.13.1)   http://paste.ubuntu.com/24039459/
docker-compose version: docker-compose version 1.11.1, build 7c5d5e4

Maybe creating a separate branch for this would be convenient?
Note that I ran make for testing and it failed with a connection failed error, but then again I ran make on master, and it also failed.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Thanks for your interest and the PR!
The trouble with this approach is that curl requests will fail as x-pack (inc. security) is included in the Elasticsearch image and thus those API calls require authentication.
So for that to work we'd need to tweak the default x-pack configuration and allow anonymous access specifically to a chosen health check endpoint (e.g. _cat/health).
Additionally, for every new feature added, we need acceptance tests, so we'll need to add python tests to ensure this is working as expected.
Incidentally, I just tried make on both the master and 5.2 branches and all tests pass.

You're right about the authorization issue with xpack. I did not face it because I was just overriding the configuration with one that doesn't use xpack. What do you suggest as next steps for this PR?


Passing the url parameters from layout to the components

I was wondering, if there is any way how to pass the parameters from the layout to the components. Let’s say I have a layout with table and some charts and I would like to directly include the filtering in the parameters of this layout. For example http://localhost/my_layout?data=something resulting in passing these to all the components, serving like a filter.
Many thanks,
Martin

It's possible, it's actually something I've been trying to solve in a more elegant way.  In the next month or so I'm going to try to release a bunch of things I've been working on for a different project, I'll try to include it.

Thanks for the answer, looking forward to look at it later!
( I'm not familiar much with React, so in the meantime I gave up and created the dashboard only using Flask. )


[Feature Request] Compact View

Thank you so much for creating such a gorgeous app!
Could you consider a compact view, that drops the images? Would be nice to be able to view more links at once.

Yes, I had thought about placing a small photo on the left and the title on the right, and maybe a menu button on the right for all the options.

I would like to request this as well.
A simple "list view" is much better for me.
When you scroll through hundreds of bookmarks it makes a difference.

Thank you so much for creating such a gorgeous app! Could you consider a compact view, that drops the images? Would be nice to be able to view more links at once.

I've created a compat view

Any suggestion is welcome.


[<IP_ADDRESS>] timeouts after dropping measurement

trying to test new continuous queries, realize I made a mistake and try to drop the query, then the measurement it inserted into causes problems. soon as I issue the drop on the measurement, the logs fill up with:
"failed to write point batch to database"
and the server becomes unresponsive until I restart (and the measurement is never dropped). I can still run commands like "show measurements" but anything that queries data never returns (like it locks the entire db).
behavior is the same in 0.9.5_nightly_b079d20-1

@parisholley can you provide explicit steps to reproduce, or a copy/paste of your curl or CLI session that leads to the issue?

@beckettsean doesn't seem to be an issue on nightly using the "bz" engine

@parisholley thanks for the update!


stevedore-np: Add version 0.8.0

Adds stevedore to the bucket.
Pretty much a straight, updated copy of TheRandomLabs/scoop-nonportable#296 (done with the original author's permission).

[x] I have read the Contributing Guide.

/verify


Fix v0.1.35 broken build

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Currently only one platform is available on anaconda.org for this version: https://anaconda.org/conda-forge/deno-dom/files?version=0.1.35
Failing builds are all except osx_arm64: https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=668906&view=logs&j=58ac6aab-c4bc-5de2-2894-98e408cc8ec9&t=933f325c-924e-533d-4d95-e93b5843ce8b
closes #10

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

On second thought, should we instead patch to set a pin? If so, any idea what version to pin to?

merging and punting on this for now. I'm sure it'll bite us later, but let's get this working right now.


last_called is too slow?

Hi
I'm trying to make of the last_called attribute. Ie things like "Alexa Turn On Lights" / "Alexa Turn On TV" and then based on the echo that was used turn on the lights for that room.
This https://github.com/keatontaylor/custom_components/pull/65 suggest that @brianhanifin got it working. But in my test there a good 10 seconds delay until the correct echo is reported back correctly.
Is there any way to force a refresh of the last_called attribute?
Cheers

Not at this time. We will be exposing a service in the future as the current polling method results in too much polling of Alexa servers.

Yes that would be a much better option

FYI, the Alexa App already allows you to do location specific control of at least lights. You'll need to use HA's emulated Hue and ensure within the AlexaApp it treats that switch as a light. I think you could also do the TV too, but I haven't tested it.

Really, how? I've not seen this option before, do you mean via a routine?
There only the option "Alexa will respond from" which give device specific controls

https://www.amazon.com/gp/help/customer/display.html?nodeId=201749250.  Might be region specific.

Alexa enabled groups. You have to  make a group in the Alexa app and add an Alexa device.
For example;
I made a group with all the lights downstairs and included the living room Echo. Same thing for all the upstairs light and the bedroom echo.
If I ask Alexa to turn off/on the lights, when I'm downstairs, she will only switch the downstairs lights. And same for upstairs.
If you add the switch for TV as a emulated Hue, you can include that in the group.

Nice that works thank for help

@alandtse I'd like to use the last_called sensor for the following case. Unfortunately as the topic says currently it's too slow.
I like to turn on an input boolean with a routine in alexa by my voice. This than will trigger an automation which sends a tts to the last_called alexa. This way it must be fast or it will send the tts to wrong alexa.
Will this be possible with the service you are planning to implement in future. Will the service be part of the component we already use?

This branch has an exposed service for the adventurous who want to test it.
alexa_media.update_last_called service can take no args or can specify specific accounts to find the last_called for.
{
  "email" : ["email1", "email2"]
}


Version check logic in dev/bots/test.dart is incorrect

This does not account for hotfix branches.
https://github.com/flutter/flutter/blob/master/dev/bots/test.dart#L676

I'm marking this as a PM blocker since we're going to want this fixed before the next beta release.

While we're at it we should change the - to a + in the hotfix process (and associated code).

We've landed the fix to the tool to correctly describe the semvar of hotfix branches. We still need to update version check test regexp. That should be fairly easy and I will take a look this week.
Separately, we will need to re-test the hotfix process.

This is done


Need  element documenting/in demo

I went searching for the select element in the demo and via the nav sidebar of the Predix-UI site and couldn't find the <select> element anywhere. We should probably add that.

I've added several keywords (input, select, multiselect, textarea, etc) to the search functionality on the Predix-UI site. Do we really want to highlight the select element in forms-design, or should we encourage them to use px-dropdown instead?

Added to demo in v0.3.28.

👍


getPrepperForImage: strip prefixes prior to reference validation

Strip the daemon:// and remote:// prefix prior to validating the
reference.  Otherwise, the validation will fail unconditionally.
Fixes: #183
Signed-off-by: Valentin Rothberg<EMAIL_ADDRESS>

@dlorenc Awesome, I'm closing this one then.


Pin DMs plugin not working[Bug]

I can pin/unpin DMs anymore the only way to unpin is to stop the plugin
Discord Version:Stable 114407

File a report on the repository for the plugin. This is not the place to report issues with plugins.


Fullscreen video crashes hyprland

Since this is on NixOS, I'll tag @fufexan as well.
I am using echo360 for watching my university lectures. Whenever I fullscreen the video, after a while (say 5 mins) the screen goes black. Moving the mouse brings it back, but I am unable to do anything to that video anymore.
The keyboard inputs work for pausing/playing the video, but pressing esc does not do anything. It feels like a single screen (like cage) and does not accept mouse input or anything. Switching to another tty is possible, but that's it. There's no way to get back into the hyprland session except issuing a restart command from another tty.
This has occurred every single time, as I can reproduce it. It does not happen with youtube videos tho (at least I haven't experienced that. I think that it disconnects the monitor when the screen goes black, but I am not sure.
I have attached the log from my last session after restart.
a.txt

I am experiencing a similar issue with my T480 laptop system Arch Linux kernel 6.4.1-arch2-1, which has Intel UHD Graphics 620. Whenever I attempt to play a video on Firefox, Chromium or VLC , my system freezes within five minutes, rendering it unresponsive. Strangely, this problem does not occur when I watch videos using mpv or when I stream DRM-protected content on platforms like Disney+ or Netflix.

Same for me. Running NixOS 23.05 on a Zephyrus G14 laptop with an AMD GPU. After a few minutes of watching YouTube on fullscreen, the screen will freeze, flicker to a black screen or between two frames with audio still playing/glitching. No keyboard or mouse movement works including when trying to switch TTY. This didn't happen before in NixOS 22.11.

Does not happen to me, so please bisect.

Also, does this happen with Hyprland from Nixpkgs or from the flake?

Also, does this happen with Hyprland from Nixpkgs or from the flake?

I am using the flake; don't know about the nixpkgs. I guess the specific NixOS version doesn't matter then, I'm not too sure.

I'm on unstable. From what I've seen, mismatched mesa versions could cause issues. But I've taken care of that, so now whenever there's a new mesa version in nixpkgs, Hyprland's nixpkgs input is updated.

Same issue when i watch any video on youtube on full screen, sometimes when i skip or rewind the video for few seconds it gets stuck on the same position, although the video slider is moving and voice is also coming fine just the display gets stuck, rest everything is functioning fine.
I am using hyprland on fedora, any help will be appreciated


GetXRButton

Allows you to pick a controller, button, and button action (buttonup, buttondown) to trigger the next action with the choice of saving the result to an FSM Variable. Utilizes the existing unity events in the package. I was used to this functionality through my last package, wanted to replicate it.

This is my first real GIT code merge. Hope you dig it.

Looks good. Thanks for the commit. My first merge from someone I didnt meet before :)

Nice!  I just got vibration working on a test script, I may try to send you something for that if you don't have anything yet.
Chris
Sent from Yahoo Mail on Android
On Sat, Jul 18, 2020 at 11:16 AM, Eric Vander<EMAIL_ADDRESS>wrote:
Looks good. Thanks for the commit. My first merge from someone I didnt meet before :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


mdns error when connect device

https://github.com/project-chip/connectedhomeip/blob/2dd4fdb065720a1faba65ed6d0fdbc22f953bcf6/src/lib/dnssd/minimal_mdns/Server.cpp#L344
>>> onoff toggle 0x12344321 1
[1687870707.009955][1160548:1160548] CHIP:TOO: Command: onoff toggle 0x12344321 1
[1687870707.010566][1160548:1160550] CHIP:TOO: Sending command to node 0x12344321
[1687870707.010880][1160548:1160550] CHIP:CSM: FindOrEstablishSession: PeerId = [1:0000000012344321]
[1687870707.010907][1160548:1160550] CHIP:CSM: FindOrEstablishSession: No existing OperationalSessionSetup instance found
[1687870707.010965][1160548:1160550] CHIP:DIS: OperationalSessionSetup[1:0000000012344321]: State change 1 --> 2
[1687870707.011042][1160548:1160550] CHIP:DIS: OperationalSessionSetup[1:0000000012344321]: State change 2 --> 1
[1687870707.011071][1160548:1160550] CHIP:-: src/lib/dnssd/minimal_mdns/Server.cpp:344: CHIP Error 0x00000046: No endpoint was available to send the message at ../commands/clusters/ModelCommand.cpp:62
[1687870707.011094][1160548:1160550] CHIP:TOO: Run command failure: src/lib/dnssd/minimal_mdns/Server.cpp:344: CHIP Error 0x00000046: No endpoint was available to send the message

step:

close all wifi connections.
enter chip-tool's interactive mode.
run onoff toggle 0x12344321 1.
error is "No endpoint was available to send the message, this is in expections.
let wifi connect to AP.
run onoff toggle 0x12344321 1 again
error is "No endpoint was available to send the message. I think this is not in expections.

in some cases, for example, mdns initialization completes before wifi connection, it unable to return to normal status.

@nykh2010 Are you on Linux or Mac?  It sounds like the platform is not dispatching relevant "network changed" notifications...

@bzbarsky-apple yes, platform is Linux.

Expectation for platform implementations is that minmdns would be restarted on networking changes.
Since notifications of networking changes is platform-specific, this probably has to be handled by the app on a platform by platform basis. I had a PR open for using a single listening socket which would have probably covered this as ANY would keep updating, however it ended up not playing well with LWIP (#27219).
I am unlikely to have time to add platform-specific dnssd restart logic to chip-tool or example apps for minmdns, however I would be happy to review patches for this.


Add pauseAudio() method

Adds a playAudio() method to the plugin. The stopAudio() does not
maintain the position when called, the new pauseAudio() method maintains
position and allows resuming.
Related issues
https://github.com/nchutchind/cordova-plugin-streaming-media/issues/59

Looks like this adds the behaviour for iOS only?

Yep, sorry...I couldn't find the code in the Android version for the stopAudio() method...That would have given me a starting point to implement it there.
Also, in my branch, I have a resumeAudio() method coded and working.  I did all that so the iOS audio controls would work (with the help of the RemoteControls plugin).
If you point me in the right direction, I can implement the Android version of this.  I'll totally understand if you reject this PR.

I'm not going to reject it, just verifying my assumption.
Could you please (if you haven't already) update the README describing the new functionality, and that it is available for iOS only?


Auto validate/format on change

Thanks for the great tool!
i wonder how many people actually type their json strings to validate them and how many paste strings from some other source either to validate them or properly indent them ? For the latter, clicking on validate button every time becomes repetitive pretty quickly...
Autovalidation would be really welcome! Or at least a keyboard shortcut
Thanks in advance

@Starojitski I think this will cause UX problems for some users. The best way to get rid of clicks is to use Ctrl+Enter. What do you think?
@circlecell do we need to add another URL parameter called "autovalidate"? Eg
jsonlint.com/?autovalidate=yes

After thinking about it for a while, the fastest is to have one big button at the top of the screen: 'Validate and Format clipboard'.
As the name suggests, clicking on that button pastes clipboard contents into the text area, triggers validation and immediately copies and pastes the text area content into clipboard... Just one click!
And if this button has autofocus attribute it would make it just one one Enter or Space keypress for keyboard us

My apologies, i pressed the wrong button and by mistake closed issue
I was thinking about it and i thought the fastest way would be to have one button called 'Format Clipboard' that would do everything: pasting clipboard into the text area, validating json and, if it's valid, copying formatted version into clipboard. If that button is largish and near the top center of the screen it would be easy to click. Setting autofocus on that button make it ideal for keyboard use - just one Enter or Space press!
Unfortunately, while writing to clipboard is possible in all modern browsers, reading from it seems impossible. Correct me if i am wrong

@Starojitski thanks for your ideas. The main goal for us is to keep jsonlint.com as simple as possible. That's why it's used by millions of people from all over the world. I don't think that adding extra buttons is good idea.
You can use shortcuts for your goal:

Ctrl + V to paste your code
Ctrl + Enter to run validation
Ctrl + A to select formatted code
Ctrl + C to copy it to your clipboard


Add multiple Todoist tasks from highlighted checkbox list

Is your feature request related to a problem? Please describe.
I really like how the Add Todoist task command uses any highlighted text as the title of the Todoist task. However, if I highlight multiple lines in a checkbox list (where each line represents a single task) and then use the Add Todoist task command, only one Todoist task is created. The text from all highlighted lines (and the checkboxes themselves) are concatenated to form a single title.
Describe the solution you'd like

Highlight multiple lines of text in Obsidian, where each line might begin with a checkbox, - [ ]
Use the Add Todoist task command
Multiple new Todoist tasks are created, where the title of each new Todoist task corresponds to a single highlighted line. For example, if 3 lines of text were highlighted, 3 separate Todoist tasks would be created.
The Markdown checkbox syntax - [ ] should be excluded from the title of each task.

Describe alternatives you've considered
I currently workaround this by highlighting one line at a time, and calling the Add Todoist task command multiple times. However, this pretty inconvenient and tedious.
Additional context
Thanks for developing this awesome plugin!

+1 to this. Came over here to propose the same thing. I like how quickly I can create Todos in Obsidian, I would love to be able to highlight a list of them and bulk add them with the same project, due date, and priority


Add message if data source file is not found

Task linked: QF-1852 Show more useful feedback if layer source file cannot be found


added login error box

the original ticket, but i set up everything from scratch because of a git workflow issue i had ( now fixed)
now i should be based on the latest master as well

Next time when you open a new PR, doesn't matter if it's connected to another PR:

Commit like this format: https://www.conventionalcommits.org/en/v1.0.0/
Code modification check list before pushing: https://github.com/PythonFreeCourse/lms#code-modification-check-list

Awesome job, @lenagrech!


[PyOpenSci] Basic Usage notebook needs cleanup / better flow

Generic Issue

xclim version: 0.40.0
Python version: Any
Operating System: All

Description
One of the suggestions coming from the PyOpenSci review process was to clean up the Basic Usage notebook to make it a bit more consistent with the rest of the documentation (https://github.com/pyOpenSci/software-submission/issues/73). The flow of the text needs a bit of work.
For context, this issue is not blocking the PyOpenSci review process, but would be nice to have.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@NickleDave
Changes that were just merged to the main branch (#1308) included modifications to the usage page (one of your initial comments). If you're interested in taking another look at that, please let me know. Otherwise, I'll consider this issue closed in the next few days.
Thanks again!

Hi @Zeitsperre, I took a quick look and see in the PR where revisions were made in places where there were minor typos / unclear language. LGTM!!! Thank you for noting this and integrating it into the review.

Awesome! All the best!


Add SRT Ingest URL to Dashboard

We now have SRT ingest! We should include the SRT ingest URL on the dashboard.
SRT URLs look like this:
srt://rtmp.livepeer.com:2935?streamid={stream-key}
The stream key field should be auto-populated with the stream key. The stream key should be hidden on the dashboard view, as we do with the playback URL. However, when the copy button is clicked the full URL is used.
Like this:

Tooltip should read:

SRT ingest is currently in beta. Ensure that your encoder and player is compatible with SRT.

How to get this port number? @victorges @pglowacky

@FullStackTiger Port number should stay the same for all ingest URLs, no matter what the stream key is.


Mix:watch webpack config not being cleaned - Windows OS

Winter CMS Build
dev-develop
PHP Version
8.1
Database engine
MySQL/MariaDB
Plugins installed
No response
Issue description
When running the command php artisan mix:watch on Windows OS native terminal, the halt signal is not well handle and the mix.webpack.js is not automatically cleaned up.
The getSubscribedSignals method of the Winter's trait is never fired on Windows.
My investigations :
In the command process execution the Symfony console in his constructor set a signalsToDispatchEvent private variable if the SignalRegistry::isSupported() test is successful.
Application.php#L99-L102
The SignaRegistry::isSupported() method perform a test for PHP pcntl_signal support.
SignalRegistry.php#L40-L43
The PHP PCNTL documentation says :

Currently, this module will not function on non-Unix platforms (Windows).

So ... No way to get halting signals on Windows OS while running Symfony console v6.2.xxx
In the next Synfony console version (6.3.xxx) a new architecture was introduced to better handle exit signals from command that works with Winter's commands.
But for now the 6.3.x Symfony console can't be installed due to Winter.Storm requirements :
"symfony/console": ">=6.0.9 <6.3.0",
Steps to replicate
Run a php artisan mix:watch ... command and press CONTROL+C to escape the run.
Workaround
Use WSL on Windows OS .

We'll have to address this in 1.3 as Symofny/Console 6.3 doesn't support PHP 8.0 so we can't switch to it yet on 1.2: https://github.com/wintercms/storm/commit/facb27d7486406306dab491ed3d9eb95245fd0b2


Questions about using pprof to obtain information related to k3s

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

I am exploring the performance of the k3s cluster, and I use the pprof tool for data collection. My work steps are as follows:
Start the proxy:
kubectl proxy --address='<IP_ADDRESS>' --accept-hosts='^*$'
Obtain performance metrics (I used the following two commands and got almost identical results):
go tool pprof http://<IP_ADDRESS>:8001/debug/pprof/heap
go tool pprof http://<IP_ADDRESS>:8001/api/v1/nodes/${NODENAME}/proxy/debug/pprof/heap

Describe the solution you'd like

I use the top command to obtain the memory usage of the entire k3s cluster, and I want to further analyze the memory usage of the internal components of k3s.

I would like to know the answers to the following questions:

Why do I get similar memory usage information through two different methods? If I want to analyze the performance of kubelet specifically, what is the correct approach?
If possible, could you please tell me the correct method to obtain pprof data for each component in k3s?
The above data was obtained on the server node; how can I obtain data on the agent nodes

Describe alternatives you've considered

Additional context

k3s version:
k3s version 1.28.2+k3s1
go version
go1.21.9
Node(s) CPU architecture, OS, and Version:
Linux 6.8.0-40-generic #40~22.04.3-Ubuntu SMP PREEMPT_DYNAMIC Tue Jul 30 17:30:19 UTC 2 x86_64 x86_64 x86_64 GNU/Linux

You can't get info for separate components, because k3s runs all the components - including the kubelet - in a single process. This is the entire value proposition of k3s and is covered in the docs.

You can't get individual info for separate components, because k3s runs all the Kubernetes components - including the kubelet - in a single process. This is the entire value proposition of k3s and is covered in the docs.
You can get it from agent in the same way you do on the server - by enabling pprof on the command line and hitting the pprof debug endpoint on the supervisor port.

Thank you very much for your reply. I understand that I cannot directly obtain the pprof data for each component, but theoretically, the pprof data I obtained in the manner described should reflect the overall memory usage of k3s. However, the actual values are significantly lower than those reported by the 'top' command. Could you please explain what the data I access at /debug/pprof represents?

The standard golang profiling data: https://pkg.go.dev/net/http/pprof


need help

hey mate i dont understand the docker part could you help me or be more specific?

Error response from daemon: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error mounting "/run/desktop/mnt/host/c/Users/hvnull/Desktop/server/ServerStatusBot/ServerStatusBot/appsettings.Production.json" to rootfs at "/app/appsettings.json": mount /run/desktop/mnt/host/c/Users/hvnull/Desktop/server/ServerStatusBot/ServerStatusBot/appsettings.Production.json:/app/appsettings.json (via /proc/self/fd/14), flags: 0x5000: not a directory: unknown: Are you trying to mount a directory onto a file (or vice-versa)? Check if the specified host path exists and is the expected type

Hi, inside the bot folder have you created the appsettings.Production.json?

Closing as there is no response

sorry for not answering. is much to ask if you could make a quick video of the steps i keep getting errors.

Hi, as it's pretty simple to set up I am not not planning on making a YouTube video. Please if you need help answer the question I asked you.


fix typo in howto

fix typo

correct is ingress-nginx  https://github.com/kubernetes-incubator/kubespray/blob/master/roles/kubernetes-apps/ingress_controller/ingress_nginx/templates/ingress-nginx-controller-ds.yml.j2#L8
label name is wrong there is no app=ingress-nginx but k8s-app=ingress-nginx


ci: speedup upgrade integration test

Fixes no issue
Description

randomizes the pod names to avoid having to delete pods between tests on master and test branch

Checklist

[x] PR is rebased to/aimed at branch develop
[x] PR follows Contributing Guide
[x] Added tests (if necessary)
[x] Extended README/Documentation (if necessary)
[x] Adjusted versions of image and Helm chart in values.yaml and Chart.yaml (if necessary)

Codecov Report

Merging #405 (126b887) into develop (b91d745) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #405   +/-   ##
========================================
  Coverage    96.99%   96.99%
========================================
  Files           21       21
  Lines         1064     1064
========================================
  Hits          1032     1032
  Misses          32       32

Continue to review full report at Codecov.

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


python3Packages.qiskit: 0.23.5 -> 0.24.0

Motivation for this change
https://qiskit.org/documentation/release_notes.html#qiskit-0-24-0
Things done

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

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 115211 run on x86_64-linux 1

21 packages built:

python37Packages.qiskit
python37Packages.qiskit-aer
python37Packages.qiskit-aqua
python37Packages.qiskit-ibmq-provider
python37Packages.qiskit-ignis
python37Packages.qiskit-terra
python37Packages.retworkx
python38Packages.qiskit
python38Packages.qiskit-aer
python38Packages.qiskit-aqua
python38Packages.qiskit-ibmq-provider
python38Packages.qiskit-ignis
python38Packages.qiskit-terra
python38Packages.retworkx
python39Packages.qiskit
python39Packages.qiskit-aer
python39Packages.qiskit-aqua
python39Packages.qiskit-ibmq-provider
python39Packages.qiskit-ignis
python39Packages.qiskit-terra
python39Packages.retworkx


BUG: Dynamic import should return different promise objects, even if the specifier is the same.

Test: https://github.com/tc39/test262/blob/master/test/language/expressions/dynamic-import/always-create-new-promise.js
Result:

the returned promises are not the same, regardless the reference and specifier pair Expected SameValue(«[object Promise]», «[object Promise]») to be false

Hmm currently internally the promise for a dynamic import is a property of the SourceTextModuleRecord object - if you import the same module a second time the SourceTextModuleRecord object from the first import() is retrieved and the promise from it is returned.
This means that when the import() completes it has an easy reference to the promise it needs to resolve, so to fix this would probably have to change this mechanism - perhaps introducing a list of promises.

It might not be necessary to keep a list; CC could keep the single internal promise and wire up the ones returned from import to it.  This way you resolve the internal one and all the ones attached to it get resolved automatically.

I've submitted a fix, this commit https://github.com/Microsoft/ChakraCore/pull/5803/commits/866386bbf9fb7d5f4a14720ba9da16ed5c9083a2 in PR 5803.
It adds the new promises as reactions to the existing promise with an internal equivalent of promiseObject.then(a => a, e => throw e);


Questions about BC Version and Docker Tags Images

HI,
I created a Docker on a an Azure VM on 26th of October issued from microsoft/bcsandbox:base and version installed is exactly "13.0.24623.24800"
Questions (may be dummy but...) :

I was not able to find this exact release on https://hub.docker.com/r/microsoft/bcsandbox/tags : any reason why ?
how can I be sure to provision the same docker locally, even after next/spring release...? (tried to pull microsoft/bcsandbox:13.0.24623.24800 but not working for me)
bonus question : what is the meaning of ".24800" exactly in the version ?

Thanks,
Xavier.

Hi Freddy,
I know that you were quite busy with all the builds to do but can I get answers as I'm still confused about BC version... :-(
Another point : I can create a docker for microsoft/bcsandbox:13.0.24630.24844-w1-ltsc2019 but not with microsoft/bcsandbox:13.0.24630.24844-w1-ltsc2019:base ? (Error DockerDo : invalid reference format).
Does-it mean that flag :base is obsolete ?
Thanks, Xavier.

base is instead of the language - meaning that you replace w1 with base in the tag - not append another tag.
The list of tags on docker hub only shows the latest x tags created - and I might have updated dynamics-nav images since.
Have you read this: https://blogs.msdn.microsoft.com/freddyk/2018/04/16/which-docker-image-is-the-right-for-you/
I think that answers your questions - let me know if not.

So base flag is only valable for w1 version. Assuming that country versions are shipped with Cronus data.
can you explain how to retrieve the image from version 13.0.24623.24800 and/or explain what are the meaning of 24623 and 24800 ('cause we can't simply use only the "last" version)

And yes I read almost (all) your articles ;-)
Thanks again and congrat for this amazing work.

The numbers you are refering to is a the number you find if you in NAV20XX type HELP, About. Or in Business Central type System Information. For instance if you see the number .24623 it is the RTM release of BC.
For NAV 2018 you can see this list:
https://support.microsoft.com/en-us/help/4072483/released-cumulative-updates-for-microsoft-dynamics-nav-2018
So if you want build 23572 - then you can see it is update 8
You can then do a docker pull microsoft/dynamics-nav:2018-cu8-XX where you can replace the XX with one of the following: AT, AU, BE, CH, CZ, DE, DK, ES, FI, FR, IS, IT, NA, NL, NO, NZ, RU, SE, UK

I think BC RTM was 24630 (at least that is that I get).
I am writing up a blog post to de-mystiphy version numbers as we speak. Will be out today.

ahh there is actually a little bit of confusion with the RTM version.
The Development Client (Classic) says 24623 and System Information says 24630

Exactly, I have the same problem:

Then I am not able to get the image for this version (based on build number of NST).

What you are looking at there is the platform version, which differs from the build version.
Traditionally these have been the same, with Business Central this changes.
We can have multiple releases with the same platform, that we want to give different build numbers.
I hope my blog post will shed light over this.

Ufff, a lot of things have changed, lot of broken stuff on our side.
@freddydk, please, don't understand me wrong, but the effort to automate looks a bit pointless when I consider all changes coming down with the BC images.

New endpoint (ACR instead of Docker Hub).
Missing HTTP API (I know that HTTP API of Azure Container Registry has nothing to do with D365BC images but it's just next complication - Docker Hub was perfect, it allowed to use Docker Image Mirrors etc.; why do we change something that works perfectly fine and looks like a nice standard including features not available today in ACR???).
New naming conventions (I am fine with it, the product name has changed so OK then).
Build numbering has change (a new info for me and now I can understand why something that worked doesn't work anymore - and again, the bad thing is we can't see the list of images via HTTP API).

I know the world changes but I can see too many new changes that make it harder to get further (you want to solve one problem and will find next two new issues).
I hope you understand my point ;)

I understand, but none of these things have anything to do with NAV/BC on Docker.
The version numbering is driven by the build team. The reason for this is that we need more frequent updates of the app without changing the platform for online.
The move to mcr was driven by Microsoft - I simply wasn't allowed to create another repo on docker hub by the time we released Business Central.
As far as I know, the mcr guys are working on an API, having said that - I still don't understand why you would need to query the docker hub and search for releases.
In the end - nothing we can do about it, I hope the upcoming blog post will shed some light on the version numbering and that you find a way.
Remember you are always free to shoot me an email or file an issue here if you find things that looks strange - then you maybe can avoid redoing too many things.

@Koubek  in short your get-item in powershell from now on is usesless. You have to go into your client and type System Information if you really need that information.
I think it is only a matter of 3-9 months then the numbers are "useless".  For instance my BC has a AL.vsix file with version 0.15.18771 - and I have no idea of what version that is linked to into BC. So unless MS makes a sheet of links with what version that works with what - then all the versioning knowledge is useless for most people.
if you pull the daily builds from insiders...then you will also se the numbers changes daily.
...And the versioning is totally out of @freddydks hands. It is in the dev-teams hands.
Versioning ends up being a matter is "how old is my installation" :) Would be nice though to get an update list for BC, VSIX files etc.

OK, thanks for the feedback. As I told before, don't understand me wrong. I don't blame @freddydk at all. He does a great job for us. I would say he really tries hard to support us (community) and makes our life easier.
I am waiting for a blog post about the build numbers... ;)
And btw, why for example do we need API? Well, one of the things we do is we build a Git repo to see changes between all NAV versions, countries etc. We do it automatically and it's easy to do it up to NAV 2018. We use images and just using naming patterns we know if there is a new release or not. This isn't possible now for BC. As we trying to move all C/SIDE code to BC and convert then switch to AL, we of course still need C/SIDE code base. This repository makes it easier but now we need to update it manually for BC as we can't identify easily whether there is a new release (CU) or not.

Thanks,
For that - wouldn't you just go for:
businesscentral/onprem:cu1
and later cu2, cu3 etc.
Blog post in review right now.

Yes, this is, in fact, the only option I have now. But I need to change the implementation. I was waiting for changes here: https://github.com/AzureCR/specs/issues/3

https://blogs.msdn.microsoft.com/freddyk/2018/12/05/business-central-build-numbers/


Tests fail

Hi,
I'm currently in the process of adding the package to conda-forge and would have two questions regarding the test setup. I'm running the full test suite under .github/workflows/run_tests.sh but am getting the following error:
OSX:
test_dhf (test_df_jk.KnownValues) ... .github/workflows/run_tests.sh: line 39: 21683 Segmentation fault: 11  nosetests pyscf/ -v --with-timer --with-cov --cov-report xml --cov-report term --cov-config .coveragerc --cov pyscf --exclude-dir=examples --exclude-dir=pyscf/pbc/grad -e test_bz -e h2o_vdz -e test_mc2step_4o4e -e test_ks_noimport -e test_jk_hermi0 -e test_j_kpts -e test_k_kpts -e high_cost -e skip -e call_in_background -e libxc_cam_beta_bug -e test_finite_diff_rks_eph -e test_finite_diff_uks_eph -e test_pipek -I test_kuccsd_supercell_vs_kpts\.py -I test_kccsd_ghf\.py -I test_h_.*\.py -I test_P_uadc_ea.py -I test_P_uadc_ip.py --exclude-test=pyscf/pbc/gw/test/test_kgw_slow_supercell.DiamondTestSupercell3 --exclude-test=pyscf/pbc/gw/test/test_kgw_slow_supercell.DiamondKSTestSupercell3 --exclude-test=pyscf/pbc/gw/test/test_kgw_slow.DiamondTestSupercell3 --exclude-test=pyscf/pbc/gw/test/test_kgw_slow.DiamondKSTestSupercell3 --exclude-test=pyscf/pbc/tdscf/test/test_krhf_slow_supercell.DiamondTestSupercell3 --exclude-test=pyscf/pbc/tdscf/test/test_kproxy_hf.DiamondTestSupercell3 --exclude-test=pyscf/pbc/tdscf/test/test_kproxy_ks.DiamondTestSupercell3 --exclude-test=pyscf/pbc/tdscf/test/test_kproxy_supercell_hf.DiamondTestSupercell3 --exclude-test=pyscf/pbc/tdscf/test/test_kproxy_supercell_ks.DiamondTestSupercell3 -I .*_slow.*py -I .*_kproxy_.*py -I test_proxy.py

Link: https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=411933&view=logs&j=e35d4f76-8ff2-5536-d795-df91e63eb9f7&t=fa7b4b17-b6ff-5c9c-8cfc-15f888c92310&l=7426
Linux:
AttributeError: /home/conda/staged-recipes/build_artifacts/pyscf_1636984882810/test_tmp/pyscf/lib/libcc.so: undefined symbol: AO2MOnr_e2_drv

Link: https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=411933&view=logs&j=6f142865-96c3-535c-b7ea-873d86b887bd&t=22b0682d-ab9e-55d7-9c79-49f3c3ba4823
Does anyone here know where this could come from?
Thanks!

This is also observed in github actions. macos sometimes raises segfault error for a test in pbc.df module. But I couldn't reproduce the error on macos desktop

@sunqm Thanks for the info. I actually sort of figured that this just happens randomly which is why I closed this and opened #1118 for Windows only (Still working on that by the way). If you need more info or want me to run some tests, let me know.


Incorrect path for for files in  cif2016_augmented_experiments.sh

Hi Hansika,
There is incorrect file location in the cif2016_augmented_experiments.sh

instead it should be  --binary_train_file datasets/binary_data/CIF_2016/augmented_data/stl_12i15.tfrecords

Hi Shankar,
Thanks for pointing this out. cif2016_augmented_experiments.sh file is an execution file used previously for certain testing. This is not used anymore. I will remove it from the repository.


updated getting started guide with corrected VM name in vagrant syntax

Step 10 in the Getting Started guide references the vagrant VM by the name cilium-master. Due to the current state of the Vagrantfile, the VM name should be cilium-n, where n is the number of nodes specified to deploy.
I updated the document so that the Vagrant syntax references cilium-1 instead.

Thanks @TrevorRobertsJr!


Add code coverage workflow.

🐛 Category
🛝 GitHub Actions
🔢 Version
🐣 Pre-Release
🤔 What's hapening?
I'd like the project to include coverage information provided by Codecov.
📝 What was your expectation?
No response

Code coverage insight is offered by Codacy. We may revisit this need if we determine that Codecov offers needed benefits.


[Feature Request] CPU affinity

Is your feature request related to a problem? Please describe.
When you use many servers on the same node, you may prefer set the cpu affinity per server.
Describe the solution you'd like
Make new field in Model > General tab where you can specify the cpu single/range affinity based on host CPU

This one isn't going to really happen by default. This is where we will defer to Docker and it's usages for limiting CPU. This is supported, albeit it through manually editing the JSON.

It could be even ok by having this feature available for manual editing. Can you please tell me where and how?
I saw the server json in /var/lib/pufferpanel/servers/xxxxxx.json, and I was using taskset in linux to set the cpu affinity on some scripts.

As for docker installation, I'm prefering this panel over others mostly because docker installation is an optional, I want avoid a level of containerization for my games, unfortunatelly some games like CS2 require less bottleneck possible.


upstream/fix/logpath fix/logpath ec5d458be  SoftHardTagBranchMore fix: normalize logpath to avoid core dump during handler registration

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.笃敏 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[SPARK-20283][SQL] Add preOptimizationBatches

What changes were proposed in this pull request?
We currently have postHocOptimizationBatches, but not preOptimizationBatches. This patch adds preOptimizationBatches so the optimizer debugging extensions are symmetric.
How was this patch tested?
N/A

Test build #75671 has started for PR 17595 at commit 263dc68.

LGTM - pending jenkins.

Test build #75671 has finished for PR 17595 at commit 263dc68.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75671/
Test FAILed.

Merging this since as long as it compiles the change should be fine.

@rxin @srowen @dongjoon-hyun there is no way to plugin a preOptimizationBatches to a spark session. Do we plan to expose it.?


Updates to interactive telemetry

Fixes: https://github.com/Azure/azure-cli/issues/5655
Closes: https://github.com/Azure/azure-cli/issues/5631
-brings interactive telemetry to same subscription as rest of cli
-interactive sends cli telemetry at start of session for successful entry
-sends cli events for commands as they are used
-sends aggregate data about interactive session upon session end
This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[x] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

(see Authoring Command Modules)

View a preview at https://prompt.ws/r/Azure/azure-cli/5666
This is an experimental preview for @microsoft users.


TypeError: this.getFunctionFromCache is not a function

TypeError: this.getFunctionFromCache is not a function. (In 'this.getFunctionFromCache('set')', 'this.getFunctionFromCache' is undefined)

I am using MMKV as a storage to persist my zustand store
import create from 'zustand';
import {persist} from 'zustand/middleware';
import {localStorage} from '../localStorage';

export const useCollectionState = create(
  persist(
    set => ({
      count: 0,
    }),
    {
      name: 'test-storage',
      getStorage: () => localStorage,
    },
  ),
);

and the localStorage is an object, i have a hunch that, the context of this inside the set function is getting reset while exporting
import {MMKV} from 'react-native-mmkv';
const mmkv = new MMKV();
let localStorage = {
  ...mmkv,
  setItem: mmkv.set,
  getItem: mmkv.getString,
  removeItem: mmkv.delete,
};

export {localStorage};

solved by binding the value of this, curious to know if there's any other way of solving it
import {MMKV} from 'react-native-mmkv';

const mmkv = new MMKV();

export let localStorage = {
  ...mmkv,
  setItem: mmkv.set.bind(mmkv),
  getItem: mmkv.getString.bind(mmkv),
  removeItem: mmkv.delete.bind(mmkv),
};

(This issue can be closed now)

Yes the this is not bound here. An MMKV instance should not be unboxed, it's an instance object


Obsolete and unbuildable Python and pyOpenSSL dependencies

Installing Obelisk (using the command in the README on the master branch at commit 7ad33cbe3e84b209e83c505ce25486445bbd602e) brings in a dependency on python2.7-pyOpenSSL, which does not build because its tests rely on an expired certificate. In any case, as a person trying to decide whether to use the code, it's a bit worrying to see any dependencies at all on Python 2.
This is masked if you follow the official installation instructions, because the package will be downloaded from the binary cache.  By the way, I tend to think that the binary cache is dangerous and probably should not be recommended, maybe not provided at all.

Thanks for the report. python2.7-pyOpenSSL doesn't appear to be one of our direct dependencies, so we'll have to determine why it's in the dependency closure.

Based on the log in #836 - which looks to be the same issue - cabal2nix is bringing in this dependency.

Any news on this or #836? I have the same problem, love to install obelisk on wsl ubuntu 20.04 lts according to manual but fail with same error msg.


pivot_table performance is extremely slow compared to Pandas

Issue:
I am noticing that the koalas DataFrame.pivot_table is performing much slower than the pandas version.
The pandas pivot_table works almost instantaneously, whereas the fastest I can get the koalas pivot_table to run on my local
machine, is roughly 8 and a half minutes.
Problem Background
I am trying to create a pivot table from some data that looks roughly like:
The data has shape (5800, 3) roughly, and looks like the following:

destination_vm
port
num_sources

0
vm1
80
.0234

1
vm2
88
.123

3
vm3
3100
.0045

The desired result is something like the following, but with a much larger shape (2058, 1146):

port
80
88
3100

destination_vm

vm1
.0234
0
0

vm2
0
0.123
0

vm3
0
0
.0045

Running a pivot to transform the data into a sparse matrix like the above, I run the following:
spark = SparkSession.builder.appName("my_job").config(         "spark.driver.memory", "8g").config(         "spark.executor.memory", "6g").config(         "spark.sql.execution.arrow.pyspark.enabled", "true").getOrCreate()
ks.set_option('compute.default_index_type', 'distributed-sequence')
self._logger.info("Creating pivot table")
df_listening_vms_by_port = df_listening_for_distribution.pivot_table(                     values=["num_sources"],                  index=['destination_vm'],                  columns='port',                  aggfunc={'num_sources': 'sum'},                  fill_value=0              )
self._logger.info("Done creating pivot table")
Output
2021-07-23 16:56:57,914 - feature_engineering - INFO - Creating pivot table
2021-07-23 17:05:28,163 - feature_engineering - INFO - Done creating pivot table
The ~8m30s gap seems like a very long time to pivot a DF that is not too large.
The output shape is (2058, 1146), so the size has gotten much bigger. However, Pandas does a similar pivot almost instantaneously.
Things I've tried
I have made sure to use the distributed-sequence  default index, but this doesn't seem to improve the performance.
In local mode, I have tried doing a local_checkpoint on the input DF to try and clear out overhead from previous processing steps.
I have tried running this in cluster mode with 8 executors with 4GB ram each, but it doesn't speed up much, which makes me think there is some Window bottleneck or something.
I've also tried exporting the input DF as a csv, reading it into a koalas DF in a local pyspark shell, and then watching the Job via the spark UI. The pivot job finishes quickly, but the actual process doesn't return in the pyspark shell for a long time, and there is a long gap in between where it appears like nothing is happening.

Here is more detail on the DAG when running the pivot in a local pyspark shell:

Is there something obvious I am missing here to help speed this up?

Am I doing something obvious that would cause the behavior I am seeing where there is a long gap between the pivot Job and the later toPandas job? (See the first smaller screenshot). Is there some way I can gain visibility into what is happening in the interim?
If I set the sparkContext loglevel to DEBUG, there appears to be nothing happening for long periods of time.
I am noticing this pattern across my application, not just with the pivot_table operation. Can you help me understand what triggers a toPandas job and why there might be long gaps where no jobs are even scheduled, then the toPandas jobs will come suddenly. This happens even when running on a cluster, with 4 executors, 8 cores each, and using the default number of partitions, 200. My data size is not very large.
Consider the following snippet:
        self._logger.info('Concatenating kdf1 and kdf2 frames')
        ks.set_option('compute.ops_on_diff_frames', True)
        df_all_port_features = ks.concat([kdf1, kdf2], sort=False, axis=1)
        ks.reset_option('compute.ops_on_diff_frames')
        self._logger.info('Filling na')
        df_all_port_features.fillna(0, inplace=True)
        self._logger.info('foo')
The time between the "Concatenating..." message and the "Filling na" message is ~10 minutes, with nothing happening in the interim in the Spark UI. Here are the relevant dependencies of the container running the application; maybe I have something misconfigured.:
pip3 install kafka-python requests==2.25.1 kubernetes pyOpenSSL protobuf sklearn==0.0 scikit-learn==0.24.2 scipy==1.5.4 numpy==1.19.5 pandas==1.1.5 xgboost==1.4.2 pyarrow==4.0.0 pyspark==3.0.1 koalas==1.8.0

cc @xinrong-databricks and @itholic; maybe you guys need more information from me?
I have also tried with spark 2.4, and get the same results.

Hi, @danielknight . Sorry for the late.
I just tried to reproduce the issue, and I ran the code below:
from databricks import koalas as ks
import random
num_data = 5800

df_listening_for_distribution = ks.DataFrame(
    {
        "destination_vm": [f"vm{random.randrange(1, 2058)}" for _ in range(num_data)],
        "port": [
            random.randrange(10000, 11146)
            for _ in range(num_data)
        ],
        "num_sources": [random.random() for _ in range(num_data)],
    }
)

result_df = df_listening_for_distribution.pivot_table(
    values=["num_sources"],
    index=["destination_vm"],
    columns="port",
    aggfunc={"num_sources": "sum"},
    fill_value=0,
)

result_df.shape  # (1931, 1138)

And it took just 0.04 secs for computing the pivot_table.

Seems like the shape of table and the shape of the result table looks similar to your example case.
I used 8 executors with 8GB ram, though.
Could you take a look into my code if did I reproduce your case correctly ??

Hi, @danielknight . Sorry for the late.
I just tried to reproduce the issue, and I ran the code below:
from databricks import koalas as ks
import random
num_data = 5800

df_listening_for_distribution = ks.DataFrame(
    {
        "destination_vm": [f"vm{random.randrange(1, 2058)}" for _ in range(num_data)],
        "port": [
            random.randrange(10000, 11146)
            for _ in range(num_data)
        ],
        "num_sources": [random.random() for _ in range(num_data)],
    }
)

result_df = df_listening_for_distribution.pivot_table(
    values=["num_sources"],
    index=["destination_vm"],
    columns="port",
    aggfunc={"num_sources": "sum"},
    fill_value=0,
)

result_df.shape  # (1931, 1138)

And it took just around 2 mins for computing the pivot_table (It's seems like still not fast, though)
Seems like the shape of table and the shape of the result table looks similar to your example case.
I used 8 executors with 8GB ram, though.
Could you take a look into my code if did I reproduce your case correctly ??

I agree that your code is similar to this pivot I am doing. In a local pyspark shell, running your code exactly in local mode returns after ~10 minutes.
On my cluster, the most recent run with 8 executors (4G memory each) ran against a KDF with shape (28618, 3) and took around 3m17s, which I can infer from these log messages:
2021-07-29 22:17:17,237 - feature_engineering - INFO - Creating pivot table
2021-07-29 22:20:34,545 - feature_engineering - INFO - Done creating pivot table

Obviously (28618, 3) is a larger data-size than the (5800,3) the pivots we both tried, so it is at least a good sign that it only costs an additional minute and a half, even with less RAM per executor. So maybe I am getting the best performance possible for the time being out of the pivot_table?
I am encountering some additional issues later in this same feature engineering code though. As I build the KDF's from the pivot_table operations, I then need to concatenate them along axis=1, and this operation takes another 13.8 minutes. Later I do a fillna.
self._logger.info('Concatenating TCP and UDP frames')
ks.set_option('compute.ops_on_diff_frames', True)
df_all_port_features = ks.concat([TCP_one_hot, UDP_one_hot], sort=False, axis=1)
ks.reset_option('compute.ops_on_diff_frames')
# this takes 13.8 minutes
# some other stuff happens that potentially creates NA values, but the data shape doesn't change.
self._logger.info('Filling na')
df_all_port_features.fillna(0, inplace=True)
# this takes another 11.3 minutes

For the concat, I am probably experiencing the cost of doing a large join. Any way around this? I will try a merge when I have availability to test that and post my findings
For the fillna I am probably experiencing the cost of moving the big concatenated DF into a single parition, as suggested by the warning in the docs here, https://koalas.readthedocs.io/en/latest/reference/api/databricks.koalas.DataFrame.fillna.html
This leads to move all data into single partition in single machine and could cause serious performance degradation. Avoid this method against very large dataset.
In general, maybe I am running up against the performance constraints of koalas? I am not using this in a notebooked environment, but rather trying to make a POC of a production service. It will run a k8s-based pyspark job which will run on a certain schedule, thus it is important to us that it is both reliable and performant, as to not hog resources from the cluster. Maybe I will encounter the same overhead in raw pyspark, and our use-case will require more resources regardless of using koalas or raw pyspark.

I had similar issue when pivot table of shape 1639320 rows × 4 columns. It took me like 2 minutes to create the pivot.
I noticed that I had some values which have been None and some with np.nan.
After command .fillna(np.nan) creating pivot took only 1s


Consistently downcase the masterKubeConfig path

This PR is a short fix for a bug I encountered when trying to provision a node whose fqdn contains uppercase letters (eg.MY-OSE-SERVER.example.com).
The symptoms
The chef run failed with a broken origin-node service.
The interesting snippet was in the origin-node service logs :
May 21 12:19:15 MY-OSE-SERVER origin-node[32335]: I0521 12:19:15.969796   32335 start_node.go:309] Reading node configuration from /etc/origin/node/node-config.yaml
May 21 12:19:15 MY-OSE-SERVER origin-node[32335]: Invalid NodeConfig /etc/origin/node/node-config.yaml
May 21 12:19:15 MY-OSE-SERVER origin-node[32335]: masterKubeConfig: Invalid value: "/etc/origin/node/system:node:MY-OSE-SERVER.example.com.kubeconfig": could not read file: stat /etc/origin/node/system:node:MY-OSE-SERVER.example.com.kubeconfig: no such file or directory
May 21 12:19:15 MY-OSE-SERVER systemd[1]: origin-node.service: main process exited, code=exited, status=255/n/a

The investigation
Apparently, the cookbook downcases the path of the master kube config path when it creates it on the disk, but it does not downcase the path when referencing that file in /etc/origin/node/node-config.yaml.
The fix
The fix is just to downcase the masterKubeConfig entry in the template that is rendered into /etc/origin/node/node-config.yaml
Before this PR:
$ grep --color -ry system:node.*fqdn.*kubeconfig *
templates/default/openshift_node/node.yaml.v1.erb:masterKubeConfig: system:node:<%= node['fqdn'].downcase %>.kubeconfig
templates/default/node.yaml.erb:masterKubeConfig: system:node:<%= node['fqdn'] %>.kubeconfig

After this PR:
$ grep --color -ry system:node.*fqdn.*kubeconfig *
templates/default/openshift_node/node.yaml.v1.erb:masterKubeConfig: system:node:<%= node['fqdn'].downcase %>.kubeconfig
templates/default/node.yaml.erb:masterKubeConfig: system:node:<%= node['fqdn'].downcase %>.kubeconfig

Please do NOT merge, there are other broken behavior when the fqdn contains non downcase letters, for example origin-ndoe does not start because "Could not find an allocated subnet for node: MY-OSE-SERVER.example.com, Waiting..." ... I will force downcase fqdn and reinstall.


clear cache when adding or updating post

Is it possible to clear the cache when adding a new post or changing?

Hello,
+1 for this one
if possible, please add this feature to the CLP-WP Varnish plugin
thanks!

It's already the case.
You can check the PHP controller:
https://www.cloudpanel.io/docs/v2/frontend-area/varnish-cache/developer-guide/php-controller/


Expose getLocationPermission function

Currently there is no way to easily request a new location, the whole passing through of refs in the parent is quite in-elegant. And this only seems to work once...
I have the following usecase: Map that shows the user's location and everytime they press 'relocate to my location' It should popup the location permission box if denied earlier.

Hi, the ref-based solution should work (and it should work repeatedly, if it's not, please provide an example). I don't know how else I should expose the function, do you have any ideas?

My issue was that I only had a single component file, just the app.js. It is a very small app. And since stateless components don't have access to their own 'this'....
For now I fixed it by wrapping it in another container component just to get the ref.
If you want, I can take a look this weekend and see if I can make a PR to expose the function better? I have a few ideas that might work.

Oh, I see. I'm glad you made it work :) PR would be welcome if you have the time :)

Closing as inactive


95 updates to installation

Hi @atanikan,
I changed and fixed the issues you mentioned in #95.
Would that changes go into the direction you'd expect?
Best
Cedric

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 18.18182% with 9 lines in your changes missing coverage. Please review.

Project coverage is 79.82%. Comparing base (f07d3c0) to head (e78e7e5).
Report is 3 commits behind head on master.

Files with missing lines
Patch %
Lines

deep_river/classification/rolling_classifier.py
16.66%
5 Missing :warning:

deep_river/classification/classifier.py
20.00%
4 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master      #97   +/-   ##
=======================================
  Coverage   79.82%   79.82%
=======================================
  Files          26       26
  Lines        1254     1254
=======================================
  Hits         1001     1001
  Misses        253      253

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


Could not initialize offscreen context

I bought an intel realsense depth camera D435i and trying to install realsense .
I exactly followed the tutorial
When I am running realsense-viewer , I am getting the following error :
Could not initialize offscreen context!
hwinfo
graphics card:
Acer Incorporated [ALI] GeForce 710M
Intel 3rd Gen Core processor Graphics Controller
Primary display adapter: #14
-Computer-
Processor		: Intel(R) Core(TM) i3-3120M CPU @ 2.50GHz
Memory		: 3855MB (2398MB used)
Machine Type		: Notebook
Operating System		: Linux Lite 4.8
-Version-
Kernel		: Linux 4.15.0-74-generic (x86_64)
Version		: #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019
C Library		: GNU C Library / (Ubuntu GLIBC 2.27-3ubuntu1) 2.27
Distribution		: Linux Lite 4.8
-Display-
Resolution		: 1366x768 pixels
OpenGL Renderer		: (Unknown)
X11 Vendor		: The X.Org Foundation
-Audio Devices-
Audio Adapter		: HDA-Intel - HDA Intel PCH
-Input Devices-
HDA Intel PCH Mic
HDA Intel PCH Headphone
HDA Intel PCH HDMI/DP,pcm:3
HD WebCam: HD WebCam
Intel(R) RealSense(TM) Depth Ca
-Display-
Resolution		: 1366x768 pixels
Vendor		: The X.Org Foundation
Version		: 1.19.6
Current Display Name		: :0.0
-Monitors-
Monitor 0		: 1366x768 pixels
-OpenGL-
Vendor		: (Unknown)
Renderer		: (Unknown)
Version		: (Unknown)
Direct Rendering		: No
-Extensions-
Composite
DAMAGE
DOUBLE-BUFFER
DPMS
DRI2
GLX
Generic Event Extension
MIT-SCREEN-SAVER
MIT-SHM
Present
RANDR
RECORD
RENDER
SECURITY
SGI-GLX
SHAPE
SYNC
X-Resource
XC-MISC
XFIXES
XFree86-DGA
XFree86-VidModeExtension
XINERAMA
XInputExtension
XKEYBOARD
XTEST
XVideo
default screen number:    0

Hi @mkeyno
This error is not related to the RealSense camera - you should be able to run most examples and use the API. Viewer requires OpenGL 3 support, hence the error. It seems you don't have graphic drivers correctly installed, in particular:
-OpenGL-
Vendor : (Unknown)
Renderer : (Unknown)
Version : (Unknown)

@dorodnic
thanks for your comment, I reinstall driver with other method and software but no success
I think this repo is not mature or hard to implement for Linux base  OSes, because I also couldn't successful using my camera over raspberry pi 4
I kinda regret to buy this product

Sorry you feel this way.
Not sure what having OpenGL drivers installed has to do with the product or repo maturity

@dorodnic
realsense-viewer runs smoothly, when raspberry pi 4 is connected with HDMI to a monitor. But through VNCserver gives output "could not initialize offscreen context" . Help sovling th problem please.


[QUERY] How to get total count of blobs size in a container?

Hi,
I have a requirement to get the sum of all blobs size for each container. I know I can calculate the size by listing all blobs and summing up all of them to get the container size, but that is too complex on the code level as each blob can have lot of versions and soft deleted blobs within the container etc. I would like to know if there is any way, where in i can just make one api call per container to get the total size of the container (sum of all blobs size)?
Thanks in advance.

Hi @kirankumar994
There isn't an API available today that you can use to get a total count of all blobs in the container. You can specify whether you want to include deleted blobs or versioned blobs when calling listBlobs(ListBlobsOptions, String, Duration). Please take a look at ListBlobsOptions and BlobListDetails,
https://github.com/Azure/azure-sdk-for-java/blob/main/sdk/storage/azure-storage-blob/src/main/java/com/azure/storage/blob/models/ListBlobsOptions.java
https://github.com/Azure/azure-sdk-for-java/blob/main/sdk/storage/azure-storage-blob/src/main/java/com/azure/storage/blob/models/BlobListDetails.java
Let me know if this helps or if you have any further questions!


Screenshotter bot no longer working

ZEIT is now Vercel and it no longer supports server apps, i.e., bots. During the transition, it seems the private key environment variable for the GitHub App got lost, so I cannot deploy to another platform.
@kevinbarabash Would it be possible to temporarily grant me the App management permission for the KaTeX org and admin permission for this repo?

@ylemkimon you now have access.

@kevinbarabash Thank you and thank you for all the reviews!
I'm trying to figure out a way to run the bot on GitHub Actions.

GitHub Actions still doesn't have a way to access PRs from forked repo, so I deployed our probot to Heroku.

I hope that GitHub addresses that limitation soon.  I assume the Heroku instance is their free tier.

It seems GitHub introduced new pull_request_target event five days ago: https://github.blog/2020-08-03-github-actions-improvements-for-fork-and-pull-request-workflows/. I'll try using that.

Super cool!  Their solution seems like a very sensible solution for the security issues.


Fix styles of documentation about interpreters

Due to the documentation of interpreters written by various authors, styles of documentations are vary and has broken tags and inconsistent spaces. This fix cleans those little glitches to make documentation more readable and look cleaner to users.

Clean up unnecessary or wrong HTML tags ex:
 after titles
Fix too wide spaces between paragraph and tables
Remove unnecessary spaces at end of paragraph and lines

Thanks @yoonjs2 for the good contribution.
Just merged another documentation improvement  #578 from @AhyoungRyu. That seems make this branch conflict to master. Do you mind resolve the conflict once again?

Hi @Leemoonsoo, I just merged my branch with #578.
And also I fixed & matched more unnecessary spaces and indention in every interpreter documentations to make them more looks like written by one author. But while I merging documentations I found contents and numbering of titles are also varies between documents.

Some documents has overview but some are not.
Some documents use numbering at titles but some are not.
Some documents has spaces between titles and paragraph but some are not.
Some documents use
 tags to make line spaces but some are not.

So It may need to build some standard guidelines in documentation styles to give ground rules for users to start documentation on their proposing features. If that guidelines goes well with the community, It will be easy to add nice design and document format in the future. (like http://getbootstrap.com or other OSS projects)
Any suggestions?

@yoonjs2
It's really good idea to make a standard guideline. I always think that we need it!

The change looks good to me. Merge if there're no more discussions.
And big +1 for guidelines for documentation style.

looks good


Please support Enterprise Linux 7 (EL7) again

Proposed feature
python_pip should support EL7 again
Rationale
I am not quite happy how this was removed here and from my perspective this is more a bug then a feature: https://github.com/robertdebock/ansible-role-python_pip/commit/4962eb1f587abb295613ada5137919fddecd22a3#commitcomment-63877727
Additional context

"The interpreter used by Ansible (see ansible_python_interpreter) requires the setuptools package, regardless of the version of pip set with the executable option."

Optional Bug Description
If this would be a bug.
Error:
TASK [robertdebock.development_environment.python_pip : update pip] **********************************************************************************************************************
An exception occurred during task execution. To see the full traceback, use -vvv. The error was: ImportError: No module named pkg_resources
fatal: [localhost]: FAILED! => changed=false
  msg: Failed to import the required Python library (setuptools) on localhost's Python /usr/bin/python. Please read module documentation and install in the appropriate location. If the required library is installed, but Ansible is using the wrong Python interpreter, please consult the documentation on ansible_python_interpreter

To fix this please ensure requirements first "setuptools package".

https://github.com/snapcore/snapcraft/pull/3428 - the relevant drop support information.
something like this could help:
- name: update pip
  ansible.builtin.pip:
    name: "{{ item }}"
    state: latest  # noqa package-latest
    executable: "{{ python_pip_executable | default(omit) }}"
  loop:
    - "pip{{ '<21' if ansible_facts['os_family'] == 'RedHat' and ansible_facts['distribution_major_version'] == '7' }}"
    - setuptools
  when:
    - python_pip_update | bool

code https://github.com/robertdebock/ansible-role-python_pip/blob/master/tasks/main.yml#L34-L42

Fixed with the latest release. Thanks for your feedback!


update incidents name

closes https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/10296

@altsou
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@altsou
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@altsou
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@altsou can you please review and merge? cc @tiburd


Getting ** (CaseClauseError) no case clause matching: {:id, :id, :id}

Hi guys! I am receiving the following error while running my test after upgrading my elixir version from 1.4 to 1.5.1, nevertheless, I couldn't be able to make things run again even after downgrade the elixir version and cleaning the cache by deleting the _build and deps folders.
  1) test deliver/2 returns an {:ok, "ok"} tuple (Notifications.WebHook.PerformTest)
     test/webhook/perform_test.exs:22
     ** (CaseClauseError) no case clause matching: {:id, :id, :id}
     code: organization_hook = insert(:organization_hook)
     stacktrace:
       (ex_machina) lib/ex_machina/ecto.ex:253: ExMachina.Ecto.drop_autogenerated_ids/2
       (ex_machina) lib/ex_machina/ecto_strategy.ex:37: ExMachina.EctoStrategy.cast_all_fields/1
       (ex_machina) lib/ex_machina/ecto_strategy.ex:31: ExMachina.EctoStrategy.cast/1
       (ex_machina) lib/ex_machina/ecto_strategy.ex:17: ExMachina.EctoStrategy.handle_insert/2
       test/webhook/perform_test.exs:23: (test)

This is happening using any of the following lines
insert(:organization_hook)

or
OrganizationHooks.changeset(%OrganizationHooks{}, params_for(:organization_hook))

or even
params_for(:vehicle_track)

Those lines and others were working before. Reviewing the function involved, I saw that the case statement does not handle the {:id, :id, :id} tuple. It works after a bit of handcrafting:
  defp drop_autogenerated_ids(map, struct) do
    case struct.__schema__(:autogenerate_id) do
      nil -> map
      {name, _type} -> Map.delete(map, name)
      _ -> map
    end
  end

I have to say that my solution might be a little bit poor or even I am doing something wrong. So would like to ask you guys for any thoughts in order to dive further and eventually create a PR if necessary.
Thank you in advance!

I ran in to this as well. Your hack fixed it for me too.

Same for me, the hack works.

Ran into this myself.  Looks like the issue is a result of using ex_machina with Ecto 2.2, the two PRs to add support #241 and #238 fix the issue.
@paulcsmith any chance of those being merged anytime soon?

#238 does solve the problem, and we need to cut a new release for this. Otherwise we have to depend on this git repo at the current master ref.

@doomspork yes, hoping to test it and get it out on Friday. I don't want to cut a new release without doing some testing on our projects first. That way we can be reasonably sure we didn't inadvertently introduce any new issues

Cool, yea I tested this out on our umbrella application with ecto 2.2.1 and it resolved this issue that was cropping up. Can you mention me when you cut a release @paulcsmith?

@warmwaffles I will try to remember, but if you subscribe to this issue you will be notified when it closes 👍

Cool, thanks!

There is also https://tellmewhenitcloses.com for people who want to be
notified when the issue closes but don't care about all the comments.
On Wed, Aug 30, 2017 at 1:35 PM, Matthew Johnston<EMAIL_ADDRESS>wrote:

Cool, thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/thoughtbot/ex_machina/issues/240#issuecomment-326064235,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAcePSI-kVT6xl25inu1ea_hmmFZyeQKks5sdZ14gaJpZM4PAm52
.

Could someone from thoughtbot could publish some beta version to Hex?
I can't upgrade non of my packages because of this, it will break with the latest ecto version

@paulcsmith hey mate, could you at least publish some beta version to Hex? Tests are passing!
It wouldn't take you that much and will help a lot of people like me. I really need this upgrade for be able to run my test and worst for me, I have deadline to meet 😢  I really need your support on this, please.

@yordis I will put it up on hex soon. You can also use the version on master right now and it should work for you 👍
{:ex_machina, github: "thoughtbot/ex_machina"}

You may also want to consider holding off on upgrading Ecto until ExMachina is ready. Hoping to get it out soon though

@paulcsmith I would do that but have more side effects than take 2 minutes for publish a beta version to Hex.
Hex will download the hold github project, also just imagine that somebody break the package in the next commit, I can't control that once I pull from master
I know I could do that, but once again, I am asking for some beta version, it wouldn't hurt and takes you little time for that. I don't see what is the problem with that, so people who want to use beta version understand what is going on and take the responsibility.
Just be on my position for a moment please, it's 2 minutes from your side vs a lot of side effects from your proposal. Please.

Also from March 3 you didn't release. Issues like ex_machina complaining about having ecto twice on the high level projects will be fix as well.

If you point to master it will lock it to the version currently there until you explicitly update it. Try it and take a look at mix.lock, you'll see it locks to a particular commit. It is a perfectly valid solution that can hold you over until tomorrow when I publish the full version. There is very little to gain by me doing a beta release only to do the real one tomorrow with no other changes.
Please be patient with me. It takes more than 2 mins because I have to go through commits and create a changelog. I could post without it, but I'd rather wait until tomorrow and do it right.

@yordis you can lock to a commit explicitly in mix.exs like this as well:
{:ex_machina, github: "thoughtbot/ex_machina", ref: "8f494f1"}
This won't break even if you run mix deps.get again.

2.1.0 is up on hex! Feel free to give it a shot and open up an issue for anything you find


Fixes #5217

Fixes #5217, a bug when you can install hg by typing ... is shown like git branch. Also saves time for those who don't want hg to be installed and have command-not-found enabled as there is no need to search repo for hg each time prompt is being printed.

@stevenspasbo I agree. Updated pull request.

The idiomatic zsh way is using $commands[hg] in the following way:
if (( $+commands[zsh] )); then
...

In your case, this would be enough:
```zsh
(( $+commands[hg] )) || return

@mcornella Okay, updated once again


Tidyup

also fix:
https://docs.oracle.com/javase/7/docs/api/java/sql/Connection.html#close()
states
"Calling the method close on a Connection object that is already closed is a no-op."
but HikariCP throws exception

CI failure is :
java.lang.NoSuchMethodError: org.hamcrest.Matcher.describeMismatch(Ljava/lang/Object;Lorg/hamcrest/Description;)V
at com.zaxxer.hikari.metrics.prometheus.HikariCPCollectorTest.connection1(HikariCPCollectorTest.java:54)

Coverage increased (+0.1%) to 75.42% when pulling d024380e6165c9728c0a84f197d893a20b84d821 on nitincchauhan:dev into 61ae2ddcb67c8f9443451b883126a9c96ce041cf on brettwooldridge:dev.

Coverage increased (+0.07%) to 75.393% when pulling 8f41140a62413a5eb9d2a61808c8c549a3ad8a61 on nitincchauhan:dev into 61ae2ddcb67c8f9443451b883126a9c96ce041cf on brettwooldridge:dev.

Cherry-picked.


Add ability to perform actions from scroll handler

This covers a use case when actions need to be performed in relation to scrolling in Hub view controller (for example centering new item).

I'm not sure if this is such a good idea - it gives the scroll handler a bit too many responsibilities. Is there no way to perform these actions from a content operation or component instead?

The only object that is notified of scrollingDidEnd events is the scroll handler. Scroll handler has no way to access any component, so it can't be the component who performs the action. Performing this action from content operation seems completely irrelevant.
Another approach would be to notify components conforming to HUBComponentContentOffsetObserver of scrollingDidEnd events in addition to scrollViewDidScroll. Would it be a better solution?


Commands: refactor commands in classes

Summary
Refactoring of commands into appropriate classes.

@mickahell @bala-ceg for your visibility


If all shards fail, fail the request despite shards.tolerant

Description
If the request is tolerant to errors, logic in SearchHandler simply sets partialResults to true upon detecting an error in ShardResponse. This means a failure to query any shard would be reported back as partialResults=true, instead of a query failure.
Why: Improve the error handling and reporting in the search system by making sure that query failures are distinct from partial results
Solution
Evaluate if all shards failed while fetching top results. If yes, fail the request by throwing a SolrException, if not set partialResults=true
Tests
Please describe the tests you've developed or run to confirm this patch implements the feature or solves the problem.
Added a test in TestTolerantSearch: testAllShardsFail() that performs distributed search across 2 shards, but both shards have failures. The test performs the distributed search first by setting shards.tolerant=false and then by toggling it to true. The test confirms that in both cases, a SolrException is thrown. Also ensured existing tests in TestTolerant pass with the new code changes.

torvalds.dev is analyzing the ticket

The issue at hand is related to the error handling and reporting in the search system. Currently, if a request is tolerant to errors, the logic in SearchHandler sets partialResults to true upon detecting an error in ShardResponse. This means that a failure to query any shard would be reported back as partialResults=true, instead of a query failure. The goal is to improve this system by making sure that query failures are distinct from partial results.
The proposed solution is to evaluate if all shards failed while fetching top results. If all shards fail, the request should fail by throwing a SolrException. If not all shards fail, partialResults should be set to true.
The relevant files for this issue are:

solr/core/src/test/org/apache/solr/TestTolerantSearch.java: This file contains the TestTolerantSearch class which includes tests for the tolerant search feature. The test testAllShardsFail() is particularly relevant as it performs a distributed search across 2 shards, both of which have failures. The test confirms that a SolrException is thrown in both cases when shards.tolerant is set to false and when it is set to true.

solr/core/src/test/org/apache/solr/cloud/TestDownShardTolerantSearch.java: This file contains the TestDownShardTolerantSearch class which tests the behavior of the search system when one shard is down. It asserts that shards.tolerant=true works even if one shard is down and that a meaningful exception is thrown when shards.tolerant=false.

The actions that should be taken are:

Implement the proposed solution in the search system. This would involve modifying the logic in SearchHandler to throw a SolrException when all shards fail while fetching top results.

Update the tests in TestTolerantSearch and TestDownShardTolerantSearch to reflect the new behavior of the search system. Ensure that the tests pass with the new code changes.

Review the changes and test the search system thoroughly to ensure that the new error handling and reporting mechanism works as expected.


VaadinRun doesn't generate themes?

I had a friend of mine clone a repo of mine and ran vaadinRun on it just to get a 404 on the page.
I told him to ran:

gradle vaadinCreateTheme --name mytheme

And everything was working properly as expected. Could vaadinRun include this behaviour in the future?
Console showed

No themes were found in themes directory directory '...webapp/VAADIN/themes', skipping theme compiler dependencies

And I used annotation @Theme("mytheme") in the project.
Thanks!

vaadinRun should just run the project, its not meant for generating anything. If vaadinRun would generate a theme for you then you would get a new theme every time you run the project.
When you create a project you should use vaadinCreateProject, that will create all the necessery things like the theme, a UI annotated with the correct annotations like @Theme etc.


[FEATURE] Filter gitlab project for sync by path / namespace

fix #30

Thanks for these PRs @NeoBlack ! I will review them as soon as I can, and get them merged.

Any objections regarding this patch?

Thanks for this! And thank you for your patience :+1:


Make sure that all plugins have nice auto-generated documentation in Sphinx

All the plugins should have nice documentation published on Sphinx.

@rw1nkler - What is the correct approach for this repo which is mostly C code? Something like the Verilog to Routing set up?

@tmichalak -- Who should take on this work?

@rw1nkler - What is the correct approach for this repo which is mostly C code? Something like the Verilog to Routing set up?

The setup for C/C++ code is simple. I can prepare it for this repository. The most important part is to document the code using Doxygen code comments.


[mgmt] update codeowners

fixes https://github.com/Azure/azure-sdk-for-js/issues/27472

@JimSuplizio ,please help review this pr, thanks

@kazrael2119, I'm going to override check-enforcer as there's nothing that'll run at the moment that'll test these changes. I ran a tool locally to sanity test these changes and verified that every ci.mgmt.yml file in the repository gets picked up correctly (I'd also done that before suggesting the change).

/check-enforcer override

/check-enforcer override


Respawn Event Listener

https://github.com/ClickPT/EasySetSpawn/blob/master/src/me/clickpt/easysetspawn/Events.java#L128-L135
In the respawn event listener you are delaying the teleportation by 5 ticks. However, this is bad practice as this will always override another listener for respawning that might have a higher priority. You should be using the setRespawnLocation method for the event.

Thanks for advising. I will fix it in possible future update.

Fixed in the new update (2.1).


Brackets mistakes in Docs of ContainerProperties Class

Link to article:
[https://docs.microsoft.com/en-us/dotnet/api/microsoft.azure.cosmos.containerproperties?view=azure-dotnet#examples](ContainerProperties Docs. Example Section)
Problem:
When i've explored docs of ContainerProperties, i noticed that in example section you have few mistakes with brackets.
Instead of using ) bracket to close signature of method call , you use ] bracket.
Example:
    Container container = client.GetDatabase("dbName"].Containers["MyCollection"];     await container.DeleteAsync();

Fixed.
#please-close