idea: document the best practices for plugins users to manage keybindings

Hi again, hope you don't mind me raising so many issues 😅.
I'm looking for a way to document / redesign how users could manage keybindings in my project. I liked the suggestion of avoiding a fancy DSL you have listed in the best practices.
But what is a good alternative? In my plugin, I would like to support the following cases:

the user wants to change an existing keybinding's action (what will be done when they press the key)

I guess it's possible for users to simply redefine the mapping?

the user wants to change an existing keybinding's key (the key that will trigger the action)

not sure if there's a built in way to do this. I can structure my plugin to publish the implementation of the action, so the user can define a lua function that calls it in their own mapping... but this is not easy to do for people and I would like to find a better way

the user wants to disable an existing keybinding

Maybe this is common knowledge, but I wonder if these could be laid out in the documentation.
Thanks, and have an awesome day!

I'm having trouble understanding this (it's late where I am).
Do you have some more concrete examples?

the user wants to change an existing keybinding's action

If the plugin doesn't automatically create any keymaps (which I advise against), but instead leaves defining keymaps up to the user (possibly providing <Plug> mappings), there shoudn't be a need to provide a way to change existing keymaps.

the user wants to change an existing keybinding's key (the key that will trigger the action)

This is what <Plug> mappings are for. As a plugin author, you create a <Plug> mapping (or a Lua function if it's something with lots of options), which defines the action, the keymap options and the modes it can be used in, etc. And you leave the actual keys that bind to the mapping up to the user to choose.
The user just has to create a keymap with <Plug>MyAction on the rhs.

the user wants to disable an existing keybinding

Should be a no-op if you don't automatically create any keymaps.

Here's a concrete example:
My plugin creates a floating terminal window for a terminal file manager. The terminal window needs custom keybindings to enable neovim specific functionality (such as opening the selected file in a new split with <c-v>)- in these cases the key should not be sent to the underlying terminal application, but should be completely handled in neovim instead.
The current implementation is run after the floating terminal window has been created, and the keymappings are only created for that specific terminal buffer:
function M.default_set_keymappings_function(yazi_buffer, config)
  vim.keymap.set({ 't' }, '<c-v>', function()
    keybinding_helpers.open_file_in_vertical_split(config)
  end, { buffer = yazi_buffer })

  vim.keymap.set({ 't' }, '<c-x>', function()
    keybinding_helpers.open_file_in_horizontal_split(config)
  end, { buffer = yazi_buffer })

  -- ...
end

The current implementation isn't very nice for users that want to partially customize the mappings.
I could solve it by creating a DSL such as this:
---@type table<string, fun(config: YaziConfig): nil>
local normal_mode_keybindings = {
  ['<c-v>'] = function(config)
    keybinding_helpers.open_file_in_vertical_split(config)
  end,
  ['<c-x>'] = function(config)
    keybinding_helpers.open_file_in_horizontal_split(config)
  end,
}

Now the user can customize the keybindings by mutating this table (or a copy of it):

to add a keybinding, add it to this table
to remove a default keybinding, remove it (some DSLs also allow setting it to
nil)
to change the action of a keybinding, change the function

This guide advises against using this kind of DSL, and I think it's much easier for plugin authors not to do it (so I agree). But it's also convenient for users to get a "batteries included" experience.
Sorry for the long post. I'm wondering if it makes any sense, and if you'd have any advice.

I see.
In the guide, I mention not to create default keymaps unless they are not controversial.
If they're specific to a certain buffer (in this case buffer = yazi_buffer) it's probably not controversial, as you will be unlikely to conflict with user-defined keymaps.
To replace a DSL, here's a way you could potentially allow users to override the default keypaps using <Plug> mappings:

Search for user keymaps where the rhs is your <Plug> mapping
Unfortunately, Neovim doesn't have a nice API for this.
If the user has defined a keymap for you <Plug> mapping, define the <Plug> mapping.
Otherwise, define a keymap with the default rhs.

It could look something like this:
-- Search for user keymaps
local user_keymaps = {
  open_vertical_split = false,
  open_horizontal_split = false,
}

for _ , map in pairs(vim.api.nvim_get_keymap("t")) do
  if type(map.rhs) == "string" and map.rhs:find("YaziOpenVertical") ~= nil then
    user_keymaps.open_vertical_split = true
  end
  if type(map.rhs) == "string" and map.rhs:find("YaziOpenHorizontal") ~= nil then
    user_keymaps.open_horizontal_split = true
  end
  -- You may want to add a check for breaking out of the loop early if everything is set
end

if user_keymaps.open_vertical_split then
  -- User has set a keymap. Define a <Plug> mapping.
  -- These <Plug> mappings expose the functionality only for mode "t"
  -- and only for the yazi_buffer.
  -- This means that users cannot create YaziOpen* keymaps for normal mode,
  -- or other buffers
  vim.keymap.set("t", "<Plug>(YaziOpenVertical)", keybinding_helpers.open_file_in_vertical_split, { buffer = yazi_buffer })
else
  -- User has not set a keymap. Define a default one.
  vim.keymap.set("t", "<c-v>", keybinding_helpers.open_file_in_vertical_split, { buffer = yazi_buffer })
end

if user_keymaps.open_horizontal_split then
  vim.keymap.set("t", "<Plug>(YaziOpenHorizontal)", keybinding_helpers.open_file_in_horizontal_split, { buffer = yazi_buffer })
else
  vim.keymap.set("t", "<c-x>", keybinding_helpers.open_file_in_horizontal_split, { buffer = yazi_buffer })
end

In the user's config, to override a default keymap, all they need to do is:
vim.keymap.set("t", "<leader>vv", "<Plug>(YaziOpenVertical)")

(they don't need to worry about things like buffer = yazi_buffer, because your <Plug> mapping handles that.


Created a Random-Color-Generator Website

Zero To Mastery - Webblocks 2022 📦📦
👉 Please remove the below and replace with your own values, leaving the headers where they are. 👈
PR contains:

This is a Random-Color-Generating Website. I have built this website using HTML, CSS, and JAVASCRIPT. The website comprises separate style.css(CSS file) and index.js(JAVASCRIPT file), which are both linked to the "index.html". The website has also been made Size Responsive using CSS (Attached screenshots for the same).

The website has two buttons. On pressing the first button, it changes the color of a specified box(which is initially White in color). The other button, on clicking, changes the color of the specified box to its initial color( i.e., White).

Breaking changes

NONE

Screenshots

Image showing Website is Size Responsive:

I request you to please mark this Pull Request under the label Hacktoberfest accepted.


Add Hyperledger Foundation

@tkuhrt any reason all y'all aren't using https://insights.lfx.linuxfoundation.org/foundation/lf-decentralized-trust which is using clomonitor under the covers currently?

@tkuhrt any reason all y'all aren't using https://insights.lfx.linuxfoundation.org/foundation/lf-decentralized-trust which is using clomonitor under the covers currently?

Interesting. This must be new. Does the .clomonitor.yaml file for ignoring checks work in LFX Insights?

unclear, I suggest you work with the LFX team, in particular Kieran is very helpful


Add container-feeder.service file

was this file in the rpm?

Yes, this file was already fixed in https://bugzilla.suse.com/show_bug.cgi?id=1030451

I think it makes sense to add the service file to the git repo, but we would need to adapt the packaging automation to submit that file to obs automatically, which is not implemented yet. maybe this could go into a subfolder called sources and we pick it up from there?

btw we have the same concept for patches already, see https://github.com/kubic-project/velum/tree/master/packaging/suse/patches

I would put this on hold until we have the mechanism described by @MaximilianMeister implemented...

I would put this on hold until we have the mechanism described by @MaximilianMeister implemented...

@mjura you can go ahead now and add the service file to packaging/suse/sources and please also add the sysconfig source that is in obs to that folder
when we merge this one it should be picked up now. thanks

@MaximilianMeister : I've moved files to packaging/suse/sources and added also container-feeder sysconfig file

@mjura thanks!
the unit test says that rpm is missing, maybe you have to provide it in the .travis.yml ?

Hmm problem is that container-feeder require some rpm based os to run. But as I have checked travis uses Ubuntu and rpm tool is missing, because it has apt-get. Container-feeder uses rpm --verify command to check if files was installed by RPM package. I looked for how to change OS distro for Travis, but it can be used only Ubuntu.

i think running the tests/building inside an opensuse docker container would be a better idea. or even alpine, since it's smaller. however you need make, gcc, go et all to be able to build and set the gopath and etc for alpine. that's why i was never a fan of travis. also at the time i checked that (a few years back) they used quite an old ubuntu version

i tried to do a POC for what i've said above but it's really too much work to make it actually work. so you can ignore what i've said.  this will circumvent the issues with testing.

@PI-Victor : I agree with you and having our own CI system would be the best solution.


Improve error message when failing to serve variant

Describe the bug
If there is an issue with serving a variant (such as in  https://github.com/Agenta-AI/agenta/issues/542). The error message is really uninformative. We need to improve the messaging there.
Context
The problem lies in agenta-cli/agenta/cli/variant_commands.py:serve_cli which has
    except ConnectionError:
        error_msg = (
            "Failed to connect to Agenta backend. Here's how you can solve the issue:\n"
        )
        error_msg += "- First, please ensure that the backend service is running and accessible.\n"
        error_msg += (
            "- Second, try restarting the containers (if using Docker Compose)."
        )
        click.echo(click.style(f"{error_msg}", fg="red"))
    except Exception as e:
        click.echo(click.style(f"Error message: {str(e)}", fg="red"))

Basically when serving fails we just say "Error message:.."
Expected behavior

We need to modify "Error message.." in serve cli to something like "Failed to start container with LLM app. \n Error message:"
We need to modify the backend code to provide something more clear about where the problem happened in addition to the error. For instance in #542 it just says "Expecting value.." instead it should say "Failed at backend.scriptname.funcname.step Error ..." or something similar

Alright, let's goo.


Provide a JSON schema

It would be nice to have JSON schema describing the JSON output of the tool, with all the possible values

I don't have any experience writing formal json schema (seems like it would require some study to do it properly), but would something such as below be sufficient?
Rough summary of JSON schema
Enabling output
Use the -oj argument to enable JSON output format
Example output:
Invocation:./gbtoolsid -oj test/zgb_template_2022.0.gb
{
"file": "zgb_template_2022.0.gb",
"toolsName": "GBDK", "toolsVersion": "2<IP_ADDRESS>+",
"engineName": "ZGB", "engineVersion": "2022.0+",
"musicName": "hUGETracker", "musicVersion": "SuperDisk",
"soundfxName": null, "soundfxVersion": null
}

Notes:

For every Tool type there are matching ...Name and ...Version field names.

Possible Name fields: ["toolsName", "engineName", "musicName", "soundfxName"]
Matching Version fields: ["toolsVersion", "engineVersion", "musicVersion", "soundfxVersion"]

If a Tool type is not detected then non-quoted null will be used for both the name and version field values.
If a Tool type entry has been detected but lacks version data then an empty quoted string "" will be used for the version field value.
While multiple entries for each Tool type can be detected, only the first entry detected for each category will be rendered in the JSON format (the default console format will render all detected entries). This scenario is uncommon and mostly occurs for game ROMs which have crack-tros with additional music drivers appended to them.

Current Possible Values

Only Tool type Names are enumerated below.
All possible Tool type Versions are not enumerated, however the values will all be quoted strings (when the value is not null)

{
    "file": [<string quoted filename>],
    "toolsName": [null, "GBDK", "Turbo Rascal Syntax Error"],
    "toolsVersion": ["<various>", "", "Unknown"],
    "engineName":  [null, "GBStudio", "ZGB"],
    "engineVersion": ["<various>", "", "Unknown"],
    "musicName": [null, "GHX", "DevSound", "DevSound", "Visual Impact", "MusyX", "Freaq", "LSDJ", "hUGETracker", "Trackerboy engine", "Black Box Music Box", "Lemon", "GBT Player", "Carillon Player", "MPlay", "GBSoundSystem"],
    "musicVersion": ["<various>", "", "Unknown"],
    "soundfxName": [null, "FX Hammer", "CBT-FX", "VAL-FX", "VGM2GBSFX"],
    "soundfxVersion": ["<various>", "", "Unknown"]
}

Closing this as resolved


Way to override JacksonService

I'd like to change implementation of JacksonService. I want to do it, because it's only way to pass modules to jackson's ObjectMapper which used for config parsing.

Any specifics? You mean like adding "contribute" methods to BQCoreModule?

Motivation: to support kotlin data classes in config factories i need to register jackson kotlin module in ObjectMapper.

You mean like adding "contribute" methods to BQCoreModule?

Yep, it can be done with this approach.

Still undecided on exposing even more of Jackson APIs in the Bootique core. An alternative solution to integrating Kotlin would be creating bootique-kotlin module that overrides JacksonService with its own subclass of DefaultJacksonService. Not as clean as contribution, but should work never the less.

I'm found that our jackson configuration (or jackson itself) doesn't support references, for example:
- step: &id001
    instrument:      Lasik 2000
    pulseEnergy:     5.0
    pulseDuration:   10
    repetition:      500
    spotSize:        2mm
- step: *id001                   # refers to the first step (with anchor &id001)

So i'm thinking about contributing ConfigurationFactory itself. It can be EtcdConfigurationFactory, GroovyConfigurationFactory, KotlinScriptConfigurationFactory, etc.

Interesting point on references. I think we need a separate bug report on this

Bootique provide ability to override module:
fun Bootique.withKotlinConfig(): Bootique {
    return this.override(BQCoreModule::class.java)
        .with(KotlinConfigModule::class.java)
}


GEODE-5667: Fix manual-start default value

GEODE-5667: Fix manual-start default value

Fixed minor warnings in test classes.
The create gateway-receiver command now sets the manual-start
property as true when the parameter is specified without a value.

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[X] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[X] Is your initial contribution a single, squashed commit?

[X] Does gradlew build run cleanly?

[X] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

@davebarnes97: I'm trying to check why the integrationTest CI build is failing but, unfortunately, the link provided by GitHub doesn't work (it shows a never ending loading page) so I'll need to run this locally to check what's happening.
Will add a new squashed commit once I find the problem.
Cheers.

@jujoramos When you merge this PR, please keep the JIRA ticket open and reassign it to me. I'll update the user guide.


shellinabox: fix ssl/tls support

shellinboxd was not finding libssl.so in it's library path, so it was
falling back to ssl disabled mode. Also, the path to openssl was
hardcoded to /usr/bin/openssl, so shellinaboxd could not generate SSL
certificates once libssl.so was added to LD_LIBRARY_PATH.
I also added myself as a maintainer as I'm interested in making a module for this package and want to monitor it's health.
(sorry for the double pull requests. :frowning: I mucked up the first one and deleted the branch, but now I realise I could have just edited it in place. Lesson learned for next time).
Cheers

Thank you!


StagingError occurs during pushing APP to cf with diego enabled

When pushing APP to latest cf instance with diego enabled.
I can see StagingError
:hello stanleyshen$ cf enable-diego stanley
Setting stanley Deigo support to true
Ok

Verifying stanley Deigo support is set to true
Ok

:hello stanleyshen$ cf start stanley
Starting app stanley in org test / space test as admin...
Downloading staticfile_buildpack...
Downloading python_buildpack...
Downloading binary_buildpack...
Downloading php_buildpack...
Downloading go_buildpack...
Downloading java_buildpack...
Downloading ruby_buildpack...
Downloading nodejs_buildpack...
Downloading admin_console_buildpack...
Downloaded java_buildpack (160.7K)
Downloaded binary_buildpack (8.3K)
Downloaded staticfile_buildpack (2.5M)
Downloaded nodejs_buildpack (50.1M)
Downloaded python_buildpack (254M)

FAILED
StagingError

TIP: use 'cf logs stanley --recent' for more information

The env is
:hello stanleyshen$ bosh stemcells
Acting as user 'admin' on 'Bosh Lite Director'

+---------------------------------------------+---------------+---------+--------------------------------------+
| Name                                        | OS            | Version | CID                                  |
+---------------------------------------------+---------------+---------+--------------------------------------+
| bosh-warden-boshlite-ubuntu-trusty-go_agent | ubuntu-trusty | 3147*   | d0b3e660-03e4-440d-48ee-afd1378a29ee |
+---------------------------------------------+---------------+---------+--------------------------------------+

(*) Currently in-use

Stemcells total: 1
:hello stanleyshen$ bosh releases
Acting as user 'admin' on 'Bosh Lite Director'

+-------------------+-----------------+-------------+
| Name              | Versions        | Commit Hash |
+-------------------+-----------------+-------------+
| cf                | 233+dev.1*      | 4ef5b279+   |
| diego             | 0.1460.0+dev.1* | 972164ef    |
| etcd              | 38*             | af52789b+   |
| garden-linux      | 0.334.0*        | 88f7ec39    |
+-------------------+-----------------+-------------+

The APP I pushed is a very simple nodejs application.
It looks like the downloading of buildpack is not stable. If I start the APP again, and I can see it works this times.

:hello stanleyshen$ cf start stanley
Starting app stanley in org test / space test as admin...
Downloading ruby_buildpack...
Downloading go_buildpack...
Downloading staticfile_buildpack...
Downloading java_buildpack...
Downloading nodejs_buildpack...
Downloading php_buildpack...
Downloading python_buildpack...
Downloading binary_buildpack...
Downloaded staticfile_buildpack (2.5M)

FAILED
StagingError

TIP: use 'cf logs stanley --recent' for more information
:hello stanleyshen$ cf start stanley
Starting app stanley in org test / space test as admin...
Downloading staticfile_buildpack...
Downloading go_buildpack...
Downloading java_buildpack...
Downloading nodejs_buildpack...
Downloading php_buildpack...
Downloading binary_buildpack...
Downloading ruby_buildpack...
Downloading python_buildpack...
Downloaded binary_buildpack (8.3K)
Downloaded nodejs_buildpack (50.1M)

FAILED
StagingError

TIP: use 'cf logs stanley --recent' for more information
:hello stanleyshen$ cf start stanley
Starting app stanley in org test / space test as admin...
Downloading nodejs_buildpack...
Downloading python_buildpack...
Downloading go_buildpack...
Downloading php_buildpack...
Downloading java_buildpack...
Downloading binary_buildpack...
Downloading ruby_buildpack...
Downloading staticfile_buildpack...
Downloading python_buildpack failedDownloading staticfile_buildpack failedDownloading java_buildpack failedDownloading php_buildpack failedDownloaded ruby_buildpack (244M)
Downloaded binary_buildpack (8.3K)
Downloaded java_buildpack (160.7K)
Downloaded nodejs_buildpack (50.1M)
Downloaded go_buildpack (366.2M)

FAILED
StagingError

TIP: use 'cf logs stanley --recent' for more information
:hello stanleyshen$ cf start stanley
Starting app stanley in org test / space test as admin...
Downloading nodejs_buildpack...
Downloading java_buildpack...
Downloading staticfile_buildpack...
Downloading ruby_buildpack...
Downloading php_buildpack...Downloading python_buildpack...
Downloading binary_buildpack...
Downloading go_buildpack...
Downloaded binary_buildpack
Downloaded go_buildpack
Downloaded python_buildpack
Downloaded php_buildpack
Downloaded staticfile_buildpack
Downloaded ruby_buildpack
Downloaded nodejs_buildpack
Downloaded java_buildpack (160.7K)
Creating container
Successfully created container
Downloading app package...
Downloaded app package (53.6K)
Staging...

It happens on a clean Cell I think, once all the buildpacks are downloaded, if I delete the app and push it again, I cannot see this issue again.
Let me know if you need some logs.

Thanks for the report, @StanleyShen. We do see this issue intermittently on our own BOSH-Lite deployments. It's coming up as an issue at all primarily because the cell rep downloads buildpacks into its download cache only on demand when requested by a buildpack-app staging task. When an auto-detect staging task comes in to a newly created or updated cell, it therefore causes it to download all the buildpacks at once. This can be slow, but we're looking into why it sometimes fails even before the staging timeout is reached. We'll be investigating these failures end-to-end in https://www.pivotaltracker.com/story/show/116232733, but we suspect it may be a combination of a relatively short 5-second idle timeout on the downloader and lack of responsiveness from Cloud Controller or the backing blobstore on constrained infrastructures such as BOSH-Lite. Since we already have this investigation scheduled, I'll close this issue for now.
Best,
Eric


Improve call stack on ObtainSnapshot_cancel

Description
Tweaks error surfacing in getVersions by regenerating the error stack. This event is a frequent cause of document access issues for partners on odsp--e.g. auth failure gets surfaced via this error. Having a more clear stack that's understandable for application authors seems like a nice value add.

@jatgarg @markfields Curious how you feel about something like this. I went down the rabbithole a bit looking into why the call stacks we get in this codepath are so useless :P. I've mostly been playing around with TS playground for a faster inner loop vs. modifying this code, so I still need to confirm this works. I'd expect it does though.

@Abe27342, I think it's worth asking - why do we care? I.e. what is the scenario where it matters?

I'm interested in giving more hints for FF users to self-diagnose issues based on telemetry. I think historically FF hot has been looped into a few incidents from partner teams where clearer call stacks from the driver layer would have provided strong hints to the application what the problem was and that it wasn't on FF side (one was yesterday, motivating this). Not sure it can be linked from here, but the Loop backlog also has an item tracking this as a pain point. If you look at the FF hot shift table, the row "UnhandledRejections thrown from odsp-driver" has some more details and a link to that item.

Ok I'm just seeing this PR, haven't looked at the change yet. I'll take a look today. @Abe27342 this problem has bugged me for years, eager to see what you had in mind!

I'm open to re-opening and merging if we want. It does make the code a bit uglier IMO, but maybe that's worth it for better diagnostics. I think the right fix long-term for this code and some other places where we do explicit promise management for the purpose of racing is to use promise.any but that requires ES2021.


DynDNS - Add RFC1918 check

Seems that sometimes DynDNS attempts to use an RFC1918 address to update the server. Added a check for that to reject RFC1918 addresses being used. Issue #2745

I feel like we are walking around in circles here:
https://github.com/opnsense/plugins/blob/6a7babcf20398b1fa845deff35edcda498384925/dns/dyndns/src/etc/inc/plugins.inc.d/dyndns/phpDynDNS.inc#L371-L374
If the service needs the dummy update we need to do the dummy update. If it's prevented we prevent the service from working.


TST: Enable LoftQ 8bit tests

Due to PR #1276, the bug that prevented use of LoftQ with 8bit quantization has now been fixed. Therefore, the tests no longer need to be skipped.
Note
I tested locally with GPU and the tests passed.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Cannot Find Custom Playnite Install

I have Playnite installed to a custom folder outside of the original programmed path and Boilr is not detecting it, even if I try to select it as a portable install and direct it to the folder it's installed in, any idea why?
PS: Adding support for uninstalled games through Playnite was a huge brain move, cannot wait to get it working!

You have to specifiy the Path to the Desktop exe in the Path for the Portable Instance. Not the folder.
Then it works.

You have to specifiy the Path to the Desktop exe in the Path for the Portable Instance. Not the folder. Then it works.

I've tried:
"C:\Users\Bloo\Games\Playnite\Playnite.DesktopApp.exe"
C:\Users\Bloo\Games\Playnite\Playnite.DesktopApp.exe
C:\Users\Bloo\Games\Playnite
C:\Users\Bloo\Games\Playnite\
None have worked.
"Import from Playnite" & "Use Playnite Portable Version" are both ticked.
Program restarted between each attempt.

Playnite has to be closed. Thats another point.

@Blootrix could you upload your games.db file?
Might be that the parser is not able to find the games in it for some reason, but hard to know without the db file.

@Blootrix I think that it should be possible to give a better error message if playnite is still open (since it always gives the same error message)

Yeah. It throws a Access Denied.

Ran into this just now, the solution was a mix of all of the above, along with an extra step.
Essentially — my library wasn't in the same folder as the Playnite.DesktopApp.exe, it was in AppData\Roaming\Playnite instead. The solution was to:

Shut down Playnite
Move the library folder to the folder where I had my portable Playnite installation
Turn on Playnite, open Settings > Advanced, and update the Database Location to the new location

This is how the it looks like in the config now:
portable_launcher_path = "C:\\Games\\Playnite\\Playnite.DesktopApp.exe"

It would be useful to have a separate field for the library location for situations like these 😄


Fix lowpass and highpass parameters of filter

To address issue #30, the keyword fc is now used to indicate cutoff frequencies.
The f_lo and f_hi keywords can still be used (so as to not break old code), but warnings will appear instructing to change to using the fc parameter.

Hello @srcole, Thank you for submitting the Pull Request !

In the file neurodsp/filt.py, following are the PEP8 issues :

Line 95:62: E203 whitespace before ':'
Line 100:62: E203 whitespace before ':'

In the file neurodsp/shape/cyclepoints.py, following are the PEP8 issues :

Line 186:13: E722 do not use bare except'
Line 205:13: E722 do not use bare except'

Do see the Hitchhiker's guide to code style


Fix incompatible types of objects

This pull request fix a error.

Explanation

Changing the object type as it is causing android app installation errors.

How to reproduce

Install react latest versions: react 17.0.2, react-native 0.66.3;
Install lottie-react-native@latest version and it peerDependencies<EMAIL_ADDRESS>Run npx react-native run-android;
The error described at #820 will be displayed.

Thanks @Nimbo1999  !

I have stumbled on this issue 20 minutes ago, and I have literally found this patch 5 minutes ago.
Speaking of timing... just perfect.
Thank you! @Nimbo1999 / @emilioicai


[FEATURE] - Allow auto-tunneling for mobile network without location access

Hello, and first thank your for making this nice app!
I usually only enable location access when I need it and normally don't even use google location services for privacy reasons.
I don't know if this is possible at all, but as I'm in foreign networks very rarely and happy enabling vpn manually in this cases I would love to able to enable auto-tunneling for mobile network only without enabling location access.

Thank you for using the app! This is a good idea and should not be hard to implement. I will get to work on it.

That's really nice to hear, thanks in advance!

Just wanted to chime in and say I love the app, and it would be great to have this feature as I've noticed it pings location quite often and thus shows the location icon in the top right a lot (Grapheme OS), it would be great if it could work just based on the network name.  Keep up the great work!

Just wanted to chime in and say I love the app, and it would be great to have this feature as I've noticed it pings location quite often and thus shows the location icon in the top right a lot (Grapheme OS), it would be great if it could work just based on the network name. Keep up the great work!

Hello! This feature is already in the app. Please let me know if you are having any issues with it. To use this feature, you would just keep "tunnel on untrusted wifi" disabled and enable tunnel on mobile data and then start auto tunneling. It should allow auto tunneling without any location permissions.

Hello! This feature is already in the app. Please let me know if you are having any issues with it. To use this feature, you would just keep "tunnel on untrusted wifi" disabled and enable tunnel on mobile data and then start auto tunneling. It should allow auto tunneling without any location permissions.

How come it needs location permissions for tunneling on untrusted wifit? I would like to have this enabled so that when I travel and use hotel wifi etc. I would get the autotunneling, without having to keep location on.

Hello! This feature is already in the app. Please let me know if you are having any issues with it. To use this feature, you would just keep "tunnel on untrusted wifi" disabled and enable tunnel on mobile data and then start auto tunneling. It should allow auto tunneling without any location permissions.

How come it needs location permissions for tunneling on untrusted wifit? I would like to have this enabled so that when I travel and use hotel wifi etc. I would get the autotunneling, without having to keep location on.

I know it is pretty ridiculous that this is the case, however, Android considers wifi name to be precise location data. Without these location permissions, Android will not allow an app to read the currently connected wifi name from the system.


Prompt for Ipykernel install then Change Kernel hangs a notebook.

Applies To

[ ] Notebooks (.ipynb files)
[ ] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?

Open a notebook
Select an interpreter that does not have ipykernel installed
Run a cell and at the prompt to install ipykernel, choose Change Kernel
Pick the same kernel (that's what I did in the repro, not sure if it matters that it's the same or whether the new kernel should also be lacking ipykernel)
Nothing happens, other than the cell continues to be "running". To work around the problem you have to reload VS Code or the notebook.

VS Code Version
Version: 1.67.0-insider (user setup) Commit: bd95a8d4516d590256eefecfce465a2f10e8f5e5 Date: 2022-04-21T05:16:20.746Z Electron: 17.4.0 Chromium: 98.0.4758.141 Node.js: 16.13.0 V8: <IP_ADDRESS>-electron.0 OS: Windows_NT x64 10.0.22598
Jupyter Extension Version
v2022.4.1001111723
Jupyter logs
No response
Coding Language and Runtime Version
No response
Language Extension Version (if applicable)
No response
Anaconda Version (if applicable)
No response
Running Jupyter locally or remotely?
No response

This only happens if the user chooses the same kernel that didn't have ipykernel installed. Not worth fixing.


PIP-245: Make subscriptions of NonPersistentTopic non-durable

Motivation
There are two types of subscriptions for a topic: Durable and Non-durable.
We create a Consumer with a Durable subscription and a Reader with a Non-durable subscription.
But for NonPersistentTopic, creating a Durable subscription is meaningless, NonPersistentSubscription doesn't have a ManagedCursor to persistent its data. After its consumer disconnected, the subscription couldn't be removed automatically if we didn't set the value of subscriptionExpirationTimeMinutes greater than 0.
For subscriptionExpirationTimeMinutes, it controls the subscription expiration of NonPersistentTopic and PersistentTopic, if we set the value of subscriptionExpirationTimeMinutes greater than 0, it may lead to data loss(The durable subscriptions of PersistentTopic also can be removed).
And the Non-durable subscriptions will be removed automatically after all the consumers disconnected, it's the existing logic.
For the purpose of removing the subscriptions which have no active consumers of NonPersistentTopic and the above reasons, we can make all the subscriptions of a NonPersistentTopic Non-durable.
Goal
Make the subscriptions of NonPersistentTopic Non-durable.
API Changes
Implementation
1. Client-side:
Add the following logic to the constructor of ConsumerImpl before this.subscriptionMode=conf.getSubscriptionMode():
        TopicName topicName = TopicName.get(topic);
        if (!topicName.isPersistent() && conf.getSubscriptionMode().equals(SubscriptionMode.Durable)) {
            conf.setSubscriptionMode(SubscriptionMode.NonDurable);
            log.warn("[{}] Cannot create a [Durable] subscription for a NonPersistentTopic, " +
                    "will use [NonDurable] to subscribe. Subscription name: {}", topic, conf.getSubscriptionName());
        }

Check the topic type, if the topic is a NonPersistentTopic, change the value of subscriptionMode to NonDurable and print WARN log, Users don't need to change their code.
2. Broker-side:
Change the constructor of NonPersistentTopic from
   public NonPersistentSubscription(NonPersistentTopic topic, String subscriptionName, boolean isDurable,
   Map<String, String> properties) {
       this.topic = topic;
       this.topicName = topic.getName();
       this.subName = subscriptionName;
       this.fullName = MoreObjects.toStringHelper(this).add("topic", topicName).add("name", subName).toString();
       IS_FENCED_UPDATER.set(this, FALSE);
       this.lastActive = System.currentTimeMillis();
       this.isDurable = isDurable;
       this.subscriptionProperties = properties != null
               ? Collections.unmodifiableMap(properties) : Collections.emptyMap();
   }

to
   public NonPersistentSubscription(NonPersistentTopic topic, String subscriptionName,
   Map<String, String> properties) {
       this.topic = topic;
       this.topicName = topic.getName();
       this.subName = subscriptionName;
       this.fullName = MoreObjects.toStringHelper(this).add("topic", topicName).add("name", subName).toString();
       IS_FENCED_UPDATER.set(this, FALSE);
       this.lastActive = System.currentTimeMillis();
       this.isDurable = false;
       this.subscriptionProperties = properties != null
               ? Collections.unmodifiableMap(properties) : Collections.emptyMap();
   }

Remove the constructor argument isDurable, and set its value to false.
Compatibility
The PIP will deprecate Durable subscriptions on NonPersistentTopic in 3.0.0, if users want to create a Durable subscription on NonPersistentTopic we will print WARN logs and change isDurable to false.
We will not be compatible with the next version after 3.0.1.
In the next release after 3.0.0, if users want to create Durable subscriptions on NonPersistentTopic, will throw an exception.
Alternatives
No response
Anything else?
No response

Discuss thread: https://lists.apache.org/thread/2ltmyglnb25jy8nk58twkwbglws43bst

The default subscription mode is durable. So users will see the warning log if they upgrade to the new version client? Why not just change isDurable to false for the non-persistent topic subscriptions. And if users are explicitly changed to true, then see the warning log.

The default subscription mode is durable. So users will see the warning log if they upgrade to the new version client?

Yes

Why not just change isDurable to false for the non-persistent topic subscriptions. And if users are explicitly changed to true, then see the warning log.

Because PersistentTopic and NonPersistentTopic subscriptions are using ConsumerConfigurationData,
if we change isDurable to false, when create consumers on PersistentTopic, we also need to add logics to change isDurable to true.
@codelipenghui

Vote thread: https://lists.apache.org/thread/phrktmgln08pgpddvx8vvwzvgnq9kd0w


load_yaml! doesn't seem to work in binary in a library project.

with features = ["yaml"] enabled, this doesn't seem to propagate to binaries in a library project.
Cargo.toml:
[package]
name = "foo"
version = "0.1.0"
authors = ["Daniel Firth<EMAIL_ADDRESS>
[dependencies]
clap = { features = ["yaml"] }

src/lib.rs
#[macro_use]

mod foo {
}

src/bin/bar.rs
#[macro_use]

extern crate foo;
extern crate clap;

use clap::App;

fn main() {
    let yaml = load_yaml!("cli.yml");
}

Results in
src/bin/bar.rs:10:16: 10:25 error: macro undefined: 'load_yaml!'
src/bin/bar.rs:10     let yaml = load_yaml!("cli.yml");

The project is here for convenience: https://github.com/locallycompact/foo-clap

It works if you use
#[macro_use]

extern crate clap;
extern crate foo;

instead of
#[macro_use]

extern crate foo;
extern crate clap;


Writing CHGCAR fails to leave spaces between numbers

Using format %18.11E when writing CHGCAR leads to missing spaces between numbers when values are negative or have many integers, e.g.,
-3.73037629970E-02-6.77633790800E-02-7.55722442840E-02-5.78415084650E-02-1.93558247350E-02

Such files produce an error when they are read.

VASP (or Fortran?) does some magic when writing numbers such that no more than one integer is written, and when the value is negative, numbers are shifted to decimals and no leading zero is wrriten, i.e.,
-3.73037629970E-02 # chem
# vs
 -.37303762997E-01 # VASP

This ensures that there is always a leading space within the 18 characters.


Update Important-leetcode.md

Thank you for giving me this Opportunity, Please give me Hactober accepted label and merge my PR, Thank You so much.

Great work !
keep it up , try out with some other folders too .

Thank You So Much.

Brother why this is showing, Please labeled it as hacktoberfest-accepted

done
have a look
On Mon, Oct 3, 2022 at 6:39 PM Shaikh Minhaj @.***>
wrote:

[image: image]
https://user-images.githubusercontent.com/107562768/193584329-1c6d1937-3670-4154-b1d9-75f9be6da81c.png
Brother why this is showing, Please labeled it as hacktoberfest-accepted
—
Reply to this email directly, view it on GitHub
https://github.com/abhishekjha812/Hacktoberfest2022/pull/21#issuecomment-1265415228,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANMISZBIJIKSCQXTY4J5FHDWBLLJZANCNFSM6AAAAAAQ2OYRBQ
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

done have a look
…
On Mon, Oct 3, 2022 at 6:39 PM Shaikh Minhaj @.> wrote: [image: image] https://user-images.githubusercontent.com/107562768/193584329-1c6d1937-3670-4154-b1d9-75f9be6da81c.png Brother why this is showing, Please labeled it as hacktoberfest-accepted — Reply to this email directly, view it on GitHub <#21 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANMISZBIJIKSCQXTY4J5FHDWBLLJZANCNFSM6AAAAAAQ2OYRBQ . You are receiving this because you modified the open/close state.Message ID: @.>

still showing.
i think it takes time for updating. we will wait for it.
Thank you for your valuable responce


Undo: Blacklist apple.com

Undo requests blacklist of domains:

apple.com - MS search

apple.com has been seen in 13 true positives and 24 false positives.


Make each SQLiteConnection have its own SQLite handle

Currently all SQLiteConnections share a single underlying SQLite connection stored in an SQLiteDatabase object. This can cause database queries from different threads to be serialised and mixed together, resulting in potential data corruption.
This PR makes each SQLiteConnection have its own SQLite connection handle so that queries from different threads are run on different SQLite connections.

This new version is causing errors in migrations. The error is
[logging] table "fluent" already exists

I traced it to this error
⚠ [FluentError.aggregate: The driver closed successfully without a result] [....../Fluent/QueryBuilder/QueryBuilder+Aggregate.swift:114:34] [Suggested fixes: Check the result set count using `count()` before requesting an aggregate.]

I suspect is has to do with this part of the code
https://github.com/vapor/sqlite/blob/ad2e9bc9f0ed00ef2c6a05f89c1cec605467c90f/Sources/SQLite/Database/SQLiteConnection.swift#L87-L100
Changing the last line to the following makes the error disappear. But I doubt if it is the right solution.
            self.eventLoop.execute {
                promise.succeed(result: ())
            }

@MrLotU Could you please review the PR? Thank you!

Looks like this patch isn't building on Linux.
...
Compile Swift Module 'SQLBenchmark' (5 sources)
Compile Swift Module 'SQLite' (30 sources)
/root/project/Sources/SQLite/Database/SQLiteConnection.swift:62:16: error: value of type 'UnsafePointer<Int8>' has no member 'map'
        return sqlite3_errmsg(handle).map(String.init(cString:))
               ^~~~~~~~~~~~~~~~~~~~~~ ~~~
error: terminated(1): /usr/bin/swift-build-tool -f /root/project/.build/debug.yaml main output:

Let me know if you need any help debugging this.

Thanks a ton for this patch @dpgao! I'm going to do a minor bump on this one since it technically breaks public API by making isClosed get-only. We should really have this bug fix before the next major version though. 👍

Hey @dpgao, you just merged a pull request, have a coin!
You now have 1 coins.


unfold before calling iconv_mime_decode

proposed solution for #187
resolves: #187

@glensc thanks! And thanks for the streak of impeccable patches (honestly: very well researched and tested patches!)
Manually merged via zendframework/zend-mail@452735048ff41d177bdac89a117edf50e6e27797


input: Add more keys like PrintScreen

#580
We'll add numpads soon
branch: https://github.com/hajimehoshi/ebiten/tree/numpad


Not all html files are minified

Hello, some html files are minified but other doesn't. Do you have any idea? This is my config:
ngtemplates:    {
evalwebApp:          {
src:        'App/Partial//.html',
dest:       'Scripts/templates.js',
options:    {
//htmlmin:  { collapseWhitespace: true, collapseBooleanAttributes: true }
htmlmin: {
collapseWhitespace: true,
removeComments: true, // Only if you don't use comment directives!
}
}
}
}
"grunt-angular-templates": "0.5.7",
"grunt-usemin": "3.0.0"

I found the anwser, I have template blocks within my html. This blocks aren't minified.


[FEATURE] Do not remove explicitly re-exported imports even if unused

Mypy will prevent users from doing implicit re-exports. So if in module a you do from b import obj, and attempt to do from a import obj elsewhere, that will error. See https://mypy.readthedocs.io/en/stable/command_line.html#cmdoption-mypy-no-implicit-reexport for more context.
There are two ways in which you can explicitly re-export. They are:

When you import and set the __all__ attribute in an __init__.py. pycln as far as I remember has no problem with this.
When you use the explicit re-export syntax that is supported by Mypy, which is from b import obj as obj. This signals to Mypy that you are intentionally, deliberately re-exporting this, and are permitting imports of obj through module a.

It appears that 2 is not respected by pycln. It will remove these re-exports, because they are unused. So the feature request here is that pycln respects this convention and does not remove these. I understand that these are easily silenced through the comment directives, but the ask here is to make that superfluous.

Yes please. Explicit re-exports using as as a standard is kinda new, and has been quite popularized in stubs. But it's a useful signal in .py files too ! You're already telling tools "this is meant to be used outside this module", pycln should understand it too.


ScaladocParser: add support for fenced code blocks

Implements https://spec.commonmark.org/0.29/#fenced-code-blocks. Supersedes #2332.

Great, thank you! I'm closing my PR then.


Make binding to ItemsControl.Items a compile error.

What does the pull request do?
Fixes the behavior described in #10946 whereby using <ItemsControl Items="{Binding Foo}" would result in a Binding object being added to the ItemsControl.Items collection instead of resulting in a compile error.
Depends on https://github.com/kekekeks/XamlX/pull/85: this PR disallows invoking "collection behavior" for property setters when using attribute syntax.
Required adding a Flyout.FlyoutPresenterClasses setter. This property was previously buggy without the setter (see https://github.com/AvaloniaUI/Avalonia/issues/10946#issuecomment-1506563268). Added tests for setting classes that I put in place when investigating this.
Breaking changes
Can no longer add items to a collection in XAML using attribute syntax. This shouldn't have been possible before, but it's possible that someone was depending on it working.
Fixed issues
Fixes #10946

You can test this PR using the following package version. 11.0.999-cibuild0033202-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]


Clot mobile wallet support

Description

WC_CLOT_MOBILE WalletType
getWCClot method
Clot mobile connect button

Additional info
Clot Wallet AppStore URL link
Main use case: connecting with stargaze.zone using WalletConnectV2 protocol

Released in v0.1.18


Removed unneeded comments and declarations. Start of code coverage increase. Error checking in more places.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: huebnerr:x: Roger HuebnerRoger Huebner seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Trying to add CustomAttribute vallue getting an error.

The issue tracker is for reporting product deficiencies. "How do I?" questions should be posted to the discussion forum at https://groups.google.com/group/google-apps-manager. When in doubt, start at the discussion forum and return here only when instructed to do so.
Please confirm the following:

I have upgraded to the latest GAM release from https://github.com/GAM-team/GAM/releases and I still have this issue.
I am typing the command as described in the GAM Wiki at https://github.com/GAM-team/GAM/wiki

Full steps to reproduce the issue:
I have created two Custom attributes
1>> Category: AzureAD, Name: ImmutableID , Info type: text, Visibility: Visible to user and admin No of Value: Single Value.
I have used this cmdlet to add the value (gam update user<EMAIL_ADDRESS>AzureAD.ImmutableID ZNnP9o/CHUSCtrdse7fgCw==) this command works perfectly.
2>>a) Category: Enhanced desktop security, Name: Local windows accounts, Info type: text, Visibility: Visible to user and admin No of Value: Multi Value.
b) Category: AD accounts, Name: Local windows accounts, Info type: text, Visibility: Visible to user and admin No of Value: Multi Value.
for this I have tried below formats
*gam update user<EMAIL_ADDRESS>"Enhanced desktop security.AD accounts" Domain.org\UPN
*gam update user<EMAIL_ADDRESS>"Enhanced_desktop_security.AD_accounts" Domain.org\UPN
Error: ERROR: 400: Invalid Input: custom_schema - invalid
So I hope there is something missing, would it be possible to get the right cmdlet.

Do: gam show schemas
I believe that you want the following but seeing the schemas will help.

gam update user @.** @.> "Enhanced_desktop_security.Local_windows_accounts" Domain.org\UPN
gam update user @. @.**> "AD_accounts:Local_windows_accounts" Domain.org\UPN

Ross

Ross Scroggs
@.***

On Aug 8, 2024, at 9:30 AM, iabud-24 @.***> wrote:
The issue tracker is for reporting product deficiencies. "How do I?" questions should be posted to the discussion forum at https://groups.google.com/group/google-apps-manager. When in doubt, start at the discussion forum and return here only when instructed to do so.
Please confirm the following:
I have upgraded to the latest GAM release from https://github.com/GAM-team/GAM/releases and I still have this issue.
I am typing the command as described in the GAM Wiki at https://github.com/GAM-team/GAM/wiki
Full steps to reproduce the issue:
I have created two Custom attributes
1>> Category: AzureAD, Name: ImmutableID , Info type: text, Visibility: Visible to user and admin No of Value: Single Value.
I have used this cmdlet to add the value (gam update user @.*** @.***> AzureAD.ImmutableID ZNnP9o/CHUSCtrdse7fgCw==) this command works perfectly.
2>>a) Category: Enhanced desktop security, Name: Local windows accounts, Info type: text, Visibility: Visible to user and admin No of Value: Multi Value.
b) Category: AD accounts, Name: Local windows accounts, Info type: text, Visibility: Visible to user and admin No of Value: Multi Value.
for this I have tried below formats
gam update user @.** @.> "Enhanced desktop security.AD accounts" Domain.org\UPN
gam update user @. @.**> "Enhanced_desktop_security.AD_accounts" Domain.org\UPN
Error: ERROR: 400: Invalid Input: custom_schema - invalid
So I hope there is something missing, would it be possible to get the right cmdlet.
—
Reply to this email directly, view it on GitHub https://github.com/GAM-team/GAM/issues/1695, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCTYL4US2RL5MEOD3TFTGLZQOMK5AVCNFSM6AAAAABMG2467KVHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ2TMMRQGM4TKMI.
You are receiving this because you are subscribed to this thread.

