Fix round with RoundingMode Ceiling

Problem 1:
If you round off a number consisting of only nines with Ceiling mode - exponent does not increase
Actual: 999.99 -> 100
Expected 999.99 -> 1000
Problem 2:
If suffix (in placeADotInStringExpanded method)  contains only zeros - result contains dot at end
Actual: 1000.00 -> 1000.
Expected: 1000.00 -> 1000

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Matvei Afanasev seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi Matvei, thank you very much for your contribution, it looks great!


[controller][test] Fix race condition during controller startup and fix flakies

Fix race condition during controller startup and fix flakies/reduce noise

Setup admin topic for a cluster before running cluster initialization routines
Close cluster wrapper at the end of test in PartialUpdateClusterConfigTest
Close local broker created during TestActiveActiveIngestion::testAAIngestionWithStoreView
Use volatile PubSubProducerAdapter::forceClosed flag to make update visible to a callback thread
Use unique data paths for venice-server-data in AbstractClientEndToEndSetup to avoid race condition

How was this PR tested?

Internal CI and GHCI
Does this PR introduce any user-facing changes?

[x] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.

Thanks for the review, @FelixGV & @nisargthakkar!


Class navigation button does nothing

Java docs have a class navigation button that doesn't work.  Would be nice to have the button do what the help says it will
Class/Interface
Each class, interface, nested class and nested interface has its own separate page. Each of these pages has three sections consisting of a class/interface description, summary tables, and detailed member descriptions:

Class inheritance diagram
Direct Subclasses
All Known Subinterfaces
All Known Implementing Classes
Class/interface declaration
Class/interface description
Nested Class Summary
Field Summary
Constructor Summary
Method Summary
Field Detail
Constructor Detail
Method Detail

For reference its the header CLASS shown in the image

Closing as this is how Javadocs work. What you're seeing is the Overview, Package, and Class are an only one can be active at once.
Overview shows the entire overview of the Javadocs, normally this is either all modules or packages in the Javadocs.
Package shows information about a specific package, normally this is all the classes in the package.
Class shows information about a specific class, this is the fields, constructors, and methods of the class.
Class is only available when you've selected a specific class.


浏览器无法连接问题

配置好之后，访问出现如上图片所示的问题，求解

看异常信息 ，是DB 异常了，可以先检查下DB 是否能正常访问。

已解决，确实是db的问题，现在是实时状态那里一直都是加载状态，不知道什么原因！！！！！
On Mon, Aug 14, 2017 at 11:24 AM, wjx82850707<EMAIL_ADDRESS>wrote:

Closed #55 https://github.com/Tencent/Tars/issues/55.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Tencent/Tars/issues/55#event-1204142525, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AObKJAt2Mdft0Nbw98nwN02sx0L7KPIgks5sX733gaJpZM4O1Tjl
.


Text mode enhancements!

Work in progress:
Fixed scrolling a bit by allowing the scroll to happen if we're about to write to a position outside the buffer. Before it only scrolled on newlines. Also added a bit of code to update the cursors location!
As stated above this is a WIP, I was just messing around a bit but it works well enough! I just copied the outb asm function from the interrupts lib, so that's something that might need to change.
Let me know what you guys think!

Interesting! :smile: :+1:
So, we also have an outb already: https://github.com/intermezzOS/kernel/blob/master/src/interrupts/src/lib.rs#L196-L199 This probably means that this is a good thing to extract to another crate...

So, we also have an outb already: https://github.com/intermezzOS/kernel/blob/master/src/interrupts/src/lib.rs#L196-L199 This probably means that this is a good thing to extract to another crate...

Yeah! I was going to do so, but I wasn't really sure where to put it or what to call it. Was thinking maybe common/src/lib or common/src/asm

So, I pulled this down and tried it, but I'm not actually seeing the cursor. Hm.

Yeah! I was going to do so, but I wasn't really sure where to put it or what to call it. Was thinking maybe common/src/lib or common/src/asm

I'm thinking it might just be its own crate entirely. But.... http://gz.github.io/rust-x86/x86/io/fn.outb.html :wink:

http://gz.github.io/rust-x86/x86/io/fn.outb.html
Oh, awesome! Looks good to me

So, I pulled this down and tried it, but I'm not actually seeing the cursor. Hm.
Hm... weird. I did this last night on os x, but i don't think that should matter. Now I'm trying to remember if I ever saw the cursor on my linux box. It used to just sit around the 3rd or 4th row and idle.

Okay, so this is interesting... The only reason I added the cursor code was because on my mac there would be this static blinking cursor that annoyed me, but I just hopped on my linux box and I don't see a cursor at all; not on master, nor for my branch. First thought is a qemu difference and they are on my machines (Linux is 2.2.0, os x is <IP_ADDRESS>). Maybe a BIOS difference could cause the cursor to be disabled by default? I'll do a little digging.

