Unity 2018.2: XCode error when building for iOS (IL2CPP) with .NET 4.x using obfuscated C# DLL

Hello,
I have been using Obfuscar with success for managed DLLs obfuscation that I'm using from Unity 2018.2 (using .NET 2.0). Recently, I have been forced to update my project Scripting Runtime version and Api Compatibility Level to .NET 4.x due some external dependencies requirements. Since then I haven't been able to build my Unity project for iOS due a compiling time semantic issue with the IL2CPP generated code. Every time it throws the same error on the  .cctor() for the managed DLL.
// System.Void <PrivateImplementationDetails>{DD48BE6E-5DB1-4824-AEC7-412718C95E39}.81E3B0DD-52D3-43EB-8D19-FBBCC40F54CB::.cctor()
extern "C" IL2CPP_METHOD_ATTR void U381E3B0DDU2D52D3U2D43EBU2D8D19U2DFBBCC40F54CB__cctor_m3364342854 (RuntimeObject * __this /* static, unused */, const RuntimeMethod* method)
{
	static bool s_Il2CppMethodInitialized;
	if (!s_Il2CppMethodInitialized)
	{
		il2cpp_codegen_initialize_method (U381E3B0DDU2D52D3U2D43EBU2D8D19U2DFBBCC40F54CB__cctor_m3364342854_MetadataUsageId);
		s_Il2CppMethodInitialized = true;
	}
	int32_t V_0 = 0;
	{
		StringU5BU5D_t1281789340* L_0 = (StringU5BU5D_t1281789340*)SZArrayNew(StringU5BU5D_t1281789340_il2cpp_TypeInfo_var, (uint32_t)0);
		((U381E3B0DDU2D52D3U2D43EBU2D8D19U2DFBBCC40F54CB_t4239457874_StaticFields*)il2cpp_codegen_static_fields_for(U381E3B0DDU2D52D3U2D43EBU2D8D19U2DFBBCC40F54CB_t4239457874_il2cpp_TypeInfo_var))->set__2(L_0);
		ByteU5BU5D_t4116647657* L_1 = (ByteU5BU5D_t4116647657*)SZArrayNew(ByteU5BU5D_t4116647657_il2cpp_TypeInfo_var, (uint32_t)0);
		ByteU5BU5D_t4116647657* L_2 = L_1;
		RuntimeFieldHandle_t1871169219  L_3 = { reinterpret_cast<intptr_t> (U381E3B0DDU2D52D3U2D43EBU2D8D19U2DFBBCC40F54CB_t4239457874_____0_FieldInfo_var) };
		RuntimeHelpers_InitializeArray_m4077124945(NULL /*static, unused*/, (RuntimeArray *)(RuntimeArray *)L_2, L_3, /*hidden argument*/NULL);
		((U381E3B0DDU2D52D3U2D43EBU2D8D19U2DFBBCC40F54CB_t4239457874_StaticFields*)il2cpp_codegen_static_fields_for(U381E3B0DDU2D52D3U2D43EBU2D8D19U2DFBBCC40F54CB_t4239457874_il2cpp_TypeInfo_var))->set__1(L_2);
		V_0 = 0;
		goto IL_0040;
	}

IL_0025:
	{
		ByteU5BU5D_t4116647657* L_4 = ((U381E3B0DDU2D52D3U2D43EBU2D8D19U2DFBBCC40F54CB_t4239457874_StaticFields*)il2cpp_codegen_static_fields_for(U381E3B0DDU2D52D3U2D43EBU2D8D19U2DFBBCC40F54CB_t4239457874_il2cpp_TypeInfo_var))->get__1();
		int32_t L_5 = V_0;
		ByteU5BU5D_t4116647657* L_6 = ((U381E3B0DDU2D52D3U2D43EBU2D8D19U2DFBBCC40F54CB_t4239457874_StaticFields*)il2cpp_codegen_static_fields_for(U381E3B0DDU2D52D3U2D43EBU2D8D19U2DFBBCC40F54CB_t4239457874_il2cpp_TypeInfo_var))->get__1();
		int32_t L_7 = V_0;
		NullCheck(L_6);
		int32_t L_8 = L_7;
		uint8_t L_9 = (L_6)->GetAt(static_cast<il2cpp_array_size_t>(L_8));
		int32_t L_10 = V_0;
		NullCheck(L_4);
		(L_4)->SetAt(static_cast<il2cpp_array_size_t>(L_5), (uint8_t)(((int32_t)((uint8_t)((int32_t)((int32_t)((int32_t)((int32_t)L_9^(int32_t)L_10))^(int32_t)((int32_t)170)))))));
		int32_t L_11 = V_0;
		V_0 = ((int32_t)il2cpp_codegen_add((int32_t)L_11, (int32_t)1));
	}

IL_0040:
	{
		int32_t L_12 = V_0;
		ByteU5BU5D_t4116647657* L_13 = ((U381E3B0DDU2D52D3U2D43EBU2D8D19U2DFBBCC40F54CB_t4239457874_StaticFields*)il2cpp_codegen_static_fields_for(U381E3B0DDU2D52D3U2D43EBU2D8D19U2DFBBCC40F54CB_t4239457874_il2cpp_TypeInfo_var))->get__1();
		NullCheck(L_13);
		if (((((int32_t)L_12) < ((int32_t)(((int32_t)((int32_t)(((RuntimeArray *)L_13)->max_length))))))? 1 : 0))
		{
			goto IL_0025;
		}
	}
	{
		return;
	}
}

The RuntimeFieldHandle_t1871169219 is not defined (the RuntimeHelpers_InitializeArray_m4077124945 doesn't seem to be defined either...). I double checked the obfuscar config (although it is not modified with respect to when it was functional for .NET 2.0) and I have been playing with the project settings (enabling "unsafe" code, disabling code stripping, etc...) without success.
If I set my "RootNamespace*" for being skipped from obfuscation, the "PrivateImplementationDetails" doesn't get generated on the assembly, and everything works just fine, but if I enable obfuscation for anything, this class is built in the DLL, and the same error is thrown from XCode when building a iOS release: 20897:3: Unknown type name 'RuntimeFieldHandle_t1871169219'.
Building an Android release works just fine (as well as does the desktop version).
Any clue that can shed light on this matter, please?
Thanks in advance,

I don't have a Unity environment to test, so it is not trivial to me what might be the cause.
Can you disable string hiding and try again?

Thanks for the quick response.
You nailed it, disabling string hiding fixes the problem.

String hiding weaves some extra code into the assemblies, which sounds like IL2CPP cannot handle properly.
I am not sure if string hiding is in fact needed for Unity apps, as IL2CPP should obfuscate things automatically.

Yeah, but I'm using the same DLL managed assembly for several architectures, and not all of them use IL2CPP. I'm interested in obfuscate things for this assembly, but not for all the source code.
Until now I was able to keep using only one managed DLL even using IL2CPP. With the .NET 4.x seems this is not possible anymore (because of the string hidding as you said), but anyways, I could use different assemblies if I want to keep strings hidden on other scripting backends different from IL2CPP.
Thank you,

Apologies if this is an obvious question, is the "string hiding" you mentioned disabling in Unity or in Xcode? After a fair amount of searching I haven't been able to find it. I'm having an issue building to iOS that this may help. Appreciate any advice!

Apologies if this is an obvious question, is the "string hiding" you mentioned disabling in Unity or in Xcode? After a fair amount of searching I haven't been able to find it. I'm having an issue building to iOS that this may help. Appreciate any advice!

@rozatlab  Not Unity or Xcode. The "string hiding" I have switched off is an Obfuscar setting on the xml configuration file.
<Var name="HideStrings" value="false" />


Run application in docker as non-root user

For security reasons it's always a good idea to run the application inside docker as non-root user.
Its possible to do this in docker by

creating a user & group
creating a new directory
giving group access to the directory and make it executable
using the newly created user to run the application

Something like this:
RUN addgroup --gid 6555 ids-nonroot
RUN adduser --gid 6555 --uid 6555 ids-nonroot
RUN mkdir /usr/src/app
RUN chgrp -R ids-nonroot /usr/src/app
RUN chmod -R 770 /usr/src/app

USER ids-nonroot

WORKDIR /usr/src/app

.. copy & run app ..

 Dominik Pinsel<EMAIL_ADDRESS>Daimler TSS GmbH, legal info/Impressum

Fixed in b3188f8


Mac: Feature: Toolbar Icon

Hi, I'd like to propose something that was probably overlooked since macOS isn't as popular. The app needs to have a toolbar icon and/or run as a service so I can quit the app and have it off my dock.
Also, how do I uninstall it? It made system-level adjustments and now if it's not running my Internet connection is completely gone.

Please see https://github.com/anfragment/zen/issues/15#issuecomment-1840986929.

OK, but how do I uninstall it? Currently, if I close it, my connection is gone.

You could try starting and stopping the proxy inside the application. If it doesn't help, please refer to this instruction to navigate to the proxy settings and turn these two switches off.


Test time increase linearly then crashed for RecursionError

I test several videos, but the predict time increase linearly, may increase from 200ms to 3s (even more). Then the test script crashed for RecursionError: maximum recursion depth exceeded. I this the reason is out of memory in net. Any suggestions?

What is the test script you used? And what is the error log?

What is the test script you used? And what is the error log?

I wrote a test script based on test_models.py to test multiply videos(download from url, decode with ffmpeg). I found it is my fault (repeated initialization) when test videos in a for loop which cause out of memory (RecursionError: maximum recursion depth exceeded). I have figured out the issue. Anyway, thanks for your replay.


StorageExplorer takes forever to load on Windows 7

Storage Explorer Version:  1.0.0, 1.1.0, 1.3.1
Platform: Windows 7
Architecture: 32 Bit
Build Number:
Commit:
Regression From:
Steps to Reproduce:

Start Storage Explorer
Wait for about 15-30 minutes

Expected Experience:
Storage Explorer opens in no time.
Actual Experience:
Storage Explorer opens very very slowly or non at all. During this loading phase there are way to many processes allocating RAM and CPU.

Wow, that is indeed a lot of processes and quite a long startup time. 😲 Could you share a bit more about your machine/vm specs? I want to try and recreate your situation as close as possible when we investigate this. Thanks!

It's a German Windows 7 x64 (Domain joined) with all updates (automatic updates) installed.
Is there anything more specific I can look for? Are there any log files or system specs that can lead to this issue?

CPU and hard drive speed would probably be the biggest factors when it comes to startup time. I don't have any ideas at the moment though for what could be causing the massive number of processes though.

Element
Value

Prozessor
Intel(R) Core(TM) i5-3427U CPU

Drive
SSD SATA 6gb/s; Read 500MB/s; write: 175 MB/s

Just curious, do you have .NET 4.7.x installed on your machine?

Element
Value

.net
.NET Framework 4.7.1 (Release: 461310)

Considering several other issues have been created with this same problem, it's safe to say this isn't a Windows 10 issue. We've seen other cases where many processes are launched, running the CPU up unnecessarily high.
We use separate processes to, among other things, run several resource-heavy operations, such as blob downloads and uploads. This keeps Storage Explorer from locking up, especially if a problem occurs during those operations. The number of these child processes should be limited to 5 or so. If for some reason a child process doesn't respond within a certain period, I think we try again by spawning a new process. This may explain where the flood of processes comes from.
Another factor that may hurt performance is if your system has hooks that make inter-process operations take longer. For example, antivirus software may monitor process creation and communication, slowing things down.
See #281 for more info.
❓ If you watch Task Manager at startup, how fast does the list of processes grow?
We haven's seen this issue pop up in any of our dev/test environments, making this difficult to pin down. We have some ideas for the future that may help, though, including better logging and auto-termination of processes that aren't able to communicate with Storage Explorer.

As soon as I start Storage Explorer (Windows 10 Latest Updates) I get 55% increase in CPU and it runs continuously loading three SQL Server processes at 15 - 20% each; this is without actually using the Storage Explorer or reading from the Emulator - it's just background loading.

@ChrisJWoodcock You sure it's Storage Explorer launching SQL Server processes? Storage Explorer doesn't automatically launch any storage emulators when you connect to them. What version of Storage Explorer are you running?

@ChrisJWoodcock You sure it's Storage Explorer launching SQL Server processes? Storage Explorer doesn't automatically launch any storage emulators when you connect to them. What version of Storage Explorer are you running?

You are correct I've updated my original comment to highlight the mistake.

Closing due to inactivty (in regards to original error). Please let us know if problems persist.


Handle PNG images that are not paletted yet have a PLTE header chunk.

Thanks for the great fix. :)


