Error Compile

When I try to compile I get the following error:
./lib/uWebSockets/src/Node.h:195:9: error: there are no arguments to ‘freeaddrinfo’ that depend on a template parameter, so a declaration of ‘freeaddrinfo’ must be available [-fpermissive]
195 |         freeaddrinfo(result);
|         ^~~~~~~~~~~~
make: *** [Makefile:52: build/database.o] Error 1

What compiler do you use and what version is it?

I use Cygwin 64 with this installed:


get_kubeadmin_token: pass hide_log_command to run_command

In order to prevent kubeadmin password in the command log
depends on https://github.com/RedHatQE/openshift-python-utilities/pull/276

/lgtm

/verified


Writes to Remote Session failing because session host application does not support ReadEntityBodyMode.Classic

Describe the bug
Writes to the session made by a .Net8  application do not persist in the session server hosted by a Classic Framework 4.8.1 Web Forms application
To Reproduce
I can share source code but I'd like to start by describing the situation and seeing if anyone can spot an obvious issue.  Im hoping someone recognizes the very specific errors I'm getting below.
Mostly my code matches whats in this example anyway, except that Im migrating to MVC/.NET8 rather than Blazor:
https://github.com/dotnet/systemweb-adapters/tree/main/samples/WebFormsToBlazor
My WebForms app is OLD and I suspect its missing some key setup but I can't figure it out.

I am able to run the Session server on the WebForms side and I have the Client going on the .NET8 side.

I am able read the session on the .NET8 side but...

The writes are failing and it's not a ReadOnly session
Here is my controller in .Net8:
[Session(IsReadOnly = false)]
public class HelloWorldController : Controller
{
	public string Welcome()
	{
		// This retrieves an item that was set in the Session server on the WebForms app.  It works.
		object setOnWebFormsSide = System.Web.HttpContext.Current.Session["SetOnWebFormsSide"];

		// This should set a value in the Session server on the WebForms app.  It doesn't work.
		System.Web.HttpContext.Current.Session["Message"] = "Hello";

		return "Hi";
	}}

I got the debug symbols for the systemweb-adapters package to trace what happens after I set a session value on the NET8 side:
Inside RemoteAppSessionStateManager.SetOrReleaseSessionData() the line _backchannelClient.SendAsync() returns "StatusCode: 404, ReasonPhrase: 'Not Found'" (which is the result passed up from the nested System.Net.Http.HttpClient.SendAsync() call that it makes):

At first, I could see that my ASP.NET app does not accept PUT requests at all.
I can get it to accept PUT requests by copying the <handlers> section of the web.config from the example linked above, specifically this line:
<add name="ExtensionlessUrlHandler-Integrated-4.0" path="*." verb="*" type="System.Web.Handlers.TransferRequestHandler" preCondition="integratedMode,runtimeVersionv4.0" />
But now the same _backchannelClient.SendAsync() call from above returns "StatusCode: 500, ReasonPhrase: 'Internal Server Error', ..."

It feels like old Webforms apps couldn't handle buffered requests and maybe I need to do some extra setup step to make it support this?  Or I have pipelines setup somewhere that are forcing non-buffered streams but Im not sure where to look.  The WebForms app is very large and hosts web pages and WCF services Im trying to migrate over.  There is a lot in the config files and I don't know where to look.
My WebForms app did not have either the Microsoft.AspNet.Web.Optimization.WebForms or Microsoft.Web.Infrastructure packages like the example but adding these doesn't change the situation.
Any ideas on where to start?
Exceptions (if any)
See the HttpException thrown in the last screenshot.  I think this is from the HttpRequest code inside the WebForms application.
Further technical details
Please include the following if applicable:
ASP.NET Framework Application:

Technologies and versions used (i.e. MVC/WebForms/etc): WebForms/WCF
.NET Framework Version: 4.8.1 and .Net Standard 2.0)
IIS Version:  IIS Express (WebForm application was originally IIS so there could be unknown cruft).   Its also hosted as https on development machines.
Windows Version: 10

ASP.NET Core Application:

Targeted .NET version: 8
.NET SDK version: 8.0.400

This is required, so that's expected

.. it throws an exception "Incompatible_with_input_stream" when it sees the ReadEntityMode is ReadEntityMode.Classic:

Hmmm... didn't realize that would throw. That's unfortunate. Can you try the fix in #550


[Usage] using inside cmake project

Good job! But I still have a question to use it in CMAKE project.
Suppose I have project in the following structures
- root
      - CMakeLists.txt
      - cmake
      - modules
      ...
      - vendor
            - variant

How should I include the variant libraries for codes implemented in modules?

Moreover, I ran bench and found boost::variant is averagely faster than mapbox::variant with better data.


Improved networking stack

The current implementation works, but suffers from disconnecting via websocket. This can be greatly improved with a custom implementation.

As a placeholder, adding ping to the websocket channels should suffice to keep the connections alive.


Package Project development,Play with RTSP ,But not work

In Editor mode is work,But when Package Project ,run the exe , It not work with black texture.How to Package Project ?
unreal 14.3
windows 10

In log There are two warnings ：
It not use VlcMedia to read rtsp://
[2017.06.29-12.01.54:238][ 11]LogWmfMedia:Warning: Failed to finish resolve rtsp://<IP_ADDRESS>:8554/1:
and
LogPluginManager:Warning: Plugin 'VlcMedia' exists at '../../../vlc_NC/Plugins/VlcMedia/VlcMedia.uplugin' and '../../../vlc_NC/Plugins/VlcMedia/VlcMedia.uplugin' - second location will be ignored
LogModuleManager:Warning: ModuleManager: Module 'XMPP' not found - its StaticallyLinkedModuleInitializers function is null.

You forgot to set the desired player plug-in on your media source asset. UE4 will pick a suitable player automatically if not set. Since both VlcMedia and WmfMedia support RTSP streams, UE4 chose to play it with WmfMedia in your case. However, WMF may not support the actual data format in the stream, so the resolving of the media source seems to fail (i.e. it can't be opened).
Set VlcMedia on your media source asset as the default player and try again. If you are opening the RTSP stream using a URL string from code, then set the DesiredPlayerName property to "VlcMedia" on your MediaPlayer asset before opening the URL.

I think I did set the  VideoLAN Media Player (VLCMedia) on my media source asset, I just set Windows player,other platform players set to Auto,  But when package project   still the same problem,In Editor mode everthing is ok.  Did I missing something else?

I got a new Warning in package game.
LogVlcMedia:Warning: Failed to create VLC instance ((null))

Ok, I know what's happening. There was actually a bug that prevented VlcMedia from working in packaged builds. It is fixed for 4.17. You will need these changes: https://github.com/ue4plugins/VlcMedia/commit/cf82a58b1fe433f793b16b9a31a715f7d6a3cab1

OK，It‘s  work！ Thanks！


Media Circus: Vessel Recovery reputation too high, easily gamed

The Vessel Recovery bonuses are really high, and can easily be gamed (by recovering off the launch pad).

Done for 1.1.0


⚠ Processors has degraded performance

In dbacfe5, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 136 ms

Resolved: Processors performance has improved in 535ac0f.


Big ERROR: Why in one place the sentence is translated in one way, and in another place on the file the same sentence is translated differently?

Hello. Why in one place the sentence is translated in one way, and in another place on the file, the same sentence is translated differently?
For example this sentence: "Put in work your power of penetration of Evil and Good."
In one html tag it is translated like this:
<meta name="description" content="Введено в работу вашу силу проникновения зла и хорошего."/>
And in another tag, the same sentence  is translated differently:

PUT В работе ваша сила проникновения "зла" и "хорошее".

Same behavior happens on translate.google.com. This can happen because of contextual translation.

Yes, but very often the translateion of googletrans does not match the one on https://translate.google.com/. I made many test, and I don't know why are not the same. You should make an update or something.
So far, only DeepL has a very good translation with API, but does not use google.


Firefox 41+ supports cut and copy commands

@mystor implemented the cut and copy commands in Firefox 41 (bug 1012662):
https://bugzilla.mozilla.org/show_bug.cgi?id=1012662

I signed it!

@petele, how often are the WebFundamentals and HTML5 Rocks updated?
https://developers.google.com/web/updates/2015/04/cut-and-copy-commands
http://updates.html5rocks.com/2015/04/cut-and-copy-commands

Web Fundamentals is published once or twice a week. It depends on how many PRs have been queued, and if there are any urgent things. I'll be pushing WF later today.
HTML5Rocks Updates isn't updated any more, we've moved all of that content to Web Fundamentals. I thought we'd setup the redirect, but looks like not. I'll follow up on that.


Log trace of uncaught exception in load generator

With this commit we enhance the handlers for uncaught exceptions
in the load generator to log the full exception trace before forwarding
the exception message to the coordinating process. Previously, the
coordinating process would then log the exception message but we lost
the actual trace which makes diagnosis harder.

@elasticmachine test this please

@elasticmachine test this please


Stabilize tests at midnight

Ensure that tests that compare times around now use identical now.

I was running the tests at midnight and noticed some failures.

Even though this change seems useful, it turned out not to fix my original issue: when TZ environment variable is set to a value not supported by pytz/tzlocal but supported by libc/Python, such as TZ=NZST, dateparser miscalculates dates.


CI: Visual regression testing challenges

Ideal developer experience:

Open pull request
AT runs

Reports back with clear pass/fail
Reports easy-to-find details about the failures

Visual regression testing runs

Reports back with clear pass/fail
Reports easy-to-find details about the failures
If failures are actually correct, make flagging those new baselines easier

Need to do a little comparison shopping - how are other projects reporting test results back?  What's the best developer experience here?  Should it print a comment in the PR?  Should it add a review with an inline comment (I've seen plugins that do this and it seems very cool)?


[PROTOCOL] Fix inline DV example in PROTOCOL.md

Which Delta project/connector is this regarding?

[ ] Spark
[ ] Standalone
[ ] Flink
[ ] Kernel
[X] Other (protocol.md fix)

Description
The previous example was incorrect. It had the magic encoded in big-endian, but it should be little-endian. This fixes it.
To see that the previous one was incorrect, you can go to https://www.dcode.fr/ascii-85-encoding, and decode wi5b=000010000siXQKl0rr91000f55c8Xg0@@D72lkbi5=-{L (be sure to select the zeromq encoding). The first 4 bytes of this decode to 0x64, 0x39, 0xD3, 0xD0.  This is<PHONE_NUMBER> (the correct magic), but only if you decode as Big endian.
The new value decodes to 0xD1 0xD3 0x39 0x64, which is<PHONE_NUMBER> in little Endian.