Hello,
see below schema's
C:\Windows\system32>gam show schemas
Schema: AzureAD
schemaId: 3y8R8zOfQG-UtXsoeMbUyQ==
displayName: AzureAD
Field: ImmutableID
fieldId: KJupEVoAR8y1YhJ5W4POxw==
fieldType: STRING
displayName: ImmutableID
multiValued: False
readAccessType: ADMINS_AND_SELF
Schema: Enhanced_desktop_security
schemaId: Md6sNo8VQCSSoYWaXx2WPA==
displayName: Enhanced desktop security
Field: Local_windows_accounts
fieldId: sg86RjVXQwC-GBVEi-hMkw==
fieldType: STRING
displayName: Local windows accounts
multiValued: True
readAccessType: ADMINS_AND_SELF
Field: ADaccounts
fieldId: vq3irX-nT7nQWsZ8WPtRyg==
fieldType: STRING
displayName: AD accounts
multiValued: True
readAccessType: ADMINS_AND_SELF
On Thu, Aug 8, 2024 at 8:42 PM Ross Scroggs @.***>
wrote:

Do: gam show schemas
I believe that you want the following but seeing the schemas will help.

gam update user @.** @.>
"Enhanced_desktop_security.Local_windows_accounts" Domain.org\UPN
gam update user @. @.**>
"AD_accounts:Local_windows_accounts" Domain.org\UPN