New Shows not recognized

As mentioned with #40, new shows are not being detected correctly prior to adding to the interaction model.  Ran across this issue again with a new show that does not have a number in the title.
The show is titled "Letterkenny", which when requesting to play via the skill is continuously playing different shows like "Leverage" or "Luther".  When attempting to wikipedia Letterkenny via Alexa there is no issue what so ever with the recognition of the word.
Upon addition of "Letterkenny" to the interaction model of the skill there is no longer any issue recognizing the title of the show.

I just tried it with a movie I hadn't added to the slots and it works fine here.
I'm guessing it just wasn't able to match "Letterkenny".  What did Alexa hear?

Alexa is not showing what it actually heard, it is showing the skill response of "Playing next unwatched episode of [Incorrect show]".  Again tested this with Letterkenny with the wikipedia skill and it heard it fine.  Tested it again after adding Letterkenny to the interaction model and it worked fine.  Tested it 10+ times prior to adding to the interaction model and it played the incorrect show every single time.

Click on More on the card to see what she actually heard.
I'm going to guess Letterkenny is going to get translated by her to something odd and if the fuzzy matching routines can't get close enough to Letterkenny, your solution for such problematic shows is to add them to the Slot.

Shows as Alexa hearing the incorrect show, just seems very odd that the first time after I add it to the slot it hears it without issue, and that with other skills it hears it correctly as well.

I need to know what she actually heard (before you added the show to the Slot) if you want help.
Please find that Card, click More, and copy and paste the text after "Alexa heard:" under Voice Feedback.
It isn't odd that it works after adding it to the slot.  What's happening is that without the entry in the Slot, Alexa is hearing something else that may sound a bit like "Letterkenny."  This is then passed on to the Kodi skill verbatim, where it then goes through a few ways of matching what it heard to your library.
If Alexa heard something like "Let her coney," "Leather kidney," etc etc, it may not match.
When you add it to the Slot, she matches it exactly to "Letterkenny" which is then passed to the Skill, where it obviously has no trouble matching that exactly.
In a nutshell: the matching isn't 100% perfect.  It's really good, IMHO, but when you encounter an exception that doesn't want to match, that's exactly what the Slots are for:  exceptions.

The two "What Alexa Heard" for this across all 10+ tests prior to adding the name to the slot were:
Alexa Heard: 'tell kodi play the next episode of luther'
and
Alexa Heard: 'tell kodi to play the next episode of leverage'

So my guess is that the skill matched "Luther" and "Leverage" more strongly to another show you have.
The solution here is to add it to the Slot.
Amazon no longer supports the AMAZON.LITERAL type, so there isn't much we can do about her converting what you said.

That being the case, would there be a way to possibly auto-update the slot when new shows/movies are added?

Unless Amazon has some API for modifying the models via a script, I don't see how.
But, this shouldn't be something that occurs frequently anyway. You just picked a particularly odd one here :)

Alright sounds good! Thanks for the insight!


code documentation (typing and docstring)

Description
Progress

[ ] tests
[ ] documentation
[ ] implementation

[ ] Typing
[ ] Setting classes/methods doc

Associated issues

css are missing
seems like sphinx won't call str or repr upon objects ?

Of course, it would have been too easy... https://stackoverflow.com/questions/58359236/how-to-publish-both-sphinx-generated-docs-and-markdown-on-github-pages-at-the-sa

Will be implemented with #140.


MINOR: Annotate KafkaAdminClientTest.testAlterClientQuotas() with @Test

KafkaAdminClientTest.testAlterClientQuotas() is uncalled. It is clearly intended to be a test method, but lacks @Test.

@ijuma, @cmccabe would one of you be able to review this? Thanks.

@bdbyrne Can you take a look and review?  This was a KIP-546 change.

LGTM - thank you!

ok to test

The results of the failed test are long gone, but this is literally just a test annotation, and 2/3 test runs passed. I'll just go ahead and merge this.


Updates for fastify v5

Drop node <= 16 from workflows
Add node 21 to benchmark-pr
Add node 21 to ci-package-manager for yarn
Fixes #106, #92.
Checklist

[ ] run npm run test and npm run benchmark
[ ] tests and/or benchmarks are included
[x] documentation is changed or added
[x] commit message and code follows the Developer's Certification of Origin
and the Code of conduct

The v4 tag has been created.

Let's hope this does not get auto-updated across the entire fleet, otherwise it would be a nightmare to revert.

It shouldn't. But what in the actual eff Dependabot? https://github.com/fastify/releasify/pull/283
@Fdawgs I believe you have setup the majority of the Dependabot stuff. Do you know why it would automatically update the major versions of workflows?

https://github.com/fastify/github-action-merge-dependabot working as intended.

@Fdawgs would this land automatically then?

@Fdawgs would this land automatically then?

Unfortunately, yes. Can see it making its way through all the reps now. 😸

We really should not be updating at least our own workflows's major version automatically. The whole point of the versioning, particularly in this repo, is to indicate breaking changes in the workflow.

How do we stop the linked tool from updating all of our repos automatically for this?

@nrayburn-tech I have removed the v4 tag for right now. To continue your work, use the git SHA. I will restore the v4 tag once we have fixed the above discussed problem.


