Uconomy broke after today's update

so i have been using Uconomy plugin with rocketMod since very long time on my servers. But after today's update it is unable to load with some missing library error.
The libraries installed on correct location. and it was working fine before the update.
here is the exception.
10/13/2023 4:44:57 PM] [Info] [registered] /pay (fr34kyn01535.Uconomy.CommandPay/pay) [10/13/2023 4:44:57 PM] [Error] mscorlib >> Caught exception resolving dependency: I18N.West, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=0738eb9f132ed756 -> Exception in DoAssemblyResolve: System.BadImageFormatException: 0x0x7f3ecef34020 File name: 'In Memory assembly'   at (wrapper managed-to-native) System.AppDomain.LoadAssemblyRaw(System.AppDomain,byte[],byte[],System.Security.Policy.Evidence,bool)   at System.AppDomain.Load (System.Byte[] rawAssembly, System.Byte[] rawSymbolStore, System.Security.Policy.Evidence securityEvidence, System.Boolean refonly) [0x0000e] in <2161c9885dd04ff291b79db9349c4b9b>:0    at System.AppDomain.Load (System.Byte[] rawAssembly, System.Byte[] rawSymbolStore, System.Security.Policy.Evidence securityEvidence) [0x00000] in <2161c9885dd04ff291b79db9349c4b9b>:0    at (wrapper remoting-invoke-with-check) System.AppDomain.Load(byte[],byte[],System.Security.Policy.Evidence)   at System.AppDomain.Load (System.Byte[] rawAssembly) [0x00000] in <2161c9885dd04ff291b79db9349c4b9b>:0    at (wrapper remoting-invoke-with-check) System.AppDomain.Load(byte[])   at System.Reflection.Assembly.Load (System.Byte[] rawAssembly) [0x00005] in <2161c9885dd04ff291b79db9349c4b9b>:0    at Rocket.Core.Plugins.RocketPluginManager.<Awake>b__12_0 (System.Object sender, System.ResolveEventArgs args) [0x00049] in <59a13feb721848c8b0cd173f6d0e820b>:0  [10/13/2023 4:44:57 PM] [Error] Could not find dependency: I18N.West, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=0738eb9f132ed756 [10/13/2023 4:44:57 PM] [Error] Failed to load Uconomy, unloading now... :System.IO.FileNotFoundException: Could not load file or assembly 'I18N.West, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=0738eb9f132ed756' or one of its dependencies. File name: 'I18N.West, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=0738eb9f132ed756'   at fr34kyn01535.Uconomy.Uconomy.Load () [0x00006] in <3c081c2693ff4a59b9facc1a1b5ac850>:0    at Rocket.Core.Plugins.RocketPlugin.LoadPlugin () [0x00033] in <59a13feb721848c8b0cd173f6d0e820b>:0  [10/13/2023 4:44:57 PM] [Info]  [unloading] Uconomy
and here are the installed libraries

Someone advised me to remove both dlls with .v4, but it didnt solved the problem.
I have checked other servers who are using uconomy, they are facing the same problem.
Did the recent update changed something? or any idea how to fix it?

It seems some of the dependency libraries need to be updated. Here's the most recent I18N dlls from Unity:
I18N.zip
The next dependency seems to be MySQL.Data, but I'm not sure which package that is from. Still looking around but if you know that'd be much appreciated - helpful for others too I think!

thank you, this solved the issue. and yes MySQL.Data wasnt needed.

Glad to hear that did the job!
Quickly putting System.Drawing here for someone else:
System.Drawing.zip

The System.Drawing.dll fixed the problems on my servers. Thank you.

So I tryed to run System.Drawning.dll and I18N.dlls at the same time and singly but it didnt fixed the ZaupShop or other plugins.
Im not rlly experienced in this area does anyone know what the issue might be?

So I tryed to run System.Drawning.dll and I18N.dlls at the same time and singly but it didnt fixed the ZaupShop or other plugins. Im not rlly experienced in this area does anyone know what the issue might be?

paste the error logs

paste the error logs
I couldnt seem to find a error after restarting the server, you can see yourself here https://youtu.be/59_Hlk5mkVw

So I tryed to run System.Drawning.dll and I18N.dlls at the same time and singly but it didnt fixed the ZaupShop or other plugins. Im not rlly experienced in this area does anyone know what the issue might be?

paste the error logs

So I tryed to run System.Drawning.dll and I18N.dlls at the same time and singly but it didnt fixed the ZaupShop or other plugins. Im not rlly experienced in this area does anyone know what the issue might be?

paste the error logs

I checked out again if theres any error logs and I found this
Error in MulticastDelegate PlayerUpdateStat: System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> System.NullReferenceException: Object reference not set to an instance of an object
and
Could not find dependency: PlayerStats, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null

how did you fix it?

The System.Drawing.dll fixed the problems on my servers. Thank you.

[loading] Uconomy
[registered] /balance (fr34kyn01535.Uconomy.CommandBalance/balance)
[registered] /pay (fr34kyn01535.Uconomy.CommandPay/pay)
Assembly-CSharp >> Caught exception resolving dependency: I18N.West, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=0738eb9f
132ed756 -> Exception in handleAssemblyResolve: System.BadImageFormatException: 0x000001E358050020
File name: 'In Memory assembly'
at (wrapper managed-to-native) System.AppDomain.LoadAssemblyRaw(System.AppDomain,byte[],byte[],System.Security.Policy.Eviden
ce,bool)
at System.AppDomain.Load (System.Byte[] rawAssembly, System.Byte[] rawSymbolStore, System.Security.Policy.Evidence securityE
vidence, System.Boolean refonly) [0x0000e] in <47fc8c70fa834cbf8141d7c1a7589125>:0
at System.AppDomain.Load (System.Byte[] rawAssembly, System.Byte[] rawSymbolStore, System.Security.Policy.Evidence securityE
vidence) [0x00000] in <47fc8c70fa834cbf8141d7c1a7589125>:0
at (wrapper remoting-invoke-with-check) System.AppDomain.Load(byte[],byte[],System.Security.Policy.Evidence)
at System.AppDomain.Load (System.Byte[] rawAssembly) [0x00000] in <47fc8c70fa834cbf8141d7c1a7589125>:0
at (wrapper remoting-invoke-with-check) System.AppDomain.Load(byte[])
at System.Reflection.Assembly.Load (System.Byte[] rawAssembly) [0x00005] in <47fc8c70fa834cbf8141d7c1a7589125>:0
at Rocket.Core.Plugins.RocketPluginManager.OnAssemblyResolve (System.Object sender, System.ResolveEventArgs args) [0x00049]
in <09b6a6e399e5417eb969381ca3201ce1>:0
Could not find dependency: I18N.West, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=0738eb9f132ed756
mscorlib >> Caught exception resolving dependency: I18N.West, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=0738eb9f132ed75
6 -> Exception in DoAssemblyResolve: System.BadImageFormatException: 0x000001E358101020
File name: 'In Memory assembly'
at (wrapper managed-to-native) System.AppDomain.LoadAssemblyRaw(System.AppDomain,byte[],byte[],System.Security.Policy.Eviden
ce,bool)
at System.AppDomain.Load (System.Byte[] rawAssembly, System.Byte[] rawSymbolStore, System.Security.Policy.Evidence securityE
vidence, System.Boolean refonly) [0x0000e] in <47fc8c70fa834cbf8141d7c1a7589125>:0
at System.AppDomain.Load (System.Byte[] rawAssembly, System.Byte[] rawSymbolStore, System.Security.Policy.Evidence securityE
vidence) [0x00000] in <47fc8c70fa834cbf8141d7c1a7589125>:0
at (wrapper remoting-invoke-with-check) System.AppDomain.Load(byte[],byte[],System.Security.Policy.Evidence)
at System.AppDomain.Load (System.Byte[] rawAssembly) [0x00000] in <47fc8c70fa834cbf8141d7c1a7589125>:0
at (wrapper remoting-invoke-with-check) System.AppDomain.Load(byte[])
at System.Reflection.Assembly.Load (System.Byte[] rawAssembly) [0x00005] in <47fc8c70fa834cbf8141d7c1a7589125>:0
at Rocket.Core.Plugins.RocketPluginManager.OnAssemblyResolve (System.Object sender, System.ResolveEventArgs args) [0x00049]
in <09b6a6e399e5417eb969381ca3201ce1>:0
Could not find dependency: I18N.West, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=0738eb9f132ed756
Failed to load Uconomy, unloading now... :System.IO.FileNotFoundException: Could not load file or assembly 'I18N.West, Version
=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=0738eb9f132ed756' or one of its dependencies.
File name: 'I18N.West, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=0738eb9f132ed756'
at fr34kyn01535.Uconomy.Uconomy.Load () [0x00006] in :0
at Rocket.Core.Plugins.RocketPlugin.LoadPlugin () [0x00033] in <09b6a6e399e5417eb969381ca3201ce1>:0
[unloading] Uconomy

@IHTPSS it looks like you need a newer version of I18N.West.dll. I think the ones I provided before were from the unityjit folder. Hopefully it's the right one:
I18N.West.zip


[Bug]: Rivet page needs to be refreshed if left running for long period.

What happened?
Rivet shows "Page is having a problem" after running for awhile. I don't have exact steps to reproduce, but have seen this happen on a canvas with just a few text nodes and nothing else.
What was the expected functionality?
Rivet should run for long periods without issue.
Describe your environment
Windows, Node 20.10.0
Relevant log output
No response
Relevant screenshots

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Here is memory usage on Windows. Just running a Rivet with an empty project and I'm not doing anything else, and memory usage is increasing at a constant rate. About 100mb per minute. This will be my steps to reproduce and I assume if I can stop the memory increasing then I've fixed the bug.
https://github.com/Ironclad/rivet/assets/50146659/ad193e1b-dccb-4761-9524-3adf090caea9

Appears to be a WebView2 related bug and I noticed (without knowing why) if you resize the Rivet window it releases the memory. Did some digging and it seems WebView2 has a 2GB limit so it doesn't matter how much Ram your PC has. I couldn't find any docs yet on reconfiguring the mem limit for WbeVIew2. I know it's not a fix but still would be nice to change it.

if you resize the Rivet window it releases the memory.

this definitely sounds like a bug in the library - you should probably try to isolate the issue and report it?

Fixed in e90d292c6013fa7be5aa8015519b5a805b004cfd


Reduce GitLab API Calls

TL;DR
We run a self-hosted GitLab free instance (12 vCPUs, 50GB RAM) with around 6500 Projects.
Wen running the analysis over the whole instance we sometimes get 502 failed to parse unknown error format: GitLab is not responding and a slow web UI with 502 errors, too.
To me it seems that the amount of API calls are responsible for that and that they maybe hit the default limits defined under Settings > Network
So one idea could be to reduce the load by excluding API calls which aren't provided in GitLab free (and to increase the allowed API calls limit.)
Detailed design
The Call https://gitlab.our-company.com/api/v4/metadata shows if it is an enterprise or free version.

