bedrock:cohere.command-r-plus-v1:0 - ValidationException

using this model on 0.63 returns these validation errors:
API call error: ValidationException: Malformed input request:
#: extraneous key [logit_bias] is not permitted
#: extraneous key [truncate] is not permitted
#: extraneous key [stream] is not permitted
#: extraneous key [num_generations] is not permitted
#: extraneous key [return_likelihoods] is not permitted
#: extraneous key [prompt] is not permitted
#/p: 1 is not less or equal to 0.99, please reformat your input and try again.

Should be fixed in 0.63.1


Integrate transpiling into the CLI

As of writing, we are running the transpiler through an #eval call, which is slow and polluting the pipeline. Integrating transpilation into the CLI will keep the codebase organized and discoverable.

Good call! Since we're not writing/reading stores from files yet, we can use the pipe subcommand for now, which exists precisely for that


Add health.md to readme.md

No reason to hide this info, might as well have it all nice and accessible from the first page you see when you go to the repo!

Closed by #74


Failed to open input method

Which operating system does the issue occur on? Ubuntu 18.04
If on linux, are you using X11 or Wayland? X11
the error:
RUST_BACKTRACE=1 alacritty
thread 'main' panicked at 'Failed to open input method: PotentialInputMethods {
    xmodifiers: None,
    fallbacks: [
        PotentialInputMethod {
            name: "@im=local",
            successful: Some(
                false
            )
        },
        PotentialInputMethod {
            name: "@im=",
            successful: Some(
                false
            )
        }
    ],
    _xim_servers: Err(
        GetPropertyError(
            TypeMismatch(
                0
            )
        )
    )
}', /home/skariel/.cargo/registry/src/github.com-1ecc6299db9ec823/winit-0.15.1/src/platform/linux/x11/mod.rs:90:17
stack backtrace:
   0: <unknown>
   1: <unknown>
   2: <unknown>
   3: <unknown>
   4: <unknown>
   5: <unknown>
   6: <unknown>
   7: <unknown>
   8: <unknown>
   9: __libc_start_main
  10: <unknown>

Probably best to report this upstream with winit/glutin. I don't think there's anything Alacritty can do about this.
However would you mind giving the #1403 PR a shot first? It's a big glutin/winit version jump, so it's possible that this resolves the issue.

This looks like it could be the same as tomaka/winit#636

Thanks for keeping use updated. Looks like that might be related.

At my setup this error was caused by setting /etc/locale.conf to an invalid language that was activated inside /etc/locale.gen on Arch.
After fixing this, it works fine.

I started getting this error right after some package upgrades a few days ago. At the same time I lost my DPI scaling in the environment. So I suspect that it's related. I have Ubuntu LTS 18.04.1 on X11. Btw, I have another computer with the same OS but different graphics card, and I don't have this error there right now.

Having the same issue on LM19 tara, any fix?

I'm also running Ubuntu 18.04 on X11 and having the same issue.
Later I found that my /etc/default/locale was:
LANG="en_HK.UTF-8"
LANGUAGE="en_HK:en"

After changing to:
LANG="en_CA.UTF-8"
LANGUAGE="en_CA:en"

problem was fixed.

Noticed the same behavior on manjaro 18.0.4 with x11 and actual 0.3.3 alacritty installed via manjaro repositories.
Fixed it by moving the binary file to a different path /opt/alacritty/alacritty and creating a shell script under /usr/bin/alacritty, which simply sets the language without the .UTF-8-ending, as suggested by @happysmash27 :
#!/bin/bash

LANG=en_dk /opt/alacritty/alacritty $@

$@ expands to the script arguments
My quick test to verify a still utf-8 compatible terminal was to create a utf8.txt file with some utf-8-only chars:
äöüß!"'§$%&/()=?°^,.-;:_<>€ƒ†…‡ˆ‰Š‹Œ•—™¶Ø

Running cat utf8.txt correctly displayed all chars in alacritty and everything else seems to work normally.
After launching alacritty it is possible to export the original LANG value with .UTF-8-ending, to ensure compatibility with other programs, f.e. by modifying the .bashrc.


Issue in only specific server

Note: I Have this error only in my hub, also I've tried the "Storage system errors". But it works correctly on other servers.
my error: https://pastebin.com/NjRQpQVX

It doesn't matter when it works on other servers.
When it doesn't work on that server, then it doesn't work.
Double-check that you followed the same steps, that you use the same version (preferably latest) and that everything listed here is correct:
https://github.com/lucko/LuckPerms/wiki/Storage-system-errors#luckperms-cannot-connect-to-my-mysql-server

Please update to the latest version
https://luckperms.github.io
That alone will fix many problems.

I updated it for every server, and bungee seems to be having that issue now.  Storage system errors I fixed earlier and I've had Bungee issues earlier too. Is Hexacord incompatible?

Update: I even tried switching database (Everything from cannot connect to the database).

What storage method are you using? Are you sure the database is working? You can test it with another plugin and see if it works.

MySQL, other plugins it works. I've never had issue that i need to make thread here.
It is working with other servers and plugins.

Use MariaDB Instead of MySQL, And double check if you entered your data right, If that doesn't fix it guess you have to wait for Luck to see the issue here

I could if my eu host would have that option, my us host (only for 1 server and its for other kind of server) may have it as it is multicraft?

I checked it alot of times.

MariaDB works with any MySQL database. Its just a better storing method for LP than MySQL

Oh, i've always though they were completely different.

Nope, it works fine with any database

Okay, rightnow i'm not at home, i can try later.

Still, tried to switch to MariaDB.

Is the server that can't connect on a different host / different machine?
Could perhaps be a firewall issue?
Either way, unfortunately (or maybe fortunately?!) this isn't a LuckPerms bug/issue.
All of the guidance I have for this issue is listed here:
https://github.com/lucko/LuckPerms/wiki/Storage-system-errors#luckperms-cannot-connect-to-my-mysql-server


Min/max zoom constraint event on layers

Describe the bug
Hi all,
This is not that much of a bug as it might be me overlooking some functionality in OpenLayers.
Is there a way to get an event when a layer stops/starts rendering due to min/max zoom constraints being reached?
Currently the only way I was able to get this information is by accessing rendered property of Layer class and combining it with moveend event on map. The first problem of course is that rendered property is protected, thereby I needed a small "shortcut" to even access it without TypeScript screaming at me. Secondly, this approach seems rather inefficient to me. Is there any other possibility to get an event when layer goes outside it's zoom level constraint and stops rendering on map. I will provide an example in code sandbox to better describe what I mean.
I tried adding all sorts of event listeners to layer but I can't seem to figure out if there is some event I could subscribe to or at least follow some other property in layer properties. If you wondered what is the main use for this functionality - it is signaling to users in layer control that some layer is not available at current zoom level. Layer control is a custom implementation.
To Reproduce
Steps to reproduce the behavior:

Code sandbox provided at - codesandbox

Expected behavior
Some kind of event signaling that layer went from actively rendering on map to not rendering.
I tried searching here and on Stack Overflow but couldn't find a similar issue, I apologize if I overlooked some similar issue on this repository.

Recent versions of OpenLayers provide an isVisible(view) method on each layer. Maybe this one works for you? If you need it in an event based manner, I'd call it upon the map's rendercomplete event.

Recent versions of OpenLayers provide an isVisible(view) method on each layer. Maybe this one works for you? If you need it in an event based manner, I'd call it upon the map's rendercomplete event.

I totally overlooked isVisible property on a layer. And this does fix my original problem.
Although, looking through API docs, all Layer implementations do have an "change:visible" event. Shouldn't this fire when this property changes? This is probably the main reason I overlooked this property in the first place.

You're right, the isVisible() method considers more than just the visible property of the layer. The events prefixed with change: are only fired when observable properties are changed. In the case of the layer, setVisible() changes the observable property visible of the layer. That event won't fire in any other case.

@ahocevar thanks for your help!
Now it is much clearer how isVisible works and what it considers for visibility.
This does answer my question and I will close this issue accordingly.
I'm wondering though if an event on "map" visibility change of an layer would qualify as a feature request sometime in the future? As I mentioned previously, I think this would be of great help when doing custom layer controls and signaling that a layer is not visible on map under current view conditions (zoom/extent/group visibility/visible property).


Feature flagging of add/edit phrases

There is a single path to adding phrases to custom categories that is now behind the EditPhrasesEnabled environment variable. The path is Settings -> Categories -> Your Custom Category -> MISSING_LOCALIZTION_Add Phrases.
Refactored EditPhrases to handle editing any category. It was previously hardcoded to just 'My Sayings.'

Notes for Test Engineering:

It is possible to successfully add phrases to a custom category those phrases are not showing up when selecting the custom category in the main screen. Updating that screen is part of the refactor and out of scope for this PR.
The refactoring of EditPhrases should be tested in My Sayings and following the user path outlined above.

Xcode Environment Variables:

"Add new category" button should be controlled by the same feature flag


Node: fix segfault with incorrect status argument types

This ensures that values that get destroyed in the destructors of status-related Op wrapper objects also get dummy values created in the constructors of those objects.
This fixes #11984.

test this please (some of the test logs seem to have disappeared).

The Mac failure appears to be a Jenkins failure. The multilanguage tests failed and aren't running again. The Windows failure is #11109.


Resize and not preserve scale

Hi,
I have a PDF which was not correctly generated. Pages should be in landscape but they are in portrait mode. All images are crush horizontally and stretch vertically. Is there a way to resize all pages but not rotate them, it's not a problem of rotation, and stretch each page to the new format ?
Thanks in advance.

There are many things you can try, but depends on the pdf and what you are doing.
Are you scaling, resizing or both?
Some things you may want to try

Flip detection [on / off / auto] (default is auto)

Turn it off to see what happens, using -f d
You can also try to force it with -f f

Use ghostscript autorotation (default is PageByPage)

Try disabling with -a d
Try to force with -a a

You can read the detailed explanation and other options at the help info.
pdfscale --help

If you can provide the pdfscale call and the log of the pdfScale run (use -v to print it), then it will be easier to guess what is happening.
Apart from that I would need the original PDF to try here.

I tried cmd='pdfscale -v -v -r  "custom mm 594 841" -s 0.9  {} {}'.format(infile,outfile) for the resizing the PDF. I was the facing the same error and I tried the below command
cmd='pdfscale -v -v -r -f d --vertical-alignment top --horizontal-alignment top "custom mm 594 841" -s 0.9  {} {}'.format(infile,outfile)
But I am getting the following error
pdfscale v2.5.3

ERROR! Invalid Paper Type: -f

Usage: pdfscale [-v] [-s <factor>] [-m <mode>] [-r <paper> [-f <mode>] [-a <mode>]] <inFile.pdf> [outfile.pdf]
Help : pdfscale -h

please Help me out.,
I am using python3 to trigger the command in Ubuntu 18.04

