Updated SegmentationDataset to support 4D images

This PR is related to constantinpape/elf#36
I think using greater or equal could be a compromise. It allows using higher-dimensional datasets, when performing the slicing it will still have the desirable property of crashing when indexing a lower-dimensional dataset with a slice of greater dimensions, and the assertion catches when using a patch_shape smaller than the raw.ndim, which could lead to silent undesirable behavior.
Let me know if you have any suggestions, cheers.

Hey @JoOkuma,
sorry for taking so long to come back to you about this. I have implemented support for 4d data now in #42, so I will go ahead and close it.
I will send you example code for how to use this soon.


Temp/Humidity stop being refreshed

What issue do you have? Please be as thorough and explicit as possible.
Temperature and Humidity is not refreshed since this week. Was previously working like a charm.
Details of your setup.

Do you use (1) Homebridge UI-X (2) Homebridge CLI or (3) HOOBS?
Homebrudge UI-X

Which version of Homebridge/HOOBS do you have?
1.3.8

Which platform do you run Homebridge/HOOBS on (e.g. Raspberry Pi/Windows/HOOBS Box)? Please also mention your version of Node.js/NPM if known.
Raspberry Pi 0W
Node.js Version  v14.17.1
NPM Version v6.14.11

Which version of this plugin (homebridge-govee) do you have? Has the issue started since upgrading from a previous version?
Issue start from last release update 4.18.0 and rollback to previous (4.17.0) don't change anything.

Which Govee devices do you have that are causing issues? Please include product models if applicable.
Thermo/Hygro Wifi/Bluetooth H5075

Please paste any relevant logs below.

[01/12/2021, 18:53:51] [Govee] Disabling HTTP client as timeout of 10000ms exceeded at createError (/usr/local/lib/node_modules/homebridge-govee/node_modules/axios/lib/core/createError.js:16:15).
[01/12/2021, 18:53:59] [Govee] Disabling BLE client as no enabled devices configured.
[01/12/2021, 18:53:59] [Govee] ***** Disabling plugin. *****
[01/12/2021, 18:53:59] [Govee] ***** No devices found to initialise. *****

If I change my account password, it start working again, fetch data for few minutes and stop again.
Device data stay visible  in Home with last knowned values.

If I change my account password, it start working again, fetch data for few minutes and stop again

When you say “stop again” what do you mean? what errors are displayed?

There is no errors. Values stop being refreshed in Home app.
If I change the password and connect again, values will be updated for a short moment and value are stuck.
If I check update status in Evde app I will see there is no data from 4 days.
As you can see in screenshot I'm using two Govee devices.

I’m a little bit stuck here because it’s most likely some account issue or other issue with how the govee account system works… but…
since all the work i have done is basically reverse-engineered, it’s not something that I can go to Govee themselves about…
Long shot, but could you try creating a new govee account and adding the devices to this? and updating the plugin config with the new details?

It's working again after creating account. Hope it won't stop working after few hours/days.
Now log is back to normal :
[02/12/2021, 18:38:38] [Govee] Initializing Govee platform...
[02/12/2021, 18:38:38] [Govee] Initialising plugin v4.18.0 | Node v14.17.1 | HB v1.3.8...
[02/12/2021, 18:38:39] [Govee] Plugin initialised. Setting up accessories....
[02/12/2021, 18:39:04] [Govee] Disabling BLE client as no enabled devices configured.
[02/12/2021, 18:39:04] [Govee] [Garage] initialising with options {"logging":"standard","lowBattThreshold":20}.
[02/12/2021, 18:39:04] [Govee] [Garage] initialised with id [A6:00:E3:32:81:12:E0:CF].
[02/12/2021, 18:39:04] [Govee] [Garage] initialising with options {"logging":"standard","lowBattThreshold":20}.
[02/12/2021, 18:39:04] [Govee] [Garage] initialised with id [29:02:E3:32:81:20:50:9D].
[02/12/2021, 18:39:11] [Govee] [Garage] has been reported [online].
[02/12/2021, 18:39:11] [Govee] [Garage] current battery [90%].
[02/12/2021, 18:39:11] [Govee] [Garage] current temperature [13.3°C].
[02/12/2021, 18:39:11] [Govee] [Garage] current humidity [64.7%].
[02/12/2021, 18:39:11] [Govee] [Garage] has been reported [online].
[02/12/2021, 18:39:11] [Govee] [Garage] current battery [87%].
[02/12/2021, 18:39:11] [Govee] [Garage] current temperature [13.1°C].
[02/12/2021, 18:39:11] [Govee] [Garage] current humidity [66%].

I'm gonna close this issue as I'm assuming that everything has been working properly? If you're still having problems feel free to reopen.

Yes everything is working. Thanks for help


Address pending comments from https://github.com/CSE-25/amrita_pyq/pull/1

[ ] Resolve comments from https://github.com/CSE-25/amrita_pyq/pull/1

I reviewed the comments in #1 and identified some code and comment enhancements that need to be made. If these align with the requirements, I'm ready to take on this issue, @Abhinav-ark, @Ashrockzzz2003.

Assigned @FLASH2332


[doc] Add detailed contribution guide

This document gives detailed guidance on contributing to the OpenTitan
project. It has gone through many rounds of review in GDoc form (and
this PR simply converts that to Markdown, with the odd grammatical fix
along the way).
The contribution guide effort was kicked off by Pirmin and has since
seen suggestions, edits, and commits from a wide number of people across
the project. Thank you to everyone for your input.

I've updated the PR to hopefully resolve the comments re TODOs in code. Thank you to everyone who has reviewed so far - please let me know if you have any additional review comments to address, or if this looks good to land.

Ping to reviewers - it would be a good to get another +1 before merging. Thanks!


fix(instagram): set insta input width

Set the max and min width of the div that contains the instagram post url input field and the submit button beside it
fix #67
Fixes Issue
**My PR closes #67 **
👨💻 Changes proposed(What did you do ?)
✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title and description of the PR is clear and explains the approach.

Note to reviewers

📷 Screenshots

This is the new width of the Instagram input

@Dun-sin  - Hey. Pardon for the inconvenience. But the max-width value 625px was applied on purpose and not by mistake.
Notice how the div that has the class max-w-[625px] contains both the input box and the submit button.
Also, notice that the width of the input box is 80% of the outer div.
Therefore, the max-width of "THE INPUT BOX" will be 80% of 625px = 500px.
Similarly, 80% of 375px = 300px, which is the minimum width.
I had tested my proposed solution and verified that the width of the input box was in accordance with the asked changes :)
Makes sense?

@Dun-sin - Hey. Pardon for the inconvenience. But the max-width value 625px was applied on purpose and not by mistake. Notice how the div that has the class max-w-[625px] contains both the input box and the submit button. Also, notice that the width of the input box is 80% of the outer div.
Therefore, the max-width of "THE INPUT BOX" will be 80% of 625px = 500px.
Similarly, 80% of 375px = 300px, which is the minimum width.
I had tested my proposed solution and verified that the width of the input box was in accordance with the asked changes :)
Makes sense?

that's not what my problem is, this is open source which means collaboration and communication comes first, you should have talked to me about it first and we would have had a discussion or at least filled in the template, there's literally a space called note to reviwers which you could have given me the reasons for the change, i can't know what's on your mind if you don't tell me.

@Dun-sin  - Allright. Noted and I will keep it in mind from now. Thanks for the learning you gave :)
Let me know the next steps - Do you want me to revert your changes?

@Dun-sin - Hey. Pardon for the inconvenience. But the max-width value 625px was applied on purpose and not by mistake. Notice how the div that has the class max-w-[625px] contains both the input box and the submit button. Also, notice that the width of the input box is 80% of the outer div.
Therefore, the max-width of "THE INPUT BOX" will be 80% of 625px = 500px.
Similarly, 80% of 375px = 300px, which is the minimum width.
I had tested my proposed solution and verified that the width of the input box was in accordance with the asked changes :)
Makes sense?

now about your change, if you told me about it, you would have known that I meant the whole thing input + button = 500, because this is what your change does on mobile, not good for mobile

but if I change it to what was said in the issue it looks like this on mobile

@Dun-sin - Allright. Noted and I will keep it in mind from now. Thanks for the learning you gave :)
Let me know the next steps - Do you want me to revert your changes?

don't worry, I already fixed it from the PR itself, and the other question still stands, do you want me to merge this when Hacktoberfest starts?

@Dun-sin  - Understood. Thanks :)
Appreciate it if you merge when Hacktoberfest starts.


Markdown heading link doesn't seem to scroll the heading correctly on screen

Testing #141327
Create a simple notebook like the following:
Cell 1:
# Main Header
## Second Header

Cell 2:
[Main Link](#main-header)
Both cell are markdown cells. When clicking the Main Link it would scroll the Main Header cell off the screen which doesn't seem right for that action. Best illustrated with GIF below:

When I click Main Link my expectation would be that Main Header would end up onscreen, like when I click it from the outline. Instead the cell containing the link is scrolled to the top and the Main Header cell is pushed offscreen.

This works in Jupyter and was also recently run across by Rong. So adding nb-golden.


Have to press Ctrl+Alt+Shift+Back/Forward twice

After the latest Chrome OS update, crouton hasn't been working quite right.
Here is my current version info:
Version 42.0.2311.87 (64-bit)
Platform 6812.75.2 (Official Build) stable-channel swanky
Firmware Google_Swanky.5216.238.5

When switching between Chrome OS and my LXDE Debian (Jessie) install, I have to push the button combination twice.
If I'm in Linux and press Ctrl+Alt+Shift+Back, Chrome OS shows up but is completely unresponsive.
If I'm in Chrome OS and try to switch to Linux, the first time I push the buttons it "locks up" Chrome OS, then the second time it switches to Linux.
However, I've had a few times where I can no longer reliably switch between systems. Earlier today while trying to switch, the screen just went black and I rebooted in order to get back to what I was doing.
I updated crouton to see if it would fix the problem, but it didn't.

I've been switching back and forth, and now it takes more button combination presses to switch between environments, and instead of Chrome OS appearing to be locked up, it goes straight to a black screen.

Same issue here, but I have other info: I like to re-map the chroot switching key combo to ctrl+esc since that's much easier to do with one hand. The line in /etc/crouton/xbindkeysrc.scm is (xbindkey '(control Escape) "croutoncycle next"). This used to work as expected to switch back and forth. Now it will only switch in one direction: from chroot back to Chrome OS. Once in Chrome OS, that key combo no longer does anything.
crouton: version 1-20150423153254~master:2f55723c
release: trusty
architecture: amd64
xmethod: xorg
targets: keyboard,extension,gnome
host: version 6812.75.2 (Official Build) stable-channel swanky
kernel: Linux localhost 3.10.18 #1 SMP Tue Apr 14 20:43:12 PDT 2015 x86_64 x86_64 x86_64 GNU/Linux
freon: no

