Fix unstable vz driver network issue

This should hopefully fix the network freeze when using vz driver
https://github.com/lima-vm/lima/issues/1200#issuecomment-1327740698
I didn't add fix issue id's mainly to try and see if this really fixes or not.
What is happening, why this change ?
When we use file.FD(), whenever the file gets closed or garbage collected (finaliser closing file) the FD give before becomes invalid.
In our case, we use file.FD() for both client and server connections. Since these are DGRAM connection, if we try to read on a closed FD we won't get any error instead it get stuck indefinitely. This could be the reason for the freeze of vz network
Before this change, i verified that that file getting garbage collected in ~2mins. So logically FD would become invalid at some point.

Lets hold the merge.
Unfortunately this doesn't fix it :( I tried a long running instance in M1 and the freeze still happened

@AkihiroSuda
I have done a additional change to wrap gvisor-tap-vsock connection with UDPFileConn.go
In this we are calling setReadDeadline before every read. This will make sure if the UDP connection is not closed. Before trying to read. Reading on closed UDP connection will end up getting stuck in that read
We can merge these current set of changes if it looks good mainly to get more info when error happens again

I am testing in my M1 from yesterday, with both long running VM and as well as load testing network with multiple torrent download. Unfortunately freeze didn't till now.

Thanks

Unfortunately freeze didn't till now.

s/Unfortunately/Fortunately/ ? 🙂

s/Unfortunately/Fortunately/ ?

I was expecting it to freeze so unfortunate :D


Adding Tavily Search API as a tool.

Description
Adding Tavily Search API as a tool. added tavily-python library as a requirement.
Type of Change

[ ] New Loader/Tool

How Has This Been Tested?

[ ] Added new notebook (that tests end-to-end)

@rotemweiss57  could you take a look into linting and 1 comment


777 registry for vrf service on l2

Description
This PR implements Registry and L1 endpoint for L2 VRF consumers
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] If it is a core feature, I have added thorough tests.

Deployment

[x] Should publish npm package

@KelvinThai I am not sure if it related with your latest changes, but there are 2 failed tests. Could we try to fix them before merging the PR?
https://github.com/Bisonai/orakl/actions/runs/5892717796/job/15982632417

Yes, man. will check it today.

Test case updated, please review it again @bayram98


Font size cannot be changed, too small on high-dpi displays

Manjaro/Arch x86_64, splatmoji-git is r92.b8d14b4-1.
splatmoji text is small and difficult to read on laptop screens with very high resolutions. On my laptop I use a desktop environment-wide scaling of 2.0x and a font DPI of 192.
Example
Is there any chance of getting a --fontsize command-line flag? Or maybe code to obey the desktop environment's settings? (I believe KDE uses QT_SCREEN_SCALE_FACTOR. GTK seems to read GDK_SCALE.)
Or failing that, instructinos on how to modify the source to hardcode a larger font size.

Splatmoji uses rofi for the pop-up menu, and in my testing rofi does not behave any differently no matter my X DPI. But it is possible to explicitly pass the -dpi flag to rofi in your splatmoji.config!
Here is the relevant default line for splatmoji.config, but with an extra flag for DPI 90:
# ~/.config/splatmoji/splatmoji.config
rofi_command=rofi -dmenu -p '' -i -monitor -2 -dpi 90

It is also possible to use a subshell to derive that value dynamically, if that works better for you:
# ~/.config/splatmoji/splatmoji.config
rofi_command=rofi -dmenu -p '' -i -monitor -2 -dpi $(/some/command/to get dpi)

I will add something like this to the README/help output/whatev. Thanks for bringing the need to my attention!

Learned more at https://github.com/davatorium/rofi/issues/254
Rofi actually does look at the following X Resources rule
rofi.dpi: 90

My version of rofi still doesn't seem to honor other dpi rules though
So now you have that more X-centric option than providing an override in the config, if that suits you 🙂

For example, paraphrased from my ~/.Xresources:
! I set this in a conditional block to only apply to my hidpi laptop
#define DPI 90
Xft.dpi: DPI
rofi.dpi: DPI


Feature:  Install in local as user

Hello,
I just want to do a local install as user, it seems it is only to install system wide requiring rights
when I trigger build-aux/luke lukefile install all with a lukefile
package  = 'peagisub'
version  = '$USER'

defines  = {
   PACKAGE           = '"$package"',
   VERSION           = '"$version"',
   NDEBUG            = 1,
   _FORTIFY_SOURCE   = 2,
}

modules  = {
   ['peagisub']         = 'src/peagisub.lua',
}

it does a copy to /usr/local/share/lua/5.1/
luarocks-admin indicates in my case:
   Rocks trees in use:
      /home/pi/.luarocks  <--- I want here the instal or a user choosen one
      /usr/local

is this configurable?
EDIT: no as PREFIX seems tobe hardcoded in lib/luke/environment.lua
I mean a thing like, would be cool:
PREFIX="/home/pi/.luarocks" ./build-aux/luke lukefile install all
or better grab PREFIXES from luarocks-admin script

Solved , see the patch https://github.com/gvvaughan/luke/pull/6

Hi @sosie-js.  Thanks for the report, and for your PR#6.
You can already pass a value for PREFIX as an argument to the luke command line invocation.  This is typically how I integrate a luke-based build into a rockspec.  For example, see https://github.com/luaposix/luaposix/blob/master/luaposix-git-1.rockspec#L35-L38
If you are calling luke directly from the shell, or make or similar, then you would:
build-aux/luke "PREFIX=$HOME/.luarocks" all install

note: no need for leading ./ - when running a command in a subdirectory the command PATH is not searched
note 2: no need to name lukefile since that's the default
HTH!  Please close this issue and PR#6 if that resolves your problem, otherwise do add any additional questions here or in the PR.

