Support for RPC calls

Hi everyone! Are there any plans to have this plugin also able to make RPC calls?
As our current PoC is being develop, we're looking into a way for having a REST API exposed to clients (eg: web app or third party devs) while having all our backends behind rabbitmq.
In order to do so, and because the plugin right now only supports fire n forget, we're looking into enabling RPC inside this plugin so we can leverage all Rabbit's flavors (such as autoscaling and so on) instead of plugin directly our services to kraken.
Hope this makes sense, i'll start working on a fork of this repo, but wanted to first check out if you guys had any plans for this.
Thanks again!

Hi @kpacha any ideas on this? as i said, i'm currently working on something to do this, but would be awesome to have some degree of official support for this as well...
Thanks a lot!

Hi @kpacha any ideas on this? as i said, i'm currently working on something to do this, but would be awesome to have some degree of official support for this as well...
Thanks a lot!

In all the times I've been thinking about what features should a pure api gateway offer, I never thought adding RPC over amqp. This is a comment and not a down vote for the feature proposal. The required deps are already included, so if the implementation makes sense, I'm ok merging it to the main branch.
In my opinion, the implementation will raise some questions regarding the semantics (at least one, at most one) and the queue management (e.g.: what happens when the client is disconnected when the response is available at the queue?). What are your ideas on that?

In all the times I've been thinking about what features should a pure api gateway offer, I never thought adding RPC over amqp. This is a comment and not a down vote for the feature proposal. The required deps are already included, so if the implementation makes sense, I'm ok merging it to the main branch.
In my opinion, the implementation will raise some questions regarding the semantics (at least one, at most one) and the queue management (e.g.: what happens when the client is disconnected when the response is available at the queue?). What are your ideas on that?

Thanks for your answer @kpacha!
It raises a lot of questions for sure, we're currently developing a PoC so this is not the only approach we're using -basically we want to compare kraken -> gprc -> service, kraken -> amqp-rpc -> service, kraken -> rest-json -> service, and the same transport layers with Kong as our gateway- benchmark everything and make a decision based on performance, effort, scalability, and so on.
Regarding your example, we're assuming it's the same as if the client is disconnected in a regular rpc/rest-json comm, in order to avoid population of not consumed msgs and queues we're going to set these ones as not durable/autodelete so they get deleted if no one consumes them.
Background reasoning is to try to avoid having different types of communication if possible (eg: services having both REST APIs and message-driven APIs), but maybe it ends up being a no-go, we're still unsure tbh.

Thanks for your answer @kpacha!
It raises a lot of questions for sure, we're currently developing a PoC so this is not the only approach we're using -basically we want to compare kraken -> gprc -> service, kraken -> amqp-rpc -> service, kraken -> rest-json -> service, and the same transport layers with Kong as our gateway- benchmark everything and make a decision based on performance, effort, scalability, and so on.
Regarding your example, we're assuming it's the same as if the client is disconnected in a regular rpc/rest-json comm, in order to avoid population of not consumed msgs and queues we're going to set these ones as not durable/autodelete so they get deleted if no one consumes them.
Background reasoning is to try to avoid having different types of communication if possible (eg: services having both REST APIs and message-driven APIs), but maybe it ends up being a no-go, we're still unsure tbh.

@kpacha i've been hitting an issue with krakend and i'm not sure how to move forward.
2021/02/01 11:53:05  WARNING: loading plugins: plugin loader found 1 error(s):
opening plugin 0 (./krakend-amqp-rpc.so): plugin.Open("./krakend-amqp-rpc"): plugin was built with a different version of package errors

My local go bin version is 1.12, same one as required by krakend as far as i can see. Do you have any ideas on why is this happening?
I've also tried building krakend from the repo, but the make build command is throwing a lot of "non found" dependency issues
Thanks for any input!

there is a small tool for checking the deps and versions for krakend plugins: http://plugin-tools.krakend.io/validate
by the way, go1.12 is too old for building krakend plugins. krakend 1.0.0 was compiled with go1.13.0