Ross

Ross Scroggs
@.***

On Aug 8, 2024, at 9:30 AM, iabud-24 @.***> wrote:
The issue tracker is for reporting product deficiencies. "How do I?"
questions should be posted to the discussion forum at
https://groups.google.com/group/google-apps-manager. When in doubt, start
at the discussion forum and return here only when instructed to do so.
Please confirm the following:
I have upgraded to the latest GAM release from
https://github.com/GAM-team/GAM/releases and I still have this issue.
I am typing the command as described in the GAM Wiki at
https://github.com/GAM-team/GAM/wiki
Full steps to reproduce the issue:
I have created two Custom attributes
1>> Category: AzureAD, Name: ImmutableID , Info type: text, Visibility:
Visible to user and admin No of Value: Single Value.
I have used this cmdlet to add the value (gam update user @.***
@.***> AzureAD.ImmutableID ZNnP9o/CHUSCtrdse7fgCw==) this command
works perfectly.
2>>a) Category: Enhanced desktop security, Name: Local windows accounts,
Info type: text, Visibility: Visible to user and admin No of Value: Multi
Value.
b) Category: AD accounts, Name: Local windows accounts, Info type: text,
Visibility: Visible to user and admin No of Value: Multi Value.
for this I have tried below formats
gam update user @.** @.> "Enhanced desktop security.AD
accounts" Domain.org\UPN
gam update user @. @.**>
"Enhanced_desktop_security.AD_accounts" Domain.org\UPN
Error: ERROR: 400: Invalid Input: custom_schema - invalid
So I hope there is something missing, would it be possible to get the
right cmdlet.
—
Reply to this email directly, view it on GitHub <
https://github.com/GAM-team/GAM/issues/1695>, or unsubscribe <
https://github.com/notifications/unsubscribe-auth/ACCTYL4US2RL5MEOD3TFTGLZQOMK5AVCNFSM6AAAAABMG2467KVHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ2TMMRQGM4TKMI>.

