Fix nccl regression on PyTorch 2.3 upgrade

As per title, fixes https://github.com/NVIDIA/nccl/issues/1251 in TGI's cuda image, regression introduced in https://github.com/huggingface/text-generation-inference/pull/1730 & https://github.com/huggingface/text-generation-inference/pull/1833
We hit this issue e.g. with llama 3 70B model with TP=4 or TP=8 on H100 & default cuda graphs, one can e.g. repro the hanging with text-generation-benchmark --tokenizer-name meta-llama/Meta-Llama-3-70B-Instruct --sequence-length 128 --decode-length 10 --warmups 2 --runs 100 -b 1, where shards hang in
Thread 1302975 (active): "MainThread"
    sched_yield (libc.so.6)
    ncclLaunchKernelBefore_NoUncapturedCuda (enqueue.cc:968)
    doLaunches (group.cc:161)
    groupLaunch (group.cc:339)
    ncclGroupEndInternal (group.cc:418)
    ncclGroupEndInternal (group.cc:368)
    ncclEnqueueCheck (enqueue.cc:1981)
    ncclAllReduce (collectives.cc:49)
    c10d::ProcessGroupNCCL::collective<c10d::ProcessGroupNCCL::allreduce_impl(at::Tensor&, c10d::AllreduceOptions const&)::{lambda(at::Tensor&, at::Tensor&, ncclComm*, c10::cuda::CUDAStream&)#1}, c10d::ProcessGroupNCCL::collective<c10d::ProcessGroupNCCL::allreduce_impl(at::Tensor&, c10d::AllreduceOptions const&)::{lambda(at::Tensor&, at::Tensor&, ncclComm*, c10::cuda::CUDAStream&)#1}>(at::Tensor&, at::Tensor&, c10d::ProcessGroupNCCL::allreduce_impl(at::Tensor&, c10d::AllreduceOptions const&)::{lambda(at::Tensor&, at::Tensor&, ncclComm*, c10::cuda::CUDAStream&)#1}, c10d::OpType, char const*, bool)::{lambda(c10::cuda::CUDAStream&, c10::intrusive_ptr<c10d::ProcessGroupNCCL::WorkNCCL, c10::detail::intrusive_target_default_null_type<c10d::ProcessGroupNCCL::WorkNCCL> >&)#1}, c10d::ProcessGroupNCCL::collective<c10d::ProcessGroupNCCL::allreduce_impl(at::Tensor&, c10d::AllreduceOptions const&)::{lambda(at::Tensor&, at::Tensor&, ncclComm*, c10::cuda::CUDAStream&)#1}>(at::Tensor&, at::Tensor&, c10d::ProcessGroupNCCL::allreduce_impl(at::Tensor&, c10d::AllreduceOptions const&)::{lambda(at::Tensor&, at::Tensor&, ncclComm*, c10::cuda::CUDAStream&)#1}, c10d::OpType, char const*, bool)::{lambda(c10::cuda::CUDAStream&, c10::intrusive_ptr<c10d::ProcessGroupNCCL::WorkNCCL, c10::detail::intrusive_target_default_null_type<c10d::ProcessGroupNCCL::WorkNCCL> >&)#2}> (libtorch_cuda.so)
    c10d::ProcessGroupNCCL::allreduce_impl (libtorch_cuda.so)
    c10d::ProcessGroupNCCL::allreduce (libtorch_cuda.so)
    c10d::ops::(anonymous namespace)::allreduce_CUDA (libtorch_cpu.so)

PyTorch 2.3 has a hard requirement on nccl 2.20.5 so I am not completely sure this fix is fine. We could also choose to downgrade.
interesting read as well https://pytorch.slack.com/archives/C3PDTEV8E/p1713223950622429?thread_ts=1712807088.459829&cid=C3PDTEV8E
Will wait for the build to run to check TGI's benchmark again & any potential regression.

Thanks a lot for the find, the fix and the details.
I'm more on the fence of waiting for torch to fix it (2.3.1 hasn't fixed it yet) since afaik this does NOT affect production.
If it did, 100% on your solution (seems better than downgrading for the time being since torch 2.3 still received some nice ugprades).

As you'd like. I am using this fix to benchmark.

Nice fix @fxmarty !
I confirm that upgrading NCCL as proposed fixes the systematic hang on 8xH100 P5 instances. TGI freezes without crashing. Pytorch 2.4 should be released this month, let's check if NCCL gets updated, otherwise it would be nice to merge that patch.


SW-4096 Use latest observations data in plants dashboard

we were sorting by completedDate (yyyy-mm-dd format), there were 2 observations on that date, the earlier one was used
sort by completedTime instead which has millisecond level granularity

The server always returns observation results in reverse completed-time order, so you should be able to always use the first one from the API response. (The order is part of the API contract, not just an implementation detail.)

If all you want is the latest results, you can include limit=1 in the query string and it will give you back the results of the most recently completed observation (so you don't have to download the details of older ones).

If all you want is the latest results, you can include limit=1 in the query string and it will give you back the results of the most recently completed observation (so you don't have to download the details of older ones).

Thanks for the info, Steve.
The app fetches observation results for the entire org and keeps that in the app redux store, after a bit of post-processing (merging planting zone, subzone names, etc) - does not use the search API.
Code in this PR deals with data at the other end of post-processing/searching of in-memory redux observations state. I suppose the server generated order is still maintained, but could break if something is changed in between.


Support for Jottacloud

I was wondering if you would consider implementing Jottacloud into rclone? Jottacloud is an cloud provider located in Norway/Europe, which is among the few cloud providers not located in the US (and therefor not under any obligation to comply with NSA etc.). They operate with an "unlimited" plan (in reality 10 TB as per the TOS) for private customers charging £7.50 a month.
There is no official API documentation, altho an official reply has been made which greenlights hacking/sniffing the API.
Thuss an unofficial python wrapper lib has been made, which works sometimes.
The soruce of the wrapper interface can be found here. Since it's unstable and not actively maintained I'm wondering if rclone would consider adopting Jottacloud into the mix?
The lack of Linux support from Jotta has been frustrating to say the least (ref threads @ Jotta forums: 1 2 3).
Jottacloud website

I'll certainly consider it - thanks for the suggestion.

Hello, jottalib maintainer here.
I just wanted to weigh in and say that while jottalib certainly is maintained (new release every 3 months), and being used daily for backups, @jkaberg is right in one regard: support for rsync like functionality is admittedly not superb. And it's not on the roadmap in the near future either, unless someone steps up to write the code.
So I certainly see the value in rclone support for Jottacloud.  And maybe a go implementation of the Jottacloud protocol would inspire something similar in jottalib. So if you decide to have a crack at it, @ncw, I'll be happy to help out, or answer any questions you may have.

This would really make a massive improvement to jottacloud, making it comparable with the 'high-end-providers' in terms of functionality! :)

Just checked in on this an noticed it's a dupe of https://github.com/ncw/rclone/issues/307, closeing


docs and some small treats for wdi5 2.0

reworked the documentation
add an esm usage example for wdi5
syntactic sugar for switching between SAP Build WorkZone's shell and app area
devx: jsconfig.json samples in CJS- and ESM-flavor

Hey @wridgeu and @marianfoo - sorry for the review ambush 🥷  Added you as reviewers in the hopes that some fresh eyes on documentation will find gaps, shortcomings and typos better :)

btw: failed test is a flaky one, as the wz shell is ...well... anyway, ignore it please
https://github.com/ui5-community/wdi5/actions/runs/6051525181/job/16423173682

closing this in favor of 2 new ones for better separation of concerns.
thx @marianfoo and @wridgeu for the reviews!


frp: 0.45.0 -> 0.46.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/fatedier/frp/releases.
meta.description for frp is: "Fast reverse proxy"
meta.homepage for frp is: "https://github.com/fatedier/frp"
meta.changelog for frp is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.46.0 with grep in /nix/store/1scmw8xlpxmmr5dg7sfg4nqacshgdz2m-frp-0.46.0
found 0.46.0 in filename of file in /nix/store/1scmw8xlpxmmr5dg7sfg4nqacshgdz2m-frp-0.46.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
frp

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/1scmw8xlpxmmr5dg7sfg4nqacshgdz2m-frp-0.46.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A frp https://github.com/r-ryantm/nixpkgs/archive/53ef61410fd7184da23792e6cd600b7e565dbc31.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/1scmw8xlpxmmr5dg7sfg4nqacshgdz2m-frp-0.46.0
ls -la /nix/store/1scmw8xlpxmmr5dg7sfg4nqacshgdz2m-frp-0.46.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

frp

Maintainer pings
cc @Br1ght0ne for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/fatedier/frp/releases.
meta.description for frp is: Fast reverse proxy
meta.homepage for frp is: https://github.com/fatedier/frp
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.46.1 with grep in /nix/store/2saakxly5kpyg5b4xyljsv977jqahm4k-frp-0.46.1
found 0.46.1 in filename of file in /nix/store/2saakxly5kpyg5b4xyljsv977jqahm4k-frp-0.46.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
frp

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/2saakxly5kpyg5b4xyljsv977jqahm4k-frp-0.46.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A frp https://github.com/r-ryantm/nixpkgs/archive/bd4aede85c5f7d4a1e73279c43fea73e00060092.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/2saakxly5kpyg5b4xyljsv977jqahm4k-frp-0.46.1
ls -la /nix/store/2saakxly5kpyg5b4xyljsv977jqahm4k-frp-0.46.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

frp

Maintainer pings
cc @Br1ght0ne for testing.


Creating interpolation search

Adding interpolation search

Create some unit tests for the algorithm, please.

Yeah, I guess it suits.


Add error messaging when attempting to add >33 models

Is this feature request related to a new or existing Amplify category?
api
Is this related to another service?
AppSync
Describe the feature you'd like to request
Currently, there is an AppSync limitation where there is a hard cap of 100 subscriptions. Each model creates 3 subscriptions. This means apps created using amplify cli can have a maximum of 33 models. Given that this is a known limitation, we should add error messaging to users attempting to create more than 33 models, and provide guidance on collapsing the models down to 33 or fewer.
The current error message users get is not descriptive or actionable:
ERR: "Service: AmazonDeepdish; Status Code: 409; Error Code: ConcurrentModificationException; Request ID: aaf0773d-e100-430a-b6c4-a0bd3373a2c3; Proxy: null)

Describe the solution you'd like
A human-readable and actionable error message in this case
Describe alternatives you've considered
N/A
Additional context
No response
Is this something that you'd be interested in working on?

[ ] 👋 I may be able to implement this feature request
[ ] ⚠ This feature might incur a breaking change

Closing this as the issue is believed to have been fixed on the service side.


Add lots of error checking to parser.

In the parser's current state there are not very helpful error messages. The parser operates completely fine on functioning code but doesn't make many validation checks like those found in the CPython parser. These need to be updated as much as possible.

Updated error checking places with get_token_expect().


