Covalent Docking Broken

Environment Information
Open Babel version: 3.1.0
Operating system and version: windows 11 running ubuntu in WSL running Anaconda3
Expected Behavior
I am running the script prepareCovalent.py to prepare a ligand for covalent docking in AutoDock Vina. I have tried with my own files and the tutorial files provided. The script imports pybel.py, when it attempts to perform _formatstodict it crashes. Instead of the expected output of a .pdb file with the ligand properly oriented in 3D space for covalent docking I get an error message.
Actual Behavior
This is the error message I get:
(vina) gorrie06@DMGLaptop:~/docking/covalent$ python ~/docking/covalent/adcovalent/prepareCovalent.py --ligand /3upo_test/ligand.mol2 --ligindices 1,2 --receptor /3upo_test/3upo_protein.pdb --residue B:SER222 --outputfile ligcov.pdb
Traceback (most recent call last):
File "/home/gorrie06/docking/covalent/adcovalent/prepareCovalent.py", line 36, in
import pybel
File "/home/gorrie06/anaconda3/envs/vina/lib/python2.7/site-packages/pybel.py", line 89, in
informats = _formatstodict(_obconv.GetSupportedInputFormat())
File "/home/gorrie06/anaconda3/envs/vina/lib/python2.7/site-packages/pybel.py", line 68, in _formatstodict
broken = [(x, y.strip()) for x, y in broken]
ValueError: need more than 1 value to unpack
Steps to Reproduce
This file contains the prepareCovalent.py script and the tutorial files for covalent docking
adCovalentDockResidue_v1.2.tar.gz
This is the prepareCovalent script
prepareCovalent.txt
This file contains pybel.py
original-pybel.txt
If you run the prepareCovalent script with the following inputs:
python  ~/local/adcovalent/prepareCovalent.py --ligand ligand.mol2
--ligindices 1,2
--receptor 3upo_protein.pdb
--residue B:SER222
--outputfile ligcovalent.pdb
you get the error message

Hi @gorrie06
I don't think we can use this to prepare covalent docking with vina 1.2.x at the moment. But this should at least work for autodock4 as in the tutorial (README from https://autodock.scripps.edu/resources/covalent-docking/).
The problem with import pybel might have something to do with openbabel version or your operating system. prepareCovalent.py expects openbabel 2.x. The usage of modules has changed slightly:
https://open-babel.readthedocs.io/en/latest/UseTheLibrary/migration.html#python-module
For the covalent docking tutorial with autodock4, you can try running prepareCovalent.py with $MGLROOT/bin/pythonsh which has openbabel 2.4.1.
Consider other ways to do covalent/reactive docking with the AutoDock programs:
Docking a Covalently Bound Ligand with ADFR
https://ccsb.scripps.edu/adfr/tutorial-covalent/
Reactive Docking with AD-GPU, which might need ligand preparation with Meeko
https://github.com/forlilab/Meeko?tab=readme-ov-file#reactive-docking
If you prefer openbabel 3.0, try changing the way you import python modules and keep an eye on other necessary updates/behavior changes. I'm also watching openbabel and saw you posted there

See also: https://github.com/forlilab/Meeko?tab=readme-ov-file#docking-covalent-ligands-as-flexible-sidechains

Great! Thank you

So the docking covalent ligands as flexible side chains requires Prody which is unavailable. The help file warns you about this. If you try to run with correct inputs you get an error that Prody is required and unavailable.
The ADFR tutorial doesn't make sense, the example shows the covalent bond as being between the alpha and beta carbon. This is pointless, the covalent bond should be between the ligand and the receptor, not between two carbons on a sidechain. Secondly, it seems to require that you have ligand atoms modeled into your receptor structure, although this is not totally clear because of what I said previously.
I'm now going to attempt to perform covalent docking with AD-GPU and update here. It may not be currently possible to covalently dock using this software suite.

Hi @gorrie06
The ADFR tutorial doesn't make sense, the example shows the covalent bond as being between the alpha and beta carbon. This is pointless, the covalent bond should be between the ligand and the receptor, not between two carbons on a sidechain.
Actually you can choose where to place the covalent bond. Placing it near CA allows additional flexibility of the residue sidechains in docking.
Secondly, it seems to require that you have ligand atoms modeled into your receptor structure
The receptor has to contain the covalent bond atoms but should not include the ligand.
This repository is not really for ADFR. But if it's something you would be interested in using (considering the method, scoring function, etc.) feel free to reach out or post on the list server to connect with other users :)

You could install Prody

Added the following to Meeko's README
Prody is required for preparing ligands as flexible sidechains. Often, pip install prody suffices to install Prody in the currently active virtual environment (e.g., conda). For more detailed installation instructions visit http://prody.csb.pitt.edu/downloads/.


Emojis won't load

Hi. First of all, great app. I've tried many (I think all of them), and this is the one, so thanks for developing this.
This is the issue, though:

The other two tabs load well. I've updated to last version and reinstalled it etc but still happening.

@nmaxcom
New version fixed it.


feat: add theme switch intervals

Users can now specify what interval chameleon should switch to the next theme

Needs a review: I wrote and ran the tests for this feature and they passed but I couldn't test it out manually since I'll have to wait for an hour/hours for the next theme to switch.

Thanks @danBamikiya !

You're welcome @timdeschryver !


Technical: Publish / Subscribe

A plugin should be able to publish information (like sensor information).
Also external devices should be able to publish.
Use MQTT.
x SP
Tasks:

[ ] task

only primitive data can be sent
the data is passed by value not by reference


Support multiple languages for names of months and week days.

According to Vlad's comment:
Support multiple languages for names of months and week_days is a killer feature. I want it as well. It's Feature#3

Also look forward to this feature. Thanks


Add functional test coverage for ANCM shutdown

Following up to https://github.com/aspnet/IISIntegration/issues/385. Once the ANCM changes for https://github.com/aspnet/AspNetCoreModule/issues/115 are available, we should add a functional test for ANCM shutdown.

There are other bugs tracking this.


Add variant without bundled definitions

Currently, one can choose between the "full" emoji-set by using index.js and the light emoji-set by using light.js.
In case of providing custom definitions, you might want to keep the bundle-size low by not importing the unused light.json or full.json data-sets.
I therefore propose the following:
Add a new file "bare.js" (or another name) that is similar to index.js or light.js but does not import the json data-sets.

IMO it's not a big deal to copypaste light.js & customize, but if anyone wish to do PR, it will be accepted.


TCHAR* <-> UTF8/ANSI macros used incorrectly; causing strings to go out of scope

Please see this commit on our own fork.
https://github.com/recogni/UnrealEnginePython/commit/a1431aaf5bfbf641072b32f9a4e572ba3900045f
If you have questions, ping me. You can check the source code for the UE engine (StringConv.h) which describes the short-lived-ness of these macros.  Feel free to cherry-pick or re-implement.
If I am super bored later, Ill send you a patch w/ our custom stuff stripped out.

Thanks a lot, the explanation is super clear, i will fix it soon.

Also, I know you mentioned before about implementing the RunFile variants with multiple args.  I took a shot at this and am using it to some decent success in our custom application.  Feel free to pull from here if you want:
https://github.com/recogni/UnrealEnginePython/commit/be9c8a84910601504f9c54371ee0dd80fe668cc0
Some things I am planning on adding - once the py.exec (or in my case py.exec_args) is done, it would be nice if we could return the result of the exec (if any) to the caller. Maybe its overkill, but just thoughts :)


Allow setup step to be called multiple times

I have the problem from CI that I don't know if pm2 deploy has already been called with setup. Calling it a second time currently results in an error when it tries to clone the repo:
Local PM2 version: 4.2.0

  ○ hook pre-setup
  ○ running setup
  ○ cloning git@deployment:pke/xstate.io.git
  ○ full fetch
fatal: destination path '/var/www/restlr.io/source' already exists and is not an empty directory.

  failed to clone

Deploy failed

Would it be possible to gracefully ignore this if the directory already exists?

just make sure that dest is not already there


[incubator] Add virtualcluster project owners

The incubator project has different owners.

/assign tashimi

/lgtm
/approve


Suggestion: add "files" to the package.json template

Library Version: code at b48b68a

Related: https://twitter.com/bitandbang/status/1088787376967700480

Hi, is someone working on this?

Not at all, go ahead @alemagio

:tada: This issue has been resolved in version 2.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


3D tracking challenge submission

My aim is to evaluate my 3D tracker. I know that I can inference my tracker through validation set and test set in order to submit my predictions. At this point, I could not see any data type which indicates the segment of the dataset in the proto file. In sum, how Waymo determine which segment I used? Maybe timestamp info is enough. Can I use any segment I wish?

These two fields uniquely identify a frame.

@peisun1115 thanks for your reply. I have two questions about 3D tracking validation. I'm trying to use create_prediction_file_example.py in order to generate a prediction file, and it requires the object type but I guess it's redundant for 3D tracking tasks. Am I right? My second question is about submission.txtpb file. It contains those instance:number_past_frames_exclude_current,number_future_frames_exclude_current. I couldn't understand what those parameters are. If you express them, I'll be very appreciated


BUFFER_DEF / BUFFER_THREAD_UNSAFE: performance issue

When defining a buffer using BUFFER_DEF with the policy BUFFER_THREAD_UNSAFE, the number of elements in the queue is still an atomic_ulong whereas it is not needed (an unsigned long is more than good enough). This has a bad impact on performance.
To overcome this limitation, we need to do preprocessing conditional code (M_IF) and a solution like the one in m-shared (see SHAREDI_ATOMIC_OPLIST & SHAREDI_INTEGER_OPLIST). To factorize in m-core.h. But it needs to be able to handle preprocessing conditional code on policy (buffer_policy_e) which is not supported.
Another solution is to duplicate the field number[] and to use the right one in function of the policy, which increases the size of the structure with not needed fields (in particular for BUFFER_THREAD_UNSAFE).
Another problem is that the size "unsigned long" seems too big. There is no need for a fixed ring buffer communication of more than 2^32 elements on a 64 bits system. So an 'unsigned int' should be enough. On ARM32, the cost as an atomic operation on 64 bits integer is way higher than on 32 bits integer. To consolidate.

Widening the structure size is totally fine for me, as long as its internal structure size. 4-8 bytes is nothing comparing to the size of the buffer itself, so don't hesitate to do it that way.


Option to disable rounding

rounding=0 in the config does not disable rounding

?

That is still rounded (and looks kinda jank tbh)

the window is not.

but the border has a diagonal line

yes

what I would normally expect is to have 90° corners

Rather than two 45°