You are receiving this because you are subscribed to this thread.

—
Reply to this email directly, view it on GitHub
https://github.com/GAM-team/GAM/issues/1695#issuecomment-2276244508, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BKM6ZVU3XMO5SMMHEN6H3R3ZQONXRAVCNFSM6AAAAABMG2467KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZWGI2DINJQHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

 ::= .
 ::=
...
 [multivalued [type home|other|work|(custom )]]
The last two custom schema fields are multivalued so you have to add an additional keyword
update AzureAD.ImmutableID xxx
update Enhanced_desktop_security.Local_windows_accounts multivalued xxx
update Enhanced_desktop_security.ADaccounts multivalued xxx
update Enhanced_desktop_security.Local_windows_accounts multivalued type work xxx
update Enhanced_desktop_security.ADaccounts multivalued type work xxx
Ross

Ross Scroggs
@.***

On Aug 9, 2024, at 4:16 AM, iabud-24 @.***> wrote:
Hello,
see below schema's
C:\Windows\system32>gam show schemas
Schema: AzureAD
schemaId: 3y8R8zOfQG-UtXsoeMbUyQ==
displayName: AzureAD
Field: ImmutableID
fieldId: KJupEVoAR8y1YhJ5W4POxw==
fieldType: STRING
displayName: ImmutableID
multiValued: False
readAccessType: ADMINS_AND_SELF
Schema: Enhanced_desktop_security
schemaId: Md6sNo8VQCSSoYWaXx2WPA==
displayName: Enhanced desktop security
Field: Local_windows_accounts
fieldId: sg86RjVXQwC-GBVEi-hMkw==
fieldType: STRING
displayName: Local windows accounts
multiValued: True
readAccessType: ADMINS_AND_SELF
Field: ADaccounts
fieldId: vq3irX-nT7nQWsZ8WPtRyg==
fieldType: STRING
displayName: AD accounts
multiValued: True
readAccessType: ADMINS_AND_SELF
On Thu, Aug 8, 2024 at 8:42 PM Ross Scroggs @.***>
wrote:

