Import order affects the compile result when using faker and fastapi together

Nuitka version, full Python version, flavor, OS, etc. as output by this exact command.

python -m nuitka --version

2.5rc12
Commercial: None
Python: 3.12.7 (main, Oct  1 2024, 11:15:50) [GCC 14.2.1 20240910]
Flavor: Arch Python
Executable: /home/abc/.virtualenvs/test/bin/python
OS: Linux
Arch: x86_64
Distribution: Arch None
Version C compiler: /usr/bin/gcc (gcc 14.2.1).

How did you install Nuitka and Python

Use pip install -e . on develop branch (commit 2bd0a4f14) in a virtualenv.

The specific PyPI names and versions

With a brand new virtualenv, I only run pip install fastapi faker and pip install -e ., which results the following package installed:

python -m pip list -v

Package           Version     Editable project location Location                                                 Installer
----------------- ----------- ------------------------- -------------------------------------------------------- ---------
annotated-types   0.7.0                                 /home/abc/.virtualenvs/test/lib/python3.12/site-packages pip
anyio             4.6.2.post1                           /home/abc/.virtualenvs/test/lib/python3.12/site-packages pip
Faker             30.8.2                                /home/abc/.virtualenvs/test/lib/python3.12/site-packages pip
fastapi           0.115.4                               /home/abc/.virtualenvs/test/lib/python3.12/site-packages pip
idna              3.10                                  /home/abc/.virtualenvs/test/lib/python3.12/site-packages pip
Nuitka            2.5rc12     /home/abc/Repos/Nuitka    /home/abc/.virtualenvs/test/lib/python3.12/site-packages pip
ordered-set       4.1.0                                 /home/abc/.virtualenvs/test/lib/python3.12/site-packages pip
pip               24.2                                  /home/abc/.virtualenvs/test/lib/python3.12/site-packages pip
pydantic          2.9.2                                 /home/abc/.virtualenvs/test/lib/python3.12/site-packages pip
pydantic_core     2.23.4                                /home/abc/.virtualenvs/test/lib/python3.12/site-packages pip
python-dateutil   2.9.0.post0                           /home/abc/.virtualenvs/test/lib/python3.12/site-packages pip
six               1.16.0                                /home/abc/.virtualenvs/test/lib/python3.12/site-packages pip
sniffio           1.3.1                                 /home/abc/.virtualenvs/test/lib/python3.12/site-packages pip
starlette         0.41.2                                /home/abc/.virtualenvs/test/lib/python3.12/site-packages pip
typing_extensions 4.12.2                                /home/abc/.virtualenvs/test/lib/python3.12/site-packages pip
zstandard         0.23.0                                /home/abc/.virtualenvs/test/lib/python3.12/site-packages pip

Also supply a Short, Self Contained, Correct, Example

When using faker and fastapi together, the import order will affect the compile result. Specifically, if I import fastapi before faker, faker will result an error.
Here are some examples.
When using faker only, it works:
import faker
_ = faker.Faker(locale="en_US")

When using faker before fastapi, it works:
import faker
_ = faker.Faker(locale="en_US")

import fastapi

When using faker after fastapi, it doesn't work:
import fastapi

import faker
_ = faker.Faker(locale="en_US")

The error is:
Traceback (most recent call last):
  File "/tmp/test/test6.dist/test6.py", line 4, in <module>
  File "/tmp/test/test6.dist/faker/proxy.py", line 72, in __init__
  File "/tmp/test/test6.dist/faker/factory.py", line 44, in create
AttributeError: Invalid configuration for faker locale `en_US`

For the second and third case, nuitka both compiled 845 C files.
The error comes from faker package from here:
        if locale not in AVAILABLE_LOCALES:
            msg = f"Invalid configuration for faker locale `{locale}`"
            raise AttributeError(msg)

while AVAILABLE_LOCALES is imported from .config which is:
from importlib import import_module

from .utils.loading import find_available_locales, find_available_providers

DEFAULT_LOCALE = "en_US"

META_PROVIDERS_MODULES = [
    "faker.providers",
]

PROVIDERS = find_available_providers([import_module(path) for path in META_PROVIDERS_MODULES])

AVAILABLE_LOCALES = find_available_locales(PROVIDERS)

Perhaps this issue is somehow related to importlib? Since faker uses importlib to find modules.

Provide in your issue the Nuitka options used

python -m nuitka --standalone ./test6.py

File based module scans are wrong and not successful with Nuitka, but the path for normal Python in list_module ought to work with Nuitka.
If you provided the reports (and maybe compared them yourself), you might get closer to why it happens. I don't see a way other than if path is somehow becoming wrong.

It looks to me, as if you believe .config is compiled, and then it's not, because it's maybe only implicitly picked up by fastapi, and then has no dependencies resolved. If that's true, this would definitely be a weakness of the fastapi support for Nuitka. I am not yet sure how it would cause issues you see though.

I actually think I managed to reproduce this, and then I didn't anymore, after fixing an implicit dependency on attrs for the pydantic schema, not sure what's going on there.

So, what happens is that sys.frozen is set for multiprocessing, and remains set. We need multiprocess to handle things differently, but when the plugin kicks in and multiprocessing or anyio is loaded, this gets set and is not removed, and activates code paths that are intended for non-Nuitka, as the later will typically work properly, but certainly not always.
Removing that will be a bigger thing. You can add del sys.frozen as a workaround. I have changed the multiprocessing plugin to do that itself, and I will probably want to do it, such that it is not necessary. We can now just modify the source of multiprocessing on the fly to never have this set, but that will be a beauty thing.
Probably a bunch of software has never been compiled without that bug. But as can be seen, also a bunch is compiled without that, since faker is assumed to be working, but anything mixing in multiprocessing usage would have changed that to the worse, and it comes down to import order.

Thanks for your report, this is fixed on the factory branch, which is a development version under rapid development. You can try it out by going here: https://nuitka.net/doc/factory.html
Feedback on whether this is working is very welcome. Please do not share plans to do it; only confirm or deny that it is working.

This now on the develop branch and will be part of the 2.5 release.

Part of the 2.5 release I just made.


evolution: improve out-of-gnome use

Motivation for this change
Evolution didn't work.
Things done

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

[x] NixOS
[ ] OS 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/)
[x] Fits CONTRIBUTING.md.

Add evolution_data_server to propagatedUserEnvPkgs. Evolution needs
${gnome3.evolution_data_server}/libexec/evolution-source-registry to
be running to be able to find or create any account settings at all, and
it apparently doesn't know to start it if it's not in the user env.
Add libgnome_keyring and dconf to buildInputs, add
glib_networking and dconf to GIO_EXTRA_MODULES in the wrapper
script. libgnome_keyring appears to be necessary for evolution to
remember a password even for a single session, even if it doesn't get
added to the keyring permanently. dconf is necessary to persist
preferences. glib_networking is necessary to connect to mail servers.
Also fix wrapping libexec binaries, even though I'm not sure that
improves anything.

Closes #12756.

I don't know enough to give an opinion on this PR. Nevertheless, maybe wrapGAppsHook should be used.

That seems likely. I'll look into it, thank you!

wrapGAppsHook is pretty rad.
travis-ci didn't manage to build this because it takes too long, I guess

Looks good. Building locally, will merge if/when it succeeds. Planning to backport to release-16.09.

Pushed to master (240ca8bebed5b25d57a6992b27423a98e83c0777). Thanks. (Will wait a bit before backporting to release-16.09.)


"The image file may be corrupt or invalid" When attempting to open 4.6GB EXR file.

Does the viewer have some sort of file size limit? The image is definitely fine, I have tested it in Photoshop.

This limit looks like 32 bit pointer somewhere.

The entire process should be 64 bit including the DirectXTex and OpenEXR dependencies used for EXR loading. But perhaps there is a resolution limit somewhere - what resolution image is this?

The image size is 30720x12800.

Is this a RAM issue? Putting that much of a graphics bitmap into memory would be way more than 4.6GB from my experience.
If you upload it somewhere, I can test it for you. I have 64GB of RAM.

Potentially resolved by #49.


build: remove babel dependency

Closes #16

Removed the tslint stuff. Looking to make that a separate PR.

Oh and yes using eslint


jni error

Build fingerprint: 'Xiaomi/libra/libra:5.1.1/LMY47V/5.11.1:user/release-keys'
Revision: '0'
ABI: 'arm'
pid: 10250, tid: 10273, name: ImageListener  >>> org.tensorflow.tensorflowdemo <<<
signal 6 (SIGABRT), code -6 (SI_TKILL), fault addr --------
Abort message: 'jni_utils.cc:92 'asset' Must be non NULL'
r0 00000000  r1 00002821  r2 00000006  r3 00000000
r4 e29c4dd8  r5 00000006  r6 00000000  r7 0000010c
r8 e29c3ad4  r9 e29c39d8  sl 00000000  fp 00000000
ip 00002821  sp e29c3980  lr f7072e89  pc f709972c  cpsr 600f0010
backtrace:
#00 pc 0003a72c  /system/lib/libc.so (tgkill+12)
#01 pc 00013e85  /system/lib/libc.so (pthread_kill+52)
#02 pc 00014aa3  /system/lib/libc.so (raise+10)
#03 pc 0001131d  /system/lib/libc.so (__libc_android_abort+36)
#04 pc 0000f54c  /system/lib/libc.so (abort+4)
#05 pc 0053d03c  /data/app/org.tensorflow.tensorflowdemo-1/lib/arm/libtensorflow_demo.so
#06 pc 0053d1bc  /data/app/org.tensorflow.tensorflowdemo-1/lib/arm/libtensorflow_demo.so
#07 pc 0053d1d8  /data/app/org.tensorflow.tensorflowdemo-1/lib/arm/libtensorflow_demo.so
#08 pc 000746b4  /data/app/org.tensorflow.tensorflowdemo-1/lib/arm/libtensorflow_demo.so
#09 pc 00075594  /data/app/org.tensorflow.tensorflowdemo-1/lib/arm/libtensorflow_demo.so (ReadFileToProto(AAssetManager*, char const*, google::protobuf::MessageLite*)+1144)
#10 pc 00077bdc  /data/app/org.tensorflow.tensorflowdemo-1/lib/arm/libtensorflow_demo.so (Java_org_tensorflow_demo_TensorFlowClassifier_initializeTensorFlow+1084)
#11 pc 000a2511  /system/lib/libart.so (art_quick_generic_jni_trampoline+32)
#12 pc 00136c1c  /dev/ashmem/dalvik-main space (deleted)