fixed in 3947fe9e9f8fb2912b969a20fbbec3f2ab62cef0


Port of governing documents from wiki

This also corrects the license file.

If you want to see a rendered version:
https://ryjones.github.io/tsc/

@tkuhrt I changed the index to include links to the TSC member github pages - good idea or no? https://ryjones.github.io/tsc/ shows it.

@tkuhrt @lehors I think I have incorporated all of your feedback. Take a close look at:
https://github.com/ryjones/tsc/blame/master/docs/project-lifecycle.md#L89-L91
and
https://github.com/ryjones/tsc/blame/master/docs/project-incubation-exit.md#L102-L105

@shemnon @lehors I changed the theme to add a table of contents on the side. this will make adding a full list of previous TSC members (and other documents) easier. Take a look?


rfkill: Cannot open RFKILL control device

s6-rc: info: service s6rc-oneshot-runner: starting
s6-rc: info: service s6rc-oneshot-runner successfully started
s6-rc: info: service fix-attrs: starting
s6-rc: info: service fix-attrs successfully started
s6-rc: info: service legacy-cont-init: starting
s6-rc: info: service legacy-cont-init successfully started
s6-rc: info: service legacy-services: starting
s6-rc: info: service legacy-services successfully started
Starting Hass.io Access Point Addon
# Setup interface:
Add to /etc/network/interfaces: iface wlp0s29f7u2 inet static
Run command: nmcli dev set wlp0s29f7u2 managed no
Run command: ip link set wlp0s29f7u2 down
Add to /etc/network/interfaces: address <IP_ADDRESS>
Add to /etc/network/interfaces: netmask <IP_ADDRESS>
Add to /etc/network/interfaces: broadcast <IP_ADDRESS>
Run command: ip link set wlp0s29f7u2 up
# Setup hostapd:
Add to hostapd.conf: ssid=test
Add to hostapd.conf: wpa_passphrase=********
Add to hostapd.conf: channel=6
Add to hostapd.conf: ignore_broadcast_ssid=0
Add to hostapd.conf: macaddr_acl=0
Add to hostapd.conf: interface=wlp0s29f7u2
# DHCP not enabled. Skipping dnsmasq
## Starting hostapd daemon
rfkill: Cannot open RFKILL control device
Driver does not support configured HT capability [DSSS_CCK-40]
wlp0s29f7u2: interface state UNINITIALIZED->DISABLED
wlp0s29f7u2: AP-DISABLED
wlp0s29f7u2: Unable to setup interface.
wlp0s29f7u2: interface state DISABLED->DISABLED
wlp0s29f7u2: AP-DISABLED
wlp0s29f7u2: CTRL-EVENT-TERMINATING
hostapd_free_hapd_data: Interface wlp0s29f7u2 wasn't started
nl80211: deinit ifname=wlp0s29f7u2 disabled_11b_rates=0

Setup:
HomeAssistant OS on a Proxmox. USB Wifi Adpater attatch per Port-Mapping to the VM of HomeAssitant
How can I debug my error ?

Driver does not support configured HT capability [DSSS_CCK-40]

It seems that your network requires High Throughput (HT) mode. I am not 100% sure, but I think it is related to the 40MHz bandwith settings you often have in 802.11n routers. Can you check if your router has such a setting?

I changed the line with the ht_capab from ht_capab=[HT40][SHORT-GI-20][DSSS_CCK-40] to ht_capab=[HT40][SHORT-GI-20].
This solved the problem with my wifi apdater

Yeah ok sorry, this is AP. So yeah, definitely make sense, lots of devices have limited AP support, so this one seems too. Glad you've been able to resolve it.


Add retry option for updateHostsFile.py

Sometimes I get a network timeout when running python updateHostsFile.py -as
Please add an option to retry downloading from the source when that happens instead of just skipping it.

This is a good suggestion, @rEnr3n.  I like it.

.

A good approach to this would be checking connectivity before starting to fetch hosts.

@codeswhite Partially. It's not good enough when network issues arise while it is downloading.


Still getting proxy errors

Hello!
First of all, thanks for the great tool! Installation was a breeze, but I still get errors from Netflix detecting I'm behind a proxy.
As far as I can tell, I've done everything correctly (http://ipinfo.io/ shows the IP of my digital ocean droplet in the NY3 zone).
One thing I'm not a 100% sure off is whether or not I correctly whitelisted my IP addresses on the proxy admin interface (I can just keep adding the same, but I don't see a list of whitelisted IPs anywhere).
Is there any log file I could use to diagnose what the problem actually is?
Thanks a lot :)!

You could look at the netlfix-proxy.log to see what (if any) errors are present.
Ultimately it will most likely come down to IPv6 tunnel configuration. Do you get IPv6=Enabled or Disabled at the end of the build?

I got the following at the end of the script:
netflix-proxy-admin site=http://XXX.XXX.XXX.XXX:8080/ credentials=admin:REDACTED
WARNING: IPv6=Disabled
caching-resolver=Disabled
Hulu region(s) available to you: {'JP': false, 'US': true}
Change your DNS to XXX.XXX.XXX.XXX and start watching Netflix out of region.
Done!

No error as far as I can tell in netflix-proxy.log.
When I created the tunnel, I chose France as the location as it is the closest to me.
When I check the configuration of the tunnel on https://tunnelbroker.net, I see the IPv4 of my droplet in the Client section of the tunnel info.
If it's easier for you, I'm on IRC.

Thank you very much for your help, it's working now!


samv7: add support for RX DMA and RS-485 mode to serial driver

Summary
These three commits add support for RX DMA and RS-485 mode to serial driver.
The first commit b66dd73 introduces the support of RS-485. In this mode the hardware automatically sets RTS pin high when
data are transfered and low then no transfer occurs.¨
Commit 5444f06 adds functions sam_dmarxsetup_circular() and sam_dmarxstart_circular() that create API for operating with two or more circular buffers.
These functions are used in commit 0f27516 that adds RX DMA support to serial driver. It uses two circular
buffers which size can be setup by SAMV7_SERIAL_RXDMA_BUFFER option. The idle bus interrupt is enabled to ensures data are read even if the buffer is not yet full. The timeout can be setup by SAMV7_SERIAL_DMA_TIMEOUT option. Only RX DMA is supported so far.
Impact
SAMV7 only. Serial driver and DMA driver only.
Testing
Tested on SAME70-XPLAINED board and custom board with the SAME70 MCU..

Hi @pkarashchenko, thanks for your input! I processed your suggestions and force pushed the commits. I had to keep few type conversions as compiler was having troubles without them but other than that it should be all done.

I´ve just updated the description in Kconfig and default value for timeout (3750 was way too long for default baud rate). No other changes.


AccessRights is not on ServiceBusTrigger anymore

Using the latest Azure Functions, The ServiceBus trigger does not have a AccessRights as in the example for triggers

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

ID: 0440205f-adcc-0fc5-3adb-db4dbe1d1292
Version Independent ID: 14ad1300-590c-8102-6887-366078e1882a
Content: Azure Service Bus bindings for Azure Functions
Content Source: articles/azure-functions/functions-bindings-service-bus.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Just found this in the body text "omit the access rights parameter". Making my question obsolete

Hi @rasmuschristensen Glad to hear that you found the details you were looking for! We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


feat(api, spi, storage): policy profile implementation

What this PR changes/adds
This PR adds the odrl:profile attribute to the policy model.
This includes the necessary changes to the api, validation, transformation, spi and storage.
Why it does that
The change is necessary to conform to ODRL v2.2
Further notes
For better validation, a new array validator named 'MandatoryIdArray' was added that ensures the array is a list of 'id' values.
To accomodate the transformation, the 'AbstractJsonLdTransformer' was extended with the 'transformIdStringArray' method.
Linked Issue(s)
Closes #3691
Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 48.31461% with 46 lines in your changes missing coverage. Please review.

Project coverage is 75.10%. Comparing base (7f20ba5) to head (436f908).
Report is 338 commits behind head on main.

Files
Patch %
Lines

...nld/spi/transformer/AbstractJsonLdTransformer.java
0.00%
36 Missing :warning:

...lidator/jsonobject/validators/OptionalIdArray.java
84.61%
2 Missing and 2 partials :warning:

...ore/schema/postgres/PostgresDialectStatements.java
0.00%
2 Missing and 1 partial :warning:

...main/java/org/eclipse/edc/policy/model/Policy.java
62.50%
3 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4332      +/-   ##
==========================================
+ Coverage   71.74%   75.10%   +3.35%
==========================================
  Files         919     1054     +135
  Lines       18457    21221    +2764
  Branches     1037     1193     +156
==========================================
+ Hits        13242    15938    +2696
- Misses       4756     4763       +7
- Partials      459      520      +61

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

@ndr-brt

@saschaisele-zf I am currently working on adding the /v3.1alpha version to the management API, because I will need that for another work item too, and I want to avoid duplicated work.
I assume i'll have this done shortly. Could you wait a little, and rebase on that once i'm finished?

Can do that.
I will just revert the last commit then.

#4343 is merged, good to rebase


Consider adding events for introspection events

As far as I can tell, we don't have any... do we?

ApiResource name
scopes
Active/not Active
sub?

done


fix movement directions

Hi @electronstudio , thanks for the PR! I was experimenting with inverted movement to add difficulty, but I forgot to change it back.


Client request retry logic using tenacity

Added tenacity as dependency.
Had to use the Retrying object instead of decorator to dynamically configure retries at runtime.
Added client_retry_status_codes and client_exponential_backoff_max to further configure retries.
Now uses async httpx client instead of the synchronous one.
Removed timeout debug code, but left config option for future use.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

42 of 48 (87.5%)  changed or added relevant lines in 6 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+1.5%) to 76.263%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/ghga_datasteward_kit/s3_upload/uploader.py
10
11
90.91%

src/ghga_datasteward_kit/s3_upload/entrypoint.py
2
4
50.0%

src/ghga_datasteward_kit/s3_upload/utils.py
12
15
80.0%

Files with Coverage Reduction
New Missed Lines
%

src/ghga_datasteward_kit/s3_upload/downloader.py
1
90.91%

Totals

Change from base Build<PHONE_NUMBER>4:
1.5%

Covered Lines:
755

Relevant Lines:
990

💛  - Coveralls


Update golang to 1.13 version

Changes

Update golang to 1.13 version
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[n/a] Includes tests (if functionality changed/added)
[n/a] Includes docs (if user facing)
[x] Commit messages follow commit message best practices

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task and TaskRun to build and release this image

