ERROR: expected number of inputs between 1 and 3 but got 9 inputs for model

{"timestamp":"2024-04-15T05:20:55.796456Z","level":"ERROR","error":"AppError(error message received from triton: [request id: <id_unknown>] expected number of inputs between 1 and 3 but got 9 inputs for model 'myserving')","target":"openai_trtllm::routes::completions","span":{"headers":"{"host": "localhost:3030", "user-agent": "OpenAI/Python 1.17.1", "content-length": "55", "accept": "application/json", "accept-encoding": "gzip, deflate", "authorization": "Bearer test", "content-type": "application/json", "x-stainless-arch": "arm64", "x-stainless-async": "false", "x-stainless-lang": "python", "x-stainless-os": "MacOS", "x-stainless-package-version": "1.17.1", "x-stainless-runtime": "CPython", "x-stainless-runtime-version": "3.10.5"}","name":"non-streaming completions"},"spans":[{"http.request.method":"POST","http.route":"/v1/completions","network.protocol.version":"1.1","otel.kind":"Server","otel.name":"POST /v1/completions","server.address":"localhost:3030","span.type":"web","url.path":"/v1/completions","url.scheme":"","user_agent.original":"OpenAI/Python 1.17.1","name":"HTTP request"},{"headers":"{"host": "localhost:3030", "user-agent": "OpenAI/Python 1.17.1", "content-length": "55", "accept": "application/json", "accept-encoding": "gzip, deflate", "authorization": "Bearer test", "content-type": "application/json", "x-stainless-arch": "arm64", "x-stainless-async": "false", "x-stainless-lang": "python", "x-stainless-os": "MacOS", "x-stainless-package-version": "1.17.1", "x-stainless-runtime": "CPython", "x-stainless-runtime-version": "3.10.5"}","name":"completions"},{"headers":"{"host": "localhost:3030", "user-agent": "OpenAI/Python 1.17.1", "content-length": "55", "accept": "application/json", "accept-encoding": "gzip, deflate", "authorization": "Bearer test", "content-type": "application/json", "x-stainless-arch": "arm64", "x-stainless-async": "false", "x-stainless-lang": "python", "x-stainless-os": "MacOS", "x-stainless-package-version": "1.17.1", "x-stainless-runtime": "CPython", "x-stainless-runtime-version": "3.10.5"}","name":"non-streaming completions"}]}
use client/openai_completion.py

I think I know the problem, my trition backend use trition with vllm.
Do we have a plan to support it?

it's not planned yet, but I think it's trivial to adapt the codes for your use case.

it's not planned yet, but I think it's trivial to adapt the codes for your use case.

Do you have any suggestions? I can try to implement it, and if I can, I can contribute this part of the code

Can you provide how to calling vllm-based triton backend? The grpc interface, the parameters for example to call the service.

You can see an example here.
https://github.com/triton-inference-server/vllm_backend/blob/a01475157290bdf6fd0f50688f69aafea41b04c5/samples/client.py#L192
if __name__ == "__main__":
    parser = argparse.ArgumentParser()
    parser.add_argument(
        "-m",
        "--model",
        type=str,
        required=False,
        default="vllm_model",
        help="Model name",
    )
    parser.add_argument(
        "-v",
        "--verbose",
        action="store_true",
        required=False,
        default=False,
        help="Enable verbose output",
    )
    parser.add_argument(
        "-u",
        "--url",
        type=str,
        required=False,
        default="localhost:8001",
        help="Inference server URL and its gRPC port. Default is localhost:8001.",
    )
    parser.add_argument(
        "-t",
        "--stream-timeout",
        type=float,
        required=False,
        default=None,
        help="Stream timeout in seconds. Default is None.",
    )
    parser.add_argument(
        "--offset",
        type=int,
        required=False,
        default=0,
        help="Add offset to request IDs used",
    )
    parser.add_argument(
        "--input-prompts",
        type=str,
        required=False,
        default="prompts.txt",
        help="Text file with input prompts",
    )
    parser.add_argument(
        "--results-file",
        type=str,
        required=False,
        default="results.txt",
        help="The file with output results",
    )
    parser.add_argument(
        "--iterations",
        type=int,
        required=False,
        default=1,
        help="Number of iterations through the prompts file",
    )
    parser.add_argument(
        "-s",
        "--streaming-mode",
        action="store_true",
        required=False,
        default=False,
        help="Enable streaming mode",
    )
    parser.add_argument(
        "--exclude-inputs-in-outputs",
        action="store_true",
        required=False,
        default=False,
        help="Exclude prompt from outputs",
    )

my trition backend also use trition with vllm，have a plan to support it？

Would be great if vllm option was supported.

I made some change to let it support vllm backend.
https://github.com/ChaseDreamInfinity/openai_triton_vllm


Can we add shouldIncludeTime optional ?

Hey,
Can we add shouldIncludeTime optional just like that;
utilities.js
let shouldIncludeTime;

  if (obj.shouldIncludeTime) {
    shouldIncludeTime = isLessThan7Days(obj.startTime, obj.endTime);
  }

Sample usage;
googleTrends.interestByRegion({
  keyword: 'Donald Trump',
  startTime: new Date('2017-02-01'),
  endTime: new Date('2017-02-03'),
  resolution: 'CITY',
  shouldIncludeTime: false
})

In that way we can get only daily data from google trends. Even less than 7 days.

@aonurbilgin I'm not seeing the daily data when I do this for the interestByRegion method. Could you post the results you are getting when you do this with the flag set to false versus set to true?

@pat310 Actualy I'm talking about interestOverTime method.  Could get results per DAY.
Just like that;
https://trends.google.com.tr/trends/explore?date=2017-10-01 2017-10-04&geo=TR&q=ios,android
My sample usage;
const googleTrends = require('google-trends-api');

googleTrends.interestOverTime({
    keyword: ['ios', 'android'],
    startTime: new Date('2017-10-01'),
    endTime: new Date('2017-10-04'),
    geo:'TR',
    hl:'tr',
    resolution: 'DAY',
    property: 'backend',
    category: 0
}, function(err, results) {
    if (err) {
        res.status(400).send('oh no error!', err);
    } else {
        let data = JSON.parse(results);
        for(key in data) {
        	console.log(data[key]);
        }
    }
});

I'm just want to get results as daily not hourly. If my date range less than 7 days isLessThan7Days function converts date range as a hourly. So in that case i cant get daily data if the date range less than 7 days.

@aonurbilgin Gotcha, no problem I will get that in there

@aonurbilgin Added an optional boolean granularTimeResolution to the input object. Will be available in the next release (version 4.3.0)

@pat310  Thank you


Looking for reference for MSAL.JS