How make a button that update 2 or more forms?

How make a button that update 2 or more forms????
In PrimeFaces just put styleClass="mystyle" and to the commandButton update="@(.mystyle)"
But how I do in Bootsfaces?

From a technical point of view, I'm sure you can do the same with BootsFaces 0.8.0 and above. We  support the update="@(.mystyle)" syntax, too.
However, you should be aware you're using a feature that's not supported by HTML. Even worse, you're using a feature that's forbidden by the HTML specification. HTML doesn't allow you to submit multiple forms with a single click. So you build your application on weak foundations. In the case of BootsFaces I'm sure this won't break anytime soon, but it's perfectly possible that the PrimeFaces team fixes the bug in a future version.

Oh, and please report your error in the open-source-project (BootsFaces-OSP).

This is how I'm doing it with plain old JSF:
                <script type="text/javascript">

                    function onContentReload(data) {
                        if (data.status === 'success') {
                            var date = new Date();
                            var dateStr = date.getHours() + ":" + date.getMinutes() + ":" + date.getSeconds();
                            console.log(dateStr);
                        }
                    }

                    function refreshPage() {
                        console.log("refreshing page");
                        $("#reloadContentForm_reloadContentButton").click()
                    }

                    $(document).ready(function () {
                        setInterval(refreshPage, 60000);
                    })

                </script>
                <h:commandButton styleClass="hidden" id="reloadContentButton" value="reload">
                    <f:ajax execute="@form" render="applicationEmergencies hostEmergencies"
                            onevent="onContentReload" />
                </h:commandButton>


 key / lineEnd command bug with RTL

When in visual RTL mode, if the last word in a line is an English one, the End key (and the lineEnd command) jump to the wrong place.
Way to recreate, in an RTL CodeMirror instance, e.g. https://codemirror.net/demo/bidi.html:

Write a text in an RTL language (e.g. Hebrew) that ends with an LTR language (e.g. English). I used "ניסיון ABC". Unlike here in this text, the ABC should appear to the left of the Hebrew word.
In that line, press the <End> key. The cursor is actually before the English word instead of after it.
If at this point you press <Enter>, the English word will move to the next line. The expected behavior is of course that <End> followed by <Enter> will start a new empty line.

The bidi handling in CodeMirror 5.x is a bit of a tangled (and, in parts, misguided) mess. I just checked the upcoming version 6, which seems to handle this properly.
It is unlikely that I'll patch this in 5.x, but if you want to try your hand at a pull request, that'd be welcome. Upgrading to the new code is not really recommended yet unless you're willing to deal with beta-quality, somewhat-unstable software, but will be the recommended way to deal with this on the long term.

I tried to spend some time going over the relevant code but like you say, it's somewhat tangled.
We'll wait for version 6 in this case.


[2.x] Add arch test function

Adds a new arch function that is a synonym for test()->group('arch'). This allows users to easily isolate arch tests for inclusion/exclusion when running their test suite.
// Before
it('tests my controllers', function () {
  expect('App\Http\Controllers')->toUseStrictTypes();
});

// After
arch('controllers use strict types', function () {
  expect('App\Http\Controllers')->toUseStrictTypes();
});

If we promote this way of declaring arch tests as the "standard" (in documentation/demos etc), it will allow us to rely on this feature in other tooling (such as mutation testing) to automatically exclude/include architecture tests as required.
It also allows us to add to this function for other cool integrations down the line as we see fit.
One consideration is that currently the PHPStorm plugin does not include support for running functions starting with arch, but as it is open source we could PR that if we want to go ahead with this 👍

I've updated the PR to add support for --arch and --exclude-arch, which is basically shorthand for --group=arch and --exclude-group=arch respectively.


Worker Polling Intervals Guidance

I know "it depends" but can you share some "real-world-experience" how often worker microservices should poll for new tasks via GET /tasks/poll/{tasktype} at Conductor server?
What would be a guidance? Should it be every second or each 5 seconds, etc.?
Or to say it differently:

Are there any performance numbers available how many request a conductor server of a specific hardware can handle (before it  gets overwhelmed)?
What would be a good monitoring way to pro-activley monitor Conductor Server to see whether it gets overwhelmed?

Many thanks

Polling interval should be set at less than a second, the poll API also supports long polling, which means, it can wait for set amount of time if no element is available in the task's queue before returning empty response.  So I would use it in conjunction to set a very low polling interval (~100-500 millisecond) and long poll timeout to say 1 second.  This combination works most effective as if there are items available, then you do not suffer from large polling interval, but if you do not have work items in the queue - excessive polling is avoided.
On the server side, Conductor server can scale horizontally and its main limits are going to be the number of IO operations your Dynomite cluster is configured to handle.  We run a sustained load of 1000 requests per second on Conductor server with a 3 to 6 node cluster.
For the monitoring, you want to use the metrics published by the servlet container/application server for  few items:

% of thread utilization.  Are the server threads starving?
JVM heap utilization
Time spent on GC

For the thread utilization, depending upon the application server in use, you can get the metric out using JMX or similar mechanism.  We use Atlas for our metric collection.

Thanks for sharing your experience!


Typo "Microsoft Storage Explorer"→"Microsoft Azure Storage Explorer"

https://docs.microsoft.com/en-us/azure/marketplace/partner-center-portal/get-sas-uri

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Correct per Cloud Style Guide; merging.


Create sponsor creation feature

This should read all the event data files. If a file contains a sponsor that doesn't already exist, it should create the sponsor file.

Added in https://github.com/mattstratton/probablyfine/commit/45a9e360dcc7a20d564a68b1f7ecda9744a5e568


What does the directory slyd/splash_utils do?

I know that portia uses splash to render the webpage, right? But what's the specific role of the directory slyd/splash_utils? How does it work with other parts of portia?

The files in the splash_utils directory are used to inject javascript into each page that is loaded so we can interact with them. The javascript that it injects allows us to run actions on the page (if you click something in your browser then that event is sent to splash to run on the page) and it also allows page changes to be sent from the server to your browser using mutation observer.


Write a Scala program to find square of a number

Description
Write a Scala program to find square of a number
Input  : 5
Output : 25

How to contribute

Save the solution in findSquareOfANumber.sc file
Add findSquareOfANumber.sc file in find-square-of-a-number folder

!assign


Converting to BinanceKlines gets this error: java.lang.String and java.lang.Boolean are in module java.base of loader 'bootstrap'

Using version 5.1.1 gets the following error when fetching Binance Klines:
java.lang.ClassCastException: class java.lang.String cannot be cast to class java.lang.Boolean (java.lang.String and java.lang.Boolean are in module java.base of loader 'bootstrap')
at org.knowm.xchange.binance.dto.marketdata.BinanceKline.(BinanceKline.java:42) ~[xchange-binance-5.1.1.jar:na]
at org.knowm.xchange.binance.service.BinanceMarketDataServiceRaw.lambda$klines$3(BinanceMarketDataServiceRaw.java:83) ~[xchange-binance-5.1.1.jar:na]
at java.base/java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:197) ~[na:na]
at java.base/java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1625) ~[na:na]
at java.base/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:509) ~[na:na]
at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:499) ~[na:na]
at java.base/java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:921) ~[na:na]
at java.base/java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234) ~[na:na]
at java.base/java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:682) ~[na:na]
at org.knowm.xchange.binance.service.BinanceMarketDataServiceRaw.klines(BinanceMarketDataServiceRaw.java:84) ~[xchange-binance-5.1.1.jar:na]
Code:
private List<BinanceKline> geKlinesFromBinance(
      CurrencyPair currencyPair, KlineInterval interval, long startTime, long endTime) {
    try {
      return binanceMarketDataService.klines(currencyPair, interval, 1000, startTime, endTime);
    } catch (IOException e) {
      e.printStackTrace();
    }
    return new ArrayList<>();
  }

Hello, @mtdayacap
The given code is not enough to understand the reason.
There is a class BinanceKlineExample in Binance-stream with an example, it works, I just checked it.

Hi @rizer1980 . Thanks for checking. I’ll try to replicate it and post the code here so you can also test.

I have the same issue. It's situated in the constructor at line 42:
this.closed = (Boolean)obj[11];
obj[11] is a String that's being attempted to be converted into a Boolean and fails.