Do: gam show schemas
I believe that you want the following but seeing the schemas will help.

gam update user @.** @.>
"Enhanced_desktop_security.Local_windows_accounts" Domain.org\UPN
gam update user @. @.**>
"AD_accounts:Local_windows_accounts" Domain.org\UPN

Ross

Ross Scroggs
@.***

On Aug 8, 2024, at 9:30 AM, iabud-24 @.***> wrote:
The issue tracker is for reporting product deficiencies. "How do I?"
questions should be posted to the discussion forum at
https://groups.google.com/group/google-apps-manager. When in doubt, start
at the discussion forum and return here only when instructed to do so.
Please confirm the following:
I have upgraded to the latest GAM release from
https://github.com/GAM-team/GAM/releases and I still have this issue.
I am typing the command as described in the GAM Wiki at
https://github.com/GAM-team/GAM/wiki
Full steps to reproduce the issue:
I have created two Custom attributes
1>> Category: AzureAD, Name: ImmutableID , Info type: text, Visibility:
Visible to user and admin No of Value: Single Value.
I have used this cmdlet to add the value (gam update user @.***
@.***> AzureAD.ImmutableID ZNnP9o/CHUSCtrdse7fgCw==) this command
works perfectly.
2>>a) Category: Enhanced desktop security, Name: Local windows accounts,
Info type: text, Visibility: Visible to user and admin No of Value: Multi
Value.
b) Category: AD accounts, Name: Local windows accounts, Info type: text,
Visibility: Visible to user and admin No of Value: Multi Value.
for this I have tried below formats
gam update user @.** @.> "Enhanced desktop security.AD
accounts" Domain.org\UPN
gam update user @. @.**>
"Enhanced_desktop_security.AD_accounts" Domain.org\UPN
Error: ERROR: 400: Invalid Input: custom_schema - invalid
So I hope there is something missing, would it be possible to get the
right cmdlet.
—
Reply to this email directly, view it on GitHub <
https://github.com/GAM-team/GAM/issues/1695>, or unsubscribe <
https://github.com/notifications/unsubscribe-auth/ACCTYL4US2RL5MEOD3TFTGLZQOMK5AVCNFSM6AAAAABMG2467KVHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ2TMMRQGM4TKMI>.

You are receiving this because you are subscribed to this thread.

—
Reply to this email directly, view it on GitHub
https://github.com/GAM-team/GAM/issues/1695#issuecomment-2276244508, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BKM6ZVU3XMO5SMMHEN6H3R3ZQONXRAVCNFSM6AAAAABMG2467KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZWGI2DINJQHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub https://github.com/GAM-team/GAM/issues/1695#issuecomment-2277719153, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCTYL64VHWZ6IBCDTG5AV3ZQSQIDAVCNFSM6AAAAABMG2467KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZXG4YTSMJVGM.
You are receiving this because you commented.


Exclude .Rproj by default for R projects?

This is a nice-to-have, but we should probably exclude .RProj and other similar files (like .Rhistory) by default for R content

Only package files, and entrypoint files are included by default so this has been taken care of.


Update gs script

simpleEvents doesn't exist, I tried switching to formattedEvents and it worked!

I can also confirm that everything else is still working, thank you!

wow, can't believe I didn't notice this! Thanks for the PR.


chore: update data packages aug 23 hotfix

Description of changes
Bump API Deps
Issue #, if available
N/A
Description of how you validated changes
Tests, e2e in api repo, will run e2e's in cli repo as well.
Checklist

[x] PR description included
[x] yarn test passes

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

Closing i nfavor of https://github.com/aws-amplify/amplify-cli/pull/10930


Dropping outside the list has no effect

When I start dragging the placeholder is set to the position where the dragged item will go when dropping. But when I release the mouse outside the list, the item moves back to its initial position. This is very confusing for the user as he excepts the item to go to where the placeholder is.
So either the the item moves to last position of the placeholder or the placeholder moves back to the initial position of the item, when the item is dragged outside the list.

This is intended behaviour, as a drop outside a list is a cancel event. However I agree that as an enhancement the placeholder could be moved back to the original position when moving outside. If somebody is interested in adding this features, please open a PR.


⚠ Bofrost.fr has degraded performance

In e0f415a, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1509 ms

Resolved: Bofrost.fr performance has improved in e6567b4.


ZCPU Editor alignment of buttons fixed

