dev: Bind mount /dev/vfio

We need the agent to detect devices that have been created under
/dev and mount them for the container to have accesss to device nodes.
In the meantime, bind mount just /dev/vfio to support vfio
and assigning devices to vfio driver. This will be removed once
complete support has been added.
Signed-off-by: Archana Shinde<EMAIL_ADDRESS>

@sboeuf Any idea why the CI is failing before I go look into it?  Is there some known issue?
I am just bind-mounting /dev/vfio here, it should not cause the tests to fail.

@amshinde nope no idea on the top of my head...

@sboeuf I tried running the CI tests locally and I dont see any faliures!
I am going to run the jenkins tests one last time and see if it helps. I am finding it absurd that adding an extra harmless mount would cause the tests to fail.

@amshinde - I'm seeing the following in the logs which looks like it might be relevant:
Nov 21 21:56:36 jenkins-azure-vm-ubuntu-16-04-templatea8f580 cc-proxy[56002]: time="2017-11-21T21:56:36.797909557Z" level=debug msg="{"container-pid":"3","error":"container_linux.go:295: starting container process caused "process_linux.go:399: container init caused "rootfs_linux.go:58: mounting "/dev/vfio" to rootfs "/tmp/hyper/8aLYnNeB44wCCjoHMWLu/root/rootfs" at "/dev/vfio" caused "stat /dev/vfio: no such file or directory"""","level":"error","msg":"Could not run process","name":"cc-agent","pid":169,"time":"2017-11-21T21:56:36.785246381Z"}" name=cc-proxy pid=56002 source=qemu vm=8aLYnNeB44wCCjoHMWLu

@jodh-intel  Yes, I realized I was using a custom kernel in my local setup. Need to wait for the kernel with vfio module to be released, for this to work.

@jcvenegas Can you merge this one ?

lgtm


Add tests for Worker template

We added a new template, but it has no tests. Does it contain all the files we expect? Does it compile and run, much less produce the expected results? We have lots of infrastructure to make this easy. Use this as a template for how to use it to test running results. You can add "which files were produced" tests by just adding entries here.
CC @muratg

@ryanbrandenburg already did most of this in #8628. I just need to re-enable the test now that the SDK has been updated.

Blocked by SDK issue: https://github.com/aspnet/AspNetCore-Internal/issues/2009#issuecomment-497166047


Disconnected peers not removed

I dump regulary
Logger?.Trace("Connected Clients: {0}/{1}/{2}", _server.GetPeersCount(ConnectionState.Connected), _server.ConnectedPeersCount, _connectedPeers.Count);
_connectedPeers is my own list list of connected peers where peers get removed on disconnect event.
It will output "Connected Clients: 4/4/3" with 3 peers connected.
Had that fixed 2 years ago, but lost my fix when updating to latest master ;)
#161 is back again ;=)

Are you sure that's this not because your list updated on PollEvents, whereas netManager counters updated immediately in other thread? Because test for old bug in executed every build with success.

Am not using pollevents. It's all multithreaded eventdriven
There are no connection events incoming. And it's constantly reporting 4 peers connected

Might it be that the netmanager has a problem if 2 of them are running?
I have one netmanager for server-to-server connection (1 connected) of the shards
and one for client connections (3 connected currently)

You can run tons of netmanagers simultaneously. There is no any problems with this.

Maybe you can create minimal program with bug?

Values after ALL player left the server and definitly no connection is running anymore:
22:50:10.346 | Trace | PureVoice | Connected Clients: 2/2/1

@c0nnex i will investigate this.

Seems to be related to , if a Client-netmanager is recreated. I currently do this on the client if it gets disconnected, because i had problems with the client receiving old data if reusing it after the connection was dropped due e.g. to timeout-

@c0nnex seems that ConnectedPeersCount in some cases not updated fast enough because of some value caches, internally value already volatile but this doesn't helps.

Well server was running 2 hours with us 3 connected, and it reports wrong values for 2 hours ;) I only dump the values every 5 seconds, so it should be updated in the second call by all means if a change occured at same time

@c0nnex ConnectedPeersCount fixed. This was really strange multithreading with caches problem. Bug with _server.GetPeersCount(ConnectionState.Connected) i cannot reproduce.

I found a place where i kept a peer-Reference, I'll fix that. I'll let you know if problems are fixed now.

@c0nnex peer reference doesn't affect connected count.

@c0nnex should be totally fixed in last commit. Reopen if you found it again


Removed python 3.6 from python PR gate

Removed 3.6 from PR gate due to deprecation and issues with torch versions above 1.12.x (eg - 1.13.1).
Description

Checklist

[ ] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

Codecov Report

Merging #2249 (ab30371) into main (529af52) will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2249      +/-   ##
==========================================
+ Coverage   88.10%   88.15%   +0.05%
==========================================
  Files         129       17     -112
  Lines        7337      287    -7050
==========================================
- Hits         6464      253    -6211
+ Misses        873       34     -839

Flag
Coverage Δ

unittests
88.15% <ø> (+0.05%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 112 files with indirect coverage changes


Suggesion: Add support for rest parameters within v-for

What problem does this feature solve?
I think it would make template composition a lot easier in many cases, particularly when using scoped slots via renderless componetns. For example, we could do:
<Form>
  <template slot-scope="{ value, events, handleSubmit, ...props }">
   <form @submit="handleSubmit">
      <!-- pass other props to the component -->
      <error v-bind="props"></error>

      <input :value="value" v-on="events" :class="{ 'is-valid': props.valid }">
      <button @click="handleSubmit" type="submit">Submit</button>
    </form>
  </template>
</Form>

Rather than:
<Form>
  <template slot-scope="props">
   <form @submit="props.handleSubmit">
      <error v-bind="getErrorProps(props)"></error>

      <input :value="props.value" v-on="props.events" :class="{ 'is-valid': props.valid }">
      <button @click="props.handleSubmit" type="submit">Submit</button>
    </form>
  </template>
</Form>

What does the proposed API look like?
No changes to the api

That already works: http://jsfiddle.net/qxnerfwu/ 🙂
If what you are asking it vue-loader handling the transformation, that should be done in vue-loader, but I remember it was asked/added some time ago 🤔

@posva hmm i tried adding appropriate buble  transforms in vue-loader's options My config now looks like this, but still can't make it work 🤔
{
  loader: 'vue-loader',
  options: {
    compilerOptions: {
      preserveWhitespace: false
    },
    transpileOptions: {
      transforms: {
      destructuring: true,
      parameterDestructuring: true,
      spreadRest: true,
    }
  }
}

Hi @posva sorry for pinging you but you closed the topic so quickly and feel that I didn't get and I feels that I haven't got proper answer to my question. I spent half a day trying to solve this problem by tweaking vue-loader's config but without success. Can you tell me what I'm doing wrong here or if this functionality event supported in SFC? Apreciate your help and time in this matter :)

Hi @posva sorry for pinging you but you closed the topic so quickly and feel that I didn't get proper answer to my question. I spent half a day trying to solve this problem by tweaking vue-loader's config but without success. Can you tell me what I'm doing wrong here or if this functionality event supported in SFC? Apreciate your help and time in this matter :)

If I recall correctly it wasn't transpiled from SFC, so it only works on browsers that support it but at the same time, I remember seeing the feature request in vue-loader, yet I couldn't find it...

Also can't find anything about this in vue-loader's repo :(. Can we move this discussions to vue-loader's repo and you will pin feature request label to it? :)


tensorflow ends with a ValueError exception

I am experimenting with tensorflow/examples/get_started/regression/linear_regression_categorical.py and according to https://www.tensorflow.org/api_guides/python/regression_examples it ought to be fairly straightforward.
My script (for qsub submission) is as follows:
#PBS -N my_project_tf_l0
cd /home/u22835/tensorflow-master/tensorflow/examples/get_started/regression/
echo start
./linear_regression_categorical.py
echo end
The error file shows the traceback with the culprit function:
ValueError: Tensor(“buffer_size:0”, shape=(), dtype=int64, device=/device:CPU:0) must be from the same graph as Tensor(“MapDataset_1:0”, shape=(), dtype=variant).
to exclude an issue with tensorflow version, I went ahead and upgraded tensorflow to 1.70 as explained in https://access.colfaxresearch.com/?p=learn, namely:
export CC=/glob/development-tools/versions/gcc-6.4.0/bin/gcc
export LD_LIBRARY_PATH=/glob/development-tools/versions/gcc-6.4.0/lib64/:$LD_LIBRARY_PATH
source /glob/development-tools/parallel-studio/compilers_and_libraries/linux/mpi/bin64/mpivars.sh
pip install –user /glob/deep-learning/versions/intel-tensorflow/tensorflow-1.7.0/tensorflow-1.7.0-cp36-cp36m-linux_x86_64.whl
as a result:
python -c ‘import tensorflow as tf; print(tf.version)’
…
ImportError: /lib64/libstdc++.so.6: version `CXXABI_1.3.8′ not found (required by /home/u22835/.local/lib/python3.6/site-packages/tensorflow/python/_pywrap_tensorflow_internal.so)
Failed to load the native TensorFlow runtime.
I assumed the exception reported in the previous post might be related to the tensorflow version (I saw a tutorial hinting in that direction). The upgrade completed but now I have a broken environment, and so assuming what I am doing here is nonsense, how do I revert back to the previous tensorflow version?
If indeed the error reported before is related to an (older) version, which version is then required?

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!

i would still want to know the reason of the exception listed below. My
original post was confusing, tensorflow runs correctly (based on other test
cases)
ValueError: Tensor(“buffer_size:0”, shape=(), dtype=int64,
device=/device:CPU:0) must be from the same graph as
Tensor(“MapDataset_1:0”, shape=(), dtype=variant).
Am Do., 7. Feb. 2019 um 20:36 Uhr schrieb ymodak<EMAIL_ADDRESS>
Automatically closing due to lack of recent activity. Please update the
issue when new information becomes available, and we will reopen the issue.
Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/24811#issuecomment-461564803,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AiHT-siXiQegln1kPXZASCD3XQkMceprks5vLIApgaJpZM4Z4STO
.

--
Regards,
Joseph Pareti - Artificial Intelligence consultant
cell +49 1520 1600 209
cell +39<PHONE_NUMBER>

I was able to execute tensorflow/examples/get_started/regression/linear_regression_categorical.py successfully using TF 1.13.0.rc0 . TF 1.7 is a very old version. Please switch to a newer version. Thanks!

thanks for the hint, however it fails just as before even with tensorflow
1.13.
see the log file with highlighted keypoints
Am Fr., 8. Feb. 2019 um 00:23 Uhr schrieb ymodak<EMAIL_ADDRESS>
I was able to execute
tensorflow/examples/get_started/regression/linear_regression_categorical.py
successfully using TF 1.13.0.rc0 . TF 1.7 is a very old version. Please
switch to a newer version. Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/24811#issuecomment-461628622,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AiHT-jz2q73KqpauZ12JeurnW3ABZm2Zks5vLLV2gaJpZM4Z4STO
.

--
Regards,
Joseph Pareti - Artificial Intelligence consultant
cell +49 1520 1600 209
cell +39<PHONE_NUMBER>


An an extra page/tab to HUMAN dashboard for Fortune UI

[ ] Integrate Fortune Launcher into dashboard


Stuck waiting for Character response

Hi,
I started using this library cause I had problems when developing my own library.
I am using the synchronous standard example provided in github.
It sometimes works, but mostly does not.
When using the headless = false option it is always stuck at the point shown in the picture.

Any ideas what is happenig?
There is no error messagen, its just waiting for the response:

send your code

from characterai import PyCAI
client = PyCAI('#####################################', headless=False)
char = 'VvRwRK_-FuuBkX3X1YqqJFsfKbXw3uL1NLet2emisv0'
Save tgt and history_external_id
to avoid making a lot of requests
chat = client.chat.get_chat(char)
history_id = chat['external_id']
participants = chat['participants']
In the list of "participants",
a character can be at zero or in the first place
if not participants[0]['is_human']:
tgt = participants[0]['user']['username']
else:
tgt = participants[1]['user']['username']
while True:
message = input('You: ')
data = client.chat.send_message(
    char, message, history_external_id=history_id, tgt=tgt
)

name = data['src_char']['participant']['name']
text = data['replies'][0]['text']

print(f"{name}: {text}")

from characterai import PyCAI

client = PyCAI('#####################################', headless=False)

char = 'VvRwRK_-FuuBkX3X1YqqJFsfKbXw3uL1NLet2emisv0'

# Save tgt and history_external_id
# to avoid making a lot of requests
chat = client.chat.get_chat(char)

history_id = chat['external_id']
participants = chat['participants']

# In the list of "participants",
# a character can be at zero or in the first place
if not participants[0]['is_human']:
    tgt = participants[0]['user']['username']
else:
    tgt = participants[1]['user']['username']

while True:
    message = input('You: ')

    data = client.chat.send_message(
        char, message, history_external_id=history_id, tgt=tgt
    )

    name = data['src_char']['participant']['name']
    text = data['replies'][0]['text']

    print(f"{name}: {text}")

This is my code
I will provide whats happening more detailed in a second

Hey, sorry for wasting your time...
When trying to record a timeout happend and I relaized that character.py was trying to open urls with the: "https://beta.character.ai" but I am a plus user (forgot to cancle) and so I need to replace all urls with: "https://plus.character.ai". And now everything works like a char !!!!

Hey, sorry for wasting your time...
When trying to record a timeout happend and I relaized that character.py was trying to open urls with the: "https://beta.character.ai/" but I am a plus user (forgot to cancle) and so I need to replace all urls with: "https://plus.character.ai/". And now everything works like a char !!!!


fedpkg still can't clone using auth a dist-git repo

[2019-08-26 06:19:54,504: DEBUG/ForkPoolWorker-1] cmd = 'fedpkg --user packit -q clone packit /tmp/packit-dist-gitbbtanuot'
[2019-08-26 06:19:56,652: DEBUG/ForkPoolWorker-1]
[2019-08-26 06:19:56,653: ERROR/ForkPoolWorker-1] No user exists for uid<PHONE_NUMBER>
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
Could not execute clone: Failed to execute command.

[2019-08-26 06:19:56,653: ERROR/ForkPoolWorker-1] Command ['fedpkg', '--user', 'packit', '-q', 'clone', 'packit', '/tmp/packit-dist-gitbbtanuot'] failed
[2019-08-26 06:19:56,653: ERROR/ForkPoolWorker-1] Command ['fedpkg', '--user', 'packit', '-q', 'clone', 'packit', '/tmp/packit-dist-gitbbtanuot'] failed.
[2019-08-26 06:19:56,653: INFO/ForkPoolWorker-1] cleaning up the mess
[2019-08-26 06:19:56,653: DEBUG/ForkPoolWorker-1] Initialize kubernetes client
[2019-08-26 06:19:56,653: INFO/ForkPoolWorker-1] loading incluster config
[2019-08-26 06:19:56,685: INFO/ForkPoolWorker-1] removing sandbox pod
[2019-08-26 06:19:56,715: DEBUG/ForkPoolWorker-1] (404)
Reason: Not Found
HTTP response headers: HTTPHeaderDict({'Cache-Control': 'no-store', 'Content-Type': 'application/json', 'Date': 'Mon, 26 Aug 2019 06:19:56 GMT', 'Content-Length': '270'})
HTTP response body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"pods \"docker-io-usercont-sandcastle-20190826-061956653569\" not found","reason":"NotFound","details":{"name":"docker-io-usercont-sandcastle-20190826-061956653569","kind":"pods"},"code":404}

[2019-08-26 06:19:56,715: DEBUG/ForkPoolWorker-1] remove contents of the PV
[2019-08-26 06:19:56,715: INFO/ForkPoolWorker-1] volume was not empty
[2019-08-26 06:19:56,716: DEBUG/ForkPoolWorker-1] content of the volume: [PosixPath('/sandcastle/README.md'), PosixPath('/sandcastle/Dockerfile'), PosixPath('/sandcastle/.git'), PosixPath('/sandcastle/.pre-commit-config.yaml'), PosixPath('/sandcastle/.gitattributes'), PosixPath('/sandcastle/setup.cfg'), PosixPath('/sandcastle/docs'), PosixPath('/sandcastle/files'), PosixPath('/sandcastle/packit.spec'), PosixPath('/sandcastle/ansible.cfg'), PosixPath('/sandcastle/.packit.yaml'), PosixPath('/sandcastle/Makefile'), PosixPath('/sandcastle/.gitignore'), PosixPath('/sandcastle/CHANGELOG.md'), PosixPath('/sandcastle/CONTRIBUTING.md'), PosixPath('/sandcastle/release-conf.yaml'), PosixPath('/sandcastle/recipe-tests.yaml'), PosixPath('/sandcastle/setup.py'), PosixPath('/sandcastle/packit'), PosixPath('/sandcastle/.zuul.yaml'), PosixPath('/sandcastle/fedora-tests'), PosixPath('/sandcastle/tests'), PosixPath('/sandcastle/Vagrantfile'), PosixPath('/sandcastle/LICENSE'), PosixPath('/sandcastle/.git_archival.txt')]
[2019-08-26 06:19:56,737: ERROR/ForkPoolWorker-1] Task task.steve_jobs.process_message[5adc02ce-9360-4854-a7fd-3e64c649c459] raised unexpected: PackitException("Command ['fedpkg', '--user', 'packit', '-q', 'clone', 'packit', '/tmp/packit-dist-gitbbtanuot'] failed: Command ['fedpkg', '--user', 'packit', '-q', 'clone', 'packit', '/tmp/packit-dist-gitbbtanuot'] failed.")
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/celery/app/trace.py", line 385, in trace_task
    R = retval = fun(*args, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/celery/app/trace.py", line 648, in __protected_call__
    return self.run(*args, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/packit_service/worker/tasks.py", line 42, in process_message
    return SteveJobs().process_message(event=event, topic=topic)
  File "/usr/local/lib/python3.7/site-packages/packit_service/worker/jobs.py", line 126, in process_message
    jobs_results = self.process_jobs(event_object)
  File "/usr/local/lib/python3.7/site-packages/packit_service/worker/jobs.py", line 92, in process_jobs
    handlers_results[job.job.value] = handler.run()
  File "/usr/local/lib/python3.7/site-packages/packit_service/worker/github_handlers.py", line 200, in run
    create_pr=False,
  File "/usr/local/lib/python3.7/site-packages/packit/api.py", line 164, in sync_release
    assert_existence(self.dg.local_project)
  File "/usr/local/lib/python3.7/site-packages/packit/distgit.py", line 98, in local_project
    anonymous=not cccolutils.has_creds(),
  File "/usr/local/lib/python3.7/site-packages/packit/fedpkg.py", line 91, in clone
    utils.run_command(cmd=cmd)
  File "/usr/local/lib/python3.7/site-packages/packit/utils.py", line 90, in run_command
    raise PackitException(f"Command {shell.args!r} failed: {error_message}")
packit.exceptions.PackitException: Command ['fedpkg', '--user', 'packit', '-q', 'clone', 'packit', '/tmp/packit-dist-gitbbtanuot'] failed: Command ['fedpkg', '--user', 'packit', '-q', 'clone', 'packit', '/tmp/packit-dist-gitbbtanuot'] failed.
[

Traceback from prod right now.

[2019-09-10 13:34:20,134: ERROR/ForkPoolWorker-1] No user exists for uid<PHONE_NUMBER>
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
Failed to set locale, defaulting to C.UTF-8
Could not execute clone: Failed to execute command.

Could this be also the reason?
/usr/bin/run_worker.sh: line 9: /home/packit/passwd: Permission denied

nice catch, seems like that's the problem

I can't reproduce locally:
$ oc rsh pod/packit-worker-0

sh-5.0$ getent passwd packit
packit:x:1000140000:0:Packit Service:/home/packit:/bin/bash

sh-5.0$ kinit -k -t /secrets/fedora.keytab<EMAIL_ADDRESS>
sh-5.0$ cd /tmp

sh-5.0$ fedpkg --user packit -q clone packit
The authenticity of host 'pkgs.fedoraproject.org (<IP_ADDRESS>)' can't be established.
RSA key fingerprint is SHA256:Q12OTyTeOHWlS54dTzy2BNu7wB8UKNf18+7WHIDsORc.
Are you sure you want to continue connecting (yes/no/[fingerprint])? yes
Warning: Permanently added 'pkgs.fedoraproject.org,<IP_ADDRESS>' (RSA) to the list of known hosts.

sh-5.0$ ls -lha
total 4.0K
drwxrwxrwt. 1 root   root   20 Sep 16 12:30 .
drwxr-xr-x. 1 root   root   62 Sep 16 12:29 ..
-rwx------. 1 root   root 1.4K Aug 25 07:48 ks-script-_kvupbo7
drwxr-xr-x. 4 packit root  141 Sep 16 12:30 packit


Add html to generic event details

Add html content for generic message html. This is to support html for the new fusion broadcast that will be generic event details instead of broadcast event details.

Looks good but seems is not yet onprod`. This causes the build fail, shall we merge it after being available.

Yep!


Move continuous integration to GitHub Actions, maybe?

As Travis-CI.org was shut down, the Travis badge in README leads to a page, where last build was made year ago.
https://travis-ci.org/github/yourlabs/django-autocomplete-light/builds
Github Actions seem simple to implement.
Do we want to go this route, or do we want to use another free CI?
Waiting for your suggestions @jpic and I'll set something up.
Do you have any suggestions for other @yourlabsopensource projects?

We use my gitlab in general because I got a fancy server.
But I don't have anything against GitHub actions, and surely anything will
be better than nothing at all !
On Fri, May 13, 2022, 21:08 Michał Pasternak @.***>
wrote:

As Travis-CI.org was shut down, the Travis badge in README leads to a
page, where last build was made year ago.
https://travis-ci.org/github/yourlabs/django-autocomplete-light/builds
Github Actions seem simple to implement.
Do we want to go this route, or do we want to use another free CI?
Waiting for your suggestions @jpic https://github.com/jpic and I'll set
something up.
Do you have any suggestions for other @yourlabsopensource
https://github.com/yourlabsopensource projects?
—
Reply to this email directly, view it on GitHub
https://github.com/yourlabs/django-autocomplete-light/issues/1296, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AJGMUCSXCRNQUAOHUNSPIOTVJ2SDLANCNFSM5V4HH2MA
.
You are receiving this because you were mentioned.Message ID:
@.***>

I can help too @mpasternak - it sounds like @yourlabsopensource is open for it!  Let me know if you need a hand, I'd like to see this too...

sure, we're open for pull requests
Le sam. 3 sept. 2022 à 17:29, Andy Shapiro @.***> a
écrit :

I can help too @mpasternak https://github.com/mpasternak - it sounds
like @yourlabsopensource https://github.com/yourlabsopensource is open
for it! Let me know if you need a hand, I'd like to see this too...
—
Reply to this email directly, view it on GitHub
https://github.com/yourlabs/django-autocomplete-light/issues/1296#issuecomment-1236143802,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJGMUCVRCBRB7BNU37MIPWTV4NVGDANCNFSM5V4HH2MA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Add ability to use Nayjest/Grids without a database call and also add methods to allow for hiding and showing columns based on screen size

Thanks for making this package it's been very helpful.
I wanted to be able to use the table builder function without making an additional database call as the data already exists in my application and a second call is redundant.
I created a new DataProvider called CollectionDataProvider that accepts an Illuminate\Database\Eloquent\Collection object instead of a query.
The code itself is essentially copied from EloquentDataProvider so it shouldn't be too hard to understand.
Sorting is a bit weird with this Collection as it seems to be done via the query, but this wasn't a problem for me since my data was already pre sorted to begin with.
I also added some tests for the CollectionDataProvider, though there weren't any examples to work off of so they are quite basic. Essentially they just check to make sure no methods throw an error
I also added the ability to hide columns based on screen size. This method simply adds the bootstrap class  hidden-*  to both the header and the row column items when building the table. Here is an example of how to use it
    # Instantiate & Configure Grid
    $grid = new Grid(
        (new GridConfig)
            # Grids name used as html id, caching key, filtering GET params prefix, etc
            # If not specified, unique value based on file name & line of code will be generated
            ->setName('my_report')
            # See all supported data providers in sources
            ->setDataProvider(new CollectionDataProvider($result))
            # Setup table columns
            ->setColumns([
                # simple results numbering, not related to table PK or any obtained data
                new IdFieldConfig,
                (new FieldConfig)
                    ->setName('title')
                    # will be displayed in table heder
                    ->setLabel('Title')
                    # sorting buttons will be added to header, DB query will be modified
                    ->setSortable(true)
                    # hide column on mobile
                    ->hideXs()
                    # hide column on small screen
                    ->hideSm()
                    # hide column on medium device
                    ->hideMd()
                    # hide column on large will also work

            ])
            # Setup additional grid components

    );

The tests are failing because it needs to be installed in a laravel application and the style checks seem to be not passing, but a quick look shows that it was also the case for all other pull requests so let me know if that's an issue.

Hi Andrew,
Thanks for contribution.
Regarding CollectionDataProvider -- At all it's good idea & I can accept such contribution, but I need to test it with Laravel 4.
Also, I already have ArrayDataProvider in presentation/framework.
Regarding hideXs / hideSm -- I can't merge it becouse generally grids was designed to work without twitter bootstrap too. I think, this is case when better to leave it for customizations outside main repository.
Regarding tests with Laravel, see how it's done here: https://github.com/presentation-framework/laravel/tree/master/tests there is "laravel/framework" dev-dependency & specific bootstrap for tests (it's a draft, some code may be not working).
And for future, please, create separate pull-requests for different features, becouse if maintainer refuse one feature, it's not possible to merge another features from samerequest.

Pull request must contain no code align / ident changes for code not related to new features.

Thanks for the information about Laravel tests. I couldn't find any information on how to properly create them except for this [https://websanova.com/blog/laravel/creating-a-new-package-in-laravel-5-part-5-unit-testing] which seemed very clunky.
You mention an "ArrayDataProvider" but the reason I wrote the code for the "CollectionDataProvider" class is because I couldn't find anything related to this in the codebase. There is an "ArrayDataRow" but no ArrayDataProvider. Is it possible it disappeared after a pull request or something?
As for the hideXs, Sm, etc, it actually does not require bootstrap, it simply adds the hidden-* class to each column header and data row.  If you are not using bootstrap you would simply have to add this css manually but I can understand why you would not want this. What if I modified this function to add "style" attributes instead?
I have not tested Laravel 4.0 but it should work perfectly fine as the code is essentially the same as the EloquentDataProvider class without the database call.

I'm looking for a simple PHP Array data provider so I'm trying out this one. While testing I came across the following issue:
On https://github.com/Nayjest/Grids/blob/master/src/Components/ShowingRecords.php#L38 it will call $paginator->total() but that method doesn't exist, however $paginator->count() works.


Event type for removed driveItem

Is there any event type for removed driveItem or is Microsoft.Graph.DriveItemUpdated published also for remove driveItem?

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

ID: 330ac686-e9e5-af44-f0d6-0134a9017c56
Version Independent ID: 54b574ae-6ad7-9361-0a5e-230f9a47e0e0
Content: Microsoft OneDrive events - Azure Event Grid
Content Source: articles/event-grid/one-drive-events.md
Service: event-grid
GitHub Login: @spelluru
Microsoft Alias: spelluru

@MartinM85
Thanks for your feedback! We will investigate and update as appropriate.

@MartinM85 The Microsoft.Graph.DriveItemUpdated event will be triggered for the drive item update and not deleted as documented.
As of now there is no event for deleted drive items. For any feedback or feature request you can pass the feature request/feedback here.
As the issue is not related to document issue we will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Fix katas using package type setting

This is a workaround to fix kata behavior in the playground, where they were incorrectly treated as exe documents and failed compilation due to missing entry point. The katas need to be treated as if they are lib document. This change adds a new parameter to updateDocument on the language server so that the caller can indicate what type the document is.
Fixes #516

Just some import/export tweaks, else looks good. Thanks!

Yeah, I just played around with it and it is a little tricky. Previously all of the compiler and language service only used types from the generated wasm, and the actual JavaScript (non-type code) was only loaded by the platform specific entry point (browser.ts or main.ts) as the wasm JavaScript is platform specific.
An enum is a value not a type. So by using an enum from the generate wasm in the compiler/language service code, we now need to suck the runtime specific JavaScript into those supposedly platform-agnostic abstractions, and not just types.
I'll try and think of a cleaner way to do this. My instinct is to just stay away from enums (they're a TypeScript construct. There are no enums in JavaScript).

I just added one commit which avoids the loading or platform specific runtime JavaScript when not desired. It does mean using the underlying value rather than the enum in a couple places. We can look at options there later, but this fixes the issue without undesirable side-effects for now.
Note: I had to build locally with '--no-check'. Seems there's quite  a bit of code in this branch failing cargo fmt or clippy for me.

Actually, I wonder if just a flag/bool isExe might be much simpler for that arg? I don't see us supporting more than type types (exe and lib) any time soon, and as I say, avoiding enums in TypeScript is best. They're kind of a minefield.

Actually, I wonder if just a flag/bool isExe might be much simpler for that arg? I don't see us supporting more than two types of projects (exe and lib) any time soon, and as I say, avoiding enums in TypeScript is best. They're kind of a minefield.

Yeah, at this point I'm ready to give in and just use a Boolean. I was applying Rust sensibility to this part, which prefers named enums for clarity. We can keep the enum in the Rust implementation, but just use a bool for the wasm boundary and ts parts.


omni rpc too slowly，bitcoin rpc seems working fine

Synchronous completion，omni_getinfo / omni_getbalance .. are too slow or no response. The bitcoin original RPCs seems working fine.Who knows why？thinks.
server=1
txindex=1
rpcthreads=10
version: v0.7.0

still exist in 0.8.2


when using pnpm and publishConfig.directory publish errors out

Affected Packages
I guess cli/publish
Problem
I'm in a nx monorepo with a pnpm workspace. nx is configured to build in a dist dir at the root of the monorepo.
When specifying publishConfig.directory in the package.json the code of changesets will get the correct directory but then following pnpm doc, pnpm will interpret the path again leading to a bad path (in my case I end up with one part of the path doubled).
Removing the field publishConfig.directory from the package.json in the dist dir fix the issues.
Proposed solution
🤷 Maybe we could remove the publishConfig.directory field to prevent pnpm from interpreting it but it's a bit involved.

Could you list the exact steps taken by different tools involved in the process? How a package directory looks before & after each tool gets executed in the pipeline?

@Andarist Sure.
Some of the code in the monorepo
package.json root package.json
packages/package/package.json package.json of the package to be published with a publish config
"publishConfig": {
    "directory": "../../dist/packages/tmp"
  }

packages/package/project.json nx config file for the package
packages/package/src/*
I can now run changeset version if I have any changesets.
Then I run nx package:build
I end up with the dist dir at the root of the monorepo containing the build of my package
dist/packages/package/package.json
dist/packages/package/CHANGELOG.md
dist/packages/package/README.md
dist/packages/package/src/index.js
...
Then if I run changeset publish changeset pickups the publishConfig in the packages/package/package.json package.json and then runs pnpm publish inside the dist directory where pnpm pick ups the publishConfig inside the dist/packages/package/package.json package.json.

Hmm, so I wonder what we could really do about this here. It really seems that Changesets is not involved in the process all of that much - we just invoke pnpm publish in the given directory and that's all. And you need us to do this because that publishConfig.directory is where build artifacts are and those should be published.
I think the problem is in the packing stage of this workflow - so within Nx itself, I don't see how to solve this without removing publishConfig.directory from this dist directory. It feels like this kind of setup would always suffer from this issue - having Changesets in the mix doesn't make it pop up, it would always be present for monorepos using Nx + publishConfig.directory + pnpm. So I think that this should be addressed in Nx.
Do you have a link to the docs for the publishConfig.directory in Nx? I can't quite find it

There is no link to nx.
changeset is reading publishConfig.directory to know where it should call pnpm publish which then also read publishConfig.directory.
The only thing related to nx is that i decide to build my packages in a root dist directory but that can happen with any build tool (and it's the case here, at the end it's only nx that's telling swc to build where I want it to build.
I could have a monorepo without nx and just webpack / any other build tool  that build in a dist folder of my choice.

Hmm... so I guess what you are saying is that we shouldn't handle publishConfig.directory on our own when using pnpm and let it resolve it on its own.
So it would read the packages/package/package.json#publishConfig.directory, use the target directory as something that needs to be published but it would ignore the dist/packages/package/package.json#publishConfig.directory in such a situation?

Yeah I think that's it. Thinking a bit more about it, it's just pnpm who needs to know where to run its publish command. So if we just had a way for changeset to known where to run pnpm publish that isn't publishConfig.directory it should solve the problem :)

Yeah I think that's it.

Have you confirmed (perhaps by patching Changesets) that it would work for you?

Do note that it's a bit different from letting pnpm handle it.

Hmm, I'm rereading this and I'm somewhat unsure how this is different. Could you clarify? Both read roughly in the same way to me.

Sorry for the late answer.
The only problem for me is that both changesets and pnpm interpret the same config key (publishConfig.directory)
This is the interesting code for me: https://github.com/changesets/changesets/blob/a3fa3ece513f96f3e2ce1fb1f228e450e40cbdd2/packages/cli/src/commands/publish/publishPackages.ts#L129-L141
I want changesets to run in the directory I care about, what happens after is purely a side effect for me, I really don't want pnpm to interpret directory again because changesets is already cwding pnpm in the correct dir.

@Andarist sorry to bother you but is it more clear from your side?


option to import just the Broker class

useful if you want multiple broker instances for any reason

I'm not sure this would work out very well. There are a lot of parts of wascally that use cached modules from require calls... for example, in the index.js file: https://github.com/LeanKit-Labs/wascally/blob/master/src/index.js#L8-L10
var dispatch = postal.channel( 'rabbit.dispatch' );
var responses = postal.channel( 'rabbit.responses' );
var signal = postal.channel( 'rabbit.ack' );

Because of this, creating a new instance of a broker would not completely isolate that instance from any others. You would likely end up with unexpected behaviors, due to the shared postal channels.
Is there a use case where you need to have multiple broker instances?


fix docker-entrypoint.sh ,SW_EXPORTER_ENABLED not effective

shuod ${var_application_file} not ${var_application_file}
Please answer these questions before submitting pull request

Why submit this pull request?

[x] Bug fix

[ ] New feature provided

[ ] Improve performance

Related issues

Bug fix

Bug description.

How to fix?

New feature or improvement

Describe the details and related test reports.

Coverage decreased (-0.002%) to 16.827% when pulling 082e2aae8a8e377eaf6bb52a8cc30da1ef401be1 on Qiliang:patch-1 into 48bca2418fd7ee583c7fa01a8b2e9aa7a96733ad on apache:master.


Dimming a Sonos speaker volume

I am using https://github.com/jishi/node-sonos-http-api which has an api to change the volume of a sonos speaker.
Set Volume:
http://localhost:5005/livingroom/volume/15 (will set volume for room Living Room to 15%)
However, using http://localhost:5005/livingroom/volume/${intensity.percent} always sets the volume all the way down. Guess I am not understanding how ${intensity.percent} works. Isn't this a number between 0-100?

It is a number between 1 and 100. Turn on debug for the httpHandler

Here's the log:
01-29-2017 23:09:01.713 DEBUG Making outbound call in doHttpRequest: GET http://<IP_ADDRESS>:5005/LivingRoom Sonos/volume/${intensity.percent} HTTP/1.1 com.bwssystems.HABridge.plugins.http.HTTPHandler
01-29-2017 23:09:01.766 DEBUG GET execute on URL responded: 200 com.bwssystems.HABridge.plugins.http.HTTPHandler

Ah i just figured it out its the URL percent encoding that was screwing it up it needs to be ${intensity.percent} and not $%7Bintensity.percent%7D


Excluded entities were showing up

I have the following snippet and expected entities in the 'exclude' section not to show up but they do. Am I missing something?
  - url: /local/monster-card.js?v=0.2.3
    type: js

      - type: custom:monster-card
        show_empty: false
        card:
          type: entities
          title: Lights On
          show_header_toggle: false
        filter:
          include:
            - domain: light
              state: 'on'
            - domain: switch
              state: 'on'
          exclude:
            - entity_id: switch.garage_camera_night_mode
            - entity_id: switch.garage_camera_infrared_led
            - entity_id: switch.garage_camera_infrared_led_filter
            - entity_id: switch.garage_camera_motion_detection
            - entity_id: switch.garage_camera_rtsp_h264
            - entity_id: switch.garage_camera_rtsp_mjpeg

My bad. There were other switch entities with no underscore in the name. I fixed them and everything works perfectly.
      - type: custom:monster-card
        show_empty: false
        card:
          type: entities
          title: Lights On
          show_header_toggle: false
        filter:
          include:
            - domain: light
              state: 'on'
            - domain: switch
              state: 'on'
          exclude:
            - entity_id: '*garage_camera*'
            - entity_id: '*garagecamera*'


remove component and resource suffixes from reflect structs

Objective
Remove suffixes from reflect component and resource methods to closer match bevy norms.
Solution
removed suffixes and also fixed a spelling error

Fixes https://github.com/bevyengine/bevy/issues/5202

Can you put the "Fixes" comment into your PR description? That will cause the issue to get linked and automatically closed.

bors r+


:bug: Wait for connection end to get completed data;

When env contains lots of data,eg: custom LS_COLORS or a very long long PATH, the connection will receive data multiple times in chunk, need to wait for its end to get completed data

Thanks for contributing!


Restrict api versions

What this PR does / why we need it:
This PR:

restricts the mutating and validating webhooks to only accept v1 cert-manager.io and acme.cert-manager.io resources
removes the v1beta1 versions from accepted AdmissionReview versions for mutating and validating webhook and accepted ConversionReview version for conversion webhook. (both admissionregistration.k8s.io/v1 API for AdmisionReview and apiextensions.k8s.io/v1 for ConversionReview is available since Kubernetes v1.16, so this should not cause any issues).

The first part is needed to prepare for the deprecated API removal in v1.6 to avoid issues during upgrade, the second part is just a cleanup.
Release note:

cert-manager's webhook now supports only `v1` `AdmisionReview` and `ConversionReview` resources.

/kind cleanup

/test pull-cert-manager-e2e-v1-16

I forgot to mention that I went looking at ClusterAPI to see whether other projects also specify a single API version in their webhooks. They do:
https://github.com/kubernetes-sigs/cluster-api/blob/a5e1b5fe003a29ec09a433cc5f6118e76993caa2/config/webhook/manifests.yaml#L22-L23

Thanks for the review and for digging into this @wallrj !

Will it also result in the /convert endpoint being called twice for each CREATE / UPDATE?

It does look like it gets called for both validating and mutating webhook- I thought that perhaps the validating webhook would simply be sent the converted version. But I also think that users should look at converting their legacy resources in v1.5, so maybe it's not an issue.

So if some user had patched our webhook configuration to use Exact (and I have no idea why they'd do that) then this change might mean that they are no longer validating cert-manager CRs.....so maybe it'd be worth adding a note to the website release notes and upgrade notes, explaining this....or maybe it'd just cause confusion.

I tried to add a release note that is not too alarming, but could potentially catch the attention of anyone doing that.
I imagine that this change might be visible to users getting webhook errors since now it is the conversion endpoint that gets called first.

previously my v1alpha2 resources would have been sent direct to the webhook/validate endpoint, without pre-conversion

Yes, I think it would have been converted to the internal type inside the webhook, mutated/validated and converted back to v1alpha2 ? See this comment.
I have also updated the PR description.

/hold cancel


Use DAP's new "StackFrame.canRestart" property to enable the "Restart Frame" command

With https://github.com/microsoft/debug-adapter-protocol/issues/147 a new optional property on the StackFrame has become available that indicates whether the "restart" request is available on a stack frame.
This feature request asks for using "StackFrame.canRestart" in VS Code.
I suggest this strategy:

if the DAP's supportsRestartRequest capability is true, the "Restart Frame" becomes available in the CALL STACK view's context menu
the "Restart Frame" command is enabled if a StackFrame has a canRestart property with value true or if the property is missing.

With this approach we make good use of the semantically overlapping properties supportsRestartRequest and canRestart by achieving UI stability.
/cc @connor4312 is this ok with you?

Cool, thanks for the merge! I'll adopt this in js-debug today

@weinand makes good sense
@connor4312 let me know once this is in js-debug nightly so I can work on the client to show it. Or should it just be there tomorrow?

I've adopted it in the linked commit and it'll be in the next nightly

Great, thanks

As Andre suggested if the supportsRestartRequest we show the action.
And the stackFrame.canRestart controls if it is enabled or not.
Try it out and let me know what you think


Expanded table layout changed after navigating to a specific route then navigating back

I have a table with with children expanded row (The row with caret is parent row and below it are its children) like the image below:

In this image, the dots (regardless green or red) of parent rows and children rows are in ordered. But, after navigating to a specific route (call it anotherRoute) on my React app and navigate back to this route, the table looks like this:

In the anotherRoute, I also have a table and I realize, after removing out the code for rendering table on that route, this weird bug is gone.
Is there any possiblity the reason for that weird bugs are from caching or rendering something of the table ?
P/s: Link for thí question on SOF: https://stackoverflow.com/questions/58360636/table-layout-changed-after-navigating-to-a-specific-route-then-navigating-back

@thelonglqd I don't have too much idea about this issue. But I guess it's not related to react-bootstrap-table. Let close it first until I got enough cases.


gunicorn dependency is missing in requirements.txt

gunicorn is not a pip install but an apt-get install. also it is not strictly required for running this code, it's just to serve it in production but will vary per environment.


Flower Power items?

Hi again,
Finally had time to start using this script - thanks!
I do have some queries that this document http://developer.parrot.com/docs/flowerpower/FlowerPower-BLE.pdf doesn't quite explain.
I assume if I have sensors capable of providing calibrated data, then this will be more accurate?
Also, what are calibrated	Ea, calibrated	Ecb and calibrated	Ec porous??
Thanks again!

@mlava

I assume if I have sensors capable of providing calibrated data, then this will be more accurate?

Correct

Also, what are calibrated Ea, calibrated Ecb and calibrated Ec porous??

These are the values from the Soil Electrical Conductivity sensor (http://www.veristech.com/the-soil/soil-ec). The Flower Power app uses them to calculate the fertilizer value.
cc/ @JeromeBouvard

Thanks @sandeepmistry


Full Stack Notes App

Project title
Notes App
Project Description
A Notes taking app where you can manage all of your notes.
Doc Domain

[] Solidity , Html, Css , JavaScript , Web3.js / Ether.js
[] Solidity , ReactJs /NextJs , Web3.js /Ether.js
[] Solidity , Python3 , Web3.py
[] Other , feel free to mention below
[] Spring Boot, Hibernate

Are you contributing under any open-source program ?
Yes

@dontwike closing this issue as we can't accept front-end projects anymore. Hope you'll understand!


Fix content type validation when charset is provided

The content types matching should only take into account the
type and subtype, not any of the content type parameters.
Content-Type grammar: https://tools.ietf.org/html/rfc7231#section-<IP_ADDRESS>
Fixes #185.

Codecov Report

Merging #312 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #312      +/-   ##
==========================================
+ Coverage   87.32%   87.35%   +0.02%
==========================================
  Files          19       19
  Lines        1815     1819       +4
  Branches      211      212       +1
==========================================
+ Hits         1585     1589       +4
  Misses        178      178
  Partials       52       52

Impacted Files
Coverage Δ

chalice/app.py
95.26% <100%> (+0.09%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5bb2239...a5946cf. Read the comment docs.


[fix][broker] Check replication cluster before starting the replicator

Fixes https://github.com/apache/pulsar/issues/20010
Motivation
PersistentTopicTest.testCreateTopicWithZombieReplicatorCursor is flaky because the cursor could still be created again in startReplicator, which could be called by:
onPoliciesUpdate
  checkReplicationAndRetryOnFailure
    checkReplication

In addition, onPoliciesUpdate is asynchronous, while testCreateTopicWithZombieReplicatorCursor updates the namespace policy nearly the same time, so there is a race with the order of updating AbstractTopic#topicPolicies.
Sometimes the policies update might fail because the topic might be deleted in PersistentTopic#checkReplication:

Deleting topic [xxx] because local cluster is not part of global namespace repl list [remote]

Modifications

Call checkReplicationCluster before calling startReplicator.
Add the local cluster to the replication cluster list
Sleep for a while in the test to reduce the flakiness caused by the asynchronous update of the policies

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/BewareMyPower/pulsar/pull/24

Before this patch:

After this patch:

Example logs of updating replication clusters:
2023-04-07T10:11:27,615 - INFO  - [pulsar-3575-5:BrokerService@2361] - [prop/ns-abc] updating with Policies(auth_policies=AuthPoliciesImpl(..., replication_clusters=[remote]
2023-04-07T10:11:27,619 - INFO  - [pulsar-3575-4:BrokerService@2361] - [prop/ns-abc] updating with Policies(auth_policies=AuthPoliciesImpl(..., replication_clusters=[],

We can see the interval is only 4 milliseconds but they happened in different threads (pulsar-3575-5 and pulsar-3575-4). Sometimes the 2nd update might happen first.

Codecov Report

Merging #20037 (bb254f1) into master (dd05408) will increase coverage by 38.28%.
The diff coverage is 81.70%.

@@              Coverage Diff              @@
##             master   #20037       +/-   ##
=============================================
+ Coverage     34.65%   72.94%   +38.28%
- Complexity    12429    31840    +19411
=============================================
  Files          1606     1865      +259
  Lines        125026   138040    +13014
  Branches      13667    15167     +1500
=============================================
+ Hits          43332   100693    +57361
+ Misses        76081    29364    -46717
- Partials       5613     7983     +2370

Flag
Coverage Δ

inttests
24.19% <0.00%> (-0.20%)
:arrow_down:

systests
24.96% <0.00%> (?)

unittests
72.23% <81.70%> (+39.13%)
:arrow_up:

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

Impacted Files
Coverage Δ

...uth/KubernetesServiceAccountTokenAuthProvider.java
77.27% <77.27%> (ø)

...ervice/AbstractDispatcherSingleActiveConsumer.java
92.37% <100.00%> (+13.74%)
:arrow_up:

...tent/NonPersistentDispatcherMultipleConsumers.java
71.60% <100.00%> (+32.09%)
:arrow_up:

...ersistentStickyKeyDispatcherMultipleConsumers.java
74.32% <100.00%> (+16.97%)
:arrow_up:

...ersistentStickyKeyDispatcherMultipleConsumers.java
85.44% <100.00%> (+3.67%)
:arrow_up:

...functions/auth/KubernetesFunctionAuthProvider.java
80.00% <100.00%> (+80.00%)
:arrow_up:

...s/runtime/kubernetes/KubernetesRuntimeFactory.java
80.42% <100.00%> (+80.42%)
:arrow_up:

...ime/kubernetes/KubernetesRuntimeFactoryConfig.java
97.36% <100.00%> (+97.36%)
:arrow_up:

... and 1514 files with indirect coverage changes

As discussed on the mailing list https://lists.apache.org/thread/w4jzk27qhtosgsz7l9bmhf1t7o9mxjhp, there is no plan to release 2.9.6, so I am going to remove the release/2.9.6 label


Fix chttp2 ping parser

The chttp2 ping parser parsed the opaque data in reversed order, then the PING response carried the reversed opaque data. This was not caught by our tests, since both the test server and client are using the same parser, and the reversed opaque data in the PING response was reversed back to the correct order by the ping initializer.
cc @yang-g: this should be the reason why our clients could not work with other servers.

Known issues:
Basic Tests Mac: #10003
Basic Tests Windows: #9620
gRPC_performance_pull_requests: #9399

Cherry-picked to 1.2.x by #10087.
Upmerged to master by #10105.


Error message rendered with b' prefix in py3

Improvement: in py3 retwist will no longer render the b' prefix before the param error message.

Added the sawkita/tox docker image because I did not manage to make it work for all envs and I remember Steffen also said he had issues with it. Instead now we have the kiwicom/tox docker image, same command, seems to be working for all tox envs.

Added the sawkita/tox docker image because I did not manage to make it work for all envs and I remember Steffen also said he had issues with it. Instead now we have the kiwicom/tox docker image, same command, seems to be working for all tox envs.


Exception: 'charmap' codec can't decode byte 0x9d in position 2392: character maps to

are u using ubunto or debain


Check whether it has a prototype in isStateless

This fixes #290 for me locally

This worked well.  Now I can delay converting the rest of my forms a little bit longer.  How can we get this pushed to NPM?


Call Access Token Factory once and cache the result until a 401 occurs

To improve the ability to "refresh" expired tokens, we should consider caching the access token provided by the factory. Then, when an HTTP request gets a 401, we call the factory again before re-issuing the request. That way the user can configure a process to "refresh" the token without forcing the connection to be reestablished

For the WebSockets transport, this has no effect. There is only ever a single request. This logic would not cover reconnecting in the event of something like #1159 (where the WebSocket is terminated when the token expires)
For the SSE transport, this only affects POST (send) requests. We would call the token factory again and re-issue the send. The unsent data would stay buffered in the pipe
For Long Polling, this affects POST requests like SSE, and the GET (poll) requests. The client would assume that a 401 error indicates that the data is still in the pipe for them to read. The server would be expected to keep data in the pipe in the case of a 401

We need to make sure that if two requests are outstanding simultaneously, the access token factory is only called once. So we should use a shared component and lock properly

Also need to make sure any new new requests wait while the access token is refreshed.


[debian bookworm] Python Feature fails to install with 'error: externally-managed-environment'

ref: #576
Test run: https://github.com/devcontainers/features/actions/runs/5293497423/jobs/9581722477#step:4:1608
[2023-06-16T19:24:31.534Z] #18 36.34 Processing triggers for libc-bin (2.36-9) ...
#18 36.39 Installing Python tools...
#18 36.42 Updating pip...

[2023-06-16T19:24:31.810Z] #18 36.77 error: externally-managed-environment
#18 36.77
#18 36.77 × This environment is externally managed
#18 36.77 ╰─> To install Python packages system-wide, try apt install
#18 36.77     python3-xyz, where xyz is the package you are trying to
#18 36.77     install.
#18 36.77
#18 36.77     If you wish to install a non-Debian-packaged Python package,
#18 36.77     create a virtual environment using python3 -m venv path/to/venv.
#18 36.77     Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
#18 36.77     sure you have python3-full installed.
#18 36.77
#18 36.77     If you wish to install a non-Debian packaged Python application,
#18 36.77     it may be easiest to use pipx install xyz, which will manage a
#18 36.77     virtual environment for you. Make sure you have pipx installed.
#18 36.77
#18 36.77     See /usr/share/doc/python3.11/README.venv for more information.
[2023-06-16T19:24:31.810Z]
#18 36.77
#18 36.77 note: If you believe this is a mistake, please contact your Python installation or OS distribution provider. You can override this, at the risk of breaking your Python installation or OS, by passing --break-system-packages.
#18 36.77 hint: See PEP 668 for the detailed specification.

Affirmative on azure-cli install being effected by this change on Debian bookworm as well: https://github.com/Azure/azure-cli/issues/26737 and https://github.com/devcontainers/features/issues/589
Reverting to -bullseye, this is (for now) working fine.

Add "version": "latest"
"ghcr.io/devcontainers/features/azure-cli:1": {
			"version": "latest"
		}

From what I can tell in the logs, specifically in this section:
[2023-06-16T19:23:54.049Z] #17 11.11 error: externally-managed-environment
#17 11.11
#17 11.11 × This environment is externally managed
#17 11.11 ╰─> To install Python packages system-wide, try apt install
#17 11.11     python3-xyz, where xyz is the package you are trying to
#17 11.11     install.

That happens when installing a package directly with pip onto system-installed Python. It looks like there is a suggestion right afterwards:
You can override this, at the risk of breaking your Python installation or OS, by passing --break-system-packages

This type of situation is usually avoided in development environments where developers will install libraries and packages in a virtualenv (which is also suggested).
I tried looking around the scripts that are running this test but couldn't figure out what exactly is doing these installs, but happy to offer some more help here if needed

Azure/azure-cli#26737 seems to use pipx as a resolution
Other alternatives are listed here.
Another alternative would be to use the system packages if version == "os-installed".

I have similar issues when I try to install docker-outside-docker. I get the error:
#13 32.44 error: externally-managed-environment
#13 32.44
#13 32.44 × This environment is externally managed
#13 32.44 ╰─> To install Python packages system-wide, try apt install
#13 32.44     python3-xyz, where xyz is the package you are trying to
#13 32.44     install.
#13 32.44
#13 32.44     If you wish to install a non-Debian-packaged Python package,
#13 32.44     create a virtual environment using python3 -m venv path/to/venv.
#13 32.44     Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
#13 32.44     sure you have python3-full installed.
#13 32.44
#13 32.44     If you wish to install a non-Debian packaged Python application,
#13 32.44     it may be easiest to use pipx install xyz, which will manage a
#13 32.44     virtual environment for you. Make sure you have pipx installed.
#13 32.44
#13 32.44     See /usr/share/doc/python3.11/README.venv for more information.

Even if I install python3-full before running the script I still get the same error. I use the following base image: mcr.microsoft.com/vscode/devcontainers/base:1.0.4-bookworm

@SebastianBalle Do you know how to override this version to make the installation work? I tried editing devcontainer.json to look like this based on these docs:
  "features": {
    "ghcr.io/devcontainers/features/docker-in-docker:2": {
      "dockerDashComposeVersion": "v2"
    },

But that doesn't seem to work.

@jtbandes
We store the script hardcoded in our internal repository. We simply changed the default value as follow:
DOCKER_DASH_COMPOSE_VERSION=${7:-"v2"} # v1 or v2

In our devcontainer.json we have "dockerComposeFile": "docker-compose.yml", in which we have specified the Dockerfile that contains the above mentioned default value.

In general the default version for docker compose should be v2 as v1 is no longer supported. The default value should either be modified by the maintainers or overwritten by the user.

Opened https://github.com/devcontainers/features/issues/745 for tracking v1 deprecation.
I should be able to get to ^ and https://github.com/devcontainers/features/issues/745 soon. Thanks for your patience.

As Docker v1 is deprecated, and Docker v2 works fine with debain:bookworm, we can simply close this issue once https://github.com/devcontainers/features/issues/745 is fixed.
Reasoning: As docker v1 is out of support by Docker, similarly, we can deprecate it's support and ask users to switch to v2.

As Docker v1 is deprecated, and Docker v2 works fine with debain:bookworm, we can simply close this issue once #745 is fixed.

That's not sufficient.  The python feature still fails to install on its own, regardless of whether or not any docker feature is installed.  That is, the python feature is broken as well.

Side note: Attempting to get a "partially working" feature install (mostly by disabling tool install) results in a broken setup.  In particular, pipx is installed, but attempts to reference global locations when called, meaning it fails due to permissions issues.  At the moment, installing python/pipx via the apt-packages feature is working for me.

Yeah, I'm getting the same error with bookworm and v2:
{
	"name": "Rust",

	"image": "mcr.microsoft.com/devcontainers/rust:bookworm",
	"features": {
		"ghcr.io/devcontainers-contrib/features/apt-get-packages:2": {
			"packages": "software-properties-common, libzstd-dev"
		},
		"ghcr.io/devcontainers/features/docker-in-docker:2": {
			"dockerDashComposeVersion": "v2"
		}
	},
	"overrideFeatureInstallOrder": [
		"ghcr.io/devcontainers-contrib/features/apt-get-packages:2",
		"ghcr.io/devcontainers/features/docker-in-docker:2"
	],
	"appPort": "9042:9042",
	"remoteUser": "root",
	"onCreateCommand": "",
	"privileged": true,
	"customizations": {
		"vscode": {
			"extensions": [
				"1YiB.rust-bundle",
				"panicbit.cargo"
			]
		}
	},
}


Expose validator settings as properties

I noticed that sometimes the settings of validators are accessed for use, but at the moment they are private attrbiutes, so this PR makes them public properties. For those validators that has their settings as public attributes, setters for the properties are added.

bors merge

bors merge

bors merge

Bors merge


Support OpenTelemetry

All client types

[x] We should introduce a new package that instruments the clients by wrapping the transport, which means we'll get consistent spans for all client types. Done in #438.
[ ] Metrics?

gRPC

[x] The @opentelemetry/instrumentation-grpc doesn't work for us because we don't use a generated client class (we invoke RPCs directly through Client#makeUnaryRequest). Fixed in #438.
[ ] We need to propagate the span context via the request metadata using the binary OpenCensus header format.

HTTP

[ ] The HTTP client should propagate span context automatically if @opentelemetry/instrumentation-fetch is registered, but we should test that and document it. Also need to check if it joins up correctly via grpc-gateway.

An initial release of @cerbos/opentelemetry (v0.1.0) is now available, which adds spans for Cerbos RPCs. It doesn't propagate the span context to the Cerbos server yet (that turned out to be non-trivial), but I'll look to add propagation in a future release.


Add option to use gpt-3.5-turbo

Add gpt-3.5-turbo functionality that uses the Chat Completions endpoint
Acceptance criteria:

[x] Add gpt-3.5-turbo to the list of radio buttons
[x] Query GPT-3.5 or Da Vinci 003 depending on which radio button was selected
[x] Return an error message still with GPT-3.5 if no table could be generated
[x] Add a tooltip that offers advice on whether a user should use Da Vinci or 3.5

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


perf: add checks for the internal data held by a message

There might be optimizations to be done if the internal data is held as
bytes already (like reusing containers to save allocs when parsing lots of data).

Thanks! Wi wait for Ash to weigh in

LGTM, blobbery approved


@conda-forge-admin, please add user @Stranger6667

Additional comment:
No response

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added user @Stranger6667 in conda-forge/schemathesis-feedstock#6.


Invalid param name --secure-key when runnning model locally

When new job were defined through GUI such message appear:
aetros start 1WPX06QP0 --secure-key=YOUR_KEY

but if we try to run this command we receive such exception:
aetros start: error: unrecognized arguments: --secure-key=my_key_id

Reading --help message for start I understood that it should be called with --api-key= param like:
aetros start 1WPX06QP0 --api-key=my_key_id

I think param name inside GUI or inside backend should be changed.

Thanks for reporting! Is fixed


fix(credential-ld): let verifiers use all supported verification methods

What issue is this PR fixing
fixes #1329
What is being changed
The VeramoLdSignature.getSupportedVerificationType() can now return string[] as well as string
This allows VeramoLdSignature suite implementations to signal compatibility with multiple Verification Method types for verification.
Quality
Check all that apply:

[X] I want these changes to be integrated
[X] I successfully ran pnpm i, pnpm build, pnpm test, pnpm test:browser locally.
[X] I allow my PR to be updated by the reviewers (to speed up the review process).
[ ] I added unit tests.
[ ] I added integration tests.
[X] I did not add automated tests because _________, and I am aware that a PR without tests will likely get rejected.

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

Comparison is base (7896cea) 89.83% compared to head (cde68aa) 89.83%.

Files
Patch %
Lines

...s/credential-ld/src/suites/JsonWebSignature2020.ts
66.66%
1 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             next    #1331   +/-   ##
=======================================
  Coverage   89.83%   89.83%
=======================================
  Files         176      176
  Lines       27010    27017    +7
  Branches     2120     2124    +4
=======================================
+ Hits        24265    24272    +7
  Misses       2745     2745

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


Fox Page - Other Opportunities

Overview
We should create a component to display the opportunities other than the main for FOX/FOXy, with all their info and CTAs to get started with each opportunity.
Any work on this should be considered presentational for now. https://github.com/shapeshift/web/issues/1834 will wire it up.
References and additional details
See #1785
Acceptance Criteria

A "Other Opportunities" heading is displayed
"Liquidity Pools", "Farming", and "Borrowing & Lending" accordions are displayed

They can be collapsed/expanded, showing the relevant list of opportunities (hardcoded)

Each item contains the relevant icon/name/APY/"Get Started" CTA for the opportunity

Need By Date
No response
Screenshots/Mockups

Mockups: https://www.figma.com/proto/WgALgik9YuWzq7VIM0JRdb/FOX-Page-Bounty?node-id=71%3A533&scaling=min-zoom&page-id=71%3A532
Estimated effort
4 hours

Already worked on it, the dumbest possible presentational component should be done already

Closed by https://github.com/shapeshift/web/pull/1862


Refactor exception handling and update comments in code

This commit refactors exception handling across the code by replacing generic exception throwing with specific Exception subclasses. It also includes updates to comments to improve clarity and readability.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

30 of 48 (62.5%)  changed or added relevant lines in 18 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.745%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Csv/CsvFile.php
0
1
0.0%

src/Rules/AbstarctRule.php
0
1
0.0%

src/Rules/Aggregate/AbstractAggregateRuleCombo.php
0
1
0.0%

src/Rules/Cell/AbstractCellRuleCombo.php
0
1
0.0%

src/Rules/Cell/ComboDateInterval.php
1
2
50.0%

src/Utils.php
0
1
0.0%

src/Rules/AbstarctRuleCombo.php
2
4
50.0%

src/Schema.php
2
7
28.57%

src/SchemaDataPrep.php
1
6
16.67%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3151

Relevant Lines:
3257

💛  - Coveralls


[New chain]: Q Mainnet

Summary
I would like to add the Q Mainnet network
Chain ID
35441
Chain detail URL
https://chainlist.org/chain/35441
RPC URL
https://rpc.q.org
Blockscout Client URL
https://explorer.q.org/api/v1/graphql
Etherscan Client URL
No response
Etherscan Client API URL
No response
Version
1.4.1
Address (Master copy)
0x41675C099F32341bf84BFc5382aF534df5C7461a
Deployment Tx hash (Master copy)
0xafa5befd0ecff012da32c33a5511c4d7e47c380a1d15a49cdee32d9e012db63d
Block explorer URL (Master copy)
https://explorer.q.org/address/0x41675C099F32341bf84BFc5382aF534df5C7461a
Address (Master copy L2)
0x29fcB43b46531BcA003ddC8FCB67FFE91900C762
Deployment Tx hash (Master copy L2)
0xaa47e53eb87a62caf9093275684618fc3b94b7342a4759632db2dad0f2a18770
Block explorer URL (Master copy L2)
https://explorer.q.org/address/0x29fcB43b46531BcA003ddC8FCB67FFE91900C762
Address (Proxy factory)
0x4e1DCf7AD4e460CfD30791CCC4F9c8a4f820ec67
Deployment Tx hash (Proxy factory)
0xbf39d10ec106bdd28cd00d058d86000dc7987b7dea495c53d9c7abd86eaa77a9
Block explorer URL (Proxy factory)
https://explorer.q.org/address/0x4e1DCf7AD4e460CfD30791CCC4F9c8a4f820ec67

I am not sure what I did wrong with the Blockscout Client URL

As I was able to execute the request successfully

Sorry for spamming. I tried to fix the link to the explorer but failed

/execute

/execute

/execute

Hello @KyrylR
I've been checking and the URL returns an error 403 when invoking from the script (even locally).
I have validated it manually and it is ok.
The PR is already merged.
Thank you.


How to force vcpkg to install dependencies with --head parameter, when using in manifest mode?

According to documentation of vcpkg_from_github:
At least one of REF and HEAD_REF must be specified, however it is preferable for both to be present.

I do not want to specify REF and SHA512 in my portfile.cmake. Instead I would like to build my library based on HEAD_REF  by adding --head when installing my dependency:
vcpkg install mypackage --head

This works in user-wide mode, however when using vcpkg in manifest mode it is assumed that REF parameter is specified which results in package installation error:
vcpkg.targets(104,5): error MSB3073: The command ""vcpkg.exe" install  --x-wait-for-lock --triplet "x64-windows" --vcpkg-root "<some_path>" "--x-manifest-root=<som_other_path>" "--x-install-root=<other_path>\vcpkg_installed\\" " exited with code 1.

@lukcz, could you try the the solution provided by https://github.com/microsoft/vcpkg/pull/16613#issuecomment-799801106 and see if it works for you?

Duplicated to https://github.com/microsoft/vcpkg/issues/16624.


number format pattern

We need to start working on the template. I hear quite a few complaints about the GUI presenting an ID column in a number format (e.g. with , separating thousands).
@karlcz @robes is there any js number formatting template that we can use?

Isn't this what the column-display pre_format field is for?

@hongsudt not sure what you mean by js number formatting template.
Sounds like if the concern is not formatting the number when it is a ID column, maybe we either need a heuristic for that, or we need a turn off formatting annotation?

I think this issue is mostly an ermrestjs issue?  We need to pick a posix-style formatting library so we can implement the pre_format display annotation. Then we can update the annotations doc with more precise dialect info for the format directives.

Issue moved to informatics-isi-edu/ermrestjs #299 via ZenHub


Drag problem when game object in transformed camera or container.

Drag problem when game object in transformed camera or container.
Version

Phaser Version: v3.19.0

Operating system: Windows 10 OS Version 1903 (Build 18362.356)

Browser: Chrome 64bit 77.0.3865.75

Description
Drag problem when game object transformed. Scaling or rotating a container, zooming or rotating a camera will cause this bug.

Additionally, when camera scrolled during dragging, the game object won't be transformed. input/dragging/camera move and rotate.js

Example Test Code
This code can be run directly at http://labs.phaser.io/edit.html
Camera
var config = {
    scene: {
        create: create
    }
};

var game = new Phaser.Game(config);

function create () {
    var rect = new Phaser.GameObjects.Rectangle(this, 100, 100, 100, 100, 0x00ff00);
    this.children.add(rect);
    rect.setInteractive({
        draggable: true
    });
    rect.on('drag', (pointer, dragX, dragY) => {
        rect.x = dragX;
        rect.y = dragY;
    });
    this.cameras.main.setOrigin(0, 0);
    this.cameras.main.setZoom(2);
    // this.cameras.main.setRotation(0.5);
}

Container
var config = {
    scene: {
        create: create
    }
};

var game = new Phaser.Game(config);

function create () {
    var rect = new Phaser.GameObjects.Rectangle(this, 100, 100, 100, 100, 0x00ff00);
    rect.setInteractive({
        draggable: true
    });
    rect.on('drag', (pointer, dragX, dragY) => {
        rect.x = dragX;
        rect.y = dragY;
    });
    var container = this.add.container(0, 0);
    container.add(rect);
    container.setScale(2);
    // container.setRotation(0.5);
}

Additional Information
Maybe related to #1542
Related doc: Phaser.Input.Events.html#event:GAMEOBJECT_DRAG
GAMEOBJECT_DRAG

Name
Type
Description

dragX
number
The x coordinate where the Pointer is currently dragging the Game Object, in world space.

dragY
number
The y coordinate where the Pointer is currently dragging the Game Object, in world space.

dragX and dragY should be coordinate in world space. But is not apply the transform from camera.
When within a container, we cannot using world coordinate directly, but this may still be a bug.
https://github.com/photonstorm/phaser/blob/e0ec646496fbc8a6083f8ef88813dd9a90233150/src/input/InputPlugin.js#L892-L902
hitTestPointer doesn't have bug. But processDragStartList processDragMoveEvent Directly using pointer.x.
https://github.com/photonstorm/phaser/blob/e0ec646496fbc8a6083f8ef88813dd9a90233150/src/input/InputPlugin.js#L1117-L1126
I think there should be a transformation. And I think dragX and dragY shouldn't using world space, parent container space may be better.
Thank you.

I'm having the same issue on 3.22.0

Thank you for submitting this issue. We have fixed this and the fix has been pushed to the master branch. It will be part of the next release. If you get time to build and test it for yourself we would appreciate that.


Add Unauthorized Login and Command

-> System compromise

Discussion in LUX. WG suggested to consider this already unders system compromise and add honeypot in the compromise description.


Control of components' state

Right now, Dataproc deploys Hadoop, Spark, PySpark, SparkSQL, Hive and Pig among other things like HDFS. It would be great to have a AI script which will control the state of each component (on/off).
Some of us need only Spark working with Google Cloud Storage and having other components installed and running just wastes the resources of workers.
Maybe we should have a script with small "settings" section which will control which components we need to be ON and which are not necessary and turn them OFF

This suggestion makes sense. We'd probably want to control the individual daemons with the scripts. I am assuming you'd want the ability to set/unset specific packages/daemons to be on or off for the lifetime of the cluster, correct?

@evilsoapbox, that's correct!

Hey @Spark2Ignite - We released an update today and I thought you might be interested based on this issue. We now allow you to set cluster properties to set XML and conf properties.
While it's not a direct answer to your issue, I thought you might find it interesting!
James

That's awesome, @evilsoapbox!


[Bug] Broken NPM dependencies

Version
1.9
Reproduction link
https://github.com/rstar2/material-dashboard-react
Operating System
Ubuntu  20.4
Device
Linux
Browser & Version

Steps to reproduce

Clone the Github repo https://github.com/creativetimofficial/material-dashboard-react.git  (or use the zip from CreativTim)
2, Try "npm install"

What is expected?
The project to install
What is actually happening?
zzzz:~/work/react-mui-dashboard$ npm i
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@babel/core
npm ERR<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/react-scripts
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@svgr/webpack
npm ERR<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!     node_modules/react-scripts
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!   9 more (babel-jest, babel-loader, ...)
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@babel/preset-env/node_modules/@babel/plugin-bugfix-v8-spread-parameters-in-optional-chaining
npm ERR<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@babel/preset-env
npm ERR<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!     node_modules/@svgr/webpack
npm ERR<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!       node_modules/react-scripts
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See /home/rumen/.npm/eresolve-report.txt for a full report.
npm ERR!  I have similar results with the React Argon template
Additional comments

Actually a little more search revealed the problem to be in NPM 7+ .
If someone else experience this see https://github.com/facebook/create-react-app/issues/10749
In my case it's easier to install the project with  yarn install

I had a similar problem, with node 14 and 16
npm ERR! code ERESOLVE npm ERR! ERESOLVE could not resolve npm ERR!  npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@emotion/react npm ERR<EMAIL_ADDRESS>from the root project npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@emotion/styled npm ERR<EMAIL_ADDRESS>from the root project npm ERR!     peerOptional<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!     node_modules/@mui/material npm ERR<EMAIL_ADDRESS>from the root project npm ERR!       1 more (@mui/icons-material) npm ERR!     1 more (@mui/styled-engine) npm ERR!   1 more (@mui/styled-engine) npm ERR!  npm ERR! Could not resolve dependency: npm ERR! peerOptional<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@mui/material npm ERR<EMAIL_ADDRESS>from the root project npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@mui/icons-material npm ERR<EMAIL_ADDRESS>from the root project npm ERR!  npm ERR! Conflicting peer dependency<EMAIL_ADDRESS>npm ERR! node_modules/@emotion/react npm ERR!   peerOptional<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@mui/material npm ERR<EMAIL_ADDRESS>from the root project npm ERR!     peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!     node_modules/@mui/icons-material npm ERR<EMAIL_ADDRESS>from the root project npm ERR!  npm ERR! Fix the upstream dependency conflict, or retry npm ERR! this command with --force, or --legacy-peer-deps npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
Fixed via installing with yarn as documented here

Hi Developers
For these sort of issues please use yarn.
All the best
Sajad


Comparisons with int and float (and probably the other APy*-type)

https://github.com/apytypes/apytypes/blob/e09260aa4217023ffba7afbf1bbd0d05935bc994/lib/test/test_floating_point.py#L420-L427
https://github.com/apytypes/apytypes/blob/e09260aa4217023ffba7afbf1bbd0d05935bc994/lib/test/apyfixed/test_arithmetic.py#L193-L200
One can probably remove is_zero, is_negative, etc when this works.

Floating-point comparison support to APyFixed was added in b5a37b63b417f097398693235406eaf6259ab602.

Floating-point and integer comparison support to APyFloat was added in 1c65c07a6765e8533fb558d8650b966d48ee591e.

Should be OK for now.


pip wheels metadata should indicate that they are built for x86-64

Right now the wheels are published as array_record-0.4.0-py310-none-any.whl ; which gets them installed on linux arm64 even though they provide x86 binaries.

@cantonios any update on this?

The release was done.
We need to bump the requirement on packges that depend on array_record to not have it install the older packages. But now we can prevent it. Look at https://github.com/tensorflow/datasets/pull/5109 for an example.


integrating meet.one wallet

Hi, I'm adding meet one the supported wallets on a web app. Is it posible to load an app on meet.one from the browser ?
On token pocket you can use this protocol https://git.io/fjMOQ I want to accomplish the same with Meet.one.  I would also like to switch to fullscreen on load?

I'm sorry if I've kept you waiting.
load an app on meet.one from the browser
We have already supported invoke wallet in external browser(not in dapps browsers in MEET.ONE client)
Open url with MEETONE

But the schema maybe unreadable, so we add one parameter external to invoke wallet, and wallet.isExternal is detected wheter in external browsers or not. (meet-js-sdk version >0.1.1 avaliable).
Supported schema is wallet.webview() and wallet.navigate()
For example :
testWebview = async e => {
  executing(e)
  try {
    let result = await wallet.webview(
      'https://meet.one?nav_alpha=1',
      // if true, invoke the wallet client and execute schema, if false, will execute schema only
      wallet.isExternal
    )
    if (result.code === 0) {
      success(e)
    } else {
      failed(e)
    }
    return true
  } catch (error) {
    failed(e)
  }
}

using meet-js-sdk to control fullscreen
No yet, we haven't supported now, but this Library will update continuously, some new schemas will update next version(control fullscreen need wallet client update). If we support this feature, I will mention you in this issue.

Great, thanks


Add Spanish video resource

This PR will add a new Spanish @midudev 's video resource which is very helpful, especially for the Spanish/LATAM community since @midudev has a huge impact in it.

Great contribution! Thank you.


ResourceAttributes

Currently the package @opentelemetry/resources and @opentelemetry/semantic-conventions expose ResourceAttributes which might create the name collisions and confusions what is what. I would think of renaming one of them.
At first I thought of @opentelemetry/resources but it simply sounds already very good so maybe semantic conventions should be exposed under different name for example SemanticResourceAttributes ?

Yeah, sounds good to me

How about ResourceAttributesNames, or ResourceAttributeKeys for semantic-conventions?
as these are not the attributes themselves, only their names.
BTW, isn't this package auto-generated from the proto files in the specification repo?

Should mirror the naming scheme of hte span attributes and semantic conventions


Adding ruff to dependencies as well as removing some rules

Summary
Added ruff as a dev dependency
Details
Removed some rules I personally don't care about, however you as the project manager should decide on rules you like and don't like. If you enable Ruff right now the project will light up like a Christmas tree. You can then easy see which rules you don't like and then add them to the list in the project.

I'll dig a bit deeper into it. But a linter would indeed be nice.


Multi-Wallet Settings and Currency Selector

This PR implements the ability to set a preferred list of currencies in settings/store.
The currency selector component can be set to radio buttons or checkboxes. Changing the selected currencies fires an event, parent views can use that, or get the state of the component to retrieve the list of active currencies (for example when saving).
Moderators are checked to see if they have at least one valid currency, and either flagged as invalid or not shown at all in the case of the browse moderators list. This should happen very rarely until we add the ability to use plug-ins and easily turn off currencies used by the wallet, until then it would require both the user and the moderator that was loaded to have edited config files.
This PR also adds a check on each moderator when the preferred list is set, to warn users if the moderator doesn't support any of the preferred currencies (ie: if you prefer to sell listings for BTC, moderators that only support BCH will have a warning message shown on them). This should primarily happen when no everyone has upgraded to 2.3 yet, though it's possible it will happen later if moderators edit their config file to disable some currencies.
This closes #1491 and closes #1555.

The thickness of the checkboxes is larger than the designs:

Also, can the hover and click area be on the entire row as opposed to just the box and label?

Looks like the padding is a little off on the radio style option:


#11 rename from miner to scraper

FIX #11

rename miner to scraper everywhere
Improvement of code quality

Codecov Report

Merging #12 into master will increase coverage by 0.34%.
The diff coverage is 95.65%.

@@            Coverage Diff             @@
##           master      #12      +/-   ##
==========================================
+ Coverage   87.61%   87.96%   +0.34%
==========================================
  Files           2        2
  Lines         105      108       +3
==========================================
+ Hits           92       95       +3
  Misses         13       13

Impacted Files
Coverage Δ

test/test_scraper/test_security.py
97.77% <100%> (ø)

src/scraper/security.py
80.95% <75%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 982f1eb...07f17e9. Read the comment docs.


ME Disabled (HAP) will no longer affect lspci output

Device
Novacustom v5x
RTE version
--
OSFV version
9603436e11cf52ab699fe4e928cdb6acb106198a
Affected component(s) or functionality
ME-neuter test
Brief summary
ME Disabled (HAP) will no longer affect lspci output, so we need to verify it in some other way.
How reproducible
100%
How to reproduce
Run the MNE004.001 Intel ME mode option Disabled (HAP) works correctly (Ubuntu) case
Expected behavior
It should pass
Actual behavior
It will always fail, since the HAP disabling doesn't affect lspci output anymore - ME will still appear as 00:16.0
Link to screenshots or logs
--
Additional context
No response
Solutions you've tried
No response

We have a change in progress here: https://github.com/Dasharo/open-source-firmware-validation/pull/346
But it seems it's been applied to static test, can we apply this to dynamic checks as well @wiktormowinski ?
Perhaps we can have a keyword to check ME state, and reuse it across these tests.

I just did the same in 3 test cases from me-neuter. By ''the same'' i mean the no-keyword version ... for now of course.
Same place:
https://github.com/Dasharo/open-source-firmware-validation/pull/346

still working on fix here: https://github.com/Dasharo/open-source-firmware-validation/pull/375

If ME is enabled, we can check Operation Mode field in HFSTS1. When the HAP bit is set, the ME should go into Debug Mode, indicated by the Current Operation Mode field set to 2.
Or check CBMEM logs:
[DEBUG]  ME: HFSTS1                      : 0x80022054
[DEBUG]  ME: Current Operation Mode      : 2
[INFO ]  HECI: coreboot in recovery mode; found CSE in expected Debug Mode state, skipping EOP

Newest version working on the new Develop branch https://github.com/Dasharo/open-source-firmware-validation/pull/403

I'm having issue on MSI where disabling ME (both HAP and Soft) wont output correct output in linux when running setpci -s 16.0 40.L - this breaks me-neuter tests

maybe on MSI roms Intel ME doesn't get initialized at all if we disable it.
Same thing happens on Z690-A DDR5, ME shows up on lspci only when set to Enabled.

I think we already do that here: https://github.com/Dasharo/open-source-firmware-validation/blob/develop/lib/me.robot#L27

@BeataZdunczyk @JanPrusinowski Should be relatively easy to improve this test by adding one more branch:

If we disable ME, we check if PCI device is present first

if it is not, the test can PASS, as the ME is disabled
if it is present, we must check the setpci command output as we do now.

This would make it work automatically for next test iteration on MSI.

ok the description problem is being resolved here: https://github.com/Dasharo/open-source-firmware-validation/issues/536
while this problem got fixed: https://github.com/Dasharo/open-source-firmware-validation/pull/579


Extra space generated in minified output

Using clean-css 3.1.2 the following source:
li:not(.foo).bar {
    display: block;
}

creates this output:
li:not(.foo) .bar{display:block}

Note the extra space after the closing parenthesis.

Ouch! You can use li.bar:not(.foo) in the meantime. Fix is coming.

Thank you for the quick fix!

Sure thing :+1: It's out in 3.1.3.

in 3.1.5 - issue is back again

I've just checked and it works fine for me. As a matter of fact I moved the code responsible for that to a different level so it's not applied to selectors anymore, see:
[root@81bc99fe636b app]# echo "li:not(.foo).bar {color:red}" | ./bin/cleancss
li:not(.foo).bar{color:red}
[root@81bc99fe636b app]# ./bin/cleancss --version
3.1.5

sorry, you're right

No probs!

Is it possible that this affects the space after the right closing bracket for background as well?
url(../../../logo_mark-small.png)50% 50% no-repeat; /* note the lack of space between ) & 50%*/

This unfortunately breaks IE8 :/

That's why you have spaceAfterClosingBrace compatibility switch, see: https://github.com/jakubpawlowicz/clean-css#how-to-set-compatibility-mode

Thank you @jakubpawlowicz I should have the docs more carefully :)

No problem! :)


Was 1.22 retagged?

Currently the SHA-256 checksum for https://github.com/dvorka/hstr/archive/1.22.tar.gz is 384fee04e4c80a1964dcf443131c1da4a20dd474fb48132a51d3de0a946ba996.
Originally it was c4995e7041dc66e2118f83bd4c6c7f4cff5b4c493ca28bd7e4aef76edeff71ba. See https://github.com/Homebrew/homebrew-core/pull/9590.
Was 1.22 retagged at some point?

@dvorka any idea what happened?

@dvorka gentle ping on this. Currently users cannot install hh in Homebrew due to this issue. We can't update the checksum until we understand the circumstances here.

My apologies for delayed response. There were inconsistencies related to tags and releases, but I'm not exactly sure whether I changed the tag (don't see anything suspicious in log and distro).
Perhaps I can release 1.23 - would this solve the problem?

Perhaps I can release 1.23 - would this solve the problem?

yes that would be perfect :)

I just released https://github.com/dvorka/hstr/releases/tag/1.23
Please let me know whether it helped. Thank you for letting me know and Homebrew distribution support!

Thank @dvorka (and you're welcome)! Voilà: https://github.com/Homebrew/homebrew-core/pull/18770


mkchain should be able to accept more than 1 bootstrap peer

Currently the mkchain arg --bootstrap-peer only accepts one peer. It should be able to accept multiple. Either by specifying the arg multiple times and/or being able to pass many peers to the single arg.

closed by #137


Implement K-NN in Python

This issue is about [Algo/DS Name/Question Name](link to resource for the Algo/DS/Question)

[x] I searched or browsed the repo’s other issues to ensure this is not a duplicate

[x] Issue Title is in form [Algo/DS Name] in [Language] (ex:- BubbleSort in Java)

[x] I will send my PR only after I'm assigned to this issue by any maintainer

[x] Assign this issue to me (I want to work on this)

[x] I ensure that I am not already assigned to 2 or more issues

Assigned the issue to you @AbhinavShaw09! You can work on this now!


Make the activitybar thinner with smaller icons

Issue Type: Feature Request
Please reduce width of activity bar, and reduce icons size as well :).
Also provide options in theme to change color of sidebar so that we don't miss sublume theme ;)
VS Code version: Code 1.33.1 (51b0b28134d51361cf996d2f0a1c698247aeabd8, 2019-04-11T08:27:14.102Z)
OS version: Windows_NT x64 10.0.17763

@Girijapathi there are several feature requests here. Please file a single issue per request. As for the sidebar color, this is currently themeable with https://code.visualstudio.com/api/references/theme-color#side-bar

Maybe title request is a duplicate @bpasero @misolori

I attempted to do this with https://github.com/microsoft/vscode/pull/52641 but need to consider a few things:

Allow this to be configurable via a setting
Options to include:

Responsive (based on dpi and window size)
Small/large
None (current)

The activity bar's icons are humorously large, and the bar has lots of empty wasted space.And the icons are not even that intuitive. Why not provide an option to make it look like Visual Studio or Android Studio, where the labels are thin vertical text? It will be more consistent with other IDE's, taking less space, and easier to see what they are, instead of cryptic images.

What is the status on this? I only see multiply closed feature-requests.
Also adding options doesn't limit anyone.

What is the status on this? I only see multiply closed feature-requests. Also adding options doesn't limit anyone.

Agree. I've been using this solution but it creates other problems.

Waste of space... So unfortunate.

I wish they were like in Obsidian, it's the ideal size (or even a little smaller). I can't believe this can't be changed, has anyone tried to tweak it?

For those who looking for solution, APC extension can modify activity bar size like this:
  "apc.activityBar": {
    "size": 32
  }

https://github.com/microsoft/vscode/issues/129771#issue-955874182

Sadly the APC extension mentioned above stops working starting from VSCode 1.93. I don't understand why this issue has been closed and is not being solved.
Any benefits of having a wide activity bar if VSCode is not targeting touch screens?

https://github.com/microsoft/vscode/issues/73713#issuecomment-2345378753

@Alecton4, see my previous comment - https://github.com/microsoft/vscode/issues/73713#issuecomment-2334038322 - because it contains a URI to a duplicate issue.