How was this patch tested?
Tested in this pr in the rust kernel

Does this PR introduce any user-facing changes?

Is this duplicated? #2050

Is this duplicated? #2050
Fix #1905

Ohh yep, did not see that one. We should merge that one :)


What do I connect to vref?

In the schematics vref is named. But what should I connect it to?
And do you have some better pictures of your breadboard setup or a parts list from Farnell or something?

You only need to connect every line that is named U_Ref. No exteral connections.
It is an internal reference voltage.
For a picture of the breadboard see the Documentation page again, I added one.
I do not have a parts list, there are only a few parts in the schematic, it is not that difficult.

So the u refs are connected to each other or to the lm393?

Draw a line from each U_Ref to the other and remove the label, maybe that will make it more clear.
The labels are there just so do not have to show crossing wires.

Ah ok I get it. Now it's clear for me thanks. Will buildo it soon to test if it works.

So now its clear I will close the issue.

Sorry bbqkees i have another question. I see you used all through hole components. Which diodes (schottky) you used instead of a smd Bat54s+ and BAT46 package??

You can probably use 1N4148 diodes for all.
I used two different diode types (1N4148 and the other of unknown type but identical or similar), but only because I had those at hand.

another question,
i'm not that electric but i think i can make it work, but does the lm393 also need vcc and ground connected?? on pin 8 and 4???

Yes you need to connect those pins too, otherwise the chip itself has no power.

thanks will do that, stupid mistake

Yes it works now. Thanks for the help.
Is it possible to get more data from the EMS bus??? In the documentation i read a lot of other info, but how can i find out if this is all the data my Nefit spits out?

question, how did you figure out the addresses, can't seem to find any similar entry's in the wiki.
for instance 0x08,0x18,0x12,0x02,

So if you read this page VERY carefully:
https://emswiki.thefischer.net/doku.php?id=wiki:ems:telegramme
You see that the offset for the first databyte is 5. So the first databyte is at the 5th byte of the message.
Now check the UBAMonitorFast table.
You can see f.i. 08 | 00 | 18 | 23 | 1 | ASCII | Service-Code 1. Zeichen
Sender: 0x08
Type: 0x18
Byte offset from start: 23 DEC = 0x17
Description: Service Code 1st sign.
Now note the sequence of your question: 0x08 | 0x18 | 0x12 | 0x02 | //#9   8 status code 1st letter.
Sender: 0x08
Type: 0x18
Byte offset from first databyte: 0x12 (remember it is +5 bytes from the start so 0x12+5=0x17)
Datatype: 0x02 = ASCII (Only used in the sketch)
Description: //#9   8 status code 1st letter
So if you check the other parameters in the sketch you see that most of them are just a decoding of the UBAMonitorFast and UBAMonitorWWMessage message.


Replace remaining uses of [project] in documentation with [package]

The [project] section is not currently referenced anywhere else in the documentation, so the use here confused me, and I spent a while trying to add it to a Cargo.toml that already had a [package].
[project] seems to be a deprecated alias for [package], which is documented, so let's swap it out.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @matklad (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

:hourglass: Testing commit fd1fd6cfe2bee8a1a66441ae47fb7d900d169344 with merge 15e6361e59a7ed3a8771f6a96a025cb338b6ce25...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 15e6361e59a7ed3a8771f6a96a025cb338b6ce25 to master...


CAMEL-10546: Continued the deprecation of CamelContext.getProperties()

properties and globalOptions coexist in schemas
globalOptions override existing properties
I first deprecated PropertyDefinition but rolled this back in the third commit when I realized it is duly used for endpoint level properties
I couldn't help with re-writing GlobalOptionsDefinition.asMap()
I didn't completed camel-spring-boot as it looks like the functionality behind "properties" would be implemented in CamelConfigurationProperties with spring boot. Not sure about that however, guidance is welcome if need be.

spring-boot dont have support for a map like properties, but have individual options of the few ones that may be used like
https://github.com/apache/camel/blob/master/components/camel-spring-boot/src/main/java/org/apache/camel/spring/boot/CamelAutoConfiguration.java#L116

Thanks for the PR it has been merged. Do you mind closing this?

Thanks for the merge


Bump Falco to module-version-2.11-rc2

Description
Bumping the Falco version on top of upstream 0.17.3 release.
Checklist

[ ] Investigated and inspected CI test results
[ ] Updated documentation accordingly

Automated testing

[ ] Added unit tests
[ ] Added integration tests
[ ] Added regression tests

If any of these don't apply, please comment below.
Testing Performed
CI should be enough.

@erthalion any clues why the benchmark comment might be empty? Looks like there might not be a baseline file?
Baseline file is empty, nothing to compare.

Closing in favor of #1824.


Update 2 broken links in documentation

Found this while trying to configure Path Stripping and getting a 404.
Tested all other links and found the compression one was also broken. All others are working fine.
Cheers ❤

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