in fact he script depend on what is set on your config file for rocktrees: string or table. Mine is for string yours is for table
to force to use yours a file config.lua with this need to be created in /etc/luarocks or in ~/.luarocks
--https://github.com/luarocks/luarocks/wiki/Config-file-format
rocks_trees = {
   {
       name= "user",
       root = home..[[/.luarocks]],
       bin_dir = home.."/.local/bin"
       --lib_dir = home.."/.local/lib/lua/5.1",
       --lua_dir = home.."/.local/share/lua/5.1
   },
   {
       name="system",
       root = [[/usr/local]],
       bin_dir = [["/usr/local/bin"]]
       --lib_dir = [["/usr/local/lua/5.1"]],
       --lua_dir = [["/usr/local/share/lua/5.1]]
   },
}
``


Python dependencies are incompatible with the new HA version

Region and Brand of car
US, Hyundai
Version of the integration
2.11.0
Describe the bug
I updated HA from 2023.3.1 -> 2023.7.3 and the component stopped working. See the same error on each HA restart.
Debug logs if an error occurred
2023-08-06 19:24:04.720 ERROR (MainThread) [homeassistant.loader] Unexpected exception importing component custom_components.kia_uvo
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/loader.py", line 813, in get_component
    ComponentProtocol, importlib.import_module(self.pkg_path)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1204, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1176, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1147, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/config/custom_components/kia_uvo/__init__.py", line 25, in <module>
    from .utils import DEFAULT_DISTANCE_UNIT_ARRAY, get_default_distance_unit
  File "/config/custom_components/kia_uvo/utils.py", line 13, in <module>
    from .KiaUvoApiEU import KiaUvoApiEU
  File "/config/custom_components/kia_uvo/KiaUvoApiEU.py", line 11, in <module>
    import push_receiver
  File "/usr/local/lib/python3.11/site-packages/push_receiver/__init__.py", line 1, in <module>
    from .push_receiver import *
  File "/usr/local/lib/python3.11/site-packages/push_receiver/push_receiver.py", line 11, in <module>
    from .mcs_pb2 import *
  File "/usr/local/lib/python3.11/site-packages/push_receiver/mcs_pb2.py", line 33, in <module>
    _descriptor.EnumValueDescriptor(
  File "/usr/local/lib/python3.11/site-packages/google/protobuf/descriptor.py", line 796, in __new__
    _message.Message._CheckCalledFromGeneratedFile()
TypeError: Descriptors cannot not be created directly.
If this call came from a _pb2.py file, your generated code is out of date and must be regenerated with protoc >= 3.19.0.
If you cannot immediately regenerate your protos, some other possible workarounds are:
 1. Downgrade the protobuf package to 3.20.x or lower.
 2. Set PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=python (but this will use pure-Python parsing and will be much slower).

More information: https://developers.google.com/protocol-buffers/docs/news/2022-05-06#python-updates
2023-08-06 19:24:04.732 ERROR (MainThread) [homeassistant.setup] Setup failed for custom integration kia_uvo: Unable to import component: Exception importing custom_components.kia_uvo
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/loader.py", line 813, in get_component
    ComponentProtocol, importlib.import_module(self.pkg_path)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1204, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1176, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1147, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/config/custom_components/kia_uvo/__init__.py", line 25, in <module>
    from .utils import DEFAULT_DISTANCE_UNIT_ARRAY, get_default_distance_unit
  File "/config/custom_components/kia_uvo/utils.py", line 13, in <module>
    from .KiaUvoApiEU import KiaUvoApiEU
  File "/config/custom_components/kia_uvo/KiaUvoApiEU.py", line 11, in <module>
    import push_receiver
  File "/usr/local/lib/python3.11/site-packages/push_receiver/__init__.py", line 1, in <module>
    from .push_receiver import *
  File "/usr/local/lib/python3.11/site-packages/push_receiver/push_receiver.py", line 11, in <module>
    from .mcs_pb2 import *
  File "/usr/local/lib/python3.11/site-packages/push_receiver/mcs_pb2.py", line 33, in <module>
    _descriptor.EnumValueDescriptor(
  File "/usr/local/lib/python3.11/site-packages/google/protobuf/descriptor.py", line 796, in __new__
    _message.Message._CheckCalledFromGeneratedFile()
TypeError: Descriptors cannot not be created directly.
If this call came from a _pb2.py file, your generated code is out of date and must be regenerated with protoc >= 3.19.0.
If you cannot immediately regenerate your protos, some other possible workarounds are:
 1. Downgrade the protobuf package to 3.20.x or lower.
 2. Set PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=python (but this will use pure-Python parsing and will be much slower).

More information: https://developers.google.com/protocol-buffers/docs/news/2022-05-06#python-updates

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

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/setup.py", line 215, in _async_setup_component
    component = integration.get_component()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/loader.py", line 821, in get_component
    raise ImportError(f"Exception importing {self.pkg_path}") from err
ImportError: Exception importing custom_components.kia_uvo

Additional context
I run HA as a container.

Push receiver was removed from our code some time ago.   Line 11 of the eu library doesn't contain this.
Something seems wrong with your system updating libraries.   Maybe try a remove and install.


WAIUA no longer shows rank with new update.

WAIUA no longer shows people's rank since the new episode 6 update.

The current steps to fix this are:

Navigate to %localappdata%/WAIUA/ValAPI using file manager.
Delete the version.txt file
You might also need to manually re-select your language in settings

The current steps to fix this are:

Navigate to %localappdata%/WAIUA/ValAPI using file manager.
Delete the version.txt file
You might also need to manually re-select your language in settings

Can confirm that this worked for me. Do you know what causes this issue to happen?

The current steps to fix this are:

Navigate to %localappdata%/WAIUA/ValAPI using file manager.
Delete the version.txt file
You might also need to manually re-select your language in settings

Can confirm that this worked for me. Do you know what causes this issue to happen?

It's because valorant-api.com had the wrong game version for a bit. I cache the version numbers but I might remove it to prevent more issues like this.


Update hard75Challenge.py

added support for role assignment on completion of 75 days.

@vedantmishra69 the TITLE could be made better.


ci: remove docker from build pipeline

Current dependencies on/for this PR:

main

PR #9   👈

This stack of pull requests is managed by Graphite.


Pre-trained Model

Hello There,
Question 1:
url = 'https://storage.googleapis.com/chakki/datasets/public/models.zip'
What is this model trained on for NER tagging?
Question 2:
Why is the trained model performing significantly better with the trained model, compared to when I train a model with Conll2003 data and evaluate on the test set?
Cheers,
TZ

Thank you for your question.
My answers are as follows:
Answer 1:
This model is based on Bidirectional LSTM-CRF.

Neural Architectures for Named Entity Recognition

This is trained by CoNLL2003 dataset.
Answer 2:
Because the published model is trained by CoNLLL2003 all(train + valid + test) datasets.
In anaGo 1.0.0, I released new trained dataset. This is trained by CoNLL2003 train + valid datasets. I think it is comparable to some papers score.

https://storage.googleapis.com/chakki/datasets/public/ner/models_en.zip

with best regards


rancher-compose results in "Check API key"

Rancher Version:
v1.2.0
Docker Version:
Docker version 1.12.3, build 6b644ec
OS and where are the hosts located? (cloud, bare metal, etc):
4.4.24-rancher on EC2
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Single node rancher. One host containing both server and agent/host. External DB on same host.

Creating an docker-compose.ymlfile containing :

version: '2'
services:
 Httpd:
   image: xx/drupal:test

RANCHER_URL="http://<MY IP>:8000" RANCHER_ACCESS_KEY="449E26A86E76F4B70109" RANCHER_SECRET_KEY="<SECRET KEY>" rancher-compose --debug up

Results:
DEBU[0000] Environment Context from file : map[]
DEBU[0000] Opening compose files: docker-compose.yml
DEBU[0000] Opening rancher-compose file: /Users/.../env-status/rancher-compose.yml
DEBU[0000] Not found: /Users/.../env-status/rancher-compose.yml
DEBU[0000] [0/1] [Httpd]: Adding
ERRO[0000] Failed to open project env-status: Can not create a stack, check API key [449E26A86E76F4B70109] for [http://<IP_ADDRESS>:8000]
FATA[0000] Failed to read project: Can not create a stack, check API key [449E26A86E76F4B70109] for [http://<IP_ADDRESS>:8000]

Expected:
That ranger-compose would be able to talk to the API by using an Account API key. But for some reason it is not able to do that.
Have looked at the comm and ranger-compose makes two GET requests; one to http://<MY IP>:8000/v2-beta and one to http://<MY IP>:8000/v2-beta/schemas.
When doing a ranger env I get an overview of my stacks as expected. So the Account API key is working for something..
rancher-compose -v returns rancher-compose version v0.12.0 and rancher -v returns rancher version v0.4.0.

Rancher CLI uses account API keys as you can manipulate any environment.
Rancher Compose uses environment API keys, which are hidden under the "advanced options" in API keys as you need to specify which environment you want to use within Rancher.
Rancher CLI also allows you to do rancher up which is the same as rancher-compose up.

Great. That fixed it. The error message could be more helpful about this. I made a change proposal to the doc to make this even more explicit. Thank you for the help :-)

Hey @blamh - Thanks!
Can you link to the PR/branch?

@justsml  https://github.com/rancher/rancher.github.io/pull/566

There is no Advanced Options under API keys???

guys i am using following built but when i click on account and then Api ^ keys there is no option for advance?
am i looking at wrong place?

Component
Version

Rancher
v2.2.3

User Interface
v2.2.41

Helm
v2.10.0-rancher10

Machine
v0.15.0-rancher6-1

This is a multiple year old issue for 1.x and is completely not relevant for  2.x.  There is no rancher-compose for 2.x.

Thanks for your reply I was wondering if you can  give me a guide to deploy
ELK stack. I'm new to rancher ...
On Sat, 1 Jun 2019, 7:05 pm Vincent Fiduccia<EMAIL_ADDRESS>wrote:

This is a multiple year old issue for 1.x and is completely not relevant
for 2.x. There is no rancher-compose for 2.x.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rancher/rancher/issues/6965?email_source=notifications&email_token=ALD76MXTHS5Y75MMFS6NAHTPYI3TZA5CNFSM4CY2RD72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWW4KFQ#issuecomment-497927446,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALD76MXIA4QNG64TTW2CJEDPYI3TZANCNFSM4CY2RD7Q
.


Use default asset roots on default.config.js

We need to cherry pick this one as well.
cc @ide

@facebook-github-bot shipit


glfwGetKeyScancode ambiguity

Quoting from Input guide:

You can query the scancode for any named key on the current platform with glfwGetKeyScancode.

Quoting from glfwGetKeyScancode function documentation:

If the key is GLFW_KEY_UNKNOWN or does not exist on the keyboard this method will return -1.

I'm confused.

If glfwGetKeyScancode returns -1, does that mean that the named key does not exist on the keyboard or on the platform? Either or? Both? How can I differentiate?
If glfwGetKeyScancode returns not -1, does that mean that the named key does exist on the keyboard or on the platform? On both?

The reason I'm bringing this up, other than the platform/keyboard ambiguity stated above, is that I got a non -1 scancode, from glfwGetKeyScancode, given a named key that does not exist on my keyboard. Specifically, they were the F13-F24 keys. Surprisingly, F25 returns -1, or should I say, as expected.
Digging a little bit further, turns out you can press the F13-F24 keys of a F1-F12 keyboard, just by using the Shift + scheme. Still I didn't manage to replicate that. Is that the reason why I don't get -1 on those high F keys?
I think it could be useful to know what keys the keyboard actually has, and not just wait for a non-existent key to be pressed. Of course, maybe I'm asking too much, and it's impossible to have such an information from the OS.
Going a little bit further, why the mouse does not have a scancode as well? Joystick? Is it just an only keyboard thing. possibly my ignorance starts shining now
GLFW 3.3.6
Windows 10
Keyboard Logitech K120

Thank you for reporting this! I had completely missed how vague and contradictory those sections were. 9959dc69cadd1f7790060846ffbc44e7da56fc56 is my attempt to improve them somewhat, in these and a few other ways.


Implement reset ready banner for participants list

Design: https://xd.adobe.com/view/31dc12cb-4503-47eb-8ffd-1d5f5a5147b9-648f/

so what happens when you close the banner, does it reappear whenever the ready state of a user changes?


Implement removal of orphan nodes

Refactors the storage of values into a separate KV store, indexed by hash(value)+key. As noted in #14 loses the benefit of deduplication, while enabling pruning of records.
Completes work from https://github.com/celestiaorg/smt/pull/24.
Revises work from https://github.com/celestiaorg/smt/pull/36.

In light of the above discussion, I'm favouring using the original PR https://github.com/celestiaorg/smt/pull/36 and changing the specification for the following reasons:

Implementing a path -> value store for faster gets, should actually be the responsibility of the caller, not the SMT, which should only be responsible for the tree itself. Hence, the idea of separating the state commitment library from the storage library. Adding this KV store, would cause them to be coupled.
Accepting the above, adding a new path -> value store just to implement pruning is wasteful, when we can just make a breaking change to the spec.
(See discussion above.) Without introducing a breaking change or a lot of extra complexity, it doesn't seem possible to allow people to optionally disable pruning, without forcing them to use BadgerDB (the only well-known key-value-version store), or implement their own versioning.

Adding a summary of some nuance discussed on Discord.
More important than breaking the specs (which is unimportant in the grant scheme of things) the real issue is that the current hash(value) -> value map, along with the key || hash(value) -> value map that replaces it in this PR (and the equivalent in #36), are redundant to the key -> value map that will be used in ADR-40 to allow for constant-time leaf reads. Redundant = space overhead that must be paid by all users of the library, not just users that want historical queries.
In other words, regardless of pruning orphaned nodes, this abstraction must be reworked from the current assumption that a single KV exists for both internal nodes and leaves to the assumption that two KVs exist, one for internal nodes and one for leaves. This second abstract coincidentally aligns exactly with the requirements for constant-time reads, and thus does not require any redundancy.
A suggested alternative would be to store key -> hash(value), then hash(value) -> value, adding a level of indirection. But this would then require reads to do two sequential disk accesses instead of one to get a leaf in constant time, which would be an enormous overhead.
Unfortunately, removing redundancy/this mandatory overhead means it becomes impossible to support historical queries, and pruning orphaned nodes must be enabled at all times.

This may be a silly question, but what about just removing Get value access entirely? The tree could just store its internal structure and value hashes, assuming only the responsibility of proof generation. That would completely avoid redundancy and leave value storage and versioning up to the caller.

This may be a silly question, but what about just removing Get value access entirely? The tree could just store its internal structure and value hashes, assuming only the responsibility of proof generation.

This will completely change how this library works. Right now it's a self-contained, general purpose Merkle Tree. Removing Get is a fundamental change, meaning that library can't be used "alone", without external store for values.

That would completely avoid redundancy and leave value storage and versioning up to the caller.

This will only push the problem up to the caller ;)
I agree with @adlerjohn that two KV stores should be used in library (which of course can be implemented as just prefix-buckets in single database).

This will only push the problem up to the caller ;)

Well, that's the idea. The library user can optimize for whatever they need, and the library doesn't have to try to solve for all cases. You could still provide a self-contained tree map that includes value access via one of the methods discussed. Anyway, just a thought.

I agree that given a trade-off between duplicating data storage, remove the Get functionality, and removing the ability to access old state roots, the latter is the least undesirable.
If there's no other options, I'm in favour of having two KV stores, where one of the KV stores is a path -> value store, and dropping the functionality of accessing old state roots.

What is the timeline for the proposed changes. I would like to coordinate an audit of the library soonish.

@marbar3778 changes are pushed :) I will tag the owners for re-review.

The core PR LGMT, however there is some other things that should be do as a result of this PR:

Update the example in the readme.
Remove the API to get old roots.
Update Get so that it reads from the path->value store directly, instead of traversing the tree.

I'll handle these in follow-up PRs so as to unblock @roysc. Good work on this PR!


Enter repo root

What does this PR do?
Invoke-BuildScript -BuildOutOfSource 0 sets CWD to the root of the datadog-agent repository
Motivation
Allow script to be called without requiring CWD to be the root of the repo, as the .bat files did
Describe how you validated your changes

script should work when run like
docker exec -it agentbuild powershell -c "cd C:\Windows; C:\mnt\tasks\winbuildscripts\Build-AgentPackages.ps1"

(or by a container with a different CWD)
Possible Drawbacks / Trade-offs
Additional Notes

/merge


agent should verify docker connectivity before accepting builds

before an agent starts accepting builds we should

ping the docker daemon (with a backoff) to ensure connectivity and
ping the agent server (with a backoff) and exit with an error if N ping attempts fail

closed by https://github.com/drone/drone/commit/daa071d497c33cdb1b15e0a6d52a74dd8fc416fd


Is there a case of Objective-C?

Is there a case of Objective-C?

@SUWW all of the APIs in the SDK are fully compatible with Objective-C. We do not plan on providing Objective-C examples here in the new future, but you can translate the Swift examples and they will work.


Relative load in test files for code under test

Would a simpler (merge "..." *load-pathname*) be sufficient?

Unfortunately if merge-pathnames is passed NIL as its second argument you get an error, and *load-pathname* is nil unless loading is in progress. To make your suggestion friendly to interactive evaluation, we'd have to wrap it in an (eval-when (:load-toplevel :compile-toplevel) ...) form, which I try to avoid where possible.

Sorry I didn't just test it out myself before asking.
I don't like, what I feel is, the ugliness of the expression. I'm thinking of the experience of the person new to Common Lisp coming to the track - will this expression leave the person with the feeling "Wow - Common Lisp is really complicated" or will they just ignore it as some ritualistic boilerplate?  (But in the latter case - will they start copying it everywhere?
I guess the ugliness was already started with the feature check...
As for utility function use in other tracks - I'll check and repot back.

@spacebat can you explain your workflow - because in my testing I'm not sure if I'm seeing the issue. Perhaps because the way I use my environment (SBCL & SLIME) is different.
From the issue #126 you mention that you set *default-pathname-defaults* to some location. In my checking no matter what my *default-pathname-defaults* is set to if I set my working directory in SLIME (via cd) it sets *default-pathname-defaults*. I don't normally set the defaults so either I'm starting SLIME in the correct directory or changing to the correct directory.
I want to keep the code that exercism submitters see as simple and clear as possible; mostly they are not experienced Common Lisp users. You appear to be an experienced Lisp user (thanks for submitting to the track!) and I want to be careful that we don't make changes to help the expert at the detriment of the beginner.

@verdammelt I just run M-x slime-load-file and use ido to browse to the test file (which might be the default if I have run the command with the test file in the current buffer), which then fails because I haven't run slime-cd first. My goal is to make the test file just work however it is loaded or evaluated, but I understand your concern - its more code and doesn't look approachable for new users. If we can't easily create an xlisp library for this sort of utility, perhaps the instructions should just mention the need to need to run cd in slime or start lisp in the exercise directory - I haven't found that mentioned anywhere but could have missed it.
Thanks for thinking about it though :)

I think if we created some xlisp library with a helper function for loading the 'production' code - then we'd still have the problem of how to load that library :)
Unless we push it to quicklisp - but that seems over-the-top.
Let me review the README - maybe we can put some documentation about SLIME usage.

Lets update the docs then. I'll close this PR.


Login error (1013): request time is invalid

With the last integration have this problem, When i try to login with HASS

Same error also


chore(testcontainers): export waitForCondition

What this PR does / why we need it:
Which issue(s) does this PR fixes?:

quick fix for #1140
Additional comments?:

@ivan-zynesis fyi, I think you exported this as well


Set minimum Python version to 3.6

Adds a shiny  that will get updated to the right Python versions once a new release is made.
Closes #654 and closes #645

👍

On second thoughts I've dropped the badge in favor of a 3.6+ in the README.
(Python versions ought to always be forward compatible, so we shouldn't really have to rely on markers to indicate eg. Python 3.8 support.)


Move website from gh-pages branch to master under website/ folder

What type of PR is this?
Improve website build and deployment process
What this PR does / why we need it:

Currently, we are using "Deploy from a branch" as source for building and deploying the website to GitHub Pages.
There's also the workflow "Deploy GH Pages" at .github/workflows/pages.yml which is responsible fo copying files from master to gh-pages branch.

Which issue(s) this PR fixes:
Fixes #3715
This PR includes the following:

Moving the website/ source code from the gh-pages branch to master under website/ folder.
Removing useless website/CNAME file (This file won't work with "GitHub Actions" as source, refer to the docs for more details)
Removing unused website/.circleci/ folder (CircleCI is not being used as you can see here)
Removing unneeded website/.mergify.yml file (Not needed because we won't use a branch anymore)
Renaming website/README.txt to website/README.md and convert to Markdown format (this would allow GitHub to display a formatted page)
Renaming docs/docs-readme.md to docs/README.md (This would allow GitHub to display a formatted page)
Update Jekyll _config.yml to exclude README.md, copy.sh, Gemfile, Gemfile.lock from output (For example, we won't have this file anymore https://armadaproject.io/Gemfile)

Special notes for your reviewer:
Once this PR is merged, we would also need to:

[ ] Set "Source" to "GitHub Actions" in "Settings > Pages > Build and deployment > Source".
[ ] Set "Custom domain" to armadaproject.io in "Settings > Pages > Build and deployment > Custom domain".
[ ] Get rid of the gh-pages branch (Optional).

Replaced by #3850


Add negate kernels (#4488)

Which issue does this PR close?

Closes #4488
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Will add interval tests shortly, realise I forgot

Will add interval tests shortly, realise I forgot

Should I wait to review this PR until you wrote those tests? Or did you add them to https://github.com/apache/arrow-rs/pull/4493 ?


live server install issue

I'm using laravel 5.4 version and followed the steps to install it on live server folder example: htttp://example.com/laravelapp
But when I'm trying to open admin URL as per documentation, it is not working.

Here you can see screenshot. Can you let me know how to access admin.

I also debug it but when trying to login getting this error:

in a live serve you need put all folder in the root / and in the public_html you need put the public folder, and change the url in the index.php

That's unrelated @coke00

@nishasharma18, if you're getting NotFoundHttpException when trying to get to the login page, then I question if your routes are set up correctly.  Can you show the output of artisan route:list?

Here is my index.php file which i moved from public folder to root folder:

 */

/*
|--------------------------------------------------------------------------
| Register The Auto Loader
|--------------------------------------------------------------------------
|
| Composer provides a convenient, automatically generated class loader for
| our application. We just need to utilize it! We'll simply require it
| into the script here so that we don't have to worry about manual
| loading any of our classes later on. It feels great to relax.
|
*/

require __DIR__.'/bootstrap/autoload.php';

/*
|--------------------------------------------------------------------------
| Turn On The Lights
|--------------------------------------------------------------------------
|
| We need to illuminate PHP development, so let us turn on the lights.
| This bootstraps the framework and gets it ready for use, then it
| will load up this application so that we can run it and send
| the responses back to the browser and delight our users.
|
*/

$app = require_once __DIR__.'/bootstrap/app.php';

/*
|--------------------------------------------------------------------------
| Run The Application
|--------------------------------------------------------------------------
|
| Once we have the application, we can handle the incoming request
| through the kernel, and send the associated response back to
| the client's browser allowing them to enjoy the creative
| and wonderful application we have prepared for them.
|
*/

$kernel = $app->make(Illuminate\Contracts\Http\Kernel::class);

$response = $kernel->handle(
    $request = Illuminate\Http\Request::capture()
);

$response->send();

$kernel->terminate($request, $response);

I'm getting this error:

Can you let  me know where to add permission

add the voyger routes in web.php

Wait.... you moved the index.php file from public to root.......? No, don't do that...

Okay then let me know how i can access it

Set up a vhost to point at your public directory.  Moving the index file means the rest of your laravel install is under the web root, including your .env file...

I'm using bluehost shared hosting and my laravel app is under sub folder like: http://example.com/laravelapp
So how can i setup vhost to point to public folder

That really doesn't have anything to do with this package.  Please ask for help in a forum that's more generic to laravel, such as laracasts forums or larachat slack group.

Thanks for the information.

have you checked the  .env file  configuration


Make Avalonia.Win32.Input.KeyInterop public

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

@grokys Are we fine with exposing this in our public API? I can see the value for embedding scenarios where everybody ends up copy pasting this dictionary anyway.

We are trying to not expose internals of platform specific code too much.
This class might be big, but totally not a problem to adapt in developer's code.


Get pathname instead of full URL

Is there any way to only retrieve the path of the link?
Instead of
console.log(url.url())
// http://mydomain.com/my/path/name

I want
console.log(url.path())
// /my/path/name

Not sure if there is already a built in way to do this.
Or is it possible to disable baseUrl, baseDomain, etc from the generated JS file?
I basically only need everything after the domain.
Thanks

Yes, you can set absolute to false
https://github.com/tightenco/ziggy/blob/master/src/js/route.js#L128


Added 3i Infotech Company

You forgot to change count.

Just a Second.

Count increase by 1

Thank you.

Next time don't forget to add description.

Thank you for telling this.. Hope I will never forget.


Update python39

The example needed some updates because it was only supporting Python 2.7.
The example now works with Python 3.9 and uses the ezc3d library to read the c3d files instead of the depreacted btk library.
The installation needs to be done with conda to make sure that users use a specific environment when running it.

[ ] Tested
[x] Documentation updated

It works for me, but I had some issues with conda not being initiated at command line and I could not activate python-example.

Andrey tried and it worked for him. I could not find a reason why you had this issue. Since you made it worked and Andrey made it work as well, then I will close this ticket.


The requested record was not found or you do not have sufficient permissions to view it.

I keep getting this error "The requested record was not found or you do not have sufficient permissions to view it." I already re do the app several times, I change the SP guid. thanks in advance

@gaarabyakuya - you need to place "Sharepoint site record Id" from CRM not sharepoint site GUID . I have same error and resolved it by placing sharepoint site record Id.
Let me know if it works.

@anupamkhasia Thank you, that indeed solve that error. Could tell me how did you map the fields? because the control keeps saying loading and nothing happens. I tried any combination that I can think of but no luck.

@gaarabyakuya - I am also still trying to load control as it looks there are additional sharepoint permissions needs to be given .

@anupamkhasia  have you find any solution for file loading issue? For me also it keeps loading nothing happen...

@ThoshanTharanga - Yes , we have to configure sharepoint site and than you will be able to use this component. Let me know if you are facing any other errors.

@anupamkhasia  Thanks for the reply.... Could I know what kind of configuration needs to be done there..... seems it is giving error when retrieving the sharepoint folder on onFileDropped funtion.

SharePoint setup mean, is it kind of setting up permission..? Thanks :)

@ThoshanTharanga - I mean to have dynamics crm and sharepoint integration configuration we need to do. please refer to following link
https://www.crmsoftwareblog.com/2020/02/how-to-setup-a-dynamics-365-integration-with-sharepoint/
Let me know if you have done this.
Thanks
Anupam

@anupamkhasia Yeah that integration part is done and selected the based on entity as Account. Still it keep loading... and can see the console error which I have mentioned previously.
Thanks,
Thoshan

@ThoshanTharanga - can you give me screenshot of PCF control configuration you have setup ?

@anupamkhasia Sure... for the client id I have given the client id of the app which I have registered in Azure App Directory and for the site id I have given the "sharepointsiteid" field from api/data/v9.0/sharepointsites response.

@ThoshanTharanga - not sure and can not determine how to look into your error but here is my configuration . please try this and let know.

Hi @ThoshanTharanga :- Thanks ! The code change works and we are able to upload documents however its does not pick the Title before the folder name and creates folder with just ID(2DA820A3E83BEC118C620012476F46B0).
We get the below console error for GET.
https://xyz.sharepoint.com/sites/abc/_api/web/getFolderByServerRelativePath(decodedUrl='abcd%2F_2DA820A3E83BEC118C620012476F46B0')
For a normal documents control, it does pickup Title before the ID and creates it properly.

HI @kshitijkhanna . Yes, this error is coming because sometimes the input we passing as a parameter in getSharePointFolderName() funtion is undefined in SharePointUploadControlIVM.ts file. Could you try by adding below change to it?
if this fix work for you, I will create a PR to that change.
Thank you :)

Hi, I am also having the same issue and tried this code. unfortunately it doesn't work. do you have any idea why? I am still getting the same 'replace' error.

Did anybody solved all the errors? :)

Hello, please can any one tell me how i can import the solution i downloaded the repository and then i run
npm install and then npm fix after that pac pcf push command but it didnt work. I also tried to build the project butit gave me the same results.
can anyone help me please with the steps after downloading the repo.
thanks a lot
Best regards


How to convert Mat from Bitmap with use OpencvSharp.Extensions in .net core 2.1 ?

Summary of your issue
I can not acces to OpencvSharp.Extensions namespace in .net core 2.1.1
Environment
What did you do when you faced the problem?
I need to convert Mat from Bitmap.
I downloaded 3.4.1 from nuget but can not access to OpencvSharp.Extensions
Example code:
paste your core code

Output:
paste your output

What did you intend to be?

Thanks. This will fix by #566.


Fix hashing with unicode fails.

Fix TypeError: Unicode-objects must be encoded before hashing
found with python 3.6dev

Fixed.


Remove apt-get install cargo in build-travis.sh

Once @lukealonso has updated the docker image, this will no longer be necessary.

Rolled into #582


fix(update-vscode): add check/docs for git-subtree

Closes #3084.

@jsjoeio me too, very interesting


Integrate with Helm

Application defined in Kedge should be able to refer Helm Charts from Kedge files. And kedge CLI should be able to use it and delegate it to helm and deploy the application and the chart associated with it.
Example: A Python web application uses PostgreSQL as database. The developer should be able to refer an existing Postgres Helm chart from the kedge file that defines the application.

Agreed, we should have something like this.
A few notes regarding this:
Chart shouldn't be tied to any given Kedge app. That means it can't be done in the way we are doing secrets or config maps.
It should be done in the way that allows deploying everything together using one kedge call.
Charts can have parameters. We should be able to refer a Chart and also define what parameters has to be set.
Maybe separate file defining what chart to deploy?
This is just my brain dump on this issue. Later I will try to make this more concrete defintion.

Just saying:
Even Helm supports plugin model https://github.com/kubernetes/helm/blob/master/docs/plugins.md
I am not sure if we wanna leverage this one, but it is out there.

Is there an issue for generating Helm Charts ? Because this issue talks about only using Helm charts from kedge!
@pradeepto and what is the priority of this ?

Not for 0.3.0 or 0.4.0.

So there is issue about generation of Helm charts https://github.com/kedgeproject/kedge/issues/272


DIPPR 801 2019?

it is posible to add the DIPPR 801 2019 to the list of available databases?, i have access and  can produce the corresponding TSVs

Hi Andrés,
It is definitely not possible. That is a proprietary database, to my knowledge any license to use it would include an EULA forbidding replicating it unless given an exemption. Sometimes they publish their data publicly in to books (like Perry's), and sometimes in research papers by authors with exceptions. It is only from those legitimate sources that data will be included into chemicals, barring the DIPPR deciding to make their database non-commercial :)
Sincerely,
Caleb


Problem bad gateway 502

I did all the steps ok, but in the finish i close and open the server and in the externa ip see "error 502 bad gateway) and the down is nginx <IP_ADDRESS>. A no clue to solve this?

Bad gateway usually means nginx can't talk to nodejs properly. Did your meteor server start up properly? Try checking the pm2 logs.

i can't see pm2 logs, and i don't have a help with google, any tutorial for this you know?

i can't start startup.sh, but i haven't an error in any place.

Try something like sudo pm2 logs. See for more documentation: http://pm2.keymetrics.io/docs/usage/log-management/


Logs during process

It seems as the sdk logs to the console and I see no option to turn it off.
(I'm using the git version and not the released one)

Yes you are right, I have left some println!s around. I will try and clean them up ASAP.
Sorry!

Thanks, that was fast :)

Let's hope is enough! Please reopen the issue if this keeps happening!


Fails to build on GHC 7.8.*

This is an FTP breakage issue. In <IP_ADDRESS>, module Data,OrdPSQ.Internal specifies deriving Traversable, but Traversable is not in scope. Also, does the DerivingTraversable extension even exist in GHC 7.8.*?

Yes, according to the docs for GHC 7.8.4 DerivingTraversable is already supported there. So it's only the FTP breakage problem.

I'm seeing this with one of my packages as well.

Released as psqueues-<IP_ADDRESS>

Thanks @erikd and @jaspervdj !


[bugfix]modify the typo

Signed-off-by: yunfeiyangbuaa<EMAIL_ADDRESS>

Type of change

Bug fix
modify the typo
New feature
Improvement (improvement to code, performance, etc)
Test update
Documentation update

Description

Additional details

Related issues

please review this pr  thanks!

please review this pr  thanks!


[LA-176][Logs] Add Nginx integration saved views

What does this PR do?

Add Nginx integration saved views

Motivation
First integration saved views ever added in HISTORY 👏
Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Pic or didnt happen

@hithwen Here is a screenshot of our Staging environment with the saved views installed:  https://share.getcloudapp.com/DOu8wXAJ


gen_l10n code for Number, Data and Currency should support overrides

Use case
Building on the excellent work in #47008
Hardcoding a formatter to a locale is too restrictive for apps that deal with single locales, but multiple currency values. Just because my locale is "en_US", doesn't mean that I want my currencies formatted to en_US. Perhaps my invoice contains amounts in Canadian dollars or Euros in addition to USD.
Proposal
{
  "@@locale": "en",
  "autopayAmount": "Rent amount: {amount}",
  "@autopayAmount": {
    "placeholders": {
      "amount": {
        "type": "double",
        "format": "currency",
        "example": "$1,250.00",
        "description": "The periodic rent amount",
        "optionalParameters": {
          "decimalDigits": 2,
          "name": "USD",
          "symbol": "$"
        }
      }
    }
  }

This currently generates code like this
  String autopayAmount(double amount) {
    final intl.NumberFormat amountNumberFormat = intl.NumberFormat.currency(
      locale: localeName,
      decimalDigits: 2,
      name: 'USD',
      symbol: '\$'
    );
    final String amountString = amountNumberFormat.format(amount);

    return '$amountString';
  }

Proposal
Keep the same pattern as today, but generate a function with placeholder overrides.
For example, Currency support would generate like so
ARB Context
{
  "@@locale": "en",
  "autopayAmount": "Rent amount: {amount}",
  "@autopayAmount": {
    "placeholders": {
      "amount": {
        "type": "double",
        "format": "currency",
        "example": "$1,000.00",
        "description": "The periodic rent amount",
        "optionalParameters": {
          "name": "USD",
        }
      }
    }
  },

Would generate...
  String autopayPaymentAmountRentAmount(
    double amount, {
    String? locale,
    int? decimalDigits,
    String? name,
    String? symbol,
    String? customPattern,
  }) {
    final intl.NumberFormat amountNumberFormat = intl.NumberFormat.currency(
      locale: locale ?? localeName,
      decimalDigits: decimalDigits,
      name: name ?? 'USD',
      symbol: symbol,
      customPattern: customPattern,
    );
    final String amountString = amountNumberFormat.format(amount);

    return 'Rent amount: $amountString';
  }

This will allow the caller to override placeholder values using properties from their currency object.
var currency = Currency(name: 'BHD', value: 36);
MyLocalizations.of(context).autopayAmount(currency.value, name: currency.name),

Results in BHD36.000

@thkim1011

The proper way to support this would be through implementing the message format syntax for formatting numbers via number skeletons. Unfortunately, I won't have the bandwidth to work on this, so I'm unassigning myself.


Add tests for CLI/REPL behavior

We need unit tests for the CLI/REPL behavior to make sure that we won't mess it up by accident.
The tests should contemplate simple use cases for the commands and their arguments.
To consider: https://docs.rs/trycmd/latest/trycmd/

We already have

tests/lurk-cli-tests.rs
tests/lurk-lib-tests.rs


Stop assuming that all transparent addresses are on mainnet

Currently, From<Script> for Address assumes that all addresses are on mainnet.
But Zebra needs to support mainnet and testnet.
We should delete From<Script> for Address, and use the From<ScriptAndNetwork> for Address in PR #1170.
We should also check for any other places in Zebra where we've assumed Mainnet.

Basically, given a script hash alone (as lock_script, the one we get from transaction outputs) it is not possible to know to which network it belongs, so i had to create a new type ScriptForNetwork and implement impl From<ScriptForNetwork> for Address to do the conversion.
Originally posted by @oxarbitrage in https://github.com/ZcashFoundation/zebra/pull/1170#discussion_r508454881

Blocked on #1170
Do we need to fix this bug in the first alpha release, or can it wait until later?

Blocked on #1170
Do we need to fix this bug in the first alpha release, or can it wait until later?

I think we can get away with not fixing this for the first alpha, but probably not any betas and certainly not the first major/stable release.

It was easier than I thought, see #1191.


Some mobs going round in circles

Ever since https://github.com/tenplus1/mobs_redo/commit/4bcfa6b802fe389535dbc62522df9dba37e7c467 , some mobs from NSSM circle round the player instead of coming straight for them
Steps to reproduce:

make new world in default game
activate nssm, mobs_redo (any version from 7th January 2017 - previous versions are OK)
start new world
set time 0 (easy mob to reproduce this with needs darkness)
spawnentity nssm:signosigno and walk back from it
see that it runs around in circles, very gradually getting nearer the player, circle uses as radius its distance from player and player as central point, over time gradually reducing the radius

This only affects a subset of mobs from nssm, consistently. Removing the rotation parameter from signosigno (commenting it out) does not fix the issue.
I'm not yet clued up enough on yaw calculations to suggest anything, just looking around L1778 (master 19th Jan) is where I can seem to affect the movement -- changing it to:
yaw = (atan2(vec.z, vec.x) - pi / 2) - self.rotate

(added rotate subtraction) restores the correct directional movement (comes straight to player) but the mesh is now rotated to the side.

Check today's build to see if it persists, if so then I'll regress the codebase.

3ba7def - yes that's the on I am testing with. just tried after a pull on master - same circles...

Okies, try the pull I just upped, seems to work fine here...

Confirmed fixed... going to study that and maybe finally get this yaw stuff understooded.... thanks.


pfe-card -- fix storybook issue with incorrect attribute

What has changed and why
In Storybook, the incorrect attribute of color is shown, instead of pfe-color

https://patternfly.github.io/patternfly-elements/demo/
Testing instructions
Be sure to include detailed instructions on how your update can be tested by another developer.

npm run storybook
view card, ensure pfe-color is shown
Ensure color property changes still work with the storybook widget

Ready-for-merge Checklist

[x] Expected files: all files in this pull request are related to one feature request or issue (no stragglers)?
[ ] Did browser testing pass?
[ ] Did you update or add any necessary documentation (README.md, WHY.md, etc.)?
[ ] Was this feature demo'd and the design review approved?
[ ] Did you update the CHANGELOG.md file with a summary of this update?

Be sure to share your updates with the<EMAIL_ADDRESS>mailing list!

@castastrophe I updated this branch w/ your suggestions (thanks!) & the latest from master. Ready to merge?

@starryeyez024 Needs a changelog


Coalesce documentation links

Combine the various documentation links into a single section, eliminating duplicates and a broken link.

Thank you!

Sure. I just submitted a follow-up PR - a tweak I missed somehow the first time.


feat: add evmSlotValue strategy support

Summary
Depends on: https://github.com/snapshot-labs/sx.js/pull/243
Depends on: https://github.com/snapshot-labs/mana/pull/125
Depends on: https://github.com/snapshot-labs/sx-api/pull/206
This PR adds support for evmSlotValue strategy.
Closes: https://github.com/snapshot-labs/sx-ui/issues/827
How to test
Set up all dependencies first and make sure you are using local mana and sx-api. Check mana/sx-api test plans for more details. There is lots of dependencies so it's a bit cumbersome to set up right now.

Create new space on Starknet (goerli) - mainnet is not supported by Herodotus using EVM Slot value strategy. You can use this token (0xd96844c9b21cb6ccf2c236257c7fc703e43ba071 + slot index 0 as you can mint it yourself).
Create new proposal.
Check mana logs for internalId and webhook URL (will be automated on deployment).
Check https://api.herodotus.cloud/batch-query-status?apiKey=API_KEY&batchQueryId=INTERNAL_ID_RETURNED_BY_MANA until status is DONE.
Open webhook URL.
Refresh page, you should have VP.
You should be able to vote.

Can we remove EVM slot proof option in the proposal validation section? This is not something we can support.

When I open the proposal page a request is failing to Herodotus API (timestamp and accumulates_chain are wrong):
https://ds-indexer.api.herodotus.cloud/binsearch-path?timestamp=0&deployed_on_chain=SN_GOERLI&accumulates_chain=undefined
When I cast a vote the request fail with this endpoint (timestamp is correct now but not accumulates_chain)
https://ds-indexer.api.herodotus.cloud/binsearch-path?timestamp=1702408665&deployed_on_chain=SN_GOERLI&accumulates_chain=undefined
I can see the message to sign doesn't include proof

When I hardcode accumulates_chain=5 when I cast a vote the request to Herodotus API works but I still not see any proof or extra data in the message to sign, UserVotingStrategies is empty
When I confirm the vote, request to Mana fail with unauthorized error message with this logs:
 40: Contract error: {"revert_error":"Execution error at transaction index 0: Error in the called contract (0x06abd599ab530c5b3bc603111bdd20d77890db330402dc870fc9866f50ed6d2a):\nError at pc=0:15647:\nGot an exception while executing a hint.\nCairo traceback (most recent call last):\nUnknown location (pc=0:233)\nUnknown location (pc=0:5191)\nUnknown location (pc=0:11307)\n\nError in the called contract (0x048b33fe56e9b9354d4278ffdd5f6d546b13aa3d8c33149db2e2e2fdb48a369e):\nError at pc=0:2855:\nGot an exception while executing a hint: Custom Hint Error: Execution failed. Failure reason: 0x5573657220686173206e6f20766f74696e6720706f776572 ('User has no voting power').\nCairo traceback (most recent call last):\nUnknown location (pc=0:554)\nUnknown location (pc=0:1844)\n"}

Also I can't see my voting power, it show 0 in proposals or proposal page.

Is it possible to get the L1 block number for the contract directly (https://testnet.starkscan.co/contract/0x02ee57d848297bc7dfc8675111b9aa3bd3085e4038e475250770afe303b772af#read-write-contract) instead of calling Herodotus API?

Is it possible to get the L1 block number for the contract directly (https://testnet.starkscan.co/contract/0x02ee57d848297bc7dfc8675111b9aa3bd3085e4038e475250770afe303b772af#read-write-contract) instead of calling Herodotus API?

We would need to pass tree to make this contract all which requires request to Herodotus (but this requests already gives us block number).

Is the tree related to block number or you mean the storage proof?

Binary tree here is something Herodotus specific: https://docs.herodotus.dev/herodotus-docs/protocol-design/timestamp-to-block-number-mapper#binary-search

To get current voting power on "proposals" page you can query storage from a L1 node with something like this: https://gist.github.com/bonustrack/bc5c770ba23b0d0026fbf2c9bd966bc8 with latest as blockNum

I was able to cast a vote 🎉

http://localhost:8080/#/sn-tn:0x039c7d023cfa2115b27b01d727820b6a2f13a7eadf04709cfb2a598fec41fe90/proposal/1/votes

Voting power on proposal page working now. It just not visible on proposals page, this can be added later in another PR if you prefer.

Found these 2 issue in the Mana logs, the 2nd trigger a restart of the server

And voting power on proposals page works well now

Yup, debugging it right now.

Looks like webhook doesn't work like we expect it to work. It just forwards the response for the request we make (right away), not when request is DONE.
I guess we can disable webhooks for now on mana and merge this PR so we can interact with it (kind of manually) via mana logs, but we will need to handle that webhook manually.

Sound good

Looks like something wrong with sx-api too

It's stuck in a loop

Fixed by re-deploy, we might need to increase delay between starting indexer as it seems that 1 minute is not enough.
Looks like Herodotus API is down.

Looks like API is back up.

When I manually trigger Herodotus it restart Mana server here is the error:


Search ingest/delete API notifications

Setup (Slack) notifications in case the search ingest or delete API calls done in the corresponding GitHubAction fail or have not the expected result.

Notifications will be posted in https://ncchat.slack.com/archives/C04Q369V3JM


Allow packages to be pinned

Would you consider allowing pinning of packages? I have been using brew bundle for a long time, but I am in a situation that I have to use an older version of a package. So, either allowing to specify a particular version of a package and having it pinned or specifying a specific URL for a formula (like https://raw.githubusercontent.com/Homebrew/homebrew-core/300531275e320640923614a3b150b8a8f8677004/Formula/asdf.rb) - will automatically pin it. Need this asap - brew bundle becomes unusable for me till the formula author fixes their issue.

Homebrew Bundle pins only taps, not formulae. Homebrew supports the installation of only the latest version of a formula.
If you need a specific version of a formula, you can use brew extract to put that formula in a tap and then pin that tap with brew tap-pin and adding tap: true to that tap's line in Brewfile. Pinning a tap prioritizes its formulae over homebrew-core for formula names are supplied by the user.
See also #439.


Update libmbus

Please merge @lategoodbye s excellent work back into the main repository

@arska
Unfortunately it's not so easy. There has been a discussion about some commits (changes in serial handling) and Robert had some concerns about that. I want to respect that.
Another problem is that i don't have a real test environment anymore.
Here is my suggestion:
Could you please write down the hashes of important changes from my repository here and i will create a clean pull request and merge it.

I think it is a pity to split the (little) attention this project receives into different repositories. When I have a problem with a new device (about once a year) it is painful to merge and the time used for that is gone for improvements, new test cases, etc :(
From looking at #67 I don't see any massive conflicts of opinions, to resolve that I propose to leave the default as-is and add an user configurable option (e.g. command line flag) to "use alternative serial port configuration" for platforms where that is helpful.
@lategoodbye (and @rscada as well of course) I can give you access to my personal testbed (raspi, usb/serial, 4 m-bus slaves) and I have access to ~10 installations, please send me a ssh public key via email.
Best regards,
Aarno

Please go ahead and sync up this repo with the latest from @lategoodbye
Unfortunately I don't have time to work on this project anymore (not even working in this industry any more). I think @lategoodbye has done a lot of great work and I think too it would be a pity if there is a split between the repos, as that only creates a confusing situation for existing and future users that might want to pick up this project for further development.
I think the best way forward would be to sync this repo with @lategoodbye latest repo, even if it cannot be fully tested (maybe @arska can help with that?)

@arska I hope it's okay for you if i create a clean pull request to merge my change into this repository. I can't merge this pull request because a lot of commits contains more changes than expected from the log. Also the author of a commit should be kept.
The reason of forking is to play with new feature. Forking is not the problem, it is the missing clean pull request to merge the results back.

@lategoodbye absolutely. I'll open a new pull request with my two commits. Do you have time to work on this today ?
I agree on the missing clean pull request, although it might motivate contributions if the project doesn't look abandoned :)
Cheers,
Aarno


Option to preserve comments

I noticed this in the README:

HTML comments are not preserved.

Is there a particular reason why this is the case?  It seems to me like it would be easy to add an option and use oncomment() in htmlparser2, and I wouldn't mind implementing it myself, but I was wondering if you had some behavioral or security motivation for not including this feature.
Thanks

The feature would be welcome as long as there are unit tests and the
default behavior stays the same (no comments).
On Thu, Jan 8, 2015 at 11:55 PM, Calvin Montgomery<EMAIL_ADDRESS>
wrote:

I noticed this in the README:
HTML comments are not preserved.
Is there a particular reason why this is the case? It seems to me like it
would be easy to add with oncomment() in htmlparser2, and I wouldn't mind
implementing it myself, but I was wondering if you had some behavioral or
security motivation for not including this feature.
Thanks
—
Reply to this email directly or view it on GitHub
https://github.com/punkave/sanitize-html/issues/38.

--
*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

If you could, please take a look at https://github.com/calzoneman/sanitize-html/commit/81c4fb4c51ce61b483351b7cc2b59ece2355c5e2.
Known quirks:

<![CDATA[ something ]]> becomes <!--[CDATA[ something ]]-->.  This is because htmlparser2 calls oncomment() unless the recognizeCDATA option is true.  Setting the recognizeCDATA option to true would change existing behavior-- when enabled, data in <![CDATA[ ]]> tags would be given to ontext().
Since HTML does not allow nested comments, <!--<!--nested-->--> in the browser would be interpreted as the comment <!--<!--nested--> followed by the literal string -->.  The same is true of htmlparser2.  This already happens in the current version of sanitize-html (<!--<!--nested-->--> becomes -->).  I can't think of a way to use this literal --> to execute an attack, however it is a bit of ugly markup.

Hmm. Regarding the first quirk, you seem to be saying that in its current form this code would break support for CDATA that is functional now? Any thoughts on the best workaround?

As you do not currently explicitly handle comments or CDATA, in its current form, sanitize-html strips it out:
> sanitizeHTML('<![CDATA[test]]>')
''

For cases where allowComments is false, this behavior remains the same.  The quirk is that if allowComments is true, then instead of removing the CDATA entirely, it becomes an HTML comment:
> sanitizeHTML('<![CDATA[test]]>', { allowComments: true })
'<!--[CDATA[test]]-->'

There's not really a good way to guard against this; you could check whether the comment text starts with [CDATA[ and ends with ]], but since oncomment() strips the leading <! or <!-- there's no way to know whether you're processing <!--[CDATA[test]]--> or <![CDATA[test]]>.

I see. But it's not an exploitable bug?
On Mon, Jan 12, 2015 at 12:42 PM, Calvin Montgomery <
<EMAIL_ADDRESS>wrote:

As you do not currently explicitly handle comments or CDATA, in its
current form, sanitize-html strips it out:

sanitizeHTML('test')
''

For cases where allowComments is false, this behavior remains the same.
The quirk is that if allowComments is true, then instead of removing the
CDATA entirely, it becomes an HTML comment:

sanitizeHTML('test', { allowComments: true })
''

There's not really a good way to guard against this; you could check
whether the comment text starts with [CDATA[ and ends with ]], but since
oncomment() strips the leading <! or <!-- there's no way to know whether
you're processing  or test.
—
Reply to this email directly or view it on GitHub
https://github.com/punkave/sanitize-html/issues/38#issuecomment-69611103
.

--
*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

That's what I'm not sure about.  My understanding is that the purpose of CDATA is to allow you to type text literally without it being parsed as HTML (e.g. using < and > without having to escape them as < and >), however I am not an expert on XML parsers so I couldn't say for sure.

OK yes, that is exploitable, because you can put:
-->OMG
In the original CDATA and once it gets reformatted as a busted CDATA
without the proper delimeters nested inside a comment, it'll break out of
the comment and output an unauthorized h2 element.
So, we need a better fix for that issue.
On Mon, Jan 12, 2015 at 1:30 PM, Calvin Montgomery<EMAIL_ADDRESS>
wrote:

That's what I'm not sure about. My understanding is that the purpose of
CDATA is to allow you to type text literally without it being parsed as
HTML (e.g. using < and > without having to escape them as < and >),
however I am not an expert on XML parsers so I couldn't say for sure.
—
Reply to this email directly or view it on GitHub
https://github.com/punkave/sanitize-html/issues/38#issuecomment-69619258
.

--
*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

:+1: npm users want their README comments preserved.

I'm sure. So we need to figure out a fix for the exploit so we can accept this feature.

Any update here?

Alas, no. Contributions welcome but they must guard against exploits and have passing tests.

There's still interest in this. I'll open it back up to make it more visible, but will let the stalebot close it if there is no action on it.

Also browser directives like <!--[if IE 9]><video></video><![endif]--> might be a valid use case.

I have the same issue with . Do we have any option to avoid sanitizing it?


Improve doc about using esqueleto with Yesod

On Fri, Sep 7, 2012 at 7:42 AM, Arthur Clemens<EMAIL_ADDRESS>wrote:

This looks promising.
Trying out, I notice that I need to hide ==. from Yesod and
Database.Persist.Sqlite, so my imports become:
import Yesod hiding ((==.))
import Database.Persist.Sqlite (withSqliteConn, withSqlitePool)
import Database.Persist.TH

Yesod exports Yesod.Persist, which in turn exports Database.Persist, which in turns exports Database.Persist.Query, which conflicts with a lot of symbols from esqueleto.
Hmm... I guess I'll have to say on the docs that either you manually import Yesod modules, or you import esqueleto qualified.

I'm using this:
import Import hiding ((==.), (!=.), (||.))
import Database.Esqueleto;
import qualified Database.Persist as P

For some reason (^.) doesn't need to be hidden.  I'm not sure about the other operators.  If there's no more elegant way of hiding persistent, then maybe something like this should be in the docs, as some noob friendly boilerplate.


effuso, forgiveless

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[e,f]

/trunk merge


Update atoms-computation.ipynb

fixes issue #46

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


Upgrade spdy to 4.x.x and support Node v6, v8, v10

Well it's been about two years in making but this PR allows Zetta to run on the latest version of Node.js with the spdy and ws module updated to their newest versions. Sorry for all the delay, folks.
This release adds support for the versions below and drops supports for node <= v4. v0.10, v0.12, etc.. Sorry for anyone who uses it but those versions are long retired. Fixes #336, #333 and will make progress with #337. Once merged we can move much fast to modernize the codebase.
Peering will work between old versions of zetta and this version and vise-versa.
Node.js Release Support (Node Releases)

v6 EoL=April 2019
v8  EoL=December 2019
v10 EoL=April 2021

v11 is going to require some work on the node-spdy front. see: https://github.com/spdy-http2/node-spdy/issues/350. Or by then we can move off node-spdy in favor of the http2 core module.
Major Changes to Spdy

In zetta we were monkey patching our way into the spdy module to listen for spdy pings from the spdy clients (cloud -> hub). This broke for good reasons in the new version. I have a PR https://github.com/spdy-http2/node-spdy/pull/353 open to add support for receiving pings offically until then https://github.com/zettajs/node-spdy supports it.
Spdy server pushes are not emitted on the agent but instead on the originating request.
Lots of changes related to server pushes in handling the steam and closing it.

There were also changes in ws that broke getting the request url from the ws object. Instead we need to pass the original request to the PeerSocket.
TODO

[ ]  - Profile CPU/Mem usage, especially with peers constantly reconnecting while cloud is subscribed to data. This caused issues in the past when the push streams were not cleaned up.
[ ]  - Profile peer connection limits.
[ ]  - Upgrade argo and titan to a version with no security vulnerabilities.
[ ]  - Validate running on Beaglebone Black and Raspberry Pi.

Opened a new PR since i lost access to the repo after leaving Google. https://github.com/zettajs/zetta/pull/379
I also published version<EMAIL_ADDRESS>that incorporates all these changes.


Implement MapBuilderService.WriteStateToFile()

@wally-the-cartographer merge

Error: PUT https://api.github.com/repos/googlecartographer/cartographer/pulls/1424/merge: 405 Pull Request is not mergeable []
Log:
=> handle work item: PR 1424 on googlecartographer/cartographer.
 => Switching to branch master.
 $ git reset --hard HEAD [data/cartographer]
HEAD is now at f995744c Implement MapBuilderService.WriteStateToFile() (#1424)
 $ git clean -fd [data/cartographer]
 $ git checkout master [data/cartographer]
Already on 'master'
Your branch is up to date with 'origin/master'.
 $ git pull [data/cartographer]
Already up to date.
Current branch master is up to date.
 => Checking out cschuet/write_state_to_file as local branch pr_1424.
 $ git remote add cschuet git@github.com:cschuet/cartographer.git [data/cartographer]
fatal: remote cschuet already exists.
 $ git fetch cschuet [data/cartographer]
 $ git branch --track pr_1424 cschuet/write_state_to_file [data/cartographer]
Branch 'pr_1424' set up to track remote branch 'write_state_to_file' from 'cschuet'.
 => Switching to branch pr_1424.
 $ git reset --hard HEAD [data/cartographer]
HEAD is now at f995744c Implement MapBuilderService.WriteStateToFile() (#1424)
 $ git clean -fd [data/cartographer]
 $ git checkout pr_1424 [data/cartographer]
Switched to branch 'pr_1424'
Your branch is up to date with 'cschuet/write_state_to_file'.
 $ git pull --force [data/cartographer]
Already up to date.
Current branch pr_1424 is up to date.
 => merge master.
 $ git merge master [data/cartographer]
Already up to date.
 => Running clang-format on all *.cc and *.h files
 $ git commit -am Ran clang-format. [data/cartographer]
On branch pr_1424
Your branch is up to date with 'cschuet/write_state_to_file'.

nothing to commit, working tree clean
 $ git push --force cschuet HEAD:write_state_to_file [data/cartographer]
Everything up-to-date
 => Abandoning branch pr_1424.
 => Switching to branch master.
 $ git reset --hard HEAD [data/cartographer]
HEAD is now at 84a1205e Merge branch 'master' into pr_1424
 $ git clean -fd [data/cartographer]
 $ git checkout master [data/cartographer]
Switched to branch 'master'
Your branch is up to date with 'origin/master'.
 $ git branch -D pr_1424 [data/cartographer]
Deleted branch pr_1424 (was 84a1205e).


added location controller and tests for controller

In this PR we add an endpoint '/api/locations/get' that can be used to get a list of locations that match a given location name.
dev deployment: http://team01-srambhatla30-dev.dokku-03.cs.ucsb.edu/
Closes #9

LMGT

LGTM


Game activity timer needs to update if the round is completed by resignation

If the round is completed due to forced resignation of one or more idlers then the game's activity timer isn't updated, so then the Tsar also has only 60 seconds to pick the winner, which is clearly undesirable.
This bug was introduced by the fix to #126.

I need to find a way to update the timer only the first time the round is completed. The problem in #126 was that subsequent resignations were updating the timer, so extending the Tsar's time out by days. Now it's never being updated so the Tsar is getting no time at all.


more tests for LanguageExtensionFinder

@woppa684 Does this cover your issue?

@woppa684 Does this cover your issue?

Not entirely, jetbrains.mps.lang.structure keeps throwing an error. After adding that to the BuiltinsUsage I can get one language OK. But another language (that depends on this other language) still throws the error:
Request handler 'LionwebLanguage' throws exception

io.lionweb.mps.converter.m2.MpsLionCoreConverterException: language canon.cpp.mylang depends on de.slisson.mps.richtext, which is not included in this conversion

I found yet another internal API for dependencies. Does this look better?


Unable to Start D2R?

Hi,
I compiled the project, and when I run the project, I get the error listed above.

You have to download Handle from Microsoft and extract it in same Directory.
https://learn.microsoft.com/en-us/sysinternals/downloads/handle

I added hint window at startup with download link and instructions.


faster LENS calculation using FastNS

Hello,
I'm an undergraduate researcher at Prof. Taras Pogorelov's lab at UIUC. I am using LENS for my research, and I found a faster way of calculating LENS parameters, so I thought I would share it. I didn't find any issues and the results were identical to your original code, can you please check as well? Thanks!

This looks interesting! Thank you. Why have you closed it?


fix(number-input): updated NumberInput

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

34 of 44 (77.27%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.08%) to 80.568%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/number-input/src/Component.tsx
31
41
75.61%

Files with Coverage Reduction
New Missed Lines
%

packages/masked-input/src/Component.tsx
2
88.33%

Totals

Change from base Build<PHONE_NUMBER>:
0.08%

Covered Lines:
7382

Relevant Lines:
8347

💛  - Coveralls


could not download chart: chart "aws-fsx-openzfs-csi-driver/aws-fsx-openzfs-csi-driver" not found in https://kubernetes-sigs.github.io/aws-fsx-openzfs-csi-driver repository

/kind bug
What happened?
I am encountering an issue when trying to install the aws-fsx-openzfs-csi-driver Helm chart using Terraform with the following code:
resource "helm_release" "fsx_csi_driver" {
  count = var.irsa_fsx_openzfs_csi_enabled ? 1 : 0
  name       = "aws-fsx-openzfs-csi-driver"
  repository = "https://kubernetes-sigs.github.io/aws-fsx-openzfs-csi-driver"
  chart      = "aws-fsx-openzfs-csi-driver/aws-fsx-openzfs-csi-driver"
  namespace  = "kube-system"

  set {
    name  = "controller.serviceAccount.create"
    value = "true"
  }
}
**What you expected to happen?**

I expected the Helm chart to be installed without errors.

**How to reproduce it (as minimally and precisely as possible)?**
Steps to Reproduce:

Run the provided Terraform code to install the aws-fsx-openzfs-csi-driver Helm chart.

Expected Behavior:

I expected the Helm chart to be installed without errors.

Actual Behavior:

I encountered the following error:

Error: could not download chart: chart "aws-fsx-openzfs-csi-driver/aws-fsx-openzfs-csi-driver

**Anything else we need to know?**:

**Environment**
- Kubernetes version (use `kubectl version`): 1.26.9
- Driver version: 1.1.0

Certainly, here's the corrected Terraform code with explanatory comments:
resource "helm_release" "aws-fsx-openzfs-csi-driver" {
Conditional resource creation based on the "irsa_fsx_openzfs_csi_enabled" variable.
count = var.irsa_fsx_openzfs_csi_enabled ? 1 : 0
Name for the Helm release.
name       = "aws-fsx-openzfs-csi-driver"
URL of the Helm chart repository.
repository = "https://kubernetes-sigs.github.io/aws-fsx-openzfs-csi-driver"
Name of the Helm chart to install.
chart      = "aws-fsx-openzfs-csi-driver"
Namespace in which to install the Helm chart.
namespace  = "kube-system"
Set values for the Helm chart.
set {
name  = "controller.serviceAccount.create"
value = "true"
}
}


hello_server.go:30:2: "./helloworld" is relative, but relative import paths are not supported in module mode

unzip the docs folder & goto \docs\docs directory
docs.zip

./scripts/runserver.sh

Expected: hello server should be up & running, but the actual is error: hello_server.go:30:2: "./helloworld" is relative, but relative import paths are not supported in module mode

$ go version
go version go1.19 windows/amd64
$ protoc --version
libprotoc 3.21.4
$ go env
set GO111MODULE=
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\b.rajalakshmi\AppData\Local\go-build
set GOENV=C:\Users\b.rajalakshmi\AppData\Roaming\go\env
set GOEXE=.exe
set GOEXPERIMENT=
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GOMODCACHE=C:\Program Files\Go\pkg\mod
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\Program Files\Go
set GOPRIVATE=
set GOPROXY=https://proxy.golang.org,direct
set GOROOT=C:\Program Files\Go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLDIR=C:\Program Files\Go\pkg\tool\windows_amd64
set GOVCS=
set GOVERSION=go1.19
set GCCGO=gccgo
set GOAMD64=v1
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=C:\Users\b.rajalakshmi\Documents\Bhala\grpc\docs\docs\go.mod
set GOWORK=
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fno-caret-diagnostics -Qunused-arguments -Wl,--no-gc-sections -fmessage-length=0 -fdebug-prefix-map=C:\Users\B73ED~1.RAJ\AppData\Local\Temp\go-build1437050701=/tmp/go-build -gno-record-gcc-switches

The error message says it all.
Closing as working as intended.
Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For questions please refer to https://github.com/golang/go/wiki/Questions

@seankhliao , what's the solution?


Fix ToT failure on avahi

Problem
The current ToT code calls PublishService twice in a row during startup - once during a call to open the pairing window, one during the general server startup. There's also a Stop called between them. On ToT this was failing. Root caused to Stop not depopulating the list of services, so we were attempting to update a service that had already been stopped.
Fixes #9695
Change overview
Add a clear to the Stop call in the avahi mdnsImpl
Testing
No unit tests for avahi impl - tested on linux lighting-app, can see error no longer prints, can find on avahi-discover.

/rebase

@saurabhst @LuDuda @msandstedt @jepenven-silabs @chrisdecenzo @jmartinez-silabs  ?
This seems to be stuck on restyler (??? ... @woody-apple is there a way to kick it) however if we get enough checkmarks I can admin-merge it. Waiting >1day over a one line fix seems harsh

@andy31415 Restyled issues can be filed here: https://github.com/restyled-io/restyler/issues I'll label this restyled-ignore, and merge.

Filed: https://github.com/restyled-io/restyler/issues/156


Help and support section disappear

Hi,
from version v1.5.12, it disappear the contents of section help and support:

Bye

Will be out in 1.5.13


Support embedding LucidCharts or iFrames in documents

Hi,
While where's transitionning our docs "as-code", we haven't found a way to insert LucidCharts documents, or even insert <iFrame> HTML tags so the resulting Confluence page displays the thing correctly.
Is there a way to achieve this ?
Thanks

I do not believe this extension can do much for LucidCharts documents. There is no Sphinx extension that appears to provide LucidCharts capabilities to integrate with. Also, if LucidCharts did provide a way to host remote charting capabilities, there does not appear to be any available documentation online on how to achieve this.
As for iframes, it is possible to utilize the iframe macro in Confluence Cloud (I do not believe this macro is available on Confluence server/data center). There does not exist a directive at this time, but can be achieved by doing something such as following in reStructuredText:
.. raw:: confluence_storage

    <ac:structured-macro ac:name="iframe">
        <ac:parameter ac:name="src">
            <ri:url ri:value="https://sphinxcontrib-confluencebuilder.readthedocs.io/" />
        </ac:parameter>
        <ac:parameter ac:name="width">700px</ac:parameter>
        <ac:parameter ac:name="height">1000px</ac:parameter>
    </ac:structured-macro>

Or, in a Markdown document:
```{raw} confluence_storage
<ac:structured-macro ac:name="iframe">
    <ac:parameter ac:name="src">
        <ri:url ri:value="https://sphinxcontrib-confluencebuilder.readthedocs.io/" />
    </ac:parameter>
    <ac:parameter ac:name="width">700px</ac:parameter>
    <ac:parameter ac:name="height">1000px</ac:parameter>
</ac:structured-macro>
```

thanks @jdknight. I tried the raw syntax yesterday but it wasn't doing anything in the final docs. But I will test with your specific syntax.
Will report here.

MANY THANKS !
Using your code snippet and using the embed URL from Lucidchart worked !!
Just go to file -> embed then Activate the embed code and copy the URL over (URL only, you don't need the HTML around) :
```{raw} confluence_storage
<ac:structured-macro ac:name="iframe">
    <ac:parameter ac:name="src">
        <ri:url ri:value="https://lucid.app/documents/embedded/40000066-a000-000-0000-000000dc2e" />
    </ac:parameter>
    <ac:parameter ac:name="width">100%</ac:parameter>
    <ac:parameter ac:name="height">800px</ac:parameter>
</ac:structured-macro>
```


can we use the store created using built_redux with redux_persist

no but i think you can easily write you own persist using a middleware


Incorrect candidate Balance

If I do not use Metamask, the balance of candidate displays incorrect

Fixing pull request:
https://github.com/tomochain/tomomaster/pull/241


docs: project is generally available

Codecov Report

Merging #110 (5b5fdc3) into master (57208c8) will decrease coverage by 1.90%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #110      +/-   ##
==========================================
- Coverage   82.78%   80.88%   -1.91%
==========================================
  Files          14       14
  Lines         999      999
==========================================
- Hits          827      808      -19
- Misses        123      142      +19
  Partials       49       49

Impacted Files
Coverage Δ

internal/server/server.go
53.90% <0.00%> (-14.85%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Minor changes to compile with OpenACC and latest cuda + pgi compiler:

- acc routine seq in header causes pgi internal errors
- sm_20 is outdated, keep minimum sm_35

@fouriaux  : yes, you are right. There are few cuda files in coreneuron that don't do large compute:

starting stopping profiling
pick random123 stream

Hence for arch compatibility / avoid failure, minimum sm version was used.
Now I used sm_30 and sm_60. If sm_60 is not supported, it will fallback to older version to avoid failure. Is that reasonable?


[BUG] Total media requested is not adhering to set value

Describe the bug
I have the "Max Similar Movies" and "Max Similar TV Shows" set to 1 each, per day, with max content fetch for each content is 10.
However, when I hit "Run Now" to test it, it request 122 items lol

To Reproduce
Steps to reproduce the behavior:
My setup is Emby, I selected 3 libraries (Anime, Movies, TV Shows), additional settings is as follows.

Expected behavior
I thought if I set each max similar value to "1" it would only download/request 1 similar item per library I selected
I am running Suggestarr in Docker, in Unraid, on the latest tag/release

Hi @solidsnakex37! Thanks for reporting the issue.
How many users do you have on your Emby server?
The “1 film and 1 TV show” refers to one content item per recently viewed film or show for the Max Content Checks setting.
For example, if you’ve recently watched 8 films and 2 TV shows, SuggestArr will request 8 films (1 for each film watched) and 2 TV shows (1 for each show watched), as long as they haven’t already been requested.
I completely understand that this can be a bit tricky to manage right now, but I’m working on implementing more granular filters to give you better control. In the meantime, the best solution to prevent too many automatic requests and downloads is to create a local Jellyseer account without auto-approval.

hey @giuseppe99barchetta! That makes sense, this can be boiled down to my lack of understanding on the settings lol
I have 17 users on my Emby server currently, so does it check for suggestions across what all my users are watching?
Thankfully I set up a local user account without auto approval so removing the 122 requests won't be a problem. When I first tried this without the Emby support I thought there was an option to choose which user on the media server you want to perform the suggestions on, unless I am misremembering. Will that be brought back?

hey @giuseppe99barchetta! That makes sense, this can be boiled down to my lack of understanding on the settings lol
I have 17 users on my Emby server currently, so does it check for suggestions across what all my users are watching?
Thankfully I set up a local user account without auto approval so removing the 122 requests won't be a problem. When I first tried this without the Emby support I thought there was an option to choose which user on the media server you want to perform the suggestions on, unless I am misremembering. Will that be brought back?

This feature has never been available, but I’ll implement it as soon as possible!

Merged with
https://github.com/giuseppe99barchetta/SuggestArr/issues/38


Audio not always played ?

Hi,
I have a bevy app running via an iframe in a Dioxus app (maybe that's the core of my issues).
The issue I face is that the audio is not always triggered, same for the animation.
As for the animation , I have to reload multiple times my window for it to work fine (load properly, otherwise, it's a black screen).
But the audio may happen once every 20-30 tries.
Latest chromium on Debian Linux.
Is the issue the fact that I run Bevy via an iframe ?
Simplified excerpt:
let mut app = App::new();

        app.add_plugins((
                DefaultPlugins
                        .set(AssetPlugin {
                                meta_check: AssetMetaCheck::Never,
                                ..default()
                        })
                        .set(WindowPlugin {
                                primary_window: Some(Window {
                                        fit_canvas_to_parent: true,
                                        canvas: Some(canvas_name),
                                        ..Default::default()
                                }),
                                ..default()
                        })
                        .set(bevy::log::LogPlugin {
                                // Uncomment this to override the default log settings:
                                // level: bevy::log::Level::TRACE,
                                // filter: "wgpu=warn,bevy_ecs=info".to_string(),
                                ..default()
                        })
                        .set(AudioPlugin::default()),
        ));

app.add_systems(Update, play_sound.run_if(runifs_common::run_if_play_audio));
app.run();

my audio setup :
#[derive(Resource, Default)]

pub(crate) struct Audio {
        pub(crate) url:               Vec<String>,
        pub(crate) current_offset:    usize,
        pub(crate) currently_running: bool
}

pub(crate) fn play_sound(mut commands: Commands, asset_server: Res<AssetServer>, mut audio_bank: ResMut<Audio>) {

        info!("AUDIO IN");

        audio_bank.currently_running = true;

        commands.spawn(AudioBundle {
                source: asset_server.load(format!("sounds/{}", audio_bank.url[audio_bank.current_offset])),
                ..default()
        });

        audio_bank.current_offset += 1;

        audio_bank.current_offset %= audio_bank.url.len();
}

the console:
found the format marker [49, 44, 33] @ 0+2 bytes. log.target = "symphonia_core::probe";log.module_path = "symphonia_core::probe";log.file = "index.crates.io-6f17d22bba15001f/symphonia-core-0.5.3/src/probe.rs";log.line = 275;
found the format marker [ff, fb] @ 119+2 bytes. log.target = "symphonia_core::probe";log.module_path = "symphonia_core::probe";log.file = "index.crates.io-6f17d22bba15001f/symphonia-core-0.5.3/src/probe.rs";log.line = 275;
using xing header for duration log.target = "symphonia_bundle_mp3::demuxer";log.module_path = "symphonia_bundle_mp3::demuxer";log.file = "index.crates.io-6f17d22bba15001f/symphonia-bundle-mp3-0.5.3/src/demuxer.rs";log.line = 122;

and the cargo.toml for bevy:
bevy = { version = "0.14.2", features = ["mp3", "jpeg"] }
Are there people running bevy via an iframe experiencing the same thing ?
Bevy inside a canvas in dioxus is not a clean take.

Again, the sound may play once, then when reloading the window, the speaker symbol disappears. Sound gone.

I simplified the stack and remove the iframe in favor of the canvas approach. Animations work better, but still no sound:
The AudioContext was not allowed to start. It must be resumed (or created) after a user gesture on the page.
and
Uncaught (in promise) NotAllowedError: A user gesture is required to request Pointer Lock.
Even clicking on screen won't change anything.
Please note that I animate the camera myself programmatically.
Networking pane show that everything is properly loaded from the server.
When the sound works, which if it has decied to work, only works on the first run of a newly created app.
The beginning of the music is weird (distorted and hashed) and the sound becomes normal 3 seconds  later approx.
I am pretty sure now that the sound is considered autoplay and not muted, requiring approval of the user ? But even clicking on the screen doesn't solve this.
Bevy in release mode, on Chromium, linux.

Adding a button to force user interaction has no effect either..

Putting
<audio controls autoplay muted>

      <source src="/sounds/my_sound.mp3" type="audio/mpeg">
      Your browser does not support the audio element.
</audio>

in the root html plays the sound when the control is clicked.
The last point of investigation is the combo Dioxus + Bevy.
Maybe Dioxus catch the sound signal after all.
Troubles..

Progress:

The issue is that Chromium puts the permission on auto, which I can't enable by code or user interaction. Once I click on "allow", it works without issue.
So I have to find a way to force this option to allow somehow.

O-K. Let me avoid anyone the full headache I went through and here is the answer:
in your js file, put this code at the top and let the magic operate.
I will close now and move on.


Fix some events logic around do_deactivate_user

https://github.com/zulip/zulip/pull/20768 didn't add sending of invites_changed event when revoking invitations. Also spotted a bug in apply_event logic for user deactivation event.

Nice, merged, thanks @mateuszmandera!


Fun find-fountains

Hello, I'm trying to run the Fun pipeline and I have some trubles with the last command that I have to use.
I used my own data to perform this pipeline and also I download one mcool file from mendeley database that is avialbe and I have the same trouble in both cases.
This is the command line that I used:
Fun find-fountains mcool_matrix/Repli-HiC_K562_WT_BrdU_label_release_0h.mcool::resolutions/10000 --ext_length 500000     --half_width 2 --norm VC_SQRT     --region_path output_dir_test/SoN_summits/Summits_10000_merged.bed    --extension_pixels 10 100 5     --offset 50000 --interval_length 50000  --signal_noise_background 1.3 1.3 1.3 1.3 1.3   --coverage_ratio 0 --p_value 0.05    --output output_dir/fountains_10kb
And I get this error:
TypeError: ufunc 'floor_divide' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''
Then I find that there was a problem with the bed file and I remove header line from the bed file (the first line) and now I have this error:
File "/usr/local/ohpc/pub/apps/Fun/1.0.1/lib/ks_test.py", line 87, in
i, scipy.stats._stats_py.KstestResult
AttributeError: module 'scipy.stats' has no attribute '_stats_py'
I don't understand what may happen here, If someone has an idea of how to solve it I would be very thankfull.
Thanks.

Hey, It appears that "_stats_py" is not in your scipy.stats, please confirm your version of scipy, and if it doesn't work, I will go check my program again.

Hi! My scipy version is 1.6.3, I don't know if I need another version and if its the case which one.

My current version is 1.9.1, maybe you can try to update it, please let me know if you fix it.

I tried to run the command line with this version, and it didn't work. What Python version do you use? I have 3.8. Also, could you provide me with the requirements.txt file to see the versions of all the modules in Python? Thanks.

Of course, you can email me a screenshot of the specific error message, and I will help you resolve the issue in my spare time.

this is all the error, so do you have a reqyurements.txt?

This is the screenshot of the specific error, so do you have the requirements.txt?

I will test the code and then provide the versions of the libraries used in the environment. I will notify you in the next few days if I manage to pinpoint the issue.

Okay! Thanks a lot, I'm looking forward to hearing back from you with the library versions used.

Apologies for the wait. After testing, my current environment is functioning normally. The main dependency versions I am using are as follows:
numba: 0.56.2
pandas: 1.4.1
numpy: 1.23.3
bioframe: 0.3.3
scipy: 1.9.1
cooler: 0.8.11
click: 7.1.2
I noticed that when you execute the Fun command, you do not specify the version of Python. Is it possible that the Python version you installed the dependencies with is different from the version you are using to execute Fun? To avoid this problem, perhaps you can specify the Python version, for example, by using python3.8 ./Fun find-fountains.