consider splitting memory allocation and computation functions in the c_rewrite branch

e.g. see: https://github.com/kwgoodman/bottleneck/blob/6a4a315f7c5426b8071903697e5e65e27d0c905d/bottleneck/src/move2_template.c#L27
If the split happens now, it will make it easy to parallelize these functions later.  Also, it makes it easy to benchmark the computation part only, as you can avoid repeated memory allocation.

My plan was to do allocation of the output array in the mover function. That way the allocation is done in one place instead of duplicating the allocation code in every low-level function.
The problem was that mover does not currently know the dtype of the output array. We'd need to pass in a mapping of input dtype to output dtype. The mapping could be a C array like this
{NPY_FLOAT64, NPY_FLOAT32, NPY_FLOAT64, NPY_FLOAT64}

where it is understood that that four positions represent input arrays of type float64, float32, int64, int32. The mapping above is for move_mean.
It seemed messy so I wasn't sure that it was an improvement.
What did you have in mind for how to split?

I didn't look too closely, but I wouldn't mind having the ability to pass in a buffer from outside bottleneck.
So, I wish I could do something like
bn.move_mean(out = my_ndarray, ...)
Then perhaps bn.move_mean would still check to make sure that the output argument matches exactly (dtype, shape, etc).  I am not sure if we should force the ordering though, but forcing dtype/shape/etc. makes some sense.
It would be nice to do this sooner than later, since it would make benchmarking things easier.  Right now, I have to modify the function a little bit before I can start benchmarking it.
And if we can get rid of all memory allocation, numpy interaction from the C implementation, perhaps other languages might want to integrate the C implementation.  Maybe Rust or Julia, but I am not that familiar with them.

Can you explain the benchmarking comment? I don't understand.

I guess, I should've said profiling instead of benchmarking
For profiling, I would like to isolate and analyze just the computation part.  Then make changes and examine the effect of that change.  With the current signature, we measure not just the hot loop, but the python memory allocation routines, etc.
So, what we can do is the move https://github.com/kwgoodman/bottleneck/blob/6a4a315f7c5426b8071903697e5e65e27d0c905d/bottleneck/src/move2_template.c#L27 line of code out of the INIT, and change the function signatures to pass in y.  Then I just wrote a separate C program that did not do memory allocation in the loop I used for benchmarking/profiling.

I'd like to remove anything that needs the GIL from the low level functions. That includes the creation of the output array. But I might save it for bottleneck 1.3. Porting to C is a big project.

Line-by-line profiling would help: #136.

I'm closing this for now.


Needs to be added to spectrafit example juypter notebook

https://github.com/raman-noodles/raman-spectra-decomp-analysis/blob/8c624e4b7e46165dd67ff1b2906c780c339052cc/ramandecompy/spectrafit.py#L585

Done.


U2F/2FA, how to enable it and enroll myself?

Hi,
The documentation on this is really sparse!
I can only find some 'hint' that it even is possible in Gitea, but any documentation on how to enable it seems to be either completely missing or named so weirdly that you can't find it.
Some examples:
https://docs.gitea.io/en-us/authentication/ (makes no mention of it, at all!)
https://docs.gitea.io/en-us/ (mentions it in the features, but that's all)
https://github.com/go-gitea/gitea/blob/master/custom/conf/app.ini.sample (here calls it U2F but no documentation on how to use it)
In the user settings under security this is mentioned: "Your account must be enrolled in two-factor authentication to use security keys." but absolutely no way of figuring out how to enroll a user to use that.
I don't want to criticize Gitea, it's a marvelous project :)
But having some documentation on how to use this feature would be really welcome! I haven't figured it out yet.
Best regards,
Mark
Edit
Apparently on the security settings page (fur the user) there is a big ass button to enroll in 2FA right in the first section. I apparantly just didn't catch that at all at first. I only noticed it when watching this video about the feature: https://www.youtube.com/watch?v=sA4Id3Sc72M
Still leaves the above as valid, just not "as bad" as i thought it was.

The secret it that you don't really need anything to do with config to enable U2F. All default settings works.
The only condition is that you must have https between browser and server. U2F works only in this case.
Than go to settings/security and enable OTP, after that U2F become available.


Compile error under OS X Yosemite

I tried to build dynomite from the master branch and :
Running gcc-4.6 (Homebrew gcc46 4.6.4) 4.6.4
Your Travis use 4.6.3
Build output:
Making all in event
gcc -DHAVE_CONFIG_H -I. -I../..  -I ../../src  -Wall -Wshadow -Wpointer-arith -Winline -Wunused-function -Wunused-variable -Wunused-value -Wno-unused-parameter -Wno-unused-value -Wconversion -Wsign-compare -Wstrict-prototypes -Wmissing-prototypes -Wredundant-decls -Wmissing-declarations -Werror=implicit-function-declaration -g -O2 -MT dyn_epoll.o -MD -MP -MF .deps/dyn_epoll.Tpo -c -o dyn_epoll.o dyn_epoll.c
mv -f .deps/dyn_epoll.Tpo .deps/dyn_epoll.Po
gcc -DHAVE_CONFIG_H -I. -I../..  -I ../../src  -Wall -Wshadow -Wpointer-arith -Winline -Wunused-function -Wunused-variable -Wunused-value -Wno-unused-parameter -Wno-unused-value -Wconversion -Wsign-compare -Wstrict-prototypes -Wmissing-prototypes -Wredundant-decls -Wmissing-declarations -Werror=implicit-function-declaration -g -O2 -MT dyn_kqueue.o -MD -MP -MF .deps/dyn_kqueue.Tpo -c -o dyn_kqueue.o dyn_kqueue.c
dyn_kqueue.c:401:40: warning: implicit conversion loses integer precision: 'intptr_t' (aka 'long') to 'int' [-Wshorten-64-to-32]
                          strerror(ev->data));
                          ~~~~~~~~ ~~~~^~~~
../../src/dyn_log.h:103:41: note: expanded from macro 'log_error'
        _log(__FUNCTION__, __LINE__, 0, __VA_ARGS__);                           \
                                        ^
dyn_kqueue.c:472:12: error: use of undeclared identifier 'st'
        cb(st, &nreturned);
           ^
dyn_kqueue.c:467:40: warning: implicit conversion loses integer precision: 'intptr_t' (aka 'long') to 'int' [-Wshorten-64-to-32]
                          strerror(ev->data));
                          ~~~~~~~~ ~~~~^~~~
../../src/dyn_log.h:103:41: note: expanded from macro 'log_error'
        _log(__FUNCTION__, __LINE__, 0, __VA_ARGS__);                           \
                                        ^
2 warnings and 1 error generated.
make[3]: *** [dyn_kqueue.o] Error 1
make[2]: *** [all-recursive] Error 1
make[1]: *** [all-recursive] Error 1

Thanks in advance :)

Hi,
after merge I am struggling with
gcc -DHAVE_CONFIG_H -I. -I../..  -I ../../src  -Wall -Wshadow -Wpointer-arith -Winline -Wunused-function -Wunused-variable -Wunused-value -Wno-unused-parameter -Wno-unused-value -Wconversion -Wsign-compare -Wstrict-prototypes -Wmissing-prototypes -Wredundant-decls -Wmissing-declarations -Wimplicit-function-declaration -g -O2 -MT dyn_entropy_rcv.o -MD -MP -MF .deps/dyn_entropy_rcv.Tpo -c -o dyn_entropy_rcv.o dyn_entropy_rcv.c
dyn_entropy_rcv.c:32:10: fatal error: 'sys/sendfile.h' file not found
#include <sys/sendfile.h>

<sys/sendfile.h> seems to be linux specific, isn't it ??
http://stackoverflow.com/questions/8252698/sys-sendfile-h-not-found-gcc
thank you for your effort !
Cheers.

Hi,
after merge I am struggling with

dyn_crypto.c:214:14: error: implicit declaration of function 'fmemopen' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
    stream = fmemopen(buffer, encodedSize+1, "w");

seems that it is more common https://github.com/btrask/stronglink/issues/102
thank you for your effort !
Cheers.

Hi Can you give me the set of commands to set up the environment to compile on mac? homebrew and stuff? One of us will try to compile it locally instead of doing back and forth

hmm, honestly I setup only gcc 4.9
brew install gcc49
then
alias gcc='/usr/local/bin/gcc-4.9'
export CC=/usr/local/bin/gcc-4.9
It is everything that I changed to be up-to date with Travis setup.
If you want to know more just ask, willingly answer.