If any maintainer is reading this, I will not sign the CLA (https://drewdevault.com/2018/10/05/Dont-sign-a-CLA.html). Feel free to use my changes anyway, make them yourself or copy it.


Use I3 for xiwi instead of ratpoison

Improves floating window behavior and compatibility with software that requires a reparenting window manager.
Window title monitoring is also more efficient.
Fixes #1483
Colors are also set to look nice with stock Chromium OS.

I haven't tested this version yet, will xiwi i3 still work with this?

What are you trying to do?  If you just want to launch an i3 environment, you're better off configuring your .xinitrc and using xinit like on a normal system.
The I3 config used by xiwi will not touch your .i3/config.

I'm running Popcorn-Time that kicks-off in a background process using the i3xiwi branch!

Merge first, ask forgiveness/apply @drinkcat's suggestions later :shipit:

@dnschneid : Had nothing to say but the copyright date...
Another thing that might be worth it is to allow the user to have a custom i3 config file (e.g. ~/i3xiwi.conf), that gets preference over the generic /etc/crouton/xiwi.conf...

I think you can put i3 commands inside of the .xiwirc file, so you can effectively already do that.


The terminal cannot be used due to error report

When starting, you can see the error message "enoent: no such file or directory, scanner'c:: (users, a1181, appdata, local, Microsoft, windowsapps')" in Ctrl + Shift + I under the console option. This problem also causes the terminal to be unable to initialize

Please provide a screenshot or a gif illustrating the problem (attaching images won't work via email, you need to go to github.com). See this wiki page for more general info on how to report actionable terminal issues.
/label "needs more info" integrated-terminal


openssl crash SIGSEGV SEGV_MAPERR at ossl_sa_doall_arg

openssl version: 3.0.7 with some cve patches
platform: aarch64
kernel version: Linux version 5.10.97
crash occurs rarely
Reason:Signal:SIGSEGV(SEGV_MAPERR)@0x0022217f78d8eb2e
 Thread name:i.hms.framework
#00 pc<PHONE_NUMBER>26ee08 sa_doall at third_party/openssl/crypto/sparse_array.c:86
     (inlined by) ossl_sa_doall_arg at third_party/openssl/crypto/sparse_array.c:148
#01 pc<PHONE_NUMBER>24aeac ossl_sa_ALGORITHM_doall_arg at third_party/openssl/crypto/property/property.c:96
     (inlined by) ossl_method_store_do_all at third_party/openssl/crypto/property/property.c:498
#02 pc<PHONE_NUMBER>200f34 evp_generic_do_all at third_party/openssl/crypto/evp/evp_fetch.c:663
#03 pc<PHONE_NUMBER>1dbf58 ossl_decoder_ctx_setup_for_pkey at third_party/openssl/crypto/encode_decode/decoder_pkey.c:346
#04 pc<PHONE_NUMBER>1dc688 OSSL_DECODER_CTX_new_for_pkey at third_party/openssl/crypto/encode_decode/decoder_pkey.c:450
#05 pc<PHONE_NUMBER>2a941c x509_pubkey_ex_d2i_ex at third_party/openssl/crypto/x509/x_pubkey.c:208
#06 pc<PHONE_NUMBER>153590 asn1_template_noexp_d2i at third_party/openssl/crypto/asn1/tasn_dec.c:682
#07 pc<PHONE_NUMBER>1522a0 asn1_item_embed_d2i at third_party/openssl/crypto/asn1/tasn_dec.c:422
#08 pc<PHONE_NUMBER>153590 asn1_template_noexp_d2i at third_party/openssl/crypto/asn1/tasn_dec.c:682
#09 pc<PHONE_NUMBER>1522a0 asn1_item_embed_d2i at third_party/openssl/crypto/asn1/tasn_dec.c:422
#10 pc<PHONE_NUMBER>151bbc asn1_item_ex_d2i_intern at third_party/openssl/crypto/asn1/tasn_dec.c:118
     (inlined by) ASN1_item_d2i_ex at third_party/openssl/crypto/asn1/tasn_dec.c:144
#11 pc<PHONE_NUMBER>2397a0 PEM_X509_INFO_read_bio_ex at third_party/openssl/crypto/pem/pem_info.c:168
#12 pc<PHONE_NUMBER>27f9c8 X509_load_cert_crl_file_ex at third_party/openssl/crypto/x509/by_file.c:231
#13 pc<PHONE_NUMBER>27fbd0 by_file_ctrl_ex at third_party/openssl/crypto/x509/by_file.c:72
#14 pc<PHONE_NUMBER>2995d8 X509_STORE_load_file_ex at third_party/openssl/crypto/x509/x509_d2.c:52
#15 pc<PHONE_NUMBER>072a00 ossl_connect_step1 at third_party/curl/curl-7.79.1/lib/vtls/openssl.c:3060
     (inlined by) ossl_connect_common at third_party/curl/curl-7.79.1/lib/vtls/openssl.c:4071
#16 pc<PHONE_NUMBER>075b1c Curl_ssl_connect_nonblocking at third_party/curl/curl-7.79.1/lib/vtls/vtls.c:373
#17 pc<PHONE_NUMBER>0365b0 https_connecting at third_party/curl/curl-7.79.1/lib/http.c:1603
     (inlined by) Curl_http_connect at third_party/curl/curl-7.79.1/lib/http.c:1529
#18 pc<PHONE_NUMBER>04b928 protocol_connect at third_party/curl/curl-7.79.1/lib/multi.c:1696
     (inlined by) multi_runsingle at third_party/curl/curl-7.79.1/lib/multi.c:1997
#19 pc<PHONE_NUMBER>04a9c4 curl_multi_perform at third_party/curl/curl-7.79.1/lib/multi.c:2568
#20 pc<PHONE_NUMBER>0202f4 OHOS::NetStack::Http::HttpExec::SendRequest() at foundation/communication/netstack/frameworks/js/napi/http/http_exec/src/http_exec.cpp:404
#21 pc<PHONE_NUMBER>020168 OHOS::NetStack::Http::HttpExec::RunThread() at foundation/communication/netstack/frameworks/js/napi/http/http_exec/src/http_exec.cpp:383
#22 pc<PHONE_NUMBER>0215f4 decltype(std::declval<void (*)()>()()) std::__h::__invoke[abi:v15004]<void (*)()>(void (*&&)()) at include/libcxx-ohos/include/c++/v1/__functional/invoke.h:394
     (inlined by) void std::__h::__thread_execute[abi:v15004]<std::__h::unique_ptr<std::__h::__thread_struct, std::__h::default_delete<std::__h::__thread_struct>>, void (*)()>(std::__h::tuple<std::__h::unique_ptr<std::__h::__thread_struct, std::__h::default_delete<std::__h::__thread_struct>>, void (*)()>&, std::__h::__tuple_indices<>) at include/libcxx-ohos/include/c++/v1/thread:284
     (inlined by) void* std::__h::__thread_proxy[abi:v15004]<std::__h::tuple<std::__h::unique_ptr<std::__h::__thread_struct, std::__h::default_delete<std::__h::__thread_struct>>, void (*)()>>(void*) at include/libcxx-ohos/include/c++/v1/thread:295
#23 pc<PHONE_NUMBER>0e868c start at third_party/musl/intermidiates/linux/musl_src_ported/src/thread/pthread_create.c:294
#24 pc<PHONE_NUMBER>087380 __clone at third_party/musl/intermidiates/linux/musl_src_ported/src/thread/aarch64/clone.s:28

I couldn't find similar crashes, is there any clues?

I'm honestly not sure how the code in question ever worked. The segfault happened in this code:
static void sa_doall(const OPENSSL_SA *sa, void (*node)(void **),
                     void (*leaf)(ossl_uintmax_t, void *, void *), void *arg)
{
    int i[SA_BLOCK_MAX_LEVELS];
    void *nodes[SA_BLOCK_MAX_LEVELS];
    ossl_uintmax_t idx = 0;
    int l = 0; /* set l to zero */

    i[0] = 0; /* set the first element of our stack declared int array to 0, fine*/
    nodes[0] = sa->nodes; /*set the zeroth element of our stack declared void* array to sa->nodes, fine */
    while (l >= 0) {
        const int n = i[l]; /* set the local int n to the l'th element of the i array, fine */
        void ** const p = nodes[l]; /* set the local void **p to the l'th element of the nodes array fine, this also doesn't look right */

        if (n >= SA_BLOCK_MAX) {
            if (p != NULL && node != NULL)
                (*node)(p);
            l--;
            idx >>= OPENSSL_SA_BLOCK_BITS;
        } else {
            i[l] = n + 1;
            if (p != NULL && p[n] != NULL) {  /* This is troubling */
                idx = (idx & ~SA_BLOCK_MASK) | n;
                if (l < sa->levels - 1) {
                    i[++l] = 0; /*prefix increment l here */
                    nodes[l] = p[n];
                    idx <<= OPENSSL_SA_BLOCK_BITS;
                } else if (leaf != NULL) {
                    (*leaf)(idx, p[n], arg);
                }
            }
        }
    }
}

It looks wrong that we are declaring a void *p and setting it to nodes[l] where nodes is an array of void, though that appears to just be some typecasting tomfoolery.
What I think might be happening here is, if p[n] ever equals null, we call the leaf function, without updating the l index.  this line however:
i[l] = n+1
still unilaterally updated i[i] to an incremented value.
On the next iteration of the loop n gets updated to be that incremented value, but l remains constant, so we're effectively indexing the next element in the sa->nodes array (or some variant thereof if a prior iteration set nodes[l] = p[n].  There is a potential there to index off the end of the sa->nodes array
I'm not going to pretend I fully understand this code, but  I think the right fix might be to change this line:
if (n  >= SA_BLOCK_MAX)
to instead check
if (n == sa->nelem)
I think that will properly constrain the loop to not index beyond the number of elements in the sa's node array.

I would be very surprised if there was a bug like this. I am not trying to dive-in right now though. Maybe @paulidale will have some comments later.


You will be unbanned, but should you be caught on our server with any hack mods again, you will be permanently banned.

thanks my guy appreceate it ill do your server right this time much love xoxo

thanks my guy appreceate it ill do your server right this time much love xoxo

i didnt know how to uninstal it my friend did it for me now it is unistalled but i am still banned...

i didnt know how to uninstal it my friend did it for me now it is unistalled but i am still banned...

i didnt know how to uninstal it my friend did it for me now it is unistalled but i am still banned...

i didnt know how to uninstal it my friend did it for me now it is unistalled but i am still banned...

Unbanned again. Don't let it happen anymore.

Unbanned again. Don't let it happen anymore.


Remove outdated workaround for data type issue with da.stack

🚀 Pull Request
Description

This removes a workaround for https://github.com/dask/dask/issues/2273, which was solved in 2017.

Consult Iris pull request check list

Add any of the below labels to trigger actions on this PR:

https://github.com/SciTools/iris/labels/benchmark_this

I think pre-commit is experiencing a bug


run master on cron

Codecov Report

Merging #1871 (1273318) into master (a229950) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1871      +/-   ##
==========================================
- Coverage   95.05%   95.02%   -0.04%
==========================================
  Files         353      353
  Lines        3314     3314
  Branches      407      407
==========================================
- Hits         3150     3149       -1
  Misses         81       81
- Partials       83       84       +1

Impacted Files
Coverage Δ

packages/client/lib/cluster/cluster-slots.ts
59.80% <0.00%> (-0.99%)
:arrow_down:

Continue to review full report at Codecov.

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

@leibale reminder. Feel free to kill off if it's now unwanted. It's from a conversation we had back when.


File download hash mismatch

Required Info

Operating System & Version
{Win (10)

Platform
PC

SDK Version
{ 2.0 }

Issue Description
When I clone the repository and run CMake (cmake .. -DBUILD_CSHARP_BINDINGS=ON -DBUILD_SHARED_LIBS=ON), I keep getting hash mismatch error(below).
I tried downloading/compiling multiple times with always the same result.
Earlier versions (e.g. since 2.22) compiled fine.
What could be the cause?
Thank you.
CMake Error at third-party/libtm/fw/CMakeLists.txt:22 (file):
file DOWNLOAD HASH mismatch
for file: [D:/SoftwareMix/RealSenseLib/librealsense/build/third-party/libtm/fw/target-<IP_ADDRESS>9.mvcmd]
  expected hash: [b7b722b785fcf5f8c31877173c8ba02fed120e03]
    actual hash: [da39a3ee5e6b4b0d3255bfef95601890afd80709]
         status: [22;"HTTP response code said error"]

Call Stack (most recent call first):
third-party/libtm/fw/CMakeLists.txt:47 (target_binary)

@AOndracek , iirc the hash da39a3ee5e6b4b0d3255bfef95601890afd80709 corresponds to zero-sized file, which can mean that the download has failed, file being locked or access permissions not valid.
Try to verify the internet connection (proxies, firewall), then remove all the intermediate files, including the one mentioned above, and make a clean cmake build.

Ok, the file size is 0, it doesn't seem to be locked or with restricted access permissions. I ran the cmake from cmd "as admin" as well, but to no avail. There is also no proxy. I'm not sure how to check the internet connection however - mainly the firewall, could you give me some advice on that please?

Direct link http://realsense-hw-public.s3-eu-west-1.amazonaws.com/Releases/TM2/FW/target/<IP_ADDRESS>9/target-<IP_ADDRESS>9.mvcmd

Thank you very much - while using the link, I discovered that there was a HTTP filter I didn't know about,  which was blocking the download.
So now it is solved.

I get the same problem but the file is a non-empty  png which i can load in preview fine

Hi @becker666  There is further advice about the file DOWNLOAD HASH mismatch error at https://github.com/IntelRealSense/librealsense/issues/4833#issuecomment-529836674


Add the support for image in the builder

I think it has been missing for quite some times now - will support the image inside the notification

Hi !
Feel free to re-open this pull-request to code-lts/Laravel-FCM
This is a fork that will be maintained as long as possible by @code-lts members :muscle:
:rocket: 1.4.0 was released See how to install it (composer require code-lts/laravel-fcm)

Released as v1.5.0
You can close this PR now ;)


Explain why do we need a tools package

https://github.com/open-telemetry/opentelemetry-service/tree/master/tools
Can this be removed?

@owais can you shine on this? I don't remember what for was this exactly.

Also if this is internal consider to move it under the internal directory.

It was added so we could pin tool dependencies to go.mod and have better guarantees about everyone using the same version of the tools. It doesn't have to be a separate package and can be moved to root.
More on it here: https://github.com/golang/go/wiki/Modules#how-can-i-track-tool-dependencies-for-a-module

Probably can be in the internal directory. Trying to avoid having too many packages/files in the root if not necessary.

Makes sense. Will move to internal.

Also please add a comment in the file why is it needed. That ensures others will not get confused like me.

Will add the comment. Do you think keeping it in internal (as opposed to root) might incorrectly suggest that the tools are only required by the internal package?

I don't, I think we have no intention to export public anything from this file so it should not be in the root.


storyshoots and graphql add-on do not work together

Issue details
When I have graphql addon in place and try to add/use storyshots addon - graphql test do not process graphql sample story
Steps to reproduce

Add graphql and storyshots addons,

setup graphql addon sample

setup storyshoots addon

run tests

expected: graphql story pass or ignored but error occured about including css file.
Even when I disable including css in jest using something like this
moduleNameMapper: {
    "\\.(css|jpg|png)$": "<rootDir>/etc/empty-module.js",
  },

It still do not pass with something like this
Error: Uncaught [TypeError: document.body.createTextRange is not a function]
          at reportException (/Users/petrsnobelt/Projects/kiwi-on-nitro/node_modules/jsdom/lib/jsdom/living/helpers/runtime-script-errors.js:66:24)
          at invokeEventListeners (/Users/petrsnobelt/Projects/kiwi-on-nitro/node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:209:9)

Please specify which version of Storybook and optionally any affected addons that you're running

@storybook/react 3.3.15
@storybook/addon-graphql 3.3.15
@storybook/addon-storyshots 3.3.15

I know I can disable some stories using regexp in storyshoots config, or have separate config for storyshoots, but it is not fine to maintain two configs or maintain regexp

As far as I know, the GraphQL addon hasn't been updated in a while. It definitely needs some love.
Do you think you can put in some time to fix this?


Make CodeGenSchemaMember functional

It would help to have this for:

defaulting property values
renaming properties

This should all be possible in azure-sdk-for-net.


Demo not working

Currently the demo under https://quasarframework.github.io/app-extension-qcalendar/#/ isn't working because of wrong js files
GET https://quasarframework.github.io/js/app.40b76b92.js net::ERR_ABORTED 404

Thanks. I am working on it. Almost there.

I changed from vue router using hash mode, to history mode so I can get anchor links to work. This caused issues with Github because of the url path is longer than the domain.

k, it's working now. That was a WOAD (Waste of a day)


Is it possible for multiple face tracking?

I notice that the function track_shot could track only one random face. What if I have multiple faces in a shot? Is it possible for multiple face tracking?

I believe the pipeline works for multiple faces in a shot. Have you tried running and visualizing the output?

The pipeline does works for multiple faces. And I notice that the pycrop file store all detected face tracks and its corresponding audios.


Multiple calls of AppRef.tick

Hi, it's me again! Thanks for your work.
You use runOutsideAngular for event handlers, it's great for app performance.
But i still can see multiple AppRef.tick calls when scroll events firing.
I think virtual-scroll it's about DOM changes, not angular's models changes and there should not be so many calls to update the entire application.
Just place console.log in tick() sources or subscribe to ngZone.onMicrotasksEmpty (i already did it in stackblitz example)
https://stackblitz.com/edit/angular-mpmwhq?file=src%2Fapp%2Fapp.component.html
[upd] https://github.com/rintoj/angular2-virtual-scroll/blob/master/src/virtual-scroll.ts#L676
i found the code
can we run the update by another way? all emits will work without this.zone.run.
maybe just recalculate this.viewPortItems and then changeDetectorRef.detectChanges() ?

That's a good idea. Would you mind submitting a pull request?

@speige Any thoughts on this? What do you suggest? This guy is on holiday. I can do this work.

Ok , he'll do it asap :) ty

I'll let you guys work it out. As soon as you have a PR ready, I'm happy to look at it.
Thanks.

The PR has been merged. Thanks for working on this! Please re-open if you continue to have issues.

@sharikovvladislav @nickbullock
I found an issue with this change in relation to the OnPush change detection strategy, I have an app which consumes ngx-virtual-scroller. In this app I use OnPush on all my components. I use markForCheck (not detectChanges) whenever I change anything on a component's databinding model.
I upgraded to the newest ngx-virtual-scroller which uses detectChanges instead of zone.run. Suddenly, my UI felt sluggish. After researching, I determined that markForCheck wasn't working & the DOM wasn't being updated when my data-bindings changed. I reverted to the earlier version of ngx-virtual-scroller & everything was fixed. Strangely, this affected components all over my app, not just children of ngx-virtual-scroller.
I'm not entirely sure what the root cause is, or how to fix it. But, it seems to be a bug in Angular where detectChanges breaks Components which use OnPush. I found a related bug where the Angular team dismissed it as "intentional", but it seems like a bug to me. I must not understand OnPush or detectChanges well enough.
https://github.com/angular/angular/issues/22560
Anyways, I'm going to add a flag to use this new detectChanges code, rather than forcing it on everyone. I'm going to default it to "false". I still think the @SkipSelf() parentCDR: ChangeDetectorRef is a good idea.
Let me know if you guys have any ideas what's happening & how it can be corrected. Thanks.

There are two things:

ChangeDetection strategy OnPush/Default and how this forces us to work with components.
How Angular starts change detection cycles. Do you remove all ticks and stuff or you rely on Angular mechanisms to start CD cycles and stuff.

markForCheck (as I know) does not start change detection. It marks component like "this component should be change detected in some future". And when something another starts change detection cycle this component will be change detected.
detectChanges starts change detection for this component and his children immediately.

I'm not certain, but it seems like detectChanges somehow breaks markForCheck so that it no longer works.

This issue may be connected that after you changed this code (stopped relyaing on zone.run) nothing in your application triggers change detection cycle (2 point at the top of my messate).
So before this flag, when ngx-virtuall-scroller called zone.run code, change detection in whole application was called by this zone.run code. After you removed it, you have to run change detection by yourself. Any component should not rely on 'some magic thing that will trigger change detection in the future' as I think. If component need to be updated you have to say it by yourself (detectChanges) or you have to understand whole pipe how change detection works and you have to understand what exact will happen to run change detection after markForCheck was called.
Let us try this new changes in our application. We don't rely on magic and we trigger change detections manually in different components. So we don't rely on global change detection cycles.
Our strategy in general:

Controls like select/date-picker/autocomplete/grid and stuff do not trigger change detection cycle for whole application. No at all. They trigger appRef.tick (to run global change detection cycle) when their outer model changes (from inside of the component).
Business components are less "hard". They rely on magic a bit (so requests/many events and stuff triggers change detection). Since we use hard OnPush strategy anyway this ticks should not kill our perfomance anyway.

I am not sure is it good or bad :) We are trying this strategy in really big enterprise application.

I don't think zone.run in virtual-scroller was causing markForCheck in my app to work. If I remove the zone.run, everything works fine. If I remove zone.run & add parent.detectChanges, everything breaks.
I'll try to reproduce this in a simple app via stackblitz.

I don't think zone.run in virtual-scroller was causing markForCheck in my app to work.

Here is the example: https://stackblitz.com/edit/angular-tsw4gp
By this I want to say, that calling zone.run function starts change detection cycle in the app. Play with that demo. Specifically, with flag shouldUpdateInZone. If your code runs in zone you will see:

Logs run cd
You will see application is updating
Log about set interval fired.

Without zone.run you will see only set intreval fired logs.

This is because all of your components are with on push strategy. After initialization and first check they get state like "don't check me, no needed". You have to change their state to "I need to be change-detected". You can do this by:

Changing the input of the component (if you have object, you must change ref to that object, but not just mutate the object). And according to OnPush strategy after changing of some input component will move to state "I need to be change-detected". markForCheck will be called automatically.
Calling markForCheck manually.

If I remove the zone.run, everything works fine

Then something else triggers change detection.

If I remove zone.run & add parent.detectChanges, everything breaks.

That is strange. Assuming that something else starting the change detection (like we assumed in previous paragraph).

I'm still investigating, but I'm debating reverting all of the code from your PR. I'm not an expert on any of this, so maybe I'm coming to the wrong conclusion, but it feels like it's the wrong approach.
My main concern is that I believe Angular documentation states that all event handlers will be run inside zones. (I need to remember where I read that). By calling vsUpdate.emit / vsStart.emit / vsEnd.emit / vsChange.emit outside an Angular zone, we're breaking expected Angular behavior.
Also, this stackblitz you created to demonstrate the performance improvement of removing zone.run https://stackblitz.com/edit/angular-virtual-scroller-problem-cuz-of-zone-run  actually proves that the performance issue has nothing to do with virtual-scroller.
Essentially hello.component.ts has some really nasty code that makes change detection really slow:
  get binding7() {
    for (var i = 0; i < 100000; i++) {
      this.temp += 1;
    }

    return this.binding1;
  }

So, you have a slow app. You're fixing the slow app by preventing change detection from running. But, the root cause isn't the fact that change detection is running, it's the fact that you have a really poorly coded component which is slow. If you want to disable change detection, you should only do it on the slow component (hello.component.ts), not everywhere. By making this code change in virtual-scroller, you're not fixing the root cause, but instead masking it. You're disabling change detection without the consuming app knowing or choosing to do so. You're breaking expected the behavior of Angular ChangeDetectionStrategy.Default (CheckAlways) for most users. You're essentially making some parts of a users app behave like OnPush even though they didn't specifically choose that strategy. Because the real problem hasn't been fixed, the problem will likely creep up again constantly. The correct fix would be to speed up hello.component's bindings and possibly convert hello.component to OnPush strategy.
It does seem strange to me that .emit or scrolling in virtual-scroller has to evoke change detection on the entire app. In most cases, this will only affect children of virtual-scroller, so why can't it be smarter about change detection? I think it's because there's a chance the programmer might change elements outside of virtual-scroller when these vsChange/etc events are emitted & Angular has no way of knowing that without re-running the whole change detection. Unfortunately, Angular was designed by default to allow users to expect data-binding to automatically work without explicitly invoking it. This makes it easy on the programmer, but makes slower apps.
I know you guys really want the flag in order to speed up your app. But, it really feels like the wrong approach to me. You should investigate which components are slow & why & fix them. However, I understand that some of those components may be hard to fix or may be 3rd party, so if the wrong fix is the easiest fix, maybe that makes it the right fix. I think I'll keep the code, with a flag, & add a strong warning saying it's not the correct fix & to use it at your own risk :)