As I understand it, you are trying to receive data not from binance-stream, but through a regular binance api
something like that:
List<BinanceKline> kline = ((BinanceMarketDataServiceRaw) binanceExchange.getMarketDataService()).klines(new CurrencyPair("BTC/USDT"),m1);
The code there is from 2018 years, and also not quite ready, since there is no method in the marketdata, only in marketdataraw.
When they wrote in 2023 klines for binance-stream, they break the usual binance-api klines.
And of course the code needs to be reworked a little.
Because this is crooked:
@Path("api/v3/klines")
   List<Object[]> klines(

Need to do a normal klines data parse.

As I understand it, you are trying to receive data not from binance-stream, but through a regular binance api something like that: List<BinanceKline> kline = ((BinanceMarketDataServiceRaw) binanceExchange.getMarketDataService()).klines(new CurrencyPair("BTC/USDT"),m1);
The code there is from 2018 years, and also not quite ready, since there is no method in the marketdata, only in marketdataraw.
When they wrote in 2023 klines for binance-stream, they break the usual binance-api klines. And of course the code needs to be reworked a little.
Because this is crooked:
@Path("api/v3/klines")
   List<Object[]> klines(

Need to do a normal klines data parse.

Yes, exactly. The code is:
Exchange binance = ExchangeFactory.INSTANCE.createExchange(BinanceExchange.class.getName());         BinanceMarketDataServiceRaw binanceMarketDataServiceRaw = (BinanceMarketDataServiceRaw) binance.getMarketDataService();         List<BinanceKline> klines = binanceMarketDataServiceRaw.klines(CurrencyPair.BTC_USDT, KlineInterval.d1);
Is that legacy/deprecated code ? I haven't really looked into the binance-stream api yet, I thought that was more for advanced stuff like websockets.

Is that legacy/deprecated code ?
not legacy/deprecated, but usually we have public methods in MarketDataService, and MarketDataServiceRaw is for internal use only. Since in this case there is no method in the MarketDataService, I conclude that something is not completed.
Stream - for websockets, yes.

for quick fix you can simply delete two lines in BinanceKline.class
private final boolean closed;
and
this.closed = (Boolean) obj[11];
after this simple api works.
but this break a little steams...

Same issue, pls fix it quickly
marketDataService.klines(((CurrencyPair) cp), KlineInterval.m15);


Section commands such as devspace run [command] do not work when the k8s cluster is unavailable

What happened?
Section commands such as devspace run [command] do not work when the k8s cluster is unavailable.
And when the k8s cluster was stopped or unavailable, we can't run the devspace commands and get an error:
fatal error trying to load remote cache from current context and namespace: Get "https://<IP_ADDRESS>:49711/api/v1/namespaces/dev/secrets/devspace-cache-sb": dial tcp <IP_ADDRESS>:49711: connect: connection refused
Latest devspace version when it works - 6.1.0
Link to a recent fix (https://github.com/loft-sh/devspace/pull/2429) for another issue that requires a working cluster.
What did you expect to happen instead?
The command can be run when the cluster is unavailable
How can we reproduce the bug? (as minimally and precisely as possible)
Install version >6.1.0
stop local cluster like k3d
try to execute command, like devspace run [command]
My devspace.yaml:
version: v2beta1

require:
  devspace: 6.1.0
...
commands:
  cluster_stop:
    command: |-
      devspace run destroy
      k3d cluster stop dev
    section: environment
  cluster_start:
    command: |-
      k3d cluster start dev
      devspace run dev
    section: environment

Local Environment:

DevSpace Version: >6.1.0
Operating System: mac
ARCH of the OS: AMD64 | ARM64 | i386
Kubernetes Cluster:
Cloud Provider: other
Kubernetes Version: 1.23

Anything else we need to know?

Hey, @atovstonog! 👋
Thanks for raising this up.
Give us some time to reproduce it and we'll be back with updates.
Appreciate your patience,
See you around! 💯

@FabianKramm this issue come back again )


Customize Mount Name

Right now, mounts are uniquely identified based on the devcontainer ID (${devcontainerId}-gh-cli). The benefits are that every login is separate, so you can log into different accounts in different devcontainers.
The main drawback is that you have to re-login for every devcontainer. Customizing the mount name would allow users to specify a name like "devcontainer-gh-cli" that is shared among all devcontainers that also have the same name. This allows for greater flexibility with sharing github cli logins.
I just have to figure out if the mount values can vary based on the inputs...

There are a few ways I can think of doing this
Method 1: mountName option (infeasible)
My first thought was to have a "mountName" option that could either be the default ${devcontainerId}-gh-cli or a custom name chosen by the user. The mounts.source option will them be defined as ${options.mountName}. I am set on using the ${devcontainerId} variable in the default as it is a unique identifier for the container and stable across rebuilds (source).
Unfortunately, the devcontainerId variable can only be used in parts of the config not used for building the image - which includes inside the devcontainer options. Thus, this method will not work.
Method 2: shared mount
I can have a second mount called "shared" or something that is shared between all containers with my extension. If the user specifies a mount name (that isn't the default value, say "DEFAULT_MOUNT"), we will use the mount defined by the devcontainerId (${devcontainerId}-gh-cli). When the user specifies an alternate mount name, say "default", we can link the ~/.config/gh folder to a folder called "default" inside this shared mount. This is a bit more roundabout way to define a similar feature, but I think it can work considering our limitations.
The only considerations I have with this is that all containers with my extension (that uses the same Docker instance) can have access to all other Github CLI authentications. This can potentially be pretty bad as, by default, the Github CLI stores the OAuth token in plaintext. What if someone creates a malicious feature that uploads your token to a random server?
The token scopes for an authenticated Github CLI OAuth token, by default, is something like "gist, read:org, repo", maybe with "workspaces" as well. I ran gh auth status a couple times in some of my devcontainers but got different values...
I think I should look at how someone can exploit this before further looking into implementing this feature.

Update: https://github.com/orgs/devcontainers/discussions/56 👀


HTML not rendering in toast

Describe the bug
Apologies - Unsure if bug or a feature request. HTML doesn't appear to render within the toast body, it brings the HTML out as a string.
Expectation there is that if HTML is included, it's rendered as HTML!
Reproduction
https://stackblitz.com/edit/bootstra-vue-next-template-pzkwxb?file=src%2FApp.vue
Used Package Manager
npm
Validations

[X] Have tested with the latest version. This is still alpha version and sometime things change rapidly.
[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.

Take a look at using the component property - I think that will do what you want.

Thanks @dwgray - Appreciate your pointer, that's got me what I need :)
    var message = ["Switching to ", h("strong", source), " failed"];

    this.show?.({
        props: {
            title: 'Switching Update',
            variant: 'default',
            value: 2500
        },
        component: h(BToast, null, {default: () => message}),
    })


Update commercial-marketplace-lead-management-instructions-https.md

Freshness review update

hi @sudharocks - you'll need to make metadata changes in the private repo. thanks.

Microsoft authors and contributors need to make content contributions through the private repository so the content can be staged and validated by the current validation rules.
After you make your changes in the private repo, review the articles on staging, and fix any validation issues, you can add the sign-off comment to let PR reviewers know that the updates are ready to be merged.
Read about authors working in the public repo for more information.
If you need help getting set up to work in the private repo, see Set up Git repository locally for documentation.
We'll close this PR. Thank you.


Log and raise a better exception for Ethereum RPC errors

Log and raise a better exception for Ethereum RPC errors

Also, fixed an error for GP tests

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

12 of 15 (80.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.006%) to 90.937%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

gnosis/eth/ethereum_client.py
12
15
80.0%

Files with Coverage Reduction
New Missed Lines
%

gnosis/protocol/gnosis_protocol_api.py
1
91.4%

Totals

Change from base Build<PHONE_NUMBER>:
0.006%

Covered Lines:
2950

Relevant Lines:
3244

💛  - Coveralls


Edit review section

Fixes:  #55
Description
I have made changes in UI of Review form
Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)

Checklist:

[x] I have made this change from my own.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

Please see the conflicts highlighted in the image above. Resolve the conflicts in the mentioned files. Ensure you pull all the latest changes before pushing your feature/bug fixes to the main branch. After resolving the conflicts, please open a new PR for review.
Thank you! @Bhum-ika


bug: The rendering of the elements hidden behind the keyboard is delayed while scrolling and then appears when scrolling stops.

Bug Report
Ionic version:

[x] 4.x
Current behavior:
If I have a row of images (or even a div with a gradient) that is offscreen/behind the keyboard when I scroll down (or up) to it, they do not render properly when the keyboard minimizes. The elements do not appear until I lift my finger off the screen or when the scroller finishes all its animations.
This is causing a super noticeable problem for me, making the whole thing look choppy. Is there any way in which I can prevent this choppy-ness from happening?
Here is a gif of the issue.

Expected behavior:
The expected behavior is for the elements behind the keyboard to quickly render and appear on the screen when the keyboard minimizes.

In the Template:
<ion-content [scrollEvents]="true" (ionScrollStart)="closeKeyboard();">
In the .TS
closeKeyboard() {Keyboard.hide();}
Using the Keyboard Plugin
const { Keyboard } = Plugins;
It goes black like in the GIF above and seems like it's actually closing on scrollend? Am I implementing the desired action incorrectly or could it be an issue with the scroll event firing timing with the keyboard?

We were able to work around this by using touchstart instead.
<ion-content (touchstart)="closeKeyboard()">
with
closeKeyboard() {Keyboard.hide();}
Although, this created an issue with blocking inner ion-content click events so we ended up modifying it to trigger the keyboard close by losing focus on the ion-searchbar. (It would be useful to have exposed a setBlur() event like the existing setFocus()) We implemented a "setBlur()" by creating a function:
loseFocus() { const activeElement = document.activeElement; if (activeElement && activeElement.blur) { activeElement.blur(); } }
inspired by this solution: https://forum.ionicframework.com/t/dismiss-keyboard-on-touch-outside-it/66998/4
We then just set the (touchstart)="loseFocus()" and placed it in a closer encapsulating div to reduce any conflicts. If you implement this, I would recommend directly addressing the input or searchbar instead of doing it globally like this -- it just happened to be the case that this doesn't cause any conflicts in our usage.

Thanks for the issue. This is likely due to the fact that the webview resizes on keyboard open/close by default. Can you try updating your capacitor.config.json file to turn off the resize?
See: https://capacitor.ionicframework.com/docs/apis/keyboard#keyboard-configuration-ios-only-


Draft event survey - CicivCamp

survey is drafted
The plan is to send the survey to everyone who registered and have the first question ask if they attended the event or not.
If yes, there are questions about the event and suggestions for the future.
If no, we ask why not and also some questions about the future.
For everyone that provides feedback, they will be entered in a drawing for a free DataPalooza registration.

survey link included in post-event message to attendees


[BUG] VM's user data added via Edit as YAML will not display on config tab

Describe the bug
When user add/update userData or networkData by Edit as YAML, UI will not display those content.
This would be a bug when user create the VM by YAML, then update it by the form, cloudinit's data will lost.
To Reproduce
Steps to reproduce the behavior:
https://user-images.githubusercontent.com/5169694/181270084-b7bb1177-85ba-4218-af42-1083624cb294.mp4
Expected behavior
User should able to edit them via form or yaml.
Environment:

Harvester ISO version: v1.0.3-rc1
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): Baremetal DL360 4 nodes

After edit by form, the will not able to start.

Verified this bug not fixed in the main case:
https://user-images.githubusercontent.com/5169694/186124384-a8cd6a97-aae8-4002-ba53-56235a936e01.mp4
Test Information

Environment: qemu/KVM single node
Harvester Version: master-dc6a3089-head
ui-source Option: Auto

Verify Steps:

Follow Steps to reproduce in https://github.com/harvester/harvester/issues/2584#issue-1319636738

@guangbochen, @WuJun2016 I think this issue might need some backend fix?
the latest fix still encountered same error:

Test Information

Environment: qemu/KVM single node
Harvester Version: master-4ea12f4f-head
ui-source Option: Auto

Verify Steps:

Install Harvester with any nodes
Create an Image for VM creation
Create VM vm1 by form with customized User Data
Update vm1 by Edit YAML, replace spec.spec.cloudInitNoCloud.secretRef to UserData (plain text)^1
Restart vm1, VM should started successfully

@lanfon72  cloud-init will only take effect the first time you apply it, but the later changes will not take effect unless you go into os and make it take effect manually.

@WuJun2016 yeah the problem here is, VM will not able to start after update VM's cloudconfig via Edit YAML.

@lanfon72  Sorry, I didn't notice the reproduction step
@futuretea   Can you help to see if you support saving cloudInit in both ways ?

currently,

networkData and userData use same secret
if user didn't specify any content in networkData input box. there are no networkdata key in the secret

@WuJun2016 solution: UI add networkdata key to the secret always
wrong:
apiVersion: v1
data:
  userdata: I2Nsb3VkLWNvbmZpZwpwYWNrYWdlX3VwZGF0ZTogdHJ1ZQpwYWNrYWdlczoKICAtIHFlbXUtZ3Vlc3QtYWdlbnQKcnVuY21kOgogIC0gLSBzeXN0ZW1jdGwKICAgIC0gZW5hYmxlCiAgICAtIC0tbm93CiAgICAtIHFlbXUtZ3Vlc3QtYWdlbnQuc2VydmljZQo=
kind: Secret
metadata:
  labels:
    harvesterhci.io/cloud-init-template: harvester
  name: vm1-4xsyw
  namespace: default
type: secret

correct:
apiVersion: v1
data:
  networkdata: ""
  userdata: I2Nsb3VkLWNvbmZpZwpwYWNrYWdlX3VwZGF0ZTogdHJ1ZQpwYWNrYWdlczoKICAtIHFlbXUtZ3Vlc3QtYWdlbnQKcnVuY21kOgogIC0gLSBzeXN0ZW1jdGwKICAgIC0gZW5hYmxlCiAgICAtIC0tbm93CiAgICAtIHFlbXUtZ3Vlc3QtYWdlbnQuc2VydmljZQo=
kind: Secret
metadata:
  labels:
    harvesterhci.io/cloud-init-template: harvester
  name: vm1-4xsyw
  namespace: default
type: secret

Move the issue to v1.1.1 since this is not a common operation with a relative low priority task for now, thanks.

@guangbochen @WuJun2016
find a new case to tigger the Server error. command SyncVMI failed: "ReadCloudInitVolumeDataSource failed: no cloud-init data-source found at volume: cloudinitdisk
1 . create a vm, and keep network data and userdata empty

@lanfon72    This issue has been fixed in issue 2503

Verified this bug has been fixed.
Test Information

Environment: baremetal DL360G9 5 nodes
Harvester Version: master-bbc2b33f-head
ui-source Option: Auto

Verify Steps:

Follow Steps to reproduce in https://github.com/harvester/harvester/issues/2584#issue-1319636738


Add mock textclassifier service

Used by T2C demo along with youtube-viewer.

This change is

All valid comments, I plan on addressing them in a follow up. The text classifier should really return labels + ranges in the original string.


Design for Front Window Poster!

We need to come up with a design to use for a poster to put in the front window of the Ace Hotel during our event!
My idea for it would be something like:

Blue background with sprinkles
"Donut.js" with the donut underneath it
underneath that the words "a donut themed event for the js community" or something like that that briefly describes the event for passerbys

If anyone is willing to jump on this it would be much appreciated! I don't have AI or Sketch on my laptop 😭 Feel free to have fun with it, that's just my preliminary idea!

@emily-plummer I can probably help. 😄

yasss @paulcpederson you da best!

@emily-plummer what size are you thinking?

Ok, I took a stab at this. I made it 24"x32" as that is a pretty typical poster size, but I can change the size if that's too big.
View/Download:
poster.pdf

Thank you so much Paul!! This looks awesome!
One grammar note: 'donut themed' should be 'donut-themed' (add a hyphen).
And I think the description is still too vague for someone who doesn't already know what the JS community is. How about something like: "a night of donuts and talks at the corner of code and creativity"

@matthewmcvickar cool. Here is a version with new copy:
poster2.pdf

PAUL IS DA BEST

This looks great!

just ordered the poster from costco! It will be at the next event! Thanks paul!

woot woot! 🎉


Pass Azure endpoint in options, instead of environment vars

Hi
I'm using the library with Azure and would like to define the endpoint without having to deal with Env variables.
Any chance we can get something similar to https://github.com/wkok/openai-clojure/issues/10 but for the AZURE_OPENAI_API_ENDPOINT ?
thank you!

Hello!
This is now supported in v0.6.0
See: https://cljdoc.org/d/net.clojars.wkok/openai-clojure/0.6.0/doc/usage-azure-openai#options


Adopt Smithy 1.42.0

Describe the feature
Smithy 1.42.0 is releasing on 12/7/2023.
Update to this version, adopting any breaking API changes.
Verify all protocol tests pass.  If new tests are introduced that do not pass (or cannot be made to pass easily), file a ticket to track those tests.
Use Case
To stay on latest Smithy features and bug fixes.
Proposed Solution
No response
Other Information
No response
Acknowledgements

[ ] I may be able to implement this feature request
[ ] This feature might incur a breaking change

Released in aws-sdk-swift 0.33.0


For Apple Developers, Apple will refuse to delete your personal data and AppleID data forever.

Backstory

After reading this post on r/privacy on Reddit HERE If you ever bought a developer license, Apple will refuse to delete your personal data and AppleID data forever
Data

[Apple] Apple.com:
[impossible  ] difficulty:

Submitted twice Accidentally. CLOSED


lv_win strange behaviour

I noticed the following weird things in lv_win behavour:

You have to set the window size in advance or else you will get a window as big as the screen. WHy?? This way you can not create a window which will resize itself to include/involve all the children that are added to it AND also setting layots on the window is kinda useless.

The above problem is because there is code in lv_win_create which sets the size to the size of the screen. So.. I removed it :) hoping that it will solve the problem. The window got smaller (default obj size but it DID NOT autofit to involve the children. What happend instead was that when scrolling the content the entire window body was scrolling, instead of just the area containing the scroll bar??

"Ok, no problem! I need to call set fit on the container of the content" - I thought. So I did lv_win_get_content, then set_fit on that object, then used that object as parent when adding components to the window. Still no auto fit.

What DID happen due to the above, however, was that the window's border around the content area disappeared?

= = =
So to summarize, IMO the expected behaviour should be:

If you create a window without setting it's size it should be the default object size. Adding objects to it should make thw window bigger, if necessary (to involve all children).

When scrolling the window the border should always be visible and only the area inside it (with the scrollbars) should scroll.

Perhaps 1. above should auto fit itself only when the inner page of the wndow is set to auto fit?

Adding objects to the content page of the window should not remove the border.

I illustrated these with screenshots (used a simplified lv_test_win_1() which only draws one window). Last img is expected behaviour.

This is my modified lv_test_win_1 func
void lv_test_win_1(void)
{
    static lv_style_t header;
    lv_style_copy(&header, &lv_style_plain);
    header.body.main_color = LV_COLOR_RED;
    header.body.grad_color = LV_COLOR_MAROON;
    header.body.padding.inner = 0;
    header.text.color = LV_COLOR_WHITE;

    lv_obj_t * win3 = lv_win_create(lv_scr_act(), NULL);
    lv_obj_align(win3, NULL, LV_ALIGN_CENTER, 0, 0);
    lv_win_set_style(win3, LV_WIN_STYLE_HEADER, &header);
    lv_win_set_style(win3, LV_WIN_STYLE_BTN_REL, &lv_style_transp);
    lv_win_set_style(win3, LV_WIN_STYLE_CONTENT_BG, &lv_style_plain_color);
    lv_win_set_style(win3, LV_WIN_STYLE_CONTENT_SCRL, &lv_style_plain);
    lv_win_set_style(win3, LV_WIN_STYLE_BG, &lv_style_plain_color);
    lv_win_set_btn_size(win3, LV_DPI / 3);

    lv_obj_t * bgnd = win3;

#if 0
    lv_obj_t * page = lv_win_get_content(win3);
    lv_cont_set_fit(page, true, true);
    bgnd = page;
#endif

    lv_obj_t * label = lv_label_create(bgnd, NULL);
    lv_obj_set_pos(label, 10, 10);
    lv_label_set_text(label, "Styled window\n\nThe content background has\ndifferent color");
}

It's the intended behavior. The window can't resize itself to involve al children. Instead, its content area will be resized and scrollbars will appear. That's how windows work on PCs too. You have a fix sized area with a header and scrollable content area.
The default size is set to "screen size" because it's the most probable use case.
It seems the window is not the best object type for your needs. Please open a new topic in the Forum's How-to category and continue there.


[Create New Issue]: Stacks in c++

Description
First of all  i will explain what basically stack is then its syntax , its member types and all the functions that can be performed in it.
Domain
C/CPP
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

\assign

@Saviour1001 please assign me this issue.

\assign

/assign

/assign

/assign

I have made a pr on this issue can u plzz take some another issue

Okay no problem


onSuccess and onCancel prop is not triggering

onSuccess={({ checkoutSessionId }) => {
  console.log(
    `Stripe checkout session succeeded. session id: ${checkoutSessionId}.`
  );
}}
onCancel={() => {
  console.log(`Stripe checkout session cancelled.`);
}}

payment already done and it redirect to another screen
it's not trigger onsuccess and oncancel
edit: onCancel is fine :D onSuccess too,

@yudhaaw96 , @A-Tokyo,
Same issue here. Any solution.
Thanks

@akhilsanker
this is my solution, i don't remember exactly what did i do, i think this props is a magic ->> renderOnComplete
<StripeCheckout   stripePublicKey={"KEY"}   checkoutSessionInput={{     sessionId: session,   }}   onSuccess={(res) => {     navigation.goBack();   }}   onCancel={(res) => {     navigation.goBack();   }}   webViewProps={{ test: "test" }}   renderOnComplete={(props) => {     return <></>;   }} />;

@yudhaaw96 ,
Awesome. That worked like a charm 🥇.
In addition  is there any possible way to pass locale, en/de ?
Thanks for the solution 👍

@yudhaaw96
Still don't work for me This is my backend code :
try {
const  session = await stripe.checkout.sessions.create({
    payment_method_types: ['card'],
    mode: 'payment',
    line_items: [{

        amount: req.body.amount* 100,
       currency: 'usd',
       quantity: req.body.quantity,
       name:'Games'
    }],
    success_url: `http://<IP_ADDRESS>:8080/public/uploads/success.html`,
    cancel_url: `http://<IP_ADDRESS>:8080/public/uploads/cancel.html`,
  })
  console.log(session.id)
  res.send({
    sessionId: session.id,session:session
  });

} catch (e) {
  res.status(400);
  return res.send({
    error: {
      message: e.message
    }
  });
}

and this is my frontend
<StripeCheckout
stripePublicKey={stripePublicKey}
checkoutSessionInput={{
sessionId: sessionId,
}}
            onSuccess={() => {
                console.log("hi")
            }}
            onCancel={() => {
                navigation.goBack();
            }}
            webViewProps={{ test: "test" }}
            renderOnComplete={(props) => {
                return <></>;

            }}

        />

@akhilsanker Pass locale in the checkoutSessionInput  prop, docs: https://stripe.com/docs/js/checkout/redirect_to_checkout


How to make a crosstab？

I can't find the charting function to do crosstab

Hi @TdzWork - can you elaborate? I'm not sure what a crosstab is.

@TdzWork may be pivot table?

Hi @TdzWork - can you elaborate? I'm not sure what a crosstab is.

like this.
The value of the column is the original value, not the aggregate value

@TdzWork may be pivot table?

pivot table column values are aggregate rather than raw, and there is no concept of dimensions or measures, I don't think it meets my needs.

We recently added a new pivot table to Superset. If it does not address your needs feel free to open.


feat: cache more end points

Ordered list alphabetically
Added additional end points to the cache:

ApiDictionary::SELF_SERVICE_ARTICLE_RELATED,
ApiDictionary::CORE_BRAND,
ApiDictionary::TICKET_DEPARTMENT,
ApiDictionary::TICKET_PRIORITY,
ApiDictionary::TICKET_SETTINGS,
ApiDictionary::TICKET_STATUS,
ApiDictionary::TICKET_CUSTOMFIELD,
ApiDictionary::USER_USERGROUP,
ApiDictionary::USER_CUSTOMFIELD,

I don't think any of these new end points have POST equivalents? So they shouldn't suffer from the same issue selfservice/comment did?
Are there any others which can be cached or have I covered everything possible here?

We could add
Core

IP bans
IP whitelists
Spam rules

User

Organisation custom fields
Operator groups
Operator roles
Operator permissions

Ticket

Canned responses
Filters
Tags
Channel settings

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Dateparser does not currently recognized time before date format

Dateparser currently does not recognize 'times' before 'dates' formats. When parsing through a document sometimes 'times' are before 'dates'.  It currently brings back a null value if 'times' are before 'dates'. I suggest incorporating something like the following code to find dates and move them to be in front of times. Something like the example below will help to keep your dateparser format working if times are before dates.
import re
import logging
import dateparser
from dateparser import parse

def dateparser(dt_input):
        try:
            # Move 'Dates' to be in front of 'Times'
            date = re.search(r'(\d+(/|-|\.){1}\d+(/|-|\.){1}\d{1,4})', dt_input)
            dt_input = re.sub(date[1], '', dt_input).rstrip()
            dt_input = str(date[1] +' '+ dt_input)
            dt_input_parsed = parse(str(dt_input))
            return dt_input_parsed
        except Exception as e:
                logging.warning(f"Error in finding time in {dt_input}: {e}")

example:

dateparser("10:56:58 PM UTC+2:00 2/22/2018")
datetime.datetime(2019, 2, 22, 22, 56, 58, tzinfo=<StaticTzInfo 'UTC+02:00'>)

Hi @reyesjd86, thank you for opening the issue, could you provide some real examples of texts with the date and times in that order?
Thanks!


Enable property for TS4231 device in ConfigureHeadstage64 operator must be set to True by default

This would make sense and match the default settings for the Enable property of the ConfigureTSS4231V1 operator.

This is the reason why and its a mistake as far as I can tell. Just delete:
https://github.com/open-ephys/bonsai-onix1/blob/450f82922b26917745f2e2a0a54803efb3d62fff/OpenEphys.Onix1/ConfigureHeadstage64.cs#L79

Also, @self check for this in other headstages.


suggestion: Maintain backwards-compatible database format for ValueBalance

This is a suggestion for @upbqdn for PR #8729

We decided to do #8745 instead.


Prepare for 2022-04-13 release

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Currently waiting on a smithy-rs CI update to go through before releasing this


Event Rule with Target creates circular reference.

Describe the bug
When creating a event rule, with pattern and target, the target's source arn is generated with rule's arn. Thus creating a circular reference.
To Reproduce
Create a function, create an event, create a pattern and register both to the event. The resulting cloudformation has the circular reference.
The target's arn can't be overwritten to use the correct arn.
Synth result:
    Type: AWS::Events::Rule
    Properties:
      EventPattern:
        detail:
          - '{"State":["running","shutting-down","stopped"]}'
        detail-type:
          - EC2 Instance State-change Notification
        id:
          - eventPatternForLambda
        region:
          - us-west-1
        source:
          - aws.ec2
      State: ENABLED
      Targets:
        - Arn:
            Fn::GetAtt:
              - MyLambdaCfnEventRuleF85763DB
              - Arn
          Id: StackLambdaDDNSMyLambdaCfnEventRule7AA3E130
MyLambdaCfnEventRuleF85763DB:
    Type: AWS::Events::Rule
    Properties:
      EventPattern:
        detail:
          - '{"State":["running","shutting-down","stopped"]}'
        detail-type:
          - EC2 Instance State-change Notification
        id:
          - eventPatternForLambda
        region:
          - us-west-1
        source:
          - aws.ec2
      State: ENABLED
      Targets:
        - Arn:
            Fn::GetAtt:
              - MyLambdaCfnEventRuleF85763DB
              - Arn
          Id: StackLambdaDDNSMyLambdaCfnEventRule7AA3E130

Expected behavior
The Arn on the Target should be the ARN of the function defined in the code.
Version:
0.29.0

The bug comes from line 121 in rule.ts in aws-events module.
    const targetProps = target.asEventRuleTarget(this.ruleArn, this.node.uniqueId);

As a side thing - you'll find the synth output will display properly if you use a ``` instead of `
As it stands, it's not really easy to parse when trying to understand the bug.

Thanks 👍

I don't think this is still relevant.
Closing, please reopen if I am wrong.


[Language question answering] Delete questionanswering-generate API addition

Data Plane API - Pull Request

API Info: The Basics
Most of the information about your service should be captured in the issue that serves as your engagement record.

Link to engagement record issue:

Is this review for (select one):

[ ] a private preview
[ ] a public preview
[ ] GA release

Change Scope
This section will help us focus on the specific parts of your API that are new or have been modified. Please share a link to the design document for the new APIs, a link to the previous Open API document (swagger) if applicable, and the root paths that have been updated.

Design Document:
Previous Open API Doc:
Updated paths:

❔Got questions? Need additional info?? We are here to help!

 Contact us!
The Azure API Review Board is dedicated to helping you create amazing APIs. You can read about our mission and learn more about our process on our wiki.

💬 Teams Channel
💌 email

Click here for links to tools, specs, guidelines & other good stuff
Tooling

Open API validation tools were run on this PR. Go here to see how to fix errors
Spectral Linting
Open API Hub

Guidelines & Specifications

Azure REST API Guidelines
OpenAPI Style Guidelines
Azure Breaking Change Policy

Helpful Links

Azure DevTools Wiki

Is this being pulled out of 2022-10-01-preview? There's already a PR to publish that and this should be removed there as well. /cc @peytonfraser
Correct. @wangyuantao kindly deleted the required files in the other PR.
https://github.com/Azure/azure-rest-api-specs/pull/20865#pullrequestreview-1120870173
Hence this PR can be closed.


fix(futuredsp): FIR assertions and types

Fix assertion for cutoff of filter.
Add more generic implementation of taps so as to handle Complex32 IQ stream with Complex32 taps.

Great, thank you!


Handshake Protocal Enhance

add revision
handshake rule update

Multi-protocols support has been applied. New rule will be applied on stage of developing new version protocols.


Syntax Problems and TR decomposition alone without NN Layers

Hi
I am trying to implement your MNIST TR classifier:
# Import Necessary Pytorch Modules
import torch
import torch.nn as nn
from torch import Tensor
from tednet.tnn import tensor_ring as tr

# A Simple MNIST Classifier based on Tensor Ring.
class TRClassifier (nn.Module) :
    def init (self):
        super (TRClassifier, self).init()

        # Define a Tensor Ring Convolutional Layer
        self.trcnn = tr.TRConv2D ([1] , [4, 5] , [ 6, 6, 6, 6], 3)
        # Define a Tensor Ring Fully−Connected Layer
        self.trfc = tr.TRLinear ([20, 26, 26],[10], [6, 6, 6, 6])

    def forward (self, inputs: Tensor ) -> Tensor :
        # Call TRConv2D to process inputs
        out = self.trcnn (inputs)
        out = torch.relu (out)
        out = out.view (inputs.size (0), -1)

        # Call TRLinear to classify the features
        out = self.trfc (out)
        return out

First, the syntax for passing inputs and outputs is not clear in any of your tutorials. I tried the following without the output yet:
TRCls = TRClassifier( X_train)
TRCls

and I am getting the error: TypeError: init() takes 1 positional argument but 2 were given
Second, I would like to do a TR decomposition of a ndarray, such as the TT decomposition syntax elsewhere:
from tensorly.contrib.decomposition import tensor_train_cross
factors = tensor_train_cross(T1, rank)
reconstruction_t = np.round(tt_to_tensor(factors), decimals=10)
reconstruction_t
TT_RMSE = math.sqrt(np.square(np.subtract(T1,reconstruction_t)).mean() )
print ("TT   for rank " + str(rank) + " RMSE = ", TT_RMSE)

or
import t3f
a_tt = t3f.to_tt_tensor(T1, max_tt_rank=2)
reconstruction_t = t3f.full(a_tt)
T3f_TT_RMSE = math.sqrt(np.square(np.subtract(T1,reconstruction_t)).mean() )
print ("T3F TT   for rank " + str(rank) + " RMSE = ", T3f_TT_RMSE)

I could not find in your APIs something similar to this. Can you please advise, if you have a TR decomposition or any other Python package provided one?
Third, it is not clear to me from the tutorials, how a linear layer takes an input shape, and an output shape, and then a specific rank for what? If you provide an example with some explanations, this will be clearer.
rank=[2, 2]
model = tr.TRLinear(T1.shape, [2, 2], ranks=rank)
tn_type = "tr"
model.tn_info["type"] = tn_type

thank you,
Manal

Hi Manal,
Thanks for your interest in our package.
To answer your first question,  'TRClassifier' is a Python class, and your 'TRClassifier' has no parameters declared.
You may use:
TRCls = TRClassifier()
Output = TRCls(X_train)

The neural network you defined is called "TRCls," which is distinct from "TRClassifier()."
Regarding your second inquiry, our package does not contain the decomposition implementation. Our package only supports the TR formats for neural network design. However, as far as we know, there is not a python package that supports the tensor ring decomposition algorithm (TR-svd) well. You can try the Matlab package at https://qibinzhao.github.io. You might also find the https://github.com/KhrulkovV/tt-pytorch useful.
For your third question, this is the term used in neural networks. Input shape refers to the input data's feature shape, and output shape means the shape of the feature map after processing.
If you have other questions, feel free to contact us.
Best regards,
Morin

Since this package does not include tensor decomposition algorithms, how can I perform tensor decomposition on layers of neural networks?

Since this package does not include tensor decomposition algorithms, how can I perform tensor decomposition on layers of neural networks?

You can utilize TensorLy for tensor decomposition of the parameter weights and subsequently reload the corresponding tensors into our specified format~(like TT-Layer). Our package exclusively implements these formats, and all our experiments are conducted with random initialization.


Log colours not setting correctly.

To test:
Run sandbox, click on button.
Colours does not go back.

not due to constexpr


Fix authorization header for proxy requests.

The proxy expects just the token in the Authorization header, not Bearer {token} like the GitHub API.
(Sorry for all the thrashing, I think we're almost there.)

 :) Turns out I was testing with bearer {token} locally. The GitHub API is okay with that and the Proxy is not.


Review/referesh for NC Datapalooza website

@wildca01t :) is working on this.

@zachambrose Discussed between us, I've knocked out all items, plus combed through all pages/sub-pages.  Few areas where we may want further refine, once we know more on final details.  I completed vs. heading over to CivicSpark. I'll check out a few other mktg items later.  Enjoy your wkds, guys!

@zachambrose Please upload "data.world" logo for sponsor page. Just let me know what they said before scraping placeholder from their website.

Are they sponsoring open pass or just datapalooza?
On Sat, Aug 12, 2017 at 2:39 PM, wildca01t<EMAIL_ADDRESS>wrote:

@zachambrose https://github.com/zachambrose Please upload "data.world"
logo for sponsor page. Just let me know what they said before scraping
placeholder from their website.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/codeforraleigh/NCOpenPass/issues/457#issuecomment-321998514,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADgoV6d_RzHD6g-RtTEFtS2LDMUxPLK0ks5sXfFTgaJpZM4OdZ6j
.

Should be Open Pass.  I've asked Nick to upload the logo and update the
twitter handle.
On Sun, Aug 13, 2017 at 10:43 AM, Jason Hibbets<EMAIL_ADDRESS>wrote:

Are they sponsoring open pass or just datapalooza?
On Sat, Aug 12, 2017 at 2:39 PM, wildca01t<EMAIL_ADDRESS>wrote:

@zachambrose https://github.com/zachambrose Please upload "data.world"
logo for sponsor page. Just let me know what they said before scraping
placeholder from their website.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<https://github.com/codeforraleigh/NCOpenPass/issues/457#issuecomment-
321998514>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ADgoV6d_RzHD6g-
RtTEFtS2LDMUxPLK0ks5sXfFTgaJpZM4OdZ6j>
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codeforraleigh/NCOpenPass/issues/457#issuecomment-322046228,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQ6IxlEkSUUX-HBEGVOpFrGilGykHdshks5sXwt4gaJpZM4OdZ6j
.

--
Zach Ambrose
<EMAIL_ADDRESS>919-438-2752 (office)
919-616-9969 (cell)
www.AmbroseStrategy.com
https://www.linkedin.com/in/zambrose

@jhibbets @ChrisTheDBA Please let me know your preference on adding logos for CityCamp and NC Open Pass to NC Datapalooza website.  Thanks.

We need the approved logo uploaded to the sponsorship log folder in Drive,
and then open a card on Waffle.io with the URL. Assign it to both of us and
we'll get it done.
On Mon, Aug 14, 2017 at 3:19 PM, wildca01t<EMAIL_ADDRESS>wrote:

@jhibbets https://github.com/jhibbets @ChrisTheDBA
https://github.com/christhedba Please let me know your preference on
adding logos for CityCamp and NC Open Pass to NC Datapalooza website.
Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codeforraleigh/NCOpenPass/issues/457#issuecomment-322283454,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADgoV0Y2zx4HLaHtCNfQ4ULVWdoSJNgdks5sYJ3SgaJpZM4OdZ6j
.

git 'er done!
Chris Mathews
On Mon, Aug 14, 2017 at 3:24 PM, Jason Hibbets<EMAIL_ADDRESS>wrote:

We need the approved logo uploaded to the sponsorship log folder in Drive,
and then open a card on Waffle.io with the URL. Assign it to both of us and
we'll get it done.
On Mon, Aug 14, 2017 at 3:19 PM, wildca01t<EMAIL_ADDRESS>wrote:

@jhibbets https://github.com/jhibbets @ChrisTheDBA
https://github.com/christhedba Please let me know your preference on
adding logos for CityCamp and NC Open Pass to NC Datapalooza website.
Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/codeforraleigh/NCOpenPass/issues/457#issuecomment-
322283454>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
ADgoV0Y2zx4HLaHtCNfQ4ULVWdoSJNgdks5sYJ3SgaJpZM4OdZ6j>
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codeforraleigh/NCOpenPass/issues/457#issuecomment-322284646,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOGta2lW_yKL5pOdQZMQ8iJxnQ7H-9dks5sYJ7zgaJpZM4OdZ6j
.

Completed,@zachambrose


Connecting to external MySQL DB issues

Problem/Motivation
Reported via @balloob, connecting from the add-on to an external DB causes issues.
References
robmarkcole/HASS-data-detective#49

This can be closed.


Add new DataView pull for applications

From Reddit user @samdoesreddit:
A potential feature might be to add another dataview pull with all notes created about that application? maybe ever pull note tags to sort by with minimal. I want to see all the stakeholder end of project review notes for instance. Sort all notes by type ect.

Took a stab at this with release 0.0.1. Please give it a try and let me know if the flow works well for you.

Closing, if the structure for applications needs to be revisited, we should open another issue.


Always Overloaded

Is there an existing issue for this?

[X] I have searched the existing issues

Used image version
f1b97f0461172c352038b2a5f6ff424f2f73dce757d4878b952fdc9ae2524518
Used docker compose file or docker command
No response
What you expected to see, versus what you actually saw
After my server is started, as soon as someone enters, its latency becomes extremely high, and it shows server overload. However, in reality, it only occupies 3-4GB of memory and some CPU. I saw someone on Reddit with a similar situation, and below is someone's reply:

EDIT: The wonderful Vapok in the official Enshrouded Discord gave me a configuration that uses Proton instead of Wine made by the amazing gOOvER... what a difference. All lag gone, CPU usage sliced in half, RAM usage dropped, no more warnings in-game.
Here's a link to the GitHub that has the PROTON egg for those interested.

Could you please take a look?
Logs from the container
No response

Hey, I'm already on it. Will probably be a alternative tag to use.
I try to cleanup my scripts and test it in a few hours. I also will need some help from the community to see the load difference hopefully. I will ask for that here Discord.

Hey, I'm already on it. Will probably be a alternative tag to use.
I try to cleanup my scripts and test it in a few hours. I also will need some help from the community to see the load difference hopefully. I will ask for that here Discord.

Thx

I keep the issue open, you can try out the new dev image which uses proton (dev-proton).

I keep the issue open, you can try out the new dev image which uses proton (dev-proton).我仍在处理这个问题，你可以试试使用质子的新开发镜像 ( dev-proton )。
This is my log, something may be wrong?
_enshrouded_logs.txt

ah, jeah I stumbled over the same problem. I will fix that in a future update. For now you have to force an update.
docker compose exec enshrouded supervisorctl start enshrouded-force-update

I too was seeing high CPU usage on an idle server, and around 2-3GB of idle mem:

Tried the dev-proton image and the pod seems to get stuck after bootstrap and updating (cleared my prior server install on storage just to be sure)
2024-01-27 18:36:23.027 supervisord: enshrouded-bootstrap INFO - proton bootstrap finished
2024-01-27 18:36:23,272 INFO spawned: 'enshrouded-updater' with pid 106
2024-01-27 18:36:23,273 INFO success: enshrouded-updater entered RUNNING state, process has stayed up for > than 0 seconds (startsecs)
2024-01-27 18:36:23.275 supervisord: enshrouded-bootstrap enshrouded-updater: started
2024-01-27 18:36:23.294 supervisord: enshrouded-updater INFO - Running enshrouded-updater
2024-01-27 18:36:23.299 supervisord: enshrouded-bootstrap INFO - Bootstrap complete
2024-01-27 18:36:23,300 INFO exited: enshrouded-bootstrap (exit status 0; expected)
2024-01-27 18:36:23.897 supervisord: enshrouded-updater INFO - Enshrouded server is already the latest version
2024-01-27 18:36:24.123 supervisord: enshrouded-updater INFO - Enshrouded server is not running - starting
2024-01-27 18:36:24,329 INFO spawned: 'enshrouded-server' with pid 117
2024-01-27 18:36:34,346 INFO success: enshrouded-server entered RUNNING state, process has stayed up for > than 10 seconds (startsecs)
2024-01-27 18:36:34.348 supervisord: enshrouded-updater enshrouded-server: started
2024-01-27 18:36:34,376 INFO exited: enshrouded-updater (exit status 0; expected)
2024-01-27 18:36:34.375 supervisord: enshrouded-updater INFO - enshrouded-updater complete

Hope the move to proton helps. Will move back to the latest tag for the time being. Thanks for all the effort this is awesome!

@smizdev nice that it worked out. I will implement a fix when switching to the proton image.
I always get the window error, because I don't have setup the x server properly. But the enshrouded server doesn't really need a window. It falls back to console output anyways.

Is this fix implemented in the current release? I've been able to play fine with only a single user that's on the same network as the server, but as soon as my friend, who's connecting remotely, tries to connect the performance tanks for both him and I. Sometimes he can't even get connected at all.
The same as with the reported issue, CPU and memory usage remain low.

@zdegraaf there is a proton version available yes. But you have to specifically use the image tag (eg. stable-proton).
But that also doesn't solve all the problems. Enshrouded still needs a lot of cpu power to run smoothly.

@cp-fabian-pittroff
I tried running this using stable-proton and overall it was a better experience and was usable playing with two people. I still got the occasional yellow server overloaded message, but I didn't notice any performance drops or disruptive stuttering like I was getting with the Wine version. It still was only using a fraction of the resources I provided to it, though I didn't get a chance to note whether or not any single core. My environment for reference:
Hypervisor OS: Proxmox 6.4-13
Hypervisor CPU: AMD Threadripper 1950X 16 Cores
VM OS: Ubuntu 22.04.3 LTS
VM Specs: 12 cores, 16 GB Ram
Docker Version: Community 25.0.1
Thanks for the great container build, not just with good core features, but also with great stuff like the backup feature.


NOT DEFERRABLE/DEFERRABLE

Fixes #8338.

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/139bef89403d74c0e3194a527abe954096a88335/
Edited pages:

v20.2/begin-transaction.md
v20.2/set-transaction.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/50dcadb24bab8b27e64522f998c9f77d4e3f2a73/
Edited pages:

v20.2/begin-transaction.md
v20.2/set-transaction.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/dcdaedf56867a52ae2e879300fc67022a413f180/
Edited pages:

v20.2/begin-transaction.md
v20.2/set-transaction.md


Unable to view or assign Pod labels or annotations in Edit context

Setup

Rancher version: 2.6.x
Browser type & version:

Describe the bug

Pod labels and annotations aren't visible or editable via the pod edit page

Note - Pod labels and annotations are visible as expected by the pod detail mast head

To Reproduce

Navigate to a Pod detail page and ensure it has some labels in the masthead (if not apply some via yaml)
Navigate to the Config view for the Pod and then the Pod --> Labels & Annotations tabs. Note the missing labels
Navigate to the Edit page for the Pod and attempt to assign some labels. Note that the Save action completes
Navigate to the Pod details page and the masthead does not contain the new label

Result
Expected Result

Edit page should show and allow edit of both types

Additional context

I've reproduced this from https://github.com/rancher/dashboard/pull/7093
I tested Deployment (and pod) labels and they worked fine

TIcket #7094 - Test Results - ✅
With Docker on a single-node instance:
Reproduced on rancher v2.6.9-rc3:

Fresh install of rancher v2.6.9-rc3
Provision a downstream cluster - [ AWS Node driver k3s w/ k8s 1.24.4+k3s1 ]
Once Active, Deploy a workload, with at least one annotation and one label added to pod
Once Active, From Pod Detail page, observe annotation is added
Verified - Annotation successfully added
Edit YAML of pod, and observe both annotation and label accurately configured
Verified - YAMLs accurately represent pod label + pod annotation added
Edit Config of pod, and observe label + annotation, accurately configured
Reproduced - Pod label + pod annotation missing from Edit Config

Screenshots:
Step 5

Step 6 - Pod Annotation YAML

Step 6 - Pod Label YAML

Step 9 - Edit Config of Pod > Labels + Annotations

Verified on rancher v2.7-d8334c8184c93baad8c73834186972d8e26c973b-head:

From reproduction env above, upgrade rancher to v2.7-head
Post-upgrade, Navigate to Pod Details > Labels + Annotations
Verified - Pod Label + Pod Annotation successfully configured
Edit Config of Pod and observe label + annotation successfully configured
Verified - Pod Label + Pod Annotation successfully configured + accurately displayed
Edit the annotation and label on the pod, then Save
Verified - Pod Annotation + Pod Label successfully and accurately updated
Deploy a new workload w/ Pod label + Pod annotation added
Verified - Pod label + Pod annotation successfully configured + accurately editable

Screenshots:
Step 3

Step 5

Step 7


Merge main to develop

SonarQube Quality Gate Status: ERROR

Status: ERROR, MetricKey: new_critical_violations, ActualValue: 33
Status: ERROR, MetricKey: new_major_violations, ActualValue: 7

See Sonar Scan Result at: http://<IP_ADDRESS>:29000/dashboard?branch=merge-main-dev&id=tapdata

exception case
The bug is not from the new code submitted this time, it is caused by a SonarQube's bug.


feat: Go version 1.19

Pull Request Template
Description
Update Go version to 1.19 and slight modification to lint settings.
Fixes #13
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

Dont see a problem since there is no functionality that depends on 1.19.


Add support for details type box for FAQs

This should allow setting box_type: details on FAQs per @bebatut request

I think I already did it in the PR with the TtT material...

are you sure? I don't see it?

My bad, it is the beer tutorial...

ahh ok, we can wait until that one then, will close this again


Inquiry on Adapting NeRF Coordinate System to OpenCV Style for Custom Dataset Training

Hi @donydchen,
Thank you for your fantastic work!
You mentioned that if we want to train on our own dataset, the coordinate system should follow the OpenCV style, where:
The X-axis points to the right,
The Y-axis points downwards,
The Z-axis points in the direction the camera faces (towards the screen).
I would like to inquire about how to train on my own dataset, which uses the coordinate system from the NeRF repository, defined as:
The X-axis points to the right,
The Y-axis points upwards,
The Z-axis points behind the image (i.e., the camera is facing away from the screen).
How should I modify this to match the OpenCV style?
Alternatively, is there any other approach to modifying the code so that I can train on my own dataset?
Thank you very much for your fantastic work, looking forward to your response.

Hey @YaoXingbo, this can typically be done by multiplying the pose (c2w matrix) with a transformation matrix, similar to blender2opencv as done in MatchNeRF. However, I suggest you confirm this further with ChatGPT, which could save you a lot of time.
Once it is converted, you can verify its correctness by plotting the epipolar line, as detailed in this comment. You can verify it on the MVSplat project, which might be easier. And it uses the exact same camera coordinate system as this project.
Typically, there is not much to be changed for another dataset. Ensure you also convert your datasets to the torch format similar to ours. You can refer to more details from convert_dl3dv.py and convert_dtu.py. You can also find more references for different data loaders from MVSplat and NoPoSplat, which all uses the same code base.
Have fun tuning. Cheers!

Hi, @donydchen Thank you for your response! I haved tranformed the coordinate to opencv-style by COLMAP.
But there‘s a confusing issue:
I noticed that the camera extrinsics in DL3D-10K dataset follow OpenCV conventions (+X right, +Y down, +Z into the screen) as described here.
However, in your mvsplat30/src/script/convert_dl3dv.py, the code includes a blender2opencv transformation matrix:
blender2opencv = np.array(
[[1, 0, 0, 0], [0, -1, 0, 0], [0, 0, -1, 0], [0, 0, 0, 1]]
)
Could you kindly clarify why this transformation is needed？As the extrinsics in DL3D-10K are already in OpenCV style.
Thank you again for your response!


Remove dependabot ignore rules

We think most of these are only there because of Python version
incompatibilities which should no longer apply now we're running 3.8.
In future we should include comments with each ignore rule so we know
why each is there. (I'm assuming the intial version of this file was
auto-generated so not finger-pointing here!)
Possibly addresses #2927

This is a mix of the pre-existing items, which I don't know much about, and new ignores which are based only on test failures, so if tests pass I think we should just remove these ignores. The rationale for some of them is documented in the PR - e.g. https://github.com/ebmdatalab/openprescribing/pull/2874 "google-cloud-bigquery-2.0.0 drops support for python 3.5" - but that requires a bit of digging to get to, an inline comment would probably have been easier...?

That makes sense. I guess for now it's OK to remove them all and then add back as and when dependabot creates failing PRs.


gitea dump -c failed to connect to database: Unknown database type:

When I try to backup using the command documented in the official documentation for backup and restore I get the following:
$ su git
git@ed-mh-pi01:/root$ gitea dump -R -c /etc/gitea/app.ini -f /home/git/gitea-dump.zip
2020/08/12 10:38:52 ...dules/setting/git.go:93:newGit() [I] Git Version: 2.20.1, Wire Protocol Version 2 Enabled
2020/08/12 10:38:52 ...dules/setting/log.go:233:newLogService() [I] Gitea v1.12.3 built with GNU Make 4.1, go1.14.6 : bindata, sqlite, sqlite_unlock_notify
2020/08/12 10:38:52 ...dules/setting/log.go:279:newLogService() [I] Gitea Log Mode: Console(Console:info)
2020/08/12 10:38:53 ...les/setting/cache.go:70:newCacheService() [I] Cache Service Enabled
2020/08/12 10:38:53 ...les/setting/cache.go:81:newCacheService() [I] Last Commit Cache Service Enabled
2020/08/12 10:38:53 ...s/setting/session.go:63:newSessionService() [I] Session Service Enabled
2020/08/12 10:38:53 main.go:111:main() [F] Failed to run app with [gitea dump -R -c /etc/gitea/app.ini -f /home/git/gitea-dump.zip]: Failed to connect to database: Unknown database type:

I'm using a mariadb database. I recently migrated to a new server, the command worked fine on the old server.
I'm encountering this issue with gitea 1.12.3
Gitea version 1.12.3 built with GNU Make 4.1, go1.14.6 : bindata, sqlite, sqlite_unlock_notify
This is the same issue as #6393, encountered in an older version, but this issue is marked as closed.

@crobarcro how do your DB config in the app.ini looks like?

@crobarcro how do your DB config in the app.ini looks like?

[database]
DB_TYPE  = mysql
HOST     = <IP_ADDRESS>:3306
NAME     = giteadb
USER     = gitea
PASSWD   = <snip>
SSL_MODE = disable
CHARSET  = utf8
PATH     = /var/lib/gitea/data/gitea.db

two things (but this should not solfe the problem - I'll test some things localy ...):
you can delete PATH as it is only used for sqlite3
and CHARSET should be utf8mb4 when upgrading, if you like proper emoji support.

I have the same problem. Just upgraded to 1.12.3 from 1.5.0 originally deployed on AWS using Turnkey Linux
admin@gitea /home/git$ sudo -u git ./gitea dump -c ./custom/conf/app.ini
2020/09/07 16:38:34 ...s/setting/setting.go:539:SetCustomPathAndConf() [W] Using 'custom' directory as relative origin for configuration file: '/home/git/custom/custom/conf/app.ini'
2020/09/07 16:38:34 ...dules/setting/git.go:93:newGit() [I] Git Version: 2.11.0
2020/09/07 16:38:34 ...dules/setting/log.go:233:newLogService() [I] Gitea v1.12.3 built with GNU Make 4.1, go1.14.6 : bindata, sqlite, sqlite_unlock_notify
2020/09/07 16:38:34 ...dules/setting/log.go:279:newLogService() [I] Gitea Log Mode: Console(Console:info)
2020/09/07 16:38:34 ...les/setting/cache.go:70:newCacheService() [I] Cache Service Enabled
2020/09/07 16:38:34 ...les/setting/cache.go:81:newCacheService() [I] Last Commit Cache Service Enabled
2020/09/07 16:38:34 ...s/setting/session.go:63:newSessionService() [I] Session Service Enabled
2020/09/07 16:38:34 main.go:111:main() [F] Failed to run app with [./gitea dump -c ./custom/conf/app.ini]: Failed to connect to database: Unknown database type:

This is my app.ini
[database]
NAME     = gitea
USER     = gitea
PASSWD   = <omit>
LOG_SQL  = false
DB_TYPE  = mysql
HOST     = <IP_ADDRESS>:3306
SSL_MODE = disable
PATH     = data/gitea.db

Otherwise everything else looks fine

@danielemoroni is your config realy stored in '/home/git/custom/custom/conf/app.ini' ?

I only get this error when gitea get's a wrong config file to work with

@danielemoroni @crobarcro make sure you use the same config-file who is used wenn you run gitea normaly!!
can be checked if INSTALL_LOCK is set in it

@6543 You're right, it's getting the wrong config file. This simply works
admin@gitea /home/git$ sudo -u git ./gitea dump

and yes, I do have this in the app.ini
[security]
INSTALL_LOCK   = true

https://github.com/go-gitea/gitea/pull/12760 will add a message for the user to check the path to config

@crobarcro how do you start/run gitea normaly?

there is a service, so I use
sudo systemctl stop/restart gitea

@danielemoroni what Configuration File Path is displayed at http(s)://your-gitea-instance.com/admin/config

there is a service, so I use
sudo systemctl stop/restart gitea

Same for me

@6543 Sorry, I don't have access to the admin webpage. I'm kind of new to gitea. Is there an equivalent way from the admin CLI or where can I look in the app.ini?

@danielemoroni your dump works so only @crobarcro has to be "fixed" ...
but to get the path of the actual used app.ini, if gitea dump works, then gitea doctor shoult tell you the paths it use

Indeed, it turns out the path to my app.ini was incorrect, as you guessed above. I guess I copied the command from the old server where it worked because the file was actually in that location. So it was a user issue.
However, to avoid people like me taking up your time I would suggest you consider issuing a more relevant error message in this case that helps diagnose the problem.
Thanks for your help!

@crobarcro happy to help :)
and gitea >= 1.12.5 whill have this new error message :)