Premium only API endpoints which can be excluded are:
https://docs.gitlab.com/ee/api/projects.html#push-rules
https://docs.gitlab.com/ee/api/merge_request_approvals.html#get-project-level-rules
https://docs.gitlab.com/ee/api/groups.html#hooks

Additional information
No response

Hey @mawl,
thanks for reporting the issue!

IIUC, the server response you attached suggests that we don't meet a rate limit, but rather overload the server with requests. The GitLab client should be able to handle rate limits by waiting; Could you try to set a lower limit in the settings and see if the behavior changes?

Are you aware of specific endpoints that aren't available in your free plan and are still queried by legitify (e.g. from the error log)? I noticed group hooks that might be relevant and seems to have a dedicated API call, but I wonder if there's experimental data that we can use to streamline the process

Hey @gal-legit,

Sounds like a good idea - I will test it and let you know if that helps.

Here are the premium endpoints:

2023/08/07 06:19:20 2023/08/07 06:19:20 failed to get project push rule GET https://git.our-company.de/api/v4/projects/5261/push_rule: 404 {error: 404 Not Found}
2023/08/07 06:19:58 2023/08/07 06:19:58 failed to get project approval configuration GET https://git.our-company.de/api/v4/projects/4054/approvals: 404 {error: 404 Not Found}
2023/08/07 06:19:58 2023/08/07 06:19:58 project 'XYZ' collection failed with error: GET https://git.our-company.de/api/v4/projects/4950/approval_rules: 404 {error: 404 Not Found}
2023/08/07 06:19:17 2023/08/07 06:19:17 project 'XYZ' collection failed with error: GET https://git.our-company.de/api/v4/projects/3484/hooks: 403 {message: 403 Forbidden}

I haven't found any errors saying hooks: 404 so it seems to be a permission problem which you can safely ignore, we can solve it on our side.

@mawl
I experimented with these API endpoints using a free user/group on GitLab SaaS, and the only endpoint that returns 404 is the push_rule endpoint.
I wonder if a free instance behaves differently, or maybe that's a matter of the user permissions for the specific groups.
That also makes handling the case a bit more complicated for some of these endpoints.
Waiting for your response on the rate limit approach to decide on the best response for that.

After enabling Settings > Network > User and IP rate limits

The CI job needs 43 minutes instead of 25 seconds to succeed.
But there are still 502 failed to parse unknown error format: GitLab is not responding in the error.log in the first two minutes.
And as you have said, the GitLabClient is capable of handling rate limits, as shown here:
2023/08/07 14:17:45 2023/08/07 14:17:45 failed to collect user XYZ - GET https://git.our-company.de/api/v4/users/533: 429 {message: {error: This endpoint has been requested too many times. Try again later.}}

So, are there any other ways to throttle the GitLab Client?
I have seen an option to define a custom rate limiter for go-gitlab here: https://github.com/xanzy/go-gitlab/blob/3b7542aae122ab4bd9328234c250c0a1f34aae4f/gitlab.go#L86
Maybe it would be nice to just set an option during legitify analyze to throttle the api load.
As a workaround we can group the calls by org and namespace, but it would be nice to have everything in one file.

@mawl
I pushed a branch gofri/gitlab_limits that enables custom rate limit for gitlab. You need to set both of these env vars:
GITLAB_RATE_LIMIT
GITLAB_BURST_LIMIT

which correspond to the rate.NewLimiter(rate, b) values.
could you check this out?

As a workaround we can group the calls by org and namespace, but it would be nice to have everything in one file.

I wanted to add support for a "merge" command for a while, meaning, allowing users to merge multiple distinct reports.
Would having that command encourage you separate different orgs/namespaces to different runs? I guess that's not as smooth as running all at once, but that should assist in handling cases where there's just too much data to collect without meeting rate limits. anyway, I'd be happy to hear your thoughts on this

No activity, closing this issue for now.


Large number of chunks causes OSError: [Errno 24] Too many open files

🐛 Bug
If an optimized dataset has too many chunks (can be replicated using small chunk size or reasonable chunk size with lots of data), while reading (e.g. using StreamingDataLoader), a Too many open files error is encountered at some point. This is because each chunk is mmap'ed as it is loaded, and the mmap'ed handle is never released, leading to too many open files across the system:
https://github.com/Lightning-AI/litdata/blob/7efd76197e0191c0a07c9f0583ce9aec4b80a49e/src/litdata/streaming/item_loader.py#L320-L322
One could ofcourse use ulimit to increase it, but at some point the number of chunks can really be too high.
It is easily resolved by keeping the number of mmap'ed files under a certain limit (for example by occasionally deleting some entries in the above dict, either randomly or smartly using FIFO or so).
I wanted to understand if this fix is alright, I am happy to send a PR for this if it makes sense.
To Reproduce
Steps to reproduce the behavior:

Optimize a dataset that results in a large number of chunks
Use StreamingDataset and StreamingDataLoader to load the data
At some point, OSError: [Errno 24] Too many open files will be encountered

Minimal code sample below to reproduce the issue.
Code sample
import glob
import numpy as np
import random

from pathlib import Path

from litdata import optimize
from litdata.streaming import StreamingDataLoader, StreamingDataset, TokensLoader

# Fake tokenizer
def tokenize_fn(filepath):
    yield np.array([random.randint(0, 10000) for _ in range(random.randint(100, 1000))])

def main():
    Path("fake_file.txt").touch()
    outputs = optimize(
        fn=tokenize_fn,
        inputs=["fake_file.txt" for i in range(10000)], # increase number of files if error is not encountered on a specific machine
        output_dir="./optimized/",
        chunk_bytes="10KB",
        num_workers=1,
        item_loader=TokensLoader(block_size=1024),
    )

    train_dataset = StreamingDataset(
        input_dir="./optimized/",
        item_loader=TokensLoader(block_size=1024),
        shuffle=True,
        drop_last=False,
    )

    train_dataloader = StreamingDataLoader(
        train_dataset, batch_size=1, pin_memory=False, num_workers=1, drop_last=False
    )

    total_tokens = 0
    total_batches = 0
    for sample in train_dataloader:
        total_batches += 1
        total_tokens += np.prod(sample.shape)

        print(total_batches, total_tokens)

    print("Batches:", total_batches)
    print("Tokens:", total_tokens)

if __name__ == "__main__":
    main()

Expected behavior

Environment

LitData version: 0.2.26
OS (e.g., Linux): Linux
How you installed: pip
Python version: 3.12

Hi @fdalvi, Thank you for reporting the issue.
I was trying to reproduce the issue using the provided script to understand the bug, but so far, I’ve had no luck on both macOS and Linux (default GitHub Codespace).
Could you please take another look at the provided script and verify its details?

Hi @bhimrazy
Thanks for taking a look! Can you please increase the inputs array by a factor of 10 (so 100000 'files' instead of 10000)? I think this should trigger the issue.
In the meantime I'll try to find another machine to test this on as well.
Best,
Fahim

Hi @bhimrazy
I was able to replicate the same issue on an macOS, so I checked and both my systems have a ulimit -n value of around 1024. My assumption is that in your case, these values are much higher. You can temporarily lower the value of this to confirm the bug by adding this to the top of the script:
import resource
resource.setrlimit(resource.RLIMIT_NOFILE, (1024, 1024))

I am not sure of the practical implications of increasing this value by arbitrary values to be honest, and it might need escalated privileges on shared clusters. Given that this may not be in a user's control, would it be useful to read this value (resource.getrlimit(resource.RLIMIT_NOFILE)[0]) and manage the mmap's appropriately?
Its also useful to think if we really need to have handles to each and every chunk of a huge dataset.
Best Regards,
Fahim

Thank you, @fdalvi. I was able to replicate the issue.

Hi @tchaton, Do you have any suggestions here ?

We need to close the memap in the TokensLoader. There used to be some issues but worth try it again.

The way I have currently resolved this issue locally is when the number of items inside _mmaps reaches a certain limit, I remove half its entries (randomly at the moment). However, since dictionary order is now guaranteed to be insertion order, we can just remove the first half/close those handles.
The limit can be something we supply in the TokensLoader initialization or even automatically derived from the system limit. Do you think this is a good way to go?

Yes, this is a good option. A slightly better one would be to call the item_loader.close(...) there: https://github.com/Lightning-AI/litdata/blob/main/src/litdata/streaming/reader.py#L307 and add a new method to the item loader to close the handle: https://github.com/Lightning-AI/litdata/blob/main/src/litdata/streaming/item_loader.py#L361


What about filtering that list before download?

How can we filter so that we only download  certain types of groups - like only the Office 365 Groups for example. Asking for a friend... :-)

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

ID: 0acd5ee1-4e4b-8441-8548-f97a295a039c
Version Independent ID: f0960ef0-2d14-e0f0-3dd5-40a9ae68673f
Content: Download a list of groups in the Azure Active Directory portal
Content Source: articles/active-directory/users-groups-roles/groups-bulk-download.md
Service: active-directory
Sub-service: users-groups-roles
GitHub Login: @curtand
Microsoft Alias: curtand

@CarolinaMiC
Thanks for your feedback! We will investigate and update as appropriate.

@CarolinaMiC use the filter button at the top and filter by Group type.

Then select the group type.

If you need more granular filtering you can make a request for this in User Voice. https://feedback.azure.com/

We will proceed to close this thread. If you have further questions feel free to reach out on Microsoft Q&A  or send me an email (on my profile) and we will gladly continue the discussion.

That's what I did. However, after filtering the downloaded groups included all groups...security, mail enabled security and Office 365. I will try again tomorrow to verify. On Jan 6, 2020 6:32 PM, Marilee Turscak - MSFT<EMAIL_ADDRESS>wrote:@CarolinaMiC use the filter button at the top and filter by Group type.
Then select the group type.
If you need more granular filtering you can make a request for this in User Voice. https://feedback.azure.com/
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.


Added loading catalog in CRM after ICML generation

Codecov Report

Merging #193 (572204f) into master (c27a932) will decrease coverage by 0.09%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #193      +/-   ##
============================================
- Coverage     30.52%   30.43%   -0.10%
- Complexity     2628     2634       +6
============================================
  Files           114      114
  Lines          8120     8145      +25
============================================
  Hits           2479     2479
- Misses         5641     5666      +25

Impacted Files
Coverage Δ

retailcrm/lib/RetailcrmCatalogHelper.php
10.63% <0.00%> (-6.03%)
:arrow_down:

retailcrm/lib/events/RetailcrmIcmlEvent.php
0.00% <0.00%> (ø)

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


Update go modules we depend on to their latest versions

There should be no functional/user-visible changes.

/hold
Sync openshift/k8s modules with oc:release-4.9 makes go 1.16 a requirement as the updated k8s/client-go requires go 1.16 because it uses fs.io. Our CI is still using golang 1.15. Since there's a released RHEL version with golang 1.16, it should be fine to switch to the newer go version.

/retest