Also, this stackblitz you created to demonstrate the performance improvement of removing zone.run https://stackblitz.com/edit/angular-virtual-scroller-problem-cuz-of-zone-run actually proves that the performance issue has nothing to do with virtual-scroller.
Essentially hello.component.ts has some really nasty code that makes change detection really slow

This code is very synthehic. I said that before. This example is just an example with NOT MANY AND SLOW components. The same will be with A LOT OF FAST components. We have that situation.  There are a lot of fast components :) And they are shown at one moment.
Imagine: 100 components with loop from 0 to 10 in them. It is absolutely same as 10 components with loop from 0 to 100.

You're essentially making some parts of a users app behave like OnPush even if they didn't specifically choose that strategy.

Hm. We don't do this. Because we understand this point and we don't break behavior of virtual-scroller by default.

You're breaking expected the behavior of Angular

Since, we have flag for this we do not break anything.
Our initial proposal was to add flag & disable zone.run call. Now it is a bit different. You call detectChanges on parent component changeDetectorRef. This does not work in our application. It is not working, since content which is updated is not at parent component. It is higher. We have special wrapper for ngx-virtual-scroller which simplifies API of virtual-scroller for cases we are using.

In most cases, this will only affect children of virtual-scroller

What do you mean? Using zone.run code will trigger change detection in zone application. Calling zone.run code will add microtask to NgZone. After task is completed NgZone triggers onMicrotaskEmpty emitter. After emit appRef.tick is called. Here is the code: https://github.com/angular/angular/blob/master/packages/core/src/application_ref.ts#L385