In TensorflowClassifier.java i got an error at jni native function: "cannot corresponding JNI function Java_org_tensorflow_demo_TensorflowClassifier_initializeTensorflow..." Please help me :(

How did you resolve this error?
In TensorflowClassifier.java i got an error at jni native function: "cannot corresponding JNI function Java_org_tensorflow_demo_TensorflowClassifier_initializeTensorflow..."

Download android-yolo.pb (link is in README) and put it into assets folder

It is the link for android_graph.pb. I have downloaded it and I put it in the assets folder but the error is still there in TensorflowClassifier.java and ImageUtils.java.
Is it the same as android-yolo.pb?

Oh I was wrong, android_yolo.pb is correct

Can you please provide me the link? because when I open this link -
https://drive.google.com/file/d/0B2fFW2t9-qW3MVJlQ29LRzlLT2c/view
I get android_graph.pb and not android_yolo.pb

It's correct link. Maybe I remember wrongly or the repository owner changed the file name

Ohh okay. Actually the error is not removed. Do you know what will be affected if it is not corrected? And any idea about how to provide the definitions for JNI function?

So sorry but actually I don't know. I just put .pb file to assets and after that, the code run well

Ohh okay no problem. Can you tell me where to put the c++ files and header files so that the functions can be recognized?

I have the same problem. could please help us somebody?


fix: use join instead of resolve for paths related with fixture option

Problem
Documented behaviour:
testDir, default value: '~~/test'
fixture, "a fixture directory (under testDir)", default value: 'fixture'
rootDir, default value: <testDir>/<fixture>
All works as expected if setupTest is called this way:
setupTest({
  fixture: 'some-fixture'
})

rootDir === '<process.cwd()>/test/fixture/some-fixture' // true

But it will not work with leading slash:
setupTest({
  fixture: '/some-fixture',
})

rootDir === '/some-fixture' // true

path.resolve is called inside resolveRootDir which returns rootDir value.
Solution
path.join instead of path.resolve does the right job.
By the way, expectFileToBeGenerated and expectFileNotToBeGenerated have the same issue. Calling expectFileToBeGenerated('/index.html') will not check if the file exists inside generate.dir, but will look for it at system root. Again path.join solves the problem.
Elsewhere path.resolve works as expected and there is no need for any change. In the other hand, path.join is enough, nuxt.ts file would have one import less, and also path.join has shorter name, this makes bundle size smaller. Minor details, but it is fun to keep them in mind.
Alternative Solution
Update docs with a warning to avoid leading slash or to use absolute paths. Also possible.

I think it was an expected behavior and should revert this or @ricardogobbosouza. Depending on implicit relative paths is not good idea. We can instead support passing rootDir to setupTest to disable fixture addition.

@pi0 You are right. I should improve this PR. Better to leave join only in places where first argument is always absolute path. Otherwise resolve does better job.

@pi0 what's your idea about it ? I did not understand

We can instead support passing rootDir to setupTest to disable fixture addition.


Bootstrap perf

reducing the generated code for struct, if the field elements of the struct are of same primitive data type.
(Please fill in changes proposed in this fix)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)
Please review https://cwiki.apache.org/confluence/display/SPARK/Contributing+to+Spark before opening a pull request.

Can one of the admins verify this patch?


Update request: vimPlugins.cord-nvim 2024-07-19 → 2025-09-25

Package name: vimPlugins.cord-nvim
Latest released version:  2025-09-25

Current version on the unstable channel:2024-07-19
Current version on the stable/release channel: NONE

[x] Checked the nixpkgs pull requests

Notify maintainers

Note for maintainers: @redyf

Add a :+1: reaction to issues you find important.

I could look doing a pr that updates the package.

No worries :+1: https://github.com/NixOS/nixpkgs/pull/344478

Nvm all good if you need me to I can try and test it on macos.

I also suggest doing an update script so the @r-ryantm bot updates based on new commits on the new branch.

Nvm all good if you need me to I can try and test it on aarch64-darwin.

That would be great 🙏

PR: https://github.com/NixOS/nixpkgs/pull/344478 Merged.


fix: Pre-selected values cannot be deleted from FluentAutocomplete

🐛 Bug Report
When you have a predefined value assigned to a bound property of a FluentAutocomplete it cannot be deleted unless you have selected another value.
This used to work in 4.10.2 but it does not in 4.10.3
💻 Repro or Code Sample
I've created a small sample repository which can be found here:
https://github.com/MarvinKlein1508/FluentUI-Autocomplete-issue
🤔 Expected Behavior
You should be able to delete the predefined value, just like it in 4.10.2.
😯 Current Behavior
The value can only be deleted whenever you select another value first.

When I remove the added if block here everything works for me just like before.
https://github.com/microsoft/fluentui-blazor/blob/78ff71e3d8514c6a8b59be86db1da4b907b10b3c/src/Core/Components/List/ListComponentBase.razor.cs#L539

Or alternativly:
if (_selectedOptions.Count != 0 || SelectedOptions?.Any() == true)

See #2891


feat!: install polaris collection via git src

Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[x] feat: non-breaking change which adds new functionality
[ ] fix: non-breaking change which fixes a bug or an issue
[ ] chore(deps): changes to dependencies
[ ] test: adds or modifies a test
[ ] docs: creates or updates documentation
[ ] style: changes that do not affect the meaning or function of code (e.g. formatting, whitespace, missing semi-colons etc.)
[ ] perf: code change that improves performance
[ ] revert: reverts a commit
[ ] refactor: code change that neither fix a bug nor add a new feature
[ ] ci: changes to continuous integration or continuous delivery scripts or configuration files
[ ] chore: general tasks or anything that doesn't fit the other commit types

Please indicate if your PR introduces a breaking change

[x] Breaking change: fix or feature that would cause existing functionality to not work as expected

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the Contributing doc
[x] I have read the Creating and reviewing pull requests at LinkORB guide doc
[ ] Lint and unit tests pass locally with my changes
[ ] I have added/updated necessary documentation in the README.md or doc/ directories (if appropriate)

Converted to draft PR, as we are not going to switch to git dependency resolution model right now.


Why is the number of threads limited to 12?

I'm asking since the headless renderer for my VFX compositing system will often run on 32 or 64 core machines and the images are typically at least 4k. I.e. at 256x256 sized blocks that's ~160 blocks for a 16:9 image. Why limit this to 12 cores?
OIIO uses all threads=cores by default and they do get maxed out on my 24 core desktop when processing 4k images.
And on that note: would you accept a PR that allows a number on ThreadingPolicy::Adaptive that means: reserve this many cores from all available? Or alternatively (to not break the API) ThreadingPolicy::AdaptiveReserve.
I.e. on a 32 core system. Adaptive(0) = 32 cores but Adaptive(3) = 29 cores (32-3). This is very useful when running multithreaded code on a DCC app where the user interacts with a UI or the like.
When the CPU is maxed out the system can be come unresponsive and sometimes even the mouse cursor does not move. Reserving/not using 1--2 cores usually solves this.

Actually this is pretty good question.
But in my common practice just increase amount of threads is not adding any gains when threads count goes somewhere up to 10-12.
Sure, I can accept PRs :)
Also it will be quite interesting if you'll launch benchmark on such machine just to fill my curiousity :)

On that note, I would also rename get_threads_count() to thread_count() in my PR.
The get_ prefix is against Rust API naming policies and threads_count is wrong English, the counted thing is always singular. :grin:.
Is that ok?

That’s fine

See #6.

I believe this is resolved.


Installation fails  // -s: 36: export: pms_header(): bad variable name

Script
nginx-proxy-manager
Describe the bug
Installation fails on a fresh LXC with Ubuntu 20.04
System info:

Proxmox Version: 8.2.2
Ubuntu 20.04

Screenshots
First this one

and after some more tryings...

Additional context
Just trying to install in a fresh LXC with Ubuntu 20.04, any idea?
Thanks for your work and help.

After more than 30 retryings, with same number of crashes at random points of the installation process, I finally succeded.
I suspect that's going to be the same with the update process.


Wikis that are unindexable and uneditable may not expose a raw URL

Check for handling of this case and fallback to GitHub.
https://gist.github.com/nnja/9136152c163091614e70defcf3753d06#gistcomment-4469816

I think this is was a false concern. The example mentioned doesn't work because the page is gone.


fix: removing skip from shrexeds tests and updating them to current logic

@walldiss pointed out that there are several tests in shrexeds that were being skipped. This was from the removal of retry logic inside the protocol itself, but now I have unskipped the tests and adapted the logic

lint @distractedm1nd

ah probably from the auto merge, will fix

Codecov Report

Merging #2075 (ad120cf) into main (6bd5adc) will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2075      +/-   ##
==========================================
+ Coverage   55.65%   55.70%   +0.04%
==========================================
  Files         208      208
  Lines       13161    13161
==========================================
+ Hits         7325     7331       +6
+ Misses       5090     5086       -4
+ Partials      746      744       -2

see 5 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


problem with window GLFW_DECORATED = true

Window with decoration has wrong dimensions of frame buffer (probably).
Very simple example:
System.out.println("LWJGL version : " + org.lwjgl.Version.getVersion());
System.out.println(" GLFW version : " + glfwGetVersionString());
glfwInit();
glfwDefaultWindowHints();

glfwWindowHint(GLFW_DECORATED, GLFW_TRUE);  <-- PROBLEM IS HERE

long window = glfwCreateWindow(500, 500, "", 0, 0);
glfwMakeContextCurrent(window);
GL.createCapabilities();
glfwShowWindow(window);
glfwSetWindowPos(window, 1200, 200);
glClearColor(0, 0, 0, 1);

int[] width = new int[1];
int[] height = new int[1];
glfwGetFramebufferSize(window, width, height);
System.out.println("FrameBuffer size : width = " + width[0] + ", height = " + height[0]);
glfwGetWindowSize(window, width, height);
System.out.println("     Window size : width = " + width[0] + ", height = " + height[0]);
while (true) {
	glClear(GL_COLOR_BUFFER_BIT);
	glfwGetFramebufferSize(window, width, height);
	glViewport(0, 0, width[0], height[0]);
	GL11.glColor4f(1, 1, 0, 1);
	GL11.glBegin(GL11.GL_QUADS);
	GL11.glVertex2f(-0.9f, -0.9f);
	GL11.glVertex2f(-0.9f, 0.9f);
	GL11.glVertex2f(0.9f, 0.9f);
	GL11.glVertex2f(0.9f, -0.9f);
	GL11.glEnd();
	glfwSwapBuffers(window);
	glfwPollEvents();
}

PROBLEM:

WITHOUT DECORATION FOR REFERENCE:

Tested on:
WINDOWS 10 Pro 64bit, GF 960M,
Possible same issue: #1148

@NkD What is GL.createCapabilities();?

