Fix error in Rails/Presence when ternary operators are used in multiple lines

When I run rubocop-rails against the following code, I get an error from Rails/Presence cop.
a.present? ?
  a :
  b

After investigation, I have identified the problem and would like to fix it.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.
[ ] If this is a new cop, consider making a corresponding update to the Rails Style Guide.

The actual error I encountered was as follows:
$ bundle exec rubocop --debug example.rb
For /app: configuration from /app/.rubocop.yml
configuration from /bundle/ruby/2.7.0/gems/rubocop-rails-2.17.4/config/default.yml
configuration from /bundle/ruby/2.7.0/gems/rubocop-rails-2.17.4/config/default.yml
Default configuration from /bundle/ruby/2.7.0/gems/rubocop-1.37.1/config/default.yml
Inheriting configuration from /app/.rubocop_todo.yml
Use parallel by default.
Skipping parallel inspection: only a single file needs inspection
Inspecting 1 file
Scanning /app/example.rb
An error occurred while Rails/Presence cop was inspecting /app/example.rb:3:0.
undefined method `keyword' for #<Parser::Source::Map::Ternary:0x000055aaee81c4c0>
/bundle/ruby/2.7.0/gems/rubocop-rails-2.17.4/lib/rubocop/cop/rails/presence.rb:116:in `current'
/bundle/ruby/2.7.0/gems/rubocop-rails-2.17.4/lib/rubocop/cop/rails/presence.rb:110:in `message'
/bundle/ruby/2.7.0/gems/rubocop-rails-2.17.4/lib/rubocop/cop/rails/presence.rb:95:in `register_offense'
/bundle/ruby/2.7.0/gems/rubocop-rails-2.17.4/lib/rubocop/cop/rails/presence.rb:82:in `block in on_if'
/bundle/ruby/2.7.0/gems/rubocop-rails-2.17.4/lib/rubocop/cop/rails/presence.rb:64:in `redundant_receiver_and_other'
/bundle/ruby/2.7.0/gems/rubocop-rails-2.17.4/lib/rubocop/cop/rails/presence.rb:79:in `on_if'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/cop/commissioner.rb:100:in `public_send'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/cop/commissioner.rb:100:in `block (2 levels) in trigger_responding_cops'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/cop/commissioner.rb:160:in `with_cop_error_handling'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/cop/commissioner.rb:99:in `block in trigger_responding_cops'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/cop/commissioner.rb:98:in `each'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/cop/commissioner.rb:98:in `trigger_responding_cops'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/cop/commissioner.rb:69:in `on_if'
/bundle/ruby/2.7.0/gems/rubocop-ast-1.23.0/lib/rubocop/ast/traversal.rb:20:in `walk'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/cop/commissioner.rb:86:in `investigate'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/cop/team.rb:155:in `investigate_partial'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/cop/team.rb:83:in `investigate'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/runner.rb:315:in `inspect_file'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/runner.rb:259:in `block in do_inspection_loop'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/runner.rb:293:in `block in iterate_until_no_changes'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/runner.rb:286:in `loop'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/runner.rb:286:in `iterate_until_no_changes'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/runner.rb:255:in `do_inspection_loop'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/runner.rb:138:in `block in file_offenses'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/runner.rb:163:in `file_offense_cache'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/runner.rb:137:in `file_offenses'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/runner.rb:128:in `process_file'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/runner.rb:109:in `block in each_inspected_file'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/runner.rb:108:in `each'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/runner.rb:108:in `reduce'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/runner.rb:108:in `each_inspected_file'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/runner.rb:94:in `inspect_files'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/runner.rb:47:in `run'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/cli/command/execute_runner.rb:26:in `block in execute_runner'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/cli/command/execute_runner.rb:52:in `with_redirect'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/cli/command/execute_runner.rb:25:in `execute_runner'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/cli/command/execute_runner.rb:17:in `run'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/cli/command.rb:11:in `run'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/cli/environment.rb:18:in `run'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/cli.rb:72:in `run_command'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/cli.rb:79:in `execute_runners'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/lib/rubocop/cli.rb:48:in `run'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/exe/rubocop:19:in `block in <top (required)>'
/usr/local/lib/ruby/2.7.0/benchmark.rb:308:in `realtime'
/bundle/ruby/2.7.0/gems/rubocop-1.37.1/exe/rubocop:19:in `<top (required)>'
/bundle/ruby/2.7.0/bin/rubocop:23:in `load'
/bundle/ruby/2.7.0/bin/rubocop:23:in `<top (required)>'
/bundle/gems/bundler-2.2.32/lib/bundler/cli/exec.rb:58:in `load'
/bundle/gems/bundler-2.2.32/lib/bundler/cli/exec.rb:58:in `kernel_load'
/bundle/gems/bundler-2.2.32/lib/bundler/cli/exec.rb:23:in `run'
/bundle/gems/bundler-2.2.32/lib/bundler/cli.rb:478:in `exec'
/bundle/gems/bundler-2.2.32/lib/bundler/vendor/thor/lib/thor/command.rb:27:in `run'
/bundle/gems/bundler-2.2.32/lib/bundler/vendor/thor/lib/thor/invocation.rb:127:in `invoke_command'
/bundle/gems/bundler-2.2.32/lib/bundler/vendor/thor/lib/thor.rb:392:in `dispatch'
/bundle/gems/bundler-2.2.32/lib/bundler/cli.rb:31:in `dispatch'
/bundle/gems/bundler-2.2.32/lib/bundler/vendor/thor/lib/thor/base.rb:485:in `start'
/bundle/gems/bundler-2.2.32/lib/bundler/cli.rb:25:in `start'
/bundle/gems/bundler-2.2.32/exe/bundle:49:in `block in <top (required)>'
/bundle/gems/bundler-2.2.32/lib/bundler/friendly_errors.rb:103:in `with_friendly_errors'
/bundle/gems/bundler-2.2.32/exe/bundle:37:in `<top (required)>'
/bundle/bin/bundle:113:in `load'
/bundle/bin/bundle:113:in `<main>'
.

1 file inspected, no offenses detected

1 error occurred:
An error occurred while Rails/Presence cop was inspecting /app/example.rb:3:0.
Errors are usually caused by RuboCop bugs.
Please, report your problems to RuboCop's issue tracker.
https://github.com/rubocop/rubocop/issues

Mention the following information in the issue report:
1.37.1 (using Parser <IP_ADDRESS>, rubocop-ast 1.23.0, running on ruby 2.7.5) [x86_64-linux]
Finished in 0.37355209999805083 seconds

This looks good. Can you squash commits into one?

OK, I'll squash them 👌

Thanks!


Allow __ to stand for the argument which accepts a do block

WIP - still needs tests + docs
It's also unclear whether this is a desired behavior.
I feel like — though this could be rather useful — it's strangely inconsistent with the meaning of __ as used in other contexts.  Further discussion at https://discourse.julialang.org/t/ann-underscores-jl-placeholder-syntax-for-closures/36482

On consideration I think this is semantically inconsistent, so I don't think we should do this with __, as tempting as it is.


Does not work with node_modules outside of project folder

I am using the latest version. If I install all modules inside my project folder, it works fine. But if I move the node_modules folder up in the folder tree (The project is in my DropBox, the node_modules outside of it), it stops working – the project globs' css and js values become empty. Any ideas?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/40206138-does-not-work-with-node_modules-outside-of-project-folder?utm_campaign=plugin&utm_content=tracker%2F8853550&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F8853550&utm_medium=issues&utm_source=github).

Hi @austinpray , any thoughts on this yet?


kconf: refactor

Description of changes

Changed directory to respect new packages convention + added completions for common shells
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

formatting error

I don't understand i tried every formatter i know of, what's causing the test to fail ?

turns out I had to format it using nixfmt-rfc-style :/


EE install shell variable DOCKERURL in setup-yum-repo does not pass through sudo command

Problem description
export DOCKERURL='<DOCKER-EE-URL>'
sudo sh -c 'echo "$DOCKERURL/rhel" > /etc/yum/vars/dockerurl'

Environment variables do not pass through sudo by default, so /etc/yum/vars/dockerurl ends up containing /rhel in this case.
Problem location
https://docs.docker.com/engine/installation/linux/docker-ee/rhel/#install-using-the-repository
Project version(s) affected
EE 17.03 and later
Suggestions for a fix
Add -E to pass the env var through sudo:
export DOCKERURL='<DOCKER-EE-URL>'
sudo -E sh -c 'echo "$DOCKERURL/rhel" > /etc/yum/vars/dockerurl'

ping @mstanleyjones ^^

On it 🙀

@mstanleyjones I see @trapier opened a PR already https://github.com/docker/docker.github.io/pull/4658


Minor bug fix in the keys() function on a project.

Invalid variable name on the project->keys() function.

Haha woops! Nice catch! Thanks!


createPositiveAction - for promise Actions that doesn't need to wait until executes, but can be undone when something wrong

Look of explanation:

We have unread message counter and we need to update it on server
When we hover on message, we send request to server, but we doesn't need to wait until request done for update state.
So we're use the same logic like createPromiseAction, but send event about success just when we call the action.

It's can be similar to
import {createPositiveAction} from 'krux'

export let readMessage = createPositiveAction(Comment.read, 'COMMENT_READ')

event with structure:
{
  "type": "COMMENT_READ",
  ...arguments[0]
}

will be fired in the same time when we call method. And method with structure:
{
  "type": "COMMENT_READ_FAIL",
  ...arguments[0],
  ...response
}

when promise is rejected

Finished with tests.


[PackMan] Introduction of IPackageManager interface

This is a copy of the ILISA interface, for now changes are only introduced to be able to generate the JSON RPC code from it, later on changes might be introduced to support the Firebolt packager

Hi Marcel: BD is complaining about the RDKM header.  If this is really an RDKM file, then please add these lines to NOTICE in the top level of the component and I will clear it.  Otherwise, please use Metrological.  Thanks, Martin
Copyright 2023 RDK Management
Licensed under the Apache License, Version 2.0

Hi Marcel: BD is complaining about the RDKM header. If this is really an RDKM file, then please add these lines to NOTICE in the top level of the component and I will clear it. Otherwise, please use Metrological. Thanks, Martin
Copyright 2023 RDK Management Licensed under the Apache License, Version 2.0

Hi Martin,
I discussed this internally before adding and as this is introduced not for Metrological itself the RDKM one seemed more appropriate. I'll add the text you refer to above.

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/ThunderInterfaces/238/rdkcentral/ThunderInterfaces

Commit: 25b2731a9848f1bf03677a8f3190be3e152dbd14

Report detail: gist

Hi Marcel: Crossed wires :-)
I want the copyright notice added to the file called NOTICE in the top level directory of the component, not in the file itself.
Thanks!

Ah sorry misunderstood :) Let me add it there

Fixed! (at least I hope it is correct now :) )

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/ThunderInterfaces/238/rdkcentral/ThunderInterfaces

Commit: 9a3c1cf47637b2f9a2212a81679a9429f0af3216

Report detail: gist

WARNING: A Blackduck scan failure has been waived
A prior failure has been upvoted

Upvote reason: Thank you!

Commit: 9a3c1cf47637b2f9a2212a81679a9429f0af3216


About the maximum length

Hi Young-Jin,
What did you set the model's maximum length to? When I set it to 1024, it caused a CUDA out-of-memory issue.
Best regards,
Young-Jin

The default max new tokens is 256.


db.collectionNames() includes "system.indexes" into the collections list

After there statements:
val client = new Fongo(testName)
val mongoDb = client.getDB(testName + "_db")
val collection = mongoDb.createCollection(testCollectionName, new BasicDBObject())
db.getCollectionNames() returns a Set of (testCollectionName, "system.indexes")
whereas Casbah driver working against the real Mongo instance returns just Set of (testCollectionName)
Fongo 2.0.6
MongoDB: 3.2.8
Scala Casbah driver: 3.1.1
Java driver: 3.2.2

Hi,
same things here

  @Test
  public void should_createCollection_create_the_collection() {
    DB db = fongoRule.getDB("db");
    db.createCollection("coll", new BasicDBObject());
    **assertEquals(Sets.newHashSet("coll", "system.indexes"), db.getCollectionNames());**
  }

I will check if MongoDB change this.

I'm using Casbah driver for Scala
Fongo: 2.0.6
Casbah version: 3.1.1
Java driver: 3.2.2
Real Mongo version: 3.2.8 - I'm comparing against

Hi,
can you test against the master branch ?

It works fine now with 2.0.10-SNAPSHOT version


Horizontal Server List breaks with Server Folders

Describe the bug
Using Server Folder breaks the Server folders
To Reproduce
Use Horizontal Server List & Server Folders
Screenshots

Additional Info
Discord channel: Stable
OS: OSX
Mod: BetterDiscord
Discord language: English, US

Issue Fix: 2.5 should work fine.


Hi

Hi, your project is so nice, I wanna make it but in my country motor drive hat doesn't available. Can you write code for l298n motor drive instead of motor drive hat.thanks

Hello Mustafa, thank you for the interest in the project. Unfortunately I cannot help you with your request. I have not used an L298N motor driver before, the Motor Driver HAT is the first driver I've worked with.
But have you tried checking out videos by Articulated Robotics? Josh uses the L298N for his differential drive robot. Here are some of his links that might help put:
https://articulatedrobotics.xyz/mobile-robot-1-project-overview/
https://www.youtube.com/watch?v=-PCuDnpgiew
https://www.youtube.com/watch?v=OWeLUSzxMsw
I hope these help you with what you are working on.


Missing pdf data source error

I copied the first example of the library:
<script lang="ts">
  import { PdfViewer } from "svelte-pdf-simple";
  const pathToPdf = "./example.pdf";
</script>

<PdfViewer url={pathToPdf} />

I switched up the pathToPdf to my path and I even tried moving it everywhere:
first I tried: const pathToPdf = '$lib/assets/menu.pdf';
then I tried: const pathToPdf = 'src/lib/assets/menu.pdf';
then I moved it inside the same directory of the route const pathToPdf = './menu.pdf';
I even tried to import the file directly: import pdf from '$lib/assets/menu.pdf';
But I always got a blank page rendered with this in the console:

And I don't know if it's useful but vs code throws the following error:

It's probably 100% my fault, but honestly, I can't wrap my mind around this error. Thanks

Ran into the same issue today...

I solved by doing the following:

I solved by importing the library and binding the viewer in the onMount function:
<script>
	import { onMount } from 'svelte';
	let PdfViewer;
	import pdf from '$lib/assets/file.pdf';

	onMount(async () => {
		const module = await import('svelte-pdf');
		PdfViewer = module.default;
	});
</script>
	<svelte:component
		this={PdfViewer}
		url={pdf}
	/>


Adjusted IModifyTags to try and prevent Modded and Oxide server tags being removed by plugins.

Thanks for the PR, but I've already got this and a few other changes to these locally. :)

Ahh excellent news. Thanks for letting me know. Keen to see the implementation.

For plugins, they just don't have hooks to use anymore. IModifyTags is internal and hasn't been callable for a few months now, and BuildServerTags is removed. It's not actually an issue with plugins though, custom builds are the ones you see with the tags removed most of the time.

I was hoping to use IModifyTags to catch the missing tags on the way back, but removing the functionality all together from the plugins is an excellent next step. Appreciate the feedback and notes. Cheers.

Pretty sure that adding them back wouldn't help, as the plugin called right after it would just remove them again. Either way, not really possible via plugins anymore, but custom builds are still a thing.


Joystick Hat Support

This code implements joystick hat support per issue #889.
The API change is to add the following function (and #defines for each of the hat positions such as GLFW_HAT_RIGHT_UP):
GLFWAPI const unsigned char* glfwGetJoystickHats(int jid, int* count);

The events test (events.c) has been updated to reflect the number of hat switches when a joystick is connected.
The joystick test (joystick.c) has been updated to add a new row for hat switches that displays the position in human-readable text (e.g., "right_up").
Code for macOS has been provided and tested.
Code for Windows has been provided, but has not been compiled or tested since I do not have access to a Windows system.
Code for Linux has also been provided, but relevant portions are compiled out with the #defines JSIOCGHATS and JS_EVENT_HAT -- following the JSIOCGAXES / JSIOCGBUTTONS and JS_EVENT_AXIS / JS_EVENT_BUTTON syntax, respectively -- since <linux/joystick.h> does not currently provide support for hat switches; if such support is added to <linux/joystick.h>, the code should work however.  This code also has not been compiled or tested since I do not have access to a Linux system.
A remark about the lack of support for hat switches on Linux appears in the documentation for glfwGetJoystickHats in glfw3.h.

This should definitely be supported and I like the interface. The implementation will be somewhat different both due to the work being done in the gamecontrollerdb branch for #900 and because pulling hats out of the button array will break backward compatibility.
Instead I propose being able to query hat mappings, where each hat is four button indices. Every platform puts the hat buttons at the end of the array already, which should help people transition. Once it's time for GLFW 4, of course, this can be cleaned up.
Waiting for the Linux joystick API to change is probably futile, but have a look at evdev.

It's annoying to work with a closed issue. I've opened a replacement.

Linux evdev joystick input is pending in #1005.

Linux evdev joystick input has been merged. Hats are now properly supported on Linux (X11, Wayland and Mir).


config: give full path to autoconf.err if 'checking for psol' fails.

@jmarantz could you review this?

lgtm


CONTRIBUTING.md improvements

Add a link to the CONTRIBUTING.md file to the README.md file
Add a section on how to run the project to the CONTRIBUTING.md file

Items came up in the discussion on #2 .

On It!

There we go #5
Had a hard time understanding why my fork wasn't updated. But hey! you learn something new everyday

I would like to work on this issue

Hey @Abhishekchauhan089 , thank you for the support, but right now @tomassirio is already working on it.

Did I miss something? I create the pull request yesterday?

@tomassirio nothing, really, I haven't had time until now to manage the repo.


bug: Connection closed without receiving any events

IDE Information
PyCharm 2024.1.4 (Professional Edition)
Build #PY-241.18034.82, built on June 24, 2024
Licensed to david timerman
Subscription is active until July 4, 2025.
For educational use only.
Runtime version: 17.0.11+1-b1207.24 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Windows 11.0
GC: G1 Young Generation, G1 Old Generation
Memory: 4054M
Cores: 28
Registry:
ide.experimental.ui=true
Non-Bundled Plugins:
nl.rubensten.texifyidea (0.9.7)
com.intellij.ml.llm (241.18034.12)
com.sourcegraph.jetbrains (6.0.16)
com.firsttimeinforever.intellij.pdf.viewer.intellij-pdf-viewer (0.15.0)
Bug Description
Whenever I ask Cody a follow up question, I receive this error.
Error performing this action  Please retry sending your message. If you tried to run a command, try it again. If the issue persists, please create a support ticket.  Error: Connection closed without receiving any events
Additional context
Stacktrace: Connection closed without receiving any events

I have the same problem. It's pretty much unusable at the moment.
PyCharm 2024.1.5 (Professional Edition)
Build #PY-241.18968.29, built on August 5, 2024
Licensed to mulle-objc / Mulle Nat
For non-commercial open source development only.
Runtime version: 17.0.11+1-b1207.30 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Linux 6.5.0-45-generic
GC: G1 Young Generation, G1 Old Generation
Memory: 2048M
Cores: 64
Non-Bundled Plugins:
  com.smallcloud.codify (1.3.4)
  com.sourcegraph.jetbrains (6.0.16)
Current Desktop: ubuntu:GNOME

Same problem using vscode...

Same thing


[sample_encode] Add BitrateLimit to cmd line options

-Added -BitrateLimit:<on,off> option to sample_encode command line options.
BitrateLimit equals MFX_CODINGOPTION_UNKNOWN(0) by default,
for encoder this means enabling (same behavior as MFX_CODINGOPTION_ON).
-Added warning in case when BitrateLimit is ON and bitrate changed by Query.
-Updated sample_encode and mediasdk docs.

Please set the default behavior to OFF, limiting artificially the bitrate will cause wrong behavior for developers taking into account that not everybody (and most users are included here) are not trying to reach compliance. It should be given a warning about it if level and profile don't match the requested bitrate but should not be a limitation for the user.

@dmitryermilov, @stasefremov
What do you think about changing of the default value for BitrateLimit to off.

What do you think about changing of the default value for BitrateLimit to off.

I'm fine about it.

Small chance that any customer intentionally rely on current default behavior (BitrateLimit == on). For other usages we can properly communicate changed MSDK behavior (i.e. argue that customers app behavior is buggy and requires changes). So I can agree with making BitrateLimit = off by default in MSDK. It should SOLVE more problems than CAUSE (i.e. it's not a purely good change, but effect can be measured as positive).

GlaUgrub == stasefremov :)

Just to be on the same page: @stasefremov confirmed he would like to see  BitrateLimit==OFF inside MSDK. @dvrogozh , @spossos , what did you mean, samples level or MSDK internal behavior?

I actually meant samples level at the first place. That's where things are easier: I think we are more or less free to change defaults here especially when we first introduce the option.
I also would prefer to have this changed in mediasdk by default. Yes, some people might step into the changed behavior. However, if default does not make sense or is a bad default, I think we should be more flexible and be able to change it. The fact is that a lot of people (especially new to the mediasdk) tend to use default settings trusting they are good and when they are finding they are not good... well, this is not good:)...
So, I suggest 2 steps:

Let's have bitratelimit=OFF in samples, both in SE and SMT - that's this PR
Let's create another PR and suggest a change to the mediasdk default and discuss more there.

@GlaUgrub, @dmitryermilov : are you ok with such plan?

@GlaUgrub, @dmitryermilov : are you ok with such plan?

Yes

@pgribov , please confirm you can do for SMT and then we'll merge this PR

@dvrogozh , @dmitryermilov
in this PR smt changes have already been added


User/ofirsharon/azure sdk kusto 04 2024

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.Kusto


One more function that we know can be safely removed

Super low hanging fruit found while looking at HACL* code quality.


Telegram Group?

Maybe we should make a Telegram Group
so we can discuss and get answers faster?

for real? can i make the group?

I think the best way is to have everything here on Github.
It will also help newcomers with their questions, as well as encourage new people to take tasks and open pull requests for this repo.
I don't have a lot of time to maintain the repo, so I am trying to go through the tasks, but it's hard to catch up with all of them.
If you see any specific reason on why a chatting group would help, let me know and we can consider this.

Hello,
I appreciate all your help
But as you said you cant keep up,but we as a community we can keep up.
I see many apis have their own Telegram Group.
For example if a newcomer doesnt know how to iterate the pages to get following list etc that could be a very fast response in Telegram than Here ..
plus we can discuss about other stuff too, create a bond between us
Its easier to reply on Telegram trust me.. i have a 3.500 Group and 4 different Teelgram accounts for bussiness use.. its easy to be active there than here

i still believe it would be much easier


Unable to get current active subs

I have version 2.4.6 installed on windows 11
every time i try and search for active renewal dates, active subscriptions, paid only
it gets to the end of the search Boolean and the script just exits
and doesn't display anything
if i try and leave the search defaults all i get is expired accounts and not ones i've subbed to

version 2.4.6 is a really old version of the scraper. Its currently on 3.3.11. I would first try upgrading to a newer version and see if that helps.

please update to a newer version
make a new issue if this persist


Double  in quote breaks it

I'm trying to create a quote from HTML code like this:
    const {editorState} = this.state;
    const contentState = editorState.getCurrentContent();
    data.text = 'This is the first line
This is the second line and it\'s ok

This should be the third line';
    const blocksFromHTML = convertFromHTML(data.text);
    const state = ContentState.createFromBlockArray(
        blocksFromHTML.contentBlocks,
        blocksFromHTML.entityMap,
    );

    let newState = EditorState.push(editorState, state, 'added-quote')
    newState = RichUtils.toggleBlockType(
        newState,
        'blockquote'
    );

    this.setState({
        editorState: newState,
    });

And I'm getting this result:

What is going on:
Blockquote element correctly process one '
' tag, but when it finds double '
' tag it breaks blockquote and paste least part of text in normal block.
Are there any solutions to resolve this problem, because quote can contain differet number of newline?
p.s. '\n' symbols instaed of '
' are ignored.

The reason here is that
 and

 are interpreted as different things.
 is a soft line break - it acts as a line break within a single block.

 acts as a break between blocks. This is a pretty reasonable heuristic, in my opinion, given the visual difference between
 and

.
Finally, \n is ignored because it is interpreted as a single whitespace character in HTML.

So is there any way to insert double newline?

It depends on what your goal is. The difference between
 and

 is handled as part of the conversion from HTML. There are no hooks to change this. However, depending on your needs have various avenues available:

If you want everything in the same block always then you can generate combine the various blocks after converting from HTML while being sure to separate them with a \n.

If you just want

 to be interpreted as two newlines you could override the second parameter of convertFromHTML with a function that converts the string to a DOM and, in the process, separate all consecutive
 tags with some sentinel text. Once you get the blocks back from the conversion you remove the sentinel text.

There's probably other options available as well.

Thank you for reply, solved this by replacing

 with

!


amplify pull  throws FeatureFlags is not initialized error

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
No response
If applicable, what version of Node.js are you using?
No response
Amplify CLI Version
10.4.1
What operating system are you using?
Windows
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
no
Amplify Categories
Not applicable
Amplify Commands
pull
Describe the bug
When running Amplify pull headless and amplify pull on a existing studio app in powerhell the following error is thrown
🛑 FeatureFlags is not initialized

When using a headless pull, noticed the application does not prompt for a allow modify backend, throws error. Then pull fails entirely.
On existing application create by amplify CLI and then studio enabled the pull will throw the error after the UI components have been synched.
Edit your schema at C:\amplify\studioApps\taskapp\amplify\backend\api\taskapp\schema.graphql or place .graphql files in a directory at C:\amplify\studioApps\taskapp\amplify\backend\api\taskapp\schema
√ Successfully pulled backend environment dev from the cloud.
Browserslist: caniuse-lite is outdated. Please run:
  npx browserslist@latest --update-db
  Why you should do it regularly: https://github.com/browserslist/browserslist#browsers-data-updating
✅
√ Synced UI components.
⚠ The components ItemCardCollection were synced with an older version of Amplify Studio. Please re-sync your components with Figma to get latest features and changes.
✅ GraphQL schema compiled successfully.

Edit your schema at C:\amplify\studioApps\taskapp\amplify\backend\api\taskapp\schema.graphql or place .graphql files in a directory at C:\amplify\studioApps\taskapp\amplify\backend\api\taskapp\schema
🛑 FeatureFlags is not initialized

** Note: when testing this in WSL the application works as expected.
Expected behavior
pull works in powershell
Reproduction steps
Amplify headless

create studio app
open powershell
amplify pull

Amplify pull

open powershell
amplify init
enable studio
create a data model
create forms in UI builder
amplify pull

GraphQL schema(s)

# Put schemas below this line

Project Identifier
6c986368d58e7b7a74f2a89d48b57947
Log output

# Put your logs below this line

Additional information
No response

I still see this bug on version 11.0.2 when pushing the backend. I've seen this bug occasionally last a few months. My workaround was to upgrade cli version.


Infra actions 1

Enables CI testing in a conda environment. Resolves #43

@ryanmwhitephd is there a reason the CI is pulling fwfr from ke-noel instead of artemis-analytics?


[pulsar-client] print unknown exception stacktrace

Motivation
right now, pulsar-client producer throws unknown exception while publishing messages and it doesn't log stacktrace and it makes very difficult to debug issue with just client log.

rerun integration tests

rerun java8 tests

run java8 tests

run java8 tests


fix(): IAM status messages switched fail and pass text and some grammar

Context
The following IAM status messages are swapped for FAIL and PASS text:

https://github.com/prowler-cloud/prowler/blob/master/prowler/providers/aws/services/iam/iam_password_policy_lowercase/iam_password_policy_lowercase.py
https://github.com/prowler-cloud/prowler/blob/master/prowler/providers/aws/services/iam/iam_password_policy_minimum_length_14/iam_password_policy_minimum_length_14.py
https://github.com/prowler-cloud/prowler/blob/master/prowler/providers/aws/services/iam/iam_password_policy_number/iam_password_policy_number.py
https://github.com/prowler-cloud/prowler/blob/master/prowler/providers/aws/services/iam/iam_password_policy_symbol/iam_password_policy_symbol.py

https://github.com/prowler-cloud/prowler/tree/master/prowler/providers/aws/services/iam
Description
I fixed the wrong status messages, which was the important part, and fixed some status message grammar in other IAM checks.
License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hi @acknosyn we'll check this out and let you know !!

Good job @acknosyn, thank you for contributing!

All good, thanks @n4ch04 @sergargar for fixing the linting errors


make it possible to serialize cancellers

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Git config file not written when creating new branch

I found that everytime I was to finish a feature, the message always show that is merged into null, no matter which base branch I used. After looking into it more closely I found that when I create the feature nothing is added inside ./git/config file. Adding the proper lines makes it work like expected.
OS: macOs High Sierra 10.13.2
If there is any way to debug this issue, I will be happy to help.

Gitflow4idea is a gui for the gitflow command line tool.
Do you get the same result when you use  git flow finish feature from the CMD?
If so - it's not an issue with the plugin, but rather something to do with gitflow on your machine
If not - please post the output of your intellij VCS console


2 servers on one network

If two machines are set as server then clients will find one and then send data to that server. Would be useful in the config to define a specific IP /name that the client should attach to.

Not sure why you’d want to run two servers.
It’s designed to be run with one server and multiple clients. That’s configured the first time you run it and stored in a config file.
Can you elaborate why you’d want two servers. Thinking I’ll close the issue otherwise.

Currently I’ve run 2 so I can see in more detail one of the trends. On the main system I have internal and external trends and when the external temperature gets low then the resolution of the internal temperatures is not great. If you could turn pens on and off as per my other request then this requirement would disappear.
--
Regards,
Alistair Fielden
From: Toby<EMAIL_ADDRESS>Sent: 25 October 2019 15:34
To: tobyweston/temperature-machine<EMAIL_ADDRESS>Cc: AlistairFielden<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [tobyweston/temperature-machine] 2 servers on one network (#97)
Not sure why you’d want to run two servers.
It’s designed to be run with one server and multiple clients. That’s configured the first time you run it and stored in a config file.
Can you elaborate why you’d want two servers. Thinking I’ll close the issue otherwise.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/tobyweston/temperature-machine/issues/97?email_source=notifications&email_token=AMDBJDYZIFJGKPC2YQIQJKTQQL7WHA5CNFSM4JFDK3XKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECIRLBQ#issuecomment-546379142 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AMDBJDYHNJ5SPLBOXRERJYDQQL7WHANCNFSM4JFDK3XA .  https://github.com/notifications/beacon/AMDBJD6TEUV6IBNU2XRH3X3QQL7WHA5CNFSM4JFDK3XKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECIRLBQ.gif

Sorry I’m still not clear. You have server 1 with 2 probs (outside and inside) and server 2 with its own outside prob? Because the chart lines will compete and skew the scale?


 When I went to the settings page, I found out that it deleted my settings entirely. After re-entering them, I still cannot keep the extension on, and re-opening the settings page shows that the settings are cleared again.
When I open the console and attempt to add a deadname, and click the arrow, "Uncaught TypeError: n is null" appears in the console. Clicking save produces the same error message.
I am using Firefox browser running on Arch Linux.

This appears to be fixed now.


Display errors in Engilsh

Hi, I am using this package in my project. And As I see, it is working fine as per my requirement. But the problem is when I try to display the errors, it is displaying errors in some other language. As shown below. Can you please help me to display these errors in English.

In Attachment You will find Validation Class after modified display Error to English not Italian language, and edit text regex to "[a-zA-Z0-9.\s\d\w]+"  in last it was not accepted any string and numbers and white spaces in string and display error "invalid range in character class at offset 13"
Validation.zip


Limits on configuration count?

Is there a limit on configuration count?
After 100 configurations, I can't add the new configuration.

Microsoft.Azure.Devices.Common.Exceptions.MessageTooLargeException: ErrorCode:ConfigurationCountLimitExceeded;Number of deployments exceed the maximum quota 100 for this IotHub.

Could you point me where it is documented?
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6f635b47-2f9b-b55c-6dc8-3e680bb62b92
Version Independent ID: ee5ed6ae-b5dd-f218-21ec-b9fb5e087dbd
Content: Automatic device management at scale with Azure IoT Hub
Content Source: articles/iot-hub/iot-hub-automatic-device-management.md
Service: iot-hub
GitHub Login: @kgremban
Microsoft Alias: kgremban

ok, found it here https://docs.microsoft.com/uk-ua/azure/iot-hub/iot-hub-devguide-quotas-throttling#other-limits
It would be great to mention it somewhere on this page. What do you think?

@Marusyk Thanks for the feedback. Since a separate doc is already available with all other limits, There is no specific need of mentioning each limits in the doc.
However, I will ask author @kgremban to review this feedback and add a reference to it if needed.

@JimacoMS4 Can you update the portal and corresponding CLI docs with a quick reference to the configuration limit and/or a pointer to the quotas article?
#reassign:jimacoms4

Hi @Marusyk -- thanks for your feedback! We've updated the documentation in a couple of places:

https://docs.microsoft.com/en-us/azure/iot-hub/iot-hub-automatic-device-management (bottom of the intro) and https://docs.microsoft.com/en-us/azure/iot-hub/iot-hub-automatic-device-management#create-a-configuration

https://docs.microsoft.com/en-us/azure/iot-hub/iot-hub-automatic-device-management-cli (bottom of the intro) and https://docs.microsoft.com/en-us/azure/iot-hub/iot-hub-automatic-device-management-cli#create-a-configuration

Please let me know in the next few days if you think more changes are needed. Otherwise, I'll go ahead and close this issue.
Thanks,
Jimaco

Cool, it is very helpful.
Thank you


Update to tokio 1.0

Based on #77

Is there anything blocking this? The checklist shows it as one of the last remaining things for Reqwest to update D:

Is there anything blocking this? The checklist shows it as one of the last remaining things for Reqwest to update D:

If I read the diff of this PR correctly, it seems we are waiting hyper-tls v0.14 (hyperium/hyper#2352), which is waiting for updates of h2 (hyperium/h2#499), http and http-body.

Closing in favor of https://github.com/hyperium/hyper-tls/pull/79


Generate File{Input,Output,IO}Stream automatically

CC @MathieuDuponchelle @EPashkin

@sdroege Thanks.

Then let's get it in, almost no manual code left for async functions then :)

@GuillaumeGomez anything missing from your side?

Nope, all good for me as well! Thanks!


Rewrite for when fields are missing from ADB output

When trying to parse the output from "adb devices -l" while my phone was in recovery mode, there was an "Invalid device list data" error because TWRP did not provide a model. I don't understand regex, so this is how I fixed it for my own personal use. This might need some more work, but it's good enough for my personal use.

@tng222 Thanks, can you add at least one unit test with the device output data you got on your machine (or just post that output here)?
I'm not married to the regex but it has been battle-proofed for a lot of different use cases, so just want to make sure we have unit tests for all different scenarios that we need to cover.
Thanks!

Here's the output of adb devices -l for my device in recovery mode:
ZY3222LBDC             recovery usb:337641472X product:omni_cedric device:cedric
Pretty much it's just the model missing, but the CreateFromAdbData method should be able to work as long as the serial and state are available, then leave the other properties as null. I'd sort out the tests myself if I could make it work on Visual Studio Mac or if I had time to make a Windows VM. I'm sure regex is fine if the method were reworked to handle this situation, I just didn't know how to directly fix the long expression, so this is what I did to be able to continue working on the program I needed this library for.

@tng222 Can you have a look at #86 and let me know if that works for you?

@tng222 I'm closing in favor of #86 for now but let me know if the issue persists and we'll re-open.


Ng-add does not add ui5 theming provider

Bug Description
When you install the package with ng add and use theming, it should also add theming provider service module in app.module alongside theming module
Expected Behavior
Working theming

@g-cheishvili actually, schematics are importing Ui5ThemingModule which contains instance of theming service, so providing the service itself is not needed anymore: https://github.com/SAP/ui5-webcomponents-ngx/blob/main/libs/ui5-angular-theming/src/ui5-theming.module.ts#L15


feat: add issuing support for iccp and galileo

Description
Title

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[x] Not Applicable

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

:tada: This PR is included in version 1.69.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


C#: Fix marshaling generic Godot collections

Add support for generic Godot collections to ConvertManagedObjectToVariant.

Fixes https://github.com/godotengine/godot/issues/65883

This was removed on purpose. I've thought of a way to add it back, though, including for type parameters of Godot generic collections and delegate invocations. Let's leave this unmerged for now, until I make these changes, then I can decide whether this is the proper solution for ConvertManagedObjectToVariant (if this method even remains).

This change is only about the default value, right? If that's the case, I can change GetGodotPropertyDefaultValues to return Variant. That would be one usage less of ConvertManagedObjectToVariant. The last usage would be in DelegateUtils, which I have an idea how to replace too.

The DelegateUtils usage seems complicated to replace. It uses System.Object because it needs reflection to access the fields of compiler generated classes (<>c__DisplayClass) for closures (needed by the editor during assembly reloading). I'll accept this PR, as we can't fully get rid of ConvertManagedObjectToVariant yet.


Artemis can't get results from fake zocalo when it has been restarted

Problems are intermittent - it was working for me until I restarted fake_zocalo, @noemifrisina has also observed this error
Artemis tests involving fake_zocalo fail due to timeout on results, but fake_zocalo is recieving the queries (based on its console output)

Seems to be a problem with RMQ startup - sometimes the results exchange is not created and bound to the xrc.i03queue

this hasn't happened in a while, closing, can be reopened if necessary


Order tree by size

This is a great tool.
Can you add an option to order the folder structure by size?

@wagoodman I am picking up this, do let me know if you already started working on this :)

@darxtrix haven't started yet, have at it!

Cool, thanks will send in a PR when done.

@wagoodman currently the file search is implemented at complete FileTree level and not at a particular FIleNode level, so are we going to implement the sorting operation on the FileTree level or FileNode level ? A FileTree level sort will be more resourcing consuming and I guess at a particular point users are mostly interested in sorting at a FileNode (directory).

I took this issue to be sort the current directory in the tree by size, but what you're really saying is globally sort the tree contents by size, regardless of which directory it is in? If that's true, then this is a heavier lift since it implies replacing the filetree UI (right pane) entirely.
Since each directory is showing the aggregated content size, would sorting the dir contents be good  enough? Later tonight I can jump back on and go into a bit more detail.

Hi @wagoodman, just following up. Any thoughts on this ?

Sorry for the delay, you're right about needing to add a total size parameter on the FileNode struct... I'll call it branchSize here, or the total number of bytes on this tree branch --from this node and it's children. My first thought is to populate branchSize at either FileTree.AddPath or FileNode.AddChild. The upside is that this is the only place that we are affecting the tree structure. The downside is that there may be a performance hit when calling something like FileTree.StackRange.
The next problem to solve is ensuring that there is parity with rendering the tree string. Currently it presumes that each dir contents is sorted alphabetically (https://github.com/wagoodman/dive/blob/master/filetree/tree.go#L73), so now there will need to be a way to indicate the current sort preference and reference that here.
As for ASC/DESC, I'd start with DESC for now, we can always expand it to ASC later (this becomes more of a screen realestate problem than a technical problem... the bottom hint bar that shows keybinding hints is getting full).

Cool, I will go ahead and implement this. Thanks !

I see this issue was removed from "UI Enhancements". What is the status of this? Is it just low priority?
Thanks.

@wagoodman Is this abandoned?

It seems like this specific issue is probably up for grabs, yup.

Would (still) be greatly appreciated in 2021 :) Love the tool 👍

+1

+1, this would be super beneificial.
Currently, I find myself going into an interactive terminal, and running something like ls -lh node_modules | grep M  to find anything that's at least a MB.

I'll join the choir and say that this would be a terrific feature. Also, I would like to add that dive is a fantastic tool and I truly appreciate all the work you put in.

Still would be a great feature in 2022 :)

In 2023 we would also really appreciate if this is implemented

looks like this was added in v0.11.0. If you're focused on the current layer contents and hit CTRL + o, it'll sort by size.

Awesome, we did it guys. (by bumping a feature request for half a decade and absolutely no one making a single PR). Another win for open source 😎

FYI: This issue has been resolved by https://github.com/wagoodman/dive/pull/424


Debug session selector grows upwards when at top of screen making it difficult to use

When I go to select a debug session and there are several, this dropdown grows upwards but has limited space on the screen so sends up being quite difficult to use.
This is on macOS, stable channel.

This is just native electron behavior. Pleae file it against electron. Thanks

Ah, cool - I've opened it at https://github.com/electron/electron/issues/13279. Thanks!


KAFKA-6032: Unit Tests should be independent of locale settings

Allows to run the tests on linux servers without needing the locale set to US.

retest this please

retest this please


No 'id' attributes anymore and remove read_element_id()

id() is a build-in function and it is terrible style to overwrite it for some contexts!
Besides these large but rather trivial change, the read_element_id() utility function is removed: This function was introduced to reduce the amount of duplicated code, but in my experience, the code which read_element_id() replaces is rather trivial and the function obfuscates things quite a bit and it does not play nicly with the mypy type checker. (Besides that, its name was IMO pretty confusing because it did not create an OdxLinkId, but parsed the elements featured by the ELEMENT-ID XSD     group...)
Andreas Lauser<EMAIL_ADDRESS>on behalf of MBition GmbH.
Provider Information

@kayoub5: I hope to have addressed the issues you've found...

lacking vetos, I intend to merge this PR tomorrow...

Got an error, bit_position = None
unsupported operand type(s) for +: 'int' and 'NoneType'
odxtools\diagcodedtypes.py", line 110, in _to_bytes
    offset = (8 - ((bit_length + bit_position) % 8)) % 8

unsupported operand type(s) for +: 'int' and 'NoneType'
odxtools\diagcodedtypes.py", line 110, in _to_bytes
offset = (8 - ((bit_length + bit_position) % 8)) % 8

stuff like that was the reason for the _int properties ;). can you provide a full traceback?

Caused by calling struct.encode(...)
  File "odxtools\diagcodedtypes.py", line 110, in _to_bytes
    offset = (8 - ((bit_length + bit_position) % 8)) % 8
  File "odxtools\diagcodedtypes.py", line 496, in convert_internal_to_bytes
    return self._to_bytes(internal_value,
  File "odxtools\dataobjectproperty.py", line 100, in convert_physical_to_bytes
    return self.diag_coded_type.convert_internal_to_bytes(internal_val, encode_state, bit_position=bit_position)
  File "odxtools\multiplexer.py", line 159, in convert_physical_to_bytes
    key_bytes = self.switch_key._dop.convert_physical_to_bytes(
  File "odxtools\parameters\valueparameter.py", line 54, in get_coded_value_as_bytes
    return self.dop.convert_physical_to_bytes(physical_value,
  File "odxtools\parameters\parameterbase.py", line 108, in encode_into_pdu
    byte_value = self.get_coded_value_as_bytes(encode_state)
  File "odxtools\structures.py", line 146, in convert_physical_to_internal
    coded_rpc = param.encode_into_pdu(encode_state)
  File "odxtools\structures.py", line 266, in encode
    return self.convert_physical_to_internal(params,

See multiplexer.py, line 160, pass self.switch_key.bit_position to convert_physical_to_bytes without None checking

See multiplexer.py, line 160, self.switch_key.bit_position passed to convert_physical_to_bytes without None checking

ok. should be fixed now...

let's get this in...


Enable "Alternate Screen Mode" in OSX

Ok, how do you enable the alternate screen modes in less / man and similar outputs (enables scrolling with the mouse wheel)?
The best I could find is setting export LESS='-g -i -M -R -w -z-1' in .zshrc

You'll have to read the LESS man page. That variable is set in zprofile.


WiFi immediately disconnected if there is no internet (Android 9 Pie)

Issue type
Select all that apply

[x] Bug
[ ] Enhancement
[ ] Task
[ ] Question
[ ] Other

Description
Android 9 (Pie), once connected to Wifi (which has no internet) it immediately disconnects and sometimes does't connect at all.
Using WifiWizard2.connect(SSID, true)
Expected behavior: With bindall set in the connect it should be able to connect to wlan with no internet connection
Actual behavior: Cannot connect to wlan without internet
Reproduces how often: 100%
Versions
3.1.0
Additional Information
No issues with Android version 5 to 8 (Tested)

Even I am facing the same issue. If I connect to the WiFi SSID with no internet for the first time, it stays connected. But if I connect back to it again, it disconnects immediately and connects back to the previous WiFi profile which has internet.

Similar issue, I test on Google Pixel 3 Android 10.
After I call WifiWizard2.connect(SSID, true, PASSWORD, 'WPA', false), it will try to connect the no internet access wifi for a while, then switch to another wifi which has internet access.
It will not keep the connection on the wifi I want.

@mknnaren by the way, I found a workaround is to call disableWifi() first, then do connect().
This workaround work for SAMSUNG Galaxy A40s Android 9.
Unfortunately, it not works for Google Pixel 3 Android 10.


Add cloud provider outage notifications

Add messaging around cloud provider outages.

This looks fine to me, but I'll let others comment as well.

Seems like the messages could be combined & a variable used for specifying the cloud-provider. Is the goal to just limit the number of -p flags you have to use?

Looks good to me too

/lgtm
/approve


Wfcore 4045 health error codes

https://issues.jboss.org/browse/WFCORE-4045 which is needed to resolve https://issues.jboss.org/browse/WFLY-10861.
If the first three 'housekeeping' commits are too many, they can be squashed

Core - Full Integration Build 7846 outcome was UNKNOWN using a merge of 01f4989b83ce4890a431581aca20a108fd5020cd
Summary: Canceled (Exit code 137 (new)) Build time: 00:00:28

Core - Full Integration Build 7845 outcome was UNKNOWN using a merge of 01f4989b83ce4890a431581aca20a108fd5020cd
Summary: Canceled (Tests passed: 3408, ignored: 105; exit code 137 (new)) Build time: 00:34:41


Deploying kind minimal shows message: To upgrade this instance run 'oz upgrade'

Deploying kind minimal shows message: To upgrade this instance run 'oz upgrade' when a minimal proxy is not upgradeable.
Example minimal proxy deployment:
$ npx oz deploy
Nothing to compile, all contracts are up to date.
? Choose the kind of deployment minimal
? Pick a network development
? Pick a contract to deploy @openzeppelin/contracts-ethereum-package/ERC20PresetMinte
rPauserUpgradeSafe
Minimal proxy support is still experimental.
✓ Deploying @openzeppelin/contracts-ethereum-package dependency to network dev-1589968356945
All implementations are up to date
? Call a function to initialize the instance after creating it? Yes
? Select which function initialize(name: string, symbol: string)
? name: string: MyToken
? symbol: string: MYT
✓ Instance created at 0x067805E69e62E8bE56e8D13f4EBf53372D3dD02e
✓ Deployed ProxyFactory at 0x6eD79Aa1c71FD7BdBC515EfdA3Bd4e26394435cC
To upgrade this instance run 'oz upgrade'
0x067805E69e62E8bE56e8D13f4EBf53372D3dD02e

hi,there,
I want to know what is the difference between minimal and upgradeable.

Hi @shiqinfeng1,
I have posted this question with an answer in the Community Forum: https://forum.openzeppelin.com/t/what-is-the-difference-between-minimal-and-upgradeable-proxies/2964/2?u=abcoathup
If you have further questions we can continue the discussion in the Community Forum.  Please ask all the questions that you need.


Emmet like behaviour

Is your feature request related to a problem? Please describe.
I first have to type {% before I see some liquid suggestions.
Describe the solution you'd like
I want to see suggestions right away when I'm typing. I'm now still using my own extension, but I'd like to see this within the Shopify extension.
Also make sure that it's autocompleted correctly with a {%- liquid -%} tag (so without delimiters etc.).
{%- assign variable_name = 'value' -%}

{%- liquid
  assign variable_name = 'value'
-%}

Please checkout the extension to see it in action.
Checklist

[ ] I have checked and made sure that the proposal adheres to this plugin's principles

Hey @stijns96, I agree, this would be a nicer development experience. I'll raise it with the team but for now, I can label it as something open for community contributions :)

Sure thing! I will dive into it as well :)


QueryRewrite: add navigation expansion support for GroupBy

This is complicated because navigations that appear in the grouping selector need to be expanded before the groupby call itself. This probably requires a number of "pending" operations to be stored on NavigationExpansionExpression, just like we do for projections and order by.

IQueryable<<>f__AnonymousType276<Customer, int>> Join(
|__ outer: DbSet<Customer>,
|__ inner: IQueryable<<>f__AnonymousType273<string, int>> Select(
|   |__ source: IQueryable<IGrouping<string, Order>> Where(
|   |   |__ source: IQueryable<IGrouping<string, Order>> GroupBy(
|   |   |   |__ source: DbSet<Order>,
|   |   |   |__ keySelector: (Order o) => o.CustomerID),
|   |   |__ predicate: (IGrouping<string, Order> g) => int Count(g) > 5),
|   |__ selector: (IGrouping<string, Order> g) => new {
|           CustomerID = g.Key,
|           LastOrderID = int Max(
|           |__ source: g,
|           |__ selector: (Order o) => o.OrderID)
|        }),
|__ outerKeySelector: (Customer c) => c.CustomerID,
|__ innerKeySelector: (<>f__AnonymousType273<string, int> a) => a.CustomerID,
|__ resultSelector: (Customer c | <>f__AnonymousType273<string, int> a) => new {
        c = c,
        LastOrderID = a.LastOrderID
     })

After nav expansion
IQueryable<<>f__AnonymousType276<Customer, int>> Select(
|__ source: IQueryable<TransparentIdentifier<Customer, IGrouping<string, Order>>> Join(
|   |__ outer: DbSet<Customer>,
|   |__ inner: IQueryable<IGrouping<string, Order>> Where(
|   |   |__ source: IQueryable<IGrouping<string, Order>> GroupBy(
|   |   |   |__ source: DbSet<Order>,
|   |   |   |__ keySelector: (Order o) => o.CustomerID),
|   |   |__ predicate: (IGrouping<string, Order> Param_0) => int Count(Param_0) > 5),
|   |__ outerKeySelector: (Customer Param_0) => Param_0.CustomerID,
|   |__ innerKeySelector: (IGrouping<string, Order> Param_0) => Param_0.Key,
|   |__ resultSelector: (Customer Param_0 | IGrouping<string, Order> Param_0) => new TransparentIdentifier<Customer, IGrouping<string, Order>>(
|           Outer = Param_0,
|           Inner = Param_0
|       )),
|__ selector: (TransparentIdentifier<Customer, IGrouping<string, Order>> join) => new {
        c = join.Outer,
        LastOrderID = int Max(
        |__ source: join.Inner,
        |__ selector: (Order Param_0) => Param_0.OrderID)
     })

Nav expansion moving aggregates inside joins make it hard to translate.


update v1.0.3

You may be interested, my English is really poor, and there may be some places where translation software cannot express my intention clearly.

This looks like a very big commit, I'll have to take some time to inspect.

I'm sorry but after inspecting this PR, there are too many additions that doesn't meet with the initial goals of this plugin. Thus, I have to close this PR without merging.


Fix root path in breadcrumb

If blank, set to root.  My this.$route.matched sets the root (Home) path to a blank string

Hi!
Thank you for your PR!
Breadcrumb was refactored in version 2 so this PR is outdated.
Regards


Alert when Orka cluster is unavailable

The Orka cluster underwent maintenance but we're not getting any alerts about it being unavailable. We need to put a mechanism in place for the Orka orchestrator to report back connectivity issues.

I think the only way this could work is if we add Prometheus metrics to the Orka driver and scrape it from the prod Prometheus.

The problem with that is that the Orka orchestrator is spawned via Gitlab per job, i.e. it is not a long-running, scrapable process. We'd have to have it push metrics to a push gateway.
We do that already for the batch jobs like reconciliation and analysis but that push gateway is local to the docker swarm network and currently not accessible from outside the network.
We could make it available but then we'd need to deal with some form of authentication. It's probably better/easier to bring up another push gateway next to the Orka orchestrator that we then push to from the orchestrator and scrape from Prometheus.


Accept num type for minValue and maxValue

The constructors were created accepting minValue and maxValue as int. This PR fixes this issue by acceptig num as the type instead.

Thank you so much! @tainanfochesatto
Your changes have been released!


[cli] flag provided but not defined: -url

Rancher versions:
cli version 2.0
Operating system and kernel: mac os x high sierra
Steps to Reproduce:
When using the latest CLI version passing the --url flag returns this error
flag provided but not defined: -url
ex:
SERVICES="$(rancher --url $RANCHER_URL --access-key $RANCHER_KEY --secret-key $RANCHER_SECRET ps | grep :staging | awk '{print $1}')"

I am experiencing the same issue with version 2.0.2 on Windows and Linux.
It works if I only call "rancher --url...". However, as soon as I add any prefix like a full path or ./ (required in PowerShell) I get this error.
I reverted back to 0.6.9 and it works fine.

Any CLI version that's tagged as v2.x is made for [Rancher 2.0, where as CLI versions v0.x is for Rancher v1.x.
These are 2 different server products, so please make sure to use the download link inside Rancher.


Now we can write a complete technology package for KLayout using the KLayoutTechnology class.

Thank you Thomas!


window._sharedData

Could not find "window._sharedData" in html response. [retrying; skip with ^C]

incomplete bug report.
looks like old bug, check version and update.
closing soon.

my version is 4.8.4


feat: Preliminary support for Python 3.13

No RC yet, just beta releases, but we can already start setting stuff up.
3.13 release schedule: https://peps.python.org/pep-0719/#schedule

Do these allow-prereleases: true have any affect on non-3.13 tests? I would be hesitant to merge this if so

Will double check this.

Edit: Just noticed the "Do not merge" label, is that still accurate?

No, removed now, thanks for noticing!

@szokeasaurusrex allow-prereleases will only kick in if the specified Python version is not generally available (all older Python versions are, so no effect there), see here.


Moving to Adjacent Topics

This package is currently published under my name on npm. Will moving this repository to @Adjacent-Topics cause any problems?

Sorry I didn't see this earlier! You should be able to continue publishing this under your NPM username.

Okay! I think that's probably the best setup, too, because it doesn't make sense to publish the package as @Adjacent-Topics/sb-edit. (I think if you publish a package under an organization it has to include the org's name...)


[Question] How to make external post request from FASTAPI?

for example:
BASEDOMAIN = "https://.astro.seb.backend.qat.gaincapital.cloud/v1"
$ curl -v -X POST https://houston.BASEDOMAIN/v1 -H "Authorization: Bearer "

If you don't have a need for aio you can use regular requests also.

Check this one for best practices..
https://github.com/tiangolo/fastapi/issues/236

If you don't have a need for aio you can use regular requests also.

But presumably you shouldn't do this as you'll end up blocking the event loop whilst the request completes.

If your endpoint(s) is synchronous, (just def, not async def), then you'll want to use a synchronous HTTP library, the most popular obviously being requests.
If your endpoint(s) are asynchronous (async def), then you'll want to use an asyncio HTTP library, like aiohttp.
If you have a mix of async and sync endpoints, then httpx is probably your best option, as it supports both sync and async modes. Obviously if you only have one or the other httpx can be a good option as well, though it is arguably less battle-tested than the other two libraries listed.
In the end, it's just important you don't block your async endpoints with synchronous requests (i.e. if you have an async def endpoint and you are making an HTTP request without calling await or async with, you're probably doing it wrong). Otherwise any HTTP lib will do.

Thanks for the help here everyone! 👏  🙇
If that solves the original problem, then you can close this issue ✔

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


AttributeError: 'dict' object has no attribute 'iteritems'

i@d:~/rl/pytorch-maml-rl$ python main.py --env-name HalfCheetahDir-v1 --num-workers 8 --fast-lr 0.1 --max-kl 0.01 --fast-batch-size 20 --meta-batch-size 40 --num-layers 2 --hidden-size 100 --num-batches 1000 --gamma 0.99 --tau 1.0 --cg-damping 1e-5 --ls-max-steps 15 --output-folder maml-halfcheetah-dir --device cuda

Traceback (most recent call last):
  File "main.py", line 138, in <module>
    main(args)
  File "main.py", line 29, in main
    config = {k: v for (k, v) in vars(args).iteritems() if k != 'device'}
AttributeError: 'dict' object has no attribute 'iteritems'

I followed your readme but the error occurs. Any tips, Thanks.

@dragen1860 are you using Python 3+?

yes. @dendisuhubdy

@dragen1860 iteritems() was removed from Python3, the codebase I believe is running on Python 2.7. Change main.py line 29 to config = {k: v for (k, v) in vars(args).tems() if k != 'device'}, let me know.

Your config is a tuple without a update attribute, config is supposed to be a dict. Did you remove the { ... } from config = { ..... }

oh, I solved it .
Can you update the readme to append a guide of how to install mujoco?
gym.error.DependencyNotInstalled: No module named 'mujoco_py'. (HINT: you need to install mujoco_py, and also perform the setup instructions here: https://github.com/openai/mujoco-py/.)

Oh my bad, I did that change locally but didn't committed it. Thank your for the feedback!
As for installation for Mujoco, this is also something I forgot in the requirements (gym[mujoco] instead of gym). I will make that change. As for the installation of the software itself, I suggest going through the steps from the mujoco-py readme.


Need to be able to pass more ssl options into websocket connection

Currently, we can only pass ssl_verify through lensesio.lenses.main down through to
https://github.com/lensesio/lenses-python/blob/master/lensesio/data/sql.py
And from there into websocket.create_connection:
    if self.verify_cert:
        conn = websocket.create_connection(
            self.lenses_exec_sql_endpoint
        )
    else:
        conn = websocket.create_connection(
            self.lenses_exec_sql_endpoint,
            sslopt={"cert_reqs": ssl.CERT_NONE}
        )

Unfortunately my deployment requires additional sslopt for websocket.create_connection, namely:
check_hostname=False

ssl_version=ssl.PROTOCOL_TLSv1_2

I propose that lenses.main as it exists be extended:
def __init__(
        self,
        auth_type="basic",
        url=None,
        username=None,
        password=None,
        krb_service=None,
        service_account=None,
        verify_cert=True):

To:
def __init__(
        self,
        auth_type="basic",
        url=None,
        username=None,
        password=None,
        krb_service=None,
        service_account=None,
        verify_cert=True,
        ssl_options={}):

As well as SQLExec.init, to change:
def __init__(self, active_threads, verify_cert=True)

To:
def __init__(self, active_threads, ssl_options={})

And accordingly pass these sslopts "as-is" into the websocket.create_connection call.
I am happy to work up and create a pull request if needed. Currently I am monkeypatching websocket prior to importing lenses:
# monkeypatch websocket tls
import ssl, websocket
_websocket_create_connection = websocket.create_connection
def _create_connection(url, timeout=None, class_=websocket.WebSocket, **options):
    _ssl_options = options.get("sslopt", {})
    _ssl_options = dict(_ssl_options, check_hostname=False, ssl_version=ssl.PROTOCOL_TLSv1_2)
    _options = dict(options, sslopt=_ssl_options)
    return _websocket_create_connection(url, timeout, class_, **_options)
websocket.create_connection = _create_connection
# end monkeypatch websocket tls

And while this is working is (obviously) far from ideal.

Wanted to update this issue: beyond more SSL options, we also need to add a cookie to pass into websocket create_connection. The current monkeypatch looks more like this:
_options = dict(options, sslopt=_ssl_options, header=["Cookie: AWSALB=CookieValue"])

So probably what the real "fix" looks like is letting us pass any options we want down into create_connection.

Hello @montsamu. I am sorry, but this library has been abandoned for quite some time now and hasn't followed the development of the Lenses REST API.
I will close this issue as we are going to archive the repository. I apologize for the inconvenience both this action and the delay of communication may have caused you.


Comply with PSDL

This addresses security issues.

Looks good. I will approve this after you pass the e2e-leg-5 tests.
It seems you tried to fix all the critical and high vulnerabilities. You can just fix the critical one if other fixes cause the failure in the test.


Windows Precompiled Binaries are not up to date

Required Info

Camera Model
{ R200 / F200 / SR300 / ZR300 / D400 }

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
{Win (8.1/10) / Linux (Ubuntu 14/16/17) / MacOS

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC/Raspberry Pi/ NVIDIA Jetson / etc..

SDK Version
{ legacy / 2.. }

Language
{C}

With every new source version of librealsense, the precompiled binaries are left untouched. Why should the binary installer be different from the source version of the library ?

[Realsense Customer Engineering Team Comment]
hi @Chimdi511,
sorry i don't get you, do you mean tools or examples from Intel.RealSense.SDK.exe of v2.12.0, are not the same from Source code (zip) of v2.12.0?

I mean the tools and examples from Intel.RealSense.SDK.exe. For example, the pcl examples aren't in the execeutable.

Same for the rs-convert tool.
I had to download the source and compile everything to get this tool :)

Hi @Chimdi511, @Julz-s
We will gradually keep expanding Windows / Linux and PiPy installers, but all new features are first introduced and tested in the open-source environment and only later integrated into the different packages (so the newly added rs-convert should be added to the installer soon).
Another consideration is what dependencies we want to redistribute. I don't think PCL examples will ever be part of the installer, since PCL itself has no stable ABI (we would have to redistribute a separate binary for each version of Visual Studio)


Bug in forward.py

Hello,
the following error occurred when I execute the forward.py file
Traceback (most recent call last):
File "forward.py", line 11, in
new_saver.restore(sess, [checkpoint_fn(layers)])
File "/home/msf/.local/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 1428, in restore
{self.saver_def.filename_tensor_name: save_path})
File "/home/msf/.local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 767, in run
run_metadata_ptr)
File "/home/msf/.local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 944, in _run
% (np_val.shape, subfeed_t.name, str(subfeed_t.get_shape())))
ValueError: Cannot feed value of shape (1,) for Tensor u'save/Const:0', which has shape '()'
Could you please help me?
Thanks much!
Omid

@omidme I encountered the same issue , How did you fix it? plaease

@hhxjzyr copy the pretrained .ckpt .meta to the folder of forward.py


Potentially Flaky test: CompactionJobQueuesTest.testGetAsync()

Test name(s)
CompactionJobQueuesTest.testGetAsync()
Describe the failure observed
GitHub actions failed on 94167151e2de1be87ac60e25751edf1b557d3a3b for this test. Thought it might have been broken by this commit, but ran locally and was not able to reproduce a failure. Reran the GH action and it passed. Failed here:
https://github.com/apache/accumulo/blob/94167151e2de1be87ac60e25751edf1b557d3a3b/server/manager/src/test/java/org/apache/accumulo/manager/compaction/queue/CompactionJobQueuesTest.java#L402
Testing Environment:
4.0/main

I just saw this happen for the first time locally during a build:
[ERROR] Tests run: 4, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 4.079 s <<< FAILURE! -- in org.apache.accumulo.manager.compaction.queue.CompactionJobQueuesTest
[ERROR] org.apache.accumulo.manager.compaction.queue.CompactionJobQueuesTest.testGetAsync -- Time elapsed: 1.647 s <<< FAILURE!
org.opentest4j.AssertionFailedError: expected: <true> but was: <false>
	at org.junit.jupiter.api.AssertionFailureBuilder.build(AssertionFailureBuilder.java:151)
	at org.junit.jupiter.api.AssertionFailureBuilder.buildAndThrow(AssertionFailureBuilder.java:132)
	at org.junit.jupiter.api.AssertTrue.failNotTrue(AssertTrue.java:63)
	at org.junit.jupiter.api.AssertTrue.assertTrue(AssertTrue.java:36)
	at org.junit.jupiter.api.AssertTrue.assertTrue(AssertTrue.java:31)
	at org.junit.jupiter.api.Assertions.assertTrue(Assertions.java:183)
	at org.apache.accumulo.manager.compaction.queue.CompactionJobQueuesTest.testGetAsync(CompactionJobQueuesTest.java:402)
	at java.base/java.lang.reflect.Method.invoke(Method.java:569)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)

@cshannon Thanks for the confirmation and stack trace. I was not able to reproduce it again.

I think the root issue is that the sleep that is supposed to wait for future6 to timeout may not wait quite long enough sometimes, so future6 gets the job when added to the queue and not future7. I have a small PR to push up shortly to make the test more reliable


Runaway energy usage by dotnet process after quitting VSCode

The bug
When I run some c# code in an interactive notebook in VS Code and then quit the application, within moments a new dotnet process is started. That dotnet process is instantly using 100% cpu and never seems to quit by itself.

Forgetting to force quit this process has now made it that VSCode is the number one power impact app on my laptop even when VS Code is not running:

Info
Fontent: VS Code

Version: 1.0.355307+4cc71ae4e275fd80852b9c2faa317bc3a0033d2c
Library version: 1.0.0-beta.22553.7+4cc71ae4e275fd80852b9c2faa317bc3a0033d2c
Build date: 2022-11-04T02:20:42.0000000Z

OS: Ventura 13.0.1
MacBook Pro M1

We think this was addressed in #2466, but that fix is only in the version of the extension for VS Code - Insiders.  If you have that installed (here) could you try it there?

I have the same issue on Linux:
Does this issue occur when all extensions are disabled?: No
This issue is caused by the Polyglot Notebooks extension (currently in Preview)
VS Code Version:
code --version
1.74.2
e8a3071ea4344d9d48ef8a4df2c097372b0c5161
x64

OS Version:
OS: Ubuntu 22.04.1 LTS x86_64
Host: MINIPC PB50 0602
Kernel: 5.15.0-57-generic
Uptime: 3 hours, 36 mins
Packages: 2641 (dpkg), 52 (snap)
Shell: bash 5.1.16
Resolution: 2560x1440
DE: GNOME
WM: Mutter
WM Theme: Yaru-dark
Theme: Yaru-olive-dark [GTK2/3]
Icons: Yaru-olive [GTK2/3]
Terminal: gnome-terminal
CPU: AMD Ryzen 7 3750H with Radeon Vega Mobile Gfx (8) @ 2.300GHz
GPU: AMD ATI Radeon Vega Series / Radeon Vega Mobile Series
Memory: 6787MiB / 30020MiB

Steps to Reproduce:

Open a folder containing *.ipynb files in VSCode
Open an  *.ipynb file by double-clicking on it
Verify that a the notebook kernel is loaded and the expected content is visible
Close the *.ipynb file in VSCode
Close VSCode

You should see one or more orphaned/runaway processes like this (in System Monitor):

Also saw this recently on MacOS after returning to my computer after being away from a few hours

Polyglot notebooks v1.0.4059041
VS Code : 1.75.0-insider (Universal) Commit: e7b5449579249db344d473370d56a5df9446c826
Quitting VS Code insiders didn't stop the dotnet processes, I had to manually kill them


A finicky RPG from Hong Kong

Hi, I hope I won't bother you too much, but I wonder if there could be any hope for playing this DX7 game on a modern OS, like Win7/10?
This is the game plus official patch: https://drive.google.com/drive/folders/1OTGcDUkIkMU14oDvkK2CWTo7AlIHap70?usp=share_link
Without applying updated EXE the game boots and plays first video in fullscreen, but the second video starts in a tiny window in the top-left corner of the screen.
After the second video ends, that small window stays black while playing the game's main menu's music, and some invisible error messages start popping up (this all happens with or without DDrawCompat).
Those errors don't appear when Win98 compat mode is enabled, but using it together with DDrawCompat makes the game freeze on second video somehow.
Out of other DDraw wrappers I think the closest one to work was WineD3D: the game will actually go into menu and gameplay, but the videos are black (although they're fullscreen).
Apparently, with WineD3D the windows for videos also don't close off properly and linger in the background, sometimes interferring with the inputs.
The DxWnd folks are having their try at this too, but it looks like they're not having much luck either, as mitigating one issue just seems to bring some another.
Then there's also that second EXE (official patch) that doesn't work at all for some reason.

I tried using alternative load methods from the wiki in case if the game ignores local ddraw.dll, but that didn't help.
Not sure if there's any use in these, but here's logs from loading both updated and old EXEs (without any other compat settings enabled): DDrawCompat_logs.zip
I don't really mind if I would have to use DDrawCompat together with some other tool (like DxWnd), but I worry that the game might actually end up being a lost cause with all those issues and it won't ever work on anything higher than WinXP…

I quickly sneak in just to say that the DxWnd team (hem..) would be glad to partecipate for everything may be unrelated to ddraw problems alone. My guess is that the game doesn't have ddraw problems only and maybe a combined configuration DxWnd+DDrawCompat could be a winning team. Regards, gho.

I suspect that this game could be a collection of oddities. Anyway ...
Trying to debug the game (attached to DxWnd, but in the initial steps this should make no big difference) and in the hope not to tell inaccurate informations, the initial error message comes from the general purpose error handling routine that meets an unexpected error 0x80004002.
The error is returned from here:
CPU Disasm
Address   Hex dump          Command                                  Comments
0041946E  |.  FF11          CALL DWORD PTR DS:[ECX]
but do not expect this to be a ddraw error, in effect it is generated by the game executable here:
CPU Disasm
Address   Hex dump          Command                                  Comments
66CB237A  |.  B8 02400080   MOV EAX,80004002
after two memcmp operations:
CPU Disasm
Address   Hex dump          Command                                  Comments
66CB233C  |.  6A 10         PUSH 10                                  ; /count = 16.
66CB233E  |.  68 1824C866   PUSH 66C82418                            ; |buf2 = DDRAW.66C82418 -> 00
66CB2343  |.  FF75 0C       PUSH DWORD PTR SS:[EBP+0C]               ; |buf1
66CB2346  |.  E8 7802FEFF   CALL <JMP.&msvcrt.memcmp>                ; \MSVCRT.memcmp
66CB234B  |.  83C4 0C       ADD ESP,0C
66CB234E  |.  85C0          TEST EAX,EAX
66CB2350  |.- 74 2F         JZ SHORT 66CB2381
66CB2352  |.  6A 10         PUSH 10                                  ; /count = 16.
66CB2354  |.  68 542DC866   PUSH 66C82D54                            ; |buf2 = DDRAW.66C82D54 -> 85
66CB2359  |.  FF75 0C       PUSH DWORD PTR SS:[EBP+0C]               ; |buf1
66CB235C  |.  E8 6202FEFF   CALL <JMP.&msvcrt.memcmp>                ; \MSVCRT.memcmp
66CB2361  |.  83C4 0C       ADD ESP,0C
66CB2364  |.  85C0          TEST EAX,EAX
66CB2366  |.- 74 19         JZ SHORT 66CB2381
my guess is that the game is checking the ddraw GUID to match two possible values.
In effect, using a DxWnd tweak that forces a NULL GUID for the DirectDraw session seems to bypass this initial error message so that you can enjoy the following ones ... :(

Wait a minute! I made a little confusion here, mixing the original and patched releases ....
This is what I found: the "official fix" was built with no direct reference to ddraw:DirectDrawCreate, instead the developers used the ordinal 9 that probably WAS a reference to DirectDrawCreateEx but it no longer is and points now to DirectDrawCreateClipper.
So, to cut the thing short, I edited the KLS.exe PE replacing the ordinal 9 with ordinal 8 and the game starts and work as a beauty!
I still have to check if the same problem happens also for DirectSound and DirectInput, but at least I got the game finally running!

DirectInput and DirectSound references seem ok, so there should be no need for further patching here.
In attach the patched executable, good as long as Microsoft won't change the ordinal numbers again. Maybe a better patch would eliminate the ordinal numbers and replace them with function names, but not today.
The game seems perfectly playable, but of course you can head to DxWnd for a locale translation (it seems to require a traditional Chinese Big5 locale support) and running it in window.
KSL.patched.zip

@ghotik, thank you so much for finally solving this case!
With your patched executable (and .dxw config from the thread at SourceForge) it really does work wonderfully!
Since DxWnd alone seems to be enough, I guess I should not waste Narzoul's time any more and close this issue now. ^^

the "official fix" was built with no direct reference to ddraw:DirectDrawCreate, instead the developers used the ordinal 9 that probably WAS a reference to DirectDrawCreateEx but it no longer is and points now to DirectDrawCreateClipper.
I still have to check if the same problem happens also for DirectSound and DirectInput, but at least I got the game finally running!

@ghotik, Does the same issue happen with DirectSound and DirectInput?  What was the final outcome of this game?

@elishacloud: as I said the final result seems good and there are no troubles with DirectSound and DirectInput ordinal numbers, but the game being Chinese doesn't let me make a final judgement. Someone should try to test it a little more thoroughfully.


Add a Gitter chat badge to README.md

PUGX/botrelli now has a Chat Room on Gitter
@giorrrgio has just created a chat room. You can visit it here: https://gitter.im/PUGX/botrelli.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

mmm :-1:  about this... who will talk on?
@giorrrgio you will spend time on it?

:-1: my fault, clicked yes accidentally :(


Upload filenames are saved with broken encoding

Bug Report
When someone uploads a file with special filename, its name is being saved with wrong encoding. For example: ščžříá řá šíř pplůúůú.jpg is saved as Å¡Ä\x8DÅ¾Å\x99ÃÃ¡ Å\x99Ã¡ Å¡ÃÅ\x99 pplÅ¯ÃºÅ¯Ãº.jpg. The former is sample of my alphabet :smile:

Expected Behavior
Save upload filename name as it should be.

Current Behavior
It saves wrong filename, but it works when I try to access it through URL.

Possible Solution
I do not know, how to fix it. In fact, I do not know what causes this.

Steps to Reproduce

create Payload app
create upload with some weird characters

Detailed Description
I can simulate the same result with
Buffer.from('ščžříá řá šíř pplůúůú.jpg', 'utf-8').toString('latin1')

Is there at least a way of renaming the file before saving? I tried request handler, but it did not help.

discuss issues

Hey @lksfrn — this is a great question. But there's an easy solution! Payload exposes config.upload options which allow you to customize the options that Payload uses behind-the-scenes while uploading files. Being that Payload uses busboy, you can pass any options that busboy supports. With those options, you can keep utf8 filename encoding.
 Love when stuff is easier than it seems!

Thank you very much, it works! 😍


User's name not being spoken in some cases

In GetLastMessageFromChannelIntent sometimes the name of the user isn't present in the response.
Ideally, in the below example it should say, Ashish Jha says, hi.
But the actual result is, says, hi.

Fixed This Issue in PR #40

Thank you.


fix: remove event listener for document and history

fix: https://github.com/alibaba/lowcode-engine/issues/1610

@alilc/lowcode-plugin-undo-redo 1.0.1 已发


How to change the highlight pen's color from yellow to another color ?

How to change the highlight pen's color from yellow to another color ?
I see the setstroke() line but even when i change color to anything else, it didn't change anything.
Thanks so much.

Did you find a solution?


IN array condition when using yii\db\Query

What steps will reproduce the problem?
$statuses = [1, 2, 3];
$rows = (new \yii\db\Query())
    ->select('*')
    ->from('user u')
    ->leftJoin('profile p', new \yii\db\Expression('p.id = u.profile_id AND p.status IN (:statuses)', [':statuses' => $statuses]))
   // OR ->leftJoin('profile p', 'p.id = u.profile_id AND p.status IN (:statuses)', [':statuses' => $statuses])
    ->createCommand()
    ->getRawSql();

What is the expected result?
SELECT * FROM `user` `u`
LEFT JOIN `profile` `p`
ON p.id = u.profile_id AND p.status IN (1, 2, 3)

What do you get instead?
SELECT * FROM `user` `u`
LEFT JOIN `profile` `p`
ON p.id = u.profile_id AND p.status IN (1)

Additional info
It seems to be more a PDO issue but maybe Yii could have a way to circunvent it when calling Command::bindPendingParams().

Q
A

Yii version
2.0.8

PHP version
7.0.4

Operating system
Ubuntu 16.04

onCondition

->leftJoin()->onCondition()

onCondition() is not available for Query but for ActiveQuery.

try:
->leftJoin('profile p', ['p.id' => 'u.profile_id', 'p.status' => $statuses])

It really works @pstdd! Thank you so much!
In my real test i just had to make an adjustment because x.id was used two (or more) times, so, just in case someone else need it, I did it using and because the key duplication:
// ...
->leftJoin('table x', ['and', ['x.id' => new \yii\db\Expression('t.x_id')], ['x.id' => $arrayIds]])

And it is generating this SQL:
... LEFT JOIN `table` `x` ON (`x`.`id`=t.x_id) AND (`x`.`id` IN (2, 4))


WIP: atril: init at 1.19.0

Motivation for this change
add atril from MATE desktop. Compiles fine on Fedora 25, but for some reason cannot display PDF documents yet.
Things done

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

[ ] NixOS
[ ] macOS
[X] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

On unstable NixOS it compiles and displays PDF ok.
Maybe you want to add yourself as maintainer.

Due to time concerns I wiould rather have someone else as maintainer :)

Alrighty, changed the description. Anything else :-)


Exclude gccode URL from test

Fix test on gccode which is on Intranet only.

@smellems, @remyb2canada proposed we could whitelist some of the known SaaS solutions that seem to be failing the tests like Youtube, Trello (today) as exceptions. We can set a review of these at certain intervals. Thoughts?


☠  Coda Daemon crashed.

2021-02-23 14:55:03 UTC [Warn] Shutdown before Coda instance was created, not saving a visualization
☠  Coda Daemon crashed.
The Coda Protocol developers would like to know why!
Please:
  Open an issue:
    <https://github.com/CodaProtocol/coda/issues/new>

  Briefly describe what you were doing and attach the crash report /root/.coda-config/coda_crash_report_2021-02-23_14-55-03.393135.tar.gz

Closing this issue as it is stale for a long time.


Casmnet 1258

Summary and Scope
Description:
Fix for:
[1.2.5-develop]
Add NMN network for 1.0 to ssh allowed into switches because of BGP DOCS in 1.0 allowing it.
Remove router ospfv3 from 1.0/1.2.
Fixed ACL, OSPF, BGP config files
Fixed test templates for ACL, OSPF, BGP

PR checklist (you may replace this section):

[x] I have run nox locally and all tests, linting, and code coverage pass.
[x] I have added new tests to cover the new code
[x] My code follows the style guidelines of this project
[ ] If adding a new file, I have updated pyinstaller.py
[x] I have updated the appropriate Changelog entries in readme.md
[x] I have incremented the version in the readme.md
[x] I have incremented the version in canu/.version

Issues and Related PRs

Resolves CASMNET-1258
Relates to Issue
Requires Issue

Testing
Tested on:  local NOX

List of virtual system tested.
List of physical systems tested.

Need review


Crash upon accessing anything from Lumos.getAllClasses()

Doing something like this:
let classes =  Lumos.getAllClasses().filter { b in
      return true
}

print(classes[0].description())

crashes with:
[ description] sent to deallocated instance 0x123195e58
The filtered array wasn't empty, sure thing.  Xcode 9.4.1.

Hmm thats odd, where are you calling the method from?

Just straight in didFinishLaunchingWithOptions.

Might be a runloop issue, try:
DispatchQueue.main.async {

    let classes =  Lumos.getAllClasses().filter { b in
        return true
    }

    print(classes[0].description())

}

@reenboog Do let me know in the next two days if that does not fix your issue.


Show npm version if package.json exist in current directory

Hello, I wan't to issue a feature request as title show. Would be nice to have that one. Let me know if that makes sense

@RusinovAnton hi. Do you mean npm -v or version of package in current directory?

@denysdovhan, I meant npm -v yeah, but version of the current package is pretty good idea too


Bug in retrieving the apiVersion with convention routing

Hi,
I'm trying to match candidates on actions rather than controllers in the ResolveController and i' m struggling with a strange bug.
I'm  trying to retrieve the versions of an action decorated with the MapToApiVersion attribute with the convention Routing.
For that, I 've changed the way of finding the matches in the ResolveController method of the ConventionRouteControllerSelector class. See below my linq request. I needed the ApiVersionActionSelector to find the actionDescriptors of a controllerDescriptor.
...
static ApiVersionActionSelector apiVersionActionSelector = new ApiVersionActionSelector();
...
static HttpControllerDescriptor ResolveController(IEnumerable<HttpControllerDescriptor> candidates, ApiVersion requestedVersion, Lazy<Exception> ambiguousException)
...
 var matches = candidates
                    .Select(candidate => new { ControllerDescriptor = candidate, ActionDescriptors = apiVersionActionSelector.GetActionMapping(candidate).SelectMany(a => a) })
                    .SelectMany(b => b.ActionDescriptors.Select(actionDescriptor => new { b.ControllerDescriptor, ActionDescriptor = actionDescriptor, Versions = actionDescriptor.GetApiVersions() }))
                    .Where(e => e.Versions.Contains(requestedVersion) && e.ControllerDescriptor.GetDeclaredApiVersions().Contains(requestedVersion));

Below my example :
     [ApiVersion("1.0")]`
    [ApiVersion("2.0")]
    public class PersonController : ApiController
    {
        [MapToApiVersion("1.0")]
        // GET: api/Person
        public IEnumerable<string> Get()
        {
            return new string[] { "V1", "V1" };
        }
    }

    [ApiVersion("2.0")]
    [ControllerName("Person")]
    public class Person2Controller : ApiController
    {
        [MapToApiVersion("2.0")]
        // GET: api/Person
        public IEnumerable<string> Get()
        {
            return new string[] { "V2", "V2"};
        }
    }