It does seem strange to me that .emit or scrolling in virtual-scroller has to evoke change detection on the entire app. In most cases, this will only affect children of virtual-scroller, so why can't it be smarter about change detection?

Initial code of virtual-scroller doing this. That emits trigger change detection in whole application. It is triggered by zone.run. You say, that it is strange for you, that virtuall-scroller triggers change detection for whole application, but the code of virtual-scroll (without flag) was doing this. And it is still doing this (since it is default behavior).

I think you misunderstood me. I agree that the original code to virtual-scroller was causing a refresh of the entire application. However, I disagree with Angular's design choice here. I think Angular should have made OnPush the only option. Without OnPush they've made it easier for programmers to code, but also made it easier for them to make slow apps.
I agree in "theory" that a lot of fast components could cause this same issue. However, I still think it's the wrong fix. 1st) If you have too many fast components that it's causing slowness you're probably misdiagnosing the problem. Either your components aren't as fast as you think they are. Or you have way too many components & are probably doing something wrong. 2nd) Angular's default ChangeDetection will NEVER be fast for a real-world enterprise application. If you really want speed, you MUST switch as many components as possible to OnPush.
By removing zone.run from virtual-scroller you're ignoring the real problem & fixing it the wrong way. This same performance issue will re-occur every few weeks as you make minor code changes until you fix the real problem.
Regardless of my personal opinion, I'm willing to accept your pull request. It's safe because it's behind a flag. I'll just add a warning in the readme that it's not the recommended approach & that the programmer needs to understand the side-effects of making that choice.

you MUST switch as many components as possible to OnPush.

All our components have OnPush strategy. Well, not all, but like 99%.
We do not need "a lot of fast components". We are writing grid at the moment. Grid component containt a lot of internal components like GridRow, GridColumn, GridHeader, GridCell and stuff. All of them have OnPush strategy. We use Storybook as something where we can test and show to our designers our UI kit. So we have demo for grid in storybook. In two words, storybook is some sandbox, where components are very isolated. You have clean Angular application with 1 component in it. In my case, we have only grid on one page.
Also, we use custom scrollbar in the grid. It is using OnPush strategy + listeners are added in runOutsideAngular. So it does not influence Angular change detection mechanism. Also, we are using ngx-virtual-scroller to improve perfomance. We have big amounts of data in our system. So it is essential. Also, we use live scroll, to fetch additional data.
Only one thing that causes change detetction for whole application and whole components is virtual-scroller. If we remove zone.run from the code, everything works insanely fast. This is real world example. Not really many components.

I've made the flag available to you in the latest version of npm. Let me know if it doesn't work.
I'm glad the flag is helpful for your scenario.
I think the flag with warning in the readme is a good compromise because it allows you (& other developers) to use it if they choose, but it explains the side-effects of using it & suggests doing other performance optimizations first.

@speige thank you! we are trying hard now to get true way for OnPush strategy in our application

@nickbullock @sharikovvladislav I'm curious if you still need executeRefreshOutsideAngularZone or if using OnPush was able to fix your performance problems? Originally you said you were already using OnPush in 99% of your code & still had performance issues, but later you said you were going to re-code to use OnPush correctly.
I'm asking because I'd like to deprecate executeRefreshOutsideAngularZone unless I have evidence that it's actually beneficial, as my opinion is still that OnPush should fix it.

Also, I wonder if there's a way in your code to modify the injector only for ngx-virtual-scroller to use  NoopNgZone. If so, that'd give the same benefit, but without needing the extra flag inside ngx-virtual-scroller. I bring this up because the flag has caused confusion for other developers & I've always seen this as a bad hack that should never be used, not as a feature I want to support inside ngx-virtual-scroller.


Issue with Api

i have a problem using multiple roles. I have a lot of roles in my mods folder. All work but one only work if there isn't any mod. Here's my code but how to make work all?
using BepInEx;
using BepInEx.IL2CPP;
using DillyzRoleApi_Rewritten;
using HarmonyLib;
using UnityEngine;
using UnityEngine.UI;
namespace HackerBuonoVSHackerCattivo
{                //Custom Url | Mod name | Version
[BepInPlugin("com.github.TechDevOfficial.HackerBuonoRole", "Hacker", "1.0")]
[BepInDependency(DillyzRoleApiMain.MOD_ID)]
public class HackerBuonoVSHackerCattivoMain : BasePlugin
{
    public Harmony harmony = new Harmony("com.github.TechDevOfficial.Hacker");
    public override void Load()
    {
        harmony.PatchAll();
        Log.LogInfo("The mod as loaded");

        //string.name | Role name | Discription of the mod | if i can see color | if color is public | Mod RGB color | Can see team color | What side the CustomRole is on | If able to vent | If it as the original kill button | If its show was the role name and not impster
        CustomRole phaserole = DillyzUtil.createRole("Hacker Buono", "Phase through wall and kill the crew.", false, false, new Color32(150, 125, 125, 255), false, CustomRoleSide.Crewmate, VentPrivilege.None, false, false);
        //how the role is intruduced
        phaserole.a_or_an = "an";

        CustomRole hackercattivo = DillyzUtil.createRole("HACKER CATTIVO", "ATTRAVERSA I MURI E UCCIDI I CREWMATE", true, false, new Color32(150, 125, 125, 255), false, CustomRoleSide.Impostor, VentPrivilege.Impostor, true, true);
        //how the role is intruduced
        hackercattivo.a_or_an = "an";

        CustomButton hackerabutton = DillyzUtil.addButton(System.Reflection.Assembly.GetExecutingAssembly(), "hackera", "HackerBuonoVSHackerCattivo.Buttons.hackerabuono.png", 10f, false, new string[] { "Hacker Buono" }, new string[] { }, delegate (KillButtonCustomData button, bool success)
        {
            // succes is if the ocoldown is over and they can use it yet
            if (!success)
                return;
            //DillyzUtil.RpcCommitAssassination(PlayerControl.LocalPlayer, PlayerControl.LocalPlayer);
            PlayerControl.LocalPlayer.gameObject.GetComponent<CircleCollider2D>().enabled = false;
        });

        //Mod RGB Color
        hackerabutton.textOutlineColor = new Color32(150, 125, 125, 255);

        hackerabutton.SetUseTimeButton(5f, delegate (KillButtonCustomData button, bool interrupted)
        {
            // interrupted is if the use time was stopped by something other than it being manually called as running out (such as meetings or death)
            PlayerControl.LocalPlayer.gameObject.GetComponent<CircleCollider2D>().enabled = true;
        });

        CustomButton hacker = DillyzUtil.addButton(System.Reflection.Assembly.GetExecutingAssembly(), "hackera", "HackerBuonoVSHackerCattivo.Buttons.hackera.png", 10f, false, new string[] { "HACKER CATTIVO" }, new string[] { }, delegate (KillButtonCustomData button, bool success)
        {
            // succes is if the ocoldown is over and they can use it yet
            if (!success)
                return;
            //DillyzUtil.RpcCommitAssassination(PlayerControl.LocalPlayer, PlayerControl.LocalPlayer);
            PlayerControl.LocalPlayer.gameObject.GetComponent<CircleCollider2D>().enabled = false;
        });
        //Mod RGB Color
        hacker.textOutlineColor = new Color32(150, 125, 125, 255);

        hacker.SetUseTimeButton(5f, delegate (KillButtonCustomData button, bool interrupted)
        {
            // interrupted is if the use time was stopped by something other than it being manually called as running out (such as meetings or death)
            PlayerControl.LocalPlayer.gameObject.GetComponent<CircleCollider2D>().enabled = true;
        });
    }
}

}

Hi, please consider using Github formatting when submitting code.
Example:
// prints to unity console (not for mods)
Debug.Log("debug");

Also, I don't understand the exact problem because the code is fine.
Please detail your Among Us installation's version name, which version of DillyzRoleApi you're using, where the problem lies (ie. loading or functionality), and verify your mod's new versions are in game.

and i use the latest version of your api

I would like to create a mod bundle, but I can't get the one in my bundle to work


Get-ALZGithubRelease.ps1 doesn't retrieve all releases from Github

Expected Behavior
The function should be able to donwload any release of ALZ-Bicep
Current Behavior
The function can only find 30 releases. Since this list returned by Github is not consistent, releases are not found randomly. In my case It's unable to donwload the latest release, v0.16.5
Possible Solution
Implement pagination
Steps to Reproduce
Get-ALZGithubRelease -githubRepoUrl 'https://github.com/Azure/ALZ-Bicep' -releases $('v0.16.5')
WARNING: Release v0.16.5 was not found on GitHub Repo: Azure/ALZ-Bicep
Context (Environment)
I was trying to download the latest release of ALZ-Bicep using New-ALZEnvironment

Operating System and version as reported by $PSVersionTable.OS: Microsoft Windows 10.0.19045
PowerShell versions as reported by $PSVersionTable.PSEdition: Core

Get-ALZGithubRelease.ps1 doesn't retrieve all releases from Github
Detailed Description
The function Get-ALZGithubRelease.ps1 will only retrieve 30 releases from Github, as per https://docs.github.com/en/free-pro-team@latest/rest/releases/releases?apiVersion=2022-11-28#list-releases:~:text=per_page integer,Default%3A 30
Since there are currently 44 releases for the ALZ-Bicep only a portion of them can be found by the function.

Hey @MarkFarkas,
Can you confirm the version of the ALZ PowerShell module you have installed?
get-module -name ALZ -ListAvailable

The latest is v0.2.20 and you can install that with Update-Module -Name 'ALZ'
There also looked to be an oddity on the GitHub side with v0.16.5 which I think I have just resolved so please try again and you should be all good 👍
Let us know
Thanks

Hello @jtracey93 , I am using version v0.2.20. I have confirmed that this in indeed an issue with the module and already made a PR: https://github.com/Azure/ALZ-PowerShell-Module/pull/86

Hi @markfarkas, the function was restructured yesterday to pull specific release versions instead of all of the releases available for a particular repo, so this should no longer be an issue. I will also go ahead and close your PR for now, but if you end up running into any other issues, please let us know and thank you for bringing this up!


If this version provide concurrent number options like v8?

If this version provide concurrent number options like v8?
I checked this doc, but can not found the option.
By the way, if here are force override options in azcopy v8? I just hope a version contains override flag and concurrent number settings.
Any suggestions?

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

ID: d1bdc29f-175d-09b9-771d-cd84dbffbb49
Version Independent ID: f5fe11b9-8224-aa19-602d-bcc3ed7c9452
Content: azcopy copy
Content Source: articles/storage/common/storage-ref-azcopy-copy.md
Service: storage
Sub-service: common
GitHub Login: @normesta
Microsoft Alias: normesta