Hi guys,
I gave this a try again this week(re-enabling decoration), and apparently it's fixed.
I didnt change anything code wise, but i updated my nvidia driver.. so maybe a driver issue after all.

I confirm that the issue has disappeared after updating the NVIDIA drivers. In version 390.77 everything works as it should. Problem solved. Heh.


vert-1 crashing

I was working on vert-1 and it stopped recompiling when I'd save. I refreshed the browser (Chrome) and the following error was shown on screen and logged to the command line. I'm using vs code as my text editor fwiw. I restarted shader-school but the error returned immediately. Finally, I deleted the shader file and repopulated the directory. That seems to have fixed it

Error: did not use all tokens while parsing file: C:\Users\Keenan Woodall\AppData\Roaming\npm\node_modules\shader-school\exercises\vert-1\index.js
at module.exports (C:\Users\Keenan Woodall\AppData\Roaming\npm\node_modules\shader-school\node_modules\glslify-stream\node_modules\glsl-parser\lib\expr.js:121:11)
at parseexpr (C:\Users\Keenan Woodall\AppData\Roaming\npm\node_modules\shader-school\node_modules\glslify-stream\node_modules\glsl-parser\lib\index.js:595:9)
at parse_expr (C:\Users\Keenan Woodall\AppData\Roaming\npm\node_modules\shader-school\node_modules\glslify-stream\node_modules\glsl-parser\lib\index.js:572:14)
at Stream.write (C:\Users\Keenan Woodall\AppData\Roaming\npm\node_modules\shader-school\node_modules\glslify-stream\node_modules\glsl-parser\lib\index.js:182:18)
at Stream.stream.write (C:\Users\Keenan Woodall\AppData\Roaming\npm\node_modules\shader-school\node_modules\glslify-stream\node_modules\glsl-parser\node_modules\through\index.js:26:11)
at Stream.ondata (internal/streams/legacy.js:17:31)
at Stream.emit (events.js:223:5)
at drain (C:\Users\Keenan Woodall\AppData\Roaming\npm\node_modules\shader-school\node_modules\through\index.js:36:16)
at Stream.stream.queue.stream.push (C:\Users\Keenan Woodall\AppData\Roaming\npm\node_modules\shader-school\node_modules\through\index.js:45:5)
at token (C:\Users\Keenan Woodall\AppData\Roaming\npm\node_modules\shader-school\node_modules\glsl-tokenizer\index.js:61:14)

I had this same issue. Turns out it was because I copy pasted some code from the internet which used a different character for -. I solved this by pasting my shader into this online linting tool to see the problem. https://evanw.github.io/glslx/


Duplicate roles/scopes after adding role permissions in admin portal

Description:
In Admin Portal, after adding role permissions to a role using role alias, duplicate entries of the role alias are created every time you navigate between the pages of the table. This would go away when refreshed but will appear again when you move between pages.
Affected Product Version:
3.2.0
@tmkasun @chanaka3d

Unable to reproduce in latest pack hence closing the issue.


Does this work on L-2 norm based attacks?

I didn't test L2 norm attacks, but I think the results should be the same. The only change we need to make is to replace linf_sgd.py in /attacker directory with the standard SGD in pytorch.


Resolve oVirt IP addresses

Since version 4 oVirt rejects connections that use directly the IP address instead of the host name. This is a side effect of the new SSO authentication mechanism. That means that when a oVirt provider is added using an IP address it will not work. Users don't do this frequently, but the system does it automatically when the provider is added using the discovery mechanism, as this mechanism uses IP addresses only. To avoid this issue this patch changes the provider so that when it receives an IP address it tries to convert it to the corresponding fully qualified host name.
There may be situations where the user really wants to use an IP address, and not the host name. For those cases this patch also introduces a new resolve_ip_addresses setting that can be used to disable resolving. This setting will be stored in the config/settings.yml file, and its default value will be true:
:ems:
  :ems_redhat:
    :resolve_ip_addresses: true

https://bugzilla.redhat.com/1417757

@masayag @borod108 @durandom please review.

 CF fails to discover RHV 4.0 - IP issue

@durandom I requested a release note in the bug. Please review it and tell me if something else is needed in order to trigger docs.

The only change in the new version of the pull request is moving the require 'resolv' to the top of the file.

cool, thanks @jhernand - now waiting for @oourfali thoughts on the config setting name and the docs.

@jhernand could you re-base on current master, amend the last commit or close/open the PR to trigger the CI?

Rebased, no changes.

@jhernand This isn't a clean cherry-pick due to some refactoring done on master branch. Can you make an Euwe-specific PR (referencing this one) or suggest other PRs to backport.

@simaishi Yes, I'll prepare an Euwe-specific PR.

@simaishi Here is the back-port for Euwe: ManageIQ/manageiq#14183. Please don't merge it yet. I will verify that it works correctly in the Euwe branch and then I will let you know.

Backported to Euwe via https://github.com/ManageIQ/manageiq/pull/14183


fix: remove fouce effect styles from default theme

Includes:

Removing FOUCE effect style,
Adding FOUCE effect warning to docs.

Closes #344.

I see UI Test of Drawer component is failing because of the prioritization of iframe in it. I think we can ignore it for now by delegating fixing this to #330


Update reference_policies_examples_aws_deny-requested-region.md

Issue #, if available:
Description of changes:
I added more information to make it more obvious to the reader what this does in relation to the RequestedRegion condition.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks so much for submitting a PR! We've made updates to the docs based on your suggestion.


Integrate gradle license plugin

Adds gradle license plugin and generated license headers.
Resolves #13

Could you add a NOTICE file also? With something like "This software is based on intellij-lsp (link) by author-name (link) " and other words required by apache license?