If i called http://localhost:52429/api/Person?api-version=2, the system is able to find the action, but if i called http://localhost:52429/api/Person?api-version=1 and after i call http://localhost:52429/api/Person?api-version=2, the system throws a ambiguousException.
After investigating, i found that the the method GetApiVersions(), doesn't work correctly, because the action Get of the V1 controller returns 2.0
Thanks.

The GetApiVersions() method is just a shortcut for the declared API versions on an action a la GetApiVersionModel().DeclaredApiVersions. The GetApiVersionModel() extension method for an action enumerates the attributes on the action and stores it in the associated property bag, which can be found in ApiVersionModel.cs.
It would seem you have duplicates or crisscrossed controller/action mapping in your query. While I won't rule out the possibility of a bug, I'd need evidence of this behavior without any of the modifications. The order in which services are called not only shouldn't matter, but will certainly happen out of order.
I'm not sure if I have a test for this exact scenario, but I would be very surprised to find this type of bug after so much time and adoption. It seems to be such an obvious flaw. In fairness though, we call them bugs for a reason and I won't rule out the possibility just yet.
One more vital thing to keep in mind trying to evaluate the API versions. The GetApiVersionModel is super useful for service authors, but the complete model is not assembled during evaluation. The full ApiVersionModel is not aggregated until a match has been made. Building the complete model for callers higher in the stack are one of the value propositions the framework provides, but also is where there are challenges in the implementation.
I struggled with that part of the implementation myself, which is why there are both unit tests and acceptance tests. I ran into a few issues that only surfaced when running through the entire Web API stack. However, trying to test and troubleshoot basic behaviors purely through the full stack is also quite daunting. I suggest starting with simple unit tests and then moving up to verifying end-to-end scenarios.  You can look at ApiVersionControllerSelectorTest.cs and ApiVersionActionSelectorTest.cs for ideas on how to setup new tests.