Okie dokie! So apparently QEMU sets up VGA registers differently for different machines, or versions (I don't know). Anyway, so after looking at this very helpful document VGA Regs I was able to write an initialize function that setup the Cursor Start/End Register so that the cursor will work on both of my machines.
Looks like the version on my mac had the cursor visibility flag set to true by default with the scanlines going from 0x0-0xF, and my linux box had the visibility flag set to false by default with the scanlines going from 0x0-0x0. I don't know much of anything about the VGA hardware (a little more now 👍 ), so there is probably more that should be done.

Awesome. Works great here. :+1:

Eventually, we can deduplicate outb and such, but for now, I'm not worrying about it too much. Thanks a ton for this :+1:


Chat screen: image thumbnails management is broken

The full res image is used by default, which is a pity

Validated in 0.3.12 develop #1708


ts-node needed as a production dependency

Codecov Report

Merging #99 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop      #99   +/-   ##
========================================
  Coverage    65.12%   65.12%
========================================
  Files           91       91
  Lines         2440     2440
  Branches       308      308
========================================
  Hits          1589     1589
  Misses         685      685
  Partials       166      166

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 e0561e7...2b51977. Read the comment docs.


Memory leak used with Arduino ESP32

Substantial memory leak when used with Arduino ESP32 with version 2.6.10.
Downgrading to 2.6.9 solves the problem.

You should update the library which this was already fixed.


🐛 Bug Report: Sign Out Url seems to be wrong, when the baseUrl has a path

📜 Description
If you sign out from Backstage, you will be redirected to the baseUrl without a path.
For example if backstage is deployed in a kubernetes cluster and the baseUrl is configured with the following value:
app.baseUrl: http://mykubernetes.installation.de/subpath/to/backstage
The Logout within the "Settings" Page will lead to the Url http://mykubernetes.installation.de/ instead of http://mykubernetes.installation.de/subpath/to/backstage.
I think the bug is in the file packages/core-app-api/src/app/AppRouter.tsx
There the Method getBasePath() is used to set the signOutTargetUrl. I think instead of getBasePath() the method readBasePath() should be used.
👍 Expected behavior
The Logout should lead to the url defined in the config parameter app.baseUrl including the basePath.
👎 Actual Behavior with Screenshots
The Logout leads to the url app.baseUrl without the basePath. For example if you defined app.baseUrl: http://localhost:3000/test the Logout leads to http://localhost:3000/ instead of http://localhost:3000/test.
👟 Reproduction steps

git clone https://github.com/backstage/backstage.git
Edit app-conifg.yaml. Set the config parameter app.baseUrl to "http://localhost:3000/test"
yarn install
yarn tsc
yarn dev
Login to http://localhost:3000/test/ with the guest-Account
Navigate to "Settings" and logout using the "Sign out" button within the "Profile" card
The logout will lead to the site http://localhost:3000/ with no content. I would expect, that the logout should lead to http://localhost:3000/test/

📃 Provide the context for the Bug.
No response
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

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

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
Yes I am willing to submit a PR!

Pull request: https://github.com/backstage/backstage/pull/20519


HDDS-10385. Memory leak for thread local usages in OMClientRequest

What changes were proposed in this pull request?
Instance object is having thread local object, which can have memory leak as thread local is not explicitly released. Further there do not seems any usecase having thread local definition for OmLockDetails.
So removed thread local definition.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-10385
How was this patch tested?
Existing test case to cover impact

Normal objects will increased OM's memory usage and GC time, threadlocal variable is to resolve this issue.

@sumitagrawl can you measure the GC pressure issue highlighted by @symious ?


[required attention] Huge delay in showing purchase panel

Hello,
we are under review form Apple and they noticed that purchase panel takes too much to show up.
 await purchaseSku(selectedPlan).then((transaction) { }

 // purchaseSku function
 static Future<GlassfyTransaction?> purchaseSku(GlassfySku sku) async {
    try {
      return await Glassfy.purchaseSku(sku);
    } catch (e) {
      return null;
    }
  }

https://github.com/glassfy/glassfy-flutter/assets/83366073/eceb559e-4a11-498f-8ec8-606378c17afe
When user taps the button we directly call  Glassfy.purchaseSk() passing the sku, this methods that takes 3secs against a common delay of 300-900ms.
Probably there is a network delay caused by Glassfy servers or maybe delay with method.channel (since we noticed you communicate with native), can you please have a look? It has a bad impact on user experience and Apple doesn't like it.

Added loading animation to button to let you see delay
https://github.com/glassfy/glassfy-flutter/assets/83366073/65ec6d53-280b-4e64-80fd-867ac96220a1

We also notificed that an eraly version of the plugin didn't suffer about this issue.
In this video can be seen that purchase panel shows up almost instantly at 10:59 https://youtu.be/baIL-QCnkI0?si=Ld8msI8q2qCjjcwO&t=659
So, if it is not due to Sandbox test or Glassfy server delay, taking a look at latest changes to code should be helpful.

@iosephmagno It's been a while, I've been trying to contact the Glassfy team. Nobody seems to respond from their side.
We are planning to migrate to revenue cat soon.


stupidterm: init at 20170315-752316a

Motivation for this change
Stupidterm is a simple and fast VTE-Based terminal emulator which is
configured by a simple ini-file.
Things done

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

[X] NixOS
[ ] macOS
[ ] Linux

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

@Mic92 What happened to my signature there?

I added https://github.com/NixOS/nixpkgs/commit/95f3e6ca320556e8ced5582fd0ca46d726282ade#diff-6228b0f570fed1908462e2c0b15f709cR6

I see, you could have told me and I would have updated it and learn something from it instead of removing my signature.
Oh well. Now it's in there at least.


XiaomiGateway3 The firmware version has been updated(1.5.0_0027)

Discussed in https://github.com/AlexxIT/XiaomiGateway3/discussions/379

Originally posted by PWMFI July  3, 2021
The version number of the new version of the firmware is (1.5.0_0027). After Mi Home is updated, all HA entities do not respond. I hope the plugin author will update the plugin as soon as possible, thank you! ! !

downgrade firmware

The same thing happened to me, I have downloaded the version of the gateway but it is still not recognized by Home Assistant.

How do you solve it, can you only downgrade the firmware? Can the plug-in author support this version by updating?

I have now tested with v1.5.0_0026 and it seems to be stable.
Changing the version is very easy, follow what they put here:
https://github.com/zvldz/mgl03_fw/tree/main/firmware

我现在已经用 v1.5.0_0026 测试过，它似乎很稳定。
更改版本非常简单，请按照他们在这里放置的内容进行操作：
https://github.com/zvldz/mgl03_fw/tree/main/firmware

The current firmware is 1.5.0_0027, do you also follow this method?

Yes, but the user is "admin" and the password in "admin" again.

Yes, but the user is "admin" and the password in "admin" again.

Okay thank you, do you need to disassemble the gateway for operation?

No, you can use the Telnet method.

Is very simply. Download putty and conect by telnet to the Gateway(admin/admin).
Later, you run this 3 comand:
--> wget -O /tmp/curl "http://master.dl.sourceforge.net/project/mgl03/bin/curl?viasf=1" && chmod +x /tmp/curl
--> export PATH="$PATH:/tmp"
--> curl -s -k -L -o /tmp/update.sh https://raw.githubusercontent.com/zvldz/mgl03_fw/main/firmware/mgl03_update.sh && sh /tmp/update.sh
Select the firmware 1.5.0_0026(4 option) and wait.

thank you very much

Your welcome.

Duplicate #370


Feature Request: Reverse wrap option

An additional bool field in the block settings. When set, columns break in reverse order on smaller screens.
Added this in my fork: https://github.com/mbarde/volto-columns-block/commit/146988173220dc45df525f01c1c3eac842d884ac
If you would like to adopt this feature I can create a PR.

@mbarde I've only seen your message now. Meh... sorry about this. A PR would sure be appreciated, and for sure accepted. Thanks!


🚀 Feature: create private repos

Bug Report Checklist

[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
I'm experimenting with creating a new project.  For now I want it to be private, I'll probably make it public later when I'm happy with what i've got.  At present, the only type of repos you can create are public.  Would it be good to be able to create private repos as well?
Additional Info
One thing to bear in mind, is that some of the bundled tooling (Renovate etc) will only work on public repos for free I think.  That may be an issue - or maybe something that can enabled later?

Agreed, this is definitely something that should be supported. I don't know what the full list of changes would be, so that'd be good to figure out. Once that's here I'd be pretty confident this will be accepting PRs.


Support for CFF/CFF2 outlines in write-fonts

I don't know if I missed something, but write-fonts does not seem to export anything related to CFF/CFF2 outlines. Is this part of write-fonts unfinished with more work needing to be done, or was this part simply forgotten to be exported? I mean, write-fonts/generated/generated_cff.rs and write-fonts/generated/generated_cff2.rs exist, but aren't actually include!'d the way they are in read-fonts. Also, write-fonts/tables/cff.rs and write-fonts/tables/cff2.rs don't exist.

You are correct, CFF writing is not done yet.

What needs to be done to implement it?


Run node in the background.

How can i run start.sh in the background?. Thing is i want to run start.sh as a background process. ctrl+c exits the process and removes the container. Is there any way to run it in the background?.

I ran it in the background by removing -it option from start.sh(main() and startNode() functions). I think we can give an option to run it in the background.

docker containers can be send to background using Ctrl + p , Ctrl + q .
https://stackoverflow.com/questions/19688314/how-do-you-attach-and-detach-from-dockers-process


tls.options not applying to entrypoint

Do you want to request a feature or report a bug?
Apologies if this is not a bug but a user problem but I have read the documentation and looked at the logs and everything seems to be fine so figured I'd report it in case it was a bug.
What did you do?
I want to configure two different entrypoints:

:65443 with mtls
:65444 without it

I have the following sample docker-compose.yaml:
---
version: "3.1"
services:
    fltraefik:
        image: "traefik:2.4"
        command:
        - --log=true
        - --log.level=DEBUG
        - --providers.file.directory=/config
        - --providers.file.watch=true
        - --providers.docker=true
        - --providers.docker.exposedbydefault=false
        - --entrypoints.nomtls.address=:65444
        - --entrypoints.nomtls.http.tls.options=`nomtls@file`
        - --entrypoints.withmtls.address=:65443
        - --entrypoints.withmtls.http.tls
        - --providers.file.debugloggeneratedtemplate

        ports:
          - "<IP_ADDRESS>:65443:65443"  # requires mtls
          - "<IP_ADDRESS>:65444:65444"  # no mtls
        volumes:
          - "./certs/:/certs"
          - "./config/:/config"
          - "/var/run/docker.sock:/var/run/docker.sock:ro"
    whoami:
        image: traefik/whoami
        labels:
          - "traefik.enable=true"
          - "traefik.http.routers.whoami.rule=PathPrefix(`/whoami`)"

And the following file inside the config folder:
---
tls:
    stores:
        default:
            defaultCertificate:
                certFile: /certs/server.crt
                keyFile: /certs/server.key
    options:
        default:
            clientAuth:
                clientAuthType: RequireAndVerifyClientCert
                caFiles:
                    - /certs/ca.crt
        nomtls:
            clientAuth:
                clientAuthType: NoClientCert

What did you expect to see?
I would expect the nomtls entrypoint to follow the configuration defined under tls.options.nomtls in the yaml file
What did you see instead?
No matter what I try only the default tls options work.
Output of traefik version: (What version of Traefik are you using?)

I provided this info above, if you need anything else don't hesitate to ask
If applicable, please paste the log output in DEBUG level (--log.level=DEBUG switch)
Logs don't show any warning or error, unfortunately.

Hello @dbarrosop and thanks for your interest in Traefik,
The configuration of the TLS options in an entrypoint means that the configured TLS options will be applied to all routers associated with the named entry point. Moreover, as described in the following documentation, the TLS options reference is mapped to the hostname found in the Host part of the rule.
Therefore, as there is no Host rule in your whoami router the default TLS option is always applied.


Error on collectstatic using django 1.11 python 3.6

Hi ,
I'm trying to move my Django project (which uses collectstatic) from python 2.7 to python 3.6.
When running collectstatic from the command line I get an error (see further). As everything else works fine, I'm a bit lost on what went wrong here.
I did not change my settings.py and all worked fine under python 2.7
Thanks for pointing me in the right direction!
Regards,
Joris
The error:
(platform)baze@baze-mintbox:~/dev/www/projects/ieper/platform > python manage.py collectstatic

Traceback (most recent call last):
  File "manage.py", line 22, in <module>
    execute_from_command_line(sys.argv)
  File "/home/baze/dev/virtualenvs/platform/lib/python3.6/site-packages/django/core/management/__init__.py", line 363, in execute_from_command_line
    utility.execute()
  File "/home/baze/dev/virtualenvs/platform/lib/python3.6/site-packages/django/core/management/__init__.py", line 355, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/home/baze/dev/virtualenvs/platform/lib/python3.6/site-packages/django/core/management/base.py", line 283, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/home/baze/dev/virtualenvs/platform/lib/python3.6/site-packages/django/core/management/base.py", line 330, in execute
    output = self.handle(*args, **options)
  File "/home/baze/dev/virtualenvs/platform/lib/python3.6/site-packages/collectfast/management/commands/collectstatic.py", line 70, in handle
    super(Command, self).handle(**options)
  File "/home/baze/dev/virtualenvs/platform/lib/python3.6/site-packages/django/contrib/staticfiles/management/commands/collectstatic.py", line 199, in handle
    collected = self.collect()
  File "/home/baze/dev/virtualenvs/platform/lib/python3.6/site-packages/collectfast/management/commands/collectstatic.py", line 61, in collect
    ret = super(Command, self).collect()
  File "/home/baze/dev/virtualenvs/platform/lib/python3.6/site-packages/django/contrib/staticfiles/management/commands/collectstatic.py", line 124, in collect
    handler(path, prefixed_path, storage)
  File "/home/baze/dev/virtualenvs/platform/lib/python3.6/site-packages/collectfast/management/commands/collectstatic.py", line 109, in copy_file
    self.do_copy_file(args)
  File "/home/baze/dev/virtualenvs/platform/lib/python3.6/site-packages/collectfast/management/commands/collectstatic.py", line 86, in do_copy_file
    self.storage, path, prefixed_path, source_storage):
  File "/home/baze/dev/virtualenvs/platform/lib/python3.6/site-packages/collectfast/etag.py", line 93, in should_copy_file
    remote_storage, source_storage, path, prefixed_path):
  File "/home/baze/dev/virtualenvs/platform/lib/python3.6/site-packages/collectfast/etag.py", line 83, in has_matching_etag
    storage_etag = get_etag(remote_storage, path, prefixed_path)
  File "/home/baze/dev/virtualenvs/platform/lib/python3.6/site-packages/collectfast/etag.py", line 58, in get_etag
    etag = get_remote_etag(storage, prefixed_path)
  File "/home/baze/dev/virtualenvs/platform/lib/python3.6/site-packages/collectfast/etag.py", line 38, in get_remote_etag
    normalized_path = storage._normalize_name(
  File "/home/baze/dev/virtualenvs/platform/lib/python3.6/site-packages/django/utils/functional.py", line 239, in inner
    return func(self._wrapped, *args)
AttributeError: 'StaticFilesStorage' object has no attribute '_normalize_name'\

For reference, my pip freeze output:
boto==2.46.1
boto3==1.4.4
botocore==1.5.40
certifi==2017.4.17
chardet==3.0.4
Collectfast==0.5.2
defusedxml==0.5.0
dj-database-url==0.4.2
Django==1.11
django-allauth==0.32.0
django-postgrespool==0.3.1
django-storages==1.5.2
docutils==0.13.1
futures==3.1.1
gunicorn==19.7.1
idna==2.5
jmespath==0.9.2
newrelic==<IP_ADDRESS>
oauthlib==2.0.2
psycopg2==2.7.1
python-dateutil==2.6.0
python3-openid==3.1.0
pytz==2017.2
requests==2.17.3
requests-oauthlib==0.8.0
s3transfer==0.1.10
six==1.10.0
SQLAlchemy==1.1.10
urllib3==1.21.1

What's your STATICFILES_STORAGE setting?

Your comment pointed me in the right direction, thanks!
STATICFILES_STORAGE was not (properly) defined due to sloppy refactoring. This caused the error.
setting it back to
STATICFILES_STORAGE = 'storages.backends.s3boto.S3BotoStorage'
resolved the issue
Sorry to bother you with this trivial mistake, and thanks again!
Joris

No worries, I'm happy to help. Maybe we should add a friendlier error message or warning if it's not set to boto?

That would be handy. The error message is a bit cryptic for those, like me, who are not very experienced with django/python. That made me wrongly think it had to do something with upgrading to python 3.6.
I might submit a pull request if I find time this summer.
Regards,
Joris

Very cool, let me know if I can assist you to make that happen. It could probably be something along the lines below added in https://github.com/jazzband/collectfast/blob/master/collectfast/management/commands/collectstatic.py#L36
if not hasattr(self.storage, '_normalize_name'):
    raise AttributeError(
        "The `STATICFILES_STORAGE` class needs to implement a `_normalize_name` method. Note that `storages.backends.s3boto.S3BotoStorage` from django-storages is the only class that Collectfast officially supports.")

Perfect. After my current deadline (1st of july), I will dive into it. Collectstatic is and has been very helpful to me, so it will be a pleasure to do something back, even if it's small. Regards and talk to you later!


Update ClearCommandTest.java

Added testing for emptyAddressBook failure.

Your codecov test failed (that is, your test coverage has decreased). Maybe you could re-assess your current coverage in your PR?


Updated bad Code(still bad though)!

Hi Parag,
Ignore the previous request, kindly check this one!
Thanks
Rajesh

@rajesh9625 Please decline or withdraw the previous pull request.

@chenlica  Declined the previous request!
Thanks
Rajesh

@renacimiento  deleted the unnecessary code, could you kindly review once again!

@renacimiento  made the suggested changes, merged and closed the branch!


Shared Zooniverse header [placeholder?]

PR Overview

This PR adds the shared Zooniverse header to the project.

Caveat: this isn't the "real" Zooniverse header, just a placeholder.

We're expecting a new Zooniverse shared header from the Zooniverse-React-Components, but we don't have an ETC at the moment.
One option is to create a functional shared Zooniverse header specific for this project, then simply replace it when the 'real' header from the ZRC is confirmed.

In any case, this PR also updates app-themes.styl to closer reflect the updated designs on Zeplin from the past 24 hours.

This will be superseded by a new style-update PR - coming soon to a repo near you!


Makes route prefix setting routeNamePrefix optional

Description
Describe what this change achieves.
Issues Resolved
https://github.com/opensearch-project/opensearch-sdk-java/pull/827#issuecomment-1629622060
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

While renaming will "fix" the sample, it doesn't solve the underlying problem:

There is no documentation to users on what is a valid name (and the OpenSearch error message isn't helpful enough to debug the problem without referencing the source code)
It disallows the existing names for every plugin, many of which we would migrate, e.g., opensearch-alerting, opensearch-anomaly-detection, opensearch-knn, opensearch-ml, etc.

The name here isn't actually used for anything programmatic. It's purely decorative and is only used as the response to the initialization command where it is displayed in a log message, and as the "node name". Node names in OpenSearch use hyphens.
Our test cases still use sample-extension as well, and don't fail on an illegal name there. If we're really set on disallowing a hyphen those need to validate the names.
I strongly prefer one of the following two options:

Simply remove the routePrefix() method and replace it with a constant in each REST handler, or a local method in a REST handler that uses an allowed prefix
If you keep the route prefix method, add a setRoutePrefix() method that validates it's a legal prefix.

I agree. However routePrefix only comes into play if an extension developer decides to use it while registering a route.
i.e. HelloWorld extension can also be registered without calling routePrefix method.
new NamedRoute.Builder().method(GET)
                .path("/hello")
                .handler(handleGetRequest)
                .uniqueName(routePrefix("greet"))   -----> can be written as `.uniqueName("greet")` OR `.uniqueName("helloWorld:greet")`
                .legacyActionNames(Set.of("cluster:admin/opensearch/hw/greet"))
                .build()

This would still work. routePrefix is completely optional

Added a validation check when extensionName is set to be later used by routePrefix

@dbwiddis I've update the routePrefix setting to be completely optional in extension-settings.yml

@joshpalis @owaiskazi19 @dbwiddis Would you please re-review this?

If this is about adding support for hyphen then why not update the pattern here?

@DarshitChanpura I'm still confused, isn't this PR purpose is to add hyphen to the extension name? Since we want to support other plugins like: opensearch-alerting, opensearch-anomaly-detection, opensearch-knn, opensearch-ml.

We are not meddling with extension name any more. This PR talks about a new optional setting routeNamePrefix which has to be alphanumeric and can contain _ only. This lifts the restriction from the extension name setting.

Cherry-picking this manually into https://github.com/opensearch-project/opensearch-sdk-java/pull/866


Training Time

How much time did it take to train the initial self supervised model using UCF101 and fine tuning again with UCF101 using 124*124 resolution?
I am having a plan to train it but availability of GPU for the required time is a constraint.

e.g. if using 2 Nvidia P40 GPUs, under 128x128 resolution, pretraining on UCF101 would take less than 1 day, finetuning again takes less than 1 day. I would estimate totally takes about 2 days.
Better or more GPUs can accelerate, for sure.

Thank you so much.


fix memory leak in actions

related to #332
methodology

trigger generate_indexes action
watch memory consumption with htop
try running with and without plugins (both are leaking)
use tracemalloc
modify code to return early from the action function
test with synchronous actions (wait for results, better but still leaking)

conclusions
there are a couple of memory-related issues:

workers stay alive after task execution and keep allocated memory (modules etc.)
python interpreter may dispose of allocated memory as it wishes (for example, keep memory may not release memory recuperated from garbage-collected objects),
database session is not released after task is done
other resources may not be released (notably file descriptors, and http connections)
we don't await results of the execution (not sure if this plays any major role, but the result have to be kept in memory)
likely memory leaks or implicit caches in some libraries (json and jinja are candidates)
even without memory leaks, actions such as generating indexes and synchronising mirrors can take huge amount of memory (>2GB), which should be managed more conservatively (for example, run some operations "on disk")

Fix
this pr works around some of the issues by recreating process executor for each new actions. This may be suboptimal in terms of performance, but it's more conservative with respect to resource management (memory).
we should make it more flexible:

executor should be kept around for executing multiple related tasks (such as tasks for job framework which is done one per package version)
we should set number of workers depending on the use case (for jobs, n should be > 1, for actions most likely n=1)

Codecov Report

Merging #345 (b23b126) into master (3a29438) will decrease coverage by 1.50%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #345      +/-   ##
==========================================
- Coverage   85.15%   83.65%   -1.51%
==========================================
  Files          66       66
  Lines        5040     5040
==========================================
- Hits         4292     4216      -76
- Misses        748      824      +76

Impacted Files
Coverage Δ

quetz/tasks/workers.py
95.72% <100.00%> (-0.86%)
:arrow_down:

quetz/database.py
78.94% <0.00%> (-21.06%)
:arrow_down:

...s/versions/53f81aba78ce_use_biginteger_for_size.py
56.25% <0.00%> (-18.75%)
:arrow_down:

quetz/pkgstores.py
45.62% <0.00%> (-16.35%)
:arrow_down:

...ns/versions/30241b33d849_add_task_pending_state.py
80.00% <0.00%> (-13.34%)
:arrow_down:

...ons/a3ffa287d074_case_insensitive_channel_names.py
68.75% <0.00%> (-6.25%)
:arrow_down:

...ions/versions/8dfb7c4bfbd7_new_package_versions.py
82.35% <0.00%> (-5.89%)
:arrow_down:

...s/versions/8d1e9a9e0b1f_adding_download_metrics.py
72.22% <0.00%> (-5.56%)
:arrow_down:

quetz/database_extensions.py
56.36% <0.00%> (-3.64%)
:arrow_down:

quetz/dao.py
87.69% <0.00%> (-2.42%)
:arrow_down:

... and 2 more

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 3a29438...b23b126. Read the comment docs.

Thanks Bartosz! I think it might make sense to also take a look at the syncing of packages (without even triggering index generation) as there is another suspected memory leak (filling up ~17 GB of RAM after syncing 250 GB of packages).
One idea was that it's in the lazy CondaFile and we're not properly closing it at the end of handle_package.

I don't think it's lazy CondaInfo. More likely, it's because we dont' manage (read: close) properly the open files


Update customweapons.nut to not use ExtraItems

Looking up ExtraItems, which exists in the script, means it can only use weapons listed within the script. Changing it to CustomItems instead allows outside scripts to be used.

just gonna comment the example table out of the main file instead, but yeah I can see why this was a headache to figure out lol.


[Permissions] onPermissionResult could send the PermissionStatus instead of just Boolean

Description
Currently, when using rememberPermissionState, the onPermissionResult callback is invoked with a boolean. Unfortunately, this doesn't help for the case where I want to know the value of shouldShowRationale. I also don't have access to the PermissionState yet within that callback. This makes it difficult to, for example show a message to the user that directs them to the system permission settings.
Steps to reproduce
Current API use:
rememberPermissionState(PERMISSION) { result: Boolean ->
  // result doesn't have enough information
}

Expected behavior
onPermissionResult should take the complete PermissionStatus instead
rememberPermissionState(PERMISSION) { result: PermissionStatus ->
  // use permission status here
}

For multiple permission request, the API could similarly use a Map<String, PermissionStatus> instead of just Map<String, Boolean>
Additional context
My use case is that: when a requested permission gets denied and showRationale=false, I can safely assume that the permission has likely been denied permanently, which means the only way to enable it is to go to the app settings. I can of course check this value before launching the permission request, but at this point, a permission that was never requested is indistinguishable from a permission that was permanently denied.

https://stackoverflow.com/questions/73898807/how-to-handle-runtime-permissions-in-jetpack-compose-properly-by-accompanist @kingsleyadio


feat(words): add caret_analysis related words

Signed-off-by: Bo Peng<EMAIL_ADDRESS>

@bopeng-saitama
Thank you for your contribution.
However, we cannot accept this PR as it is.
Could you refer to the following PR before making PR?
You have to explain the reason why you want to ignore the words. It is strongly recommended for you to add a few words to the ignorant library per PR.
https://github.com/tier4/autoware-spell-check-dict/pull/310
Words used only in CARET_trace should be replaced by other words which are commonly used.

@takam5f2
Thanks for the comment. I will remake the PR with a reason.


Hybrid AAD failing, not able to join On-Premises Domain.

Hi,
We have configured Hybrid AAD profile & when the machine is trying to enroll and join the On-prem AD, we are getting an error:80004005.
It is working fine when connected to Organisation network.
on Public network , on wi-fi it is failing.

Please try https://support.microsoft.com/en-us to help resolve this issue.


importlib.metadata.PackageNotFoundError when using pydantic-argparse

Hello,
I am encountering a variation of #1793, which seemed to be solved in 1.1 . Importing pydantic_argparse will fail with importlib.metadata.PackageNotFoundError: pydantic-argparse.

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

python -m nuitka --version

1.1.6
Commercial: None
Python: 3.9.13 (main, Oct 19 2022, 16:06:06)
Flavor: Unknown
Executable: /tmp/env/bin/python
OS: Linux
Arch: x86_64
Distribution: Ubuntu (based on Debian) 18.04.5

How did you install Nuitka and Python

I am using pyenv to install Python 3.9.13. I am using venv to create a clean environment.

The specific PyPI names and versions

python -m pip freeze

Nuitka==1.1.6
pydantic==1.10.2
pydantic-argparse==0.6.0
typing_extensions==4.4.0

Example

# file: program.py
import pydantic_argparse

print("hello")

After compilation, the execution stops with this error:
Traceback (most recent call last):
  File "/tmp/program.dist/program.py", line 1, in <module>
  File "/tmp/program.dist/pydantic_argparse/__init__.py", line 12, in <module pydantic_argparse>
  File "/tmp/program.dist/pydantic_argparse/__version__.py", line 18, in <module pydantic_argparse.__version__>
  File "/tmp/program.dist/importlib/metadata.py", line 559, in metadata
  File "/tmp/program.dist/importlib/metadata.py", line 196, in from_name
importlib.metadata.PackageNotFoundError: pydantic_argparse

The line that triggers this inside pydantic_argparse is:
__title__: str = metadata.metadata(__package__)["name"]

Provide in your issue the Nuitka options used

python -m nuitka --standalone --follow-imports program.py

Note if this is a regression

This does not seem to be a regression, the bug is present in 0.9, 1.0.8, 1.1.5 and 1.1.6

I edited the description as it was referring to a similar issue (#1745), but not the one I was thinking of (#1793).

i don't get what could be causing this. Code looks very innocent and should be covered, as a hard import with compile time resolution.

So, there is a mismatch between the distribution name, and the package name. Nuitka correctly conserves the distribution object, but in order create find things like a runtime path of it, it searches for the name inside the embedded modules, and pydantic-argparse is not one of them, but somehow it was resolved to that.
I need to figure out where that is happening, and if a trivial - -> _ replacement will generally work, or if there is another thing to persist, namely the package name to use for internal lookup.

Actually, we were using the metadata name, and just need to keep track of the original name and use that instead, then it works.

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 if this is working is very welcome, just please do not share plans of doing it, but rather confirmations or denials of it working.

This is hotfix material, but there might be no 1.1.8,since 1.2 is right around the corner.

Thanks for the amazingly quick fix! I confirm that this is working using factory branch.

Part of the current 1.1.8 hotfix release that I just made.


[P2] javascript.fbp test failed on Galileo Gen2

version: v1-beta10 (sha1 114cf120a7f4720c91e8ce1f5b3bdb7020694888 )

Output returned:
ERR:sol-flow ./src/lib/flow/sol-flow-parser.c:706 parse_declarations() Couldn't handle declaration 'MyScript' of metatype 'js'

Complete output:
https://gist.github.com/anonymous/833c3df837f4e8d40304

who compiled soletta? did you use git submodules to download duktape? It is missing and thus was not compiled (js.so)

I was not aware of that, because I based the build in the documentation available for the galileo:
https://github.com/solettaproject/soletta/wiki/Galileo-and-Galileo-Gen2-Instructions
After duktape and other submodules were installed the issue reported here has disappeared.
Thank you.

@mbelluzzo or @anselmolsm, could you update the build instructions to reflect submodules usage?

Added to https://github.com/solettaproject/soletta/wiki/Galileo-and-Galileo-Gen2-Instructions and  https://github.com/solettaproject/soletta/wiki/Build-requirements.


Show all search result in a leaderboard (show all possible matches)

Closes #1775
UI elements

The PR is ready to review


Plot Hawkes process estimated intensity

Do half of #133
Here is the plot generated by the example.
@sharonxu  Do you have any suggestion?

Hey @Mbompr thanks! This is exactly what I had in mind, it looks great.


tenacity.RetryError: RetryError[<Future at 0xffff670e6bb0 state=finished raised ValidationError>] when using --code_review True

I'm getting this error when trying to run with --code_review True
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/tenacity/_asyncio.py", line 50, in __call__
    result = await fn(*args, **kwargs)
  File "/app/metagpt/metagpt/actions/action.py", line 84, in _aask_v1
    instruct_content = output_class(**parsed_data)
  File "/usr/local/lib/python3.9/site-packages/pydantic/main.py", line 341, in __init__
    raise validation_error
pydantic.error_wrappers.ValidationError: 4 validation errors for task
Logic Analysis -> 0 -> 1
  str type expected (type=type_error.str)
Logic Analysis -> 1 -> 1
  str type expected (type=type_error.str)
Logic Analysis -> 2 -> 1
  str type expected (type=type_error.str)
Logic Analysis -> 3 -> 1
  str type expected (type=type_error.str)

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/app/metagpt/startup.py", line 72, in <module>
    fire.Fire(main)
  File "/usr/local/lib/python3.9/site-packages/fire/core.py", line 141, in Fire
    component_trace = _Fire(component, args, parsed_flag_args, context, name)
  File "/usr/local/lib/python3.9/site-packages/fire/core.py", line 466, in _Fire
    component, remaining_args = _CallAndUpdateTrace(
  File "/usr/local/lib/python3.9/site-packages/fire/core.py", line 681, in _CallAndUpdateTrace
    component = fn(*varargs, **kwargs)
  File "/app/metagpt/startup.py", line 68, in main
    asyncio.run(startup(idea, investment, n_round, code_review, run_tests, implement))
  File "/usr/local/lib/python3.9/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/usr/local/lib/python3.9/asyncio/base_events.py", line 647, in run_until_complete
    return future.result()
  File "/app/metagpt/startup.py", line 47, in startup
    await company.run(n_round=n_round)
  File "/app/metagpt/metagpt/software_company.py", line 60, in run
    await self.environment.run()
  File "/app/metagpt/metagpt/environment.py", line 67, in run
    await asyncio.gather(*futures)
  File "/app/metagpt/metagpt/roles/role.py", line 240, in run
    rsp = await self._react()
  File "/app/metagpt/metagpt/roles/role.py", line 209, in _react
    return await self._act()
  File "/app/metagpt/metagpt/roles/role.py", line 168, in _act
    response = await self._rc.todo.run(self._rc.important_memory)
  File "/app/metagpt/metagpt/actions/project_management.py", line 184, in run
    rsp = await self._aask_v1(prompt, "task", OUTPUT_MAPPING, format=format)
  File "/usr/local/lib/python3.9/site-packages/tenacity/_asyncio.py", line 88, in async_wrapped
    return await fn(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/tenacity/_asyncio.py", line 47, in __call__
    do = self.iter(retry_state=retry_state)
  File "/usr/local/lib/python3.9/site-packages/tenacity/__init__.py", line 326, in iter
    raise retry_exc from fut.exception()
tenacity.RetryError: RetryError[<Future at 0xffff670e6bb0 state=finished raised ValidationError>]

Sorry for not providing more details, I'm absolutely clueless about the code base. I'd love to help, but I don't quite know how I can be of more help here.

The following piece of code is a result of LLM not returning the desired output as per the prompt's requirements. I recommend using GPT-4.
pydantic.error_wrappers.ValidationError: 4 validation errors for task
Logic Analysis -> 0 -> 1
  str type expected (type=type_error.str)
Logic Analysis -> 1 -> 1
  str type expected (type=type_error.str)
Logic Analysis -> 2 -> 1
  str type expected (type=type_error.str)
Logic Analysis -> 3 -> 1
  str type expected (type=type_error.str)

I am using GPT 4?
...
OPENAI_API_MODEL: "gpt-4"
...
### for Research
MODEL_FOR_RESEARCHER_SUMMARY: gpt-3.5-turbo
MODEL_FOR_RESEARCHER_REPORT: gpt-3.5-turbo-16k

It looks like you're using GPT-4 for the configuration.
This version of Software Company is executed step by step in a waterfall fashion, and if any step in the middle encounters an error, the entire process terminates abnormally. A version that supports iteration is still under development and has not been released yet.


My templates disappeared from the global .fttemplates folder

Hi,
All has been well until I ran CleanMyMac this morning. I just discovered that all my templates in the global . fttemplates directory have disappeared or were deleted by CleanMyMac.
Is it possible to have this fttemplates directory in another folder that probably contains all my projects (since the template is reused by all projects), instead of duplicating the template in each project folder?

Hi,
All has been well until I ran CleanMyMac this morning. I just discovered that all my templates in the global . fttemplates directory have disappeared or were deleted by CleanMyMac.
Is it possible to have this fttemplates directory in another folder that probably contains all my projects (since the template is reused by all projects), instead of duplicating the template in each project folder?

Apologies, CleanMyMac is not the culprit. I created some folders again then ran CleanMyMac, and the folders are still there. Any idea why the initial ones disappeared?

This is the second time i've heard about this in a short timespan. I don't know yet what is causing this and thought this might have been a user error but maybe it isnt.
@green-anger sorry to ping you but did you have global templates as well? Or were your deleted templates local ones in a project

Hey @Huuums, no problem.
I work on Win10 and yes I'm using the global folder, current path is ~.vscode\extensions\huuums.vscode-fast-folder-structure-3.6.1.fttemplates
The plugin folder name includes the current version (-3.6.1 in this case), I'm 100% sure it was different with the previous version (don't remember which exactly, maybe -3.6.0). So, my guess is VS Code removed the previous plugin version folder and created a new one and it's empty by default. I have no idea if it can be managed in the settings or has something to do with the plugin itself.
Hope that helps.

Thank you very much. I hope you have backed up the templates this time because a 3.7.0 update rolled out. Man that's a bummer.
I will investigate a little further and hopefully can figure out how to prevent this. But just to be on the safe side I will also give the option to configure your own global template folder path with the next version.
I'm really sorry this happened to you two and anyone else that might come in here.

I think I kind of understand what's happening but I'm also not 100% sure because I can't test this properly without potentially breaking more things for more users.
I think that because I didn't use the .vscodeignore file correctly and had an .fttemplates folder in my extension project this might have overridden existing folders because my empty folder was also published. That would mean that this problem arose out of pure ignorance on my side because I didn't know how the publishing process works in detail.
I thought the publishing process bundled everything into a single js file which is then the extension that gets installed. Turns out that's not how it is and it takes everything that I don't ignore in that file and publishes it as part of the extension.
Once again I can only say I'm sorry.
That being said. Now that I did put that into the ignore file I'm still not 100% sure that it won't happen again. It might still have to do with the versioning number of the extension folder jumping around and deleting things due to that.
So with the next version which will drop shortly you will be able to set a custom global template folder path which I highly encourage to use.
Please also backup your global templates to another folder for now.

Thank you so much @Huuums for your time. I'll look forward to the update.  Your plugin is really a life saver.
In the meantime, I have set the global folder to automatically sync to my Google Drive.
Cheers & Happy Holidays.

No problem. Thanks for taking the time to reach out.
Happy holidays to you too.
Update should be out. You can set your new custom global directory with the command "Set custom global folder templates directory" or just put the path into the settings directly

@Huuums FYI, I didn't notice your message about the new update, ran VS Code and the plugin was updated automatically, the previous version didn't have any global templates, so it's not clear if it'd have been copied or moved. However, now I have two different folders:
huuums.vscode-fast-folder-structure-3.7.0
huuums.vscode-fast-folder-structure-3.8.0
So, maybe it wouldn't have been removed this time. I added some content to the default global template folder and will see how it'll work next time.
Cheers!


Detect read-only mode and apply consequences

In read-only mode, user can only --list tags, and it happens by default.
In non-read-only mode, user selects the path to the output file (or --in-place), and is able to run any operation. The thing is that --list no longer happens by default.

This needs to be implemented, probably as a sort of a post-action in options.cc.

Closed by e1d954388e68add825e0fd84b5a7c680a7127098


Add a new test case of rescind the VF device from guest

Test result:
03/29/2019 08:48:22 : [INFO ]
[LISAv2 Test Results Summary]
Test Run On           : 03/29/2019 08:36:38
ARM Image Under Test  : canonical : ubuntuserver : 18.04-lts : Latest
Total Test Cases      : 1 (1 Passed, 0 Failed, 0 Aborted, 0 Skipped)
Total Time (dd:hh:mm) : 0:0:11
ID TestArea             TestCaseName                                                                TestResult TestDuration(in minutes)
1 SRIOV                SRIOV-HOTPLUG-VF-DEVICE                                                           PASS                 7.24

@xian123 please also test this with RHEL 7.x and SLES15.
Not necessarily to post test results, but more to ensure that the automation works with those distros as well.

As @chvalean said, please test RHEL 7.5 and SLES 15. We will merge after

@chvalean The test supports Ubuntu, RHEL 7.x and SLES15.


Feature-gate zstd so we can target wasm32-unknown-unknown

Another option might be switching to zstd-rs, but I haven't tested this and it seems that there would be a large performance penalty.
People who need compression and wasm can probably recompress with flate, so this seems like a good solution to me.

There is also this approach, using a pure rust zstd: https://github.com/YushiOMOTE/rs-tiled/commit/acf85975f85fcc3e6636e2b0c989431634aa5952

There is also this approach, using a pure rust zstd: https://github.com/YushiOMOTE/rs-tiled/commit/acf85975f85fcc3e6636e2b0c989431634aa5952

Thanks!


fix: webpack 5 compatibility

Please explain the changes you made here.
Webpack 5 now requires the file extension when importing (see docs) so added the file extension to relevant imports. Create React App is now using Webpack 5 and adding compatibility would help resolve issues seen in the react-square-web-payments-sdk library that Square featured in its latest Square Developer Sandbox 101 video.
I'm mainly a frontend/backend Swift developer so I'm a bit out of my element here but after I made these changes and confirmed it fixed a sample Create React App project, Jest started to throw errors that it couldn't find the modules I added the .js file extension to. I added jest-ts-webcompat-resolver to Jest's config which resolved the issue but completely open to a better solution.
Does this close any currently open issues? No

[x] Individual Contributor License Agreement (CLA) signed

Thank you so much for the PR. I ran the test suite and there's a few failures. Would you be able to resolve those?
Happy to re-review once they are passing.

@JHKennedy4 Think I got it now — could you re-run the test suite?


chore: support Python 3.13

Active support for Python 3.13

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.37%. Comparing base (012b7a3) to head (4ca26ff).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #370   +/-   ##
=======================================
  Coverage   84.37%   84.37%
=======================================
  Files          13       13
  Lines        2432     2432
=======================================
  Hits         2052     2052
  Misses        380      380

Flag
Coverage Δ

coverage
84.37% <ø> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


CustomSerializer for rx_requests

Hi there,
I have not a single project that responses to me via JSON, both for errors and valid responses. So I need to use custom serializer as I did when I used Alamofire.
But, when I start using RxAlamofire, all its requests are using default response serializers.
I have such kind of responses:
{
  "error": 1,
  "errors": {
    "password": "Incorrect username or password."
  },
  "session_id": "null"
}

and for valid result:
{
  "error": 0,
  "session_id_alive":<PHONE_NUMBER>,
  "session_id": "key"
}

Is there any way of creating such rx code, create custom NSError for error > 0 in JSON response (or maybe some other validations in JSON)?
networkManager.rx_JSON(.GET, url)
      .subscribe(onNext: { json in
               log.info(son)
          }, onError: { (error) in
               log.error(error)
      }).addDisposableTo(disposeBag)

Take a look at implementation of rx.json, that should give you an idea how to do this.
public func json(_ method: Alamofire.HTTPMethod,
        _ url: URLConvertible,
        parameters: [String: Any]? = nil,
        encoding: ParameterEncoding = URLEncoding.default,
        headers: [String: String]? = nil
        )
        -> Observable<Any>
    {
        return request(
            method,
            url,
            parameters: parameters,
            encoding: encoding,
            headers: headers
            ).flatMap { $0.rx.json() }
    }


Adding Binary Search in Matrix

🚀 Feature Request
I'm frustrated when I have to search for a target value in a large matrix where both rows and columns are sorted. The traditional linear search takes too long, especially when working with large datasets (e.g., matrices with thousands of rows and columns). The inefficiency of manually checking each element slows down processing time.
I would like an optimized solution using Binary Search in a Matrix. This solution should:
Allow for efficient searching of a target element in a matrix where rows and columns are sorted.
Reduce the search complexity from O(n * m) to something more efficient like O(log(n * m)) or O(n + m).
Be able to handle both 1D-like search or divide-and-conquer strategies in 2D matrices.
Linear Search: Checking each element in the matrix, but this becomes impractical with large matrices, leading to O(n * m) time complexity.
Row-wise Binary Search: Applying binary search row-by-row, but this doesn't fully utilize the matrix's sorted property in both rows and columns.
Heuristic Search: Jumping through rows and columns to minimize search space, but it’s not as optimal or intuitive as binary search.

can u assign me i want to work on this
Would you like to work on this feature?

[ x] Yes
[ ] No

and can u remove the invalid tag


TF Rule: SG should not allow ingress rule from anywhere

Add a new terraform rule:
AWS security group should not allow ingress from anywhere (<IP_ADDRESS>/0 or ::/0)

This rule is covered in cli/assets/terraform.yml and cli/assets/terraform12.yml with rule SG_WORLD_INGRESS


Using label-studio without Internet

Can I use LS without Internet? Or if there are some commands or special arguments that can pass through the login page?

Got it! It seems that when starting LS via label-studio start --username <username> --password <password>, the  should be an email address. Then everything will be OK.


[Snyk] Fix for 2 vulnerabilities

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
:sparkles: Snyk has automatically assigned this pull request, set who gets assigned.
As this is a private repository, Snyk-bot does not have access. Therefore, this PR has been created automatically, but appears to have been created by a real user.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

react/package.json
react/package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

479/1000  Why? Has a fix available, CVSS 5.3
Regular Expression Denial of Service (ReDoS) SNYK-JS-CSSWHAT-1298035
Yes
No Known Exploit

490/1000  Why? Has a fix available, CVSS 5.3
Regular Expression Denial of Service (ReDoS) SNYK-JS-RAMDA-1582370
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.

Commit messages

Package name: @storybook/builder-webpack5
    The new version differs by 250 commits.

05070ca v6.5.0
88c6a62 Update root, peer deps, version.ts/json to 6.5.0 [ci skip]
ca93284 6.5.0 changelog
ff28cd9 6.5.0-rc.1 next.json version file
9e1f519 Update git head to 6.5.0-rc.1, update yarn.lock
3f09d4e v6.5.0-rc.1
82b7ac1 Update root, peer deps, version.ts/json to 6.5.0-rc.1 [ci skip]
4b50e28 6.5.0-rc.1 changelog
0a6e347 Merge pull request #18220 from storybookjs/improve/detection-webpack5
95a5c80 move nextjs detection up
152e441 Merge pull request #18248 from storybookjs/18177-fix-conditional-args-fail-gracefully
2948cae ArgsTable: Gracefully handle conditional args failures
f837e31 Merge pull request #18246 from storybookjs/chore_docs_adds_mdx2_steps
73cf6ba adds MDX 2 docs and gotchas
64b07fe Merge pull request #18244 from storybookjs/chore_docs_cleanup_broken_links
fef1a32 Fixes broken links
82ce9de Merge pull request #18231 from Tomastomaslol/issue-18143-reset-button-broken-for-URL-values
ebf9341 Merge pull request #18038 from bisubus/fix-vue3-tsx
9583cea remove repeated test
1b396fd 6.5.0-rc.0 next.json version file
6cc69b5 Update git head to 6.5.0-rc.0, update yarn.lock
c27fd9e v6.5.0-rc.0
b56b1ce re add reset of args that was not set initially. Extend tests for onResetArgs
19ba77b Update root, peer deps, version.ts/json to 6.5.0-rc.0 [ci skip]

See the full diff

Package name: @storybook/react
    The new version differs by 250 commits.

05070ca v6.5.0
88c6a62 Update root, peer deps, version.ts/json to 6.5.0 [ci skip]
ca93284 6.5.0 changelog
ff28cd9 6.5.0-rc.1 next.json version file
9e1f519 Update git head to 6.5.0-rc.1, update yarn.lock
3f09d4e v6.5.0-rc.1
82b7ac1 Update root, peer deps, version.ts/json to 6.5.0-rc.1 [ci skip]
4b50e28 6.5.0-rc.1 changelog
0a6e347 Merge pull request #18220 from storybookjs/improve/detection-webpack5
95a5c80 move nextjs detection up
152e441 Merge pull request #18248 from storybookjs/18177-fix-conditional-args-fail-gracefully
2948cae ArgsTable: Gracefully handle conditional args failures
f837e31 Merge pull request #18246 from storybookjs/chore_docs_adds_mdx2_steps
73cf6ba adds MDX 2 docs and gotchas
64b07fe Merge pull request #18244 from storybookjs/chore_docs_cleanup_broken_links
fef1a32 Fixes broken links
82ce9de Merge pull request #18231 from Tomastomaslol/issue-18143-reset-button-broken-for-URL-values
ebf9341 Merge pull request #18038 from bisubus/fix-vue3-tsx
9583cea remove repeated test
1b396fd 6.5.0-rc.0 next.json version file
6cc69b5 Update git head to 6.5.0-rc.0, update yarn.lock
c27fd9e v6.5.0-rc.0
b56b1ce re add reset of args that was not set initially. Extend tests for onResetArgs
19ba77b Update root, peer deps, version.ts/json to 6.5.0-rc.0 [ci skip]

See the full diff

Package name: svgo
    The new version differs by 199 commits.

daf2f88 2.3.1
25d0f87 Upgrade css-select from `^3.1.2` to `^4.1.3` (#1485)
59876d8 Remove unused artifacts
68798eb Mark convertStyleToAttrs as `disabled` in README (#1472)
3d79f57 Convert convertPathData to visitor
06110b4 Convert mergePaths to visitor
368a67b Convert removeHiddenElems to visitor
17aaf36 Cleanup collapseGroups tests
e381ccc 2.3.0
09aec37 Implement exclude pattern cli option (#1409)
e3f37ec Add mergeStyles to readme
27bef1a Add "visitor" plugins support (#1454)
19c77d2 Add mergeStyles plugin (#1381)
d89d36e Split regression extracter and runner (#1451)
eb934b4 Serve svg for regressions and cache w3c test suite
7389bcd Override default floatPrecision with global
d08815c Implement simple node clone (#1450)
3d4adb6 Simplify number rendering and fix -0 in path
316a002 Remove hasAttr and hasAttrLocal usages (#1447)
447f82c Convert addAttributesToSVGElement to item plugin (#1448)
13a0ad0 Specify --ignore-path for ESLint (#1443)
e8d563c Fix invalid radix in cli (#1446)
bc5c4ea Add a test for removeXMLNS plugin (#1444)
3390df1 removeOffCanvasPaths: Add one more test (#1445)

See the full diff

Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
👩💻 Set who automatically gets assigned
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

fixed in recent release when we got rid of node-sass


WiFi not connecting if iGate is turned on before WiFi AP

I have my WiFi router and iGate connected to the same power strip. I noticed that if I unplug the power strip and plug it in iGate would not connect to WiFi. After inspecting everything I noticed that this only happens in case when iGate boots up before my router.
When I give power to my iGate it turns on its AP which is normal behaviour. My WiFi takes around 1 minute more than iGate to turn on. After the WiFi Timeout which I have set to 10 min iGate does not connect to WiFi.
In case I plug iGate after my WiFi is turned on iGate normally connects to WiFi and works as it is supposed to.
I have also Beta DiGipeater backup mode turned on if it maybe helps you with diagnosis.
Maybe something like this:
iGate turns on, if there is no WiFi turn on AP.
After the AP timeout check again.
Do you think this could work?
Thanks!
73 de S54B

there is a solution already in github code (2024.06.04)
file : /src/LoRa_APRS_iGate.cpp
line58: uncomment //#define STARTUP_DELAY 5 //min
this will make the igate wait 5 min to try to connect to wifi and wait for the router to have valid internet info

I turned on the delay ad it fixed my issue.
Thank you,
Tilen


Hazard Analysis Peer Review

DUE: Oct 21, 2022 + 2 days of grace period
Team to Review: Team 6, Board Gamers
Repository: https://github.com/Dorps/aiboardgame

Create at least 5 "deep" issues

Not typos or formatting mistakes
Focus on observations based on the Hazard Analysis rubric

Issues are prefixed with Team3 Review:
Issues have the label Review-HazA-Team03
Tag the TA grading Team 6

The requirements are not phrased as statements that the system shall do
Conflicting Scope

"Physical Computer" is listed under the system boundaries and out of scope sections. If you are not in control of the "Physical Computer", we think you should exclude it from your boundaries.

The "Training Model" Recommended Actions do not cover all the Causes of Failure
Missing Assumptions

From the slides, an example assumption: "the operator of your machine isn’t intentionally trying to misuse it".
We think it ie better to state what you assume the "Physical Computer" is capable of

Failures caused by user actions are not failures

You define a hazard as: "A hazard for our AI-based game simulation engine will be any threat, vulnerability, system failure, or potential errors that our system is susceptible to." You also state: "... user error will be considered out of the scope for this document.". However in your FMEA table, you list user actions as causes of failure. You definitions conflict with this therefore, this should not be a cause of failure.

Peer Review Issues:

https://github.com/Dorps/aiboardgame/issues/13
https://github.com/Dorps/aiboardgame/issues/14
https://github.com/Dorps/aiboardgame/issues/15
https://github.com/Dorps/aiboardgame/issues/16
https://github.com/Dorps/aiboardgame/issues/17
https://github.com/Dorps/aiboardgame/issues/18
https://github.com/Dorps/aiboardgame/issues/19
https://github.com/Dorps/aiboardgame/issues/20


Set Initial Page

If the user change the default initial page like this:
final PageController _pageController = PageController(initialPage: 1);
The first page opened is the second one inside the list

@patana93 it is not working when it has 5 tabs and passes initial index 4 and 5

I have fixed the issue.
Please, check.


start refactoring

This is an initial step to refactor the parser. Ultimate goal is to include cli completion and better error messages which part of the docopt are invalid or colliding.

Thanks for doing this, but... did you see the giant "THIS CRATE IS UNMAINTAINED" message at the top of the README?
In general, before submitting huge PRs like this to projects, please file issues to make sure such things are wanted. As it is, since I don't spend much time maintaining this crate, I really cannot justify spending the bandwidth to land a huge refactor. Moreover:

The broader docopt project is unmaintained. I suggest not using it.
This library has worse problems than poor error messages and missing CLI completion. Namely, some inputs are very slow to parse and only String arguments are handled.

In general, I would recommend either moving on from docopt to something else (like clap), or if you insist, fork the project and carry it forward that way.

Not insisting, I just happened to run into some limitations so I just gradually went into the weeds. I did see the disclaimer. Moving on.


change to append instead of create in systemd file / add logrotate

change to append instead of create in systemd file / add logrotate for aggregated dcos logs
Description
https://jira.mesosphere.com/projects/DCOS_OSS/issues/DCOS_OSS-1811
Urgency

[ ] Blocker
[ ] High
[x] Medium

Requirements

Test all commands and procedures. :white_check_mark:
Build content locally and test for formatting/links. :white_check_mark:
Add redirects to dcos-website/redirect-files.
Change all affected versions (e.g. 1.7, 1.8, 1.9, and 1.10). :white_check_mark:
See the contribution guidelines. :white_check_mark:

Can one of the admins verify this patch?

I reviewed this and made some changes:

Remove the journalctl --vacuum-size=10M since this complicates debugging and RCAs
Added Example config for logrotate
Stream-lined the filebeat-journald service

@adyatlov @fabianbaier Could you take a look, it this is correct now?

lgtm


Subject Line

https://sendgrid.com/docs/User_Guide/Transactional_Templates/Using_handlebars.html
One big change that is forced on without any notice:
In old templates, I passed the subject as a sub-property of personalizations.
But now I should pass subject as part of the dynamic_template_data.
This is very big change, also this is bad concept.
Subject is part of mail header, while the handlebar template is building the mail body. You are mixing up things that should NOT be mixed at all.
And this big change is not documented anywhere.
Old style:
{
    "personalizations": [
           "to" : [...to array],
           "cc": [...cc array],
           "substitutes": [..substitutes],
           "subject": "SUBJECT LINE"
    ]
}

New Style:
{
    "personalizations": [
           "to" : [...to array],
           "cc": [...cc array],
           "dynamic_template_data": [..substitutes + it now includes subject!],
    ]
}

hey @axelcho - find the new v3 mail send functionality documented here: https://dynamic-templates.api-docs.io/3.0/mail-send-with-dynamic-transactional-templates/v3-mail-send
You should still be able to specify a subject in personalizations. However, if you have a subject defined in the dynamic template data, it overwrites what is defined in the personalizations - this has always been the customization/substitution behavior.
All existing transactional templates are still viable - you can send, create, and edit them, you will just be given the option to switch from substitutions to dynamic data the next time you go in the UI to edit a template.
If you are still having trouble getting this to work, please get in touch with our awesome support team! They can help out with individual implementation issues. You can access support contact options by logging into https://support.sendgrid.com.


Event button below Join Us does not lead to events section.

In footer section the event button below Join us does not take us to events section.

I fixed this bug please review and give me some suggestions.
PR #108


Upgrade to Smithy 1.26.x

Issue #, if available:
N/A

Description of changes:
Upgrade to smithy 1.26.x using a version range.

Testing:

smithy-go: make smithy-clean smithy-build smithy-publish-local smithy-clean
aws-sdk-go-v2: make

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

Closed in favor of #411.


Project config improvements

I made a few changes to make maintaining this package a better experience.

Simplifications to the project configuration
Changes to linter configuration
Removed redundant linter
Configured dynamic versioning

Looks reasonable, except I don't fully understand how from where the version number in dynamic version comes from.

If the package is built from a commit tagged v1.0.0, the package name will be FVHIoT-1.0.0.tar.gz.
If a change is then made and committed and a package built, it will be FVHIoT-1.0.1.dev1+g9114acc.

The package version is incremented by one minor revision by default since it's something newer than the latest git tag v1.0.0
dev1 means there's one commit on top of v1.0.0
9114acc is the hash of the commit the package was built from
If more commits are made, a timestamp is appended as well

Is it ok to merge? 😃


Ux improvements

This is the first iteration skeleton for the new UI of Forest, its currently in a subdirectory ready to be transitioned into the main application and make use of the Bokeh components.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


inventory changer?

Does anyone have the code for an inventory changer? I want to put it in Osiris because I personally prefer inventory changer to skin changer.

there is a thread on uc (inventory changer no protobuf) very in depth and detailed. do it on your own don t expect anyone to spoonfeed you. oh i forgot to say u need to modify ur skin changer for skins to be applied good luck with that G


More fixes to the way the NASA ADS references are parsed

Hello @mattpitkin! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file psrqpy/utils.py:

Line 708:25: W291 trailing whitespace
Line 718:29: W291 trailing whitespace
Line 731:101: E501 line too long (115 > 100 characters)
Line 755:1: W293 blank line contains whitespace

With this merge it fixes a change related to v1.64 of the psrcat tarball, which reverted the references back to an older style. It also fixes finding ADS entries for a variety of the references. Of the 1063 references in the psrcat_ref file 999 of them can be found in ADS by get_references(). Some of those that aren't founds have incomplete information (missing volume or page number) or are ATel's that don't have ADS entries yet.


Scopes doesn't return for sub organization users when authorization request is initiated without openid scope for a Standard based application

Describe the issue:

Create an OIDC application using Standard based application template and share it with all orgs
Authorize scim2 Users management API and organization API
Create a role for the application with all the permissions of selected APIs in step 2
Create a sub org
Switch to the sub org and create a user
Assign the created user to that application's role in sub org space
Try application login with authorize request
Sample: Don't include "openid" scope

https://localhost:9443/oauth2/authorize?response_type=code&client_id=Eim_7Wr0qpC6xH5ZO6N_fx4yOpca&redirect_uri=https://example-app.com/redirect&scope=internal_org_user_mgt_delete internal_org_user_mgt_update internal_org_user_mgt_view internal_org_user_mgt_create internal_org_user_mgt_list internal_user_mgt_create internal_user_mgt_view internal_user_mgt_list internal_user_mgt_delete internal_user_mgt_update

Upon recieving the authz code try token request

curl --location 'https://localhost:9443/oauth2/token' \
--header 'Content-Type: application/x-www-form-urlencoded' \
--data-urlencode 'code=e4cc80bd-0802-3585-9516-358dc0d623d0' \
--data-urlencode 'grant_type=authorization_code' \
--data-urlencode 'client_id=Eim_7Wr0qpC6xH5ZO6N_fx4yOpca' \
--data-urlencode 'client_secret=ZhWuy2Y3Mw1hEqoseSZgfLFQhp6nZgCs6ze_6ZobEKEa' \
--data-urlencode 'redirect_uri=https://example-app.com/redirect'

Scopes will not be returned.
If you try the same step with a applciation created using traditional web app template / SPA, the flow works fine without openid scope
Expected behavior:
Return the relevant scopes even if the openid scope is not requested in the initial request

This happen only for the started app because,
when creating a standard app, the "username" attribute is not set as requested attribute of the application,
but for other applications tradintional apps, and SPA "username" attribute is set as requested attribute by default.
The standard app flow works fine for
https://localhost:9443/oauth2/authorize?response_type=code&client_id=Eim_7Wr0qpC6xH5ZO6N_fx4yOpca&redirect_uri=https://example-app.com/redirect&scope=internal_org_user_mgt_delete internal_org_user_mgt_update internal_org_user_mgt_view internal_org_user_mgt_create internal_org_user_mgt_list internal_user_mgt_create internal_user_mgt_view internal_user_mgt_list internal_user_mgt_delete internal_user_mgt_update

kind of only API scopes if the root app has requested at least one claim of the profile scope,


fix: update required ibm provider version to < 1.7

/run pipeline

/run pipeline

/run pipeline

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


High loss of accuracy when using TFLite on Android

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Only on Android (modified the TFLiteClassifier.java class to output the results in a more convenient manner and commented the call to applyFilter() so the model doesn't base its detection on other images than the one I am giving to it)
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): OSX 10.13.3
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: Samsung Galaxy S6
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): v1.8.0-0-g93bc2e2072 1.8.0
Python version: Python 3.6.0 :: Anaconda 4.3.1 (x86_64)
Bazel version (if compiling from source): Build label: 0.14.1-homebrew
GCC/Compiler version (if compiling from source): Apple LLVM version 9.1.0 (clang-9<IP_ADDRESS>)
CUDA/cuDNN version: NA
GPU model and memory:  NVIDIA GeForce GT 650M 1 GB
Exact command to reproduce:

cd ../tensorflow-for-poets-2/
python -m scripts.retrain --bottleneck_dir=bottlenecks --how_many_training_steps=$STEPS --model_dir=/Users/jean-baptistebuisson/new_training_dir/model/ --summaries_dir=/Users/jean-baptistebuisson/new_training_dir/training_summaries/mobilenet_1.0_224 --output_graph=/Users/jean-baptistebuisson/new_training_dir/$NAME.pb --output_labels=/Users/jean-baptistebuisson/new_training_dir/retrained_labels.txt --architecture=mobilenet_1.0_224 --image_dir=/Users/jean-baptistebuisson/tf_files/tw --tfhub_module https://tfhub.dev/google/imagenet/mobilenet_v1_100_224/feature_vector/1
cd ../new_training_dir/
cd ../tensorflow/
bazel-bin/tensorflow/contrib/lite/toco/toco --input_file=../new_training_dir/$NAME.pb --input_format=TENSORFLOW_GRAPHDEF --output_format=TFLITE --output_file=$NAME.tflite --inference_type=FLOAT --inference_input_type=FLOAT --input_arrays=input --output_array=final_result --input_shapes=1,224,224,3
cd ../new_training_dir/

Describe the problem
The above commands let me retrain a Mobilenet model with 2 categories (positive and negative) and outputs a .pb and a .tflite model that both work fine when tested with label_image with the same data (100 positive images and 100 negative):
For the .pb model:
True positive: 97; true negative: 98; false positive: 2; false negative: 3
For the .tflite model:
True positive: 100; true negative: 100; false positive: 0; false negative: 0
Then I import the model on Android and run on the same images I get the following result:
True Positive: 100; True Negative: 0; False Positive: 100; False Negative: 0
I don't know what is messing with the detection but on Android everything is detected as positive.
Here are the .pb and .tflite models I am using:
retrainedMNetV1_7_8000.pb.zip
retrainedMNetV1_7_8000.tflite.zip
And here is my modified TFLiteClassifier.java class:
TFLiteClassifier.java.zip

@jbuisson1 : How are you feeding images to the classifier?

Here is the code where I feed an image to the classifier (f is the File of the current image to be fed)
BitmapFactory.Options options = new BitmapFactory.Options()
options.inSampleSize = 8;
final Bitmap b = BitmapFactory.decodeFile(f.getAbsolutePath(), options);
ArrayList<TFLiteRecognition> res = new ArrayList<>();
try {
    res = Globals.lightClassifier.classifyFrame(b);
} catch (IllegalStateException e) {
    Log.d(TAG, "testImage: " + e.getMessage());
}

@jbuisson1 : My suspicion without looking at the images will be that images fed to classifier are not correct or not in the same format as during the training. You can try doing a sanity check of images that you are feeding to the classifier.
Try dumping the images to sdcard after you do
bitmap.getPixels(...) in convertBitmapToByteBuffer.  Check if they match with the images during the training phase after pre-processing.
Let me know if that helps. You can also try running the non-TFLite model on Android and check if there is an accuracy loss. I suspect you will see a loss there as well, which would mean an error in the pre-processing phase.

Hi @jbuisson1. I face the same problem right now and I'm not sure what I'm supposed to do. In your case, do you mean you change the image size in the Android or the model training? Desperate for help...Thanks in advance.

In case anyone has the same problem in the future, the solution for me was to make sure that all dimensions were the same - width, height, IMAGE_MEAN, and IMAGE_STD.
If you are using one of the tensorflow examples, you need to update all these values accordingly. Also make sure the image's size also matches the hardcoded width and height values.

@DoritoDog I see that codelab example (flower classification) shows moblenetv2 training with 0-1 based pre-processing, but in android after tflite conversion in floating point model, they use -1, 1 based preprocessing.
Could you please help by justifying or confirming that its wrong there.


Add support for exporting outputs

This adds support for exporting outputs from the Terraform state after apply.

The custom stream implementation provides a better way of capturing the outputs while also preventing the command output from being logged to stdout before sensitive values are marked as such.


[Feature Request] Add flag to disable signal/update reordering

As described in the docstring update in https://github.com/temporalio/sdk-core/pull/639, the reordering of signals/queries to come first is more a byproduct of language runtime specifics than anything else.
Languages that can control the iteration of routines/tasks/whatever, don't need this reordering, and in fact it prevents one specific case that seems like it should work from working:
Say your history has something like:
...
activity completed
signaled
WFT Sched & Start

If I reorder jobs such that lang sees [signal, activity resolve]
Then code that looks like this:
fn signal_handler() {
    if that_activity_future.is_completed() {
        // do a thing
    }
}

Will not enter the if condition, since the signal handler is invoked first. However it seemingly should, according to history, since the activity is completed before the signal is received.
Rust SDK should make use of the more accurate behavior, since it can. Ideally, Python and .NET would too since they have this control, though that could potentially represent a breaking change. If we could confirm it's nonbreaking, we could do it, but not exactly important.

I generally agree that reordering is not strictly needed, it's there for a couple of reasons (there are probably more cases I missed):

late signal handler registration - in all languages we support late (dynamic) registration, this is the main form of registration in languages like TS where handlers are registered in the body of the main workflow function. We want the signals handlers to be called as soon as they're registered as shown here.
evaluation of conditions - conditions, a.k.a Workflow.await in some languages, are evaluated at key points in the activation. Conditions may have timeouts (as shown below). We prefer letting the condition function win the race with a timer. Whether or not that is the right decision can be debated but that's a choice we made.

export async function conditionRacer(): Promise<void> {
  let blocked = true;
  setHandler(unblockSignal, () => void (blocked = false));
  const unblocked = await condition(() => !blocked, '1s');
  assert.true(unblocked);
}

Can we confirm Java SDK behavior here?
Also, if we allow relying on the ordering of non-signal/update and signal/update event processing in a single task, I think switching signal registration from predefined to first-line-in-workflow defined but left everything else the same could represent a non-deterministic change, correct? Maybe that's ok to be a non-deterministic change.

Python and .NET would too since they have this control, though that could potentially represent a breaking change. If we could confirm it's nonbreaking, we could do it, but not exactly important.

I think it would be breaking to no longer order signals/updates before other non-query-non-patch events. IIRC the only reason I introduced ordering in Python SDK was to get patches and queries in the right place, so I just mimicked TS behavior. But it is set now.


cytoscape-html-label.d.ts is not a module after importing it in a component (angular 7)

When importing cytoscape-html-label as a module, so import htmlLabel from cytoscape-html-label, there is an error in the angular saying that cytoscape-html-label.d.ts is not a module

Wrong d.ts file. Please, rebuild and republish. @kaluginserg

Any updates on this issue? I would really like to use this extension for an angular project.

Hello all
I fixed it exporting the interface CytoscapeNodeHtmlParams in cytoscape-node-html-label.d.ts file.
But I should need a new version that fix this bug in order to use it in my project.

Hello, I have the permissions to publish new versions, but not much time to do so.
Is the fix already in master? If not, please send a PR I'll try to publish a new version with the fix.

Does simply adding the export to the interface fix the problem?

Yes that's it but I could be good to have a new use case.

Hello all,
What about npm? I can't get the new version from there. Thnaks again

Released version 1.2.0

I have the same issue with angular 9. Is there any fix available? I want to use this in an angular project.

Does version 1.2.0 has the same error?

Yes, 1.2.0 and angular 9.

I confirm: still not working on 1.2.1.
import * as nodeHtmlLabel from 'cytoscape-node-html-label' does not work, with the same error cytoscape-html-label.d.ts is not a module

I confirm: still not working on 1.2.1.
import * as nodeHtmlLabel from 'cytoscape-node-html-label' doesn't work, with the same error: "cytoscape-html-label.d.ts is not a module".
Using import 'cytoscape-node-html-label' with declare var nodeHtmlLabel compiles but it says that nodeHtmlLabel is not defined and even .nodeHtmlLabel as function of cyInstance is not defined.

I found a way to solve it, this is my "cytoscape-node-html-label.js" from v1.2.1:
declare type IHAlign = "left" | "center" | "right";
declare type IVAlign = "top" | "center" | "bottom";

interface NodeRequireFunction {
  (id: string): any;
}

interface NodeModule {
  exports: any;
  require: NodeRequireFunction;
  id: string;
  filename: string;
  loaded: boolean;
  /** @deprecated since 12.19.0 Please use `require.main` and `module.children` instead. */
  parent: NodeModule | null | undefined;
  children: NodeModule[];
  /**
   * @since 11.14.0
   *
   * The directory name of the module. This is usually the same as the path.dirname() of the module.id.
   */
  path: string;
  paths: string[];
}

declare var module: NodeModule;
declare var define: any;
declare var cytoscape: typeof cytoscape;
interface CytoscapeNodeHtmlParams {
    query?: string;
    halign?: IHAlign;
    valign?: IVAlign;
    halignBox?: IHAlign;
    valignBox?: IVAlign;
    cssClass?: string;
    tpl?: (d: any) => string;
}
interface CytoscapeContainerParams {
    enablePointerEvents?: boolean;
}

declare module 'cytoscape-node-html-label'

I've added interface NodeRequireFunction and interface NodeModule from "global.d.ts" to be able to modify declare var module: any as declare var module: NodeModule.
Then I've added declare module 'cytoscape-node-html-label' and this is the main thing to be recognized as module from Angular.
Then you can import them as usual:
import * as cytoscape from 'cytoscape';
import * as nodeHtmlLabel from 'cytoscape-node-html-label';


BAU - Replace black, flake8, isort with ruff

WIP, still need to...

Manually fix issues that can't automatically be fixed
Go through unsafe fixes and ensure non-breaking

Closing as we have a bunch of conflicts now, seems simpler to spin up a new PR


Add a GPT-J example using Intel Neural Compressor

Describe your changes
Add a GPT-J example using Intel Neural Compressor. It aims to demonstrate the capabilities of INC on large language models.
Enhance format_input function olive_evaluator.py to support list type input_data.
Checklist before requesting a review

[ ] Add unit tests for this change.
[ ] Make sure all tests can pass.
[x] Update documents if necessary.
[x] Format your code by running pre-commit run --all-files
[ ] Is this a user-facing change? If yes, give a description of this change to be included in the release notes.

(Optional) Issue link

/azp run

/azp run

@guotuofeng Hi, could you please trigger the test for me? Thanks!

/azp run

/azp run

@yuwenzho can you merge main or push an empty commit so that we can start a new ci run with the latest changes in main?

@yuwenzho can you merge main or push an empty commit so that we can start a new ci run with the latest changes in main?

I rebased main. Please help to trigger the test again. Thanks a lot!

/azp run

/azp run

Closing PR to fix the pipeline run issue. Will reopen immediately.

/azp run

@yuwenzho perhaps the force push is confusing the azp bot. Can you push an empty commit so that we have a new commit. That might fix the pipeline run issue.

@yuwenzho perhaps the force push is confusing the azp bot. Can you push an empty commit so that we have a new commit. That might fix the pipeline run issue.

I pushed an empty commit.

/azp run

@yuwenzho perhaps the force push is confusing the azp bot. Can you push an empty commit so that we have a new commit. That might fix the pipeline run issue.

I pushed an empty commit. Please help to trigger the test again. Tnanks!

That didn't fix it. I think we might have to close this PR and open a new one. Looks like some bug from github or the azp bot.
Can you close this and create a new one?

@yuwenzho perhaps the force push is confusing the azp bot. Can you push an empty commit so that we have a new commit. That might fix the pipeline run issue.

I pushed an empty commit. Please help to trigger the test again. Tnanks!

That didn't fix it. I think we might have to close this PR and open a new one. Looks like some bug from github or the azp bot.
Can you close this and create a new one?

Sure


Incorrect syntax highlighting when using HTML comments in Javascript section

Given the following snippet:
console.log("This is executed (A)"); <!-- This is a line comment.
--> This is also a line comment. Even if I add this: /*
console.log("This is executed (B)");

Code Mirror renders the following:

The expected highlighting would be that past the <!-- and the whole second line are green, while the third line is styled as the first console.log
You can reproduce this issue on https://codemirror.net/demo/theme.html and pasting the above snippet.
This issue was originally reported on https://bugs.chromium.org/p/chromium/issues/detail?id=920367 which also links to the ECMAScript standard: https://www.ecma-international.org/ecma-262/#sec-html-like-comments

Similar to a MultiLineComment that contains a line terminator code point, a SingleLineHTMLCloseComment is considered to be a LineTerminator for purposes of parsing by the syntactic grammar.

I'm kind of wondering if the person reporting that ran into a real problem or was intentionally trying out corner cases, since it's been really long since I've seen such comments in actual source code, but okay, attached patch adds support.

Thank you for the quick resolution!


[FEATURE] Reactive version of the clients

Is your feature request related to a problem?
Hi there, and apologies if it is too early/it's not on your roadmap/ has already been discussed elsewhere and I have missed it.
First of all thanks for all the hard work, I have been eagerly following your work on spring-data-opensearch and already started looking into how this library evolves. We are currently using an older version of Opensearch (1.2.4) alongside the spring-data-elasticsearch library. While playing around with the the merged code, I noticed that there are no equivalents of the reactive versions of the client/configuration. Is this something that you plan to add?
What solution would you like?
Support reactive version of the clients
What alternatives have you considered?
N/A
Do you have any additional context?
N/A

@ymamakis this is a valid feature, worth keeping on roadmap for sure, thank you.

Hello, any news on this subject? We really want to use the opensearch client, but I'm leaning towards moving to the spring data elasticsearch for the reactive connection. Do we have an ETA @wbeckler?
Thanks a lot!

@cunhap we ended up with using WebClient + AWS request signing

Given that it's all Apache 2.0-licensed code, could we just copy the Reactive* clients from https://github.com/spring-projects/spring-data-elasticsearch/tree/main/src/main/java/org/springframework/data/elasticsearch/client/elc and tweak to use the OpenSearch REST high-level client?

@msfroh please check https://github.com/opensearch-project/spring-data-opensearch/pull/227, we are getting there