Reviewer Notes
If API changes
are included, additive changes
must be approved by at least two OWNERS
and backwards incompatible changes
must be approved by more than 50% of the OWNERS,
and they must first be added
in a backwards compatible way.
Release Notes
Describe any user facing changes here, or delete this block.

Examples of user facing changes:
- API changes
- Bug fixes
- Any changes in behavior

/ok-to-test

/ok-to-test
Thanks!

/lgtm

Thank you for your contribution!
Could you give some more context for this change?
This changes the version of go in the ko docker image that is used to build pipeline nightly and full releases, so it may be worth mentioning that in the commit message.
It's probably ok, but I'd like to understand if there is any specific feature in 1.13 that we're looking for.

So, the prow jobs are now running with go 1.13, so this should be able to get in :angel:
/approve
/lgtm


Unexpected behavior of one.user.chgrp method

Description
Method one.user.chgrp have two different behaviors:
Eg:
<?xml version="1.0"?>
<methodCall>
    <methodName>one.user.chgrp</methodName>
    <params>
        <param><value>oneadmin:deadbeefdeadbeefdeadbeefdeadbeef</value></param>
        <param><value><i4>356</i4></value></param>
        <param><value><i4>156</i4></value></param>
    </params>
</methodCall>

If user 356 is member of 1 group:
OpenNebula will just switch current group for user.
If user 356 is not member of 1 group:
OpenNebula will remove user from the current group and add to new one.
To Reproduce

Create user
Create new group
Run the method above

You will see: user will not be member of users group anymore.
Expected behavior
I expecting that OpenNebula will return an error, that user is not member of the group.
Or we should update the XML-RPC API documentation to describe both behaviors.
Details

Affected Component: Core
Hypervisor: KVM
Version: 5.8.5

Additional context
Add any other context about the problem here.

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

Yes this is the intended behavior in order to support primary and secondary groups. The user is removed from the old group only if the new group is not a  secondary one.
The idea is to use oneuser chgrp to change across the secondary groups (equivalent to unix newgrp) or change the primary.


Add a license

@jprouty Thanks so much for creating this project. It'd be great if you could specify a license for it -- there's some excellent guidance on choosing one and specifying it in your repo:

https://choosealicense.com/
https://help.github.com/articles/licensing-a-repository/

Thanks William! I've added MIT

Thanks @jprouty !

Just curious, did you have anything in mind WRT using this repo for a new project?

I had two things in mind that led me to ask for you to choose a license:

a general rule I try to know the license of software I use -- just a good rule of thumb for preventing problems!
I think a product possibly could be built around this backend, but if the licensing forbid it, then that would be the end of that. (Will I ever actually make such a product? Highly unlikely -- I have zero free time and this idea is somewhere in the middle of a long list of future projects. But at least I now know not to scratch it off the list due to licensing.) (You should totally make a product around it yourself, btw!)

Ya, I really want to do (2) as it would be the best way to promote the project and get the most # of people using it. But.... it's pretty difficult considering how cumbersome the transaction export (or scraping) that would be necessary (specifically AMZN) in conjunction with storing/retaining sensitive credentials to automate the process.

Yup, it's definitely got some challenges. I wondered if the credentials problem could be handled by creating an agent to run on user's own machines, pull transaction data using their credentials locally, then ship the data to a service to be cleaned up, tagged, transformed, etc. That way the credentials never need to leave the user's machine.


Discord Canary 0.0.143 fails to build with withOpenASAR = true;

Steps To Reproduce
Steps to reproduce the behavior:

build (discord-canary.override {withOpenASAR = true;})

Build log
https://gist.github.com/fufexan/0cfa71fa45e51578381a03161a9cecc3
Additional context
It is working fine either without OpenASAR or with OpenASAR on 0.0.142.
Notify maintainers

@MP2E @devins2518 @Artturin @Infinidoge
Metadata

system: "x86_64-linux"
host os: Linux 6.0.5, NixOS, 22.11 (Raccoon), 22.11.20221030.412b991
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.11.0
nixpkgs: /etc/nix/flake-channels/nixpkgs

same, but with ordinary discord

Seems to have been fixed for me, about 1-2 weeks ago.

same, but with ordinary discord
@dibusure

Ordinary Discord failing to build should probably its own separate issue.

Since my issue has been resolved, I think it can be closed.

same, but with ordinary discord

@dibusure
Ordinary Discord failing to build should probably its own separate issue.

no, it builds, but without openasar

That would also be its own separate issue.


Ensure that key dimensions are hashed correctly

In an earlier PR I accidentally dropped the DynamicMap key dimensions values from the memoization hash, which meant it never updated. Somehow this wasn't caught at all so I've added some unit tests.

Tests passing, merging.


🚀 [Feature] : Move filters to store

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Done with #778


Fix/update notebook openai

Description

#1259
Related Issues

Checklist

[ ] Is this code covered by new or existing unit tests or integration tests?
[ ] Did you run make test successfully?
[ ] Do new classes, functions, methods and parameters all have docstrings?
[ ] Were existing docstrings updated, if necessary?
[ ] Was external documentation updated, if necessary?

Additional Notes or Comments

Codecov Report
Attention: 34 lines in your changes are missing coverage. Please review.

Comparison is base (34830a7) 80.33% compared to head (0abf6d0) 67.88%.
Report is 147 commits behind head on main.

Files
Patch %
Lines

superduperdb/backends/dask/compute.py
47.05%
18 Missing :warning:

superduperdb/backends/base/compute.py
68.96%
9 Missing :warning:

superduperdb/backends/base/query.py
85.71%
3 Missing :warning:

superduperdb/backends/base/metadata.py
75.00%
2 Missing :warning:

superduperdb/backends/base/artifact.py
80.00%
1 Missing :warning:

superduperdb/backends/base/data_backend.py
80.00%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1431       +/-   ##
===========================================
- Coverage   80.33%   67.88%   -12.46%
===========================================
  Files          95      103        +8
  Lines        6602     7340      +738
===========================================
- Hits         5304     4983      -321
- Misses       1298     2357     +1059

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


🐛 [bp-2.5] Refresh external managed token secret if service account is deleted

Summary
cherry-pick https://github.com/open-cluster-management-io/ocm/pull/458 https://github.com/open-cluster-management-io/ocm/pull/504
Related issue(s)
Fixes #https://issues.redhat.com/browse/ACM-13704

/cc @qiujian16 @elgnay

/hold
/test integration

/approve

/lgtm

/unhold

/cherrypick backplane-2.4


Allow memo date change Firefox / Safari

Describe the bug
in v0.22.4 the ability to update memo dates was reinstated. This feature works in Edge and Chrome but not Firefox or Safari
For Firefox this is due to needing a specific date string format provided as the datetime-local typed input value otherwise the time section of the datetime view is not populated. (Chromium based browsers don't have this issue because you can set the time on their calendar view, but you cant do this in Firefox of Safari)
This becomes an issue as Firefox only fires the change event needed to update the date time state if the complete date and time is set on the input
I dont have access to a Mac to debug and confirm this is the same reason for Safari not working, but a quick test showed Safari sets the date to undefined
POC to test this behaviour
https://jsfiddle.net/95c2zae1/3/
Date format used when featured worked in v0.22.2
https://github.com/usememos/memos/blob/e4a09c407c6e2099462c0388e1441781829ad4b8/web/src/hooks/useDateTime.ts#L9
Date format used in v0.22.4
https://github.com/usememos/memos/blob/a94855516611ae56ff2ca9db9816866c3d18ab11/web/src/components/MemoEditor/index.tsx#L421
I've created a PR that will address fixing the date format
https://github.com/usememos/memos/pull/3783
Steps to reproduce
1 -> Create memo using Firefox/Safari
2 -> Edit memo -> change date
3 -> date does not update on UI
The version of Memos you're using.
v0.22.4
Screenshots or additional context
No response

Bump
30/08 Discord question re the issue https://discord.com/channels/1017113925776588902/1045138348165050409/1278800934012719104

Actually, on Chrome for Android is not working either

I can confirm that by editing a memo's date on Safari it will show Invalid instead of the picked date. If I save the post the app sets automatically the date to 1 January 1970.

Note 1: the below should work as a temporary solution for Firefox and Safari(? untested) browsers, but it does not look like its a solution for chrome browsers on mobile (android)
Note 2: I only really care about the day of the memo not the time so this workaround is not suited if you are requiring time. In that case you are better off using a chromium browser to have time support.
Note 3: Please don't consider this a long term solution and close the issue 😅
Go to settings -> system -> additional styles and add
input[type=datetime-local] {
    color: red;
    opacity: 100%;
}

When you edit a memo this will display the browser date picker rather than the Memos formatted representation of the date and it will set the colour to red so that I know its not ready to modify yet
e.g.

Next, go to settings -> system -> additional scripts and add
setInterval(() => {
    const matchingDates = document.querySelectorAll("input[type=datetime-local]");
    const matches = Array.prototype.slice.call(matchingDates);

    matches.forEach(match => {
        match.setAttribute("type", "date");
    })
}, 2000)

This will run a lookup every 2 seconds (2000 in code, fee free to modify milliseconds as needed) to see if a memo editor is open with an incompatible date time format. If one is found it change the format to date only.
In changing the format from date time to date only the CSS stops taking effect, moving the browser date picker back behind the Memos formatted representation of the date and then I know the date is ready to be set.
e.g.

I believe the 10am added to my dates are an adjustment for time zones

Is this going to to be added in the next release?
Great work by the way I really enjoy your tool :)

Thanks, your PR still could not fix my issue, that in ios safari, I could not change the date(the invalid date is fixed), i think it may because that the datetime-local is not support by safari-ios yet. https://developer.mozilla.org/en-US/docs/Web/API/HTMLInputElement/showPicker#browser_compatibility

I have make the change like this and it seems solve the issue above, i thought there is the conflict in the editor focus and datepicker focus. could you make the change into your PR as well.

I'll give the MUI implementation a go on the weekend

:heart_eyes:


Edit REST API doc: Fix typos, punctuation, and wording issues

Edited the RST doc and autogenerated content for the REST API documentation.
Note: This PR is a cherry-picked version of an earlier PR to fix typos in the REST API documentation.  This PR contains only the commit that changes the RST doc (docs/source/rest_api/error_codes.rst) and auto-generated content (rest_api/openapi.yaml).
I'll copy the relevant comments from #1222 , then close it.
Signed-off-by: Anne Chenette<EMAIL_ADDRESS>

Added a commit to fix the REST API description for GET /state/{address}.

Missing sign-offs


iOS/mWeb - Chat - Message compose box can be moved

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

Action Performed:

Go to https://staging.new.expensify.com/
Log in with any account
Open any chat
Try to write a message and use fingers to move the compose box

