"Sequence numbers are deprecated"

My code:
from datetime import datetime
from pytodoist import todoist
from pytodoist.api import TodoistAPI
from credentials import *

#Setup todoists
user = todoist.login(user, password)

#Setup datetime objects
today_object = datetime.today().date()

#API setup
api = TodoistAPI()
response = api.login('##EMAIL##', '##PASSWORD##')
user_info = response.json()
user_api_token = user_info['token']

def copy_due(inp, outp):
    """
    Takes arguments
    inp: Input project
    outp: Output project

    and copies due tasks from inp to outp
    """
    #Setup projects
    output_id = user.get_project(outp).id
    due_tasks = user.get_project(inp).get_uncompleted_tasks()

    for task in due_tasks:
        if task.due_date is not None:
            date_object = datetime.strptime(task.due_date, '%a %d %b %Y %H:%M:%S %z').date()

            if today_object == date_object:
                notes = task.get_notes()

                label_ids = task.labels

                response = api.add_item(user_api_token, task.content, project_id=output_id, priority=task.priority, labels=str(label_ids))

                task.complete()

copy_due("Recurrent-tickler", "Solitary-actionable")
#copy_due("Tickler", "Solitary-actionable")

The error:
Traceback (most recent call last):
  File "/Volumes/home/Work/Tickler-Todoist/test.py", line 44, in <module>
    copy_due("Recurrent-tickler", "Solitary-actionable")
  File "/Volumes/home/Work/Tickler-Todoist/test.py", line 42, in copy_due
    task.complete()
  File "/usr/local/lib/python3.5/site-packages/pytodoist/todoist.py", line 1293, in complete
    _perform_command(self.project.owner, 'item_close', args)
  File "/usr/local/lib/python3.5/site-packages/pytodoist/todoist.py", line 200, in _perform_command
    _fail_if_contains_errors(response, command_uuid)
  File "/usr/local/lib/python3.5/site-packages/pytodoist/todoist.py", line 175, in _fail_if_contains_errors
    raise RequestError(response)
pytodoist.todoist.RequestError: {"error_tag":"ACCESS_DENIED","error_code":101,"http_code":403,"error_extra":{},"error":"Sequence numbers are deprecated. Please upgrade your client to switch to sync tokens or contact support at https:\/\/support.todoist.com. Deprecated functionality will be  completely disabled on 13 Feb 2016 at 9:00:00 GMT"}

The code has been working just fine for the last ~6 months – hope you have time to look at it! 👍

I haven't been able to reproduce the error and all of my tests still pass successfully. Can you please confirm that the error is still occurring for you?
The error message suggests that there was an authentication problem, can you please double check your credentials?

It's back! D:
  File "test.py", line 48, in <module>
    copy_due("Recurrent-tickler", "Solitary-actionable")
  File "test.py", line 42, in copy_due
    task.complete()
  File "/home/martin/.local/lib/python3.5/site-packages/pytodoist/todoist.py", line 1293, in complete
    _perform_command(self.project.owner, 'item_close', args)
  File "/home/martin/.local/lib/python3.5/site-packages/pytodoist/todoist.py", line 200, in _perform_command
    _fail_if_contains_errors(response, command_uuid)
  File "/home/martin/.local/lib/python3.5/site-packages/pytodoist/todoist.py", line 175, in _fail_if_contains_errors
    raise RequestError(response)