@hh23485 Thanks for your comment. We will review your issue and will revert to you.

@hh23485
I'm curious about what you mean by "concurrent number options"
If you mean like, the # of files being transferred in parallel, those options live under our environment variables (azcopy env)

I'd recommend posting your question in https://social.msdn.microsoft.com/forums/azure/en-US/home?forum=windowsazuredata since it's not directly related to the documentation. We'll follow up there.

@hh23485  Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly reopen the issue and continue the discussion.
I'd recommend posting your question in https://social.msdn.microsoft.com/forums/azure/en-US/home?forum=windowsazuredata since it's not directly related to the documentation. We'll follow up there.


feature: Configure Dependabot Alerts

The type of this PR is: Enhancement
Description
Adds the default dependabot config and sets the platform team as the
maintainers.
PR Checklist (tick all before merging)

[x] If I've added a new file to the tree I've exported it from the common index.ts
[x] I've added comments with examples for any new interfaces or helpers and ensured that they're in the docs
[x] No platform-specific terminology has been used outside of click and tap (platform is inferred by the DB storing events)

:rocket: PR was released in v3.0.1 :rocket:


Traduction

I would like impress.js in Spanish. I can contribute with the traduction for the Spanish community. It can have a documentation in that language.

[x] I have read and complied with the Issues Contributing Guidelines.
[x] If am reporting a bug, I have a simplified example on Pastebin or JsFiddle that reproduces the issue.
[x] I have used the Search Tool to make sure someone haven't already reported my issue or feature request.

Hi @ArtNous
Currently there is no framework for translation support. I don't know if we want to introduce one either, but if we did, for sure it would be something very minimalistic. There's a design goal that impress.js remains self contained, so we don't use any external libraries like jquery, underscore... anything.
That said, impress.js itself doesn't really show that much text anyway. (There are a few UI buttons now, but that's about it.) Perhaps more interesting would be to translate DOCUMENTATION.md, README.md and so on. And of course the content - and maybe even code comments - of the demo presentations too.
If that's what you wanted to do, I recommend you simply fork the impress.js, and translate those texts in your own repo. We could then link to this Spanish translation from the wiki.

Closing for lack of response. If you have further questions, please reopen this issue.


can't take screenshot more than 1000 times at a go

I had to automate screenshot which does happen but i can't take more than 1000 screenshots using "for loop".
I have defined a function where i press few keyboard buttons then take screenshot, after  that i "for loop" it for 1000 shots.
don't get the image files but it does press the same button, any way to change this functionality ?
Any place in the source where i can change the limits of no. of screenshots ??
Thanks a lot, in advance !!

I have hacked into scrots source code which acts as dependency for pyautogui.
There is a C function used to create folder name which by default allocates 4 bytes space for naming the png files, it has to be modified and recompiled.
Good malloc allocation would be 15 bytes
Good Calloc allocation would be 15 bytes.
For any details, feel free to contact.


error GitHub Pages: docs/css/main.scss has an error on line 10

Hello
I want to use this template to build a page github, I acted step-by-step instructions at: https://mmistakes.github.io/skinny-bones-jekyll/getting-started/ , but it works only locally.
Github reports an error that I do not know how to jump?
THIS ERROR: Your site is having problems building: Your SCSS file docs/css/main.scss has an error on line 10: File to import not found or unreadable: variables. Load path: /hoosegow/.bundle/ruby/2.3.0/gems/jekyll-theme-primer-0.1.5/_sass. For more information, see https://help.github.com/articles/page-build-failed-invalid-sass-or-scss/.
Please help me

Do you have a link to a public repo I could take a look and see if something else stands out to me. Without an example all I can do is speculate.
 From that error seems to me you're missing some of the Sass partials that are in _sass and that's what the error is all about.
That or you have some bad code in one of the files.
When you build locally are you using the github-pages gem or vanilla jekyll? The former mimics GH Pages and will make troubleshooting easier. Using the vanilla Jekyll gem won't likely show you the same errors as the version could be different and GH locks various settings.

What was the solution? I am getting the same error


How can we force whisper to be always on punctuation enabled mode

I am currently struggling with punctuation being on and off
How can I force it to be completely on all the time?
I also asked the question here : https://github.com/openai/whisper/discussions/194
Also I am ok for slowness but I would like to get best results. What are the hyper parameters that I should provide
Thank you very much

Whispering does not currently have such a function.
If whisper supports it, whispering may also support it.
I have not tried it, but there seems to be a study like punctuation restoration.
https://github.com/xashru/punctuation-restoration


IOKit/IOKitLib.h file

IOKit/IOKitLib.h file not found

same here

If you are having this issue then you shouldn't be here. Wait.

why : )
i have that problem too


can i get an invitation  code ?

i want a code for the maps  site... i already have an account but i can't sign in with .. i don't know why.......
https://forum.antichat.ru/members/312328/   my profile on  forum.antichat.ru

This is not the right place to discuss such subject. Write PM at the forum.


implement altair-fastify-plugin

Implemented an altair fastify plugin, I published it as "altair-fastify-plugin" with the version "2.4.10" and invited "imolorhe" account as mantainer of the package.
I hope it's useful as it was also made for my own use in my projects.
https://www.npmjs.com/package/altair-fastify-plugin

Need to remove loose equality comparison and replace it with strict equality. Also resolving all the other comments.

Done

@imolorhe  I was going to modify the commit to include those changes for it to be cleaner on the git timeline, idk if you prefer that or just how it is right now

I made the changes so I can quickly verify if that fixes the CI. After the CI passes, feel free to rebase.


Convert to use Get/Set-Timezone Cmdlets if available - Fixes #24

I also cleaned up the code a lot so that the variable names weren't being mixed up (sometimes TimeZoneId was used and sometimes it wasn't - the code was difficult to follow and debug).
I also improved the unit tests and increased the coverage.
I also performed a clean up of the code and bought it all up to HQRM standard. So hopefully I've eliminated the technical debt on this resource.
Here is a list of the changes:

Add support for Nano Server and WMF5.1 via Get-Timezone/Set-Timezone cmdlets.
Minor changes to bring make resource ready for HQRM.
Renamed and reworked functions in TimezoneHelper.psm1 to prevent conflicts with new built-in WMF5.1 Timezone Cmdlets.
Fixed localization so that failback to en-US if culture specific language files not available.
Moved code to init C# type into Set-TimeZoneUsingNET functions
Renamed internal Timezone parameters to TimezoneId to more clearly represent value

This change is

Thanks @Indhukrishna - good suggestion - done! Moved code to a Test-Command function in TimezoneHelpers.psm1 to detect if a CmdLet exists.
Are you able to use reviewable (click the Reviewable button in the PR) for any future review comments? Thank you heaps for the review.

Reviewed 5 of 8 files at r1, 3 of 3 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Review status: all files reviewed at latest revision, 1 unresolved discussion.

DSCResources/TimezoneHelper.psm1, line 126 [r1] (raw file):
Previously, Indhukrishna (Indhu Sivaramakrishnan) wrote…
> This block exists to load a particular module (and the cmdlet). Similar block exists for Timezone cmdlets also. Can we refactor this into a utility method so all the places can call this method instead?

LGTM

Comments from Reviewable


Platform specific config

With Postgres, we have Platform specific config.
E.g: unlogged should be a valid option in dbt_project, but setting this is an invalid option.
Supporting "official" adapters such as Postgres, Snowflake, etc would probably be the approach?

@joshuataylor agreed - if you open a PR i'll happily approve it!


Update client3:circe, client3:core to 3.9.8 in scala-steward-dependencies

About this PR
📦 Updates

com.softwaremill.sttp.client3:circe
com.softwaremill.sttp.client3:core

from 3.9.7 to 3.9.8
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.softwaremill.sttp.client3" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.softwaremill.sttp.client3" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Not signed


Use 'libssl' and 'libcrypto' instead of 'openssl'.

Problem
Compile failure on Ubuntu 20.04 LTS. Apparently openssl.pc does not exist anymore.
Summary of Changes
Our docker images add libssl-dev and libcrypto-dev. Use these for package configuration.

Are you sure?
https://packages.ubuntu.com/search?searchon=contents&keywords=openssl.pc&mode=exactfilename&suite=focal&arch=any

Are you sure?
https://packages.ubuntu.com/search?searchon=contents&keywords=openssl.pc&mode=exactfilename&suite=focal&arch=any

Not really ... leedarson seems to have reported errors on Ubutun 20.04.
Then again, their last update was Thank you guys. we finally resolved the compile issue by downloading the .pc files manually. A lot of .pc file were missed.
I am not sure how one manually downloads the pc files.
However what I am sure is that they succedded apt-get on libssl and libcrypto and failed on openssl. So it seems to make sense to me to have libssl.pc and libcrypto.pc and not openssl.pc


Unable to pass List to Worker.postMessage in dartium v1.13

In dartium-lucid64-full-stable-<IP_ADDRESS> you can no longer pass a List to Worker.postMessage. I get this error:
Exception: DataCloneError: Failed to execute 'postMessage' on 'Worker': An object could not be cloned.

It works with dart2js.
In dartium if you pass the list to new JsObject.jsify it works, but in Firefox the worker then
seems to receive {_jsObject: [foo, bar], $builtinTypeInfo: [null]}

I have created a simple project to illustrate this issue:
https://github.com/rwl/dart_sdk_issue_25074

https://codereview.chromium.org/1490763005

Thanks. Should be fixed in bleeding edge as soon as that change gets built.


React.addons.classSet is deprecated.

Warning: React.addons.classSet will be deprecated in a future version. See http://fb.me/react-addons-classset

Hey Guys,
Any news about that, it got reverted and that's it??
Any chances that it will make it back in?
Thanks

not it got merged, how ever I didn't publish a new release, give me a moment

I just release v0.0.18 that address that issue


New arguments for the CommandAPI

Please describe your suggestion
Add new argument options to the CommandAPI. In particular:

ScoreboardTeamArgument - Lets you access Bukkit's Team class
ScoreboardObjectiveArgument - Lets you access Bukkit's Objective class
RecipeArgument - Lets you access Bukkit's Recipe class

Please supply examples of the desired inputs and outputs
Team Example Usage:
(sender, args) -> {
    Team team = (Team) args[0];
    team.addPlayer((Player) args[1]);
    //other team related code
}