Hi there.
You should probably have created another issue, since this one is unrelated to your problem.
My first advice is to try the command in the shell first.
When it works, transfer the logic to python.
Also, not sure why you are using -v -v if your are not gonna post the log from that here.
The first one seems OK
$ pdfscale -v -v -r "custom mm 594 841" -s 0.9 in.pdf out.pdf
2020-12-18:23:56:21 | pdfscale v2.5.3 - Verbose Execution
2020-12-18:23:56:21 |    Mixed Tasks: Resize & Scale
2020-12-18:23:56:21 |        Dry-Run: FALSE
2020-12-18:23:56:21 |     Input File: in.pdf
2020-12-18:23:56:21 |    Output File: out.pdf
2020-12-18:23:56:21 |  Get Page Size: Adaptive Enabled
2020-12-18:23:56:21 |         Method: Grep
2020-12-18:23:56:21 |   Source Width: 595 postscript-points
2020-12-18:23:56:21 |  Source Height: 842 postscript-points
2020-12-18:23:56:21 |     Print Mode: Print ( auto/empty )
2020-12-18:23:56:21 |    Fit To Page: Enabled (default)
2020-12-18:23:56:21 |    Auto Rotate: PageByPage
2020-12-18:23:56:21 |    Flip Detect: No change needed
2020-12-18:23:56:21 |   Run Resizing: CUSTOM ( 1684 x 2384 ) pts
2020-12-18:23:56:22 |      New Width: 1684 postscript-points
2020-12-18:23:56:22 |     New Height: 2384 postscript-points
2020-12-18:23:56:22 |   Scale Factor: 0.9
2020-12-18:23:56:22 |     Vert-Align: CENTER
2020-12-18:23:56:22 |      Hor-Align: CENTER
2020-12-18:23:56:22 |  Translation X: 93.55 = 93.55 + 0.00 (offset)
2020-12-18:23:56:22 |  Translation Y: 132.44 = 132.44 + 0.00 (offset)
2020-12-18:23:56:22 |    Run Scaling: -10 %
2020-12-18:23:56:22 |     Background: No background (default)
2020-12-18:23:56:23 |   Final Status: File created successfully

The second one fails the same way your does. Because it should.
You call -r and then you put the paper type a long way down the road for some reason. So you end up using -f as a paper type.
Also, you are using top as horizontal alignment, when you should use one of

left, right, center

So instead of:
pdfscale -v -v -r -f d --vertical-alignment top --horizontal-alignment top "custom mm 594 841" -s 0.9 in.pdf out.pdf

You need something like:
$ pdfscale -v -v -r "custom mm 594 841" -f d --vertical-alignment top --horizontal-alignment center -s 0.9 in.pdf out.pdf
2020-12-19:00:04:30 | pdfscale v2.5.3 - Verbose Execution
2020-12-19:00:04:30 |    Mixed Tasks: Resize & Scale
2020-12-19:00:04:30 |        Dry-Run: FALSE
2020-12-19:00:04:30 |     Input File: in.pdf
2020-12-19:00:04:30 |    Output File: out.pdf
2020-12-19:00:04:30 |  Get Page Size: Adaptive Enabled
2020-12-19:00:04:30 |         Method: Grep
2020-12-19:00:04:30 |   Source Width: 595 postscript-points
2020-12-19:00:04:30 |  Source Height: 842 postscript-points
2020-12-19:00:04:30 |     Print Mode: Print ( auto/empty )
2020-12-19:00:04:30 |    Fit To Page: Enabled (default)
2020-12-19:00:04:30 |    Auto Rotate: PageByPage
2020-12-19:00:04:30 |    Flip Detect: Disabled
2020-12-19:00:04:30 |   Run Resizing: CUSTOM ( 1684 x 2384 ) pts
2020-12-19:00:04:30 |      New Width: 1684 postscript-points
2020-12-19:00:04:30 |     New Height: 2384 postscript-points
2020-12-19:00:04:30 |   Scale Factor: 0.9
2020-12-19:00:04:30 |     Vert-Align: TOP
2020-12-19:00:04:30 |      Hor-Align: CENTER
2020-12-19:00:04:31 |  Translation X: 93.55 = 93.55 + 0.00 (offset)
2020-12-19:00:04:31 |  Translation Y: 264.89 = 264.89 + 0.00 (offset)
2020-12-19:00:04:31 |    Run Scaling: -10 %
2020-12-19:00:04:31 |     Background: No background (default)
2020-12-19:00:04:31 |   Final Status: File created successfully

Cheers!
Gus

@tavinus Thank you for the reply it was very helpful.


fix no data in insigths v5

fixed #25
We should use =~(regex match) to match multi value variables
Already pass the test by myself.

pls accept this fix


Add .csv support for country codes

Add .csv support for country codes. The iso.org xml export is no longer available for free. A parser for https://www.iso.org/obp/ui/ would be nice.

Added in 380ed685ae252f8dc3754ef3fe38110bb75aeb3d


fix removing errors most of pydantic on massive upload tests

should now upload more kinds of gpt export json content without errors

Thanks

should now upload more kinds of gpt export json content without errors

Thanks, I will try it out today!


Lists of Checkboxes are corrupted when accessed from Canvas

Please check that this issue hasn't been reported before.

[X] I searched previous Bug Reports didn't find any similar reports.

Expected Behavior
The only change to the default settings of both Obsidian and Tasks is that in Tasks I have set [/] to follow [ ]
Example: Obsidian Note with three equal headers and a Obsidian Canvas with the 3 cards. Each card is the Note but with the view narrowed to one of the headers. (One card for each header). Below each header on the Note is a list of Tasks. For example the first task under the first header could be - [ ] Task 1. And the first task under the 2nd header could be - [ ] Just another task. Using the Canvas, when Just another task is clicked it should change from [ ] to [/]
Current behaviour
From the example above: Using the Canvas, when Just another task is clicked it remains unchanged, but the - [ ] Task 1 is changed to - [/] Just another task
Steps to reproduce
# Task List 1
- [ ] TL1 - Item 1
- [ ] TL1 - Item 2
- [ ] TL1 - Item 3

# Task List 2
- [ ] TL2 - Item 1a
- [ ] TL2 - Item 2a
- [ ] TL2 - Item 3a

# Task List 3
- [ ] TL3 - Item 1b
- [ ] TL3 - Item 2b
- [ ] TL3 - Item 2b

Which Operating Systems are you using?

[ ] Android
[ ] iPhone/iPad
[ ] Linux
[ ] macOS
[X] Windows

Obsidian Version
1.1.9
Tasks Plugin Version
1.25.0
Checks

[X] I have tried it with all other plugins disabled and the error still occurs

Possible solution
Please contact me if you have any questions or if you need more details

@BluBloos FYI Re earlier discussions...

This is likely similar to https://github.com/obsidian-tasks-group/obsidian-tasks/issues/688...
See discussion earlier today:
https://github.com/obsidian-tasks-group/obsidian-tasks/issues/688#issuecomment-1427141969

@LastDavid-Zoho If you have time to do a test for us, it could be very helpful.
Could you try exactly the same thing but with only dataview installed.
What happens then? Does dataview complete the correct task?
Many thanks.

Sorry, I did not make it clearer in my original post. The error only occurs once Tasks has been given a "cycle". Without the Tasks plugin the error does not occur. Also with Tasks added, but no "cycle" provided the error does not occur. By cycle I mean when a task is clicked the default is to switch between [ ] and [x]. If you update the Tasks settings so that [ ] is followed by [/] and [/] is followed by [x] then the error will occur. (Sorry if I am not using clearer terms)

@claremacrae (I forgot the mention, I am still learning 😊)

@LastDavid-Zoho That's odd. I don't doubt it, but I can't think of any reason why it might be like that.
If you can think of a way to attach files here to enable me to quickly reproduce what you observe, I might be able to find a few minutes to test it.
I think I would need:

A markdown file with the tasks in
A canvas file which embeds the tasks
Your [vault]/.obsidian/plugins/obsidian-tasks-plugin/data.json file

This is so that I am using the same custom statuses as you
Read it in a text editor first, to make sure it doesn't have any personal info in, in folder names

This is likely similar to #688...

yep!
I haven't tested, but the PR i just pushed should fix this,
(although the feature doesn't work, just no data loss)

I'm totally confused by the statement that the problem only occurs with custom statuses though.
If true, it invalidates my assertion that the problem was the same as 688...


Create Image Upload (Drag & Drop)

A image upload system that will allow you to drag and drop files onto the main webpage itself.

an example of this: https://www.smashingmagazine.com/2018/01/drag-drop-file-uploader-vanilla-js/

[ ] The user should be able to upload the images to their site/application
[ ] We will have to either check for duplicate images or let the users browse and delete them. (We can either warn them that they are about to delete an image that might be used elsewhere. Or create a references list which will show the images used on each page and where.)

If I were to create a references JSON file, I could use the same file for export to remove unused images within the application.
{
'home': [
'89asjkdolasdijoi7', <- File Hash within the user's system.
]
}
All images will be stored in the same folder res/images/


Pres/Slide Dimensions

Hi,
Hope, you're well. I was thinking of adding a method to get dimensions of a slide, right now I've popped it on the slide.getDimensions() but maybe this can go inside getInfo().slidenumber > .info eventually?
Would you like it in your lib?
If yes:
Does this feel like the right order of precedence for what dimensions override each-other? My memory says yes but I'd need to read the spec to confirm.
What do you want to see in a PR? Is this fine (with tidy up obviously, and initing the source archive in the function) more or less as is or?
https://github.com/MP70/pptx-automizer/commit/d0e028c3d486376885e399b648a4c0ca6aea64a8

Hi Matt,
that looks great, thank you very much! I was having issues with slide dimensions too in the past, so this is very welcome.
I was playing around with your code and moved it to src/helper/xml-slide-helper.ts. It can reuse some things from there (e.g. slideXml) and some code could be reused later, too (e.g. get related master and layout).
Have a great week! :)
Thomas

And feel free to add modifications directly to this repository in the future :smiley:
I have created a new branch for this feature.

Hi Thomas,
That is the obvious home for it, thanks! Sorry, I was just focusing on getting it working first :)
I'll get this branch into a more mergable state when I have a moment, thanks for access.
It's a super useful feature in my eyes as it means we can layout things on the slide, (e.g center an element) as well as enabling some hygiene features such as detecting elements that are outside of the canvas, glad you're happy to include it.

Hi Thomas,
I'm just refreshing myself on this one. I think that in practice 99.9% of the time the only thing that practically matters is the dimensions at the template level, rather than side level. Although the pptx spec supports different slide sizes in the same file in practice the major presentation packages, generally, do not, including powerpoint.
So, I'm a bit torn on if we should just put a simpler version in at the template level e.g
 let template = pres.getTemplate('output')
 template.getDimensions()

and cut out the complexity, or leave it at the slide level (or do both).
Whats your view?

Hi Matt,
I was already wondering how to set different dimensions, and I completely agree that we can cut out complexity here. See also MS docs.
Thanks a lot again for your contribution! :smiley:


[FLINK-29472] Add first version of release scripts

This PR adds the first version of shared release utilities for connectors.
The expectation is that the release_utils branch will be mounted as a git submodule into each connector repo.
See the README for the purpose of each script.