pytodoist.todoist.RequestError: {"error_tag":"ACCESS_DENIED","error_code":101,"http_code":403,"error_extra":{},"error":"Sequence numbers are deprecated. Please upgrade your client to switch to sync tokens or contact support at https:\/\/support.todoist.com. Deprecated functionality was  disabled on 14 Feb 2016 at 17:00:00 GMT"}```

Maybe this line is significant? `Deprecated functionality was  disabled on 14 Feb 2016 at 17:00:00 GMT`

Do your unit tests still pass? My system hasn't been modified since I wrote that it was fixed.

The functionality is deprecated, but it has been so for nearly a year now with no issue.
My tests still pass therefore I think this may be an intermittent problem that I haven't yet encountered.
The issue may go away if the Todoist API is updated. If not, I could have a go at removing the deprecated functionality.

@Garee In my previous post the code includes:
Deprecated functionality will be  completely disabled on 13 Feb 2016 at 9:00:00 GMT
Whereas now it says:
Deprecated functionality was disabled on 14 Feb 2016 at 17:00:00 GMT
Perhaps Todoist messed up the year?
If you try to run my code with multiple tasks in the input project, you don't get the error?

I ran your example with hard coded credentials and it's working fine for me. I also re-ran the test suite and they all pass.
Are you still experiencing the issue @ryqiem ?

I'm getting something similar I believe, with the login_with_api_token() command.  When I run the line
user = todoist.login_with_api_token(my_api_token)
the response I get is
  File "C:\Anaconda2\lib\site-packages\pytodoist\todoist.py", line 92, in login_with_api_token
    return User(user_json)

  File "C:\Anaconda2\lib\site-packages\pytodoist\todoist.py", line 325, in __init__
    self.sync()

  File "C:\Anaconda2\lib\site-packages\pytodoist\todoist.py", line 361, in sync
    self.api_seq_no = response_json['seq_no']

KeyError: 'seq_no'

@ryqiem @amccaugh I have released a new version that uses version 7 of the Todoist sync API.
Please upgrade the package and let me know if it works or if you have any issues.
$ pip install pytodoist --upgrade

Thanks for reporting the issue.

Tested and it looks great, thanks!

After fixing a slight change in the task attributes from due_date due_date_UTC, it's working awesome here as well!
You're the man, @Garee!


No JSON object could be decoded

Describe the bug
No JSON object could be decoded.
To Reproduce
Steps to reproduce the behavior:

udemy course url: [e.g https://www.udemy.com/course/build-olx-clone-with-python-django or https://www.udemy.com/course/microsoft-azure-fundamentals-training-bootcamp] '...'

Screenshots

Python Environment (please complete the following information):

Windows 7
Python 2.7.13

Additional
I returned to check udemy-dl 0.4 and it says that Udemy Says: You don't have permision

@fyz i will remove python2 support in next update please check if the error occurs with python3.

@r0oth3x49 working with python 3.7, thanks


Issue in passing Click Coordinates in OSD

I am building one functionality where there are two things, one normal image and one OSD viewer, i need to pass click coordinates whenever image or OSD is clicked, here is this function which gets this done:
const handleMouseClick = ( viewerRef: React.RefObject<OpenSeadragon.Viewer>,
    e: any,  click_target: string = ""
  ) => {  let el = e.target;
    console.log(viewerRef.current);
    if (el) { let x, y;
      // Check if the click was on the OpenSeadragon viewer
      if (click_target === "osd" && viewerRef.current) {
        console.log("click on osd");
        // Get the click coordinates relative to the viewer
        const rect = viewerRef.current.element.getBoundingClientRect();
        x = e.clientX - rect.left;    y = e.clientY - rect.top;
        // Convert the click coordinates from viewport coordinates to image coordinates
        const viewportPoint = new OpenSeadragon.Point(x, y);
        const imagePoint =
          viewerRef.current.viewport.viewportToImageCoordinates(viewportPoint);
        x = imagePoint.x;
        y = imagePoint.y;
        // Adjust for zoom level and pan
        const zoom = viewerRef.current.viewport.getZoom(true);
        x /= zoom;
        y /= zoom;
        console.log("x: ", x, "y: ", y);
      } else {
        console.log("click on image");
        // Handle click on the image (as before)
        const rect = el.getBoundingClientRect();
        x = e.clientX - rect.left;
        y = e.clientY - rect.top;
        console.log("x: ", x, "y: ", y); }
      const imageScale = image ? image.width / el.offsetWidth : 1;
      x *= imageScale;
      y *= imageScale;
      console.log("x: ", x, "y: ", y);
      const click = getClick(x, y);
      if (click) setClicks([click]);
    }
  };

here the else statement which is when normal image is pressed, that works with my model as I am implementing SAM model and it needs click coordinates, however click on OSD which is in else statement gives weird output which means i am passing wrong coordiantes, can someone help me to fix it?

Looks like you got an answer on Discord: https://discord.com/channels/888321246431969280/888820630865186876/1212308519278215189


readDir behavior not restored by #9605

Unfortunately, it does not look as though https://github.com/gohugoio/hugo/pull/9605 restored readDir behavior from 0.92.
I am comparing:
hugo v0.92.2-CDF6A0D6 linux/amd64 BuildDate=2022-02-11T14:17:39Z VendorInfo=gohugoio
hugo v0.93.2-643B5AE9 linux/amd64 BuildDate=2022-03-04T12:21:49Z VendorInfo=gohugoio

It looks like readDir now uses the project directory as base, instead of the theme directory—even for partials in the theme.
Confusingly, I think the new behavior is probably more correct, but it is still not the same as it was in 0.92.

I'll close this as duplicate of #9609 -- I'm not able to fix this in the near term, however, but others are welcome to chime in.

If it would be helpful, I can provide a git hash of a repository that builds with 92 but not 93.
We've now worked around the issue by using assets throughout.


CLOUDSTACK-10012: ServerDaemon customization

add option to allow loading another webapp (cloudstack-ui for example)
remove jetty in param keywords as it's better to stay agnostic

cc @rhtyd


frontend: stats.ripe.net/geoloc/data.json?resource=AS1200 ERROR

Describe the bug
Artemis performs the following query when hovering over the hijack's AS number:
https://stat.ripe.net/data/geoloc/data.json?resource=AS1200 (example AS 1200 here).
This gives:

  |
-- | --
status | "error"
server_id | "app009"
status_code | 400
version | "1.0"
cached | false
see_also | []
time | "2020-03-02T15:39:26.683994"
messages |
0 |
0 | "error"
1 | "AS1200 is of unsupported type ASN. It should be an IP prefix."
data_call_status | "supported"
process_time | 0
build_version | "2<IP_ADDRESS>"
query_id | "20200302153926-5f6a0298-302c-420e-ad6e-9dc049bba5cf"
data | {}

It seems IP prefix needs to be given as the parameter no ASN.
Affected Component(s)

[ ] Back-End (Database, Microservices, Containers, etc)
[x] Front-End (Flask, API, etc)
[ ] Docs
[ ] Build System

To Reproduce
Steps to reproduce the behavior:

Go to 'hijacks'
Click on 'however over the as number of a hijack'
Fire up the the developer tools of your browser and look at the return codes of queries performed to stats.ripe.net
See error

Expected behavior
AS info should be fetched and presented when howering over ASN.

Thanks for pointing this out @pgigis is already working to fix this.

Hi,
thanks for reporting the issue. It seems that RIPE Stat modified their API.
I pushed a fix https://github.com/FORTH-ICS-INSPIRE/artemis/commit/548fbe5d232bd41bed4a21d8382adf7bd33abf5b which resolves the issue.
If you are using the docker containers please pull the new images.
Petros

There was an issue with the building of the docker containers.
The problem should be now fixed.
I am closing this issue.

Hi Guys,
thx for fixing.
it seems that RIPE has fix the API and the call I reported works again.
I just noticed it while planning to upgrade my artmis instance that things work again...
https://stat.ripe.net/data/geoloc/data.json?resource=AS1200

  |
-- | --
status | "ok"
server_id | "app026"
status_code | 200
version | "1.0"
cached | false
see_also | []
time | "2020-03-12T10:06:13.694019"
messages | []
data_call_status | "supported"
process_time | 441
build_version | "2<IP_ADDRESS>"
query_id | "20200312100613-64d806eb-c745-4cd3-9324-d6a7992714ea"
data |
located_resources |
0 |
resource | "1200"
location | "NL"
result_time | "2020-03-11T00:00:00"
latest_time | "2020-03-11T00:00:00"
earliest_time | "2005-02-18T00:00:00"
parameters |
query_time | "2020-03-11T00:00:00"
resource | "1200"


TelemetrySignals WebSocket Endpoint does not contain PowerPlantID

In the signals that we get back from the WebSocket endpoint, the PowerPlant ID is missing! Please add it:
{
"activePower" : "20.0",
"isOnOff" : "false",
"isAvailable" : "true"
}

The bug got even worse! The WebSocket is not giving back any TelemetrySignals after adding the PowerPlantId to the set of signals!

Fixed the issue!


Fail to install on Mac OS

I install this on my mac, met this error. No errors on wsl2 ubuntu

Originally posted by @ilaipi in https://github.com/Avimitin/nvim/issues/26#issuecomment-1060139468

I don't have macbook or other stuff that can run mac os. So this might be hard to figure out what happen. @ilaipi

Maybe it's because the network, after I retry PackerInstall several times, the plugins are all installed succesfully.

Maybe it's because the network, after I retry PackerInstall several times, the plugins are all installed succesfully.

Can you remove all the plugins and cache and then run a fresh installation?

I run a fresh installation at the begining by run :
# plugins directory
rm -rf ~/.local/share/nvim

# neovim cache file
rm -r ~/.cache/nvim

# neovim plugins load sequence
rm -r ~/.config/nvim/plugin

(from the installation guide)
And I just try one more time , the same error.

So this might not be a network issue. I will try to figure it out. (See if I can borrow a MacBook from someone else.)

I've asked some of my friends, and they said they got no error during the installation. So you might need to find out the issue yourself.

And yes, it is related to the network. I've read the source code from jobs.lua file, and it occurs when killing the process.

Ok, let's close this, thanks!


Can't uninstall: sh: line 65: [: ==: unary operator expected

michal@10:57:06:~/dev/bis (support/strong-params)$ curl get.pow.cx/uninstall.sh | sh
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100  4389  100  4389    0     0   9414      0 --:--:-- --:--:-- --:--:--  9398
Sorry to see you go. Uninstall Pow [y/n]?
sh: line 65: [: ==: unary operator expected

hm, my bad. I didn't enter 'y' or 'n', just hit enter.

You can close issues yourself. =]


feat(WEG-95): descriptions in cards

This PR adds the facility's descriptions to the preview cards. We crop the content after 3 rows of text.

:tada: This PR is included in version 1.2.0-rc.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

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


How We Work styling

A couple of little things to sort out:

Needs more padding at the top and bottom
The pattern background can be swapped out for trees-pattern-grey, and the other img files can be deleted
The image and the text are quite close to each other in the centre. Could they be moved apart a bit? I'm not sure exactly how they should look, but I'd use figma and the other sections as a guide and play around with it

Otherwise looks great

Fixed in another issue


Raising error for source exception

Handle cases when the file is not a valid media file.

it is work!! thank you

@mattapperson any chance we could merge this guy soon?

+1

Hey @mattapperson, any chance we could have this guy merged soon?

Any updates on this? It would be really useful for me to have access to this?


[Groups Infra] Move all API Keys to be group scoped

https://www.notion.so/dust-tt/Engineering-e0f834b5be5a43569baaf76e9c41adf2?p=5d15a65989da4b11aa99ed1c213f9fb3&pm=s
We want the auth to be bound to the user now
The system API key remains user-less and should have read/write permission for all ACLTypes

An API key is created by a user by assigned a list of groups it is part of for access verification. The user can set groups he has access to.

Existing API keys will be scoped to the workspace group by default.

We will likely want to create a follow-up task: if a user is removed from a group, api keys tied to him should be invalidated

We likely don't want to invalidate API keys created by users if they are removed from a group as it could break systems. Interesting questions. Likely more a warning for the admin?


log/slog: export Source method in Record to support custom handler implementations

Go version
go version go1.23.1 linux/amd64
Output of go env in your module/workspace:
Not relevant

What did you do?
Implement a custom handler for the log/slog package. My custom handler requires access to the source location (file and line) of log entries for detailed logging.
What did you see happen?
The source method in slog.Record is not exported, meaning it’s inaccessible to custom handlers. As a result, I had to re-implement logic to retrieve source location information, which led to redundant code and reduced consistency with Go's built-in logging behavior.
What did you expect to see?
Expected an accessible Source method in slog.Record that allows custom handlers to retrieve source location information without re-implementing the existing internal logic. This would simplify the creation of custom handlers and ensure consistent behavior when accessing log entry sources.

Related Issues

log/slog: Source type is public but Record.source() function is private? #66791 (closed)

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

Do we need to export it when it's just a single call to runtime.CallersFrames?

@seankhliao Yes, it's just a single call, but having it exported prevents each developer creating a custom handler from having to re-implement the same logic.

CC @jba

I can't think of a good reason not to export it. It's not much code, but it's also not code that we would expect most people to be comfortable writing. The fact that it's a method and not a field is enough of a clue that it could be expensive.
I'm in favor of this.
/cc @aclements @rsc


noise makers from ben

https://www.facebook.com/groups/puredata/posts/10159793833909495/

repositories are here:
https://github.com/ben-wes/perlin.3d-
https://github.com/ben-wes/simplex.3d-
We could also add 2d versions, which will be faster but less flexible, of course.
The control rate versions might also be interesting, but I might need to improve them a bit since I think they're far from ideal performance-wise.
The simplex noise doesn't have a help file yet - but it's almost similar. For simplex, the values on each axis repeat after a value of 768 (whereas it's 256 for perlin).

Not sure if this is relevant here - but maybe interesting to know: simplex noise was patented. But it's obviously expired right in time:
https://patents.google.com/patent/US6867776B2/en


api tweaks

Can leave the old functions in while updating the docs and adding new aliases.
parentSel
I usually do var sel = d3.select('.graph').html() at the start of my render function. d3.conventions({sel}) is cleaner than d3.conventions({parentSel: sel}).
appendMany
I think dataAppend is easier to teach; the first argument is the data, the second is what you're appending. Or manyAppend?
ƒ
Not sure this is necessary anymore with ES6. Maybe remove from the examples?

Maybe arrayAppend?

how about just appendMany("div", data) ?

for a grace period we could add in some code that detects whether the string argument is first or second and output a console.warning("DEPRECATED: jetpack's appendMany order of arguments has changed. it's appendMany("div", data) now");


WebGL: INVALID_OPERATION: delete: object does not belong to this context when hot-reloading

Just a minor issue. Not sure if this is critical now or later.
Steps to Reproduce

Create a new Flutter project.
Choose Chrome as debug device.
Play around. See on browsers console. (maybe only on save/auto-reload)

Expected results:
No warning on console.
Actual results:
The warning appears mulitple times on console. It seems this only happens when app hot reloads.

WebGL: INVALID_OPERATION: delete: object does not belong to this context (canvaskit.js:231)

No demo / minimal project required. Just start a new project. There are no changes made. I just moved the home page to a new file. But this is meaningless.
I have found another issue about that (#112467) but won't help. The guy wrote that it was his fault, not flutter. But I doubt it. Don't sure what he mean with "data" in this context. I've just created a fresh new project on a fresh installed OS and fresh installation of Chrome.
Windows 11 Pro 22H2
VS Code 1.74.2
Flutter 3.3.10
Chrome 108.0.5359.125

Hi @twoco, does this issue always reproduce, or is it intermittent?
Can you also share a recording of how you reproduce the issue and the full error logs?
Thank you

I can always reproduce it. I don't think that a record helps in this case. And I have no error log. The error occurs only on Chrome. It's maybe a (security) change of the browser. Maybe something set to deprecated and should be changed in the future.
Steps to Reproduce
Below I describe my case. The most important things are running Flutter project in Chrome.

Check/Update your Software. See "Test Environment" below.
Create a new Flutter project. (Existing project may also work.)
Select "Chrome (web-javascript)" as debug device and run the project (F5 in VS Code).
Wait  until it's fully loaded and open the browser dev tools console.
 (CTRL + S)
Take a look to the browser console. You should now see the warning. It appears on every reload.
A hard-reload removes the warning. It only appears on hot reload.

Result
On CTRL + S:

WebGL: INVALID_OPERATION: delete: object does not belong to this context (canvaskit.js:231)

Test Environment
Windows 11 Pro 22H2
VS Code 1.74.2
Flutter 3.3.10
Chrome 108.0.5359.125
Maybe the OS and IDE doesn't matter. You only need the latest Flutter and Chrome browser.
Additional Information
Searching the web with this warning message brought me more information:
I found another Flutter issues (may not be related): #91881
There are also other threads on various forums. But I haven't found any official documentation about it yet.
It's the Flutter CanvasKit at methods like deleteProgram, deleteBuffer and deleteFramebuffer. I'm not sure (no experience in WebGL), the warning message is about doing delete objects which do not belong to the current context. What does this mean? Keeping context where the objects were created? For these delete methods? Maybe related to Skia? This is the lib behind, isn't it?

Hi @twoco, thanks for the detailed info. I am able to reproduce this on the stable branch. However, when I attempt to reproduce this on the master branch, I see the error below(it sort of comes in a flash, I had to use VLC to see the error properly).
Do you see the same thing on the master channel?
Error during script reloading. Firing full page reload. HotReloadFailedException: 'Mismatched anonymous define() module: function() { return CanvasKitInit; }' http://requirejs.org/docs/errors.html#mismatch'


add Brooklyn Plugin

send CLA by email

@robertgmoss Thanks for the plugin.
Because of how the commands are namespaced cf brooklyn <command>, the help won't be properly displayed. cf brooklyn login -h will just return the help for command brooklyn. In order for the help to be displayed, the commands will have to in args[0], for example, cf brooklyn-login, cf brooklyn-push.

@robertgmoss Two things:

To clarify Simon's comment above - You can opt to do nothing and those commands won't have a callable help associated with them or you can change the command names as suggested to enable the invocation of help for them.  We are waiting to hear which you want to go with.

For the brooklyn command itself, it says typing cf brooklyn will show help:  "Brooklyn plugin command's help text" but typing cf brooklyn actually returns
runtime error: index out of range because the CLI is looking for the next argument which it expects you to supply on cmdline.

You've got the CF CLI team hear to help so let us know what we can do.

Thanks.  This is a good reminder that I need to sort the help out.
I was thinking of implementing the help command in the plugin. I think it would be nicer than changing all the commands to brooklyn-*, especially for push.

hopefully this is acceptable now

Thanks, looks good. Merged!


Can the text field and the search field be combined?

It would be great if the text field is also the search field, ~otherwise after clicking the text field, the user needs to click again to actually start searching.~
I see there is a searchAutofocus, but visually it would be better if they are the same field.
It works even better when combined with #30.
Thanks!

@Kimi-Arthur , you can enable search on the main textfield by setting readonly to false.


Disable windows support for k8s 1.14

Per Will, we would like to only support k8s 1.15 and above for Windows cluster

Need to verify the upgrade case for Windows cluster from k8s 1.14 to k8s 1.15 before disable it

May I request to know the reason we want to disable Windows Support for k8s 1.14.x? or could you enter this in the issue description?

Rancher version: master-head (v2.3) (08/29/2019)
Windows version looks enabled for kubernetes version 1.15.x and up in latest master.
If I pick Kubernetes version 1.14.x or below Windows support is always disabled regardless of Network provider. ✅


ReferenceError instead of NameError?

Feature

When toying around with the demo, I saw that ReferenceError is raised when referring to an undefined variable.
>>>>> abc
ReferenceError: "name 'abc' is not defined"

Python Documentation

While on the surface this reads fine, ReferenceErrors are raised in the context of weakref references which "go away". ReferenceError's docstring says: "Weak ref proxy used after referent went away."
CPython3 (and 2) uses NameErrors:
>>> abc
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
NameError: name 'abc' is not defined

Not sure if the demo's out of date or if this is still an issue.

Ah, no, it's because rustpython_wasm converts errors to Javascript errors when they're thrown, and the demo just displays those errors. The ReferenceError there is actually a Javascript ReferenceError (mdn)

Ah, I was guessing something similar might be in effect.

Should this be closed then?

@dralley I'm still not sure if the behavior incidental or by design. Probably worth a line in some documentation somewhere. If it's by design and already documented, then feel free!


[T3A4][T09-A2]Clarence Chee

Ready for review.

@cheec Some comments added. Please close the PR after reading comments

@nirandiw doesn't the boolean variable re-instantiate with each function call? if so then there should be only one assignment per call


Rename cluster-autoscaler helm destination repo

Issue #, if available:
Epic: https://github.com/aws/eks-anywhere/issues/3186
Description of changes:
Renames cluster-autoscaler's helm repo to align with convention.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/approve


[import/order] weird behavior when upgrading to 2.27.0

I was on 2.26.0 and was trying to update to 2.27.4 but started getting a weird error in 2 of the files on my project.
Here's my config for the import/order rule:
		"import/order": ["error", {
			"groups": [	"builtin", "external", "internal", "sibling", "index", "parent", "type"],
			"newlines-between": "always",
			"alphabetize": { "order": "asc", "caseInsensitive": true }
		}],

Here is what my file currently looks like - this was fine in 2.26.0

This is what the plugin wants it to look like

There are many issues with this:

"../commands/Utility/poll" appears right after the builtin modules, before external ones
"../config.json" appears after the discord.js external module import but before mongodb
"../events/crowdinverify" appears after mongodb but before node-cron, both external modules

On a different file I have a less obvious but equally weird error:
This is what the file looks like currently

This is what the plugin wants the file to look like:

Here I believe the bug happens because index and config are in the same directory, however, I only want index to show up before sibling files, and config should show up after as it is in the parent directory
I have files with similar structures where this is not an issue, so I'm not sure why it is on these 2 files.

This may or may not be related to #2669. Since I'm not sure, I decided to make a new issue about it.

I think this was fixed by #2674. Can you try v2.27.5 and see if it works?

There are still issues with this rule but those existed before 2.27.0 so I'll consider this fixed. Thanks!


zmwfilter very slow

Hi,
Filtering 14,000 ZMW holes from an 800 Gb subreads BAM file using 'zmwfilter' is taking > 3 hours. That seems too slow. Is there any speed improvement, or any other way to do this?

It's very likely your file system. There's nothing I can think of. You could parallelize by having maybe 100x 140 ZMWs and then samtools merge?

I don't think so. I'm on an SSD drive. Filtering a small number of variants is super fast, i.e. < 1 minute. Increasing to thousands of variants causes a stand-still.
Also reported by another user: https://github.com/PacificBiosciences/zmwfilter/issues/1

I'll file it internally. We'll come back when we have a solution.

Thanks. This is a bit urgent for me, because I can't complete my analysis without this. So I appreciate any help.

Note some more data, and a work-around I found in the meantime:

Some more tests show the time per hole extraction is linear, ~0.2 sec/hole on the fastest disk/machine. However, this is too slow for large numbers of holes. It would take 5.5 hours for 100,000 holes.

A single pass through the full subreads.BAM using this awk script is faster than trying to extract about > 20,000 holes with zmwfilter. This script below takes the same amount of time regardless of the number of holes that are extracted, but is slower for about < 20,000 holes than using zmwfilter.

samtools view -@8 -h subreads.bam | awk 'BEGIN{idx=1;matched=0}{if(FNR==NR){holes[NR]=$1}else{split($1,hole,"/");if(hole[2]==holes[idx]){print $0;matched=1}else{if(matched==1){idx++;matched=0;if(hole[2]==holes[idx]){print $0;matched=1}}}}}' holes.txt - | samtools view -b - > output.sam

#holes.txt = numerically sorted hole numbers, one per line

I've rebuild with the latest dependencies. 20k ZMWs are now being filtered in 45s on my machine. Please let me know if that works and I'll upload to bioconda:
https://github.com/PacificBiosciences/zmwfilter/releases/tag/v1.0.0

Works great! Please let me know once on bioconda.

Actually, there's some issue I think. I'm trying to 'samtools index' the resulting BAM, but it isn't working. It outputs a 16 byte BAM index. Maybe something is malformed in the BAM output.

samtools index on an unaligned BAM always results in an empty BAI.

Ah, yes, correct. My mistake.

It's in bioconda. Still as version 1.0.0, though the internal version is 1.2.0. Will correct that at some point.

Just confirming, 1.0.0 on bioconda now is the same as the release version you linked to in github?
(i.e. 1.2.0 is just a labeling issue, and not a different version?)
Thanks.

Oh, nevermind. I see that it is labeled in bioconda as 1.0.0, but the binary version output is 1.2.0.


License?

Just getting into Bolt, and rather than reinvent the wheel was looking at this, just wondering if the code is licensed under any OSS license?

The license is MIT. I'll add a license.md, because it seems i've forgotten to do so. Thanks!

Awesome!


407 Error when using who am I function

Hello, I was testing out this link: https://learn.microsoft.com/en-us/power-apps/developer/data-platform/webapi/use-postman-web-api. My goal is to use the REST API for Dataverse to be able to update and manage tables. Whenever I make a WhoAmI call in postman, I get a 407 error. The error is as follows

I configured postman correctly and recieved a bearer token:

I believe I followed the tutorial for the environment correctly. What might this bug be?

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

ID: d25c7db3-badf-d848-1cca-9e5fe3390504
Version Independent ID: 70c9e985-9dbc-6415-5983-c551b05b0c87
Content: Set up a Postman environment (Microsoft Dataverse for Apps) - Power Apps
Content Source: powerapps-docs/developer/data-platform/webapi/setup-postman-environment.md
Service: powerapps
Sub-service: dataverse-developer
GitHub Login: @divka78
Microsoft Alias: dikamath

Thanks for your comment, @ElijahMansur. This issue has been assigned to the doc author for review.

@JimDaly the stack exchange didn't work when I tried it. Maybe my computer has a network restriction that causes this error? I'm not entirely sure. What else should I check?

You should be able to send the GET request via your browser and it should work. Have you tried that?
I'm not a network security expert, but the error seems to be coming from a proxy server managed by honda.com. Is that your domain? You may want to check with your network administrator to understand if this is intentionally added to control access.

Yeah, when I try using the generic get request by using the browser, I get this window.

@ElijahMansur
I'm going to assume that this works on your personal network like #3678


Please invite me to the GitHub Community Organization

Name
Psami
Discord Username (if applicable)
No response
Additional Context
No response

Welcome to Chryzhub @Psami-wondah 🥳🎉


Only include the renderer-preference in builds where SVGGraphics is defined

After PR #13117 it's now (finally) possible for different build targets to specify individual options/preferences, and we can utilize that to only expose the renderer-preference in builds where SVGGraphics is actually defined.
Note that for e.g. MOZCENTRAL-builds, trying to enable SVG-rendering will throw immediately and the preference thus doesn't make sense to include there.
Also, update the dummy SVGGraphics to use a class, tweak the PDFJSDev-check in src/display/svg.js to agree fully with the option/preference, and remove an unnecessary eslint-disable.

Thanks!


:bug: Fix: Rerelease all packages

Description
Concise description of proposed changes
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

#1200  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @roninjin10 and the rest of your teammates on  Graphite


Stop polluting ::Array and ::Hash

Requiring simlpecov globally adds the merge_resultset method to the built-in Array class and Hash class via monkey-patching, which means this gem widely changes the test subject's behavior.
 We should immediately stop doing this.
I suppose the same merging logic could be more safely and cleanly implemented by two or three static methods instead of extending the core classes, but for now, I'd suggest a patch that changes the Result class to dynamically extend the result objects using the existing Modules.
This way we would be able to minimize the conflict with other PRs working on the merging_helpers e.g. #436 and #441.

The build failed because of Rubycop, there is #448 for that.

As to the actual topic: I totally agree with you, we must stop doing this, thanks for pointing it out!
We still need to remove the actual includes from the source https://github.com/colszowka/simplecov/blob/master/lib/simplecov/merge_helpers.rb#L37-L38 - would you mind doing this? I would expect that with the existing changes this should keep working?

@colszowka

We still need to remove the actual includes from the source

I believe that was actually included in the previous commit, and at least the specs and features were passing locally with that changeset!

I'm an incapable of reading diffs it seems. Thanks! Merging! I also created #450 for the followup work

Lol, awesome branch naming :monkey_face: :gun:


Make a small automatic screenshot delay when someone get's a pet

Is your feature request related to a problem? Please describe.
When you got a pet it will stand in your character, so you can't see the pet at the moment runelite takes a screenshot
Describe the solution you'd like
Make like a 2 seconds delay when someone get's a pet
Describe alternatives you've considered
N/A
Additional context
N/A

Never mind, it only appears with some pets that not have enough space, close pls


Suggestion :: add  culture support

It isn't an issue, but a suggestion for multilanguage use.
Can you add omnisharp.Culture option in vscode extension with format "en-En" or "ru-RU" and add a capability to parse it into your omnisharp-server?
something like
///<summary>
/// This is Description
///<summary/>
///<summary xml:lang="ru-RU">
/// Это - описание
///<summary/>
public class Test {}

If you will leave null in omnisharp.Culture hover will show "This is Description". Otherwise, if you'll set omnisharp.Culture = "ru-RU" hover will show "Это - описание"
And you will make it for other tags, such as param and other - it would be awesome!

DocComments are part of the C# language. You could work up a proposal and share it on csharplang for consideration. See https://github.com/dotnet/csharplang?tab=readme-ov-file#proposals


cardinality of tsm level labels too high

https://github.com/influxdata/influxdb/issues/11969 was not quite implemented correctly in https://github.com/influxdata/influxdb/pull/13353
We should not have a level label for any TSM level as that could significantly increase the cardinality for that metric, and not add a huge amount of value.
Instead, as stated in the ticket, we should have levels {1, 2, 3, 4+}.

Fix in https://github.com/influxdata/influxdb/pull/13698 and one more in https://github.com/influxdata/influxdb/pull/14181


Add instance url scopeModel

What is the purpose of the change
Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #9290 (334946f) into 3.0 (858f8a9) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                3.0    #9290      +/-   ##
============================================
- Coverage     64.70%   64.67%   -0.03%
  Complexity      329      329
============================================
  Files          1206     1206
  Lines         51844    51848       +4
  Branches       7716     7717       +1
============================================
- Hits          33546    33535      -11
- Misses        14685    14696      +11
- Partials       3613     3617       +4

Impacted Files
Coverage Δ

...ache/dubbo/registry/client/InstanceAddressURL.java
51.17% <ø> (-0.23%)
:arrow_down:

...vent/listener/ServiceInstancesChangedListener.java
79.91% <100.00%> (+0.41%)
:arrow_up:

.../remoting/transport/netty4/NettyServerHandler.java
58.13% <0.00%> (-9.31%)
:arrow_down:

.../common/threadpool/serial/SerializingExecutor.java
70.37% <0.00%> (-7.41%)
:arrow_down:

.../org/apache/dubbo/rpc/protocol/tri/WriteQueue.java
68.75% <0.00%> (-6.25%)
:arrow_down:

...org/apache/dubbo/rpc/protocol/AbstractInvoker.java
69.15% <0.00%> (-2.81%)
:arrow_down:

...ng/zookeeper/curator5/Curator5ZookeeperClient.java
63.28% <0.00%> (-2.42%)
:arrow_down:

...ting/zookeeper/curator/CuratorZookeeperClient.java
68.75% <0.00%> (-1.93%)
:arrow_down:

...a/org/apache/dubbo/monitor/dubbo/DubboMonitor.java
85.71% <0.00%> (-1.91%)
:arrow_down:

...he/dubbo/registry/zookeeper/ZookeeperRegistry.java
65.26% <0.00%> (-1.20%)
:arrow_down:

... and 8 more

Continue to review full report at Codecov.

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

https://github.com/apache/dubbo/issues/9403

I think getModuleModelsFromUrl is a little bit weird. It can be better if the module model can be passed through the API method.


[BUG] MSIX Install Link Failure

Describe the bug
New installations using the "one click" install link are nonfunctional due to settings.json not existing. This has been reproducible on <IP_ADDRESS>3
To Reproduce

On a fresh Win 10 instance, use the one click link to install
Error:

Additional context
Underlying issue has been identified and a fix is in progress. Due to its nature, this is added to 1.2 requirements.

Fixed in f8e87b3156e78664650c47795dafddd3a25fe379.

Fixed in f8e87b3156e78664650c47795dafddd3a25fe379.


Adding a contributing readme

Simpathey, you should look into adding a contributing.md to the root of the repo. When creating a PR Github will link to it, and good stuff to have in there are any code of conducts if you have one, coding conventions, how to run the project, how and when you should create a PR (e.g. create and discuss in an issue or something like that before creating a PR), etc. etc.

Looks like Vicious is on it with #4 so review that

Yup. :)


[feature request] Stdin Support for URLs(Target)

In current version of ffuf we only have stdin support for wordlist, it would be easy to pipeline ffuf if we can have a stdin input for URLs also. Something like below:
echo "https://target.com/FUZZ" | ffuf -u - -w wordlist.txt

Hi @BitThr3at,
ffuf already supports that mode, you just need to read from stdin. Here's an example but you can build a wrapper around it if you want a cleaner method to call it.
$ echo "https://api.github.com/FUZZ" | for item in `cat /dev/stdin` ; do ffuf -k -w wordlist/foo:FUZZ -fc 404 -mc 200 -t 1 -u $item ; done

I think this is more of an process issue. The URL part can already be read from the stdin with the default functionality but it applies the restriction that you cannot specify the FUZZ target in the input itself.
Like:
echo 'https://example.org/url/path' | ffuf -w -:HOST -w wordlist.txt -u HOST/FUZZ ...


Cookbook: demonstrate SignalPlotConst Update()

SignalPlotConst requires the contents of the arrays are constant (hence the name)
However, Update() functions allow data in the arrays to be changed. Add a cookbook examples demonstrating this.
Suggested by @damien-ls in #2222

whoa that already exists https://scottplot.net/cookbook/4.1/category/plottable-signalconst/#signalconst-data-updates


✨ NEW: Use external markdown-it plugins

Replaced the (very adhoc) local markdown-it plugin for directives, with https://github.com/executablebooks/markdown-it-docutils, and in general it works 🎉 ! (some notes and todos below)

TODOS:

[x] markdown-it-docutils has not yet implemented code/code-block/code-cell directives; once that is done there will be > parity with the plugin it is replacing

I noted that, incorrectly, {code-block} does not get parsed as a directive, which needs to be fixed, i.e. the regex needs to include -

[x] The required markdown-it plugins should be added (i.e. front-matter and footnote)
[x] The additional myst extensions should be added (dollarmath, amsmath, deflist, ...), and turned on/off via a setting
[x] Currently I have just copy/pasted markdown-it-docutils/dist/css/style.min.css to ./styles.css, but it would be good to have this updated automatically
[x] For some reason the admonition icons are not showing correctly, I also noted this when looking at https://executablebooks.github.io/markdown-it-docutils/ on my mobile (ios)

Note this is not the case for Furo itself (where the icons come from), so I must be missing something from there 🤷

[ ] when you place a cursor, or highlight some text, in the source text, you can see above that the line it is on is also indicated in the preview. However, for admonitions their styling appears to obstruct it.

Note that it does show up for "indented" components like lists;

This also highlights the importance of correctly setting Token.map for all the (block) nodes

[x] Currently the CSS (taken from https://github.com/pradyunsg/furo, see https://github.com/executablebooks/markdown-it-docutils/blob/main/src/style/_variables.scss) uses @media (prefers-color-scheme: dark) to change between light and dark themes. This doesn't necessarily play well with the VS Code themes, so should think about either (a) if this is the best approach in the first place and/or (b) if we should override some of these variables.

@rowanc1 it would be great if you give this a try (the dev instructions are in the README); it would be good for you to understand this use case for the markdown-it plugins, and maybe even get some idea for curvenote 😄
also cc @pradyunsg (as mentioned I borrowed some of your sass from Furo 😄)

Just seeing this in the css -- https://caniuse.com/css-masks is not in chrome/webkit which this uses.
Mine currently look like this in Chrome here: https://executablebooks.github.io/markdown-it-docutils/:

(changed the mask property on the second one...)

Ah cheers, that could be it then 👍

Ah so furo uses https://github.com/postcss/autoprefixer (via gulp) during the css compliation, which auto-adds vendor prefixes

After update to markdown-it-docutils v0.10.0:

Sorry to dig up this old PR. It doesn't look like {toctree} directives are supported or properly rendered.


fix(log-level): use default value if value is empty

Problem:
Update log-level value and then update it back to empty. We will get error message in status.
Solution:
Use default log-level value if the value is empty.
Related Issue:
https://github.com/harvester/harvester/issues/3340
Test plan:

Update log-level setting to debug
Update log-level setting to empty.
There should not have error in status.

@mergifyio backport v1.1


fix: punch a hole in Compose UI to blend with HTMLElement

#215  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Improve timestamp-cli

Description

Review the output
Review the input params
Make sure there's no leftovers from Rekor
Add timestamp verification into CLI

Changing to a p0 - I'm going to define p0 as something we want to get done before an initial release of the project.

Remaining task is to add an inspect flag


"Do not pretend to be the owner" error

Setup step 3 shows "Do not pretend to be the owner" if authorized user is not the same with the config, but the check is originally on the client side and not using the server-side environments.

fixed in e7eb86db5c435b2cc230097251eedbea3b197800


Free code camp won't let me pass this task: Give a background color to a Div

Challenge Give a Background Color to a Div Element has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:53.0) Gecko/20100101 Firefox/53.0.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

<link href="https://fonts.googleapis.com/css?family=Lobster" rel="stylesheet" type="text/css">

CatPhotoApp

Click here for cat photos.

Things cats love:

cat nip

laser pointers

lasagna

Top 3 things cats hate:

flea treatment

thunder

other cats

<form action="/submit-cat-photo">
  <input type="radio" name="indoor-outdoor" checked> Indoor
  <input type="radio" name="indoor-outdoor"> Outdoor
  <input type="checkbox" name="personality" checked> Loving
  <input type="checkbox" name="personality"> Lazy
  <input type="checkbox" name="personality"> Energetic
  <input type="text" placeholder="cat photo URL" required>
  <button type="submit">Submit</button>
</form>

I did everything as asked but it won't let me pass no matter what I do.

Your class name is not correct. You have used a point as separator in silver-background.

omg thanks a lot

Hi,
Thanks for reporting this issue.
Here are some next steps that you should follow before we can help you with your query:

Check these troubleshooting steps first.
Please use the Chat Rooms or try looking through our forum for help with a specific challenge and any coding assistance.

IMPORTANT:
Please NOTE that this tracker is for reporting bugs and enhancements to freeCodeCamp's code base only.
Happy Coding!


[Bug] The topic list div covers the cards below it

Description
The div which contains the topic, medium, and language list in the all.html file covers/flows on top of the thumbnail box below them.

Reproduction

visit the link on a laptop: https://resources.creativecommons.org/all/?

Expectation
The site is expected to adjust the positioning and sizing of elements according to screen size for better accessibility.
The issue arises because the list div overlaps the cards, leading to accessibility issues. One of the critical problems with the current CSS properties is that the list div overlaps the cards on different screen sizes, causing a poor user experience. To resolve this issue, we can implement responsive design techniques with CSS styling to add some space between the cards and the elements above them. This will ensure that the list div does not overlap the cards
Screenshots

Environment

Device: Laptop
OS: op!_ OS 21.10
Browser: Chrome
Version: Version 111.0.5563.64 (Official Build) (64-bit)
Other info: display resolution: 1920 x 1080

Resolution

[x] I would be interested in resolving this bug.

I am not able to reproduce this on my PC. Could you try hard reloading the page? and see if the error still exists.

@adityakm24 I am not able to produce this error as well. Could you provide more details?

I just tried it using my laptop running OS: op!_ OS 21.10 and in incognito mode using chrome. The issue still persists

@adityakm24 Chrome incognito mode with the same dimensions.

I have adjusted and tried all dimensions but wasn't able to reproduce. I think the bug only exists on your machine which is weird. Could you check the website in a different browser? Maybe firefox?

@Xaid-vfx Same here.
@adityakm24 Could you share a screen recording for this, following these steps -->

Load the https://resources.creativecommons.org/  page.
Click on See all.
See if there is any error in the console.

I haven't been able to reproduce either, yet. Marking as work required just in case it is occuring.

Hello @possumbilities ,
Since Submit a resource  link is added, there is enough space and padding between the cards and the filter links. I could not reproduce this issue as well. Lemme know your thoughts.

This should be resolved with the new Vocabulary implementation now.


Possible bug when parsing a float with more than one decimal

Hi!
It seems that the number: "5.0535017175947585" (String) does not get validated as a Number, the regex: /^[0-9]+(?:.[0.9])?$/ seems to only parse one decimal (source: regex101.com)
A possible fix could be: ^[0-9]+(?:.[0-9]*)?$  , although I must say i'm not an expert at regular expressions.
Greetings!

A better regex can be found here: http://ntt.cc/2008/05/10/over-10-useful-javascript-regular-expression-functions-to-improve-your-web-applications-efficiency.html (negative numbers als well)

Thanks @Fonz001. This is definitely a bug. I'll fix that ...

Fixed in 34af404f97c8b2b1840540f168b5c1048da5f220


Project GetRelativePath improvement

Optimize GetRelativePath for trivial cases (very intensive usage in PM tree)
Do not consider path sharing the same drive letter as relative (ie. on C:\ common) - should fix #655

I could test this if you could me few examples of a bad paths.

I was thinking this would be worth starting adding unit tests :)
The tests should be:

faster: paths are equals,
faster: path is contained in base directory (typically relative paths of files within the project folder),
faster not relative: paths have different drive letter,
consider not relative: paths only share the drive letter (typically classpath in very different location, like a haxelib VS your project's folder).

The last case is one reason we can have very long relative paths because the paths share nothing but the drive letter so I think it makes sense to not consider them as relative. For example:

relative path of C:\g\h\i\j\k\l from C:\a\b\c\d\e\f is ..\..\..\..\..\..\g\h\i\j\k\l
recombined as absolute path becomes C:\a\b\c\d\e\f\..\..\..\..\..\..\g\h\i\j\k\l
this essentially doubles the path length can lead to the exception.

I was thinking this would be worth starting adding unit tests :)

If so, please, consider XUnit or NUnit before MSTest.

@Neverbirth feel free to recommend your preference.


[BugFix] _compaction_running would not be reseted when _check_for_apply() can't submit task

Signed-off-by: zombee0<EMAIL_ADDRESS>What type of PR is this：
bug
Which issues of this PR fixes ：
Fixes:
TabletUpdates::compaction() will check _compaction_running and set it to true, so that only one compaction can run at the same time.  do_apply() of compaction_commit_task will change the _compaction_running to false, and then next compaction can run.
But when _check_for_apply() returned without submit the compaction_commmit_task because of _apply_running, do_apply would not execute, in this scenario,  _commit_compaction() would still return ok, and then _do_compaction() would execute _wait_for_version() and wait until timeout and then return ok, _do_compaction() would return ok and would not deal with _compaction_running, and compaction() finished and left _compaction_running = true, the next compaction can’t go on.
Problem Summary(Required) ：
fixed the bug

I have a question.  If _do_compaction return OK, which means we have pushed a new edit version into _edit_version_infos , if _check_for_apply return directly because of _apply_running is true, which means apply_thread is running do_apply(). And this function does not return until all committed versions have been applied. It seems that the committed compaction_version will be applied at last and _compaction_running will be set to false at last.

I have a question. If _do_compaction return OK, which means we have pushed a new edit version into _edit_version_infos , if _check_for_apply return directly because of _apply_running is true, which means apply_thread is running do_apply(). And this function does not return until all committed versions have been applied. It seems that the committed compaction_version will be applied at last and _compaction_running will be set to false at last.

you are right, I missed this.


Use attrs for lots of the return values

There are lots of functions that currently return a tuple. Let's think about them returning dataclasses with attrs.

I like and will open a branch

I have some second thoughts here. While I like the idea of returning data classes. I would prefer increases in speed, if possible. In case Cython or mypyc #25 dont result in proper speed-ups and/or seem like a pain to maintain relative to performance I am happy to go in the direction of attrs. What are your thoughts on this @thht ?

ok this makes sense!


Update flyway-core to 7.5.2

Updates org.flywaydb:flyway-core from 6.5.7 to 7.5.2.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.flywaydb", artifactId = "flyway-core" } ]

labels: library-update, semver-major

Superseded by #94.


Perspective work kill workspace

I have been using perspective for a few weeks now. Pretty neat package, but I have encountered a problem. Whenever I try to invoke persp-kill I get the following error:
run-hooks: Wrong number of arguments: (1 . 1), 0

Here's my configuration
(use-package perspective
  :config
  (setq persp-state-default-file "~/.emacs.d/persp-confs/persp-auto-save")
  (persp-mode)
  :bind (("s-b" . persp-counsel-switch-buffer)
         ("s-M-b" . persp-ivy-switch-buffer)
         ("C-x b" . persp-switch-to-buffer*)
         ("C-x k" . persp-kill-buffer*)))

(add-hook 'kill-emacs-hook #'persp-state-save)

Sure, persp-killed-hook returns (treemacs--on-scope-kill) which returns nil when evaluated.
The perspective installation is up to date. I tried reinstalling fresh copies more than once before I submitted this issue.
Based on the return value, and having treemacs and treemacs-perspective packages installed, do you think this might be an integration issue between these three packages?

Sure, persp-killed-hook returns (treemacs--on-scope-kill) which returns nil when evaluated.
The perspective installation is up to date. I tried reinstalling fresh copies more than once before I submitted this issue.
Based on the return value, and having treemacs and treemacs-perspective packages installed, do you think this might be an integration issue between these three packages?

It crashes because of the way treemacs-perspective calls the hook.  :)
Try this implementation of persp-kill, which will just overwrite the one in Perspective with one that calls persp-killed-hook with an argument:
(defun persp-kill (name)
  "Kill the perspective given by NAME.

Killing a perspective means that all buffers associated with that
perspective and no others are killed."
  (interactive "i")
  (if (null name) (setq name (persp-prompt (persp-current-name) t)))
  (with-perspective name
    (condition-case nil
        (run-hooks 'persp-killed-hook)
      (wrong-number-of-arguments (run-hook-with-args 'persp-killed-hook name)))
    (mapc 'persp-remove-buffer (persp-current-buffers))
    (setf (persp-killed (persp-curr)) t))
  (remhash name (perspectives-hash))
  (persp-update-modestring)
  (when (and (persp-last) (equal name (persp-name (persp-last))))
    (set-frame-parameter
     nil 'persp--last
     (let* ((persp-sort 'access)
            (names (persp-names))
            (last (nth 1 names)))
       (when last
         (gethash last (perspectives-hash))))))
  (when (or (not (persp-curr)) (equal name (persp-current-name)))
    ;; Don't let persp-last get set to the deleted persp.
    (persp-let-frame-parameters ((persp--last (persp-last)))
      (persp-switch (persp-find-some)))))

Let me know if it works. Even if it does, though, IMO it's a bug in treemacs-perspective. It expects treemacs--on-scope-kill to be useable directly as a hook, but persp-killed-hook never advertised an interface with a parameter. It's a simple fix on the treemacs-perspective side to wrap treemacs--on-scope-kill with (lambda () (treemacs--on-scope-kill persp-curr)) (or better yet, a dedicated named function).

It crashes because of the way treemacs-perspective calls the hook.  :)
Try this implementation of persp-kill, which will just overwrite the one in Perspective with one that calls persp-killed-hook with an argument:
(defun persp-kill (name)
  "Kill the perspective given by NAME.

Killing a perspective means that all buffers associated with that
perspective and no others are killed."
  (interactive "i")
  (if (null name) (setq name (persp-prompt (persp-current-name) t)))
  (with-perspective name
    (condition-case nil
        (run-hooks 'persp-killed-hook)
      (wrong-number-of-arguments (run-hook-with-args 'persp-killed-hook name)))
    (mapc 'persp-remove-buffer (persp-current-buffers))
    (setf (persp-killed (persp-curr)) t))
  (remhash name (perspectives-hash))
  (persp-update-modestring)
  (when (and (persp-last) (equal name (persp-name (persp-last))))
    (set-frame-parameter
     nil 'persp--last
     (let* ((persp-sort 'access)
            (names (persp-names))
            (last (nth 1 names)))
       (when last
         (gethash last (perspectives-hash))))))
  (when (or (not (persp-curr)) (equal name (persp-current-name)))
    ;; Don't let persp-last get set to the deleted persp.
    (persp-let-frame-parameters ((persp--last (persp-last)))
      (persp-switch (persp-find-some)))))

Let me know if it works. Even if it does, though, IMO it's a bug in treemacs-perspective. It expects treemacs--on-scope-kill to be useable directly as a hook, but persp-killed-hook never advertised an interface with a parameter. It's a simple fix on the treemacs-perspective side to wrap treemacs--on-scope-kill with (lambda () (treemacs--on-scope-kill persp-curr)) (or better yet, a dedicated named function).

Yes this worked thanks.
I thinks it's a treemacs-perspective issue too.

Yes this worked thanks.
I thinks it's a treemacs-perspective issue too.

Great. I want to open a Treemacs PR to sort this out. Here's my patch: https://github.com/gcv/treemacs/commit/c536ec2e24594c311ffab1a0fdfc9f1af6f848a3
Could you please give it a quick spin before I open the pull request? It seems to work for me, but I never used treemacs-perspective before (though I sometimes use just treemacs by itself) so it'd be good to have you look over it. You'll have to revert the definition of persp-kill from the one I posted above to the one Perspective ships with.

Great. I want to open a Treemacs PR to sort this out. Here's my patch: https://github.com/gcv/treemacs/commit/c536ec2e24594c311ffab1a0fdfc9f1af6f848a3
Could you please give it a quick spin before I open the pull request? It seems to work for me, but I never used treemacs-perspective before (though I sometimes use just treemacs by itself) so it'd be good to have you look over it. You'll have to revert the definition of persp-kill from the one I posted above to the one Perspective ships with.

@gcv just tested it and it seems to be working great.

@gcv just tested it and it seems to be working great.

@Randy1Burrell: This bug should now be fixed in treemacs.

@Randy1Burrell: This bug should now be fixed in treemacs.

@gcv great, thank you.

@gcv great, thank you.

Closing!!

Closing!!


API client can view minimal stats for a Process

Is there a related GitHub Issue?
#175
What is this change about?
Adds a new stats endpoint for process
Does this PR introduce a breaking change?
No
Acceptance Steps
Follow acceptance on #175
Tag your pair, your PM, and/or team
@acosta11 @Birdrock

Perhaps fixup the two commits together?

@Birdrock I have removed the kustomize and makefile changes. This PR now purely addresses #175


Search results are not clickable

I don't know if this is an issue or an enhancement, but to me it would make sense to click the result item on the search view and be taken to the item page/view by. Currently, when I click on an item of the search results list what happens is that the keyboard goes away. Only then the items become clickable.
Sorry if the explanation is bad, I can try to improve it upon request 🙂

Hey @regisfrias, thanks for pointing this out. I would have sworn that you click immediately already, but when I tried it out it does indeed take two clicks to go to the search result. You're absolutely right. Let me see if I can take a look at why this is going wrong.

Well, it's no big deal 🙂 I just have been using the app a lot and, since I can't help with the code I'll overflow the issues page 😆

No worries, thanks for catching it! Should be something pretty simple


ER_BAD_FIELD_ERROR: Unknown column 'john' in 'where clause'

Version: 2.18.1
I receive the titled error while attempting to perform a prepared query containing WHERE ... IN (...)
let users = [ "john", "doe", "becky"]

await MysqlConnection.query(`
    SELECT *
    FROM users
    WHERE users.username IN (${  users.fill(0, users.length, '?').join(',')  });