The mawk process below, which seems to be started with the chroot, appears to be related to this issue. If I kill that process, I can no longer cycle into the chroot's desktop with ctrl+alt+shift+forward.
It's a very long command. Much scrolling involved.
 mawk -W Interactive      function output(s) {         print s         fflush()     }          function update() {         c = lc || rc; s = ls || rs; a = la || ra         if (!cmd && c && s && a && p) {             cmd = "p"         } else if (!cmd && c && s && a && n) {             cmd = "n"         } else if (cmd && !c && !s && !a && !p && !n) {             system("/usr/local/bin/croutoncycle " cmd)             cmd = ""         }     }     $5 == 1 && $6 == 29   { lc = $7; update() }     $5 == 1 && $6 == 42  { ls = $7; update() }     $5 == 1 && $6 == 56    { la = $7; update() }     $5 == 1 && $6 == 97  { rc = $7; update() }     $5 == 1 && $6 == 54 { rs = $7; update() }     $5 == 1 && $6 == 100   { ra = $7; update() }     $5 == 1 && $6 == 59         {  p = $7; update() }     $5 == 1 && $6 == 60         {  n = $7; update() }

Am having the exact same problem...now must hit the button combo twice to cycle between Chrome OS and the chroot desktop.  There seems to be a middle environment w/ a frozen Chrome OS screen between the two.
I'm a bit of a novice and would love to hear any suggestions on how to get the button sequence back to normal / one press effectiveness.

I think you both experiencing #1475 I had this just before my device switched to freon. It is also a problem on "beta-channel parrot" see #1644

same problem here. Isn't a consistent thing. Sometimes multiple key combinations required. Sometimes I can't switch at all to Linux and have to clobber the running chroot

1rocketdude, head over to issue #1475 and try out my proposed solution towards the bottom of that thread. Post on that issue whether the change to croutoncycle works for your case.

I appreciate the heads up. I updated using crouton -r trusty -u and the problem appears fixed. Lots of magic going on that I'm not aware of.
crouton: version 1-20150430123324~master:90ad47ff
release: trusty
architecture: amd64
xmethod: xorg
targets: lxde,audio,xorg,chrome
host: version 6812.83.0 (Official Build) stable-channel lumpy
kernel: Linux localhost 3.4.0 #1 SMP Mon Apr 27 10:29:31 PDT 2015 x86_64 x86_64 x86_64 GNU/Linux
freon: no

I've updated again and continue to have the problem. However, a workaround that seems to be stable for me (so far) is...
To Linux: Ctrl+Alt+F2 then Ctrl+Alt+F3
To ChromeOS: Ctrl+Alt+F1
Hopefully it works as well for someone else.

@dnschneid, would you please update this issue with a link to the issue it is a duplicate of?

Sorry, it's #1475

@Asuza I have exact same problem, I think I got it with last ChromeOS update. The "middle" VT that I see with Ctrl+Alt+F2 is the one from where I entered the chroot & started xfce4. It would be best to switch directly to the chroot display, also because brightness gets messed up

Same here, arm samsung chromebook, not freon
debian jessie + lxde or + xfce, both without or with crouton integration extension installed (i.e, before, and after adding target xiwi).
Before updating with -t xiwi, it showed xmethod xephyr
Will try 1475.


DynamoDB: Generate AttributeValue.IsXXSet properties for SS, BS, and NS

Description
Adds IsSSSet, IsBSSet, and IsNSSet properties to AttributeValue to allow empty collections to be stored.
Motivation and Context

Fixes https://github.com/aws/aws-sdk-net/issues/3297 - It is not currently possible to store empty string sets, number sets, or binary sets without running into:

Amazon.DynamoDBv2.AmazonDynamoDBException : Supplied AttributeValue is empty, must contain exactly one of the supported datatypes

Testing
I'm not sure how to do testing, though I assume the net change of adding to the mapping file as suggested by @normj in https://github.com/aws/aws-sdk-net/issues/3297#issuecomment-2075401652 would only be adding the correlating properties and customization as defined by the generator.
Screenshots (if appropriate)
N/A
Types of changes

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

Checklist

[x] My code follows the code style of this project
[x] My change requires a change to the documentation - (Though I assume most of this is generated based on the change)
[ ] I have updated the documentation accordingly
[x] I have read the README document
[ ] I have added tests to cover my changes
[ ] All new and existing tests passed

License

[x] I confirm that this pull request can be released under the Apache 2 license

@normj after you run the generator will that also generate the implementation similar to https://github.com/aws/aws-sdk-net/blob/main/sdk/src/Services/DynamoDBv2/Generated/Model/AttributeValue.cs#L198-L205 ?

@danielmarbach Yes. The IsLSet that you pointed to was generated in the past because the customization file that @DavidBoike file edited already had the L property in the list.

@normj myquestion was if the generator will generate a similar access pattern for the new property but I now derive from your answer that it will be the case

The PR was released as part of version 3.7.302.23 of AWSSDK.DynamoDBv2.

Thanks. @normj!


Android mode

It would be cool if veraball would get android controls and one could get it into f-droid.

I haven't succeeded in exporting to Android, the resulting APK refusing to install or crashing on start.


[bug 1134475] Cleanup all references to input-dev environment

Remove all references to input-dev.allizom.org

Oops, I didn't notice that. Fixed now

Awesome! Thank you!

c1fe25a [bug 1134475] Cleanup all references to input-dev environment


rule to ensure that default export matches module name

It would be great to have a rule that  ensure that the default export matches module name:
eg
// good
//doSomething.js

export default function doSomething {
    ...
}

// bad
//dosomething.js

export default function doSomething {
    ...
}

I like the idea, though it's a really nitpicky style issue, as the name of the default import is rarely used anywhere else :sweat_smile:.
I usually name my files in kebab-case, as I like it that way, and because apparently that causes less problems/confusion for people with case-insensitive operating systems (--> mac users).
I think we could simply transform the name of the file to camelCase and see if that matches the exported value. In the case files with dots in their name (foo-bar.helper.js), we could check if the name matches any part of the transformed name (fooBar, helper or fooBarHelper). For index.js files, I'd recommend to skip the check though.
If the exported value has no name (=== arrow function, object/array/primitive/...), then I assume this is fin to ignore?

I like the idea, though it's a really nitpicky style issue, as the name of the default import is rarely used anywhere else :sweat_smile:.

I like to use:
import doSomething from './doSomething';

//doSomething.js
export default function doSomething() {
   ...
}

everywhere

True, but that's more of an import style issue (partially export too of course). You may be interested in this discussion https://github.com/sindresorhus/eslint-plugin-unicorn/issues/6.

@jfmengels I've seen https://github.com/sindresorhus/eslint-plugin-unicorn/blob/master/docs/rules/filename-case.md but it doesn't work for me, because I want:
// FooBar.js

export default class FooBar {
}

to work too.

@jfmengels sorry I posted my earlier comment before I'd finished it. See the completed version now...

Also, foo/index.js should have an import named "foo" (following the same casing rules as if it was foo.js)

@ljharb foo/index.js can have a default export named "foo", if the default export has a name.

oops yes sorry, default export, not import. totally agreed.

I think it will be extraordinarily difficult to come to any consensus on a versatile implementation.
That said: I wouldn't use this rule. I'm still not super into the style guide enforcement aspect of import linting. So feel free to ignore my curmudgeonry, I'm not the target user of this rule. 😎
If y'all are able to figure out something you think would be both useful and not cause a million nitpick issues to be filed, I'm cool with it. I'm concerned that it will be something pretty niche that people will want to customize, which makes me lean towards this being is a spin-off module.

I'm pretty much in agreement with @ljharb
I'm also OK with a config that chooses between an exact match or camel-case, or etc

I believe that eslint-plugin-filenames already does what is being suggested here

@peet that looks about right

@ljharb https://github.com/airbnb/javascript/issues/1185

I just wanted to let you guys know that eslint-plugin-filenames has been abandoned according to the author. I am one of those guys who would like to ensure that the exported entity name (even for non-default exports) matches the filename.

I created a plugin "eslint-plugin-consistent-default-export-name" (https://github.com/minseoksuh/eslint-plugin-consistent-default-export-name)
that  combines  "match-exported" rule from eslint-plugin-filenames and "default-import-match-filename" rule from this repo.
Try it out if you want to.

@minseoksuh itd be great if you were willing to make a PR to eslint-plugin-import.

@ljharb thanks for the response.
Do you mean I make a PR to eslint-plugin-import for the rule "match-exported"? (which in my plugin is named default-export-match-filename)
If it is ok with the theme of the repo "eslint-plugin-import", than I would love to do it.
In fact, I think much more people will be able to use, if the rules are in a popular plugin.
Let me know.
Thanks!

@minseoksuh yes, if you can handle all the edge cases and have good test coverage, it seems like a great rule to me :-)

@minseoksuh I'd prefer your new names, they are more explicit 👍


ref(docker): create a standalone docker artifact to run lightwalletd

Please ensure this checklist is followed for any pull requests for this repo. This checklist must be checked by both the PR creator and by anyone who reviews the PR.

[ ] Relevant documentation for this PR has to be completed before the PR can be merged
[ ] A test plan for the PR must be documented in the PR notes and included in the test plan for the next regular release

As a note, all CI tests need to be passing and all appropriate code reviews need to be done before this PR can be merged

Motivation
This PR is meant to align lightwalletd docker execution to what we previously had working in https://github.com/ZcashFoundation/zebra/pull/8323 (but we no longer need in our repo). Allowing users to use an standalone lightwalletd docker image without further configurations, allowing users to just do:
docker run --detach -it -p 9067:9067 -p 9068:9068 electriccoinco/lightwalletd

And have a working lightwalletd server out-of-the-box (at least for testing/development purposes)
Changes

Upgrade to golang:1.19 as https://github.com/zcash/lightwalletd/commit/1002f42497c17693fae1a09aafba6b68b2e640b1 made this a requirement, considering some dependencies no longer work with golang:1.17
Use go instead of make to build inside the image (this is the expected approach with Docker)
Remove instructions with docker_img_run_zcashd as we don't have zcashd nor zcash-cli` inside the container
Add a .dockerignore file to just have the required file inside the container
Use a multistage Dockerfile to have a resulting image with the bare minimum: lightwalletd, and with a considerably reduced image size
Set some variables as ARGs, so people can change their values if they'd like to build the image locally.
Create a default zcash.conf to use this image as a standalone artifact without further configurations
Use the available cert.key and cert.pem from the repo to avoid running it with --no-tls-very-insecure by default

Pending

Confirm if the default configuration for zcash.conf is the right one
Confirm if having zcash-cli and zcashd inside the image is expected
Adapt documentation (?)
Confirm the docker-compose files still work with this Dockerfile