Expected Result:
Compose box should be static and should not be able to move
Actual Result:
The compose box can be moved independently.
Workaround:
Unknown
Platform:

Where is this issue occurring?

Mobile Web

Version Number: 1.1.60-0
Reproducible in staging?:  Y
Reproducible in production?: Y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/168422372-6889bbda-82d1-4862-8114-08e3df19a279.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause
Slack conversation:
View all open jobs on GitHub

This is a duplicate of https://github.com/Expensify/App/issues/5894 – the attempted solution iosSafariAutoScrollback never really worked and created other issues, so was reverted here.
There's some relevant info here

Going to close this out as a dupe and reopen https://github.com/Expensify/App/issues/5894


Support for weighted edit distance

I'm not sure if SymSpell already has support for weighted edit distance. If so, please tell me how to use it.
Otherwise, I suggest to add this as another possible distance metric, in addition to Levenshtein and Damerau-Levenshtein. The implementation itself shouldn't be problematic: just use the weight matrix instead of the default unit cost. The matrix is input to the constructor, and for command line use it can be stored in a file. (I could in principle do it myself, but I don't know C#)

There is a third-party SymSpell implementation with weighted Damerau-Levenshtein edit distance / keyboard-distance: https://github.com/searchhub/preDict

Weighted edit distance can also be added as a post-processing step.  The preliminary SymSpell results could be filtered/re-sorted according to your preferences.

It is planned to add a weighted edit distance to SymSpell in the future, but there is no timeline yet.

Thanks, I'll look into that!
Yes, but then the problem is the maximum edit distance parameter: I would like this parameter to be the true weighted cost, otherwise I have to use a too large value.
I hope you'll get the time for it.


Values with accents and other symbols in IFC and UTF decoding

Hi, I have characters with accents in the IFC file, and when I read the value in the IFC it appears encoded as UTF-16 I think and I can't decode it.
Sorry for the wording of the question, I had to use automatic translator.
Thanks!

https://github.com/tomvandig/web-ifc/issues/58

Thanks @tomvandig , I will read it carefully

Thanks @tomvandig !


Select component max-h

How I can give a max height to the ul that is inside the select?

menuProps={{className: 'bg-red-500 max-h-20'}}


Automatically update list of outdated translations

Resolves #2547

I think the best way to go about resolving #2547 would be to generate the list of outdated translations and push the new version after the PR has been merged. Essentially, the process would look like:

Pull request opened
PR eventually gets merged
Workflow detects a new commit to master
The outdated translation generator runs
If a new outdated translations file is generated, then it's committed directly to master

(Steps 3, 4, and 5 would be done entirely by GitHub Actions)
See documentation for pushing a commit using Checkout V2 GitHub Action

I think the best way to go about resolving #2547 would be to generate the list of outdated translations and commit the new version to master after the PR has been merged. Essentially, the process would look like:

Pull request opened
PR eventually gets merged
Workflow detects a new commit to master
The outdated translation generator runs
If a new outdated translations file is generated, then it's committed directly to master

(Steps 3, 4, and 5 would be done entirely by GitHub Actions)
See documentation for pushing a commit using Checkout V2 GitHub Action

However, not every PR will be merged into master. So the solution would need to ensure that this process takes place for any remote branches controlled by the riverpod repository.

Running your pull_request workflow when a pull request merges

I don't think this is the way to go.
Rather, I would change the build mechanism such that this file no-longer needs to be committed.
One think we could do is change "npm run build" such that it implicitly generates this file before building the website.


Add openregistry.assets.basic docs

Coverage remained the same at 100.0% when pulling 9b82f763cb52914d04cafeb08669102e11035a9c on ivanka-jakimchuk:master into be8bde7b5f8d4160325fc8dae789c360924f23ec on openprocurement:master.


StringCatalogEnum: Make the project testable and add unit tests

Currently the project is not unit testable. Refactor the project, create testable structure, and add unit tests.

Issue addressed by @charJin , thank you!


run() should be synchronous

Discussion that started here https://gitter.im/cyclejs/cycle-core?at=569934f75de13b3f15e34af6
The problem is:

We may want to use sources or sinks that Cycle.run returns to inspect something
AND the main() and the drivers may run synchronously (like they do in some tests or e.g. server side rendering)

Current solution to that is to delay (with setTimeout) actual execution internally in run(). The discussion shows hows that's not nice in some cases.
One possible solution is to scrap the const {sinks, sources} = run(main, drivers); API and use instead just const subscription = run(main, drivers);, if people need to inspect the sinks and sources, they can wrap main in a function mainWithInspections which intercepts sources and sinks.
Feedback? @Frikki @TylorS
And @Widdershin this could be relevant to restart since I know you were using {sinks, sources} from run().

One possible solution is to scrap the const {sinks, sources} = run(main, drivers); API and use instead just const subscription = run(main, drivers);, if people need to inspect the sinks and sources, they can wrap main in a function mainWithInspections which intercepts sources and sinks.

Currently I am doing:
// Cycle 1
const drivers = {
  DOM: makeDOMDriver('#app-main'),
  preventDefault: preventDefaultDriver,
  keydown: keyboardDriver,
};
// Cycle one emits a stream of "pla data"
const {sinks: {plaData$}} = Cycle.run(kv, drivers);

// Cycle 2
const driversAssistent = {
  DOM: makeHammerDriver(makeDOMDriver('#app-assistent')),
  preventDefault: preventDefaultDriver,
  keydown: keyboardDriver,
  props$: () => O.just({}),
  // Cycle 2 cosumes the stream of pla data
  data$: () => plaData$,
};
Cycle.run(pla, driversAssistent);

How would I do that with the mainWithInspections  api?

Another idea:
const myCycle = Cycle(main, drivers);
const {sinks, sources} = myCycle;
// could be done whenever I want, inside setTimeout or inside setAnimationFrame:
myCycle.run();

Or If I just do not care
Cycle(main, drivers).run();

Your usage isn't recommended. You should have just one run. Usually we should just have one DOM driver, but I think it's better to have two DOM drivers than to have two run instances.

#200 should be success)
If results of run can used only acyncroniously they probably shoud be deferred before can be accessed, using subscription in case of cycle. I don't really understand the problem though, what is wrong with {sinks, sources} returned as synchronious result of run without setTimeout currently used?

@whitecolor the Problem is that if setTimeout is not used, there will be items already emitted by the source and sink observables before a subscriber has a chance to listen (subscribers can only be added after run already returned). Those items will be lost.
@staltz I see that my usage s not recommended for developing a applications. What I am currently working at is a set of small widgets that can be embedded in otherwise static pages (like blogposts, tutorials, articles). Imagine something like jsbin or jsfiddle embeddings but without the possibility to edit the code or something like those widgets on this page: http://worrydream.com/LadderOfAbstraction/
I am developing each widget as it's own cycle application / data flow component but the whole page in which they will be included in the end will not be a cycle application. I need some of the components, when embedded on the same page being able to communicate.
The current possibility to take the sinks of one cycle and pass it into another cycle as sources  is almost perfect for that use case. And I think it makes perfect sense for a cycle to provide it's sources and sinks to the outside as kind of "waste heat".
I understand that not being the recommended usage my use case might not drive your decisions in terms of api design but I did not want to miss the chance at least giving some support for the current api :smile:

I do indeed rely upon sources and sinks in cycle-restart, but it could always be adapted to work with a synchronous run. As long as there is still some way to get a reference to sources and sinks, I'm happy.

@laszlokorte Yeah in general I'd still like to support exporting sinks and sources for others to fiddle as they wish.
@Widdershin I saw the src for restart, and basically you're using sources.Foo.log() but not sinks.
Another option, which is not pretty, is prerun:
const {sources, sinks} = Cycle.prerun(main, drivers);

// ... Use sources and sinks ...

Cycle.run(sources, sinks);

And yet another option is a callback given to run:
Cycle.run(main, drivers, (sources, sinks) => {
  // ... Use sources and sinks ...
});

Other ideas please?

I just noticed the prerun idea is essentially the same as @laszlokorte suggested.

@staltz restart does use sinks, but only for the sake of disposal. If sources.dispose also disposes of sinks then everything would be fine. (I feel like I might not really understand what sources.dispose does vs sinks.dispose)

source.dispose might or might not be available, it is implemented in drivers (DOM Driver has dispose, others may not have). sink.dispose is getting rid of the proxySink subject.
On the other hand, we can just return a Disposable from run(), which can be used to dispose everything.

Right now I'm starting to prefer @laszlokorte's solution, adapted to be backwards compatible:
"I want the sources and sinks from execution":
const {sources, sinks, run} = Cycle(main, drivers);
// ... use sources and sinks here ...
run(); // starts it synchronously

"I don't care about sources and sinks from execution" (backwards compatible):
Cycle.run(main, drivers); // starts it synchronously

Opinions on this? @TylorS @erykpiast @Frikki @laszlokorte @Widdershin

That seems like a sound way to go to me. Just so I'm clear; testing would be as follows

const {sources, sinks, run} = Cycle(main, drivers)

sources.DOM.observable.skip(1).take(1).subscribe(root => {
  // assert stuff
})

run() // makes all the stuff above work (?)

Also, does it feel cluttered when we need dispose() as well with the (proposed) adapter API?
const {sinks, sources, dispose, run} = Cycle(main, drivers, options)

First snippet was correct. Second was wrong. run() returns a disposable.
const {sources, sinks, run} = Cycle(main, drivers)

sources.DOM.observable.skip(1).take(1).subscribe(root => {
  // assert stuff
})

const disposable = run() // makes all the stuff above work

disposable.dispose();

Sounds like a solid plan to me. Seems to cover all the bases.

synchronous
access to sinks
access to sources
disposable

Yeah I can try to come up with teachable concepts out of that.
const {sources, sinks, run} = Cycle(main, drivers);
// Plugs together main and drivers in a cycle

run();
// Execute the cycle

Won't this
const {sources, sinks, run} = Cycle(main, drivers); run()
be sthe same
const {sources, sinks} = Cycle.run(main, drivers); // where `run` executes synchoriously last

?

No, not really. We are talking about this problem
const {sources, sinks} = Cycle.run(main, drivers);
sources.DOM.observable.skip(1).subscribe(el => {
  // Never gets anything because by the time we called run, it synchronously
  // executed and COMPLETED as an Rx Subject, so when subscribed later,
  // nothing happens.
});

@staltz Ok I got and removed the question. So then I think
const {sources, sinks, run} = Cycle(main, drivers);

while preserving existing API looks good, up to you.

Cool, that's probably what we are going to do