`, users)

The exact SQL from exception is:
\n\t\n\tSELECT *\n\tFROM users\n\tWHERE users.username IN (john,doe,becky);\n\t\t\t\t
... which leads me into thinking the escaping is not done correctly. It should be:
\n\t\n\tSELECT *\n\tFROM users\n\tWHERE users.username IN ('john','doe','becky');\n\t\t\t\t
Is there a workaround for this?

Hi @tgbv I'm assuming you think users.fill(0, users.length, '?').join(',') returns the string ?,?,?, but it does not:

In other words, your statement
    SELECT *
    FROM users
    WHERE users.username IN (${  users.fill(0, users.length, '?').join(',')  });

Is actually
    SELECT *
    FROM users
    WHERE users.username IN (john,doe,becky);

and you have no ? placeholders in there at all. You can see this if you add in your code the following:
let users = [ "john", "doe", "becky"]

console.log(`
    SELECT *
    FROM users
    WHERE users.username IN (${  users.fill(0, users.length, '?').join(',')  });
`)

But you don't even need to form a list of placeholders for an array in this module. I'm not sure if you are using this module, though, as await is not usable here, so I don't want to provide an example that may not actually work for you. But you can probably see above, though, at least that the invalid string is actually being created by your code before the .query method gets it.

This is embarrassing! 😂  Thank you for explanation

No problem at all. And don't feel embarrassed; we all have to start somewhere and if you hesitate to ask questions because you think it may be embarrassing, it may be harder to progress in your pursuit for knowledge.


Image component is overly generous asking for permissions

Describe the bug

The Image component's Picture property asks for READ permission for any path on the "external" storage partition. However, for Android SDK 19+, this is not needed for any files within the app-specific storage directory. This can lead to apps asking for permission when none is needed.
The easiest way to mitigate would likely be to replace the conditional of the if statement at the top of the Picture method with a call to RUtil.needsFilePermission, which encodes a more robust set of rules about when permission is needed.
Affects

[ ] Designer
[ ] Blocks editor
[x] Companion
[x] Compiled apps
[ ] Buildserver
[ ] Debugging
[ ] Other... (please describe)

Expected behavior

App Inventor should not prompt the user for permission to read external files if none is actually needed.
Steps to reproduce

See this thread

#2716

I want to work on this issue, please assign me this issue.


Enable Envoy proxy sidecar to terminate TLS traffic and initiate a mTLS connection with the other sidecars in the mesh

Describe the feature request
We are a major financial organization that leverages micro-services in a Kubernetes cluster communicating with each other using TLS currently (Note - not mutual auth).

The problem we are trying to solve is that we would like to make use of mTLS capabilities of Istio, between micro-services and using SSL between the micro-service container and the Envoy Proxy sidecar.

In this scenario, we would also like to be able to scrape HTTP metrics at the envoy sidecar for observability.

Use case: Service A calling Service B.

Current approach: Service A initiates a one-way TLS connection, which is terminated at the Envoy sidecar.  Envoy proxy sidecar of Service A then initiates a mTLS connection with the envoy proxy of Service B.  And then, the envoy proxy of Service B terminates the mTLS connection and initiates a one-way TLS with Service B container using self-signed certs.

Service A --> TLS --> Envoy Proxy of Service A --> mTLS --> Envoy Proxy of Service B --> TLS --> Service B

For enabling this capability in Istio, we are extending Istio by making use of the custom EnvoyFilter configuration as seen in the code block below.

Ideally, we would like have this capability built into Native Istio to avoid having to use custom EnvoyFilter configurations.  Custom configurations can go out of date quickly in a version upgrade, and require special maintenance between versions.

If this could be built into Native Istio, we can avoid having to do this.

apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
  name: custom-envoy-filter
spec:
  configPatches:
    - applyTo: CLUSTER
      match:
        context: SIDECAR_INBOUND
        cluster:
          portNumber: 443
      patch:
        operation: MERGE
        value:
          transport_socket:
            name: tls
            typed_config:
              "@type": type.googleapis.com/envoy.api.v2.auth.UpstreamTlsContext
              common_tls_context:
                tls_certificates:
                  - certificate_chain:
                      filename: /etc/certs/custom/cert.pem
                    private_key:
                      filename: /etc/certs/custom/key.pem
                validation_context:
                  trusted_ca:
                    filename: /etc/certs/custom/ca-chain.cert.pem
    - applyTo: FILTER_CHAIN
      match:
        context: SIDECAR_OUTBOUND
        listener:
          portNumber: 443
      patch:
        operation: MERGE
        value:
          transport_socket:
            name: tls
            typed_config:
              "@type": type.googleapis.com/envoy.api.v2.auth.DownstreamTlsContext
              common_tls_context:
                tls_certificates:
                  - certificate_chain:
                      filename: /etc/certs/custom/cert.pem
                    private_key:
                      filename: /etc/certs/custom/key.pem
                validation_context:
                  trusted_ca:
                    filename: /etc/certs/custom/ca-chain.cert.pem

Here the custom certs are mounted in the sidecar proxies using the Istio annotations.

Describe alternatives you've considered

As described above, we are using the EnvoyFilter configuration available in Istio to accomplish this task.

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ X ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ X ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Additional context

subscribe

I think this is the main user feature request of the sidecar API? @rshriram would be nice to compile all PRs ref this one. Or duped this one closed.

This didn't make it into 1.6. Moving to 1.7 where it will have to be re-evaluated.

I'm not sure why you want to use a sidecar - what you are describing is a Gateway. Do you need the envoy to be running
alongside an app ? Technically this should work - with changes to the injection template or manual injection - we can make it easier if it's common, but running an envoy configured as Gateway next to a workload is possible, and is pretty similar with 'whitebox' mode where iptables are not used.

Chiming in here because this is related to my Envoy and Istio objectives. I think the benefits of using a sidecar over a gateway is the simplicity granted to the application container, allowing it to be completely ignorant of TLS from a hosting perspective, while ensuring that no unencrypted traffic flows across a node (only within a pod). One of my objectives was to allow for simple or mTLS between all pods communicating within the mesh, with the sidecar terminating TLS when it proxies the call into the application container. If possible, it would obviate the need for our developers to add any kind of TLS support at the application level.

@sanketdayma96 , may I know how can you inject the cert into the envoy proxy ? Are you manually inject the cert into the envoy proxy into container or have any other setting to config envoy proxy can inject cert when deploy ?

@sanketdayma96 , may I know how can you inject the cert into the envoy proxy ? Are you manually inject the cert into the envoy proxy into container or have any other setting to config envoy proxy can inject cert when deploy ?

I created a configmap consisting of the certs and mounted it in the sidecar using the userVolume and userVolumeMount annotations. Hope this helps.

@sanketdayma96 , may I know how can you inject the cert into the envoy proxy ? Are you manually inject the cert into the envoy proxy into container or have any other setting to config envoy proxy can inject cert when deploy ?

I created a configmap consisting of the certs and mounted it in the sidecar using the userVolume and userVolumeMount annotations. Hope this helps.
@sanketdayma96 ,
Would you mind tell me what is the parameter in configmap to consist the certs ? Is it need to create a new one or using existing istio configmap ?
In addition, I have tried below method but cannot inject cert into sidercar ..
https://stackoverflow.com/questions/57343583/how-can-i-use-custom-client-certificate-for-external-service-with-istio

@sanketdayma96 , may I know how can you inject the cert into the envoy proxy ? Are you manually inject the cert into the envoy proxy into container or have any other setting to config envoy proxy can inject cert when deploy ?

I created a configmap consisting of the certs and mounted it in the sidecar using the userVolume and userVolumeMount annotations. Hope this helps.

Thanks you! I have success to insert the cert into sidecar proxy. For MTLS between A and B istio proxy, are you apply the PeerAuthentication , AuthorizationPolicy and the envoy filter then can enable the HTTPS between istio proxy ?

I'm not sure why you want to use a sidecar - what you are describing is a Gateway. Do you need the envoy to be running alongside an app ? Technically this should work - with changes to the injection template or manual injection - we can make it easier if it's common, but running an envoy configured as Gateway next to a workload is possible, and is pretty similar with 'whitebox' mode where iptables are not used.

Can you explain this?

@sanketdayma96 Hey, I am trying to use your filter for the same purpose, but for some reason envoy is not terminating SSL from my Service A to Service B. Sidecar is doing Passthrough since the traffic encrypted not by sidecar

What is the status on this issue of sidecar being deployed in hybrid mode? I see that the PRs have been merged. Is it now available in an Istio version?
I was also looking for guidance on how to configure the Envoy sidecar in this new hybrid mode to terminate TLS and originate mTLS / simple TLS to the destination service. Is there sample documentation for how to achieve this sidecar configuration?


Release branches need proper controls

I created 0.5 release branch, and am able to push to it without review.  Maybe a bad thing?

All release branches now have protection.


Horreum: Include environment related information

Implement logic for including environment related information in JSON file, i.e. the used database type, Multi AZ vs single-AZ, with or without external Infinispan, Keycloak version, etc.
Update the schema Labels in Horreum correspondingly.

This task also includes the update to the existing JSON structure.

Environment data is properly into Horreum report using configmap. Please find attached the example of JSON report which was generated yesterday during test run.
result-20240114-231327-4ec405b5-28f5-4444-9c4e-f8176394d287.json

Here is the updated one updated with additional aurora database information:
result-20240115-161429-2cbca6d0-d864-4893-a68e-ad7c38d15f80.json

Sent a PR: https://github.com/keycloak/keycloak-benchmark/pull/679


Provide Detailed README Content

Problem Description
The current README file consists of only three dots (...), which does not offer any information or guidance for users. A comprehensive README is essential for users to understand the purpose of the project, how to use it, and any other relevant information.

Thank you for raising the concern regarding the minimal content in the current README. We appreciate your feedback, and we acknowledge the importance of providing clear and comprehensive information for users.
In response to this, we plan to enhance the README content in the upcoming commits. The revised README will include detailed sections covering the project's introduction, installation instructions, usage examples, contributing guidelines, licensing information, and support details.

...


error for HM-CC-TC when service climate.set_hvac_mode is set to OFF

Home Assistant version (if applicable):
2020.12.7
0.118.3
callstack:
Logger: homeassistant.components.websocket_api.http.connection
Source: components/homematic/climate.py:148
Integration: Home Assistant WebSocket API (documentation, issues)
First occurred: 5:24:23 PM (3 occurrences)
Last logged: 5:26:49 PM

[139771418984256] %i format: a number is required, not NoneType
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/websocket_api/commands.py", line 135, in handle_call_service
    await hass.services.async_call(
  File "/usr/src/homeassistant/homeassistant/core.py", line 1451, in async_call
    task.result()
  File "/usr/src/homeassistant/homeassistant/core.py", line 1486, in _execute_service
    await handler.job.target(service_call)
  File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 204, in handle_service
    await self.hass.helpers.service.entity_service_call(
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 499, in entity_service_call
    future.result()  # pop exception if have
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 664, in async_request_call
    await coro
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 536, in _handle_entity_call
    await result
  File "/usr/src/homeassistant/homeassistant/components/climate/__init__.py", line 430, in async_set_hvac_mode
    await self.hass.async_add_executor_job(self.set_hvac_mode, hvac_mode)
  File "/usr/local/lib/python3.8/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/src/homeassistant/homeassistant/components/homematic/climate.py", line 148, in set_hvac_mode
    self._hmdevice.turnoff()
  File "/usr/local/lib/python3.8/site-packages/pyhomematic/devicetypes/thermostats.py", line 47, in turnoff
    self.writeNodeData("SET_TEMPERATURE", self.OFF_VALUE)
  File "/usr/local/lib/python3.8/site-packages/pyhomematic/devicetypes/generic.py", line 346, in writeNodeData
    return self._setNodeData(name, self.WRITENODE, data, channel)
  File "/usr/local/lib/python3.8/site-packages/pyhomematic/devicetypes/generic.py", line 363, in _setNodeData
    LOG.error("HMDevice.setNodeData: %s not found with value %s on %i" %
TypeError: %i format: a number is required, not NoneType

Describe the bug
when homeassistant service climate.set_hvac_mode is used with older wall thermostat HM-CC-TC
an error is thrown
TypeError: %i format: a number is required, not NoneType
reason is perhaps, that HM-CC-TC  does not understand SET_TEMPERATURE, but only SETPOINT
Expected behavior
temperature is set to minimum value (6°C)
Additional context
only applies to HM-CC-TC, not the newer HM-CC-RT-DN

I think the issue has at least 2 different underlying causes:

The error logging in _setNodeData could handle the channel=None case more gracefully, so the error message would be more useful. Maybe we can just a %s instead of %i in line 363?
https://github.com/danielperna84/pyhomematic/blob/483e593d4ce29598b10dbccbe8c5b2b5fdab53d8/pyhomematic/devicetypes/generic.py#L351-L365

The homematic component handles the automatic detection of the correct datapoint to use when setting the temperature ("SETPOINT" vs "SET_TEMPERATURE"):
https://github.com/home-assistant/core/blob/4ec81d4b6749932d3a998440972da0c554d264bb/homeassistant/components/homematic/climate.py#L185
        self._state = next(iter(self._hmdevice.WRITENODE.keys()))
and then uses it when setting the temperature:
https://github.com/home-assistant/core/blob/4ec81d4b6749932d3a998440972da0c554d264bb/homeassistant/components/homematic/climate.py#L133-L139
    def set_temperature(self, **kwargs):
        """Set new target temperature."""
        temperature = kwargs.get(ATTR_TEMPERATURE)
        if temperature is None:
            return None

        self._hmdevice.writeNodeData(self._state, float(temperature))

That's why regular setting of the temperature works in the homeassistant component. But it would not work using pyhomematic directly because SET_TEMPERATURE is hardcoded:
https://github.com/danielperna84/pyhomematic/blob/483e593d4ce29598b10dbccbe8c5b2b5fdab53d8/pyhomematic/devicetypes/thermostats.py#L36-L43
Same for the turnoff function that is called when setting HVAC_MODE_OFF:
https://github.com/danielperna84/pyhomematic/blob/483e593d4ce29598b10dbccbe8c5b2b5fdab53d8/pyhomematic/devicetypes/thermostats.py#L45-L47
I don't have a quick solution right now (for the 2nd issue), but I hope this helps in deciding how to go about resolving the issue. The code in the homeassistant component and pyhomematic for the thermostats is pretty duplicated (probably to fix an issue directly in the component that should have been fixed in pyhomematic), but let's see what the most pragmatic way is to tackle this ;-)

Should be fixed with the new integration.


changes in CGNv6 librararies and examples

Hi Mike,
I performed quite a lot of changes in libraries and created examples for some basic  CGN use case, which I put under  ansible-etc/
Many of the changes in library scripts can be addressed by SDKgenerator, but some should be considered more carefully.

This PR is far outdated at this point, so I'm going to close it. Changes to modules need to be made to the objects.j2 template in the sdk-generator.
Example playbooks and module changes should also be submitted in the form of 2 distinct PRs.


Remove unneeded "root" classes from the oscal classes

Issue description / feature objectives
Simple oscal classes consisting of a single __root__ object in common.py have had all their references simply refer to the regex defined by the root field.  Currently the unused classes are still present in common.py and serve to show the original class structure and provide context for the regex patterns.  If they cause confusion since they aren't actually referenced in the code they can be removed in the future.

All the root's have been removed related to classes in the common.py file but roots remain in the other oscal class files.  They can be removed later if desired, but they are much less of a nuisance than the ones in common.py, so this issue is closed.


rancher upgrade causes rke2 cluster to get stuck in provisioning after upgrade

Rancher Server Setup

Rancher version: Upgraded from 2.6.2 to 2.6.3-rc4
Installation option (Docker install/Helm Chart): Helm

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): RKE2

Proxy/Cert Details:

Information about the Cluster

Kubernetes version: v1.21.6+rke2r1
Cluster Type (Local/Downstream): Downstream; amazon rke2 with rhel os

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):

Describe the bug
After upgrade, cluster goes to 'provisioning'/waiting and does not recover.  The 3 machines in my cluster stay in provisioning state, with machine pools showing 'waiting on probes calico, etcd, kube-apiserver and various related messages.
Provisioning log full of:
12:32:19 am | [INFO ] provisioning etcd node(s) ron-rhel-3-ron-rhel-3-78c8b55fcc-mqvkc: waiting on probes: kube-controller-manager 12:32:21 am | [INFO ] provisioning etcd node(s) ron-rhel-3-ron-rhel-3-78c8b55fcc-h6fcq,ron-rhel-3-ron-rhel-3-78c8b55fcc-mqvkc 12:33:03 am | [INFO ] provisioning bootstrap node(s) ron-rhel-3-ron-rhel-3-78c8b55fcc-vtpz7: waiting on probes: kube-apiserver 12:33:15 am | [INFO ] provisioning bootstrap node(s) ron-rhel-3-ron-rhel-3-78c8b55fcc-vtpz7: waiting on probes: kube-controller-manager 12:33:21 am | [INFO ] provisioning bootstrap node(s) ron-rhel-3-ron-rhel-3-78c8b55fcc-vtpz7: waiting on probes: calico, kube-controller-manager
To Reproduce

Rancher 2.6.2 HA cluster
create amazon rke2 cluster with RHEL 8.4/20.10.x
Perform helm upgrade to 2.6.3-rc4

Result
cluster is in provisioning state and does not change to active
Expected Result
cluster is upgraded and active/availalbe
Screenshots

Additional context

Closing this - the issue was due to me runnning the incorrect pytests on my rke2 cluster which resulted in my pods being out of resources.
To verify this is not an issue I

created a 2.6.2 ha rancher
created 3 rke2 clusters - 2 with rhel 1 with ubuntu
ran preupgrade tests
upgraded rancher to 2.6.3-rc4
upgraded k8s version
ran post upgrade tests


Error on localhost when updating user profiles but not on production server

I apologize for all of my tickets recently. I feel like all my Userena problems are happening at once! :)
When I try to update a user on my local development, I get this:
MultiValueDictKeyError at /admin/auth/user/129/
"u'userena_signup-0-id'"

I do not get this error on my production environment for whatever reason.
My full traceback is here: http://dpaste.com/28C95YR
I have no idea where to begin; it's weird to me seeing as it's happening local but not on my live site for whatever reason.

I fixed this one. Wasn't a Userena problem. Ended up being an issue with Django Suit.
Details here for anyone who may come across this: http://stackoverflow.com/questions/19741314/multivaluedictkeyerror-in-django-admin


Code Smell: Long Method (Bloater)

There's a Long Method code smell in /Tone/source/oscillator.ts. The offender is the _getRealImaginary method which is over 300 whopping lines of code. We can fix it using the Extract Method refactoring technique.

Sorry, I forked this repo for a class assignment and accidentally added this issue to the actual repo.


Add ukrainian university to list

Adding Poltava State Agrarian University ( Ukraine) into university list . Thx for this opportunity for students

Adding poltava agrarian university

@Poprop First stage of verification passed, official domain is pdau.edu.ua as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@Poprop Please provide us with a proof, which shows that the school recognizes the domain pdau.poltava.ua, that you are submitting as an official email domain for the students.
You can provide us with any of the following:

A copy of instruction (a hard-copy or electronic one) given to you by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that students use this email domain
A reference to a recognized journal where an article was published by someone from the school, with explicit affiliation to the school, and e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be verified independently.

Sorry i made mistake in domain , it must be - @st.pdau.edu.ua
1.Page of university - https://www.pdau.edu.ua/
2.Page with instruction how to use corporative university mail domain   -
https://moodle.pdau.edu.ua/
video instruction page -https://youtu.be/qhMuSlfgnbs
3.Screen shot of my university domain mail box with invitations to online
lessons in google meet
[image: image.png]
ср, 22 нояб. 2023 г. в 18:33, Philip Torchinsky @.***>:

@Poprop https://github.com/Poprop Please provide us with a proof, which
shows that the school recognizes the domain pdau.poltava.ua
http://pdau.poltava.ua, that you are submitting as an official email
domain for the students.
You can provide us with any of the following:

A copy of instruction (a hard-copy or electronic one) given to you
by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that
students use this email domain
A reference to a recognized journal where an article was published
by someone from the school, with explicit affiliation to the school, and
e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that
the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be
verified independently.

—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/19377#issuecomment-1823101693,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6MGHREIFJATKE6YUQUSMJLYFYSNZAVCNFSM6AAAAAA7RT2NECVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRTGEYDCNRZGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

@Poprop No problem, thank you for the update. Pull request merged. Feel free to request free licenses with emails ending with @st.pdau.edu.ua. Please start requesting licenses in an hour, to let the changes to propagate through our systems. Thank you!

Thank you for your help!
пт, 24 лист. 2023, 09:19 користувач Philip Torchinsky <
@.***> пише:

@Poprop https://github.com/Poprop No problem, thank you for the update.
Pull request merged. Feel free to request free licenses with emails ending
with @st.pdau.edu.ua. Please start requesting licenses in an hour, to
let the changes to propagate through our systems. Thank you!
—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/19377#issuecomment-1825241876,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6MGHRANJQ5JJACH4T74FKDYGBC7JAVCNFSM6AAAAAA7RT2NECVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRVGI2DCOBXGY
.
You are receiving this because you were mentioned.Message ID:
@.***>


Put JSX 4 PPX support functions into a separate module

Currently, the React module has the functions createElementWithKey and createElementVariadicWithKey.
In my understanding, these are helper functions required for the JSX 4 PPX, but are not meant to be used by a user of the React bindings.
I would therefore propose to move them to a separate module, something like ReactPPX4Support. What do you think @mattdamon108?

@mattdamon108 there was a bit of back and forth about where to put the source of truth for these things.
Since the source of truth for types is now the compiler's Jsx e.g. type element = Jsx.element, I guess it makes sense to also put these functions there, unless there are some issues with this, which were discussed but I have forgotten?

Sorry for missing the notification.
First of all, adding Jsx is a first step to embracing other JSX libraries besides the React. Therefore, I added the primitives such as types there. Second, we removed the function such as addKeyProp was the inline issue, as I remember. https://github.com/rescript-lang/rescript-compiler/pull/5677#discussion_r975274485

Ok, so my understanding is:

the Jsx stuff is meant to be framework-neutral and is part of the compiler repo
the PPX is React-specific and is also part of the compiler repo
the PPX requires some helper functions that are currently part of @rescript/react, not of the compiler repo
addKeyProp needs to be colocated with createElementWithKey and createElementVariadicWithKey

Correct?
So my suggestion would be to move addKeyProp, createElementWithKey and createElementVariadicWithKey out of @rescript/react and into a new "PPX support module" in the compiler repo. Which means that we'd also need to duplicate the corresponding React.createElement bindings there. But that should be fine.
This way the PPX + its support code can evolve independently from @rescript/react, and users of the React bindings are not tempted to invoke some PPX-internal functions.

Yes, correct.
I agreed. If we move the helper functions out of rescript-react, then non-zero cost js output would be cleaned from it. It would make the rescript-react a binding module.

Sounds great.

@cknitt Can we close it?

Yes, done 😄


Lindalu fhl vba scrub

Docs Build status updates of commit 23b0641:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/VBA-Docs, working directory: W:\trea-s, target branch: HEAD, target branch commit: cadf72ca376985875b13c83d237814a613297553, source commit: 23b06411de80da84bc4f4c692d19b7983674678b.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 63f1b4a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Language/Reference/User-Interface-Help/asc-function.md
:white_check_mark:Succeeded
View

Language/Reference/User-Interface-Help/chr-function.md
:white_check_mark:Succeeded
View

api/Project.actualstartdrivers.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


osbuild: change legacy type from bool to string

This work is based on previous PRs, namely:
https://github.com/osbuild/osbuild-composer/pull/501
and
https://github.com/osbuild/osbuild/pull/327
The problem here is that we used to treat legacy as a boolean before we
started introducing support for alternative architectures, but now we
need to specify exact strings for the grub2 stage, for example for
ppc64le the legacy parameter looks like this:
"legacy": "powerpc-ieee1275"

This patch will allow us to introduce support for ppc64le and fix
associated issues:
https://github.com/osbuild/osbuild-composer/issues/693

Schutzbot failed on different stages this time. Last time it was only Fedora 31 image, this time it is Fedora 31 and 32 base. Should we bypass the check, @teg ?

F31 base tests failure is due to a flaky mirror. That i think we can safely ignore. The other two failures were both due to boot failing in AWS, but it also does work on one, so I think we can safely assume this to be unrelated to this PR as well. I'd have preferred not to circumvent the tests, and hopefully we'll eventually get to a place where we don't have to.


Add an account switcher

Let users switch between multiple accounts

It'd be awesome if this could support actually being logged into multiple accounts (potentially on different servers), and show the total unread counts for the accounts you're not logged into in the switcher.  I think it may make sense to implement this without that unread counts feature at first, but it's worth thinking about when we design the switcher to ensure there's space to put the unread counts :)

I am working on it right now (in connection with storing multiple accounts information).
It will be available both on startup and while logged in.
My todo:

[ ] On Start, AccountList not empty => login into last logged in account
[ ] On Logout if AccountList not empty => show list of accounts to login + button 'Add new account'
[ ] If Login in stored account fails => Set realm to the one in it, but redirect to Email + Password page
[ ] On Start, if AccountList is empty => go directly to RealmScreen

@timabbott Not sure if supported, but the 'Teams' in Slack have icons and names associated, which are needed to display a nicer list.

We have names for organizations, available as realm_name in the events register response.  We don't have icons organizations at this time.