Codeclimate failures are occured due to a licence plugin issue. (https://github.com/hierynomus/license-gradle-plugin/issues/166)

You can config this behavior by license.mapping in gradle. will you give it a try?

Sure will do. But IMO this should be fixed by the plugin itself. Anyway will configure for now.

LGTM!
I'll try to tweak codeclimate check.


fix(orchestrator): revert remove field 'variables' from ProcessInsanceDTO (#2229)

#2229 broke the build

/lgtm


Move keyboard focus to runtime environment when triggering code to run

In the test page the user can trigger the code to run by pressing Shift+R from the Blockly workspace, or by pressing the "run code" button. In either case the p5.js context should receive following key-presses.
For testing purposes I'm making an "on space pressed" block that can trigger actions in the p5 context.
In the broader scope, most blockly-based applications have a code workspace and a runtime. Many of them expect interaction with the runtime: micro:bit has a simulator that can be clicked and tilted, code.org has clicking in flappy bird, and so on. That means we need to have a standard way to move between the workspace and the runtime, since we don't want keypresses aimed at (e.g.) playing a game in the runtime to actually move blocks around in the blockly workspace.

See https://github.com/google/blockly-keyboard-experimentation/discussions/49 for the broader discussion and https://github.com/google/blockly-keyboard-experimentation/issues/27 for the inverse action.


improve downtime during promotion

we have downtime whenever we create a new prod and switch to it. Outgoing connections seem relatively quick, incoming takes ten minutes or more for some reason. The new prod is not finding a valid device and is sending channel unavailable. This is odd because the same device finds asterisk for outgoing connections.
I don't think asterisk can send a message to the SIP device if it isn't registered. We need to wait for the device to tell asterisk it is there.
Don't require registry as much? Decrease the timeout so the device needs to re-register more often?
Use Route53 for faster DNS updating?

Run our own nameserver on the VPN and update that explicitly when we promote?
Use some kind of frontend just for SIP connection handling, like http://www.opensips.org?

how do the people who know what they are doing do this? A load balancer of some kind?
all incoming/outgoing connections are to frontend server
stand up new asterisk server without decommissioning existing server
route all new incoming connections to new asterisk server
wait for existing incoming and outgoing connections to drain
decommission existing asterisk server
OpenSIPS looks active


When using two tabs, a logout on tab A doesn't affect tab B

Describe the bug

Open A2, login
Open something (Admin) in a second tab
Logout in tab 1
Nothing happens in the second tab, and session refresh continues to work (although it's using the new token now, since there's only one store)

this work has been deprioritized, so closing.


Build example pages in Sanity to use for testing

In order to have a stable environment for testing, we need to set up one or more example pages in Sanity

[ ] Identify what kind of pages are needed
[ ] Build the pages
[ ] Document the pages

Related to #419 and to some degree #418.

Is this still relevant @nilsml

A test dataset could be used for this


Fixed #34983 -- Deprecated django.utils.itercompat.is_iterable().

See ticket-34983.

I did a search for try-iter-except cases not using is_iterable() and found the following:
django/core/serializers/base.py:347:        pks_iter = iter(field_value)
django/http/multipartparser.py:675:        iterator = iter(stream_or_iterable)
django/http/response.py:485:            self._iterator = iter(value)
django/http/response.py:495:            return iter(self.streaming_content)
django/template/defaultfilters.py:691:                        iter(next_item)
django/urls/resolvers.py:737:            iter(patterns)
django/utils/http.py:62:                itr = iter(value)
tests/prefetch_related/tests.py:447:            iter(obj_iter)

I'm not sure whether we want to switch some of these to use isinstance(..., Iterable) also?

I did a search for try-iter-except cases not using is_iterable() and found the following:
django/core/serializers/base.py:347:        pks_iter = iter(field_value)
django/http/multipartparser.py:675:        iterator = iter(stream_or_iterable)
django/http/response.py:485:            self._iterator = iter(value)
django/http/response.py:495:            return iter(self.streaming_content)
django/template/defaultfilters.py:691:                        iter(next_item)
django/urls/resolvers.py:737:            iter(patterns)
django/utils/http.py:62:                itr = iter(value)
tests/prefetch_related/tests.py:447:            iter(obj_iter)

I'm not sure whether we want to switch some of these to use isinstance(..., Iterable) also?

Don't know if it's worth doing :thinking: There are also some hasattr(..., "__iter__") checks. We can discuss this separately.

We can discuss this separately.

That's fair. Will open a follow up PR.

@ngnpope Thanks :+1:

We can discuss this separately.

That's fair. Will open a follow up PR.

See #17523.


Disallow near-identical words

Don't want multiple near-identical words appearing in the word search
Already take one step in this direction by disallowing words that are superstrings of other words
However this does not capture cases of subtly different spelling e.g. color/colour, ebonise/ebonize, etc
Should add a Levenshtein-distance metric enforcing edit distance be > 1

Not enough to limit to Levenshtein-distance > 1, need to also exclude words with share too-high proportion of their letters (or something equivalent to this)
Nearest neighbors for oligodendrocyte include:

oligodendrocytes : excluded by substring-matching only if we include seed word
oligodendrocytic : not excluded
Oligodendrocyte : excluded by downcasing equivalence with oligodendrocyte
oligodendroglial : excluded as superstring with oligodendroglia
oligodendrogenesis : not excluded
oligodendroglia : not excluded

Fixed for simple cases via addition of Levenstein constraint + 70% matching constraint. However still wouldn't for the long suffixes on "oligodendrocytes". Marking as fixed until otherwise noted

Fixed for simple cases via addition of Levenstein constraint + 70% matching constraint. However still wouldn't for the long suffixes on "oligodendrocytes". Marking as fixed until otherwise noted

Still has issues with near-identical short words: "marked" vs "marking", "time" vs "timing"
Use an out-of-the-box stemmer to identify base words and remove duplicates that way

Still has issues with near-identical short words: "marked" vs "marking", "time" vs "timing"
Use an out-of-the-box stemmer to identify base words and remove duplicates that way

Add word-length constrain to Levenstein distance to only apply to words of length ≥5. Currently it's misfiring on e.g. "time" vs "tide"

Add word-length constrain to Levenstein distance to only apply to words of length ≥5. Currently it's misfiring on e.g. "time" vs "tide"

Added Levenstein min-length and stemming


Define block size more specifically

This needs to be clarified for implementors to use blocks correctly. Should it be equal to req. size or should it be a usable size (>= req. size)?

Additionally, if the block size is an exact size, should we also store the usable size somewhere? Is that up to the allocators themselves?

Block size will be the requested size.


Add function to get size of a Texture

I want a function to get the size of a texture so I can implement shaders to do general sprite sheets. Right now I have to hard code in the expected dimensions of the texture or require the user to specify, but that's not really ideal.

Can you take a look at this function? Thanks.

does this still work with the new updates to 0.15?

This makes sense. Will work to integrate into next release.


DOC: Note that morlet and cwt don't work together

See http://dsp.stackexchange.com/a/18104/29
http://stackoverflow.com/q/23730383/125507

by the way, there's an apparently-abandoned CWT implementation at
http://www.phy.uct.ac.za/courses/python/examples/moreexamples.html#wavelet-analysis-continuous-wavelet-transform
and a slightly-modified mirror at https://github.com/emanuele/cwt/blob/master/Wavelets.py that might make sense to take over as part of scipy.signal?  it has more wavelet functions built-in than any other implementation I've found, and is BSD licensed.

Merged, thanks @endolith

The link to a CWT implementation above should probably be added to a new or existing issue on CWT to not lose it. @aaren or @Dapid may be familiar with it already.

Actually I found aaren's wavelet implementation today that might be as good or better, I haven't installed it yet https://github.com/aaren/wavelets

I'd not seen the code linked above before. There are a number of CWT implementations around and they mostly end up looking quite similar (including mine). Not sure about licensing with others but I can say that mine was written from scratch.
signal.cwt is due a sort out, but I'm not sure how much we want to think about api compared to the planned scipy.wavelets (currently pywt).

Pywt is only discrete wavelets though, right?

That's right, pywt is only discrete. It might make sense to keep CWT separate and in signal, but I'm not sure on that.


test: move site info header creation to function and write unit test for function

Summary
As per request, I moved some logic to its own function and added a unit test.
I'd love to get input! Also on the style that I did it in. For example, would it have been better in its own file? (if so, how do we want to structure it? 🤔) and what do we think of the naming. (is it creating a header? or would it be better to name it createSiteInfoBase64?

For us to review and ship your PR efficiently, please perform the following steps:

[x] Open a bug/issue before writing your code 🧑💻. This ensures we can discuss the changes and get feedback from everyone that should be involved.
[x] Read the contribution guidelines 📖. This ensures your code follows our style guide and
passes our tests.
[x] Update or add tests (if any source code was changed or added) 🧪
[x] Update or add documentation (if features were changed or added) 📝
[x] Make sure the status checks below are successful ✅

A picture of a cute animal (not mandatory, but encouraged)

Small note: we should prefix all commits with chore:, fix: or feat:. In this case, it could probably be a chore:. One thing to note is that only fix: and feat: commits will generate a release.


Avoid reporting EPC on unannotated using declarations

fixes https://github.com/microsoft/TypeScript/issues/57836

@typescript-bot test this


[Suggestion] Make the armour interfaces support capabilities

It would be easier to add in support for other mods which add types of armour which would work for breathing/spacesuit/etc if you also supported having the interfaces added via a capability.
Where you do this:
https://github.com/zmaster587/AdvancedRocketry/blob/MC1_10/src/main/java/zmaster587/advancedRocketry/atmosphere/AtmosphereVacuum.java#L54-L60
you could also add a check for if the item has the appropriate capability, then use the capability for the IFillableArmor instead of the item itself.  This will allow adding support for other mod items by simply adding the capability to the item itself.
Here is an example of where I use capabilities for something similar:
The Interface in question:
https://github.com/Thutmose/Thut/blob/1.9/Misc/ThutWearables/src/main/java/thut/wearables/IActiveWearable.java
Registering a default application of the capability:
https://github.com/Thutmose/Thut/blob/1.9/Misc/ThutWearables/src/main/java/thut/wearables/ThutWearables.java#L110-L150
An item that directly uses the interface (uses the super for the interface, but IWearable is the main needed thing in my case, could be IActiveWearable instead):
https://github.com/Thutmose/Thut/blob/1.9/Misc/ThutBling/src/main/java/thut/bling/ItemBling.java
A support class to make other mod items use the capability:
https://github.com/Thutmose/Pokecube/blob/1.9/Pokecube Addons/src/main/java/pokecube/compat/wearables/WearableCompat.java
And where the capability or item itself is checked for use:
https://github.com/Thutmose/Thut/blob/48221b1be029329308cca51d533a0feae79504f3/Misc/ThutWearables/src/main/java/thut/wearables/EnumWearable.java#L43-L46
I should probably mention why this would be relevant to me:
I have a feeling that at some point pokecube, or one of the addons for pokecube, will be adding in aqua/magma armour, which canonically can be used as space suits, and I would like to be able to make them actually viable space suits when used with this mod, and this would be the most convenient way to do so, as if it is added by an addon, I might not be able to add the interface to the items themselves and use @Optional notation to make it not a hard dependency.

Added the Capability zmaster587.advancedRocketry.api.capability.PROTECTIVEARMOR
The Capability uses the IProtectiveArmor interface where the method protectsFromSubstance(IAtmosphere atmosphere, ItemStack stack, boolean commitProtection); returns true if the atmosphere is to have no effect on the player.

Thanks, when those armours get added, I will definitely see about getting them supporting this :D


ResourceExhaustedError : OOM when allocating tensor with shape

The problem
I try to compute a very costly loss function using FFT2D, rolling of the tensor and neural network with 3 layers.
You can find the Python script here : TestFFT2D_2.py
My loss function is the sum of 448 terms. Each of this term is computed with a rolling of the initial tensor, FFT2D, a multiply of two tensor and a IFFT2D. That is why it is very costly and why it needs a lot of memory.
You can find the end of the error message here :
MemoryError.txt
Do you know why the GPU is not able to deal with a memory demanding code ? Do you know a way to avoid this error message even if we need to pay a computing time cost ?
Do you think I need to do a feature request for a better management of the GPU memory ?
System information

**Have I written custom code **: TestFFT2D.py
OS Platform and Distribution: Linux Ubuntu 16.04
TensorFlow installed from: Source
TensorFlow version: 1.2
CUDA/cuDNN version:
Cuda compilation tools, release 7.5, V7.5.17 and  cuDNN  5 with the GeForce GTX 680
GPU model and memory: Tested on GeForce GTX 680 with 2Go
** Python version**: Python 3.6.1 |Anaconda 4.4.0 (64-bit)|
Exact command to reproduce: python TestFFT2D_2.py

Warning: As this doesn't appear to be a bug with Tensorflow, the devs may ask for this to be moved to Stack Overflow.
Having a quick look at your code and your hardware I think that it's down to not having enough GPU memory. 2GB is really not very much. Can you please run nvidia-smi alongside your code to see whether anything else has a lock on your GPU memory? Perhaps something else is using it?

As far as I know, Tensorflow smartly manages the GPU memory and has ways of dealing with these kind of issues. However in my experience I have found that I will get an OOM error when something else is using the GPU and there isn't enough memory to even get started/initialised. Unfortunately I think that the answer is you need more memory or check that it's free. You'll probably get more luck on Stack Overflow.

This question is better asked on  StackOverflow since it is not a  bug or feature request. There is also a larger community that reads questions there. Thanks!


Feature/update readme badges

Description of change
Updating the README.md badges to point to the correct repositories.
Manual QA steps

ci/cd pipelines pass
Quick run of the pipeline in a local repo. poetry run tox

Risks

Low

Rollback steps

revert this branch

Wrong Repo.


Simplification and rebuild of "Create Event" form

Business Value
Since 90% of user events are very simple 1 ticket type events. Our create event form should prioritise that, and allow some minor complexity as a secondary option
To-Do

[x] Review wireframes and test
[x] Research sensible defaults
[x] Look at recurring events vs term and usage and value
[ ] Custom question, allow a text entry and a text response. No data types.
[ ] Move any version of sessions under "advanced"

Wireframes
https://share.goabstract.com/08aa190c-d579-498e-908a-b86a96346eaf

Linked Issues

[ ] #1189
[ ] #1061
[ ] #1062
[ ] #1059

Question from Slack:
Design for the date: datepicker vs 3 number fields ?
Design for the hours: hourpicker vs 2 number fields ?
What are the default values for the options from the current create event form? (notify on applicant and such)

From https://github.com/CoderDojo/community-platform/issues/1259#issue-433321614, can you confirm that "Any unexpected ticket (coming from the old event UI) should be resetted to the 2 default ones" ?

Design for the date: datepicker vs 3 number fields ?
I don't really mind here. Are you saying 3 number fields is better or 3 number fields is easier (both valid)
Design for the hours: hourpicker vs 2 number fields ?
Same as above. Which do you rather and why. I just went with common styles used in google calendar.

From #1259 (comment), can you confirm that "Any unexpected ticket (coming from the old event UI) should be resetted to the 2 default ones" ?
Asked for clarification on that ticket

Number fields are annoying for international, because of the ordering. However, date pickers are a generally hated component when it comes to UX.
Same for time, annoying due to PM/AM only being an english thing.
Thoughts @DanielBrierton ?

Talked about in Slack. Particularly gregwood confirmed that

date pickers (calendar dropdown type things) are dead annoying, inaccessible, hard to use on a phone, etc
three fields for day, month and year are way easier to use. the month should be the names, not numbers
maybe consider separating the Event date label heading into two: Event date and Event time

So I'll take another run at a version of the wireframe tomorrow.

Bump on

What are the default values for the options from the current create event form? (notify on applicant, ticket approval, public event)

Notify on applicant (false)
ticket approval (false)
public event (true)

Make sense?

We've looked into the librairies, and it seems that https://chronotruck.github.io/vue-ctk-date-time-picker/ is a decent solution if we're going the library way.
My only issue with normal html fields comes down to handling PM/AM for time, and offering an experience which ease the input.

This is basically what I'm thinking. A seperate date selector (as it's a single date not a range), and a time range selector. Does this work with the above library. Splitting them

Natives splitting are documented here :
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/time
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/date
That might be an issue since native input for date/time don't work on IE. We can validate it, but that'll be an issue presenting the error message.
For vue-ctk-date-time-picker, the example for both scenario are displayed on the exact link I sent on the previous message.
In the end, this doesn't describe the issue I'm facing @conoramurphy : what are we expecting the user to do with those inputs? Are they supposed to

Enter> scroll day > tab or click >scroll month > tab or click > scroll year
or
Enter, click day based on a visual calendar
or
Enter>type day>automatic slash added>type month>...
or ..?
Notice that for the native inputs, it's up to the browser how to display it. We have few to no capacity to change it.

The lack of Safari support would be more a blocker for native for me.
For the library can you point me to where that does a time range picker by itself?
Ideally the calendar picker would be

click day based on a visual calendar.

But I'm not overly opinionated here once it's clean and we're not making them do anything they shouldn't need to.

Apologies, I didn't understand you wanted a time-range picker. This doesn't exists, not in native, not in this lib either.
It's going to be 2 fields in both scenario.

Ok, then I'm happy with trying three different inputs.
A
[Calendar picker] [Start time picker] to [End time picker]

It's observed as working. Waiting until the end of August for monitoring results.

Statistical review
72.3% of events created on Zen in Sep/Oct are with the new form. However of that 27.7% using the old form 66% could use the new form. Leaving 91% of events accounted for functionally with the new form.
The percentage of new clubs using events has gone up nearly 10% (depending on how you calculate it).
Review of the non-converted
Out of the last 50 events created using the old form (all of these are in October)


Tslint Configured Code

Remove Package.lock.json from  structure and add this to .gitignore files as it will create on the fly in our structure.


Respond to feedback

Two changes here:

Fixes #20488  Target type new requires parentheses.
Add Extension GetEnumerator both to the article, and publish the speclet for the feature. See https://github.com/dotnet/docs/pull/20446#pullrequestreview-483029684

The link in the proposal is fixed in dotnet/csharplang#3865


Naming of NRTThreaded for novelty slice is inconsistent

All the other algorithms have:
NRTThreaded[Whatever] but novelty slice is NRTThreading[Whatever]. Compiler picked it up when trying to use it but you might want to change this for consistency with the other algorithms if that's your jam.

Cheers. One day :-D


Improve UI Variables hide datasource and add dependency from prometheus monitored instances for ClickHouse_Queries_dashboard.json

Improve UI for ClickHouse_Queries_dashboard.json

set basicAuth: false and usePost: true by default for all installed ClickHouse datasources

Signed-off-by: Eugene Klimov<EMAIL_ADDRESS>

What is the reason to have variable named 'exported_namespace' and not just a 'namespace'? Any name conflict?

@alex-zaitsev expected_namespace it same variable name which we use in Main Prometheus dashboard and it came from directly from metrics_exporter


Stop inserting fake OSM tags early during importing

This PR cleans up the import process a bit. Previously, when we make the initial pass over raw OSM data, we'd look at the sidewalk and parking tagging on every road. When tags were missing, based on different bits of config, we'd insert fake tags (including a made-up abst:sidewalks_inferred = true to mark the assumption). Now, we leave the OSM tags alone, and instead do this inference right before going through the tags -> lane specification process -- both the current one and the experimental osm2lanes one.
This cleanup helps with the osm2lanes cutover (https://github.com/a-b-street/osm2lanes/issues/248) by making the old and new code-path do the same kind of last-minute tag transformation. This also helps with #87, by grouping strange old assumptions about footway and shared-use paths in less places in the code.
The big behavioral change with this PR is removing some roads we were previously importing as shoulders+bike lanes. All of those cases weren't being represented very well, so removing them before the next step seems fine. Examples are https://www.openstreetmap.org/way/923062091 (a highway=track) and https://www.openstreetmap.org/way/539534605 / https://www.openstreetmap.org/way/573276662 (highway=pedestrian that definitely shouldn't be a bike lane-focused representation).
I'm regenerating all A/B Street data with this change. I don't expect any new problems after some initial testing, but I'll hold off on merging until things are done there.

Lots of mini cleanups is cool!

No issues in A/B Street land, so merging this!


Errorneous link in docs

NavbarSubmenu links forwards to
https://windows-ui.github.io/docs/navbarSubmenu.html
but page is at
https://windows-ui.github.io/docs/navbarsubmenu.html

Hi @pjylkka,
Thanks for letting me know :)
Issue has been fixed for the NavbarSubmenu link.