Fix method equality to be able to validate a JWT

Fix method equality for being abile to validate a JWT parsed from JWS
token.
Comparing the two SigningMethod interfaces produce false negative for
the most of the cases where a token is received over th wire and it has
to be parsed to JWT from a JWS as so far there isn't any other way; the
equality fails because the types which implements the interface are
defined on pointers, hence they cannot be assigned to type values and do
a comparison over the values, and the addresses never will be the same
because the one assigned to the internal field of the JWS values is one
that it's not exposed, so a new one SigningMethod (pointer) must be
provided to the Validate, provoking that the condition always fails

Any chance in the next days to review this?
Thanks

lgtm. sorry it took a a bit to get back to you on this. thank you for the PR :)


Does ttok need an internet connection?

ttok either does not work or is very very slow when the internet connection is bad/non-existent. Is this a known design behavior or a bug?
Related:

#10

@simonw Bumping this; I am considering disabling token counting, but I am not sure if the culprit is really this.

Bumping this again.

ttok seems to use HuggingFace, which is most probably the culprit. See this issue. I suspect some similar env flag must be set for offline usage.


⚠ Samsung has degraded performance

In e0a1f06, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2083 ms

Resolved: Samsung performance has improved in e50d7e2.


Traffic Manager hangs after reload

Hello,
In the lastest version (0.9.9), client.get_trafficmanager() will hang if the world is reloaded and a traffic manager has been created with sync mode before. Am I missing something? A minimal example to reproduce this is
import carla

