Consider using info field for reward vector

Hello,
Thanks for this repository, it will be very useful to the MORL community :-).
I was just wondering if you think it would be a good idea to enforce gym compatibility by specifying rewards as scalar and giving the vectorial rewards elsewhere. The idea would be to use a field in the info dictionary as they do in PGMORL. This would allow to use existing RL algorithms and logging libraries out of box (e.g. stable-baselines, tensorboard logs, ...).
For example: In a DST env, if you return the treasure reward only in the reward field, you can use the DQN implementation from baselines and have insights on the average reward, as well as the episode length in the tensorboard logs. Of course, you can extract the full vectorial reward from the info dictionary in order to learn with MORL :-).
With kind regards,
Florian

Hi @ffelten!
I am glad that mo-gym is being useful :)
Regarding scalarizing rewards, that is why I created the LinearReward wrapper (see the example in the Readme as well). It makes the env return the reward as scalar and the vector reward in the info dict.
I believe it is possible to use stable-baselines when using this wrapper.
Do you think it would be better the other way around? I.e. the default is the linear reward and a wrapper makes it vectorized?
Best regards,
Lucas

Oh, I missed that one. I think we should be fine with a wrapper such as the one you mentionned. Good job :-).


Cryptodev linux

This pull request corrects issues with the current engine for /dev/crypto.
Also in:
http://rt.openssl.org/Ticket/Display.html?id=2770&user=guest&pass=guest

+1

+1

is this still an issue?

To be honest I no longer work with cryptodev. Maybe someone else would take on the patches if he's interested on the issue.

@richsalz. The OP has moved on, and no one else has stepped up to own these patches. They would need some significant work now to bring them up to date with all the latest opacity changes etc. I think we should close this. Do you agree?

Isn’t Ben interested in this?  Not saying don't close, but perhaps ask him.

Ping @benlaurie. Your opinion?

I'm interested in this and can take the lead.  However, this should be made a post 1.1.0 feature request.

Because of refactoring the patches don't apply with newer version of openssl. I adjusted Nmav's patch and it is available here: https://github.com/archlinuxarm/PKGBUILDs/blob/master/core/openssl-cryptodev/fix_sig-gen_and_allow_copy_evp_contexts.patch

I also adjusted the patch and improved it by enabling more algorithms and styling the code: https://github.com/Voker57/openssl/tree/cryptodev-patches

If anyone is interested in submitting a new PR for this, that would be great.  Please see the CONTRIBUTING file for CLA etc.

Closing this, as it became part of #1784 (through #1792)


feat: hope we can modify the tags

now tags can only generated by AI, we  can't control it , you must select it in a drop menu,which is correlated with tags table
can we  have our own tags ? for example,  when we try to add a new one we can input something ,instead of select onw from the drop menu

I am on 0.4.2 and am able to add my own tags, maybe a screenshot or somethign would help?

I am unable to see  add tag... i see that if we use tags, it will internally add as metadata in chromadb for filtering data.
I'm on the latest openwebui docker
@tkg61


31 send data to collector

this is not directly related to your story, but for now we have an interval when sending data, but also expect to be called via payload when sending data. This makes no sense it should be one or the other. So I think for now it is ok to expect we frequrently send data in an interval. But then we don't receive the payload from the command argument. Instead we would need to query the data from a source at the device. So please remove the payload parameter and simply for now use a static data payload inside the index.ts where you are currently parsing it payloadObject = JSON.parse(payload)


 Add more prop expand/collapse

Card so great, but sometime we need expand/collapse them. Maybe use this prop with item too (when we touch on mobile, it expands more button for option, it'll be better dropdown, maybe)
P/s : I tried with accordion, but the Accordion's title only get string, not  as I wish

Please be more specific so we can help.  It is most helpful to use the issue template provided.

Closing this for housekeeping.
My assumption was that the request is for adding styles and/or markup to allow expanding/collapsing the Card.  This sounds like core SUI feature request as we only handle generating valid SUI markup.  Adding styles is beyond our scope here.


Is visible in category

Hi,
This is not an issue per se, I was facing problem in Magento 2.2.6, that any new products added were not displaying in the frontend.
So I found your extension, installed and in fact it is showing an issue for the products which are not displayed in the frontend.
That is the colum is visible in Category is blank for the products which are not visible in frontend, but I couln't find any resolution, how I can address this and what does this mean, as the products are already in the category.
if you can help me understand the usage of this column It will help me address the issue.
I have already cleared caches, reindexing etc. with no help.
thank you.

Hi @pkjoshi
The column displays if the product is present in the category collection (yes/no). It should say (no) instead of (blank), that bug will be fixed soon. Can you post me a screenshot with your overview so i can take a look? Thanks, Ben

HI, I have the same problem on 2.2.6.
This is screenshot. The products are all Configurable Product

Seems there is a Bug on Magento 2.2.6
I have exported and imported all products and now all are visible.
@pkjoshi do you have used Configurable Products using Save & Duplicate?

Hi, @zenomordiante @pkjoshi
Fix is provided in new release 1.0.7. Feedback is welcome!
Thanks,
Ben


[RFC] MCAP WebSocket protocol

MCAP WebSocket
JSON-encoded control messages in ws-protocol are converted to MCAP Metadata records. All messages are binary. MCAP native record types are used for Schemas, Channels, and Messages. MCAP records are sent as-defined including the size prefix, even though it is often redundant with the WebSocket framing. This allows fragmented delivery of large messages or attachments.
Server Messages
serverInfo { name, capabilities } -> Metadata { op: "server_info", name, capabilities }
Client Messages
subscribe { [subscriptions]: { id, channelId } } -> Metadata { op: "subscribe", subscription_id, channel_id }
unsubscribe { [subscriptionIds] } -> Metadata { op: "unsubscribe", subscription_id }
... -> Metadata { op: "client_info", name, capabilities }
Common Messages
advertise { [channels] } -> Schema, Channel
unadvertise { [channelIds] } -> Metadata { op: "unadvertise", channel_id }
message { timestamp, subscriptionId, data } -> Message
status { level, message } -> Metadata { op: "status", level, message }
TBD:

Parameters
Services
Actions

Including client-published messages? In https://github.com/foxglove/ws-protocol/pull/250 we have a spec update that uses JSON for client-published messages.

My thinking is to always use MCAP as the serialization format. It's valid to publish JSON messages, but from a WebSocket transport point-of-view they would be binary payloads.

This alone would allow fragmentation but still requires all fragments to be delivered in order with no interruption. Will there be a way to tag fragmented messages with an id so they can be interspersed with other smaller messages?

The current protocol is based purely on MCAP records, and I'm reticent to drift away from that since it has the nice property of being able to stream from a file for DVR-like playback or stream to a file for efficient recording. It has the limitation of being unable to interleave individual records (same as the MCAP file format). My own two cents is this is acceptable for mcap-websocket and something that goes away with mcap-webtransport where every channel has its own QUIC channel.

I think generally this is a sensible direction to go in. We should discuss whether now is the right time, or whether there are more tactical changes we should make first.
A similar analogy is Apache Arrow (columnar table format) vs Arrow Flight (gRPC API for streaming Arrow tables).
We have heard feedback that for customers with a custom robotics stack, implementing both mcap for recording and fg-websocket for live viz is duplicative (and the protocols are unnecessarily divergent).
We also know that we have features on the horizon, such as:

DVR (alternate between live and recorded data in the same connection)
WebTransport / WebRTC support (lossy streaming over low bandwidth links)

It would be excellent if we could retain the property that the on-wire format and the file format are the same, i.e. I could pipe all data received on a socket to a file, and that mcap file would be perfectly legitimate to open in Foxglove Studio. Likewise, I could pipe a file directly into a socket, and while the other end might be surprised to receive some messages on channels it had not subscribed to, it would be a legitimate series of packets.
Here is a (non-public) document I wrote about 6 months ago with some similar ideas in it.

For posterity:

I thought some more about this.
I feel like there are a lot of unanswered questions about how this protocol will evolve (e.g. how are we going to solve ROS concepts like parameters/services/actions, how might we solve DVR, support different underlying transport protocols, etc). Therefore, if we want to go down this path, we are going to need a long baking time before we are ready to publish it as part of the official MCAP spec, during which time we may want to break backwards compatibility (with older servers, not the existing MCAPv1 spec).
Unless I'm overthinking this, I think that is at odds with shipping a quick solution for ROS better performance ROS 2 connections.

Decision for now was to move forward with more incremental updates to the foxglove websocket protocol for initial release of foxbridge.

Closing for now, we can reevaluate after reaching feature parity with rosbridge


⚠ SceneTime has degraded performance

In 223cf3b, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1573 ms

Resolved: SceneTime performance has improved in 58a8c25 after 6 minutes.


ssl connection failed

Just follow the steps, but once I did ssl connection, it crashed, which stack is as bellow. anything wrong with my settings. Tried different browser no help.
python SimpleExampleServer.py --example chat --ssl 1 --cert ./cert.pem
Traceback (most recent call last):
File "SimpleExampleServer.py", line 71, in
server.serveforever()
File "/root/kw/wss/SimpleWebSocketServer.py", line 696, in serveforever
super(SimpleSSLWebSocketServer, self).serveforever()
File "/root/kw/wss/SimpleWebSocketServer.py", line 654, in serveforever
client.handleClose()
File "SimpleExampleServer.py", line 38, in handleClose
clients.remove(self)
ValueError: list.remove(x): x not in list

You sure you posted this issue on the right github repository;)? This repository is a C++ library.


Update local.py