@kpacha This simple plugin is not using any external dependencies (only go pkg ones) so there's no .sum file, and even if i add any, the validator (which i've already used) says it is ok.
Depending on my go version (i've tried 1.11, 1.12, 1.13 and 1.15) i get the same error but against diff dependencies.
right now it says
2021/02/01 13:10:27  WARNING: loading plugins: plugin loader found 1 error(s):
opening plugin 0 (./krakend-amqp-rpc.so): plugin.Open("./krakend-amqp-rpc"): plugin was built with a different version of package internal/cpu

Still clueless :( any help appreciated

The validator also tells you which go version should you use to compile the plugin depending on your krakend version. http://plugin-tools.krakend.io/versions/v1.2.0
1.11.0, 1.12.0 and 1.15.0 will never work because these versions haven't been used in our compilations

Got it, issue was related to a minor version diff between my go and go version used for krakend.
KrakenD 1.2.0 was built using Go version: 1.15.2, mine was 1.15.5 thus it was throwing all kinds of errors. Now i'm on track (i hope). Thanks for all your help! will keep posted regarding this plugin


Incorrect output.path leads to compiler not finishing

I could not really figure out how this happens but it bugged me for days
I accidentally set the output.path to a string "./C:\some\path" which is obviously wrong
Unfortunately, the compiler instance did not finish (neither calling the callback function nor exiting nor crashing).
This should really be reported as an error by the compiler

@TheLarkInn I'd like to tackle this one! :shipit:

Output now should validate via schema.


Chained conditionals

Does Handlebars.net support chained conditionals (http://handlebarsjs.com/block_helpers.html)?
{{#if isActive}}

{{else if isInactive}}

{{/if}}

This is not currently supported, but it would be pretty easy to add to the existing conditional block builder.

Updated NuGet to 1.6.1

Cursory tests look great. Thanks -- really appreciate your work on this project.


Error get_num_tokens_from_messages() with gpt-3.5-turbo-0613

System Info
Windows(server on linux), python, poetry
Who can help?
@hwchase17
@agola11
Information

[X] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[X] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[X] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[X] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
Langchain has problems with working with model casts
 File "C:\Users\zabre\PycharmProjects\sales-bot\bot\src\sales_bot\text_handler.py", line 445, in get_response
    memory.prune()
  File "C:\Users\zabre\AppData\Local\pypoetry\Cache\virtualenvs\sales-bot-f2_0xw7_-py3.11\Lib\site-packages\langchain\memory\summary_buffer.py", line 65, in prune
    curr_buffer_length = self.llm.get_num_tokens_from_messages(buffer)
                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\zabre\AppData\Local\pypoetry\Cache\virtualenvs\sales-bot-f2_0xw7_-py3.11\Lib\site-packages\langchain\chat_models\openai.py", line 420, in get_num_tokens_from_messages
    raise NotImplementedError(
NotImplementedError: get_num_tokens_from_messages() is not presently implemented for model gpt-3.5-turbo-0613.See https://github.com/openai/openai-python/blob/main/chatml.md for information on how messages are converted to tokens.

For example, when working with gpt-3.5-turbo-0613, he is not able to count the number of tokens, because the moment of checking the model is hard-coded
Please correct the following in def get_num_tokens_from_messages(langchain\chat_models\openai.py)":
        if model == "gpt-3.5-turbo":
            # gpt-3.5-turbo may change over time.
            # Returning num tokens assuming gpt-3.5-turbo-0301.
            model = "gpt-3.5-turbo-0301"
        elif model == "gpt-4":
            # gpt-4 may change over time.
            # Returning num tokens assuming gpt-4-0314.
            model = "gpt-4-0314"

Replase:
        if "gpt-3.5-turbo" in model:
            # gpt-3.5-turbo may change over time.
            # Returning num tokens assuming gpt-3.5-turbo-0301.
            model = "gpt-3.5-turbo-0301"
        elif "gpt-4" in model:
            # gpt-4 may change over time.
            # Returning num tokens assuming gpt-4-0314.
            model = "gpt-4-0314"

Expected behavior
Sorry, because the project is closed, there is no way to provide sources, but the problem arises when using blind models and when calculating the number of tokens
memory = ConversationSummaryBufferMemory(
    llm=self.analyse_llm,
    ai_prefix="1",
    human_prefix="2",
    max_token_limit=self.max_memory_token_limit,
)

#  loaded old memory, and check length with prune causes an error

memory.prune()

Sorry, my version of the library was older, there is no such error in the new one


Touch screen 90 degrees out

I don't know whether this is the right place to ask this but I'm having some trouble with this touch screen (https://www.amazon.co.uk/gp/product/B075JFT9SF) which I want to use with my Raspberry Pi 2 and pi-scan. The touch screen works fine with a normal raspian installation and is calibrated very well however does not work at all with the 'touch screen' version you supplied (I understand this is for the raspberry pi touch screen only) or the mouse version. I have removed the read only permission from the / and /boot mounts and installed the drivers for the screen (https://github.com/goodtft/LCD-show) and the screen resolution is now fine but the touch screen is 90 degrees out - when I press the top right, it shows in the bottom right and when I press on the bottom right it shows in the bottom left etc. I realise this is a very specific issue but could you help at all?

I don't know what the interaction is between the Kivy libraries (this is what is used for the touch-screen UI) and your screen drivers. So I can't help directly. However, if you want to take a vanilla image and install all the pre-requisites for Pi Scan yourself, you can do the following:

Set up locale/keyboard stuff (unless you want to leave it as Great Britain keyboard locale)

Install these apt-get packages:

python-setuptools python-dev pkg-config python-pip build-essential python-pil
git-core
dbus python-dbus udisks2 exfat-fuse
liblua5.2-0 lua5.2 liblua5.2-dev libusb-dev
build-essential libltdl-dev libusb-dev libexif-dev libpopt-dev libudev-dev pkg-config git
automake autoconf autopoint gettext libtool wget

Install gphoto2 (for latest version, install here instead of from apt-get https://github.com/gonzalo/gphoto2-updater )

pip install -I Cython==0.25.2

pip install lupa --install-option="--no-luajit"

ldconfig
udev_version=$(udevd --version)
if   [ "$udev_version" -ge "201" ]
then
udev_rules=201
elif [ "$udev_version" -ge "175" ]
then
udev_rules=175
elif [ "$udev_version" -ge "136" ]
then
udev_rules=136
else
udev_rules=0.98
fi

Install wiringpi

Install Kivy dependencies:

libsdl2-dev libsdl2-image-dev libsdl2-mixer-dev libsdl2-ttf-dev
libgl1-mesa-dev libgles2-mesa-dev
libgstreamer1.0-dev
gstreamer1.0-plugins-bad gstreamer1.0-plugins-base gstreamer1.0-plugins-good gstreamer1.0-plugins-ugly
gstreamer1.0-omx gstreamer1.0-alsa libmtdev1 libmtdev-dev
xclip

Install Kivy

apt_file="/etc/apt/sources.list.d/mitako.list"
apt_entry="deb http://archive.mitako.eu/ jessie main"
apt_key="http://archive.mitako.eu/archive-mitako.gpg.key"
apt_cache="/var/lib/apt/lists/archive.mitako.eu_dists_jessie_main_binary-armhf_Packages"
echo "Registering pipaOS repository: $apt_entry"
echo "$apt_entry" | sudo tee $apt_file > /dev/null
echo "Adding repository key $apt_key"
curl -L "$apt_key" | sudo apt-key add -
echo "Refreshing available packages"
apt-get update
apt-get install python2-kivypie

Install configuration files. See attached.

files.zip
Now you should be able to play around with the kivy options in .kivy/config.ini (default options are at: https://github.com/Tenrec-Builders/pi-scan/blob/master/config/touch.ini ). The read-only stuff isn't installed either so you don't need to muck around with remounting to modify stuff. But you do need to remember to shutdown cleanly. Once you have figured out the driver/kivy issues, please post here and I may integrate those options back into the main code base.
Best of luck.

First of all, thanks for the comprehensive response! I am a bit of a noob with Linux so I may need to pick your brains about the finer points of the installation. Am I OK doing that on here or via email?

Probably best to useemail if you want to get into the weeds. You can ask
any questions at help at tenrec dot builders.
On Mon, Apr 16, 2018, at 12:35 PM, computinginschools wrote:

First of all, thanks for the comprehensive response! I am a bit of a
noob with Linux so I may need to pick your brains about the finer
points of the installation. Am I OK doing that on here or via email?> — You are receiving this because you commented. Reply to this email
directly, view it on GitHub[1], or mute the thread[2].>

Links:

https://github.com/Tenrec-Builders/pi-scan/issues/16#issuecomment-381705539
https://github.com/notifications/unsubscribe-auth/AA1sTJwxvLkou-Ncg9905gqbIQsWVUlCks5tpORpgaJpZM4TSlVi

Just one other thought on this. As a workaround, just rotate the screen and control it with a keyboard. Every button on the interface has a labelled hotkey. So you can do everything you need without actually using the touch screen at all.


VP/BE 220 rpc supply balance and borrow balance

Description:
Implement custom RPC for fetching supply balance and borrow balance

Update documentation: https://minterestfinance.atlassian.net/wiki/spaces/MINTEREST/pages/134807557/Minterest+API.
Master branch updated to Substrate 3.0.0. You also need to update your PR.


v4 Draft: Add url prefix. like http://localhost:10086/url_prefix

Hi @donaldzou
I'm looking for way to configure this app with reverse proxy and use it wit some configurable url prefix like
http://localhost:10086/url_prefix
I am not good enough at VueJS but can you check this pull request and see if it is possible or acceptable for you to add such functionality

The python is easily configured by using

sudo "$venv_gunicorn" --config ./gunicorn.conf.py --env SCRIPT_NAME=${URL_PREFIX}

But front-end needs prefix like variable and not absolute path for some resources

So please look at this, and maybe it will be possible to implement this. This is a very common and useful feature that allows use application on some common host but with custom URL prefix.
PS: I tested this variant - it's works, but the code maybe done not in proper way.

Hi! I followed your idea made an update to v4 with adding an app_prefix into the wg-dashboard.ini without the need to change every files. Currently it can only be change manually.
More details: https://github.com/donaldzou/WGDashboard/commit/a30843cff979fecbd6bed9d385dcc9763a2b9c78
To to add the prefix, you can simply edit the wg-dashboard.ini, add a new entry under the [Server] section
[Server]
...
app_prefix = /donaldzou/1234

Remember to not add slash after the prefix. The app will add one automatically.
Let me know if this works :)

And it even works with cross-server

Hi, @donaldzou
Thanks a lot for your job, it works.
I added it inside docker, added env vars to contorl this and tested on real host.
And it perfectly works with IP bit not host name
Because of CORS
Can you suggest how this can be fixed?

@donaldzou
Thanks a lot for your job.
It works.
I am testing it now using docker and traefik rules. will let you know.
Thanks

@donaldzou
Can you check the protocol (http/https) in prefix URL in front-end
In case it proxy over http -> https - it requests API calls to http
Mixed Content: The page at 'https://myhost/wgdashboard/#/signin' was loaded over HTTPS, but requested an insecure resource 'http://myhost/wgdashboard/api/isTotpEnabled'. This request has been blocked; the content must be served over HTTPS

Hi @shuricksumy, just pushed a small changes on the frontend code, let me know if it works :)

@donaldzou
Unfortunately, this did not help((
Maybe the redirection is done by server side or some other logic.
It's still ask over http
Mixed Content: The page at 'https://myhost/wgdashboard/#/signin' was loaded over HTTPS, but requested an insecure resource 'http://myhost/wgdashboard/api/getDashboardTheme'. This request has been blocked; the content must be served over HTTPS.
wgdashboard/#/signin:1 Mixed Content: The page at 'https://myhost/wgdashboard/#/signin' was loaded over HTTPS, but requested an insecure resource 'http://myhost/wgdashboard/api/isTotpEnabled'. This request has been blocked; the content must be served over HTTPS.

Hmmmm this is weird. With v4, every front-end activity is done on front-end with Vue.js. Just wondering if you can execute this in your browser when you open https://myhost/wgdashboard/#/signin please?
console.log(`${window.location.protocol}//${window.location.host}${window.location.pathname}`)

Hmmmm this is weird. With v4, every front-end activity is done on front-end with Vue.js. Just wondering if you can execute this in your browser when you open https://myhost/wgdashboard/#/signin please?
console.log(`${window.location.protocol}//${window.location.host}${window.location.pathname}`)

it returns login form
I did even in postman

What did this returned from your browser when you execute in the console?
console.log(`${window.location.protocol}//${window.location.host}${window.location.pathname}`)

What did this returned from your browser when you execute in the console?
console.log(`${window.location.protocol}//${window.location.host}${window.location.pathname}`)

also did u git fetch the latest commit?

I've fetch the latest code

this is JS code from browser

This is weird.... even your browser is returning https from the console.. but somehow the js is not using the browser's protocol.. I just pushed another changes to the branch and it will print out the url front-end is fetching. Could you please let me know what did it print out please?

@donaldzou
Updated but no successes and it looks really weird

Do you see any redirection in your Network tab from Chrome's dev tool?

yes, it's blocked

Ohhhh.. I think the frontend code is correct but somehow the reverse proxy redirect https back to http

ok, I will play a bit with traefik rules
I use ordinary config - maybe need to add something
So I will let you know
Thanks for helping

No problem. Always happy to help 😁

@donaldzou
I found the issue
it's double // in API calls - it produce redirection on server side
I've changed logic to
${window.location.protocol}//${window.location.host}${window.location.pathname.replace(//+$/, '')}
and it works
need to delete trailing / in window.location.pathname

ahah mystery solved! Will update the branch in a sec.

@donaldzou
It's not a critical, just to nice to have - is it possible folder /static also use after the url_prefix ?
It's quite easy to add this in redirection rules but still it needs 2 routes in redirection but maybe one is better ))

Just updated the branch, let me know if it works :)

@donaldzou It's not a critical, just to nice to have - is it possible folder /static also use after the url_prefix ?
It's quite easy to add this in redirection rules but still it needs 2 routes in redirection but maybe one is better ))
it actually used for loading web application

I think this could be done ;)

@donaldzou
So help a lot for what you are doing and this project.
I will close this pull request
I have docker builder project for your app -> please fill free to reuse any ready idea Docker builder

closed

@donaldzou
So thanks a lot for what you are doing and this project. I will close this pull request I have docker builder project for your app -> please fill free to reuse any ready idea Docker builder

No problem! Honestly I think I will just link your repo when I release v4 since I know nothing on Docker so I'll leave it to experts like u ;)


Replace characters “ and ” with "

They may cause problems if the current codepage doesn't have them. For instance, in my case, VS issues a warning, and the warning is treated as an error.

Is it ok this way?

@Nekotekina, yes, that will do nicely.  Thanks for the fix!


Unable to unpickle an object created from a class that inherits from shapely.geometry Polygon

Expected behavior and actual behavior.
Error on the line: p2 = pickle.loads(pickled):
TypeError: init() missing 2 required positional arguments: 'my_attr' and 'bound_wkt'
Steps to reproduce the problem.
import pickle
from shapely.geometry import Polygon
from shapely.wkt import loads as shapely_wkt_loads
class MyPoly(Polygon):
def init(self, my_attr, bound_wkt):
self.my_attr = my_attr
super(MyPoly, self).init(shapely_wkt_loads(bound_wkt))
def main():
bound_wkt = 'POLYGON  (( -79.240 39.277, -79.239 39.277, -79.239 39.278, -79.240 39.278, -79.240 39.277))'
p = MyPoly('abc', bound_wkt)
pickled = pickle.dumps(p)
p2 = pickle.loads(pickled)
print (p2)

if name == 'main':
main()
Operating system
Windows 7
Shapely version and provenance
Shapely (1.6.4.post1) installed from whl file downloaded from (https://www.lfd.uci.edu/~gohlke/pythonlibs/)

@matejt I'm sorry to say it, but shapely's pickling support doesn't support subclassing. See the implementation at https://github.com/Toblerity/Shapely/blob/master/shapely/geometry/base.py#L244.  I don't think we are going to change this soon.
If you can't alter your program to use shapely via composition instead of inheritance, you will need to override __reduce__ and __setstate__ in your class.


Whatif parameter does not work!

Whatif parameter does not work. Be careful!

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

ID: 67a1fb9d-8199-7574-276a-d32482fd1f7d
Version Independent ID: d6d3f5ba-d91b-2968-d6a8-588ec3c65e86
Content: Update-SPSolution (sharepoint-server)
Content Source: sharepoint/sharepoint-ps/sharepoint-server/Update-SPSolution.md
Service: unspecified
GitHub Login: @techwriter40
Microsoft Alias: kirks

@YuriySamorodov Thank you for this feedback
I am going to get this over to the PowerShell team so we can figure this out and get the article updated.
I was able to get this into the backlog for the PowerShell team. The team will get this prioritized into a sprint plan. Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@YuriySamorodov Thank you for reporting this feedback here. This list is focused on content issues and I want to make sure your immediate product issue does not get lost in this list. If you would please open a service ticket the support engineers will be able to identify if there is a bug (highly likely) and escalate to the appropriate product team.


Add JDK 16

Tool name
Adopt OpenJDK 16.0.1+9
Tool license
GPLv2+CE
Add or update?

[X] Add
[ ] Update

Desired version
16.0.1+9
Approximate size
No response
Brief description of tool
No response
URL for tool's homepage
No response
Provide a basic test case to validate the tool's functionality.
No response
Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[X] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11
[ ] Windows Server 2016
[ ] Windows Server 2019

Can this tool be installed during the build?
No response
Tool installation time in runtime
No response
Are you willing to submit a PR?
No response

Hi @haileh,
based on our policy, we pre-install only LTS Java versions, so the next installed version will be Java 17.
For other versions, please, use setup-java action, it supports installation of any Java version.
I am closing this issue. Please, let me know if you have any concerns.
Thanks.


s390x sha3 support

Add z14 sha3 hw support.
Labeled WIP, because i still need to find time to take a closer look @dot-asm 's code .

There's no rush, since we're in feature-freeze for our next release (scheduled in May).

Managed to look at @dot-asm and shared my results. Removed 'WIP' from the title. From my side this is ready to go.

I'd like to argue in favour of including this into 1.1.1. With following rationale. Even though it can be viewed as feature, it has literally zero impact on API. This kind of contradicts definition of feature, something that is in first hand visible. Secondly, and more importantly, it will have to be backported to 1.1.1a, because it provides significant (more than significant) advantage, so it can as well be done now.

Integrated your feedback (thanks) and rebased.

it has literally zero impact on API. [...] it provides significant (more than significant) advantage

True. I interpreted @richsalz  's comment from april https://github.com/openssl/openssl/pull/5935#issuecomment-380795787 as something like "whats not a fix is considered a feature". However, if significant performance improvements can still be accepted for 1.1.1 (as long as they are "transparent") this would apply to nearly all of my s390 openssl work and i would prioritize getting it upstream (over other work).
Secondly, are backports of "add hw support"-patches to previous releases accepted? Im asking because im doing those backports anyway and they are shipped on top of supported distros' packages, so i could as well contribute them .

I am in favor of this PR getting merged.  It's borderline, to me, but doesn't affecdt anything else.
As for changes to released branches, I am less in favor, but not necessarily opposed.
This is what we call "wishy washy"

this would apply to nearly all of my s390 openssl work and i would prioritize getting it upstream (over other work)

It's still kind of trade-off. I mean any modifications close to release incur formally unwanted risk, and it has to be weighed against benefits. And of course complexity plays role. I reckon that this modification is simple enough and gives enough benefit to tolerate the risk so close to release.

Merged. Thanks!


[UI] - Helper icon for prompt component

Current Behavior

Desired Behavior

Add helper icon for the prompt component.
When hover over the icon tooltip message have to appear.

Implementation

Acceptance Tests

Mockups

Contributor Guides and Handbook

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Assign me to fix this issue.

assign me this issue

@Yashsharma1911, please track this and assist with verbiage as necessary.
@officialrajdeepsingh, please use the layer5/sistent-svg NPM package for the icon. // @nebula-aac

@officialrajdeepsingh, also delete the whitespace between "Environment" and "?" in the title: "Delete Environment ?"

Hi, I follow the guidelines to install meshery locally. But I face an error when I'm running.
make ui-setup

Error is look like this.
❯ make ui-setup
make: /bin/bash: No such file or directory
make: /bin/bash: No such file or directory
make: /bin/bash: No such file or directory
make: /bin/bash: No such file or directory
cd ui; npm i; cd ..
make: /bin/bash: No such file or directory
make: *** [Makefile:252: ui-setup] Error 127

meshery on  master via 🐹 v1.21.4

I already install the go and make CLI (gnumake42).
> go version
go version go1.21.4 linux/amd64

> make -v
GNU Make 4.4.1
Built for x86_64-pc-linux-gnu
Copyright (C) 1988-2023 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <https://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Current operating system NixOS.
# System Details Report
---

## Report details
- **Date generated:**                              2024-01-17 16:10:11

## Hardware Information:
- **Hardware Model:**                              HP HP Pavilion Laptop 15-eg2xxx
- **Memory:**                                      8.0 GiB
- **Processor:**                                   12th Gen Intel® Core™ i5-1240P × 16
- **Graphics:**                                    Intel® Graphics (ADL GT2)
- **Disk Capacity:**                               512.1 GB

## Software Information:
- **Firmware Version:**                            F.03
- **OS Name:**                                     NixOS 24.05 (Uakari)
- **OS Build:**                                    24.05.20231211.a9bf124
- **OS Type:**                                     64-bit
- **GNOME Version:**                               45.2
- **Windowing System:**                            Wayland
- **Kernel Version:**                              Linux 6.1.67

hey @senali-d
What should the hover message be?

@sudhanshutech, can you please assign me this issue, I will change the styles, as per the theme only, and will remove the ${environment.name}, as said it was already shown in the modal.

These modals are now been replaced with sistent standard modals.Hence this is now a completed issue.


feat: balance tx fails fast on txcoll or collRet

balanceTransaction fails gracefully if Tx body contains either total collateral attribute or collateral return outputs attribute.
Issue Number
ADP-1653

bors r+


What if i work with multiple modules ?

Thank you for ng-classify, really useful
extends a controller or a service for exemple, will generate only one module name
but in my case i am working on multiple modules, and it will be useful if you develop the
configuration/options side, so then we can define a prefix for every module name
best regards.
Exemple:
App.controller => angular.module('app')
Dashboard.controller => angular.module('dashboard')

Hi @belarfaoui-zakaria,
Can you clarify your request?  I'm not quite following.
Before and after code would be great.
Thanks,
Cary

I just stumbled onto this myself and I think what @belarfaoui-zakaria may mean is when you pass a string to the class you are extending, that module has to have already been defined somewhere.  ng-classify will generate angular.module('myModuleName').config... and with just that, if that module doesn't exist, you'll get errors.  (With Angular 1.5.3, anyway).  In this case, the correct syntax needs to be angular.module('myModuleName', []).config... where you pass the array of dependencies as the second argument, which for right now, seems to define the module and return it.  I'm not sure how that will behave though in the case the module already does exist, would it trash that module's dependencies, or does it append any new ones?
A workaround I have found is that when you know you are using ng-classify to create a 'one-off' module, you can define the module like this angular.module 'moduleName', [] above your class definition.
I find I do this a lot as its an easy way to swap in interceptors, decorators, etc by just changing the main module's dependency list instead of having to add a build flag to conditionally include the files in the build.


[FLINK-27042][metrics] Fix instability of StreamTaskTest#testMailboxMetricsScheduling

What is the purpose of the change

Fix instability of StreamTaskTest#testMailboxMetricsScheduling

Brief change log

Remove assertion for latency measurement from StreamTaskTest#testMailboxMetricsScheduling as it
causes instability and is already covered in StreamTaskTest#testMailboxMetricsMeasurement.

Verifying this change
Please make sure both new and modified tests in this PR follows the conventions defined in our code quality guide: https://flink.apache.org/contributing/code-style-and-quality-common.html#testing
This change is already covered by existing tests, such as:

StreamTaskTest#testMailboxMetricsScheduling
StreamTaskTest#testMailboxMetricsMeasurement

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency):  no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery:  no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@flinkbot run azure


Couple of quick pointers pls

Hi
thanks, this is quite an awesome library. I am using the library Jib.jl to subscribe to data from Interactive brokers.
quick ques: in Rxpy, there is a method run() to block execution. is there anything similar in Rocket.jl please?
thanks

You should use subscribe! to start an execution and unsubscribe! to stop an execution.

@Roh-codeur see https://github.com/oliviermilla/Lucky.jl for a trading framework leveraging Rocket.jl
It's WIP.
Jib.jl isn't integrated yet though. Feel free.


Introduced PullRequestSyncService and refactored SyncCommand to use service

Behaviour should be extracted away from the hexagonal architecture edge (namely the SyncCommand) towards a service which is responsible for this behaviour

@LonelyPullRequests

Looks good IMO

@LonelyPullRequests Please help me get quality feedback from the community on this pull request.


Would "dachar" work better as a stand-alone library if fixes were language agnostic?

dachar version: *
Python version: 3.7
Operating System: Linux

Description
We have created fixes that are specific to our daops library, e.g.:
{
    "post_processors": [
        {
            "args": null,
            "func": "daops.data_utils.coord_utils.squeeze_dims",
            "kwargs": {
                "dims": ["lev"]
            }
        }
    ]
}

An alternative would be to define all fixes in a language-agnostic form, such as:
{
  "fixes": [
     {
         "operation": "squeeze singleton dimensions",
         "operands": {
               "dimensions": ["lev"]
      }
  ]
}

The reason for taking this approach would be that the dachar library would exist as a stand-alone component that was entirely independent of any implementation of fixes.
Benefits of this approach would be:

Any external tools using the information generated by dachar could use the fix definitions.
dachar does not hard-code specific implementations.

Drawbacks of this approach would be:

An extra language/encoding of the fixes is required - this must be written by the project. (There might be existing standards/approaches we could use).
Our daops library will need to define mappings from the language-agnostic form to our own daops functions.

@cehbrecht and @huard: what are your thoughts on this issue?

Agree with the benefits.
For the drawbacks:

Could you consider an extension to NcML that could become NcML-2.3 or NcML-3.0 ?
daops.mappings would essentially be xarray.open_ncml, and I think you'd get community support.
https://github.com/pydata/xarray/issues/2697
https://github.com/pangeo-data/esgf2xarray/issues/2

@huard, thanks for your response. We will have to look into it. NcML is certainly an option - rather than making up our own bespoke approach.

Sorry to harp on this, I'll be happy to be proved wrong on this.

@huard, I might be happy for you to be proved right :-)


feat(mat-select): add "select other" optional functionality

Bug, feature request, or proposal:
Feature request
What is the expected behavior?
Add "select other" / "select custom" optional functionality to mat-select. This would provide a seamless way for a dev to let a user select from a predefined list of drop down items or to provide a custom selection. I imagine if someone chose the "other" option of a mat-select dropdown, a matInput text field would appear and accept input. Crucially, user updates to this "other" matInput field would change the value of the FormControl associated with the mat-select field.
What is the current behavior?
Currently, this functionality can already be achieved however it is not easy. In my app I have made a custom "select-other" form control by combining a mat-select and matInput controls, both of which have their own FormControl. The component needs to flatten output from both of these controls so that I can interact with the component by just giving it one FormControl as input. It works, but it's not super clean. I'm assuming that a "select other" option (similar to a Google Form's "other" radio button option) is a pretty common design need.
What is the use-case or motivation for changing an existing behavior?
Increase developer happiness.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
5.x
Is there anything else we should know?

What about an autocomplete? It seems it fits in your case. Take a look at the example  pages at the doc site: https://material.angular.io/components/autocomplete/examples
It seems to me that it almost does what you want (just need to implement the saving part to add the custom option the already existing list).

@julianobrasil an autocomplete is a good suggestion for a stand in. I don't think it's quite the same as a "select other" option though (from a UX standpoint), because it makes it too easy for the user to add a custom option. In my case, I'd prefer it if the user select one of the predefined options, but if they must go with something else, they can. In the select other scenario, the user is forced to first select the "other" option from the drop down before entering a custom option.
While it's not my use case, an example of this might be a "race/ethnicity" question on a sign up form. In an attempt to normalize statistics, you might prefer users choose from the predefined options, but still want to give them a custom option. I think an autocomplete would noticeably increase the likelihood someone would submit a custom response (note, I'm speculating).
Anyway, it's possible "select other" functionality is nitch and doesn't make sense for the Angular Material library, but my guess is that this is a fairly common need.

I agree with @thefliik, it indeed is a needed feature.
@thefliik can you please share your solution?

@Sreya1 Ok, here's the source code. I imagine there are still many rough edges I'm unaware of. Also note, this component is customized to my particular use case. I imagine a number of the options aren't generally applicable (e.g. hasAnyOption).
You can learn more about creating custom form control components, in general, in the ControlValueAccessor docs, as well as the form field control guide.
Example usage:
<form-select-other
    *ngSwitchDefault
    [formControl]='formControl'
    placeholder="Main option placeholder"
    otherPlaceholder="Other option placeholder"
    [options]="[]"
    [allowCustomOption]="true"
    [hasDeselectOption]="true"
    >
</form-select-other>

@thefliik thanks :))


feat: bump hdwallet to latest

Description

Included changes: https://github.com/shapeshift/hdwallet/pull/614
Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[x] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

N/A
Risk

Testing

Ensure MetaMask Txs use EIP-1559 fees scheme for signing, vs. legacy
currently

Engineering

:point_up:

Operations

:point_up:

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #4366   👈

This comment was auto-generated by Graphite.


feat: add support for min ready instances

Fixes #1375.

Would this make more sense as a query parameter on the endpoint? Might be less invasive if the change is limited there as well

Great idea. I love it.


Put GITHUB_TOKEN in env to mitigate API throttling

GitHub has quite strict limits on API requests if you are an unauthenticated user, which the tests can run into since they download Pulumi plugins from GitHub. In the context of GitHub Actions, there is an access token available, which will give elevated API quotas. Actions can use it by default, but it needs to be made available in the environment for it to be in effect in tests.
Hence: add an environment entry for GITHUB_TOKEN to any workflow that runs the tests.

Thanks Kyle :star:


Unable to run mbppo_lagrangian.py

Hi @akjayant ,
I followed the installation pipeline from the README.md but I get the following conflicts when running python3.8 mbppo_lagrangian.py --exp_name="test_1" --seed=0 --env="Safexp-PointGoal2-v0" --beta=0.02:

Traceback (most recent call last):
File "mbppo_lagrangian.py", line 9, in
from utils.logx import EpochLogger
File "/lustre03/project/6002409/cmartel/safe_rl/mbppo/mbppol/src/utils/logx.py", line 16, in
from .mpi_tools import proc_id, mpi_statistics_scalar
File "/lustre03/project/6002409/cmartel/safe_rl/mbppo/mbppol/src/utils/mpi_tools.py", line 1, in
from mpi4py import MPI
ModuleNotFoundError: No module named 'mpi4py'
(mbppo) [cmartel@beluga2 src]$ module load mpi4py
(mbppo) [cmartel@beluga2 src]$ vim ~/.bashrc
connect <IP_ADDRESS> port 6000: Connection refused
(mbppo) [cmartel@beluga2 src]$ source .bashrc
-bash: .bashrc: No such file or directory
(mbppo) [cmartel@beluga2 src]$ source ~/.bashrc
(mbppo) [cmartel@beluga2 src]$ python3.8 mbppo_lagrangian.py --exp_name="test_1" --seed=0 --env="Safexp-PointGoal2-v0" --beta=0.02
2023-07-03 19:00:17.595867: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcudart.so.11.0
Creating environment:  Safexp-PointGoal2-v0
Traceback (most recent call last):
File "mbppo_lagrangian.py", line 795, in
env = SafetyGymEnv(robot=robot, task="goal", level='2', seed=10, config=env_config)
File "/lustre03/project/6002409/cmartel/safe_rl/mbppo/mbppol/src/env_utils.py", line 57, in init
self.env = gym.make(env_name)
File "/lustre03/project/6002409/cmartel/venv/mbppo/lib/python3.8/site-packages/gym/envs/registration.py", line 156, in make
return registry.make(id, **kwargs)
File "/lustre03/project/6002409/cmartel/venv/mbppo/lib/python3.8/site-packages/gym/envs/registration.py", line 101, in make
env = spec.make(**kwargs)
File "/lustre03/project/6002409/cmartel/venv/mbppo/lib/python3.8/site-packages/gym/envs/registration.py", line 72, in make
cls = load(self.entry_point)
File "/lustre03/project/6002409/cmartel/venv/mbppo/lib/python3.8/site-packages/gym/envs/registration.py", line 17, in load
mod = importlib.import_module(mod_name)
File "/cvmfs/soft.computecanada.ca/easybuild/software/2020/avx512/Core/python/3.8.10/lib/python3.8/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1014, in _gcd_import
File "", line 991, in _find_and_load
File "", line 975, in _find_and_load_unlocked
File "", line 671, in _load_unlocked
File "", line 848, in exec_module
File "", line 219, in _call_with_frames_removed
File "/lustre03/project/6002409/cmartel/safe_rl/mbppo/safety-gym/safety_gym/envs/mujoco.py", line 9, in
from safety_gym.envs.engine import *  # noqa
File "/lustre03/project/6002409/cmartel/safe_rl/mbppo/safety-gym/safety_gym/envs/engine.py", line 9, in
import mujoco_py
File "/lustre03/project/6002409/cmartel/venv/mbppo/lib/python3.8/site-packages/mujoco_py/init.py", line 3, in
from mujoco_py.builder import cymj, ignore_mujoco_warnings, functions, MujocoException
File "/lustre03/project/6002409/cmartel/venv/mbppo/lib/python3.8/site-packages/mujoco_py/builder.py", line 506, in
cymj = load_cython_ext(mujoco_path)
File "/lustre03/project/6002409/cmartel/venv/mbppo/lib/python3.8/site-packages/mujoco_py/builder.py", line 101, in load_cython_ext
mod = load_dynamic_ext('cymj', cext_so_path)
File "/lustre03/project/6002409/cmartel/venv/mbppo/lib/python3.8/site-packages/mujoco_py/builder.py", line 125, in load_dynamic_ext
return loader.load_module()
File "mujoco_py/cymj.pyx", line 1, in init mujoco_py.cymj
ValueError: numpy.ndarray size changed, may indicate binary incompatibility. Expected 96 from C header, got 88 from PyObject

It seems to be related to the numpy version required by the different packages. I am currently trying to run using numpy==1.21.4. Would it be possible to provide:

The commit id of the mujoco-py repository or the version of the mujoco-py package used to run the script
The commit if of the safety_gym repository used to run the script

@chrismartel were you able to fix the issue?


Potential deadlock in application startup

Internal issue: b/218923102
The following call https://github.com/flutter/engine/blob/12609b45685bd988b296595be2bd3ebacb80ba9c/shell/platform/android/external_view_embedder/external_view_embedder.cc#L271-L273 expects to run on the platform thread, but it might be blocked during application startup.
Culprit: https://github.com/flutter/engine/pull/31122

I removed the cp: label since I haven't been able to repro this issue locally, and the change hasn't made it to g3 yet.
I'd like to close b/218923102, and see if there's a customer issue that sounds like caused by the change pointed out above.


Rename VaporOS Flatpak Manager

The current name is too long and too specific to VaporOS. It also doesn't communicate that it only supports applications from the Flathub repository.

Suggestions are welcome. I'm thinking of "Flathub Theater", but I'm not sure yet.


GraphQL endpoint field is now persisted

A section for GraphQL related fields is added to the global store
A new reducer setGQLState is added to the global store for updating the GQL store
GraphQL endpoint field is now persisted through the GQL store

Hey @AndrewBastin,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: fa35dbc0-0c1d-11ea-9cd6-8f216fa7db85


Use mqtt-stack & mqtt-spec?

Hi Matteo,
Just saw that you started that project. Are you considering using the mqtt-stack and mqtt-spec stuff I started? There is not much missing, I just hadn't the time to finish it yet.

Aedes is all about speed. So, no framework here. It is going to support
MQTT and MQTT only. Similarly to Mosca, it would support some pluggability.
Regarding mqtt-spec, I plan to use it for the integration tests, probably
to replace what it is in Mosca now.
Il giorno dom 8 mar 2015 alle 12:38 Joël Gähwiler<EMAIL_ADDRESS>ha scritto:

Hi Matteo,
Just saw that you started that project. Are you considering using the
mqtt-stack and mqtt-spec stuff I started? There is not much missing, I
just hadn't the time to finish it yet.
—
Reply to this email directly or view it on GitHub
https://github.com/mcollina/aedes/issues/1.

I think it is possible to build that high-speed MQTT only broker also with mqtt-stack. Since the library is just a collection of these middlewares that provide a simple API for implementing the various MQTT features. There is even already an architecture for "plugins": https://github.com/mqttjs/mqtt-stack/blob/master/test/support/memory_backend.js. Do you think that the 2, 3 additional function calls will slow down that much? And on the other hand, making mqtt-stack faster would be also a really cool thing!
I just hopped we could spare some duplicate work. ;)

I don't know what will slow things down. I am doing this in Benchmark-Driven-Development, adding feature and checking if I can keep the same level of performance. Normally function calls do not slow things down, but creating functions and/or objects might do. In the composability vs speed battle, here I pick speed.
This approach is paying off: Aedes performs at 70k msg/sec on iojs on my macbook pro. The worst piece of performance hack is this: https://github.com/mcollina/aedes/blob/master/lib/client.js#L51-L61. That alone gave something 20-30% of perf increase.
Aedes is the reason why I did mqtt-packet. The MQTT.js Connection object was too slow. Not having a MQTT.js Connection is like a 100% perf increase over Mosca ;).
The goal is to support the full MQTT spec anyway, and I hope to keep things very decoupled here (another Mosca mistake) so they can be reused.

I like that approach, this kind of research is interesting to follow and already paying off: I used parts of your client implementation to replace mqtt-connection. That way mqtt-stack seems handling the bombing benchmark now well (before it just corked up :D).
On my macbook-air aedes performs with 26'000 msg/s and mqtt-stack 12'000 msg/s. However the comparison might be more true when aedes is implementing the same features.
Would be cool to compare both implementations in the future to see what things slow down and could be made better.

Definitely leave room for authentication and authorization plugins of some sort, though.

@RangerMauve @256dpi if you want to contribute, I am definitely accepting pull requests ;).


Browser in android's twitter app can't view webpages of fadeit.dk

Steps to reproduce:

open twitter app on android
go to @fadeitdk
press on fadeit.dk wesite link which will take you to app's browser

@ozooner says that it's because this particular browser doesn't execute javascript

Twitter has fixed it.


git dependency missing on aquilon

Following documentation, aquilon user need git-shell shell but git is not installed as dependency of aquilon.
Regards,

Of the RPM I assume?

you assume well

I still do not see git here: https://github.com/quattor/aquilon/blob/upstream/setup.cfg
Assume this is not fixed? @guillaume-philippon @jrha


Add a dummy benchmark

The goal is to have a placeholder for quick experimenting. Did not want to put into examples, as that may confuse users, or tests.

One question is should that setup have google benchmark linked in?
I am asking because if we don't use anything then we might as well have this as an example instead of a benchmark

I don't know if the google benchmark should be linked in. Either way is fine. Regarding examples, I'm not sure what the users would think when stumbling on an empty example, and thus decided not to put it there.

Ignoring HIP failure (not started because of the long queue).


Crash before launch with "Up And Down And All Around" mod

Forge 2838
VanillaFix 1.0.10-99
UpAndDownAndAllAround-1.12.2-2.9.0
Trying to launch minecraft results in a crash in the launcher with default configs. Turning bugFixes=false lets the client launch just fine though. Not sure how gutted that leaves VanillaFix as a result...
https://pastebin.com/fibCpp4d

I've opened a issue at the UpAndDown devs tracker and he replied with the following comment about this issue https://github.com/Mysteryem/Up_And_Down_And_All_Around/issues/96#issuecomment-509017689

Whenever I flip upside down, my screen flips but my character and gravity stay the same. Is there any way to fix this?


Split up rows with "multiple" bugs

bson | multiple bugs, including arithmetic overflow
capnproto-rust | Multiple bugs, including a memory safety bug

these should be split up into multiple rows, one per bug found

Also svgparser, cpp_demangle, pulldown-cmark


TexText class is not working correclty (for me)

When I try to write plain text outside the $$ symbols that are for latex equations I get the following WARNIN:
WARNING  Unsupported element type: <class 'svgelements.svgelements.Use'>                  svg_mobject.py:174
and also only the elements that are in between of the $$ appear. The rest of the words do not. Why is thishappening?

I also have this kind of problem,how did you solve it?


Address accessibility issues in Issue #3

Closes #3
Original issues:

[x] Document language missing
[x] Site logo image missing alternative text
[x] Enter address form field missing label
[x] Submit input missing value
[x] incorrect a / h1 wrapping

Additional changes:

[x] Correct skip link issues
[x] Clean up/format markup

Excellent! Thanks for this, @michaelck. If only Google Translate would get it together, we'd have 0 errors from the WAVE toolbar.


feat: add validation to binding name input in vscode extension

validates that binding names are unique and not empty
fixes DEVX-1547

Tests

[ ] TODO (before merge)
[ ] Tests included
[x] Tests not necessary because: difficult to test vscode ui and getting binding logic is directly copied from wrangler, where it is tested

E2E Tests CI Job required? (Use "e2e" label or ask maintainer to run separately)

[ ] I don't know
[ ] Required
[x] Not required because: no e2e tests

Public documentation

[ ] TODO (before merge)
[ ] Cloudflare docs PR(s):
[x] Documentation not necessary because: unreleased

For posterity, I manually tested this locally:


Unifi Integration settings window has no text

Home Assistant release with the issue:
0.98.0
Last working Home Assistant release (if known):
unknown
Operating environment (Hass.io/Docker/Windows/etc.):
Docker on Debian Buster
Component/platform:
https://www.home-assistant.io/components/unifi/
Description of problem:
Clicking on the gear wheel in the integration settings spawns a settings popup without text. (See here: https://imgur.com/a/XijbgaM). I have inspected the source code of this popup window to make sure it's not a browser problem, and the settings options look like this:
<div>
            <paper-checkbox role="checkbox" tabindex="0" toggles="" checked="" aria-checked="true" aria-disabled="false" active="" dir="null" style="--paper-checkbox-ink-size:48px;"></paper-checkbox>
          </div>

There is no content in the paper-checkbox tag.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
No configuration.yaml applicable, it's the Unifi Integration added via web frontend
Traceback (if applicable):
No applicable traceback
Additional information:
Checked with Chrome on Arch Linux, as well as Firefox. Firefox had no Extensions installed to provide a vanilla experience.

Deconz integration too

And second integration window has text, but doesn't have padding

For some reason the new text strings haven't propagated to the release.

same for me. But only Unifi. Deconz is fine

I have same problem in both deconz and unifi.  Tested both Chrome, Firefox with clean cache and my iPad. All same so not a browser cache problem

After upgrade to 0.98.1 it now works with Unifi but still fails with deconz.

Same.

Upgraded to 0.98.2.  Deconz integration options still haven't text.

Yes. Issue is still there in 0.98.2

@balloob it is still a thing

It works in dev …

@balloob your screenshot is from unifi. Unifi was fixed. But that was only half the problem solved. The same issue was present for both Unifi and deconz. Unifi was fixed.  It is still broken for deconz.

This issue is for unifi.

Deconz is here, where you commented too https://github.com/home-assistant/home-assistant-polymer/issues/3554


[HOLD for payment 2024-06-28] [Payment card / Subscription] [$250] Subscription - Keyboard opens when neither of the fields are in focus after selecting currency

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

Version Number:  1.4.80-3
Reproducible in staging?: y
Reproducible in production?: new feature
Issue found when executing PR: https://github.com/Expensify/App/pull/42771
Issue reported by: Applause - Internal Team
Action Performed:

Go to staging.new.expensify.com
Paste staging.new.expensify.com/settings/subscription/add-payment-card in any chat.
Tap on the link.
Enter all the details.
Tap Currency.
Select a currency.

Expected Result:
Keyboard will not open.
Actual Result:
Keyboard opens when neither of the fields are in focus after selecting currency.
Workaround:
n/a
Platforms:

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

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/df6599d6-ca29-431a-afdd-92db59f14576
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~017794b8ac770cfc92
Upwork Job ID:<PHONE_NUMBER>782491823
Last Price Increase: 2024-06-06
Automatic offers:

ShridharGoel | Contributor | 102630012

Issue OwnerCurrent Issue Owner: @mananjadhav

We think that this bug might be related to #wave-collect - Release 1

Not a backend bug

I can see it's open for External contributors.
Looks like @ShridharGoel's proposal would work.
🎀 👀 🎀 C+ reviewed.

@stephanieelliott, please handle the payment for this issue, please 🙇
@mananjadhav is the C+ who reviewed the PR
@ShridharGoel is the contributor who authored the PR

@stephanieelliott Can you please add the payout summary?

Summarizing payment on this issue:

Contributor: @ShridharGoel $250 via Upwork -  PAID
Contributor+: @mananjadhav $250 please request via ND

Upwork job is here: https://www.upwork.com/jobs/~017794b8ac770cfc92

$250 approved for @mananjadhav

@stephanieelliott We're good to close this issue.


[DO NOT MERGE] INTPLAT-418: test pr for new github-webhook and prbot in APW

What does this PR do?

Motivation

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
[ ] If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

Added Do Not Merge label to remove from Docs review queue. Thanks! 😄


Asymptotic complexity of actions on .NET collections

Hello,
I was wondering if it would be possible to expand the System.Collections documentation so it spells out the big-O cost of the different operations.  (Alternatively, it would be enough to state the underlying algorithm, because then the complexity is easy to determine.)
For example, I've just had a need for a sorted list of objects, which must support efficient insertion and deletion.  I assume that SortedSet is a red-black tree and will do what I want, but this doesn't seem to be spelt out in the documentation.  Another reasonable implementation would be a sorted list, which would support more efficient retrieval (by binary search) but less efficient insertion and deletion.
Thanks for all your work on the documentation and .NET Core itself.
Pete

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

ID: 26b9b071-fac4-e286-3dd0-107ed2687a03
Version Independent ID: 2356f35d-5784-d763-125b-8c5625600a63
Content: System.Collections.Generic Namespace
Content Source: xml/ns-System.Collections.Generic.xml
Service: unspecified
Product: .net
GitHub Login: @mairaw
Microsoft Alias: mairaw

Some methods and types have their asymptotic complexity documented, so I think it would help if you listed where exactly do you think complexity is missing.
For example, the documentation for SortedSet<T>.Remove says that it's an O(log n) operation. Though the documentation for SortedSet<T>.Add does not mention complexity, so that's something that should be added.
As for the implementation, it is indeed a red-black tree.

@svick Yes some of the methods do mention complexity, but as you say not all do, and that's my point really.
Rather than add the asymptotic complexity to all the method descriptions, perhaps a simpler approach would be a summary paragraph in the description of each of the container classes.  For example: 'SortedSet is a balanced tree structure which is designed to support O(log n) insertion, retrieval, and deletion.'

I think this would be ridiculously helpful. I can't tell you the number of times I wish libraries would document this.
I wouldn't mind giving this issue a little bit of my time. Can PR's come in incrementally i.e. one per class or would you prefer one PR with all the changes.

@ezzy1337 That would be awesome!  One per class would be a great scope. That would help us review and merge in a timely fashion.  Ping us if you have any questions as you start making changes.

Just to be sure this is the .NET framework and not .NET Core right? I didn't see anything specifically spelling out which.

@ezzy1337 To be useful, I think it would need to cover .NET Framework and .NET Core.  Adding @safern who is responsible for the System.Collections namespace(s).
My thought is the big-O metrics should be similar across implementations, but he would be the authority.

@BillWagner It looks like the specific documentation is going to live in the dotnet-api-docs repo i.e. for SortedSet.Remove. So after making the change in that repo is there a follow-up PR needed for this repo?

@ezzy1337 A PR in dotnet-api-docs will update the docs. No need for a second PR. You can reference this issue there, using dotnet/docs#<issueNumber>

This is still an issue with the documentation I believe.  @BillWagner , I saw your comment that you would reopen issues when this was necessary.  Would you be able to do that with this one please?  I don't have the necessary access to do it myself.  I'm also including @dotnet/docs but I'm not sure if that will work; Github's completion didn't recognise it as valid.

This is still an issue with the documentation I believe.  @BillWagner , I saw your comment that you would reopen issues when this was necessary.  Would you be able to do that with this one please?  I don't have the necessary access to do it myself.  I'm also including @dotnet/docs but I'm not sure if that will work; Github's completion didn't recognise it as valid.

reopening and moving to dotnet-api-docs

reopening and moving to dotnet-api-docs

Issue moved to dotnet/dotnet-api-docs #5282 via ZenHub

Issue moved to dotnet/dotnet-api-docs #5282 via ZenHub


[Impreove][Core]remove no meaningful duplicated check code

Purpose of the pull request
remove no meaningful duplicated check code
Brief change log
remove the duplicated check code
Verify this pull request
This pull request is code cleanup without any test coverage.
This change added tests and can be verified as follows:

remove the duplicated check code.

just the dulicatede code

Codecov Report

Merging #8552 (ad8ab20) into dev (ca6d148) will decrease coverage by 0.16%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #8552      +/-   ##
============================================
- Coverage     45.41%   45.24%   -0.17%
- Complexity     4039     4040       +1
============================================
  Files           688      690       +2
  Lines         26773    26873     +100
  Branches       2873     2884      +11
============================================
+ Hits          12158    12160       +2
- Misses        13466    13564      +98
  Partials       1149     1149

Impacted Files
Coverage Δ

...r/plugin/registry/zookeeper/ZookeeperRegistry.java
48.21% <0.00%> (-7.15%)
:arrow_down:

...cheduler/api/service/impl/ExecutorServiceImpl.java
41.96% <0.00%> (-5.14%)
:arrow_down:

...inscheduler/api/controller/ExecutorController.java
40.00% <0.00%> (-1.94%)
:arrow_down:

...dolphinscheduler/remote/future/ResponseFuture.java
81.96% <0.00%> (-1.64%)
:arrow_down:

...lphinscheduler/service/process/ProcessService.java
31.88% <0.00%> (-0.08%)
:arrow_down:

...a/org/apache/dolphinscheduler/dao/entity/User.java
74.13% <0.00%> (ø)

.../org/apache/dolphinscheduler/dao/entity/Queue.java
51.61% <0.00%> (ø)

...org/apache/dolphinscheduler/dao/entity/Tenant.java
72.72% <0.00%> (ø)

...rg/apache/dolphinscheduler/api/dto/gantt/Task.java
0.00% <0.00%> (ø)

...rg/apache/dolphinscheduler/dao/entity/Session.java
41.93% <0.00%> (ø)

... and 37 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 ca6d148...ad8ab20. Read the comment docs.

Well I don't think it is duplicate.if it removes, the below codes that the commands.size return NPE .

hi @caishunfeng  can you help check this pr ?


Styling issue

Can you fix the styles please?
Border and Focus styles are not matched with the default style system.

hi @RealMrHex
thank you for telling this issue.
the commit a5db040249b6594 solved the issue.


Question: How to HMR React Front End and Express Backend?

Hello, I just came across your repo and I am interested in using this to HMR my back-end API. What is not clear from the docs is how to set this up for both the front-end and back-end simulatenously.
My current setup is very similar to the react-hot-boilerplate example, in that I have a server.js file which sets up the express server for HMR, this however is purely used for development.
This works well for the React app, but now I wish to work on a express server to create a REST API for the application.
My question: Do I have two different servers running HMR, one for the front-end React app, and one for the back-end express API? Or can I set this up in such a way as to have one HMR which will handle changes both to the front-end and the back-end?
A simple example demonstrating this setup would be most appreciated.

Hello,
You will have two HMR, one for react app (browser), another for express app.
Try this:
Add webpack config for your express app, for example, webpack.config.server.js. Look at simple-example. Don't forget to configure externals for node app in webpack config. And then just run node-hot --config webpack.config.server.js. node-hot-loader compile you entry (server.js) with webpack and then start it.
Also look at README for correct hot reload express app.

Thanks for getting back to me so quickly. Okay, that makes sense. I will setup another webpack config for the server. Thanks for the help!


AddOrUpdate generate query with inlined parameter

AddOrUpdate make query to check if entity already exists in db
query look like:
SELECT TOP (2)
    [Extent1].[Id] AS [Id],
    ... other fields ...
    FROM [Table] AS [Extent1]
    WHERE N'id-value' = [Extent1].[Id]

I expecte query to look like:
SELECT TOP (2)
    [Extent1].[Id] AS [Id],
    ... other fields ...
    FROM [Table] AS [Extent1]
    WHERE @p__linq__0 = [Extent1].[Id]

Looks like we are generating constants in https://github.com/aspnet/EntityFramework6/blob/master/src/EntityFramework/Migrations/DbSetMigrationsExtensions.cs#L136.
@anpete do you remember if this was done on purpose or should we consider changing it to be parameters?

EF Team Triage: This issue is not something that our team is planning to address in the EF6.x code base.  This does not mean that we would not consider a community contribution to address this issue.
Moving forwards, our team will be fixing bugs, implementing small improvements, and accepting community contributions to the EF6.x code base. Larger feature work and innovation will happen in the EF Core code base (https://github.com/aspnet/EntityFramework).
Closing an issue in the EF6.x project does not exclude us addressing it in EF Core. In fact, a number of popular feature requests for EF have already been implemented in EF Core (alternate keys, batching in SaveChanges, etc.).
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.


Use Argon2d instead SHA2 as key derivation function

Hello,
First of all, thank you for developing PINcredible which look a great project.
I quickly read the code, in particular the import/export function.
As far as I can understand, the password typed by the user is hashed by SHA2 function and the hash is used as AES key to encrypt serialized data. Unfortunately modern GPUs can do billion SHA2 hash per second.
In my opinion, a good improvement could be to use Argon2d function instead of SHA2, which will make dictionary attacks harder on exported file. Argon2d is build to be hard to crack even if user password have poor entropy and it is very efficient against GPUs usage.
Links :

https://crypto.stackexchange.com/questions/50135/is-sha-2-suitable-for-key-derivation#answer-50137
https://en.wikipedia.org/wiki/Argon2
https://github.com/lambdapioneer/argon2kt (example of implementation in Kotlin)

@LoHub Argon2ID is now implemented, it will be included in the next release.
I've used the following specifications:

mode: Argon2id
65.536 KibiByte memory usage
10 iterations
salt of size 32 Bytes
parallelism of 2 threads

The only thing that's weird is that the key derivation is blazing fast compared to the implementation of the lambdapioneer/argon2kt author's demo app.
Where his app needs 2,5 seconds mine is done in less than 400 milliseconds.

Implemented in Version 0.7.0.
Previously saved PINs still work with SHA, if you create a new one, the new default is Argon2id.


Patch upgrade

Description
Changes proposed in this pull request:

Strip a resource from blacklisted properties and merge it with the newest resource version on the cluster

Related issue(s)

/retest

/retest

/retest

/retest

/test pre-busola-integration-test-k3s-namespace

/retest

/retest


Number text representation

Within the framework of the project I am currently working on I have to develop a library that will convert a number to its text representation in Russian and English.
For instance, given the number 1235 the library will output "one thousand two hundred and thirty five" for English and "одна тысяча двести тридцать пять" for Russian.
I can design this library in such a way that it is localizable for any language and language extension grammar files are shipped via nltk_data repository. In the future the reverse process, i.e. parsing of text to number, can also be included within this library which will be a handy tool for discovering numbers in natural language.
Do you think it makes sense to include such a library into NLTK?

Maybe you'll fine this helpful: http://pastebin.com/WwFCjYtt
But i'm not sure whether which module it should go under within NLTK.

Looks like I have to make a PR when I have this module ready. For now closing.


(Feature) User should see error message while typing in edit profile

This PR closes #166492509 and #166492443 by:

Adding isValidValue and  validateProfile methods to userProfile.
Adding tests to this functions.
Adding validation to EditProfile screen while typing and disabling save when is not valid.

Demo

Fixing edge case when the profile changes after saving is clicked by avoiding changes when saving === true
const handleProfileChange = newProfile => {
    if (saving) {
      return
    }

    setProfile(newProfile)
  }


Merge planning

In order to merge zig-ar with this, some minor changes/addition needs to be done.

Add header and contents field to ParsedFile

pub const ParsedFile = struct {
    name: []const u8,
    header: Header,
    contents: []u8,
};

Contents need to be stored

pub fn parse(self: *Archive, allocator: *Allocator, stderr: anytype) !void {
    ...
    // try reader.context.seekBy(try fmt.parseInt(u32, mem.trim(u8, &archive_header.ar_size, " "), 10));
    var contents = try self.allocator.alloc(u8, size);
    try reader.readNoEof(obj_file.contents);
    ...
}

Merging these things (related to 1 actually), though it is not that mandatory

archive_headers: std.ArrayListUnmanaged(format.ar_hdr),
parsed_files: std.ArrayListUnmanaged(format.ArchivedFile),

Minor nitpick

pub const ar_hdr = extern struct {
    ar_name: [16]u8,
    ar_date: [12]u8,
    ...
};

We can just have normal names in Zig convention here. The reason is that freebsd manpage lists this struct in terms of its own symbols, its not any kind of contention afaik. I decided to have mine as follows:
pub const Header = extern struct {
    name: [16]u8,
    mtime: [12]u8,
    ...
}

Let me know if something seems odd, I will start working on merging tomorrow

On some side notes, zar is currently missing any hints to thin archives.
Also, this superceeds 3

Cheers! All of these changes look sensible to me. I'm happy to go with zig-naming, I was only copying the convention for that struct in the compiler, but you are right that it differs based on the implementation.
I will make the changes as requested on my end tomorrow morning to help prepare for the merge if that works. :)

Actual code merge is done :fireworks: #4 !
I think before this is considered wholly complete - the results of the merge need to be cleaned-up so that everything works using similar conventions.
This is both in style (using same names for variables etc.) and functionality (using file-based as opposed to in-memory representation for files). I'm happy to help with either of these. But during the week my time availability is going to be limited (I will be able to work on this for a couple of hours some mornings).
But having the actual code merged-in is a great first step! :D

I agree with the content representation part. I will continue to work on this project as a whole and try to resolve as much as I can.

One more thing is that I suggest marking the struct Archive managed. This is because

Explicitly passing the allocator in all function is tedious
We currently aren't dellocating any memory, which is bad

OK - so I'm not sure how sensible this is, but my current plan was to structure the program around the fact that ar is a simple program. As long as we aren't allocating more memory than is needed persistently anyway, a single arena allocator that is simply cleared when the program terminates should be fine I think - which means that explicit deallocations simply aren't needed.
I will double check with @kubkon to draw on their experience as well just in case.

Well not only that but we aren't closing file handles either right now...


No hot reloading when change font scale

Description
In old arch, we can change font scale in realtime with command option +/-.
In new arch, it has to be refreshed manually with pressing R to get new font scale applied.
Steps to reproduce

gh repo clone yzhe554/new-arch-font-scale
cd new-arch-font-scale
npm i
npm run ios
change font scale with option command +/-

React Native Version
0.74.3
Affected Platforms
Runtime - iOS
Areas
Fabric - The New Renderer
Output of npx react-native info
System:
  OS: macOS 14.5
  CPU: (10) arm64 Apple M1 Max
  Memory: 107.00 MB / 32.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 18.18.0
    path: ~/.nvm/versions/node/v18.18.0/bin/node
  Yarn:
    version: 1.22.18
    path: /usr/local/bin/yarn
  npm:
    version: 9.8.1
    path: ~/.nvm/versions/node/v18.18.0/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.15.2
    path: /opt/homebrew/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.5
      - iOS 17.5
      - macOS 14.5
      - tvOS 17.5
      - visionOS 1.2
      - watchOS 10.5
  Android SDK: Not Found
IDEs:
  Android Studio: 2023.2 AI-232.10300.40.2321.11567975
  Xcode:
    version: 15.4/15F31d
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 17.0.9
    path: /usr/bin/javac
  Ruby:
    version: 2.7.6
    path: /Users/yuzheng/.rbenv/shims/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.74.3
    wanted: 0.74.3
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: Not found
  newArchEnabled: Not found
iOS:
  hermesEnabled: true
  newArchEnabled: true

Stacktrace or Logs
N/A

Reproducer
https://github.com/yzhe554/new-arch-font-scale
Screenshots and Videos
In Old Arch
https://github.com/user-attachments/assets/b1e2d7a7-1070-4d3d-ac2c-a7e75952ef03
In New Arch
https://github.com/user-attachments/assets/e7ef203c-c355-4674-80de-e508c046e61f

Hey @cortinico, there is one more side effect connected to this issue. I'm the author of react-native-unistyles, and Unistyles automatically re-renders the view on different events like font scale changes. In the new architecture, there is different behavior, and the text is cut off at the edges. It's not initially visible because the user needs to reset the app as mentioned above, but with Unistyles, it's visible after the first re-render.
Here is link to the unistyles issue: https://github.com/jpudysz/react-native-unistyles/issues/256
Repro: as above linked by @yzhe554
Minimal steps to reproduce:
export default function App() {
    const [counter, setCounter] = useState(0)

    return (
        <View
            style={{
              flex: 1,
              backgroundColor: '#fff',
              alignItems: 'center',
              justifyContent: 'center',
            }}
        >
          <Text style={{fontSize: 24}}>RN 24 Label Testing!!!</Text>
          <Text style={{fontSize: 18}}>RN  18 Label Testing!!!</Text>
          <Text style={{fontSize: 16}}>RN 16 Label Testing!!!</Text>
          <Button onPress={() => setCounter(prevState => prevState + 1)} title="Re-render" />
          <StatusBar style="auto" />
        </View>
    )
}

Try to change font scale and then press re-render button.
Result for new architecture:
https://github.com/user-attachments/assets/e1a4f765-e055-4455-93f2-795f50c05269
Old arch:
https://github.com/user-attachments/assets/3158c494-ff04-4ca7-aedd-e27afb2fb0c6

Hi there! These are actually two separate issues. So, let's tackle them separatedly.
First, @yzhe554 thanks for reporting the issue.
I looked into it and discussed it with some colleagues that have more context.
This is a known limitation, as of today, because in the New Architecture we don't have a good mechanism to notify React that all the nodes received the updated text multiplier. It is also an edge case, as in production it is very rare that a user go in the settings to change the preferred text size and then it returns to the app. Furthermore, the issue solves itself if you navigate to another screen or you do a state update that force a re-render
We are exploring how to implement some features that can be used as foundations to fix this one as well, so the fix to this will wait until that thread of work is done.

Then, @jpudysz, this is a different issue. I'd love if you can do a test for me.
What happens if the user:

change the text
press on re-render
reload the app from Metro (r in the simulator)

The reason I ask is that my gut feeling is that this scenario will "fix" your issue. Text multiplier affects layout as well and, from the video, it looks like that the rerender from unistyle is not updating the layout correctly: the left margin in the New Architecture is unchanged, while, in the old architecture, the left margin shrinks while the font increase its size.

@cipolleschi Thanks for the quick feedback.
I am working on an enterprise level project that is trying to introduce RN as a brownfield project. This would be a key point whether we can go with new arch. If you have a rough timeline on the fix, it will be really helpful for us to plan.

@yzhe554 we don't have a rough timeline as of today, unfortunately.
Is it a use case you face often, of people going to the setting page, come back to the app and expect to see the app updated?

@yzhe554 we don't have a rough timeline as of today, unfortunately. Is it a use case you face often, of people going to the setting page, come back to the app and expect to see the app updated?

Nah, I agree it's an edge case. But just need to convince other team(non tech). I will follow this for now. Thanks again.


Update peerDependencies to support Grunt 1.0

Update peerDependencies to support Grunt 1.0
Hello,
This is an automated issue request to update the peerDependencies for your Grunt plugin.
We ask you to merge this and publish a new release on npm to get your plugin working in Grunt 1.0!
Read more here: http://gruntjs.com/blog/2016-02-11-grunt-1.0.0-rc1-released#peer-dependencies
Also on Twitter: https://twitter.com/gruntjs/status/700819604155707392
If you have any questions or you no longer have time to maintain this plugin, then please let us know in this thread.
Thanks for creating and working on this plugin!
(P.S. Close this PR if it is a mistake, sorry)

Could you please publish a new release to npm? Thank you very much.

Bump, would be super nice if this was merged!

This one is old guys. Need some help?


CSS

CSS
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Stuy Guide website
WHEN I view the study guide
THEN I see a dark blue header and footer, and four boxes with a shadow

Finished adding the CSS style sheet


<IP_ADDRESS> Nuget?

I see https://www.nuget.org/packages/FaceRecognitionDotNet/ is outdated.

Now, there are not high priority issue.
And on development and developed

feature/knn (on Development)
helen dataset support (DONE)
DlibDotNet update (Unit test is already PASS)

but it is major update and it should be carefully

Do you have any idea or feature request?

ok


mimxrt1060_evk test/kernel/mem_slab/mslab_threadsafe meets Imprecise data bus error

Describe the bug
Imprecise data bus error
To Reproduce
Steps to reproduce the behavior:

mkdir build; cd build
cmake -DBOARD=mimxrt1060_evk ..
make
See error

Expected behavior
Test PASS
Impact
slab memory usage
Screenshots or console output
***** Booting Zephyr OS 1.14.0-rc1 *****

Running test suite mslab_threadsafe

===================================================================

starting test - test_mslab_threadsafe

PASS - test_mslab_threadsafe

===================================================================

Test suite mslab_threadsafe succeeded

===================================================================

PROJECT EXECUTION SUCCESSFUL

***** BUS FAULT *****

  Imprecise data bus error

***** Hardware exception *****

Current thread ID = 0x800002fc

Faulting instruction address = 0x60005060

Fatal fault in ISR! Spinning...

Environment (please complete the following information):

OS: (e.g. Linux)
Toolchain ( Zephyr SDK)
Commit SHA or Version used: v1.14.0_rc1

@MaureenHelm

close same root cause as #13727


dataset processing issue

Hi, I've noticed that batch_load_scannet_data.py is the same as one in Scan2Cap. So there is the same issue.
Traceback (most recent call last):
File "batch_load_scannet_data.py", line 84, in
batch_export()
File "batch_load_scannet_data.py", line 79, in batch_export
export_one_scan(scan_name, output_filename_prefix)
File "batch_load_scannet_data.py", line 29, in export_one_scan
mesh_vertices, aligned_vertices, semantic_labels, instance_labels, instance_bboxes, aligned_instance_bboxes = export(mesh_file, agg_file, seg_file, meta_file, LABEL_MAP_FILE, None)
File "/root/autodl-tmp/Vote2Cap-DETR/data/scannet/load_scannet_data.py", line 56, in export
mesh_vertices = scannet_utils.read_mesh_vertices_rgb_normal(mesh_file)
File "/root/autodl-tmp/Vote2Cap-DETR/data/scannet/scannet_utils.py", line 99, in read_mesh_vertices_rgb_normal
assert(os.path.isfile(filename))
AssertionError
So how should I fix it? By the way, here is the original link: https://github.com/daveredrum/Scan2Cap/issues/11

This error implies that the directory you choose does not contain the .ply mesh file. To fix this issue, please modify the data directory in this line: https://github.com/daveredrum/Scan2Cap/blob/main/data/scannet/batch_load_scannet_data.py#L16C1-L16C22 to where you store the scannet raw datasets.
For instance, if you have downloaded the scannet under /remote-home/scannet, the directory should look like this:
/remote-home/scannet
          scannet_meta_data
          scannetv2-labels.combined.tsv
          scans
          scans_test
          tasks

Here, you shall set SCANNET_DIR = '/remote-home/scannet/scans'.

Thanks for your reply! But I'm afraid that I did not give a clear question. I knew how to modify the data directory, but the main problem is that after loading 706 scenes this error I mentioned before occurs. It's not a problem about data directory. I guess it's because scannetv2.txt contains 806 scenes while Directory 'data/scannet/scans/' contains only 706 scenes for train and val. So after loading 706 scenes, the program cannot find the 707th.  I highly recommend you read the original link: https://github.com/daveredrum/Scan2Cap/issues/11.

You can dismiss that error as long as you can find the corresponding .npy files in the scannet_data directory, since the preprocessing script (batch_load_scannet.py) reads a list of scan_names rather than the folder names.
If you wish to process the last 100 scenes (for test), you can manually set:
SCANNET_DIR = '/remote-home/scannet/scans_test'

Hope this will help you.

Thanks!  So I guess loading the last 100 scenes is not compulsory?

If you wish to evaluate your models on the validation set only, it is not required to process the test set data. But if you wish to submit results to the official website (scene0707_00 - scene_0806_00), processing the test set is necessary.

The simplest solution is to set:
SCAN_NAMES = os.listdir(SCANNET_DIR)


Update docker samples

⚠ Don't merge yet, it's not entirely working 😄

This PR is to update the docker sample to latest Identity Server and .NET Core 1.1.0.

Hi @tugberkugurlu, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Working now 🎉

It's ready for review.

I am not in the position to review that, but if @alexzeitler gives a thumbs up - it's good enough for me ;)

thanks!


Implement the elliptic curve for faster factorization

Problem:
The current factorization algorithms we are using doesn't give us an output until everything is factored out into primes. This becomes a bigger problem as our numbers get too large to factor at a reasonable time.
Solution:
Thus, the solution is to implement a version of Lenstra's ECM that quits after it deems a number "unfactorable". We will likely use the factor package to do this.
Strategy:

Implement a custom version of ECM (that factors everything.)
Benchmark it against the package's ECM and a previous factoring algorithm we were using to optimize.
Integrate it into our full test and check how much further we can go, and how many unique fibpsps we generate.

The factor package was deemed unsuitable for this. A solution was almost found with the arithmoi package, but an implementation of the ECM couldn't be done due to private modules. The current maintainer has suggested that using a timeout function would work, so I decided to go that route instead.


[ACR / Build] Expose --no-cache for quick build

Feature request:
Create a new CLI parameter, --no-cache and add this when creating a new QuickBuildRequest so that we can ignore all layer caching when running a quick build.

Please assign this to @northtyphoon

Looks like https://github.com/Azure/azure-cli/pull/7255 has handled this
@northtyphoon, please reactivate otherwise.


Is it possible to intercept every resource request? Providing some hooks before handle the request or after.

No, it's impossible to intercept request.


the problem for ver 0.3.8

Hi, Rock,
I installed wifi by pip successful, but when I run wifi or wifi scan command, the error is same and as below:
Traceback (most recent call last):
File "/usr/local/bin/wifi", line 202, in
args.func(args)
File "/usr/local/bin/wifi", line 51, in scan_command
print_table([[cell.signal, cell.ssid, 'protected' if cell.encrypted else 'unprotected'] for cell in Cell.all(args.interface)])
File "/usr/local/lib/python2.7/dist-packages/wifi/scan.py", line 44, in all
cells = map(Cell.from_string, cells_re.split(iwlist_scan)[1:])
File "/usr/local/lib/python2.7/dist-packages/wifi/scan.py", line 54, in from_string
return normalize(cell_string)
File "/usr/local/lib/python2.7/dist-packages/wifi/scan.py", line 137, in normalize
while lines[0].startswith(' ' * 10):
IndexError: list index out of range
but wifi  --help, works fine.

Hi!
Which os/distro are you using?
Thanks,
-rocky
2016年3月30日 上午12:28於 "rackmount"<EMAIL_ADDRESS>
Hi, Rock,
I installed wifi by pip successful, but when I run wifi or wifi scan
command, the error is same and as below:
Traceback (most recent call last):
File "/usr/local/bin/wifi", line 202, in
args.func(args)
File "/usr/local/bin/wifi", line 51, in scan_command
print_table([[cell.signal, cell.ssid, 'protected' if cell.encrypted else
'unprotected'] for cell in Cell.all(args.interface)])
File "/usr/local/lib/python2.7/dist-packages/wifi/scan.py", line 44, in all
cells = map(Cell.from_string, cells_re.split(iwlist_scan)[1:])
File "/usr/local/lib/python2.7/dist-packages/wifi/scan.py", line 54, in
from_string
return normalize(cell_string)
File "/usr/local/lib/python2.7/dist-packages/wifi/scan.py", line 137, in
normalize
while lines[0].startswith(' ' * 10):
IndexError: list index out of range
but wifi --help, works fine.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/rockymeza/wifi/issues/95

Ok I guess you are on your pi. Let me look into it this weekend.
-rocky
2016年4月1日 上午6:58於 "Rocky Meza"<EMAIL_ADDRESS>
Hi!
Which os/distro are you using?
Thanks,
-rocky
2016年3月30日 上午12:28於 "rackmount"<EMAIL_ADDRESS>
Hi, Rock,
I installed wifi by pip successful, but when I run wifi or wifi scan
command, the error is same and as below:
Traceback (most recent call last):
File "/usr/local/bin/wifi", line 202, in
args.func(args)
File "/usr/local/bin/wifi", line 51, in scan_command
print_table([[cell.signal, cell.ssid, 'protected' if cell.encrypted else
'unprotected'] for cell in Cell.all(args.interface)])
File "/usr/local/lib/python2.7/dist-packages/wifi/scan.py", line 44, in
all
cells = map(Cell.from_string, cells_re.split(iwlist_scan)[1:])
File "/usr/local/lib/python2.7/dist-packages/wifi/scan.py", line 54, in
from_string
return normalize(cell_string)
File "/usr/local/lib/python2.7/dist-packages/wifi/scan.py", line 137, in
normalize
while lines[0].startswith(' ' * 10):
IndexError: list index out of range
but wifi --help, works fine.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/rockymeza/wifi/issues/95

what is the output of sudo /sbin/iwlist wlan0 scan?

HI,Rocky,
Sorry for the late reply.
Yes, you are right, I'm using it on a pi.
Raymond

On Mar 31, 2016, at 4:04 PM, Rocky Meza<EMAIL_ADDRESS>wrote:
Ok I guess you are on your pi. Let me look into it this weekend.
-rocky
2016年4月1日 上午6:58於 "Rocky Meza"<EMAIL_ADDRESS>
Hi!
Which os/distro are you using?
Thanks,
-rocky
2016年3月30日 上午12:28於 "rackmount"<EMAIL_ADDRESS>
Hi, Rock,
I installed wifi by pip successful, but when I run wifi or wifi scan
command, the error is same and as below:
Traceback (most recent call last):
File "/usr/local/bin/wifi", line 202, in
args.func(args)
File "/usr/local/bin/wifi", line 51, in scan_command
print_table([[cell.signal, cell.ssid, 'protected' if cell.encrypted else
'unprotected'] for cell in Cell.all(args.interface)])
File "/usr/local/lib/python2.7/dist-packages/wifi/scan.py", line 44, in
all
cells = map(Cell.from_string, cells_re.split(iwlist_scan)[1:])
File "/usr/local/lib/python2.7/dist-packages/wifi/scan.py", line 54, in
from_string
return normalize(cell_string)
File "/usr/local/lib/python2.7/dist-packages/wifi/scan.py", line 137, in
normalize
while lines[0].startswith(' ' * 10):
IndexError: list index out of range
but wifi --help, works fine.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/rockymeza/wifi/issues/95

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

It works fine to show all Essids.
Raymond

On Apr 2, 2016, at 7:20 PM, Rocky Meza<EMAIL_ADDRESS>wrote:
what is the output of sudo /sbin/iwlist wlan0 scan?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

@rackmount, can you paste in the output of sudo /sbin/iwlist wlan0 scan please?

wlan0     Scan completed :
Cell 01 - Address: C4:6E:1F:B3:91:52
ESSID:"CSBearings"
Mode:Managed
Frequency:2.422 GHz (Channel 3)
Quality:4/5  Signal level:-66 dBm  Noise level:-92 dBm
IE: IEEE 802.11i/WPA2 Version 1
Group Cipher : CCMP
Pairwise Ciphers (1) : CCMP
Authentication Suites (1) : PSK
IE: Unknown:
DD9C0050F204104A0001101044000102103B0001031047001054504C696E6B53636F70C46E1FB391521021000754502D4C494E4B10230009544C2D57523130343310240003322E301042001473636F7270696F6E4334364531464233393135321054000800060050F204000110110019576972656C65737320526F7574657220544C2D575231303433100800022008103C0001011049000600372A000120
IE: WPA Version 1
Group Cipher : CCMP
Pairwise Ciphers (1) : CCMP
Authentication Suites (1) : PSK
Encryption key:on
Bit Rates:1 Mb/s; 2 Mb/s; 5.5 Mb/s; 11 Mb/s; 6 Mb/s
9 Mb/s; 12 Mb/s; 18 Mb/s; 24 Mb/s; 36 Mb/s
48 Mb/s; 54 Mb/s
Cell 02 - Address: 28:C6:8E:C5:B0:69
ESSID:"YMI-NETGEAR"
Mode:Managed
Frequency:2.412 GHz (Channel 1)
Quality:5/5  Signal level:-57 dBm  Noise level:-92 dBm
IE: IEEE 802.11i/WPA2 Version 1
Group Cipher : CCMP
Pairwise Ciphers (1) : CCMP
Authentication Suites (1) : PSK
IE: Unknown:
DD1D0050F204104A0001101044000102103C0001011049000600372A000120
Encryption key:on
Bit Rates:1 Mb/s; 2 Mb/s; 5.5 Mb/s; 11 Mb/s; 6 Mb/s
9 Mb/s; 12 Mb/s; 18 Mb/s; 24 Mb/s; 36 Mb/s
48 Mb/s; 54 Mb/s
Cell 03 - Address: 00:1C:C2:1C:E3:EE
ESSID:"IDriveWiFi-E3EE"
Mode:Managed
Frequency:2.437 GHz (Channel 6)
Quality:2/5  Signal level:-73 dBm  Noise level:-92 dBm
IE: IEEE 802.11i/WPA2 Version 1
Group Cipher : TKIP
Pairwise Ciphers (2) : TKIP CCMP
Authentication Suites (1) : PSK
IE: WPA Version 1
Group Cipher : TKIP
Pairwise Ciphers (2) : TKIP CCMP
Authentication Suites (1) : PSK
Encryption key:on
Bit Rates:1 Mb/s; 2 Mb/s; 5.5 Mb/s; 11 Mb/s; 9 Mb/s
18 Mb/s; 36 Mb/s; 54 Mb/s; 6 Mb/s; 12 Mb/s
24 Mb/s; 48 Mb/s
Cell 04 - Address: 18:1B:EB:E3:73:67
ESSID:"\x00\x00\x00\x00\x00"
Mode:Managed
Frequency:2.437 GHz (Channel 6)
Quality:5/5  Signal level:-45 dBm  Noise level:-92 dBm
IE: IEEE 802.11i/WPA2 Version 1
Group Cipher : TKIP
Pairwise Ciphers (2) : CCMP TKIP
Authentication Suites (1) : PSK
IE: WPA Version 1
Group Cipher : TKIP
Pairwise Ciphers (2) : CCMP TKIP
Authentication Suites (1) : PSK
Encryption key:on
Bit Rates:1 Mb/s; 2 Mb/s; 5.5 Mb/s; 11 Mb/s; 18 Mb/s
24 Mb/s; 36 Mb/s; 54 Mb/s; 6 Mb/s; 9 Mb/s
12 Mb/s; 48 Mb/s
Cell 05 - Address: 98:FC:11:A8:07:CD
ESSID:"CSbearings"
Mode:Managed
Frequency:2.462 GHz (Channel 11)
Quality:5/5  Signal level:-57 dBm  Noise level:-92 dBm
IE: IEEE 802.11i/WPA2 Version 1
Group Cipher : TKIP
Pairwise Ciphers (2) : CCMP TKIP
Authentication Suites (1) : PSK
IE: Unknown:
DD8C0050F204104A00011010440001021041000100103B00010310470010D2CC9B6BFFDB40FDAFABEE9D1EEDB9DF10210012436973636F2D4C696E6B7379732C204C4C4310230017576972656C6573732D4E2041636365737320506F696E74102400075741503631304E104200033030301054000800060050F2040001101100075741503631304E100800020086
IE: WPA Version 1
Group Cipher : TKIP
Pairwise Ciphers (2) : CCMP TKIP
Authentication Suites (1) : PSK
Encryption key:on
Bit Rates:1 Mb/s; 2 Mb/s; 5.5 Mb/s; 11 Mb/s; 6 Mb/s
9 Mb/s; 12 Mb/s; 18 Mb/s; 24 Mb/s; 36 Mb/s
48 Mb/s; 54 Mb/s
On Sun, Apr 10, 2016 at 4:34 AM, Rocky Meza<EMAIL_ADDRESS>wrote:

@rackmount https://github.com/rackmount, can you paste in the output of sudo
/sbin/iwlist wlan0 scan please?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/rockymeza/wifi/issues/95#issuecomment-207966554


Include full DateTime for Logger

Pretty self explanatory. Would like to get the full datetime for log entries.

No reason why not… merging :-)


Disable PhEDExInjector; Enable all tiers to be injected in Rucio - wmagent branch

wmagent branch version of

Disable PhEDExInjector; Enable all tiers to be injected in Rucio #9980

 Fix #9991 - Disable PhEDExInjector; Enable all tiers to be injected in Rucio


Create more commands in the file db.json

The app needs more commands to help people beginners.

Created a PR that adds another flavor of git log. https://github.com/mariorodeghiero/git-commands/pull/6

@cbolgiano Good job, if you know more commands, feel free to contribute.


[New] support all config

Instead of define each config separately, sometimes its convenient to just use all config. So instead of:
{
  "plugins": ["effector"],
  "extends": ["plugin:effector/recommended", "plugin:effector/scope", ...]
}

we can do
{
  "plugins": ["effector"],
  "extends": ["plugin:effector/all"]
}

Should be merged before #132

Sounds reasonable, but. I am worried about conflicts. E.g., this new preset enabled https://eslint.effector.dev/rules/prefer-sample-over-forward-with-mapping.html AND https://eslint.effector.dev/rules/no-forward.html at the same time.
Shall we have a little more planning and testing before this change? 🥲

Well, these 2 rules still can be enabled currently without new all preset, or do I miss something? Not sure how it may be fixed by new preset, any suggestions?

Well, these 2 rules still can be enabled currently without new all preset, or do I miss something? Not sure how it may be fixed by new preset, any suggestions?

Now it is on the user to enable both rules. After this change, it would be on the plugin, which is not good.
I have no idea how to fix it 🙃

How user should know this behavior? I just checked my config and have both rules run same time. I mean most of the users (which don't know these behavior) just have it by default