Doesn't work with PDRPC

Running wine-discord-ipc-bridge with Project Diva Megamix+ (with the PDRPC mod) doesn't pass the rpc status to Discord
PDRPC doesn't detect Discord
[PDRPC]: Waiting for Discord...

My launch options: WINEDLLOVERRIDES='dinput8.dll=n,b' gamemoderun mangohud /home/e404nnf/wine-discord-ipc-bridge/winediscordipcbridge-steam.sh %command%

Solved it, if you have ShareDiva, uninstall it, then put the winediscordipcbridge-steam.sh right before gamemoderun, not right before %command%


WIP: HttpsConnectionMiddleware

Porting the HttpsConnectionAdapter to HttpConnectionMiddleware. There are a few things that are interesting:

The HttpProtocol assumes that closing the output will close the input. I don't love that assumption but I preserved it here.
Creating pipes is messy when you need to flow settings around (for back pressure and scheduling). It would be better if we just had a custom pipe implementation that decorates the SSLStream.

@dotnet-bot test OSX 10.12 Release Build


Improve README for Windows users

As discussed in #226. First pass, happy to update/move/reword this as needed.

You might want to exclude README.md changes from CI - I didn't realise how many builds this would trigger!

Heh, it's okay the runs are fine. The failing windows builds is a known issue and should be getting fixed upstream soon.


Add HTTP proxy support when UAA is TLS-enabled

According to the net/http package documentation, the DefaultTransport used by the DefaultClient takes HTTP proxy configuration from the HTTP_PROXY and NO_PROXY environment variables:

DefaultTransport is the default implementation of Transport and is used by DefaultClient. It establishes network connections as needed and caches them for reuse by subsequent calls. It uses HTTP proxies as directed by the $HTTP_PROXY and $NO_PROXY (or $http_proxy and $no_proxy) environment variables.

When we pass an http endpoint to the UAA Go client constructor (NewClient()), it uses the http.DefaultClient to do the requests, and therefore uses the aforementioned proxy settings.
However, when we give an https endpoint to the constructor, an HTTP client is built with newSecureClient(). A custom http.Transport is defined which does not set any proxy settings.
This PR makes the custom http.Transport used when TLS is enabled use the same HTTP proxy configurations as the DefaultTransport. This change allows the client to access a TLS-enabled UAA when going through an HTTP proxy is necessary.

@markstgodard, @shashwathi or @leochu, could you please take a look or point me to the right person who could review this PR?

Hey @musse,
Given that we are trying to deprecate this library, we're going to close this issue at this point. Please feel free to reopen with your use case if you urgently need it.
Thanks!
Nitya and @utako


Forbidden access when accessing cluster with Azure AD

I followed all the steps as described in the document without any errors.
After changing the kubectl credentials I get a "forbidden" error
az aks get-credentials --resource-group myResourceGroup --name $aksname --overwrite-existing
~ kubectl get pods --all-namespaces To sign in, use a web browser to open the page https://microsoft.com/devicelogin and enter the code XXXXXXX to authenticate. Error from server (Forbidden): pods is forbidden: User "xxxxxxxxxxxxxxxxxx" cannot list resource "pods" in API group "" at the cluster scope
Did something change to Azure AD that causes the listed steps not to work anymore? (article is over a year old...)

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

ID: 6115d555-45a1-6539-4af6-fbc20a99f008
Version Independent ID: 21f80c83-ba10-9b7e-c54f-51a9c140517c
Content: Quickstart: Deploy an Azure Kubernetes Service cluster - Azure Kubernetes Service
Content Source: articles/aks/kubernetes-walkthrough.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

Same issue as: https://github.com/MicrosoftDocs/azure-docs/issues/28105

Thanks for sharing the update.


pdf2txt.py -t xml not extracting all information for extended encoding?

I'm trying to get the text out this mixed Arabic-English document:
https://www.dropbox.com/s/rfoisx2bw651l08/a.pdf?dl=0
A command like 'pdf2txt.py -t xml -p 22 a.pdf' gets all the English text information.
For the Arabic text, it seems it uses different embedded fonts/font-size combinations to render it, but I don't think I get all the information I need to reconstruct the Arabic. The number of characters seems roughly OK, but there simply is not enough variety in the values, as many of them map to colons or periods.
This suggests to me that there's some non-ascii encoding going on, and that pdf2txt somehow truncates to the most significant bits.
Could this be the case? If so, how could I get all the raw info out of the pdf, so that I can continue analysing it and mapping the characters to the correct unicode?

Do you know the origin of this pdf document? It looks like it is a scanned book with optical character recognition (OCR) applied to it. The OCR software probably only recognized the Latin alphabet correctly, and not the Arabic characters.
A couple of observations that support this:

The cover of the book is a noisy picture. Other images of the cover are sharp. There is also a tiny white line at the top of the first page, because the book was slightly missaligned in the scanner.
If you zoom in onto the characters you can see noise around all characters.
There are some traces of use, e.g. on the third page there is a stain. That's probably not in the digital edition. ;)
If you copy the text you get characters that are almost always correct for the English text, but not always. If you copy "Thematic vocabulary list | vi" on the sixth page, you get an I (capital i) instead of an | (pipe character).

Why is this important? Pdfminer.six extract the text from pdf using the "source code"  of the pdf. It does not  use OCR of any sort. This is a scanned document; basically images with some OCR information on top of it. You see the scanned document, but if you extract the text (e.g. by copying, or by using pdfminer.six) you get the result from the OCR.
If you want to extract the Arabic text you should probably use an OCR tool that does understand Arabic.

Makes sense, thanks for the detailed explanation!


Can I rewrite this thing in TypeScript?

I'd like to learn more about libp2p because it's awesome.
I'd also like to improve my TS skills + type safety is a desirable thing to have for a framework as sophisticated as this one.
Thoughts? :)

