Refs #23919 -- Replaced doc references to django.utils.http.urlquote() and urlquote_plus() with standard library alternatives.

Follow up to fee42fd99ee470528858c2ccb3621135c30ec262.

Ah yes, that unicode page! I guess that until someone finds the time to rewrite the whole page, we'll stick to partly updates. Thanks for these changes!


refs #1082 - putting existing PR on top of master

SlimerJS has some options that start with - instead of -- and CasperJS will now forward them with this commit.
This is blatant theft from #1082 with just a rebase on top of the current master.

:+1:

Thanks for the rebase. Merging in favor of  identical, outdated #1082.


[PROPOSAL] annotated gitignore files

This is a work-in-progress, based off the discussions in this gist around tackling some of the issues with maintaining a large suite of gitignore templates.
The Problem
Over time, the context provided in gitignore templates is lost into the aether. By providing some structure to these files, hopefully we can maintain (and improve) this situation.
There's other parties who depend on this repository, so I'm
The Goals
First off, thanks to everyone who participated in that gist, in particular @joeblau and @arcresu - who have much more context on this issue than myself. I've taken that feedback and turned that into a first step down this path.
I want to enrich our existing templates with metadata and add some simple structure, without breaking compatibility (that is, it should remain consumable as a gitignore template) or introducing any new tooling.
I believe this will help on a few fronts:

maintainers: will have extra context available to them, when reviewing proposed changes
contributors: will be able to view context alongside template changes, without having to navigate a long history of PR discussions. Hopefully contributors will also be able to discuss and clarify this context over time as well, but this depends on things like adoption.
users: will have the context around a template available at consumption time, so they can understand why a template is configured as such a way, and be empowered to make changes because they understand why a rule exists

The Proposal
This PR makes some changes to the VisualStudio template as a testbed for what data we need to capture. As I've been doing this, a bunch of concepts have crystalized for me, which I'll formalize in the CONTRIBUTING document alongside this (it also feels very much like a grammar syntax is the end result, but let's see how the discussion evolves).
I'm using the VisualStudio template as a starting point for a couple of reasons:

It's one that I'm very familiar with, so taking on the challenge to provide more context is more achievable than choosing a template (or stack) I'm not as familiar with.
I've had various discussions with people in the community about recent template changes (like #1505), so it'll be a good test case for how feasible this idea is.

I'll write up some further comments about this format in this PR.

cc @MichaelMarner @bkeepers @M-Zuber @mikemcquaid who also commented on the gist

cc @adamralph @fekberg @damianh @akoeplinger @filipw who were all bugging me on Twitter about this yesterday.

This looks very nice.
The remove/include sections is something I have wanted for a long time.
Overall it really changes the feel from a random list of file extensions to a document that is self expressing.
It would be very interesting to see the grammar that develops and to see what tooling rises out of that.

@shiftkey This is neat :+1: I think the key will be having a clear test which fails if it's not in the right format (and that people can run locally using just a gitignore checkout).

I'm currently in the process of re-writing gitignore.io's generator in golang.  Do you think people will want to see a documented output file or undocumented as default when someone types gi visualstudio?  I assume that the default uncommented lines in these templates are fine and would prefer and undocumented default, but I would like to get some feedback from others if we're going to move forward with this format.

Do you think people will want to see a documented output file or undocumented as default when someone types gi visualstudio?

I think if you're using a service like gitignore.io you're just interested in getting the contents, rather than the document. Perhaps a option like --verbose for those who want to get the annotations as well?

I would definitely want an option like --verbose as I use gitignore.io to get the template and then make my changes, and having the extra information would be very useful

As part of getting some focus this year, I'm dropping this. Happy to help someone else if they want to pick it up.

If it's closed then it's unlikely anyone will ever pick it up...

@adamralph ok then

@shiftkey Any reason for not merging the VisualStudio changes (separately) and keeping the proposal?

@fekberg merge conflicts mostly

@shiftkey oh sorry, this is a PR, I thought was an issue. Another option would be to close this, but copy out the proposal into an issue.

@adamralph issues have been disabled for this repo :cry:

:footprints: :gun:

@shiftkey FWIW I think it's completely reasonable to close this out.

If someone else in here wants to take some time out cleaning up the VisualStudio.gitignore template, it'd be :heart:. But as I said above I'm putting some of the big-picture stuff I have in mind on hold for the moment...

@shiftkey this raises an argument for enabling issues, since then you could open one and mark it up for grabs. Very few people are ever going to search closed pull requests for way to help out.

this raises an argument for enabling issues, since then you could open one and mark it up for grabs.

I think this is a bigger problem that won't necessarily be solved by enabling issues on the repository, but I'll think about it. Meanwhile, the branch is still open for reference.


Generated model requires unknown imports

Describe the bug
At the moment the generators have no way of telling the user which imports they require.
For example, for the Java generator, it generates enums with the expectation of com.fasterxml.jackson library to be present, however, the user has no way of knowing it is.
Expected behavior
I expected to be able to generate the model as is, without having to require me to dig into the generator to figure out which imports/required libraries are needed.

We can add helper array in each generation of model where dev/user can pass needed dependencies in presets/generators for given model.

We can add helper array in each generation of model where dev/user can pass needed dependencies in presets/generators for given model. Something like we have for getNearestDependencies() function, but as an array/object :)

Good idea 👍
I just see a dilemma down the line. You now have a list of dependencies that are required for the model. But in your project (where the user output writes the models to) we require you to manually add those (dynamic) dependencies to your project dependencies... Then the user would need to "know" about the dependencies anyway 😕 Of course it helps in importing the dependencies in the model output file, just not for the project itself.

I just see a dilemma down the line. You now have a list of dependencies that are required for the model. But in your project (where the user output writes the models to) we require you to manually add those (dynamic) dependencies to your project dependencies... Then the user would need to "know" about the dependencies anyway 😕 Of course it helps in importing the dependencies in the model output file, just not for the project itself.

We can log all unique dependencies when someone will generate models as post-generation hook and only inform that project needs given deps to property work.

Could do yea 🤔
Well, an issue for a later day I guess.


Hiding of elements with non-unique names

When trying to hide an element with a non unique-name with the context menu only the first one in features array is hidden. That is not necessarily the one which was clicked on. Furthermore it is not possible to hide the other one.
Reproduce with demo:

Right click on the ycf2 feature in the second inverted repeat of O. europaea.
Select Hide.
The ycf2 in the first inverted repeat disappears while this one remains.
Repeat the procedure multiple times.
Nothing else happens.

The problem occurs when you select a feature which appears more than once on the same genome. For example ycf2 appears twice on O. europaea.
For fixing the bug it may help to add exact Ids to all features in the perl-script (in the same way links have an id). Then the exact hiding of features should be possible.


Yarn install step for deploy action sporadically breaks

Similar case to this, currently trying to bypass with retrying after clearing cache and removing lockfile.

switched to npm 👍


remove bmag_mid as input to collisions table for normNu

Currently if we want to do normNu collisions we have to specify the input
.bmag_mid = ...,

to the collisions table. This should be calculated internally, as a use may forget to change this argument after changing the magnetic field profile.

Solved in PR #353


PR #91994 - tenant: add apiv2 support for sql-over-http

Exalate commented:
Related PR: https://github.com/cockroachdb/cockroach/pull/91994
Commit: https://github.com/cockroachdb/cockroach/commit/94f14ab84dafcc9222003db4ba1c103df5e293c9

Release note (ops change): sql tenants now support the HTTP endpoint under
/api/v2/sql which allows the caller to execute an HTTP request containing SQL
statements to execute. The JSON response contains the results. This endpoints
works identically as on a non-tenant server, except that it naturally scopes to
the target tenant for SQL execution.
Epic: CRDB-17356
Jira Issue: DOC-6229

serverless feature. no docs required.


Go: 424-Longest Repeating Character Replacement & 567-Permutation in String

File(s) Modified: 424-Longest-Repeating-Character-Replacement.go, 567-Permutation-in-String.go
Language(s) Used: Golang
Submission URL: Longest Repeating Character Replacement - https://leetcode.com/submissions/detail/798561063/, Permutation in String - https://leetcode.com/submissions/detail/798711212/

Thanks, @imrushi!


Error in Hue integration after updating to 2022.2.7 and 2022.2.8 - Hue does not startup anymore

I saw the solution provided (related to the motion sensor) but it doesnt exactly relate to me. I dont have any Philips Hue motion sensors, only Lights and Buttons.
I do have in fact a TV Zone created in my Hue App. Should i maybe delete it ? will that help ?
Here is my error :
`Logger: homeassistant.components.hue.bridge
Source: components/hue/bridge.py:75
Integration: Philips Hue (documentation, issues)
First occurred: 12:25:43 PM (2 occurrences)
Last logged: 12:32:58 PM
Unknown error connecting to Hue bridge
Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/components/hue/bridge.py", line 75, in async_initialize_bridge await self.api.initialize() File "/usr/local/lib/python3.9/site-packages/aiohue/v2/init.py", line 104, in initialize await self.fetch_full_state() File "/usr/local/lib/python3.9/site-packages/aiohue/v2/init.py", line 242, in fetch_full_state await asyncio.gather( File "/usr/local/lib/python3.9/site-packages/aiohue/v2/controllers/base.py", line 312, in initialize await resource_control.initialize(initial_data) File "/usr/local/lib/python3.9/site-packages/aiohue/v2/controllers/base.py", line 80, in initialize resource: CLIPResource = dataclass_from_dict(self.item_cls, item) File "/usr/local/lib/python3.9/site-packages/aiohue/util.py", line 206, in dataclass_from_dict **{ File "/usr/local/lib/python3.9/site-packages/aiohue/util.py", line 207, in field.name: _parse_value( File "/usr/local/lib/python3.9/site-packages/aiohue/util.py", line 144, in _parse_value return dataclass_from_dict(value_type, value) File "/usr/local/lib/python3.9/site-packages/aiohue/util.py", line 206, in dataclass_from_dict **{ File "/usr/local/lib/python3.9/site-packages/aiohue/util.py", line 207, in field.name: _parse_value( File "/usr/local/lib/python3.9/site-packages/aiohue/util.py", line 147, in _parse_value return [ File "/usr/local/lib/python3.9/site-packages/aiohue/util.py", line 148, in _parse_value(name, subval, get_args(value_type)[0]) File "/usr/local/lib/python3.9/site-packages/aiohue/util.py", line 144, in _parse_value return dataclass_from_dict(value_type, value) File "/usr/local/lib/python3.9/site-packages/aiohue/util.py", line 206, in dataclass_from_dict **{ File "/usr/local/lib/python3.9/site-packages/aiohue/util.py", line 207, in field.name: _parse_value( File "/usr/local/lib/python3.9/site-packages/aiohue/util.py", line 147, in _parse_value return [ File "/usr/local/lib/python3.9/site-packages/aiohue/util.py", line 148, in _parse_value(name, subval, get_args(value_type)[0]) File "/usr/local/lib/python3.9/site-packages/aiohue/util.py", line 144, in _parse_value return dataclass_from_dict(value_type, value) File "/usr/local/lib/python3.9/site-packages/aiohue/util.py", line 206, in dataclass_from_dict **{ File "/usr/local/lib/python3.9/site-packages/aiohue/util.py", line 207, in field.name: _parse_value( File "/usr/local/lib/python3.9/site-packages/aiohue/util.py", line 184, in _parse_value raise TypeError( TypeError: Value 1 of type <class 'int'> is invalid for Position.y, expected value of type <class 'float'>`