client = carla.Client('<IP_ADDRESS>', 2000)
world = client.load_world('Town01')
print ("Loaded world")

tm = client.get_trafficmanager()
tm.set_synchronous_mode(True)
print ("TM set to sync")

world = client.load_world('Town01')
print ("Loaded world")

tm = client.get_trafficmanager() # This will hang
print ("Got TM")

I am using Python 3.7 and Ubuntu 18.04.

Hi,
Thanks for reporting the issue! This is indeed happening (even though you are slightly misusing the commands) because the map change/reload happens with synchronous mode on. You should briefly deactivate it before the map change/reload, then reactivate it. Let me know if this solves your issue!

Thanks for your quick response! Setting sync mode to False indeed solves my problem. Also it seems like the CARLA server should be launched without the -benchmark flag for it to work.

@jackbart94 I have reproduced the issue, however, by setting the traffic manager to async mode (i.e., tm.set_synchronous_mode(False)) before reloading the world does not fix the issue for me!

Hello,
unfortunately Carla still crashes after a few successful loops (in my case 4) with the following error:
RuntimeError: failed to connect to newly created map
while 1:
	tm = client.get_trafficmanager()
	tm.set_synchronous_mode(True)
	print ("TM set to sync")

	settings = world.get_settings()
	settings.fixed_delta_seconds = 0.1
	settings.synchronous_mode = True
	world.apply_settings(settings)
	print("Carla sync mode ON")

	# do stuff ...

	tm.set_synchronous_mode(False)
	world.apply_settings(carla.WorldSettings(synchronous_mode=False))
	world = client.load_world('Town01')
	# after some successful loops ... RuntimeError: failed to connect to newly created map
	print ("Loaded world")