Hello @james2mid
We currently generate typescript declaration files on release, using types defined in jsdocs per #802 , so that libp2p users within typescript projects can use it easily. However, we are not looking for an entire rewrite of the codebase to typescript for 2 main reasons:

time needed to so such a major revamp to all libp2p modules, which would mean more important features would not be done
from a contribution perspective, it will be easier to contribute to libp2p as a JS project because all TS developers know JS, but not all JS developers know TS.

If you are still interested in helping out with other types of tasks let us know. There are a lot of issues labeled with ready and easy that you can have a look if you like

Hello @james2mid
We currently generate typescript declaration files on release, using types defined in jsdocs per #802 , so that libp2p users within typescript projects can use it easily. However, we are not looking for an entire rewrite of the codebase to typescript for 2 main reasons:

time needed to so such a major revamp to all libp2p modules, which would mean more important features would not be done
from a contribution perspective, it will be easier to contribute to libp2p as a JS project because all TS developers know JS, but not all JS developers know TS.

If you are still interested in helping out with other types of tasks let us know. There are a lot of issues labeled with ready and easy that you can have a look if you like

I appreciate your reply. I'm keen to help out so I'll check out the available issues :)
Aside from the libp2p documentation, are there any learning resources you can recommend surrounding P2P and other related topics?

I appreciate your reply. I'm keen to help out so I'll check out the available issues :)
Aside from the libp2p documentation, are there any learning resources you can recommend surrounding P2P and other related topics?

I appreciate your reply. I'm keen to help out so I'll check out the available issues :)

That would be super helpful. Let me know what would you like to get and we can discuss it further if you need. By the way, there is https://github.com/libp2p/js-libp2p/issues/659 and https://github.com/libp2p/js-libp2p/issues/830 in the context of adding types to jsdocs to create type declarations on build that you can have a look if you are interested in typescript thematic.

Aside from the libp2p documentation, are there any learning resources you can recommend surrounding P2P and other related topics?

Have a look at this issue: https://github.com/libp2p/libp2p/issues/56 It has a lot of nice resources for the P2P space

I appreciate your reply. I'm keen to help out so I'll check out the available issues :)

That would be super helpful. Let me know what would you like to get and we can discuss it further if you need. By the way, there is https://github.com/libp2p/js-libp2p/issues/659 and https://github.com/libp2p/js-libp2p/issues/830 in the context of adding types to jsdocs to create type declarations on build that you can have a look if you are interested in typescript thematic.

Aside from the libp2p documentation, are there any learning resources you can recommend surrounding P2P and other related topics?

Have a look at this issue: https://github.com/libp2p/libp2p/issues/56 It has a lot of nice resources for the P2P space


Straight through notify

PLEASE DO NOT MERGE THIS PR UNTIL ALL REVIEWERS HAVE HAD A CHANCE TO REVIEW AND APPROVE
- What I did
Removed all avoidable server-side latency under normal conditions when delivering notifications from one atSign to another. (Normal conditions means that both secondary servers are available, connectable and functioning normally)
Lower bound of latency inside the sending secondary server is now approximately 2.5 milliseconds. With regards to total end-to-end latency, for short encrypted messages, the combination of Client-1 encryption time, client-1-to-server-1 communication time, server-2-to-client-2 communication time and client-2 decryption ( excluding other network latency caused by e.g. speed-of-light), adds about another 2.5 milliseconds, giving an overall lower bound for end-to-end latency of approximately 5 milliseconds.
- How I did it
(All of the 'how I did it' info here comes from this PR's commits' messages. As this PR also contains some logging and de-linting changes, reviewers will likely find it easier to review the changed files one commit at a time)
In the at_persistence_secondary_server package:

in AtNotificationKeystore, publish the new notification event before persisting
In the at_secondary_server package:

notify_connection_pool.dart

added pool.init(size) method. init(size) is called by AtSecondaryImpl when it is starting.
pool.get(atSign): Removed redundant second call to _hasCapacity() on the underlying OutboundClientPool

at_secondary_impl.dart

Removed some dead code
Moved initialization of the notifications ResourceManager to after the initialization of the OutboundClientManager
Uses new init(size) method on the ResourceManager, which in turn calls the new init(size) method on NotifyConnectionPool

notification_manager_impl.dart

NotificationManager_storeNotificationInQueue (called by the public NotificationManager.notify method) now

calls queueManager.enqueue(notification) before persisting it
nudges the notifications ResourceManager to let it know there is a new notification ready to send

update_verb_handler.dart

queues the notification for delivery (via NotificationManager.notify()) before persisting it
de-linted

resource_manager.dart

Enhanced the queue processing so that it still periodically runs on a timer, but also can be 'nudged' into action at any time (e.g. NotificationManager and UpdateVerbHandler as described above)
Added boolean guard _isProcessingQueue to ensure that the main queue processing loop, in the _processNotificationQueue() function, is not re-entrant
Added boolean _nudged so that when _processNotificationQueue() completes, it knows that it should immediately run again
Added new init(outboundConnectionsLimit) method (called by AtSecondaryImpl while it is starting) which in turns initializes the NotifyConnectionsPool with size _outboundConnectionsLimit
Removed this line of code from the main processing for a given atSign within _processNotificationQueue() : AtNotificationMap.getInstance().removeWaitTimeEntry(atSign); and moved it into _sendNotifications in order to fix a race-condition bug where

if a new notification for a given atSign was enqueued by the NotificationManager at the same time as _processNotificationQueue() was processing previously-queued notifications for that same atSign (the window for this to occur is provided by var outboundClient = await _connect(atSign); which can be very long)
then the next time _processNotificationQueue() would run, with no other notification having been enqueued for that atSign in the interim, it would find nothing to process because the atSign had been removed from the waitTimeMap

Changed _sendNotifications so it no longer closes the outbound connection after delivering all of the notifications for a given atSign. Keeping the connection open means that we avoid the (significant) latency added by creating and authenticating a new connection each time.

at_notification_map.dart : Bug-fix: Made the empty list strongly typed in AtNotificationMap.remove() which was previously untyped and therefore being returned as a ListIterator<dynamic>, which was in turn causing problems elsewhere when casting to Iterator<AtNotification>

Made OutboundClientPool usable as an LRU pool

Added lastUsed DateTime field to OutboundClient class; field is updated to DateTime.now() every time a method is called
Added removeLeastRecentlyUsed() method to OutboundClientPool. Sorts the clients in the pool by lastUsed, ascending
Modified NotifyConnectionPool to call removeLeastRecentlyUsed() on its OutboundClientPool if it is at capacity
Modified OutboundClientManager to call removeLeastRecentlyUsed() on its OutboundClientPool if it is at capacity
Added some tests for direct use of removeLeastRecentlyUsed() method
Added some tests on NotifyConnectionsPool to verify LRU behaviour of its underlying OutboundClientPool

- How to verify it
All existing and new (in this PR) tests should pass.
NB This branch includes changes to at_persistence_secondary_server, so at_secondary_server's pubspec.yaml in this branch has a dependency override to get at_persistence_secondary_server from this same branch rather than from pub.dev.
When this PR is merged, it should immediately be followed by
(1) publish new version of at_persistence_secondary_server (its pupspec.yaml and CHANGELOG.md are ready to go)
(2) Create a new PR which removes the dependency override for at_persistence_secondary_server from at_secondary_server's pubspec.yaml
- Description for the changelog
Removed all avoidable server-side latency under normal conditions when delivering notifications from one atSign to another.

@cconstab tagging you in your role as bug-finder general to find where this breaks ... branch name is straight_through_notify

Original fix to the waitTimeMap race condition bug introduced a different (albeit very minor) bug. Latest commit fixes that bug also.

Noticed a potential bug in ResourceManager._enqueueErrorList(). I think that the waitTimeEntry and quarantineEntry for a given atSign should only be cleared when all pending notifications have reached maximum retries, but right now it happens if any of them have reached maximum retries :
      if (atNotification.retryCount == maxRetries) {
        AtNotificationMap.getInstance()
            .removeWaitTimeEntry(atNotification.toAtSign);
        AtNotificationMap.getInstance()
            .removeQuarantineEntry(atNotification.toAtSign);
        logger.warning(
            'Failed to notify ${atNotification.id} from ${atNotification.fromAtSign} to ${atNotification.toAtSign}. Maximum retries ($maxRetries) reached');
        continue;
      }

On Wed, May 4, 2022 at 9:27 AM Muralidharan @.***>
wrote:

@.**** commented on this pull request.
In
at_secondary/at_secondary_server/lib/src/connection/base_connection.dart
https://github.com/atsign-foundation/at_server/pull/664#discussion_r864571563
:

@@ -47,11 +47,23 @@ abstract class BaseConnection extends AtConnection {
throw ConnectionInvalidException('Connection is invalid');
}
try {

 logger.info('SENT: [${getMetaData().sessionID}] ${BaseConnection.truncateForLogging(data)}');

@gkc https://github.com/gkc this may cause lot of noise in prod since
we log on every socket write. e.g image data will cause lot of encoded
chars to be logged
Yes I added the 'truncateForLogging" method to cut down on the amount of
noise. Having used this configuration for over a week now, I am convinced
that logging all @ protocol interactions (and very little else) at INFO
level makes the logs far more useful.

Message ID: @.***
com>

Hi @murali-shris Before I merge to trunk, please can you do another review please, of the commits since May 5th?


[F#] Add information about how to achieve different equality comparison with records

Add additional information how to create records with class like comparison / reference equality.
Sometimes records are faster to write than classes and so the reference equality is still a need.
To avoid further search for beginners I would like to see this in the documentation.
cc @cartermp

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@cartermp sorry totally forgot about this


Converting everything to swift-3

Fixes #23 #20

fixes #23

Thanks @lucoceano. Will approve by tonight! 🎉


Make it so the controllable instance doesn't leak through to the DSL

Originally I was thinking it was a big problem that a new instance of Controllable was created each time controllable was called.  It turns out that since what is created is really an anonymous inner class with no state of its own, its not a huge problem but just a slight waste of resources due to the extraneous object creation.  However, I also realized that the controllable method was being leaked through to the DSL.  This PR addresses both by making it a val rather than a def and making it private to the surge package.

Codecov Report

Merging #142 (b81c634) into main (1d6f07e) will decrease coverage by 0.09%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #142      +/-   ##
==========================================
- Coverage   71.74%   71.65%   -0.10%
==========================================
  Files         158      158
  Lines        3143     3143
  Branches       82       82
==========================================
- Hits         2255     2252       -3
- Misses        888      891       +3

Impacted Files
Coverage Δ

...n/scala/surge/kafka/streams/SurgeHealthCheck.scala
92.00% <ø> (ø)

...src/main/scala/surge/core/KafkaProducerActor.scala
87.80% <100.00%> (ø)

...surge/internal/core/SurgePartitionRouterImpl.scala
72.72% <100.00%> (ø)

...a/surge/internal/domain/SurgeMessagePipeline.scala
87.27% <100.00%> (ø)

...e/internal/persistence/PersistentActorRegion.scala
83.33% <100.00%> (ø)

...l/akka/kafka/KafkaConsumerStateTrackingActor.scala
100.00% <100.00%> (+3.22%)
:arrow_up:

...afka/streams/AggregateStateStoreKafkaStreams.scala
81.13% <100.00%> (ø)

...health/windows/actor/HealthSignalWindowActor.scala
91.72% <0.00%> (-2.26%)
:arrow_down:

...a/surge/kafka/KafkaPartitionShardRouterActor.scala
67.78% <0.00%> (-0.68%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1d6f07e...b81c634. Read the comment docs.


Update meeting info section to point to "org" repo

We should keep the OCI contributor/maintainer dev meeting information in
the new "org" repo and all other repos should point there. This
minimizes places we need to update in the future if our meeting
technology or meeting frequency changes again.
Signed-off-by: Phil Estes<EMAIL_ADDRESS>

LGTM

LGTM


Play better with mobile divices, e.g. touch events

Closes #149

Nice to have a static example that requires no server!


runtime(rust): Respect no_plugin_maps and no_rust_maps globals

Makes it possible to opt-out of rust ft buffer mappings the same way it works for other ft plugins.

@chris-morgan Hello, I hope it is ok. This problem has been bugging me for quite a while.

Thanks. There isn't much activity over at https://github.com/rust-lang/rust.vim so let me include it now.


[Bug] CryptoProviderFactory.cs should implement IDisposable and dispose the CryptoProviderCache.

Which version of Microsoft.IdentityModel are you using?
6.32.2
Where is the issue?

[x] M.IM.Tokens

Is this a new or an existing app?
This is captured by a test and prevent a partner from upgrading.
public class CryptoProviderFactory  // does not implement IDisposable
...
        public CryptoProviderFactory()
        {
            CryptoProviderCache = new InMemoryCryptoProviderCache() { CryptoProviderFactory = this };
        }
...

whereas InMemoryCryptoProvider is IDisposable.
Expected behavior
Implement IDisposable and dispose the in memory crypto provider cache.
Actual behavior
CryptoProviderFactory is not disposable whereas InMemoryCryptoProvider is IDisposable.
Note: This needs to be fixed in 6.x and in 7.x

Triage: We will need simple repro steps to convert into a unit test to showcase the issue we are fixing repro's before fix is applied.

There is static analysis that may be able to detect (some?) of these cases:
https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca2213

We are not going to do this now.


The string does not print completely

Hello, I am using your "ns16550s" library, I want to print a string every few seconds "This is uart device test.\n\r ", but when I debug, the terminal only prints "This is uart devi", and there is no line feed. Please look at this problem in your spare time.

Hello, I fixed the bug in the latest release, please let me know if it works now (I couldn't test it, I don't have the running environment anymore)


08/23/2018 21:55 : Create Pull Request for branch master -> prTesting1_2018082321

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon1.md
:white_check_mark:Succeeded

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


API stability proposal

This is an extension of the conversation started in:
https://docs.google.com/document/d/1eDQQSxqx2khTXfa2vVm4vqkyRwXYkPzZCcbjxJ2_AvA/edit?ts=5af97a4e
Ultimately, we would like to stabilize the API and have a deprecation period like we do for config changes. Getting there is going to take a bunch of thought and a solid plan. We also need to make sure we don't reduce velocity when the project is still relatively young.
Related to https://github.com/envoyproxy/envoy/issues/827
cc @brian-pane @htuch @alyssawilk @envoyproxy/maintainers

I think we should wait for this until the project is more mature. Or at most make this a best-effort idea. I don't want to restrict refactoring and code-cleanup due to out-of-tree code.

I think we should wait for this until the project is more mature. Or at most make this a best-effort idea. I don't want to restrict refactoring and code-cleanup due to out-of-tree code.

That was my feeling when I filed #2925 and I agree. Tough I would love to see some level of API stability for a subset of them.

FYI I think the current plan here is to adopt the WASM ABI as our stable filter API.

@mattklein123 any update to thinking here or should we close this out with https://github.com/envoyproxy/envoy/issues/3390#issuecomment-562327638 as PoR?

@mattklein123 any update to thinking here or should we close this out with https://github.com/envoyproxy/envoy/issues/3390#issuecomment-562327638 as PoR?

Sorry which comment as the PoR? I'm not sure we have any good plan here still.

The one I linked to (https://github.com/envoyproxy/envoy/issues/3390#issuecomment-562327638). Basically, we will use Wasm ABI as the only stable ABI for filters (Wasm, C++ NullVm etc).

I don't know that we have made any progress on this one way or the other. Practically, the Wasm ABI is the only way today to have any hope of a stable ABI. I would be fine just saying this is the way it always will be but I don't feel super strongly about it.


Can you offer lower Compile Environment project?

My OS is Win7 and VS2013. When cmake successful, but it have very errors. Or can you offer more detail document which can guide me to run the project correctlly.

The problem is probably that we use C++11 which is not completely supported by the VS2013 compiler.

I just modified the Readme file to make the C++11 requirement clearer.


MessageSent Event return wrong class

Laravel Version: 9.0.2
PHP Version: 8.0.12

Description:
The Event Illuminate\Mail\Events\MessageSent return a Symfony\Component\Mime\Email as message, but the corrected should be Symfony\Component\Mailer\SentMessage.
This object (Symfony\Component\Mailer\SentMessage) contains informations about the sent mail that can't be retrieved with  \Symfony\Component\Mime\Email.
One example is the MessageId. In SwiftMailer, we could define the messageId, but now it's no more possible with Symfony Mailer.
The only way to the the message ID now is from Symfony\Component\Mailer\SentMessage that is not returned by MessageSent event.
One problem with the current class returned is when we use the mailer webhooks to log the mail delivery/open/etc.
If we sent the message using queue, we don't have access to SentMessage and the MessageId.
This simple code change solve the problem
public function send($view, array $data = [], $callback = null)
{
...
if ($this->shouldSendMessage($symfonyMessage, $data)) {
          $sentMessage = $this->sendSymfonyMessage($symfonyMessage);
-         $this->dispatchSentEvent($message, $data);
+         $this->dispatchSentEvent($sentMessage , $data);
            ...
        }
}

*ps: In this code sample, I'm considering just the class returned, not the validations required, the chanfe on MessageSent class, etc
It's not just more semantical, but give a lot of usefull data that can't be get from another way

We're checking if https://github.com/laravel/framework/pull/40963 could fix things.

Thanks. We'll try to patch this soon!

Amazing. Thanks @driesvints


HBASE-26856 BufferedDataBlockEncoder.OnheapDecodedCell value can get corrupted

Reapplying reverted commit, plus spotless:apply.

FYI @arshadmohammad @pankaj72981

Thanks @ndimiduk  for bringing this.


feat: adds some nav cards to the root index

Adds some cards used to navigate to the demos.
I used TextRewrite hahaha

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


added pyproject.toml

Thank you @joshua-gould for your contribution! I'll go ahead and merge the github actions and pyproject.toml to the main repo.
I'll be setting up the deployment actions to make it available to install from PyPI soon.


[ROS2] Loading plugins from a non-ROS package

In ROS 1, the paths to all packages used in the project workspace are added to the ROS_PACKAGE_PATH environment variable, including pure CMake packages. This allows pluginlib to find libraries defined in non-ROS libraries, since the ROS 1 implementation of ClassLoader uses ros::package::path().
However, in ROS 2 there are different environment variables for CMAKE_PREFIX_PATH and AMENT_PREFIX_PATH. The first one contains all CMake packages used in my project, while the second one is a subset of the first only containing ament CMake packages. The ROS 2 implementation of ClassLoader uses ament_index_cpp::get_package_prefix(), which apparently only looks for packages in AMENT_PREFIX_PATH. This means that (so far) I can no longer load libraries defined in non-ROS packages.
The Tesseract motion planning pipeline is an example of something affected by this difference in behavior. The tesseract_plugins package defines several plugins which use libraries created by the ROS-agnostic tesseract_collision package, and these plugins are loaded at runtime by classes in the tesseract_monitoring ROS package. This works just fine in ROS 1 but in my ROS 2 port I get errors like:
[my_project_planning_manager_node-1] terminate called after throwing an instance of 'pluginlib::ClassLoaderException'
[my_project_planning_manager_node-1]   what():  package 'tesseract_collision' not found, searching: [/home/joe/workspaces/ros2_ws/install/xacro, /home/joe/workspaces/ros2_ws/install/opencv_tests, /home/joe/workspaces/ros2_ws/install/joint_state_publisher, /home/joe/workspaces/ros2_ws/install/vision_opencv, /home/joe/workspaces/ros2_ws/install/my_project_planning, /home/joe/workspaces/ros2_ws/install/tesseract_monitoring, /home/joe/workspaces/ros2_ws/install/tesseract_rosutils, /home/joe/workspaces/ros2_ws/install/tesseract_plugins, /home/joe/workspaces/ros2_ws/install/tesseract_msgs, /home/joe/workspaces/ros2_ws/install/srdfdom, /home/joe/workspaces/ros2_ws/install/robot_state_publisher, /home/joe/workspaces/ros2_ws/install/rct_ros_tools, /home/joe/workspaces/ros2_ws/install/rct_image_tools, /home/joe/workspaces/ros2_ws/install/rct_optimizations, /home/joe/workspaces/ros2_ws/install/octomap_msgs, /home/joe/workspaces/ros2_ws/install/mutable_transform_publisher, /home/joe/workspaces/ros2_ws/install/mutable_transform_publisher_msgs, /home/joe/workspaces/ros2_ws/install/image_geometry, /home/joe/workspaces/ros2_ws/install/cv_bridge, /opt/ros/dashing]

Any thoughts? I'd be open to package configuration workarounds on my end as well.
(pinging @Levi-Armstrong as the Tesseract package maintainer)

A slightly messy workaround is to manually append the non-ROS package containing the libraries I want to load to the AMENT_PREFIX_PATH environment variable, since ament_index_cpp::get_resource() uses this variable to populate its search paths (via ament_index_cpp::get_search_paths()).


Updating to AuTest 1.8.1.

This new AuTest version fixes an import issue for the hosts package.

Cherry picked to 9.0.x


Small Code improvement

Added Typing and minor code change to improve readability.

Did you tested the code? For some reason, saving wallpaper isn't working for. You can take a look if you are free. I am currently trying to find out what went wrong in image save command.


Is there support for onBlur?

onChange is too slow for what I'm doing with react-numeric-input. I need to use onBlur which the callback function is only triggered after the user moves the focus away from the input.
I didn't see onBlur in the documentation but is there a substitute for it?

Yes there is.


DIGITALTWIN - add a description page for each solutions

[ ]  create .md files with description for each solution
[ ] make cards clickable opening description page in same tab

Had a discussion with Zia and Alex about this.

Alex will follow up with the Jimber team to get a list of features per experience.

Zia will come up with a template we can apply to each page.

I think we should also have a simple graphic per experience explaining what is happening in the background vs what happens currently – e.g. a video connection going between peers rather than going to a data center first

considered as top prio right now

text template is put to the document: https://secure.threefold.me/pad/#/2/pad/edit/NiMhcN6gp+0wnD2jVBTdZDqm/

This will not be finished this sprint as we didn't get the information necessary quickly enough. Will update today on what's realistic.

AI file sent to @Vilnite


Home assistant can't restart

I put everything as you said in configuration..yaml and in the folder custom_components. When I try to restart home assistant , I get this errors and It doesn't restart :
Error executing service <ServiceCall homeassistant.restart (c:0807283bd57f4011bd263a421d8363ff)>
23:04 loader.py (ERROR)
Error parsing manifest.json file at /config/custom_components/sonoff/manifest.json: Expecting value: line 7 column 1 (char 6)
23:04 loader.py (ERROR)
I have the latest version of hassio
Thank you

I have too same problem

I dont have the custom_component folder, how can i get it?

This also broke for me with the upgrade to 96 (or possibly the upgrade to Python 3.6 I did it at the same time). It seems like it doesn't want to load the component anymore.

I dont have the custom_component folder, how can i get it?

You need to create it. Use this file organisation (from the README) :
 custom_components
    └── sonoff
        └── __init__.py
        └── switch.py
        └── sensor.py

Where do i need to put that ?

Does the component still Works in the latest home assistant ? It wont start with me ...
Thx
Outlook voor Android downloaden
On Mon, Jul 29, 2019 at 10:10 AM +0200, "polyedre"<EMAIL_ADDRESS>wrote:
I dont have the custom_component folder, how can i get it?
You need to create it. Use this file organisation (from the README) :
custom_components
└── sonoff

    └── __init__.py

    └── switch.py

    └── sensor.py

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@thostie2002 you need to put the custom_components directory inside your config dir for Home Assistant. If you used pip to install it by default it is in the .homeassistant folder.
@Gabber2222 It seams that there is a problem with your manifest.json file, could you put it here ? I tried with this example and it worked:
{
  "domain": "your_domain_name",
  "name": "Your Integration",
  "documentation": "https://www.example.com",
  "dependencies": [],
  "codeowners": [],
  "requirements": []
}

Dear Peterbuga,
I don 't understand what you mean with the manifest.json file. I copied it from the website.
Thx  in advance,
Ivan
Outlook voor Android downloaden
On Mon, Jul 29, 2019 at 10:30 AM +0200, "polyedre"<EMAIL_ADDRESS>wrote:
@thostie2002 you need to put the custom_components directory inside your config dir for Home Assistant. If you used pip to install it by default it is in the .homeassistant folder.
@Gabber2222 It seams that there is a problem with your manifest.json file, could you put it here ? I tried with this example and it worked:
{
"domain": "your_domain_name",
"name": "Your Integration",
"documentation": "https://www.example.com",
"dependencies": [],
"codeowners": [],
"requirements": []
}
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

So i need to make a new file in the config folder named custom_components?
@gabber2222 are you dutch?

A new folder
Outlook voor Android downloaden
On Mon, Jul 29, 2019 at 11:02 PM +0200, "thostie2002"<EMAIL_ADDRESS>wrote:
So i need to make a new file in the config folder named custom_components?
@Gabber2222 are you dutch?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Yes, i am Dutch :-)
Outlook voor Android downloaden
On Mon, Jul 29, 2019 at 11:02 PM +0200, "thostie2002"<EMAIL_ADDRESS>wrote:
So i need to make a new file in the config folder named custom_components?
@Gabber2222 are you dutch?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I have added the sofoff: section to my configuration.yaml file, and I have added the .py files to my config/custom_components/sonoff folder but I et the following error when checking the config from within HASS
Log Details (ERROR)
Tue Aug 13 2019 21:51:34 GMT-0700 (Pacific Daylight Time)
Error loading custom_components.sonoff. Make sure all dependencies are installed
Traceback (most recent call last):
File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/loader.py", line 355, in _load_file
module = importlib.import_module(path)
File "/usr/lib/python3.7/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1006, in _gcd_import
File "", line 983, in _find_and_load
File "", line 967, in _find_and_load_unlocked
File "", line 677, in _load_unlocked
File "", line 728, in exec_module
File "", line 219, in _call_with_frames_removed
File "/home/homeassistant/.homeassistant/custom_components/sonoff/init.py", line 28, in
import websocket
ModuleNotFoundError: No module named 'websocket'
Any pointers to what I have done wrong?

@markbooker here https://github.com/peterbuga/HASS-sonoff-ewelink/issues/130

@peterbuga thank-you, this resolved the issue, would it be possible to add that as a requirement or setup step to the page, I think it would help a lot of people.

@markbooker in a normal installation everything is added automatically upon first boot with this component enabled, who knows what kind of other problems you guys have around there :/

@peterbuga I think what @markbooker is suggesting is a great idea, as I know these constant websocket questions are tiring. If there was a simple step in the guide to ensure the pre-reqs are installed, your time won't be wasted. I can tell these posts frustrate you (there's a multitude of examples), so by ensuring the detail is available, and simple, for users to follow, it would reduce some of this angst.

i have it now working, the only thing is it changes often to unavailable but that stop when I switch how can I solve that?

Docker image would be great.


can we configure appliationname option based on web application

Hi,
is configuration option "ApplicationName" available for application so that we can seperate traces according to different application instance. For example, i define the option in the entrance file of php framework.
Thanks.

Untill now, php.ini is the only place to set "ApplicationName".
Do you want to set the "ApplicationName" in your application?

@eeliu
yes, since it is common to run several applications on same server while we want to see call details seperatly.


Set up ESLint with Standard and React config

Did a first pass on setting up the rules. I'll enforce and fix the issues once we've settled on the rules. I used Standard config, combined with a more specific React config.

@rafeememon Can you give your opinion about this?

Current coverage is 92.55%

Merging #383 into master will not affect coverage as of 1d14141

@@            master    #383   diff @@
======================================
  Files            9       9
  Stmts          188     188
  Branches        38      38
  Methods         88      88
======================================
  Hit            174     174
  Partial          4       4
  Missed          10      10

Review entire Coverage Diff as of 1d14141

Powered by Codecov. Updated on successful CI builds.


suggest add table function generate_series and change the results of current version

Is your feature request related to a problem or challenge?
as https://github.com/apache/arrow-datafusion/discussions/9315 mentioned,  generate_series is alias of range, it's result does not include the upper bound. it results the problem of Portability from other DBMS.
Describe the solution you'd like
the result  of generate_series is same as that of postgresql, the range() funciton keeps the current result
 select generate_series(1,3);
 generate_series
-----------------
               1
               2
               3

select * from generate_series(1,3);
 generate_series
-----------------
               1
               2
               3

Describe alternatives you've considered
No response
Additional context
No response

databend's numbers function + 1 is similar with generate_series
ctx.sql("select number+1 from numbers(3)")
┌──────────────┐
│ (number + 1) │
│    UInt64    │
├──────────────┤
│            1 │
│            2 │
│            3 │
└──────────────┘

I'm just wondering what is the purpose of datafusion to design in such a way. If this feature is needed, I can do this one.

Here is the duckdb syntax: https://duckdb.org/docs/sql/functions/nested.html#range-functions
It looks to me like generate_series and range can be both scalar functions (return list) or table functions (return a table)

take


Use kubernetes patch instead of get-update in subscriber for updating cm and secret

Currently, for updating configmap and secret, we are reading the secret and writing the secret. This is not an efficient way as it makes it complicated if other services are using it. Better to patch the required field.
Ref-https://github.com/litmuschaos/litmus/blob/b3c315542482d0f447deff308e18b1b2f4a8c415/litmus-portal/cluster-agents/subscriber/pkg/k8s/operations.go#L160

/assign

May I know if there is a maintainer who can assign this to me? Thanks.

@afzal442 you can start on the issue and raise the PR, we keep the issue open for anyone to work on. Just put a message that you are working on it and start. If there isn't a PR after a few weeks someone else can take it up no need to assign or reassign issues.

Oh sorry for that @gdsoumya . I'll take care abt that nxt time

No worries @afzal442, just put a comment here once you start on the pr.

I would like to understand and properly work on it.
Starting it.

I have made the pr

Hey @gdsoumya @imrajdas,
I have studied the changes you requested in this pull request and made the necessary adjustments.
I noticed that "isClusterConfirmed" is always true, so there is no need to include it as a parameter in the function.
func ClusterRegister(accessKey string) (bool, error) {
	clientset, err := GetGenericK8sClient()
	if err != nil {
		return false, err
	}

	// Define a patch object for the ConfigMap with only "IS_CLUSTER_CONFIRMED".
    configMapPatch := map[string]interface{}{
        "data": map[string]interface{}{
            "IS_CLUSTER_CONFIRMED": true,
        },
    }

	configMapPatchBytes, err := json.Marshal(configMapPatch)

	if err != nil {
		return false, err
	}

	// Patch the ConfigMap.
	_, err = clientset.CoreV1().ConfigMaps(AgentNamespace).Patch(
		context.TODO(),
		AgentConfigName,
		types.StrategicMergePatchType,
		configMapPatchBytes,
		metav1.PatchOptions{},
	)

	if err != nil {
		return false, err
	}

	logrus.Info("%s has been updated",AgentConfigName)

	newSecretData := map[string]string{
		"ACCESS_KEY": accessKey,
	}

	secretPatch := map[string]interface{}{
		"stringData": newSecretData,
	}

	secretPatchBytes, err := json.Marshal(secretPatch)
	if err != nil {
		return false, err
	}
	_, err = clientset.CoreV1().Secrets(AgentNamespace).Patch(
		context.TODO(),
		AgentSecretName,
		types.StrategicMergePatchType,
		secretPatchBytes,
		metav1.PatchOptions{},
	)
	if err != nil {
		return false, err
	}

	logrus.Info("%s has been updated", AgentSecretName)

	return true, nil
}

If this seems correct to you i would like to raise a PR for it.