The debugger's buttons are higher than the other buttons. There's also much space between the debugger buttons.
![Old Buttons] (http://i.imgur.com/4EQ77bI.png)
![New Buttons] (http://i.imgur.com/eNVM3OZ.png)

umm ok


Publish v1.0.0

Inspirations

Hyde
In the novel, The Strange Case of Dr. Jeykll and Mr. Hyde, Mr. Poole is Dr. Jekyll's virtuous and loyal butler. Similarly, Poole is an upstanding and effective butler that helps you build Jekyll themes. It's made by @mdo.

Jelly Bean versus Dr. Jekyll & Mr. Hyde June, 1989
— by C. W. Meisterfeld (Author), Darlene Perez (Author), Richard Flinn (Author)

Frozen 2013 film

[ ] Tumblr
[ ] Twitter
[ ] Instagram
[ ] Facebook ??

Tumblr http://rimeofficial.tumblr.com/post/144947670810/jelly-bean-v100-frozen-a-slenderize-two-column

Jelly Bean v1.0.0 (Frozen) https://github.com/RimeOfficial/jelly-bean inspiration Disney Frozen 2013 Film #rime #disney #frozen #jellybean #jellybelly #release #github #jekyll #hyde


Add analytic code resource type

Purpose
Add the resource type, mirage, and icons for the Analytic Code resource type
Summary of Changes

Add AnalyticCode to resource types enum
Add hasAnalyticCode boolean to registration model
Fix mirage for SHARE and Registrations
Add placeholder icon
Add translation string for Resource List

@futa-ikeda That'll get done in my other PR that I'm working on. I need that same functionality, so I'm refactoring that function and a similar one for disabled icons into a helper file.


CollectionViewCell wont swipe (Actions are shown below the cell)

CollectionView setup (Not using CVController like in the example):
let layout = UICollectionViewFlowLayout()
layout.scrollDirection = .vertical

collectionView = UICollectionView(frame: .zero, collectionViewLayout: layout)
collectionView.delegate = self
collectionView.dataSource = self
collectionView.backgroundColor = .white
collectionView.bounces = true
collectionView.alwaysBounceVertical = true
collectionView.allowsSelection = true
collectionView.allowsMultipleSelection = false
collectionView.showsVerticalScrollIndicator = true
collectionView.translatesAutoresizingMaskIntoConstraints = false
collectionView.contentInsetAdjustmentBehavior = .never

UICollectionViewDataSource: cellForItemAt setup (ItemCell is a subclass of SwipeCollectionViewCell):
    func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
        guard indexPath.section == 1 else { return UICollectionViewCell() }

        let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "ItemCell", for: indexPath) as! ItemCell
        cell.delegate = self
        cell.setupViews()
        cell.configureCell(item: Item())
        return cell
    }

SwipeCollectionViewCellDelegate setup:
extension MainViewController: SwipeCollectionViewCellDelegate {
    func collectionView(_ collectionView: UICollectionView, editActionsForItemAt indexPath: IndexPath, for orientation: SwipeActionsOrientation) -> [SwipeAction]? {
        guard orientation == .right else { return nil }
        let deleteAction = SwipeAction(style: .destructive, title: "Delete") { action, indexPath in
            action.fulfill(with: .reset)
        }
        deleteAction.image = UIImage(named: "Trash")
        deleteAction.transitionDelegate = ScaleTransition.default
        return [deleteAction]
    }
}

Result:

Add your cell content to the contentView.

Add your cell content to the contentView.

what means about Ad cell content to the contentView.


Feature/0.12.0 rc compat

Resolves #17

Good stuff, thanks!

@nickhudkins - added you as a collaborator on this :)

@brentvatne thanks so much :) I'll be working on the android version when I get a little bit of time (hopefully before the end of the week)


update jQuery version in example

I know it's only an example, but best to have the most recent version here which doesn't have a known security vulnerability.
REF:

https://snyk.io/vuln/npm:jquery
https://blog.jquery.com/2019/04/10/jquery-3-4-0-released/

LGTM. Make sense. Thanks for contributing, @coliff!


Add comments support

Comments stating with # are not highlighted differently. Also, vim plugins that use the syntax file to discover the comment marker and allow the user to use a mapping to quickly comment out lines, do not work and revert to their defaults (e.g. /* */ style comments).

Invalid


[Pinterest] 'NoneType' object is not subscriptable

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I remove or skip any mandatory* field

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2022.11.11 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Worldwide
Provide a description that is worded well enough to be understood
I've tried downloading other videos and they seem to work just fine. I'm not sure why this video is not working.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
$ yt-dlp -vU https://pinterest.com/pin/678002918919219984/
[debug] Command-line config: ['-vU', 'https://pinterest.com/pin/678002918919219984/']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.11.11 [8b64402]
[debug] Python 3.10.8 (CPython x86_64 64bit) - Linux-6.0.12-arch1-1-x86_64-with-glibc2.36 (OpenSSL 3.0.7 1 Nov 2022, glibc 2.36)
[debug] exe versions: ffmpeg 5.1.2 (setts), ffprobe 5.1.2, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.14.1, brotlicffi-<IP_ADDRESS>, certifi-2022.09.24, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.2
[debug] Proxy map: {}
[debug] Loaded 1723 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.11.11, Current version: 2022.11.11
yt-dlp is up to date (2022.11.11)
[debug] [Pinterest] Extracting URL: https://pinterest.com/pin/678002918919219984/
[Pinterest]<PHONE_NUMBER>19219984: Download Pin JSON metadata
ERROR: 'NoneType' object is not subscriptable
Traceback (most recent call last):
  File "/usr/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1485, in wrapper
    return func(self, *args, **kwargs)
  File "/usr/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1561, in __extract_info
    ie_result = ie.extract(url)
  File "/usr/lib/python3.10/site-packages/yt_dlp/extractor/common.py", line 674, in extract
    ie_result = self._real_extract(url)
  File "/usr/lib/python3.10/site-packages/yt_dlp/extractor/pinterest.py", line 136, in _real_extract
    return self._extract_video(data)
  File "/usr/lib/python3.10/site-packages/yt_dlp/extractor/pinterest.py", line 34, in _extract_video
    for format_id, format_dict in data['videos']['video_list'].items():
TypeError: 'NoneType' object is not subscriptable

Fixed in f549b18512570d0c000179df9147415e4eba1649
You can wait for the next release, use the master branch or try the unofficial daily builds
Closing as duplicate of #1772

Ah, I didn't find that issue when I was searching. Thanks for fixing it!


Bratseth/allow non allocatable limits

Well this was a lot more work than anticipated but now it's possible to just configure any limits that contains an AWS flavor. Another use for autoscaling: When you don't want to think about which flavors exist.

Thanks!


Added blockhash, cppformat, efl, embulk, exact-image, kafkacat, librd…

…kafka, moe, natalie and rexster

Merged with some additions. Thank you again for all your help the last few months!


Maybe uefi::Result should allow for custom errors

Providing extra data about an error has been needed a couple of times now. Often, the error data is only available for specific error types, so it should be optional to fill it out for other error types. It would be nice if we didn't need to fallback to core::result::Result whenever that happens, because in doing so we lose ergonomics conveniences, treat warnings as errors, and confuse users by giving them two different Result types.
Here are some rough ideas:
// Directly maps the semantics, but implementing a custom error requires work
type Result<T, E: From<Status>> = core::result::Result<Completion<T>, E>;

// We could ease this work by providing something like this
impl<ErrData> From<Status> for (Status, Option<ErrData>) { ... }

// Or, if we feel the Option will always be good enough, flatten the abstraction
type Result<T, ErrData> = core::result::Result<Completion<T>, (Status, Option<ErrData>)>;

// The question is, do we sometimes want this, which is closest to today's practice ?
type Result<T, ErrData: Default> = core::result::Result<Completion<T>, (Status, ErrData)>;

I think at this point we're getting into the more general, unresolved Rust issue of how to design composable error types.
The community already went through the standard Error type, error-chain, failure-rs, etc. and I still don't know which solution works best.

From what I have seen so far, every popular attempt at a general-purpose error handling abstraction uses dynamic memory allocation for type erasure. I don't think this is a luxury that we can afford at the core library layer (though we might integrate it in a higher-level layer like uefi-services), so we may need an abstraction that is more specialized towards UEFI's error-handling needs.


Add note about the allowJs tsconfig compiler option in content-collections.mdx

What kind of changes does this PR include?

New or updated content

Description
To be merged after https://github.com/withastro/astro/pull/6229
Add note about the allowJs tsconfig compiler option and how it is necessary to set it to true when using a javascript file as the Content Collection's configuration file.

Yeah I wasn't sure where to put this really, but I felt like putting this note anywhere else would be a good fit. I'll see how I can move it there, in the mean time, I'm happy to take your suggestions 🙂

Just a ping @MoustaphaDev that I think we were all happy with this! There now seems to be a conflict to resolve, but I think as soon as all the checks pass, this is yours to merge! 🥳

I completely missed your comment @sarah11918 , I'm going to get this merged soon

Thanks for your reviews @sarah11918 and @delucis!


Workspace - Extra option on issue cards page

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

Action Performed:

Login with<EMAIL_ADDRESS>Navigate to workspace editor
Click on issue cards page

Expected Result:
There should be 2 options “Issue and manage cards” and “Reconcile Cards”
Actual Result:
There are 3 options available: “Issue and manage cards” and “Reconcile Cards” and "Settlement frequency "
Workaround:
Unknown
Platforms:

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

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
https://expensify.testrail.io/index.php?/tests/view/3447292
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

QA team is Failed step in TR on Step 4

Coming back from OOO! Reviewing today and will action next steps

Posted in channel to clarify expected options, maybe this needs to be updated in testrail!

Hello @dylanexpensify! Please submit a "QA - Add/Update test to TestRail" ticket. Thank you


How to change language to English ?

I see this apps support only Indonesian Language, Can we change translation language to english ?
thanks

Not available on English for now @zukinaru


Revert "Actually add Compute Library tests to the Jenkins File (#8394)"

This reverts commit a00d21156ea749d65c9a59b3a79dac501723086b till we
can understand why this is failing in CI but not on developer machines.
@tqchen @leandron @masahi
Sorry about the breakage but here's a revert if that can be merged asap.

I'm not 100% sure that #8394 needs to be reverted, considering that there is evidence of the tests passing today: https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-8402/3/pipeline.
It might as well be a transient. Needs investigation, but maybe doesn't justify reverting that PR as a whole.

I did see another of my PRs fail yesterday with the same sort of failure -.https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-8400/1/pipeline
I've tried respinning that one.
Lets hope it is transient 🤞

I think this should be dropped as the CI appears to be holding.

Lets not merge this if CI holds - I'll leave it here for a day or so before Closing this out.

I've hit the failure today https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-8366/21/pipeline/258

I also hit the failure today https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-7518/79/pipeline/

we would require the jenkinsfile to be in main in order to take effect. please send a PR to first cleanup the arm compute_lib.sh to be nop, then we can merge and delete the step

tracking issue https://github.com/apache/tvm/issues/8417


Minor modifications

Exclude the Cloudtable class from the generated Swagger, since it can be a function input parameter (not from the http triggered parameters)
If the return type of the function is an array type, its generated schema is slightly different. Added support for that in GenerateResponseParameterSignature

Hi Mikael. I hope it's ok that I'm sending you an unsolicited PR. I had good use of your code today, and wanted to contribute my modifications.

@wobba +1 for this PR, I am using the functionality for arrays! Please consider merging. Thank you @rwatjen !!

Thank you :)


Fix permissions error (EPERM) on Windows