I have the same problem. Tried all combinations of setting the world and/or traffic manager to sync/async mode before loading the new world, but the traffic manager always hangs after.
@jackbart94 can we reopen this?

Hi @TimLued @adscib ,
Can you confirm the exact version of Carla that are you using?

I can reproduce @adscib's issue in CARLA <IP_ADDRESS>!

The error occurs for me in Carla <IP_ADDRESS>

I was also running CARLA <IP_ADDRESS> from the official pre-compiled release running on Ubuntu 20.04 with Python 3.7.3. I had a different issue with the traffic manager in CARLA 0.9.9 so didn't get that far, but I can check if the same issue occurs there if that would be helpful.

I am reopening this because I find the solution does not work if there are actors spawned before closing the reloading the world. This happens to me at <IP_ADDRESS>

I'm encountering the same issue on Carla <IP_ADDRESS>.
Interestingly, if I restart the Python script, it reconnects correctly. However, dropping the client handle and creating a new one between connection does not work.

I can also confirm this issue is still present in the 0.9.10 pre-release and in the nightly build I downloaded yesterday.

I also encounter the same issue in CARLA <IP_ADDRESS> and I use Ubuntu 20.04 and Python 3.7.
What I've tried and found:

Setting TM's sync mode to False either before or after reloading the world will cause the TM to hang.
Leaving the server running and restarting the script, it works well (similar to @Beskhue's description).
Starting two servers with different ports, using the same code to create two clients, the first works but the second client's TM will hang (similar to https://github.com/carla-simulator/carla/issues/3220#issuecomment-685976275).
Without TM, everything works fine.

When dropping the client, some network connections are kept open. This is noted in #3212. When these connection are completely dropped the script can successfully connect again, even when using traffic manager and spawning agents.
A workaround is to create the Carla client inside a new process (from multiprocessing). When this process completes, the client and all connections are dropped. Now, launching a new process and setting up Carla again will succeed.
For example:
from multiprocessing import Process
import carla

def run():
    client = carla.Client("localhost", 2000)
    world = client.load_world("Town01")

    traffic_manager = client.get_trafficmanager()
    traffic_manager.set_synchronous_mode(True)

    settings = world.get_settings()
    settings.fixed_delta_seconds = 0.1
    settings.synchronous_mode = True
    world.apply_settings(settings)

    # ... Do things ...

    traffic_manager.set_synchronous_mode(False)

    settings = world.get_settings()
    settings.fixed_delta_seconds = 0.0
    settings.synchronous_mode = False
    world.apply_settings(settings)

while True:
    p = Process(target=run, args=())
    p.start()
    p.join()

See also: https://github.com/carla-simulator/carla/issues/3212#issuecomment-686979191

Hi @Beskhue,
Thanks for your comment! It works when multiprocessing is used. However, in my case, after close(), which closes a client, in any process being called, if I hope to re-instantiate (re-create) a client in the same process, the connection will fail.
So, does it imply that (1) I should create "sub-process" within each process (though I don't know if it is possible) to re-instantiate a client after closing a previous one OR (2) I should never call close() to close a client within a process?
And also here is a code snippet I used to close a client:
def close():
    # clean up actors, sensors, etc

    traffic_manager.set_synchronous_mode(False)

    settings = world.get_settings()
    settings.fixed_delta_seconds = 0.0
    settings.synchronous_mode = False
    world.apply_settings(settings)

@yjiang05 Neither (1) or (2). Using this workaround, you do not re-use the processes. You should create a process per client setup and teardown. So, you create a process, and in it you setup the client, run the things you want to run, and close the client. To run another thing, you repeat the same steps.

Sorry if it took so much to open a PR for this issue, it appears we have found a way to fix it!
It will soon be merged in dev and then to master when 0.9.10 comes out, but if you want to test it (and give us feedback!) the branch is traffic_manager/map_change_fix
Thanks for your patience!

This issue still exist for me. For everybody who is looking for a workaround, you can simply create another tm with a different port.
For example if you created the TM in the first map with port 8000, just create another one with port 8001.
Not ideal, but worked for me.

This issue still exist for me. For everybody who is looking for a workaround, you can simply create another tm with a different port.
For example if you created the TM in the first map with port 8000, just create another one with port 8001.
Not ideal, but worked for me.


Remove unnecessary checks on update with etag

This PR fixes https://github.com/firebase/firebase-admin-python/issues/62 by removing unnecessary checks in function _update_with_etag.
Why remove the checks instead of updating the docs?

To be more consistent with Java/Node API
It is specified in the docs that transaction is a set() operation, however, the current code checks the value as if it's an update() operation.

Code of set(), notice how it doesn't do this check at all and is using put like transaction()
https://github.com/firebase/firebase-admin-python/blob/99c48ef28692a263ef4171b1b0c15489584b9aab/firebase_admin/db.py#L146-L161
Code of update(), instead of using put, it's using patch. The original author might have borrowed the code from here:
https://github.com/firebase/firebase-admin-python/blob/99c48ef28692a263ef4171b1b0c15489584b9aab/firebase_admin/db.py#L186-L200
/cc @alexanderwhatley @hiranya911

I noticed that there's already a PR that changes the API in https://github.com/firebase/firebase-admin-python/pull/59. I'm not sure if that PR fixes this particular problem, but I think it makes sense to merge this PR if public API review takes more time because currently there's no way to update a single leaf element.

Thanks @andyfangdz. This looks pretty good. Would you be able to provide an integration test for this as well? See integration/test_db.py and the integration testing guide for more information.

@hiranya911 Indeed! Here's the integration test. I also took the liberty of bumping a minor version since this change technically does not break anything already working with 2.2.0.

Thanks @andyfangdz for the contribution.

@hiranya911 Can you do a release for 2.2.1 on pypi? Thanks!

@andyfangdz I was waiting to see if we can get #59 also approved, and do a 2.3.0 release sometime next week. I will do a release next week regardless of the status of #59. Sorry about the delay.

@hiranya911 No worries! Thanks for the fast review :smile:. Looking forward to #59!

This was released today.

Thanks!


Make import_docs.sh work on Mac

This just adds a suffix to tempfiles - what was going wrong?
LGTM

sed on BSD requires an argument after -i.


Added extra parameter arguments

Now you can also send the sender_address

idea to drop the servicePointId parameter and replace it by arguments?

@bockhauzen Hey! Sorry for not getting back on this sooner. I forgot to watch this repository and only just now discovered about this pull request =S
If this is still desired, I'm more inclined to add the arguments as additional parameters so that user-error is less likely to occur. That's how the rest of the client is set up. It would then look something like:
public function getShippingMethods(?int $servicePointId = null, ?int $senderAddressId, ?bool $returnMethodsOnly = null): array

Would that work for you?
I'd still have to look into what the "all" option for "sender_address" exactly does. If it's the same as just plain leaving the option out then it's not necessary to implement in the client either.

Hi,
I agree. That would def. be a cleaner solution.
One other question;
There also seem to be an issue with:
client->createLabel
When trying to use DE as country code it seems to fail.
Im trying to debug it, probaly my brains are just failing.
Thanks anyway for your response!

Update: There seems to be a difference between passing "all" and leaving the parameter out entirely. Not passing the parameter will use the default sender address. On my test account not passing the parameter yields 336 shipping methods and passing "all" yields 984. The latter takes over a solid half minute to complete. I'll probably have to increase the client's timeout over this.

@bockhauzen This is now available in v3.8.0! Note that I've moved your other issue to a separate thread (#10) that still requires reproduction steps.


feat(client): support swcli job command

Description

depend on: https://github.com/star-whale/starwhale/pull/2071
changes:

refactor e2e for github action
support swcli job sub commands
refactor pfp、speech examples with @predict and @evaluate decorator

command example:> swcli job --help
Usage: swcli job [OPTIONS] COMMAND [ARGS]...
Job management, list/info/cancel/resume job
Options:
  --help  Show this message and exit.

Commands:
  cancel       Cancel job
  info         Inspect job details
  list (ls)    List all jobs in the specified project
  pause        Pause job
  recover      Recover removed job
  remove (rm)  Remove job
  resume       Resume job

Modules

[x] Client

Checklist

[x] run code format and lint check
[x] add unit test
[x] add necessary doc

others lgtm