Note that we could just do a review round, and then try them out for the ES connector release before merging them.

Thanks @zentol and @zentol for driving this, but I saw dev mail list received all PR updates information from this repo, is there some settings incorrect?

Thanks @zentol and @MartijnVisser for driving this, but I saw dev mail list received all PR updates information from this repo, is there some settings incorrect?

Yes, we need to add an .asf.yaml file to this repo. Will do in a moment :)

Happy to see you start working @zentol , Could you take a look my comments here? https://github.com/apache/flink/pull/21227 sorry to reply under this issue but I've tried ping you on slack but no response.


Extend 64-bit tests to 32-bit platforms

Some tests are gated by statements like the one seen below, only so that I could move on to more important matters. Like building the thing I'm testing. Now I’m running out of things to do, however, so it's time to iron out these wrinkles.
guard MemoryLayout<UInt>.size == 8 else { throw XCTSkip() }

Hm. I was thinking of a testing pattern along the lines of:
func testStuff() {
    try on64BitOr32BitPlatforms {
        // testing 64-bit stuff
    }   x32: {
        // testing 32-bit stuff
    }
}

But I like the way XCTSkip() tells you whether the code executed. So maybe the following is preferred:
func testStuffX64() {
    guard MemoryLayout<UInt>.size == MemoryLayout<UInt64>.size else { throw XCTSkip() }
    // testing 64-bit stuff
}

func testStuffX32() {
    guard MemoryLayout<UInt>.size == MemoryLayout<UInt32>.size else { throw XCTSkip() }
    // testing 32-bit stuff
}

Cool, now I just need a 32-bit simulator. Does Xcode even have one of those? 🫠

Xcode... y u no 32-bit simulator? 🫠

I'll circle back to this when I find a 32-bit simulator...


Make 'Translation' box a bit bigger

Using this translations box often - it's a small target and I miss it sometimes. Would be nice to make it a line or two bigger by default.

You can customize the field height though using a custom style.  From the home screen, settings > settings, add the following to the "custom styles" field:
textarea#translation { height: 150px; }

Then click save.  When you open the reading form, it will look like this:

as opposed to the default:

Champion, thanks!
Is there a full list of the things I can style?

I see that this page with all the styles exists: https://github.com/jzohrab/lute-v3/blob/master/lute/static/css/styles.css
Still, I'm not a coder and it's mostly a mystery to me. So, perhaps it should be a goal to create a visual guide for the clueless? I'd be happy to do them one-by-one once I figure out what they are and take screenshots and show changes.

Right, I understand, CSS can be pretty cryptic! :-)
Many things can be styled.  The best way to sort it out is to get some kind of "elements" view of the page -- e.g. in chrome dev tools, you can right click an element, click Inspect, and see stuff about it.  (I'm getting buried with stuff and can't spend much time showing you this :-) ) -- but eg. with an id like "blah" in something you can style the span with that id like this:
span#blah { font-size: 200%; }
or, for all elements with a certain class (e.g. hello -- this has 3 classes!) you can say
span.x { ...styling ...}
There are some more examples at https://jzohrab.github.io/lute-manual/usage/themes/custom-styles.html

I'll do some experimenting and at a later date, we can look into creating a more in-depth guide. I'm happy to offer my own help when the time comes.

I'm getting buried with stuff and can't spend much time showing you this :-)

Sorry for contributing to this!!!
I promise that I'm basically done. I've reported 90% of the stuff that I've found I've found so far and there are only a few things I'm still considering and a few that would be dependent on other features becoming reality first (I can plant seeds though... hehe).


Big box not updated by Combine tool

There was a point post office.
So I made an area, then used the Combine tool to combine them.
All worked fine, except the words in the big box under "Edit Feature" did not become "Post office" but instead stayed "Area", even though the tags were updated at the bottom correctly.

Looks like this has already been fixed in the development branch by the new multiselection inspector.


Edited demo/screencast-qat.sh.

Replaced 'master' with 'main' on line 159. Related to issue 499.
Signed-off-by: DougTW<EMAIL_ADDRESS>

closed in favor of rebased version: #582


Update API for point in time keep alive

Background
Currently the opensearch only support updation of point in time keep alive by specifying the new keep alive in the search requests which are using the PIT.
For example:
GET /_search
{
  "size": 10000,
  "query": {
    "match" : {
      "user.id" : "elkbee"
    }
  },
  "pit": {
    "id":  "46ToAwMDaWR5BXV1aWQyKwZub2RlXzMAAAAAAAAAACoBYwADaWR4BXV1aWQxAgZub2RlXzEAAAAAAAAAAAEBYQADaWR5BXV1aWQyKgZub2RlXzIAAAAAAAAAAAwBYgACBXV1aWQyAAAFdXVpZDEAAQltYXRjaF9hbGw_gAAAAA==",
    "keep_alive": "100m" <--- specifying the new point in time keep alive
  },
  ...
}

In order to extend the keep alive of an Open PIT, right now the only way is to use the search api with PIT. In cases, when user wants to extend the PIT, they have to use a search query, which is a workaround and doesn’t suit all the usecases.
For example, in dashboards. If a user is seeing the list of PITs and want to extend the keep alive time. They have to trigger a search using that PIT with keep_alive field. The update PIT will come handy in such situations.
Describe the solution you'd like
A new API to update the Point in time keep alive.
API contract:
Currently we do not see the need to make multiple PIT update in a single request. Current usecases demands only a single PIT keep alive update. We can incremently support the list of PITs, in case we found the PIT update for multiple PIT Ids is needed.
Request:
PUT /_search/point_in_time/
{
"pit_id":<pit_id>,
"keep_alive":"3hr"
}

Response for success:
{
    "pit_id": "o463QQEKbXktaW5kZXgtMRYwTzBpQzU2eVJycWxjUXZJZS1LUFl3ABZXclY3UG5xUlM1LWktN002bjV4WldnAAAAAAAAAAABFjdXcktkc3ExUkoyODd4MG1jaFRHd0EBFjBPMGlDNTZ5UnJxbGNRdkllLUtQWXcAAA==",
    "_shards": {
        "total": 1,
        "successful": 1,
        "skipped": 0,
        "failed": 0
    },
   "creation_time":<PHONE_NUMBER>048 --> original time the PIT was created
   "expiry_time":<expiry_time> --> The time when the PIT will expire after the update
}

What it does?
The user can pass the time by which they want to extend the expiry_time. Though the user will recieve a 403 error if they try to extend the keep_alive time more than the point_in_time.keep_alive( defaults to 24h). The point in time id and Keep alive are mandatory fields for this api.
Example

Let’s say a user create a PIT at 10th Dec 1pm:

POST my-index-1/_search/point_in_time?keep_alive=4hr

This will have the expiry_time at 10th Dec 5pm.

Now at 3pm, the user wanted to extend the PIT keep alive to 5hr

PUT /_search/point_in_time/
{
"pit_id":<pit_id>,
"keep_alive":"5hr"
}

The response will be as given below:

{
    "pit_id": "o463QQEPbXktaW5kZXgtMDAwMDAxFkhGN09fMVlPUkVPLXh6MUExZ1hpaEEAFjBGbmVEZHdGU1EtaFhhUFc4ZkR5cWcAAAAAAAAAAAEWaXBPNVJtZEhTZDZXTWFFR05waXdWZwEWSEY3T18xWU9SRU8teHoxQTFnWGloQQAA",
    "creation_time": <time when it was created>( 10th Dec 1pm)
    "_shards": {
        "total": 1,
        "successful": 1,
        "skipped": 0,
        "failed": 0
    },
    "expiry_time": <timestamp when it will expire>( corresponds to 10th Dec 8pm--> last access_time(3pm + keep_alive(5hr))
}

The expiry of a PIT depends on the last access time and keep_alive
If the customer wants to extend the keep alive beyond 24 hours(depends on the point_in_time.max_keep_alive) from the time the API is fired, error will be thrown.

API sequence diagram

General questions/answers related to the new API

What if the updated is failed on some of the shards?

We will allow partial failure similar to search requests. In the response we will have details related to successful and unsuccessfull shards. The user can trigger the same request in case they want to update the PIT on the failed shards. For more details see here.

Should we have update point in time API for multiple PITs?

Currently we do not see a use case of using the update Point in Time for multiple PITs. Therefore we will be going with one PIT update for now.

Does the update Point in time Api support reducing the Keep_alive?

No the current plan to only allow user to extend the PIT( we do max(new_value, old_value) similar to how the update through a search request works to have consistent behaviour for update using search request and update API)

Example of update API in case of failed shards
Let’s assume we have 3 shards(s1,s2 and s3) and the user triggers an update of keep alive to 2hr and it fails on shard s3.
s1 → 2hr
s2 → 2hr
s3 → (old value)
The user can trigger the same command to update the Keep alive on the failed shards.
s1 → 2hr
s2 → 2hr
s3 → 2hr

Note: The keep alive does not change with time. The PIT get expired based on last_access_time and keep_alive

Additional context
About point in time: https://opensearch.org/docs/latest/opensearch/point-in-time/

Closing this issue as after internal discussion, we decided to not implement this API as of now.


[WIP] Move to GitHub for CI and CD

Summary of the PR
Fixes Azure DevOps being a weenie
Related issues, Discord discussions, or proposals
Links go here.
Further Comments
Still uses AzDO for package storage, as I don't wanna change the feed for experimental consumers.

@HurricanKai ci is green, can you green plz kthx bye

Also consider filtering for /src/ changes


How to trigger 'resize' event?

Hello, can't figure it out.
I have similar event like here in https://snazzymaps.com/style/42/apple-maps-esque
Initially map width is 70% of the screen and when hiding a side panel it's width 100%.
how to fire 'resize' event of google map, like this:
google.maps.event.trigger(map, 'resize');
?

Sorry I've taken so long to reply, it should be as simple as:
google.maps.event.trigger(maplaceObj.oMap, 'resize');

Thanks a lot for reply) Though, sure, I've handle it so long time ago.
I guess it would be useful to have it in docs :-)

Related to #173

Hi there,
Can I have one example to trigger resize event on map. I am unable to resize the map
your help will be really appreciated.


Struggling with 'OUT_OF_MEMORY' error