Also, as this is now using a multistage Dockerfile, this can be easily extended to add a tests stage in it, to run with CI

Thank you @daira! I actually ended up removing the whole section with comments, as this was recently removed anyways https://github.com/zcash/lightwalletd/commit/38b751e5513dd998cfda2a691800e8ed092f09c1

I don't understand how to run this (to test it). I'm sure part of the problem is that I'm unfamiliar with docker. For example, am I expected to start a zcashd instance manually, or does this happen within docker? If I start it manually, should I give any non-default command line (or zcash.conf) options? Should I be using the make targets, and if so, which ones? Can you document the exact steps need to run this? Actually, I think updating docs/docker-run.md might be a good place to do this, so people don't have to refer to this PR.
Also I see that this PR needs to be rebased onto the latest master branch. (I did that here locally, it wasn't difficult, but if you'd like I can push my branch to GitHub.)

I see this has been approved now, thank you! Is there anything in particular that is preventing this PR from merging?

@LarryRuane I'll be adding the process and documentation so it's clearer and easier to test this changes.

@gustavovalverde

Is there anything in particular that is preventing this PR from merging?

No, we can merge it, but I just wanted to do some testing myself as an extra safety margin. But since this PR makes only docker-related changes, it's safe for anyone not using docker.
Now that @gustavovalverde will be adding some documentation, we could wait for this and have me test it. Or would you rather have me merge it right away? I'm okay with either plan, I don't want to hold anyone up.

Hi @LarryRuane!
I've added extra testing steps into the PR body, and also in the documentation.
I added an extra docker-compose file to keep the changes as minimum as possible.


Bookmark styling

Been using this plugin for weeks now. I like the extra options it gives. Here's a few simple improvements, that would make it a lot better IMO.

Instead of highlighting the whole line, I would like it to highlight the selected code instead. Like default bookmark manager.
Outline is very annoying. I suggest making it 0.4 opacity OR clearing it from view when something else is selected OR just having the manager select the text normally OR instead of outline just use line-highlight, just like the ST3 setting ["highlight_line": true].
Some documentation how to set a hotkey for "GoTo".

I also find the Outline to be very annoying, it's the only thing stopping me from using this plugin constantly. KristerV's suggestions are all very good, and I think the best would be an option to just remove it and have the gutter arrow.

Regarding point 2, instead of outlines, would it be possible to have a user-configurable background highlight colour that extends the full width of the bookmarked line?  This would make it easy to visually locate bookmarks rapidly.  See below for rough idea.

I might be late to the party but new to this package. The outline was also something that bothered me. Although I have no clue how to make a user configurable option (which would be very nice) I changed the appropriate lines in "sublimebookmark.py". I did have to extract the package first with a e.g. Package Resource Viewer.
In that file at line no. 285 is the add_regions() function which draws the regions. I have set mine to stippled underline which works for me. This is how it looks now:

If one requires a different style take a look at https://www.sublimetext.com/docs/3/api_reference.html and look for the add_region() function to see the available flags. Removing it completely is easily done by replacing sublime.DRAW_STIPPLED_UNDERLINE above by sublime.DRAW_EMPTY.
I am not sure though how to color the whole line. I think that the whole line is technically not a region since there are no characters or white space. One would have to look into how the  ["highlight_line": true] is done how @KristerV  mentioned ...
Hope this might be helpful for anybody, sad to see this thing not maintained but none the less thanks @bollu for making a bookmark feature in Sublime Text that is actually useful!

Hi @DrBones, would you mind sharing if you are on Linux / MacOS / Windows ?
I am getting an Assertion error that the UID is not set correctly ( on windows ).

I managed to make some progress by commenting out all the async operations 🥲 .
They seem to be throwing errors and making the bookmark plugin non-functional.

Hi @sourcevault, sorry for the delay. Unfortunately, I am not on that system anymore. It was on windows though.
I am currently on a Mac and tried my hack with a sublime 3.2 version. It did not draw my desired stippled underline, however it did not throw any errors or required further modifications either. I could only remove the highlight alltogether...strange but probably the API changed at some point.
However, sublime 3 and this bookmarks plugin is rather old/unmaintained/deprecated e.g. for sublime text 4, this plugin does not work at all.
Sorry, I can't help any further. Best of luck! :)

Hi @DrBones 😃 nice to meet you !
I am on windows 10/11 | Sublime Text 4143.
For now, this change is working for me as any information on the code itself proves too much of distraction:
view.add_regions(str(uid), [region], "text.plain",
"bookmark", sublime.HIDDEN |
sublime.DRAW_EMPTY |
sublime.DRAW_NO_FILL |
sublime.DRAW_EMPTY_AS_OVERWRITE)

https://github.com/bollu/sublimeBookmark/commit/0edc70972ce72eb82729c619033c4208a5734bb0
I mainly like this plugin for it's search function, I can name bookmarks, and then search through them, really handy with dealing +4000 lines of code.
ofcourse, we would all like to see further development, but not being a python coder myself, I do not see how I can be of much help, there is also not a lot of tutorials on how to build such plugins ( I mean you need to understand the entire suite - debugging, testing, writing idiomatic python - to do it on a high enough quality standard ).
Thanks you for documenting your finding, if I didn't come across it I would still be lost.


ItemImage with local source

I know that Zeego currently doesn't support remote urls with with the ItemImage component.
But is it possible to display an image with a local image source, so with an uri like assets://*?

Just found the solution for my problem.
It is possible, but only with a url that starts with file://.
Previously I tried the uri field from Asset but you have to use the localUri field from AssetInfo.
You have to set the source property to source={{ uri: localUri }} even when the typing is not up to date, but this way the image will be loaded from the local source.


🚑 fix: Update requirements for Google Colab compatibility

Fixed module versions:
× Cannot uninstall blinker 1.4

× google-colab 1.0.0 requires ipykernel==5.5.6, but you have ipykernel 6.29.5 which is incompatible.

Installing build dependencies ... done
Getting requirements to build wheel ... done
Preparing metadata (pyproject.toml) ... done
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 111.5/111.5 kB 2.6 MB/s eta 0:00:00
Preparing metadata (setup.py) ... done
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 977.7/977.7 kB 17.1 MB/s eta 0:00:00
Preparing metadata (setup.py) ... done
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 121.0/121.0 kB 10.9 MB/s eta 0:00:00
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 20.8/20.8 MB 22.4 MB/s eta 0:00:00
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100.2/100.2 kB 9.6 MB/s eta 0:00:00
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 18.5/18.5 MB 92.1 MB/s eta 0:00:00
Building wheel for blinker (setup.py) ... done
Building wheel for ipywidgets-bokeh (setup.py) ... done
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
flask 3.0.3 requires blinker>=1.6.2, but you have blinker 1.4 which is incompatible.
holoviews 1.19.1 requires bokeh>=3.1, but you have bokeh 2.4.3 which is incompatible.
holoviews 1.19.1 requires panel>=1.0, but you have panel 0.14.4 which is incompatible.

  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 111.5/111.5 kB 2.6 MB/s eta 0:00:00
  Preparing metadata (setup.py) ... done
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 977.7/977.7 kB 17.1 MB/s eta 0:00:00
  Preparing metadata (setup.py) ... done
   ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 121.0/121.0 kB 10.9 MB/s eta 0:00:00
   ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 20.8/20.8 MB 22.4 MB/s eta 0:00:00
   ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100.2/100.2 kB 9.6 MB/s eta 0:00:00
   ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 18.5/18.5 MB 92.1 MB/s eta 0:00:00
  Building wheel for blinker (setup.py) ... done
  Building wheel for ipywidgets-bokeh (setup.py) ... done
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
flask 3.0.3 requires blinker>=1.6.2, but you have blinker 1.4 which is incompatible.
holoviews 1.19.1 requires bokeh>=3.1, but you have bokeh 2.4.3 which is incompatible.
holoviews 1.19.1 requires panel>=1.0, but you have panel 0.14.4 which is incompatible.

testing...

%%shell
apt-get remove python3-blinker
pip install blinker==1.8.2
pip install -Uq ipykernel==5.5.6 ipywidgets-bokeh==1.0.2

Use this at colab to install


PKI CI is failing

https://github.com/NationalSecurityAgency/skills-service/runs/4234846682?check_suite_focus=true

PR: #970


Feature request: Move completed items automatically

I would like that as soon as I check a task as complete, it moves automatically to the archive page.

Hello, @YousufSSyed! Thank you for your input! There is a new command in 0.10.0 that allows you to complete and archive a task under cursor (Toggle task done and archive it)

@ivan-lednev How exactly do I use it? Do I just run the command for each task I want to mark as complete?
What I want is for it to automatically archiv my tasks when I click the checkbox with my mouse.

@YousufSSyed, you can assign some hotkey to the command (e.g. ctrl + Enter), this way when the caret is on a line that contains a task, you press your hotkey, and the task is going to get checked & archived.
I'll reopen the ticket and investigate a way to do it the way you described.

since the tasks plugin now has statuses, just clicking will archive tasks that I move to a "in progress" state and aren't completed.  perhaps make this "when I click and the task is completed ('x') is set as status".

@pcause Are you saying you archive tasks with the Tasks plugin?

No.  I am saying that in your plugin you can't rely on click as was suggested.  tasks and other plugins use other symbols to indicate various states that are not "done".  Done means an x in the box.  What I was saying is nthat you have to see what the value in the box is set to before archiving.

Seconded, I think that it would be beneficial for the achiver to automatically trigger when I tick a box, rather than needing to be triggered manually every time.


Entering tables of values or matrices

Ideally there should be a question type which is for a table or matrix of values to be entered. It is very time consuming to use gapfills to do this.

Bringing this to the fore again as not only time consuming, but on Reveal the solution can look very messy.

The matrix part now exists!


High Severity Vulnerability

Installing this package via npm resulted in a high severity vulnerability. Result of npm audit are as follows:
                       === npm audit security report ===

                                 Manual Review
             Some vulnerabilities require your attention to resolve

          Visit https://go.npm.me/audit-guide for additional guidance

  High            Arbitrary File Write

  Package         decompress

  Patched in      No patch available

  Dependency of   download

  Path            download > decompress

  More info       https://npmjs.com/advisories/1217

found 1 high severity vulnerability in 1042 scanned packages
  1 vulnerability requires manual review. See the full report for details.