Looks like this needs changes in github.com/openshift/release, at least
diff --git a/ci-operator/config/code-ready/crc/code-ready-crc-master.yaml b/ci-operator/config/code-ready/crc/code-ready-crc-master.yaml
index 739eb9b789..25b1ce7ca5 100644
--- a/ci-operator/config/code-ready/crc/code-ready-crc-master.yaml
+++ b/ci-operator/config/code-ready/crc/code-ready-crc-master.yaml
@@ -15,6 +15,10 @@ base_images:
     name: release
     namespace: openshift
     tag: golang-1.15
+  openshift_release_golang-1.16:
+    name: release
+    namespace: openshift
+    tag: golang-1.16
 binary_build_commands: make cross
 build_root:
   image_stream_tag:
@@ -25,9 +29,9 @@ images:
 - dockerfile_path: images/openshift-ci/Dockerfile
   from: base
   inputs:
-    openshift_release_golang-1.15:
+    openshift_release_golang-1.16:
       as:
-      - registry.svc.ci.openshift.org/openshift/release:golang-1.15
+      - registry.svc.ci.openshift.org/openshift/release:golang-1.16
   to: crc
 resources:
   '*':

/hold till CI issue resolve.

https://github.com/openshift/release/pull/23829 created this for https://github.com/code-ready/crc/pull/2842#issuecomment-971734187 one but looks like it is deadlock now, we have to merge this one to get other merged.

openshift/release#23829 created this for #2842 (comment) one but looks like it is deadlock now, we have to merge this one to get other merged.

Maybe we can temporarily use something like this in openshift/release?
From ab80e9424e4bf6dd27aa17e958bdcf897d1eb173 Mon Sep 17 00:00:00 2001
From: Christophe Fergeau<EMAIL_ADDRESS>Date: Mon, 22 Nov 2021 12:28:52 +0100
Subject: [PATCH 1/2] crc: Drop golang-1.13

---
 ci-operator/config/code-ready/crc/code-ready-crc-master.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ci-operator/config/code-ready/crc/code-ready-crc-master.yaml b/ci-operator/config/code-ready/crc/code-ready-crc-master.yaml
index 0ba6313fe7..8246401df9 100644
--- a/ci-operator/config/code-ready/crc/code-ready-crc-master.yaml
+++ b/ci-operator/config/code-ready/crc/code-ready-crc-master.yaml
@@ -12,7 +12,7 @@ build_root:
   image_stream_tag:
     name: release
     namespace: openshift
-    tag: golang-1.13
+    tag: golang-1.15
 images:
 - dockerfile_path: images/openshift-ci/Dockerfile
   from: base
--
2.33.1

From affaaf1f408c5f0a2cad4dd892e079e40e5d0ce8 Mon Sep 17 00:00:00 2001
From: Christophe Fergeau<EMAIL_ADDRESS>Date: Mon, 22 Nov 2021 12:30:19 +0100
Subject: [PATCH 2/2] crc: Tag golang 1.16 in addition to 1.15

This should allow to keep CI green while crc is being updated to the
1.16 image. Once that is done, we can drop the 1.15 tagging.
---
 .../config/code-ready/crc/code-ready-crc-master.yaml       | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/ci-operator/config/code-ready/crc/code-ready-crc-master.yaml b/ci-operator/config/code-ready/crc/code-ready-crc-master.yaml
index 8246401df9..544ec1da37 100644
--- a/ci-operator/config/code-ready/crc/code-ready-crc-master.yaml
+++ b/ci-operator/config/code-ready/crc/code-ready-crc-master.yaml
@@ -7,6 +7,10 @@ base_images:
     name: release
     namespace: openshift
     tag: golang-1.15
+  openshift_release_golang-1.16:
+    name: release
+    namespace: openshift
+    tag: golang-1.16
 binary_build_commands: make cross
 build_root:
   image_stream_tag:
@@ -20,6 +24,9 @@ images:
     openshift_release_golang-1.15:
       as:
       - registry.svc.ci.openshift.org/openshift/release:golang-1.15
+    openshift_release_golang-1.16:
+      as:
+      - registry.svc.ci.openshift.org/openshift/release:golang-1.16
   to: crc
 releases:
   initial:
--
2.33.1

openshift/release#23829 created this for #2842 (comment) one but looks like it is deadlock now, we have to merge this one to get other merged.

Maybe we can temporarily use something like this in openshift/release?
From ab80e9424e4bf6dd27aa17e958bdcf897d1eb173 Mon Sep 17 00:00:00 2001
From: Christophe Fergeau<EMAIL_ADDRESS>Date: Mon, 22 Nov 2021 12:28:52 +0100
Subject: [PATCH 1/2] crc: Drop golang-1.13

---
 ci-operator/config/code-ready/crc/code-ready-crc-master.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ci-operator/config/code-ready/crc/code-ready-crc-master.yaml b/ci-operator/config/code-ready/crc/code-ready-crc-master.yaml
index 0ba6313fe7..8246401df9 100644
--- a/ci-operator/config/code-ready/crc/code-ready-crc-master.yaml
+++ b/ci-operator/config/code-ready/crc/code-ready-crc-master.yaml
@@ -12,7 +12,7 @@ build_root:
   image_stream_tag:
     name: release
     namespace: openshift
-    tag: golang-1.13
+    tag: golang-1.15
 images:
 - dockerfile_path: images/openshift-ci/Dockerfile
   from: base
--
2.33.1

From affaaf1f408c5f0a2cad4dd892e079e40e5d0ce8 Mon Sep 17 00:00:00 2001
From: Christophe Fergeau<EMAIL_ADDRESS>Date: Mon, 22 Nov 2021 12:30:19 +0100
Subject: [PATCH 2/2] crc: Tag golang 1.16 in addition to 1.15

This should allow to keep CI green while crc is being updated to the
1.16 image. Once that is done, we can drop the 1.15 tagging.
---
 .../config/code-ready/crc/code-ready-crc-master.yaml       | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/ci-operator/config/code-ready/crc/code-ready-crc-master.yaml b/ci-operator/config/code-ready/crc/code-ready-crc-master.yaml
index 8246401df9..544ec1da37 100644
--- a/ci-operator/config/code-ready/crc/code-ready-crc-master.yaml
+++ b/ci-operator/config/code-ready/crc/code-ready-crc-master.yaml
@@ -7,6 +7,10 @@ base_images:
     name: release
     namespace: openshift
     tag: golang-1.15
+  openshift_release_golang-1.16:
+    name: release
+    namespace: openshift
+    tag: golang-1.16
 binary_build_commands: make cross
 build_root:
   image_stream_tag:
@@ -20,6 +24,9 @@ images:
     openshift_release_golang-1.15:
       as:
       - registry.svc.ci.openshift.org/openshift/release:golang-1.15
+    openshift_release_golang-1.16:
+      as:
+      - registry.svc.ci.openshift.org/openshift/release:golang-1.16

Well in this case release repo test will happen with 1.15 not 1.16 so I am in favor to merge first on release side and then here so we know our changes on release repo is correct one.

to: crc
releases:
initial:
2.33.1

Well in this case release repo test will happen with 1.15 not 1.16 so I am in favor to merge first on release side and then here so we know our changes on release repo is correct one.

Not clear what release repo tests you are referring to? Imo it's fine to do things in steps, first keep using 1.15 in release, but allow to use 1.16, then update crc to use 1.16, and finally drop 1.15 from release. "release repo tests" will be done with 1.16 at the latest step at worse, and while we transition our branches to 1.16, CI will keep working as before.

Well in this case release repo test will happen with 1.15 not 1.16 so I am in favor to merge first on release side and then here so we know our changes on release repo is correct one.

Not clear what release repo tests you are referring to? Imo it's fine to do things in steps, first keep using 1.15 in release, but allow to use 1.16, then update crc to use 1.16, and finally drop 1.15 from release. "release repo tests" will be done with 1.16 at the latest step at worse, and while we transition our branches to 1.16, CI will keep working as before.

@cfergeau This make sense also, do you have PR on release branch which we can review/merge?

@cfergeau This make sense also, do you have PR on release branch which we can review/merge?

Done in https://github.com/openshift/release/pull/23888
I need to respin this PR to use registry.ci.openshift.org rather than registry.svc.ci.openshift.org.

/cherry-pick dev


Checking of a string in pattern.regex

You use the check_pattern_startswith_slash method on a string and the whole thing breaks.
First you have to look if this "pattern" variable is a RegexURLPattern and then you can access in the method check_pattern_startswith_slash
pattern.regex.pattern

Closed because of stupidity


Can not publish to two topics