Hello,
I'm using STM32F769-Disco board and I'm struggling since two days with 'OUT_OF_MEMORY' error returned by pv_picovoice_init()!
I'm using Picovoice in an RTOS context (FreeRTOS).
I've increased all possible memory regions as following:
1- heap and stack size of the application in linker file:
_Min_Heap_Size = 0xF000 ; /* required amount of heap: 60kB /
_Min_Stack_Size = 0xF000 ; / required amount of stack: 60kB */
2- FreeRTOS heap size:
#define configTOTAL_HEAP_SIZE ((size_t) (250 * 1024))
3- Stack size of the task managing picovoice:
#define configPICOVOICE_TASK_STK_SIZE ( 0x8000 )
4- Picovoice memory buffer, relocated at the end of SDRAM and increased its size to 128kB:
#define SDRAM_DEVICE_ADDR ((uint32_t)0xC0000000)
#define SDRAM_DEVICE_SIZE ((uint32_t)0x1000000) /* SDRAM device size in MBytes */
#define MEMORY_BUFFER_SIZE (128 * 1024) // 128kB instead of 70kB
//static int8_t memory_buffer[MEMORY_BUFFER_SIZE] attribute((aligned(16))); // Commented out
In the task:
int8_t* memory_buffer = NULL;
uint32_t membuffer_start_address = 0;
/* memory_buffer will be located at the end of the SDRAM */
membuffer_start_address = (SDRAM_DEVICE_ADDR+SDRAM_DEVICE_SIZE) - MEMORY_BUFFER_SIZE;
membuffer_start_address = membuffer_start_address - (membuffer_start_address%16);
memory_buffer = (int8_t *)membuffer_start_address;
status = pv_picovoice_init(
ACCESS_KEY,
MEMORY_BUFFER_SIZE,
memory_buffer,    // In SDRAM
sizeof(KEYWORD_ARRAY),
KEYWORD_ARRAY,
PORCUPINE_SENSITIVITY,
wake_word_callback,
sizeof(CONTEXT_ARRAY),
CONTEXT_ARRAY,
RHINO_SENSITIVITY,
true,
inference_callback,
&handle);
Any help please?
PS:

Issue faced with 1.1.0 and 2.1.0 library versions.
For point 4- : If I relocate again the memory buffer in the internal RAM using static int8_t memory_buffer[MEMORY_BUFFER_SIZE] attribute((aligned(16))); , I still having the same issue!

Thank you for your help.

Thank you @safsoun for the report. Can you do me a favour and try to reproduce this using the official demo? Once we can then I can get someone to work on it. Otherwise, it would be harder to help out but we do the best we can.

Thank you for your fast reaction.
Unfortunately, I didn't reproduce the case with the demo as it is ...
This FW (reproducing the issue) is an evolution of another one having the same architecture and having the exact implementation of Picovoice task . The older one works perfectly.. So the behevior is strange with the new FW mainly after extending all memory region sizes!
I wanted to know from which memory region pv_picovoice_init() is requesting the memory? heap or stack? or is there something in the compiled library in the demo limiting the memory usage?
Thank you in advance.

If the stack is small then you get a stack overflow and I don't think you get a clear error message. I doubt it's a heap issue so that leaves us with the buffer you are passing in. I think the buffer is the main issue but I can't tell you what is exactly wrong. Is it possible to work from the demo and move towards where you are to see what breaks?

Thank you for the answer.
Ok, I will try to remove as much as possible other tasks and stop at the point where I get the error with the minimum firmware, otherwise I will go from the other direction: starting from scratch then adding tasks until I get the issue. This certainly will take awhile but I will come back to you asap with the results.
If I get a minimal FW that reproduces the issue, can I send it to you for debug?

Happy to help out there when you can reproduce. It is almost impossible (resource-wise) to get external code and debug per community member. I hope you understand. Let's get you to a place where we can at least know what triggers the issue and we do our best to help.

Hello,
Sorry, but I couldn't see how could I get a help if you can't get a simple firmware to reproduce it from your side?
I've already reproduced it (running on STM32F769-Disco board) in a very simple FreeRTOS task with the exact call sequence as the demo and with the original pv_params.h file and got 'OUT_OF_MEMORY' error message but could not find which triggers the issue since I don't have idea how memory is managed in the library!
1- I've already increased heap and stack in the linker file:
_Min_Heap_Size = 0xA000 ;	/* required amount of heap  /
_Min_Stack_Size = 0xA000 ;	/ required amount of stack */
2- Increased task stack size: 0x6000
3- Increased Picovoice memory buffer size:  #define MEMORY_BUFFER_SIZE (128 * 1024)
I start thinking that is not a memory lack but it's a limitation in Picovoice library which could not run in an RTOS context :-/

BTW, is this a commercial application or a hobby project?

Finally, I found/fixed the issue.
It's not due to Picovoice library nor to my application but due to the sbrk() implementation provided in STM32CubeIDE that cannot work with FreeRTOS. sbrk() is defined in in sysmem.c file located under \STM32CubeIDE\Application\User
This is an article describing the issue and providing the fix: https://nadler.com/embedded/newlibAndFreeRTOS.html
Thank you anyway for your time.


Huge RSS when queryRaw(unsafe) large rowset

Bug description
We use prisma to update our database row by row and once a day we generate a large CSV file to export actual data.
const rows = await prisma.$queryRaw() get 600k rows, the exported file size is about 40Mb, but the memory allocation is very large.
time: 32251ms
heapUsed: +642Mb/748Mb
heapTotal: +652Mb/780Mb
rss: +1847Mb/2067Mb
arrayBuffers: -463kb/2Mb
external: -463kb/3Mb

Heap usage is under a normal value, but out K8S kills ou process, because RSS exceeds 2Gi limit. This is only for a 40Mb result set.
With "native" pg (npm i pg) RSS value stays at very low. It here any problem with prisma rust engine under-the-hood?
How to reproduce
Run a 600k resultset with queryRaw or queryRawUnsafe
Expected behavior
RSS would be stay low.
Prisma information
generator client {
  provider        = "prisma-client-js"
}

datasource db {
  provider = "postgresql"
  url      = env("DATABASE_URL")
}