May I ask which version of xml2js are you using? Cuz i couldn't find decompress in my dependencies, which are:
<EMAIL_ADDRESS>C:\OpenSource\node-xml2js
+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| `--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   | `--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   | +--<EMAIL_ADDRESS>|   | +--<EMAIL_ADDRESS>deduped
|   | `--<EMAIL_ADDRESS>deduped
|   +--<EMAIL_ADDRESS>|   | +--<EMAIL_ADDRESS>|   | | +--<EMAIL_ADDRESS>|   | | +--<EMAIL_ADDRESS>|   | | +--<EMAIL_ADDRESS>|   | | `--<EMAIL_ADDRESS>|   | |   `--<EMAIL_ADDRESS>|   | `--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   | +--<EMAIL_ADDRESS>|   | +--<EMAIL_ADDRESS>|   | | +--<EMAIL_ADDRESS>deduped
|   | | +--<EMAIL_ADDRESS>|   | | +--<EMAIL_ADDRESS>|   | | `--<EMAIL_ADDRESS>|   | |   +--<EMAIL_ADDRESS>deduped
|   | |   +--<EMAIL_ADDRESS>|   | |   `--<EMAIL_ADDRESS>deduped
|   | `--<EMAIL_ADDRESS>|   |   +--<EMAIL_ADDRESS>|   |   | `--<EMAIL_ADDRESS>deduped
|   |   +--<EMAIL_ADDRESS>deduped
|   |   +--<EMAIL_ADDRESS>|   |   | `--<EMAIL_ADDRESS>deduped
|   |   +--<EMAIL_ADDRESS>|   |   | `--<EMAIL_ADDRESS>deduped
|   |   +--<EMAIL_ADDRESS>|   |   | +--<EMAIL_ADDRESS>deduped
|   |   | `--<EMAIL_ADDRESS>deduped
|   |   +--<EMAIL_ADDRESS>|   |   | `--<EMAIL_ADDRESS>deduped
|   |   +--<EMAIL_ADDRESS>|   |   +--<EMAIL_ADDRESS>|   |   `--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   | `--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   | +--<EMAIL_ADDRESS>|   | `--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   | `--<EMAIL_ADDRESS>deduped
|   `--<EMAIL_ADDRESS>deduped
+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>deduped
| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| `--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>deduped
| | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>deduped
| | | +--<EMAIL_ADDRESS>deduped
| | | +--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>| | |   `--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| |   +--<EMAIL_ADDRESS>deduped
| |   +--<EMAIL_ADDRESS>| |   `--<EMAIL_ADDRESS>deduped
| +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>deduped
| | +--<EMAIL_ADDRESS>deduped
| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>deduped
| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>deduped
| | `--<EMAIL_ADDRESS>| |   `--<EMAIL_ADDRESS>deduped
| +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| |   +--<EMAIL_ADDRESS>| |   | `--<EMAIL_ADDRESS>| |   |   `--<EMAIL_ADDRESS>| |   `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>| | | | +--<EMAIL_ADDRESS>| | | | `--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>deduped
| | `--<EMAIL_ADDRESS>deduped
| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>deduped
| | | `--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>| | |   +--<EMAIL_ADDRESS>| | |   `--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>deduped
| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| |   `--<EMAIL_ADDRESS>| |     `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>deduped
| | | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>| | | | `--<EMAIL_ADDRESS>| | | |   +--<EMAIL_ADDRESS>| | | |   | +--<EMAIL_ADDRESS>deduped
| | | |   | +--<EMAIL_ADDRESS>| | | |   | `--<EMAIL_ADDRESS>| | | |   |   `--<EMAIL_ADDRESS>deduped
| | | |   +--<EMAIL_ADDRESS>deduped
| | | |   `--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>deduped
| | | `--<EMAIL_ADDRESS>deduped
| | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>deduped
| | | +--<EMAIL_ADDRESS>deduped
| | | +--<EMAIL_ADDRESS>| | | | +--<EMAIL_ADDRESS>| | | | | `--<EMAIL_ADDRESS>deduped
| | | | +--<EMAIL_ADDRESS>deduped
| | | | `--<EMAIL_ADDRESS>deduped
| | | +--<EMAIL_ADDRESS>| | | | `--<EMAIL_ADDRESS>deduped
| | | +--<EMAIL_ADDRESS>deduped
| | | +--<EMAIL_ADDRESS>deduped
| | | +--<EMAIL_ADDRESS>deduped
| | | +--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>deduped
| | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>deduped
| | | `--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>deduped
| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>deduped
| | +--<EMAIL_ADDRESS>deduped
| | `--<EMAIL_ADDRESS>| |   `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>deduped
| | +--<EMAIL_ADDRESS>deduped
| | +--<EMAIL_ADDRESS>deduped
| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| |   +--<EMAIL_ADDRESS>| |   +--<EMAIL_ADDRESS>| |   | +--<EMAIL_ADDRESS>| |   | `--<EMAIL_ADDRESS>| |   +--<EMAIL_ADDRESS>| |   `--<EMAIL_ADDRESS>| |     +--<EMAIL_ADDRESS>| |     `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>deduped
| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>deduped
| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>deduped
| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>deduped
| | +--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>deduped
| | +--<EMAIL_ADDRESS>deduped
| | `--<EMAIL_ADDRESS>| |   `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>deduped
| | +--<EMAIL_ADDRESS>deduped
| | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>deduped
| | | `--<EMAIL_ADDRESS>| | |   +--<EMAIL_ADDRESS>| | |   | +--<EMAIL_ADDRESS>deduped
| | |   | +--<EMAIL_ADDRESS>| | |   | | +--<EMAIL_ADDRESS>| | |   | | | `--<EMAIL_ADDRESS>| | |   | | `--<EMAIL_ADDRESS>| | |   | +--<EMAIL_ADDRESS>| | |   | `--<EMAIL_ADDRESS>deduped
| | |   +--<EMAIL_ADDRESS>| | |   | +--<EMAIL_ADDRESS>| | |   | +--<EMAIL_ADDRESS>| | |   | | `--<EMAIL_ADDRESS>| | |   | +--<EMAIL_ADDRESS>deduped
| | |   | `--<EMAIL_ADDRESS>| | |   |   +--<EMAIL_ADDRESS>| | |   |   | +--<EMAIL_ADDRESS>deduped
| | |   |   | `--<EMAIL_ADDRESS>| | |   |   `--<EMAIL_ADDRESS>| | |   |     +--<EMAIL_ADDRESS>| | |   |     `--<EMAIL_ADDRESS>deduped
| | |   `--<EMAIL_ADDRESS>| | |     `--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>deduped
| | | +--<EMAIL_ADDRESS>| | | | `--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>| | |   +--<EMAIL_ADDRESS>| | |   | `--<EMAIL_ADDRESS>| | |   |   `--<EMAIL_ADDRESS>| | |   +--<EMAIL_ADDRESS>deduped
| | |   `--<EMAIL_ADDRESS>deduped
| | +--<EMAIL_ADDRESS>deduped
| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>deduped
| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>deduped
| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>deduped
| `--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   `--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>`--<EMAIL_ADDRESS>

@Omega-Ariston I'm using 0.4.23. I'm assuming this is the latest published version in npm.

@Omega-Ariston I'm using 0.4.23. I'm assuming this is the latest published version in npm.

Then we're using the same version. Would you mind providing the dependency list so that we can do a comparison? You can acquire it through the command npm list :)

Oh my bad. It was from another package I installed before this. I incorrectly thought this package had it as well. Sorry.


cmake: default to linking 'app' with the interface library 'FS'

This patch removes the need for application build script code to
explicitly link 'app' with a filesystem implementation.
It does this by introducing a zephyr interface library called 'FS'
that contains the usage requirements for linking with the filesystem
library subsys__fs and using Kconfig to default to linking the 'app'
library with this interface library.

Codecov Report

Merging #8533 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #8533   +/-   ##
======================================
  Coverage    63.8%   63.8%
======================================
  Files         426     426
  Lines       40890   40890
  Branches     6921    6921
======================================
  Hits        26089   26089
  Misses      11666   11666
  Partials     3135    3135

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 13eba39...93cb439. Read the comment docs.

in subsys/settings/src/CMakeLists.txt, is it necessary to keep following line:
zephyr_include_directories($ENV{ZEPHYR_BASE}/ext/fs/nffs/include)

recheck


Cast between array elements

Is your feature request related to a problem or challenge?
Arter Pull Request: https://github.com/apache/arrow-datafusion/pull/6384, In arrow-datafusion casting between arrays elements stopped working. The feature could be of great help when working with different types of data.
Describe the solution you'd like
All rules should be simillar to the cast function in arrow-rs (See https://docs.rs/arrow/latest/arrow/compute/kernels/cast/index.html)
For example, if we create new array with Int32Type and Int64Type should return Int64Array.
select make_array(Int32(1), Int64(2));
----
Int64(1), Int64(2)

Describe alternatives you've considered
No response
Additional context
Simillar Issues:
https://github.com/apache/arrow-datafusion/issues/6119
https://github.com/apache/arrow-datafusion/issues/6075
Simillar PR:
https://github.com/apache/arrow-datafusion/pull/6384

How to determine the final data type? Can with have a data_type argument in make_array(array, target_data_type) to check whether the elements are all possible to cast to target_data_type?

I will take this


Remove Pillow Dependency

Pillow is used as a dependency only insofar as it is used to load the font for fontio. I left that in place because I was thinking that by relying on the bitmap font library, it would create a circular dependency, but am now realizing that is not the case. CircuitPython uses the bitmap font library already, so it makes sense to list it as a dependency. Both this library and that would depend on Blinka, but that's it. The font file would also need to be included as a pcf file instead of pbm/pil files.

This ended up being a bit trickier than I thought because of a circular dependency of bitmap_font needing the Glyph class from fontio. Several pillow functions were also being utilized. In the end, this greatly simplified fontio because bitmap_font does pretty much all the heavy lifting.


fix: schema class can set Meta.unknown

This PR enables the Schema class of a model to determine the default unknown behavior. Without this, if you have a schema where you expect extra values to be present, every time deserialize or serialize is called, you must set unknown=INCLUDE. Now, in the Meta class of the Schema, unknown can be set to some value that will be respected as the default if not overrided by method parameter.
While I was at it, I also touched up some of the typing. I'm not fond of using @overload but I think it's the best way to address typing a method where a bool flag impacts the return type.

The circleci failure is a flake8 check complaining about missing docstrings on the @overload decorated methods. I don't think the overloads should be required to have a docstring since the primary method itself does. I'm also unable to get the check to fail locally :thinking: Does anyone with more knowledge of the circleci config than myself have a suggestion for a fix?

Codecov Report

Merging #1885 (a967fd0) into main (d407c48) will decrease coverage by 0.01%.
The diff coverage is 85.10%.

@@            Coverage Diff             @@
##             main    #1885      +/-   ##
==========================================
- Coverage   93.79%   93.78%   -0.02%
==========================================
  Files         539      539
  Lines       34084    34117      +33
==========================================
+ Hits        31970    31996      +26
- Misses       2114     2121       +7


Aarch64 and ppc64le

Fixes #9 9

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge/arm-arch @conda-forge/gtk2 This is ready for review/merge.


Installing a package with a hyphen results in picking the wrong package

This doesn't seem consistent, but it appears that pipenv is looking up packages differently than pip.  e.g. msgpack-python fails to install, but google-api-python-client works fine (resolving to the version with all underscores).
± pipenv install msgpack-python
Installing msgpack...
Collecting msgpack

Error:  An error occurred while installing msgpack!
  Could not find a version that satisfies the requirement msgpack (from versions: )
No matching distribution found for msgpack

Thanks for opening this @rjpower, this is in fact a byproduct of a bigger issue noted in #90, which I'm currently working on addressing. Due to some inconsistencies in how we're interacting with the pypi repository, certain packages are experiencing name mangling. We should hopefully have a fix for this soon.

Closing for #90


creative mode idea / command block idea

Describe your idea
You should be able to attach a command block on the back of a pocket computer if command block peripheral is turned on in the cc tweaked config
Describe alternatives you've considered if you've any
command block pocket computer 😳
Additional context
No response
Linked Issues
No response

also turtles should be able to have a command block attach to them

Idea: suggest to CC: Tweaked not a addon

I recommend submitting this directly to CC:Tweaked

the cc:tweaked dev, said not really any use, command pocket computer is useful, but they are too lazy to implement, would like if someone did it


api/social-image Serverless Function error

Description

Notion Test Page ID
This is the rootPageId: 22b650d3913540ddade133474cf9fc88
Build fails on Vercel. I've tried reducing the image quality setting, but whatever I try I always get "api/social-image" is 50.23mb in Vercel
🤔

I've removed the headless chrome OG approach in favor of Vercel's new edge-based OG image generation, which is quite a bit faster, simpler, and best of all doesn't include any additional dependencies. https://github.com/transitive-bullshit/nextjs-notion-starter-kit/pull/383


More style-break-free text substitution

For mixed style text, to keep an EL {....} to be one "run" structure is very frustrating work.
How about handle EL which is divided into some "run"s like
run[0].text = 'Hello, {'
run[1].text = 'contry'
run[2].text = '} !!'

with the key 'contry' has value "World",
to
run[0].text = 'Hello, World'
run[1].text = ''
run[2].text = '!!'

closed by #14


Should be good to know size of panel at anytime

For exemple: knowing if the panel is expanded or not

I think that's possible by changing the access control of the property private var currentStickyPointIndex: Int to open.
Would this solve your problem?

Closing due to inactivity


Improved the unique validation rule test cases

Resolves some inconsistencies noticed by @Flynsarmy.
@bennothommo @Flynsarmy if you guys could do a sanity check on this for me please that would be great.

@bennothommo is this good to go?

@LukeTowers yep, I believe so. :)


[$250] Task - Users avatars change to default in share somewhere after assigning task from FAB

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.74-0
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail: n/a
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
Action Performed:

Open the app and log in
Tap FAB > Assign task
Create a task and share it somewhere
Return to LHN
Tap FAB > Assign task
Enter title and proceed to the confirmation screen
Tap Share somewhere

Expected Result:
Users' avatar are displayed next to reports
Actual Result:
Users' avatars change to default ones in Share somewhere after creating a task the first time
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/38435837/bb3cd6a0-50b1-4f33-8f0d-d3ba07ea1629

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01cc2420bf539eea63
Upwork Job ID:<PHONE_NUMBER>287768064
Last Price Increase: 2024-05-15

Issue OwnerCurrent Issue Owner: @fedirjh

@NikkiWines FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to  #vip-vsb

This can definitely be external, I'll check the deploy diff to see if we modified something related to tasks or avatars in the latest deploy.

Seems like something that miiight be tied to the recent participants migration (https://github.com/Expensify/App/pull/40254), thoughts @s77rt @marcaaron?

Oh, I thought we deployed that already haha ::puts on game face:: 😅

It definitely could be related - hard to say without digging into the code.

I'm checking that atm

Not related to the participants migration. Offending PR: https://github.com/Expensify/App/pull/41636/files#r1602409674
cc @tienifr @c3024

Fix is in staging and works well!


Upload base64 encoded image

Doesn't seem like the uploadMedia method supports base64 encoded images - is there an alternative way to use this type of image?

Hi,
Why should uploadMedia support base64?
It already support binary through Buffer objects. It is more efficient to send than base64 and if you get media through base64 format, you can convert easily to binary using a decoder.

@alkihis
Hey, so I've been trying to upload base64 image but still no luck, here is the string I have, which opens in browser (simple white image)
const image = 'data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAZAAAADICAYAAADGFbfiAAAAAXNSR0IArs4c6QAABxZJREFUeF7t1bENAAAIwzD6/9P8kNnsXSyk7BwBAgQIEAgCCxsTAgQIECBwAuIJCBAgQCAJCEhiMyJAgAABAfEDBAgQIJAEBCSxGREgQICAgPgBAgQIEEgCApLYjAgQIEBAQPwAAQIECCQBAUlsRgQIECAgIH6AAAECBJKAgCQ2IwIECBAQED9AgAABAklAQBKbEQECBAgIiB8gQIAAgSQgIInNiAABAgQExA8QIECAQBIQkMRmRIAAAQIC4gcIECBAIAkISGIzIkCAAAEB8QMECBAgkAQEJLEZESBAgICA+AECBAgQSAICktiMCBAgQEBA/AABAgQIJAEBSWxGBAgQICAgfoAAAQIEkoCAJDYjAgQIEBAQP0CAAAECSUBAEpsRAQIECAiIHyBAgACBJCAgic2IAAECBATEDxAgQIBAEhCQxGZEgAABAgLiBwgQIEAgCQhIYjMiQIAAAQHxAwQIECCQBAQksRkRIECAgID4AQIECBBIAgKS2IwIECBAQED8AAECBAgkAQFJbEYECBAgICB+gAABAgSSgIAkNiMCBAgQEBA/QIAAAQJJQEASmxEBAgQICIgfIECAAIEkICCJzYgAAQIEBMQPECBAgEASEJDEZkSAAAECAuIHCBAgQCAJCEhiMyJAgAABAfEDBAgQIJAEBCSxGREgQICAgPgBAgQIEEgCApLYjAgQIEBAQPwAAQIECCQBAUlsRgQIECAgIH6AAAECBJKAgCQ2IwIECBAQED9AgAABAklAQBKbEQECBAgIiB8gQIAAgSQgIInNiAABAgQExA8QIECAQBIQkMRmRIAAAQIC4gcIECBAIAkISGIzIkCAAAEB8QMECBAgkAQEJLEZESBAgICA+AECBAgQSAICktiMCBAgQEBA/AABAgQIJAEBSWxGBAgQICAgfoAAAQIEkoCAJDYjAgQIEBAQP0CAAAECSUBAEpsRAQIECAiIHyBAgACBJCAgic2IAAECBATEDxAgQIBAEhCQxGZEgAABAgLiBwgQIEAgCQhIYjMiQIAAAQHxAwQIECCQBAQksRkRIECAgID4AQIECBBIAgKS2IwIECBAQED8AAECBAgkAQFJbEYECBAgICB+gAABAgSSgIAkNiMCBAgQEBA/QIAAAQJJQEASmxEBAgQICIgfIECAAIEkICCJzYgAAQIEBMQPECBAgEASEJDEZkSAAAECAuIHCBAgQCAJCEhiMyJAgAABAfEDBAgQIJAEBCSxGREgQICAgPgBAgQIEEgCApLYjAgQIEBAQPwAAQIECCQBAUlsRgQIECAgIH6AAAECBJKAgCQ2IwIECBAQED9AgAABAklAQBKbEQECBAgIiB8gQIAAgSQgIInNiAABAgQExA8QIECAQBIQkMRmRIAAAQIC4gcIECBAIAkISGIzIkCAAAEB8QMECBAgkAQEJLEZESBAgICA+AECBAgQSAICktiMCBAgQEBA/AABAgQIJAEBSWxGBAgQICAgfoAAAQIEkoCAJDYjAgQIEBAQP0CAAAECSUBAEpsRAQIECAiIHyBAgACBJCAgic2IAAECBATEDxAgQIBAEhCQxGZEgAABAgLiBwgQIEAgCQhIYjMiQIAAAQHxAwQIECCQBAQksRkRIECAgID4AQIECBBIAgKS2IwIECBAQED8AAECBAgkAQFJbEYECBAgICB+gAABAgSSgIAkNiMCBAgQEBA/QIAAAQJJQEASmxEBAgQICIgfIECAAIEkICCJzYgAAQIEBMQPECBAgEASEJDEZkSAAAECAuIHCBAgQCAJCEhiMyJAgAABAfEDBAgQIJAEBCSxGREgQICAgPgBAgQIEEgCApLYjAgQIEBAQPwAAQIECCQBAUlsRgQIECAgIH6AAAECBJKAgCQ2IwIECBAQED9AgAABAklAQBKbEQECBAgIiB8gQIAAgSQgIInNiAABAgQExA8QIECAQBIQkMRmRIAAAQIC4gcIECBAIAkISGIzIkCAAAEB8QMECBAgkAQEJLEZESBAgICA+AECBAgQSAICktiMCBAgQEBA/AABAgQIJAEBSWxGBAgQICAgfoAAAQIEkoCAJDYjAgQIEBAQP0CAAAECSUBAEpsRAQIECAiIHyBAgACBJCAgic2IAAECBATEDxAgQIBAEhCQxGZEgAABAgLiBwgQIEAgCQhIYjMiQIAAAQHxAwQIECCQBAQksRkRIECAgID4AQIECBBIAgKS2IwIECBAQED8AAECBAgkAQFJbEYECBAgICB+gAABAgSSgIAkNiMCBAgQEBA/QIAAAQJJQEASmxEBAgQICIgfIECAAIEkICCJzYgAAQIEBMQPECBAgEASEJDEZkSAAAECAuIHCBAgQCAJCEhiMyJAgAABAfEDBAgQIJAEBCSxGREgQICAgPgBAgQIEEgCApLYjAgQIEDgAQdcAMlo3X8zAAAAAElFTkSuQmCC'
So, I'm taking this string and creating buffer like this:
const bufferedImage = Buffer.from(image, 'base64');
And, then uploading media like this:
const mediaId = await uploadClient.v1.uploadMedia(bufferedImage, { type: 'png' });
but it returns that wrong media type has been uploaded, can you give me a tip how to upload base64 image? (or why this image is rejected)

You try to decode a content type prefix data:image/png;base64,, that isn't part of the file 😅
Just take the real base64 part of the string (after the first comma).
It should be fine this way.

@alkihis
Ah, I think I tried this way too but had another problem which I forgot, thanks, will try it like this!

Let me know if you still encounter an issue. Buffer media upload works (it is unit tested with real files :D), but your media file can be not supported by Twitter, or their is a problem in your base64 string.

@alkihis
Basically, I'm trying to capture canvas as an image and post it with twitter.
I'm capturing it like canvas.toDataURL('image/png') and I will trim starting string, no problem. I have just one big problem, I wanted to use only api v2 as they have scopes and user won't be scared by using my application that I can do anything with their accounts, but I can't upload media like this, right? Do you know any way to upload media without v1.1 access?

For now, v2 do not provides a way to upload medias 😔 You need to use OAuth 1.0a and v1.1 to do this, sorry!


ci: bump eksctl version

Fixes #N/A
Description
Bumps the eksctl version used in CI to v0.191.0 to support EKS 1.31.
How was this change tested?
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.291%

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
5578

Relevant Lines:
6697

💛  - Coveralls


Trying use @babel/plugin-syntax-optional-chaining

Hey there,
I'm trying to include @babel/plugin-syntax-optional-chaining in my project, after install it, I've added to preact.config.js:
const babelLoader = helpers.getLoadersByName(config, 'babel-loader')[0];

babelLoader.rule.options.plugins.unshift(require.resolve('@babel/plugin-syntax-optional-chaining'));

and tested with the following code:
export default function() {
	const a = {
		b: 1
	};

	return (

{a?.b}

	);
}

the expected babel behaviour is convert:
a?.b -> a && a.b
but the following error has been returning:
✖ ERROR ./src/components/App.js
Module build failed: SyntaxError: Unexpected token (7:9)

   5 |
   6 | 	return (
>  7 |
{a?.b}

     | 		       ^
   8 | 	);
   9 | }
  10 |

Someone can help me? Thanks

you might want https://github.com/babel/babel/tree/master/packages/babel-plugin-proposal-optional-chaining

that package looks to have support only for babel 7 and after, preact-cli need to be updated to newest versions, I'm keep using lodash/get , closing for now!

that package looks to have support only for babel 7

yes, optional chaining support is available from babel 7 onwards only.

preact-cli need to be updated to newest versions

we are waiting for the stable version release. Once that comes out, we will be updating it 😄

good to know! thank u


Support running PCL tests

Not sure if this is still a requirement. Moving to backlog, but feel free to close if you no longer require this.

Closing. We'll figure out our own (EF-local) strategy for testing on various platforms and runtimes.


LanguageData and Queries

This project makes using tree-sitter much easier. But, one big missing piece is query definitions.
As far as I can tell, resources added to the subframeworks in a xcframework are not easily accessible from package clients. Further, the binaryTarget does not support bundling resource. So, I opted for a new, regular target that does allow resources.
Since I was already doing this, I created a simple enum type LanguageData that gives you very easy access to the bundled parsers and their queries. This also give you easy access to both compile- and runtime-information about what languages are available via this package. But, of course, I'm open to other ideas!
usage:
import tree_sitter_language_resources

let swift = LanguageResource.swift

let url = swift.highlightQueryURL
let parserPtr = swift.parser

Renamed to LanguageResources, and adding a little bit to the README.

thank you!


XMSS/XMSS^MT testfiles are using deprecated StateAwareSignature-class

I'm testing PQC-Signatures and worked with XMSSTest.java and XMSSMTTest.java (found in folder https://github.com/bcgit/bc-java/blob/master/prov/src/test/java/org/bouncycastle/pqc/jcajce/provider/test/) but the StateAwareSignature-class is deprecated since version 1.64.
Could you kindly share some examples with the new extraKeyShard() method as I cannot find any documentation and/or examples?
Thanks in advance.
Best regards
Michael

org.bouncycastle.pqc.jcajce.provider.test.XMSSTest.testShardedKeyExhaustion() is probably a good place to start.

Closing as resolved.


Resolve internal server error

Ticket No

https://trello.com/c/26CTbZVe/5-resolve-internal-server-error

Summary of changes
-Added a line of code in src-data(type.tsx) and DashboardFilter
How should this be tested? [Screenshots, Video or Type instructions]

After logging onto the dashboard, reload the page and the internal server error should not be appearing again.

Notes for reviewers

Out of scope

Combined into this single PR


Make IS_NODE check more failure-proof

Description

The node-libs-browser project (from Webpack) mocks the process object, but leaves process.versions undefined. This causes the IS_NODE check to throw an error.

For repository owners only:
Please remember to apply all applicable tags to your pull request.
Tags: FEATURE, BREAKING, BUG, PERF, DEV, DOC, SECURITY
For more info see: https://github.com/tensorflow/tfjs/blob/master/DEVELOPMENT.md

This change is

Thank you!


Move wiki

Description
Move the wiki to markdown files.
Before merging: please put links to the files on the wiki pages they are replacing.
Closes #1722

I think you should also edit this here https://github.com/Chatterino/chatterino2/blob/8e202c37556ae969718d9be6e736e700be647bb1/src/widgets/helper/EditableModelView.cpp#L99-L100

In my opinion fourtf should change that redirect to the file, since changing this in a new version would break the link in older ones. If the redirect is changed the link will still work in older versions.
@fourtf I summon you to this conversation ;)

@Mm2PL Just change the url :+1:
Wiki shouldn't be purged either, it should just maybe link to the markdown version at the top in big text


Is this plugin still needed now eslint-plugin-babel supports class properties?

It seems babel/semi might make this defunct?

Probably not. If you have time, feel free to update the settings if you can verify that it won't modify weaken the existing linting, but consider it non-essential and our sprint tickets top priority.

Yes it does. Will deprecate it.


Create HTML5 version

The project builds and runs on HTML5 but it is not really useful:

Add ability to save a file. It should be possible to use something like this:

    <script>
        // https://stackoverflow.com/a/18197341/1266551
        function download_as_file(text) {
             var filename = "defold.convexshape";

            var element = document.createElement('a');
            element.setAttribute('href', 'data:text/plain;charset=utf-8,' + encodeURIComponent(text));
            element.setAttribute('download', filename);
            element.style.display = 'none';
            document.body.appendChild(element);
            element.click();
            document.body.removeChild(element);
        }
    </script>

Add ability to load an image. Use HTML5 Drag and Drop File API? https://developer.mozilla.org/en-US/docs/Web/API/HTML_Drag_and_Drop_API/File_drag_and_drop

I will try to add save file support and submit a PR soon.

PR for saving: https://github.com/rgrams/defold_polygon_editor/pull/6


Test client does not handle WSGI iterator correctly

As described in the new relic forums [1] it seems that tests run with certain libraries such as the New Relic Python agent expose a bug. The client doesn't convert an iterable to an iterator.
I've tried this with versions
Flask==0.10.1
Werkzeug==0.9.6
[1] https://discuss.newrelic.com/t/werkzeug-cant-work-properly-when-using-agent-2-40-0-34/13318/4

There is nothing on that page.  Can you post the relevant description of the problem here rather than linking to another place?

@davidism Scroll down to Graham's response.
This is definely a bug.

It must be my browser, there is literally no content on the linked page.

Just for clarity, with a particular WSGI application, a NewRelic customer encountered this particular traceback:
  File "/Users/kakashi/Programs/project/.env/lib/python2.7/site-packages/werkzeug/test.py", line 873, in run_wsgi_app
    buffer.append(next(app_iter))
TypeError: _WSGIApplicationIterable object is not an iterator

As noted further down in the thread, this happens because werkzeug.test.run_wsgi_app expects an WSGI apps' return value to be an iterator, while according to the WSGI spec, it might as well be an iterable.
--
However, simply returning an iterable isn't enough for this bug to be triggered, the start_response call must also happen after the iterable has been returned.

See #713


ParticleEmitter.update doesn't accept negative values

https://github.com/libgdx/libgdx/issues/3963
I'm trying to make an Action that produce particles, this is successful with the default liner Interpolation, however, using other Interpolations like Bounce would fail because the calculation of the delta will be sometimes negative ! i.e. this will not work (ParticleEmitter.update(-0.16)) and will freeze the particles for each frame with such value.
It would have also been a great if we can just set the time, i.e. setTime(...);
Reproduction steps/code
@Override
  protected void update(float percent) {
      float particleEffectTime = getDuration() * percent;
      float delta = particleEffectTime - particleEffectLastTime;
      for (int i = effects.size - 1; i >= 0; i--) {
          PooledEffect effect = effects.get(i);
          if (percent == 1.0f) {
              effect.allowCompletion();
          } else {
              effect.setPosition(getCenter().x, getCenter().y);
              /**
               * This can be negative because of the Interpolation like Bounce !
               */
              effect.update(delta);
          }
      }
      particleEffectLastTime = particleEffectTime;
      this.percent = percent;
  }

Found and fixed


Extra bussnes day in a week

Hi
depend on locale i have 5 (work day)+ 2 (holiday) extra holiday i add via
var days = [day, day ...]
moment.updateLocale('ru', {holidays: days, holidayFormat: 'MM.DD.YYYY'});
But some week in a month have 6 + 1(holiday), so is it possible add extra work day for certain day?

If I understand correctly you want to define custom business days just like you specify custom holidays.
Right now I don't think it's possible, without specifying always 6 business days and then for the remaining 11 months specify custom holidays for the weekend.

Yes you right.
I little bit drill source and find out it :( So work should be done and i modified one function similar functionality isHoliday.
`moment.fn.isBusinessDay = function () {
var locale = this.localeData();
var defaultWorkingWeekdays = [1, 2, 3, 4, 5];
var workingWeekdays = locale._workingWeekdays || defaultWorkingWeekdays;
if (locale._businessdays) {
if (locale._businessdays.indexOf(this.format(locale._businessdayFormat)) >= 0) {
return true;
}
}
if (this.isHoliday()) return false;
if (workingWeekdays.indexOf(this.day()) >= 0) return true;
return false;
};
moment.updateLocale('ru', { holidays: hdays, holidayFormat: 'MM.DD.YYYY', businessdays: bdays, businessdayFormat: 'MM.DD.YYYY' });
`
May be not good code but works for me


perf(debounce): remove tryCatch/errorObject for custom tryCatching

This PR applies approach to remove custom trycatcher on debounce operator.
Numbers are
before
debounce - immediate |  5,530 (±7.51%) |      23,628 (±3.99%) |   4.27x |   327.3%

after
debounce - immediate |  5,593 (±9.91%) |      27,302 (±2.89%) |   4.88x |   388.1%

not sure if this can be considered noticeable improvement.

I'll check this in around later today if there isn't any suggestion.

Updated PR to separate trycatch as function.

Merged with 90bf3f1.


Diagram: Add fillto property to part & scene

Add fillto property to part & scene:

Loading & saving the property

There is already filler tag, see issue #85
What we want, we want edit those fillers in the Diagram view: changing the padding, adding and removing fillers in specific parts.

Filler elements were replaced by container target word counts. There is need to edit them in Arc View, but that is done in separate issue.


请问能否写清楚如何利用自己的输入做模型推理，没看明白doc

谢谢您的建议，我们稍后更新一下ReadMe :)

谢谢您的建议，我们稍后更新一下ReadMe :)

Thanks to the scientists for their work. I am looking forward to giving the documentation, how should I deploy and use it as an ordinary developer?
I hope my 16g GPU can reason.

Hi, we have updated the readme.
To change sampling prompts, modify the txt file passed to --prompt-path


Show modals on source collapse

Closes #1187

Thanks for the PR ~ I took your change and integrated it into #1193 since I'm moving the underlying file to another location


Fix ./set_env_vars.sh script for Linux

Before opening, please confirm:

[X] I have searched for duplicate or closed issues and discussions.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

Bug Category
Other
Describe the bug
Per Matt's comment, #38 broke the cedar-drt/set_env_vars.sh script on Linux. (Although it currently works as expected on macOS.)
Also: the CI didn't catch this because the CI doesn't directly use the set_env_vars.sh script. We need to update ci.yml to use the same scripts we mention in the README for consistency.
From Matt's comment:

This is not working for me when I follow the instructions to run DRT. I get the usual error: fuzz/target/x86_64-unknown-linux-gnu/release/rbac: error while loading shared libraries: libjvm.so: cannot open shared object file: No such file or directory

Indeed, you are correct. Looks like the LD_LIBRARY_PATH code isn't doing anything on my macOS (i.e., DRT runs fine without it). So just skipping the code in the case of macOS sounds like a good solution to me.
Would it be feasible to build on macOS in our CI in addition to Linux? Either way, let's also add a note about what OS we build on for CI so users know what system they should use.

Closing this issue since it's stale. Our build has changed significantly since originally filed.


Unable to copy large part of world

Greeting, i am trying to copy a part of my world but mcedit is unable to copy files that is superior to 4 gb. An error message appear and tell me to use ZIP64. I made some research and are unable to find a solution for it (i really dont want to copy small part of my world)
I really need help to know how to use Zip64 for McEdit.
Thank you very much

Can you give us a screenshot of the console window? And what version of MCEdit are you using?

I use the <IP_ADDRESS> version, i cant get the screen now because the select-all --> copy take 20 hours.
I tried it 2 times : one with default setting and another with the cache at 50000 MB, same message
Therefore there is some information on zip64 on mcedit on this page : http://www.minecraftforum.net/forums/mapping-and-modding/minecraft-tools/1260525-mcedit-minecraft-world-editor-now-open-source?page=451

I think I know what the issue is. I need you to send me the map which I know will be an hard thing to do, but I hope you can try using dropbox or something, otherwise I can't test it...

@naor2013 I send you the download link via email

Just giving an update, I think I fixed it, I'm testing at the moment, it will take a few hours.
BTW, explaining that cache setting you tried to change, what it means is basically asking you how much of your ram do you agree to use for copying the map before you want it to be saved in your computer as a zip file. So, even setting it to 50000 MB won't work since you don't actually have 50 GB of ram.
Just out of curiosity, why are you actually trying to copy a 32GB map for?

Great news
I understand know for the cache thanks you
For the copy it is to transpose it in a void world, deleting water and do other stuff like deleting tilesticks

Hi, thank you for adding this in the commit.
I tried to changed the old file with the new file and i get this error :
Console --> http://pastebin.com/vsPeFgb3
Msg in mcedit --> http://prntscr.com/efgbhx
Is it because i changed only the shematic.py file ?

Are you running from source?
And did you see his second commit? The one after this one? He changed one more line that could be the cause of your issue


Docker-compose setup

A Docker-compose setup for local development will drastically simplify the process of getting the project up and running, especially with external data systems like Qdrant.

Closing due to Chidori having changed substantially


Start screen is erased when nvim is opened and this plugin loads

The start/help screen that displays in the middle of the window when you first open nvim is erased immediately when this plugin is loaded. Typically this is a misbehaving redraw or autocmd handler, but haven't been able to pinpoint exactly where it's occurring.
-- plugins.lua (using packer.nvim)

use {
    'jose-elias-alvarez/buftabline.nvim',
    --opt = true,
    requires = 'kyazdani42/nvim-web-devicons',
    config = [[require'config.buftabline']]
}

Toggling opt (so disabling it on start) fixes the issue, so I know the issue is this plugin somehow.

You're right, and it looks like it's the initial render triggered by BufEnter. However, it looks like changing tabline after Vim / Neovim has started will clear the start screen. You can try it by running nvim -u NONE -c "set showtabline=2", then calling :set tabline="".
It looks like setting tabline at startup (from the command line or init.vim) keeps the start screen, but since all of the plugin's updates are scheduled, it'll always get cleared shortly after start. Not sure what we can do about this, and I don't know if I consider it a priority, but I'm open to fixes.


interesting people RPC initial implementation CORE-5414

Here is an initial implementation of the "interesting people" RPC to be used for Search V3. This implements the following algorithm:

Obtain lists of users (ranked by score within list). For this specific implementation we use the following lists:

Recent chat participants - Ordered based on the number of conversations each participant show up in, weighted by the recency of the last message sent in that conversation.
People you follow. This is mostly to bolster the size of the result, these are given much less weight than chat activity.

Assign each list a weight.
For each user in the union of all the lists, obtain a score by computing the rank in each list multiplied by the weight of that list, and summing all these value up for all lists. This should hopefully allow us to add more types of lists (KBFS, viewing profiles) without much trouble.

The rank is computed as the position in the list, divided by the size of the list.

I implemented most of this in the service package because it was the easiest place to not get any circular dependencies with the chat package.
cc @malgorithms

Also, cc @chrisnojima


Spelling correction

Spelling correction
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Quality checks

[ ] My content adheres to the style guidelines
[ ] I ran make test or make e2e-test and it was successful

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Appreciate you picked up on this @mkarakas


about TaskInfo KillPolicy property

mesos TaskInfo message  protobuf include “KillPolicy”
but your action neglect this property, which might lead to difference in executing killtask action
mesos TaskInfo message  protobuf
https://github.com/apache/mesos/blob/964bab98372c51185063f78d5b4f09cca03ed8a7/include/mesos/mesos.proto#L1284

Thanks for this. Looks like it hasn't landed in a mesos point release yet, but probably will in mesos 0.29


[WebToolsE2E][Aspire] After updating to Aspire 8.0 Preview 3, opening eShopLite.sln fails with: "Package downgrade detected: Microsoft.Extensions.Options from 8.0.1 to centrally defined 8.0.0"

REGRESSION INFO: Worked fine with Aspire 8.0.0-preview.2.23619.3
INSTALL STEPS

Clean machine: Win11 x64 23h2 ENU
Install VS 17.10 Preview 1[34511.213.main]
Install Aspire Preview 3 latest build 8.0.0-preview.3.24064.1
Apply NuGet Feeds
Install Docker Desktop

REPRO STEPS

Clone aspire-samples repo from main branch
Update NuGet.config to use the dotnet8 feed

Update Directory.Packages.props to use aspire version 8.0.0-preview.3.24064.1
Update global.json to use sdk version 8.0.200-preview.24060.48
Open solution eShopLite.sln

NOTE

This issue also reproduces on Metrics / AspireWithNode solutions.

ACTUAL
NU1109	Detected package downgrade: Microsoft.Extensions.Options from 8.0.1 to centrally defined 8.0.0. Update the centrally managed package version to a higher version.
eShopLite.ServiceDefaults -> Microsoft.Extensions.ServiceDiscovery.Abstractions 8.0.0-preview.3.24064.1 -> Microsoft.Extensions.Options (>= 8.0.1)
eShopLite.ServiceDefaults -> Microsoft.Extensions.Options (>= 8.0.0)

EXPECTED
Opening solution without any errors.
OTHER INFO

I don't think this is a bug in aspire. Updating a project to a new version of Aspire is more than just updating Aspire.Hosting package versions. I do agree that the samples should be changed to target the latest shipped servicing (meaning reference 8.0.1) which would get rid of this warning.
In general, I'm expecting that going forward each time that we update the version of Aspire that the samples repo depends on we should ensure that we are also updating the platform so that it at least matches the version that Aspire references, if not newer.

@joperezr I investigated this some more today. I'm unable to get rid of this warning even when I update the runtime versions to 8.0.1 in Directory.Packages.props file of this repo.
From solution explorer, it seems to indicate that Aspire.Hosting 8.0.0-preview.3.24071.6 depends on Microsoft.Extensions.Hosting 8.0.0 depends on Microsoft.Extensions.Options 8.0.0. But other dependencies such as Aspire.StackExchange.Redis 8.0.0-preview.3.24071.6 depend on the 8.0.1 versions. Therefore, we see the downgrade warnings.
Question: Does the Aspire.Hosting package need to be updated to depend on the 8.0.1 versions?

@balachir that error is saying that you still have Microsoft.Extensions.Options set to 8.0.0 in Central Package Management.

@DamianEdwards I'm unable to figure this out. With clean install of latest build of VS 17.10 P1, we get SDK 8.0.200-preview with 8.0.0 runtime. But VS 17.10 P1 will actually ship with SDK 8.0.200 with 8.0.2 runtime. So, I installed a build of SDK 8.0.200 with 8.0.2 runtime on my machine, pinned SDK version to 8.0.200 in global.json and updated all the versions in my Directory.Packages.props to 8.0.2.

But I still get that same downgrade error

What feeds are you using? Are there any other NuGet messages (info, warning)?

Note that the Microsoft.Extensions.* packages do not version together, e.g. right now there is no version of Microsoft.Extensions.Hosting higher than 8.0.0 on nuget.org, despite there being a Microsoft.Extensions.Options with version 8.0.1. It's likely you'll have to manually configure versions for each Microsoft.Extensions.* package and get rid of the RuntimeVersion property.

@DamianEdwards @davidfowl I tried explicitly setting the version of Microsoft.Extensions.Options to 8.0.2 but I still can't get eshopLite to build. Other samples in the aspire-samples repo also have this same problem.
This is blocking all our validation using the samples from aspire-samples repo. Can you try it out and help unblock us? I've added the preview 3 milestone label to this issue for better visibility.

After some more trying, I got the solutions in aspire-samples repo to build successfully now. Following are the changes that I needed to make for Aspire 8.0 P3.

Update global.json to point to .NET SDK 8.0.200

Update some versions in Directory.Packages.props

Tried to make following changes with latest Aspire 8.0.0-preview.3.24105.21, now can build solutions successfully.

Update global.json

Update nuget.config to use the dotnet8rtm feed and dotnet-tool feed.

Update Directory.Packages.props

Fixed with #108. Closing this issue.


Expose cache size as an operational metric

Codecov Report

Merging #398 (02e5c20) into main (203a766) will increase coverage by 0.75%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #398      +/-   ##
==========================================
+ Coverage   60.95%   61.70%   +0.75%
==========================================
  Files          91       91
  Lines        6300     6403     +103
==========================================
+ Hits         3840     3951     +111
+ Misses       2225     2215      -10
- Partials      235      237       +2

Flag
Coverage Δ

unittests
61.70% <100.00%> (+0.75%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/pipeline/encode/encode_prom.go
78.03% <100.00%> (+0.26%)
:arrow_up:

pkg/pipeline/extract/aggregate/aggregates.go
90.19% <100.00%> (ø)

pkg/pipeline/utils/timed_cache.go
96.03% <100.00%> (+0.29%)
:arrow_up:

pkg/pipeline/extract/conntrack/store.go
79.11% <0.00%> (-3.29%)
:arrow_down:

pkg/api/conntrack.go
88.81% <0.00%> (-0.87%)
:arrow_down:

pkg/pipeline/utils/multiorderedmap.go
100.00% <0.00%> (ø)

pkg/pipeline/extract/conntrack/metrics.go
100.00% <0.00%> (ø)

pkg/pipeline/decode/decode_protobuf.go
77.55% <0.00%> (+0.46%)
:arrow_up:

pkg/pipeline/extract/conntrack/conntrack.go
93.37% <0.00%> (+1.11%)
:arrow_up:

pkg/pipeline/extract/conntrack/conn.go
78.15% <0.00%> (+2.45%)
:arrow_up:

... and 3 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Added support for bytes buffer pool

Added support for bytes buffer pool, which allows us to cache buffers that have been allocated but no longer used to reduce GC pressure.
In istio, I think most places that use strings.Builder and bytes.Buffer can be modified to use this buffer pool. As an experiment, I modified a few places in this PR. Benchmark see the CR comments.

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test all

@wulianglongrd: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
c950420147cde447b4cae0613edcdeb29ebdb525
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@wulianglongrd: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
703e0db18ae1eea5b6b84c25b179eeeb7f4c0a47
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@wulianglongrd: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
703e0db18ae1eea5b6b84c25b179eeeb7f4c0a47
link
true
/test unit-tests-arm64

unit-tests_istio
703e0db18ae1eea5b6b84c25b179eeeb7f4c0a47
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

/test unit-tests-arm64

Is there a practical profile showing that this is the bottleneck

The simple answer is, "No."
To be honest, this PR is a bit far-fetched. Here’s how it came about: We noticed that the serviceMatchingListenerPort function was consuming 20% of memory (fixed in #53281). While addressing that issue, I stumbled upon AddConfigInfoMetadata(I revert this change due to the comments above saying it was overly complicated), which was using about 2% of memory (they were close on the flame graph). Yes, 2% is almost negligible, but since fixing it seemed straightforward and I’d recently seen buffer pool optimizations in various open-source projects, so.... I created this PR. Looking at it in isolation, its overall optimization impact is minimal, but it might be useful elsewhere or as an option for similar issues in the future—that's all there is to it.

I am concerned this is not a hardcoding implement with an assumption of the buffer size.

Can you give an example to explain this? I didn't understand.

Thanks for the review, I don't feel confident enough to move this PR forward, so I'm closing it. 😥


fix(collectors): do not process resource if no subscribers are available

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

/kind flaky-test

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area library

/area broker

/area protocol-buffers

/grafana-dashboard

/area collectors

/area tests

/area examples

What this PR does / why we need it:
When processing a resource, if there are no subscribers subscribed for it then remove it from the cache if present and return.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

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

Needs approval from an approver in each of these files:

OWNERS [alacuku]

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

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

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,alacuku]

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


Device Themes

The ability for users to switch between the different iPod Classic generation themes (1-7).
A nice to have would also be the ability for them to upload their own skins/mix n' match themes.

I think for the upload skins thing. I think the only real way it makes sense is "creating a custom file format" that is really just a zip. with maybe just the background the clickwhell and a json file with the name and auther and date etc, also depending on how far you want to go with assets you could make the entire thing a bunch of images so you could skin everything down to the selected gradients.

Was gonna try to play around with themes next week if you have any advice on how the skins get implemented. (First I need to setup iOS Dev Env, I've never done any Swift!)

Was gonna try to play around with themes next week if you have any advice on how the skins get implemented. (First I need to setup iOS Dev Env, I've never done any Swift!)

The app's built with Flutter, so no Swift/SwiftUI is being used here! But absolutely would appreciate any help/ideas on this :)


:lock: Remove plaintext saved password support

Problem Statement
OKTA_PASSWORD is not stored securely. Having this feature in is a
security risk for users.
Solution

Remove documentation for OKTA_PASSWORD

Remove support for OKTA_PASSWORD

Resolves #171
Will revisit with support for a password command as recommended in #12

@loopingz can you give this a code review for me?


Explanation of usage of functional components

In this project milestone, I was meant to refactor my code to functional components although I used the functional components right from the beginning.

Having plenty of experience with react, I just typed in rafce and an extension that I have installed known as intelisense completes the functional component.
I then import calculate from logic/calculate then use the useState hook which uses an object to store the state of the numbers and operations.
I use object destructuring to access the individual properties of the object.
Finally I place all the JSX in a return statement. An opening bracket has to be on the same line as the return then all the JSX has to be wrapped in one overall tag.

Here is a screenshot showing how the functional component appears

STATUS: APPROVED ✔ ✔
Hi @Fene-87,
✔ Implementation of requirements
Your project is complete!  There is nothing else to say other than... it's time to go ahead with the project :shipit:
Congratulations! 🎉
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the Issue thread if something is not 100% clear. And please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Hey @TedLivist Thank you for your time.


Localization of the status page

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

Description
When you change the default language element to another and then back to English, the element with the class "data-v-7d4a7f28" does not change its value.
👟 Reproduction steps

You need to change the default language, for example, to your native language.
Set the status page, add "Services".
Go back to the settings and set the default language (English)
Go to the status page.
The Services header element will still be in your native language:
spaneditable="false" data-v-7d4a7f28="" data-v-c15243e4="">Services

👀 Expected behavior
Change the language of the elements, according to the general settings
😓 Actual Behavior
The language change does not occur on all elements
🐻 Uptime-Kuma Version
1.19.3
💻 Operating System and Arch
20.04
🌐 Browser
Chromium 108.0.5359.99
🐋 Docker Version
20.10.22
🟩 NodeJS Version
No response
📝 Relevant log output
Services

Yes, work as expected. It is free text, not an fixed name. You can rename it.

That's right, how could I not see it.
But it turns out when I change the language, this text is translated into my native language, but when I change it back it remains the same.
Thanks!
But if I can edit it doesn't matter anymore


GKE: private endpoints

How to test:
Scenario 1: Public endpoint, private nodes, no outgoing access to the internet from nodes
On GKEClusterConfigSpec, set
  privateClusterConfig:
    enablePrivateEndpoint: false
    enablePrivateNodes: true

Nodes will not have access to the internet, so if created from Rancher, so the air-gapped installation instructions need to be followed to ensure the nodes have access to the necessary charts and images. Rancher itself does not necessarily need to be installed in the same VPC since it can access the cluster's public endpoint.
Scenario 2: Public endpoint, private nodes, Cloud Nat
On GKEClusterConfigSpec, set
  privateClusterConfig:
    enablePrivateEndpoint: false
    enablePrivateNodes: true

No other steps are needed.
Scenario 3: Public endpoint, private nodes, authorized networks
On GKEClusterConfigSpec, set
  privateClusterConfig:
    enablePrivateEndpoint: false
    enablePrivateNodes: true
  masterAuthorizedNetworks:
    enabled: true
    cidrBlocks:
    - displayName: example
      cidrBlock: <rancher server IP range>

Add Rancher's public IP to authorized networks. This blocks requests from other networks outside of GCP. Testing should ensure that Rancher still has access to the cluster.
Scenario 4: Private endpoint, private nodes,no outgoing access to the internet from nodes
On GKEClusterConfigSpec, set
  privateClusterConfig:
    enablePrivateEndpoint: true
    enablePrivateNodes: true

Nodes will not have access to the internet, so if created from Rancher, so the air-gapped installation instructions need to be followed to ensure the nodes have access to the necessary charts and images. Rancher needs to either run in the same VPC or be connected with [Cloud VPN]https://cloud.google.com/network-connectivity/docs/vpn/concepts/overview() or Cloud Interconnect VLAN.
Scenario 5: Private endpoint, private nodes, Cloud Nat
On GKEClusterConfigSpec, set
  privateClusterConfig:
    enablePrivateEndpoint: true
    enablePrivateNodes: true

In the GKE console, create a VPC network and subnet https://console.cloud.google.com/networking/networks/list

In the GKE console, create a Cloud NAT gateway for your network https://cloud.google.com/nat/docs/using-nat#creating_nat

Create your cluster in Rancher, with Spec.Network and Spec.Subnetwork referring to the network you created, and Spec.PrivateClusterConfig.EnablePrivateEndpoint and Spec.PrivateClusterConfig.EnablePrivateNodes set to true and Spec.PrivateClusterConfig.MasterIpv4CidrBlock set to a /28 range. Keep track of the cluster name, region or zone, and node pool name for later.

Wait for the cluster to be ready in GKE.

Once the cluster is created, the cluster agent needs to be manually installed in order to allow Rancher to access the cluster. You will need to download a kubeconfig and be able to connect to the private control plane endpoint in some way. This could by creating a compute instance with your kubeconfig credentials on the same VPC, or connecting to the VPC via a VPN. If you don't want to create an additional node and you don't have a VPN, you can create an SSH tunnel through one of the cluster nodes.

The Rancher UI should prompt you run a kubectl command on the cluster to register it with Rancher.

Bugs:
https://github.com/rancher/rancher/issues/32251
https://github.com/rancher/rancher/issues/32241

Supported Scenarios (https://confluence.suse.com/display/EN/GKE+Private+Cluster+Scenarios):
2, 3 only with NAT and 5. Only these three will be tested, as decided in todays meeting with Caleb, Ricky and Colleen.


AuthenticationHandler not plugin, dependency plugins don't specifiy t…

@WolframHempel @timaschew
I was going through the documentation and testing some things, and the valve permissions never actually printed where they were being loaded from.
I know this is way too close to the 1.0 release, but I tested this and it works across all authentication handlers ( minor change ) since all of our connectors isReady events are sync ( file loads so fast it worked regardless ), and also works if they are async.
I think this should go in, but I want to get some thumbs up first incase it:

Isn't urgent enough to bypass code freeze
Has any implications I can't think of

Good catch!
LGTM 👍

Coverage decreased (-0.1%) to 96.375% when pulling 642f8010f3eb9b481075ca8e7b260d4c363758e5 on bug/auth-not-plugin into 91054a1571d95db7257534e57c3c49d09b676780 on master.