This is related to the issue we had with the motion sensors. The data does not match with the models of the API.
This is either because Signify needs to update their documentation or the data is in fact invalid due to a bug or a device issue etc.
I've looked in the code and indeed the only resource that uses the Position object is the entertainment space.
So as temporary solution you could just remove the entertainment zone from the Hue app. I'll submit a bugfix to update the model.

Maybe it is enough just to change the position of the lights a tiny bit so the value will no longer be exactly 1

Ok, I deleted my entertainement areas, then update to 2022.2.8 and its working fine now. Thanks for your help. Will wait for the next update to recreate my areas.

OK, thanks for reporting back and nice that the workaround is working now.
Please keep the issue open for others as reference. We'll close this issue once the fix is released. Thanks!


Allow users to undo keybindings they set in the Keyboard Shortcuts tab with Ctrl + Z.

Allow users to undo keybindings they set in the Keyboard Shortcuts tab with Ctrl + Z.

Do you mean following?

Open Keyboard Shortcuts Editor
Set a keybinding
Press ctrl+z
Remove the assigned keybinding?


MODE SELECT crashes in various scenarios

This ticket refers to commit a23d642 (current develop branch).
While testing new tools for the PiSCSI board I stumbled upon some bugs in the mode page handling.
When you try to change the sector size of a CD-ROM drive with no medium inserted piscsi crashes with a assertion.
When a medium is present, changing the sector size to 512 bytes works fine with MODE SELECT(6) and these command parameters for the DATA OUT phase:
00:00:00:08:00:00:00:00:00:00:02:00

When doing the same with MODE SELECT(10) piscsi crashes with an exception. The command used was equivalent to the MODE SELECT(6) above, i.e. it should have the same effect and work. These were the command parameters for DATA OUT:
00:00:00:00:00:00:00:08:00:00:00:00:00:00:02:00

MODE SELECT(6) crashes with an exception when trying to set a valid but unsupported sector size:
00:00:00:08:00:00:00:00:00:00:04:00

It also crashes when the requested sector size is not a multiple of 512:
00:00:00:08:00:00:00:00:00:01:02:01

I also tested with a real CD-ROM drive, where everything worked as expected, including the error handling. Note that the real drive does not require the SP bit to be set when changing the sector size, whereas piscsi requires this bit to be set.

@uweseimet Thanks for reporting the bugs / corner cases. What do you recommend is a proper handling of the two latter invalid inputs? Ignore the command, log a warning and continue?
@kkaempf This has to do with the CD-ROM sector size changing functionality. Several scenarios that we didn't consider. Would you be able to spend a little time adding some more logic here?

@rdmark Regarding the invalid input the commands should be rejected with INVALID FIELD IN PARAMETER LIST. Also see the SCSI specification. This is also what a real drive does.

By the way, lauching piscsi with the requested sector size for the respective drive (-b option) would quite likely also work, without any code change in the v23.11.01 codebase.

@kkaempf I plan to tag a new stable piscsi software release in 2 weeks time. Do you think you'll have a chance to look at this before that?

Are you running your tests with assertions enabled?

Are you running your tests with assertions enabled?