model InvolvedParty {
  id               String    @id @db.VarChar(100)
  bankCode         String    @db.VarChar(20)
  systemCode       String    @db.VarChar(20)
  legacyId         String    @db.VarChar(100)
...

const prisma = new PrismaClient();

const rows = await prisma.$queryRaw`

Environment & setup

OS: MacOS and K8S
Database: PostgreSQL
Node.js version: v18.16.0

Prisma Version
prisma                  : 4.15.0
@prisma/client          : 4.15.0
Current platform        : darwin-arm64
Query Engine (Node-API) : libquery-engine 8fbc245156db7124f997f4cecdd8d1219e360944 (at<EMAIL_ADDRESS>Migration Engine        : migration-engine-cli 8fbc245156db7124f997f4cecdd8d1219e360944 (at node_modules/@prisma/engines/migration-engine-darwin-arm64)
Format Wasm             : @prisma/prisma-fmt-wasm 4.15.0-28.8fbc245156db7124f997f4cecdd8d1219e360944
Default Engines Hash    : 8fbc245156db7124f997f4cecdd8d1219e360944
Studio                  : 0.484.0

We do not use transaction, we do not use query params (SQL is a simple string), we do not use any preview feature.

SQL is like that:
    SELECT
        ip."technicalGroupId" as super_id,
        ip."systemCode" as system_code,
        ip."legacyId" as system_id,
        a.giro as account,
        a.iban,
        a."accountClass" as account_class,
        'active' as status
    FROM "InvolvedParty" ip

Each row is about 70 bytes.

.ENV
DATABASE_URL="postgresql://user:pass@<IP_ADDRESS>:5432/ms-name?schema=ms-name"

It looks like that: https://github.com/prisma/prisma/issues/16912

Do I understand correctly that you run 1 SELECT SQL query via prisma.$queryRaw that returns roughly 40MB of data, but Prisma jumps to use ~2GB of RSS usage when doing so @BCsabaEngine?
This is happening with Prisma 4.15.0, on MacOS and K8S (What OS and version?) running Node v18.16.0 with PostgreSQL (which version?). Can you reliably reproduce this locally?
If so, could you maybe share your Prisma Schema, Prisma Client query and an example of the data (optimally in a way so we can easily import it) with us? If you can't share that publicly, I am my first name @prisma.io for email. (We can also do NDA if necessary)

I have created a small repo to demonstrate the potential bug: https://github.com/BCsabaEngine/prisma-test
Every package is uptodate, app runs on a mac with node v18.17.1 and connected to a dockered PG (PostgreSQL 13.7
PostgreSQL 13.7 on x86_64-pc-linux-gnu, compiled by gcc (Debian 10.2.1-6) 10.2.1 20210110, 64-bit), so can reproduce it locally in my machine.

Thanks @BCsabaEngine, I can reproduce this. I also see that the Prisma Client executed raw query is a lot slower than the pg one, which is also unexpected. We're going to investigate this.

With prisma v5.4.2 the error still exists, but RSS reduced to lower
Start RSS: 214 Mb
Extract finished - 1000000 rows
Concat - 25 Mb
End RSS: 1879 Mb

With prisma v5.7.0 the error still exists
Start RSS: 223 Mb
Extract finished - 1000000 rows
Concat - 25 Mb
End RSS: 1958 Mb

With prisma v5.7.0 the error still exists
Start RSS: 223 Mb
Extract finished - 1000000 rows
Concat - 25 Mb
End RSS: 1958 Mb

Is this still an issue? We seem to be running into a similar issue.  In our case, the RSS size is never reclaimed until we allow the Prisma object to be garbage collected. Otherwise, memory grows with each request.
We're using $queryRaw with toSql for vectors in PostgreSQL and pgvector. We're trying to understand why Prisma appears to be retaining massive amounts of memory in our AWS Lambda instances.   Our raw queries are bulk inserts (due to vectors). Inserting around 1,500 vectors of 1.5k dimensions each is resulting in huge amounts of RSS being allocated.
We can fix the issue by discarding our Prisma objects on each request, but that means we have to reconnect to the DB each time and that's slow.

Prisma client 5.12.1
Node 20.x on AWS Lambda with 1024 MB memory allocated.

Hey folks, we're about to release Prisma 5.17.0 and it comes with perf & memory usage improvements on queryRaw. I won't close this issue because the memory usage improvement is mostly a side effect of the work we've done, but perhaps you might want to give it a try nonetheless once it's out.

With prisma 5.17.0 the result is:
Start RSS: 227 Mb
Extract finished - 1000000 rows
Concat - 25 Mb
End RSS: 845 Mb

so memory consumption is much lower.
With native pg lib:
Start RSS: 245 Mb
Extract finished - 1000000 rows
Concat - 25 Mb
End RSS: 515 Mb

The difference is no considerable. The speed of this query is considerable but this issue is/was about memory.


bug: Incorrect return format "useContractRead"

Is there an existing issue for this?

[X] I have searched the existing issues

Package Version
0.8.6
Current Behavior
Hi,
when trying to get read from a contract using "useContractRead", I get this type of result:

How do you parse such a thing?
Expected Behavior
I should have a normal array like so,
[ { betsCount: 1, maxBet: 1, name: "Team 1", dieResult: 0} ]
(simple array of objects)
Steps To Reproduce
Make a solidity smart contract with one function that returns an array
Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
https://codesandbox.io/s/epic-rgb-72gyx7?file=/src/App.js
Anything else?
No response

This is because contract functions in Ethers return a hybrid object/array type.
You should still be able to access values by named key still though (e.g. teams[0].name, teams[1].betsCount, etc).

Expanding on what Jake said, if you use TypeScript and convert the ABI to a const assertion, you can get the return type inferred automatically. More info here.

@tmm Thanks, incredible <3
@jxom Thanks! Good to know
But why are the "BigNumber" not automatically casted to "number" for me ?
Seems to be kind of a hustle to always have to map the numbers yourself like so :
results.map((item: Team) => ({
  dieResult: Number(item.dieResult),
  maxBets: Number(item.maxBets),
  betsCount: Number(item.betsCount),
  name: item.name,
}))

With web3-react it is done internally.
Thanks again!


Fix travis build

https://travis-ci.org/aspnet/Identity

Has it ever passed? I woud expect the same kind of issues ee had in EF tests that use databases.

Not that I'm aware of. It's one of the few repositories that don't pass on travis. We should skip the localdb tests on mono.

@davidfowl I skipped the tests, but looks like the travis build is failing on setup first... any ideas what's busto?
19.06s$ ./build.sh --quiet verify
Installing 'KoreBuild 0.2.1-beta7-15004'.
Successfully installed 'KoreBuild 0.2.1-beta7-15004'.
Unable to find version '0.2' of package 'Sake'.
Determining latest version
Latest version is 1.0.0-beta8-15118
Downloading dnx-mono.1.0.0-beta8-15118 from https://www.myget.org/F/aspnetvnext/api/v2
Download: https://www.myget.org/F/aspnetvnext/api/v2/package/dnx-mono/1.0.0-beta8-15118
######################################################################## 100.0%
Installing to /home/travis/.dnx/runtimes/dnx-mono.1.0.0-beta8-15118
Adding /home/travis/.dnx/runtimes/dnx-mono.1.0.0-beta8-15118/bin to process PATH
Setting alias 'default' to 'dnx-mono.1.0.0-beta8-15118'
Cannot open assembly 'packages/Sake/tools/Sake.exe': No such file or directory.

2161f42086ca0f5b9c0c04460a0dc45e72b678bd


Need a way to pass parameters by reference in Calc

Say I want to pass in a list/array or map/object to be modified inside a function. Currently that doesn't work:
list = [] -> [ ]
Defining function addTo(a, n) = {
   a += n
}
addTo(list, 1)
Error: Cannot change Parameter value a at line 3.

Suggest allowing var or const (default) as  a modifier in front of the name, as in:
define a(var a, const n) = { ... }

where the default would be const (as now), but either one is allowed in order to be explicit.
Of course, this would imply that the actual value for a non-const parameter could not be a constant value...

Okay, apart from the difficulties I'm having just implementing this, there is at least one (likely) insoluble problem: when an empty set/map is converted to one or the other when the first element is added. Even in the best case when I could get the real object and make a new one, the original reference (in the symbol table of one of the callers) is never referenced anywhere, so I could never update the original reference with the new object, unless I could pass in the original LValueContext of it ... (good luck with that).

Questions:

At what point do we find the actual symbol table reference(s) to the lvalue to do the assignment, += or whatever?
What do we need to pass as the "value" in the ParameterValue to get that to happen?
How to indicate the proper lookup context (i.e., say the "real" variable is named "s" in the global context, but the parameter name is "s" inside the function, how do we reference the "real" "s" value [that is, we need the caller's context to do the lookup)?

Maybe we need to change the grammar thusly:
: 'const' ? expr
| 'var' var
;

(with suitable additions for id '=' recently introduced). But that way we automatically get the var reference and can never pass in a constant value for a non-constant parameter.


messing logs for SPBC8E4.02c but I don't see a problem?

warning in 9a6cbb66c43cb971: in extension for PomBase-genotype-1044, can't find feature with identifier: SPBC8E4.02c - can't find feature for: SPBC8E4.02c
https://www.pombase.org/gene/spncrna.9001
(it is a non-coding RNA though)

SPBC8E4.02c is now an alternate ID for spncrna.9001 - I think that's why it's spawning a log message. I've fixed that one.


[Bug]W E B upper case object doesn't work on mac os big sur

As you can see , it just delete one character

Hi @androidfans
Thanks for the reporting, I will pay attention to this issue and fix it as soon as possible.
Thanks and regards

Hi @androidfans
This issue has been addressed and fixed. Please update your plugin to the latest version.
Thanks and regards

wow, so quick.  I will try it when i get home @jeffreytse

It works!  thx a lot


Make import form type easier to override

Allows overriding get_import_form() to change the form used.

LGTM, but not ready to merge until build re-stabilizes


Question about the data format

Hi, thank you very much for your reply in the afternoon, I'm sorry to trouble you again.
I want to use optical flow in VQE, however, the input of the optical flow network has RGB channel instead of Y channel, so I think that I should make yuv channel lmdb instead of y lmdb, and then convert it to rgb channel before inputted to the optical flow network. So the only change of the origin code is set 'only_y=False' in the import_yuv fuction, is it right?
BTW, there are several algorithm to convert data from yuv420 to rgb. Which one do you think is better?
1.R = Y + 1.402 * (V - 128)
G = Y - 0.34413 * (U - 128) - 0.71414 * (V - 128)
B = Y + 1.772 * (U - 128)
2.out_img = np.matmul(img, [[0.00456621, 0.00456621, 0.00456621],
[0.00791071, -0.00153632, 0],
[0, -0.00318811, 0.00625893]]) * 255.0 + [-276.836, 135.576, -222.921]
The second one is from BasicSR~

Maybe right. You may first have a try. If you have RGB input such as PNG images, you can simply set the in_nc and out_nc being 3, and use disk IO instead of LMDB IO. See: https://github.com/RyanXingQL/PythonUtils/blob/4c2aa9b7ae9c72c8aa9200b5e3bb569a54c92f35/dataset.py#L266.
To convert YUV420P to RGB, I suggest: YUV420P -> YUV444P ->  RGB. See:
https://github.com/RyanXingQL/PythonUtils/blob/4c2aa9b7ae9c72c8aa9200b5e3bb569a54c92f35/conversion.py#L177
https://github.com/RyanXingQL/PythonUtils/blob/4c2aa9b7ae9c72c8aa9200b5e3bb569a54c92f35/conversion.py#L132

Okay ,thank you very much. I'll have a  try.


Update baseline doc to allow formatting rules

Since detekt allows to suppress formatting issues and add them to the baseline, the docs are obsolete.
Reference: #2083

Codecov Report

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

@@            Coverage Diff            @@
##             master    #2101   +/-   ##
=========================================
  Coverage     80.84%   80.84%
  Complexity     2008     2008
=========================================
  Files           336      336
  Lines          5784     5784
  Branches       1059     1059
=========================================
  Hits           4676     4676
  Misses          546      546
  Partials        562      562

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 acc7d14...516f0c3. Read the comment docs.


compatibility with sparklyr?

hello and thanks for this very interesting project. I wonder if someone can use it with sparklyr? any ideas?
thanks!

Hi, thank you for your interest.
We don't support R binding or sparklyr currently. Sorry.

haaa thats sad. R/sparklyr are pretty hot these days... maybe in the future? how difficult it is for you to do it?
thanks!!

Just learned of this on the databricks blog. If the Scala API is at parity with PySpark, it should be something we can support without too much trouble.

really cool


[Add]  Want to add new animation button

Details
I want to add new animation button on the website
Screenshots
No response
Type of Contribution

[ ] Update to an existing Animated Button
[X] Adding a new Button
[ ] Resolving a bug
[X] Proposal to the Repository
[ ] Changes related to documentation or README.md
[X] Other Changes

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue
[X] I am a GSSoC'23 contributor

Please check #1346


Can't install or open it[BUG]

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
Since I've downloaded bloxstrap, it worked for like a month then it started to crash and normal roblox too, if I use another windows acc it works, but it's annoying to switch between windows accounts

if I use another windows acc it works, but it's annoying to switch between windows accounts

You've been flagged by Hyperion. You've already said that this happens with normal Roblox too. Not my problem.


Fix LIBCLOUD-887

If a user sets LIBCLOUD_DEBUG, they will receive an error and no requests will be made. This is happening on trunk and v2.0.0rc1-tentative
In ./libcloud/utils/loggingconnection.py
LoggingConnection.request does not accept the stream keyword which causes an error:
TypeError: request() got an unexpected keyword argument 'stream'

Reported by @supertom

Passed. Merging.


Handle application/vnd.elasticsearch+json;compatible-with=7

Handle application/vnd.elasticserach+json;compatible-with=7 in backward compatible way

See discussion at https://github.com/opensearch-project/opensearch-go/issues/53

This is not relevant to OpenSearch.


[Enhancement]  Method invocation may produce 'NullPointerException'

Search before asking

[X] I had searched in the issues and found no similar issues.

Enhancement Request

located at:
eventmesh-sdk-java/src/main/java/org/apache/eventmesh/client/http/producer/RRCallbackResponseHandlerAdapter.java line 87 104
Describe the solution you'd like
reference:

Are you willing to submit PR?

[x] Yes I am willing to submit a PR!

I want to fix it.


virt-manager: 1.4.2 -> 1.4.3

Motivation for this change
Things done

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

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

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

Will you backport?

Yes, will backport.


request: Additional sub commands for some modules.

Additional sub commands for some modules.
AFK Command/Module
Add other sub commands to the AFK command or make AFK command a separate module.

[ ] Ignored Channels: a command to add channel as blacklisted.
[x] Allowed Role List: only these roles can use the command other than mods.
[ ] Reset: to set yours/someone's AFK reason to default.
[ ] Clear: removes the AFK status of a member.
[ ] List: list of members who have set AFK status.

Giveaway

[ ] End: command to end the ongoing giveaway whenever mods what to.

Thanks for reading and you might implement these changes/updates to Skyra someday. <3

Allowed Role List: only these roles can use the command other than mods.

This is already covered with permission nodes:

skyra pnode add everyone deny afk
skyra pnode add my-allowed-role allow afk


Diff data too large

I'm experiencing a strange bug. When opening a commit, it shows "Diff Data Not Available." and when checking the gogs.log it contains [W] Diff data too large
When checking with an external client (e.g. Github for Windows) i can see the full Diff.

Hi, please checkout this: https://gogs.io/docs/advanced/configuration_cheat_sheet#git-(git)

The strange thing is, that it just affects commits with many changed files, not all.

Added
[git]
MAX_GITDIFF_LINES = 3000000

and restarted gogs. No change.

Which file did you add those lines?

That did the trick, ty. However, this leads me to a feature request :D

this leads me to a feature request

What is it...? :trollface:


Add version: t1m0thyj.WinDynamicDesktop version 4.3

Removing old versions to reduce clutter
Microsoft Reviewers: Open in CodeFlow

/azp run


Workers on 1.10 even when EKS is provisioned as 1.11

I have issues
I'm submitting a...

[x] bug report

What is the current behavior?
Created new cluster, specifying cluster_version = "1.11" with two worker groups (using new Launch Templates support).
Provisioned node reports version 1.10.11
This is a problem with ami filter probably.
If this is a bug, how to reproduce? Please include a code sample if relevant.
Create new cluster version 1.11 with at least one worker groups launch template, example:
worker_groups_launch_templates = [
    {
      name                   = "r-xlarge-spot"
      instance_type          = "r4.xlarge"
      override_instance_type = "r3.xlarge"

      asg_desired_capacity                     = 1
      asg_max_size                             = 1
      asg_min_size                             = 1
      autoscaling_enabled                      = true
      key_name                                 = "dev"
      on_demand_percentage_above_base_capacity = "0"
      public_ip                                = true
      root_volume_size                         = 50
      spot_instance_pools                      = 1
      spot_instance_pools                      = 2
      subnets                                  = "${join(",", local.subnets)}"
    },

What's the expected behavior?
Node version is in sync with cluster version.
Are you able to fix this problem and submit a PR? Link here if you have already.
If we identify the issue then probably yes.
Environment details

Affected module version: latest
OS: MacOS
Terraform version:

Terraform v0.11.11
+ provider.aws v1.54.0
+ provider.local v1.1.0
+ provider.null v1.0.0
+ provider.template v1.0.0

Any other relevant info
Launch Template reports AMI: amazon-eks-node-1.10-v20190109 (ami-0de10c614955da932)
➜  ~ kubectl version
Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.2", GitCommit:"81753b10df112992bf51bbc2c2f85208aad78335", GitTreeState:"clean", BuildDate:"2018-04-27T09:22:21Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"11+", GitVersion:"v1.11.5-eks-6bad6d", GitCommit:"6bad6d9c768dc0864dab48a11653aa53b5a47043", GitTreeState:"clean", BuildDate:"2018-12-06T23:13:14Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}
➜  ~ kubectl get nodes
NAME                                         STATUS    ROLES     AGE       VERSION
ip-10-10-72-147.eu-west-1.compute.internal   Ready     <none>    22h       v1.10.11

No update and lots of changes in this module since. So I will close be feel free to reopen after testing on latest version of module.


Error Creating Contract "Unable To find Type: ot2_app_contract"

Hi everyone,
I am currently to trying to create a new Contract in the App Contract Approval in OpenText but when I give clic in the button Add I show me the next error

SomeBody why I show me that error ???, I would thank you a lot, This project I download of github

It seems that you did not deploy the ot2_app_contract cms type in you tenant.
The step how to do that are described in the 'Sample app - Contract Approval' example as described on the OpenText developer forum: https://developer.opentext.com/learn/tutorials/d4c0a773ef486fef1876cbd3709161f2
Note that this example does not yet use the OpenText Cloud Developer Tools extensions for VS Code. I propose that you have look at the updated contract approval app tutorial as described in https://mimage.opentext.com/ot/2/devx/vscode/22.4.1/vscode-tutorial.pdf
This is a complete step-by-step guide showing and explaining you how to develop an application with the OpenText Developer Tools extensions in VS Code and how to use the OpenText Cloud APIs.
The tutorial also explains how to setup a local development environment with nodejs and how to install the OpenText Developer Cloud VS Code extensions.

Thanks a lot!!!! triffit, Sure I will ty your recommendation and the following days I will upload the conclusion


Support for audio

Can this be used for transmitting audio stream as well? If yes, is there any documentation for it. If not, is there any suggested tweak?

@Anupamsdesk You can check this
https://github.com/socketio/socket.io-p2p/tree/master/examples/streaming


Add design Assets and add related configurations to tailwind file

@Onyelaudochukwuka can you assign me this? i need some of the fonts config from this issue

What is the status of this @Onyelaudochukwuka

It's been quite slow on my end


Pay Genius TDSv2 Upgrade

We are upgrading our Pay Genius integration to use TDSv2, which ensures better fraud detection, a faster checkout experience and decreased payment failures. There is the possibility of payment failures occurring if the deployment does not go as planned, but the upgrade is expected to take no longer than 15 minutes, so the impact should be minimal. Should you experience an increased payment failure rate, please reach out to our support team by emailing<EMAIL_ADDRESS>or by calling +27615413986.

The upgrade has been completed, and all tests have passed.


OVN overlay interface not listed

hotsos output does not show interface information for the OVN overlay interface

this is because ml2-ovs used the same network as used for neutron local_ip config but thats no longer the case for ml2-ovn. Should be easy enough to fix.


Added For creating FloatingActionsMenu programically

Will be great if you will add option to create fab programatically
this will also enable developer to set color for Main Fab

I think the #49 implementation would solve this problem in a more idiomatic way.


problem with build gocrack

i have problems when build gocrack:
go build -tags """" -o builds/linux/amd64/gocrack_worker  -ldflags \
	"-X github.com/fireeye/gocrack/worker.CompileRev=e3b20e4 \
	 -X github.com/fireeye/gocrack/worker.CompileTime=1569172867 \
	 -X github.com/fireeye/gocrack/worker/engines/hashcat.HashcatVersion=" \
	cmd/gocrack_worker/*.go
# github.com/fireeye/gocat
../gocat/gocat.go:239:7: could not determine kind of name for C.EVENT_OPENCL_SESSION_POST
../gocat/gocat.go:237:7: could not determine kind of name for C.EVENT_OPENCL_SESSION_PRE
Makefile:14: recipe for target 'build' failed
make: *** [build] Error 2

Can you confirm you're using Hashcat 5.1.0? I got an identical error when using the HEAD of master instead of 5.1.0

i used de master branch of hashcat, it turns out that the stable version of hashcat has some bugs with some hash  types

Please do not use the master branch of hashcat as that's considered the "beta/bleeding edge" builds and often between versions of Hashcat, the C API changes which forces us to go update our cgo bindings. It should work when using the v5.1.0 tag


flightradar24 card is not selectable as a new card

Hello,
I've installed successfully from HACS and restarted.
I can't select the card as a new card as it doesn't appear.
Is this a bug or am I missing another step to finally install?
Help is much appreciated.

The same thing happens to me

The card must be configured as a custom card.
Ref: https://github.com/Springvar/home-assistant-flightradar24-card?tab=readme-ov-file#configuration


Possibility to create CPack installers

I have never worked with CPack before, but I think this could be useful.

I agree it could be useful. I've only dabbled with CPack previously, I found that for anything more complicated than a shell-archive or ZIP file, it was not very intuitive to use and debugging the output in a cross-platform way was compounded by the fact that many installer types are platform-specific.
That being said, actual usage of it from my memory is pretty simple (mainly just setting a bunch of variables and invoking CPack).

Initial support for CPack implemented in #35.


feat: QuantitySelector component

What's the purpose of this pull request?
This PR adds the QuantitySelector component.
How it works?
It adds the following files that compose the QuantitySelector component logic, its documentation and tests:

Core logic files

useQuantitySelector.ts, which defines the QuantitySelector context and the hook for using it. The values that are made available by this context are: name, currentValue, onClick
QuantitySelector.tsx, which defines the provider which wraps the children inside the shared context.
QuantitySelectorButton.tsx, which defines an IconButton component that uses the onClick function from the shared context
QuantitySelectorInput.tsx, which defines an Input component that uses the currentValue from the shared context
index.tsx, which exports all of the aforementioned files
QuantitySelector.test.tsx, which adds tests for the component

Storybook/Assets/Style related files

QuantitySelector.mdx, which is the Storybook page with two versions of the QuantitySelector: a readonly styleless version, and other not-readonly, styled version.
QuantitySelector.stories.tsx, which defines both QuantitySelectorStylelessTemplate and QuantitySelectorDefaultTemplate, binding them to the storybook
Icons.tsx, which defines two simple react components that renders the plus and minus signs from an SVG, to be used with the IconButton component
quantity-selector.css, the QuantitySelector stylesheet using it data-attributes and Tailwind when possible.

How to test it?
yarn test QuantitySelector inside packages/ui folder
base.store Deploy Preview
[WIP]

References
[WIP]
Kent Dodds blog post about inversion of control
Kent Dodds blog post about compound components
Tailwind docs about hover / focus / etc
Useful unity converter from px to rem
Mozilla developer docs about data attributes
W3C spinbutton ARIA practice docs
Page on digitalA11Y about the spinbutton role
Jest queries
And
@gvc,
@renatamottam,
@icazevedo,
@ArthurTriis1
For all the patience and handholding.

We have a style selector. Maybe remove this styless story?

Not sure if I understood what you meant, @tlgimenes. I followed the same pattern used on the IconButton page, showing the raw, unstyled version first, and then a version with a custom style.

Not sure if I understood what you meant, @tlgimenes. I followed the same pattern used on the IconButton page, showing the raw, unstyled version first, and then a version with a custom style.

We have a style pop-up selector on the top of the page. Adding a story only for this case is not necessary. IconButton should be changed too

@tlgimenes oh, got it, thanks! That does sound better. I'm going to change that and later I'll open a PR for the IconButton too :)

@emersonlaurentino past Heitor would agree with you, since your exampled looks a lot like my first implementation for this component (you can check it out at https://github.com/vtex/faststore/pull/1035/commits/493bed206c921d2381c315aeb83c4d4c96d284ec). However, after seeing @ArthurTriis1 #1033 which uses the compound components pattern, I felt more inclined to make this new abstraction. I agree that we are exporting more components, but I think more about the gains when we wrap it all inside a context. In my head, we do that, we are essentialy telling the dev "ok, so this is the QuantitySelector universe. Whatever you put inside, it can access these elements from the shared state (quantity, etc...)".
Think about it this way: we are not giving the full assembled LEGO kit to the developer, but instead giving the pieces and telling how they can be assembled together. But the developer is free to use its imagination and assemble variants of the component using the same pieces. It's impossible to know all the use cases upfront, but imagine if:

The dev wants to put the two buttons at the left side of the input.
The dev wants to add three buttons (a reset one, for instance)
The dev wants to manipulate/pass attributes individually to the input or to any of the buttons
The dev do not want to increase/decrease by a single unit
The dev want a special action to trigger (e.g. onChange) in one of the elements

These are some use cases that comes to my mind right now, and we would be losing that flexibility by wrapping two buttons and an input inside a div.
Let me know if I managed to convince you, otherwise I would love to discuss more. 😄

Hmmm, I'm kind with past Heitor on this one. This is personal preference, but I'd like to keep this compound component architecture to a minimal. I say that because we did everything following compound component architecture on v0.x and we end up with a very complex library.
Since we are using atomic design now, I think that if the developer wants one of these use cases you. mentioned, the dev can use the atoms to build their fancy behavior on their own project. I think that, in most cases I've seen for this component, the simple implementation is the best

Hmmm, If I can add my two cents in here, I'm kinda with past Heitor on this one too. This is personal preference, but I'd like to keep this compound component architecture to a minimal. I say that because we did everything following compound component architecture on v0.x and we end up with a very complex library. Since we are using atomic design now, I think that if the developer wants one of these use cases you mentioned, the dev can use the atoms to build their fancy behavior on their own project. I think that, in most cases I've seen for this component, the simple implementation is the way people go with

I agree with you, but we will face a new problem "pass the children properties. How are we going to tackle it?

After many conversations and debate, I'm closing this in favor of #1083. I'm closing this branch and will not revert to the initial atomic design so we can keep the discussion for future generations. Thanks everyone!


[HUDI-5349] Clean up partially failed restore

Change Logs
If a table was attempted w/ "restore" operation and if it failed mid-way, restore could still be lying around. when re-attempted, a new instant time will be allotted and re-attempted from scratch. but this may thwart compaction progression in MDT. so we need to ensure for a given savepoint, we always re-use restore instant if any.
Impact
MDT compaction progression no longer thwarted?
Risk level (write none, low medium or high below)
low
Documentation Update
N/A
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

can you check CI failures

@nsivabalan CI failing test was in ITTestDataStreamWrite

@nsivabalan can you clarify why landed this without CI passing?


Subregions in icepak meshing

draft

Codecov Report
Attention: Patch coverage is 30.21390% with 261 lines in your changes are missing coverage. Please review.

Project coverage is 34.40%. Comparing base (22380a4) to head (7010f70).

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #4302       +/-   ##
===========================================
- Coverage   81.73%   34.40%   -47.34%
===========================================
  Files         107      107
  Lines       51787    52008      +221
===========================================
- Hits        42328    17891    -24437
- Misses       9459    34117    +24658

Closing as PR is not updating with latests commits (that are available in the branch)


Include directive gets dropped when key layout is saved

In my repo I have the following:
#include <behaviors.dtsi>
#include <dt-bindings/zmk/keys.h>
#include <dt-bindings/zmk/bt.h>
#include <dt-bindings/zmk/ext_power.h>

After using the tool it gets changed to:
#include <behaviors.dtsi>
#include <dt-bindings/zmk/bt.h>
#include <dt-bindings/zmk/keys.h>

The keys include gets moved, which is ok, but the ext_power include gets deleted, which isn't.
Doesn't seem likely, but just in case it's relevant, the initial keymap had some encoder mappings which I deleted because my build doesn't have encoders.

The keys include gets moved, which is ok, but the ext_power include gets deleted, which isn't.

Why not? The editor is managing header include statements needed for behaviors you're using in the keymap. If you added some RGB behaviors it would add the appropriate header, and if you then remove the behavior it would delete the header as well. In this case you aren't using &ext_power as far as I can see so the header can be omitted.

Unusual build probably, but I'm using SuperMini NRF52840 controllers in a wired configuration with USB plugged into the left half of my lilly58 build, a TRRS cable between the halves and no batteries. When #include <dt-bindings/zmk/ext_power.h> is removed my keyboard stops working, but if I add it back in then it works fine. I haven't tried to figure out why. I just know it works when it's there, and it doesn't when it's not.

Thanks for the hint. I added an EP_TOGGLE key and it retains the header. Good enough for me.

You mean the compiled firmware doesn't work? I didn't see that your build was failing when the header was removed, it looked like the jobs on those commits were all cancelled.
Not that I could feasibly help debug, but the header just contains some preprocessor defines so I'm really surprised that would have any effect.

Yeah it compiled fine, but didn't work when flashed. My best guess is that without one or more of the defines, this hardware just defaults to only using power from the battery terminals and not the USB connector.


python3Packages.simplisafe-python: init at 9.6.9

Motivation for this change
Python library the SimpliSafe API
https://simplisafe-python.readthedocs.io/
This is a Home Assistant dependency.
Things done

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

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

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

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

1 package failed to build:

home-assistant

2 packages built:

python38Packages.simplisafe-python
python39Packages.simplisafe-python

Result of nixpkgs-review pr 117069 at 8ba0c688 run on aarch64-linux 1

3 packages built successfully:

home-assistant
python38Packages.simplisafe-python
python39Packages.simplisafe-python

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

3 packages built:

home-assistant
python38Packages.simplisafe-python
python39Packages.simplisafe-python


df-126 -> DELETE support for DoTValve WELL

This PR includes:

support for WELL DELETE
upgraded DoTValve unit test suite to handle server errors gracefully
expanded use of ValveException in DotValve for passing descriptive failure information to SOAP clients
improved test input XML files

This connects to #126

Summary of file changes
XML Files:

all xml files are changed to use distinct UID values (they overlapped before)

IStore:

IStore was upgraded with a deleteFromStore method per the WitsmlSpec.

StoreImpl:

added deleteFromStore method
added support for valve exception reporting in method responses.

DeleteFromStoreResponse:

created this based on the GetFromStoreResponse to structure responses to SOAP clients.

IValve:

made deleteObject, getObject, and createObject all throw ValveExceptions.
made deleteObject accept a QueryContext.

ValveException:

created this as a custom catch-all exception for shuttling error messages up to the StoreImpl for detailed SOAP client reporting.

ValveDelegator:

added deleteObject handler
transitioned from HttpResponse<JsonNode> responses to HttpResponse<String> responses because when the REST body returned is not valid json, the JsonNode implementation throws obscure parsing errors that are hard to understand.
implemented ValveException throwing in the other delegator methods.

DotValve:

implemented deleteObject
implemented valve exception throwing throughout
added DELETE WELL to getCap

DotValveTest:

added delete tests for 1311 and 1411 wells
added checking in all create tests that accept an already exists error from DoT as a valid, test-passing behavior
updated all UID checks with the updated values in the test files


Default 'Content-Type' header is added if the request header is created with lowercase 'content-type'.

Should allow for lowercase 'content-type' header, without adding the default 'Content-Type': 'text/plain;charset=UTF-8'. If two content-type headers are present on the request it usually results in an error.
      if (!headers["Content-Type"] && !headers["content-type"]) {         headers["Content-Type"] = "text/plain;charset=UTF-8";       }

Looks like this issue was fixed, just not updated in NPM package.


Installable as a rock

This PR updates the module to be a proper rock.

Made the module a proper rock by updating the rockspec, and installing in the right place, with the right name
The rock installs itself as a new command for LuaRocks 3 (start it with luarocks gui)
Updated dependencies in the rockspec (datafile, xavante, etc)
Fixed the datafile usage, it no longer relies on the local repository files, but now uses the files installed along with the rock

Notes:

still shows the problems with LuaRocks 3 (indexing arg) to be fixed in LuaRocks
requires some bugfixes to datafile, see https://github.com/hishamhm/datafile/pull/22
requires the luarocks-api branch of the LuaRocks repo

Ping @hishamhm @jiteshpabla

added another commit, to auto-start the webbrowser with the GUI


Bump identity-carbon-auth-rest version

Bump identity-carbon-auth-rest version 1.4.29 -> 1.4.30

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/512142227

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/512142227

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/512142227
Status: success

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/512142227
Status: success


Properly escape constructed routes in StripeClient.Cards.

Routes constructed with string.Format throw exceptions because the route tokens are not properly escaped in the format string. This problem only appears to exist in StripeClient.Cards.cs. This PR resolves this issue.

@nberardi - FYI - this PR fixes breaking changes introduced by @Giovani-Brady in commit 0b86c3e04e18c059081fa44fcb07befa146e94ce.

@JasonS
Adding some unit tests around these methods to avoid future failures such as the one I created.
Also as a side note the UpdateCard method will fail due to the incorrect naming of the parameters being added

@JasonS
please review PR #68
it includes all the changes noted here. additional bug fixes and unit tests around these methods.


CopyToRemote fails on update when FileArchive is a directory

What happened?
I'm trying to upload some configuration files to my ec2 instance. However whenever I make a change pulumi fails to deploy because CopyToRemote can't update. It attempts to make a target directory when the target directory already exists and bails out
Example
    const copyConfigs = new remote.CopyToRemote(`${stack}-tp-${proxyId}-configs`, {
        connection,
        source: new pulumi.asset.FileArchive("configs"),
        remotePath: "/home/pulumi",
    }, { dependsOn: waitForCloudInit, parent: proxy });

results in:
Updating (test):
     Type                               Name                              Status                  Info
     pulumi:pulumi:Stack                tile-proxy-test                   **failed**              1 error
     └─ aws:ec2:Instance                test-tp-0
 ~      └─ command:remote:CopyToRemote  test-tp-0-configs                 **updating failed**     [diff: ~source]; 1 error

Diagnostics:
  pulumi:pulumi:Stack (tile-proxy-test):
    error: update failed

  command:remote:CopyToRemote (test-tp-0-configs):
    error: failed to create remote directory /home/pulumi/configs: sftp: "Failure" (SSH_FX_FAILURE)

config is just some sshd_configs, a docker-compose.yml file and some nginx configs.
Output of pulumi about
CLI
Version      3.124.0
Go Version   go1.22.5
Go Compiler  gc

Plugins
KIND      NAME       VERSION
resource  aws        6.41.0
resource  cloudinit  1.4.4
resource  command    1.0.0
language  nodejs     unknown
resource  tls        5.0.4

Host
OS       arch
Version  "rolling"
Arch     x86_64

This project is written in nodejs: executable='/usr/bin/node' version='v20.15.1'

Current Stack: organization/tile-proxy/test

TYPE                                                URN
pulumi:pulumi:Stack                                 urn:pulumi:test::tile-proxy::pulumi:pulumi:Stack::tile-proxy-test
ops:tile-proxy:InstanceConfig                       urn:pulumi:test::tile-proxy::ops:tile-proxy:InstanceConfig::test-tp-instance-config
ops:tile-proxy:LoadBalancer                         urn:pulumi:test::tile-proxy::ops:tile-proxy:LoadBalancer::test-tp-lb-group
pulumi:providers:aws                                urn:pulumi:test::tile-proxy::pulumi:providers:aws::default_6_41_0
pulumi:providers:tls                                urn:pulumi:test::tile-proxy::pulumi:providers:tls::default_5_0_4
aws:iam/role:Role                                   urn:pulumi:test::tile-proxy::ops:tile-proxy:InstanceConfig$aws:iam/role:Role::test-tp-role
tls:index/privateKey:PrivateKey                     urn:pulumi:test::tile-proxy::ops:tile-proxy:InstanceConfig$tls:index/privateKey:PrivateKey::test-ssh-key
aws:ec2/securityGroup:SecurityGroup                 urn:pulumi:test::tile-proxy::ops:tile-proxy:LoadBalancer$aws:ec2/securityGroup:SecurityGroup::test-tp-lb-sg
aws:lb/targetGroup:TargetGroup                      urn:pulumi:test::tile-proxy::ops:tile-proxy:LoadBalancer$aws:lb/targetGroup:TargetGroup::test-tp-lb-targets
aws:iam/instanceProfile:InstanceProfile             urn:pulumi:test::tile-proxy::ops:tile-proxy:InstanceConfig$aws:iam/instanceProfile:InstanceProfile::test-tp-instance-profile
aws:alb/loadBalancer:LoadBalancer                   urn:pulumi:test::tile-proxy::ops:tile-proxy:LoadBalancer$aws:alb/loadBalancer:LoadBalancer::test-tp-lb
aws:ec2/securityGroup:SecurityGroup                 urn:pulumi:test::tile-proxy::ops:tile-proxy:InstanceConfig$aws:ec2/securityGroup:SecurityGroup::test-tp-sg
aws:ec2/instance:Instance                           urn:pulumi:test::tile-proxy::aws:ec2/instance:Instance::test-tp-0
aws:lb/targetGroupAttachment:TargetGroupAttachment  urn:pulumi:test::tile-proxy::aws:ec2/instance:Instance$aws:lb/targetGroupAttachment:TargetGroupAttachment::test-tp-0-attachment
aws:alb/listener:Listener                           urn:pulumi:test::tile-proxy::ops:tile-proxy:LoadBalancer$aws:alb/listener:Listener::test-tp-lb-listener-ssl-redirect
aws:alb/listener:Listener                           urn:pulumi:test::tile-proxy::ops:tile-proxy:LoadBalancer$aws:alb/listener:Listener::test-tp-lb-listener
pulumi:providers:command                            urn:pulumi:test::tile-proxy::pulumi:providers:command::default_1_0_0
command:remote:Command                              urn:pulumi:test::tile-proxy::aws:ec2/instance:Instance$command:remote:Command::test-tp-0-wait-for-cloud-init
command:remote:CopyToRemote                         urn:pulumi:test::tile-proxy::aws:ec2/instance:Instance$command:remote:CopyToRemote::test-tp-0-configs

Found no pending operations associated with test

Backend
Name           obelisk
URL            s3://opsready-pulumi-infrastructure-backend
User           oehpr
Organizations
Token type     personal

Dependencies:
NAME             VERSION
@pulumi/aws      6.41.0
@pulumi/command  1.0.0
@pulumi/pulumi   3.121.0
@pulumi/tls      5.0.4
@types/node      18.19.39
typescript       5.5.2

Pulumi locates its logs in /tmp by default

Additional context
I can sorta work around this issue by making sure to move the directory first thing when I remote into the ec2 instance, but it's brittle and if something goes wrong the pulumi script can not recover.
CopyToRemote should not die when it is performing an update. It should update.
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Thanks for reporting this @Ghoughpteighbteau; I agree it should be updating in place. We'll take a look.

Hi @Ghoughpteighbteau, sorry you're running into this. Can you confirm that both the local configs and the remote pulumi are directories, not files?
If yes, could you try specifying the remote path as /home/pulumi/, with a trailing slash?

yep yep!
this is my file tree
/home/oehpr/dev/lightship/devops/infrastructure/tile-proxy
├──configs
│  ├──docker-compose.yml
│  ├──nginx
│  │  ├──conf.d
│  │  │  ├──cantopotiles.conf
│  │  │  ├──healthCheck.conf
│  │  │  ├──proxy_params
│  │  │  └──usgstopotiles.conf
│  │  ├──mime.types
│  │  └──nginx.conf
│  └──opsready-sshd.conf
├──index.ts
├──package-lock.json
├──package.json
├──Pulumi.prod.yaml
├──Pulumi.test.yaml
├──Pulumi.yaml
├──tile-caching-system.drawio
└──tsconfig.json

i've updated the target to include a trailing slash
    // now for configuration files, including nginx and sshd settings
    const copyConfigs = new remote.CopyToRemote(`${stack}-tp-${proxyId}-configs`, {
        connection,
        source: new pulumi.asset.FileArchive("configs"),
        remotePath: "/home/pulumi/",
    }, { dependsOn: waitForCloudInit, parent: proxy });

and I reran pulumi up
obelisk in infrastructure/tile-proxy on  tile-caching [!] via  v20.15.1 via  test took 28s
at 09:32:21 ❯ pl up
Previewing update (test):
     Type                               Name                     Plan        Info
     pulumi:pulumi:Stack                tile-proxy-test
     └─ aws:ec2:Instance                test-tp-0
 +-     ├─ command:remote:Command       test-tp-0-apply-configs  replace     [diff: ]
 ~      └─ command:remote:CopyToRemote  test-tp-0-configs        update      [diff: ~source]

Resources:
    ~ 1 to update
    +-1 to replace
    2 changes. 15 unchanged

Do you want to perform this update? yes
Updating (test):
     Type                               Name               Status                  Info
     pulumi:pulumi:Stack                tile-proxy-test    **failed**              1 error
     └─ aws:ec2:Instance                test-tp-0
 ~      └─ command:remote:CopyToRemote  test-tp-0-configs  **updating failed**     [diff: ~source]; 1 error

Diagnostics:
  command:remote:CopyToRemote (test-tp-0-configs):
    error: failed to create remote directory /home/pulumi/configs: sftp: "Failure" (SSH_FX_FAILURE)

  pulumi:pulumi:Stack (tile-proxy-test):
    error: update failed

:shrug:

Oh, sorry you asked about the remote as well, here's that tree as it currently stands.
pulumi
└── configs
    ├── docker-compose.yml
    ├── nginx
    │   ├── conf.d
    │   │   ├── cantopotiles.conf
    │   │   ├── healthCheck.conf
    │   │   ├── proxy_params
    │   │   └── usgstopotiles.conf
    │   ├── mime.types
    │   └── nginx.conf
    └── opsready-sshd.conf


Unable login to api interface

Node red debug info:
There was an error during login operation into Alpha ESS monitoring portal: TypeError: Cannot read properties of undefined (reading 'AccessToken')
We got the following unprocessable body:
{"code":9006,"info":"Network exception, please try again later!","exMessageInfo":"Please use web browser to view the data"}

Please update to newest version and confirm fix.

It works After upgrade to 1.8.0


Mega Menu

I didn't find any solution how to make a complex mega menu style navigation with wagtailmenus

.
Is there any way to make a multi level menu  (min 3 levels) that can behave like this, and maybe have sub menus from a snippet or a custom django model for example this type of tree:
Shop

Category 1 (this is a snippet)

Product 1

Product 2

Category 2 (this is a snippet)

Product 1

Product 2

Category 3 (this is a snippet)

Product 1

Product 2

No out of the box, @meaelien. You'll probably have to subclass the menu models and create one suited to your needs.
https://wagtailmenus.readthedocs.io/en/stable/advanced_topics/custom_menu_classes.html#overriding-the-models-used-for-main-menus


Avoid failing if file doesnt exist / PyOxidizer support

PyOxidizer breaks on stdlib-list as  it doesnt provide __file__   See indygreg/PyOxidizer#69 for more info about why, but the tl;dr version is __file__ is an optional attribute and should not be relied upon.
e.g. at the moment this is what it looks like
>>> import depfinder
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "depfinder", line 34, in <module>
  File "depfinder.main", line 41, in <module>
  File "stdlib_list", line 6, in <module>
  File "stdlib_list.base", line 6, in <module>
NameError: name '__file__' is not defined

I never used PyOxidizer before. Good to know. Do you want to send a PR?


feat(azurearcdata): version stable/2024-01-01

ARM (Control Plane) API Specification Update Pull Request

NOTE: This is an exact copy of the stable.2024-01-01 recently merged to PRSaaSMaster, we are promoting to GA/stable as part of this PR.

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

[!NOTE]
As of January 2024 there is no PR assignee. This is expected. See https://aka.ms/azsdk/pr-arm-review.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Click here to see the details of Step 1, Breaking Changes review
If you are in purview of Step 1 of the diagram, follow the Breaking Changes review process.
IMPORTANT! This applies even if you believe your PR was mislabeled, for any reason, including tool failure.

Click here to see the details of Step 2, ARM review
See https://aka.ms/azsdk/pr-arm-review.

 Click here to see the diagram footnotes
Diagram footnotes
[1] See ARM review queue (for PR merge queues, see [2]).
[2] public repo merge queue, private repo merge queue (for ARM review queue, [1])
The ARM reviewer on-call engineer visits the merge queue twice a day, so the approximate ETA for merges is 12 - 24 hours.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] New resource provider.
[x] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been generated using OpenAPI Hub).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with PR workflow diagram Step 2 (ARM review), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

/azp run

Breaking changes previously reviewed and approved in https://github.com/Azure/azure-rest-api-specs-pr/pull/15309

Regarding

The same 1:1 PR went first into []RPSaaSMaster here](https://github.com/Azure/azure-rest-api-specs-pr/pull/17272)   which is just an aggerate to all prior preview PRs that went in ok before hand, this is just a promoted new version to public main. 2. The failing PR into main  (here) is the exact PR that went into PRSaaSMaster.   No changes were made 1:1. Why would it not fail going into PRSaaSMaster and then fail in public https://github.com/Azure/azure-rest-api-specs/pull/28388 main which is identical?    I think this is a tooling issue?

The PUT body is an "extension" (or whatever the swagger terminology is) of a tracked resource:
"allOf": [
{
"$ref": "../../../../../common-types/resource-management/v3/types.json#/definitions/TrackedResource"
}
],
The TrackedResourehas a tagsproperty.
The "update" resource in PATCH doesn't extend the TrackedResource because it doesn't contain a location property. Hence, for "update" resource we add tagsin the body explicitly.
So, both PUT and PATCH resources shuold contain a tags property, but the PUT resource does it through the TrackedResource. I wonder if this is something that their tooling is not smart about (and something new since it hadn't complained before)?

Regarding

There is no Unicode regex support for ResourceName's .

/pr RequestMerge


Copy of boost-ext::ut ?

Hi,
This unit test framework seems eerily similar to boost-ext::ut, as was also presented at cppcon22.
Maybe you could contribute to their instead of maintaining a similar, parallel repo (with far less user support).
Or if you are copying their work - maybe close your repo?
Best,
Alexander

Clean Test is inspired by UT and therefore offers a similar API. UT is great, but only Clean Test can

introspect generic expressions with short circuiting,
execute tests in parallel and handle simultaneous failures correctly,
change settings at runtime,
run in CI (UTF-8 support, JUnit reports, etc.).

Details have been presented at Meeting C++ 2022. You are welcome to contribute to Clean Test.


Add namespace to index mount directories to avoid DM collision

Use unique index mount directories that consist of
<namespace>-<index>. These unique directories guard against potential
data loss when doing data movement operations. Having the namespace
(i.e. rabbit name) included in the mount path ensures that these
individual compute mount points (index mount directories) are always
unique.
For example, on a 2 rabbit job with 1 compute node each, you would see
the following directories across the 2 rabbits:
 /mnt/nnf/7b5dda61-9d91-4b50-a0d3-f863d0aac25b-0/rabbit-node-1-0
 /mnt/nnf/7b5dda61-9d91-4b50-a0d3-f863d0aac25b-0/rabbit-node-2-0

If you did not include the namespace, then the paths would be:
 /mnt/nnf/7b5dda61-9d91-4b50-a0d3-f863d0aac25b-0/0
 /mnt/nnf/7b5dda61-9d91-4b50-a0d3-f863d0aac25b-0/0

On a copy_out directive, if the user specifies the entire directory as
the source (e.g. source=$DW_JOB_MY_GFS2), then data movement will pick
up each of those 0/ directories and drop them in the same spot in the
destination. This would lead to potential data loss.
Signed-off-by: Blake Devcich<EMAIL_ADDRESS>

No, don't force-push.  Then we can't diff before-and-after.

No, don't force-push. Then we can't diff before-and-after.

I rebased from master. No changes.