TypeError: cannot pickle 'pydantic_core._pydantic_core.Url' object

Initial Checks

[X] I confirm that I'm using Pydantic V2 installed directly from the main branch, or equivalent

Description
Maybe this is more of a pydantic-core bug?
Example Code
import pickle
import pydantic

class A(pydantic.BaseModel):
    url: pydantic.HttpUrl

a = A(url="https://github.com/pydantic/pydantic")
pickle.dumps(a)

Python, Pydantic & OS Version
pydantic version: 2.0b2
        pydantic-core version: 0.38.0 release build profile
                 install path: ~/workspace/pydantic/pydantic
               python version: 3.10.6 (main, Mar 10 2023, 10:55:28) [GCC 11.3.0]
                     platform: Linux-5.19.0-42-generic-x86_64-with-glibc2.35
     optional deps. installed: ['email-validator', 'typing-extensions']

@commonism if you wanted to take a stab at this on the rust side, you might follow the discussion in https://github.com/PyO3/pyo3/issues/100 and add the relevant methods to pydantic_core._pydanic_core.Url. If you aren't comfortable doing that, I would suggest opening a feature request on pydantic-core for making this type picklable.
@davidhewitt is there a better way to make the type picklable short of defining those methods? (__getstate__ etc.)

Not yet. Shortcuts to implement the dunder methods for #[pyclass] is something we're just exploring for PyO3 0.20, I'll make a note to consider if we can do something smart for pickling.

Have not had rust on my hands before, but … gave it a shot.
So far I got either compiles but does not work, or does not compile.
    fn __getstate__(&self) -> PyResult<String> {
        Ok(self.lib_url.to_string())
    }

    fn __setstate__(&mut self, url: &PyAny) -> PyResult<Self> {
… it'll look easy when done
    }

Try implementing __reduce__ and returning essentially (type(self), str(self). In other words, serialize it as a string and rehydrate by calling the Url constructor.

Seems like this was fixed in https://github.com/pydantic/pydantic-core/commit/0ff7563c3eeafbbbf9355352cc66b5868d52c359


Configure updates to main to trigger builds automatically

Docs currently not automatically building - updated webhook. Check connection on next deploy

Currently should be building on superstaq.readthedocs.org


Profile Page Showing even when not logged in

Describe the bug
Profile Page Showing even when not logged in
To Reproduce
Steps to reproduce the behavior:

Go to 'main'
Click on 'my profile'
See error

Expected behavior
No profile page when not logged it
Screenshots

Additional context
Make sure to add the labels :)
Thanks :)

I would like to work on this issue .Please assign it to me.
Expected is no profile page when not logged it but it is showing .


re-enabled naming metrics in ckpt name

Fixes #3018
Fixes #3000

Hello @williamFalcon! Thanks for updating this PR.

In the file tests/callbacks/test_model_checkpoint.py:

Line 186:24: W292 no newline at end of file

@williamFalcon fixes #3017 and not #3018


Feature/default version support

This adds support for setting the default version for a tool and workflow based off of the associated CWL file.
[] Check that you pass the basic compilation and unit tests by running grunt

Coverage increased (+0.01%) to 8.812% when pulling 2972f3db145cc73ea425247b9b4092097caeccd6 on feature/default_version_support into ce7742d1f619839ccf51773a17209a62d57c9186 on develop.


Definition of RACK_DIR is missing from Makefile

The Makefile should define a variable called RACK_DIR, which can be set to the location of the Rack source code. This allows compiling the plugin outside of the Rack source tree.
See API Update Thread.

mine is the same as the tutorial
TUTORIAL https://github.com/VCVRack/Template/blob/master/Makefile#L24
MINE https://github.com/jeremywen/JW-Modules/blob/master/Makefile#L4

Oops, sorry. I didn't see the definition at the top. I'll check this again.

Ah, I see what's wrong. The variable is defined, but you are not using it.
YOURS:
include ../../plugin.mk

TUTORIAL:
include $(RACK_DIR)/plugin.mk

oops, thank you!  fixed.


[CF] Add TARGET_OS_BSD in some places.

These particular uses should hopefully be uncontroversial.

CFPlatform.c: to use the _CFIsMainThread pthread implementation; to
properly implement __CFTSDSetSpecific and __CFTSDGetSpecific with the
same pthread implementation as Linux (and Mac)

CoreFoundation_Prefix.h: to exclude strlcpy and issetugid which is
normally available in the system; to include sys/param.h for the
MIN macro

CFData.c: for the __CFPageSize implementation

CFBundle_Resources.c: to include dirent.h symbols

CFXMLPreferencesDomain.c: to define sleep for the platform

CFBurstTrie.c: for mmap.

@swift-ci test and merge


Disable creating cbr0 in salt.

@dchen1107  @roberthbailey  @cjcullen  @lavalamp
This has the net effect of ensuring that Docker doesn't come up before the kubelet is ready.
The reason is this:

Docker is configured to use --bridge=cbr0
But this bridge doesn't exist, so Docker crash loops via monit
Eventually the kubelet creates cbr0 and restarts Docker, so Docker is now running on the right bridge.

There is still a thin race where monit could restart Docker, and then kubelet could restart it again, but at least the bridge doesn't change.

Ugh, this won't fix the master.  We should just use docker0 there.

I have a possible sister fix for this building now, too: I have a tiny hack to the ContainerVM docker init script that won't even start docker if cbr0 isn't created, if we detect the kube-env instance metadata. I think that would have the net effect that (a) docker won't be started, and (b) even if monit tried to restart docker, it would fail until cbr0 is ready. I might be missing a case, though. Does that help the master case?

Actually, I fixed the master by just using docker0 in that case.  This PR can now bring up healthy clusters.

Hrm. If you knew Docker would only start if cbr0 were ready, would you do it differently? Asking because the docker0 variance is slightly annoying - I was keying off of ifconfig cbr0, but with this PR, I'd leave masters docker-less.
I'd actually like to make the CVM-side change as well, because the chicken/egg-issue seems to bite us not-unoften, so it just seems like it's a waste to bring them up in the wrong order.

Also, are we talking about the same docker0 that we go to lengths to delete in configure-vm? Does docker then recreate it when it comes back up?

hmm, I am ok with your pr here. Before introduce --configure-cbr0 flag, on master node, actually we are using docker0 for master node. I guess i should let this in first for v1.0, especially for some soak time over week. But it sounds like a step-backward to me here though. :-)
Also I updated https://github.com/GoogleCloudPlatform/kubernetes/issues/9822#issuecomment-113716184. There are still some unclear puzzles to me, especially why docker doesn't restart in that case by kubelet?

Just realized that one more thing I forgot to mention above. You already introduced to return earlier at SyncLoop if container runtime is not up. If we check if the network is configured; if it is not, return earlier too. In this case, those static pods also won't be restarted, including those master components pods on master node, thus but both #10122 and #9822 are prevented. And you don't need to disable cbr0 for master too.
Also node needs to join the cluster talking to master first to have PodCIDR, then config cbr0. Master node is not need and I also pointed out above in configure-vm.sh,  we already treated master node differently and directly passed master-ip-range, why not simply configure cbr0 there directly instead of through kubelet? In this case, we just need pass flag --configure-cbr0=false to master node kubelet. But again, Kubelet should sanity check network in both cases, and not start any pods before network is configured. Anything I missed here?
I am going to OOT tomorrow, but will be back on Sunday. If this is not done yet, I can pick up this on Sunday night.

If we switched to just using docker's --bip instead(which means docker0 and
not cbr0), what would change?
On Jun 20, 2015 1:26 AM, "Dawn Chen"<EMAIL_ADDRESS>wrote:

Just realized that one more thing I forgot to mention above. You already
introduced to return earlier at SyncLoop if container runtime is not up. If
we check if the network is configured; if it is not, return earlier too. In
this case, those static pods also won't be restarted, including those
master components pods on master node, thus but both #10122
https://github.com/GoogleCloudPlatform/kubernetes/issues/10122 and #9822
https://github.com/GoogleCloudPlatform/kubernetes/issues/9822 are
prevented. And you don't need to disable cbr0 for master too.
Also node needs to join the cluster talking to master first to have
PodCIDR, then config cbr0. Master node is not need and I also pointed out
above in configure-vm.sh, we already treated master node differently and
directly passed master-ip-range, why not simply configure cbr0 there
directly instead of through kubelet? In this case, we just need pass flag
--configure-cbr0=false to master node kubelet. But again, Kubelet should
sanity check network in both cases, and not start any pods before network
is configured. Anything I missed here?
I am going to OOT tomorrow, but will be back on Sunday. If this is not
done yet, I can pick up this on Sunday night.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/pull/10153#issuecomment-113729964
.

@thockin Using --bip definitely make code cleaner, but not really solve the race based on today's mechanism on how node auto-register with master on GKE & GCE. Basically kubelet has come up first, and talk to master to get PodCIDR then configure cbr0. In this case, kubelet has to restart docker daemon, and also check if the network is configured correctly, and not run any pods too.
Please note that we only have issues with file-based pods, not with pods from apiserver because kubelet does set NotReady to nodeStatus, and no pods are bound to NotReady node. But I think what I proposed to check network readiness should solve such issue.

In general, I'm concerned about switching the master off cbr0 this close to
1.0. We've been running that way for weeks, and it feels like we're going
to find little things for a while if we switch now.
On Jun 20, 2015 7:35 AM, "thockin-cc"<EMAIL_ADDRESS>wrote:

If we switched to just using docker's --bip instead(which means docker0 and
not cbr0), what would change?
On Jun 20, 2015 1:26 AM, "Dawn Chen"<EMAIL_ADDRESS>wrote:

Just realized that one more thing I forgot to mention above. You already
introduced to return earlier at SyncLoop if container runtime is not up.
If
we check if the network is configured; if it is not, return earlier too.
In
this case, those static pods also won't be restarted, including those
master components pods on master node, thus but both #10122
https://github.com/GoogleCloudPlatform/kubernetes/issues/10122 and
#9822
https://github.com/GoogleCloudPlatform/kubernetes/issues/9822 are
prevented. And you don't need to disable cbr0 for master too.
Also node needs to join the cluster talking to master first to have
PodCIDR, then config cbr0. Master node is not need and I also pointed out
above in configure-vm.sh, we already treated master node differently and
directly passed master-ip-range, why not simply configure cbr0 there
directly instead of through kubelet? In this case, we just need pass flag
--configure-cbr0=false to master node kubelet. But again, Kubelet should
sanity check network in both cases, and not start any pods before network
is configured. Anything I missed here?
I am going to OOT tomorrow, but will be back on Sunday. If this is not
done yet, I can pick up this on Sunday night.
—
Reply to this email directly or view it on GitHub
<
https://github.com/GoogleCloudPlatform/kubernetes/pull/10153#issuecomment-113729964
.