🤦🏻 - I only compiled the test/*cpp files with DEBUG.
Now I see the "no medium in drive" problem - it's the DiskCache initializer checking blocks > 0 - to be expected.
I'll skip DiskCache initialization if no medium (== image size) is known.

I re-opened the ticket, because of the 3 bugs in mentioned only 1 appears to have been addressed ...

I'm unable to reproduce the other bugs and would need some advice.

I just double checked and can reproduce everything with the latest develop branch. With these data for the DATA IN phase of MODE SELECT(6):
00:00:00:08:00:00:00:00:00:00:04:00

the result is:
terminate called after throwing an instance of 'io_exception'
  what():  Invalid sector size of 1024 byte(s)
Aborted

With:
00:00:00:08:00:00:00:00:00:01:02:01

terminate called after throwing an instance of 'io_exception'
  what():  Invalid sector size of 513 byte(s)
Aborted

Further, from my tests it looks as if when using MODE SELECT10) instad of MODE SELECT(6) the sector size cannot be changed at all, even though MODE SELECT(10) must work like MODE SELECT(6). It is just an extended version.
Regarding the SP bit the difference between a real drive and piscsi is a bit different than I initially said, but my note on this is essentially the same: The real drive changes the sector size regardless of whether the SP bit is set or not. piscsi reports an error if the SP bit is set. Is this difference intended?

I'm still confused.
Running scsicd_test on latest develop gives me
[2024-02-27 13:46:20.075] [warning] (ID -1) - Unknown MODE SELECT page code: $00
[2024-02-27 13:46:20.075] [debug] (ID -1) - Changing sector size from 2048 to 512
[2024-02-27 13:46:20.075] [warning] (ID -1) - Unknown MODE SELECT page code: $00
[2024-02-27 13:46:20.075] [debug] (ID -1) - Changing sector size from 512 to 2048
[2024-02-27 13:46:20.075] [warning] (ID -1) - Unknown MODE SELECT page code: $00
[2024-02-27 13:46:20.075] [debug] (ID -1) - Changing sector size from 2048 to 512
[2024-02-27 13:46:20.075] [warning] (ID -1) - Unknown MODE SELECT page code: $00
[2024-02-27 13:46:20.075] [debug] (ID -1) - Changing sector size from 512 to 1024
[2024-02-27 13:46:20.075] [debug] (ID -1) - Changing sector size from 512 to 768
[2024-02-27 13:46:20.075] [debug] (ID -1) - Changing sector size from 2048 to 512
[2024-02-27 13:46:20.075] [warning] (ID -1) - Unknown MODE SELECT page code: $00

There are three 2048 to 512 byte switches. The first is via ModeSelect6, the second via ModeSelect10, and the last via ModeSelect6 again ('no media' case).

For the "non-multiple of 512 bytes" cases I'd assume that you don't want an exception raised ?!

Yes, I can confirm that the MODE SELECT(10) case indeed appears to be working. But just like MODE SELECT(6) it crashes with the sector sizes from my examples above:
2024-02-27 20:23:52.468] [debug] (ID:LUN 0:0) - Changing sector size from 2048 to 1024
terminate called after throwing an instance of 'io_exception'
  what():  Invalid sector size of 1024 byte(s)
Aborted

My bug reports are not about what I want, but more about crashes and what the SCSI spec says. piscsi terminating is something nobody would want ;-).
I have provided all MODE SELECT data I have used to reproduce the crashes (v23.11.01 does not crash), and I have shown the piscsi crash messages, I'm sorry, but I don't think I can do more than that. Just use the same data for your tests and I am sure you will see the same issues.

@kkaempf So ultimately I think it's up to us to decide if we want to treat "non-multiple of 512 bytes" cases as exceptions, or to log a warning and proceed. I'm ok either way. It seems like a highly unlikely case in real life. And if it turns out users encounter this scenario, they are surely to report it if we keep the exception.
Please let me know what you think is the best solution.

@kkaempf It should react to unsupported command parameters according to the SCSI specification. A crash (an uncaught exception is a crash, just like a segmentation fault) means something is broken. Does you real hard drive or CD-ROM drive crash when it receives a command or parameters it does not support? Do you have to switch your drives off and on in such a case? I bet not ;-).
@rdmark I don't think it is a good idea to define an uncaught exception, i.e. a crash, as a valid reaction on a SCSI command. But If you are OK with a change for an exotic platform triggering several reproducable bugs and crashing piscsi, good luck!

@kkaempf Wouldn't you say that this scenario corresponds to condition d) found under the 8.2.8 MODE SELECT(6) command section from the SCSI spec?

The target shall terminate the MODE SELECT command with CHECK CONDITION status, set the sense key to
ILLEGAL REQUEST, set the additional sense code to INVALID FIELD IN PARAMETER LIST, and shall not change
any mode parameters for the following conditions:
a) If the initiator sets any field that is reported as not changeable by the target to a value other than its current
value.
b) If the initiator sets any field in the mode parameter header or block descriptor(s) to an unsupported value.
c) If an initiator sends a mode page with a page length not equal to the page length returned by the MODE SENSE
command for that page.
d) If the initiator sends a unsupported value for a mode parameter and rounding is not implemented for that mode
parameter.
e) If the initiator sets any reserved field in the mode parameter list to a non-zero value.

Right now we allow the change to an invalid sector size and then bail out with this assertion:
https://github.com/PiSCSI/piscsi/blob/dd9a3296d4f0060b923bf1297c5849cfae297333/cpp/devices/disk.cpp#L699
So I think we should detect the invalid sector size earlier and, for instance, add a check in SCSICD::ModeSelect before calling SetSectorSizeInBytes(), along the lines of...
if (!GetSupportedSectorSizes().contains(size_in_bytes)) {
    throw scsi_exception(sense_key::illegal_request, asc::invalid_field_in_parameter_list);
}

What do you think?

Whole changeset reverted with https://github.com/PiSCSI/piscsi/pull/1451


Shortcut

Description of the new feature / enhancement
Shortcut
Custom Shortcut
Window Move Left --------------- LeftWindows + Keypad4
(There is no preset window on the left, but there is a screen that spans across multiple screens)
Window Move Right --------------- LeftWindows + Keypad6
Window Move Up --------------- LeftWindows + Keypad8
Window Move Down --------------- LeftWindows + Keypad2
Window Arrange All Windows Automatically/Restore --------------- LeftWindows + Keypad7
(Arrange preset layout display windows in sequence)
Window Maximize/Minimize --------------- LeftWindows + Keypad9
(Maximize according to the display screen)
Scenario when this would be used?
Shortcut
Custom Shortcut
Window Move Left --------------- LeftWindows + Keypad4
(There is no preset window on the left, but there is a screen that spans across multiple screens)
Window Move Right --------------- LeftWindows + Keypad6
Window Move Up --------------- LeftWindows + Keypad8
Window Move Down --------------- LeftWindows + Keypad2
Window Arrange All Windows Automatically/Restore --------------- LeftWindows + Keypad7
(Arrange preset layout display windows in sequence)
Window Maximize/Minimize --------------- LeftWindows + Keypad9
(Maximize according to the display screen)
Supporting information
No response

Window+Up arrow
Unable to use FancyZones' Window+Up arrow to move the window upwards
If it can be used, it is not possible to maximize the window using "Window+Up arrow"
Shortcut conflict
FancyZones hopes to increase
Display all windows (one click arrangement of all windows, preset windows)
And there are no shortcut keys

Your english is a bit hard to read.....please use complete sentences.
Have you tried disabling FancyZones?


Installer bug - wrong apex package installed

If you pip install torchscale then via requirements it also installs apex from pypi. However, apex on pypi is not Nvidia's apex, but is an unrelated project with many deps. As a result, many other additional deps are also pulled into the installation.
Nvidia's apex is currently not pip-installable, and therefore it should not be listed as a requirement.
cc @shumingma

Thanks for catching that! I've uploaded a fix to torchscale==0.1.2 by removing apex from the installation dependency and making it an optional package.

Many thanks. Closing this.


Company branding - does it require AAD Premium?

Hi,
The below information from this article conflicts with the information in the article „Add branding to your organization's Azure Active Directory sign-in page” (https://docs.microsoft.com/en-us/microsoft-365/admin/setup/customize-sign-in-page?view=o365-worldwide), which states you can also use company branding with the Azure Active Directory included with Microsoft 365 for business, Microsoft Dynamics CRM Online, Enterprise Mobility Suite, or other Microsoft services:
"Adding custom branding requires you to have Azure Active Directory Premium 1 or Premium 2 licenses. For more information about licensing and editions, see Sign up for Azure AD Premium"
Best regards,
Silvia

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

ID: 72f2e675-436d-dcd4-fa27-49408b201158
Version Independent ID: 4d821732-fc10-f4a6-3c7d-77bce75299d7
Content: Add branding to your organization's sign-in page - Azure AD
Content Source: articles/active-directory/fundamentals/customize-branding.md
Service: active-directory
Sub-service: fundamentals
GitHub Login: @ajburnle
Microsoft Alias: ajburnle

@SilviaEid Thanks for reaching out. We will investigate and update the thread accordingly.

Hello @SilviaEid ,
Thanks for your feedback.
Azure AD company branding feature available in three editions— Office 365 apps ( includes any of paid subscriptions like O365 E1, E3, E5, F3, Microsoft Dynamics CRM Online, Enterprise Mobility Suite), Premium P1, and Premium P2.
More information, refer this article: https://azure.microsoft.com/en-in/pricing/details/active-directory/
If you have any of these license then you can customize Azure AD company branding. We will update Doc with above information.

We will now proceed with closure of this thread. If you have further questions, please tag me in the comments and I will gladly continue the conversation.
@please-close


Pre-prod review for code, UI, and content

We need to conduct a final review for code, UI, and content before we do a deployment. This will likely surface other issues.

Issues:

https://github.com/tiki/website/issues/22
https://github.com/tiki/website/issues/23
https://github.com/tiki/website/issues/24
https://github.com/tiki/website/issues/25
https://github.com/tiki/website/issues/26
https://github.com/tiki/website/issues/27

Because my estimates got stuck, I was not able to see the UI that is shown after so this list is definitely incomplete.


Spin application keep restarting in AKS cluster

I deployed the simple-app sample to an AKS cluster, and is observing that two of four pods are continuously restarting.
The cluster consists of four nodes (two are amd62, two are arm64). The restarts are happening on one node on each architecture. All nodes run Ubuntu 22.04.
I've obtained the containerd logs (using journalctl -u containerd), and attached here. Let me know what else could be needed to troubleshoot this problem. I still have the cluster running, and the app deployed.
containerd-restarting.log
containerd-not-restarting.log

invalid metric type for ...

This line from the end of the log file is interesting and it seems to come from containerd metric collection logic: https://github.com/containerd/containerd/blob/2f807b606a58020174e1aecfdec63810201e05bc/core/metrics/cgroups/v2/metrics.go#L151
Could you please verify that your nodes are on cgroups v2 by running mount | grep cgroup?

From the node where the app is restarting:
cgroup2 on /sys/fs/cgroup type cgroup2 (rw,nosuid,nodev,noexec,relatime)
cgroup on /run/containerd/io.containerd.runtime.v2.task/k8s.io/79f773f2e2b1659aa4f3d7bb1d81a48666d3add27fce0b1b5d805a85d1021191/rootfs/sys/fs/cgroup type cgroup2 (ro,nosuid,nodev,noexec,relatime)

From the node where the app is not restarting:
cgroup2 on /sys/fs/cgroup type cgroup2 (rw,nosuid,nodev,noexec,relatime)
cgroup on /run/containerd/io.containerd.runtime.v2.task/k8s.io/7f139ffd06a9c6a7d95d1e6c019b537d90232ee53b0d4f8417598669ad76db9d/rootfs/sys/fs/cgroup type cgroup2 (ro,nosuid,nodev,noexec,relatime)

I am thinking there might be some metircs that we report at the shim level where they are not recognized by the containerd.

I just deployed another application to the cluster (to rule out the simple sample app).
This one also restarts, but only of one of the nodes, which the other app was restrating on. I've also noted that the restrats have not happenede for the last 11 hours on the node, where the old app used to restart, and this node does not restart the new app. So I don't know if something "fell into place" on that node. (Let me know if this becomes too confusing to keep track on).
However current restart stats for the two apps across the four nodes:
NAME                                                    READY   STATUS             RESTARTS        AGE   IP             NODE                                NOMINATED NODE   READINESS GATES
simple-spinapp-56687588d9-7d8zf                         1/1     Running            260 (11h ago)   38h   <IP_ADDRESS>    aks-nodepool1-31969921-vmss000001   <none>           <none>
simple-spinapp-56687588d9-s5g95                         1/1     Running            364 (21m ago)   38h   <IP_ADDRESS>   aks-nodepool2-19133108-vmss000001   <none>           <none>
simple-spinapp-56687588d9-xffsh                         1/1     Running            0               38h   <IP_ADDRESS>     aks-nodepool2-19133108-vmss000000   <none>           <none>
simple-spinapp-56687588d9-xh8l7                         1/1     Running            0               38h   <IP_ADDRESS>    aks-nodepool1-31969921-vmss000000   <none>           <none>
spinkube-live-mikkel-75cc76c678-4r7rl                   0/1     CrashLoopBackOff   7 (2m25s ago)   38m   <IP_ADDRESS>   aks-nodepool2-19133108-vmss000001   <none>           <none>
spinkube-live-mikkel-75cc76c678-f7476                   1/1     Running            0               38m   <IP_ADDRESS>    aks-nodepool1-31969921-vmss000001   <none>           <none>
spinkube-live-mikkel-75cc76c678-fm8dd                   1/1     Running            0               38m   <IP_ADDRESS>    aks-nodepool1-31969921-vmss000000   <none>           <none>
spinkube-live-mikkel-75cc76c678-j5pvj                   1/1     Running            0               38m   <IP_ADDRESS>     aks-nodepool2-19133108-vmss000000   <none>           <none>

Without having done anything, the other nodes, which previously also restarted the spin-sample, has started restarting both the simple-sample and the new app:
NAME                                    READY   STATUS             RESTARTS          AGE     IP             NODE                                NOMINATED NODE   READINESS GATES
simple-spinapp-56687588d9-7d8zf         1/1     Running            307 (6m28s ago)   43h     <IP_ADDRESS>   aks-nodepool1-31969921-vmss000001   <none>           <none>
simple-spinapp-56687588d9-s5g95         0/1     CrashLoopBackOff   407 (80s ago)     43h     <IP_ADDRESS>   aks-nodepool2-19133108-vmss000001   <none>           <none>
simple-spinapp-56687588d9-xffsh         1/1     Running            0                 43h     <IP_ADDRESS>     aks-nodepool2-19133108-vmss000000   <none>           <none>
simple-spinapp-56687588d9-xh8l7         1/1     Running            0                 43h     <IP_ADDRESS>    aks-nodepool1-31969921-vmss000000   <none>           <none>
spinkube-live-mikkel-75cc76c678-4r7rl   1/1     Running            51 (39s ago)      5h19m   <IP_ADDRESS>   aks-nodepool2-19133108-vmss000001   <none>           <none>
spinkube-live-mikkel-75cc76c678-f7476   0/1     CrashLoopBackOff   47 (4m48s ago)    5h19m   <IP_ADDRESS>   aks-nodepool1-31969921-vmss000001   <none>           <none>
spinkube-live-mikkel-75cc76c678-fm8dd   1/1     Running            0                 5h19m   <IP_ADDRESS>    aks-nodepool1-31969921-vmss000000   <none>           <none>
spinkube-live-mikkel-75cc76c678-j5pvj   1/1     Running            0                 5h19m   <IP_ADDRESS>     aks-nodepool2-19133108-vmss000000   <none>           <none>

Just added a third nodepool using Mariner / Azue Linux, both nodes in that node pool are restarting the apps.
NAME                                    READY   STATUS             RESTARTS         AGE     IP             NODE                                NOMINATED NODE   READINESS GATES
simple-spinapp-56687588d9-7d8zf         1/1     Running            307 (23m ago)    43h     <IP_ADDRESS>   aks-nodepool1-31969921-vmss000001   <none>           <none>
simple-spinapp-56687588d9-jwp9b         1/1     Running            1 (2m1s ago)     2m34s   <IP_ADDRESS>     aks-nodepool3-26262681-vmss000000   <none>           <none>
simple-spinapp-56687588d9-s5g95         0/1     CrashLoopBackOff   409 (2m6s ago)   43h     <IP_ADDRESS>   aks-nodepool2-19133108-vmss000001   <none>           <none>
simple-spinapp-56687588d9-wdt9b         1/1     Running            1 (2m1s ago)     2m34s   <IP_ADDRESS>     aks-nodepool3-26262681-vmss000001   <none>           <none>
simple-spinapp-56687588d9-xffsh         1/1     Running            0                43h     <IP_ADDRESS>     aks-nodepool2-19133108-vmss000000   <none>           <none>
simple-spinapp-56687588d9-xh8l7         1/1     Running            0                43h     <IP_ADDRESS>    aks-nodepool1-31969921-vmss000000   <none>           <none>
spinkube-live-mikkel-75cc76c678-4r7rl   1/1     Running            55 (4m46s ago)   5h37m   <IP_ADDRESS>   aks-nodepool2-19133108-vmss000001   <none>           <none>
spinkube-live-mikkel-75cc76c678-dx8zv   1/1     Running            2 (50s ago)      2m23s   <IP_ADDRESS>     aks-nodepool3-26262681-vmss000001   <none>           <none>
spinkube-live-mikkel-75cc76c678-f7476   0/1     CrashLoopBackOff   50 (3m19s ago)   5h37m   <IP_ADDRESS>   aks-nodepool1-31969921-vmss000001   <none>           <none>
spinkube-live-mikkel-75cc76c678-fm8dd   1/1     Running            0                5h37m   <IP_ADDRESS>    aks-nodepool1-31969921-vmss000000   <none>           <none>
spinkube-live-mikkel-75cc76c678-j5pvj   1/1     Running            0                5h37m   <IP_ADDRESS>     aks-nodepool2-19133108-vmss000000   <none>           <none>
spinkube-live-mikkel-75cc76c678-q88q5   1/1     Running            1 (50s ago)      2m23s   <IP_ADDRESS>     aks-nodepool3-26262681-vmss000000   <none>           <none>

I checked on one of the node where this issue is happening and observed that it is returning cgroups v1 metrics:
root@aks-nodepool1-31969921-vmss000001:/# /tmp/ttrpc-client /run/containerd/f1e518aba0e9dccf.sock 94c1433b03b96c984b1bf15773e409682ae64b6d697f7392c1bb37256f492bca

type_url:"io.containerd.cgroups.v1.Metrics"  value:"\x12\x03\x08\xee\x01\x1a\x16\n\x12\x08\xfdܙ\xae\n\x10\xe4\xfd\xa0\xf9\x03\x18\x98\xdf\xf8\xb4\x06\x12\x00\"\t\x8a\x02\x06\x10\x80\xe0\x9c\xff\x02"

trying to find out why that is the case.

another data point, but i am unable to make anything out of it right now.  (i was expecting crictl to fail as well)
root@aks-nodepool1-31969921-vmss000001:/# crictl ps | grep spin
01748f8974627       bca3c4d784227       59 minutes ago      Running             simple-spinapp          323                 94c1433b03b96       simple-spinapp-56687588d9-7d8zf
f4ff19248a4ef       f99fcd5dfb557       About an hour ago   Running             spinkube-live-mikkel    62                  b0540e08d14db       spinkube-live-mikkel-75cc76c678-f7476

crictl stats f4ff19248a4ef
CONTAINER           NAME                   CPU %               MEM                 DISK                INODES
f4ff19248a4ef       spinkube-live-mikkel   0.00                33.43MB             7.881MB             23

crictl stats 01748f8974627
CONTAINER           NAME                CPU %               MEM                 DISK                INODES
01748f8974627       simple-spinapp      0.00                10.28MB             90.11kB             26

i added some debug logging to shim and used that on the node having crashes:
Apr 23 17:05:22 aks-nodepool1-31969921-vmss000003 containerd[212274]: time="2024-04-23T17:05:22.943002063Z" level=info msg="stats: StatsRequest { id: "2008530c02eec982e25a6ebe9f5e005b0704b4ed8546f7c7b5f3e104f9c17f4
0", special_fields: SpecialFields { unknown_fields: UnknownFields { fields: None }, cached_size: CachedSize { size: 0 } } }"

Apr 23 17:05:22 aks-nodepool1-31969921-vmss000003 containerd[212274]: time="2024-04-23T17:05:22.945109028Z" level=info msg="stats resp: StatsResponse { stats: MessageField(Some(Any { type_url: "io.containerd.cgroup
s.v1.Metrics", value: [18, 2, 8, 5, 26, 16, 10, 12, 8, 159, 181, 14, 16, 154, 197, 7, 24, 133, 240, 6, 18, 0, 34, 8, 138, 2, 5, 16, 128, 128, 169, 16], special_fields: SpecialFields { unknown_fields: UnknownFields
{ fields: None }, cached_size: CachedSize { size: 0 } } })), special_fields: SpecialFields { unknown_fields: UnknownFields { fields: None }, cached_size: CachedSize { size: 0 } } }"

so it seems like it is some how receiving cgroup v1 metrics.


Practical guide to logging in to docker store and consuming images - please

File: docker-store/faq.md, CC @johndmulhausen
Please could you spell out specifically how I consume a specific image, e.g. jre8?
I have tried various combinations of:
docker login -u adrianblakey -p xxxxxxx https://store.docker.com
docker pull store/oracle/serverjre:8
docker pull  https://store.docker.com/store/oracle/serverjre:8
docker pull  store.docker.com/store/oracle/serverjre:8

TY - that works.
One more question - how do I make this a remote repo in Artifactory?
Artifactory is expecting the URL of a repo to which it can connect - so is
there a "hidden" , "default" URL that the docker client connects to when
you run docker login?
Adrian Blakey
<EMAIL_ADDRESS>Skype: adrianblakey
(510) 747-8057
On Tue, Nov 14, 2017 at 3:48 PM, Joao Fernandes<EMAIL_ADDRESS>wrote:

It seems that you got the login command wrong.
First you need to checkout the product at https://store.docker.com/
images/oracle-serverjre-8. Then:
docker login -u adrianblakey
you'll be prompted for the password
docker pull store/oracle/serverjre:8
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/docker/docker.github.io/issues/5307#issuecomment-344438803,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKYtnC-tPrEkD5AJJOwpBb-kXTV1enUks5s2ibHgaJpZM4QdvVw
.

Can you try using docker.io/store/oracle/serverjre:8 and see if it works?
I've filed an internal issues against Docker Store so that we can improve the login/pull experience. I don't think there's any point in updating the docs, since no one will search there on how to overcome this problem.
For that, I'm closing this issue, but feel free to comment.


Update pt-help.txt

added missing options + fixed weird grammar

How do I add comments to all files changed since a certain commit?
that's what just see on stackoverflow
and i see that the "code" have been changer and only the one who write the code

Thats'h hard wtiting on smartphone and anderdrand remembrer old scool commit that i learnd by thé past

Thanks for the
little stress in m'y life ,diving in New advanture


VS Code, Python, Linting, mpl_toolkits.mplot3d and cm.inferno(G)

Environment data

VS Code version: 1.25.1
Extension version (available under the Extensions sidebar): ?
OS and version: MacOS Sierra 10.12.6
Python version (& distribution if applicable, e.g. Anaconda): See below.
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A
Relevant/affected Python packages and their versions:
conda version 4.5.8
condo-build-version 3.12.0
python version: 3.6.6.final.0

Actual behaviour
I am new to vs code and python so please forgive any errors.   I have posted a previous issue and somebody asked me to add a new issue here.   The previous post was:
https://github.com/Microsoft/vscode-docs/issues/1757
I can run the following code on visual studio code and it appears to work, however there are 2 problems:

With the import statement for mpl_toolkits.mplot3d, it is saying:

Unable to resolve 'mpl_toolkits.mplot3d', intellisence may be missing for this module

The piece of code that contains facecolors=cm.inferno(G) is highlighting the cm with a red underscore and saying:

[pylint] E1101: Module 'matplotlib.cm' has no inferno member.

Expected  behavior
I believe this is a problem with linting.   The code runs and works on both vs code and spyder, but looks horrible because of the red formatting.
Steps to reproduce:

Create a python file and copy it in:

# SurfacePlot3D.py
import matplotlib.pyplot as plt
from matplotlib import cm
from mpl_toolkits.mplot3d import Axes3D
import numpy as np

X, Y = np.meshgrid(np.arange(-100, 100), np.arange(-100,100))
Z = X ** 2 + Y ** 2

G_x, G_y = np.gradient(Z)
G = np.sqrt(G_x2 + G_y2)
G = G / G.max()

#Figure contains all of the objects in the plot
fig = plt.figure()
#Get the axes of the plot
ax = Axes3D(fig)
#THIS DOESN'T WORK IN VS Code, but works in Spyder
#Uncomment the following line to plot the surface.
ax.plot_surface(X,Y,Z, facecolors=cm.inferno(G))
#Show the Plot
plt.show()

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)
##########Linting Output - pylint##########
************* Module SurfacePlot3D
20,34,error,E1101:Module 'matplotlib.cm' has no 'inferno' member

------------------------------------------------------------------
Your code has been rated at 6.15/10 (previous run: 6.15/10, +0.00)

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help)
TMSyntax.ts:47 Overwriting grammar scope name to file mapping for scope source.python.
Old grammar file: file:///Applications/Development/Visual%20Studio%20Code.app/Contents/Resources/app/extensions/python/syntaxes/MagicPython.tmLanguage.json.
New grammar file: file:///Users/mark/.vscode/extensions/magicstack.magicpython-1.0.12/grammars/MagicPython.tmLanguage

Hi ericsnowcurrently,
I have many extensions enabled, but if you mean the Python one, then, yes it is 2018.7.1
Sorry, I thought that I included an image of my extensions, but it has not uploaded properly.
Thanks
Mark

Oh and yes, the code appears to be working fine.   Intellisense and linter are reporting errors that they shouldn't be.
Thanks
Mark

How are you running the code?  Note that if you have not selected the interpreter from your conda environment then the linter and intellisense are not going to be able to find packages that you installed in your environment, using the global environment again.

Hi ericsnowcurrently,
When I run the code, I select "Anaconda custom (64-bit) Python 3.6.6 (base)" in the status bar and then right click on the code and select "Run Python File in Terminal".   It then brings up the desired graph.
However when I am editing, there is a message with mpl_toolkits.mplot3d, my file is marked red and cm.inferno(G) is underlined with red and a message saying that matplotlib.cm has no inferno member.
Is there something I can do to refresh the intelligence etc when I have anaconda selected ?
Thanks
Mark

@MarkyMark1000 there are no images; please upload them through the GitHub interface.
And what is your python.pythonPath set to? The base part suggests you are not using a conda environment but instead your Anaconda install directly which could explain why none of the tools can find the libraries you are referencing.

Hi Brett,
Within my MyPython workspace, I have a MyPython.code-workspace file which contains "python.pythonPath": "/anaconda3/bin/python"
Is this the correct way to get python.pythonPath ?
The path of python.pythonPath changes whenever I change the status bar to a different version of python.   The current status bar is set to "Anaconda Custom (64-bit) Python 3.6.6 (base)", with python.pythonPath of "/anaconda3/bin/python".
I have tried to upload the images before, but they were deleted or failed to upload.   I will attach a copy of the files to this email and then attempt to upload them again in GitHub.
Thanks for your help.
Mark
On Friday, 3 August 2018, 19:10:28 BST, Brett Cannon<EMAIL_ADDRESS>wrote:
@MarkyMark1000 there are no images; please upload them through the GitHub interface.
And what is your python.pythonPath set to? The base part suggests you are not using a conda environment but instead your Anaconda install directly which could explain why none of the tools can find the libraries you are referencing.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I will attempt to upload 2 images.
Thanks
Mark

Hi brettcannon,
I'm not quite sure what you mean with respect to the (base), so I will try to explain what I have done:
I have tried using 'Anaconda Custom (64-bit) Python 3.6.6 (base)' and the pythonPath changes to:
"python.pythonPath": "/anaconda3/bin/python"
I have also tried using 'Python 3.6.6 :: Anaconda custom (64-bit) and the pythonPath changes to:
"python.pythonPath": "/anaconda3/bin/python3"
However I doesn't really matter which one of these two I use as pylint is still saying that matplotlib.cm has no inferno member.
[I am quite new to python, so perhaps I am missing something.   Any explanation, help or suggestions would really be appreciated.]
When I run /anaconda3/bin/python -c "import mpl_toolkits.mplot3d" it doesn't really do anything, it just moves onto the next line.   I have tried this using /bin/python3 with the same result.
I was never under the impression that there was a problem with the actual import because the code executes correctly and builds the graph.   I thought the problem was with pylint, intellisence or some other feature of VS Code.   Perhaps I have miss-understood this.   If you could help explain, I would be very grateful.
Thank you for all of your help.
Mark

Hi brettcannon,
Sorry I have been a bit silly with my previous comment and should have checked the code.   It appears that there is no longer green highlighting on the following code and the corresponding message no longer appears:
original message:   Unable to resolve 'mpl_toolkits.mplot3d', intellisence may be missing for ...
code:   from mpl_toolkits.mplot3d import Axes3D
I must admit that I have recently installed the insiders version of VS Code, so perhaps that has helped.
Unfortunately the following error message is still being reported, even though the graph builds successfully:
[pylint] E1101:Module 'matplotlib.cm' has no 'inferno' member.
Thanks
Mark

I will attempt to show an image of VS Code Insiders (originally I was using VS Code).   It no longer has the mplot3d issue, but still displays the following error:
[pylint] E1101:Module 'matplotlib.cm' has no 'inferno' member.
The graph can however still be built when the code is run:

Hi Brettcannon,
Please note that I am running this on a mac, not windows.
As far as I can tell, the code executes correctly on a mac, but the issue with pylint and the following still remains:
[pylint] E1101:Module 'matplotlib.cm' has no 'inferno' member.
Am I in the correct area to report problems with pylint ?   Should this be added to another blog ?
Regards
Mark

@MarkyMark1000 Yep, I realize you're on macOS. My comment was meant to convey that I can reproduce the mpl_toolkits problem but not your other import problem.
And the Pylint issue can be reported to Pylint if you want, but that we have nothing to do with the cause of the message.

https://github.com/Microsoft/PTVS/issues/4138


Page updated

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/log-and-display-processed-messages-sipsnoop-sample.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Make stars in surface view clickable and open edit modal

Apparently needs Canvas Update:
https://docs.rs/iced/latest/iced/widget/canvas/trait.Program.html#method.update
The Game of Life Example does that.


How to hide Sidebar title?

Hello! I'm trying to hide the sidebar title but can't find the answer. I've seen the font size settings but not how to hide it completely.
Could anyone tell me how to achieve it?

Thank you very much

"apc.stylesheet": {
  ".monaco-workbench .part > .title > .title-label": "display: none"
}

TIP: You can use builtin developer-tools get id/ class of the element you want to customize.


Fix uninitialised bias parameters in RelPositionMultiHeadedAttention

Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos, doc improvements)
[x] Did you read the contributor guideline?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

What does this PR do?
Fixes #4622
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Thanks, bot!


missing Attributes with WLED 0.10.0

Describe the bug
3 attributes are missing
wled.0 | 2020-07-19 19:08:59.081 | warn | (15662) State attribute definition missing for + lor
wled.0 | 2020-07-19 19:08:58.914 | warn | (15662) State attribute definition missing for + lip
wled.0 | 2020-07-19 19:08:58.906 | warn | (15662) State attribute definition missing for + lm
To Reproduce
Update yout wled to 0.10.0
Expected behavior
A clear and concise description of what you expected to happen.

https://github.com/iobroker-community-adapters/ioBroker.wled/commit/f0c22ebabff53c48c707b210a6c35e94b968bec3


Global restructuring: Item subtree

The Item subtree currently has the following top level things:
Item

Object
3D shape
2D shape
Pattern
Face
Symbolic
Natural scene
Drawing
Film clip
IAPS [International Affective Picture System]
IADS  [International Affective Digital Sounds]
SAM  [The Self-Assessment Manikin]

This is an odd collection of top level elements.  Further, body parts appear under Participant. I think Body parts should be moved to the Item tree.
Can we do better on top-level categories?

Per Nima's suggestion I found the hierarchical ontology of dbpedia: http://mappings.dbpedia.org/server/ontology/classes/
Some relevancy I noticed from having a first pass at it:

Person: Thing > Agent > Person (their hierarchy seems to only include the Person's "Role". We also want to include "Body parts". Maybe those two can be become two categories under Person, or we might want to change "Person" to "Human")
Multimedia: Thing > Work
Vehicle: Thing > MeanOfTransportation
Building: Thing > Place > ArchitecturalStructure > Building
Plant & Animal: Thing > Species > Eukaryote (interestingly scientific)


[question] Where is the data stored?

Where does drm store the data queried from the registry?
Can I create a host-volume there to make it persistent?

Hi, sorry for the delay. It is stored in memory at the moment. I originally meant this to be easily torn up/down and writing to disk seemed like overkill. I'll look into alternatives.


Scheduler fails to start when started according to docker compose example in the docs

In what area(s)?
/area runtime
What version of Dapr?
1.14.4
Expected Behavior
Scheduler service starts sucessfully
Actual Behavior
Scheduler fails to start with error

{"level":"panic","ts":"2024-10-17T14:42:16.805372Z","caller":"backend/backend.go:189","msg":"failed to open database","path":"data/default-dapr-scheduler-server-0/member/snap/db","error":"write data/default-dapr-scheduler-server-0/member/snap/db: no space left on device","stacktrace":"go.etcd.io/etcd/server/v3/mvcc/backend.newBackend\n\t/home/runner/go/pkg/mod/go.etcd.io/etcd/server/v3@v3.5.13/mvcc/backend/backend.go:189\ngo.etcd.io/etcd/server/v3/mvcc/backend.New\n\t/home/runner/go/pkg/mod/go.etcd.io/etcd/server/v3@v3.5.13/mvcc/backend/backend.go:163\ngo.etcd.io/etcd/server/v3/etcdserver.newBackend\n\t/home/runner/go/pkg/mod/go.etcd.io/etcd/server/v3@v3.5.13/etcdserver/backend.go:55\ngo.etcd.io/etcd/server/v3/etcdserver.openBackend.func1\n\t/home/runner/go/pkg/mod/go.etcd.io/etcd/server/v3@v3.5.13/etcdserver/backend.go:76"}

Steps to Reproduce the Problem
Docker compose:
services:
  scheduler:
    image: "daprio/dapr:1.14.4"
    command: ["./scheduler", "--port", "50007"]
    ports:
      - "50007:50007"
    # WARNING - This is a tmpfs volume, your state will not be persisted across restarts
    volumes:
    - type: tmpfs
      target: /data
      tmpfs:
        size: "10000"

The above is a snippet from the official docs. I copied the sample from there, removed everything but the scheduler and pinned the image version.
With the compose shown above, run docker compose up scheduler to reproduce the problem.
Release Note

RELEASE NOTE:

Doing this worked for me.
command: ["./scheduler", "--port", "50007", "--etcd-data-dir=/var/lock/dapr/scheduler"]

Doing this worked for me.
command: ["./scheduler", "--port", "50007", "--etcd-data-dir=/var/lock/dapr/scheduler"]

Indeed, that works. So I guess this is a problem with the defaults in the container. Or, if the above is the desired way of calling it (which I doubt), it is at least missing in the docs that I referenced.

Not stale


Editing different entities on different tabs

Because the system stores the entity being edited on a SessionAttribute, if I edit two different vets on two different browser tabs, I can update the wrong vet if I post the first form instead of the last one opened.

Thanks for pointing this out.
This is a common problem and you can find an elegant solution to it here.
Petclinic aims to propose a fairly simple architecture for people who are getting started with Spring so we will not implement that solution.
However, it reminds me that we should probably move our attributes to the request scope instead. This was highlighted by @jdubois a long time ago here. I'll create a separate issue for that.
Cheers,
Michael.


Add Darwin support for directx-shader-compiler

I can't test this so if someone would be kind enough to check it runs
I'd be grateful.
Built on https://github.com/NixOS/nixpkgs/pull/104377
Things done

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

[x] NixOS
[ ] macOS
[ ] 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"
[ ] 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.

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

1 package built:

directx-shader-compiler

I merged the other PR. Please rebase this one.

I merged the other PR. Please rebase this one.

Done. Why was this necessary?

Done. Why was this necessary?

If am not mistaken it could happen that the init commit could be doubled if the hash changed through rebasing or something. Just wanted to not risk this.

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

1 package built:

directx-shader-compiler

Result of nixpkgs-review pr 107188 run on x86_64-linux 1

1 package built:

directx-shader-compiler

Is it possible to merge this, please?

Is it possible to merge this, please?

Not unless you address my suggestion above

sorry, @prusnak I was certain that I left a comment before marking as resolved. I'll re-comment now

Thanks!
On Thu, Dec 24, 2020, 9:38 PM Pavol Rusnak<EMAIL_ADDRESS>wrote:

Merged #107188 https://github.com/NixOS/nixpkgs/pull/107188 into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/107188#event-4147471722, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGRJXAG2DOVKHDDVCY2CJ3SWM75HANCNFSM4VCB3YZQ
.


webapp:[('PKCS12 routines', 'PKCS12_parse', 'mac verify failure')] when attempting to upload new certificate

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az webapp config ssl upload
Errors:
[('PKCS12 routines', 'PKCS12_parse', 'mac verify failure')]
Traceback (most recent call last):
python3.6/site-packages/knack/cli.py, ln 206, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 603, in execute
    raise ex
cli/core/commands/__init__.py, ln 661, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 654, in _run_job
    six.reraise(*sys.exc_info())
...
python3.6/site-packages/OpenSSL/crypto.py, ln 3067, in load_pkcs12
    _raise_current_error()
python3.6/site-packages/OpenSSL/_util.py, ln 54, in exception_from_error_queue
    raise exception_type(errors)
OpenSSL.crypto.Error: [('PKCS12 routines', 'PKCS12_parse', 'mac verify failure')]

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az webapp config ssl upload --name {} --resource-group {} --certificate-file {} --certificate-password {} --query {}

Expected Behavior
Environment Summary
Linux-4.9.184-linuxkit-x86_64-with-debian-9.11
Python 3.6.5
Shell: bash

azure-cli 2.0.73

Additional Context

Can you run the command with --debug & share the details please excluding any sensitive info. Thank you.

@DaveSin can you please share more details here for debugging please, after trying this with the latest version of CLI? Thank you.
Reactivate when you add more details here.

Hi All, I found I had this error when my password contained special characters such as @ or $ etc...

Hi, I am still facing this issue and my password is just alphabets.

@anuradhaarao can you open a new issue if possible - since this was originally opened on a very old version of CLI so I'd like to get more details on this if possible. Thanks!

@panchagnula  - opened up a new one here : https://github.com/Azure/azure-cli/issues/1472.


Adds Timber ACF WP Blocks to README.md

Just adds a new Timber related tool, for creating Gutenberg Blocks.

Whoa, this is pretty cool! I'll give it a spin :-)

Yeah, congrats on the new project! Excited to try it out as well


x/sys/unix: cannot build for linux/{ppc64,ppc64le}

The package is still under curation, just as a reminder.
# golang.org/x/sys/unix
/swdev/src/golang.org/x/sys/unix/flock.go:18: undefined: Flock_t
/swdev/src/golang.org/x/sys/unix/sockcmsg_linux.go:14: undefined: Ucred
/swdev/src/golang.org/x/sys/unix/sockcmsg_linux.go:27: undefined: Ucred
/swdev/src/golang.org/x/sys/unix/sockcmsg_unix.go:42: undefined: Cmsghdr

I'm getting this error too. Isn't it supposed to be fixed two years later?

@hacdias, your checkout is old.
You should see as of today,
$ cd $GOPATH/src/golang.org/x/sys/unix && git rev-parse HEAD
0f826bdd13b500be0f1d4004938ad978fcc6031e

This was fixed by https://golang.org/cl/37943

Thanks. I'll do that.
On Sun, Jul 30, 2017, 4:58 PM Brad Fitzpatrick<EMAIL_ADDRESS>wrote:

@hacdias https://github.com/hacdias, your checkout is old.
You should see as of today,
$ cd $GOPATH/src/golang.org/x/sys/unix && git rev-parse HEAD
0f826bdd13b500be0f1d4004938ad978fcc6031e
This was fixed by https://golang.org/cl/37943
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/10087#issuecomment-318910918, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFMdsP0SKJOrwS7Ie8vaSJjUn96ZU7C8ks5sTKgdgaJpZM4Dpy0i
.


Test/autogen meas

Generate measurements
Separate optimal_request function

Codecov Report

Merging #3 (19e88af) into master (395bd11) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #3   +/-   ##
=======================================
  Coverage   42.85%   42.85%
=======================================
  Files           3        3
  Lines          35       35
  Branches        6        6
=======================================
  Hits           15       15
  Misses         17       17
  Partials        3        3

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 395bd11...19e88af. Read the comment docs.

Needs testing


Network parameters table

Currently on explorer the network parameters are rendered as a JSON blob. We should turn this into something more helpful, a table showing the network parameter key and the value associated with it.

[ ] Render key/value table to show all network parameters
[ ] Adjust cypress tests to pass for the new layout (check each value is non-empty, this will be improved upon when capsule lands)
[ ] Add unit tests for page as appropriate
[ ] If possible (i.e. no invalid DOM nesting) consider rendering a syntax highlighted parsed blob for JSON properties

Needs #258


Pass parameters through Link

As in example dynamic-segments:
var routes = (
  <Route path="/" handler={App}>
    <Route name="user" path="/user/:userId" handler={User}>
      <Route name="task" path="tasks/:taskId" handler={Task}/>
      <Redirect from="todos/:taskId" to="task"/>
    </Route>
  </Route>
);
...
var User = React.createClass({
...
  render () {
...

<Link to="task" params={{userId: userId, taskId: "foo"}}>foo task</Link>

<Link to="task" params={{userId: userId, taskId: "bar"}}>bar task</Link>

...
});

But I want to use Route "task" without :taskId:
<Route name="task" path="tasks" handler={Task}/>

Is it possible to pass a parameter taskId to Task from Link, without specifying it in the path in Route? I don't want use the query for this. If I specify taskId in params, query or simply specify as different parameter in Link, I cannot get it in Task.

I learned the code and understand that on click the Link () calls transitionTo. But this function returns only location in the history. So, all parameters in Link compomemt need for a formation of the URL.
Thus, it is clear that it is necessary to define onClick of Links and manually change model/props and then to interact with it. However, it is not clear how to get parameters from another Route/Component, but this problem seems to me easier.

I solved the problem as follows:
var User = React.createClass({
	taskId: "",
	...
	handleClick: function(task) {
		var that = this;
		return function(event) {//without return event will bubble to default handler (?)
			that.taskId = task;
			console.log("set taskId: " + that.taskId);
		}
	},
	render: function() {
		console.log("verify taskId: " + this.taskId);
		return dom.div(...
			Link({to: "task", params: {userId: params.userId}, onClick: this.handleClick("foo")}, "foo task")
			...
			RouteHandler({taskId: this.taskId})
			...
	});
...
var Task = React.createClass({
	...
	render: function() {
		console.log("get taskId: " + this.props.taskId);
	...

Running ... click on link, in console:
set taskId: foo
verify taskId: foo
get taskId: foo

It's works. But all of this implementation is similar to cheap craft (((
Did I surely miss something?

Can have the parameter be optional
<Route name="task" path="/task/?:taskId?" handler={Task}>

A little late answer but what you are after is really
<Route name="task" path="/task/(:taskId)" handler={Task}>
which makes taskId optional


Use Django built-in get_valid_filename

This is more loosen restriction which allowing Unicode in filename.

@SuJiKiNen Thank you for the contribution. Unfortunately this doesn't pass the tests.
FAIL: test_downloading_from_playground (explorer.tests.test_views.TestCSVFromSQL)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/work/django-sql-explorer/django-sql-explorer/explorer/tests/test_views.py", line 506, in test_downloading_from_playground
    resp['Content-Disposition']
AssertionError: 'filename="Playground-1.csv"' not found in 'attachment; filename="Playground-1"'

The filename is tested in the view tests and as you can see, it's not returning the result expected.


Osgis.bundleFinder: support version range

Useful for brooklyn.libraries when name:version of pre-installed bundle is specified.
@m4rkmckenna as discussed - can you take a look please?

Thanks @m4rkmckenna - I've added a couple more tests, squashed it into a single commit, and rebased against master. Once jenkins confirms this builds, I'll merge it.


failed to cherry pick PR 4074 from ty-dc, to branch release-v0.8

commits 04355b4db9b9a8709da88bb39cf2163577574dd5 of ty-dc conflict when merging to branch release-v0.8, please manually cherry pick it by yourself.
PR https://github.com/spidernet-io/spiderpool/pull/4074 ,  action https://github.com/spidernet-io/spiderpool/actions/runs/11063394874
Auto-merging .github/workflows/auto-upgrade-ci.yaml
Auto-merging Makefile
Auto-merging docs/usage/install/upgrade.md
Auto-merging test/Makefile
CONFLICT (content): Merge conflict in test/Makefile
error: could not apply 04355b4d... Merge pull request #4074 from ty-dc/fix/upgrade-nighty-ci
hint: After resolving the conflicts, mark them with
hint: "git add/rm <pathspec>", then run
hint: "git cherry-pick --continue".
hint: You can instead skip this commit with "git cherry-pick --skip".
hint: To abort and get back to the state before "git cherry-pick",
hint: run "git cherry-pick --abort".
hint: Disable this message with "git config advice.mergeConflict false"

#4129


feat(ci): validate github actions with actionlint

Actionlint makes sure github actions are staying in line (as well as inline shell scripts through shellcheck). Fire off actionlint if there are changes to github actions in CI. Also, minor changes (quoting variables, adding read -r) to the release aciton to make actionlint happy.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

Closing due to security concerns of using actionlint. If you see this PR and want to contribute – file a new PR with the improvements that actionlint suggests instead!


JP - Create Docs layout

This PR brings a new Docs Layout, useful for every microsite. It changes the layout classes organisation and also adds some minor fixes.
Please, @rafaparadela could you review? Thanks.

LGTM


Not apparent what eyedropper will recolor

It's not obvious with the eyedropper tool which object on the canvas is going to change color. Perhaps some persistent visual cue as to which object was "selected" before eyedropper was selected and show new color on mouseover

Champika

I think this is an issue of selection state not being very apparent now that we hid the handles

When something is selected, then eye dropper is chosen, the thing selected appears deselected.  Picking a new color, however, auto-fills the thing you still have selected just by picking a color - Mark

Let's show the bounding box on selected shapes

Make the blue highlight larger


Support a future event (shape TBD) that marks messages unread.

See discussion.

API discussion is in this thread (long).
A server-side draft is at https://github.com/zulip/zulip/pull/19307 .

Oops, we ended up duplicating this issue as #5361.
I think that one has wound up with more things linking to it, so I'll close this one as a duplicate in favor of that one.


2.8

@rultor release tag is 2.8

@rultor release tag is 2.8

@portlek OK, I will release it now. Please check the progress here

@rultor release tag is 2.8

@portlek Done! FYI, the full log is here (took me 9min)


Use the built-in AggregateError when possible

If globalThis.AggregateError exists, use that instead of our polyfill. Alternatively, remove AggregateError altogether and expect the end user to polyfill it themselves.
Related issue
https://github.com/sodiray/radash/issues/392

I will work on it.

Hi @aleclarson
This issue can be easily resolved with a small modification to the file:
class AggregateErrorPolyfill extends Error {
  errors: Error[]
  constructor(errors: Error[] = []) {
    super()
    const name = errors.find(e => e.name)?.name ?? ''
    this.name = `AggregateError(${name}...)`
    this.message = `AggregateError with ${errors.length} errors`
    this.stack = errors.find(e => e.stack)?.stack ?? this.stack
    this.errors = errors
  }
}

export const AggregateError = globalThis?.["AggregateError"] ?? AggregateErrorPolyfill

However, this can be a bit more complicated due to the current tsconfig configuration, where globalThis might not have the AggregateError type definitions.

We could resolve this in a couple of ways:

Using a workaround:
// @ts-ignore
export const AggregateError: typeof AggregateErrorPolyfill = globalThis?.["AggregateError"] ?? AggregateErrorPolyfill

Modifying the tsconfig:
Add "ES2021.Promise" to the lib configuration:
"lib": ["es2020", "ES2021.Promise"],

Then define AggregateError with the appropriate type:
export const AggregateError: AggregateErrorConstructor = globalThis?.["AggregateError"] ?? AggregateErrorPolyfill

Regarding tests:
The current tests check the implementation of AggregateErrorPolyfill, which might differ from how the native AggregateError works.

With the current way we import the function in our tests, we will end up importing the native implementation.
However, it is possible to simulate an environment where AggregateError is not available and force the import of the polyfill version with:
vi.stubGlobal('AggregateError', undefined)
const { AggregateError } = await import('radashi')

But this would cause another type error:

In this case, we would need to either use @ts-ignore or modify the tsconfig to accommodate the test files.
That being said, it would be useful to add two more tests to ensure that when globalThis.AggregateError is undefined, our polyfill is used, and when it exists, the value of our stub is correctly returned.
What are your thoughts on this?

Add "ES2021.Promise" to the lib configuration:

Let's put /// <reference lib="es2021.promise" /> at the top of AggregateError module so we can keep tabs on why something beyond es2020 is needed.

export const AggregateError: AggregateErrorConstructor = globalThis?.["AggregateError"] ?? AggregateErrorPolyfill

I think it's fine to assume globalThis exists. The new pnpm lint:compat command doesn't complain.
Also, let's use typeof globalThis.AggregateError as the type and inline the class declaration like so:
export const AggregateError: typeof globalThis.AggregateError =
  globalThis.AggregateError ??
  class AggregateError {…}

With the current way we import the function in our tests, we will end up importing the native implementation.
However, it is possible to simulate an environment where AggregateError is not available and force the import of the polyfill version with:
vi.stubGlobal('AggregateError', undefined)
const { AggregateError } = await import('radashi')

I will take care of this part. Let me know when you open the PR 👍

Fixed in #116


changePassword throws error

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
Use the example code from aws amplify website.
Auth.currentAuthenticatedUser()
    .then(user => {
        return Auth.changePassword(user, oldPassword, newPassword);
    })
    .then(data => console.log(data))
    .catch(err => console.log(err))

Error log:
undefined is not a function (near '...user.getSession...')
- node_modules\aws-amplify\lib\Auth\Auth.js:1095:27 in <unknown>
- node_modules\promise\setimmediate\core.js:45:7 in tryCallTwo
- node_modules\promise\setimmediate\core.js:200:23 in doResolve
- node_modules\promise\setimmediate\core.js:66:12 in Promise
* null:null in userSession
- node_modules\aws-amplify\lib\Auth\Auth.js:1347:27 in changePassword
* screens\ChangePassword.js:79:8 in _callee$
- node_modules\regenerator-runtime\runtime.js:62:44 in tryCatch
- node_modules\regenerator-runtime\runtime.js:296:30 in invoke
- node_modules\regenerator-runtime\runtime.js:62:44 in tryCatch
- node_modules\regenerator-runtime\runtime.js:152:28 in invoke
- node_modules\regenerator-runtime\runtime.js:195:17 in <unknown>
- node_modules\promise\setimmediate\core.js:45:7 in tryCallTwo
- node_modules\promise\setimmediate\core.js:200:23 in doResolve
- node_modules\promise\setimmediate\core.js:66:12 in Promise
- node_modules\regenerator-runtime\runtime.js:194:27 in callInvokeWithMethodAndArg
- node_modules\regenerator-runtime\runtime.js:217:12 in enqueue
- node_modules\regenerator-runtime\runtime.js:241:8 in async
* screens\ChangePassword.js:79:8 in _callee
- node_modules\promise\setimmediate\core.js:37:14 in tryCallOne
- node_modules\promise\setimmediate\core.js:123:25 in <unknown>
- ... 8 more stack frames from framework internals

The Session key of CognitoUser object from function currentAuthenticatedUser is null.
What is the expected behavior?
Successfully update password.
Which versions of Amplify, and which browser / OS are affected by this issue? Did this work in previous versions?
iOS and Android
"aws-amplify": "^0.4.0"
You can turn on the debug mode to provide more info for us by setting window.LOG_LEVEL = 'DEBUG'; in your app.
[DEBUG] 19:25.443 AuthClass - Session is valid, directly return this session
 Object {
   "[DEBUG] 19:25.785 AuthClass - Getting the session from this user:": Object {
     "Session": null,

I believe this is related to #578

@vincen7su can you try<EMAIL_ADDRESS>It should have been fixed in that version.

@powerful23<EMAIL_ADDRESS>works correctly.
Thank you!


Fix grammar in Readme

This fixes #5 ...?
I don't know, beauty is subjective or something.
But it does fix the grammar issues! 😀

Hey @Arihant25, thank you for creating this pull request! Have a wonderful day...


Lift macro/headers to compiler pass

Headers and macro such as MIN/MAX are currently set as properties in MPI backends and should instead be lifted into a compiler pass automatically generating needed macros/headers based on expression

Fixed in #2076


Parse_kdc_rep in krb5_parser.rs seems don't work properly.

Hey there, it occurred to me that some fields would not be properly extracted when padata, the optional field, is absent.
From my perspective, it could be caused by the unwrap_or_default function. If no padata found there , i, the remaining data here, would be empty as the default return value. Consequently, the following fields would never be parsed.

Good catch, thanks! I will check that there are no similar errors in the code, and push a commit to fix this.

fix published in release 0.7.1


Bitrate switching option is not working.

Description
Bitrate switching option is not working.
Include a templateon stackblitz you can use.
stack overflow question
Results
Expected
https://i.stack.imgur.com/tc1bw.png
Error output
quality-menu.js:1 Uncaught ReferenceError: videojs is not defined
https://i.stack.imgur.com/h6Jl0.png
videojs-contrib-hls version
what version of videojs-contrib-hls does this occur with?
videojs-contrib-dash x.y.z
videojs version
Not working on these version
"video.js": "^6.8.0"
"videojs-contrib-dash": "^2.9.3",
but it wokes on the oldrer version
Browsers
all browsers are affected

@squarebracketI have update stackblitz

Ok, so you are trying to integrate contrib-dash with contrib-quality-levels. Unfortunately, they don't integrate, and likely never will, as there's no clean way to integrate them. I'll spare you the boring bits, but basically there's no straightforward way to have dash.js pick only certain rates, which would be required for the qualityLevels API.
At this point, and depending on your needs, you've got a couple choices:

Use VHS
It integrates with contrib-quality-levels, but doesn't support all the features of dash.js. If you can, I would recommend switching to VHS.
Patch contrib-dash to support a subset of the qualityLevels API
If all you need is support for manual bitrate switching, and you don't need all of the qualityLevels API support, you could use something like this patch to support manual bitrate selection. This is what I'm using, along with a custom plugin for the UI bits. I think it should work with any of the resolution switcher plugins, though I'm not sure.

@squarebracket  I have  add VHS but now the video is not playing

i got the quality levels


add sampling factor for DeterminePartitionsJob

Description

Let's first recall that there are two type of DeterminePartitionsJob.

When the input data is not assume grouped, there may be duplicate rows. We will launch two mr job. The first one do group job to remove duplicate rows. Then the second one do global sorting to find lower and upper bound for target segments.
When the input data is assume grouped, we only need to launch the global sorting mr job to find lower and upper bound for segments.

As we don't need a segment size which exactly equals the targetRowsPerSegment parameter, we can sample the input data to improve the DeterminePartitionsJob. Let me explain how to use the sampling factor to improve  the DeterminePartitionsJob.

If the input data is assume grouped, we just do sample by random at the mapper side of the global sort mr job.
If the input data is not assume grouped, we do sample at the mapper of the group job. We use hash on time and all dimensions and mod by sampling factor to do sample, we don't use random method because there may be duplicate rows.

For example, if we ingest 10,000,000,000 rows and the targetRowsPerSegment is 5,000,000, we can sample by 500, so the DeterminePartitionsJob only need process 20,000,000 rows, this helps save a lot of time.

Key changed/added classes in this PR

DeterminePartitionsJobSampler
DeterminePartitionsJob

This PR has:

[ ] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] a release note entry in the PR description.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

Thanks for the changes, @hqx871 , I will try to get this reviewed soon.

@hqx871 , let me know if you have given some thought to this.

Ideally, for determining partitions, I would have liked to do what we do with PartialDimensionDistributionTask in native batch using StringSketch and StringSketchMerger. It would probably go something like this:
* Each mapper emits a `StringDistribution` for each `Interval` that it encounters.

* The reducer merges the distributions received for each `Interval` using `StringSketchMerger`.

PartialDimensionDistributionTask uses sketch to get quantiles for shards, it is a different from DeterminePartititonsJob, which use MR to do distinct and sort to find the upper and lower bounds for shards. Am I right?

I have not read much about the quantile sketch method. but it seams to do de-duplication itself and do not need to launch another de-duplication job.

After reading the PartialDimensionDistributionTask more, I find it use bloom filter to do de-duplication.

Just considering the PartialDimensionDistributionTask, if we apply sampling method before the DedupInputRowFilter, we should use hashed based sampling method to avoid bias.

@hqx871 , I am merging this PR for now as it is functionally correct. We can do further improvements as needed with some benchmarking. Thanks for the contribution and for your patience!


Writer and Result classes for printing a QR-Code on a console

Hello,
I've created a new Writer for printing a QR Code on Command Line Interface. I find it very useful to get data or link from a server to your smartphone.
You can expect something like this for example :

(since it uses UTF-8, it should probably fail on Windows)
Usage
require_once __DIR__ . '/vendor/autoload.php';

use Endroid\QrCode\Builder\Builder;
use Endroid\QrCode\Writer\ConsoleWriter;

$result = Builder::create()
        ->writer(new ConsoleWriter())
        ->data($argv[1])
        ->build();

echo $result->getString();

Hope this helps :cherry_blossom:

Sorry, took me a while to check with PHPStan

@Trismegiste sorry for the late response. I was enjoying my vacation ;) Really nice feature, great addition! Regarding the dark mode: I would really prefer to use the current QR code foreground and background color for this. If you could make it so that those colors are used we can create a dark mode using the current interface (and if really wanted, create a separate MR for creating a global dark mode setting).

Done :coffee:

@Trismegiste thank you. I have to fix some new QA issues I see (not caused by your changes) before I tag the new version.

Tag 4.5.0


Create new makefile

Starting to dig into the workflow (and doing the README work yesterday) got me thinking that we should have something like this.  This is a very simple makefile that can be referenced from both the README and our workflow.
Biggest advantages are:

We have a living, breathing file that is the source of truth of how to build the project. There is no way of the README and the workflow file getting out of date with one another.
New users can get started with just one command

make # by default builds clang

Users cannot fat-finger or typo any command, leading to more reproducible outcomes on their side.
README and worfklow files can be simplified using these commands.

In support of #4

Nice work @cjappl ! I'm all on board with configuration as code wherever we can. This makefile looks good to me, no concerns! But before we merge this I just want to check we've considered a couple of questions.

The motivation seems to be about making the test as frictionless as possible. Would it be more straightforward to provide a pre-built binary in a zip file? I was planning on paying for an Apple developer account anyway, so could potentially use that for code signing and notarisation. I'm not hugely familiar with this process, though, so I'm uncertain how much effort it would be.
Are we sure Make is the right tool for this? How would using CMake compare? No need to draft it now, let's just back-of-the-envelope consider whether we might want to!

Thanks again 👍

The motivation seems to be about making the test as frictionless as possible. Would it be more straightforward to provide a pre-built binary in a zip file? I was planning on paying for an Apple developer account anyway, so could potentially use that for code signing and notarisation. I'm not hugely familiar with this process, though, so I'm uncertain how much effort it would be.

Absolutely 100% on board with pre-built binaries. That would reduce friction in the biggest way possible!! On that note, I was the guy who set up code signing and notarization at my last gig, so I would expect it to take about a day of work. Especially if we had a good agreement on which binaries to sign, how we wanted them named, how we wanted them stored etc.
I think that this Makefile provides something in addition to that too. Think of this as a repeatable recipe book for doing tasks we see common around the repo. In fact, notarization steps could live in here!

Are we sure Make is the right tool for this? How would using CMake compare? No need to draft it now, let's just back-of-the-envelope consider whether we might want to!

CMake is incredible for building C++, but kind of stumbles at these more basic things that fall under "shell scripting". I find its complexity kind of gets in it's own way in this usecase. I am a massive CMake evangelist for building repos of C++ code from source, but for this aforementioned "recipe book" approach, Make or shell scripts are great. From things like bootstrapping the dev environment, to generating documentation, code signing, having these in a simple "one command, that is always installed on every system" is great.
In CMake, as a user you have to:

make a build directory
generate a project
run your command (or commands)

As a dev, the syntax has a lot of boilerplate for shell commands, and a lot of it is compiling C specific.
Contrasted to Makefiles, as a user you just:

type make

Understood, and agreed. Thanks for the great contribution! I'm happy to merge this now if you are?

Understood, and agreed. Thanks for the great contribution! I'm happy to merge this now if you are?

Good with me! If you have the time to test it out on your machine, that is bonus points, but all targets and options seem to work OK for me.
Do you want me to go through and update the README with the new commands in this PR, or file a followup?

Nice one, looks great! Happy to merge?

Ship it!


Visual Studio 2015 hang when I tried to right click on the solution node in solution explorer with the latest version

Visual Studio 2015 hang when I tried to right click on the solution node in solution explorer with the latest version, please solve it, thank you.

Having the same problem in VS2017. I have temporarily disabled the extension until this is fixed.

@Jimex There must have been some special project types not supported well.Can you upload the solution as attachment ?

@xunqinbo in my case, I have several WiX Toolset projects

@jfevia @xunqinbo Yea, same, I also have 2 WIX projects in my solution.

Fixed in V2.0.2,closing for now - feel free to re-open if it needs further discussion.


Question: Would you be willing to host a version of Postal that uses the modular version of Lodash?

I have a modified version of postal that I use that uses the modular version of lodash. I use lodash in my applications but rarely use enough of it that I feel like pulling in the entire library is worth while. Would you mind putting together an npm version of postal that uses the modulized version for dep. Or mind if I do?

See https://github.com/postaljs/postal.js/pull/110

See https://github.com/postaljs/postal.js/issues/91#issuecomment-73159144.

Done! :smiley_cat: See #91 (comment).


In tutorial mode, clicking on the small sim traps you in fullscreen

When you click on the simulator in mobile/tablet view, it goes fullscreen but doesn't have a collapse icon. You can get out of it by pressing escape
Found in IE11

Won't fix.

Also repros in Chrome

Fixed.


Sample code does not import BertTokenizer

The sample code in README.md does not import BertTokenizer so throws an error. Need to include:
from transformers import BertTokenizer

thanks ! I added this line to the readme


Changing color theme

Checklist

[X] There are no similar issues or pull requests for this yet.

Is your feature related to a problem? Please describe.
This feature request is not related to a problem.
Describe the solution you would like.
I would like to see some documentation on how the color theme can be changed.
Describe alternatives you considered
I've tried searching for how other repos have done this and so far it appears like this is a custom implementation.
Additional context
No response

This is an interesting one, I think we can just add different CSS classes for different components and allow them to be configured via the Admin instance, I'll look into this but if anyone has good understanding of UI feel free to contribute.


Subsampling

hello, do you know if would be possible to obtain a subsample (randomly) from the original gef?

you can sample the data.cells.cell_name randomly to get a cell_name_subset, and then run data_subset = data.tl.filter_cells(cell_list=cell_name_subset, inplace=False) to get a subsample data, this is done on cells/bins, similarly, you can do on genes through data.genes.gene_name and data.tl.filter_genes().


Update requirements.txt for everything!

Per in-person discussion, closing this (all requirements are already in requirements.txt, and optional dependencies are now in optional-dependencies.txt).


Incorrect context submenu position caused by transform ancestor

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

VS Code Version: 1.60.1
OS Version: web

The issue occurs when trying to intergrate VSCode (or Monaco) into a transform parent element.
Steps to Reproduce:

https://github.com/microsoft/vscode/blob/33c149f3ad49db462cb451f4db2e01d64b15e40b/src/vs/workbench/browser/media/style.css#L44-L46
Replace the body.web {} with:body.web {
  position: absolute;
  left: 0;
  top: 0;
  will-change: transform;
  transform: translate(100px, 100px);
}

Run yarn watch and yarn web.
Right click the editor and move the mouse to a context menu item with submenu.
The submenu is in incorrect position.

Reason:
This is because transform ancestor behaves as the containing block of fixed descendants.
Here is the code that causes the issue:
https://github.com/microsoft/vscode/blob/83bd43bc519d15e50c4272c6cf5c1479df196a4d/src/vs/base/browser/ui/menu/menu.ts#L920-L921
A PR has been made for this issue.

Is it expected that the menu shows up far away from the hamburger icon in the top left?

@TylerLeonhardt please file a separate issue as this fix was targeted for submenus in monaco


Firefox 44.0.2 - Inactive

Latest version of Firefox (44.0.2) causing the webfontloader to set class of inactive and fall back on stack.
[http://www.eframe.co.uk/] (Example - must be opened in the latest version of FF 44.0.2)

The fonts on your site appear to be loading correctly. Are you still seeing this issue?

I have the same issue, but most of the times it loads. To reproduce this issue:

Open a test site where font loading happens with multiple font.
Open a page on a new tab, doesn't matter what
Go back to the test page, hit CTRL+F5 and switch to the other tab
When you switch back to the original tab, some fonts might be not loaded. If everything is fine, try the previous steps again and again.

Here are my classes when the fonts loaded:
wp-toolbar supports csstransforms3d csstransformspreserve3d wf-montserrat-n4-active wf-montserrat-i2-active wf-montserrat-n3-active wf-montserrat-i6-active wf-sourcesanspro-n3-active wf-sourcesanspro-n4-active wf-merriweather-n4-active wf-merriweather-i6-active wf-raleway-n4-active wf-vollkorn-i2-active wf-vollkorn-n3-active wf-vollkorn-n4-active wf-vollkorn-i6-active wf-sourcesanspro-i2-active wf-sourcesanspro-i6-active wf-merriweather-i2-active wf-merriweather-n3-active wf-raleway-n3-active wf-raleway-i2-active wf-raleway-i6-active wf-active
and when some of them not loaded with the described technique:
wp-toolbar supports csstransforms3d csstransformspreserve3d wf-loading wf-sourcesanspro-i2-loading wf-sourcesanspro-i6-loading wf-merriweather-i2-loading wf-merriweather-n3-loading wf-raleway-i2-loading wf-raleway-n3-loading wf-raleway-i6-loading wf-montserrat-n4-active wf-montserrat-i2-active wf-montserrat-n3-active wf-montserrat-i6-active n2-in-expert wf-sourcesanspro-n3-active wf-merriweather-n4-active wf-merriweather-i6-active wf-sourcesanspro-n4-active wf-vollkorn-i2-active wf-vollkorn-n3-active wf-vollkorn-n4-active wf-vollkorn-i6-active wf-raleway-n4-active
In Firebug on the network tab, every font seems OK in the Google CSS.
Also I found if I disable the API for the Fontwatcher, then it works fine.
You can do this by opening: /src/core/fontwatcher.js and there is the version 42 hardcoded:
FontWatcher.SHOULD_USE_NATIVE_LOADER = parseInt(match[1], 10) > 42;
I  just replaced it with 100.
(you can also find this 42 in the minified version: (x=/Gecko.*Firefox/(\d+)/.exec(window.navigator.userAgent))?42)

Thanks for reporting this. I think I fixed this issue in v1.6.22. Could you give that version a try and see if the issue is resolved? Thanks!

Thanks, I just did some tests and it works fine where 1.6.21 failed. I just notice a strange effect with this update, sometimes when you follow my steps and change back to the tab where the font should be loaded, it marks all fonts loaded after 0,5-1 sec. (I'm running a spinner while the fonts are loading and this spinner stays for 0,5-1 sec after I switch back to the page.) It does not happen every time, but little frustrating. I know that it is not a real world use case, but maybe it indicates problem with the implementation.

@nextend Glad that fixed it. The new issue sounds weird. Do you have a test case somewhere?
(I'm closing this issue, but feel free to open a new one for that issue).


Fix #53 with ismutable default

This makes it so that ismutable has a default now, which is to use the mutable field of a type.

I'm not sure we should do this. It's almost guaranteed to not be correct for wrapper types.

It doesn't change any of the current behavior for wrapper types. It simply provides a fall back. If we want to account for wrappers I could replace the current ismutable(::Type{<:AbstractArray}) = true with

function ismutable(::Type{T}) where {T}
    if parent_type(T) <: T
        return T.mutable
    else
        return ismutable(parent_type(T))
    end
end

...and then we could explicitly define the mutability of each type.

I added the recursive version that searches for a parent structure. I still haven't changed any of the tests, because the behavior is essentially the same.

@ChrisRackauckas, is this PR still problematic?

I think that works.

Could it get a test? Fake wrapper

So instead of testing a fake wrapped array I just tested views of StaticArrays. Turns out this actually fixes views on immutable arrays.

I'm convinced.


Add .extend and .withComponent deterministic ID generation

Will fix #1031
I added a test with the new behaviour. This should fix server-side rendering issues with .extend and .withComponent.

@philpl I modified the .extend, I wasn't taking extended rules. Making a .join('') is not a problem, you can see that it work in test. But I can also skip functions or replace it by a string "FUNC" if you prefer.

@neoziro That is looking fine. The only thing I'm concerned about is performance in this case :wink: I'll take a look later and see whether it's sufficient

@philpl done

@neoziro This is looking perfect! :tada: I added two small nits. I'll test this and see if it fixes the SSR issues, but I don't see why it wouldn't :smile:

@philpl ready to be merged

Awesome! :tada:

Yay! :tada:


Port to futures 0.3

While this requires nightly, the API is now stabilized and will become part of Rust 1.36. IMHO we should merge this now so that it can all get wider testing.

This is currently marked as WIP because it uses https://github.com/gtk-rs/glib/pull/476 and https://github.com/gtk-rs/gir/pull/753 via Cargo patching

Should be ready now

Looks good! Waiting for CI confirmation.


Fixes second callback after timeout when leading: true

I noticed while working on the context pull request test cases that my debounced function was actually getting called twice when leading: true. This was not caught in the current tests because the promise resolves immediately  before the debounce timeout has fully completed.
This pull request uses clear for the timeout when leading: true rather than run. The latter was calling the function a second time after the timeout elapsed.

Hi @idpaterson, thanks for the pull request!
I'm pretty sure the correct/expected behavior is to invoke the function twice when leading: true. Once immediately and once after the wait period is over. This is also how lodash.debounce works:
const {debounce} = require('lodash')
const run = debounce(function run() {
  console.log('called')
}, 200, {leading: true})

run()
run()
setTimeout(run, 100)

Will output:
called
called

In cases where this is not the desired behavior it could instead be covered by a trailing option that, if set to false, only invokes the given function once, at the leading end.

Ah you're absolutely correct, the test that I added did not actually test the problem that I was seeing. The original issue I noticed was that the callback is called again even if the debounced function is called only once. This was certainly a bad fix on my part.
I updated the pull request with two tests, one the exact opposite of my first ensuring that the callback is called again if the debounced function is invoked multiple times during the timeout, and one to ensure that if invoked only once the callback is called only once.

Ah, that makes a lot more sense. Thanks a lot for this!

Published as v2.0.0 as this could potentially be a breaking change

Thank you for the speedy publish!