The dotfile was getting permissions errors on windows.
This fixes the issue by buffering in memory rather than writing to a dotfile.
Memory usage was similar in my tests. (Write to dotfile: 42k, Buffer in memory: 42-44k.)
Fixes #83.
Related: #103, #104, #156.

It would be good to know why this seems to fix the issue. Buffering up the contents sounds like it might be more related to a timing issue than the sequence of operations on disk.It looks like fs.watch() has problems on windows generating spurious EPERMs: https://github.com/joyent/node/issues/3250 https://github.com/joyent/node/issues/4337 Some of the issues seem to suggest the error can just be ignored or it might work to just unregister

What happens in windows when:
    wb.pipe(fs.createWriteStream(dotfile));

is changed to:
    wb.pipe(fs.createWriteStream(outfile));

with the fs.renae() part removed?

Writing straight to the outfile works great. PR here: #161
(The EPERM error was always thrown on the dotfile.)


Object<.., RArc, ...> doesn't implement Clone

I tried using RawMessageHook_TO<'static, abi_stable::std_types::RArc<()>> where RawMessageHook_TO is a sabi_trait. But when I try to clone it, I can't!
The reason appears to be because impl<T> GetPointerKind for RArc<T> specifies it as PK_SmartPointer which only implements clone if the value type implements clone, but that is counter-productive to RArc!
Is there some way I can access the RArc to clone it? (unsafe is fine)

Currently, I'm hacking around this with some ugly transmutes to a local type with the same representation as RObject, but I would like something more official than this hack.

Well, the smallest thing that could be done is to add special cased methods to RObject and DynTrait to clone them when they wrap an RArc.
With that, you could do RawMessageHook_TO::from_sabi(raw_message_hook.obj.cloned()) to clone the trait object.
(this uses the fact that #[sabi_trait] trait objects have a pub obj: RObject/DynTrait field)

That would work! I can put up a pr to do that


Kublet log gets spammed / cannot remove sandbox csi-node-driver.

Kublet log gets spammed with error messages every minute about not being able to remove a sandbox regarding csi-node-driver.
Expected Behavior

No messages in kubelet log regarding csi-node-driver not being removed.
Current Behavior

When I check the kubelet log, I see something like the following:
Apr 29 14:48:07 master0-k8s kubelet[248689]: E0429 14:48:07.945689  248689 kuberuntime_gc.go:184] "Failed to remove sandbox" err="rpc error: code = Unknown desc = failed to delete container k8s_POD_csi-node-driver-ffgtk_calico-system_34eca578-c907-419e-9b
fe-b76e298b8bf2_0 in pod sandbox 549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f from index: no such id: '549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f'" sandboxID="549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d
4f3f0a055f"
Apr 29 14:49:07 master0-k8s kubelet[248689]: E0429 14:49:07.950178  248689 remote_runtime.go:224] "RemovePodSandbox from runtime service failed" err="rpc error: code = Unknown desc = failed to delete container k8s_POD_csi-node-driver-ffgtk_calico-system_3
4eca578-c907-419e-9bfe-b76e298b8bf2_0 in pod sandbox 549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f from index: no such id: '549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f'" podSandboxID="549a709981c68792d4ddd6824af52ae
1e7b662328c24ef196e373d4f3f0a055f"
Apr 29 14:49:07 master0-k8s kubelet[248689]: E0429 14:49:07.950217  248689 kuberuntime_gc.go:184] "Failed to remove sandbox" err="rpc error: code = Unknown desc = failed to delete container k8s_POD_csi-node-driver-ffgtk_calico-system_34eca578-c907-419e-9b
fe-b76e298b8bf2_0 in pod sandbox 549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f from index: no such id: '549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f'" sandboxID="549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d
4f3f0a055f"
Apr 29 14:50:07 master0-k8s kubelet[248689]: E0429 14:50:07.962171  248689 remote_runtime.go:224] "RemovePodSandbox from runtime service failed" err="rpc error: code = Unknown desc = failed to delete container k8s_POD_csi-node-driver-ffgtk_calico-system_3
4eca578-c907-419e-9bfe-b76e298b8bf2_0 in pod sandbox 549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f from index: no such id: '549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f'" podSandboxID="549a709981c68792d4ddd6824af52ae
1e7b662328c24ef196e373d4f3f0a055f"
Apr 29 14:50:07 master0-k8s kubelet[248689]: E0429 14:50:07.962229  248689 kuberuntime_gc.go:184] "Failed to remove sandbox" err="rpc error: code = Unknown desc = failed to delete container k8s_POD_csi-node-driver-ffgtk_calico-system_34eca578-c907-419e-9b
fe-b76e298b8bf2_0 in pod sandbox 549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f from index: no such id: '549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f'" sandboxID="549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d
4f3f0a055f"
Apr 29 14:51:07 master0-k8s kubelet[248689]: E0429 14:51:07.965152  248689 remote_runtime.go:224] "RemovePodSandbox from runtime service failed" err="rpc error: code = Unknown desc = failed to delete container k8s_POD_csi-node-driver-ffgtk_calico-system_3
4eca578-c907-419e-9bfe-b76e298b8bf2_0 in pod sandbox 549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f from index: no such id: '549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f'" podSandboxID="549a709981c68792d4ddd6824af52ae
1e7b662328c24ef196e373d4f3f0a055f"
Apr 29 14:51:07 master0-k8s kubelet[248689]: E0429 14:51:07.965196  248689 kuberuntime_gc.go:184] "Failed to remove sandbox" err="rpc error: code = Unknown desc = failed to delete container k8s_POD_csi-node-driver-ffgtk_calico-system_34eca578-c907-419e-9b
fe-b76e298b8bf2_0 in pod sandbox 549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f from index: no such id: '549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f'" sandboxID="549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d
4f3f0a055f"
Apr 29 14:52:07 master0-k8s kubelet[248689]: E0429 14:52:07.968807  248689 remote_runtime.go:224] "RemovePodSandbox from runtime service failed" err="rpc error: code = Unknown desc = failed to delete container k8s_POD_csi-node-driver-ffgtk_calico-system_3
4eca578-c907-419e-9bfe-b76e298b8bf2_0 in pod sandbox 549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f from index: no such id: '549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f'" podSandboxID="549a709981c68792d4ddd6824af52ae
1e7b662328c24ef196e373d4f3f0a055f"
Apr 29 14:52:07 master0-k8s kubelet[248689]: E0429 14:52:07.968856  248689 kuberuntime_gc.go:184] "Failed to remove sandbox" err="rpc error: code = Unknown desc = failed to delete container k8s_POD_csi-node-driver-ffgtk_calico-system_34eca578-c907-419e-9b
fe-b76e298b8bf2_0 in pod sandbox 549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f from index: no such id: '549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f'" sandboxID="549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d
4f3f0a055f"
Apr 29 14:53:07 master0-k8s kubelet[248689]: E0429 14:53:07.973043  248689 remote_runtime.go:224] "RemovePodSandbox from runtime service failed" err="rpc error: code = Unknown desc = failed to delete container k8s_POD_csi-node-driver-ffgtk_calico-system_3
4eca578-c907-419e-9bfe-b76e298b8bf2_0 in pod sandbox 549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f from index: no such id: '549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f'" podSandboxID="549a709981c68792d4ddd6824af52ae
1e7b662328c24ef196e373d4f3f0a055f"
Apr 29 14:53:07 master0-k8s kubelet[248689]: E0429 14:53:07.973083  248689 kuberuntime_gc.go:184] "Failed to remove sandbox" err="rpc error: code = Unknown desc = failed to delete container k8s_POD_csi-node-driver-ffgtk_calico-system_34eca578-c907-419e-9b
fe-b76e298b8bf2_0 in pod sandbox 549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f from index: no such id: '549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d4f3f0a055f'" sandboxID="549a709981c68792d4ddd6824af52ae1e7b662328c24ef196e373d
4f3f0a055f"

Possible Solution

Steps to Reproduce (for bugs)

Context

So far this seems not to have any impact, aside the fact, that there is clutter in the log file. All pods seem to operate normally. Also, this only affects the control-plane nodes.
Your Environment

Calico version: 3.25.1
Orchestrator version (e.g. kubernetes, mesos, rkt): kubernetes
Operating System and version: Debian 11
Link to your project (optional):

I am not sure how to reproduce this, since it just popped up and the cluster works fine, anyway.

I'm not sure what might have caused that - looks like kubelet was trying to delete something that didn't exist? Maybe just a bookkeeping error in kubelet that resolved itself?

Yes, maybe. Most interesting part for me would be how to track it down to the root cause.

That I'm not sure on, and I suspect it's not a Calico issue (it just happens to be occurring for a Calico pod). Container sandbox deletion isn't something that Calico is involved in (with the exception of tearing down pod networking) but that doesn't seem to be where the error is here.


Remove on_shutdown/set_pre_stop

It causes occational exit spam, especially in tests, when an error occurs in the exit handler due to not being able to connect etc.
I think we don't need it anymore since _run_stub's finally-block appears to trigger disconnect properly on KeyboardInterrupt now. I haven't been able to repro the old race (CC @thundergolfer who fought with this bug back in the day)

Ah I think I had tried this and ran into the same issue — so lovely to be able to scroll right to the test results though!!!


[bug]: 3.5.1 SD1.5 Inpainting Is Very Difficult to

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Windows
GPU
cuda
VRAM
8GB
What version did you experience this issue on?
3.5.1
What happened?
Hello,
I am having a lot of difficulty replicating the ease of inpainting in 3.5.1 compared to previous versions. I had figured out a pretty flawless manual inpainting method using the canvas but it does not work anymore. I am curious if anything changed under the hood in 3.5.1 concerning inpainting with SD 1.5 models?
In previous versions I could easily change outfits etc. using CFG 7 and simple prompts without much weighting or image-specific negative prompts. I found denoising strength of 0.90 for 30 steps ideal in most cases. For coherence pass I find choosing Masked is better than Unmasked in all cases. I use 30 steps at 0.3 strength for great results. Mask blur varies between 4, 8, and 16 with 8 being a good universal value in my experience. I typically use epicPhotogasmZ or X inpainting models. I do use Controlnet when necessary for difficult images but generally found it is not needed to preserve original image composition even with denoising set to .90 or even 0.95.
However, in 3.5.1 I cannot get inpainting to do much at all! I included an example image grid below with details. Just tried to change a man in a black shirt into a shirtless man as it is simple. This would have been very easy prior to 3.5.1, I feel like I need to set CFG to 8-10 and denoise to 0.95 or 1 to even make changes like this in 3.5.1 release. Unfortunately, that usually ruins the inpainting process and requires more steps and Controlnet to compensate.
Thank you! I appreciate any help and I hope this post helps others who may be experiencing the same issue.
Screenshots

Additional context
FYI: I have tested many combinations of CFG, denoising strength, mask blur radius, coherence pass modes, and how tight/loose I draw my masked areas. It is much too tedious compared to earlier releases and each image requires lengthy troubleshooting to make desired changes. The example I posted above is meant to represent what settings worked very well in earlier releases.
Contact Details
No response

After many failed attempts to fix this I did fresh install and problem magically went away.  Thank you for your amazing efforts with invokeai!


[docs] fix typos in docs