There seems to be a trivial bug with a wrong comment sign from another programming language. I stumbled upon it when testing nano-graphrag with Kotaemon (https://github.com/Cinnamon/kotaemon). After fixing it locally, it worked again.
Other mentions of this bug:
https://github.com/Cinnamon/kotaemon/issues/440
https://github.com/chroma-core/chroma/issues/931
Description of changes
Summarize the changes made by this PR.

Improvements & Bug fixes

Change "//" to "#"

This change is not correct. // is "integer division" in python. To future readers do not comment these lines out - it is a misunderstanding of python syntax.
The solution for the error you may see is documented on the linked threads.
https://github.com/chroma-core/chroma/issues/931#issuecomment-1665119867
https://github.com/chroma-core/chroma/issues/931#issuecomment-1665171946
https://github.com/zylon-ai/private-gpt/issues/1012#issuecomment-1859521496
https://github.com/zylon-ai/private-gpt/issues/1012#issuecomment-1863033587

Closing due to being incorrect.

Thank you and sorry for the noise! I will mention that solution somewhere in the other project.


[BUG] Cognitive Services RaiPolicyContentFilter required properties not marked as such

API Spec link
https://github.com/Azure/azure-rest-api-specs/blob/main/specification/cognitiveservices/resource-manager/Microsoft.CognitiveServices/preview/2023-10-01-preview/cognitiveservices.json#L5562
API Spec version
2023-10-01-preview
Describe the bug
The RaiPolicyContentFilter definition has five properties, none of which is marked as required. Based on my experiments, at least name, blocking, and source are required, however.
Expected behavior
Creating a RAI policy with a content filter with only a subset of properties should succeed, based on the spec.
Actual behavior
When not all filter properties are specified, the endpoint responds with 400,

"The request content was invalid and could not be deserialized."

Reproduction Steps
The spec's own example PutRaiPolicy.json doesn't work until one adds the missing properties.
Environment
No response

@tianxin-ms  Please help have a look, thank you.


Queries time out, have to restart DNSCrypt service

I've noticed this happens fairly regularly with v0.7.0 [dnscrypt proxy 2.0.31]:
PS C:\Windows\system32> nslookup www.bbc.co.uk
DNS request timed out.
timeout was 2 seconds.
Server:  UnKnown
Address:  ::1
DNS request timed out.
timeout was 2 seconds.
DNS request timed out.
timeout was 2 seconds.
*** Request to UnKnown timed-out
Query log shows requests timing out:
time:1580652681	host:::1	message:www.bbc.co.uk	type:AAAA	return:SYNTH	cached:0	duration:0	server:-
time:1580652682	host:::1	message:www.bbc.co.uk.localdomain	type:A	return:SERVER_TIMEOUT	cached:0	duration:5002	server:cisco
time:1580652684	host:::1	message:www.bbc.co.uk	type:A	return:SERVER_TIMEOUT	cached:0	duration:5000	server:cisco
Error log shows:
2020-02-02 14:10:29.9248 ApplicationUpdater System.Net.Http.HttpRequestException: An error occurred while sending the request. ---> System.Net.WebException: The remote name could not be resolved: 'raw.githubusercontent.com'
at System.Net.HttpWebRequest.EndGetResponse(IAsyncResult asyncResult)
at System.Net.Http.HttpClientHandler.GetResponseCallback(IAsyncResult ar)
--- End of inner exception stack trace ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Net.Http.HttpClient.d__59.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Net.Http.HttpClient.d__30.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.ConfiguredTaskAwaitable1.ConfiguredTaskAwaiter.GetResult()    at SimpleDnsCrypt.Helper.ApplicationUpdater.<DownloadRemoteUpdateFileAsync>d__2.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at System.Runtime.CompilerServices.ConfiguredTaskAwaitable1.ConfiguredTaskAwaiter.GetResult()
at SimpleDnsCrypt.Helper.ApplicationUpdater.d__1.MoveNext()
Toggling the DNSCrypt Service allows requests to work again.

i got this issue as well, working well with dns-proxy-2 cmd version


Refactor to support "pluggable" workflow engines

Is your feature request related to a problem? Please describe.
Add support for making it easy to support further workflow engines than the currently supported cwl-tes. These may or may not be Task Execution Schema-compatible.
Describe the solution you'd like
Before adding further workflow engines, a robust abstract solution should be conceived that answers at least the following questions:

In what format should the individual plug-ins be? Via functions? Via classes/subclasses? In another way?
How to deal with different types of workflow engines: (1) executables (2) Python libraries (3) WES services (4) non-WES web services (5) any other?
How are workflow types/versions mapped to a workflow engine?
How is installation of dependencies handled? Should all dependencies be installed? Only those configured? How will this be achieved, especially for non-pythonic engines?

For the actual addition of further services, after refactoring, priority should be given to engines that support TES as an endpoint, currently:

cwl-tes
Cromwell
Nextflow

Individual issues should be opened up for these.

This will be skipped in favor of having workflow language-specific implementations based on a common archetype for shared code.


FlatList - Extreme scrolling breaks touches with focused TextInput

Description
With a FlatList that has a fairly large number of items, sometimes you can scroll far/fast enough to permanently break something around TextInputs and touches. After "extreme" scrolling inside the FlatList, and after something is typed into a TextInput, the first touch outside the TextInput is swallowed. The second touch works as normal. It's hard to describe so I made a test project to reproduce and recorded a video to show every step and the resulting error.
The video is recorded using the iOS simulator, but the bug occurs on every iOS system we've tried so far, regardless of hardware or OS version. We have yet to reproduce it on Android.
We're not sure of the relevance of scroll speed, Y position, or number of items. Our live application has much fewer items than the test application in the video, but it's still easy to reproduce there. Probably because those items are more taxing on the hardware (images, different opacity etc). But that's all conjecture.
Either way this happens organically in our application and is preventing us from releasing our move away from ListView to production, which is a shame because overall the performance improvements with FlatList have been fantastic.
Reproduction
This repository contains the test application from the video above.  scrolltest.js has all the relevant code.
Clone the repository, npm install, react-native run-ios, quickly scroll through the list, type something in the text input and try the "Clear" button. I highly recommend watching the video so you'll know what I'm talking about.
Additional Information

React Native version: Current RN master https://github.com/facebook/react-native/commit/d5dda1b1365321c0bb64eca0d1d88909f0f6c7fd with React 16.0.0-alpha.3
Platform: iOS simulator and device. Android seems unaffected.

Please take a look at this whenever you have the time @sahrens since its holding us back from moving FlatList to production. Our feeling is that its not just happening on extremely fast scroll, but thats a very easy way to reproduce it. It happens otherwise too just not deterministically. Cheers!

iOS only? And you don't see the same problem with ListView and the same level of "extreme scrolling"? Very odd. I wonder if it could be related to event coalescing? @shergin - maybe you can take a look?

I gave it another shot now that my scrolling hand is Usain Scrollt and could not make it happen with ListView nor Android.
On Android I would sometimes run into https://github.com/facebook/react-native/issues/12884, but that's temporary and delays events, so different in nature from this permanent event skip.

OMG, this bug report is so amazing! Thank you, @iggyfisk!
I will try to debug it soon.

I am facing a similar issue with my project. While the text input is focused 2 things happen :
1 => scrolling works as usual, so this is great
2=> when I touch the list to select an option the first touch doesn't work, I checked onBlur is called at this point, and then the second press works just fine
I am not sure if it is the same problem you are facing, If not do you have a solution to this @iggyfisk ?
Thanks 👍

That doesn't sound like the exact same thing @CheragV, once I break things by monster scrolling a FlatList, the onBlur handler on TextInputs still works as expected. It doesn't fire on the first touch after typing since that touch gets eaten by the bug.
Either way my solution so far is to spend an hour doing retakes of a bug report video, then wait for the California boys to sort it out.

I found an actual problem. The fix is coming! 🎉

What a hero @shergin! Looking forward to it!

I'm also running into a similar issue where extreme scrolling of a FlatList breaks touches. In my app I have a list of 100-200 items where each item has two touchable areas. After a bit of scrolling elements stop responding to touches, sometimes it takes a second try for it to work but sometimes it's 3-4 touches. I don't use TextInputs.

https://github.com/facebook/react-native/issues/11809#issuecomment-285938537

Was fixed in 23c2a6cf1ae297182a62d1a8143b55b186f5e1b7. 🎉

Thanks a lot for looking into it Shergin.
On 20 Mar 2017 7:45 AM, "Valentin Shergin"<EMAIL_ADDRESS>wrote:

Was fixed in 23c2a6c
https://github.com/facebook/react-native/commit/23c2a6cf1ae297182a62d1a8143b55b186f5e1b7.
🎉
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/facebook/react-native/issues/12976#issuecomment-287687243,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABA-Y_LX8iTH4PGuy99QgX249Twa2Vvjks5rniCNgaJpZM4MfS1P
.

Fixed as far as I could test, good hustle team!


Levi/racecondition

This fixes a witnessed race condition with filesystem access when running optimized code on macOS which prevented the createFile function from being successful. This also addresses other similar potential issues by removing unnecessary calls to file system functions in the init method.

Seems to be a configuration issue... ?
2019-05-21T22:02:35.4953901Z [command]/bin/bash --noprofile --norc /__w/_temp/6763c081-68bf-460a-820c-3380ba38ce89.sh
2019-05-21T22:02:36.4074829Z warning: you may be able to install zlib using your system-packager:
2019-05-21T22:02:36.4075604Z      apt-get install zlib
2019-05-21T22:02:36.4075903Z
2019-05-21T22:02:36.4719049Z [1/4] Compiling Swift Module 'ZIPFoundation' (7 sources)
2019-05-21T22:02:37.8868394Z <module-includes>:1:10: note: in file included from <module-includes>:1:
2019-05-21T22:02:37.8868982Z #include "shim.h"
2019-05-21T22:02:37.8869186Z          ^
2019-05-21T22:02:37.8869629Z /__w/1/s/Sources/CZLib/shim.h:15:9: error: 'zlib.h' file not found
2019-05-21T22:02:37.8869755Z #import <zlib.h>
2019-05-21T22:02:37.8869894Z         ^
2019-05-21T22:02:37.8870238Z /__w/1/s/Sources/ZIPFoundation/Data+Compression.swift:208:8: error: could not build C module 'CZlib'
2019-05-21T22:02:37.8870434Z import CZlib
2019-05-21T22:02:37.8870537Z        ^
2019-05-21T22:02:37.9821225Z ##[error]Bash exited with code '1'.
2019-05-21T22:02:38.0212087Z ##[section]Finishing: Linux Swift 5.0

Seems to be a configuration issue... ?
I just updated the CI configuration.
Looks like the official Swift 5 Docker image no longer has zlib pre-installed. I switched to a user provided image that includes zlib. Please pull in from development.

Updated.

 Sorry I didn't have time to provide a test case which exhibited the race condition... as you might expect, that'd be a bit challenging.


Flutter Issue During Running App

To avoid unexpected runtime failures, or future build failures, try to migrate this app to the V2 embedding.

It clearly says what needs to be done in the terminal. So why list it as issue? @ErKaran98


EA SPORTS FIFA Football Démo [PCSB00083]

App summary

App name: EA SPORTS FIFA Football Démo
App serial: PCSB00083
App version: 1.00

Vita3K summary

Version: v0.1.6
Build number: 2957-4343481c
Commit hash: https://github.com/Vita3K/Vita3K/commit/4343481
CPU backend: Dynamic
GPU backend: both

Test environment summary

Tested by: User01
OS: Windows 10
CPU: AMD Ryzen 5 2600 Six-Core Processor
GPU: NVIDIA GeForce RTX 3060 Ti/PCIe/SSE2
RAM: 8 GB

Issues

Screenshots

Log
vita3k.log

Recommended labels

Nothing

Vita3K summary

Version: v0.1.6
Build number: 3069
Commit hash: https://github.com/vita3k/vita3k/commit/73fc1be1
CPU backend: Dynarmic

Test environment summary

Tested by: othmanead
OS: Windows 11
CPU: Intel i5-10300H
GPU: RTX 3060 mobile
RAM: 16 GB

Issue:
Game plays with audio and graphical issues
Screenshots:

Log
vita3k.log
Recommended labels

Ingame+
Audio quality
Graphics bugs


this page does not detail the requirement for when webhooks will work (does not work with slack webhooks)

Monitor alerts action group for webhooks does not work with slack webhooks
This page does not list the requirement for webhook conditions to work. Intentionally or unintentionally webhook notifications to slack channels does not work without a logic app.

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

ID: b1390bec-bae0-2fe7-a29c-a242584fd7b3
Version Independent ID: 27892ee4-2ec3-ea70-f747-80a9bc914fd7
Content: Webhook actions for log alerts in Azure alerts - Azure Monitor
Content Source: articles/azure-monitor/platform/alerts-log-webhook.md
Service: azure-monitor
Sub-service: alerts
GitHub Login: @yanivlavi
Microsoft Alias: yalavi

@vairakkumar-vairaperumal Thank you for your query, we will look into it and get back to you at the earliest.

@vairakkumar-vairaperumal Thanks again for sharing your feedback on this. You are right that to trigger complex actions with Azure Monitor alerts , slack in your example you need to leverage logic apps only.
We will further review and see how we can have a backlog item to enhance the documentation to reference the complex actions with in the webhooks  documentation to bring more clarity. For now, will go ahead and close this issue, however if you have any further queries please tag us in your reply and will be happy to reopen the thread and continue the conversation. Thank you


Use style classes instead of two styles

Using style classes in a single style will allow an animated transition, unlike the entire reload we're doing now between two styles.

At one point, we discussed shipping Light and Dark as one style using style classes, but we settled on two styles. @peterqliu may remember why.

for Light/Dark, I think there might be subtle differences in styling besides coloration. @andreasviglakis would be in a better position to answer

Chatted with @andreasviglakis today, I'll look into doing this myself.

Nope — punting on this until style classes become better supported.


Tip probe saves incorrect tip length

overview
Sometimes after tip probe, a pipette will save incorrect data (tip length too short), causing subsequent tip probe to fail.
Have seen on multiple beta customer robots, and on Sunset, but have not yet been able to reproduce reliably.

Addressed by #1816 and #1817


Cross route scrolling: Not working if anchor is dynamic content

Given a link like About anchor.
If the element
Heading
 gets generated by a ngFor loop on the about page, scrolling does not work after navigation ended.
See example app for reproduction.

Works again with angular rc.5


Fix filterchain match's incorrect early match

Fixing incorrectly matched filterchain if lMatch.FilterChain==nil.
This moves filtering unrelated to lMatch.FilterChain before the nil check.
Signed-off-by: Mandar U Jog<EMAIL_ADDRESS>In absence of this PR, If a patch has only listener match but no filterchain match, it is incorrectly applied since missing filterchain match-clause was treated as a positive match.

@mandarjog: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
d80022b9c1d92de99d3b836f7155fdea04147b57
link
true
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test release-notes_istio

/cherry-pick release-1.12

@mandarjog: once the present PR merges, I will cherry-pick it on top of release-1.12 in a new PR and assign it to you.

In response to this:

/cherry-pick release-1.12

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/cherry-pick release-1.11

@mandarjog: once the present PR merges, I will cherry-pick it on top of release-1.11 in a new PR and assign it to you.

In response to this:

/cherry-pick release-1.11

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/cherry-pick release-1.10

@mandarjog: once the present PR merges, I will cherry-pick it on top of release-1.10 in a new PR and assign it to you.

In response to this:

/cherry-pick release-1.10

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@mandarjog: new pull request created: #35838

In response to this:

/cherry-pick release-1.12

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@mandarjog: #35836 failed to apply on top of branch "release-1.11":
Applying: Fix filterchain match's incorrect early match
Using index info to reconstruct a base tree...
M	pilot/pkg/networking/core/v1alpha3/envoyfilter/listener_patch.go
M	pilot/pkg/networking/core/v1alpha3/envoyfilter/listener_patch_test.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/networking/core/v1alpha3/envoyfilter/listener_patch_test.go
CONFLICT (content): Merge conflict in pilot/pkg/networking/core/v1alpha3/envoyfilter/listener_patch_test.go
Auto-merging pilot/pkg/networking/core/v1alpha3/envoyfilter/listener_patch.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Fix filterchain match's incorrect early match
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to this:

/cherry-pick release-1.11

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@mandarjog: new issue created for failed cherrypick: #35839

In response to this:

/cherry-pick release-1.11

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@mandarjog: #35836 failed to apply on top of branch "release-1.10":
Applying: Fix filterchain match's incorrect early match
Using index info to reconstruct a base tree...
M	pilot/pkg/networking/core/v1alpha3/envoyfilter/listener_patch.go
M	pilot/pkg/networking/core/v1alpha3/envoyfilter/listener_patch_test.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/networking/core/v1alpha3/envoyfilter/listener_patch_test.go
CONFLICT (content): Merge conflict in pilot/pkg/networking/core/v1alpha3/envoyfilter/listener_patch_test.go
Auto-merging pilot/pkg/networking/core/v1alpha3/envoyfilter/listener_patch.go
CONFLICT (content): Merge conflict in pilot/pkg/networking/core/v1alpha3/envoyfilter/listener_patch.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Fix filterchain match's incorrect early match
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to this:

/cherry-pick release-1.10

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@mandarjog: new issue created for failed cherrypick: #35840

In response to this:

/cherry-pick release-1.10

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


removed azdo url check

Explicit domain check is not required as the org url is entered explicitly by the user. This may resolve the issues with private AzDO deployment sucn as #55

Checks look good can we complete this PR ?


installing brew - fatal: The remote end hung up unexpectedly

Press RETURN to continue or any other key to abort
==> Downloading and installing Homebrew...
fatal: The remote end hung up unexpectedly
Failed during: git fetch origin master:refs/remotes/origin/master --tags --force
Mac-Pro-USER:~ MacOSX$

From the issue template:

Please note we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information. If you do not read fill out the issue template repeatedly you will be blocked from ever submitting issues to Homebrew again.

If you add the necessary information this issue may be reopened.


Manually correcting segmentation

I know that the files *.pseg.png store the coordinates of the automatic line segmentation. I have seen mention on the Web of the use of GIMP for manipulating these coordinates, but without further details on how this is done. When I open GIMP on these files, I see nothing that shows the segmentation, nor anything that I can edit manually.
For background: I am trying to do OCR for some documents that are in a poor state, with smudges and faded ink, and no matter how much image preprocessing I do, automatic segmentation fails on at least some parts of the page. I see manual adjustment as the only viable way forward. That is, I would like to manually remove lines, add new lines, change the positions of lines, and ideally also change the order of lines, before the actual OCR is done. Can I insert this manual correction into the usual OCRopus workflow?
Thanks in advance for your time.
Mark-Jan Nederhof

The *.pseg.png files are normal PNG files where the color code is used to encode the information about layout, see here for more information https://github.com/tmbdev/ocropy/wiki/OCRopus-File-Formats#physical-layout .
Since some time you can now also use masks to help the layout segmentation. I think this is now yet documented well, but you can have a look at the initial pull request.


BigInt support & WalletConnect implementation

Reviewing changes. ETA ~1w.

Entering 0 for Plasma Amount under the Plasma Options causes the following white square to appear.

Entering 0 for Plasma Amount under the Plasma Options causes the following white square to appear.

I've reproduced this bug. Imo this section needs a refactor.


Building docker for Cardano

Training to build cardano in docker

My concern with this is no one will maintain it, and Dockerfile's are not known for their stability over time...
The recommended way to build a docker image is using nix:


macOS 10.13 : I get an 'Illegal instruction: 4" on launch

Still getting the same result as in #103
Fawkes 0.3.1
macOS 10.13
Crashes on click, here is what that does when launching with command line
[nico@MacNico] ()  /Applications/Fawkes.app/Contents/MacOS
$ ./Fawkes
Illegal instruction: 4

Can you try the 0.3 version? You can download that at this link.

Can you try the 0.3 version? You can download that at this link.

Can you try the 0.3 version? You can download that at this link.

Hi..
I had the exact same error with 0.3. I opened the issue #103  for this, and you told me I had to try 0.3.1
I did and encountered the same issue.

Can you try the 0.3 version? You can download that at this link.

Hi..
I had the exact same error with 0.3. I opened the issue #103  for this, and you told me I had to try 0.3.1
I did and encountered the same issue.

Can you try the binary version? If it does not work, would you paste the error messages here.

Can you try the binary version? If it does not work, would you paste the error messages here.

If by binary version you mean the dmg that's on your website, that's exactly the one I'm trying to use.

Can you try the binary version? If it does not work, would you paste the error messages here.

It says exactly the same thing:
[nico@MacNico] ()  ~/Downloads
$ ./protection-v0.3
Illegal instruction: 4
[nico@MacNico] ()  ~/Downloads
$ ./protection-v0.3 --help`
>
[nico@MacNico] ()  ~/Downloads
$ ./protection-v0.3 --help
Illegal instruction: 4
[nico@MacNico] ()  ~/Downloads
$ [nico@MacNico] ()  ~/Downloads

No installation required: https://dev.scrb.ai

I downloaded the binary v1.0 from https://sandlab.cs.uchicago.edu/fawkes/#code an got the same error:
$ ./protection -d imgs/
Illegal instruction: 4

MacOS 10.13.6
(I tried the v1.0-DMG before and that didn't work either.)


add readmes to all top level directories

I think some workspace crates are missing them

After https://github.com/artichoke/artichoke/pull/1776 lands, the only directories that are missing READMEs are:

artichoke-load-path (pending removal once I can bring mezzaluna-feature-loader to parity)
mezzaluna-feature-loader
ui-tests

For ui-tests, I think the README should looks something like:

What this workspace is
How to run the tests
Do you need to cargo install cargo-insta?
How to add a new test (and why you'd add one here)


Increase cacheContextLifetime to reduce false reports

Replace #32001.
To prevent the context cache from being misused for long-term work (which would result in continuously using invalid cache without awareness), the context cache is designed to exist for a maximum of 10 seconds. This leads to many false reports, especially in the case of slow SQL.
This PR increases it to 5 minutes to reduce false reports.
5 minutes is not a very safe value, as a lot of changes may have occurred within that time frame. However, as far as I know, there has not been a case of misuse of context cache discovered so far, so I think 5 minutes should be OK.
Please note that after this PR, if warning logs are found again, it should get attention, at that time it can be almost 100% certain that it is a misuse.

CI failed.


Code method signatures at the interface level

There's a few examples of this, but method signatures should always be:
Map instead of HashMap
List instead of ArrayList
Hope that makes sense.

Believe this is fixed.


Remove backend packages

Related Issue(s):

#432

Description:

Removes backend-specific code, configuration, and documentation
Going forward, the SQLAlchemy and PgSTAC backends will be distributed as separate packages and managed in their own repos (stac-fastapi-sqlalchemy and stac-fastapi-pgstac, respectively). The main Dockerfile and docker-compose.yml have been removed since there is no longer a deployable backend associated with this repo.
Distributes stac-fastapi.api, stac-fastapi.types, and stac-fastapi.extensions as modules under a single stac-fastapi package.
Import statements for the individual modules will not change, but downstream applications will need to update their dependency from something like...stac-fastapi.api==2.4.1
stac-fastapi.types==2.4.1
stac-fastapi.extensions==2.4.1

...to something like...stac-fastapi==2.5.0

PR Checklist:

[x] Code is formatted and linted (run pre-commit run --all-files)
[x] Tests pass (run make test)
[x] Documentation has been updated to reflect changes, if applicable, and docs build successfully (run make docs)
[x] Changes are added to the CHANGELOG.
[ ] Remove or add CI to implement validation checks

The test data at tests/data/joplin probably isn't used anywhere, but maybe its nice to keep around?

Yeah, it might make sense to keep it around for now. I'm thinking that as we expand the test suite we may want  a simple backend mocker that just reads files from a directory. Having these files available would probably make that easier to put together in the future.

Can remove the alembic exclude from the find_namespace_packages call in setup.py.

Fixed in 2d72c588151fc16eec1564553a653a039bce6c58

NB there have been some changes to backend-specific code that were dropped by 22d6a01, so we will need to go back and backport any of those to the backend-specific repos (if they haven't been backported already).

Since this appears to have stalled out, I've made an RFC with a plan to finish the backend breakout. Tl;dr: I'm proposing we punt on the separation of responsibilities (e.g. https://github.com/stac-utils/stac-fastapi/pull/453) -- while it's a good idea, it's becoming a blocker that's holding the stac-fastapi ecosystem in an undefined state. While we might end up with pain down the road to synchronize the backend repos with the main ones, it's my hope that we can use API breakages v3.0 to solve the separation of concerns in the medium-term.

@gadomski That makes sense to me, we can always come back to these changes in the future in a later major version.  I think the more important thing is getting the backends split out, regardless of the separation of concerns.


[Question] sql hive  load data inpath '/ods/ods_xx/${db_date}'  error

**For better global communication, Please describe it in English. If you feel the description in English is not clear, then you can append description in Chinese(just for Mandarin(CN)), thx! **
Describe the question
A clear and concise description of what the question is.
it happen [ERROR] 2021-07-26 16:00:29.007  - [taskAppId=TASK-8-79-125]:[167] - sql task error: org.apache.hive.service.cli.HiveSQLException: Error while compiling statement: FAILED: SemanticException Line 1:18 Invalid path ''/fake/origin_data/db/ldr_proc/?/'': No files matching path hdfs://cluster01/fake/origin_data/db/ldr_proc/%3F
Which version of DolphinScheduler:
-[1.1.0-preview]
1.3.6
Additional context
Add any other context about the problem here.
Requirement or improvement

Please describe about your requirements or improvement suggestions.

Can your load data script be executed successfully in the Linux environment? Perhaps the tenant executing the command does not have permission to this directory in HDFS

Can your load data script be executed successfully in the Linux environment? Perhaps the tenant executing the command does not have permission to this directory in HDFS

I used  '/ods/ods_xx/$[yyyy-MM-dd-1]/'   instead of  /ods/ods_xx/${db_date} ,and it executed successsfully

hi,Is your problem solved?

hi,Is your problem solved?

yes, but  I want to use  a global paramater

hi,Is your problem solved?

yes, but I want to use a global paramater

This feature will be released in 2.0, and the 1.3.X series will not release this content for the time being, you can view it on the dev branch now. this is doc https://dolphinscheduler.apache.org/zh-cn/docs/dev/user_doc/globalParams.html


Fix javadoc warnings

GearVRf-DCO-1.0-Signed-off-by: Mihail Marinov<EMAIL_ADDRESS>

Nice.  Thank you very much!


TypeError: type.getFields is not a function

Hi there, great work on this project so far! I ran across an issue in typeToInterface in fromSchema/index.ts. If a GraphQLUnionType is used in the schema, it throws TypeError: type.getFields is not a function at Line 173 in :
    let isInput: boolean = type instanceof GraphQLInputObjectType;
    const f1: GraphQLInputFieldMap | GraphQLFieldMap<any, any> =
      (type instanceof GraphQLInputObjectType) ? type.getFields() : ((type as GraphQLObjectType).getFields)();
    let f: Array<GraphQLField<any, any> | GraphQLInputField> = Object.keys(f1).map(k => f1[k]);

nice, thanks for the report.
do you have a minimal reproduction so I can take a further look?

Sure, try running the schema.json gist I posted up at: https://gist.github.com/arondesign/acab6ce16b8a2128e6431de6c6a9f0b4

thanks, found the issue 😄 #66

@arondesign should be fixed in v1.0.1

Awesome!
On Fri, May 26, 2017 at 11:49 AM, Brett Jurgens<EMAIL_ADDRESS>wrote:

@arondesign https://github.com/arondesign should be fixed in v1.0.1
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/avantcredit/gql2ts/issues/65#issuecomment-304360578,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADfk8NYoeTCfW28ZOmHfxgW502MWi-T3ks5r9x6hgaJpZM4Nn19G
.

Hi @brettjurgens, I think this still needs one adjustment. If you run the schema.json file I sent you, you'll notice in
  interface IQuery {
    __typename: string;
    Products: IProducts | null;
    ProductRoute: IProductUnion | null;
    Product: IProduct | null;
  }

IProducts is undefined. However, the following is generated
  type ConfigurationInterface = IHaloEarrings | IHaloRing;

  interface IConfigurationInterface {
    __typename: string;
    attributes: any | null;
    components: any | null;
    id: string | null;
    kind: string | null;
    images: Array<string> | null;
    price: number | null;
    msrp: number | null;
    hero: string | null;
    heroSecondary: string | null;
    name: string | null;
    thumbnail: string | null;
    description: string | null;
    route: string | null;
    shortDescription: string | null;
    configuration: any | null;
    parts: any | null;
    video: string | null;
    relatedOptions: IRelatedOptions | null;
  }

The unioned types do share a common interface (ConfigurationInterface), but the output for
  type ConfigurationInterface = IHaloEarrings | IHaloRing;

should instead read
  type IProductUnion = IHaloEarrings | IHaloRing;

Or of course, just "ProductUnion" for both.

whoops. don't know what i was thinking there. fixed in #69

@arondesign 1.0.2 should work, i ran it against your schema and it looked good


feat(IICO): add view functions for current valuations

I removed the non required initialization to keep the same style.


Use llvm-ar, llvm-ranlib for Android NDK

For newer Android NDK, target-triple-ar|ranlib has been replaced to llvm-ar|ranlib. Which could be an issue when compiling libs like https://github.com/sfackler/rust-openssl .
This PR tries to use llvm-ar|ranlib when the compiler family is Clang.

Thank you!


use secureValue for secure env variables

ContainerInstanceManagementModels.EnvironmentVariable has a property secureValue which should be used for secure environment variables. The other property value is used for non-secure environment variables.
This should address #47

@kanika1894 what do we need to do to get this reviewed?

Thanks for your contribution @el-pato. Will look into it.

I guess it will be good if we can do a slight update in action.yml.
https://github.com/Azure/aci-deploy/blob/master/action.yml#L120
We can have something like key=secureValue.
Feel free to update any relevant documentation regarding this.

IMO it'd be okay to leave the description as-is. You're still passing them in key=value format, it's just that the values are meant to be secure.
I'll add a small change to the readme to make the behavior of secure environment variables a little more clear.

@kanika1894 updated the Readme. Look good?

Sorry for late reply, LGTM, thanks @el-pato !


recent_conversations: Exclude muted user PMs from recent conversations.

Earlier muted private conversations were shown in recent conversations. This commit exclude them from recent conversations.

Fixes: #24299

Screenshots and screen captures:

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

@alya please review this PR.

Thanks! It looks good to me for conversations where all users are muted, including live updates.
I think for group PMs where some of the users are muted, we should show the names of muted users as "Muted user", like we do in the message header bar:

Glad to see us fixing this bug. :)

Thanks! It looks good to me for conversations where all users are muted, including live updates.
I think for group PMs where some of the users are muted, we should show the names of muted users as "Muted user", like we do in the message header bar:

I have updated the PR and addressed the change suggested.

@Ujjawal3 thanks for working on this! I left a couple of comments above.

We should also add a test for this new logic in web/tests/recent_topics.test.js. Thanks!

@abhijeetbodas2001 I have addressed your feedback. Please have a look.

The large amount of mocking in the recent_topics_ui.js tests is already a problem; posted a suggestion on how to avoid making it worse. But I think longer term, we're going to need to rewrite that test to start with a set of normal message objects and mocking way fewer data modules.

OK, merged; I think we do want to rewrite the "recent topics" tests to just use a larger base set of messages that include private messages too, and more generally mock 5x fewer modules (in favor of just initializing them with mock data), but that's a longer-term project that we don't want to block this bug fix. @amanagr FYI, though I think cleaning up this test suite is not worth interrupting your other work.


Update zio, zio-streams to 2.0.0-RC4

Updates

dev.zio:zio
dev.zio:zio-streams

from 2.0.0-RC1 to 2.0.0-RC4.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio" } ]

labels: library-update, early-semver-pre-release, semver-spec-pre-release, commit-count:1

Superseded by #68.


Allow binding to specific interface in GUI mode

Would like to see the option to bind to a specific network interface (-a option) to be valid generally. There's no good reason why this option should only be valid in non-GUI mode.

@phansson, I have created the pull request.
If you have time, feel free to test my implementation before @Nilhcem reviews it.
Thanks.

@phansson, @Vest,
Many thanks,
This has been merged and will be available when version 2.1 is published


Historical lines?

Hi I want to get the rsi of the stocks programmatically. So i am looking for historical line function and couldn't find it. Does any one know how to located it?

Thanks for the question!
I haven't figure out the rsi yet.
For the region id that you asked in the other issue, you can find the answer here.
https://github.com/tedchou12/webull/wiki/What-is-the-region_id%3F

Use the get_bars method which returns a pandas dataframe of ohlcv.  Then use your TA library to calculate the RSI.  I'd suggest pandas_ta.
>>>help (wb.get_bars)
Help on method get_bars in module webull.webull:

get_bars(stock=None, tId=None, interval='m1', count=1, extendTrading=0, timeStamp=None) method of webull.webull.webull instance
    get bars returns a pandas dataframe
    params:
        interval: m1, m5, m15, m30, h1, h2, h4, d1, w1
        count: number of bars to return
        extendTrading: change to 1 for pre-market and afterhours bars
        timeStamp: If epoc timestamp is provided, return bar count up to timestamp. If not set default to current time.

Found it. Thanks a lot for you help.
On Sun, Dec 27, 2020, 2:32 PM Mike M<EMAIL_ADDRESS>wrote:

Use the get_bars method which returns a pandas dataframe of ohlcv. Then
use your TA library to calculate the RSI. I'd suggest pandas_ta.

help (wb.get_bars)
Help on method get_bars in module webull.webull:

get_bars(stock=None, tId=None, interval='m1', count=1, extendTrading=0, timeStamp=None) method of webull.webull.webull instance
get bars returns a pandas dataframe
params:
interval: m1, m5, m15, m30, h1, h2, h4, d1, w1
count: number of bars to return
extendTrading: change to 1 for pre-market and afterhours bars
timeStamp: If epoc timestamp is provided, return bar count up to timestamp. If not set default to current time.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tedchou12/webull/issues/120#issuecomment-751507227,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACI7TB4ZR6FG62434YRRTRDSW6DS7ANCNFSM4VKGVKKQ
.


Reduce GC allocations

GC is killing the mobile version. It can freeze the game for around 200 ms on every 2-3 seconds.
Here are the screenshots of profiler showing what is allocating the memory (at commit 7725dfb):
Complete list

imGUI details

behaviour update details

As of version 3.2, GC is freezing the Android game on every 10 seconds, when running at 17 FPS.


feature Add new parameters to 'trim'.

Proposed changes
mysql> SELECT trim('ababccaab','ab') str;
+------+
| str  |
+------+
| cca  |
+------+

Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[x] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall

run p1


Mujoco - Unity Plugin Crash - 2.2.1 on 2020.3

I downloaded the latest 2.2.1 version of Mujoco, both windows binaries and source code.
I am running Windows 10, Unity 2020.3.36f1.
Create a vanilla 3D project from unity hub
I imported the package from source code (using package.json) as suggested. I copied the DLL in place.
I imported arm26.xml in unity (after checking it out in simulate program)
Saved the scene, hit play, crash!

Editor_crash.log
?

If it's worth noting, I installed 2021.3.6f1, same issue.
I made sure there are no floating mujoco dlls of earlier releases, deleted them all.
I double checked with 2.2.0 - same issue, restarted my pc and check again, still the same.
I know I tried an earlier plugin version successfully many months ago (5+), but I don't recall which one.

I have met the same issue, also reverting to 2.2.0 solved the problem.

Unity -- Project. Folder Package (where you put the mujoco.dll in this folder). Select this mujoco.dll. And see the Unity Inspector. Click Editor (ticked). Click Apply.
Works with (https://github.com/deepmind/mujoco/releases) mujoco 2.20.
Make sure you copy the mujoco.dll from the unzipped folder (mujoco-2.2.0-windows-x86_64\bin) to your User folder (C:\Users\YourName\MuJoCo\bin)

Closing this for now, please re-open if the issue persists.


Disable New Git View submenus

Background
I was wondering if there is a way to disable the new git submenus or if this could be added as a feature. In the new git submenus it is now more difficult and to access frequently used features like "Push" and "Pull". In the old git menu these were near the top an easily accessible. I don't think these should be given the same precedence as features that will be infrequently used by most users like "Abort Rebase", "Rename Branch".
Question
Is there a way to get the old (flat) git menu back?
For reference:
Old git menu:

New git submenu:

/duplicate of #104762, and as a workaround you may be interested in https://github.com/microsoft/vscode/issues/104762#issuecomment-674902220


onDisconnect

Is there planned onDisconnect and onConnecting events?
I am rewriting my app from Android to Flutter. And in Android package there was these states. Is there planned something like this?

There is onConnect callback on WampClient.
Is it usable?

There is onConnect callback on WampClient.
Is it usable?

Yes, it is usable. I would like to have also at least onDisconnect . For example when by some reason client disconnects, i would be able to call reconnect method.

I was previously using Jawampa on Android.

Created pull request: https://github.com/kkazuo/dart-wamp-client/pull/6
Inside it i have added onDisconnect event.

Closing, cause merged in master!


Fix WebPartTitle font-size so they change as the browser screen size changes

Q
A

Bug fix?
[X]

New feature?
[ ]

New sample?
[ ]

Related issues?
fixes #114

What's in this Pull Request?
This pull request adds CSS media queries to adjust the <WebPartTitle /> font-size at different browser screen sizes in alignment with what I've observed on out-of-the-box SharePoint WebParts.

@ThomasLamb thanks this got merged and will be available in version 1.8.0.


Memory Leak in chat completion create

Confirm this is an issue with the Python library and not an underlying OpenAI API

[X] This is an issue with the Python library

Describe the bug
After Making many streaming calls from a flask app the process memory is infinitely increasing and it is never getting reduced.
Even after performing close on response and client.
To Reproduce

Wrap the below code in flask api or you can run as a long running process as well.
Invoke this method in a loop or for 1000 you will be able to see the process gradually takes up more memory.
Even performing manual gc didn't help.

Code snippets
def make_stream_call_with_close():
    client = OpenAI(api_key=os.environ.get("OPENAI_API_KEY"))
    responses = client.chat.completions.create(
        model='gpt-3.5-turbo',
        messages=[{"role": "user", "content": "Say exactly one word."}],
        stream=True,
    )
    responses.response.close()
    print(responses.response.is_closed)
    client.close()
    print(client.is_closed())

OS
macOS,Linux
Python version
4.10.13
Library version
openai==1.3.4 , httpx==0.27.0 , pydantic==2.7.1, pydantic_core==2.18.2

After performing around 10-15 requests
Snapshot comparison before invoke and after invoking once 1.
00:00:00 Top 10 differences: /Users/jogireddy/PycharmProjects/flaskProject/.venv/lib/python3.10/site-packages/openai/_response.py:238: size=61.4 KiB (+61.4 KiB), count=525 (+525), average=120 B/usr/local/Cellar/python@3.10/3.10.13/Frameworks/Python.framework/Versions/3.10/lib/python3.10/abc.py:123: size=54.1 KiB (+54.1 KiB), count=614 (+614), average=90 B<frozen importlib._bootstrap_external>:672: size=94.6 KiB (+51.6 KiB), count=1546 (+570), average=63 B<frozen importlib._bootstrap>:241: size=30.6 KiB (+30.6 KiB), count=316 (+316), average=99 B/usr/local/Cellar/python@3.10/3.10.13/Frameworks/Python.framework/Versions/3.10/lib/python3.10/functools.py:52: size=21.4 KiB (+21.4 KiB), count=106 (+106), average=207 B/usr/local/Cellar/python@3.10/3.10.13/Frameworks/Python.framework/Versions/3.10/lib/python3.10/functools.py:58: size=15.8 KiB (+15.8 KiB), count=239 (+239), average=68 B/usr/local/Cellar/python@3.10/3.10.13/Frameworks/Python.framework/Versions/3.10/lib/python3.10/typing.py:694: size=14.8 KiB (+14.8 KiB), count=32 (+32), average=475 B/Users/jogireddy/PycharmProjects/flaskProject/.venv/lib/python3.10/site-packages/openai/_resource.py:22: size=10184 B (+10184 B), count=70 (+70), average=145 B/usr/local/Cellar/python@3.10/3.10.13/Frameworks/Python.framework/Versions/3.10/lib/python3.10/typing.py:989: size=4464 B (+4264 B), count=62 (+59), average=72 B/usr/local/Cellar/python@3.10/3.10.13/Frameworks/Python.framework/Versions/3.10/lib/python3.10/http/cookiejar.py:964: size=2720 B (+2720 B), count=6 (+6), average=453 B
Snapshot compare before invoking and after invoking for 8-10 times in loop
00:00:00 Top 10 differences: /Users/jogireddy/PycharmProjects/flaskProject/.venv/lib/python3.10/site-packages/openai/_response.py:238: size=899 KiB (+899 KiB), count=7731 (+7731), average=119 B/usr/local/Cellar/python@3.10/3.10.13/Frameworks/Python.framework/Versions/3.10/lib/python3.10/functools.py:58: size=190 KiB (+190 KiB), count=2815 (+2815), average=69 B/usr/local/Cellar/python@3.10/3.10.13/Frameworks/Python.framework/Versions/3.10/lib/python3.10/tracemalloc.py:115: size=126 KiB (+126 KiB), count=1615 (+1615), average=80 B/usr/local/Cellar/python@3.10/3.10.13/Frameworks/Python.framework/Versions/3.10/lib/python3.10/abc.py:123: size=52.7 KiB (+52.7 KiB), count=591 (+591), average=91 B<frozen importlib._bootstrap_external>:672: size=94.4 KiB (+51.4 KiB), count=1545 (+569), average=63 B/Users/jogireddy/PycharmProjects/flaskProject/.venv/lib/python3.10/site-packages/openai/_resource.py:17: size=50.0 KiB (+50.0 KiB), count=696 (+696), average=73 B/Users/jogireddy/PycharmProjects/flaskProject/.venv/lib/python3.10/site-packages/openai/_resource.py:22: size=30.9 KiB (+30.9 KiB), count=406 (+406), average=78 B<frozen importlib._bootstrap>:241: size=30.6 KiB (+30.6 KiB), count=315 (+315), average=99 B/Users/jogireddy/PycharmProjects/flaskProject/.venv/lib/python3.10/site-packages/openai/_resource.py:23: size=22.5 KiB (+22.5 KiB), count=360 (+360), average=64 B/Users/jogireddy/PycharmProjects/flaskProject/.venv/lib/python3.10/site-packages/openai/_resource.py:21: size=22.5 KiB (+22.5 KiB), count=360 (+360), average=64 B
Snapshot before invoking and after invoking for 10-15 times
/Users/jogireddy/PycharmProjects/flaskProject/.venv/lib/python3.10/site-packages/openai/_response.py:238: size=1112 KiB (+216 KiB), count=9255 (+1541), average=123 B/usr/local/Cellar/python@3.10/3.10.13/Frameworks/Python.framework/Versions/3.10/lib/python3.10/functools.py:58: size=239 KiB (+49.7 KiB), count=3550 (+736), average=69 B/Users/jogireddy/PycharmProjects/flaskProject/.venv/lib/python3.10/site-packages/openai/_resource.py:17: size=63.8 KiB (+13.8 KiB), count=888 (+192), average=74 B/Users/jogireddy/PycharmProjects/flaskProject/.venv/lib/python3.10/site-packages/openai/_resource.py:22: size=36.9 KiB (+6144 B), count=502 (+96), average=75 B/Users/jogireddy/PycharmProjects/flaskProject/.venv/lib/python3.10/site-packages/openai/_resource.py:23: size=28.5 KiB (+6144 B), count=456 (+96), average=64 B/Users/jogireddy/PycharmProjects/flaskProject/.venv/lib/python3.10/site-packages/openai/_resource.py:21: size=28.5 KiB (+6144 B), count=456 (+96), average=64 B/Users/jogireddy/PycharmProjects/flaskProject/.venv/lib/python3.10/site-packages/openai/_resource.py:20: size=28.5 KiB (+6144 B), count=456 (+96), average=64 B/Users/jogireddy/PycharmProjects/flaskProject/.venv/lib/python3.10/site-packages/openai/_resource.py:19: size=28.5 KiB (+6144 B), count=456 (+96), average=64 B/Users/jogireddy/PycharmProjects/flaskProject/.venv/lib/python3.10/site-packages/openai/_resource.py:18: size=28.5 KiB (+6144 B), count=456 (+96), average=64 B/Users/jogireddy/PycharmProjects/flaskProject/.venv/lib/python3.10/site-packages/pydantic/_internal/_core_utils.py:200: size=3952 B (+3784 B), count=21 (+20), average=188 B

Found this function at openai/_response.py:238
`
@functools.wraps(func)
def wrapped(*args: P.args, kwargs: P.kwargs) -> APIResponse[R]:
extra_headers = {(cast(Any, kwargs.get("extra_headers")) or {})}
extra_headers[RAW_RESPONSE_HEADER] = "true"
kwargs["extra_headers"] = extra_headers

return cast(APIResponse[R], func(*args, **kwargs))

return wrapped
`

Thanks, we'll look into this!

Please upgrade to the latest version, you're quite far behind and we've fixed a bunch of memory leak issues since then.

I upgraded openai version to 1.23.5
and still I can see some issues with the same function just it got moved to different place.
/Users/jogireddy/PycharmProjects/flaskProject/venv/lib/python3.9/site-packages/openai/_legacy_response.py:347: size=1389 KiB (+1389 KiB), count=12109 (+12109), average=117 B/Users/jogireddy/PycharmProjects/flaskProject/venv/lib/python3.9/site-packages/openai/_response.py:674: size=1278 KiB (+1278 KiB), count=11239 (+11239), average=116 B/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/linecache.py:137: size=1232 KiB (+1232 KiB), count=12014 (+12014), average=105 B/Users/jogireddy/PycharmProjects/flaskProject/venv/lib/python3.9/site-packages/httpx/_content.py:175: size=868 KiB (+868 KiB), count=83 (+83), average=10.5 KiB/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/functools.py:58: size=769 KiB (+769 KiB), count=12308 (+12308), average=64 B/Users/jogireddy/PycharmProjects/flaskProject/venv/lib/python3.9/site-packages/openai/_legacy_response.py:330: size=726 KiB (+726 KiB), count=6332 (+6332), average=117 B/Users/jogireddy/PycharmProjects/flaskProject/venv/lib/python3.9/site-packages/openai/_response.py:653: size=676 KiB (+676 KiB), count=5931 (+5931), average=117 B/Users/jogireddy/PycharmProjects/flaskProject/venv/lib/python3.9/site-packages/openai/_resource.py:34: size=358 KiB (+358 KiB), count=5088 (+5088), average=72 B/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/abc.py:102: size=261 KiB (+261 KiB), count=2992 (+2992), average=89 B/Users/jogireddy/PycharmProjects/flaskProject/venv/lib/python3.9/site-packages/httpx/_models.py:82: size=249 KiB (+249 KiB), count=3736 (+3736), average=68 B


Reference invoice changed to array

Issue #5 fix.

Thanks for the fix. I think it's time for me to start working on tests :)


can't find variable "HeaderStyle"

When i use the 'Grid' element, it return me 'can't find variable "HeaderStyle"'. Anyone got the same issue?

your one from module 'react-native-uikit' and your own lib folder is not the same...............

hmm will have a look.


failed to activate the status-bar package when atom-clock enabled

open Atom and and this error message happens

Atom Version: 1.12.8
Electron Version: 1.3.13
System: Mac OS X 10.14
Thrown From: status-bar package, v1.4.1
Stack Trace
Failed to activate the status-bar package
At atom.tooltips.findTooltips is not a function

TypeError: atom.tooltips.findTooltips is not a function
    at AtomClockView.setTooltip (/Users/liuran/.atom/packages/atom-clock/lib/atom-clock-view.js:142:21)
    at AtomClockView.initialize (/Users/liuran/.atom/packages/atom-clock/lib/atom-clock-view.js:19:10)
    at AtomClockView.start (/Users/liuran/.atom/packages/atom-clock/lib/atom-clock-view.js:14:10)
    at Object.consumeStatusBar (/Users/liuran/.atom/packages/atom-clock/lib/atom-clock.js:63:24)
    at Provider.module.exports.Provider.provide (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/service-hub/lib/provider.js:29:52)
    at ServiceHub.module.exports.ServiceHub.provide (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/service-hub/lib/service-hub.js:30:20)
    at Package.module.exports.Package.activateServices (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:362:71)
    at Package.module.exports.Package.activateNow (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:187:16)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/package.js:157:32
    at Package.module.exports.Package.measure (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:92:15)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/package.js:150:26
    at Package.module.exports.Package.activate (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:147:34)
    at PackageManager.module.exports.PackageManager.activatePackage (/Applications/Atom.app/Contents/Resources/app.asar/src/package-manager.js:550:34)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/package-manager.js:531:29
    at Config.module.exports.Config.transactAsync (/Applications/Atom.app/Contents/Resources/app.asar/src/config.js:337:18)
    at PackageManager.module.exports.PackageManager.activatePackages (/Applications/Atom.app/Contents/Resources/app.asar/src/package-manager.js:526:19)
    at PackageManager.module.exports.PackageManager.activate (/Applications/Atom.app/Contents/Resources/app.asar/src/package-manager.js:508:46)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/atom-environment.js:792:28

I just started to get the same error.
I had it installed from about 2 years ago, just accidentally changed a file in the package and couldn't get it back.  Uninstalled it and reinstalled it and just got the same message.  Newer Atom version different OS, same issues.
Atom Version: .1.34.0
Electron Version: 2.0.16
System: Windows 10 x64
Chrome: 61.0.3163.100
Node: 8.9.3

Thank you for reporting the issue guys. Is it still present on the newest Atom version (1.35.1)?

It does not seem to be an issue in the newest version of Atom, (1.35.1) on Win 10 x64 for me.
Thank you.

Thank you for the follow up!


Blockquotes are displayed within a box instead of indented block of text

Using the following snippet:
**NO realizar**
> Urocultivo en primer evento
> Urocultivo de control post-tratamiento

Using other interpreters, is shown like this:

or (live) here like:
`
NO realizar

Urocultivo en primer evento
Urocultivo de control post-tratamiento
`

but with flutter_markdown is redered like the image below:

Is this issue already known? Can be fixed with styles?

I can reproduce this behaviour on stable and master with flutter_markdown: ^0.6.6.
Although it can be fixed with styling as stated by the author, I believe the default behaviour is expected by most users.
Labeling for visibility.

complete code sample
import 'package:flutter/material.dart';
import 'package:flutter_markdown/flutter_markdown.dart';

String _markdownData = """
**NO realizar**
> Urocultivo en primer evento
> Urocultivo de control post-tratamiento
""";

void main() {
  runApp(
    MaterialApp(
      title: "Markdown Demo",
      home: MarkdownApp(),
    ),
  );
}

class MarkdownApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('Simple Markdown'),
      ),
      body: Markdown(
        data: _markdownData,
      ),
    );
  }
}

 flutter doctor -v
[✓] Flutter (Channel master, 2.6.0-1.0.pre.169, on macOS 11.5.1 20G80 darwin-arm, locale en-GB)
    • Flutter version 2.6.0-1.0.pre.169 at /Users/nexus/dev/sdks/flutters
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 3dc413b963 (6 hours ago), 2021-09-02 22:56:07 -0700
    • Engine revision c02b815416
    • Dart version 2.15.0 (build 2.15.0-69.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.5.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] VS Code (version 1.59.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.26.0

[✓] Connected device (3 available)
    • iPhone 12 (mobile) • F2592D43-5FE3-46BB-8A2B-A38BA468DCE9 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • macOS (desktop)    • macos                                • darwin-arm64   • macOS 11.5.1 20G80 darwin-arm
    • Chrome (web)       • chrome                               • web-javascript • Google Chrome 93.0.4577.63

• No issues found!

[✓] Flutter (Channel stable, 2.2.3, on macOS 11.5.1 20G80 darwin-arm, locale en-GB)
    • Flutter version 2.2.3 at /Users/nexus/dev/sdks/flutter
    • Framework revision f4abaa0735 (9 weeks ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5.1, Build version 12E507
    • CocoaPods version 1.11.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] VS Code (version 1.59.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.26.0

[✓] Connected device (3 available)
    • iPhone 12 (mobile) • F2592D43-5FE3-46BB-8A2B-A38BA468DCE9 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • macOS (desktop)    • macos                                • darwin-arm64   • macOS 11.5.1 20G80 darwin-arm
    • Chrome (web)       • chrome                               • web-javascript • Google Chrome 93.0.4577.63

• No issues found!

I actually like the default rendering behavior ;)
If it's changed I would love to have a small snippet that shows how to get the old behavior back.


Lint Rebirth Prep

Original PR that needed to be reverted on master: https://github.com/bolt-design-system/bolt/pull/572
For both config-stylelint and config-eslint this:

Renames index.js to ideal.js
Creates new index.js that extends ideal.js
Adds every rule that is causing warnings to index.js and disables it
Linting is now passing (b/c all the rules are disabled)
A comment in index.js contains the process to get the code base back to the ideal:

Disabling all rules with errors from `ideal.js` (which represents the ideal state)
Process:
1. Delete a line from `rules`
2. Lint
3. Fix
4. PR
When `rules` is empty, we rename `ideal.js` to `index.js` and delete this file.

I've also added npm run lint to Travis builds and npm run lint && npm run test to the git prepush hook.

Already merged in elsewhere


add train diffusion scripts

I prepared a script for training the diffusion models.
I did some quick tests and it seems to work, but haven't tested much. The code is not the cleanest though.

Hey @eloimoliner! I started merging the functionality on a separate branch, you can keep track of the changes in https://github.com/01tot10/neural-tape-modeling/pull/2. I'll still give it some final eyeballing, but I think things should be good to go! Thanks again!


Make the "Push to FHIR" part of the DeIdentify Custom Processor optional

The purpose of this task is to make the Push Deidentified resource to FHIR optional via configuration in the DeIdentify NiFi Custom Processor. There could be a boolean checkmark that indicates whether to do that or not, when the checkmark is unselected the validation for the FHIR server would be skipped.

The use of a Nifi Custom Processor for the Healh Patterns Deid wrapper has been migrated over to "deid-prep", a service that provides the same function but can be deployed independent of Nifi.  With this migration, support was added to allow toggling whether the request will get stored in a target FHIR server or not, effectively resolving this issue.


When creating a new file on treemacs after a rename of a file you can't create the new one

when i rename a file with its old name like 'foo-old' and then i go to create a new file with 'c f' with the name "foo" it keeps selecting the "foo-old" and saying that file already exists
What did you expect to happen?
to be able to create the new file with a name similar to the one i just renamed
What actually happened?
keeps selecting the old file name and trying to create that old file name
Additional details:

Screencast: https://files.gitter.im/5cdfc9cdd73408ce4fc0a585/iwbF/Sep-14-2020-09-19-51.gif

Steps to reproduce:

Open Treemacs for some project
Select a file and press R to rename it to foo-old
Press c f to create a new file
Try to create a file named "foo"

CC: @alexander-miller

I think this is because RET is accepting the suggestion you are presented with.  Try typing what you want your new file to be called, then press C-M j  -- usually control + alt + j

yep! that works. 👍   But I do think maybe there should be another selection for the new file you want to create?

It's just how ivy works by default I think.  Look at https://oremacs.com/swiper/ and search for ivy-immediate-done

oh wow, and it says it right there about matching an existing file


Android 29 running error

2021-01-22 14:43:22.956 20547-20736/com.xiangchuang.myapplication E/UsbManager: exception in UsbManager.openDevice     java.lang.SecurityException: User has not given 10227/com.xiangchuang.myapplication permission to access device /dev/bus/usb/002/002         at android.os.Parcel.createException(Parcel.java:2071)         at android.os.Parcel.readException(Parcel.java:2039)         at android.os.Parcel.readException(Parcel.java:1987)         at android.hardware.usb.IUsbManager$Stub$Proxy.openDevice(IUsbManager.java:739)         at android.hardware.usb.UsbManager.openDevice(UsbManager.java:559)         at com.intel.realsense.librealsense.DeviceWatcher.addDevice(DeviceWatcher.java:115)         at com.intel.realsense.librealsense.DeviceWatcher.invalidateDevices(DeviceWatcher.java:78)         at com.intel.realsense.librealsense.DeviceWatcher.access$000(DeviceWatcher.java:16)         at com.intel.realsense.librealsense.DeviceWatcher$1.onDeviceAttach(DeviceWatcher.java:44)         at com.intel.realsense.librealsense.Enumerator.notifyOnAttach(Enumerator.java:102)         at com.intel.realsense.librealsense.Enumerator.access$200(Enumerator.java:18)         at com.intel.realsense.librealsense.Enumerator$MessagesHandler.handleMessage(Enumerator.java:144)         at android.os.Handler.dispatchMessage(Handler.java:107)         at android.os.Looper.loop(Looper.java:213)         at android.os.HandlerThread.run(HandlerThread.java:67)

Hi @whosmyqueen   There was a case involving the error exception in UsbManager.openDevice java.lang.SecurityException: User has not given ... permission to access device in December 2019.
https://github.com/IntelRealSense/librealsense/issues/5388
The RealSense user who created the case reported that updating their camera's firmware driver resolved the problem.
https://github.com/IntelRealSense/librealsense/issues/5388#issuecomment-623078025

Hi @whosmyqueen   There was a case involving the error exception in UsbManager.openDevice java.lang.SecurityException: User has not given ... permission to access device in December 2019.
https://github.com/IntelRealSense/librealsense/issues/5388
The RealSense user who created the case reported that updating their camera's firmware driver resolved the problem.
https://github.com/IntelRealSense/librealsense/issues/5388#issuecomment-623078025

@MartyG-RealSense It doesn't solve my problem. My project is Android 29. Your SDK can only run on Android 27.

@MartyG-RealSense It doesn't solve my problem. My project is Android 29. Your SDK can only run on Android 27.

Setting TargetSDKVersion to '27' is a method used to fix problems with using Android 10 and 11 devices with the RealSense Android wrapper.
There was a RealSense user in November 2020 who was successfully using the wrapper with Android 29 on some Android devices but not others.  This fits past patterns where problems may occur with using the wrapper on Android devices of a certain model but it works on other devices.
https://github.com/IntelRealSense/librealsense/issues/7691#issuecomment-720876395
The link below has an example of problems occurring with particular models:
https://github.com/IntelRealSense/librealsense/issues/6546#issuecomment-643180328

Setting TargetSDKVersion to '27' is a method used to fix problems with using Android 10 and 11 devices with the RealSense Android wrapper.
There was a RealSense user in November 2020 who was successfully using the wrapper with Android 29 on some Android devices but not others.  This fits past patterns where problems may occur with using the wrapper on Android devices of a certain model but it works on other devices.
https://github.com/IntelRealSense/librealsense/issues/7691#issuecomment-720876395
The link below has an example of problems occurring with particular models:
https://github.com/IntelRealSense/librealsense/issues/6546#issuecomment-643180328

Hi @whosmyqueen  Do you require further assistance with this case, please?   Thanks!

Hi @whosmyqueen  Do you require further assistance with this case, please?   Thanks!

Case closed due to no further comments received.


edit pass: grpc-batch1-jd

@shirhatti   @mairaw - Please review the copy edit to these articles, and let me know if I’ve changed the technical meaning anywhere. This is for Azure DevOps work request 1646344.
Contributes to #15889

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

Thanks for the feedback, everyone. Fixed everything that appeared to be for me.

@Ja-Dunn you can batch suggestions into a single commit using the Files changed tab.

@mairaw - Thanks for the tip on batching commits! Looks like I still didn't quite get it, although I did go to the Files changed tab to try to do so. Anyway, okay, I believe I captured all the feedback.
@shirhatti - Appreciate the close reading. Re: "first class" description of support, I was originally reading that as a marketing claim, and we typically avoid that in technical docs. That said, I understood the nuance you were trying to get at in your comment. I think "built-in" is a great alternative phrasing.

LGTM. I'll let @shirhatti do the approval before merging.


fix(flow): remove src folder from npm bundle

Fix for https://github.com/popperjs/popper-core/issues/1064

In this case flow types will stop working
https://unpkg.com/<EMAIL_ADDRESS>Flow only looks for .js.flow near "main" entry point when importing like this
import { createPopper } from '@popperjs/core';

Right! I suppose I could point to the lib folder though

I updated Rollup, now the flow file points to lib
// @flow

export * from '../../lib/index.js'


fix connection phase ignoring server errors

Summary
The binding doesn't propagate errors received from the server during the connection phase. If an error response is sent after a client handshake, the error is logged and the client continues to execute; disregarding the error.
Example flow:
13:19:08 - Debug: C->S: Client 1: ClientHandshake len: 50
13:19:08 - Debug: S->C: Client 1: ErrorResponse len: 348
13:19:08 - Error: Got error level: Error Message: ...
(client continues to perform execution)
13:19:08 - Debug: C->S: Client 1: Parse len: 120
...

This PR adds a check for an error in the connection phase, propagating it as an exception if one is received.

Closing as this behavior is implemented in #42


Question about Planner part

Thanks for you great work! But there is a little question I want to know about. How does the planner trying to affect the generation part? That is, after these labels have been predicted by Planner, how would they been processed and sent to the main transformer part? I'm trying to find related parts in your code but failed. Patient response would be highly appreciated.

Hi,
These observations are converted into embeddings and sent to a Transformer.
Here is how it works:

The Dataset receives observation data and converts them into an observation graph (check here).
The graph is represented as node_ids (check here), and a Transformer encodes this graph with the visual features (check here).

Best,
Ethan

Thanks so much for your quick response, I also want to know about how to get id2tagpos.csv for mimic_cxr, thanks!

id2tagpos.csv is the observation position data, i.e., the observation plan. You can generate the data by:
Prepare:

CheXbert labels (i.e., id2tag.csv)
MIMIC-CXR annotations (annotation.json)

Run:
python ./src/plan_extraction.py

I got it, thank you!


[Feature Request]: Setup GH Action For PRs

Feature description
We should look at setting up an action or two to help with the following flows (maybe more scenarios?):

When a PR references a ticket, and it's not going into main it should still link (not sure if we can do this easily)
When a PR merges into a feature branch -- the ticket is closed (this should be possible with actions)
When a ticket is associated to a pull request, it should move into In Review so it properly shows up on our board in the PR tab

Describe alternatives you've considered
No response
Anything else?
https://docs.github.com/en/issues/planning-and-tracking-with-projects/automating-your-project/automating-projects-using-actions

act is a really useful utility for testing gh actions locally.

Saw a comment https://github.com/orgs/community/discussions/13240#discussioncomment-5440154, maybe this is a promising way to automatically close the issue when PR is targeting the feature branch.

Migrated to:  https://issues.redhat.com/browse/RHOAIENG-2523


Feature: Count sigops for Transaction

I copied over the sigop counting logic from Bitcoin Core, but I made a few adjustments.

I removed 2 consensus flags that checked for P2SH and SegWit activation. This code assumes both are activated. If we were to include that, what would be a good way to go about it? (ie. If I run this method on a transaction from the 1000th block and it just so happened to have a P2SH-like input, Bitcoin Core wouldn't accidentally count those sigops because the consensus flag will stop them from running the P2SH logic. Same goes for SegWit)
Since there's no guarantee that we have an index from which we can get the prevout scripts, I made it into a generic closure that looks up the prevout script for us. If the caller doesn't provide it, We can only count sigops directly in the scriptSig and scriptPubkey (no P2SH or SegWit).

Another pinning issue appeared?

Weird... I verified the 1.48.0 test suite passes locally. Maybe it was a fluke (needs a re-run)

re-kicked

Another pinning issue appeared?
yeah I opened an issue for this a few days ago: https://github.com/rust-bitcoin/rust-bitcoin/issues/2071
syn v2.0.33 requires rustc +1.56

Fix 1.48 CI error

@junderw nice!  How did you know what dep to update?

@junderw is this just a copy of the commit I made here: https://github.com/rust-bitcoin/rust-bitcoin/pull/2074/commits/d34df0c705670d154a915407e19f7e3e3aa57ef2?

@junderw is this just a copy of the commit I made here: d34df0c?

Yep. I'm going to remove it once a fix is merged into master.

@junderw ah ok.  Feel free to comment on https://github.com/rust-bitcoin/rust-bitcoin/pull/2074

concept ACK. I didn't review the implementation beyond checking the "pushonly" stuff since I happened to remember how weird that computation is. We are veering into consensus-code territory that is full of nasty surprises, but OTOH getting accurate sigop counts is a pretty important thing for Script authors so I think we should try to do it.

Could you fold your changes into the original commits? It's hard to review code that changes the same lines across multiple commits, and I think this PR is small enough that it won't throw off in-progress reviewers.

I found a bug in Script count_sigops and count_sigops_legacy.
f1220af34ed325cbe56fd76527acfb830b38c8a7edfe524165de16098a8d0f44
This Coinbase transaction has an early ending scriptSig that was unable to be parsed.
Upon looking at Bitcoin Core's logic:
if (!GetOp(pc, opcode)) break;
Then looking at the GetOp code, it returns false when it runs into an early end of script.
So Bitcoin Core returns early with the accumulated n when it hits an error. I have changed the logic to fit that.

I rebased the bugfix into a separate commit and made a new PR. This PR now depends on #2075

Thanks for the review. I have accepted all of your fixes with a few minor tweaks (diff locally to see what I fixed, it was mostly comments and whatnot.

Done and done!

Thanks for sticking with it!

Writing tests... This generic S argument makes it really hard to write.
tx.total_sigop_cost(None) ends in a specify with None::<T> error...

We should probably split the public functions into 2 separate ones. One with the closure and one without.

Thoughts?

I think we should use saturating add/mul everywhere in this PR. Apologies for not realising that yesterday.

@tcharding Is there a compelling reason to use saturating add/mul instead of checked add/mul here?

I think we should use saturating add/mul everywhere in this PR. Apologies for not realising that yesterday.

@tcharding Is there a compelling reason to use saturating add/mul instead of checked add/mul here?

In order to go over the 32 bit threshold of 4,294,967,295, the fastest way would be to have as many bare-multi-sigs in the outputs as possible.
1-of-1 multisig (smallest to fit as many as possible in the blocksize) is 46 bytes per output. Since it's non-segwit, we can only fit up to about 1 MB in a single transaction (if a miner mines it for us) so we could fit around ~ 21730 multisigs in there.
21730 * 20 * 4 is 1738400
We need about 2470 times that to hit the 32 bit threshold.
So until we get 2.4 GB blocksizes, we won't have to worry at all. If we ever hit that, that is just the worst-case-scenario of an obviously malicious transaction on a 32 bit machine.
By the time (if at all) we get 2.4 GB blocks, we might not even support 32 bit systems anymore.
saturating will ensure that there is no wrapping while also keeping the ergonomics of the API in a better state. (Checking Options for everything when we only have 1 MB blocks is bad ergonomics)

Maybe we could just mention this monstrosity in the docs, "oh yeah when you use None, write this hideous line of code."

@junderw I've run into this issue in a few contexts (taking a generic option). One solution is to provide a correctly-typed None constant with a name like NO_OUTPOINT_MAP. Another is to use a custom enum rather than Option (though to hold a closure it might still need to be generic which means you may have the same issue..).

@tcharding Is there a compelling reason to use saturating add/mul instead of checked add/mul here?

Because checked add/mul will require everything returns a Result even though the error variant is practically inaccessible.

@junderw I've run into this issue in a few contexts (taking a generic option). One solution is to provide a correctly-typed None constant with a name like NO_OUTPOINT_MAP. Another is to use a custom enum rather than Option (though to hold a closure it might still need to be generic which means you may have the same issue..).

What about taking Option<&mut dyn FnMut(&OutPoint) -> Option<TxOut> + 'a>? It adds a &mut to the closure unnecessarily, but it makes the None case nicer.

Using dyn and requiring Fn and only taking & could be another answer... it would cut off any users that somehow need to utilize mutable state in their lookup closure...
But tbh, they can use interior mutability, and 99% of use cases will likely just be a function with no state anyways or a non-capturing closure... or a capturing closure that only reads a HashMap etc. (which doesn't need mutable state anyways)...
This would lower the requirements for ownership of the actual closure itself, but increase the requirements by restricting what can be accessed inside the closure. Since most use cases using a state-full closure would just be temporary values anyways, not bound to a variable... so the lowering of the requirements on the closure itself seems unimportant.
Option<&'a mut (dyn FnMut(&OutPoint) -> Option<TxOut> + 'b)> where 'b: 'a feels like an ok idea.
Let me know if there's some trap I'm not thinking of here.

The more I think about it, the more I am warming up to the NO_OUTPOINT_MAP constant and keeping it generic idea.
I'm going to stop thinking about this now until I get some more feedback, my brain is too excited here. lol

I've never used dyn with Fn before so it's hard for me to say what the merits are, and what compiler weirdness we're likely to hit :).

Yeah, getting rid of the Option was the best option tbh.

Sorry for the constant dismissal of ACKs :sweat_smile:, documentation fix.

FTR I didn't put much thought into the spent parameter stuff.

@junderw looks like Github hid my comment as "outdated" which talks about the lack of Taproot support. I think we should change count_sigops_internal to also count CHECKSIGADD, which I believe is all we need to add Taproot functionality. (In Taproot you compare the sigop count to some weight*50 formula rather than a fixed maximum, but the idea of counting sigops against a limit is the same. And I believe the limit check isn't actually implemented in this PR, so there's no reason to not support Taproot.)
Do you want to update this PR or should I merge this and we'll do it in a followup?

I replied in the thread (you can click to unhide and read it)

This function is a Transaction level function, but Taproot scripts don't count toward the transaction's sigop count.
We could add a taproot variant to the Script level count function, though. Since the only time sigops are tracked during taproot verification is when running the tapscript, the script interpreter holds a running budget that it ticks down every time it performs SIGADD etc.
We could also then build upon that and add a Witness method that would calculate the budget (which is serialized witness stack size + 50) and see if the sigops of the tapscript go over that.

I think this is a separate PR.

Oof, I understand, you're right. I forgot that the pre-Taproot and Taproot limits are compared against different things (blocks and transactions vs individual txins) so you can't just aggregate them like this.


Extension stopped converting: 'currency' is not defined in the unit registry

Hey,
The extension stopped working a few days ago. I've tried multiple conversions but I get no result with any of them.
In the console I have this error: pint.errors.UndefinedUnitError: 'currency' is not defined in the unit registry
Thank you!

Hi there!
Have the same issue appeared after I installed the extension few days ago.
@krillin666 have you found any solution?

Same problem here. Not only corrency but others units to. Like celsius to fahrenheit and others.

Same here. If it helps, it happens using "internal" for me, not fixer.io.
Here's the python traceback:
Traceback (most recent call last):
  File "/home/max/.local/share/albert/org.albert.extension.python/modules/calculate-anything/calculate_anything/query/handlers/multi_handler.py", line 55, in _handle
    result = handler.handle(query)
  File "/home/max/.local/share/albert/org.albert.extension.python/modules/calculate-anything/calculate_anything/query/handlers/base.py", line 23, in _wrapper
    return func(self, query, *args, **kwargs)
  File "/home/max/.local/share/albert/org.albert.extension.python/modules/calculate-anything/calculate_anything/query/handlers/base.py", line 57, in handle
    return self.handle_raw(query, *args, **kwargs)
  File "/home/max/.local/share/albert/org.albert.extension.python/modules/calculate-anything/calculate_anything/query/handlers/units.py", line 240, in handle_raw
    with ureg.context('currency'):
  File "/usr/lib/python3.10/contextlib.py", line 135, in __enter__
    return next(self.gen)
  File "/home/max/.local/lib/python3.10/site-packages/pint/facets/context/registry.py", line 301, in context
    self.enable_contexts(*names, **kwargs)
  File "/home/max/.local/lib/python3.10/site-packages/pint/facets/context/registry.py", line 242, in enable_contexts
    self._switch_context_cache_and_units()
  File "/home/max/.local/lib/python3.10/site-packages/pint/facets/context/registry.py", line 151, in _switch_context_cache_and_units
    self._redefine(definition)
  File "/home/max/.local/lib/python3.10/site-packages/pint/facets/context/registry.py", line 175, in _redefine
    dims_old = self._get_dimensionality(basedef.reference)
  File "/home/max/.local/lib/python3.10/site-packages/pint/facets/plain/registry.py", line 660, in _get_dimensionality
    self._get_dimensionality_recurse(input_units, 1, accumulator)
  File "/home/max/.local/lib/python3.10/site-packages/pint/facets/plain/registry.py", line 681, in _get_dimensionality_recurse
    reg = self._units[self.get_name(key)]
  File "/home/max/.local/lib/python3.10/site-packages/pint/facets/plain/registry.py", line 588, in get_name
    raise UndefinedUnitError(name_or_alias)
pint.errors.UndefinedUnitError: 'currency' is not defined in the unit registry

Same here 😢

I have got the same problem, unfortunately.

@maxime-rey so do you mean it works fine with the flex.io only?

It doesn't work with internal or fixer.io. But not only currency is buggy the measures and other conversions doesn't work to

Is there any update on this? I really miss using this extension

Can you let me know if #48 fixed this?

@tchar Conversion works now. Thank you so much for fixing this!


add topic for Azure Managed Identity

second attempt, let me know if anything else is needed

Thanks @metskem!