—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/pull/10153#issuecomment-113775831
.

@zmerlynn please see my above https://github.com/GoogleCloudPlatform/kubernetes/pull/10153#issuecomment-113729964
Actually I am suggesting not switch cbr0 on master, instead of blocking any pods creation before network is configured successfully, including static pods.

I think that'd be a better change, if it's doable in a reasonable amount of time.

Assigning to @zmerlynn to take responsibility for the salt review. But I think @dchen1107 is right about the kubelet syncLoop stuff.

I will update this PR to do the network syncLoop stuff.

Ok, I updated this PR.  I broke the cbr0 reconciliation out of the node status loop (which is arguably what should have happened in the first place, anyway.
ptal.

The Salt part looks fine. Reassigning to @dchen1107 to review the sync loop and for final buy-off.

@brendandburns is out for DockerCon, and I am going to carry on this PR.

Close this one since I already carried the work in #10211


Enhancement: Improve the handling of non-coding variants in the Variant-Linker tool.

Current Behavior
The tool does not adequately handle non-coding variants, which can lead to incomplete or incorrect annotations.
Proposed Solution

Enhance the logic to identify and process non-coding variants.
Ensure appropriate annotations are generated for non-coding variants.

Acceptance Criteria

[ ] Non-coding variants are correctly identified and processed.
[ ] Appropriate annotations are generated for non-coding variants.
[ ] Unit tests are added to verify the handling of non-coding variants.

Unit tests will we implemented and handled in #5

Example non-coding variant:
ENST00000650946.1:n.439-1601_439-1600insC
9-22125503-TC-T
This works now and was fixed through #14


Publish

Pull Request Test Coverage Report for Build 14

39 of 40 (97.5%)  changed or added relevant lines in 5 files are covered.
50 unchanged lines in 21 files lost coverage.
Overall coverage decreased (-16.3%) to 74.579%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/enumerable-impl.ts
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/enumerables/assistance/intersect.ts
1
71.43%

src/enumerables/sum.ts
1
71.43%

src/enumerables/assistance/except.ts
1
71.43%

src/enumerables/take-while.ts
1
66.67%

src/enumerables/single-or-default.ts
2
50.0%

src/enumerables/max.ts
2
62.5%

src/enumerables/distinct.ts
2
50.0%

src/enumerables/last-or-default.ts
2
0.0%

src/enumerables/select.ts
2
50.0%

src/enumerables/min.ts
2
62.5%

Totals

Change from base Build 10:
-16.3%

Covered Lines:
329

Relevant Lines:
437

💛  - Coveralls


[connectivity] Add property setter to selectorVisible property. Fixes JB#54986

Add property setter to selectorVisible property.

One minor nitpick, aside from that LGTM


overwrite training_step for CustomDPOTrainer to clear cuda cache every train step

according to the issue: https://github.com/huggingface/trl/issues/1377, call torch.cuda.empty_cache() after each step to clear the cuda cache
What does this PR do?
reduce memory usage when using dpo trainer, according to https://github.com/huggingface/trl/issues/1377
Before submitting

[x] Did you read the contributor guideline?
[x] Did you write any new necessary tests?

Hi all, could you tell me why this PR is in pending?
请问下，为什么这个PR处于 pending状态呢？


Exposed ports question

Can you clarify what is meant by:

By default, all exposed ports are published on the host. The following two properties are set per exposed port:

docker.containers.[id].ports.[portname].host (f.ex 'docker.containers.id.cache.ports.80/tcp.host')
docker.containers.[id].ports.[portname].port (f.ex 'docker.containers.id.cache.ports.80/tcp.port')

You can pass those project properties over to your integration test and use them to connect to your application.

How can I pass these properties in my test? Are they exposed as system properties?
Thanks
Nick

They are exposed as project properties. You can pass them to your tests as such:
            <plugin>
                <artifactId>maven-failsafe-plugin</artifactId>
                <version>2.17</version>
                <configuration>
                    <systemProperties>
                        <app.base.1>http://${docker.containers.app.ports.8080/tcp.host}:${docker.containers.app.ports.8080/tcp.port}</app.base.1>
                        <app.base.2>http://${docker.containers.app2.ports.8080/tcp.host}:${docker.containers.app2.ports.8080/tcp.port}</app.base.2>
                    </systemProperties>
                </configuration>
                <executions>
                    <execution>
                        <goals>
                            <goal>integration-test</goal>
                            <goal>verify</goal>
                        </goals>
                    </execution>
                </executions>
            </plugin>

See also: https://github.com/wouterd/docker-maven-plugin/blob/master/src/it/sample-api-mongo-it/pom.xml

Thanks for that


fix: remove unnecessary manual focus

Please review.
Details
Fix the regression https://expensify.slack.com/archives/C01GTK53T8Q/p1619030226468300
Tests / QA Steps

open a chat on Mobile Web and then navigate back to another screen.
Now going back to the chat should not open the keyboard automatically

Tested On

[ ] Web
[x] Mobile Web
[ ] Desktop
[ ] iOS
[ ] Android

Screenshots
NA

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.


Unclear how to actually get the "IndexedDBEdit" tab to show up in Chrome developer tools

At least I assume that is what is supposed to happen, as the screenshot shows such a tab.  But after I installed the extension and reloaded the page, no such tab shows up.  Do I need to enable it for my site?  If so, how?

Probably related, but when I first click on the extensions icon, IndexedDBEdit shows up under "No access needed:  These extensions don't need to see and changed information on this site."  Which is weird, because when I installed it asked for permission saying it would be able to "Read and change all your data on all websites".  I don't see how it could work properly without access, but I also don't see any way to give it access.

When I try to manage the extension one thing it says is:
Inspect views
    service worker (Inactive)

Does this mean it can only access an IndexedDB if my page has a service worker? (which it doesn't)

I finally got it to work.  Apparently it was necessary to completely shut down and restart the browser, which is something I haven't encountered with any other extensions and is a last ditch thing to try since I have a lot of tabs open.


What value should be provided for the Description and Description-Url parameters?

I am developing a Business Central app that requires code signing with Azure Key Vault as described here in Microsoft Learn:
https://learn.microsoft.com/en-us/dynamics365/business-central/dev-itpro/developer/devenv-sign-extension#steps-for-signing-your-app-file-with-azure-key-vault
I am going to be signing with a service principal, so this is the sample code provided on that page:
sign code azure-key-vault --azure-key-vault-url "https://MyKeyvault.vault.azure.net/" --azure-key-vault-certificate "NameOfMyCertificate" --azure-key-vault-client-id "ClientIdOfServicePrincipal" --azure-key-vault-client-secret "ClientSecretOfServicePrincipal" --azure-key-vault-tenant-id "MicrosoftEntraId" --description "Some Description" --description-url "" --verbosity Information "C:/Path/To/File(s)"
It is unclear to me what should be provided to the Description and Description-Url parameters; there is no clear documentation for this.
How should I proceed?

I just read in this post that these two parameters can be set to $null:
https://github.com/dotnet/sign/issues/642#issuecomment-2148378970
I've tried this, and I get this error:
Option '-u' is required.
I am still stuck and I cannot determine what to do. Help?

@floyd-chan, the values for --description and --description-url apply to Authenticode signatures.  Here is signtool.exe's description for those values:

Command
Description

/d Desc
Specifies a description of the signed content.

/du URL
Specifies a URL for the expanded description of the signed content.

BTW, I recommend that you look at using a federated identity instead of a client secret when connecting with either GitHub Actions or Azure DevOps pipelines.  In these cases, you'd use the --azure-key-vault-managed-identity parameter instead.  Then, you wouldn't need to have a secret in your pipeline.  @clairernovotny, can you link to samples for this?

Hi @floyd-chan,
Sign CLI supports secret-less authentication using Workload Identity Federation for GitHub Actions and Azure Pipelines.
Here are a couple samples:

GitHub: https://github.com/dotnet/sign/blob/main/docs/gh-build-and-sign.yml#L37-L91
Azure Pipelines: https://github.com/NuGetPackageExplorer/NuGetPackageExplorer/blob/main/azure-pipelines.yml#L219-L243

Docs on setting up the Workload Identity Federation are here:

GitHub: https://learn.microsoft.com/en-us/azure/developer/github/connect-from-azure?tabs=azure-portal%2Cwindows#set-up-azure-login-with-openid-connect-authentication
Azure Pipelines: https://learn.microsoft.com/en-us/azure/devops/pipelines/release/configure-workload-identity?view=azure-devops

The --description and --description-url options were made optional in this pull request: #718.


&amp is still visible in /wordify

When an "&amp" piece of text is in the database, it's removed when parsed in the web view, but not the wordify API

Example:
Main Ingredient
Baked Breakfast Potatoes
Oatmeal
Old Fashioned Donuts
Bacon, Egg & Cheddar Whole Grain Breakfast Pizza


Fix deprecation warning with newer StaticArrays versions.

Should be backwards compatible; broadcasting was implemented a long time
ago.

Codecov Report

Merging #104 into master will increase coverage by 0.14%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #104      +/-   ##
==========================================
+ Coverage   93.78%   93.93%   +0.14%
==========================================
  Files           9        9
  Lines         692      692
==========================================
+ Hits          649      650       +1
+ Misses         43       42       -1

Impacted Files
Coverage Δ

src/derivatives.jl
100% <100%> (ø)
:arrow_up:

src/core_types.jl
85.57% <0%> (+0.96%)
:arrow_up:

Continue to review full report at Codecov.

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


HEOS component not showing device physical inputs in source list

Home Assistant release with the issue:

0.92.2
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):

Hass.io
Component/platform:

https://www.home-assistant.io/components/heos/
Description of problem:
I can't select device physical inputs from source list, but I can see all HEOS favorites after I logged in heos://system/sign_in?un=heos_username&pw=heos_password.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

I used discovery component to set up.
Traceback (if applicable):

Additional information:
Here is log for HEOS related stuff I could find:
2019-05-03 09:00:29 INFO (MainThread) [homeassistant.setup] Setup of domain heos took 0.0 seconds.
2019-05-03 09:00:34 DEBUG (MainThread) [pyheos.connection] Executed command 'system/register_for_change_events': '{'command': 'system/register_for_change_events', 'result': 'success', 'message': 'sequence=0&enable=on'}'
2019-05-03 09:00:34 DEBUG (MainThread) [pyheos.connection] Connected to <IP_ADDRESS>
2019-05-03 09:00:36 DEBUG (MainThread) [pyheos.connection] Executed command 'system/check_account': '{'command': 'system/check_account', 'result': 'success', 'message'<EMAIL_ADDRESS>2019-05-03 09:00:37 DEBUG (MainThread) [pyheos.connection] Executed command 'player/get_players': '{'command': 'player/get_players', 'result': 'success', 'message': ''}'
2019-05-03 09:00:37 DEBUG (MainThread) [pyheos.connection] Executed command 'player/get_play_state': '{'command': 'player/get_play_state', 'result': 'success', 'message': 'pid=68723180&state=play'}'
2019-05-03 09:00:37 DEBUG (MainThread) [pyheos.connection] Executed command 'player/get_now_playing_media': '{'command': 'player/get_now_playing_media', 'result': 'success', 'message': 'sequence=4&pid=68723180'}'
2019-05-03 09:00:37 DEBUG (MainThread) [pyheos.connection] Executed command 'player/get_volume': '{'command': 'player/get_volume', 'result': 'success', 'message': 'pid=68723180&level=12'}'
2019-05-03 09:00:37 DEBUG (MainThread) [pyheos.connection] Executed command 'player/get_mute': '{'command': 'player/get_mute', 'result': 'success', 'message': 'pid=68723180&state=off'}'
2019-05-03 09:00:37 DEBUG (MainThread) [pyheos.connection] Executed command 'player/get_play_mode': '{'command': 'player/get_play_mode', 'result': 'success', 'message': 'pid=68723180&repeat=off&shuffle=off'}'
2019-05-03 09:00:37 DEBUG (MainThread) [pyheos.connection] Command under process 'browse/browse': '{'heos': {'command': 'browse/browse', 'result': 'success', 'message': 'command under process&sequence=8&sid=1028'}}'
2019-05-03 09:00:38 DEBUG (MainThread) [pyheos.connection] Executed command 'browse/browse': '{'command': 'browse/browse', 'result': 'success', 'message': 'sequence=8&sid=1028&returned=19&count=19'}'
2019-05-03 09:00:38 DEBUG (MainThread) [pyheos.connection] Executed command 'browse/browse': '{'command': 'browse/browse', 'result': 'success', 'message': 'sequence=9&sid=1027&returned=0&count=0'}'
2019-05-03 09:00:38 INFO (MainThread) [homeassistant.components.media_player] Setting up media_player.heos
2019-05-03 09:00:39 DEBUG (MainThread) [pyheos.connection] Event received for player {Living Room (HEOS 5)}: {'command': 'event/player_now_playing_progress', 'message': 'pid=68723180&cur_pos=36600000&duration=0'}
2019-05-03 09:01:20 INFO (MainThread) [homeassistant.components.discovery] Unknown service discovered: dlna_dms {'host': '<IP_ADDRESS>', 'port': 60006, 'ssdp_description': 'http://<IP_ADDRESS>:60006/upnp/desc/aios_device/aios_device.xml', 'name': 'Living Room', 'model_name': 'HEOS 5', 'model_number': 'DWS-5000 3.0', 'serial': None, 'manufacturer': 'Denon', 'udn': 'uuid:14bc3b21-455a-1dbd-0080-0005cd474a44', 'upnp_device_type': 'urn:schemas-denon-com:device:AiosDevice:1'}
2019-05-03 09:01:20 INFO (MainThread) [homeassistant.components.discovery] Unknown service discovered: dlna_dmr {'host': '<IP_ADDRESS>', 'port': 60006, 'ssdp_description': 'http://<IP_ADDRESS>:60006/upnp/desc/aios_device/aios_device.xml', 'name': 'Living Room', 'model_name': 'HEOS 5', 'model_number': 'DWS-5000 3.0', 'serial': None, 'manufacturer': 'Denon', 'udn': 'uuid:14bc3b21-455a-1dbd-0080-0005cd474a44', 'upnp_device_type': 'urn:schemas-denon-com:device:AiosDevice:1'}
2019-05-03 09:12:56 DEBUG (MainThread) [pyheos.connection] Command under process 'browse/play_preset': '{'heos': {'command': 'browse/play_preset', 'result': 'success', 'message': 'command under process&sequence=10&preset=7&pid=68723180'}}'
2019-05-03 09:12:56 DEBUG (MainThread) [pyheos.connection] Executed command 'browse/play_preset': '{'command': 'browse/play_preset', 'result': 'success', 'message': 'sequence=10&preset=7&pid=68723180'}'
2019-05-03 09:12:58 DEBUG (MainThread) [pyheos.connection] Event received for player {Living Room (HEOS 5)}: {'command': 'event/player_queue_changed', 'message': 'pid=68723180'}
2019-05-03 09:12:58 DEBUG (MainThread) [pyheos.connection] Executed command 'player/get_now_playing_media': '{'command': 'player/get_now_playing_media', 'result': 'success', 'message': 'sequence=11&pid=68723180'}'
2019-05-03 09:12:58 DEBUG (MainThread) [pyheos.connection] Event received for player {Living Room (HEOS 5)}: {'command': 'event/player_now_playing_changed', 'message': 'pid=68723180'}

Thanks for posting the logs... so this is a key one:

Executed command 'browse/browse': '{'command': 'browse/browse', 'result': 'success', 'message': 'sequence=9&sid=1027&returned=0&count=0'}'

sid means Source ID and 1027 is the constant for AUX Inputs.  As you can see in the rest of the params, it's not returning any results (returned=0 and count=0).
This isn't a bug in the dependency or the HA integration, but may be a bug in the device firmware itself (which seems likely given your device had other HEOS API spec conformity issues as well).
Do you see inputs in the HEOS mobile app?  Music tab --> Inputs.

Yes, I can see it without problem.  I have chromecast audio plugged in that AUX input.

Thanks.  I've asked Denon to take a look at this issue and will let you know when I hear back.

I'm not sure what is changed but now I can access physical input from source list :)
2019-06-04 22:31:36 DEBUG (MainThread) [pyheos.connection] Executed command 'browse/browse': '{'command': 'browse/browse', 'result': 'success', 'message': 'sequence=9&sid=1027&returned=1&count=1'}'

Cool! Must have been a firmware update :)