When I try to publish to two topics, only the first topic is published:
if (mqttClient.publish("topic1, "payload1")) Serial.println("Publish1 ok");   else Serial.println("Publish1 failed");
if (mqttClient.publish("topic2, "payload2")) Serial.println("Publish2 ok");   else Serial.println("Publish2 failed");
The second publish return true.

Any movement on this?

Not sure what to suggest. I am able to publish multiple times without any problems.

It was my bad coding.
Now it works fine.
Thanks


sPDF setPage() does not work in case of html(), renders html source always into first page with xOffset and yOffset 0

Hi,
I need to add panel contents (Selection Criteria) to each page in PDF (header panel). I am using the new html() method to do this. I am trying to set the current page using setPage() function but the output PDF has all the header panels in first page with xOffset and yOffset 0 and hence overlapped on each other.
jsPDF v1.5.3
Please refer fiddle: https://jsfiddle.net/nzgr2a7y/3/
The header panel is coming twice but in page 1. It should be present in each page once. Also is there any way to set xOffset and yOffset to html() like in options. The options does not have these properties defined in the function definition.
As fromHTML(), addHTML() and html2pdf() are deprecated, using html() which is working fine with CSS and everything apart from this issue. Any help would be highly appreciated. Thank you.
Regards

Duplicate of #1176


Error when there is no database

Larastan Version: 0.6.4
--level used: max

Description
I'm using spatie/laravel-permission package in my app. That registers some stuffs in service provider. When i setup a pipeline this error occured.
Similar issue: https://github.com/symfony/symfony/issues/37069
Error Output
------ ---------------------------------------------------------------------
  Line   Http/Controllers/Api/RoleController.php
 ------ ---------------------------------------------------------------------
         Internal error: SQLSTATE[HY000] [2002] Connection refused (SQL:
         select column_name as `column_name` from information_schema.columns
         where table_schema = forge and table_name = roles)
         Run PHPStan with --debug option and post the stack trace to:
         https://github.com/phpstan/phpstan/issues/new
 ------ ---------------------------------------------------------------------
 ------ ---------------------------------------------------------------------
  Line   Http/Resources/PermissionResource.php
 ------ ---------------------------------------------------------------------
         Internal error: SQLSTATE[HY000] [2002] Connection refused (SQL:
         select column_name as `column_name` from information_schema.columns
         where table_schema = forge and table_name = permissions)
         Run PHPStan with --debug option and post the stack trace to:
         https://github.com/phpstan/phpstan/issues/new
 ------ ---------------------------------------------------------------------
 ------ ---------------------------------------------------------------------
  Line   Rules/EmptyRole.php
 ------ ---------------------------------------------------------------------
         Internal error: SQLSTATE[HY000] [2002] Connection refused (SQL:
         select column_name as `column_name` from information_schema.columns
         where table_schema = forge and table_name = roles)
         Run PHPStan with --debug option and post the stack trace to:
         https://github.com/phpstan/phpstan/issues/new
 ------ ---------------------------------------------------------------------

Laravel code where the issue was found
class PermissionController extends Controller
{
    public function index(): JsonResponse
    {
        $permissions = Permission::all();

        return new JsonResponse([
            'data' => $permissions->pluck('name'),
        ]);
    }
}

Another example
ConnectionFactory reads Salesforce config from .env and creates a soap client.
    public function register(): void
    {
        $this->app->singleton(SforceEnterpriseClient::class, static function (): SforceEnterpriseClient {
            $factory = new ConnectionFactory();

            return $factory->createConnection();
        });
    }

Error Output
 ------ ---------------------------------------------------------------------
  Line   Salesforce/Services/UserService.php
 ------ ---------------------------------------------------------------------
         Internal error: INVALID_LOGIN: Invalid username, password, security
         token; or user locked out.
         Run PHPStan with --debug option and post the stack trace to:
         https://github.com/phpstan/phpstan/issues/new
 ------ ---------------------------------------------------------------------

This occurs when you test a package for Laravel, not an application.
Testbench boots up Laravel which is not really static analysis :)

AFAIK Larastan *used to boot up Laraval but now it does real clean static analysis. @canvural ?

Hi,
Can you run PHPStan only on one file, Http/Controllers/Api/RoleController.php for example, with --debug flag and post the stack trace here?
@szepeviktor We still boot up the Laravel.

Sure.
$ php vendor/bin/phpstan analyse app/Http/Controllers/Api/RoleController.php --debug
Note: Using configuration file ./phpstan.neon.
Http/Controllers/Api/RoleController.php

   Illuminate\Database\QueryException

  SQLSTATE[HY000] [2002] Connection refused (SQL: select column_name as `column_name` from information_schema.columns where table_schema = forge and table_name = roles)

  at vendor/laravel/framework/src/Illuminate/Database/Connection.php:671
    667|         // If an exception occurs when attempting to run a query, we'll format the error
    668|         // message to include the bindings with SQL, which will make this exception a
    669|         // lot more helpful to the developer instead of just the database's errors.
    670|         catch (Exception $e) {
  > 671|             throw new QueryException(
    672|                 $query, $this->prepareBindings($bindings), $e
    673|             );
    674|         }
    675|

Maybe we need the xdebug extension to get a stack trace...

Can you show your phpstan config, maybe there is some autoloader configured?

@voku
includes:
    - vendor/nunomaduro/larastan/extension.neon
    - vendor/timeweb/phpstan-enum/extension.neon
    - vendor/phpstan/phpstan-strict-rules/rules.neon

parameters:
    paths:
        - app

    level: max

    ignoreErrors:
        - '#Dynamic call to static method#'
        - '#Access to an undefined property SObject::#'
        - '#Variable property access on SObject#'

    checkMissingIterableValueType: false

Larastan version: 0.6.4
--level: 5

I'm also using spatie/laravel-permission and got this similar error as mentioned in the issue. --debug gives the same stack trace posted in https://github.com/nunomaduro/larastan/issues/656#issuecomment-690099511
-- ---------------------------------------------------------------------------------------------------------------------------------------------------------
     Error
 -- ---------------------------------------------------------------------------------------------------------------------------------------------------------
     Internal error: Internal error: SQLSTATE[HY000] [2002] php_network_getaddresses: getaddrinfo failed: nodename nor servname provided, or not known (SQL:
     select column_name as `column_name` from information_schema.columns where table_schema = my_app and table_name = roles) in file
     app/Policies/RolePolicy.php
     Run PHPStan with --debug option and post the stack trace to:
     https://github.com/phpstan/phpstan/issues/new
     Internal error: Internal error: SQLSTATE[HY000] [2002] php_network_getaddresses: getaddrinfo failed: nodename nor servname provided, or not known (SQL:
     select column_name as `column_name` from information_schema.columns where table_schema = my_app and table_name = roles) in file
     app/Http/Controllers/Backend/RoleController.php
     Run PHPStan with --debug option and post the stack trace to:
     https://github.com/phpstan/phpstan/issues/new
 -- ---------------------------------------------------------------------------------------------------------------------------------------------------------

This may not help but we're striving to release Laravel 1.0
Currently Laravel is booted up
https://github.com/nunomaduro/larastan/issues/656#issuecomment-690098048 👀

Totally looking forward to Laravel 1.0 release 🥳
😏

I'm experiencing the same error. Here's a simple piece of code that will throw the error:
use Spatie\Permission\Models\Role;

public function doSomething(Role $role)
{
   return $role->permissions;
}

I did some debugging and I'm pretty sure this change is the culprit. Specifically, this part:
if (! isset(static::$guardableColumns[get_class($this)])) {
            static::$guardableColumns[get_class($this)] = $this->getConnection()
                        ->getSchemaBuilder()
                        ->getColumnListing($this->getTable());
}

Larastan seems to call the constructor for the Role class at some point, which will in turn end up performing a query here.
To temporarily fix this, one could use a Laravel version before this change was added. E.g. 7.23.2

In work projects, we are extending the \Spatie\Permission\Models\Role model and using our own Role model. So at first, I did not see this error.
But when I explicitly use that model like so:
public function doSomething(\Spatie\Permission\Models\Role $role): Collection
{
    return $role->permissions;
}

it produces the same error as in the original issue.
The error originates from here. The solution I'm thinking is wrapping this statement in try/catch block. And fallback to "guessing" the table name in catch block. Like this:
--- src/Properties/ModelPropertyExtension.php
+++ src/Properties/ModelPropertyExtension.php
@@ -73,9 +73,16 @@
         }

         $modelName = $classReflection->getNativeReflection()->getName();
+
         /** @var Model $modelInstance */
-        $modelInstance = new $modelName;
-        $tableName = $modelInstance->getTable();
+        try {
+            $modelInstance = new $modelName;
+
+            $tableName = $modelInstance->getTable();
+        } catch (\Exception $e) {
+            $tableName =  Str::snake(Str::pluralStudly($modelName));
+        }
+

         if (! array_key_exists($tableName, $this->tables)) {
             return false;

Or we can just return false from the catch block. Ignoring this model.
Any other suggestions?

@canvural
Seems good to me. Can't think of another solution.

@eislambey and @Daanra Can you test master and check if it fixed the issue for you?

@canvural That fixed it for me. Thanks!


Persona Import Blocking SwiftUI Environment Variables

import Persona2 is causing build errors on @Environment(\.presentationMode) var presentationMode. When removing 1 the other works.
Full error : 'Environment' is ambiguous for type lookup in this context

Sorry, you'll need to fully qualify usages of Environment since the name clashes with our Environment type. e.g. @SwiftUI.Environment(\.presentationMode) var presentationMode

got it. Any plans on adding any native SwiftUI stuff in the future?

Not at the moment but I'll note the interest, you can view a sample of how to integrate here https://stackoverflow.com/questions/73758537/integrating-persona-sdk-in-a-swiftui-view/73821670#73821670


refactor: use mutable refs in noir-contracts

Description
Refactoring:

Pass context everywhere by mutable reference
Push to arrays by mutable reference
And a small change that I initially made: moving call from PrivateCallStackItem and PublicCallStackItem to context.

Checklist:

[x] I have reviewed my diff in github, line by line.
[x] Every change is related to the PR description.
[x] I have linked this pull request to the issue(s) that it resolves.
[x] There are no unexpected formatting changes, superfluous debug logs, or commented-out code.
[x] The branch has been merged or rebased against the head of its merge target.
[x] I'm happy for the PR to be merged at the reviewer's next convenience.

Soon to be unblocked by https://github.com/noir-lang/noir/pull/2087
https://i.imgur.com/NG8airn.gif?noredirect

Only compiles with Jake's PR's branch. Need to wait until that's merged before merging this.

I like overall how we decided to expose a pretty low level interface and now we're cleaning it up as we go (versus making everything magic from the start, I suppose) LGTM at a glance, really like the change

One thing I hadn't noticed before is how some methods silently add the header to a note, which is now made explicit since they require a mutable reference to it. Does it make sense to use different types to represent notes with and without headers, to make it more clear which methods expect the header to be set, and which ones set it? Not for this PR of course.

I'll tag @LeilaWang on this comment, because Leila did a lot of the lovely work introducing headers.


Use pattern_name instead of url_name in RedirectView

Since Django 1.6 the RedirectView has an argument pattern_name. At the moment we use url_name. We should switch to the (new) standard.
See: https://docs.djangoproject.com/en/dev/ref/class-based-views/base/#django.views.generic.base.RedirectView.pattern_name

@ostcar is this still required for 2.0

no


PyNose not compatible with PyCharm build 212.*?

Will there be support for newer builds?

Hi, what is the incompatibility issue you are facing with? Currently we know that the plugin will cause an error saying "Slow operations are prohibited on EDT" in newer version PyCharm when running in GUI mode.

The GUI mode plugin only works for PyCharm versions 211.* or older, according to error message:

Hey, sorry for such a late response - we are currently working on it now, and a new GUI mode with inspections will be added soon! It will also be compatible with the new builds of PyCharm.

We have released a new GUI version of the plugin that is compatible with PyCharm 2021.3+ (since build 213.5744.248).
Feel free to try it and give us feedback!


Theme: Maintain preference across page switches

When you switch between pages, e.g. Home -> Socials -> Home, the theme you choose, is not persisted across page switching. Applies to both logged out and logged in.

this issue was closed by Chris, Harvey, and Roop


rain stops on the original sea level (idk how to call it)

im not sure this is supposed to be like that
the rain particles stop on some type of invisibe barrier   it stops at the water level that was before im made a big hole)

the funny thing is that in other place rain normally works

oh and i found another issue
and the isssue is :  no discord :(

also when you place beds in water    a block of water is being destroyed

I know that this will be for later
Will there be vs2 support. Beacuse rn when a ship is on water It sinks (the wpo Water not normal full block Water)


Add slug generator

Generate a URL safe slug. I think this could fall under the Internet namespace.
Example: this-is-a-slug
I'd be happy to tackle this if you think it would be valuable.

Yep, sounds good.
Ruby faker API looks reasonable:
# Optional arguments: words=nil, glue=nil
Faker::Internet.slug #=> "pariatur_laudantium"
Faker::Internet.slug('foo bar') #=> "foo.bar"
Faker::Internet.slug('foo bar', '-') #=> "foo-bar"

In the ruby example, I don't get why the default case with 0 arguments, the slugs are "glued" with _ and the 2nd example, glued with .. Is the "glue" random as well? I feel like - should be default.
But +1 for this addition

@tobyhinloopen if you fixed it to a - how would you randomize it if that's what you wanted?

This is my first attempt.
https://github.com/igas/faker/pull/80
I kept it pretty close to the Ruby implementation. The biggest difference is that you need to provide a list for the glue. This way we can provide a custom random glue.

@tobyhinloopen I feel that - is most common. but I feel like in this one better to have random, so ppl can find that they do not handle all the cases in specs for example, if they have their own slug format, they can get it with this. Thoughts?

@igas Sounds right 👍


unable to convert the .pb to .tflite as tflite_convert is giving me a problem

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

I am working on tensorflow-for-poets-2 and trying to convert the trained model to tflite so that i can port my model to mobile device. which i was not able to do and i am following the hands on from the
https://codelabs.developers.google.com/codelabs/tensorflow-for-poets-2-tflite/#0

python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" >> 1.12.0
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
Other info / logs
my log when executing the code:
(tensorflow) C:\Users\H156759>python
Python 3.6.7 |Anaconda, Inc.| (default, Dec 10 2018, 20:35:02) [MSC v.1915 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.

import tensorflow as tf
tf.version
'1.12.0'
exit()

(tensorflow) C:\Users\H156759>tflite_convert --help
Traceback (most recent call last):
File "C:\Pavans\Anaconda3\envs\tensorflow\Scripts\tflite_convert-script.py", line 6, in
from tensorflow.contrib.lite.python.tflite_convert import main
ModuleNotFoundError: No module named 'tensorflow.contrib.lite.python.tflite_convert'

Hi yaxiongma,
you mean to say that i should try tflite on Linux and this will not work on windows ? because my execution environment that i am trying is on Windows currently.

If all the ops in your graph are supported by the current TFLite, you can do this successfully in linux environment, and you'd better choose 1.12.0. Don't forget to Check the TensorFlow Lite & TensorFlow Compatibility Guide
Good luck~

Hi yaxiongma,
you mean to say that i should try tflite on Linux and this will not work on windows ? because my execution environment that i am trying is on Windows currently.

@pavanjava
Yes.
I haven't tried it in Windows.
good luck


Discriminator tutorial

Summary
Moving discriminator tutorial from qiskit-ignis to qiskit-iqx-tutorials
Details and comments
The ignis discriminator tutorial was previously in qiskit-ignis. However, it should live in qiskit-iqx-tutorials.

Missing copyright and version table, but good otherwise.

@nonhermitian Added copyright and version table. Updated code to remove deprecation warning, ran NB from top to bottom.


[tl.random]: Add support for Triton's tt.random.* operations

Test cases for the tt.random.* operation are in test_random.py which needs to be ported to XPU.

Fixed by pull request https://github.com/intel/intel-xpu-backend-for-triton/pull/184


bug: mock connector defaults to chainId one despite the RPC being for another network

Is there an existing issue for this?

[X] I have searched the existing issues

Package Version
0.6.8
Current Behavior
Auto connecting with mock connector seems to default to chainId 1 despite the connected RPC being for 137, this seems to make the auto connect behavior different from the regular connectors.
Expected Behavior
Mock connector to match behavior of regular connectors, which is to auto connect to the same chain id as the current RPC.
Steps To Reproduce
Use mock connector with Polygon RPC
Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
No response

Also experiencing the same with Aurora with the auto connect feature. The signer ends up defaulting to chainid 1 for some reason.
We use the following config for the Aurora chain for reference:
const AURORA_CHAIN: Chain = {
  id:<PHONE_NUMBER>,
  name: 'Aurora Mainnet',
  network: 'aurora',
  nativeCurrency: {
    decimals: 18,
    name: 'Ethereum',
    symbol: 'ETH',
  },
  rpcUrls: {
    default: 'https://mainnet.aurora.dev',
  },
  blockExplorers: {
    default: { name: 'Aurorascan', url: 'https://aurorascan.dev/' },
  },

  testnet: false,
}

Here's also the config for the connector for reference:
const { chains, provider, webSocketProvider } = configureChains(
  [chain.mainnet, AURORA_CHAIN],
  [
    jsonRpcProvider({
      rpc: (chain) => { return { http: chain.rpcUrls.default }},
    }),
  ]
)
const client = createClient({
  autoConnect: true,
  connectors: [new InjectedConnector({ chains })],
  provider,
  webSocketProvider
})

This issue seems to be present on 0.8.8 but not on 0.8.4

@tmm The issue occurs when you use custom chains like Aurora.
Steps to reproduce:

Create a new project using the wagmi-template
Add the following chain in the config

const AURORA_CHAIN: Chain = {
  id:<PHONE_NUMBER>,
  name: 'Aurora Mainnet',
  network: 'aurora',
  nativeCurrency: {
    decimals: 18,
    name: 'Ethereum',
    symbol: 'ETH',
  },
  rpcUrls: {
    default: 'https://mainnet.aurora.dev',
  },
  blockExplorers: {
    default: { name: 'Aurorascan', url: 'https://aurorascan.dev/' },
  },

  testnet: false,
}

Connect your wallet and console log the signer that you get from useSigner. Make sure it's not connected to Ethereum mainnet, any chain would do.
Refresh the page and check the chain the signer is connected to. It would default to chainID 1 instead of the current selected chainID.

Note: The issue is still present in 0.8.10. Would really appreciate some response here.

Opened up a PR for this that follows the behavior of other connectors in wagmi.
Could also explore making an eth_chainId call to set the chainId for MockConnector's underlying provider in the future, but this would deviate from the behavior of other connectors so holding on that for now.


Add cache for common package queries

This adds a cache for common package queries in GetPackageDescriptor. Code which needs to process a list of packages benefits from this change. This skips 350 queries in the package integration tests for example.

Maybe we can wait #22294

I don't think this is related because this cache should only be valid within one request. If you cache these objects longer than that you may get problems with expired/updated values.

I don't think this is related because this cache should only be valid within one request. If you cache these objects longer than that you may get problems with expired/updated values.

Yes,  it's the same. ContextCache means cache in one request.

Oh, my mistake. I mixed up that PR with the global cache.

I think this could be rewritten since #22294 merged.


Can't blur any videos (iOS 8.1)

I'm trying to blur a video, and have tried playing it with both AVPlayer or MPMoviePlayerController. I just see a black view. Does this library work with live video? I think it might not be able to pick up the video layer.
I also tried adding a LFGlassView directly to my ViewController's main view:

Without LFGlassView:

After further research, yeah I think this might not be possible. We settled on using the new UIVisualEffectView in iOS 8, and just reducing opacity on iOS 7.


CAMEL-18942. Copy the description of the path/operation to the genera…

@davsclaus do I need to make another PR to push this changes to 4.0?

its been cherry picked to main branch already


refactor: update default checkmk_server_version from 2.0.0p25 to v2.0.0p26 :arrow_up:

:robot: Authored by __scripts/ci.py python script on airtop-jonas by jonas from Vienna
Release Date of v2.0.0p26: 2022-06-20
GitHub Compare URL (for the interested): https://github.com/tribe29/checkmk/compare/v2.0.0p25...v2.0.0p26
This PR should result in the release of a new minor version for this role!
NOTE: There have been 12 new versions since 2.0.0p25. After this PR has been merged, the github workflow will run again and a new PR will open semi-immideally. Please ensure to create a proper tag/release for every merged ci.py PR.

Accompanying ansible-role-checkmk_agent PR: https://github.com/JonasPammer/ansible-role-checkmk_agent/pull/6

closing test pr


chore: support build.py on apple silicon macos

What problem does this PR solve?

Issue Number: close/fix/resolve/ref #xxx
What is changed and how it works?
Please describe how it works
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B
[ ] Test cases to be added
[ ] No code

Checklist:

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

thx @xmh1011


Update references to RedHat

Referring to the compiler as “the RedHat compiler” doesn’t really work anymore, following the migration to the Eclipse foundation. Do we call just say “Eclipse” instead now (e. g. “an Eclipse AST AndOp”)? Or is there some better name we can use?
Scope:

ceylon.ast.redhat, test.ceylon.ast.redhat module names
RedHatTransformer class name
documentation comments scattered around ceylon.ast.redhat and test.ceylon.ast.redhat
README
where else?

On the other hand… Ceylon isn’t Java. I don’t have the impression that there are chances of a different, independent compiler for Ceylon coming up anytime soon. So perhaps it would make sense to ignore the vendor and just talk about “the compiler AST”. Does that sound reasonable?

Yes, definitely.


Unicode magic

Fix line height for x-row containing emoji.
Fixes #154

The bug above was tested on a monitor. This doesn't seem to be an issue on my macbook display

@dotcypress @matheuss Perhaps we could bump up the line-height to 1.2em to protect from this ^ and for more readability too?

@CodeTheory @matheuss absolutely agree 👍
And 1.2em will fix this:

@dotcypress Do it i will merge when done.

@ppot Done
https://github.com/zeit/hyper/pull/1107


Redis sslprotocols=tls12 support

Azure Redis has deprecated TLS 1.0 and 1.1 and will retire these protocols in the near future.    According to Microsoft, StackExchange.Redis users need to add sslprotocols=tls12 to their connection string to support TLS 1.2.
Unfortunately, this doesn't seem to work for CacheManager.  When adding this option to our connection string, it throws back an error with the same connection string but excludes this option.   It seems like CacheManager reconstructs the string internally and doesn't support this particular option.
Here's the code we use to create the CacheManager instance:
CacheFactory.Build<string>(settings =>
	settings.WithUpdateMode(CacheUpdateMode.Up)
		.WithRedisConfiguration("redis", connectionString, redisDatabaseId)
		.WithRedisCacheHandle("redis", false)
);

Here's the redis connection string:
myrediscacheurl.net:6380,password=myredisaccesskey,ssl=True,abortConnect=False,sslprotocols=tls12
Here's the error we're getting:
Connection to 'myrediscacheurl.net:6380,password=****,ssl=True,abortConnect=False' failed.
Notice that the error excludes the sslprotocols=tls12 portion of the connection string?  Has anyone else encountered this?  Is there a known fix?

Yeah that looks like something I'd have to add to the configuration part of the Redis client.
The only work around right now would be to initializing the multiplexer yourself and pass the instance to CacheManager (there is an option to do that)

Redis .NET clients use the earliest TLS version by default on .NET Framework 4.5.2 or earlier, and use the latest TLS version on .NET Framework 4.6 or later. If you're using an older version of .NET Framework, you can enable TLS 1.2 manually
[source: https://docs.microsoft.com/en-us/azure/azure-cache-for-redis/cache-remove-tls-10-11 ]
So I guess if you are using .NET 4.6 and above it should work without changing the connection string


Band chart type for Chart JS

Hi, i've been working with chart JS for a few months now, and due to my needs i built a new chart type, the band chart, which is meant for drawing horizontal/vertical lines or bands across the chart. (see attached as an example)
do you guys think this would make a good addition to Chart JS? I've seen a few requests for something like this but not as a chart type. The file making up the chart type itself, minified is only ~3k.
also related to my development on this i added the ability to turn off labels per dataset, {label:false} and also added the ability to ignore events (so the bands don't interact with the mouse)
also i added a new hover:mode, called 'exact', like 'label', but shows you any data under the exact mouse position (also in example), not just along the x-axis
If these features would be useful, should i submit them as separate issues/pull requests?
thanks!

@nathanmal that looks awesome! Could be a great start towards a bunch of features.

thanks! yeah it was necessary for me to have bands of color on a chart, so i made it primarily for that, but it has other uses as well i think. Each datapoint can have it's own label too.
I'll fork the repo then so you guys can check out the code and if everything looks good i'll send a pull request. Would be nice to have it rolled into chartjs so i don't have to keep working off my own branch :)

Sounds like a good plan! Feel free to join our slack channel as well.

Cool, what's your slack channel? chartjs.slack.com? I haven't used slack yet but been wanting to try it. I think one of the moderators has to add my email<EMAIL_ADDRESS>thanks!

@nathanmal there should be a button to join in the readme.

I know that +1 is frown upon, but having band chart type would be amazing.
Especially since v2.0 can mix multiple chart types.

@nathanmal that looks really awesome. Great work!

@nathanmal is this chart type an extension? And if so, any chance to try it with the current release?

Heya yes unfortunately i had to take a break for a while, but i'm back and i've updated the chart type for the new Charts.js 2.0
Have a look: https://jsfiddle.net/nathanmal/f2trua1f/
still some bugs to be worked out i think, i'll post this in the slack dev group

@nathanmal that is really really nice! Do you have it as an extension? I would like to use & test it and help to find bugs/fix them.

Thanks. Where can I see more examples? I need to create first chart

Can you give me advice? I need your halp

Where can I find documentation for adding sections (or vertical lines) in a line chart?

Closing since this a new chart type that would better live as an extension to Chart.js rather than in the core.
Some of the enhancements documented here have been implemented in the core such as the new tooltip mode (PR #3400)
There is also an annotation plugin for adding arbitrary lines and rectangles to the chart. Using a plugin like this eliminates the need to have objects that are filtered out of labels, etc.

Okay, sounds good. I'm a little disappointed, i would have loved to help develop this into something or discuss if any of these ideas would be useful, i did post in slack a few times and sent nick email but could not get a response from anyone..  :/ so i kinda gave up. Thx will check out the annotation plugin.

@nathanmal I'm sorry that no one replied on Slack. That is best place for discussing this.
In terms of the annotation plugin, I'm definitely happy to merge PRs there. If you're interested in helping to maintain it, I am totally open to that as well

all good, I'm a big fan of the library and really appreciate all the work you guys put into it so far. i'll def take a look through the plugin and work on updating my charts using that as opposed to my own branch/chart type. I'd be happy to help you maintain the code once i get more familiar with it, and i'll just DM you in slack any questions or ideas that pop up. Thanks!

Hi !
Is this new band graph available now with ChartJS ?
I've been looking for something like this for sometime and it looks really great !


Lot View for iPad and small screens

Continues from #1519. This PR adds support for iPad and 3.5&4" screens. Here's what it looks like:

Nice.
There are some problems with animating to the current lot, I've disabled the animations to get things working for now and left a TODO to get to it later.
Side note: @orta I think as a mvp we could use traitCollectionDidChange: on the top-level auction view controller and just call the setup code, like we do for registration status changes. It's not perfect but it'd be fast and it would work.
Fixes #1085.

Great, and well tested

Yeah, all and all looks good to me 👍


[normalize.css]: Review whether normalize is outdated as a CSS reset file

Summary
Normalize.css is the CSS reset used within ICDS but it hasn't been updated for 5 years. This ticket is to review and update to a new CSS reset if normalize.css no longer covers the required styles.
💬 Description
Normalize is used to reset all the browser set styling so there is a better, consistent style across all browsers. At the moment, Normalize has not been updated for 5 years so it is worth while investigating if a more modern, up to date CSS reset file would be more beneficial.
💰 Use value
As browsers develop, more default browser styles may be introduced which Normalize.css does not cover. This could potentially cause styling issues between browsers. By using a more up to date, more recently maintained CSS reset file, there is confidence the CSS reset will cover all bases.
Additional info
Tell us anything else useful to help us understand your suggestion.

Include a guidance change to update the instructions

Output to be a ticket to action, let me know once done to bring it into sprint

Did some research about our options:
Do we move to modern-normalize?
Do we stay where we're at and make changes when needed?
Do we remove normalize?
Both modern-normalize and removing normalize introduced display issues in the components and would require changes to per-component or global CSS.
We decided to not change anything right now, and to keep using normalize.
We understand that changing our approach in future won't be a breaking change since the impact of normalize.css is all scoped inside component CSS files.


Add support for NOTIFY message

NOTIFY message provides information about voicemail
waiting. add onMwiInfo callback that can be used
to get information from NOTIFY message so that message
waiting info can be handled/displayed.

@mscdex just wondering if you will be willing to pull this in/update npm ?

I have noticed that the code to parse the "mwiConfig" in the acount already seems to be there so I'll take that out and then update the PR once I see if you have any other comments.

@mscdex is this still active ?

Unfortunately not really. My original use case for this ended up having problems with the addon mostly due to V8's lazy gc and other things causing memory use issues (even after setting the max old space size and such). That forced me to create a smaller, more limited addon just for the project's needs.

In that case maybe I'll fork and maintain for my use.


isso: <IP_ADDRESS> -> <IP_ADDRESS>

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for isso is: "A commenting server similar to Disqus"
meta.homepage for isso is: "https://posativ.org/isso/"
meta.changelog for isso is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found <IP_ADDRESS> with grep in /nix/store/zhchgpj9cjybq8pk8dc4gw1x5wyhkfzg-isso-<IP_ADDRESS>
found <IP_ADDRESS> in filename of file in /nix/store/zhchgpj9cjybq8pk8dc4gw1x5wyhkfzg-isso-<IP_ADDRESS>

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

2 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
isso

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/zhchgpj9cjybq8pk8dc4gw1x5wyhkfzg-isso-<IP_ADDRESS> \
  --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 isso https://github.com/r-ryantm/nixpkgs/archive/b8d3991ce58b2de158649f46061de5753dd8fccc.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/zhchgpj9cjybq8pk8dc4gw1x5wyhkfzg-isso-<IP_ADDRESS>
ls -la /nix/store/zhchgpj9cjybq8pk8dc4gw1x5wyhkfzg-isso-<IP_ADDRESS>/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:

isso

Maintainer pings
cc @fgaz for testing.

Result of nixpkgs-review pr 170057 run on x86_64-linux 1

1 package built:

isso

This should fix build failure (https://hydra.nixos.org/build/177428708)
ZHF: #172160


Consider to add support for IP2Location.io as a fallback

Hi there,
Would you like to add support of our service into your project? Our service provided 500 queries per day for keyless query, or 30000 queries per month if you sign up our free plan. You can read more on the result returned via our documentation.

Just for context:

IPinfo.io provides 1,000 requests per day without a token.
IPinfo provides 50,000 requests per month with a free tier token.
Universal access to a globally distributed load-balancing network regardless of access tier.
Higher accuracy driven by latency-based IP geolocation methods. (Blog post)

In addition to offering generous no-tier and free-tier access, we also have incredible uptime hours. If access or limits to our service is an issue, we are happy to look into it. Thanks.


clean.bat / clean.sh

provide script that cleans blockhain data in app folder in case it has been corrupted.

made first version of mac in<PHONE_NUMBER>9834a43139dca858efcad217199740 and a2c5817ec65ae3bb9d2c19dc768ffb0db761d958


Monitor changes in a SharePoint

Hello!
I would like to contribute to this project!
An interesting feature to have, would be able to monitor a SharePoint, on the changes made on the files existing in a folder (added, deleted, modified).
Looking forward to contribute.
C.

Sure please connect with me on LinkedIn
https://www.linkedin.com/in/g-pavan-kumar
On Wed, 3 Jul 2024 at 7:47 PM, jcamilo08 @.***> wrote:

Hello!
I would like to contribute to this project!
An interesting feature to have, would be able to monitor a SharePoint, on
the changes made on the files existing in a folder (added, deleted,
modified).
Looking forward to contribute.
C.
—
Reply to this email directly, view it on GitHub
https://github.com/iamgpavan/SharePoint-Connector/issues/2, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AOBVSPWCNTFZC76TFTILI3TZKQBX5AVCNFSM6AAAAABKJV5AH2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGM4DQNZWGI3DONA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


How many epochs are used for the two datasets?

Thank you for your work! How many epochs are used for the two datasets?

Thanks for your interest in our work! It takes ~16/7 epochs for ADE20K/COCO-Stuff.


Scale issues fix

Adding lock around calculate MAC
Changing pool size from 8k to 16k
Removal/modification  of some logs.

retest this please

retest this please

retest this please

retest this please


Add info about required nodejs version to README.md

The server uses Async Await. I think this was not supported until node 7.6, but I am not 100% sure about it.

Yes, default support for async/await was added in node 7.6.

#22


annoying people

these people are annoying and bad. please do something to block this link

There is currently nothing we can do (but we're working on it). For now just put their name into Block Player Chats

ok

there are so many of them it's impossible to block all so that's why i made this
make an option to block 2 or more ppl simontaniously and server list to see
make it easy

they are still there, currently lagging server. please look at the issue

We are working making moderation tools to handle this.


Picture is displayed in full size when clicked on.

Picture should display a standardized size, so as not not cover the description box.

fixed with latest commit


Best practices for foreign enums

I've got a couple of enums that come from other crates. While I see how I can go about wrapping foreign structs and enums that have variants with arguments in a newtype and defining those manually, the only way I see to handle a foreign enum is to explicitly write a new enum that has all of the exact values as the original and add the annotation to its impl.
foreign crate
pub enum ForeignEnum {
    One,
    Two,
    Three,
}

my crate
use foreign_crate::ForeignEnum;

#[derive(async_graphql::Enum)]
pub enum MyEnumWrapper {
    One,
    Two,
    Three,
}

impl From<ForeignEnum> for MyEnumWrapper {
    fn from(x: ForeignEnum) -> Self {
        match x {
            ForeignEnum::One => Self::One,
            ForeignEnum::Two => Self::Two,
            ForeignEnum::Three => Self::Three,
        }
    }
}

Not sure if, like with async_graphql::Object, I can provide a custom name to my wrapper enum like #[async_graphql::enum(name = "ForeignEnum"). I could always have the same enum name used and reference the foreign enum through an explicit path like this, I supposed:
#[derive(async_graphql::Enum)]
pub enum ForeignEnum {
    One,
    Two,
    Three,
}

impl From<foreign_crate::ForeignEnum> for ForeignEnum {
    fn from(x: foreign_crate::ForeignEnum) -> Self {
        match x {
            foreign_crate::ForeignEnum::One => Self::One,
            foreign_crate::ForeignEnum::Two => Self::Two,
            foreign_crate::ForeignEnum::Three => Self::Three,
        }
    }
}

thoughts
Would be nice if we had a fancy expansion macro that could be a friendly wrapper for a foreign enum.
use async_graphql::wrap_foreign_enum;

wrap_foreign_enum!(MyWrapperName, foreign_crate::ForeignEnum);

Let me think about it. 😁

#[derive(async_graphql::Enum)]
#[graphql(remote = "ForeignEnum")]
pub enum MyEnumWrapper {
    One,
    Two,
    Three,
}

With the above definition, automatically generating the following conversion function is the only thing I can do.
impl From<ForeignEnum> for MyEnumWrapper {
    fn from(x: ForeignEnum) -> Self {
        match x {
            ForeignEnum::One => Self::One,
            ForeignEnum::Two => Self::Two,
            ForeignEnum::Three => Self::Three,
        }
    }
}

Not sure if, like with async_graphql::Object, I can provide a custom name to my wrapper enum like #[async_graphql::enum(name = "ForeignEnum")

This is of course possible. Derive macros are not allowed to take parameters, but you can write them like this.
#[derive(async_graphql::Enum)]
#[graphql(name = "ForeignEnum")]
pub enum MyEnumWrapper {
    One,
    Two,
    Three,
}

Would certainly be handy in the future! I wrote all of mine by hand, which was quite painful. Maybe something like this as an optional extension that can be applied for those that want it?

I added a remote attribute to the Enum macro, it can automatically implement the conversion functions of impl From<Local> for Remote and impl From<Remote> for Local enumeration types.
If you are free, please help me add a paragraph describing this feature in define_enum.md. 🙂
Here is test code
https://github.com/async-graphql/async-graphql/blob/397650bb0df210e22d16eb0d711d0255c3a93663/tests/enum.rs#L75

I added a remote attribute to the Enum macro, it can automatically implement the conversion functions of impl From<Local> for Remote and impl From<Remote> for Local enumeration types.
If you are free, please help me add a paragraph describing this feature in define_enum.md. 🙂
Here is test code
https://github.com/async-graphql/async-graphql/blob/397650bb0df210e22d16eb0d711d0255c3a93663/tests/enum.rs#L75

Cool. Added a little info to the book per your description. See #285.


Add options menu and install process as gif.

How about showing the options menu as well as install process as an animated gif in the readme?

In fact I'd like to replace the existing GIF with a video, since I find GIFs a bit disturbing. So yes, having videos for demonstrating
a) the install process
b) the adaption of utPLSQL preferences
would be a good idea. But I would like to create a ReadMe with 3 concurrently playing GIFs...

@jgebal I've changed the README.md to show a PNG and call the animated GIF via link. This is better, but not good, since it is not possible to navigate within the animated GIF.
An alternative is to use a video. Unfortunately using a plain link to a video hosted on GitHub leads to a download. Embedding videos is not supported in Markdown which leaves two options:
a) call a Youtube URL, e.g. https://www.youtube.com/watch?v=uGs6Uoj-EqM
b) call an URL hosted on uplsql.org or similar where the video may be embedded without all the other stuff you get when calling the Youtube website (ads, autoplay, ...)
Suggestions?

It would be best to have self-hosted video but youtube will be a good-enough option for now I think.
For self-hosted option we have utplsql.org (generated from githubpages) and for utPLSQL/utPLSQL we generate documentation sub-pages.
It's far from good but it must be good-enough for now as we didn't have time to spend on the layout and format of the page itself.
We could have gh-pages branch for sqlDeveloper plugin, and keep the video(s) there.
Then we could probably embed it in the generated content for pages.

I agree that the self-hosted option is probably the best. In the meantime I use endless-looping animated GIFs.


Empty commits

It seems to be still possible to make empty commits
https://github.com/mobi3006/pierreinside

This commit https://github.com/GitbookIO/repofs/commit/1b36cdcc6243cf3c8b0b0fb53d588545fcf20b9f helps fix some cases.
There's still the problem of overwriting a file with the exact same content, that is detected as an UPDATE.


Update go version to 1.21

Summary
Due to the backwards compatible changes in go1.21, this should not break existing plugins, while allowing them to opt-into newer language features (see https://tip.golang.org/blog/toolchain for more details).
Ticket Link
None

@spirosoik @toninis Gentle reminder to review the PR :wink:

As an heads up, unfortunately this broke Calls main branch building. We'll be likely sending a PR to allow us to override the golang-version variable in the future.

Thanks for the heads up @streamer45 :+1: I've submitted https://github.com/mattermost/rtcd/pull/121 and https://github.com/mattermost/mattermost-plugin-calls/pull/536 to fix the broken pipeline.

Thanks @hanzei! To note, the 1.21 bump was actually included in a couple of PRs already but we needed a way to control which version is used to build the plugin especially as we had to prepare some dot releases. Ideally we'd be keeping it in sync with whatever version is used by the server release that ships the plugin.

Totally, I purely meant from the Calls side which should be possible now so we are good :)


Add docs for how to build

Wasm
native
cross

Is Wasm & Native not documented in https://github.com/artichoke/artichoke/blob/master/.circleci/config.yml?

@Lewiscowles1986 it is. There are some yarn scripts that make it nicer locally and I think having it in human readable markdown would be a better experience.
Do you want to take a stab at this?

[ ] wasm
[ ] native
[ ] cross

Building mruby-sys currently depends on a clang toolchain being installed + bintools + yacc + gperf + ruby.
https://github.com/artichoke/artichoke/blob/6ad780b1a9899155bc7fe9a71a5b8eb3d16acc2f/.circleci/config.yml#L48-L61
See GH-137.


[FEATURE] Creation and destruction API of App and Bus

Feature you want to implement
Our goal is the below:

To orgranize Swift to be easily recognizable at a glance.
To create or destroy a Bus and an App at any time.

How the feature is implemented
Swift was implemented in swift.swift module.
Original structure
class Swift(...):
  def __init__(...):
    self._init_bus(...)    # initialize global buses
    self._init_app(...)    # initialize apps
    self._show_frame(...)  # show frames of apps

Expected structure
class Swift(...):
  def __init__(...):
    self.load(...)  # initialize buses and apps, and show frames

  def createBus(...):
    pass

  def createApp(...):
    pass

  def destroyBus(...):
    pass

  def destroyApp(...):
    pass

Documentations
load(appInfos: dict, busNames: list)

Initialize the Swift and load the apps and buses based on the given information.
Initailize Buses and Apps.
Show frames of Apps.
Use the new creation and destruction API.

createBus(name: str)

Create a Bus.

createApp(name: str, pos: str, show: bool, ...)

Create an App and show its frames.

destroyBus(name: str)

Destroy the Bus.

destroyApp(name: str)

Destroy the App.
Be aware that we should remove it from both python and Qt.

I edited it, so please ask me if there is something you don't understand.

Great, you may start!

Oh, I have a question!
I think the argument busNames in load should be changed to a dict busInfos, since it may have a param, i.e. timeout.
How do you think about it?
@kangz12345

Right, I agree with you!

I would like to make createBus() and createApp() more stateless, i.e., they just create a bus or an app based on the given arguments (name, path, module, show, pos, etc.).
I don't think setup_env should be saved, because it is a setup environment information, which is only necessary for setup.
Only the current status, which includes the list (or dict, etc.) of apps and buses, should be tracked, I think.
Therefore, what I want for this new API is supporting a way of addition or removal of apps and buses in swift. Of course, this shouldn't allowed to apps directly (they might use it in a bad manner). However, for this issue, it only provides the API via methods Swift, and the apps do not have the Swift instance, hence cannot call the API.
Later, we would make a special messaging API for the apps (let's talk about the details later..), so the apps can request swift to create or to destroy an app or a bus. The authorization would then happen, for example, swift pops up a dialog for the confirmation from the user.

I have a problem..
In createApp, there are many arguments (e.g. path, module). Thus, it causes Pylint warning.
swift\swift.py:100:4: R0913: Too many arguments (9/5) (too-many-arguments)
swift\swift.py:100:4: R0914: Too many local variables (16/15) (too-many-locals)

Thus, how about changing the arguments into a dict?
@kangz12345

I didn't know such lint rules 🤣
In that case, let's use namedtuple or dataclasses or anything else you think good enough, since we want to force the info structure.
For example, define AppInfo and BusInfo in swift.swift module as a namedtuple or a dataclass:
info = AppInfo.from_dict(info_dict) # just an example; not necessarily this way
self.createApp(info)

In addition, I think dataclasses would be better since it is easier to define helpful methods inside.
Please let me know if you have opinions about this.
You can make another PR for this if you want.
How do you think?

Originally, I tried to use a dict!
How about making another issue about dataclasses?
Before dataclasses, I want to request PR.

Oh, very good idea. Please make a PR now, and let's make the dataclasses in the next PR.
If you want to make it, you can do it.
If you want to develop the next step (regarding to #62), I will do this (dataclass).

Oh, very good idea. Please make a PR now, and let's make the dataclasses in the next PR.
If you want to make it, you can do it.
If you want to develop the next step (regarding to #62), I will do this (dataclass).

Cool! Please make dataclasses after this issue.


Stuck on RefreshingAADProfile for long and then failed.

I tried above twice on an existing cluster with RBAC enabled. Instead of creating a new cluster, I have used az aks update-credential command. I tried this twice and in both tries, my cluster got stuck on RefreshingAADProfile for a long time and then went into failed state. Wierd is when I use az aks show command, i can see aadProfile updated with right client,server and tenant IDs of my registered apps. Would like to know how to move it out of Failed state. Thanks.

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

ID: a47c6df7-5103-1336-eb0b-68f95155e6c8
Version Independent ID: f4420b08-9b57-f581-fcb4-f069afdd5b0c
Content: Integrate Azure Active Directory with Azure Kubernetes Service
Content Source: articles/aks/azure-ad-integration.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

Wrote a blog here for anyone who wants to know how to fix this bug using resource explorer.
https://wordpress.com/post/devopsandcloud.wordpress.com/425


Fix invalid search location of JDK for arm64

Related commit: 0e9f74e35f1255cb9ec45be3d8960aad195a9f6e
Signed-off-by: Heemin Kim<EMAIL_ADDRESS>Description
JDK dependency location is invalid for arm64 architecture. This commit sets correct location for JDK dependency for arm64 architecture.
Execution failed for task ':test:logger-usage:test'.
> Error while evaluating property 'javaVersion' of task ':test:logger-usage:test'
   > Could not resolve all files for configuration ':jdk_provisioned_runtime'.
      > Failed to transform mac-17.0.4-arm64.tar.gz (adoptium_17:mac:17.0.4) to match attributes {artifactType=directory, jdk=true, org.gradle.status=integration}.
         > Could not find mac-17.0.4-arm64.tar.gz (adoptium_17:mac:17.0.4).
           Searched in the following locations:
               https://github.com/adoptium/temurin17-binaries/releases/download/jdk-17.0.4+8/OpenJDK17U-jdk_arm64_mac_hotspot_17.0.4_8.tar.gz

Issues Resolved
None
Check List

[X] Commits are signed per the DCO using --signoff
[X] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@saratvemulapalli Please take a quick look.

@heemin32 Care to backport to 2.x manually pls?

@heemin32 Care to backport to 2.x manually pls?

Will do.


When Camera2D enters tree, ensure first update is not lost

Currently, after initialising Camera2D, the camera is set to uninitialised again. The result is, the first call to _update_scroll() is used to initialise the camera and not used to update the scroll. This PR ensures that the camera is set to uninitialised before initialising the camera not after.
In addition, this PR ensures that changes to Camera2D's limits don't affect smoothed_camera_pos i.e. applies the #39567 and #2764 fix to 4c23fe602b7be8103bc611e2558f9bcfc6a3a57e included in #63083.
Fixes #63330.

Thanks!


Branding ApplyTheme Subsite

There appears to be issues with the branding extensions when applying to a subsite

Works as designed, documentation could be needed.


enh: add unit test for instanceID from providerID

What this PR does / why we need it:
Adds unit test for fun instanceIDFromProviderID in instances.go file
Special notes for reviewers:

Release note:

NONE

/ok-to-test

@dulek please check

/retest

bump @dulek @pierreprinetti

/lgtm

/lgtm cancel

/approve
@pierreprinetti can mark this with /lgtm once it's ready. Approve will stay.

Please check again @pierreprinetti

Just a last-minute doubt: https://github.com/kubernetes/cloud-provider-openstack/pull/2406#discussion_r1361824758

Excellent, thanks.
/lgtm


Add catch_all_405s or ability to define custom error handler for 405 MethodNotAllowed Error

Errors 404 and 405 are treated in a special way by Flask-Restful (see _should_use_fr_error_handler()).
In case of 404 NotFound, we can set the catch_all_404s=False flag, so that Flask's error handle will deal with the 404 NotFound exception.
There is no equivalent feature for 405 MethodNotAllowed. And so every time a request hits 405, the response will be {'message': 'The method is not allowed for the requested URL.'}. And that is even after registering custom error handler to Flask object:  app.register_error_handler(Exception, my_custom_error_handler)
It would be really useful to be able to set catch_all_405s=False, or ideally be able to define our own error handler for 405 error.
Currently the only way to specify custom 405 error is by defining a non-dynamic dictionary:
e.g.
    'MethodNotAllowed': {
        'type': 'MethodNotAllowed',
        'description': 'The method is not allowed for the requested URL.',
        'url': None,
        'data': None,
        'status': 405
    }

This however won't allow us to update the dict elements such as 'url' in the given example.

You can set a custom 405 message by redefining Werkzeug's HTTP exceptions. Flask is based on Werkzeug's exceptions. An example of overriding 405 is below:
errors = {        'MethodNotAllowed': {             'result': 'False',             'status': 405,             'message': ['This method is not allowed for the requested resource']     } } api = flask_restful.Api(errors=errors)
So if you raise a 405 or or use abort, this message that you define will be presented rather than the default response. This is also documented in flask_restful's docs. However the doc states that only the 'status' key matters, but the error name also matters, which is why I made a pull request to update this in #597

Is there any way to overwrite the Content-Type header using the method mentioned by @ashwinkumar01 ? I'd like my error responses to reflect the format of problem details in RFC7807, which uses 'application/problem+json' instead of 'application/json'.

@jakubczaplicki - thanks this works really well!

Thank you this help :)

@jakubczaplicki why is this issue marked as completed?


Adding rancher-backups usage guide document

SURE-6231

@btat All your suggestions have been addressed

@btat All your comments have been addressed.


Begin sketching out a new high-level fs API.

This is a very preliminary sketch of #83. It doesn't even compile yet, but it shows a possible high-level structure of such an API.

This is now complete enough to compile, and a lot of the API surface area is now in place, though most of the functionality isn't implemented yet.

Seems possible perhaps! I suspect this'd want to bake a bit more before going into libstd, but the internal details of libstd are up for grabs to change at any time!

Interesting idea. In addition to obviously baking :-), a few additional pieces we'd need:

A way for this fs API to use WASI directly, instead of going through the current wasi-common userspace implementation. Some of that may be as simple as just adding alternate implementations of the API under #[cfg(target_os = wasi)]. What should we do about WasiCtx though? In wasi-common, one can create multiple independent WasiCtxs and they're explicitly passed into the API. In WASI itself, there's currently just one implicit WasiCtx (or equivalent) below the platform boundary. One option is to change WASI itself to have a context argument and support independent contexts, I suppose.

Something similar to what libpreopen does within WASI libc, so that when Rust code calls File::open("/absolute/path"), we can remap it to directory fd and relative path. This should be straight-forward.

"Rust-idiomatic" wrappers on top of wasi-common's low-level API, similar to the wasi crate, using Results and slices rather than bare errnos and pointers. This is what the Rust libstd support for WASI currently uses, and it would be nice if this new fs layer could use something similar.

@cc newpavlov since this is one possible path to a libc-free libstd for WASI targets.

cc @newpavlov even, since this is one possible path to a libc-free libstd for WASI targets.

This builds, and although most of the functionality is not yet implemented, I'd like to land it and continue to iterate incrementally.

@sunfishcode sounds great to me! Feel free to merge it or I can do that if you want. Also, apologies for little to no activity lately but got swamped with preparation work for the upcoming Devcon5 conference :-)


Can this be used where an aspect only partially implements the interface that the parent class is implementing?

interface IPrintableComplex
{
   void Print();
   void PrintComplex();
}
public class SimplePrinter
{
   public void Print() { Console.WriteLine("OK"); }
}
public partial class Person : IPrintableComplex
{
   [BeaKona.AutoInterface(typeof(IPrintableComplex))]
   private readonly SimplePrinter aspect1 = new SimplePrinter();

   public void PrintComplex() { Console.WriteLine("Oh, K."); }
}

If I write this at the moment, I just get an error saying that SimplePrinter doesn't implement IPrintableComplex. What would be useful for me is if the generator could just generate the methods that SimplePrinter does have, then I can fill in the rest inside the Person class.

I thought about that feature and decided that it would add too much confusion because AutoInterface is primarily an interface helper tool and SimplePrinter does not implement IPrintableComplex. Match by signatures is just relaxation of strictness but all interface members are still required.
For your use-case I can add additional parameter which explicitly grants that, but currently I do not have much free time to implement it. Here is an example:
[BeaKona.AutoInterface(typeof(IPrintableComplex), AllowMissingMembers=true)]

Thanks, that does sound like it would suit my use case.
Time, yes, who has it?
I've made an attempt in a fork, I'd appreciate it if you had a look. I've got some concerns, I don't know why in the ProcessClass method 'references' is a collection. Also, looks like there would need to be a parallel change where the code is built from a template?? but I don't really get how that bit works.
https://github.com/beakona/AutoInterface/compare/main...waxingsatirical:AutoInterface:issue_17

Variable references is collection because there could be many fields/properties/indexers that should be called/accessed during forward-call.  In example multiple backers there are two fields paired to IPrintable interface.
Expression references.First().ReceiverType.IsMemberImplemented(i) would test first element and later inside block use all references as they all pass that test. What about this approach:
foreach(old expression)
{
   var validReferences = references.Where(i => i.ReceiverType.IsMemberImplemented(method, false));
   if(validReferences.Any())
   {
     //old code.. but use validReferences
     //writer.WriteMethodDefinition(builder, method, scope, @interface, validReferences);
   }
}

Filtering of references should be performed both for template (aka generator) and for default implementation (when generator is null).
I like your approach to IsMemberImplemented but tests for properties and indexers do not take into account explicit/noexplicit flag. Flag explicit/noexplicit could also be boolean because opposite of ExplicitInterfaceImplementation is not Ordinary | LambdaMethod.
It seem that in your usecase expression at BeaKona.AutoInterfaceGenerator/AutoInterfaceSourceGenerator.cs, 481 would always return false because method IsAllInterfaceMembersImplementedBySignature() would return false if SimplePrinter (variable receiverType) do not implement IPrintableComplex (variable type).

I've made nuget prerelease package 1.0.30-pre that should fix this issue.

Hi, thanks for making the change, if I can get this working it will really clean up some of my code. I've had a look at the pre release version. The behaviour is not exactly as I expected, in my example from above, if Person does not implement PrintComplex() then the source generator is putting in an empty method, in the generated partial class. like this:
PrintComplex() { }
I hoped that the source generator would not do this, so that the compiler would then raise an error tell me 'Person2' does not implement interface member 'IPrintableComplex.PrintComplex'. Leaving the developer to fill in the gaps that have not been satisfied by the child aspect.
If this is by design can we have another flag to stop it from generating the empty methods?
On another note, I'm also getting this warning in my application when trying to use AutoInterface, I haven't really looked into it though, so probably just something stupid I've done, maybe better not to spend any time on it for now
CSC : warning CS8785: Generator 'AutoInterfaceSourceGenerator' failed to generate source. It will not contribute to the output and compilation errors may occur as a result. Exception was of type 'ArgumentException' with message 'Reported diagnostic has an ID 'BK-AG09', which is not a valid identifier.

Hi, I've been testing your latest release.
I have a different problem now, where my child aspect has a property, get & set,  that implements the target interface property which is get only.
The problem stems from ITypeSymbolExtensions
public static ISymbol? FindImplementationForInterfaceMemberBySignature(this ITypeSymbol @this, ISymbol interfaceMember)
{
    string name = interfaceMember.Name;
    if (string.IsNullOrEmpty(name) == false)
    {
        foreach (ISymbol member in @this.GetMembers(name).Where(i => i.Kind == interfaceMember.Kind))
        {
            var comparer = new ComparerBySignature();
            if (comparer.Equals(member, interfaceMember))
            {
                return member;
            }
        }
    }

    return null;
}

This returns true only if the members match, but should only be interested in whether the interfaceMember is implemented.

Thanks for the update, the MemberMatchTypes has resolved that problem. Onto the next one, the generated properties have no access modifiers, despite the target implementation declaring them as public. I have also tried adding public to the interface declaration but this is not respected. Am I missing a setting somewhere?
public interface IPrintable
{
    public void Print();
}

public class SimplePrinter : IPrintable
{
    public void Print() { Console.WriteLine("OK"); }
}

public class MyPrinter : IPrintable
{
    [Beakona.AutoInterface]
    private IPrintable _simplePrinter  = new SimplePrinter()
}

I've created new project with your example, added keyword partial to the MyPrinter and here is the output:
// <auto-generated />
#nullable enable

namespace AutoInterfaceSample.Test
{
    partial class MyPrinter
    {
        void IPrintable.Print() => this._simplePrinter.Print();
    }
}

MyPrinter is generated without access keywords intentionaly because class/struct/record is partial and it will follow access you specify at other partial sites.
Print() is explicit interface member implementation, intentionaly. That is main purpose of AutoInterface to redirect all interface members to target instance allowing that MyPrinter behaves as IPrintable without interfering with other MyPrinter members.