I believe this issue is resolved by the answers provided in #223 and the #230 feature request. Feel free to come back or reopen the issue if that's not the case.


tidevice安装成功后,CMD输入tidevice list 报错怎么回事呀??

List of apple devices attached
Traceback (most recent call last):
File "d:\python\lib\runpy.py", line 198, in run_module_as_main
"main", mod_spec)
File "d:\python\lib\runpy.py", line 87, in run_code
exec(code, run_globals)
File "D:\python\Scripts\tidevice.exe_main.py", line 7, in
File "d:\python\lib\site-packages\tidevice_main.py", line 687, in main
actionsargs.subparser
File "d:\python\lib\site-packages\tidevice_main_.py", line 82, in cmd_list
for dinfo in um.device_list():
File "d:\python\lib\site-packages\tidevice_usbmux.py", line 75, in device_list
data = self.send_recv(payload)
File "d:\python\lib\site-packages\tidevice_usbmux.py", line 46, in send_recv
with self.create_connection() as s:
File "d:\python\lib\site-packages\tidevice_usbmux.py", line 43, in create_connection
return PlistSocket(self.__address, self._next_tag())
File "d:\python\lib\site-packages\tidevice_safe_socket.py", line 95, in init
super().init(addr)
File "d:\python\lib\site-packages\tidevice_safe_socket.py", line 48, in init
self._sock.connect(addr)
ConnectionRefusedError: [WinError 10061] 由于目标计算机积极拒绝，无法连接。