Thanks for this great project! I'm starting to learn more about evidently and hoping to come around and contribute some more.
I wanted to find some small ways to help out, so ran aspell over the docs like this:
cat docs/**/**/*.md | aspell list | sort -u
cat docs/**/*.md | aspell list | sort -u
cat src/evidently/**/*.py | aspell list | sort -u
cat examples/**/*.ipynb | aspell list | sort -u

This PR proposes fixing some apparent typos found with that approach.
Thanks for your time and consideration.

Hey @jameslamb ,
Thank you!


HMAC SHA 256 for TOTP verification

Hello, I was able to get the following code working.
			TOTPGenerator totpGen = new TOTPGenerator.Builder(myKey) 			        .withHOTPGenerator(builder -> { 			            builder.withPasswordLength(6); 			            builder.withAlgorithm(HMACAlgorithm.SHA1); 			        }) 			        .withPeriod(Duration.ofSeconds(30)) 			        .build();
Since SHA1 is deprecated I wanted to switch to SHA256. I switched to use your SecretGenerator class for this like so...
		byte[] secretKeyBytes = SecretGenerator.generate(256); 		Base32 base32 = new Base32(); 		String secretKey = base32.encodeAsString(secretKeyBytes);
This secret key is added to my otp string like so. I have set the algorithm to SHA256. I have tried keeping the secret as is, and removing the padding.
otpauth://totp/testCreator:user?secret=KJFVIRCKGRLVERRUI4ZUEMSUJQ2TIR2CJNITGMSQKI2FQR2JKNATMVKPGZDEOVSJG5LEENRWLJKFAWSUI5AT2PJ5HU&issuer=testCreator&algorithm=SHA256
otpauth://totp/testCreator:user?secret=LJEU4RJXIJMUWSSHINLDON2ZKRME2TKLJZCEKSSVLI2UYVKCI5LTOU2SKNMVISBSKBFVCT2IKBBEYM2FGRIT2PJ5HU======&issuer=testCreator&algorithm=SHA256
When I switch my validation code to use SHA256 as written below my codes never validate. I am guessing this is user error but if you have any ideas or an example it would be most appreciated.
			TOTPGenerator totpGen = new TOTPGenerator.Builder(myKey) 			        .withHOTPGenerator(builder -> { 			            builder.withPasswordLength(6); 			            builder.withAlgorithm(HMACAlgorithm.SHA256); 			        }) 			        .withPeriod(Duration.ofSeconds(30)) 			        .build();
Thanks!

Both your opthauth uri's work fine when i try to validate the generated codes.
However I am not sure what you are doing with your secret. You shouldn't Base32 encode your secret.

Apologies for the late reply. After further testing I believe it was the authenticator app I was using not being compatible with HMAC SHA 256. Your library worked fine with the google authenticator app. The Microsoft Authenticator does not seem to work with HMAC SHA 256.


androidx.core version 1.7.0-alpha02 dependency causing Android build errors

The audioplayers' library dependency on androidx.core:core-ktx:+ is using androidx.core:core-ktx:1.70-alpha2 (requires compileSdk 31) even though our compileSdk is 29.
Full Description
Without any new local changes or library updates, we started encountering Android build errors when trying to run the emulator: error: resource android:attr/lStar not found. From investigation, we saw others encounter issues if they had third party libraries not specify a safe core-ktx version: https://stackoverflow.com/a/69031863.
We found the androidx.core:core-ktx dependency in the audioplayers library by running gradle :app:dependencies > dependencies.txt in the command line and searching core-ktx in the .txt file:
+--- project :audioplayers
|    +--- io.flutter:flutter_embedding_debug:1.0.0-05e680e202af9a92461070cb2d9982acad46c83c (*)
|    +--- androidx.core:core-ktx:+ -> 1.7.0-alpha02

Code to Reproduce
audioplayers: ^0.18.3 in pubspec.yaml.
Log Errors
error: resource android:attr/lStar not found

Files/URLs/Sources
Related links:

https://community.revenuecat.com/sdks-51/androidx-core-version-1-7-0-alpha02-dependency-causing-android-build-errors-397?postid=1025#post1025
https://issuetracker.google.com/issues/199180389
https://stackoverflow.com/a/69031863

Platforms
Pixel 2 XL API 30 (android-x86 emulator)
Pixel 3a API 30 (android-x86 emulator)
Workaround
Since our Android compileSdk is 29, our workaround was to add the following to the bottom of app/build.gradle:
configurations.all {
  resolutionStrategy {
    force 'androidx.core:core:1.6.0'
    force 'androidx.core:core-ktx:1.6.0'
  }
}

Wasn't this fixed already?
Could you please try with the current version 0.20.0?

I was getting the same error as OP, updating audioplayers to 0.20.0 in my pubspec.yaml worked for me. Thank you!

0.20.0 also fixed for me, but then I also lost Android audio like in #1001

Awesome! I will fix #1001 right away, that is a much simpler issue.


Move url to internal

fixes: #1743

As per team's conversation and analysis, We prefer having url in public surface area


Fix misprint in hw_oldkernel.js

Looks like return statement was omitted from setPinMode,
Before change I've seen errors like this:
Can not switch pin P9_12 into OUTPUT mode: TypeError: Cannot read property 'err' of undefined
at Object.f.pinMode (/home/ubuntu/node-nest/node_modules/bonescript/index.js:161:19)
at ...
at /home/ubuntu/node-nest/node_modules/unofficial-nest-api/index.js:231:17
at IncomingMessage. (/home/ubuntu/node-nest/node_modules/unofficial-nest-api/index.js:213:25)
at IncomingMessage.EventEmitter.emit (events.js:117:20)

Coverage decreased (-0.03%) to 23.92% when pulling 5705ad39482d1c544c0538b720653a45bf6a17fe on timothybasanov:patch-1 into 6a04cd269662c2caff2ec820d07fd3b39ac7c768 on jadonk:master.


[SSL]VIRTUAL_HOST + (+.crt/.key) does not work for me

my VIRTUAL_HOST is mysite.dev.
-> So my files are mysite.dev.crt and mysite.dev.key.

But the nginx response in the log is:
no "ssl_certificate" is defined in server listening on SSL port while SSL handshaking
What means he found something. \o/ ;-)
When I rename the files to default(.crt/.key) then it works.
What is here wrong?

Hi
I had same issue and altered nginx.tpl file by replacing:
listen 443  ssl;
with:
listen 443 default_server ssl;
in this part:
server {
error_log /app/file.log debug;
    server_name {{ $host }};
    listen 443 default_server ssl;

and it works now.
Did this by making new image, Dockerfile:
FROM jwilder/nginx-proxy
COPY ./nginx.tmpl /app/nginx.tmpl

This also worked for me. I've only got one ssl service running and a few non-ssl, will this cause an issue if there are multiple ssl services? Otherwise this is worth fixing.

Also worked for me.
Took a couple attempts to get it right so here's a pastebin for clarification: http://pastebin.com/0psiBGyN
I am running multiple ssl containers without issue.

I have some problems with your suggested fix. Is anyone of you running more than one docker image with ssl?
If i start one everything is working, but if i start a second container that uses ssl i get the following error in nginx:
[emerg] 35#0: a duplicate default server for <IP_ADDRESS>:443 in /etc/nginx/conf.d/default.conf:243
and it does not connect the second container to nginx. Any ideas?

How are you testing this fix? I tried to update the template after pulling and running the trusted image and ran into the same issue. When I built from the repo (git clone https://github.com/jwilder/nginx-proxy.git) it worked with no changes (since this fix was already in the repo).

@phal0r I figured that might be an issue. I've only got the one ssl container.

@log0ymxm The behaviour is really strange. Now i am using the nginx-proxy without the fix suggested here and the following does work:
when you provide an crt and key for default AND the virtual host ssl is working. if you only have one of both ssl is not working. However i don't know which cert nginx is using, but it does not use the else branch in the template since the "Strict-Transport-Security" header is set. Maybe this is helpful for someone to investigate further.

As I mentioned, it works fine for me with multiple (9 non-ssl and 3 SSL) sites. I cloned the repo and performed a docker build before running the container. I suggest stop & start each time an SSL container is added as the dynamic doesn't seem to work correctly though I haven't had time to inspect the config files and determine why.

@Nitesedge Ah, I overread your response, sorry. I tested the trusted containter from the docker hub, which has this issue and then cloned the repo and applied this fix. This fix here is not included as far as i can see it in the nginx template, but i will try to build the Dockerfile from the repo as you suggested. Thanks for the tip.

Unfortunately, building the image from the repo didn't change anything for me.

I get the no "ssl_certificate" is defined in server listening on SSL port while SSL handshaking error as soon as I have mixed SSL and non-SSL VirtualHosts.
When I see that error in the console, only the non-SSL sites work. I get ERR_CONNECTION_RESET in the browser when trying to connect to any SSL sites.
One workaround seems to be to add a default.crt and default.key (even if it's just a copy of any of the other certificates you're using).
It seems that in nginx it is not allowed to have a listen 443 ssl; without ssl_certificate. So changing
server {
	server_name {{ $host }};
	listen 443 ssl;
	return 503;

	{{ if (and (exists "/etc/nginx/certs/default.crt") (exists "/etc/nginx/certs/default.key")) }}
	ssl_certificate /etc/nginx/certs/default.crt;
	ssl_certificate_key /etc/nginx/certs/default.key;
	{{ end }}
}

to
{{ if (and (exists "/etc/nginx/certs/default.crt") (exists "/etc/nginx/certs/default.key")) }}
server {
	server_name {{ $host }};
	listen 443 ssl;
	return 503;

	ssl_certificate /etc/nginx/certs/default.crt;
	ssl_certificate_key /etc/nginx/certs/default.key;
}
{{ end }}

did the trick for me. Of course you'll potentially get the contents of a different site when connecting with https to a site which does not have a certificate.

My use case was fixed in #91. thx for the work.


Cannot authenticate

Playnite's version
10.20
Plugin's version
1.1.3
Legendary Launcher version
?
Bug description
When I try to authenticate, it is giving me an error "Failed to authenticate user". It has imported the installed game, but due to the authentication error it cannot import the other games. OS - Windows 10. I installed legendary by downloading the legendary.exe from github. I was trying to authenticate by entering my epic account email and password.
Relevant log
The error message is "Failed to authenticate user".
Screenshots

Following is the location where legendary.exe is stored:

Legendary binary needs to be at specific places mentioned at https://github.com/hawkeye116477/playnite-legendary-plugin/wiki/Troubleshooting#legendary-launcher-is-not-installed. Otherwise correct path to legendary binary needs to be added to Advanced plugin settings or PATH environment variable. Have you done it? If yes, then show me log files content from %APPDATA%\Playnite.

Hey, sorry for the late reply, and thank you! Adding the custom path in the advanced setting worked!

You're welcome. As I see, it's another similar problem reported, so that will be "fixed" in the next version by adding more checking if user has Legendary Launcher installed, so itll be clear what is that problem.


Add support for input driving table

This builds on #279 and depends on https://github.com/neo4j/neo4j/pull/10981 and can not be merged until the dependency is merged and released.

Not sure what's up with GitHub; the commits are listed here in the wrong order compared to my local branch.