Yeah looks good to me.

I guess I don't know the background, but I wonder what is with about ReplaySubject? If missed values are the only problem it'll do the job.

When using ReplaySubject you have to choose a buffer size. You can not know how many items may be missed so you would have to use a very large buffer which is not practical in terms of performance / memory consumption.

Ok, get it. I was thinking about buffer size 1, but obviously not only the last value matters here.

This is now done with Cycle Diversity.


[ONNX] test TFIDF Vectorizer IDF and TFIDF modes

Add few end-to-end test cases for IDF and TFIDF modes of TFIDF Vectorizer.
Corresponding changes in torch-mlir: https://github.com/llvm/torch-mlir/pull/3726

@zjgarvey could you do a second round?

Seems this was approved but was unfortunately not merged. Would you be able to resolve the conflicts @samutamm?


Doctor consultation module is crashing on the cloud instance.

Not good

Turns out 404 ng-includes is what causes these infinite redirects. This has been an issue since 2016.


Broken CSS

New bug when opening news articles. CSS seems to be broken/not recognised by the website at all.

Fixed CSS, was not added to the view.php therefore the website load it.


Colour coding of tones

Use custom HTML in the Anki card styling, e.g. :
.tone1 {color: #0088D5;}
.tone2 {color: #7CA000;}
.tone3 {color: #7267C6;}
.tone4 {color: #ED000D;}
.tone5 {color: #617C84;}
This combines well with identification of tones (see issue "Identification of tones").

This would require the following fields in the CSV output from scanned text:
{hanzi}
{english}
{pinyin w/ tone class}
{hanzi w/ tone class}
{example of hanzi in usage}
Example of fields in use:
饭店
hotel
fàndiàn
饭店
北京饭店，长城饭店

The new code today takes the CC-CEDICT information and splits it into tab separated hanzi, definitions, and pinyin. We used the tone information (e.g. ni3) from CC-CEDICT to use custom HTML formatting and to replace the given pinyin with diacritic-marked pinyin (nĭ).
Custom Anki card formats can make use of the HTML information to give tones colour.
If we can't find a way to make a complete .apkg file which contains both new words and also the card formatting, then it may be acceptable to provide users with an empty card template which their new words can be imported to.


LinearOperator dtype determination in _interface.py

Reference issue
Closes gh-19207
What does this implement/fix?
When determining the dtype:

Make dtype the smallest possible.
Invoke rmatvec, matmat, and rmatmat in dtype determination.

Reducing the dtype of the test vector is sensible; otherwise, they essentially all come out as float64. But the other two are unnecessary given a correctly-implemented LinearOperator.

Using matmat substituting matvec for dtype determination fails above; see https://github.com/scipy/scipy/issues/19207#issuecomment-1712842372
So revert the change.

I have also tried to include rmatvec or rmatmat in dtype determination in
            #if self.rmatvec is not None:
            #    v = np.zeros((self.shape[0],), dtype=np.int8)
            #    rdtype = np.asarray(self.rmatvec(v)).dtype
            #    self.dtype = np.promote_types(self.dtype, rdtype)

but they are not None if not present, as I have anticipated and hoped for, instead giving the Not Implemented error... So I give up.

Ready for review please

I don't think so. LinearOperator objects will always have a .dtype attribute set (unless if one writes a particularly broken subclass).

I don't think so. LinearOperator objects will always have a .dtype attribute set (unless if one writes a particularly broken subclass).

Maybe remove it then as never being executed? It is inconsistent with my changes following the original framework.

Oh, that's the code that already exists? Yes, it should be removed and replaced with just aslinearoperator() wherever it is called. It's a very old function that predates the dtype inference now incorporated directly into LinearOperator. It's harmless, as it will never take that branch, but also useless.

Oh, that's the code that already exists? Yes, it should be removed and replaced with just aslinearoperator() wherever it is called. It's a very old function that predates the dtype inference now incorporated directly into LinearOperator. It's harmless, as it will never take that branch, but also useless.

In scipy/sparse/linalg/_eigen/arpack/arpack.py the following
function
def _aslinearoperator_with_dtype(m):
is being called 31 times... So maybe should be changed in a separate PR after this one is merged.

Oh, that's the code that already exists? Yes, it should be removed and replaced with just aslinearoperator() wherever it is called. It's a very old function that predates the dtype inference now incorporated directly into LinearOperator. It's harmless, as it will never take that branch, but also useless.

I have first thought it would be better for a separate PR but waiting for the final review have just done it.

Ready to merge?

This PR is 4 months old now. Could someone review it and consider merging?

I already forgot about this PR... Could someone please review it, otherwise it might just slip through the cracks.

Trying to jumpstart it back to life after months of deep sleep

@j-bowhay could you please have another look? It appears stalled but I would love to clear all my old PRs ready to merge before starting new ones.

Could you add a test for the type determination, eg. for this new integer functionality

There is already a new test in this PR as I can see. Could you please elaborate your request?

Could you add a test for the type determination, eg. for this new integer functionality

There is already a new test in this PR as I can see. Could you please elaborate your request?

The new tests are for the changes to svds as far as I could see. Could you add a test like the example in your original issue
    >>> from scipy.sparse.linalg import LinearOperator
    >>> def mv(v):
    ...     return np.array([2*v[0], 3*v[1]])
    ...
    >>> A = LinearOperator((2,2), matvec=mv)

@j-bowhay et al.
While waiting so long to be reviewed, the code has become outdated due to the introduction of new rules https://numpy.org/neps/nep-0050-scalar-promotion.html#proposed-weak-promotion  in NumPy 2. Too large for int8 generic NumPy and python integers used to perform the matvec on an int8 vector now result in errors 'Python integer 200 out of bounds for int8'. In the current revision of the PR, the errors come from doc/source/tutorial/optimize.rst where such integers are used in examples.
The core function
    def _init_dtype(self):
        """Determine the dtype by executing `matvec` on an `int8` test vector.

        In `np.promote_types` hierarchy, the type `int8` is the smallest,
        so we call `matvec` on `int8` and use the promoted dtype of the output
        to set the default `dtype` of the `LinearOperator`.
        We assume that `matmat`, `rmatvec`, and `rmatmat` would result in
        the same dtype of the output given an `int8` input as `matvec`.

        Called from subclasses at the end of the __init__ routine.
        """
        if self.dtype is None:
            v = np.zeros(self.shape[-1], dtype=np.int8)
            self.dtype = np.asarray(self.matvec(v)).dtype

in scipy/sparse/linalg/_interface.py thus needs some more sophisticated logic, I guess. Suggestions are very welcome to fix this new issue, please.

@j-bowhay could you please have another look? I have added the missing pytests per your request and had to massage the logic of the main function to deal with the overflow of the Python int being converted into one of numpy ints in numpy 2.
My pytests all pass know. The only failed run appears unrelated to this PR.

While idling, I forgot how to run properly pytest on my local setup so had to burn a bit of CI resources here, sorry.

@lobpcg, thanks for pushing this forward! I am away travelling for the next four months without much computer access, so unfortunately, it might be a while before I can look at this again. Maybe @rkern would kindly be able to take another look.

@rkern I do not know why one test has started failing https://github.com/scipy/scipy/actions/runs/11410097437/job/31751716717?pr=19209
after fixing the typo in the unrelated test... The fails are in tests of "asarray" which is concerning but still should be unrelated to this PR?

@rkern everything fixed

Agreed. In it goes! Thank you for your patience and endurance.


Fix row in devise shared_links.

Fix the shared links so that they always go in a row as their own element.

Fixed.


Export GPS routes with comments to PDF

It will be nice to export list all the GPS routes for the given client as the PDF document. The export button should be available in the geofencing-ui plugin [1].
[1] https://github.com/camel-labs/camel-labs/tree/master/iot/cloudlet-ui/geofencing-ui

Closing old tickets.


question: is it possible to operate outside of the current working directory?

Hi, thanks for the awesome plugin! I included it as the recommended search and replace integration in my plugin https://github.com/mikavilpas/yazi.nvim/pull/250 today.
I was wondering if it's supported to filter files outside of the current working directory. Here's an example of what I get:

To make that a bit clearer, vim.uv.cwd() points to /Users/mikavilpas/git/yazi.nvim while my Files Filter points to /Users/mikavilpas/.config/nvim/lua/config/**
I'm wondering if I have forgotten to set some value, or if this is simply not supported. I can maybe prevent confusing cases like this in my integration.
Thanks!

@mikavilpas that's awesome. I have it in my neovim config todos to try yazi.nvim someday and this integration makes even more enticing 😆
Regarding operating within a specific directory, different from cwd, that is supported by passing the directory path in Flags. Files Filter gets passed as an argument to --glob, so that's likely why it's not working.

I've been wondering if the messaging / UI around this could be improved, so let me know if you have any ideas 😄
Maybe this would be good?

actually, this might be better:

just FYI, I've noticed a bug on invalid paths that I'll be looking into:
https://github.com/MagicDuck/grug-far.nvim/issues/148

I've that change to the default placeholders to make it more clear that a dir / file path can go in Flags 😄
https://github.com/MagicDuck/grug-far.nvim/pull/149

It seems to work perfectly now! I only switched the Flags to the Files Filter field and all is good now 😄
Thanks so much for the help!

Awesome, great to hear!


NA condition for areas needs to be debugged

This work as of end of day. Don't merge until we talk about it.


fix link to Falco with helm

Signed-off-by: ramshazar<EMAIL_ADDRESS>What type of PR is this?
/kind cleanup
/kind documentation
Any specific area of the project related to this PR?
/area documentation
What this PR does / why we need it:
There is a link on https://falco.org/docs/getting-started/third-party/production/#gke that is an 404.
The PR provides the correct link.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ramshazar
To complete the pull request process, please assign mfdii after the PR has been reviewed.
You can assign the PR to them by writing /assign @mfdii in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Welcome @ramshazar! It looks like this is your first PR to falcosecurity/falco-website 🎉

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr, ramshazar
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: d8c3ab629b45178bfc6b4fb95ef429b5037676e4


Refactor

Refactored the project to improve code organization. Grouped related functionality into separate module folders.

@CodiumAi-Agent /review


Your demo cannot be displayed

I want to see how your components are used, but your demo website seems to be experiencing issues, and I am unable to see the correct call code example. It always shows loading
demo

@playertk - this should be resolved.  Thanks for letting me know!


Random 403s, 412s and 502s when bot tries to reply to Microsoft Teams conversation

I have MSBF chatbot built using .NET SDK v4. The bot is deployed to Azure and has enabled Microsoft Teams and DirectLine channels. The MS Teams tenant's location is Europe.
Since approx. last week I have noticed increased amount of errors in my logs when I handle the conversationUpdate event and try to send welcome message to the user using TurnContext.SendActivityAsync(...) call.
All of the following errors are related to calls to POST https://smba.trafficmanager.net/emea/v3/conversations/<conv_id>/activities/<activity_id>.

403 - Forbidden:
{"error":{"code":"BotDisabledByAdmin","message":"The tenant admin disabled this bot"}}

412 - Precondition Failed:
{"error":{"code":"PreconditionFailed","message":"Precondition failed, please try again."}}

502 - Bad Gateway:
{"error":{"code":"ServiceError","message":"Internal server error"}}

Additional info:

All these errors are related to handling conversationUpdate and trying to send message back to the user
Messages that I am trying to send are either simple text, list of adaptive cards or carousel of adaptive cards
The errors are random, they happened only for some users, for other users the messages were sent successfully
Requests which caused these erroneous codes all took longer than 2 seconds
As I mentioned this started to happen only recently
The conversations where I am trying to reply to, are one-to-one (bot vs user)

I have already seen #194 and this post, however I am not trying to send any typing indicators.
Any ideas what is going on? Is this related to outage in European regions?

You should only receive that 403 message when the bot is not enabled for that tenant, either because tenant admins have disabled all apps, or your app specifically. Changes to this permission can take a while to propagate across a large tenant (particularly for users who have never logged into Teams).
For the 412/502 I'd need to see the message that generated the response. Since you mention sending lists/carousels of cards my guess is that you're running into issues with maximum card size and/or maximum # of cards per carousel.

We are facing absolutely same issue. It's not related to message size as we found evidence of exactly same message going through just fine in App insights. This should be enterprise ready product so we would like to see some official acknowledgement of this. Looking at azure health console just hurts as there are failures everywhere but all is allays green there..

My apologies for the delayed response. I've received additional information from Teams:

They are investigating the increase in 412 responses, for now they recommend developers retry when getting this response.
The 403 responses are probable directly related to my previous response, and the increased propagation time for permissions during this massive usage spike.
For 502s, there was an ongoing issue with typing indicator resulting in 502s, for which the fix was rolled out WW on 4/1. You should stop receiving 502 responses within the next 24 hours.

Please let me know if the above additional information doesn't seem to align with what you're seeing.

Thanks for letting me know. Is there a reason why it is not acknowledged in Azure Service status? If that is not the right place is there some service health portal for Bot Service and Basic channels? It is really challenging to learn about failures on GitHub threads.

I've dug into the health reporting a bit, and currently the only health reporting we do is for the service all up on the Azure Service status page. We'll explore how to service potential channel health issues in the future.


Rename cancel to dialog-cancel

Ones https://github.com/OnsenUI/OnsenUI/pull/1470 is merged, this will get rid of weird error syntax message in chrome.

Thanks!


Feature Request label/fill color valueMapping

ability to use 'colorMapping' in the same way 'valueMappings' is implemented.
We would like to set the label of fill color based on the value without using the thresholds, because the values are set numers/errorcodes/whatever which might no be in correct numeral order to use thresholds, or even might be text instead of integers.
For example network interface states up/down/error/nan mapped to a specific color.

numbers have an order so if your incoming data is numerical you can get the color you want, you just have to order the thresholds from lower to higher. Threshold indexes don't have a meaning, just a traversal order.
will have a think about the string data. That as you say is unsupported.

+1 for basing the colors on string values.
the deprecated flowcharting plugin used to solve this by providing an input for color, pattern and level in the threshholds section.

This has been added using thresholds rather than mappings. Now in addition to thresholds and thresholdsRef there's also thresholdPatterns and thresholdPatternsRef. This is an example use.
With this you can use a text data-series to drive labelColor, strokeColor and fillColor via regex.
It will be in the next build.

published in 1.14.0


Design logic of IPFS hash after adding articles into circle

Design of how does system regenerate hash for public articles added into circle
Design of resolving old article link

See PoC repo here.
Issues to be discusses:

How to implement encryption on backend, Node.js 15+ experimental implementation or node-webcrypto-p11?
How to maintain client side decryption code, hard coded in string or compile and import?

Step into implementation. Feel free to reopen and add more details.


Deprecated chrome.loadTimes()

Hi,
I am using your lib to send the RUM Speed Index to Google Analytics as a metric.
Unfortunately, Chrome has been sending the following in DevTools' console for a while (I am using the Canary channel):

[Deprecation] chrome.loadTimes() is deprecated, instead use standardized API: Paint Timing. https://www.chromestatus.com/features/5637885046816768.
[Deprecation] chrome.loadTimes() is deprecated, instead use standardized API: Navigation Timing 2. https://www.chromestatus.com/features/5637885046816768.

By chance, are there plans to switch this in this library?
Thanks,

Looks like this was closed with no comment. Was there some discussion elsewhere?

@pmeenan I'm sorry I don't understand, I have the latest trunk but I still get warnings in the console.


[NEW] vercel.com

Add vercel.com to list of passkey supported sites
Reference: https://vercel.com/login you can see login with passkey

Thanks for submitting do you mind making sure the alphabetical order is correct 🙏


Fix ODS-1491 to remove Openvino from dashboard after uninstall

As of now, when ODS-1491 finishes to run, it leave Openvino as "disabled" in RHODS Dashboard. This PR wants to add the removal process
Signed-off-by: bdattoma<EMAIL_ADDRESS>

JenkinsJob: #1162 SUCCESS the failure is expected due to a product bug


MP Metrics intermittent failures

After changes based on development guidelines, I am getting intermittent failures:
Failed tests:
  MetricsTest.testGauge:113
Expected: is <42>
     but: was <1>
  MetricsTest.testGaugeMetadata:145 Unexpected gauge value (in seconds)
Expected: is <2520>
     but: was <60>

When I retry the pipeline, these get fixed (up to now this always worked).

This was caused by MetricsMpServiceTest - when a CDI container is created outside of MP server, it is not closed when the server is closed (as it does not belong to it).
I have modified the test to use the implicit CDI started by server - this should fix the problem.


Cann't delete vmi testvm

Is this a BUG REPORT or FEATURE REQUEST?:
/kind bug
What happened:
I want to delete vm:
$ virtctl stop testvm
$ kubectl delete vm testvm
but vmis testvm left:
$ kubectl get vm
No resources found in default namespace.
$ kubectl get vmis
NAME     AGE    PHASE       IP    NODENAME
testvm   4h9m   Scheduled         <IP_ADDRESS>
$ kubectl get pod
No resources found in default namespace.
I delete with --force, but fail:
$ kubectl delete vmis testvm --force --grace-period=0
warning: Immediate deletion does not wait for confirmation that the running resource has been terminated. The resource may continue to run on the cluster indefinitely.
virtualmachineinstance.kubevirt.io "testvm" force deleted
$ kubectl get vmis
NAME     AGE    PHASE       IP    NODENAME
testvm   4h9m   Scheduled         <IP_ADDRESS>
What you expected to happen:
Delete vmis testvm.
How to reproduce it (as minimally and precisely as possible):
Deploy with vm.yaml, and stop and delete vm.
Anything else we need to know?:
no.
Environment:

KubeVirt version (use virtctl version): 0.33.0
Kubernetes version (use kubectl version): 1.17.9
VM or VMI specifications:
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release): centos 7
Kernel (e.g. uname -a): 3.10.0
Install tools:
Others:

maybe you can paste some virt-controller、kube-controller's log, that would be helpful.

cc @DanielQujun
virt-handler: failed to bindmount containerDisk containerdisk
{"component":"virt-handler","kind":"","level":"info","msg":"all containerdisks are ready","name":"testvm","namespace":"default","pos":"mount.go:367","timestamp":"2020-10-13T06:25:11.342880Z","uid":"736d8612-3aa8-40d6-8bbd-9ed3e40f4ef0"}
{"component":"virt-handler","kind":"","level":"info","msg":"Bind mounting container disk at /paasdata/docker/overlay2/4e59a66089cb2360266858a2734722beef05681a7eaad1c340ede964fb021acf/merged/disk/cirros.img to /var/lib/kubelet/pods/eb6fa840-1bcf-468e-baf1-4228352fd9c5/volumes/kubernetes.io~empty-dir/container-disks/disk_0.img","name":"testvm","namespace":"default","pos":"mount.go:252","timestamp":"2020-10-13T06:25:11.346301Z","uid":"736d8612-3aa8-40d6-8bbd-9ed3e40f4ef0"}
{"component":"virt-handler","kind":"","level":"error","msg":"Synchronizing the VirtualMachineInstance failed.","name":"testvm","namespace":"default","pos":"vm.go:1334","reason":"failed to bindmount containerDisk containerdisk: Error: no such file or directory\nUsage:\n virt-chroot mount [flags]\n\nFlags:\n -h, --help help for mount\n -o, --options string comma separated list of mount options\n -t, --type string fstype\n\nGlobal Flags:\n --cpu uint32 cpu time in seconds for the process\n --memory uint32 memory in megabyte for the process\n --mount string mount namespace to use\n --user string switch to this targetUser to e.g. drop privileges\n\nno such file or directory\n : exit status 1","timestamp":"2020-10-13T06:25:11.370199Z","uid":"736d8612-3aa8-40d6-8bbd-9ed3e40f4ef0"}

ok，the issue has been fixed with following command:
kubectl -n kubevirt patch kv kubevirt --type=json -p '[{ "op": "remove", "path": "/metadata/finalizers" }]'

ok，the issue has been fixed with following command:
kubectl -n kubevirt patch kv kubevirt --type=json -p '[{ "op": "remove", "path": "/metadata/finalizers" }]'


Addin disappeared. Get addins redirects to web

I have this issue: https://github.com/OfficeDev/outlook-dev-docs/issues/514 but cant comment since it has been closed. I had sideloaded my addin correctly and was trying some things when I accidentally closed outlook. Now, after opening again, the addin is not loaded and whenever i click on the get addins button it opens a tab in the browser and redirects me to: https://appsource.microsoft.com/en-us/marketplace/apps?product=office%3Boutlook&page=1&src=office
instead of opening the dialog inside outlook.
Curiously, if i go to file -> Info -> Manage addins, it opens https://outlook.office365.com/mail/inbox and there I can see my addin under the "My addins" tab.
Whats going on? Why has it stopped showing my addin? Ive tried rebooting and it still doesnt work.

Transferred to the correct repository.

@jasonjoh Thanks for transferring.
@GMolini Thanks for raising this. I'll transfer to the office-js repo so the product team can take a look.
Thanks.


Uninformative RequirementParseError

When using tfutils, if I run a script outside of a github repo, I get the following (uninformative) error. This error should be updated to be more informative.
INFO:tfutils:module tensorflow.python.training.learning_rate_decay not in a git repo, checking package version INFO:tfutils:module __main__ not in a git repo, checking package version Traceback (most recent call last):   File "train_alexnet_kanef_imnet_example.py", line 533, in <module>     train_model()   File "train_alexnet_kanef_imnet_example.py", line 529, in train_model     base.train_from_params(**params)   File "/home/anayebi/.local/lib/python2.7/site-packages/tfutils/base.py", line 165, in __init__     self.sonified_params = sonify(self.params)   File "/home/anayebi/.local/lib/python2.7/site-packages/tfutils/utils.py", line 147, in sonify     for k, v in arg.items()])   File "/home/anayebi/.local/lib/python2.7/site-packages/tfutils/utils.py", line 147, in sonify     for k, v in arg.items()])   File "/home/anayebi/.local/lib/python2.7/site-packages/tfutils/utils.py", line 147, in sonify     for k, v in arg.items()])   File "/home/anayebi/.local/lib/python2.7/site-packages/tfutils/utils.py", line 147, in sonify     for k, v in arg.items()])   File "/home/anayebi/.local/lib/python2.7/site-packages/tfutils/utils.py", line 162, in sonify     rval = version_check_and_info(mod)   File "/home/anayebi/.local/lib/python2.7/site-packages/tfutils/utils.py", line 62, in version_check_and_info     info = version_info(module)   File "/home/anayebi/.local/lib/python2.7/site-packages/tfutils/utils.py", line 41, in version_info     info = pkg_resources.get_distribution(pkgname)   File "/home/anayebi/.local/lib/python2.7/site-packages/pkg_resources/__init__.py", line 550, in get_distribution     dist = Requirement.parse(dist)   File "/home/anayebi/.local/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2935, in parse     req, = parse_requirements(s)   File "/home/anayebi/.local/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2883, in parse_requirements     yield Requirement(line)   File "/home/anayebi/.local/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2892, in __init__     raise RequirementParseError(str(e)) pkg_resources.RequirementParseError: Invalid requirement, parse error at "'__main__'"

ok this is resolved now with a (more informative) warning.


No error message in the fsi window

I seem not unable to get error messages in the fsi window. For example,
let input = File.ReadAllText("Playground/input/Day1.tx")

Sending the code to fsi results no result, which should be telling me that the file "Day1.tx" doesn't exist.

Fixed in 1.0.7


drivers/timer: Restore non-tickless tick count behavior

The newer series of timer drivers will compare counters vs. the last
tick boundary to compute a number of ticks to announce to the kernel.
In the case of CONFIG_TICKLESS=n, this actually represents a change of
behavior from our old scheme where "ticks" always reflected the number
of interrupts received.
The distinction only matters when an interrupt is delayed more than a
full tick, of course.  But that actually makes a difference to some
timekeeping code.  Restore the old behavior.
This also has the benefit of further reducing code size when !TICKLESS
and improving performance of the ISR by removing the division
(remember Cortex M0 has no hardware divide!).
Fixes #12409
Signed-off-by: Andy Ross<EMAIL_ADDRESS>

Also the commit message suggests this will reduce code size and eliminate a division in the ISR, but I don't see that effect in this commit;

It's on line 72.  When dticks is used in practice the division can't be optimized out.  I think that's clear enough, but if you have different phrasing feel free to commit it.
As far as a test goes: that just seems silly.  We already test TICKLESS=n in CI, existing emulators are known to delay interrupts significantly.  And in any case this was a hygine issue never observed in practice.  You seriously want a test in the tree that does nothing but spin with interrupts locked just to verify that the argument to z_clock_announce() is the literal 1 that can be plainly observed in source code?
Can you please remove this -1 or else move the related but to a 1.15 target?  We're in a release rush right now and no one has time for needless busywork.

Also the commit message suggests this will reduce code size and eliminate a division in the ISR, but I don't see that effect in this commit;

It's on line 72. When dticks is used in practice the division can't be optimized out. I think that's clear enough, but if you have different phrasing feel free to commit it.

How can it be optimized out?  It's used in setting last_count.

However, this PR should include a test that demonstrates the behavioral difference between tickless and non-tickless configurations and that the change in this PR has the intended effect of restoring desired behavior. Without a test the regression may be re-introduced, possibly when a new platform is added

That's a nice idea but given the current focus on fixing a considerable number of open bugs for the 1.14 release, should not gate this patch.
If you really want that to happen, file an enhancement GH issue ticket.

As far as a test goes: that just seems silly. We already test TICKLESS=n in CI, existing emulators are known to delay interrupts significantly. And in any case this was a hygine issue never observed in practice. You seriously want a test in the tree that does nothing but spin with interrupts locked just to verify that the argument to z_clock_announce() is the literal 1 that can be plainly observed in source code?

You identified this as a concern originally.  I was fine with #12409 being closed as not-worth-fixing as long as that decision was recorded in the issue.
If the code is going to be changed to restore behavior that was unintentionally changed I don't think it's silly to expect a test to be added to make sure that doesn't happen again.
I will, however, drop my review back to a comment so Zephyr can blaze forward towards release.


XS code generators, object groups, etc.

This is the profile we use for XS with Fuzzilli. It includes code generators to stress unique aspects of the XS runtime. It has two ObjectGroups and several additional build-ins to fuzz Hardened JavaScript features. It also pulls in the RegExpFuzzer from V8.

Oops.... failed the CLA check because of wrong email. Closing.....


Guidelines for contributing to the references

Copy-paste from the google doc we reviewed and approved in the Training WG a few months ago.

@petermattson  or @bitfort , could one of you review this PR now that I've worked around the git login issues I was having before?


introduce ClientIDCallback to allow JAAS login module to retrieve Client ID

Currently there is no way to get client ID in custom login module. Its required for some MQTT cases when username is not used.

Couple of things:

You need a Jira for this change.
You need to squash all your commits.
Your commit message should conform to the pattern outlined in the Hacking Guide.

What is the JAAS login module that requires the ClientIDCallback?

What is the JAAS login module that requires the ClientIDCallback?

In my particular case custom JAAS login module contains implementation specific logic but Client ID is required to handle the case

@gtully added JaasCallbackHandlerTest


Compiler allows to use undeclared names, failing at runtime

I was adapting some code to work with modules and I found something I did not expect.
TypeScript Version:  2.1.5
Code
package.json:
{
  "name": "tsbug",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "dependencies": {
    "tslib": "^1.5.0",
    "typescript": "^2.1.5"
  }
}

tsconfig.json:
{
    "compilerOptions": {
        "strictNullChecks": true,
        "noImplicitAny": true,
        "target": "es5",
        "module": "commonjs",
        "outDir": "build",
        "moduleResolution": "node",
        "lib": [
            "es5",
            "dom"
        ]
    },
    "include": [
        "src/**/*.ts"
    ]
}

src/main.ts:
new Thing();

export function main() {
}

src/Thing.ts:
// Note this class is not exported!
class Thing {
}

Expected behavior:
Running tsc should show a big error because Thing is not defined nor imported in src/main.ts.
Actual behavior:
src/main.ts compiles and breaks at run time instead.
"use strict";
new Thing();
function main() {
}
exports.main = main;

src/Thing.ts is not a module (it has no imports / exports) so it's assumed to be global code, i.e. included via a script tag.

Interesting... Since when is there this behavior? Previously it would at least require a <ref/> tag if I recall correctly.

It's always worked this way

Alright then... I'm just surprised.
Running tsc ./src/main.ts fails, so using globals this way impedes you to perform file-by-file compilation... which sort of makes sense since in order to know if that declaration is an error you would need to compile every other file in the project that is not proven to be a module.
Furthermore, in my original code VS Code highlighted the undeclared name (I was not able to reproduce that in the test case) as an error...

use --isolatedModules if this is what you want to achieve, and every file would be expected to be a module, or you will get an error.

Thanks, I'll use that.


Add release notes for v2.7.0

Description
Adding release notes for v2.7.0 release
Issues Resolved
#356
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Closing as duplicate of https://github.com/opensearch-project/job-scheduler/pull/368

Failing due to unavailable JS 2.8.0 snapshot, will re-run checks after an opensearch distribution build is generated with JS component


feat: compassWallet / sei

Sei EVM is in continuous development, and so is Compass.
That's why it has a link for a Canary version today.

It will work. Actually, It's live on DragonSwap - https://test.dragonswap.app/swap

DragonSwap team had faced an issue while integrating this, that's why I have raised a PR so other dApps (we are reaching too), can add it easily.

It was not working with flag, that's why I asked them to change that to getInjectedConnector({ target: window.compassEvm })

@magiziz @DanielSinclair please do review this.

@hsnice16 Could we wait until it's released to production ? The extension is currently marked as "canary" and only 5 users are using it only.

Yeah, as I have mentioned above in all the replies, this is still in development, and this PR will help dApps integrate it easily and do development faster, that's why it has fewer users and we will need you to merge this PR.

Yeah, as I have mentioned above in all the replies, this is still in development, and this PR will help dApps integrate it easily and do development faster, that's why it has fewer users and we will need you to merge this PR.

We usually prefer wallets working in production before merging, otherwise we'll have to maintain it with adding new links, new steps and just watch out for it. We'll have this PR open until they've released in production.

You can merge this PR for now, and once it goes live in production, I will raise one more PR, and after that, you won't have to do anything.

@hsnice16 We will wait for the time being so that we can properly QA. You can test out your wallet with this example deployment here. Feel free to ping us here to re-open or open another PR when it gets further along, and then we'll review and merge.

Hey @magiziz @DanielSinclair, Sei EVM changes are live in production now. You can install compass wallet, and use the features - https://chromewebstore.google.com/detail/compass-wallet-for-sei/anokgmphncpekkhclmingpimjmcooifb
Let me know, what should be my next steps to add support for CompassWallet in Rainbowkit.
Many teams developing in this space are asking for it, so it would be great if we could add it ASAP.
Thanks.

@hsnice16 Sounds good. Will re-open this PR and re-review. Do you know if they support EVM chains by any chance ? I'm playing around with it rn, but doesn't seem like there is EVM support for the wallet.

Yes, they do. Sei EVM is live on Arctic-1 chain only, so you can switch to that chain. You can use DragonSwap dApp to test - https://test.dragonswap.app/swap
Let me know if you need anything.
https://github.com/rainbow-me/rainbowkit/assets/56081584/b391b6ac-5f51-4aed-96d3-a043f116d5a8

Let me know if you want me to raise a new PR (I am a Compass team member) @magiziz @DanielSinclair

team! @magiziz @DanielSinclair do let me know the next steps here.

@hsnice16 Yes we'll come back to this as soon as we can. Please bear with us 🙏

Okay. Thanks, 👍

I have updated the Chrome link to the production Chrome web store link.
Let me know, if need any other change.

@hsnice16 Sounds good. I'll take a last look and approve the PR.

@hsnice16 Just did some changes for compassWallet, added wallet to the docs, included it in our example dApp + added sei chain and fixed some merge conflicts. Before mergin this can you make sure the compassWallet.svg is sort of rounded square instead of a transparent background ? I would prefer having the dark background and white compass logo like they have in their extension.
Example of dark mode image not working as expected:

Hey @magiziz, I updated the logo. You can check.

chore: optimize compassWallet.svg

Hey @magiziz, what do you do in optimization?

@hsnice16 I just optimized it with svgo library. I approved the PR we should be able to merge this either today or tomorrow.

Thanks.

Do let me know if anything blocks the merging.

@hsnice16 We'll merge this soon. Please bear with us 🙏

Okay.

thanks for merging this PR @DanielSinclair @magiziz 🙌
do we have any ETA on the next new package release?

@hsnice16 Should be around today or tomorrow

Thanks


Implement signatureHelp and implementation

Thanks for the feedback, I really appreciate that you care about small details like the order (which I totally agree with since there are a lot of methods and it would be quite hard to navigate without an IDE).
I think i can fix everything today.

@ebkalderon Everything should be fixed

Affects #10.


ProtoSchool Taipei - Deep Dive into IPFS and Filecoin (Oct 21, 2019)

ProtoSchool Taipei - Deep Dive into IPFS and Filecoin
Activity content:
Schedule: October 21st, 2019 19:00-22:00 (GMT+8)
Location: Syntrend Creative Park 11/F
Attending guest:
Raúl Kripalani - Technical Lead, libp2p
Jeromy Johnson - Technical Lead, Filecoin
Kevin Hoo - ProtoSchool Taipei Chapter Co-organizer
Jim Lin - ProtoSchool Taipei Chapter Co-organizer
Kevin Wong - ProtoSchool Hong Kong Chapter Organizer
Activity process:
18 : 30 - 19 : 00 Door Open & Networking
19 : 00 - 19 : 10 Intrudction of Speakers
19 : 10 - 19 : 20 Welcome to ProtoSchool Taipei － Kevin Hoo
19 : 20 - 20 : 30 Technical Deep Dive into libp2p － Raúl Kripalani
20 : 30 - 21 : 40 Development of Filecoin － Jeromy Johnson
21 : 40 - 22 : 00 Networking
Meetup Link:
https://www.meetup.com/IPFS-Taiwan/events/265679767/

Closing as this probably happened already!


Fix some errors in snippets

Description
This addresses the problem of spurious failures on Cirrus (only) of the sample analysis script.
I think that the fact that it doesn't occur on LUCI is just a result of a race.  I think that there is a bug in the code that checks "snippets" (i.e. the non-application samples) where it writes all of the snippet output to a single file, overwriting it many times, and it just depends on which snippet gets saved last as to whether it passes or not.  Which means we haven't really been testing them, so some of them have a bug in them.  The reason it gives such a cryptic message is that the line where it happens is in the generated boilerplate (a ";" after the fake symbol created for the analysis), and so it doesn't have a line in the corresponding sample. I found several places where there's a spurious comma after the code, which causes this next line to have a missing statement error. So, on Cirrus, it happens to sometimes write one of these places last to that file, and so the analysis fails.
This PR addresses the sample errors, but not the bug in the analyzer. I'll address that in another PR. It also turns on the Linux analysis that runs on Cirrus again.
Related Issues

https://github.com/flutter/flutter/issues/87517

I find it weird that the "no tests" bot didn't post anything until 13 hours after I pushed.

@Piinks, actually, I was waiting for @Hixie to give a test exemption for this PR, but I guess it's important to get the analysis running again on Cirrus too. Ian, if you decide this didn't merit an exception, I'm glad to make a new PR to address anything needed.

Oh whoops! I saw the label had been removed by the bot but it was all green now so I slapped it back on there. Sorry!

test-exempt: adds tests


Support for stdin/stdout?

👋 Thanks for setup-cfg-fmt, I'd be curious if you'd accept a patching adding support for reading the file from stdin and writing the output to stdout?
The use-case is for editor integration x)

feel free to send a patch similar to what is done here:

https://github.com/asottile/pyupgrade/blob/55754c9dbd14eade499d524c37272807c25b4d07/pyupgrade/_main.py#L865-L869
https://github.com/asottile/pyupgrade/blob/55754c9dbd14eade499d524c37272807c25b4d07/pyupgrade/_main.py#L892-L893

Started poking around between meetings and noticed that some fields are filled from adjacent files (readme, license and tox). I think for those cases we have two options:

skip them when input is stdin or
add some sort of --stdin-file-name or something like that to inform what's the file being formatted?

What do you think?
(oh, for the same motivation, I was also thinking of adding --exit-zero-even-if-changed, let me know if I should stop lol)

some fields are filled from adjacent files

hmmmm yeah that would kinda make this intractable wouldn't it :(

hmmmm yeah that would kinda make this intractable wouldn't it :(

Yeah after seeing that, I'm thinking of leaving it alone (which is option 3 not listed above 😁)


WIP - squashing trie data during block import

What was wrong?
We are storing state roots into the database for every step in the transaction. We only need to store the state roots from the end of the block.
How was it fixed?
Use the HexaryTries ability to squash changes, and join all the state changes in a block to one. We don't need to index into state roots from the middle of a block (or the middle of a transaction!), so we can drop them all, except the final one.
It seems natural to do a follow-up where we don't bother calculating the hashes along the way
Also, moved BaseDB closer to implementing the collections.abc.MutableMapping interface, by adding an optional default parameter. (HexaryTrie expects the db to meet that interface)
There are a few hacks in place to make this work without the PR getting a lot bigger. Probably the biggest follow-up todo item is to rethink the trie and account db, and how they fit together, finally getting to #507
Also, we're technically storing the state root at the end of the last transaction, and then again after the block finalization (eg~ paying the miner), but we'll clean that up later.
Cute Animal Picture

Are there any cases where we wouldn't want to use a squashing trie? If not, would it make sense to use it by default everywhere?

Are there any cases where we wouldn't want to use a squashing trie?

AFAICT, it all comes down to whether we want to access historical state. We don't need to access historical state in the middle of a transaction or block, but we do need to access historical state at the end of previous blocks.

If not, would it make sense to use it by default everywhere?

If you mean to use it without a context, I think this would be problematic. Squashing in a context means we can be sure to delete all the intermediate state without accidentally deleting any state we need for historical access. But in general, most places that we update the trie should probably be squashed.

:+1: opened #657

Closing in favor of #704

If not, would it make sense to use it by default everywhere?

I figured a different solution to this, in #704 -- prune the trie always, but add a layer that ignores deletes on the wrapped database at persist() time.
All my favorite parts of this PR have now been isolated into others. Closing


RuntimeWarning: invalid value encountered in arccos

This seems a little strange; presumably some calculation is ending up trying to take arccos of a number greater than 1 (or less than -1), which might indicate a more significant issue.
from datetime import datetime
from suncalc import get_times

date = datetime(2023, 5, 24)
lon = 0
lat = 51.5
get_times(date, lon, lat)

Running this (with suncalc 0.1.3 from PyPI) gives:
/<PATH>/suncalc-env/lib/python3.11/site-packages/suncalc/suncalc.py:202: RuntimeWarning: invalid value encountered in arccos
  return acos((sin(h) - sin(phi) * sin(d)) / (cos(phi) * cos(d)))

Hi !
Got the exact same error today when using these data:
lat = 48.862725, lng = 2.287592
Was thinking it was due to float precision, because lat = 48, lng = 2 doesn't log any error but is less precise than with my first data sample.

import suncalc

paris = lat, lon = 48.862725, 2.287592
dates = pd.DataFrame(index=pd.date_range("2022-01-01", "2022-12-31", freq="D"))
lons, lats = [lon] * len(dates), [lat] * len(dates)
times = suncalc.get_times(dates.index, lons, lats)
night = times["night"]

dates.iloc[night[night.isna()].index]
>> 2022-06-12
>> 2022-06-13
>> 2022-06-14
>> 2022-06-15
>> 2022-06-16
>> 2022-06-17
>> 2022-06-18
>> 2022-06-19
>> 2022-06-20
>> 2022-06-21
>> 2022-06-22
>> 2022-06-23
>> 2022-06-24
>> 2022-06-25
>> 2022-06-26
>> 2022-06-27
>> 2022-06-28
>> 2022-06-29
>> 2022-06-30

This coordinate is in Paris, nowhere near the polar circle. I.e., the sun sets and a calculation must be wrong somewhere.

A PR is welcome, but I'm not using this package myself. You might want to check in the JS implementation if they do something different or if the same error occurs there


[stable/ambassador] Set annotations on Deployment/DaemonSet

What this PR does / why we need it:
This is helpful for supporting Reloader
to enable triggered rolling restarts of Ambassador Pods.
Signed-off-by: Steve Huff<EMAIL_ADDRESS>@Flydiverny @kflynn @nbkrause
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @nbkrause

/assign
/ok-to-test

@Flydiverny thanks for the feedback!  i have made the changes you requested

/lgtm

@hakamadare Out of interest what kind of use case do you have where you need to roll the ambassador pods with reloader?

@Flydiverny i believe we’re getting bitten pretty hard by https://github.com/datawire/ambassador/issues/1478 - the indicators are that one or more of the Ambassador pods stops updating its Envoy config, and so after a deploy of our application, Ambassador still routes some amount of requests to the previous deployed version.  We have deployed Ambassador as a DaemonSet, and it can be annoying to delete each pod in turn and wait for it to be replaced, so as a convenience I’d like to be able to trigger Reloader to do it.


Prerelease USB-C version disables bluetooth

When I open the USB-C AirPods case, it disables bluetooth (like it says unavailable), then it re-enables itself and only after this, it shows the AirPods battery popup.
iPhone 12 Pro, 16.6b1

And I can't use it in this state because trying to connect to the airpods does the same effect

Here's my bluetoothd (iPhone 12 Pro, 16.6b1): https://drive.google.com/file/d/1Hxb3CvLTnotKGtjhBUd5TuFP3tp4fyPA/view?usp=sharing