Dear Andrew,
I have the denon heos home cinema and i have the same problem.
Maybe you can help me?
Thanks
Yannay


Add warning when encountering /index related 404s

Closes #107

Could we warn people with the fix so the warning goes away, a map or path config?
Sent from my iPhone

On Feb 22, 2016, at 7:35 AM, Matthew Phillips<EMAIL_ADDRESS>wrote:
Closes #107
You can view, comment on, or merge this pull request online at:
https://github.com/stealjs/system-npm/pull/108
Commit Summary
Add warning when encountering /index related 404s
File Changes
M npm-extension.js (4)
M npm-utils.js (13)
M test/folder_index/dev.html (20)
Patch Links:
https://github.com/stealjs/system-npm/pull/108.patch
https://github.com/stealjs/system-npm/pull/108.diff
—
Reply to this email directly or view it on GitHub.

We could, but that would only work for dependencies, it's hard to configure a dependency's dependency.


Shot part details not working

Describe the bug
Hen I click "Show part details" I get a waiting cursor. Starting Kicad in a shell I get this trace back:
Traceback (most recent call last):
  File "...3rdparty/plugins/com_github_bouni_kicad-jlcpcb-tools/mainwindow.py", line 727, in get_part_details
    dialog = PartDetailsDialog(self, part)
  File "...3rdparty/plugins/com_github_bouni_kicad-jlcpcb-tools/partdetails.py", line 114, in __init__
    self.get_part_data()
  File "...3rdparty/plugins/com_github_bouni_kicad-jlcpcb-tools/partdetails.py", line 226, in get_part_data
    self.get_scaled_bitmap(
  File ...3rdparty/plugins/com_github_bouni_kicad-jlcpcb-tools/partdetails.py", line 130, in get_scaled_bitmap
    image = image.Scale(width, height, wx.IMAGE_QUALITY_HIGH)
TypeError: Image.Scale(): argument 1 has unexpected type 'float'

I think I have version 2022.02.03 of your plugin installed.
To Reproduce
Steps to reproduce the behavior:

Click "Show part details" anywhere
Busy cursor, no dialog appears.

Expected behavior
Dialog appears with content.
KiCad Version

Application: KiCad

Version: (6.0.1), release build

Libraries:
	wxWidgets 3.0.5
	libcurl/7.81.0 OpenSSL/1.1.1m zlib/1.2.11 brotli/1.0.9 zstd/1.5.2 libidn2/2.3.2 libpsl/0.21.1 (+libidn2/2.3.0) libssh2/1.10.0 nghttp2/1.46.0

Platform: Linux 5.15.10-arch1-1 x86_64, 64 bit, Little endian, wxGTK, i3, x11

Build Info:
	Date: Jan 15 2022 23:16:29
	wxWidgets: 3.0.5 (wchar_t,wx containers,compatible with 2.8) GTK+ 3.24
	Boost: 1.78.0
	OCC: 7.5.3
	Curl: 7.81.0
	ngspice: 35
	Compiler: GCC 11.1.0 with C++ ABI 1016

Build settings:
	KICAD_USE_OCC=ON
	KICAD_SPICE=ON

I know what's the problem, maybe I can fix it later this evening

Not exactly same day but hey 😏

@Bouni  Sorry for the slow response, but I can confirm that your fix appears to fix the issue!
I test it manually by cloning the repo, maybe a new release tag is needed`?

@nickoe A release is on its way but I want to wait for some more fixes to be confirmed to avoid to release to often


When i try to run this plugin it gives me error that Unexpected token

Syntax error on line 10: Unexpected token
import { traverseLayers } from "./functions/traverse-layers";

hey @sanjeevkumar143 - can you add more context about how you are running into this issue? Are you running from a browser (if so what build tool - webpack, rollup, parcel, etc), node.js, other?

no, I am using the Figma software after a lot of hours wasted I change my tsc.config file and made change "module":"udm" and it work all errors gone but now am facing another issue which is the plugin frozen https://prnt.sc/sus730 it's just loading nothing come up

https://prnt.sc/sus7l8 these error comes up in my console

nothing come up in Figma console just stuck in loading

oh I see - you are running this locally. have you tried npm install && npm run dev from the root of this project before running the plugin in development mode?

oh I see - you are running this locally. have you tried npm install && npm run dev from the root of this project before running the plugin in development mode?

yes i tried it


fix(linter): false positiver in private member expr in oxc/const-comparison

fixes https://github.com/oxc-project/oxc/issues/8163

#8164  👈 (View in Graphite)
main

How to use the Graphite Merge Queue
Add either label to this PR to merge it via the merge queue:

0-merge - adds this PR to the back of the merge queue
hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

Merge activity

Dec 28, 8:48 AM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.


very slow query performance ... influxdb unusable ???

start with new db
system: 32gb ram, 8core i7, ssd

where is the problem with this simple query ?
running time ~9min ... wow ...
$> time influx -database TestDB -execute 'select count(f1) from "TestMeas"'
name: TestMeas
time    count

0       673734
real    0m0.641s
user    0m0.008s
sys     0m0.000s
$> time influx -database TestDB -execute 'select * from "TestMeas" WHERE gState>0 ORDER BY time LIMIT 10'
name: TestMeas
...
...
...
real    0m18.440s
user    0m0.004s
sys     0m0.004s
$> time influx -database TestDB -execute 'select * from "TestMeas" WHERE gState>0 ORDER BY time DESC LIMIT 10'
name: TestMeas
...
...
...
real    8m57.146s
user    0m0.000s
sys     0m0.008s

@play113 Please fill out the issue template and include a script to insert data that can reproduce the issue.

#!/usr/bin/env perl
use strict;
use warnings;
use EV;
use AnyEvent;
use AnyEvent::Socket;
use AnyEvent::Handle;
use AnyEvent::InfluxDB;
use InfluxDB::LineProtocol qw(data2line);
use Data::Dumper;
my $db = AnyEvent::InfluxDB->new( server => 'http://localhost:8086' );
my $hdl = undef;
tcp_server undef, 8888, sub {
my ($fh, $host, $port) = @_;
$hdl = AnyEvent::Handle->new( fh => $fh );
$hdl->push_read(
    line => sub {
        my (undef, $line) = @_;
        my ($device, $length, $gwStatus, $vbStatus, $vbSensor1stat, $vbSensor2stat, $vbSensor3stat,
            $u1eff, $u2eff, $u3eff, $i1eff, $i2eff, $i3eff, $pw1, $pw2, $pw3, $f1, $f2, $f3,
            $pf1, $pf2, $pf3, $u_ges, $i_ges, $pw_ges, $tms, $ts,
            $s1, $s2, $s3, $q1, $q2, $q3, $s_ges, $q_ges, $pf_ges, $crc) = split(/,\s*/,$line);
        $db->write(
            database    => 'TestDB',
            data    => [
                data2line('TestMeas',
                    # values
                    {
                        gState      => sprintf("%032b", $gwStatus),
                        vbStatus_bin      => sprintf("%016b", $vbStatus),
                        vbS1StatusReg_bin => sprintf("%016b", $vbSensor1stat),
                        vbS2StatusReg_bin => sprintf("%016b", $vbSensor2stat),
                        vbS3StatusReg_bin => sprintf("%016b", $vbSensor3stat),
                        u1eff             => int($u1eff),
                        u2eff             => int($u2eff),
                        u3eff             => int($u3eff),
                        u_ges             => int($u_ges),
                        i1eff             => int($i1eff),
                        i2eff             => int($i2eff),
                        i3eff             => int($i3eff),
                        i_ges             => int($i_ges),
                        pw1               => int($pw1),
                        pw2               => int($pw2),
                        pw3               => int($pw3),
                        pw_ges            => int($pw_ges),
                        f1                => int($f1),
                        f2                => int($f2),
                        f3                => int($f3),
                        pf1               => sprintf("%.3f", $pf1),
                        pf2               => sprintf("%.3f", $pf2),
                        pf3               => sprintf("%.3f", $pf3),
                        pf_ges            => sprintf("%.3f", $pf_ges),
                        s1                => int($s1),
                        s2                => int($s2),
                        s3                => int($s3),
                        s_ges             => int($s_ges),
                        q1                => int($q1),
                        q2                => int($q2),
                        q3                => int($q3),
                        q_ges             => int($q_ges)
                    },
                    # tag keys
                    {
                        GATEWAY_NAME        => "TestGW1",
                        LOCATION_ShortName  => "TestLocation",
                        CONSUMER_ID         => "$device"
                    },
                    $ts
                )
            ],
            # on_success  => sub { print "$line written\n"; },
            on_success  => sub { print "ok\n"; },
            on_error    => sub { print "$line error: @_\n"; },
        );
        $hdl->on_drain(
            sub {
                $hdl->fh->close;
                undef $hdl;
            }
        );
    },
);

};
EV::run;


2.18 - fasthttp error for no content response

Prerequisites

[X] I am using the latest version of Locust
[X] I am reporting a bug, not asking a question

Description
Sending a request that generates a no content response is throwing a error when using fasthttp rest. This isn't happening under 2.17
Command line
Locust UI
Locustfile contents
File "/usr/local/lib/python3.11/contextlib.py", line 137, in __enter__
worker-1  |     return next(self.gen)
worker-1  |            ^^^^^^^^^^^^^^
worker-1  |   File "/opt/venv/lib/python3.11/site-packages/locust/contrib/fasthttp.py", line 382, in rest
worker-1  |     if resp.text is None:
worker-1  |        ^^^^^^^^^
worker-1  |   File "/opt/venv/lib/python3.11/site-packages/locust/contrib/fasthttp.py", line 467, in text
worker-1  |     return str(self.content, self.encoding, errors="replace")
worker-1  |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
worker-1  | TypeError: str() argument 'encoding' must be str, not None

Python version
3.11
Locust version
2.18
Operating system
linux

I think its due to this change https://github.com/locustio/locust/pull/2416/files
it is possible that get_encoding_from_headers will return None which will throw the above error. Need to return as a empty string as per 2.17

Oh... yea sounds about right. Or maybe we should just not try to decode at all when there is no content? I dont have time to look at this right now...

Could just throw in a condition as a quick fix so are able to still utilise the requests lib
self.encoding = get_encoding_from_headers(self.headers)   if self.encoding is None:      self.encoding = ""

https://github.com/locustio/locust/pull/2451/files


Add tests for RequestCancelWorkflowExecution in service/frontend/api/handler.go

Add tests for RequestCancelWorkflowExecution in service/frontend/api/handler.go
What changed?
tests for RequestCancelWorkflowExecution in service/frontend/api/handler.go
Why?
improve unit test coverage
How did you test it?
unit tests
Potential risks
Release notes
Documentation Changes

Pull Request Test Coverage Report for Build 018fe06a-bbb0-4e5a-b0a6-3987fbe463a9
Details

0 of 0   changed or added relevant lines in 0 files are covered.
172 unchanged lines in 16 files lost coverage.
Overall coverage decreased (-0.1%) to 69.687%

Files with Coverage Reduction
New Missed Lines
%

common/task/weighted_round_robin_task_scheduler.go
1
89.05%

service/history/shard/context.go
2
68.99%

service/matching/tasklist/db.go
2
73.23%

service/matching/tasklist/task_list_manager.go
2
76.48%

common/persistence/sql/sqlplugin/mysql/task.go
2
73.68%

service/history/task/transfer_active_task_executor.go
2
72.79%

common/util.go
2
91.78%

common/persistence/sql/sqlplugin/mysql/db.go
2
79.49%

common/log/tag/tags.go
3
50.46%

service/history/handler/handler.go
3
96.2%

Totals

Change from base Build 018fe051-df14-4702-b515-0388c6db07f3:
-0.1%

Covered Lines:
103092

Relevant Lines:
147936

💛  - Coveralls


Delete outdated /deal ordinals-bitcoin-benjies before applying #3594

delete invalid and outdated directory, it should help with #3594 by @DEALwithitDEV

@chrisconfirm done....I think


ADS issue when running ./manage_services.sh start

Getting following error

./manage_services.sh start
mongod (pid 14977) is running...
Starting all services.
http logging service started
localhost:8239
localhost:8239
ERROR:tornado.general:Uncaught exception
Traceback (most recent call last):
File "/usr/lib/python2.6/site-packages/tornado-4.5.1-py2.6-linux-x86_64.egg/tornado/http1connection.py", line 722, in _server_request_loop
ret = yield conn.read_response(request_delegate)
File "/usr/lib/python2.6/site-packages/tornado-4.5.1-py2.6-linux-x86_64.egg/tornado/gen.py", line 1055, in run
value = future.result()
File "/usr/lib/python2.6/site-packages/tornado-4.5.1-py2.6-linux-x86_64.egg/tornado/concurrent.py", line 238, in result
raise_exc_info(self._exc_info)
File "/usr/lib/python2.6/site-packages/tornado-4.5.1-py2.6-linux-x86_64.egg/tornado/gen.py", line 307, in wrapper
yielded = next(result)
File "/usr/lib/python2.6/site-packages/tornado-4.5.1-py2.6-linux-x86_64.egg/tornado/http1connection.py", line 159, in _read_message
max_bytes=self.params.max_header_size)
File "/usr/lib/python2.6/site-packages/tornado-4.5.1-py2.6-linux-x86_64.egg/tornado/iostream.py", line 256, in read_until_regex
self._try_inline_read()
File "/usr/lib/python2.6/site-packages/tornado-4.5.1-py2.6-linux-x86_64.egg/tornado/iostream.py", line 720, in _try_inline_read
self._read_from_buffer(pos)
File "/usr/lib/python2.6/site-packages/tornado-4.5.1-py2.6-linux-x86_64.egg/tornado/iostream.py", line 778, in _read_from_buffer
self._run_read_callback(pos, False)
File "/usr/lib/python2.6/site-packages/tornado-4.5.1-py2.6-linux-x86_64.egg/tornado/iostream.py", line 687, in _run_read_callback
future.set_result(self._consume(size))
File "/usr/lib/python2.6/site-packages/tornado-4.5.1-py2.6-linux-x86_64.egg/tornado/iostream.py", line 910, in _consume
b = (memoryview(self._read_buffer)
NameError: global name 'memoryview' is not defined

CentOS-6.9
python version : 2.6.x

@ksripathi
can you look into it?

The above error is occurring because of the proxy in our network.
@ksripathi Sripathi and I tried modified ovpl code ( which was modified in cluster-automation machine) on base4 cluster setup. There is no error on base4.
We should have solution for ADS running under " NETWORK-PROXY" environment.

This issue is raised because of running tornado==4.5.1 version.
Try to downgrade this to tornado==4.3
pip uninstall tornado==4.5.1
pip install tornado==4.3
This will resolve above issue


While switching cluster in kubernetes extention laptop hangs

Issue Type: Bug
When I switch to any of the listed kubernetes cluster in extension my laptop hangs and the only way to bring it back is to reboot the laptop.
Even my Microsoft Teams, Microsoft Outlook also stops responding after this.

That said, I don't know how it's even possible for us to hang the entire machine.  The only thing I can think of is that we're doing something that ends up consuming 100% CPU, or possibly something that spins off an "infinite" number of kubectl processes.
I don't know how much patience you'll have to investigate given the impact, but if you are able, it would be useful to know:

What happens to CPU usage when you switch cluster?
Do you see a large number of shell or kubectl processes running afterwards?
If you have Process Explorer open (Help > Open Process Explorer) during the switch, does it show new or CPU-heavy processes under the extension host?
Does this happen with all clusters, or only to clusters where you need to authenticate?  (E.g. if you have a KinD or Minikube cluster, are you able to switch to that okay?)

Thanks, and apologies again!

I'm having the same issue.  When I look at the Process Explorer what I see is the extension going and pulling each of the CRDs on the cluster I'm connecting to.  That cluster has a LOT of CRDs.  I don't know what all it's doing with the CRDs, but that seems to be what it's doing when it locks the system.

Ah, this sounds like it might relate to the extension suggestion feature.  @lstocchi, as an interim workaround, would your new config setting bypass the fetching of CRDs or only of putting up the suggestions message?

@itowlson no it only hides the suggestions message. But i don't think the problem is there because it just makes a call to fetch all api-versions so it should be fast.
I guess the problem comes when we fetch all CRDs to find some custom schemas to enable a better code completion :/
Maybe we should give a way to disable it or just give the users the option to target some specific CRDs ... WDYT? @itowlson
Many thanks for your comment @mark-bradshaw, this helps us finding the right issue :pray:

I hope you can fix it.  I enjoy the extension.
On April 12, 2022, Hugo Menzaghi @.***> wrote:

@itowlson https://github.com/itowlson no it only hides the
suggestions message. But i don't think the problem is there because it
just makes a call to fetch all api-versions so it should be fast.
I guess the problem comes when we fetch all CRDs to find some custom
schemas to enable a better code completion :/
Maybe we should give a way to disable it or just give the users the
option to target some specific CRDs ... WDYT? @itowlson
https://github.com/itowlson
Many thanks for your comment @mark-bradshaw <https://github.com/mark-
bradshaw>, this helps us finding the right issue 🙏

—
Reply to this email directly, view it on GitHub
<https://github.com/vscode-kubernetes-tools/vscode-kubernetes-
tools/issues/1050#issuecomment-1096333663>, or unsubscribe
<https://github.com/notifications/unsubscribe-
auth/AAIFQ2WIEQIVPENGFLDUKGTVEUWV5ANCNFSM5ROC56QQ>.
You are receiving this because you were mentioned.Message ID: <vscode-
kubernetes-tools/vscode-kubernetes-
@.***>

@mark-bradshaw could you please tell me approx. how many CRDs you have in your cluster? The one with a lot of CRDs. I am trying to figure out which should be the upper limit to use to change the plugin behavior. Thanks!

220


bad msvc performance

I've proposed a PR (#10) that allows to compile on MSVC.
However the performance is awful (0.65GB/sec instead of 11GB/sec on the same machine, with MSVC Clang)
If anybody is able to look into that.

Side note: it has nothing to do with __builtin_prefetch. MSVC Clang accepts it, does not change results.

Incidental findings: defining
#define really_inline   __declspec(noinline)
MSVC perfomance jumps to 5.6GB/sec. Much better.

The use case for __declspec(noinline) are functions that are small (so likely inlined) but rarely used. It does not make intuitive sense that it would help performance.
In simdjson, we have
 #define simdjson_really_inline __forceinline

In simdcsv, we have...
#ifdef _MSC_VER

#define really_inline inline
#define never_inline __declspec(noinline)

Yes, discovered this by chance.  MSVC results are counter-intuitive, something's wrong in their inlining.

it is find_quote_mask + cmp_mask_against_input that I need to un-inline

something's wrong in their inlining.

Though we have not worked much on simdcsv, we have worked much with simdjson and Visual Studio and I have found that, indeed, something is very wrong with the code generation. At first, we thought that Visual Studio would ignore use and not inline, but it does.
The one strange thing I have spotted in the generated code is that Visual Studio seems to sometimes get confused and copy all the registers to stack and then immediately bring them back from stack. I reported the issue to Microsoft and we even chatted with some of the people from the core Visual Studio compiler team. Unfortunately, I do not think that they investigated the issue, and I ended up giving up and moving on (after reporting the issue in details).
You can see that I did a lot of work...
https://github.com/simdjson/simdjson/issues/847#issuecomment-657858235
You can try manually inlining the functions... it should work. To be clear, manually inlining the functions give better results then relying on the compiler, under visual studio.
@reder2000  Are you able to look at the produce assembly? It could be fun to compare before/after inlining. Just do an assembly dump. I predict that you will see the registering/deregistering issue.
Note that if you use ClangCL, the problem goes away.
References
https://github.com/simdjson/simdjson/issues/847#issuecomment-653225952

it is find_quote_mask + cmp_mask_against_input that I need to un-inline

Or try manually inlining them!

I've been playing a while with it.  It turns out that MSVC performance is worst with /Ob2 than /Ob1.
With /Ob1 performance is 50% of Clang-CL at /Ob2 (and same as CLang-CL at /Ob1)
I tried replacing function with macros, one gets the /Ob2 (really bad) performance whatever the flag, so too much inlining is causing MSVC to act weirdly.
Those people seem to have the same kind of problems: https://github.com/xtensor-stack/xsimd/issues/617

Those people seem to have the same kind of problems: https://github.com/xtensor-stack/xsimd/issues/617

I think their problem is too little inlining.

Maybe I read that too quickly.
I'll submit the issue to MSFT. /Ob2 worst than /Ob1 is not good news. Even though this is a corner case.


Add CS 233 Syllabus

@joshnatis

Added you as a contributor in the README! :)

P.S. let me know if you know anyone that has other recent syllabi / where ppl hang out nowadays so I could ask them there


[Done] Implement spare gaussian process regression

@fritzo @ysaatchi This pull request is an attempt to implement sparse approximation versions of GPR (#702). The main reference for notations is Damianou's thesis. The supported methods are DTC, FITC, and VFE.

[x] Implement model and guide for DTC, FITC, VFE.
[x] Implement inference for the above sparse methods.
[x] Add support for Kernel to quickly calculate diagonal of covariance matrices.
[x] Add options to predict full cov matrix or just diagonal form, with noise or not noise (y or f), jitter.
[x] Move SpareMultivariateNormal to distribution section (make another pull request for this to easy to test).
[x] Add tests to detect if there are errors in the implementation.
[x] Fix bug in tutorial and ~add a section on sparse regression~ (move this part to another pull request).

Note that the implementations of Likelihood module and Mean function module will be in some future pull requests.

@fritzo @ysaatchi I have completed the implementation of sparse regression models. There are several tests for forward pass of these models. And I add a sparse section in the GP tutorial to illustrate the inference step. Would you mind helping me review this pull request?
p/s: I don't know if the way I unify all three models "DTC", "FITC", "VFE" is standard or not (I don't have much background, just learn about GP around two months ago, and this implementation comes from what I understand from various literature - especially this note - after writing down all computation details). As far as I can see, it is pretty fast to inference. The math in the implementation is quite clear to follow because I use the standard notions.

@fritzo Travis fails at running the tutorial due to not detecting my "pyro" kernel. I don't know how to resolve it. :(

@fritzo Travis fails at running the tutorial due to not detecting my "pyro" kernel. I don't know how to resolve it. :(

Could you try reverting the changes to the metadata, in particular removing the kernel name pyro from kernelspec? It's trying to activate the named ipython kernel pyro, but that is only on your local system. I think you can also just run python -m nbstripout --keep-output tutorial/source/gp.ipynb which will revert the non-essential changes to the notebook:
  "metadata": {
    "kernelspec": {		    "kernelspec": {
 -   "display_name": "Python 2",		 +   "display_name": "Python (pyro)",
     "language": "python",		     "language": "python",
 -   "name": "python2"		 +   "name": "pyro"
    },		    },

@neerajprad Thanks for the tip!!! I just modified it.

After https://github.com/pytorch/pytorch/pull/4923, we can use mean, variance in Distributions. I suggest we update Travis with a newer build of pytorch.

@fehiepsi we're having issues building a new PyTorch wheel for testing, but we'll update as soon as we can.

After pytorch/pytorch#4923, we can use mean, variance in Distributions. I suggest we update Travis with a newer build of pytorch.

The wheels are updated, and these methods should be available now. I restarted the tests that were failing earlier.

@ysaatchi @fritzo I addressed your reviews in the last commit. Especially, I fixed a mistake while introducing the noiseless parameter for prediction. Now, we get a reasonable plot for sparse version.
@ysaatchi I already made a preliminary version of the variational approach. Will make a pull request for it soon. It is much cleaner than the exact implementation here and is able to address general likelihood! :)


Feature Request:  Add a 'save lesson' button

Hey! :smile:
Sometimes during a lesson, I'll think to myself: "I should return to this later". I usually forget which lesson that was, and that's my mistake, but I think it'd be great to have a "save lesson" button somewhere. Perhaps even an icon, like a star, which then appears next to the link on the curriculum menu.
(Of course I could just remember to write it down, but it may be a useful feature for others as well).
Looking forward to hearing your thoughts. Thanks!

I second this! Here's an example of what's happening and what is frustrating: http://somup.com/cFXiXhn1Nm

Is this still an issue @ariellehale / @username1001 - It looks like there's been somewhat of an implementation of what you two are talking about.

I forgot all about this. Haven't really used FCC in awhile. Just realized it was updated! This implementation seems more like a progress tracker. What I meant was more along the lines of a bookmarker/lesson saver which can be viewed by the user on their profile (or within a new route). But now that I think about it, it might be useful to just use the browser's bookmarker, or even use note-taking application. :smile:


Make available all stable versions on RTD and display version number below logo

On RTD, we should be able to select stable releases:

Also, when selecting a stable release, the number should appear (instead of "stable"):

Also see https://github.com/neuropoly/spinalcordtoolbox/issues/3015 for more context and how-to

I triggered the builds for previous versions.
As for the stable release (the fact that we want the actual number), it's a feature that has been requested but not merged yet. https://github.com/readthedocs/readthedocs.org/issues/4529
For the moment, the default version is set to latest. Not sure if we want to change it to stable.

As for the stable release (the fact that we want the actual number), it's a feature that has been requested but not merged yet. readthedocs/readthedocs.org#4529

according to @joshuacwnewton it should work, see https://github.com/neuropoly/spinalcordtoolbox/pull/3021/files

according to @joshuacwnewton it should work, see https://github.com/neuropoly/spinalcordtoolbox/pull/3021/files

That change just controls whether to show text at all (blank or stable/latest/etc.)

Also, when selecting a stable release, the number should appear (instead of "stable"):

Just to clear things up, at SCT, we have this:

stable automatically fetches the latest stable version. It will always display the text "stable".
5.0.0 is a tag that was manually activated. It will always display the text "5.0.0".

5.0.0 is a tag-based build that was manually activated. It will display the text "5.0.0" under the project logo.

this is exactly what we want for ivadomed

so should we disable latest and stable and just keep the releases ?

latest --> master (we need to keep)
stable --> latest release (would be the default, we need to keep)
right @joshuacwnewton ?

in this case the current setup should work, right ?

stable should not show "stable", but the latest version tag. @joshuacwnewton do you know if this is possible? is that what you've implemented in sct?

I found the info higher
As for the stable release (the fact that we want the actual number), it's a feature that has been requested but not merged yet. readthedocs/readthedocs.org#4529

I found the info higher
As for the stable release (the fact that we want the actual number), it's a feature that has been requested but not merged yet.readthedocs/readthedocs.org#4529

indeed, it was my misunderstanding. I'm OK with the current proposed change.

We can close this comment.


Typescript types generated ?

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
I would love to use this in typescript, but has no types!
Describe the solution you'd like
A clear and concise description of what you want to happen.
Include types as part of the build
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

I am using flowtype in the project, you interested to submit a PR for typescript?

feel free to submit PR for typescript


Spike DB decision

Acceptance Criteria:
A decision is made which DB implementation to use
Options =>
Firebase
Local Mongo w/ Mongoose

Decided to use SQL w/ JSON Columns


PrisonPearl rewrite using uuids as the identifying factor instead of damage value.

Basically prison pearl differentiates the pearls based on what their damage value is.  This is no longer a viable solution.  We need to move it over to identifying pearls by its first line of lore which should be its uuid.
@gipsy-king I'm assigning you to do this, is that okay?

'tis a big one. Apparently we've got what, two tiers of backwards compatibility to maintain unless we can upconvert all pearls? (pre-uuid pearls, post-uuid pearls, and with this, lored pearls)

Well you just add another step to the preexisting script.
On Jul 23, 2015 3:28 PM, "Daniel Boston"<EMAIL_ADDRESS>wrote:

'tis a big one. Apparently we've got what, two tiers of backwards
compatibility to maintain unless we can upconvert all pearls? (pre-uuid
pearls, post-uuid pearls, and with this, lored pearls)
—
Reply to this email directly or view it on GitHub
https://github.com/Civcraft/PrisonPearl/issues/60#issuecomment-124218195
.

prisonpearl should touch every pearl every restart thanks to pearl feeding, anyone running prisonpearl recently should have all their pearls on the latest version, unless we want to care about people jumping from really old PP to modern PP without any issues.

Meant by my comment that I've got reports from Eden that some of their pearls are so old they don't have UUID's. Meaning actively pearls on Civcraft currently span versions.

That's just the lore, good catch though.  Need to make sure that it is
actually there.
On Jul 23, 2015 8:22 PM, "Daniel Boston"<EMAIL_ADDRESS>wrote:

Meant by my comment that I've got reports from Eden that some of their
pearls are so old they don't have UUID's. Meaning actively pearls on
Civcraft currently span versions.
—
Reply to this email directly or view it on GitHub
https://github.com/Civcraft/PrisonPearl/issues/60#issuecomment-124275760
.

shouldn't the pearl be updated when they pick it up?

@gipsy-king how goes this?

Not really apparently.

didn't look into PP at all, sorry for not telling you.

NP. I'll start it if you'd rather me.
On Aug 8, 2015 8:17 AM, "Benjamin Grosse"<EMAIL_ADDRESS>wrote:

didn't look into PP at all, sorry for not telling you.
—
Reply to this email directly or view it on GitHub
https://github.com/Civcraft/PrisonPearl/issues/60#issuecomment-128972543
.

@rourke750 yeah I checked it out and I don't really understand most of PP.

Status? Isn't this live?

it died, I think roruke fixed it but I have not tried again since the last debacle.

@rourke750  did you make any improvements here? Should we try again?

just tried again on a local copy, another failure, here is the stack.
[12:17:24 ERROR]: Error occurred while enabling PrisonPearl v2.5.5 (Is it up to date?)
java.lang.ArrayIndexOutOfBoundsException: 3
at com.untamedears.PrisonPearl.PrisonPearlStorage.load(PrisonPearlStorage.java:106) ~[?:?]
at com.untamedears.PrisonPearl.PrisonPearlPlugin.load(PrisonPearlPlugin.java:429) ~[?:?]
at com.untamedears.PrisonPearl.PrisonPearlPlugin.onEnable(PrisonPearlPlugin.java:129) ~[?:?]
at org.bukkit.plugin.java.JavaPlugin.setEnabled(JavaPlugin.java:321) ~[spigot.jar:git-Spigot-d97e08b-880a532]
at org.bukkit.plugin.java.JavaPluginLoader.enablePlugin(JavaPluginLoader.java:340) [spigot.jar:git-Spigot-d97e08b-880a532]
at org.bukkit.plugin.SimplePluginManager.enablePlugin(SimplePluginManager.java:405) [spigot.jar:git-Spigot-d97e08b-880a532]
at org.bukkit.craftbukkit.v1_8_R3.CraftServer.loadPlugin(CraftServer.java:357) [spigot.jar:git-Spigot-d97e08b-880a532]
at org.bukkit.craftbukkit.v1_8_R3.CraftServer.enablePlugins(CraftServer.java:317) [spigot.jar:git-Spigot-d97e08b-880a532]
at net.minecraft.server.v1_8_R3.MinecraftServer.s(MinecraftServer.java:414) [spigot.jar:git-Spigot-d97e08b-880a532]
at net.minecraft.server.v1_8_R3.MinecraftServer.k(MinecraftServer.java:378) [spigot.jar:git-Spigot-d97e08b-880a532]
at net.minecraft.server.v1_8_R3.MinecraftServer.a(MinecraftServer.java:333) [spigot.jar:git-Spigot-d97e08b-880a532]
at net.minecraft.server.v1_8_R3.DedicatedServer.init(DedicatedServer.java:263) [spigot.jar:git-Spigot-d97e08b-880a532]
at net.minecraft.server.v1_8_R3.MinecraftServer.run(MinecraftServer.java:524) [spigot.jar:git-Spigot-d97e08b-880a532]
at java.lang.Thread.run(Thread.java:745) [?:1.8.0_51]

@rourke750 status?


[Bug] Fences Connect to Angled Blocks (On All Sides)

Problem Statement
Fences always connect to angled blocks, even if it looks odd when they do.
Probably Bad:

Definitely Good:

Ok:

Steps to Reproduce

Place down a fence, preferably a vanilla one.
Place a sloped block so that the slope-y bit faces the fence. Observe how the fence appears to connect to the slope-y block.

Additional Information
In vanilla fences don't connect to some blocks (e.g. glow stone) I'm not sure whether they should connect to carpenter blocks skinned as such blocks, but it's something to keep in mind:

This also occures with BlockCraftery Fences.


Allow worker threads to wait for event bus replies

Worker threads are not able to send/receive event bus messages while they run.
While they can register as a consumer for messages, they can't interact with any other verticle running as receiving replies is asynchronous.
CompletableFuture does not work in this scenario, thus a new blocking runOnEventBus function is proposed.
JsonObject result = vertx.runOnEventLoop(future -> {
    vertx.eventBus().send("thop", cmd, reply -> {
        System.out.println(reply);
        if (reply.succeeded()) {
            JsonObject value = (JsonObject) reply.result().body();
            future.complete(value);
        } else {
            future.completeExceptionally(reply.cause());
        }
    });
});

More discussion:
https://github.com/eclipse/vert.x/issues/1386

I gave some thoughts at this point, this is actually forbidden because worker concurrency model mandates that at most one thread can execute the worker at the same time.
Multithreaded worker model works with such a case, for instance this works:
  @Test
  public void testWorkerWaitForReply(TestContext tc) {
    vertx.eventBus().consumer("the-address", msg -> {
      msg.reply(msg.body());
    });
    Async async = tc.async();
    vertx.deployVerticle(new AbstractVerticle() {
      @Override
      public void start() throws Exception {
        CountDownLatch latch = new CountDownLatch(1);
        vertx.eventBus().send("the-address", "hello", reply -> {
          latch.countDown();
        });
        latch.await(10, TimeUnit.SECONDS);
        async.complete();
      }
    }, new DeploymentOptions().
      setWorker(true).
      setMultiThreaded(true));
  }


cp Xv4 X

@HeidiSteen This PR moves the lastest version of api, client and bulk import into the subdirs with those names. This should not impact current docs because the current docs use the -v4 subdirs.


V0.1 improve code quality with isort

Description

Improve code quality with isort.
Linked issue(s)/Pull request(s)

184

Type of Change

[x] Non-breaking bug fix
[ ] Breaking bug fix
[ ] New feature
[ ] Test
[ ] Doc update
[ ] Docker update

Related Component

[x] Simulation toolkit
[x] RL toolkit
[x] Distributed toolkit

Has Been Tested

OS:

[x] Windows
[x] Mac OS
[x] Linux

Python version:

[x] 3.6
[x] 3.7

Key information snapshot(s):

Needs Follow Up Actions

[ ] New release package
[ ] New docker image

Checklist

[ ] Add/update the related comments
[ ] Add/update the related tests
[ ] Add/update the related documentations
[ ] Update the dependent downstream modules usage

Codecov Report

Merging #186 into master will decrease coverage by 0.42%.
The diff coverage is 82.82%.

@@            Coverage Diff             @@
##           master     #186      +/-   ##
==========================================
- Coverage   72.49%   72.06%   -0.43%
==========================================
  Files          86       86
  Lines        4064     4066       +2
==========================================
- Hits         2946     2930      -16
- Misses       1118     1136      +18

Flag
Coverage Δ

unittests
72.06% <82.82%> (-0.43%)
:arrow_down:

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

Impacted Files
Coverage Δ

maro/communication/utils/default_parameters.py
100.00% <ø> (ø)

maro/rl/shaping/action_shaper.py
71.42% <ø> (ø)

maro/rl/shaping/experience_shaper.py
66.66% <ø> (ø)

maro/rl/shaping/state_shaper.py
71.42% <ø> (ø)

...simulator/scenarios/citi_bike/decision_strategy.py
42.76% <0.00%> (ø)

...aro/simulator/scenarios/citi_bike/weather_table.py
100.00% <ø> (ø)

maro/simulator/scenarios/helpers.py
74.32% <ø> (ø)

maro/communication/registry_table.py
94.26% <50.00%> (ø)

maro/event_buffer/event_buffer.py
96.29% <66.66%> (ø)

maro/utils/utils.py
48.80% <66.66%> (-21.43%)
:arrow_down:

... and 32 more

Continue to review full report at Codecov.

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


How can I change navigationbar title?

Hiya! So you think it's broken or missing a feature eh?
Welllllll, it just might be.  Do me a favor and check the following.  Then feel free to delete this comment and above.

[ ] Issue does not already exist
Identify exactly what aspect your issue is about

[ ] ignite base application
[ ] ignite the cli
[x] ignite generated code

each route has the title.  Look at your routes for a title property.
https://github.com/infinitered/ignite/blob/master/ignite-base/App/Navigation/Routes.js

Oh, I mean update the title of the screen
Thanks!

Ah, from code!
Any screen has access to the title through navigator.
So you could this.props.navigator.setState({title: 'New Title'})

OK! Thank you very much! ;)

cheers!

this.props.navigator.setState({title: 'New Title'}) doing this overwrites any defined titles in NavigationBarRouteMapper.js
I got over it by adding:
componentWillUnmount() {
        const { navigator } = this.props
        navigator.setState({
            title: null
        });
}

...but it still displays the title up until the scene change animation finishes.
Is there a better way to do this?


Enforce post-release activities

Currently many post-release activities are not being enforced and subsequently not done by component owners.
The 3 main ones I've seen that are not being done are:

Not version bumping the .x branches to the next minor version / developer iteration
Not adding component to the manifest of the next minor version / developer iteration
Not publishing the github release

When Steps 1 & 2 are not done, it blocks components with dependencies from building the next version, until all of the dependencies have performed Steps 1 & 2. Version bumping is needed for constructing local build environments (like cloning core Dashboards and bootstrapping a Dashboards plugin), and manifest changes are needed to produce builds with the next version that can be consumed (like running AD tests using an OpenSearch cluster with job-scheduler and common-utils plugins).
When Step 3 is not done, it is a bad look for viewers of the plugin, when releases are missing or outdated, or inconsistencies from one plugin to the next.

I believe all these things are being automated, cc: @prudhvigodithi, how far are we from that?

Hey @ohltyler yes for OpenSearch the version increment is in place, for OpenSearch Dashboards the task to increment the version is merged, however we need to call the that task in an automated way by integrating it to same workflow that does the version increment for the OpenSearch.
But for all this work in a seamless manner the .x branch should be cut at an early stage for every plugin repo close to same time, so that when the version increment workflow is called for both OpenSearch and OpenSearch Dashboards the PR is open for review and merge, once this is there we can automate to merge the PR after having certain checks and time slots so that the end to end version increment is done, but again for this to happen there could be chances the PR CI checks could fail which has dependency with other plugins that has to be pre built and then consumed by the downstream plugins for the CI checks to pass ( catch 22 situation :) ) .
To avoid over complexity what we can do is once the version increment PR's are open and once we can confident with few data points we can proceed to auto merge, then eventually once the plugins built and pushed the downstream plugins can consume them and CI should pass.
@dblock @bbarani @peterzhuamazon WDTY?

I think we should continue pushing automation here. I don't know what the next step is, but @prudhvigodithi I believe it's yours!
The catch-22 problem was discussed in https://github.com/opensearch-project/opensearch-build/issues/2706. I suggest solving it with an auto-rebase automation for any open version increment PR.

Hi @saratvemulapalli, Looks like you are already taking of automating releases on GitHub as a part of this task? https://github.com/opensearch-project/opensearch-build/issues/1898#issuecomment-1251341348

Hi @saratvemulapalli, Looks like you are already taking of automating releases on GitHub as a part of this task? #1898 (comment)

Yup I am. Assigned it to myself.

@saratvemulapalli Are you planning to create a campaign to automate releases on GitHub across all repos?

@bbarani there has been a campaign: https://github.com/opensearch-project/opensearch-plugins/issues/201
Few plugins have already onboarded.


Allow granular vendoring

Carry of https://github.com/docker/docker/pull/22151 - moves the USAGE description from hack/.vendor-helpers.sh to hack/vendor.sh
closes https://github.com/docker/docker/pull/22151
- What I did
Allow vendoring only one dependency.
- How I did it
Passing command line parameters to hack/vendor.sh
- How to verify it
Run examples below
- A picture of a cute animal (not mandatory but encouraged)

hack/vendor.sh can now accept command line arguments
./hack/vendor.sh github.com/docker/engine-api will revendor only the
engine-api dependency.
./hack/vendor.sh github.com/docker/engine-api v0.3.3 will vendor only
engine-api at the specified tag/commit.
./hack/vendor.sh git github.com/docker/engine-api v0.3.3 is the same
but specifies the VCS for cases where the VCS is something else than git
./hack/vendor.sh git golang.org/x/sys eb2c74142fd19a79b3f237334c7384d5167b1b46 https://github.com/golang/sys.git will vendor only golang.org/x/sys
downloading from the specified URL

ping @vdemeester :smile:

LGTM :angel:
Merging when :green_heart:


status.airTemp.value issue for Canadian Genesis connection

Hyundai / Kia Connect version: Any version beyond 2.26.6
Python version: 3.12.4
Operating System:  ghcr.io/home-assistant/home-assistant:stable container

Description
After  merge #625 I have been receiving the following error on my Canadian Genesis, reverting to 3.23.0 of hyundai_kia_connect_api stops this error from occurring.
File "/usr/local/lib/python3.12/site-packages/hyundai_kia_connect_api/VehicleManager.py", line 85, in update_all_vehicles_with_cached_state
self.update_vehicle_with_cached_state(vehicle_id)
File "/usr/local/lib/python3.12/site-packages/hyundai_kia_connect_api/VehicleManager.py", line 90, in update_vehicle_with_cached_state
self.api.update_vehicle_with_cached_state(self.token, vehicle)
File "/usr/local/lib/python3.12/site-packages/hyundai_kia_connect_api/KiaUvoApiCA.py", line 180, in update_vehicle_with_cached_state
self._update_vehicle_properties_base(vehicle, state)
File "/usr/local/lib/python3.12/site-packages/hyundai_kia_connect_api/KiaUvoApiCA.py", line 252, in _update_vehicle_properties_base
and get_child_value(state, "status.airTemp.value")[-1] == "H"
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^
TypeError: 'NoneType' object is not subscriptable
What I Did
Simple start of home assistant plugin kia_uvo. This issue occurs persistently after updating from 2.26.6 of the connect plugin for home assistant. I have performed a local workaround by simply updating the check to include a check for None type however I am not sure if this would negatively impact anything else further in the code.
i.e.
        if (
            get_child_value(state, "status.airTemp.value") is not None
            and get_child_value(state, "status.airTemp.value") != "OFF"
            and get_child_value(state, "status.airTemp.value")[-1] == "H"
        ):

I added the check. That should solve it.
Could you update your manifest file and test for me?

Thanks, works fine for me.


Allow user selecting icon using app

To my understanding it still is an unresolved issue on how to easily add the images as profile icons to ageis.
I think a nice solution to this Problem could be, if this icon pack had its own app, which implements being an image source on Android. Currently aegis allows you to pick an icon, which triggers the standard Android source selection. Would this icon pack be "wrapped" in an app, people would be able to select an icon in aegis, triggering this wrapper app, and fetch store the icon in aegis.

Unfortunately I don't have programming skills for that (and I have tried, it's not my thing). Only programming I do is batch files, HTML and CSS :P
Unless someone else wants to do it, then go for it. But I'm only designer, so I only do icons.
And note that Aegis will have better icon system someday as said by developers.


Improve the inheritance architecture of the contracts with overriding public variables

See https://github.com/hifi-finance/hifi-protocol/issues/33.

Not applicable to this repo.


[Antctl] Add an output format to better display multi-line string responses.

Current output formatters (table, json, and yaml) of Antctl are not good at displaying multi-line string responses.
Added a new output formatter called "raw" whose output is similar to fmt.Print(responseString) which directly lets user redirect the output into a file and apply it afterwards.
Fixes #3426
Signed-off-by: Kumar Atish<EMAIL_ADDRESS>

Codecov Report

Merging #3589 (ea0bae8) into main (7be763e) will decrease coverage by 2.27%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #3589      +/-   ##
==========================================
- Coverage   64.16%   61.89%   -2.28%
==========================================
  Files         278      278
  Lines       27825    27827       +2
==========================================
- Hits        17854    17223     -631
- Misses       8063     8775     +712
+ Partials     1908     1829      -79

Flag
Coverage Δ

kind-e2e-tests
47.77% <ø> (-3.38%)
:arrow_down:

unit-tests
43.49% <0.00%> (-0.05%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/antctl/command_definition.go
43.89% <0.00%> (-0.30%)
:arrow_down:

...g/agent/apiserver/handlers/featuregates/handler.go
0.00% <0.00%> (-82.36%)
:arrow_down:

pkg/controller/egress/store/egressgroup.go
1.38% <0.00%> (-55.56%)
:arrow_down:

...kg/apiserver/registry/system/supportbundle/rest.go
20.45% <0.00%> (-54.55%)
:arrow_down:

...ver/registry/controlplane/nodestatssummary/rest.go
50.00% <0.00%> (-50.00%)
:arrow_down:

pkg/support/dump.go
8.19% <0.00%> (-49.19%)
:arrow_down:

...egator/apiserver/handlers/recordmetrics/handler.go
0.00% <0.00%> (-44.45%)
:arrow_down:

pkg/support/dump_others.go
0.00% <0.00%> (-44.00%)
:arrow_down:

...g/agent/apiserver/handlers/addressgroup/handler.go
0.00% <0.00%> (-40.00%)
:arrow_down:

...agent/apiserver/handlers/appliedtogroup/handler.go
0.00% <0.00%> (-40.00%)
:arrow_down:

... and 35 more

Do we have an example of such multi-line output, with a comparison of "raw" and other format?

Currently we don't have any antctl command whose output include multiple-line string responses.
You can find example in the issue #3426 . Also i changed output of an antctl command for testing purpose.
root@minikube:/# antctl trace-packet
|
  Hello
  World
   Project Antrea
root@minikube:/# antctl trace-packet -o yaml
|
  Hello
  World
   Project Antrea
root@minikube:/# antctl trace-packet -o json
"Hello\nWorld\n Project Antrea\n"
root@minikube:/# antctl trace-packet -o raw
Hello
World
 Project Antrea

Thanks for the examples. The proposal looks good to me.

Hi @tnqn @jianjuns  Could you help to check this PR and move forward? thanks.

/test-all

/test-all

/test-all


errors is not defined

It keeps saying errors is not defined and failing.

Nevermind, I'm dumb

Same issue for me while using anonymous login both on Windows and raspberry pi

actually if you are using sample code from npm/git you need to correct error variable as error is not defined.
return reject(new Error("Invalid username or password", 401));

instead of
return reject(new errors.GeneralError('Invalid username or password', 401));


Paginator should allow a bound custom pagination function so that server side pagination is possible

Describe what feature you'd like. Pseudo-code, mockups, or screenshots of similar solutions are encouraged!
At present we are able to paginate over javascript arrays. It will be quite helpful if pagination query can be handled by a custom function so that server side paging can be done.
normally such a function will have the form
(filter:any,offset:number,limit:number,size:number)=>{data:any[],offset:number,limit:number,size:number}

where
filter contains the data filter and sort options
offset is the page offset
limit is the number of records per page
size is the total number of records returned as query result
What type of pull request would this be?
Enhancement
Any links to similar examples or other references we should review?
No response

the current implementation allows the same, I just did not see it


Redirect to new API reference docs.

We'll remove the pubsub check later.

@jmdobry @lukesneeringer I had opened a PR against the docs site to add redirects a bit differently.
https://github.com/GoogleCloudPlatform/google-cloud-common/pull/239
Do we want to allow users to view older versions of the docs since I assume they weren't formatted correctly to be on the cloud site? Or do we not care about old versions anymore??

I'll leave it up to @lukesneeringer.


Revert "Remove stealth link from all newsletters"

Reverts parameter1/science-medicine-group-newsletters#74

 Remove the hidden links in templates - all sites


variable timestamp_ bits

I noticed timestamp_ in GroundTruthPackage class uses long (32bits) but in MeasurementPackage it uses long long (64 bits). Shouldn't we use same number of bits in both places? And it seems 32 bits cannot hold all widths of time stamps.
It probably won't report out error as we care only about delta in time stamps, but it will incur truncation by long(32bits) I believe.

@jinchenglee This causes a bug in windows mingw. I had hard time figuring out what was wrong. Did you create a pull request?

I did, but I do not see a response.
On Apr 10, 2017 10:00 PM, "Suresh Rathnaraj"<EMAIL_ADDRESS>wrote:

@jinchenglee https://github.com/jinchenglee This causes a bug in
windows mingw. I had hard time figuring out what was wrong. Did you create
a pull request?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/udacity/CarND-Extended-Kalman-Filter-Project/issues/17#issuecomment-293150220,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARGgLobg3VUHNL9FJxfB-FVhgpJ5gVHrks5ruwjwgaJpZM4MsfZH
.

The declaration of previous_timestamp_ in FusionEKF.h should be also be long long. (I lost time tracking that one down.)

Come on ... this issue just cost me a week of frustration.

Hi, I updated the repo with the long long declarations. @jinchenglee I don't see a pull request for this, did you close it at some point?


Add Vikunja

Changes

This PR adds Vikunja, as proposed in #35.
Resolves #35
Supporting Material

Website: https://vikunja.io
Affiliation

I am the creator of Vikunja.
Checklist

[x] I have read the Contributing guide, and confirmed my PR aligns with the requirements
[x] I have performed a self-review (valid markdown formatting, spelling and grammar)
[x] I have indicated whether I have any affiliation with any software/ services added
[x] I agree to follow the repositories Contributor Covenant Code of Conduct

Thanks for the suggestions! I've incorporated and shortened the description of Vikunja a bit.

:tada:

Thanks @kolaente  :)
🔀 Merged!


Got sampled images working. Samplers are next

Description
Writing the necessary stuff so that we can have decoupled image/samplers in bindings
Testing
I changed example 07 a little bit and it runs normally. If you remove the sampler from the equation it doesn't forward any to the vk write
TODO list:
If you still provide a sampler in the DescWrite, it will be forwarded to Vulkan (the current way stuff works is that if the type's category is image and you provide samplers, they will be sent). I think this is not an issue because if the VkDescriptorType of the write is VK_SAMPLED_IMAGE the samplers of the image info are ignored. However it makes no sense to give samplers to an image info for a sampled image descriptor. If necessary I can make it generate a warning or fail

However it makes no sense to give samplers to an image info for a sampled image descriptor. If necessary I can make it generate a warning or fail

Its necessary for us to ignore the sampler as well, because otherwise we write it into the lifetime tracking of the pool.

I'll see when it gets lifetime tracked then and have it be ignored at that point

btw your dxc submodule is not inline with the one on master, please fix so there's no difference


502 Bad Gateway?

Hello, I just wanted to access the demo of the video, and there was an error in the title. Is there something wrong?https://36d0f3ae0b147b22.gradio.app

Thanks a lot for raising this issue, I am rebooting and merge to hugging face.

https://083193edbb48f717.gradio.app try here.


Define milestones for outreach committee

needs to be coordinated with official timeline as per #3.

Looks like they don't use this repo, so closing for now.


NSwag.EXE is not signed, have to reapprove in Carbon Black after update

Every time NSwag Studio gets an update and the installer runs, I have to get NSwag.exe unblocked because the exe is not signed. Are there plans to get this exe signed?

Yes, closing as duplicate of #2906


Shutter information

Hi everyone,
I would like to know if it's possible to modify the shutter settings and how.
I saw on the SeekDevice.h file 3 device commands:

GET_SHUTTER_POLARITY
SET_SHUTTER_POLARITY
TOGGLE_SHUTTER

But I don't know what these commands do and how to use it.
Is there anyone to tell me more about the shutter settings ?
Because when I stream the video, the image freeze during shutter event and thoses events come out very often.
Thank you in advance all.
Etienne

From my research I am pretty certain this is not possible on the current cameras. I suspect it was possible on the first versions of the Seek Compact. Configuring out the shutter now causes the camera to reset.


realself.com Ripper

Hi. Apologies if this is not the proper procedure for requesting a new feature/ripper but I'm not familiar with the protocol here.
Would it be possible to have a ripper for realself.com albums?
Thanks heaps!

This is the proper procedure, but the project is currently dormant..

Provide an example album for the dev(s)

Example album: https://www.realself.com/review/miami-fl-breast-implants-pre-op-breast-augmentation-and-preparations#media-review-822974-image-201984

Sorry... Any update on this please?

Tagging #8 so it shows up in the request list. Closing this issue.


Storybook Support

Describe the bug
I am trying to use this with Storybook but i am getting erros like these:
ERR! TypeError: (0 , vite_1.default) is not a function
ERR!     at Object.viteFinal (/home/projects/vitejs-vite-pfyr4e/.storybook/main.ts:21:16)
ERR!     at eval (/home/projects/vitejs-vite-pfyr4e/node_modules/@storybook/core-common/dist/cjs/presets.js:294:28)
ERR!  TypeError: (0 , vite_1.default) is not a function
ERR!     at Object.viteFinal (/home/projects/vitejs-vite-pfyr4e/.storybook/main.ts:21:16)
ERR!     at eval (/home/projects/vitejs-vite-pfyr4e/node_modules/@storybook/core-common/dist/cjs/presets.js:294:28)

Reproduction
https://stackblitz.com/edit/vitejs-vite-pfyr4e?file=.storybook/main.ts
System Info
System:
    OS: macOS 13.0
    CPU: (10) arm64 Apple M1 Max
    Memory: 1.60 GB / 32.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.18.0 - ~/.n/bin/node
    npm: 8.19.2 - ~/.n/bin/npm
  Browsers:
    Chrome: 107.0.5304.87
    Firefox: 106.0.4
    Safari: 16.1

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

https://github.com/sxzz/vue-macros-storybook
Make sure it's the first plugin config.plugins.unshift(VueMacros()), and @storybook/builder-vite will inject the vite-vue plugin, so you needn't add it to vue macros.
Also, it's really bad for the support of Storybook with Vue and ESM.
There are lots of known bugs. 😅 So, there's no plan to support it officially and I don't promise it will work in the future.

I strongly recommend using Histoire instead.


Toxiproxy docker image 2.1.4

https://hub.docker.com/r/shopify/toxiproxy/

Starting with Testcontainers 1.15.0, the users are responsible for selecting the version, and the "default" ones are deprecated.