Objective Example Usage:
(sender, args) -> {
    Objective obj = (Objective) args[0];
    obj.setDisplayName((String) args[1]);
    //other objective related code
}

Recipe Example Usage:
(sender, args) -> {
    Recipe r = (Recipe) args[0];
    if(r.getResult().isSimilar(/* the player's hand item? */)) {
        //code
    }
}

Looks really good!
But doesn't vanilla use dynamic string arguments as scoreboard/team arguments?
Like with /scoreboard remove <name> you can type anything you want, not just the scoreboard names.
I got scoreboard arguments to work just fine with this:
    private DynamicSuggestedStringArgument objectives = new DynamicSuggestedStringArgument(() ->
    {
        List<String> names = new ArrayList<>();
        Set<Objective> objectiveSet = Bukkit.getScoreboardManager().getMainScoreboard().getObjectives();
        for(Objective objective: objectiveSet){
            names.add(objective.getName());
        }
        return names.toArray(new String[0]);
    });

Would your scoreboard/team arguments work the same way? Or will it only accept scoreboard/team names?

Looks really good!
But doesn't vanilla use dynamic string arguments as scoreboard/team arguments?
Like with /scoreboard remove <name> you can type anything you want, not just the scoreboard names.
I got scoreboard arguments to work just fine with this:
    private DynamicSuggestedStringArgument objectives = new DynamicSuggestedStringArgument(() ->
    {
        List<String> names = new ArrayList<>();
        Set<Objective> objectiveSet = Bukkit.getScoreboardManager().getMainScoreboard().getObjectives();
        for(Objective objective: objectiveSet){
            names.add(objective.getName());
        }
        return names.toArray(new String[0]);
    });

Would your scoreboard/team arguments work the same way? Or will it only accept scoreboard/team names?

It has taken me 2 hours to finally inform you that you are indeed right and that there's basically no point to me creating objectives or teams because vanilla basically uses a dynamic suggested string. Due to this, I have chosen not to implement the things outlined in this issue. However, I may create some templates for Dynamic Suggested Arguments which are prepopulated with various things which may be useful.

For completeness, this has been implemented in versions 3.0 and 2.1.


Use exact weights in generate_qpd_samples when it's possible for all of them

This changes generate_qpd_samples to use exact weights when it's possible for all of them, i.e., it's either all or nothing, for now at least.  I am working on a subsequent PR that will allow a mixture: the weights above the 1 / num_samples threshold will be evaluated exactly, while everything else will be sampled.   However, that's a bit more complex, and this will suffice for some of the basic use cases (e.g., cutting CNOTs only).

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

10 of 10 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 88.243%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
2079

Relevant Lines:
2356

💛  - Coveralls


Notebook integration tests output