#50

装itunes就可以了。


GDScript: Optimize match

Closes #63719.
Closes #75682.

Optimize match with jump tables whenever possible. I want to use two kinds of tables:

OPCODE_JUMP_TABLE_RANGE. Can be used for continuous (or with a little sparsity) ranges of integers (such as enums). For example, from 0 to 10, or from 2 to 12. Instead of doing N comparisons (in the worst case), the jump address is calculated using the value, offset, and size.
OPCODE_JUMP_TABLE_BSEARCH. Can be used to binary search in a constant array sorted at compilation. Suitable when the value is not an integer or the range is very sparse.

When using variable patterns, destructuring (and match guards in the future), the old bytecode will be preserved (or the branches will be divided into subgroups in which optimizations can be performed).
TODO
Parser

[x] (Bonus) Allow pass in match.
[x] Add warning REDUNDANT_PATTERN (multiple patterns with a wildcard pattern).
[ ] Add the ability to ignore warnings in match.

Analyzer

[x] Add error on type mismatch for typed variables. Reduces the impact of the non-obvious comparison difference from == operator described in #74462.
[ ] Add warning DUPLICATE_PATTERN.

Codegen, VM, disassembler

[x] Add opcodes OPCODE_JUMP_TABLE_RANGE and OPCODE_JUMP_TABLE_BSEARCH.

Compiler

[ ] Do not perform unnecessary operations if the type is known.
[ ] Optimize match with jump tables.
[ ] Remove unreachable branches?

Miscellaneous

[ ] Add tests.
[ ] Make benchmarks: PR vs master vs if-elif-else.

My project uses a lot of match in performance-critical sections, so it would be nice to know if I can expect a free speedup in them anytime soon. Is this PR likely to get in anytime soon?

Is this PR abandoned?

As far as I remember, I stopped because of #58878. I implemented a workaround with SafeVariantSort, but this did not seem elegant enough to me, since I had to create a separate table variant_vector_constants. Also, the compiler part is unfinished, but I think it is doable.