Core Library
MSAL.js (@azure/msal-browser)
Wrapper Library
MSAL React (@azure/msal-react)
Public or Confidential Client?
Public
Documentation Location
learn.microsoft.com
Description
I cannot find a reference for this library - I have only found samples. I just want to see what parameters a function takes and what its return value looks like. (In this case, acquireTokenSilent). Is there a reference available?
(I will now look at the code, assuming there's no reference)

You can find our TypeDocs here


fix: remove unnecessary async to on_auth_state_change

What kind of change does this PR introduce?
It fixes on_auth_state_change not be async. Somehow it got reverted on a refactor (https://github.com/supabase-community/gotrue-py/commit/e7ebc64112d970673265c7b314a1e8820fc0f7e1)
What is the current behavior?
It was async.
What is the new behavior?
It's not async anymore.
Additional context
This causes problems when using the supabase client, since it's not being awaited: https://github.com/supabase-community/supabase-py/blob/main/supabase/_async/client.py#L90

Thank you for this PR.


Update for Xcode 10 compatibility and fix hardcoded search path in otest-query-appletv target

Fixes #752

I can update this @ExtremeMan — the PR in #755 was one part, we also need this PR as well

Looks like tests are timing out.

Ooops, saw your comment after posting mine. Let me know if you need help. I might try it out too.

Yeah there was something I had to fix that I remember from when I put the PR up. Taking a look now.

So in my testing, these tests don't work with Xcode 9.4 either. I think the last supported Xcode version was 9.2?

Oh, really? Travis is using 9.3

Yep you're correct. Looks like we need to bump it to 9.4 first? I tried tackling it but I'm lacking project context to fix that one :)

I have pushed several commits similar to what you proposed! Thanks for your PR. Take a look if xctool needs more fixing but overall I feel it should work now on Xcode 10

Travis is still flaky though :( will be looking into it later

Oh, I missed that piece. Can you rebase?

Updated!

Thanks a lot for pushing to fix xctool for Xcode 10 and providing fixes!


Make our GCE image pass the GCE requirements for official images

Current situation
This page details the requirements for official GCE images: https://cloud.google.com/marketplace/docs/partners/vm/build-vm-image#test_the_vm_image
From the checks listed there we are missing:

Having python installed
Having a working gcloud / gsutil (a docker error is printed when calling these commands)
Having dhcp in the process list. I'm not sure why they want that, though. We use systemd-networkd, maybe that was not considered when creating the list.

Impact
We can't provide official images until we pass this validation
Ideal future situation
GCE images include python and gsutil/gcloud work out of the box.
/cc @blixtra

These are the errors printed when trying to execute gsutil or gcloud:
marga_kinvolk_io@flatcar-container-linux-1-vm ~ $ gsutil
Got permission denied while trying to connect to the Docker daemon socket at unix:///var/run/docker.sock: Get http://%2Fvar%2Frun%2Fdocker.sock/v1.38/images/json?filters=%7B%22reference%22%3A%7B%22google%2Fcloud-sdk%22%3Atrue%7D%7D: dial unix /var/run/docker.sock: connect: permission denied
Warning: failed to get default registry endpoint from daemon (Got permission denied while trying to connect to the Docker daemon socket at unix:///var/run/docker.sock: Get http://%2Fvar%2Frun%2Fdocker.sock/v1.38/info: dial unix /var/run/docker.sock: connect: permission denied). Using system default: https://index.docker.io/v1/
Got permission denied while trying to connect to the Docker daemon socket at unix:///var/run/docker.sock: Post http://%2Fvar%2Frun%2Fdocker.sock/v1.38/images/create?fromImage=google%2Fcloud-sdk&tag=latest: dial unix /var/run/docker.sock: connect: permission denied
/run/torcx/bin/docker: Got permission denied while trying to connect to the Docker daemon socket at unix:///var/run/docker.sock: Post http://%2Fvar%2Frun%2Fdocker.sock/v1.38/containers/create: dial unix /var/run/docker.sock: connect: permission denied.
See '/run/torcx/bin/docker run --help'.

I thought this could be due to the docker service not being running, but starting it didn't make a difference:
marga_kinvolk_io@flatcar-container-linux-1-vm ~ $ sudo systemctl start docker
marga_kinvolk_io@flatcar-container-linux-1-vm ~ $ gsutil
Got permission denied while trying to connect to the Docker daemon socket at unix:///var/run/docker.sock: Get http://%2Fvar%2Frun%2Fdocker.sock/v1.38/images/json?filters=%7B%22reference%22%3A%7B%22google%2Fcloud-sdk%22%3Atrue%7D%7D: dial unix /var/run/docker.sock: connect: permission denied
Warning: failed to get default registry endpoint from daemon (Got permission denied while trying to connect to the Docker daemon socket at unix:///var/run/docker.sock: Get http://%2Fvar%2Frun%2Fdocker.sock/v1.38/info: dial unix /var/run/docker.sock: connect: permission denied). Using system default: https://index.docker.io/v1/
Got permission denied while trying to connect to the Docker daemon socket at unix:///var/run/docker.sock: Post http://%2Fvar%2Frun%2Fdocker.sock/v1.38/images/create?fromImage=google%2Fcloud-sdk&tag=latest: dial unix /var/run/docker.sock: connect: permission denied
/run/torcx/bin/docker: Got permission denied while trying to connect to the Docker daemon socket at unix:///var/run/docker.sock: Post http://%2Fvar%2Frun%2Fdocker.sock/v1.38/containers/create: dial unix /var/run/docker.sock: connect: permission denied.
See '/run/torcx/bin/docker run --help'.

And running with sudo simply says it can't find the command:
marga_kinvolk_io@flatcar-container-linux-1-vm ~ $ sudo gsutil
sudo: gsutil: command not found

This is because they are aliases, not real commands:
marga_kinvolk_io@flatcar-container-linux-1-vm ~ $ alias -p
alias egrep='egrep --colour=auto'
alias fgrep='fgrep --colour=auto'
alias gcloud='(docker images google/cloud-sdk || docker pull google/cloud-sdk) > /dev/null;docker run -ti --rm --net=host -v /home/marga_kinvolk_io/.config:/root/.config -v /var/run/docker.sock:/var/run/docker.sock google/cloud-sdk gcloud'
alias grep='grep --colour=auto'
alias gsutil='(docker images google/cloud-sdk || docker pull google/cloud-sdk) > /dev/null;docker run -ti --rm --net=host -v /home/marga_kinvolk_io/.config:/root/.config google/cloud-sdk gsutil'
alias ls='ls --color=auto'

Summary of what was discussed in slack:
We have two packages in the coreos-overlay repo that handle GCE images: oem-gce and coreos-oem-gce. The first one is the one used to customize GCE images. The second one is used to create a container that then runs with rkt.
The aliases come from coreos-base/oem-gce/files/files/google-cloud-sdk.sh, which is copied to /etc/profile.d/google-cloud-sdk.sh by the OEM Ignition file.

The rkt container that runs with the oem-gce service is intended for the "guest tools" that run on GCE instances, it's not intended to be used by normal users to run commands in it.  Also, it's quite outdated, we probably do want to run a more updated version of it.
As we've discussed with Kai, we could ship an alias that runs python with a docker container. It would be similar to the gcloud/gsutil aliases.
I found why those aliases didn't work for me: I needed to add my user to the docker group. We should try to find a way to do this automatically, so things work out of the box.

The fixes have been cherry picked to all channels and should be included in the upcoming releases. Once they are released we can upload the new images to GCE and try again.

Awesome! Thx so much, Marga.
Chris
On Fri, Sep 25, 2020, 06:43 Marga Manterola<EMAIL_ADDRESS>wrote:

The fixes have been cherry picked to all channels and should be included
in the upcoming releases. Once they are released we can upload the new
images to GCE and try again.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flatcar-linux/Flatcar/issues/186#issuecomment-698859570,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA4DA366DYLXOKEHQM2AGLSHRX5BANCNFSM4ROU6MJA
.


Multiple Period not working

Hi all,
I am trying to configure a scheduler that contain multiple period.

9AM to 6PM
1130PM to 1230AM(next day)

The issue I am facing is the solution giving me "Any" State. Hence the scheduler is not working when i define multiple period.
My Config is as follow:

This is the logs

hi @ku1918, Thanks for reaching out!
This is a known issue that can occur when combining 1-sided start/stop periods with other periods. We're investigating possible solutions, but for now the current workaround would be to change your two midnight periods to be full periods instead of 1-sided (i.e. start 23:30, stop 24:00, and start 0:00 and stop 0:30).
Please let us know if this does not work for you.

@CrypticCabub , you mean the issue being 23:30 cross over the next day?
So I need to have 3 period under 1 schedule.

9-6
1130-12
12-1230

Is that what you saying?

@ku1918, yes, you would need to have 3 periods like you describe to achieve the behavior you want.
The current scheduling logic looks at each day as individual a 24-hour periods. So to achieve a running period that crosses midnight you would need to have a running period at the end of the first day and at the start of the following day.


Icon Doc page inconsistencies

Edit: Looks like a couple of the icon issues have been reported here too: https://github.com/Semantic-Org/Semantic-UI/issues/6175. That being said, there are still a lot of other issues I mention below
Love the new addition of Font Awesome 5 to SUI!  Very nice work!  However, I was looking through the categories and noticed several inconsistencies between the Semantic UI Icons doc page and the documentation on Font Awesome's website.

The accessible icon (the modern wheelchair icon) is missing from Accessibility, Health, Users and People, and Vehicles categories (it's in the 'Brands' section only on the SUI doc page, it would be nice to include it in these other categories)
The youtube icon is missing from the Audio & Video section
The Chess Rook icon is misspelled in SUI, it has chess rock, it should be chess rook
The 'Currency' category icons have several inconsistencies, Bitcoin and others aren't listed here
SUI has the user doctor icon instead of user md like in Font Awesome, maybe make an alias so users could use either
The 'Payments' category icons also have several inconsistencies, amazon and apply pay are missing as well as many others

I also found several other icons that appear to be inaccurate on the docs page.  I've attached images of them.
Thanks so much and let me know if I can help in any way or if you want me to submit a PR for these. Are the main SUI documentation pages something we can submit PR's for?
Best,
-Justin
These are the incorrect icons:

These are what they should look like based on the FA 5 docs:

Thanks for the feedback, the icons which aren't in the categories must have been added after I created the PR because I went through and manually added them (it was time consuming 😆 ).
The icons which are broken I have done a lot of corrections you can take a look here. The code icons are broken because the docs has its own code icon which is conflicting with them @jlukic is aware of this.
For all the other suggestions I will look into them and will try and add them to my next patch!

😄 No worries, I bet! You did a great job though! Kudos!
Yeah, it looks like you fixed the arrows alternate horizontal, arrows alternate vertical, and the exchange alternate. It looks like the code, code branch, file code, and file code outline aren't mentioned in that new PR https://github.com/Semantic-Org/Semantic-UI/pull/6181 though.
Let me know if you have any questions on the others and thanks for the swift reply! You're crushing it @hammy2899 👍

This will be out before Friday with @hammy2899’s work

Thanks @jlukic and @hammy2899, we are using Semantic UI in both of our production web apps and have been loving it, keep up the great work!

Changes got delayed, will be out this weekend.

Getting out tonight


Fix Abstract Human map method

map is confusing, use set_user instead.

Coverage remained the same at 99.443% when pulling dc3447976edf95bf74081648584860329549b742 on fix-map into 2a22546da5dfbc1a82cdd4181c9a76d43bc9d945 on master.

Coverage remained the same at 99.443% when pulling dc3447976edf95bf74081648584860329549b742 on fix-map into 2a22546da5dfbc1a82cdd4181c9a76d43bc9d945 on master.


Add suggestions url field

This adds the ability to set suggestions url from the UI. Closes #513

Nice work! One question: Does suggest_url from components/search_engines/prepopulated_engines.json get filled into the Suggestions URL box for built-in search engines?

Does suggest_url from components/search_engines/prepopulated_engines.json get filled into the Suggestions URL box for built-in search engines?

Yes, it does.

Nice. Everything looks good.


Create base library.

Changes Unknown when pulling 60d293890c5f62acd1b9a3edfa3b6e54a358c0c1 on patch-1 into ** on master**.

Changes Unknown when pulling 4d32e97f8bf9ad312cc81e98bb056808c4c3f7f7 on patch-1 into ** on master**.

Changes Unknown when pulling 4d32e97f8bf9ad312cc81e98bb056808c4c3f7f7 on patch-1 into ** on master**.

Changes Unknown when pulling 0d735582037b5e0f5902638ea1f05c1ea7c8a206 on patch-1 into ** on master**.


Version 1.12.2

Does not work under version 1.12.2

Edit litemod.json and change   "mcversion": "1.12.1" to   "mcversion": "1.12.r2" and it should work.


Support for m17n on Linux

See the m17n home page. This might be an alternative to the present Linux solution (but probably not a replacement).

See also m17n-niv for a keyboard in need for kbdgen - there has been requests for Windows and OSX versions of that keyboard.


Regex never matches port

Description
The regexp here: https://github.com/hashgraph/hedera-sdk-js/blob/628258026122ac189ed28c503221bb17799e5cee/src/ManagedNodeAddress.js#L8 never matches the port part. Because of this isTransportSecurity() always returns false: https://github.com/hashgraph/hedera-sdk-js/blob/628258026122ac189ed28c503221bb17799e5cee/src/ManagedNodeAddress.js#L109
Steps to reproduce
Read the code
Additional context
No response
Hedera network
mainnet
Version
latest
Operating system
No response

Can confirm this is a bug. I believe this was fixed in Java, but the fix was not ported to JS.

Fixed directly on the develop branch with an accompanying test: https://github.com/hashgraph/hedera-sdk-js/blob/develop/test/unit/HostAndPort.js


Cannot completely remove dialogic from project

On Godot Mono RC1, I tried to use Dialogic. It did not work, which was partially expected as it has not been updated for this beta. The issue is, when deleting it however I now receive these errors intermittently.
 Script does not inherit a CustomResourceLoader: res://addons/dialogic/Resources/CharacterResourceLoader.gd.
  Script does not inherit a CustomResourceLoader: res://addons/dialogic/Resources/TimelineResourceLoader.gd.
  Script does not inherit a CustomResourceSaver: res://addons/dialogic/Resources/CharacterResourceSaver.gd.
  Script does not inherit a CustomResourceSaver: res://addons/dialogic/Resources/TimelineResourceSaver.gd.

I have disabled dialogic in plugins, then deleted it, and have no traces of it left in my project. What is causing these errors?

@dmitsuki I believe you have to go into your project.godot file (using VScode for me), and delete the plugin's inserted code in there.  It's labeled, but to be sure, I checked it against a clean project see what I needed to take out.

Ah, the issue was, after I did that, it still gave me the errors. Sorry for
not being clear enough.
On Sun, Feb 12, 2023, 2:07 PM MetaNoder @.***> wrote:

@dmitsuki https://github.com/dmitsuki I believe you have to go into
your project.godot file (using VScode for me), and delete the plugin's
inserted code in there. It's labeled, but to be sure, I checked it against
a clean project see what I needed to take out.
—
Reply to this email directly, view it on GitHub
https://github.com/coppolaemilio/dialogic/issues/1418#issuecomment-1427145003,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANPX5SH46VI3DNIJCG5B53WXFNK7ANCNFSM6AAAAAAUYR5D2I
.
You are receiving this because you were mentioned.Message ID:
@.***>

I pushed some changes to main that should fix Dialogic running on the latest RC1. I'll close but feel free to re-open if you still find some issues.

As this is fixed and now closed, just putting this out there for posterity.
@dmitsuki if for whatever reason you still want to delete dialogic from your project cleanly, I forgot to mention the .godot folder.  You'll have to go into the hidden files within the target project folder and may have to delete references to dialogic plugin cached in there...

issue still occurs


Poincaré's embedded file

Hello, Poincaré's embedded file keeps loading and reporting errors, it seems because the dimensions in the embedded file are incorrect. Have you made any changes to the embedded files?

Could you share the errors?


plugin API for IP bans

This competes with #1263. Pros: better performance. Cons: decreased portability (only Linux/MacOS/OpenBSD), plugins are generally sketchy (golang/go#19282).
Here's the mock plugin I used for testing: https://gist.github.com/slingamn/450b19fa7bb5e325a29b48f71f3c0842

This is actually fairly problematic: https://old.reddit.com/r/golang/comments/b6h8qq/is_anyone_actually_using_go_plugins/ejkxd2k/

Figured out a performance band-aid for #1263, so we're going back to that one.


Adding IPv4Registration mode capability

Signed-off-by: André Martins<EMAIL_ADDRESS>

[ ]   File: daemon/main.go:L138-149
 I think we should push the address unconditionally if the KVStore is available. We can think about a mode which does not require a KVStore in which forwarding is performed without policy enforcement.
 But how can we make sure the address is available?
 I'm sure the KV store supports an atomic lookup-or-add operation which allows to uniquely register the address or receive the information that somebody else is using it.
 Yes and that's sort of what's doing right now, the problem is, how can we know the that the node that is registering is not the same node that registered before? For example: something crashes and cilium has to be restarted again.
 We don't have to do this in this PR but we can add an identity to each node using a key or certificate and associate the lookup with the identity so a node can reclaim its own node address.
 For now, this could be done by the node itself by storing some metadata such as the hostid or the UUID of the fstab together with the registered prefix and verify that this is the same
[ ]   File: daemon/daemon/ipv4_prefix.go:L1-189
 It would be great to store both the IPv4 prefix and the IPv6 node address here to correlate the two.
[ ]   File: daemon/daemon/ipv4_prefix.go:L1-189
 We'll need the same for IPv6 as well. It's less urgent but the current method of deriving it from the IPv4 address will not always work as it's perfectly valid to have overlapping IPv4 addresses for the host itself.
At the very least, we should verify that the IPv6 node address is not conflicting.

Closing in favour of #2374


Remove forced console.log

Logging shouldn't be forced in production code.

Yes, please. How come it's taking so long to get this done?

Is there a reason this still hasn't been merged?


[zh] Fix change-runtime-containerd.md title display error

https://kubernetes.io/zh/docs/tasks/administer-cluster/migrating-from-dockershim/change-runtime-containerd/ page title display error:
title: 将节点上的容器运行时从 Docker Engine 改为 containerd weight: 8 content_type: task

The page is out of sync?

The page is out of sync?

I saw it last night. I forgot to sync.😅
I will sync it.

/hold

/hold cancel

/lgtm
/approve


Quest controllers are detected as Oculus Go controller on start

XRTK - Mixed Reality Toolkit Bug Report
Describe the bug
The Oculus quest defaults to only use an Oculus Go controller
To Reproduce
Create a project for Oculus using their native API (creating Oculus Native Controller Mappings) and run on a Quest
Expected behavior
Oculus Quest should support Oculus Touch controllers
Actual behavior
Oculus Go is detected
Your Setup (please complete the following information)

Unity Version 2019.1.4
XRTK Version 0.1.16+

Target Platform (please complete the following information)

Oculus Quest

Additional context
Also described on the Oculus forums here:
https://forums.oculusvr.com/developer/discussion/79152/project-issues-with-latest-firmware-integration-update#latest
This is a known Oculus platform issue introduced with API 1.39 with Oculus Quests updated to the latest firmware.  Compounded by the fact that Unity packages take priority over plugins in the Unity project (where Oculus place it), causing older DLL's to be used over newer versions.

Shouldn't this be in the oculus repo?


Indicate current documentation target

As mentioned in https://github.com/rust-lang/docs.rs/pull/2170. It allows to know what is the platform being checked currently. It looks like this:

cc @Nemo157

Used content "controlled by" font instead of the browser's bullet.


@noble/secp256k1 package removed from package.json but still required in node build

node build still seems to use the @noble/secp256k1 package. I built it locally and it seems to have @noble/curves/secp256k1.
build on npm registry:

latest build locally:

is it possible that build has not been pushed to NPM?

The master code of git is v5.3.1. And it hasn't been published to npm. So the build result is different.

it seems like the current node build on NPM is broken then?

You can use "tronweb": "5.2.0" - without caret . Its working for me

You can now using npm i<EMAIL_ADDRESS>to solve this problem. For security reason, I don't recommend using<EMAIL_ADDRESS>as the release note posted.


crash on destruction of result of dot

Next code crashes on destruction of a3:
    {
        xt::xarray<double> a1{1, 2};
        {
            xt::xarray<double> a2{
                {1, 1, 1},
                {1, 2, 3}
            };
            {
                xt::xarray<double> a3 = xt::linalg::dot(a1, a2);
                std::cout << a3 << std::endl;
            } // crashes here
        }
    }

Also, I expected (3 5 7) as result but the console says {3, 5} (wrong result, wrong shape).
Is this a bug? Or, maybe I'm using xtensor-blas in wrong way, here?

Hi and thanks for the first ever bug report on xtensor-blas!
You're absolutely correct, the shape of the result was wrong. Therefore, blas probably attempts to write into unallocated memory which is also why it's crashing on destruction.
The bug is fixed and a test was added in #17. After the test suite builds on travis, I'll merge.
Cheers,
Wolf

fixed! thanks again.

if you have any other feedback about xtensor/xtensor-blas, feel free to join us on gitter: https://gitter.im/QuantStack/Lobby
We're always happy to hear about use cases and potential improvements.

Thank you for fast response. I checked new code and found one problem. has_raw_data_interface doesn't exist in xt::detail namespace. It resides xt namespace instead in latest revision of xtensor repo.

@xylosper thanks for the reports!
Quick side note: xtensor-blas has not had a release tagged yet. However, when it is, we recommend using tagged / released versions of the dependencies rather than the state at a given point in time of the repositories.


EFS Provisioner is not resilient

Based on the configuration:
https://github.com/IBM/cp4d-deployment/blob/master/aws/efs_module/efs-provisioner.yaml
the pod wouldn't resist a node/cluster restart or a simple delete.
This should be wrapped in a deployment to make it more resilient.

Changes merged to master branch.


Dynamic loading not supported

When I try to load a .so extension at startup with
./php -z extension.so
I get the error
Failed loading extension.so:  Dynamic loading not supported
is this disabled?

Yes, it's disabled. Currently it is not possible to dynamically load libraries on Linux systems. Because the current compilation mode is pure static compilation mode, and under pure static compilation of Linux system, there is no dynamic library linked, and these external extensions cannot be dynamically loaded.
You can load a .so extension on macOS, because macOS platform does not support pure static library, almost all binaries need to link libSystem.dylib and libnetwork.dylib. So if we add libffi for static-php-cli, we can use xxx.so on macOS.

I am experimenting with glfw now and I realized it was compiled as an .so extension.
I thought maybe I could add it on startup.

on mac it says this, just so you know
PHP Warning:  Module "glfw" is already loaded in Unknown on line 0
Warning: Module "glfw" is already loaded in Unknown on line 0

For macOS, we have supported glfw extension. Just compile it bin/spc build glfw --build-cli.
For other extensions, I recommend that you first compile a purely statically compiled PHP, and then use phpize && ./configure && make && make install to find xxxx.so and use it as an independent php -d extension=xxxx.so binary .

Im using both mac and linux now
I meant the errors occur when I downloaded php 8.2.8 cli for mac here https://dl.zhamao.xin/static-php-cli/
so I didnt compile it.
On linux, I have the glfw.so extension so I asked about loading it at startup.
I understand you are statically loading extensions, but I dont understand why that prevents you from allowing dynamically loading additional extensions.

I understand you are statically loading extensions, but I dont understand why that prevents you from allowing dynamically loading additional extensions.

In the Linux operating system, purely static files cannot load and use dynamic libraries, and PHP's .so extension is also a type of dynamic library. This is due to limitations of the Linux itself, not PHP or other reasons. If and only if dynamic compilation is used when compiling PHP, and the ffi extension is enabled, we can use the extension=xxx.so method to dynamically load external extensions.

I meant the errors occur when I downloaded php 8.2.8 cli for mac here https://dl.zhamao.xin/static-php-cli/

The php binary hosted on my server is just a demo, including common extensions, but does not include the ffi extension.

If you want to use dynamic extension loading, why are you using static-php-cli? I think you just want to either download pre-built binaries for your distro, or compile php dynamically. Compiling php dynamically is fairly straight forward with most extensions, you can follow the official documentation.

A guy made a flatpak of the php, it is now portable with glfw !
@DubbleClick The main problem was to make it portable with glfw.

It works with flatpak, my php game is included with php and works with opengl.


Enhance test suite

I'm interested in contributing to this issue. Could you provide some more details on what enhancements are needed for the test suite? Specifically:

What areas lack coverage?
Are you looking for more unit tests, integration tests, or both?
Are there any known edge cases that need better testing?
Any specific goals or metrics you're aiming for (e.g., code coverage percentage)?
What would be your top priority for improvement?

Thanks!

@robhudson thank you for your interest!
Regarding your questions (I will probably also update the description with few of the following points):

probably the major theme around tests is being sure every protocol has all the features tested, for example:

ASGI has a test for pathsend, but RSGI doesn't have any test for file responses
RSGI has quite a few tests on body iterators, ASGI and WSGI don't

covering encoding/decoding on all protocols with tests would also be nice (at least it would make issues like #325 immediately evident)
given Granian implements its own awaitable classes, it would be nice to have tests on those at least for edge cases like cancellations and exceptions
in terms of unit vs integrations tests: unit tests would be nicer, but at the same time that would mean implement them on the rust side of things probably, and also the way the code is structured today in Granian – expecting a lot of things in place, both on the web side like the request and the Python part like the applications – makes this quite hard to achieve (unless we refactor a lot of Rust code to expose additional interfaces); so I would say at the moment integration tests like the one already in place is the way to go
I'm personally not a big fan of coverage percentage, especially when that number is not backed by other guidelines or strategies; on the other hand coverage might help identify parts of the code which are not tested – and again, not with the aim to have all the lines tested, but just as a metric to eventually pay more attention to specific code parts – but I'm not sure how that should be achieved in a mixed code base like this one, where we have coexisting Rust and Python code. I'm open to any investigations in that regards.

Given all the above points, I don't have any specific top priority; I think my general feeling is that the investment should be made on tests preventing issues like #325 to happen in the first place, rather than increasing the general coverage.


ServiceUnavailable Defunct Connection Address - AWS Internal load balancer

Hi,
We're getting this error from Neo4j refusing to connect every couple of minutes when we point the driver to the load balancer (aws elb classic). When we point the driver to one instance internal IP it works fine
<class 'neo4j.exceptions.ServiceUnavailable'>: Failed to read from defunct connection Address
Any help would be much appreciated.

Hi @gpierrick,
Do you have a timeout on connections at your load balancer?  If so, try to adjust your timeout and config connection lifetime in the driver config settings.
Read more about connection lifetime here

Thanks @zhenlineo for the quick answer. I've updated AWS ELB to 4000 (max allowed)
Looking at the link you sent, the driver seems to have 3600 as default correct?
Would this set up work? What would happen after 4000 seconds? Can the driver handle this gracefuly?

The driver maintains a pool. When driver wants to run a query (session.run) or starts a tx (session.beginTx), it will try to obtain a connection from the pool. Then the driver will go over the connections in the pool one by one until it grabs a connection that is still in a good state. If the driver found a connection is expired, it will directly call close on this socket and then move to check next connection in the pool.
You should not see any error, and hopefully no noticeable delay.
Let us know if you got any unexpected error.
Zhen

Hi @gpierrick,
We have not hear you more regarding this issue. I am assuming that you are happy with the current configuration. I am considering this issue as solved now. Please be free to let us know if you have more problems or more doubts regarding the problem you have.
Cheers,
Zhen

Hi Zhen,
I actually had to pause production for other reasons so I haven't verified the new configuration.
Thanks


[CLOSED] Make the task less verbose by default

 Issue by GoogleCodeExporter
Monday May 11, 2015 at 22:05 GMT
Originally opened as https://github.com/twogee/missing-link/issues/13

I find a little strange that there's so much things printed by default. I think
that in the nominal case, the useful information is the url and the response
code. If something doesn't behave correctly, then the end user could turn ant
into the verbose mode, and he can gets every info about its http request and
response.

See attached a suggested patch.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 6 May 2011 at 9:00
Attachments:

log-verbose.patch

 Comment by GoogleCodeExporter
Monday May 11, 2015 at 22:05 GMT

+1, I frequently end up running Ant in -quiet because of Missing Link output.
The output is useful when I explicitly ask for it (e.g. a debugging session),
otherwise it tends to end up as noise.

Original comment by<EMAIL_ADDRESS>on 9 Aug 2011 at 1:39

 Comment by GoogleCodeExporter
Monday May 11, 2015 at 22:05 GMT

Original comment by<EMAIL_ADDRESS>on 18 Aug 2011 at 1:16

Changed state: Accepted
Added labels: Milestone-1.1.3, OpSys-All, Type-Enhancement
Removed labels: Type-Defect

 Comment by GoogleCodeExporter
Monday May 11, 2015 at 22:05 GMT

Made the debugging utilize built-in ant logging levels for brevity.  By
default, the INFO level messages will now print as such:

     [http] HTTP Request
     [http] ********************
     [http] URL:                http://www.google.com/
     [http] Method:             GET
     [http]
     [http] HTTP Response
     [http] ********************
     [http] Status:             200

While run with -verbose, will look like:

     [http] ********************
     [http] HTTP Request
     [http] ********************
     [http] URL:                http://www.google.com/
     [http] Method:             GET
     [http] Headers:            no
     [http] Query Parameters:   no
     [http] Entity:             no
     [http]
     [http] ********************
     [http] HTTP Response
     [http] ********************
     [http] Status:             200
     [http] Headers:            yes
     [http]     HTTP/1.1 200 OK
     [http]     Transfer-Encoding: chunked
     [http]     Date: Thu, 18 Aug 2011 13:53:50 GMT
     [http]     X-XSS-Protection: 1; mode=block
     [http]     Expires: -1
     [http]     Set-Cookie: NID=50=r20TP23DAHGUgQgoUp8R8Nx-7CaSKw0bZbpcm76Thjuk3h3XHyuD2XhpuMrl0bAKNwOLS3I1rRzIRzzxB1NfUGPjFFHulwEhNPMkEhUS2z8gDTJ9-93mYIufMZ6fT-OU; expires=Fri, 17-Feb-2012 13:53:50 GMT; path=/; domain=.google.com; HttpOnly

     [http]     Set-Cookie: PREF=ID=c1b6da0aafda2bee:FF=0:TM=1313675630:LM=1313675630:S=5_g2PzPikR-aBqtd; expires=Sat, 17-Aug-2013 13:53:50 GMT; path=/; domain=.google.com
     [http]     Content-Type: text/html; charset=ISO-8859-1
     [http]     Server: gws
     [http]     Cache-Control: private, max-age=0
     [http] Entity:             yes

Original comment by<EMAIL_ADDRESS>on 18 Aug 2011 at 1:56

Changed state: Fixed

 Comment by GoogleCodeExporter
Monday May 11, 2015 at 22:05 GMT

I didn't utilize the patch on this Issue for a few reasons.  First, it has
conflicts caused by other patches and updates, second is I didn't particularly
like the formatting and levels chosen for particular parts of the output.

To me the submitted patch didn't clearly distinguish between req/resp areas
enough, so my changes might be slightly more verbose then the patch, but far
quieter then the previous output.  See the previous comment for the differences
now.

Changes will be available in the next release (1.1.3)

Original comment by<EMAIL_ADDRESS>on 18 Aug 2011 at 2:00

 Comment by GoogleCodeExporter
Monday May 11, 2015 at 22:05 GMT

Maybe we can do an option on the task to make it quieter.

I feel like your comparison to the copy method is unfair, the information
encompassing a file copy can clearly be wrapped up into a few words

To me, the basic information that I would always want to see about an HTTP
request, even in a "quiet" mode, would be the request type, URL, response
status.

My formatting is slightly verbose, because I looked for a way to try and
introspect the and "Project" at run time to determine the current logging
threshold, but, alas it appears Ant does not expose this (unless I am missing
something).  The reason I wanted to do this, was so that I could change the
output for the log lines that come in both INFO/VERBOSE if VERBOSE was/wasn't
set.  But since I can't find a way to do this, I went to the lowest common
denominator approach to have some consistent log messages

Original comment by<EMAIL_ADDRESS>on 18 Aug 2011 at 3:54

 Comment by GoogleCodeExporter
Monday May 11, 2015 at 22:05 GMT

If I may, I still find the output very verbose: 8 lines for 3 pieces of
information.
For instance the _verbose_ mode of the "copy" is printing things like:
"Copying /tmp/myfile to /tmp/myotherfile"

With logs such as the http task has, it would be:
Source File
********************
source : /tmp/myfile

Destination File
********************
dest : /tmp/myotherfile

overkill I think.
But it's just my opinion.

Original comment by<EMAIL_ADDRESS>on 18 Aug 2011 at 2:12

 Comment by GoogleCodeExporter
Monday May 11, 2015 at 22:05 GMT

> Maybe we can do an option on the task to make it quieter.

That would be nice then.

> I feel like your comparison to the copy method is unfair, the information
encompassing a file copy can clearly be wrapped up into a few words

I have to admit it is quite unfair, but I think two lines are very sufficient
for the 3 pieces of information we want.

>To me, the basic information that I would always want to see about an HTTP
request, even in a "quiet" mode, would be the request type, URL, response
status.

I agree. Hence my suggested patched.

>My formatting is slightly verbose, because I looked for a way to try and
introspect the and "Project" at run time to determine the current logging
threshold, but, alas it appears Ant does not expose this (unless I am missing
something).  The reason I wanted to do this, was so that I could change the
output for the log lines that come in both INFO/VERBOSE if VERBOSE was/wasn't
set.  But since I can't find a way to do this, I went to the lowest common
denominator approach to have some consistent log messages

Indeed you can't. But are these ASCII headers really needed ?

I don't want to appear stubborn of this nitpicking formatting issues on these
nearly indispensable http ant tasks, but here is what I suggested with my patch:
In INFO:
     [http] GET http://www.google.com/
     [http] Response: 200
In VERBOSE:
     [http] GET http://www.google.com/
     [http]     URL:                http://www.google.com/
     [http]     Method:             GET
     [http]     Headers:            no
     [http]     Query Parameters:   no
     [http]     Entity:             no
     [http] Response: 200
     [http]     Headers:            yes
     [http]         HTTP/1.1 200 OK
     [http]         Transfer-Encoding: chunked
     [http]         Date: Thu, 18 Aug 2011 13:53:50 GMT
     [http]         X-XSS-Protection: 1; mode=block

not perfect, url and method is repeated for the request and should probably be
removed from the verbose mode, but I think it is nicely readable thanks to the
indentation.

Original comment by<EMAIL_ADDRESS>on 18 Aug 2011 at 4:11

 Comment by GoogleCodeExporter
Monday May 11, 2015 at 22:05 GMT

Better?

C:\work_ml\ant-http\trunk\dist\ml-ant-http-1.1.3>ant -f build-sample.xml
Buildfile: C:\work_ml\ant-http\trunk\dist\ml-ant-http-1.1.3\build-sample.xml

http-get:
     [http] HTTP GET http://www.google.com/
     [http] Response Status: 200

BUILD SUCCESSFUL
Total time: 0 seconds

C:\work_ml\ant-http\trunk\dist\ml-ant-http-1.1.3>ant -f build-sample.xml
-verbose
Apache Ant(TM) version 1.8.2 compiled on December 20 2010
Buildfile: C:\work_ml\ant-http\trunk\dist\ml-ant-http-1.1.3\build-sample.xml

http-get:
     [http] ********************
     [http] HTTP Request
     [http] ********************
     [http] URL:                http://www.google.com/
     [http] Method:             GET
     [http] Headers:            no
     [http] Query Parameters:   no
     [http] Entity:             no
     [http]
     [http] ********************
     [http] HTTP Response
     [http] ********************
     [http] Status:             200
     [http] Headers:            yes
     [http]     HTTP/1.1 200 OK
     [http]     Transfer-Encoding: chunked
     [http]     Date: Fri, 19 Aug 2011 12:01:50 GMT
     [http]     X-XSS-Protection: 1; mode=block
     [http]     Expires: -1
     [http]     Set-Cookie: NID=50=Sb3fULSoiZva6UBGOanOTrIU5R8dhBHYpIZ0a94MD1PUhGP1kLatbzsu8Df2qtj6u_qwekPTsiW3xGFcnZuqhTEFE_o6mqvxuuX0JP4x9sNJJBHZEl0L-O_HDx-JKguU; expires=Sat, 18-Feb-2012 12:01:50 GMT; path=/; domain=.google.com; HttpOnly
     [http]     Set-Cookie: PREF=ID=ef7ffbc95c291570:FF=0:TM=1313755310:LM=1313755310:S=q86ysWb9dJ23UM2_; expires=Sun, 18-Aug-2013 12:01:50 GMT; path=/; domain=.google.com
     [http]     Content-Type: text/html; charset=ISO-8859-1
     [http]     Server: gws
     [http]     Cache-Control: private, max-age=0
     [http] Entity:             yes

BUILD SUCCESSFUL
Total time: 0 seconds

Original comment by<EMAIL_ADDRESS>on 19 Aug 2011 at 12:08

Changed state: Fixed

 Comment by GoogleCodeExporter
Monday May 11, 2015 at 22:05 GMT

Nicolas,

It just hit me.. there is no spoon.  No one said we had to play be the rules,
so I stole a play from the Spring playbook (picked this up when digging into
how Spring can do Dependency Injection into private/protected members with no
getters/setters)

  protected int getLogLevel() {
    try {
      for (final Object listener : getProject().getBuildListeners()) {
        if (DefaultLogger.class.equals(listener.getClass())) {
          final Field msgOutputLevel = listener.getClass().getDeclaredField("msgOutputLevel");
          msgOutputLevel.setAccessible(true); // there is no spoon
          final int logLevel = msgOutputLevel.getInt(listener);
          return logLevel;
        }
      }
    } catch (final Exception e) {
      throw new BuildException(e);
    }
    return Project.MSG_INFO;
  }

This allows me to modify the output depending on the log level and will make me
a happy camper, so I will shorten up the output for the default level for 1.1.4

Original comment by<EMAIL_ADDRESS>on 19 Aug 2011 at 11:44

Changed state: Accepted
Added labels: Milestone-1.1.4

 Comment by GoogleCodeExporter
Monday May 11, 2015 at 22:05 GMT

seems nice !

About the implementation hack, I would suggest you to be defensive regarding
the success of getting the error log. Ant developers are very attentive to
backward compatibility, but here a private field is used, so it is possible it
changes in the future. Then throwing a BuildException may be rude for such
logging problematic. A warn log should be sufficient.

Original comment by<EMAIL_ADDRESS>on 19 Aug 2011 at 12:24

 Comment by GoogleCodeExporter
Monday May 11, 2015 at 22:05 GMT

Excellent point, I've changed it to be more defensive

Original comment by<EMAIL_ADDRESS>on 19 Aug 2011 at 12:42

 Comment by GoogleCodeExporter
Monday May 11, 2015 at 22:05 GMT

Original comment by<EMAIL_ADDRESS>on 19 Aug 2011 at 12:51

Added labels: Project-ant-http


Autolint broken after Neomake update

Today after updating Vim plugins I started to get the following error from neomake-autolint plugin on startup:
Error detected while processing function neomake#autolint#Startup[12]..neomake#autolint#Setup[11]..neomake#autolint#buffer#makers:
line   12:
E15: Invalid expression: l:ft.'_'.l:maker_name.'_autolint'
Press ENTER or type command to continue

Knowing that it worked with identical configuration just couple of hours ago I looked for recent updates to related plugins and was able to pinpoint Neomake commit that broke Autolint: 28559e
It seems the behavior of neomake#GetEnabledMakers function has changed causing Autolint to crash.
I'm not too good with VimL, so couldn't come up with the fix right away.

@zoresvit @carlitux I submitted the following fix: 11ce867. It works for me locally. If this does not address the problem for you please let me know.


Dashboard: Added animations to Entertainment pages 3 through 6

Summary
This PR adds animations to pages 3 - 6 of the Entertainment template.
These animations are first iterations, missing animation features we'll want for polish are noted in this doc:
https://docs.google.com/spreadsheets/d/1iJ8Adqs4zGWteMoxVCZ2MM4LAza0LQmtPUX6I9qdshY/edit#gid=21496487
To-do
Apply animations to the remaining Entertainment pages.
User-facing changes
None.  Animations are disabled by default.
Testing Instructions
1.) Go to Dashboard.php and set enableAnimation to true.
2.)
At some point we'll want to update the propType that templates is inheriting from edit-story for the pages to remove the warning in console about invalid templates[1].pages[0].animations[0].iterations (mostly just noting it so that it's in the back of my brain)

This should be fixed in my current PR.  I accidentally put test animations into the DIY template and they were using the old format and throwing errors.  Once my current PR is up, they should go away 🎉


baobab-deku

Hello deku. I've just started implementing a very simple helpers lib to use deku along with Baobab. You can think of it as a centralized app state store.
I am just starting a discussion here about the choices made and so I can maybe collect some feedback. It just has a createDispatcher function and very simple actions (that you can easily compose) and a branch function which is a higher order component and binds props of the given components to some part of the tree.

Here is the gist:
import Baobab from 'baobab';
import {dom, element} from 'deku';
import {createDispatcher, branch} from 'baobab-deku';

const {createRenderer} = dom;

// 1. Creating our tree
const tree = new Baobab({counter: 0});

// 2. Creating actions to mutate the counter
function increment(tree, by = 1) {
  tree.apply('counter', nb => nb + by);
}

function decrement(tree, by = 1) {
  tree.apply('counter', nb => nb - by);
}

// 3. Creating our dispatcher & renderer
const dispatcher = createDispatcher(tree),
      render = createRenderer(document.body, dispatcher);

// 4. Creating a counter component
const Counter = branch({counter: ['counter']}, ({dispatch, props}) => {
  return (

{props.counter}

        <button onClick={dispatch(decrement)}>-</button>
        <button onClick={dispatch(increment)}>+</button>

        <button onClick={dispatch(decrement, 10)}>-10</button>
        <button onClick={dispatch(increment, 10)}>+10</button>

  );
});

// 5. Rendering our app, pass the tree as context & refreshing on tree update
function refresh() {
  render(<Counter />);
}

tree.on('update', refresh);
refresh();

:+1: looks nice

This looks interesting. I think an approach like this using cursors might be the right one. However, it seems like a weird definition of dispatch that you have here.
Why not dispatch into something like redux, and have your baobab tree updated in your reducer?

You could very dispatch data-actions like redux does and reduce them but the thing is Baobab enables imperative style updates and the actions aren't pure per se. It works more like Clojure's atom swapping like Om does, for instance.
One of the reason I went this way is because hot-reloading is free. You just need to hot-reload the rendering logic whereas with redux, as displayed here, you need to hot-reload both the rendering logic and the reducers.

Another solution, which is used by baobab-react is to partially apply the actions with the tree at component level, but this makes the higher order branch more difficult to use for virtually no benefit.

this is neat, I'd probably have made it pass the tree to render(<Counter />, tree) and made branch() work like react-redux's connect() or something.

Thanks @rstacruz. I forgot to pass the tree in context indeed.

What do you mean to work like react-redux's connect()?

react-redux has a connect() function that lets you select what part of the humongous state tree you want to use in the component:
module.exports = connect(map)(component)

function map (state) {
  // return a slice of `state` that the component concerns itself with.
  return { number: state.number }
}

You don't need this with Baobab because the tree has cursors.
This is what is done by:
const Counter = branch({counter: ['counter']}, Component);

It basically says, I want a prop to be named counter and map data from the tree at path `['counter'] to it.
Plus, this is statically analyzable and enables (not yet with deku but with React it does) a re-render directly from relevant components.

Might be good to get this into the docs to show an approach using immutable structures instead of Redux :)


bool(pa.scalar(False)) evaluates to True

Describe the bug, including details regarding any error messages, version, and platform.
This can easily cause bugs: assert pa.scalar(False) will not raise AssertionError!
Component(s)
Python

False is a valid value for a Boolean Scalar in this context, so it won't return 0, it will return an object specialized as boolean.
I guess the question would be: what wouldn't be acceptable to pyarrow as a scalar? Most of the types return scalar objects - including list, tuple, strings, etc.

@zfoobar pyarrow.BooleanScalar needs to implement the __bool__ magic method and return the appropriate python boolean.

I guess the question would be: what wouldn't be acceptable to pyarrow as a scalar? Most of the types return scalar objects - including list, tuple, strings, etc.

It's not about what is acceptable as a scalar. It's about which scalars should be considered Truthy and Falsey. In python the convention is:

Boolean/Integer/Float/Complex: 0 → False, other →True
Container-Types (string/list/tuple): same as bool(len(obj))

For other types (dates, timestamps, etc.) I'm not sure a boolean conversion makes sense.

Makes total sense.

I think it is worth discussing what to do in case when the boolean is null. It might make more sense to raise an Exception in this  case rather than silently casting to False.

@randolf-scholz PR updated with your wonderful insights.

@zfoobar LGTM. Possibly pylint will complain about is the unnecessary else, but that's nitpicking.

While I certainly agree the current behaviour is not ideal, I am a bit hesitant to just add __bool__ to the boolean scalars.

Several broader questions for __bool__ itself:

If we add it to BooleanScalar, we should probably add it to all other scalars as well?
If we add it to other scalar types, what should be its behaviour? For other numeric types it probably makes sense to follow 0 being falsey. But personally I am not fully convinced we should start making ListScalars truthy/falsey depending on the length of the list scalar value.
As @randolf-scholz already mentioned, we then also need to make a decision about the truthy value of null.

But aside from bool(..) there are other aspects that people would then expect from normal "scalar" values, such as equality. Currently __eq__ is very dumb, so doing pa.scalar(None) == pa.scalar(None) gives True (while that doesn't match element-wise equality behavior for the array with equivalent values) or it requires exact types (again not being consistent with the element-wise "equal" kernel)

We could also say: if you want python-like scalar behaviour, just call as_py() on the pyarrow scalar.
(and we could also raise an error in __bool__ always, to make it clear this method isn't actually implemented)

@jorisvandenbossche

We could also say: if you want python-like scalar behaviour, just call as_py() on the pyarrow scalar.
(and we could also raise an error in bool always, to make it clear this method isn't actually implemented)

Certainly that could be better than the current situation. I opened this issue after the following code silently failed:
assert pa.compute.all(pa.compute.invert(array.is_null())

Currently __eq__ is very dumb, so doing pa.scalar(None) == pa.scalar(None) gives True (while that doesn't match element-wise equality behavior for the array with equivalent values) or it requires exact types (again not being consistent with the element-wise "equal" kernel)

That just seems really bad behavior as well. pa.compute.equal(pa.scalar(None, type=pa.bool_()), pa.scalar(None, type=pa.bool_())) returns <pyarrow.BooleanScalar: None>, so (pa.scalar(None, type=pa.bool_()), pa.scalar(None, type=pa.bool_()) should return this as well. This should be fine, since if the result of this equality will be used in context where a python boolean is expected the casting to bool will raise the TypeError, if we follow option 1 from my other comment.

Just coming at this from the lens of a newb user - not even considering the Null case:
>>> a = pa.scalar(1)
>>> b = pa.scalar(2)
>>> assert(pc.equal(a,b))
>>>

^^ It wouldn't immediately occur to me to even know what as_py() is or to use it to ensure the expected behavior in this case. A solid RTFM could be in order but to my eye it wasn't immediately clear that this should be done.
Agree that eq returning boolean object, which can then be relied on to provide bool seems intuitive.
To @jorisvandenbossche's point, I did feel odd only adding bool implementation for a single data type. =/
Happy to take another swing at this if we can align on an implementation, otherwise as_py() it is...

I'll add a +1 for implementing the intuitive usage of __bool__ and __eq__ across scalar types (maybe split into 2 PRs). I don't think requiring usage of as_py() is as user-friendly in the long term.


'React/RCTDefines.h' file not found

I'm not able to run the project when i link this library, i followed the steps but when try to compile i get
'React/RCTDefines.h' file not found
I'm using a expo detached  app.
thanks for any help

Done!, I solve the problem
I need to install manually the search headers paths
usefull links:
https://docs.expo.io/versions/latest/guides/expokit.html#changing-native-dependencies
https://forums.expo.io/t/how-do-i-manually-add-pods-headers-public-to-the-header-search-paths-configuration/2427/4


Remove init.dart

We haven't used init.dart in several releases. We can safely remove this no-op
script now.

@iansf

LGTM


feat: pr github action workflow

Signed-off-by: Skye Gill<EMAIL_ADDRESS>

Review whether actions pass after https://github.com/open-feature/open-feature-operator/pull/93 is merged


Add getLikesMaxId, misc tweaks

Mostly authored by @glutamate

Thanks!  Released as version 0.6.  Sorry for the delay.

NP, thanks for the merge.


No difference between types True and types False

Hey,
i ran some performance tests with pyavroc, however i could not find any performance difference when using types True and False. This is my (simplified) Code:
avroFile = open(self.avrofile)
reader = pymzavro.reader.PymzAvroReader(avroFile, readerType=True)
for spectrum in reader:
spectrum.user
Am i doing something wrong?
Greetings
Marius

You're not doing anything wrong, you're doing everything right.
Those numbers are from Python 2.6. The results for Python 2.7 are pretty much equal for dictionaries and objects.
I've put some benchmarking in a branch; click the links here to see the different timings: https://travis-ci.org/walshb/pyavroc
I will update the doc sometime soonish.
Cheers
Ben


distribution: create sha256sum.txt file when creating version releases

This PR closes out https://github.com/bitly/oauth2_proxy/issues/479 by:

creating a sha256sum.txt file containing sha256 checksums for each pre-built binary that we distribute with our releases. we'll have to attach this file to each release manually for now. we may or may not want to commit this file into the repo in the future, but for now this should be a good start.
updates our README.md file with some instructions on how to verify pre-build binaries using the sha256sum.txt file we provide.

hmm looks like dist.sh wasn't updated to use dep

added in an additional commit to remove the gpm references in dist.sh and replace with dep. RFR


Added Resources in Web Development

Fixes Issue #163
Description:
I have added resources links in icons,colors and UI components sub section of Web Development
Resource URL:
https://lucide.dev/icons/
https://hypercolor.dev/
https://tailwindcomponents.com/gradient-generator/
https://flowbite.com/
https://ui.aceternity.com/
Checklist:
Before submitting your pull request, ensure that you have completed the following tasks:

[x] I have carefully reviewed and adhered to the contributing guidelines before creating this pull request.
[x] I followed the prescribed PR title template. (Check this if you're adding a resource)

Thank you @komal-agarwal5


Change of URLs to the MTR section

Hi! Looks like overnight url to the MTR section of docs has changed from /microsoftteams/room-systems/ to /microsoftteams/rooms/ leading to broken URLs. Is it possible to deploy redirections to the old urls so we would have some time to get all the indexers to notice the new URL?

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

ID: 15006a6a-d58e-5c43-7b57-2ec925990883
Version Independent ID: 19c6e539-15e7-e2b1-ed92-19ed1b1af48d
Content: Manage a Microsoft Teams Rooms console settings remotely with an XML configuration file - Microsoft Teams
Content Source: Teams/rooms/xml-config-file.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

hi @Larryngolog - We performed a large number of file renames in addition to the URL change from /room-systems/ to /rooms/. Links to original file names should automatically redirect to the new /rooms/ URL with new file names. Please let us know if you are still receiving errors when trying to get to the old URL with old file names.

Hello. This link https://docs.microsoft.com/en-us/microsoftteams/room-systems/xml-config-file does not redirect to https://docs.microsoft.com/en-us/microsoftteams/rooms/xml-config-file

Are you saying you have a bookmark that points to https://docs.microsoft.com/en-us/microsoftteams/room-systems/xml-config-file? The original file name was https://docs.microsoft.com/en-us/microsoftteams/room-systems/with-xml-config-file

Yes, I had sent a partner an e-mail with that link on Jan 22nd when it was working and on Jan 23rd it was not working anymore.
You can see that it existed - it's in on the Web Archive  sitehttps://web.archive.org/web/20191218114150/https://docs.microsoft.com/en-us/microsoftteams/room-systems/xml-config-file

ok thanks. We'll get a redirect put in place for that file. thanks for the feedback!


Update README.md

Added a link to where the releases are published.

Thank you, but this seems redundant. There's already a link to the main NuGet package.
CC @flcl42

oops! thanks for checking.


Cannot close editor window using Command + W -> see screenshot of error attached

VSCode Version: 1.4
OS Version: El Capitan

Steps to Reproduce:

Command + W

.

Some commands changed their ids, see https://github.com/Microsoft/vscode/issues/6605


Resolve issue #5 and and updated board properties

This time with spaces.

Is there a reason this pull request isn't being accepted? I would like to work on it and make a fix if so. The get_all_boards function is critical to something I am trying to implement using this library.

Nah, we're just kind of slow. I was having a team member review it but I guess better to do it myself when I have time, which could be a while. At a glance it is very good and some things are applicable to BASC-py4chan as well. I guess if you do have the time could you summarize all your changes so I can figure out what it's all about?

Oh good to hear! I was worried it was dismissed for some reason and wanted to know how it could be fixed. Changes are summarized below:

Lots of basic syntax updates (eg turning "4chan" into "8chan")
Updated the get_all_boards() function to properly handle the 8chan api json response and return a list of Board objects
Included an example usage of get_all_boards() for the docs
Added all the metadata contained about each board to the Board object using the @property decorator. Fairly constant data like title and tags are cached while changeable things like number of active users and time make requests to 8chan again using  _get_metadata()

Hope that clears things up! Let me know if there's anything else that can be tweaked/improved.

Alright, basically you fixed py8chan to work with their new catalog system. Well that is awesome so I will just merge and publish it.
And hey if you want to try and do some more work on this such as building a doctest system (a very elegant Python method for test driven development) or other interesting things, building these sets of API wrappers taught me everything I had to know about Python, not my masters degree.
Primarily our team is working to build an alternative to the aging Java-based Asagi scraper at https://github.com/bibanon/eve . I think that if we integrate these API wrappers with that it would make it possible to have a near universal chan or even forum scraper system as long as it uses the same class methods. Maybe that might be good to work together with on the future?
https://docs.python.org/2/library/doctest.html

Alright I published a new version on pypi, v0.0.4, thank you for your contribution.
One regression I was having after merging woodenphone's pull request was that this unicode assertion wasn't working. What's up with it? I disabled it for this release.
https://github.com/bibanon/py8chan/pull/13
https://github.com/bibanon/py8chan/commit/6fb0fa836963f2bf5d8b9ecd35b45eab5ad2e3b3
BTW there were these two intriguing code suggestions that were never packed into a pull request so I never got around to them. Could you experiment with them and see if they are good to merge?
https://github.com/bibanon/py8chan/issues/11
https://github.com/bibanon/py8chan/issues/12

Sorry for a late reply. Stoked to see the pull request accepted! I would
definitely like to work on more features in the future. The Eve project
sounds really interesting as well. Do you see it performing the work of the
py8chan API wrapper in the future? If that's the case it might make more
sense to focus my efforts there and contribute to a more universal tool.
In regards to the specifics you pointed out:
#13 : My best guess is the assertion is a typo. Instead of testing 'board'
it should test 'uri'. It doesn't make much sense once you realize a few
lines up there is another assertion test for 'board' against a different
outcome so one of the two is bound to fail.
#11 : The fix for poster_id is solid so I implemented it as well. I think I
ran into a new issue regarding the Post structure. Basically the first Post
in a thread has a ton of extra information associated with it that the rest
of the Posts don't have (see below). Not sure what the fix for this is.
Seems like it might involve some rethinking of the Post class.
First Post Subsequent Posts
no : 12021208 no : 13125966
sub : QTDDTOT: Questions that don’t deserve their own thread resto :
12021208
com : <p clas [...] com : <p class[...]
name : Anonymous name : Anonymous
time :<PHONE_NUMBER> time :<PHONE_NUMBER>
omitted_posts : 0 sticky : 0
omitted_images : 0 locked : 0
sticky : 1 cyclical : 0
locked : 0 bumplocked : 0
cyclical : 1 last_modified :<PHONE_NUMBER>
bumplocked : 0 id : 196f83
last_modified :<PHONE_NUMBER>
id : 674207
tn_h : 61
tn_w : 255
h : 400
w : 1680
fsize : 55627
filename : QTDDTOT
ext : .png
tim : 5fe06f6ca027c1c4[...]
fpath : 1
spoiler : 0
md5 : FY89nitoGdj+XGd2VNWg6A==
resto : 0
#12 : This seems like a good fix as well so I've included it. Basic testing
worked well.
All this should go into a pull request soon. Just wanted your opinion on
how to handle those difficulties I found in #11.
On Sun, Apr 7, 2019 at 8:46 PM Antonizoon<EMAIL_ADDRESS>wrote:

Alright I published a new version on pypi, v0.0.4, thank you for your
contribution.
One regression I was having after merging woodenphone's pull request was
that this unicode assertion wasn't working. What's up with it? I disabled
it for this release.
#13 https://github.com/bibanon/py8chan/pull/13
6fb0fa8
https://github.com/bibanon/py8chan/commit/6fb0fa836963f2bf5d8b9ecd35b45eab5ad2e3b3
BTW there were these two intriguing code suggestions that were never
packed into a pull request so I never got around to them. Could you
experiment with them and see if they are good to merge?
#11 https://github.com/bibanon/py8chan/issues/11
#12 https://github.com/bibanon/py8chan/issues/12
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bibanon/py8chan/pull/15#issuecomment-480673467, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AK-mOzhPf7vtN_vSg4qrwRanhgeTSs6Xks5veruogaJpZM4cMwX9
.


Windows user feedback.

Hi, leaving feedback for dark theme on FF 57.0.1 x64 on windows 10. The only issues that I've found so far are:

the connection details sub-menu background color makes unable to read the text

a white horizontal line on the right context menu

Thanks for your work.
Best regards.

Just to add to this, I have the same issue with the download list in the toolbar, as above with the connection details.
The library (CTRL+J) is all-white.
Can confirm that white line.
Happy to contribute any code or testing as required.

Hi @chinook77, thanks for the feedback, it's really appreciated!
The connections sub-menu was overlooked on my part (it's also not correct on macOS), so that will be fixed ASAP.
@CaptaPraelium, I believe that the download drop down issue should now be resolved - can you let me know if the latest version is ok?
In regards to the white line occurring on the context menu, can either of you tell what that is coming from?
It's not appearing on my mac, so it's hard to troubleshoot unfortunately

Update: connection panel should be fixed (at least in terms of readability).
I'm debating whether I should change the color of FF's default secure connection color (green) to match the color scheme used in this repository (by default, darkcyan).  If anyone has any opinions/preferences, please let me know.

"In regards to the white line occurring on the context menu, can either of you tell what that is coming from?"
I think because of this grey separator:

Connection properties and download toolbar looking great, thank you. That was quick.
I think it would be great to see the green replaced with the user's colour scheme.
Not sure where that white line is coming from but it may help to mention that it is actually not white but the light grey colour from your scripts. It is interesting to note that, we still have the separator shown by chinook, in addition to the grey line... But the grey line covers the full width of the menu as the separator used to, and now the separator at the top there is shorter like the others.

Glad it's working!
I was thinking it should be changed too (I've changed it on my own browser), I just don't want to be to invasive with my changes.
But, then again, for those that don't like it, all they would have to do is remove a single @import line.
I'll try and get a fix out soon for the context menus - I know those don't look great for you guys

They do now! Nice work.
By the way, I highly recommend this with the https://github.com/m-khvoinitsky/dark-background-light-text-extension. There are other 'dark mode' extensions, but this one allows you to make a custom colour scheme.
I have noticed a few minor inconsistencies with the firefox dark theme but I'll make new bugs for those ASAP.

Hi @CaptaPraelium,
I'm glad they're looking better!
Thanks for the heads up about that extension, I'm excited to give it a try.
As always, I appreciate the feedback.  If possible, please submit each bug/inconsistency you notice as it's own issue (makes it easier to keep track of everything and ensures the Issue feed only consists of ongoing/unaddressed problems) :)

Resolved - 7503a89 and 512e490

Hi, this separator It's back. How to remove it?

Thanks in advance.

Hi @chinook77, check out issue #60...there's a FF 58 bug that I believe is causing it to return.  The current workaround is to use the windows version of userChrome linked there.
Let me know if that fixes things for you.

Thanks, that works.


Dedup cli-kit when bundling hydrogen

WHY are these changes introduced?
Since cli-hydrogen is an external dependency that at the same time depends on cli-kit which exists in this repo, it creates a weird dependency tree that eventually causes esbuild to bundle 2 identical copies of cli-kit
Fixes #0000

WHAT is this pull request doing?

Add a new ESBuild plugin to dedup cli-kit when bundling.

How to test your changes?
See the snapshot in the comments, if you install that and try any app and hydrogen command, everything should work.

Post-release steps

Measuring impact
How do we know this change was effective? Please choose one:

[ ] n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
[ ] Existing analytics will cater for this addition
[ ] PR includes analytics changes to measure impact

Checklist

[ ] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[ ] I've considered possible documentation changes

Is this the first time we've encountered this type of scenario? I would assume we have other scenarios where dependencies are shared in this way, and curious why this is the first time we have to write some custom dedup code for it?

pnpm does a good job on hoisting all transitive dependencies to the root node_modules when the versions are compatible. So in most (almost all) cases we only have 1 copy of each dependency.
If two of our dependencies use different (incompatible) versions of another sub-dependency, then esbuild will include both and that's correct. Example, a package depends on<EMAIL_ADDRESS>and another one depends on<EMAIL_ADDRESS>esbuild will bundle both. Not sure if this is actually happening for any specific dependency, but it could.
The case with cli-hydrogen and cli-kit is a bit special, since cli-kit is part of this monorepo, when we use it it is not installed as a normal dependency, it is read directly from source (with a symlink).
But when we install cli-hydrogen as a external dependency, it tries to use an external cli-kit too so we need to force it to use the local one (we do this with custom resolutions in the main package.json).
But, even doing this, it still thinks they are different cli-kits, so this is why we need this plugin.
This also happens with @shopify/plugin-cloudflare for the same reason: is part of this monorepo but it is also used by cli-hydrogen. I didn't include a plugin for that case because I was having some issues to make it work and is not as important: that package only has 2 small files, is not a big deal to have them duplicated for now.
Hope is more clear now 🙏

Very clear, thank you so much for that explanation!


Update regenerate-tutorials.yml

@jdbaldry would you mind reviewing this one since it adds the mimir repo

Tested in https://github.com/grafana/killercoda/actions/runs/11348355655, don't see any updates which makes sense given that it's already there in https://github.com/grafana/killercoda/tree/Jayclifford345-patch-1/mimir/play-with-mimir


Rename the id for the copy paste toolbar

For some reason, the copy paste toolbar was inheriting the styles of the
one in the contact list on the Firefox browser, this renames it and the
issue no longer seems to be present.
Tested by loading the copypaste UI in Firefox and Chrome

:+1:


Resolved as of 2.0.6

          Resolved as of 2.0.6

Originally posted by @Wikxen in https://github.com/iP1SMS/disposable-phone-numbers/issues/4#issuecomment-517272955

What's up?


Kibana 6.3.1 beta 1 - sessions timing out even with auto-refresh enabled

Since upgrading to 6.3.1 and installing Kibana plugin 6.3.1-14-beta-1, any dashboards left sitting with auto-refresh enabled will expire their session and display an "Unauthorised" banner after one hour.
This was not happening with 6.3.0.
Obviously it's not a good situation for unattended dashboards, as they need regular intervention.
Kibana / ES 6.3.1
Ubuntu 16.04
Firefox 61.0.1

This was fixed in the lates v14 release of the plugin


If zero groups are returned

If MediaFrameSourceGroup.FindAllAsync() returns zero groups then make sure you request the Webcam capability in your Package.appxmanifest file.

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

ID: fcc6c3a0-84f4-2c8d-cbb4-cfc7dd5032ff
Version Independent ID: 3954dd9a-3753-c8db-3caa-c8e25e2a4cab
Content: MediaFrameSourceGroup.FindAllAsync Method (Windows.Media.Capture.Frames) - Windows UWP applications
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.media.capture.frames.mediaframesourcegroup.yml&version=GBlive&_a=contents
Product: uwp
Technology: winrt-reference
Git Login: drewbatgit
Microsoft Alias: drewbat

Hi @dgough , thanks for the additional information. If I understand correctly, you'd like for this to be a remark added to this Method?

Yes, I think it can save people some time and frustration if the method description mentioned this. It took me awhile to figure out what I was missing when I was trying to use this method.

Excellent, I will go ahead and append this to the reference page.

The remark has been added and should appear live soon. Thanks, @dgough .


Editorconfig: switch to max line width of 80

This commit changes the max_line_width to 80 in .editorconfig. This is not mandatory nor enforced. However, 80 is also the default for commit messages and some editors guide users to follow this rule. For example, my CLion IDE automatically shows a line at width 80:

I think that a line-width of 80 is more suited than 100 from personal experience. However, I also used 100 as default for a long time.
Checklist

[ ] Sensible git history (for example, squash "typo" or "fix" commits). See the Rewriting History guide for help.
[ ] Update the changelog (if necessary)

PS: @nicholasbishop @GabrielMajeri : Merry Christmas!

This is not mandatory nor enforced.

Then I'm not sure that I understand the effects of these changes. If the line size setting is not enforced by rustfmt, will it only be visible as a hint in certain editors/IDEs?

Merry Christmas!

Thank you, sorry for the late response, I wish you Happy Holidays as well!

If the line size setting is not enforced by rustfmt, will it only be visible as a hint in certain editors/IDEs?

Right. The whole idea of the .editorconfig project is to change editor and IDE settings as you type. Because of this, editorconfig has a really high adoption even in very small editors and barely known IDEs.


Scheduled snapshots

Ok... so with the latest git push, this PR is now complete and ready for final review and integration. Please comment as you see fit.
If you want to "try it out", you can go into proxyfsd/file_server.conf and uncomment (presumably) the second of the two SnapShotPolicy: lines under [Volume:CommonVolume]. That will turn on the SnapShot Policy mechanism.
You can then go to /.snapshot/ and see your SnapShot's created and pruned per the policy. Of course the finest granularity of taking scheduled SnapShot's being a minute, watching the pot of water boil is a bit tedious.
Alternatively, you can see this same list of SnapShot's via the embedded HTTP Server inside ProxyFS. There, you can even create (and delete) SnapShot's manually as well. Using this mechanism, you could presumably create a sequence of SnapShot's more rapidly than once a minute... just follow the timestamp naming convention (RFC3339) to "impersonate" your chosen SnapShotPolicy/Schedule... and watch the "prune" function happen. Do note, however, that "pruning" only occurs at the time the SnapShotPolicy daemon() wakes up to perform a scheduled SnapShot.

Ok - latest commit resolves (1)... and (2) turns out to resolve itself with time.


Bug when using decimal points for slidesToShow

This issue was reported in #1983 and closed because of lack of jsfiddle. Here's the user's jsfiddle: https://jsfiddle.net/o8ga52nn/9/
When using decimal points for slidesToShow and scrolling all the way to the end of the slider, when resizing the window, slider disappears/doesn't initiate. This happens every time. I have tried to trigger .slick('resize') on 'breakpoint' and on window.resize, to no avail.
Here's my code below for reference:
     $slider.slick({
            infinite: false,
            arrows: true,
            nextArrow: $carouselArrowNext,
            prevArrow: $carouselArrowPrev,
            speed: 700,
            slidesToShow: 2,
            slidesToScroll: 2,
            focusOnSelect: true,
            responsive: [
                {
                    breakpoint: 1681,
                    settings: {
                        slidesToShow: 2.77
                    }
                },
                {
                    breakpoint: 1024,
                    settings: {
                        slidesToShow: 2.4
                    }
                },
                {
                    breakpoint: 521,
                    settings: {
                        slidesToScroll: 1,
                        slidesToShow: 1.3
                    }
                }
            ]
        });

====================================================================
Steps to reproduce the problem

Scroll/swipe to the far right end of the carousel
Resize window until it hits breakpoint
See carousel disappear.

This is clearly a plugin-related bug. This is what I did to fix this on my end.
Steps

Create onSlick function that re-slicks the carousel
At every breakpoint, check if slider has been initialized, if not unslick it.
At every breakpoint, the first time you reach that breakpoint, or if slick has not been initialized, call onSlick.

Code
I know this code could be improved and I'd love suggestions but here is what I've got:
        var breakPointActiveSmall;
        var breakPointActiveMedium;
        var breakPointActiveLarge;
        var breakPointActiveExtraLarge;

        function resetBreakPoints() {
            breakPointActiveSmall = false;
            breakPointActiveMedium = false;
            breakPointActiveLarge = false;
            breakPointActiveExtraLarge = false;
        }

        function onSlick(slidesToShow, slidesToScroll) {
            settings = {
                infinite: false,
                arrows: true,
                nextArrow: $carouselArrowNext,
                prevArrow: $carouselArrowPrev,
                speed: 700,
                slidesToShow: slidesToShow,
                slidesToScroll: slidesToScroll,
                focusOnSelect: true
            };

            $slider.on('init', function(event, slick, currentSlide){
                setTabbing();
            });

            $slider.slick(settings);
        }

        function toggleCarousel() {
            var slickHasInitialized = $slider.hasClass('slick-initialized');

            function handleBreakpoint(breakpoint, slidesToShow, slidesToScroll) {
                if(!breakpoint) {
                    if(slickHasInitialized) {
                        $slider.slick('unslick');
                    }
                    onSlick(slidesToShow, slidesToScroll);
                    resetBreakPoints();
                    breakpoint = true;
                }

                return;
            }

            if(Modernizr.mq('(max-width: 520px)')) {
                handleBreakpoint(breakPointActiveSmall, 1.3, 1);
            }

            if(Modernizr.mq('(min-width: 521px) and (max-width: 1024px)')) {
                handleBreakpoint(breakPointActiveMedium, 2.4, 2);
            }

            if(Modernizr.mq('(min-width: 1025px) and (max-width: 1680px)')) {
                handleBreakpoint(breakPointActiveLarge, 2.77, 2);
            }

            if(Modernizr.mq('(min-width: 1681px)')) {
                handleBreakpoint(breakPointActiveExtraLarge, 2, 2);
            }
        }

        resetBreakPoints();
        toggleCarousel();
        $window.on('resize', toggleCarousel);

this "unofficial" setting is great, it wold be fantastic if it wold be integrated as official option


Reward AD can not got reward at first times

only second or more times can got reward call back

This issue still persists, please re-open this thread.


[Enhancement] Include ES-RNN space in auto.py

If I am not mistaken, ES-RNN is not included in auto.py, so a non-ML expert cannot simply use it like N-HiTS in this example.
IMHO, it would be nice to have it there, so that ES-RNN enjoys the same automations as RNN, NBEATS, N-HiTS & MQN-HiTS.

Hi @gsamaras, we lunched a new version of NeuralForcast with an easier interface. We will include ES-RNN in the following weeks.


help building for esp32 and micropython 1.15?

Hi, very sorry to trouble you but we're using your awesome work to give us better mdns and want to use it with latest MP.  Can you provide a bit of guidance as to what you did - for example in menuconfig - to build using your mdns bits and not the defaults?  Thanks for your time.

Hey @ehagerty
I haven't actually worked with the newest MicroPython release soon. I will try to take a look into updating the build configuration for MP 1.15 on the weekend if I find time.
Currently for MP 1.13 which I used the config for the board is injected in the docker container which builds the MP image: https://github.com/cbrand/micropython-mdns/blob/main/Dockerfile#L29
It uses for this the hardware flags from here https://github.com/cbrand/micropython-mdns/blob/main/config/boards/MDNS/mpconfigboard.h#L3 which I extracted from the MicroPython code and didn't use menuconfig for the configuration as far as I remember.
If I manage to find some time on the weekend I can take a look to port the current config to 1.15. I already saw in your fork that you did some work there already though so am not quite sure if I can get it to work.

Hi @cbrand thank you so much for replying at all sir!  I apologise if my git etiquette is lacking - I'm an old guy learning new tricks!  Thank you so much for such a superb job with this repo in the first place - you not only made it possible for us to move ahead with a pilot for elderly care, but wow are you thorough!  In any case, I've done a lot of (probably crude) refactoring parts of your code to suit what I'm trying to do and probably haven't pushed any of that yet.  So that I don't step on anyone's toes - what /should/ I be doing as I adapt your repo for a somewhat different use case?  By that I mean should I be creating a different repo or trying to fit it back to yours?  Also happy to communicate outside this bug report - just tell me what is appropriate.  Again, thanks so very much for your time.

Hey @ehagerty
Feel free to issue a pull request if you manage to back port something. I am glady checking it and give feedback / merge it if it is of value for the whole project.
Ideally you should create a PR (Pull Request) describing the changes you've made and what you hope to achieve there. Most bigger projects have a CLA (Contribution License Agreement) but as this is a small project at the moment so as long as you are willing to release the code under the same license (MIT) and make it available for the public.
Just don't be overly surprised if the PR check takes a bit or is rejected. I do my best effort to explain why if that would happen. Otherwise it is always good that you work on your own base repository of course so you don't have to wait for me (or any other contributor of an OSS project) to react.

@ehagerty
I managed to successfully create an image with MicroPython 1.15 which did include the micropython-mdns library.
It successfully managed to do service discovery in my local home Wifi. You can either download the firmware in the newly created 1.0.0 release or use the Makefile or Dockerfile to build it yourself.
This can be done by doing running:
make compile-micropython-1-15

You require docker to be installed on your machine for that.
Hope that helps. Best of luck with your project!

@cbrand you are the GREATEST!  I am so very grateful for you taking time out of your life to make this possible for us - a very great help indeed.  If I can ever be of assistance to you please just reach out.


The schedule

It works everywhere except the iPhone 5. Responsive table class made it small on desktop so I didn't use that.

Looks good nick! 👍


[Aces and Eights(Shattered Frontier)] added a section for speed and accuracy modifiers

Changes / Comments
This adds functional enhancements. I added a new feature, which is a section on the weapons page that allows users to input speed and accuracy modifiers. I also made the user able to choose if they want to auto calculate their total speed and accuracy from the modifiers or not.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.

Character Sheet Info Roll20 Internal Use only.


undefined reference enif_snprintf and others

The following example results in linker errors after compilation. Rustler 0.16.0, rustc 1.26.0. It is unclear if this is caused within Rustler, or if it somehow stems from erlang_sys-nif. Any ideas are appreciated. I specifically encountered a missing enif_snprintf reference, which I tracked down to be caused from NifAtom.
#[macro_use]
extern crate rustler;

use rustler::env::OwnedEnv;
use rustler::types::atom::NifAtom;

fn main() {
    OwnedEnv::new().run(|env| {
        if let Ok(a) = NifAtom::from_str(env, "hello") {
            if let Ok(b) = NifAtom::from_str(env, "hello") {
                assert_eq!(a, b)
            }
        }
    });
}

Cargo.toml:
[package]
name = "rustler_enif_snprintf_unresolved"
version = "0.1.0"
authors = []

[dependencies]
rustler = "0.16"
rustler_codegen = "0.16"
lazy_static = "0.1"

Error Message:
/tmp/rustler_enif_snprintf_unresolved/target/debug/deps/librustler-0dc2f9159eb90b83.rlib(rustler-0dc2f9159eb90b83.rustler9.rcgu.o): In function `<rustler::env::OwnedEnv as core::ops::drop::Drop>::drop':
          /home/mo/.cargo/registry/src/github.com-1ecc6299db9ec823/rustler-0.16.0/src/env.rs:186: undefined reference to `enif_free_env'
          /tmp/rustler_enif_snprintf_unresolved/target/debug/deps/librustler-0dc2f9159eb90b83.rlib(rustler-0dc2f9159eb90b83.rustler11.rcgu.o): In function `rustler::wrapper::nif_interface::enif_alloc_env':
          /home/mo/.cargo/registry/src/github.com-1ecc6299db9ec823/rustler-0.16.0/src/wrapper/nif_interface.rs:59: undefined reference to `enif_alloc_env'
          /tmp/rustler_enif_snprintf_unresolved/target/debug/deps/librustler-0dc2f9159eb90b83.rlib(rustler-0dc2f9159eb90b83.rustler11.rcgu.o): In function `rustler::wrapper::nif_interface::enif_make_atom_len':
          /home/mo/.cargo/registry/src/github.com-1ecc6299db9ec823/rustler-0.16.0/src/wrapper/nif_interface.rs:121: undefined reference to `enif_make_atom_len'
          /tmp/rustler_enif_snprintf_unresolved/target/debug/deps/librustler-0dc2f9159eb90b83.rlib(rustler-0dc2f9159eb90b83.rustler3.rcgu.o): In function `rustler::wrapper::term::fmt':
          /home/mo/.cargo/registry/src/github.com-1ecc6299db9ec823/rustler-0.16.0/src/wrapper/term.rs:14: undefined reference to `enif_snprintf'
          collect2: error: ld returned 1 exit status

@evnu is this the actual code you're trying to use? If so, it will not work, as you can only use rustler code in the context of a NIF.
If you're trying to create a binary to call with a port, rustler won't be much use to you.

@scrogson No, I did not intend to run the code like this. I encountered the error within a unit test of a NIF library. But your following point might describe what I encountered:

If so, it will not work, as you can only use rustler code in the context of a NIF.

The original undefined reference which I tried to track down was within a test similar to this snippet:
#[cfg(test)]
mod test {
  #[test]
  fn some_test () {
    // .. some code ..
    # init() creates something (e.g. Enum) which might contain a `NifAtom`
    assert_eq!(None, init())
  }
}

Now, what I gather from your comment is that if Value::init() returns a value which contains something like NifAtom, this will break and may manifest in an undefined reference error.
I will close this issue as I believe that your explanation pointed me in the right direction. Thank you!

@evnu you're very welcome!


change theora transport dependency to compressed transport

Theora image transport seems to be not that well maintained in comparison with compressed transport.
Hopefully this fixes https://github.com/ros-sports/rqt_image_overlay/issues/20 too.
Signed-off-by: Kenji Brameld<EMAIL_ADDRESS>

Galactic CI failure is due to https://github.com/ros-perception/image_transport_plugins/issues/83


Training a 512x512 coco detector

I successfully trained a 512x512 resnet101 detector on coco, using your code, and Torch version 4.1 (python 2.7, cuda 10).  It needed a couple of minor changes to the source code.  In particular, the loss calculation in arm_loss.py and odm_loss.py requires division by float, instead of integer.
-        loss_l /= total_num
-        loss_c /= total_num
+        loss_l /= total_num.float()
+        loss_c /= total_num.float()

Using 4 GPUS and a batch size of 24, the ARM_conf_loss oscillated for several hundred iterations, but then it calmed down and trained normally.  The final accuracy after 120K iterations is pretty good:
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.333
 Average Precision  (AP) @[ IoU=0.50      | area=   all | maxDets=100 ] = 0.542
 Average Precision  (AP) @[ IoU=0.75      | area=   all | maxDets=100 ] = 0.358
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= small | maxDets=100 ] = 0.153
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=medium | maxDets=100 ] = 0.385
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= large | maxDets=100 ] = 0.476
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=  1 ] = 0.288
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets= 10 ] = 0.444
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.472
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= small | maxDets=100 ] = 0.257
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=medium | maxDets=100 ] = 0.535
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= large | maxDets=100 ] = 0.636

Thanks for sharing your code.

That's great! Thank you too.

Now，the problem of oscillated loss has been solved !  Hope it will help.


[Functions] Update CI email template to better display error log

For now the email template for CI notification email is pretty simple, it just collect maven output with [error] and send it directly, which may be confusing and will take some times to get the error project from email, we need to find a better way to generate CI test report and send notifications.

Done in pr #597


Remove ACS event grid swagger since private preview dates are not finalized.

Remove ACS event grid swagger since private preview dates are not finalized.

/azp run automation - sdk


Set max line length back to "79"

Fixes #41
Max line length is set to "79"

@sdhiaeddine Test failed, change the lines in all the files, flake8 . will show you the lines having length > 79

What's the ratio behind <120 ?

@sdhiaeddine  Do you need help with this?

@sdhiaeddine - Are you working on this ?

I messaged him on slack but he didn't answer there either :/
@CirXe0N if you are working on something else I can take it :)

Okay @velibor7 you take it

Closed with https://github.com/thenewboston-developers/thenewboston-python-client/pull/61


Add installation instructions for conda.

I created a recipe for backspinpy and submitted it to bioconda. This PR adds conda installation instructions to the README.

Thank you!