This is not a failure but unexpected output:
notebook working copy
Cannot read property 'extensionLocation' of undefined: TypeError: Cannot read property 'extensionLocation' of undefined
   at NotebookEditorWidget._setKernelsFromProviders (file:///Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/contrib/notebook/browser/notebookEditorWidget.js:635:66)
   at NotebookEditorWidget._setKernels (file:///Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/contrib/notebook/browser/notebookEditorWidget.js:585:29)

Also seeing some of these: Notebook content provider view options file name pattern is invalid undefined


[bug]: Can´t run invokeai after install because of missing module ´ldm´

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Windows
GPU
cuda
VRAM
12GB
What happened?
After running invoke.bat I get this message:
** Model Installation Successful **
You're all set! You may now launch InvokeAI using one of these two commands:
Web version:
python scripts/invoke.py --web  (connect to http://localhost:9090)
Command-line version:
python scripts/invoke.py
If you installed manually, remember to activate the 'invokeai'
environment before running invoke.py. If you installed using the
automated installation script, execute "invoke.sh" (Linux/Mac) or
"invoke.bat" (Windows) to start InvokeAI.
Have fun!

http://localhost:9090 is not working
Then I run this:
(base) C:\invokeai>python .venv\Scripts\invoke.py --web

And get this error:
Traceback (most recent call last):
File "C:\invokeai.venv\Scripts\invoke.py", line 3, in
import ldm.invoke.CLI
ModuleNotFoundError: No module named 'ldm'
Screenshots
No response
Additional context
I´m not a developer, I do have basic understanding of Python.
I suspect that I should install stable diffusion seperately? If so, how should I make sure that the references are ok between invokeAI and stable diffusion?
Contact Details
No response

I got this one I installed the conda env (from the yaml file) from within the environments directory.
As the docs state somewhere, you need to copy the env.yml file that you want to the root of the repo and run the conda env create command at root.
When I did this, this error went away.

I got this one I installed the conda env (from the yaml file) from within the environments directory.
As the docs state somewhere, you need to copy the env.yml file that you want to the root of the repo and run the conda env create command at root of this repo.
When I did this, this error went away.

Thanks! I just ran the installation another time and it solved the problem :)


Can't add a new channel on socket after closing one.

Hey, another one related to closing channels. Once you close a channel, you can't add a new one.
I changed the 'closes successfully' test to this:
test('closes successfully', () async {
  final socket1 = PhoenixSocket(addr);
  await socket1.connect();
  final channel1 = socket1.addChannel(topic: 'channel3');
  await channel1.join().future;

  final socket2 = PhoenixSocket(addr);
  await socket2.connect();
  final channel2 = socket2.addChannel(topic: 'channel3');
  await channel2.join().future;

  channel1.push('ping', {'from': 'socket1'});

  expect(
    channel2.messages,
    emits(
      predicate(
        (msg) => msg.payload['from'] == 'socket1',
        'was from socket1',
      ),
    ),
  );

  await channel1.leave().future;

  expect(channel1.state, equals(PhoenixChannelState.closed));
  expect(socket1.channels.length, equals(0));

  final channel3 = socket1.addChannel(topic: 'channel3');
  await channel3.join().future;

  channel3.push('ping', {'from': 'socket1'});

  expect(
    channel2.messages,
    emits(
      predicate(
        (msg) => msg.payload['from'] == 'socket1',
        'was from socket1',
      ),
    ),
  );
});

It proceeds fully until it gets to the channel3.join().future call which will time out eventually.
I'll continue to try digging into the code to see why this may be happening, but I haven't figured it out yet.

Current thought is that it might be related to the StreamRouter that is set up in _topicStreams in PhoenixSocket. In removeChannel() it's doing this:
  void removeChannel(PhoenixChannel channel) {
    _logger.finer(() => 'Removing channel ${channel.topic}');
    if (channels.remove(channel.reference) is PhoenixChannel) {
      _topicStreams.remove(channel.topic);
    }
  }

This is just removing from the map, but not canceling the routing of the topics on the StreamRouter. I don't actually know how a stream router works, so I'll be looking into that now to see if it could be that.

I hacked in some code to verify my theory and it does seem correct. Since the route on the stream can't be removed, it'll continue splitting those events away from the stream with no way to subscribe it to the same topic again. I don't know enough about streams in dart to know the best way to solve this. It seems to me that this should be handled within an internal routing function on the socket when new messages arrive, but I'm not sure.

Good catch! I'll see if I can fix this really quickly

Fix is part of the new 0.4.11 release! Thanks again for your investigation @hworld !

I also filed an PR upstream in google/quiver-dart#673

Works perfectly for me now. Thanks so much for tackling these so fast!


Prevent program from endlessly looping in error state

[x] I read the README file
[x]  I have checked that the code is compiling and working
[x]  I have added appropriate comments in-between the code for better understanding of the algorithm
[x]  I've updated the changes in TrackingProgress.md

Sequence name
Added new \ Update existing
Language
Added new \ fixed issues

Powers of 2
Update
Python
fix

On ValueError exception, the program will now recover after a proper input is entered. Previously, the program would crash and throw a TypeError if you typed "2.1" and then "2", for example.

Without returning the result of "user_input()" after the "ValueError" exception, the program has no way of recovering due to the fact that regardless of what the user may enter after inputting anything that cannot be parsed as an integer, that value is never actually returned.

Thanks for spotting this error.


Added typescript declaration file

I have added a declaration file, for typescript support.
I would like to add it to the package.


SPIKE: Reconsider tx5-connection "hub" concept

DEPENDS ON: https://github.com/holochain/sbd/issues/36

closing in favor of just doing this work inside https://github.com/holochain/sbd/issues/36


Cannot override mode styles on many components

The docs for overriding mode styles aren't accurate for several components, e.g.
.ios ion-item {
  ...
}

I believe certain components need be updated to include the following:
...createThemedClasses(this.mode, 'item'),

I'm not sure why this was removed as it looks intentional, but will look into this.

This has been added for the 4.3.0 release. The full list of components that have the mode class now can be seen in the table at the bottom of this PR: https://github.com/ionic-team/ionic/pull/17838
It can be used like the following:
ion-item.md {
  // MD styles
}

ion-item.ios {
  // iOS styles
}

Note that this is only a problem for components that are being used inside of a shadow element. If using light DOM you can still use the documented way to style modes.


Disable categories

I think I want to disable categories, as I want to only have one news feed for my website (on the start page). Is it possible to get rid of them so that the menus and the complete logic is constructed without categories? Or is this not a use case of djangocms-blog?
I'm thinking of a tree like this:
├─ blog posts list (url: `/`)
│   ├── blog post a (url: `/my-first-blogpost`)
│   ├── blog post b (url: `/my-second-blogpost`)
│   └── ... (url: `/...`)
├─ Site: about us or something like that (url: `/about`)
└─ Site: contact (url: `/contact`)

Sorry for asking for support, but I could find a documentation and if this is not possible, I wish this for a feature ;)
Thanks...

You can use "Posts only" as menu structure in the apphook config. You still have to assign a category to each post, though

@sedrubal how about making categories not required again (added to the comment regarding menu structure)

Yes, I'd be glad if you make categories not required :+1:
But sorry, what do you mean with apphook config? Or where can I select "Posts only" ?
I hope I don't steal your time...

In /admin/djangocms_blog/blogconfig/ you should see a blog configuration model. In the Layout section you see Menu structure field  with the "Posts only" option


Incorrect KMS CryptoKey version; current state is: DESTROYED

Question
What is your question?
Hi Seth, we recently ran into a problem with this deployment where our vault is trying to unseal with the wrong version of our GCP KMS keyring. This seems like something we're doing wrong, but we're having a difficult time tracking it down.
The current/latest version of our Key is 6, but vault is trying to use version 1:
failed to unseal core: error="fetching stored unseal keys failed: failed to decrypt encrypted stored keys: failed to decrypt envelope: rpc error: code = FailedPrecondition desc = projects/{project_id}/locations/global/keyRings/vault-server/cryptoKeys/vault-init/cryptoKeyVersions/1 is not enabled, current state is: DESTROYED."
Any ideas on what we might be doing wrong there?
Considerations
This was working as of a few days ago, and then it stopped recognizing the existing kms keyring in our terraform, so we imported it and it then started using the wrong version.

So it appears as if we have it working now, but now are trying to understand why. We trashed the contents of the backend bucket (not the bucket itself), which were immediately replaced, and vault was able to successfully unseal itself. Guessing there was a mismatch between the key used to store the objects in the bucket vs the key vault was trying to use to unseal itself?


chore: updated deps

:tada: This PR is included in version 1.3.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Cleanup and update gems

This is something I've wanted to do for a long time. With this, old stuff gets cleaned out, which should let this continue to operate with even less maintenance, as the database wont fill up.

cc @pedro @brandur @heroku/api for review

looks good to me! not familiar with DELETE USING but seems straightforward/covered by tests!

Yeah, I spent some time trying to figure out how to do it with the things sequel gives you, but couldn't get it. :/
also @petergeoghegan thinks sql is good, but clearly DELETE USING proves that is is not good

also @petergeoghegan thinks sql is good, but clearly DELETE USING proves that is is not good

Whyyyyyy SQL.

Nice!


Enable Versioning by tag

Adds versioning support according to tag. Also move build options to github actions side rather than pyproject.toml.

I just used the same version tag that was already in testpypi to avoid publishing the same thing again. It was intended to fail, meaning it was able to get <IP_ADDRESS> from v<IP_ADDRESS>.test tag.

I noticed one bug though, I will send one more commit to fix it. Currently it doesn't use the tag version number for sdist. I will add that as well.

LGTM. See the annotation warnings here - https://github.com/GenomicsDB/GenomicsDB-Python/actions/runs/6937502067. We should probably upgrade to Trusted Publishing as suggested. This could be done as part of another PR. See #39. Thanks.

Sure, I'll take a look at it.


Update hip-clang flags for hip kernels

Update hip-clang flags for  hip kernels
Add some env variable for dumping out ISA

Forrmatting.

@pfultz2 @atamazov
Can I remove these lines for  "-mcpu=XXX" and " -mcode-object-v3"? Or I should keep it for some compiler and runtime?
https://github.com/ROCmSoftwarePlatform/MIOpen/blob/2ba5a243f2190387f0343caf09bc4fb658658481/src/hip/handlehip.cpp#L355
https://github.com/ROCmSoftwarePlatform/MIOpen/blob/2ba5a243f2190387f0343caf09bc4fb658658481/src/hip/hip_build_utils.cpp#L70

@pfultz2 Need some clarification in order to properly implement this PR: Which compiler are these lines targeted? hip-clang?
https://github.com/ROCmSoftwarePlatform/MIOpen/blob/2ba5a243f2190387f0343caf09bc4fb658658481/src/hip/hip_build_utils.cpp#L101-L108

@yxsamliu @aaronenyeshi Can you take a look at this PR?

@asroy

update: Not sure if it's safe to remove these flags. So I didn't touch -mcpu, but moved -mcode-object-v3 under hcc only

This is fine.

@yxsamliu @aaronenyeshi Using ```-save-temps```` for hip-clang seems to change the behavior of hip-clang
Running this convolution:
bin/MIOpenDriver conv -F 1 -n 128 -c 128 -H 17 -W 17 -k 128 -y 1 -x 7 -p 0 -q 3 -u 1 -v 1 -l 1 -j 1 -t 1 -i 10

Using HCC from rocm3.1
stats: name, n, c, ho, wo, x, y, k, flopCnt, bytesRead, bytesWritten, GFLOPs, GB/s, timeMs
stats: fwd-conv1x7u1, 128, 128, 17, 17, 1, 7, 128,<PHONE_NUMBER>, 19398656, 18939904, 10680, 48, 0.794485
Using hip-clang-2124 without -save-temps
stats: name, n, c, ho, wo, x, y, k, flopCnt, bytesRead, bytesWritten, GFLOPs, GB/s, timeMs
stats: fwd-conv1x7u1, 128, 128, 17, 17, 1, 7, 128,<PHONE_NUMBER>, 19398656, 18939904, 9663, 44, 0.878093
Using hip-clang-2124 with -save-temps
stats: name, n, c, ho, wo, x, y, k, flopCnt, bytesRead, bytesWritten, GFLOPs, GB/s, timeMs
stats: fwd-conv1x7u1, 128, 128, 17, 17, 1, 7, 128,<PHONE_NUMBER>, 19398656, 18939904, 7016, 32, 1.209469
Even after ignoring the performance regression from HCC to hip-clang. The difference perf between -save-temps doesn't make sense to me. Any thought on what is wrong?


nixos/gitea-actions-runner: init

Description of changes
Module to set up gitea-action-runner instances.
https://blog.gitea.io/2023/03/hacking-on-gitea-actions/
Tested against Forgejo 1.19.1 using the token option. The runner by default wants to connect to a docker instance, but I was able to substitute that with the compatible podman socket.
Untested, but assumed maybe working:

tokenFile option
Multiple runners
Docker

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[x] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Can VM test be added?

If you know how to programmatically retrieve a runner registration from a gitea instance.

If you know how to programmatically retrieve a runner registration token from a gitea instance. Not keen on mocking the gitea API.

https://github.com/go-gitea/gitea/pull/23762

Apparently using docker/podman is optional.

If you know how to programmatically retrieve a runner registration token from a gitea instance. Not keen on mocking the gitea API.

go-gitea/gitea#23762
I guess we could start by running a test that checks runner registration. A test that actually tries to build something might be more difficult as that requires networking and other weird stuff...

Only in 1.20, didn't get backported into 1.19.

Have a draft for the test in the linked PR.

Looks good at first glance :)
I thought about suggesting lib.optional instead of lib.optionals, but honestly, lib.optionals looks just way better :^)
Unfortunately, I don't think I'll be able to dedicate any time to do proper in-depth tests over the next few days, as I am busy with a lot of other stuff.
But I also don't want to block this PR, so eeehhh go ahead I guess?

So when someone has docker/podman enabled in their system config this module will act differently than if they have them disabled, what if someone has docker/podman enabled but doesn't actually want to use it with gitea runner and instead use native runner? Right now what will happen is that gitea runner will be configured slightly differently and the systemd service will have to wait for docker/podman.

Updating the labels against the gitea/forgejo instance requires getting a new registration token and dropping the runner config.
Guess we need to account for that.

Not sure how we should compare the labels and re-register based on that, as it involves state.
The .runner contains the labels from registration-time, but I feel like there is a high probability the format of it will change in the future:
$ jq .labels /var/lib/gitea-runner/test/.runner
[
  "ubuntu-latest:docker://node:16-bullseye",
  "ubuntu-22.04:docker://node:16-bullseye",
  "ubuntu-20.04:docker://node:16-bullseye",
  "ubuntu-18.04:docker://node:16-buster"
]

Additionally, there are custom labels, that can be configured in the runner registration page.
They are a bit weird, IMHO.
https://blog.gitea.io/2023/03/hacking-on-gitea-actions/#what-is-the-difference-between-agent-labels-and-custom-labels-for-a-runner
Just mentioning for reference and not because we should do anything to handle them.

I think I covered all remaining issues.


Scroll bar in canvas

Is it possible to have a scroll bar in the canvas? If the graph goes out of the canvas, I would like to have a scroll bar rather than pull the part back inside the canvas

Does any one have any leads on this? Would be of great help


Forbid relative directories for resource mounts

The dir parameter of resource mounts allow relative directories which allow to access data in the host system if the manifest is broken. Ok - the manifest is trusted by a check that dir is absolute should be added to the manifest verification.

Fixed by #767


Conversion to HF checkpoint should generate a checkpoint format that can be loaded directly

The conversion we have with litgpt convert to a huggingface checkpoint creates a model.pth file. But then you have to load it like so as described in the tutorial:
import torch
from transformers import AutoModel

state_dict = torch.load("output_dir/model.pth")
model = AutoModel.from_pretrained(
    "output_dir/", local_files_only=True, state_dict=state_dict
)

But we should make it work like this:
model = AutoModel.from_pretrained("output_dir")

The only blocker for this is that from_pretrained requires the pytorch_model.bin to be loaded with weights_only=True. Our checkpoints don't satisfy this constraint, because we save checkpoints using the incremental pickle save. See #1357 for more context where we had to work around this.

Yes I agree, there were a few people on Discord recently struggling with this


New package: MTH229 v0.2.4

Registering package: MTH229
Repository: https://github.com/mth229/MTH229.jl
Created by: @jverzani
Version: v0.2.4
Commit: a669847c1018cec70845ddf08b0f5857191a9294
Git reference: HEAD

This package has been an unregistered package for students for a number of years. To ease installation under Pluto, we decided to register it. Using the same name as the course name is important to match the course materials that have been developed. Thanks for the review.


implicit val that resolves to itself

implicit val stringToInt: Function1[String, Int] = new Function1[String, Int] { def apply(s: String): Int = s.toInt }

Why is this not considered a bug ?
Adding a reference to a user question who reported the same issue.
http://stackoverflow.com/questions/37022552/scala-implicit-completing-itself/37022955?noredirect=1#comment61601791_37022955

Abide is superseded by https://github.com/scalacenter/scalafix; closing all issues


Integrate with create_view from sqlalchemy-utils?

Not sure which side should do the integration, but I think it would be really nice if this could add autogeneration for views created with sqlalchemy-utils' create_view.
(if this is something that sounds like a good idea, I'm willing to try to work on it, but I'll probably need some guidance)

I'd be against a direct integration with sqlalchemy_utils from alembic_utils. Its has a whole bunch of fantastic stuff but my experience has been that the responsiveness to community issues is very limited. For example, some documented functionality has been broken for years e.g. https://github.com/kvesteri/sqlalchemy-utils/issues/319
What functionality from create_view are you interested in?
The two capabilities that I see are

runtime creation of views
ability to query a view from the same object that created it

I could see an argument for adding either/both but want to make sure I understand what you're looking for

Closing for inactivity


feat(22770): Implement interface should not add optional properties

Fixes #22770

Can you get @DanielRosenwasser to accept this bug before we review it? I personally don't think it's a good idea.

This PR hasn't seen any activity for quite a while, so I'm going to close it to keep the number of open PRs manageable. We can re-open it once we decide what to do with the original bug.