chore(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/terraform-ibm-landing-zone
module
minor
v5.25.1 -> v5.29.0

github.com/terraform-ibm-modules/terraform-ibm-observability-instances
module
minor
v2.13.2 -> v2.14.0

terraform-ibm-modules/cos/ibm (source)
module
minor
8.6.2 -> 8.10.2

terraform-ibm-modules/kms-all-inclusive/ibm (source)
module
minor
4.13.4 -> 4.15.8

terraform-ibm-modules/observability-agents/ibm (source)
module
patch
1.28.1 -> 1.28.6

terraform-ibm-modules/observability-instances/ibm (source)
module
minor
2.13.2 -> 2.14.0

Release Notes

terraform-ibm-modules/terraform-ibm-landing-zone (github.com/terraform-ibm-modules/terraform-ibm-landing-zone)
v5.29.0
Compare Source
Features

the following optional input variables have been added to the OCP DA solution, allowing the use of existing resources for KMS and COS:- existing_kms_instance_name- existing_kms_resource_group- existing_kms_endpoint_type- existing_cos_instance_name- existing_cos_resource_group- existing_cos_endpoint_type- use_existing_cos_for_vpc_flowlogs- use_existing_cos_for_atracker (#855) (1ed856b)

v5.28.0
Compare Source
Features

added support to specify the operating system for OCP cluster workers using new input operating_system (#848) (9eda4cd)

v5.27.1
Compare Source
Bug Fixes

fixed the resource_group_names and resource_group_data outputs to ensure they include prefix (#852) (677a996)

v5.27.0
Compare Source
Features

removed support for provisioning a lite plan COS instance as this plan is no longer available in IBM Cloud (#844) (006e35f)

v5.26.0
Compare Source
Features

added new output vpc_dns (#842) (9f87d9d)

v5.25.2
Compare Source
Bug Fixes

deps: update terraform terraform-ibm-modules/landing-zone-vpc/ibm to v7.19.0 (#839) (f4b278b)

terraform-ibm-modules/terraform-ibm-observability-instances (github.com/terraform-ibm-modules/terraform-ibm-observability-instances)
v2.14.0
Compare Source
Features

added support for IBM Cloud Logs (#536) (a42eb4a)

terraform-ibm-modules/terraform-ibm-cos (terraform-ibm-modules/cos/ibm)
v8.10.2
Compare Source
Bug Fixes

fixed bug where secrets module was being called by the DA even when no service credentials were being created (#696) (f5b69b5)

v8.10.1
Compare Source
Bug Fixes

deps: update required_provider to latest for the deployable architecture solution (#690) (214b30b)

v8.10.0
Compare Source
Features

support to create service credentials in secrets manager (#687)This enhancement will enable users to create service credentials in an existing secrets manager for a COS instance. Users can choose to use an existing secret group or create a new one for the service credentials. (b10af71)

v8.9.2
Compare Source
Bug Fixes

deps: updated KMS module to 4.15.2 in the bucket DAs (#685) (412cdb5)

v8.9.1
Compare Source
Bug Fixes

deps: update terraform time to latest for the deployable architecture solution (#682) (86cc006)

v8.9.0
Compare Source
Features

added support to the bucket DAs to optionally provide an apikey from another account if the KMS instance exists in a different account to COS using new input variable ibmcloud_kms_api_key (#676) (cf6e56b)

v8.8.5
Compare Source
Bug Fixes

clean up variable descriptions (#675) (e7b5254)

v8.8.4
Compare Source
Bug Fixes

added supporting documentation for the resource_keys input variable for the instances DA variation (#677) (0ff0d45)

v8.8.3
Compare Source
Bug Fixes

deps: update terraform terraform-ibm-modules/kms-all-inclusive/ibm to v4.14.2 (#679) (35e3b51)

v8.8.2
Compare Source
Bug Fixes

deps: update terraform terraform-ibm-modules/kms-all-inclusive/ibm to v4.14.1 (#678) (5336aa5)

v8.8.1
Compare Source
Bug Fixes

add sleep around auth policy when destroying (workaround for https://github.com/terraform-ibm-modules/terraform-ibm-cos/issues/672) (#673) (208a2a1)

v8.8.0
Compare Source
Features

removed support for lite plan as this is no longer available from IBM Cloud (#674) (b37bcab)

v8.7.0
Compare Source
Features

added new outputs to the secure-cross-regional-bucket and secure-regional-bucket DA solutions: s3_endpoint_direct, s3_endpoint_private, bucket_name, cos_instance_ids (#656) (2e3c40f)

terraform-ibm-modules/terraform-ibm-kms-all-inclusive (terraform-ibm-modules/kms-all-inclusive/ibm)
v4.15.8
Compare Source
Bug Fixes

deps: update terraform terraform-ibm-modules/key-protect/ibm to v2.8.4 (#538) (ab0a0f4)

v4.15.7
Compare Source
Bug Fixes

deps: update terraform terraform-ibm-modules/cbr/ibm to v1.24.0 (#535) (f93cff7)

v4.15.6
Compare Source
Bug Fixes

deps: update terraform terraform-ibm-modules/key-protect/ibm to v2.8.3 (#531) (96e6976)

v4.15.5
Compare Source
Bug Fixes

deps: update terraform-module (#529) (3c4ba65)

v4.15.4
Compare Source
Bug Fixes

deps: update terraform ibm to latest for the deployable architecture solution (#528) (2daf902)

v4.15.3
Compare Source
Bug Fixes

deps: update terraform terraform-ibm-modules/cbr/ibm to v1.23.3 (#526) (4e57041)

v4.15.2
Compare Source
Bug Fixes

deps: update terraform terraform-ibm-modules/key-protect/ibm to v2.8.1 (#525) (71bd47c)

v4.15.1
Compare Source
Bug Fixes

deps: update terraform terraform-ibm-modules/cbr/ibm to v1.23.1 (#523) (bf747e2)

v4.15.0
Compare Source
Features

added support for creating Context Based Restrictions (CBRs) using new input cbr_rules.
The existing_kms_instance_guid input in the module has been updated to existing_kms_instance_crn (#516) (defd93d)

v4.14.2
Compare Source
Bug Fixes

deps: update terraform terraform-ibm-modules/key-protect/ibm to v2.8.0 (#520) (050e4c8)

v4.14.1
Compare Source
Bug Fixes

deps: update terraform ibm to latest for the deployable architecture solution (#514) (806370c)

v4.14.0
Compare Source
Features

The following outputs have been renamed in the module and the DA solution as they now support returning values when an existing KMS instance is passed:- kp_private_endpoint -> kms_private_endpoint- kp_public_endpoint -> kms_public_endpoint (#505) (73c3a1d)

v4.13.6
Compare Source
Bug Fixes

deps: update terraform-module (#507) (428d736)

v4.13.5
Compare Source
Bug Fixes

update the DA solutions variable descriptions (#501) (c7adca0)

terraform-ibm-modules/terraform-ibm-observability-agents (terraform-ibm-modules/observability-agents/ibm)
v1.28.6
Compare Source
Bug Fixes

deps: update icr.io/ext/logdna-agent docker tag to v3.10.1-20240807.70ab6fb75337a3c9 (#376) (55810cf)

v1.28.5
Compare Source
Bug Fixes

deps: update icr.io/ext/sysdig/agent docker tag to v13.3.3 (#377) (3dea997)

v1.28.4
Compare Source
Bug Fixes

deps: update icr.io/ext/logdna-agent docker tag to v3.10.1-20240730.2544990241b83325 (#374) (cd0ffd1)

v1.28.3
Compare Source
Bug Fixes

deps: update icr.io/ext/sysdig/agent docker tag to v13.3.2 (#370) (f821a84)

v1.28.2
Compare Source
Bug Fixes

deps: updated the logdna-db-permissions initContainers image to icr.io/goldeneye_images/ubi9-cluster-tools:stable (was previously using ubi8-cluster-tools) (#365) (5eebbec)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

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

GitHub release
v1.12.6

Your semantic-release bot :package::rocket:


 Tried with both Option and Composition API, and also tried downgrading the versions. But I am still getting these errors.
warn2 @ runtime-core.esm-bundler.js:40`
[Vue warn]: provide() can only be used inside setup().
warn2 @ runtime-core.esm-bundler.js:40`
[Vue warn]: useContext() called without active instance.
warn2 @ runtime-core.esm-bundler.js:40`
[Vue warn]: Unhandled error during execution of setup function

vue-router.esm-bundler.js:3302 TypeError: Cannot read properties of null (reading 'setupContext')
    at getContext (runtime-core.esm-bundler.js:7609:14)
    at useSlots (runtime-core.esm-bundler.js:7599:12)
    at setup (vue3-easy-data-table.es.js:1305:19)
    at callWithErrorHandling (runtime-core.esm-bundler.js:155:22)
    at setupStatefulComponent (runtime-core.esm-bundler.js:7056:29)
    at setupComponent (runtime-core.esm-bundler.js:7012:11)
    at mountComponent (runtime-core.esm-bundler.js:4922:13)
    at processComponent (runtime-core.esm-bundler.js:4897:17)
    at patch (runtime-core.esm-bundler.js:4489:21)
    at mountChildren (runtime-core.esm-bundler.js:4685:13)
triggerError @ vue-router.esm-bundler.js:3302

This is not a problem with the library. I made a reproduction for you here

I downloaded your sandox and ran on my PC, and I am still getting the same error. May I ask which node versions are you using?

@sunil-dzango you can check it in package.json here

Seems to be related to https://github.com/HC200ok/vue3-easy-data-table/issues/163, as updating vue to version 3.2.45 fixes the issue.

Same issue here, tried upgrading the library and vue, removing my lock file, my modules and nothing

Having the same issue since I've upgraded to Vue 3.3.x. Used to work fine with Vue 3.2.x


Documentation: Improve Canvas definition to help onboarding new participants

Description
To support onboarding new participants, enhance the Canvas Design documentation. There is material from the Copenhagen Masterclass that can support an improved introduction to the Canvas.

Closed with PR #135


added Escape Station as example of game using toolkit to Readme

as title suggest's added one line to readme. to include another game using this (Im the developer of said game)

There were the following issues with your Pull Request

Commit: bcfd079ee494260076ffdf63dc7e7841da2af49b
Commits must be in the following format: %{type}(%{scope}): %{description}

Guidelines are available at https://github.com/thestonefox/SteamVR_Unity_Toolkit

This message was auto-generated by https://gitcop.com

Interesting... I am working on exactly the same for quite a while already ;-)

I added it


To update document https://docs.microsoft.com/en-us/azure/devops/pipelines/process/stages?view=azure-devops&tabs=yaml#conditions

This document has date 5/3/2019 and states that "You cannot currently specify that a stage run based on the value of an output variable set in a previous stage." While in the sprint-168 it now says
Azure Pipelines
Jobs can access output variables from previous stages

https://docs.microsoft.com/en-us/azure/devops/release-notes/2020/sprint-168-update#azure-pipelines-1
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 4266f72c-c774-0046-4593-d01eb775d3c3
Version Independent ID: f20827aa-a6c5-96a8-5969-e576ffbc2e38
Content: Stages in Azure Pipelines - Azure Pipelines
Content Source: docs/pipelines/process/stages.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

Thanks @rodrigoCruzTekExperts. This was fixed previously.


PowerRun fails to find apps from Store after 0.18.1 upgrade

Environment
Windows build number: Windows 10 19041.264
PowerToys version: 0.18.1
PowerToy module: PowerRun
Steps to reproduce
With 0.18.0 installed.
Upgrade to 0.18.1
Use PowerRun and try to open apps from MS Store (like Windows terminal, Windows store itself)
Expected behavior
It should find these APPs :)
Actual behavior
PowerRun doesn't show any result, even with full name
PS.: Rebooting the OS fix the issue.
Screenshots
I Didn't take any, but find the same result in two different machines.

Closing this in favour of #3223


Merge cortex, cxgen and cxsh

Currently there are three command line tools for cortex. Even though the tools are sufficiently different to justify them being separate executables, it is not as clean when it comes to installing cortex on a machine. Ideally, there should just be one tool that is installed globally.
Proposed to merge the tools, such that:
cortex

Starts the shell.
cortex create MyApp --lang c

Runs cxgen --package MyApp --lang c
cortex MyApp --lang c

Runs cxgen MyApp --prefix MyApp --lang c

Attempt at a more structured description to merging the tools:
Create an application
cortex create MyApp

This creates a new application project with the name "MyApp". Applications are standalone executable units which right now the build system doesn't yet provide for. The following files will be created:
MyApp
├── rakefile
└── src/
    └── MyApp.c

Create a component
cortex create component MyLib

This creates a new component project with the name MyLib. The following files will be created:
MyComponent
├── rakefile
└── src/
    └── MyComponent.c

Create a package
cortex create package MyPackage

This creates a new package project with the name MyPackage. The following files will be created:
MyPackage
├── rakefile
├── MyPackage.cx
└── src/
    └── MyPackage.c

Build a project
cortex build

This will build the project. Only applicable to projects of compiled languages.
Test a project
cortex test

This will run associated tests for the project.
Install a project
cortex install

Installs a project to the global cortex environment. Requires cortex to be installed on a machine.
Run a project
cortex run

Runs the project. Only applicable to applications.

The full range of options:

Option
Description

Start cortex shell

--help
Displays help for cortex command

--version
Displays version of cortex

create [artifact] name
Create new project (for artifacts, see previous comment)

build
Build project

run
Run application

install
Install project

pp
Run preprocessor (not to be invoked by user, is ran when using build


Wrong assembly code is generated in some cases by SV generator that causes unexpected spike simulation error.

It looks that the recent fix for https://github.com/google/riscv-dv/pull/710 introduced the problem with a spike simulation.
The randomization of cfg.scratch_reg register seems to be constrained only by following constraint:
https://github.com/google/riscv-dv/blob/8a6a399ed5268130f30879d5548d97b477f21bdf/src/riscv_instr_gen_config.sv#L414-L418
This causes that scratch_reg can be in some cases randomized to be the GP register that is the same as x3:
https://github.com/google/riscv-dv/blob/8a6a399ed5268130f30879d5548d97b477f21bdf/src/riscv_asm_program_gen.sv#L1151
During the exit procedure the GP register is assigned with 1 but then it is overridden by assigning "write_tohost" subprogram address to the x3 register.
test_done:
                  li gp, 1
                  ecall
...
ecall_handler:
                  la x25, _start
                  sw x0, 0(x25)
                  sw x1, 4(x25)
                 ...
                  sw x31, 124(x25)
                  la x3, write_tohost
                  jalr x0, x3, 0

...
write_tohost:
                  sw gp, tohost, t5

_exit:
                  j write_tohost

This causes that spike ends simulation with unexpected error.  See https://github.com/riscv/riscv-isa-sim/issues/555
The complete assembly program and simulator log is attached in a zip file:
rv32i_riscv_jump_stress_test.zip

Thanks for opening the issue. Seems like we need to constrain scratch_reg to not be the same as gp register. I'll prepare a fix.

@dariuszst can you apply the fix in #726 and see if this issue is fixed?

@udinator - I've checked some test for different targets (rv32i, rv32imc, rv64imc, 64gc, multi_harts) on the branch with a fix and I didn't notice spike simulation problem as before. So I think the correction of the constraint should solve the problem and will not introduce new ones.

Thanks @dariuszst! I'll go ahead and close out this issue.


Styles are applied on components during Editor.prototype.create instead of Editor.prototype.setStyle

updating editor's style should affect existing components
imported components should get current styling

updating editor's style should affect existing components
imported components should get current styling

updating editor's style should affect existing components
imported components should get current styling


When the ProcessEvent throw an exception all subscribes will be lost

I catch the exception at top scope, subscribes still lost,  but it no happend every time,  look like it happend in some special condition,how do i fix that?
public class MqttEventBus
{
        _client = mqttFactory.CreateMqttClient();
        _mqttClientOptions = new MqttClientOptionsBuilder()
            .WithProtocolVersion(MqttProtocolVersion.V500)
            .WithCredentials("***", "***")
            .WithTcpServer("***.***.***", ***).Build();

        _ = Task.Run(
            async () =>
            {
                // User proper cancellation and no while(true).
                while (true)
                {
                    try
                    {

                        // This code will also do the very first connect! So no call to _ConnectAsync_ is required in the first place.
                        if (await _client.TryPingAsync()) continue;
                        _logger.LogInformation("Try to connect the blocker");
                        await _client.ConnectAsync(_mqttClientOptions, CancellationToken.None);
                        _client.ApplicationMessageReceivedAsync += ProcessEventAsync;
                        _client.DisconnectedAsync += DisconnectedAsync;
                        SubscribeHandlers();
                        _logger.LogInformation("Connected");
                    }
                    catch (Exception e)
                    {
                        _logger.LogException(e);
                    }
                    finally
                    {
                        // Check the connection state every 5 seconds and perform a reconnect if required.
                        await Task.Delay(TimeSpan.FromSeconds(5));
                    }
                }
            });

}

    private Task DisconnectedAsync(MqttClientDisconnectedEventArgs arg)
    {
        _logger.LogInformation("DisconnectedAsync");
        return Task.CompletedTask;
    }

 private async Task ProcessEventAsync(
        MqttApplicationMessageReceivedEventArgs mqttApplicationMessageReceivedEventArgs)
    {

            try
            {

                await TriggerHandlersAsync();
            }
            catch (Exception e)
            {
                mqttApplicationMessageReceivedEventArgs.ProcessingFailed = true;
                _logger.LogError(e, message: "ProcessEventError");
            }
        }
    }

logs

The problem still in my product environment，I try to publish rpc ping/pong message to other computer for monitor this problem on a same client instance.
Then it can receive well，I also used third part tools(mqttx) to publish a new message，it receive well too，i think the exception only stopped real device messages，so i am confused about this.
Any one have an idea for help， thanks!

If it works when you publish a message with mqttx, then my best guess is the message being sent from the actual device, sometimes has an unexpected character or value in the payload, and your TriggerHandlersAsync() isn't able to handle that.  I do quite a bit with GPS Devices and I've run into issues where, perhaps I expected the course/direction to be an integer because that was what my test packet had, but from time to time, the actual device was sending a decimal/double, or null, and that threw exceptions - perhaps this is something similar.
You should log the contents of the received message payload when there is an exception to see if there is something unusual about the payload contents.  As well, you should create additional try/catch methods within TriggerHandlersAsync() and nested methods to narrow down where the exception is being thrown.

Finally，I fix this issue via Task.Run，Process event code execute in the other thread，I have not find the reason why we can't thow exception in First thead.


Docs website down?

Trying to get started with this project, but the documentation website appears to be offline.

It's been down for a bit.  Try using the .md docs files inside the repo :)


336: PaymentsJS Implementation

@afred - Since we'll be handling environment variables through the separate script, this should be ready for review. Thanks again for your help in talking through release! Closes #336 and Closes #337.

@afred - Assuming the checks come back well, this should be ready to roll. Thanks!

Github is not updating to show these tests pass, but following the links to the Travis builds, they do. Merging.


Improvefix function array index with null param and add regress cases for ar…

fix array index with null param
add create inverted index regress cases for array nested all type in dup/mor/mow table
add query case for array all type with inverted index

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run external

run external


set  w/o restoration module

DIffbir How to set up w/o restoration module training

In our latest code, to train w/o restoration module, you just need to remove a few lines of code (as shown below):
https://github.com/XPixelGroup/DiffBIR/blob/e7974cbe899e83bddd392a9a7ea0356b8fb529eb/train_stage2.py#L80-L89
https://github.com/XPixelGroup/DiffBIR/blob/e7974cbe899e83bddd392a9a7ea0356b8fb529eb/train_stage2.py#L108
https://github.com/XPixelGroup/DiffBIR/blob/e7974cbe899e83bddd392a9a7ea0356b8fb529eb/train_stage2.py#L131
and directly use lq as condition:
cond = pure_cldm.prepare_condition(clean, prompt) => cond = pure_cldm.prepare_condition(lq, prompt)


String: Increase prefix size from 128 to 256 chars

The file prefix limit was originally 256 characters. When the code was converted to use LMP TRIE maps, the limit was set to 128 characters.
This commit changes the prefix limit from 128 to 256 characters.
Increases the character limit for prefix matches from 128 to 256.

I got raced 🏎 https://github.com/cilium/tetragon/pull/1732/commits/c02af3e1fee37f9c57ed7c3caa67e786af092a9d


Add medium type

Codecov Report

Merging #3 (6b6c048) into main (be95be7) will increase coverage by 21.05%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             main        #3       +/-   ##
============================================
+ Coverage   78.94%   100.00%   +21.05%
============================================
  Files           2         2
  Lines          19        21        +2
  Branches        1         1
============================================
+ Hits           15        21        +6
+ Misses          4         0        -4

Impacted Files
Coverage Δ

test.c
100.00% <100.00%> (ø)

example.c
100.00% <0.00%> (+26.66%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Datatables do not respect the flex column_arg

As per the documentation here and here, the flex key should be able to resize the columns to fit the page width in proportion to the flex values given.
Here's my code:
def test_pywebio():
	row = {
		"Col1": "1",
		"Col2": "asddddddda",
		"Col3": "asdasd",
		"Col4": "asdasdasd",
		"Col5": "asd",
		"Col6": "asd",
		"Col7": "asd",
		"Col8": "asdasdasdas",
		"Col9": "asdasda",
	}
	col_args = {
		"Col1": {"flex": 1},
		"Col2": {"flex": 6},
		"Col3": {"flex": 2},
		"Col4": {"flex": 2},
		"Col5": {"flex": 2},
		"Col6": {"flex": 2},
		"Col7": {"flex": 2},
		"Col8": {"flex": 6},
		"Col9": {"flex": 2},
	}

	pywebio.output.put_datatable([row], column_args=col_args, height='auto')

Here's the output:

As you can see, the flex values are not being considered.
I also tried using maxWidth, minWidth and width values, and although there are some changes to the table sizes, it does not work as documented.
Environment Information

OS and Version:  Win11
Browser and Version:  Chrome Version 111.0.5563.147 (Official Build) (64-bit)
Python Version: v3.11.2
PyWebIO Version: 1.8.0

pywebio will call gridOptions.columnApi.autoSizeAllColumns() when the grid is ready, which will make the grid ignore the flex setting on column.
I will consider to update code to disable auto size column when user provide column sizing settings.


Events prototypes

Who is this for?
The team
What is it doing for them?

Creates Option 1 for testing
Displays single date for a multi date event (either next upcoming or one passed via url parameter)
Adds show/hide for all dates

Fixes #2943

Codes good if we're happy it'll need (small amounts of) refinement.
Question around why we've decided to show the nearest date till today? When testing, wouldn't it be simpler to start with the easiest thing and make no assumptions so we can tease out what people really want, rather than just agreeing with something we've shown them?
Similar to the whats on filter, we're at risk of having an idea validated, not because it's the best way to do it, but because while people are going through the test they find nothing wrong with it?

Ping @Heesoomoon for question ☝


Elliptic curve visualizer

Closes #593
Preview

Hmmm graph looks a bit strange when I change the width of the screen. This is what it looks like at 1046x740.

Really nice. A few tips for improvement:

On mobile, the arrow and label are squeezed in between the graph and legend. Could the be positioned in the same place as on desktop?
The pulsating circle is barely noticeable. I think you can make that a bit strong visually, and also slow down the animation quite a bit (more of a calm pulse)
You shorten the pauses in between animation steps a bit more

I did not see the issue Andrew mentioned above, so I assume that's fixed.

Another thing is that I cannot run this locally. npm can't figure out the dependencies when trying to install. Is there a particular node version that I need now, or something else?

Hmm not sure about the installation problem, It does add a new dependencey with react-lottie but I don't think that should be an issue


Blockchain not configured This blockchain has not been enabled for use within Anchor.

Description
We are trying an anchor wallet SDK in unity and facing this error when we tried to run sdk of anchor wallet. We change the chain id and testnet URL in RPC but not working.
SDK Link: https://liquiidio.gitbook.io/unity-plugin-suite/v/anchorlink/examples/example_a
Error:"Blockchain not configured
This blockchain has not been enabled for use within Anchor.
In order to sign requests with this blockchain, you must first enable the blockchain and setup an account through the main interface."
Followed all the Steps for blockchain configuration but getting this error again and again.
I also changed chain id and Rpc but it's not working .If anyone know about this error Please let me know.
Here's the code where i change Chain id  and Rpc into testnet .
public async void StartSession()
{
_link = new AnchorLink(new LinkOptions()
{
Transport = this.Transport,
            ChainId = "f16b1833c747c43682f4386fca9cbb327929334a762755ebec17f6f23c9b8a12",
            Rpc = "https://api.wax-test.liquidstudios.io",
            ZlibProvider = new NetZlibProvider(),
            Storage = new PlayerPrefsStorage()
        });

        await Login();
    }

Platform
Desktop (Windows)
Steps To Reproduce
1.Using this sdk in unity.https://liquiidio.gitbook.io/unity-plugin-suite/v/anchorlink/examples/example_a.
2.With Wax testnet and getting those errors.
3. Getting this  error

Relevant log output
No response
Contact Details
<EMAIL_ADDRESS>Anything else?
No response

When this happens, can you click the "Share Request" button in the lower left, and share the contents of that here?
I have a feeling the SDK itself is sending the wrong network as the requested blockchain. Anchor itself doesn't have any issues normally selecting which network.


Add workflow for github releases.

This github workflow will generate a new release when pushing a tag with the correct format.
Meaning release now will be done with the following process:

push a release candidate tag
let all partners run tests
push a new release candidate tag if some failure have been found and fixed.
once all green, push the release tag on the green release candidate.

The release will be done.
No release branch will be created as we don't need them anymore.
Also no stable/canary branch update anymore, as we won't use them anymore.

How did you test this?

Tried it on my fork. Just re-run it on this head:
https://github.com/Keenuts/SPIRV-Tools/releases/tag/v2022.4
https://github.com/Keenuts/SPIRV-Tools/actions/runs/4377103880


Auto gg

This was suggested by me in #16.
I added a option to toggle it on and off, defaults to off.

Uses depreciated/unused mixin, so does not technically work, but can be if the mixin is enabled.

I forgot to check if the player is in pickaxe, so this is not ready yet.

I still would not merge this; as i need to delete that code or fix the /locate functionality
( I Just cant figure the regexes out)

I can look into it if you want. I have to do a bunch of regex anyway

Also this probably counts as a bot

Kinda I guess? It's an auto response, I don't know if that counts as a bot. Maybe? And I don't know if those are illegal.

Luckly it didn't work anyway


I am unable to login with Admin User

https://37b4-2a01-4f9-5a-1303-00-2.ngrok-free.app/#/login
I am entering credential
username: admin
password: admin
When i click on login it failed with Request Timeout Error
Let me know what i am doing wrong?

Hi Anildalar,
Unfortunately, the jade project is no longer in maintenance. Most of features are moved to the new project - voipbin.
Please check out the voipbin project here.

http://voipbin.net/

Thank you.
Kind regards,
Sungtae

Ok thanks for reply
On Mon, 27 May, 2024, 6:46 am Sungtae Kim, @.***> wrote:

Hi Anildalar,
Unfortunately, the jade project is no longer in maintenance. Most of
features are moved to the new project - voipbin.
Please check out the voipbin project here.

http://voipbin.net/

Thank you.
Kind regards,
Sungtae
—
Reply to this email directly, view it on GitHub
https://github.com/pchero/jade/issues/242#issuecomment-2132480989, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AGEIXIJ6GYRBSCRTN5JWBU3ZEKCORAVCNFSM6AAAAABIJRFGLCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZSGQ4DAOJYHE
.
You are receiving this because you authored the thread.Message ID:
@.***>


change kinflate to kustomize

Rebased and resolved conflict in pkg/kustomize/transformers/overlay.go

Travis is not happy.
It seems we still use kinflate somewhere.


Fix: change IF writing for surrogate pairs issue

Source issue: https://github.com/metanorma/xmlgraphics-fop/issues/4
Original issue: https://github.com/metanorma/mn2pdf/issues/90, https://github.com/metanorma/mn2pdf/commit/db41c1a9f96eed7ef786cbc5963dd9014b5ae8f1

Done in the branch fix/characters_issues.


[ISSUE #3267] Refactor RequestContext

Fixes #3267
Motivation
Explain the content here.
Explain why you want to make the changes and what problem you're trying to solve.
Modifications
Refactor RequestContext
CountDownLatch replace with CompletableFuture
Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

@jonyangx  PTAL~


[Bug]: Disabling "Allow Filtering" also disables "add new row"

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Using v1.8.12.
When "Allow Filtering" is disabled in the table properties, the "add new row" option in the table widget also disappears.
Steps To Reproduce
Create a table widget. In the widget properties, disable "Allow Filtering". The filter icon/button in the table disappears but the "add new row" also disappears. The widget property "Allow adding a row" is still enabled, but is absent from the widget.
Public Sample App
No response
Issue video log
No response
Version
v1.8.12 self-hosted docker

@sbalaji1192 @chandannkumar i thought this issue was resolved. Can you both please check and see if this is still happening? If yes, @sbalaji1192 can you estimate this issue and we will decide if we can pick it up this sprint or not

@dilippitchika We closed this issue last week. The fix will get released in the upcoming release this week. https://github.com/appsmithorg/appsmith/issues/18346

Ok, @bobbintb this issue has been fixed as part of our recent fixes which will go live this week. Please keep an eye out for v1.8.13


Demo location

Let's make it easy for people to try out command line music box and explore the files without having to download the project.
Acceptance criteria

One button click takes you to some environment where you can begin using musicbox

Ideas

Codespaces provide useful environments for people to immediately start working on the project. They could also be useful for a demo if we install music box on setup and show by default instructions for the demo

This has the benefit of an integrated terminal and you can download things from the file explorer
We may also be able to upload some waccm data by default if we make them small enough so that the script could at least be demonstrated, even if we have to regrid the files to be very sparse

Maybe a binder, the only problem is that the runnable scripts wouldn't be as easy to use

Download
https://www.mediafire.com/file/7ybpxn8lil1av70/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Complete cookies API

Create a more complete cookies API for accessing and controlling our cookie features.
Work in progress.
See #215.

Narrowing the scope on this one, because the PR is becoming too large...


SessionSecurityMiddleware Client Activity Keep-Alive

I really like the all-in-one convenience of django-security, but the SessionSecurityMiddleware implementation lacks the client-side keep-alive available in django-session-security.  The keep-alive is important to us because our product is used to guide a conversation with a customer so our users are often "active" on a page without server-side interaction.
Any interest adding a keep-alive feature to django-security?  If so, what approach would you prefer?  The licenses look compatible so it seems like any of the following would work:

Replace SessionSecurityMiddleware with the django-session-security implementation
Port the JS code to SessionSecurityMiddleware
Include both in django-security

Interesting. We built our own little keep alive thing for our own application. I don't know if a better solution is to make django-session-security a dependency of django-security and just expose that. (Seems weird to just grab all of their code.)

I ended up trading emails with James (author of DSS) to discuss a vulnerability.  In that same private email chain, I mentioned this project and he said:

If someone contributed the client side script to django-security it would be great !

So the second option is already 100% approved.  I'm a fan of a combined project so a single base of users/developers can ticket, fix, and release the combined codebase without needing commit access to both (thought I suspect James would facilitate that option as well).

Ah cool. It's a bit hectic here, but i'll see if we can have someone take a look at this.


Read the ms.service value from csv metadata

Our original ms.service assign value as follows:

If csv ServiceName exists, then Normalize Key Vault - > keyvault
Otherwise, use the service directory

The proposal:

If csv MSDocService exists, then use the value directly. PM or service owner will update this to the right value
Otherwise, use the service directory

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


KeyVaultEnvironmentPostProcessor.isKeyVaultEnabled should check the enabled flag before checking if AZURE_KEYVAULT_VAULT_URI is populated

KeyVaultEnvironmentPostProcessor.isKeyVaultEnabled should check the enabled flag before checking if AZURE_KEYVAULT_VAULT_URI is populated.
URIs like AZURE_KEYVAULT_VAULT_URI  are often constructed from other properties and in many test scenarios resolution will fail due to those properties not being resolved.
Solution:
Reorder and simplify isKeyVaultEnabled's conditionals.
https://github.com/microsoft/azure-spring-boot/blob/4543480ed26976dfb9eec71375855da5ed24fdc6/azure-spring-boot/src/main/java/com/microsoft/azure/keyvault/spring/KeyVaultEnvironmentPostProcessor.java#L31-L38
Patch in PR:
    private boolean isKeyVaultEnabled(ConfigurableEnvironment environment) {
        return environment.getProperty(Constants.AZURE_KEYVAULT_ENABLED, Boolean.class, true)
                && environment.getProperty(Constants.AZURE_KEYVAULT_VAULT_URI) != null && isKeyVaultClientAvailable();
    }

Thanks for bringing this up. Could you share more details about the test scenarios when resolution will fail due to the URI?

Our default config defines the current pattern used for our keyvault instance,
We use profiles to set the referenced properties.
We were hoping to avoid having another profile to set the uri and we want to be able to test construction of the uri without having the KeyVaultEnvironmentPostProcessor start.
The patch fixes this issue.
#default config
#this currently fails
azure.keyvault.enabled=false
azure.keyvault.uri=https://${subscription.prefix}-${env.prefix}-${region.code}.vault.azure.net

@jmax01 Thanks for your response and just for clarification:
When the user specifies azure.keyvault.enabled to false, the KeyVaultEnvironmentPostProcessor should not be started. In which case, the below code snippet does return false.
https://github.com/microsoft/azure-spring-boot/blob/4543480ed26976dfb9eec71375855da5ed24fdc6/azure-spring-boot/src/main/java/com/microsoft/azure/keyvault/spring/KeyVaultEnvironmentPostProcessor.java#L31-L38

should not be started. In which case, the below code snippet does return false. Or do I miss something?

If azure.keyvault.enabled is false KeyVaultEnvironmentPostProcessor should not attempt to resolve AZURE_KEYVAULT_VAULT_URI.
Processing should cease immediately.

The issue still exists in the new implementation. I will supply a new PR.

This problem already fixed in our new repo, so I'll close this issue.


Bug (compactor):  Investigate compactor oom bug and improve memory allocation

Background
The phenomenon of compactor oom occurs during the compactor-test.

We found that when the test is first started, compactor's memory spikes and experiences oom, and stabilizes after a while.
Investigate
I turned on the memory profiler and reran the test, and found a few things.

After the introduction of check_compact_result, it was taking up some of the memory.  I found out that the memory tracker is not maintained during the execution of check_compact_result, which could lead to a memory overflow.

Unfortunately, this only delayed the appearance of oom and did not solve the problem. so continued to analyze the metrics and the profile.There was some interesting information in the metrics that helped me investigate the cause of the problem: Why is the oom only appearing at startup? A simple way to think about it is to look for fluctuations in the same time.

multi version key

k-v pair size

epoch

According to the above metrics, we can find that in the startup phase of the test (10min), min_epoch is pinned, which leads to a large number of multi-version keys in the system, and the size of k-v pairs in the test is larger than that of a normal nexmark test.
Analyzing further, based on the profile results, we can locate the problem in the encode and decode phases of the block.
1.

From the profile analysis, BlockBuilder::add consumes a lot of memory, based on the metrics above, it may be due to the effect of multiple key versions. Reading the code further, after https://github.com/risingwavelabs/risingwave/commit/8136111a7dac6034759952a40a3b457833152d57, the builder tends to place the same user_key in the block.

TLDR: the limit of is_new_user_key could lead to huge blocks under the current test, which are not covered by the memory estimation algorithm, and trigger more memory allocations.

Improvement
Based on the information in the Memory profile, we found some points that can be optimized.

BlockBuilder::Add

BlockBuilder preallocates the buf at startup to minimize secondary memory allocation.

There are two problems:

the extra 256 allocated may not be enough after the introduction of the type index, and a large number of keys will trigger an allocation of

During compress, the buf variable is changed, invalidating the expected pre-allocation.

BlockBuilder::compress

compress function will generate a new BytesMut write each time for encoding

Block::decode_with_copy

Crete Bytes from Vec lead to memory reallocation.
    /// Converts the vector into [`Box<[T]>`][owned slice].
    ///
    /// If the vector has excess capacity, its items will be moved into a
    /// newly-allocated buffer with exactly the right capacity.
    #[cfg(not(no_global_oom_handling))]
    #[stable(feature = "rust1", since = "1.0.0")]
    pub fn into_boxed_slice(mut self) -> Box<[T], A> {
        unsafe {
            self.shrink_to_fit();
            let me = ManuallyDrop::new(self);
            let buf = ptr::read(&me.buf);
            let len = me.len();
            buf.into_box(len).assume_init()
        }
    }

Change

reserve more space for type index
change the condition for determining block full to avoid block size exceeding capacity as much as possible.
Introduce a dedicated compress buf
refactor the usage of Bytes

Result

From the profile analysis, BlockBuilder::add consumes a lot of memory, based on the metrics above, it may be due to the effect of multiple key versions. Reading the code further, after https://github.com/risingwavelabs/risingwave/commit/8136111a7dac6034759952a40a3b457833152d57, the builder tends to place the same user_key in the block.

What is the benefit of putting all versions of a user key into a single block? I guess it is a bug?


Implement python decouple

Implemented python-decouple.
All the environment-specific parameters (secret key, user, passwords, debug status etc.) are now kept into a single .env file. Also, we won't be needing  secret.py script anymore. Here are the changes made to achieve the same:

Added .env script.
Added .env to .gitignore to ensure these are not pushed with the code.
Updated requirements.txt to add python-decouple.
Made the required changes in README.md.

Fixes #85

Fixed with #180. Closing this


ELK compatible logs

Changes needed for collecting logs from the prod instance

Not needed because #152 is being merged and deployed to prod


[GH-#406] Removed last_updated field from Account structure

Resolves #406 .

@meivantodorov @HristianHristov @0legIv can you review?


TypeError: Cannot read property 'useRef' of null

TypeError: Cannot read property 'useRef' of null
TypeError: null is not an object (evaluating 'dispatcher.useRef')
This error is located at:
in SpeechRecProvider (at SpeechRecognitionRootView.js:5)
in SpeechRecognitionRootView (at App.js:260)
in RCTView (at View.js:32)
in View (created by MenuProvider)
in RCTView (at View.js:32)
in View (created by MenuProvider)
in MenuProvider (at App.js:255)
in App (at Thrive_AI_App/index.js:18)
in Provider (at Thrive_AI_App/index.js:17)
in Root (at renderApplication.js:50)
in RCTView (at View.js:32)
in View (at AppContainer.js:92)
in RCTView (at View.js:32)
in View (at AppContainer.js:119)
in AppContainer (at renderApplication.js:43)
in Thrive(RootComponent) (at renderApplication.js:60)
import {StyleSheet, Text, View} from 'react-native';
import React from 'react';
import {SpeechRecognitionRootView} from 'react-native-voicebox-speech-rec';
const App = () => {
return (

App

);
};
export default App;
"dependencies": {
"react": "18.2.0",
"react-native": "0.74.1",
"react-native-background-fetch": "^4.2.5",
"react-native-html-to-pdf": "^0.12.0",
"react-native-share": "^10.2.1",
"react-native-voicebox-speech-rec": "^1.0.4"
},
and one more thing is that when I install this library  and then install the pods it removes other pods like Removing RNSVG, Removing RNScreens and Removing react-native-safe-area-context.
Adding a custom script phase for Pod RNFBApp: [RNFB] Core Configuration
Auto-linking React Native modules for target Thrive: BVLinearGradient, CodePush, RNAppleAuthentication, RNCAsyncStorage, RNCClipboard, RNDateTimePicker, RNFBApp, RNFBAuth, RNFBFirestore, RNFBMessaging, RNFS, RNFastImage, RNGestureHandler, RNGoogleSignin, RNIap, RNImageCropPicker, RNKeychain, RNLocalize, RNNotifee, RNPermissions, RNRate, RNReanimated, RNShare, RNSound, RNVectorIcons, ReactNativeAvoidSoftinput, TextToSpeech, appcenter-analytics, appcenter-core, appcenter-crashes, react-native-biometrics, react-native-fbsdk-next, react-native-image-picker, react-native-in-app-review, react-native-netinfo, react-native-pager-view, react-native-splash-screen, react-native-track-player, react-native-version-info, react-native-voice, and react-native-webview
[Codegen] Generating ./build/generated/ios/React-Codegen.podspec.json
Analyzing dependencies
[Codegen] Found FBReactNativeSpec
/Users/mac/Documents/JSDev/Thrive_AI_App/node_modules/react-native-fbsdk-next/react-native-fbsdk-next.podspec:4: warning: already initialized constant Pod::FBSDKVersion
/Users/mac/Documents/JSDev/Thrive_AI_App/node_modules/react-native-fbsdk-next/react-native-fbsdk-next.podspec:4: warning: previous definition of FBSDKVersion was here
Downloading dependencies
Removing RNSVG
Removing RNScreens
Removing react-native-safe-area-context
Generating Pods project
Integrating client project
Pod installation complete! There are 78 dependencies from the Podfile and 113 total pods installed.

Also wondering on if there's a fix for this

same here. Any update


Add docs command

Description
This PR adds a docs command which helps produce readme files for each combination. To this end, a Go template can be provided which is executed for each combination.
Have a look at the defaultTemplate in the docs.go for an example.
Related Issue(s)

Fixes #
How to test
go run main.go docs --context example/ --all

Release Notes

NONE

Hey @csweichel , I added this to the Workspace Project, set the status in In-Progress, and assigned you (so that we don't lose sight of this issue).
cc: @Siddhant-K-code @utam0k

Hey @csweichel @Siddhant-K-code and @axonasif, is this still needed?

Hey @csweichel @Siddhant-K-code and @axonasif, is this still needed?

Yes, this PR was intended to create a markdown file containing the content of our workspace images. Which would help the better documentation for workspace images 🙏🏼

:wave: hey there, removing this from the Team Workspace project (for now). @csweichel not sure if you want to pick this back up, but, it's not currently a priority.


Automatically set pythonpath correctly for default project setups

see https://stackoverflow.com/questions/53653083/how-to-correctly-set-pythonpath-for-visual-studio-code
some 1/4 million views so far approx 150 upvotes but no implementation in vscode so far.
Please improve to make python developers happy campers and not get ModuleNotFoundError:  on trying to run the first piece of code or test - it is too frustrating when knowing that quite a few IDEs will do this out of the box with no such errors.

Please file this on the vscode-python extension, thanks


stella: 6.7.1 -> 7.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/stella-emu/stella/releases.
meta.description for stella is: Open-source Atari 2600 VCS emulator
meta.homepage for stella is: https://stella-emu.github.io/
meta.changelog for stella is: https://github.com/stella-emu/stella/releases/tag/7.0
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 7.0 with grep in /nix/store/yamxxlazxqlccsgxydqbzwgi01rsfqkq-stella-7.0
found 7.0 in filename of file in /nix/store/yamxxlazxqlccsgxydqbzwgi01rsfqkq-stella-7.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

stella

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/72f627af9e350b4b609c2cf5f0eec21538ea218e#stella

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

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

1 package built:

stella

Maintainer pings
cc @AndersonTorres for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge


#830 Improve time to display context menu with large number of nodes

Fixes: #830
Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

This may not working in all circumstances so I am not going to merge it to master.


feat(testing): Use JVM Test Suite Plugin

This PR moves the creation of each version-specific test source and task to use the JVM Test Suite Plugin. This is beneficial not only because Gradle is moving towards this DSL, but it also gives better handle of test dependencies and shared test configurations (as described in the docs). This change, however, comes with some trade-offs and considerations:

The minimum version of Gradle required to use the plugin had to be increased to v7.3

If we want to preserve compatibility before v7.3, perhaps we could create a different version of the MultiReleaseExtension that uses the legacy API and choose which one to use on runtime depending on the executing Gradle version?
However, Gradle is now on the brink of v8.4, so preserving compatibility with older versions may not be worth the effort.

The project's Gradle Wrapper was updated to v8.3 to access the JVM Test Suite Plugin API.
The documentation was updated to reflect the usage of the JVM Test Suite Plugin

An example was added about sharing test configurations
The "Version specific dependencies" section now uses the TestingExtension DSL. Should we preserve its equivalent without this new DSL?

The JVM Test Suite dependencies API was introduced on Gradle v7.6, so the code has a conditional section to allow compatibility with versions >=7.3 and <7.6.

As mentioned above, I'm not sure it's worth preserving compatibility with v7.x, so maybe it'd be better to increase the minimum Gradle version to v7.6 and remove that condition from the code. Any thoughts?

Finally, all sample build configurations were updated to use the TestingExtension DSL.

I am personally not very convinced by the test suite APIs (e.g in this PR doesn't look like it makes the code any shorter), but as long as it is compatible with the old mechanism, fine by me. Can you restore the abstract modifier for the extension?

Right! I agree it doesn't make the code shorter for the plugin, but it's great to have a single extension to handle everything related to tests on user-land. But yes, the old mechanism is still compatible, so people can still choose 🙂
I restored the abstract modifier for the extension. Thank you for this plugin! I have found it very useful so far, and the post about when to use Multi-Release JARs you wrote was very useful to make an informed decision!


Fix CUDA MMA transform dialect tests

We generalized linalg.matmul_transposed_b ops into linalg.generic ops with https://github.com/openxla/iree/pull/15678. This commits updates the tests to match linalg.generic ops accordingly.

Side note, neither transform dialect strategies, nor kernel config should require generalization (or specialization) to work IMO. Generalizing after strategy selection should always be fine though. We might want to eventually drop that generalization pass, or tune it to only the cases where it makes sense (e.g. linalg.reduce).

Side note, neither transform dialect strategies, nor kernel config should require generalization (or specialization) to work IMO. Generalizing after strategy selection should always be fine though. We might want to eventually drop that generalization pass, or tune it to only the cases where it makes sense (e.g. linalg.reduce).

Yup agreed. We should improve the generalization and strategy selection ordering issue there. But wanted to have this unblocking the CI, given that the failing tests are just happen to use those named ops to test something else.


[HUDI-6127]Flink Hudi Write support commit on an empty batch

Change Logs
Fix issue #8498
In flink multi-writer scenario, if one of the writers has no data, it will keep its inflight instant in the timeline. But incremental clean and archive will be blocked by the oldest inflight commit in the timeline.
So to make clean and archive can move forward  we should upport starting a new instant even when there is no data in the last batch.
Impact
Now we will commit the instant even when there is no data in current batch, thus there maybe some useless commits on the timeline.
Risk level (write none, low medium or high below)
None
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change
Contributor's checklist

[x] Read through contributor's guide
[x] Change Logs and Impact were stated clearly
[x] Adequate tests were added if applicable
[ ] CI passed

@hudi-bot run azure

@hudi-bot run azure

@hudi-bot run azure

@hudi-bot run azure

The failed test case ITTestHoodieDataSource#testStreamWriteReadSkippingCompaction should not be affected by this patch (flag default false), also run it in local for about 10 times and can not reproduce, assumes it is flaky and would merge the patch soon ~


Could not connect to DB of v12.0

I run the docker-compose -p opencast-allinone -f docker-compose/docker-compose.allinone.mariadb.yml up -d && docker-compose logs -f,  it showed errors as below, and the Opencast service could not start.
docker-compose-opencast-1       | Could not connect to DB
docker-compose-opencast-1       | Run opencast_main_init
docker-compose-opencast-1       | Run opencast_tz_set
docker-compose-opencast-1       |   Setting timezone to UTC
docker-compose-opencast-1       | Found custom config in /etc/opencast
docker-compose-opencast-1       | Run opencast_main_sync_config
docker-compose-opencast-1       | Run opencast_main_configure
docker-compose-opencast-1       | Run opencast_opencast_configure
docker-compose-opencast-1       | Run opencast_elasticsearch_configure
docker-compose-opencast-1       | Run opencast_db_configure
docker-compose-opencast-1       | Run opencast_mariadb_configure
docker-compose-opencast-1       | Run opencast_jdbc_configure
docker-compose-opencast-1       | Run opencast_db_trytoconnect
docker-compose-opencast-1       | Run opencast_jdbc_trytoconnect
docker-compose-opencast-1       | Try to connect to DB (1/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (2/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (3/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (4/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (5/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (6/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (7/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (8/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (9/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (10/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (11/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (12/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (13/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (14/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (15/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (16/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (17/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (18/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (19/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (20/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (21/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (22/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (23/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (24/25) FAILED
docker-compose-opencast-1       | Try to connect to DB (25/25) FAILED
docker-compose-opencast-1       | Could not connect to DB

@fishfree thank you for the report. I will look into it.

Apart from that your example command will not work (should be
docker-compose -p opencast-allinone -f docker-compose/docker-compose.allinone.mariadb.yml up -d && docker-compose -p opencast-allinone -f docker-compose/docker-compose.allinone.mariadb.yml logs -f

or simply
docker-compose -p opencast-allinone -f docker-compose/docker-compose.allinone.mariadb.yml up

), I cannot reproduce the error. Do you see log output from MariaDB?

I'm closing this for now. Feel free to reopen if this is still an issue.


change owner test done

Hi @nuggetnchill
Please review and merge. This is a major feature so please take your time with this--I've reviewed it extensively but please double check.
This is, ofcourse, lower priority than Danny's PR.
Thank you.


WIP: Add sell NFT UI + API

[x] Adds new sell NFT page (fields will include: reserve price, duration, and wallet passphrase)
[x] Adds API endpoints to create/setup smart contract auction
[x] Updates txn confirm task to handle auction app statuses
[ ] Updates/adds collectible status reflecting it's currently for sale (to avoid creating duplicate concurrent auctions)
[ ] Checks that the collectible is eligible for sale (i.e. X days have passed since initial CC purchase and that ASA is not frozen)

For separate PRs:

Auction cancellation
Bidding flow
Background task to close expired auctions
Listing of NFTs that are for sale in secondary marketplace

Closing this as other/better work has been done that obsoletes this. Will keep the branch around for a while as a reference.


feat(connection): add initialization configuration capabilities for data sources

What type of PR is this?
type-feature

What this PR does / why we need it:
this pr add initialization configuration capabilities for data sources.
User can give a sql script that executed when jdbc connection created to init the connection.
eg. if you want to set your sql execution timeout, you can give a sql script like this:
set session ob_query_timeout=60000000;
set session ob_trx_timeout=60000000;

odc will execute them one by one when you want to create a session on this datasource.
odc connects to observer by jdbc, if you want to add some your custom jdbc parameters, you can to this by this pr. odc will append your custom jdbc parameters to the url.
this init config will work in every session creation scenario.
Which issue(s) this PR fixes:

Fixes #93
Special notes for your reviewer:
I create a new table named connec_connection_attribute, this table will be not only used by this feature, but also used in other scenario.
the SessionCreatedInitializer is used to execute user's custom sql init script, and OBConsoleDataSourceFactory#getJdbcParams is used to append users' custom jdbc parameters to odc's default parameters, odc's jdbc url parameters got the highest priority.
this init for connection will work in two scenarios:

session creation: if you want to create a session for a datasource, this init will work.
datasource test: when you test a datesource,  these init measures will also work.

Additional documentation e.g., usage docs, etc.:

Does this pr also support native mysql?

sure


INITIALIZE POD: Alert layout improvements

Description
The Alert box in the InitialisePods() could be better laid out and support a scrolling widget.
In the below screenshot, there is a lot of empty space on the left inside the Alert box that does not need to be there.
On my Linux desktop and Android version I can't seem to scroll the Alert box to see the full message.

make the text scrollable, font size smaller, left and right padding smaller , title in the center

https://github.com/anusii/solidpod/pull/84 PR created

merged , can be closed


pip-tools can't handle setup with functions as kwargs

I have a setup.py where the setup function has a few arguments filled in with outputs from function calls, for example:
setup(
    name='MyPackage',
    description='Description',
    version=find_version(),
    ...

But when I call pip-compile on this project, I get things like NameError: global name 'os' is not defined. This error goes away when I comment out these function calls.
It looks like pip-compile is doing something that tries to fill in these values, but doesn't load my imports first (like import os), so the execution fails.
Environment Versions

OS Type: MacOS Latest
Python version: Python 2.7.15 :: Anaconda, Inc.
pip version:  pip 18.1
pip-tools version: pip-compile, version 3.1.0

I believe this stems from pip-compile's use of distutils to run setup.py, which was broken in python versions up through 3.5.

I'll close this based on the above, but please let us know if it doesn't resolve your issue. Thanks!


Remove pending DOI message from humanatlas.io

7th Release DOIs have been fully minted and you can now remove / comment out the pending DOI message.

@bherr2 removed the pending message card


Login with HybridAuth does not work

Trying to login with one of the social networks leads to the following Error.
2017-09-20 16:53:38 NOTICE Undefined property: Auth\Controller\IndexController::$serviceLocator (errno 8) in /var/www/zf3.yawik.org/module/Auth/src/Auth/Controller/IndexController.
2017-09-20 16:53:38 EXCEPTION Call to a member function get() on null in /var/www/zf3.yawik.org/module/Auth/src/Auth/Controller/IndexController.php on line 211
2017-09-20 16:53:38 ERR Call to a member function get() on null (errno 1) in /var/www/zf3.yawik.org/module/Auth/src/Auth/Controller/IndexController.php on line 211

@kilip can you take a look please

This works now. Thanks a lot


路径包含汉字[Bug]

Search before asking

[X] I searched in the issues and found nothing similar.

Version
windows11
iotdb 1.0.1
Describe the bug and provide the minimal reproduce step
insert into root.ln.汉字(timestamp,status) values(1,true)
Msg: 305: org.apache.iotdb.commons.exception.IoTDBException: Unexpected errors: null
What did you expect to see?
Msg: The statement is executed successfully.
What did you see instead?
Msg: 305: org.apache.iotdb.commons.exception.IoTDBException: Unexpected errors: null
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

https://github.com/apache/iotdb/pull/10051 fixed this bug.


Is login hint available under the new Google Identity Services library?

Sorry for posting an issue here, but I couldn't find any repo for the new library.
I'm making an app that connects to a google calender.
I would like make the sign-up and connecting to calender flow as seamless as possible.
My understanding is that this can be done with "login hints":
From the docs at https://developers.google.com/identity/oauth2/web/guides/how-user-authz-works

Adding a hint during authorization initialization--typically the email address of the user's Google Account--enables Google to skip the display of an account chooser, saving users a step. The ID Token credential returned by Sign In With Google contains the user's email address.

From past issues and digging around in the source, it looks I can run the "sign in with google" flow, and then pass an email from that to the authorisation flow like so:

This does redirect to a link with "login_hint=EMAIL_FROM_LOGIN_FLOW", but still displays the account selection.
Am I doing something wrong, or is this feature not yet supported in the new library?
Much thanks,
Jarred

Hi Jarred,
The hint property of TokenClientConfig  provide a hint to Google on what account to display in the consent prompt. For example, if the user has multiple Google accounts signed in with their Chrome browser, instead of showing all the accounts in the account chooser page, with hint set to an email address (most likely obtained from "Sign in with Google"), only that account would show up in the account chooser page.
If you want to skip the account chooser page for user who grant the consent previously, I suggest using the combination of hint set to your user email account and prompt set to '' (empty string). In this way, the user will be prompted only the first time your app requests access. A pop-up would still be shown to user as a notification to the user and closed itself, but user would not need to click or confirm the account to grant permission again.
Thanks,
Herman

Your code sample uses a "hint" property, though the documentation specifies the property name as "login_hint". Nevertheless, it sounds like you're still getting the proper URL param so that's apparently not the problem.
I have tried to bypass the account chooser by setting login_hint to both the user email and the id token sub field. Neither seems to work; I still get the account chooser as you do. I tried both the "popup" and "redirect" ux modes.
I can avoid the account chooser if I use OAuth2Client.generateAuthUrl() on the server side with a login_hint. So that works but isn't ideal because it only supports the redirect flow.


'MODULE NOT FOUND' Error when using removeTrait.js and provenance.js

inputting either (node utils/removeTrait.js "Trait Name") or (node utils/provenance.js) creates an error code where the module is not found.
I used (node step2_spritesheet_to_generative_sheet/utils/removeTrait.js "Trait Name") to help direct the code; however the same error message came up.
Using (make provenance) created a similar error message.
I don't believe these issues are a problem with the engine but, something I may have missed.
Thanks in advance

@lukarabonne regarding the provenance, you need to run make all before you run make provenance. I'll look into removeTrait I'm not sure I actually tested it I might have just copied it from nft chef and haven't tested it with my engine

@lukarabonne I fixed provenance and remove trait sorry about that. See Readme https://github.com/jalagar/animated-art-engine#remove-trait. Let me know if you have any issues!


Local module not replacing Global module

I have several related but distinct applications, all of which use webpack. This is my file structure:

node_modules
webpack, et al

babel, et al

app1

webpack.config.js

category

app2

webpack.config.js

app3

webpack.config.js

The any of those webpack.config.js files can require('webpack') with no issue, however the global webpack is unable to find it, as it is looking in `process.cwd()/node_modules/webpack...".
Essentially, the desired effect would be to execute require('webpack') from the cwd. To that end, I have a possible solution, by creating a new fake module (in memory only) at process.cwd(), then executing a require on it, and making sure the returned module is not the current one.
var path = require("path");
var Module = require("module");

try {
    var localModule = new Module();
    try {
        localModule.load( path.join(process.cwd(), 'dummy') );
    }
    catch (e) {}

    var localWebpack = localModule.require(path.join("webpack", "bin", "webpack.js"));
    if(localWebpack !== module.exports) {
        return localWebpack;
    }
} catch(e) {}

Would you like to send a PR making this change?

I had considered this and normally would, but this feels a bit hackish to me. I was hoping somebody might reply with some suggestions or something. =]

Finding the nearest node_modules folder could work.

@sokra I solved it but I'm not sure which way is preferred.
The first way my own implementation finding nearest node_modules folder function named resolveLocalWebpack in 35 lines (was inspired by require.resolve doc),
the second way is simply using a new package dependency resolve in 2 lines.
And I wrote about tests in PR, maybe someone could help with that or give me the right way, currently reading all content of the test folder)


Test Connection of all components / ignore-setup flag

Introduces --test argument: Allow testing of each component.

REST-API, InfluxDB and SSH-Clients

Makes a difference between critical failure and warning

Warns if no SSH-Client / No Server is set up.
Fails if REST-API or InfluxDB isn't working
Tells whether vSnap/VADP/Cloudproxy clients were detected
Warns if SSL is disabled.

Also implements a ignore_setup functionality, regarding issue #11 .
On certain args no (connecting) setup will be done, avoiding error messages.
Warns explicitly if setup is disabled.

various minor bugfixes and improvements

It sounds a lot better now, thanks!. I'm deleting the branch.


Not Working (ver 1.6)  : RHEL 7.5

I have  below ttyd versions
My script

ttyd version 1.6.1-7f42964

ttyd  -p 8091 bash -x
-- output ----
[2020/07/06 17:21:09:6333] N: ttyd 1.6.1-7f42964 (libwebsockets 3.0.1)
[2020/07/06 17:21:09:6334] N: tty configuration:
[2020/07/06 17:21:09:6334] N:   start command: bash -x
[2020/07/06 17:21:09:6334] N:   close signal: SIGHUP (1)
[2020/07/06 17:21:09:6334] N:   terminal type: xterm-256color
[2020/07/06 17:21:09:6335] W: lws_create_context: LWS_WITHOUT_EXTENSIONS but extensions ptr set
[2020/07/06 17:21:09:6335] N:  Listening on port: -1

2, ttyd version 1.5.2-985885c

ttyd  -p 8091 bash -x
------ Output -------------------
[2020/07/06 17:21:03:8409] NOTICE: ttyd 1.5.2-985885c (libwebsockets 3.0.1)
[2020/07/06 17:21:03:8409] NOTICE: tty configuration:
[2020/07/06 17:21:03:8409] NOTICE:   start command: bash -x
[2020/07/06 17:21:03:8409] NOTICE:   close signal: SIGHUP (1)
[2020/07/06 17:21:03:8409] NOTICE:   terminal type: xterm-256color
[2020/07/06 17:21:03:8410] NOTICE:  Using foreign event loop...
[2020/07/06 17:21:03:8410] NOTICE: Creating Vhost 'default' port 8091, 2 protocols, IPv6 off
[2020/07/06 17:21:03:8411] NOTICE:  Listening on port: 8091

** Can I know how to fix this? I  need to do keymapping for xterm.js

Upgrade your libwebsockets, or try the static version from release assets:
https://github.com/tsl0922/ttyd/releases/tag/1.6.1

Thank you!
It is working okay with the  release assets.


Stop bundling the openFile-buttons in the MOZCENTRAL build

Note how both of the openFile-buttons are always hidden during viewer initialization in the MOZCENTRAL build, i.e. the built-in Firefox PDF Viewer. Despite that we still include HTML, CSS, and JavaScript code for these buttons in the build.
This patch reduces the size of the gulp mozcentral output by 1679 bytes, which isn't a lot but still cannot hurt.

/botio-linux preview

Thanks!


ARROW-12031: [C++][CSV] infer CSV timestamps columns with fractional seconds

implement support for inferring CSV timestamps columns with fractional seconds
use timestamp[ns] parser which also covers [ms] and [us] to
avoid testing 2 additional timestamp types

Thank you very much. How about adding also a Python test somewhere near https://github.com/apache/arrow/blob/master/python/pyarrow/tests/test_csv.py#L510 ?

OK, would rely on the CI to test/debug this so that I don't have to bootstrap another build/test system, hence didn't include this in the first run.

Thanks @MartinNowak . You can use Archery (or run docker-compose directly) if you don't want setup a build environment yourself.
For example, to see the list of possible Python build images:
$ archery docker images | grep python
 - conda-python
 - conda-python-dask
 - conda-python-hdfs
 - conda-python-jpype
 - conda-python-kartothek
 - conda-python-pandas
 - conda-python-spark
 - conda-python-turbodbc
 - debian-python
 - fedora-python
 - python-sdist
 - python-wheel-manylinux-2010
 - python-wheel-manylinux-2014
 - python-wheel-manylinux-test-imports
 - python-wheel-manylinux-test-unittests
 - python-wheel-windows-test
 - python-wheel-windows-vs2017
 - ubuntu-cuda-python
 - ubuntu-python
 - ubuntu-python-sdist-test

and then run one of those images using docker run <image-name>.
However, since this PR is simple, I can probably fix issues directly if you have trouble building locally.

+1, thank you. Will wait for CI before merging.

Was actually quite smooth to build and test the python package. Thanks for the good docs and well structured project.


Windows365LanguagesInstaller does currently not support 21H2

The Windows365LanguagesInstaller script does not support Windows 10 21H2.
Can this be provided or can I contribute?

If you want to fork the repo and dial in the script to support 21H2, you can.


Add set

Fixes #36
Adds combinedType 'set', for lists without duplicates

I've used pytest-unordered rather than set comparison as we specifically want to check whether duplicate items have been removed.


redraft renderer

redraft is a project to render DraftJS state to React components without having to render a readOnly editor. (this is important for our use case b/c we're rendering hundreds of messages composed with DraftJS and rendering hundreds of DraftJS editors in readOnly mode is slow)
It'd be great to create a renderer from this plugin to React components using redraft. This is what a redraft renderer looks like: (shortened example from the redraft README)
const renderers = {
  // Inlines
  inline: {
    BOLD: (children, { key }) => {children},
    ITALIC: (children, { key }) => {children},
    UNDERLINE: (children, { key }) => {children},
    CODE: (children, { key }) => {children},
  },
  // Blocks
  blocks: {
    unstyled: (children) => children.map(child =>
{child}
),
    'header-one': (children) => children.map(child =>
{child}
),
    'code-block': (children, { keys }) =>
{addBreaklines(children)}
,

  },
  // Special entities
  entities: {
    LINK: (children, data, { key }) => <Link key={key} to={data.url}>{children}/>,
  },
  // Potentially add decorators
  decorators: [],
}

This seems very possible and would complement @jpuri's awesome draft-to-markdown. (which works beautifully in tandem with this plugin by the way)
What do you think?

Redraft looks cool, I am not sure if it can take care of stuff like nested lists. But that can be made possible with small changes.


banking API which banks?

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@loveyandex, could you explain the issue ?

No description of the issue, closing.


Investigate Deployment Preview Comments

Current State
The messages that appeared in previous PRs, like #93, no longer appear after the transfer. Investigate and follow up on the issue

@cmg777 After investigating this issue, I found that aside from the deploy previews, there are actual deploy notifications that correspond to the deploy previews that have to be enabled. These can be found under Site Configurations -> Notifications -> Emails and Webhooks. Do you have all of these notifications enabled on the project?

Hi @GeoffreyXue
Only the following notifications are active

@cmg777 Interesting, it seems like it's named slightly different, which is a big trange. I think trying out the notification setup that mirrors mine above might fix the problem. If you'd like, I can also go in and modify the notifications myself - up to you!

@GeoffreyXue
Sure, please go ahead and make the necessary modifications. Feel free to ask if you require any extra credentials.

@cmg777 Yeah, I believe I will need the credentials for your Netlify site. Do you have a preferred way of sending them? Any method works for me.

@GeoffreyXue
I will send them by email. What email address would your prefer?

@cmg777<EMAIL_ADDRESS>works!

@GeoffreyXue
OK, I just sent you an email to that address.

@cmg777 Unlinking + relinking the project seems to have done the trick, the deploy previews now work after pushing a fresh commit!
Ex: #106
Marking as closed.