@KowalczykBartek  doesn't your issue get solved by https://github.com/btrask/stronglink/commit/b8577d614d3b7673e95c3a69b78ddfa9929dbd3d
@shailesh33 the packages needed are `brew install cmake autoconf libtool gcc automake openssl'

Ok, so, as far I made successful make , but I had to do :

according to @ipapapa hint I added stronglink were required (under src/ directory ) and I added required include #import "fmemopen.h"

I had to remove #include<sys/sendfile.h> from dyn_entropy_rcv.c dyn_entropy_snd.c and dyn_entropy_util.c (reference : http://stackoverflow.com/questions/8252698/sys-sendfile-h-not-found-gcc)

I had to remove -rdynamic option  (reference : http://stackoverflow.com/questions/29534519/why-gcc-doesnt-recognize-rdynamic-option)

Thanks for your amazing work and hints !

Thanks guy for your amazing work and blazing fast support !
Cheers !


Leaving java_vtgate_test_helper.py while running java tests.

Its' easier to keep state that way, and terminate all
processes right away, without forking something else.
@enisoc @guoliang100 @aaijazi whoever feels more comfortable with java, please! :)

LGTM


Add vertical scrolling to bézier track editor.

While the bézier editor likely needs a much broader UX pass, this at least adds means to scroll the track list, the absence of which makes the bézier editor basically unusable with large numbers of tracks.

Closes https://github.com/godotengine/godot/issues/74577

Thanks!

@SaracenOne Sorry, but this doesn't seem to work on my setup. :( Scroll wheel of the mouse or two-finger scroll-gesture on the touchpad don't do anything unless zoom modifiers (CTRL / ALT) are pressed.
First I though this was broken again later (started working with post-RC1-version), but this new issue seemed to appear right after this commit (switched/checked out versions right before this and this).
Could this be related to OS/HW-setup? My system (From Godot's Help->Copy System Info): Godot v4.2.rc (1ba920fad) - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 2060 (NVIDIA; <IP_ADDRESS>01) - Intel(R) Core(TM) i7-10750H CPU @ 2.60GHz (12 Threads)
This is a Lenovo laptop with touchpad and an external mouse (Logitech M705). Everything else seems to work with this setup, including scrolling in other parts of the Godot's UI.
Sorry for taking so long before testing this (as the opener of the related issue), but I only just started to work again with the bézier-editor.
Don't know how this should be handled so writing this comment here...


MH-13215 WorkflowOperationTagUtil throws a null pointer

This replaces #549

@lkiesow & @mtneug This is a quite important bug that Opencast 6.x is about to introduce. Could one of you please take the review to make sure this fix gets into Opencast 6.0?

Works as intended.


Account for different module ABI name when reconstructing a type

(cherry picked from commit 04034d5b80e173bf8639aba0fc27340d238a4ecf)
Explanation: A type's mangled name will store the module's ABI name, not the module's regular name. When reconstructing a type from a mangled name, the demangler needs to take that into account, and look up the module by both it's ABI name and regular name, otherwise type reconstruction will fail for types defined outside of the current module.
Scope: Small.
Issue: rdar://126953614
Original PRs: https://github.com/apple/swift/pull/73264
Risk: Low, affects type reconstruction of modules that use the -module-abi-name flag.

@swift-ci test


Question: should class and className properties be of the same type?

At the moment HTMLAttributes and SVGAttributes definition looks like:
export interface HTMLAttributes<T> extends AriaAttributes, DOMAttributes<T> {
  // Extension
  class?: ClassNames

  // Standard HTML Attributes
  className?: string

  // ...
}

So class and className that refer to the same property have different types. And I don't know is it intentional or just copy-paste error since this approach has its own pros and cons.
Pros:

ClassNames is one of jsx-dom's cool features which is not supported by React. So in case we want to port a component from jsx-dom to React, TypeScript will remind us that this functionality must be abandoned, since React components simply don't have a class property

Cons:

Code looks inconsistent when using className and class simultaneously
ESLint + plugin:react/recommended complains about class without this rule:

"react/no-unknown-property": ["warn", { "ignore": ["class"] }]

They should be the same type. It’s an oversight on my part.

@proteriax, ok, then I'll make PR to fix it tomorrow :)


Add sorting filter to the certificates' request

Admin users on the admin part of the site can sort certificates by:

[ ] price
[ ] status
[ ] search field

Using the search field admin can search the desired certificate by the date of its creation or admin’s name who has generated it.

There are two ways to solve it:

return all data and sort/search it with a front-end part
do it on the backend part.
On mockups we have pagination, so, I think, it would be wiser to use a second option.


修订 README-cn.md

修复几个小问题

Mac的键盘修饰键Option不是Alt
命令应该使用代码块，便于阅读
beta被写成了beat

Hi, can you please describe the PR in English?

Of course.
So I fixed some little issue:

On Mac keyboard, the modifier key should be Option but not Alt
Terminal command should be placed in a Markdown code block for better reading experience
As for a release version, 'beat' should be 'beta'
Thanks.


fix(BigTable): Fixes the issue with same qualifier across families.

Closes #9870
I have also added a test case to test this scenario. I will paste the link of that PR in the comment.

Conformance Test PR: https://github.com/googleapis/conformance-tests/pull/74


um-field-error still displayed after filled out missing fields

Hi,
Repro steps:
Go to https://cloud.taipy.io/register/ and click Create account. See displayed um-field-error for proper fields.
Next, fill out missing fields (meet the requirements) - still can see um-field-error.
Screenshot

Proposed solution
Validation - if it is fine, no um-field-error message.
Best wishes,

Hello and thank you for your report. Taipy-cloud is currently undergoing a major migration. We'll take care of it right after.

The graphical interface has been redone, and the Taipy Cloud service will be shut down end of April 2024.


multi face

Hi
great work, i have almost 100 different person how i can save after recognition them i need one frame only .... actually i need data for dash ploty for graph  how many people recognized by my model counting ........thnx for help

What do you mean exactly? Can you specify your issue better, please? Thanks....

We have 100 students In my class I create flask app using tf opencv n facenet mtcnn model pre train work good …..now I need output images after recognition like 5 friends same time in frame (web,ip whatever), there is any way opencv Extract particular frame and save images with name and if Unkown person detected save his image in Unkown folder with date n time ,I have a CVS file Contain all 100 students name n one folder with name with images , if it’s possible then I can feed data to dash ploty for graph and Heroku for reports….. I really need this please help me in short I need to display a graph and monthly reports if there is any way please guide me sir

You can develop what you mentioned using OpenCV easily!
Here is the algorithm:
1.) Get the specific frames according to your requirements
2.) Perform face recognition model on the frames
3.) Crop each recognized faces and save them under the folders which are named with the recognized person's name
4.) Crop the unrecognized faces and save them under the folder which is named "unknown"
5.) Get the saved images counts and use that information to creating report
You can benefits this code to crop and save faces:
import cv2
img = input_video_frame
crop_img = img[y:y+h, x:x+w] # set the x, y, w, h considering recognized face pixel coordinates
cv2.imshow("cropped", crop_img)
cv2.imwrite(face_name + ".jpg", "./FaceDatabase" + face_name) # face_name comes from face recognizer and FaceDatabase is the location which is for storing cropped face images


Title Bar Customization

Adds the platform section to config.ini, with settings WindowBorder, WindowText, WindowCaption, and WindowDark
[platform]
WindowBorder="#494949"
WindowText="#e9e0c4"
WindowCaption="#1d1d1d"
WindowDark=true

Before
After

Windows <10, Linux DEs

[x] Nothing ✨ (I don't think you can do any of this)

Windows 11

[x] Set title bar caption color based on WindowCaption
[x] Set  title bar border color based on WindowBorder
[x] Set title bar text color based on WindowText
[x] Set dark title bar for based on WindowDark (visible if WindowCaption isn't set)

Windows 10

[ ] Set dark title bar based on WindowDark

macOS

[x] Set title bar caption color based on WindowCaption (I think you can do this)
[ ] Set title bar border color based on WindowBorder (I don't think you can do this; don't they not have borders now?)
[ ] Set title bar text color based on WindowText (I hope you can do this)
[ ] Set dark title bar for based on WindowDark (visible if WindowCaption isn't set; I have no idea if you can do this, and I don't really care)

@yuxshao In this latest commit, I had a conflict between some Cocoa stuff and the Delay namespace in RemoteAction.  Unfortunately, it's not a preprocessor definition so I can't just #undef it like I've had to do 3 or 4 times for Windows stuff thus far. Lmk

At the moment I'm just waiting to verify it works as intended in Windows 10, and after that it will be ready for review

macOS build is failing because I can't get the qmake to work with the include precedence. Something about the Objective C++ upsets the compiler in the way it's ordered when using qmake & all of the declarations made in StyleEditor.h are unavailable from the C++, since the compiler interpreted the header as Obj-C++. I never had this issue when using cmake


Updated clipboard redirection for mac ..

and Mac headers to macOS which is the proper writing.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Leave a comment

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

updated for macOS beta


Too many retries while calling session-restart, last error value is illegal. (3004)

Version of the custom_component: 0.8.13
Add your logs here.
2023-08-01 08:44:00.989 ERROR (MainThread) [custom_components.midea_dehumidifier_lan.appliance_coordinator] Error fetching Schlafzimmer data: Too many retries while calling session-restart, last error value is illegal. (3004)
There is a problem connecting to the Media Cloud. The integration can no longer connect to collect the data.
2023-08-01 08:52:35.147 DEBUG (MainThread) [custom_components.midea_dehumidifier_lan.appliance_coordinator] Trying to find appliance sn=000000P0000000Q1F0C9D1E4******** id=1869169779**** address=192.168.17***** version=3 2023-08-01 08:52:42.872 DEBUG (MainThread) [custom_components.midea_dehumidifier_lan.appliance_coordinator] Finished fetching Schlafzimmer data in 7.725 seconds (success: False) 2023-08-01 08:52:44.170 DEBUG (MainThread) [custom_components.midea_dehumidifier_lan.appliance_coordinator] Finished fetching Badezimmer data in 0.124 seconds (success: True)

Same Problem here while initial setup
Logger: custom_components.midea_dehumidifier_lan.config_flow
Source: custom_components/midea_dehumidifier_lan/config_flow.py:221
Integration: Midea Air Appliances (LAN) (documentation, issues)
First occurred: 17:41:34 (6 occurrences)
Last logged: 17:51:00

Caught flow exception during appliance step
Traceback (most recent call last):
  File "/config/custom_components/midea_dehumidifier_lan/config_flow.py", line 219, in _connect_to_cloud
    self.cloud = self.client.connect_to_cloud(cfg)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/midea_dehumidifier_lan/util.py", line 192, in connect_to_cloud
    return midea_beautiful_api.connect_to_cloud(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/midea_beautiful/__init__.py", line 87, in connect_to_cloud
    cloud.authenticate()
  File "/usr/local/lib/python3.11/site-packages/midea_beautiful/cloud.py", line 390, in authenticate
    self._get_login_id()
  File "/usr/local/lib/python3.11/site-packages/midea_beautiful/cloud.py", line 360, in _get_login_id
    response = self.api_request(
               ^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/midea_beautiful/cloud.py", line 302, in api_request
    self.handle_api_error(int(payload[error_code_tag]), payload["msg"])
  File "/usr/local/lib/python3.11/site-packages/midea_beautiful/cloud.py", line 645, in handle_api_error
    handler()
  File "/usr/local/lib/python3.11/site-packages/midea_beautiful/cloud.py", line 620, in authentication_error
    raise CloudAuthenticationError(error, message, self._account)
midea_beautiful.exceptions.CloudAuthenticationError: Cloud authentication error: this account does not exist (3102)

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/config/custom_components/midea_dehumidifier_lan/config_flow.py", line 552, in _do_validate
    return await self._validate_discovery_phase(user_input)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/midea_dehumidifier_lan/config_flow.py", line 535, in _validate_discovery_phase
    await self.hass.async_add_executor_job(self._connect_and_discover)
  File "/usr/local/lib/python3.11/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/midea_dehumidifier_lan/config_flow.py", line 498, in _connect_and_discover
    self._connect_to_cloud()
  File "/config/custom_components/midea_dehumidifier_lan/config_flow.py", line 221, in _connect_to_cloud
    raise _FlowException("no_cloud", str(ex)) from ex
custom_components.midea_dehumidifier_lan.config_flow._FlowException

I am facing the same issue

Midea has changed MSmartHome-API

is there a fix for this?

Basically yes but not (yet) for this one here. Don‘t know how active the Development for this Report here is.
I read it on a different integration for Midea devices and they updated it already.

@vinyli85  thanks for the tip, I added the following custom component to Home Assistant and am able to control my Midea Dehumidifier again.

Same problem here

+1

+1

+1

Same issue here.
2023-09-09 07:04:29.033 ERROR (MainThread) [custom_components.midea_dehumidifier_lan.appliance_coordinator] Error fetching Outside Kitchen AC data: Too many retries while calling session-restart, last error value is illegal. (3004)

I'm also seeing this. I had set it up and it was working then I added a second humidifier to my account and then it stopped working. I then removed the integration and couldn't set it up. I think this could have something to-do with an API change on their end or an issue when you have multiple devices.

@nbogojevic it may sound like we just need to upgrade the underlying python library?

+1

Following
I am having the same error "Unable to connect to Midea cloud API (Too many retries while calling session-restart, last error value is illegal. (3004))"
Looking for a solution

I have the same issue.
I don't know if this is related, but I noticed a few months back the Android app changed from MSmartHome to just plain SmartHome with a greener colored logo instead of Midea blue and I had to add my dehumidifier to my account again.
https://www.midea.com/us/smarthome
Perhaps they changed to a different API from the MSmartHome system.

Does anyone know if/when this will be fixed?

Same issue here. I hope it can be fixed soon.

Same issue. Waiting for an update. This component doesnt retrieve sensors like tank full

Same issue. Waiting for an update. This component doesnt retrieve sensors like tank full

I think it does. Have you tried this:
https://github.com/georgezhao2010/midea_ac_lan#extra-sensor-and-switch-entities
It is working ok for me.

Same issue. Waiting for an update. This component doesnt retrieve sensors like tank full

I think it does. Have you tried this:
https://github.com/georgezhao2010/midea_ac_lan#extra-sensor-and-switch-entities
It is working ok for me.

Great!! Much thanks @rhobison

I got it to work by rolling back to a previous midea app and syncing the device to it instead. I was truly unable to use the new version of mideas smart home.

I got it to work by rolling back to a previous midea app and syncing the device to it instead. I was truly unable to use the new version of mideas smart home.

Unfortunately, it's a temporary solution because the app will stop working at the end of this year, 2023. The Midea app asks for migration to MSmartHome...

I just bought one Midea Dehumifier in the hope to use with this and just noticed that can't connect to the app. :/
Any hint what is the workaround? Onboard in the old app will work?

Use that app instead:
https://play.google.com/store/apps/details?id=com.midea.aircondition

I just bought one Midea Dehumifier in the hope to use with this and just noticed that can't connect to the app. :/ Any hint what is the workaround? Onboard in the old app will work?

Why don't you use this one in Home Assistant?
https://github.com/georgezhao2010/midea_ac_lan#extra-sensor-and-switch-entities
I am using it and it works perfectly!

I just bought one Midea Dehumifier in the hope to use with this and just noticed that can't connect to the app. :/ Any hint what is the workaround? Onboard in the old app will work?

Why don't you use this one in Home Assistant? https://github.com/georgezhao2010/midea_ac_lan#extra-sensor-and-switch-entities
I am using it and it works perfectly!

I agree, I'm using that component and it works perfectly!

I used the other component and it worked without issues.

@rhobison Are you sure that the other integration is actually local?  I integrated my dehumidifier and it worked fine in home assistant.  After I deleted the device from the MSmartHome app, the integration no longer worked in home assistant.
I didn't do much investigation but curious of your thoughts.

To be honest, I don't know. I still have the Smart Home app installed. :/

To be honest, I don't know. I still have the Smart Home app installed. :/

But this is from the integration repository:

Note: During the configuration process, you may be asked to enter your Midea account and password. It's necessary to retrieve appliance information (Token and Key) from Midea cloud server. After all appliances configured, you can remove the Midea account configuration without affecting the use of the appliance.

So, I would think the account is not necessary, but I am not sure.

Still an issue

Is the problem still occuring with version 0.9.0?

No, for me this is fixed in 0.9.0
Thank you!

I still have issue in 0.9.2

It's unfortunate that this issue is closed as it's a breaking issue...

@GregHilston it was closed by a bot because no one responded to this issue within 7 days.


How to clean up partial upload chunks after final upload success?

Question
Hey folks! I'm having a bit of a difficult time understanding how best to clean up partial upload chunks from our datastore after we've successfully uploaded the final concatenated upload. Currently, our partial uploads and final uploads are getting stored to the same bucket and aren't tagged differently or getting cleaned up. Do we need to add something different in our config?  Do we need to have a different ConcaterDataStore for partials to get cleaned up properly?
Setup details
Please provide following details, if applicable to your situation:

Operating System: Linux
Used tusd version: v2.4.0
Used tusd data storage: AWS S3
Used tusd configuration:

composer := tusd.NewStoreComposer()
composer.UsesConcater = true
config := tusd.Config{
		MaxSize:                 26 * 1024 * 1024 * 1024,
		BasePath:                "/files/",
		RespectForwardedHeaders: true,
		StoreComposer:           composer,
	})

Used tus client library: tus-js-client

I think I understand what I need to do. I believe it looks like I need to setup another S3Store for the ConcaterStore on the composer with a separate bucket path.

With tus, a partial upload could be reused for a different concatenation in theory, so they are not cleaned up automatically. When we used S3 as our backend in production, we had a lifetime policy which removed all old uploads, which included partial and final uploads. Depending on your use case, it might be fine to use a similar approach and handle the lifetime of partial uploads just like any other upload.
Alternatively, you could use a post-finish hook to manually delete the partial uploads that were used for concatenation. The hook request includes information about whether the upload is a final or partial part and what partial uploads where used (see IsFinal, IsPartial, and PartialUploads).
I hope this helps.


Handle MemoryError for list and strings.

This aims to address #1750 by raising a Memory error for excessively large allocations. It adds a MAX_MEMORY_SIZE const to VM which is then used by objlist.rs and objstr.rs when doing imul or mul operations.

I'm not sure if this is the correct approach; 2**61 bytes is 2.3 exabytes, which probably wouldn't get hit before an actual OOM error occurs in the process. There is a RFC that allows for catching OOMs in a few different ways (try_reserve -> Result<(), ...> functions on Vec or String, oom=panic rustc option that will panic and unwind instead of just aborting) but none of them are stable yet. I'm not sure how to approach this problem, honestly.

That is true, but this is also a reasonable sanity check, since a list with 8 byte pointers can only contain up to usize::MAX / 8 elements. Beyond that, allocating becomes the virtual machine's problem and will need to dealt with accordingly.


Add support for ID-porten clients

Rename the okdata pubreg family of commands to okdata pubs (for "public services") to reflect that they now don't only handle Maskinporten clients, but also ID-porten clients.

Avhenger av https://github.com/oslokommune/okdata-maskinporten-api/pull/168.


Display a link to the producer platform if the account is a producer account

What

Display a link to the producer platform if the account is a producer account

Is there a endpoint to find out if a account is a producer or not

@stephanegigandet Any clue on this one?

We don't have an API for that yet.
Where would this link be displayed? On the home screen? Hidden in the parameters? Or on a product page if it's the product of the producer?

We have added a promo for everyone. Closing


[BUG] router v2 test fails if oc version is above 1.19

When running the routerv2 test with oc versions above 1.19 the following error is thrown from this line of code.
Common.sh file:
log "Adding workload.py to the client pod"
oc set volumes -n http-scale-client deploy/http-scale-client --type=configmap --mount-path=/workload --configmap-name=workload --add

Error:
Error: unknown flag: --type
See 'kubectl set --help' for usage.

OC version that failed:
# ./oc_1_20 version
Client Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.0", GitCommit:"af46c47ce925f4c4ad5cc8d1fca46c7b77d13b38", GitTreeState:"clean", BuildDate:"2020-12-08T17:59:43Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.0+2f3101c", GitCommit:"2f3101cb663d0cb102ccb9730b63753604f6d29b", GitTreeState:"clean", BuildDate:"2021-02-26T13:55:24Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"linux/amd64"}

OC version that worked:
# oc version
Client Version: 4.6.0-202103060018.p0-aaa9ca3
Server Version: 4.6.21
Kubernetes Version: v1.19.0+2f3101c

Hey @dry923, are you sure you're using oc rather than kubectl?, I've verified that the oc set volumes subommand is available in latest oc binary available.
rsevilla@wonderland /tmp $ oc version
Client Version: 4.6.7
Server Version: 4.8.0-0.nightly-2021-04-09-222447
Kubernetes Version: v1.21.0-rc.0+f381840
rsevilla@wonderland /tmp $ ./oc set -h
Configure application resources

 These commands help you make changes to existing application resources.

Usage:
  oc set COMMAND [flags]

Manage workloads:
  deployment-hook Update a deployment hook on a deployment config
  env             Update environment variables on a pod template
  image           Update image of a pod template
  probe           Update a probe on a pod template
  resources       Update resource requests/limits on objects with pod templates
  selector        Set the selector on a resource
  serviceaccount  Update ServiceAccount of a resource
  volumes         Update volumes on a pod template

Manage secrets and config:
  data            Update the data within a config map or secret
  build-secret    Update a build secret on a build config

Manage application flows:
  build-hook      Update a build hook on a build config
  image-lookup    Change how images are resolved when deploying applications
  triggers        Update the triggers on one or more objects

Manage load balancing:
  route-backends  Update the backends for a route

Manage authorization policy:
  subject         Update User, Group or ServiceAccount in a RoleBinding/ClusterRoleBinding

Use "oc set <command> --help" for more information about a given command.
Use "oc options" for a list of global command-line options (applies to all commands).

@rsevilla87 So version 1.20 of oc does not have set volumes
# ./oc_1_20 set -h
Configure application resources

 These commands help you make changes to existing application resources.

Available Commands:
  env            Update environment variables on a pod template
  image          Update image of a pod template
  resources      Update resource requests/limits on objects with pod templates
  selector       Set the selector on a resource
  serviceaccount Update ServiceAccount of a resource
  subject        Update User, Group or ServiceAccount in a RoleBinding/ClusterRoleBinding

Usage:
  kubectl set SUBCOMMAND [options]

Per our discussion on slack i'm closing this out as there is definitely something wrong w/ the binary of oc that I was using. Will reopen if needed.


Unprefixed value is used if prefixed value is not available

Hi,
The following playground link should illustrate this issue: http://play.golang.org/p/XHSJ8ihZh-
If MYAPP_USER is not defined but USER is, the value of USER will be used instead. This seems to be a side effect of https://github.com/kelseyhightower/envconfig/pull/10 , in https://github.com/kelseyhightower/envconfig/blob/master/envconfig.go#L51 we use fieldName as key without actually checking if a struct tag was actually set.
I can submit a PR for this but want to make sure the current behaviour is not intentional and this is not just a documentation issue.

Had to fix it for my project so ended up creating a pull request anyway: https://github.com/kelseyhightower/envconfig/pull/15

Had to fix it for my project so ended up creating a pull request anyway: https://github.com/kelseyhightower/envconfig/pull/15

We still get this problem when adding a tag. For example, in the code bellow I expected to fill the T.Thing only when there's a MYAPP_THING environment variable. But it accepts a THING environment variable.
package main

import (
	"fmt"
	"os"

	"github.com/kelseyhightower/envconfig"
)

type T struct {
	Thing string `envconfig:"thing"`
}

func main() {
	os.Setenv("THING", "123")
	t := T{}
	envconfig.Process("myapp", &t)
	fmt.Printf("This should be empty: %v", t.Thing)
}

This problem was introduced by the varInfo.Alt attribute in PR #61. We use the Alt whenever the varInfo.Key wasn't found, but Alt doesn't have the prefix in it. Is there any reason to why we don't add prefix to varInfo.Alt?

@rafaeljusto Attempting to think back, so please be patient ...
As I recall when I was doing the update the alt specification was documented as not using the prefix, but instead being the directly used for looking up a value. I believe, but haven't verified, that this is how the tests cases were designed as well.

 This behavior is much older than your PR. It was introduced in the commit fab02b324e628c39a622baa3b3a5c71e94aadfdd and is also documented in README:

If envconfig can't find an environment variable in the form PREFIX_MYVAR, and there is a struct tag defined, it will try to populate your variable with an environment variable that directly matches the envconfig tag in your struct definition

Thanks!


Problems on DataGrid

With the dark theme the selected row values cannot be read.
The String format doesn't work properly:
<DataGridTextColumn Header="Incidência" Binding="{Binding AffectedValue, StringFormat=\{0:N2\}}"/>
<DataGridTextColumn Header="IVA" Binding="{Binding VATValue, StringFormat=\{0:N2\}}"/>
<DataGridTextColumn Header="Desconto" Binding="{Binding Discount, StringFormat=\{0:N2\}}"/>
<DataGridTextColumn Header="Total" Binding="{Binding Total, StringFormat=\{0:C\}}" />

The last column should display 2 decimal places and the currency simbol, and the 3 columns before should have 2 decimal places.
How can i align the values to the right?

With the dark theme the selected row values cannot be read.

Upgrade to latest (preview) version. This is fixed already https://github.com/AvaloniaUI/Avalonia/pull/3564

I was using the 0.9.8 and updated to version 0.9.9 today, the problem remains on the datagrid theme.
I'm using these styles:
<StyleInclude Source="avares://Avalonia.Themes.Default/DefaultTheme.xaml"/>
<StyleInclude Source="avares://Avalonia.Themes.Default/Accents/BaseDark.xaml"/>
<StyleInclude Source="avares://Avalonia.Controls.DataGrid/Themes/Default.xaml"/>

This is fixed in the current master and not in the release. Use a recent nightly build if you want to use this fix.


[Task]: Migrate all the flags and segments from Flagsmith

Closing as this is completed using REST API: https://apidocs.launchdarkly.com/tag/Environments#operation/postEnvironment


pub get fiailed , 2.0version

tips:
Because websafe_svg >=2.0.0-nullsafety.0 depends on http ^0.13.0 which depends on http_parser ^4.0.0, websafe_svg >=2.0.0-nullsafety.0 requires http_parser ^4.0.0.
And because dio 3.0.10 depends on http_parser >=0.0.1 <4.0.0 and no versions of dio match >3.0.10 <4.0.0, websafe_svg >=2.0.0-nullsafety.0 is incompatible with dio ^3.0.10.
?how to fix it? pls

It sounds like your issue is with dio.  Upgrading to version 4.0.0 of that library will likely solve your problem.


Fixing Clear All bug arising in #99

solves #108

This change is

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

LovefieldService.js, line 502 [r1] (raw file):
Hmm, I think this condition is the wrong way round.
I think this would be easier to follow if you did something like
var deleteCommentsQuery;
if (run_id) {
    deleteCommentsQuery = <query that deletes comments for run>
} else {
    deleteCommentsQuery = <query that just deletes all comments>
}
deleteCommentsQuery.exec();

Comments from the review on Reviewable.io

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


Implement Select component

Implement Select component based on what done for SelectRingTypeAC and SelectRingTypeACMul in reference project.
Component should be generic enough to allow plugging external data-sources for Autocomplete feature.

To be more complete I also suggest to look at SelectIPDefACMul component as this contains also grouping management.


error when run on vps ubuntu 20.04 ARMv8 x64

hello. I tried to run the example code below:
const { Sticker } = require('wa-sticker-formatter')

const image = 'https://c.tenor.com/WZBvSgw5JMgAAAAC/watson-amelia-amelia-watson.gif'

const stickerMetadata = {
    type: 'full',
    pack: 'watson',
    author: 'amelia',
    categories: [
        '🌹'
    ]
}
const sticker = new Sticker(image, stickerMetadata).build()

and I returned this error that I didn't understand:
node:child_process:397
      ex = new Error('Command failed: ' + cmd + '\n' + stderr);
           ^

Error: Command failed: "/app/node_modules/wa-sticker-formatter/bin/libwebp_linux/bin/gif2webp" /tmp/0.5qhsrtj739g.gif -o /tmp/0.5qhsrtj739g.webp
/bin/sh: 1: /app/node_modules/wa-sticker-formatter/bin/libwebp_linux/bin/gif2webp: Exec format error

    at ChildProcess.exithandler (node:child_process:397:12)
    at ChildProcess.emit (node:events:394:28)
    at maybeClose (node:internal/child_process:1064:16)
    at Socket.<anonymous> (node:internal/child_process:450:11)
    at Socket.emit (node:events:394:28)
    at Pipe.<anonymous> (node:net:672:12) {
  killed: false,
  code: 2,
  signal: null,
  cmd: '"/app/node_modules/wa-sticker-formatter/bin/libwebp_linux/bin/gif2webp" /tmp/0.5qhsrtj739g.gif -o /tmp/0.5qhsrtj739g.webp',
  stdout: '',
  stderr: '/bin/sh: 1: /app/node_modules/wa-sticker-formatter/bin/libwebp_linux/bin/gif2webp: Exec format error\n'
}

is this some incompatibility with ARM systems?

@erickythierry can you share the file or url that causes the issue? I'd like to try to replicate it. I'm currently running this on an Ampere OCI and it works.

@erickythierry can you share the file or url that causes the issue? I'd like to try to replicate it. I'm currently running this on an Ampere OCI and it works.

currently is already functioning normally in arm architecture, the problem was in version 3.xx or lower, it was an incompatibility with a dependency that had no version available to arm at the time (2 years ago approximately)

Thank you for confirming. I'll close the issue, then!


Accessing Stored Processes via SAS Navigator Portlet: ERROR when opening Stored Processes with prompts or form fields

We primarily want to use the SAS Navigator Portlet to access Stored Processes. Currently, an error is displayed during execution, as it seems that prompts and form fields are not supported. It might be an idea to pass the parameters in the URL, for example: '/SASStoredProcess/do?_action=form,properties&_program=' (Documentation: https://documentation.sas.com/doc/en/itechcdc/9.4/stpug/n0m6bfw604p33tn1ly7rbrx3ghla.htm#n1b938dmvdgyi0n1oql2tyqirorz).

link changed to include the form and properties option for _action.


Need documented process for identified security vulnerabilities

Migrated from GitLab: https://gitlab.com/meltano/handbook/-/issues/7
Originally created by @aaronsteers on 2021-10-22 19:15:57

As follow-up to sdk#256 and the slack thread here, we've identified a gap in our process documentation for how to responsibly track, respond to, and disclose potential security vulnerabilities as we discover them.
The only documentation currently in the handbook does mention we will acknowledge the email disclosure (to the sender) "on the next business day" but not specifically how we will disclose.
Key things to evaluate:

When do we disclose a vulnerability and how.
If there are known exploits, how much information should be provided in publicly searchable issue descriptions.
What factors (if any) would lead to an issue being marked as confidential.
What are our internal procedures for quick-response risk assessment, sizing, and prioritization. I.e., can these happen in private forums (such as slack or zoom), and what needs to be shared back with the community.

These are questions others have had to answer, and I (AJ) would like as much as possible to lean on industry best practice, where available. Following from this, I think the first step is to locate a few other internal guidelines and procedures from established tech companies and see how they compare/contrast from each other.
Note: This is not a confidential issue; anyone can view and engage in the discussion here.

What I found as of today for this process:

https://meltano.com/docs/responsible-disclosure.html

Please email<EMAIL_ADDRESS>to report any security vulnerabilities. We will acknowledge receipt of your vulnerability report the next business day and strive to send you regular updates about our progress. If you're curious about the status of your disclosure please feel free to email us again.

https://gitlab.com/meltano/handbook/-/tree/master/src/_support#responsible-disclosure-of-security-vulnerabilities

Emails to<EMAIL_ADDRESS>also forward to Zendesk, and are automatically assigned to the Security group, which includes all current team members. As documented in our Responsible Disclosure Policy, we will acknowledge receipt of a vulnerability report the next business day and strive to send the reporter regular updates about our progress.

View 1 previous comment from the original issue on GitLab


Connected event of client does not fire.

Version
2.4.0 Nuget
Description
The Connected event of the client does not seem to fire. Everything else works just fine. I can use the server's response to achieve what I want, but figured I might as well post about my issue here.
Source
https://github.com/TalicZealot/SotnRandoTools/blob/main/SotnRandoTools/src/Coop/CoopMessanger.cs

Hi @TalicZealot do you see the same behavior using the ClientTest and ServerTest projects?  You should see the selected line in the screenshot below from those projects.

Does your Connect method return true?

I'm getting the same desired result as you with the test projects. At first I thought my event arguments had a mismatch, but I tried both variants with the same result.

Does your Connect method return true?

I've refactored that method to be void. We have verified that a connection does get established and we can send and receive data without issues for extended periods of time, this is the one event that for some reason doesn't happen. I'm not using Keepalive since it causes stability issues, but from what I've read that doesn't have anything to do with this library. Since it works in the test project I feel like I missed something small and simple.

Could you try adding client.Logger = Console.WriteLine into your Connect method?  Then check to see what (if any) messages appear?
public bool Connect(string hostIp, int port)
{
	if (client is null)
	{
		client = new SimpleTcpClient(hostIp, port);
	}

	try
	{
		client.ConnectWithRetries(4 * 1000);
	}
	catch (Exception)
	{
		Console.WriteLine("Connection timed out.");
		return false;
	}

	client.Events.DataReceived += DataReceived;
	client.Events.Connected += Connected;
	client.Events.Disconnected += Disconnected;
	client.Logger = Console.WriteLine;

	return true;
}

Received log messages on disconnecting. None on establishing a connection, sending or receiving data.

Interesting - disconnecting from should also show the IP:port.  Something is disposing the client.  Is this a forms app?  If so you should consider defining your TCP client/server outside of the form, perhaps in app.cs as globals.

Sorry I should have clarified that I erased the IP from the picture. It did show it. I disposed it manually by disconnecting. My project is a win forms extension to another win forms project.

Interesting, give me a moment.  Thanks for clarifying.

Does this line get evaluated correctly in your Connected event handler, i.e. do you see this variable gets populated correctly?
toolConfig.Coop.DefaultServer = client.ServerIpPort;

It does not.

Can you try commenting out your console writeline in the connect event handler and see if that variable gets set?

No change in behavior. I also tried telling it to dispose the client in the Connected event to no avail.

Got it.  But messages are able to be exchanged bidirectionally?

Do you mean data? If so yes, everything works both ways. While I'm here there is something else that popped up recently. I'm disposing the client when it disconnects, but when the user reconnects and the server sends data it gets received twice.

For disconnects, don't dispose.  If you dispose and then reconnect, you have to first re-instantiate the object.
I think I know why it's not working for you.  You're setting the Connected event AFTER calling ConnectWithRetries.
Change it to this:
public bool Connect(string hostIp, int port)
{
	if (client is null)
	{
		client = new SimpleTcpClient(hostIp, port);
		client.Events.DataReceived += DataReceived;
		client.Events.Connected += Connected;
		client.Events.Disconnected += Disconnected;
	}

	try
	{
		client.ConnectWithRetries(4 * 1000);
	}
	catch (Exception)
	{
		Console.WriteLine("Connection timed out.");
		return false;
	}

	return true;
}

Gonna try that now, I think using the Console.WriteLine at the top of the events can lock the thread and cause weird things to happen because it gets redirected to BizHawk's windows forms log window.

Everything works perfectly now, no double data and the Connected event is firing. Than you so much for your time!

Happy to help @TalicZealot


[Failing Test]: beam_PreCommit_Java_Kafka_IO_Direct flaky: KafkaIOIT.testWatermarkUpdateWithSparseMessages

What happened?
This is kafka version compability tests and running multiple times in a test suite run. With high probability (20%-50%) one of the run fails.
The test relies on two concurrently running pipelines which may cause the flake under unpredictable CPU load on host machine.
Issue Failure
Failure: Test is flaky
Issue Priority
Priority: 2 (backlog / disabled test but we think the product is healthy)
Issue Components

[ ] Component: Python SDK
[X] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

Looks like this is resolved by #26153 (see history of https://ci-beam.apache.org/job/beam_PreCommit_Java_Kafka_IO_Direct_Cron/) closing this for now. CC: @johnjcasey


The sequence length when training semantic token to acoustic token

Hi, I want to ask the sequence length when you train your S2A model. Because SpearTTS directly predict 3 codebook's tokens. We found that if the target audio is more than 10s, the total sequence length will be very long, and the transformer model is hard to model the long sequence.
Furthermore, whether you considering to use better codec model? We have train some codec model on TTS data, I belive it can improve the performance of this project. We have train codec on 16khz, which may more easy to train. We have release part of code on https://github.com/yangdongchao/AcademiCodec
If you are interested in this, please let me know, I am willing to help you provide better codec model. I want to finish this amazing project together!

Hey, thanks for reaching out. I've seen your AcademiCodec repo, great work! 👍 I'd love to work together with you to Open Source TTS forward faster.
The current S2A model is working from 1500 semantic tokens to 4500 acoustic tokens (1.5kbps, 2 quantizers). I've extended it to 9000 tokens (3kbps, 4 quantizers) and the training took just 2x longer. I suspect it's because of the new PyTorch 2.0 optimized attention implementation that manages to hide the O(N^2) scaling I would expect.
Unfortunately the quality from the 3kbps multi-purpose EnCodec model is not great and the 3kbps S2A model trained for the same amount of epochs makes more mistakes.
I am now looking at training a separate model that would take the synthesized codes for 2 quantizers (1.5kbps) and "upsample" it to 8 quantizers (6kbps). The https://github.com/suno-ai/bark project validated this idea and I already managed to train a simple 2->4 "upsampling" model with great results.

Hey, thanks for reaching out. I've seen your AcademiCodec repo, great work! 👍 I'd love to work together with you to Open Source TTS forward faster.
The current S2A model is working from 1500 semantic tokens to 4500 acoustic tokens (1.5kbps, 2 quantizers). I've extended it to 9000 tokens (3kbps, 4 quantizers) and the training took just 2x longer. I suspect it's because of the new PyTorch 2.0 optimized attention implementation that manages to hide the O(N^2) scaling I would expect.
Unfortunately the quality from the 3kbps multi-purpose EnCodec model is not great and the 3kbps S2A model trained for the same amount of epochs makes more mistakes.
I am now looking at training a separate model that would take the synthesized codes for 2 quantizers (1.5kbps) and "upsample" it to 8 quantizers (6kbps). The https://github.com/suno-ai/bark project validated this idea and I already managed to train a simple 2->4 "upsampling" model with great results.

Thanks for your reply.  I think your plan is very nice. I believe my audio codec model can help to improve this project. I can help you to train a audio codec works on 16khz and 320 down-sample (1 second audio will produce 50 tokens per quantizer), which means that you can train a S2A model using 1500 semantic tokens to 4500 acoustic tokens (3 quantizers, the same process with Google). We test using 3 quantizers can get good reconstruction performance.  We can discuss how to better communication with each other, so that we can push the project together.

In our internal tests AcademiCodec did provide the best quality (followed closely by Descript DAC).
Independently the Vocos vocoder was released and it proved more convenient for us since it could reuse the existing Encodec tokens and still deliver great speech quality.


[WIP] Embed tag for overriding blocks in included templates

This PR adds an embed tag, which works like Twig's embed tag. It effectively includes a template but allows that template's blocks to be overridden in the host template within the body of the embed tag.
This will address a number of open issues, it's clearly a highly-desired feature:

#224
#318
#319
#378

My initial work on this seems very promising. I am opening this PR now so the community knows it is coming soon, and to get more eyes on it and make sure it's being implemented correctly.
To-Do:

[x] More tests, especially around error-handling, and verifying it doesn't cause strange behavior when used in a template that uses other block-inheritance mechanisms (extends, block, parent(), block(name))
[ ] Documentation
[x] Verify Pebble's embed behavior matches that of Twig

I've added a bunch more tests, and am comparing them to how Twig would render them, and I can't find any differences yet. Would love to get some more eyes on the PR, especially from people who have used Twig's embed tag in real-world usage.
But besides that, I just need to add some documentation to the wiki and I think this should be ready-to-go.

Alright, I think this PR is ready to be merged at this point


Test python 3.12 and/or 3.13

Currently bundling this for Windows with Python 3.11 because that's what's on my Linux desktop, but it will probably work fine on newer. Determine if that's the case, and if so, update the install script.

So far it looks ok. Will probably switch for 0.1.8.

v0.1.11 updated to Python 3.12.


Need consistent exports for components (index.js)

The index.js is not consistently being exported since there are a lot of main components and sub-components that composes the main component.
Anyway, here's how it should be:
export FileUploader, { Filename, FileUploaderButton } from './components/FileUploader';

FileUploader is the main component
{ Filename, FileUploaderButton } are sub-components (or non-main components)

Also worth mentioning that changing the export statements in this file is probably a breaking change.
Is there a way to resolve this issue in a non-breaking way?

This seems like it's been taken care of when looking at index.js. Is this not the case?

Gonna close this since I think it's good to go 🚀
Anyone can feel free to re-open if that's not the case 😄


Extract more meaningful modules from model::cfd

I set out to start tackling #1430, but I got carried away and started trying to remove things that didn't make sense any longer to me (such as the Completed enum) rather than just moving them around.
I intend to continue with #1430 in follow-up PRs. If reviewers already have some ideas for meaningful modules, that would be helpful, although I would action that feedback in the follow-up PRs.

bors r+


how can I render a 3d layer with geojson.

Please read the documentation, and use the issue templates to raise questions.


Install fails on Debian-based distros

Bug
Installing the deb package fails on Debian-based distros  due to a missing dependency that is not available through the official distro repositories.
To Reproduce
Steps to reproduce the behavior:

Download the deb package for the latest version of LBRY
Install the deb package via dpkg -i
Run apt-get -f install to get the missing dependencies installed
Confirm whether you want to remove LBRY despite specifying "install" in the aforementioned command.

Expected behavior
apt-get installs all dependencies to make LBRY useable.
Screenshots
If applicable, add screenshots to help explain your problem.

System Configuration

LBRY Daemon version: curl: (7) Failed to connect to localhost port 5279: Connection refused
LBRY App version: 0.51.2
LBRY Installation ID: Can't open LBRY to find this due to install failure
Operating system: Devuan 4.0 Chimaera (based on Debian 11.1.0 Bullseye)

Additional context
The issue is with the dependency "libappindicator1", which is not available in the distro repositories.

Devuan appears to be ranked #30 in distros. You may have to install libappindicator1 yourself, either from source or otherwise.

How Devuan is ranked anywhere has nothing to do with this bug. The fact that libappindicator1 is not available on Debian-based distros is the only relevant thing here. Check:
https://packages.debian.org/search?searchon=names&keywords=libappindicator
You will see that both Buster and Stretch are referenced in the search, but Bullseye is not referenced. Bullseye is the latest stable version of Debian.
This is why libappindicator1 is not available on Devuan Chimaera (the latest stable version of Devuan) -- Devuan tries to match Debian as closely as possible without relying on systemd, so if a package is not available on Devuan and has no reason to require systemd, chances are said package is also not in Debian.
In essence, you are saying that because Devuan ranks 30 on some unsourced list of rankings, all users who are using the latest stable version of Debian (or any distro directly based on the latest stable version of Debian that isn't adding back dropped packages) should have to install libappindicator1 themselves, either from source or otherwise. As most people will be trying to install LBRY on the latest stable version version of their distro (rather than an older version that isn't likely to be supported for long, if it's still supported at all), I don't consider installing from source to be a good solution to satisfying the missing dependency -- many people don't know how to compile software from source, and even among those of us that do there are many of us (myself included) who consider dpkg to be a bit of a complicated mystery (no, that's not an attempt at attacking dpkg -- I just find it a bit difficult to work with for making packages), so getting libappindicator1 installed in a way that dpkg (and thus apt-get -f install) will recognized may not be an option for every person that wants to used LBRY.

You're right. I have removed this dependency. It will be out in the next release. I apologize.


Connection timed out after 120000ms

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v14.17.0
Amplify CLI Version
5.6.0
What operating system are you using?
Mac
Amplify Categories
Not applicable
Amplify Commands
push
Describe the bug
I have an amplify project with ~40 Lambda functions, a comparable number of APIs, and roughly 20 dynamodb tables. I try to create a new environment and then push the changes but I always get the following time out
✖ An error occurred when pushing the resources to the cloud

Connection timed out after 120000ms
An error occurred during the push operation: Connection timed out after 120000ms

I create a new environment from the root folder of my amplify project. The idea is to set up a duplicate backend where i can test things in isolation. The commands are pretty straightforward:
amplify env add envname
amplify push
The error message does not provide any meaningful description. In addition, this time out should be configurable.
Expected behavior
I would expect after amplify push to get my back and environment provision in AWS
Reproduction steps

amplify env add envname
amplify push

GraphQL schema(s)

# Put schemas below this line

Log output

# Put your logs below this line

? Are you sure you want to continue? Yes
✖ An error occurred when pushing the resources to the cloud

Connection timed out after 120000ms
An error occurred during the push operation: Connection timed out after 120000ms
⠴ Uploading files...100%%

Additional information
No response

@krikork some clarifying questions:

could you provide the terminal output to see at what point is it timing out?
all the files are uploaded? did you see progress messages and then failure?
if the CloudFormation was already in progress, after the CLI failure when you check the CFN console, did the stack update complete successfully?

I had this issue recently, it is related to the entire set of zip files not getting uploaded in time...there are a few solutions:

Temporarily remove a few of the functions (manually edit some of the json files by hand), then push and then add the remaining ones back
Make your zip files smaller (lessen the dependencies, and use layers, don't include aws-sdk etc.)
increase the timeout (not sure how to do this)
faster internet connection

The ideal solution would be that the timeout is set per zip file upload instead of per all the zip files...or that it is multiplied for each zip file...
An even nicer solution would be for the uploading part to show a list of all the functions/zip files and the % of each that has been uploaded...similar to the status page, but with an extra column % uploaded...this would help identify which ones are taking a long time to upload (often because of package.json issues)...

Also a note about the % uploaded that displays...as per the below ticket, the % is incorrect...(or at least it doesn't mean what you would expect it to mean)...
#8210

@GeorgeBellTMH so the upload itself times out for large files? what is the total number of files / size, could you share a ls -al output to get closer to the problem? also if you can provide the terminal output of such a failed push it would be highly appreciated.

I resolved the issue a few days ago by shrinking my dependencies and doing a partial push, so I can't get you any more detailed info...It may be the upload or the whole process, not easy to tell...but if you imagine uploading 100 zip files in parallel on a 10mb/s connection, each one only gets 100kb/s...so it could take a long time if they are all big files... it only happens when there are a lot of big zips on a slow connection...to test I would just create 40 functions and put a huge dependency in each one...I added some debugging code in aws-s3.ts->uploadFile to see when files completed and how long they were taking, that's how I found #8210...maybe only upload 5-10 at a time?

@attilah please see below answers to your questions.

@krikork some clarifying questions:

could you provide the terminal output to see at what point is it timing out?
all the files are uploaded? did you see progress messages and then failure?
if the CloudFormation was already in progress, after the CLI failure when you check the CFN console, did the stack update complete successfully?

The terminal output is everything I shared in the original log message. There's nothing else.
I do not know if all files are uploaded. How could i find out?
Cloud Formation did not appear to be doing anything - i kept checking it throughout the entire process of attempting to push.

From typing the command to getting the error message it will take anywhere between 10 to 15 minutes. It appears that the percentage upload it changes per resource. This is just my guess based on how the percentages were flickering throughout the upload process. At some point that flickering would stop and the percent applauded would show as 100. The console will stay in that state for a few minutes and then error out as I specified above.
As others have pointed out, the size of the package and speed of connection matter. The way I was able to solve this problem was the provision an EC2 instance and do everything from there.
I continue to be surprised by the low quality of Amplify and will switch to an alternative the first opportunity I get. Btw, this is not the first thread where this issue is reported but everything else I was able to find was somehow closed.


Use PATCH without optimistick locking to remove finalizer this way will avoid conflicts

For remove this makes sense in all cases.
(For add this could cause some issues, its hard to avoid "double adding" without optimistic locking" atm)
2022-05-24 14:19:26,505 i.j.o.p.e.ReconciliationDispatcher [ERROR] [flink.basic-session-job-example] Error during event processing ExecutionScope{ resource id: ResourceID{name='basic-session-job-example', namespace='flink'}, version: 854260} failed.
io.fabric8.kubernetes.client.KubernetesClientException: Failure executing: PUT at: https://<IP_ADDRESS>:55618/apis/flink.apache.org/v1beta1/namespaces/flink/flinksessionjobs/basic-session-job-example. Message: Operation cannot be fulfilled on flinksessionjobs.flink.apache.org "basic-session-job-example": the object has been modified; please apply your changes to the latest version and try again. Received status: Status(apiVersion=v1, code=409, details=StatusDetails(causes=[], group=flink.apache.org, kind=flinksessionjobs, name=basic-session-job-example, retryAfterSeconds=null, uid=null, additionalProperties={}), kind=Status, message=Operation cannot be fulfilled on flinksessionjobs.flink.apache.org "basic-session-job-example": the object has been modified; please apply your changes to the latest version and try again, metadata=ListMeta(_continue=null, remainingItemCount=null, resourceVersion=null, selfLink=null, additionalProperties={}), reason=Conflict, status=Failure, additionalProperties={}).
	at io.fabric8.kubernetes.client.dsl.base.OperationSupport.requestFailure(OperationSupport.java:682)
	at io.fabric8.kubernetes.client.dsl.base.OperationSupport.requestFailure(OperationSupport.java:661)
	at io.fabric8.kubernetes.client.dsl.base.OperationSupport.assertResponseCode(OperationSupport.java:612)
	at io.fabric8.kubernetes.client.dsl.base.OperationSupport.handleResponse(OperationSupport.java:555)
	at io.fabric8.kubernetes.client.dsl.base.OperationSupport.handleResponse(OperationSupport.java:518)
	at io.fabric8.kubernetes.client.dsl.base.OperationSupport.handleUpdate(OperationSupport.java:342)
	at io.fabric8.kubernetes.client.dsl.base.OperationSupport.handleUpdate(OperationSupport.java:322)
	at io.fabric8.kubernetes.client.dsl.base.BaseOperation.handleUpdate(BaseOperation.java:649)
	at io.fabric8.kubernetes.client.dsl.base.HasMetadataOperation.lambda$replace$1(HasMetadataOperation.java:195)
	at io.fabric8.kubernetes.client.dsl.base.HasMetadataOperation.replace(HasMetadataOperation.java:200)
	at io.fabric8.kubernetes.client.dsl.base.HasMetadataOperation.replace(HasMetadataOperation.java:141)
	at io.fabric8.kubernetes.client.dsl.base.HasMetadataOperation.replace(HasMetadataOperation.java:43)
	at io.javaoperatorsdk.operator.processing.event.ReconciliationDispatcher$CustomResourceFacade.replaceResourceWithLock(ReconciliationDispatcher.java:344)
	at io.javaoperatorsdk.operator.processing.event.ReconciliationDispatcher.removeFinalizer(ReconciliationDispatcher.java:317)
	at io.javaoperatorsdk.operator.processing.event.ReconciliationDispatcher.handleCleanup(ReconciliationDispatcher.java:283)
	at io.javaoperatorsdk.operator.processing.event.ReconciliationDispatcher.handleDispatch(ReconciliationDispatcher.java:78)
	at io.javaoperatorsdk.operator.processing.event.ReconciliationDispatcher.handleExecution(ReconciliationDispatcher.java:55)
	at io.javaoperatorsdk.operator.processing.event.EventProcessor$ControllerExecution.run(EventProcessor.java:356)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
	at java.base/java.lang.Thread.run(Thread.java:829)

Hi @popdaniel942 , yes this is related to that. Will investigate that, it should not happen in general (or just rarely when the resource is updated while the reconciliatio running), so this issue will mitigate that. But will investigate why is that happening.

@popdaniel942 there is integration test on our side, that checks this, so it does not happen at least in normal flows:
https://github.com/java-operator-sdk/java-operator-sdk/blob/49ee0a03f90fdeb72a5f5cdd8d1477ef4329eeb3/operator-framework/src/test/java/io/javaoperatorsdk/operator/ControllerExecutionIT.java#L69-L71
If you could tell more in what situation this heppens to you would be very helpful. Isn't you code open source?  thx

apparently with 3.0.1 this example works

@popdaniel942  thank you! we were able to reproduce this on an other project too, I'm on it! :)

In our full code it still doesn't work with 3.0.1. But we don't change anything in the resource In the cleanup. All we do is call some rest methods. And when DeleteControl.defaultDelete() is called it fails. And after the retry it works.

Yes, this happens because the framework tried to remove the finalizer during a default delete. And the resource it sees in cache might have a different version than on the server. This is strange because once a resource is marked for deletion it cannot be changed, only finalizer removed. So if there are no more finalizers used, it's weird. But also since the cleanup is triggered is means that the resource (in memory) is already marked for deletion. Looking into it..

@popdaniel942 could you pls turn on debug level logs, with MDC values, and thread id present, something like this:
appender.console.layout.pattern = %style{%d}{yellow} "%tid" %style{%-30c{1.}}{cyan} %highlight{[%-5level] [%X{resource.namespace}.%X{resource.name}.%X{resource.resourceVersion}] %msg%n%throwable}

and send us the log files pls?  thx!

@popdaniel942 are you using UpdateControl.patchStatus to update the resource?

Right now we are using UpdateControl.updateStatus but we got the error with both. What is the difference between them? Also I see that when the cleanup is made there is a foregroungDeletion finalizer being added and that the resourceVersion gets changed. Also in some cases, if we run the code in debug mode with breakpoints on the cleanup method we get the error, but if we run normally without debug mode, we don't get the error

It seems to be that I found the problem. It is actually not a bug, but this issue (using patch for finalizer removal) will fix it, see details here:
https://github.com/java-operator-sdk/java-operator-sdk/issues/1245

Right now we are using UpdateControl.updateStatus but we got the error with both. What is the difference between them?

its described here what can go wrong with the patch and why: https://github.com/java-operator-sdk/java-operator-sdk/issues/1245#issue-1250503818
One possible scenario that likely causes this is the following: when a resource reconciliation is in progress the resource gets deleted (like using kubectl, or other controller). The event for this delete is received. But at the end of the reconciliation to update the resource status a patch request (without optimistic locking) is executed. Since there is already an event present (the mar for delete event) the resource is submitted instantly for reconciliation. Note that the event for the status patch is not necessarily received at this point. So at the end of the cleanup the finalize removal uses a stale resource version.

Note that this not happened before because the UpdateControl.updateStatus is using optimistick locking and additional mechanisms to cache the resource that is the result of the update. This can happen just with UpdateControl.patchStatus

Also I see that when the cleanup is made there is a foregroungDeletion finalizer being added and that the resourceVersion gets changed.

In this case it probably happened also occasionally before, since the finalizer removal is using optimistic locking (OL)  ATM (this issues will change that)

Also in some cases, if we run the code in debug mode with breakpoints on the cleanup method we get the error, but if we run normally without debug mode, we don't get the error.

This is probably caused because if you debug (it takes some time) to other finalizer is removed, or the resource is changed in other way.

In case of updateStatus (since there is OL) it easy to cache the resource after the update, and ensure that next reconciliation has starts with newest one.

If interested more in depth an see why is this possible pls check this:
https://github.com/fabric8io/kubernetes-client/pull/3943#issuecomment-1097171960

Ok so the issue should be solved by this PR:
https://github.com/java-operator-sdk/java-operator-sdk/pull/1249
Feel free to try ti out already with a local build from that branch.
At the end it won't be patch, but the update will be retried, see the reasons in the related issue:
https://github.com/java-operator-sdk/java-operator-sdk/issues/1248
Hopefully the issue will be merged soon, and a new patch version released.


fix with_items: barword_variable issue

This fixes the with_items: barword_variable issue since ansible <IP_ADDRESS> treats it as an error not a deprecation

Thanks for the PR! I'm going to update the CI in this role and then you will need to pull in from master for them to pass...

Went with https://github.com/ANXS/vim/pull/3 because tests were passing...


Added details about BUILDKITE_* variables supported by build.env()

Added details about BUILDKITE_* variables supported by build.env()

Hi @nithyaasworld - thanks for this.
I'm folding the changes into my #1455 PR. My only question is does the function work only with the list of variables? Or only with those BUILDKITE_* variables but also other ones, for example defined by the user?

Thanks.


Some data are not added to the index

Meilisearch version:
0.29.1
Env:
AWS/k8s
Describe the bug
We have ~60k records that we add in 350 chunks. There are ~171 records appearing in the task queue with the action of documentAdditionOrUpdate. After this, a settingsUpdate occurs, which updates the search settings.
Problem:
The index count shows ~40k records instead of ~60k and some data are missing. There are no errors appearing in the logs.
What could be the problem? How can I debug it?
Thanks,

Hello @beshkenadze
This could be because of a recent bug that was fixed in version 0.30.5. I would recommend that you upgrade to the most recent version and reindex your data from scratch. Furthermore, the number of documents displayed by the stats route could be an issue. As a result, I recommend searching exhaustively or querying the get documents route to find the correct number of documents in the index.
Please let me know if this helps.
Thanks

@oluademola Thank you for your answer, but nothing has changed after updating to version 0.30.5.
We decided to try switching to Elasticsearch, so I close this issue.


Can't delete bot registration resource

I can't delete bot registration - msbot-i9tekgf, subscription ID - 0a05d163-70d7-4b44-a4ca-1e1df8acad43
Tried both Azure portal and AZ cli and got the same error -
{"error":{"code":"UnknownError","message":"message: Illegal characters in path. original trace:    at System.IO.Path.CheckInvalidPathChars(String path, Boolean checkAdditional)\r\n   at System.IO.Path.GetFileName(String path)\r\n   at Intercom.Models.BotStore.<FreeBotIconUrl>d__2.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at DevPortalLib.Helpers.BotHelper.<DeleteBotAsync>d__21.MoveNext() in C:\\__w\\1\\s\\DevPortalLib\\Helpers\\BotHelper.cs:line 415\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at System.Runtime.CompilerServices.ConfiguredTaskAwaitable.ConfiguredTaskAwaiter.GetResult()\r\n   at Intercom.ResourceProvider.DataProviders.ResourceDataProvider.<DeleteResourceAsync>d__10.MoveNext() in C:\\__w\\1\\s\\Intercom.ResourceProvider\\DataProviders\\ResourceDataProvider.cs:line 396"}}

Hi @orsharab do you use a custom url for the bot icon? (i.e. an external url) If so, it could be a bug in our service

Seems like it

@orsharab There are two cases:
1 is uploading a file from Portal and
2 is set an url from ARM api.
Could you verify which one is it?
You could use F12 to see the url of the displayed icon. If it is case one, it should be like background-image: url("//bot-framework.azureedge.net/bot-icons-v1/BOTID_SOMEVALUE.png");.

It was set with ARM API

@orsharab I see.. It is a bug in our service. A workaround: you could upload some arbitrary icon to replace the one you set and delete again. It should work in this way.

The workaround solved the issue, thanks @xieofxie!

Create an internal work item to fix this. Close the issue now


Add docstring

About this PR

Add docstring about dico.model.channel.Guild
Changes

docstring
Checklist

[ ] I checked this pull request runs on Python 3.7.X.
[ ] This fixes something in Issues.

Issue:

[ ] This adds something new.
[ ] There is/are breaking change(s).
[x] (If required) Relevant docstring is edited or added.
[x] This is not a code change. (Typo fix, docstring, etc.)

Thanks for the contribution!


Dont store password if no auto login

Fixes #1179

yes it is I tested it and JSON as no issues to store a null in a string


not enough values to unpack (expected 9, got 7)

Amazing work!
I find that if use mask as input, this function below will be called.
https://github.com/yangchris11/samurai/blob/c8c27f7e2acc2ed0a6e3691d32530b26af2d1cb5/sam2/sam2/modeling/sam2_base.py#L902
But the sam_output didn't contain the ious and kfious. So when unpack the sam output here
https://github.com/yangchris11/samurai/blob/c8c27f7e2acc2ed0a6e3691d32530b26af2d1cb5/sam2/sam2/modeling/sam2_base.py#L1005
will raise the error

Please see https://github.com/yangchris11/samurai/issues/3 for same discussion. We currently only support bbox as input, but you can be expecting the mask prompt version soon :)

Thanks a lot for your response.


Reuse time.Now() result in StartGC()

StartGC() in both MemoryTTL and ShardedTTL can be optimized to only call time.Now() once and pass the result to r.isValid(). For huge caches it may save a lot of overhead. Actually it can also avoid the overhead of adding a TTL to the item's setAt by subtracting r.ttl from the current time.
deadline := time.Now().Add(-r.ttl)
// ...
return setAt.After(deadline)

fixed with https://github.com/koding/cache/pull/9 Thanks for the suggestion @ash2k


The grid fades out in an empty project

In latest Godot, Windows 10 64 bits
I just created an empty project, and noticed the grid is fading in a weird way:

It depends on the camera angle, sometimes the grid is fully visible, sometimes it disappears completely.

I can not reproduce this so far having tried a lot of different angles and zoom levels. Also on latest master build, Linux

I can reproduce this on Windows (with an RX 480 card)

First of all thank you for your report and sorry for the delay.
We released Godot 3.0 in January 2018 after 18 months of work, fixing many old issues either directly, or by obsoleting/replacing the features they were referring to.
We still have hundreds of issues whose relevance/reproducibility needs to be checked against the current stable version, and that's where you can help us.
Could you check if the issue that you described initially is still relevant/reproducible in Godot 3.0 or any newer version, and comment about its current status here?
For bug reports, please also make sure that the issue contains detailed steps to reproduce the bug and, if possible, a zipped project that can be used to reproduce it right away. This greatly speeds up debugging and bugfixing tasks for our contributors.
Our Bugsquad will review this issue more in-depth in 15 days, and potentially close it if its relevance could not be confirmed.
Thanks in advance.
Note: This message is being copy-pasted to many "stale" issues (90+ days without activity). It might happen that it is not meaningful for this specific issue or appears oblivious of the issue's context, if so please comment to notify the Bugsquad about it.

I think it's irrelevant now, I never had this problem since.

Thanks for the info. I also haven't encountered it in the stable version. :+1:


Python: JSON commands in transaction

Issue link
This Pull Request is linked to issue (URL): https://github.com/valkey-io/valkey-glide/issues/2683
Checklist
Before submitting the PR make sure the following are checked:

[x] This Pull Request is related to one issue.
[x] Commit message has a detailed description of what changed and why.
[x] Tests are added or updated.
[x] CHANGELOG.md and documentation files are updated.
[x] Destination branch is correct - main or release
[x] Commits will be squashed upon merging.


Remove GuildChannel fallback, and remove GuildChannel as extendable

Please describe the changes this PR makes and why it should be merged:
PR based on comments from Gawdl3y. Requesting review from @Gawdl3y.
Previously all known channel types are accounted for before defaulting to a generic GuildChannel if the channel type is unknown. If discord adds a channel type before the library properly handles it, this could lead to bugs of unknown impact (minor -> critical).
Because of this fallback, GuildChannel was added to the extendable-structures. (this also is not ideal since extensions to GuildChannel were not inherited by TextChannel or VoiceChannel) Now that the fallback is gone, we can cleanly remove it from the extendable structures.
Semantic versioning classification:

[ ] This PR changes the library's interface (methods or parameters added)

[x] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)

[ ] This PR only includes non-code changes, like changes to documentation, README, etc.

It was suggested by Gus in chat, that an alternative approach should be taken. Instead of removing "defaulting to a generic GuildChannel", we should add "defaulting to a generic Non-GuildChannel" which is currently being discarded in all versions. (while still removing GuildChannel as an extendable).
So could I get some Official Direction on this PR?

closing in favor of #3165