Prepare for 6.0.3 release

Working on the 6.0.3 release...

[x] Update CHANGELOG
[x] Cut a rug tag
[x] Add tag release notes

Trying to get https://github.com/geerlingguy/drupal-vm/issues/2090 fixed too... got caught in a rabbit hole that ultimately led to finding this juicy issue: https://github.com/pyca/cryptography/issues/5771

Not going to hold up 6.0.3 on the failing Docker build. It's weird.

Did some digging into the docker fail, I think its an ansible issue but I didn't want to dig too far.
Here is what I got:

The failure is related to the postifx service which I am seeing other tickets about in the ansible issue queue (service module)
-- https://github.com/ansible/ansible/issues/73936
-- https://github.com/ansible/ansible/issues/73855
-- https://github.com/ansible/ansible/issues/73562
There was a recent change in the service module in ansible related to string compatibility which could be the culprit
-- https://github.com/ansible/ansible/commit/e0c75a6756f2c9d54239f882fed89cbeb31d2106#diff-455776c96408127a660e622f67e53963a3049430c75c2dc8f74f5c526b3cb6f5

Is there a way you can easily test this with a version of ansible from dec 2020? (2.10.4, 2.9.16, 2.8.18)?

I'll keep tracking the Docker issue here: https://github.com/geerlingguy/drupal-vm/issues/2090


Add icons beside home, about, testimonials, FAQ's , contact and login

Is your feature request related to a problem? Please describe.
The icons beside the home, about, testimonials, FAQ's, contact, and login are missing.
Describe the solution you'd like
I would like to add the icons to enhance the website.
Additional context
I am working under GSSoC'24
Please assign this issue to me @Priyaaa1

@Saimanjari777 this is not needed now


Pagination may not work correctly for archived workflows

Pre-requisites

[X] I have double-checked my configuration
[X] I can confirm the issues exists when I tested with :latest
[ ] I'd like to contribute the fix myself (see contributing guide)

What happened/what you expected to happen?
When we perform the merge of live workflows and archived workflows (in order to only show one of them in the UI), the listMeta  in WorkflowList is obtained from live workflows. This may cause issues for pagination for archived workflows.
mergedWfsList := v1alpha1.WorkflowList{Items: mergedWfs, ListMeta: liveWfs.ListMeta}

We need to test the pagination more thoroughly and revisit this part of the code. https://github.com/argoproj/argo-workflows/blob/master/server/workflow/workflow_server.go#L155C2-L155C86
Version
latest
Paste a small workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.
TBA

Logs from the workflow controller
kubectl logs -n argo deploy/workflow-controller | grep ${workflow}

Logs from in your workflow's wait container
kubectl logs -n argo -c wait -l workflows.argoproj.io/workflow=${workflow},workflow.argoproj.io/phase!=Succeeded

@sunyeongchoi  Would you like to help take a look at this? Your recent PRs on pagination (https://github.com/argoproj/argo-workflows/pull/11703, https://github.com/argoproj/argo-workflows/pull/11684) brought my attention to this part of the code.

@terrytangyuan
Of course!! I will analyze the issue at hand :)

@terrytangyuan
Hello. I would like to share the results of my analysis on the issue so far.
func (s *workflowServer) ListWorkflows(ctx context.Context, req *workflowpkg.WorkflowListRequest) (*wfv1.WorkflowList, error) {
	wfClient := auth.GetWfClient(ctx)

	listOption := &metav1.ListOptions{}
	if req.ListOptions != nil {
		listOption = req.ListOptions
	}
	s.instanceIDService.With(listOption)
	wfList, err := wfClient.ArgoprojV1alpha1().Workflows(req.Namespace).List(ctx, *listOption)
	if err != nil {
		return nil, sutils.ToStatusError(err, codes.Internal)
	}

// In this part, always return err. Because archived workflows use int type continue not string type(workflows use).
	archivedWfList, err := s.wfArchiveServer.ListArchivedWorkflows(ctx, &workflowarchivepkg.ListArchivedWorkflowsRequest{
		ListOptions: listOption,
		NamePrefix:  "",
		Namespace:   req.Namespace,
	})

	if err != nil {
		log.Warnf("unable to list archived workflows:%v", err)
	} else {
		if archivedWfList != nil {
			wfList = mergeWithArchivedWorkflows(*wfList, *archivedWfList, int(listOption.Limit))
		}
	}
...

In the given code, we cannot fetch data unconditionally in ListArchivedWorkflows. This is because the continue value used in ListArchivedWorkflows is of type int, whereas in Workflows, it's a string. As a result, it always leads to an error.
This error is being logged as follows:
server: time="2023-09-03T01:02:29.487Z" level=warning msg="unable to list archived workflows:rpc error: code = InvalidArgument desc = listOptions.continue must be int"
In essence, this problem arises because, as mentioned earlier, Workflows and ArchivedWorkflows use different pagination methods, and the issue occurs when we try to use the list options from Workflows in ArchivedWorkflows.
It would be a good idea for us to discuss how to address this issue together.
After considering various solutions, I will share them with you shortly. :)

@terrytangyuan
I have one question.
After version v3.4.10, there used to be a separate "archived workflows" page, but I'm curious why the current master branch does not have the "archived workflows" page.

Workflows and ArchivedWorkflows use different pagination methods, and the issue occurs when we try to use the list options from Workflows in ArchivedWorkflows.

Yes that's exactly what the issue is! Let me know if you can come up with solution to merge the two lists.

After version v3.4.10, there used to be a separate "archived workflows" page, but I'm curious why the current master branch does not have the "archived workflows" page.

Yes, we removed the archived workflows page and both types of workflows are now displayed in one list view.

Re-opening this since #12068 reverted #11761

This should be re-resolved by #12736 now


Adding transaction

Hi,
Do you have methods for adding transactions?
Thank you

Assuming you don't mean just downloading transactions, the API is currently read-only. If you make a pull request to add this feature, it would be most welcome however!


WPS connection ?

Hi,
Is it possible to make a connection with using WPS ?
thanks

Not currently, just WPAS, Open and WEP, but I would be interested in looking into this

I'm questioning if this functionality belongs in this library...  https://android.googlesource.com/platform/packages/apps/TvSettings/+/lollipop-wear-release/Settings/src/com/android/tv/settings/connectivity/WpsConnectionActivity.java
It seems like it would require UI and a lot of additional bloat.  My concern is that I don't want to make UI/UX decisions at the library level.  Do you have any recommendations on what you want to see implementation wise?  Otherwise I think I plan on not implementing this...really sucky answer I know and I apologize for that, but I have to pick my battles.

Feel free to reopen in the future...currently closing due to no activity


Generate event for exceptions

If the reconciler cannot parse the selector annotation to a valid
selector, we record an event so that the user will get notified.
Tested:
$ k create ns a
$ k hns create a1 -n a
$ k delete validatingwebhookconfigurations.admissionregistration.k8s.io hnc-validating-webhook-configuration
$ k create role test -n a --verb=update --resource=deployments
$ k annotate role test -n a "propagate.hnc.x-k8s.io/none"="**foo"
$ k get event -n a
LAST SEEN   TYPE      REASON                OBJECT      MESSAGE
41s         Warning   CannotParseSelector   role/test   invalid propagate.hnc.x-k8s.io/none value: strconv.ParseBool: parsing "**foo": invalid syntax

Fix #1165

/assign @yiqigao217
/assign @adrianludwin

/lgtm
/approve
/hold
/assign @rjbez17

AFAIR there are more places to surface the error event than just 1 place?

Oh good point. Maybe this logic should be moved into selectors.ShouldPropagate() so that we don't miss another place like this?

AFAIR there are more places to surface the error event than just 1 place?

Oh good point. Maybe this logic should be moved into selectors.ShouldPropagate() so that we don't miss another place like this?

selectors doesn't have an eventRecorder. This is actually the only place that we call shouldPropagate() in the reconciler, the other places are in the validator

/lgtm
/hold cancel


Glitch on macbook with m1 processor

I have two macbooks, one with an intel processor and the other with M1 processor. I am using a standard microphone.  Sounddevice shows  with the standard microphone in both cases a sample rate of 44100 and a block size of 512 (blocksize=0). But in the case of a MacBook on the M1 processor, I have a glitch effect. The glitch disappeared when I manually set the block size to 4096. What are the best practices for choosing the block size?

In many cases, the defaults should be fine, but sadly there is no guarantee. Sometimes you have to find the best settings by trial and error. If you don't mind the latency, a conservative setting would be latency=0.1, which I think is also used by Audacity (you can keep blocksize=0).
However, there might also be some improvements possible in the underlying PortAudio library, especially when dealing with newer hardware. This cannot be done on the Python side, you should ask the PortAudio maintainers: http://portaudio.com/contacts.html.

Thank you. But why, for example, does Zoom not have such problems? Their interface doesn't allow such settings. I understand that they use other frameworks, maybe it's their own development. Do you have any idea how they do it?

@dryarullin Sorry, could  I ask you how do you successfully install sounddevice at M1 mac
I met a problem with MemoryError: Cannot allocate write+execute memory for ffi.callback(). You might be running on a system that prevents this. How did you solve this problem

@dryarullin

But why, for example, does Zoom not have such problems?

I don't know, because I don't have access to their implementation nor to its documentation, it's a proprietary closed-source product.
But I can guess: there is a huge number of hardware out there, and testing all of it in all configurations is simply impossible. But testing more is of course better than testing less.
And maybe the Zoom engineers have tested the hardware you are using and the PortAudio contributors haven't?
Or maybe the Zoom engineers have just been lucky to choose some parameters that happen to work with that given piece of hardware.
I'm sure this problem can in principle be fixed, and a first step would be making the PortAudio developers aware of it. There is nothing that can be done about it here, in the python-sounddevice project.


setParent relies on methods outside the CommentInterface

FOS\CommentBundle\Model\Comment::setParent relies on setAncestors on itself, which does not exist (it can be fixed easily by adding a public abstract method though) and on getAncestors on the parent (which would require adding it in the CommentInterface

+1


Link: Update docs to make href required and update children proptype

Deploy preview for gestalt ready!
Built with commit 004ecf92aa2ea5bb6cc308d26cc7e25e230d90f2
https://deploy-preview-380--gestalt.netlify.com

Deploy preview for gestalt ready!

The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #380   +/-   ##
=======================================
  Coverage   72.58%   72.58%
=======================================
  Files          57       57
  Lines        1109     1109
  Branches      265      265
=======================================
  Hits          805      805
  Misses        221      221
  Partials       83       83

Impacted Files
Coverage Δ

packages/gestalt/src/Link.js
35.71% <ø> (ø)
:arrow_up:

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 32fb76c...ed4e852. Read the comment docs.

Closing out. Feel free to re-open if still relevant.


Add check for updated app physical_path

Description
Prior to this, updating an app or virtual directory's physical path did not trigger an app update, and therefore the only way to update a physical path was to delete and regenerate the app. I've added logic to app.rb that checks if the physical path has been updated, and if it has, runs appcmd to update the physical path.
Issues Resolved
[List any existing issues this PR resolves]
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[ ] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

I see this was fixed in 4.1.1. Closing.

Thanks for digging into this 👍


Can not compile

Hello,
I tried to compile my kernel with version 0.10.12 of this crate with the "rustc 1.65.0-nightly (eaadb8947 2022-08-27)" but I get:
LLVM ERROR: dwo only supported with ELF and Wasm
error: could not compile rustc-std-workspace-core
warning: build failed, waiting for other jobs to finish...
LLVM ERROR: dwo only supported with ELF and Wasm
error: could not compile core
LLVM ERROR: dwo only supported with ELF and Wasm
error: could not compile compiler_builtins.
I can compile it with rust 1.63.0-nightly.

https://github.com/rust-lang/rust/issues/101071

Closing as a duplicate.
Thanks.


"Second attempt to send HTTP request failed" message is unhelpful

Please see issue 2451.

done by: https://github.com/vibe-d/vibe.d/pull/2499

done by: https://github.com/vibe-d/vibe.d/pull/2499

done by: vibe-d/vibe.d#2499

done by: vibe-d/vibe.d#2499


woljif is now the protagonist.

Describe the bug
woljif is now the protagonist. After walking through the market square with him in a party, he begins to respond to all dialogues as the main character of the game, and upon returning to the tavern, the game is tightly blocked. When you try to change the composition of the squad on the world map, the lock icon weighs on woljif, but not on the main character.
The bug occurs only if you save and load on the market square with woljif in the party and with generalstalt classes for him and for the main character.
Ver <IP_ADDRESS> (Toy Box), 1.0.6d (gog)

So I ran into this. Use Arcemi Pathfinder WotR Save Editor to reset your character back to the main char. They added the feature when I ran into this issue. I find that if you are not actively leveling up turning off the gestalt check box prevents a lot of issues.

I had this happen when I mistakenly had all his classes checked as Gestalt which sets his level to 0 on a reload. If you then try to level him up when his level is 0, is makes him party leader (or anyone else for that matter). To avoid, only set one of his classes as gestalt (assuming 2 classes total). This avoids the mod from calculating his character level as 0 and the subsequent issues that result. It's possible you may be running into a different issue, but I wanted to put this here in case it helps.

So Have I been doing this wrong the whole time? You only select the checkbox for second class for gestalt and the select the primary on level up?

Yes, if you select both classes as Gestalt, it will not count either of them toward you character level, which results in character level 0 and all sorts of issues can come out of that.

I've added a button to enable you to restore your main character to the correct character, and also set up something to make it more difficult to cause this scenario. Should be resolved by #416


TESTING

This PR is only for testing the Kokoro PR build, don't review, don't merge, will be deleted eventually

Codecov Report

Merging #362 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #362   +/-   ##
=======================================
  Coverage   60.69%   60.69%
=======================================
  Files          65       65
  Lines        1659     1659
  Branches      249      249
=======================================
  Hits         1007     1007
  Misses        544      544
  Partials      108      108

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 d641fec...37e782a. Read the comment docs.


Switch testnets/chains on preferences

Although this has to be killed here, I liked the idea of being able to switch easily between networks. When we will be on the web maybe we could recycle something from here
Originally posted by @sabau in https://github.com/cosmos/voyager/pull/1792

absolutely, i agree. we will get there soon 😸

Consider #1312


TitleVisible affects Scrollable in Designer

If Scrollable is checked in the designer, you are able to scroll the screen, but if you change (check or uncheck) TitleVisible or StatusBarVisible properties, even though Scrollable property is checked, you are no longer able to scroll in the Designer, to be able to scroll again, you have to uncheck and check again Scrollable property.

I have tested this issue in both Chrome and Edge, and it appears that this problem has been fixed. The user is still able to scroll after pressing any combination of the two check boxes.
I believe this issue can be closed.

@SpencerHowell I'm also seeing that this isn't a problem. @moliata Can you verify that this still affects you and what browser you're using if that's the case?

No longer a problem, closing the issue.


Feature request: add ability to add third_party_services to monitors resources

add ability to add third_party_services to monitors resources
support for all types of monitor resources
https://github.com/site24x7/terraform-provider-site24x7/blob/main/docs/resources/website_monitor.md
https://github.com/site24x7/terraform-provider-site24x7/blob/main/docs/resources/rest_api_monitor.md
https://github.com/site24x7/terraform-provider-site24x7/blob/main/docs/resources/ssl_monitor.md
https://www.site24x7.com/help/api/#create-monitor

v0.0.1-beta.6 contains changes for configuring third party services to resources.
// (Optional) List of Third Party Service IDs to be associated to the monitor.
third_party_service_ids = [
"4567"
]
Refer the following example https://github.com/site24x7/terraform-provider-site24x7/blob/main/examples/website_monitor_us.tf


Update virtual-machines-managed-disks-types-overview.md

There are many sizes available in each type of disk, so that row should be "Max disk size" for clarity.

@dcbrown16 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@dcbrown16 Agreed, thanks for making this change and helping to approve our docs. :)
It all looks good so:
#sign-off


Is not sent (pristine) indicator/flag/state

Hello. Thank you for your library, it's great!
In some cases, we need to know that the request hasn't been sent yet at all. But getQuery always returns the default structure:
https://github.com/klis87/redux-saga-requests/blob/master/packages/redux-saga-requests/src/selectors/get-query.js#L121
const defaultQuery = {
  data: null,
  loading: false,
  error: null,
};

Just checking over data is null won't work as the backend may return, for example, either null or a model (yes, that happens ¯_(ツ)_/¯).

Is there a way to extend defaultQuery (not default data) or to understand, that the query is "pristine"?
It looks like, just adding pristine flag to defaultQuery should solve the issue as the default query is returned any time when there is no data for the query (it's not initialized/sent/etc.) in requests` state.

Hi, I am glad you like this library!
Actually in roadmap https://github.com/klis87/redux-saga-requests/issues/306 there is a task:
add loaded prop to query object, as without it it is not possible to know whether data is empty as request was not dispatched yet or because it is truly empty, this will also prevent flickering if requests is dispatched for the 1st time after component is rendered`
Regarding the idea, I think this actually be solved aboge getQuery, maybe in selector, but actually probably in reducer. Reducer could store loaded: false initially and on 1st success response it would be true and stay true. But resetRequests action could still make it going back to false.
For now I think you can solve it in 2 ways:

use action.meta.getData to transform server response and replace null with sth else, like 'empty' or whatever, then you will be able to dinstinguish 2 scenarios
If you use React, you could use useQuery or Query and wrap them with containing hook or component, with additional state flag, false initially, which you could change to true in useEffect hook

you could store those flags yourself in your own reducer, and wrapping getQuery with your selector, using reselect, reading query and your flags reducer, adding extra pristine flag to response

Hello! Thank you for your reply.
For now, it's resolved in request levels. An Axios interceptor runs a casting function if it's passed in the config. It just throws a simulated not found error.
The issue is not for a particular problem. It's very handy in many cases to have an ability to understand that the request is "pristine". I'm glad to know there is the roadmap with the feature coming!
Looking forward to new versions. This issue can be closed. :)

Hey @klis87 , It's not clear to me what the task in the roadmap is about as the query has already a loading prop. I think a good solution would be to replace the loading prop with a status one, everything is explained very well in this blog post: https://kentcdodds.com/blog/stop-using-isloading-booleans

@cluk3 In reducers there is even no loading prop, there is just pending counter, which is just integer, number of current pending requests of a given type. Boolean would be indeed buggy, as for instance sequence 1stREQUEST > 2ndREQUEST > 1stRESPONSE > 2ndRESPONSE would cause 1stRESPONSE to set loading to false despite the fact there is still 2ndREQUEST pending.
getQuery transforms pending integer as loading: pending > 0
This task is to give 1 extra information, to distinguish between:

I dont have any pending requests of a given type, and I did not even fire any request yet
I already fired request of this type, but response.data from server was null and I really want to know that I tried, but data was empty

This flag wont be necessary for response.data like empty, or [], but for null it is as by default data is null if nothing was fetched or a query was reset.
Regarding the attached article, I agree that booleans can be abused if you have many states, and state machines are better then, but in our case 90% people will need only loading flag, pristine will be just extra flag for 10% cases. I dont want all people to do if(query.loading === 'loading', just if(query.loading)

I understand now, thanks a lot for the answer and for the awesome work with the library!

@umbrella-kirill-karpov @cluk3 pristine added to queries in<EMAIL_ADDRESS>also resetRequests sets pristine back to true


Possible to get output to print out deep nested objects?

Is there anyway for the CLI to output the entire response including deeply nested objects?
Now the output becomes, for example:  requestContext: [Object]

@Guitarkalle There isn't right now. It's just meant to test if the request was successful or not. Are you using it to debug your output?

@jayair
So here I can't see the headers for instance :)

Authenticating with User Pool
Getting temporary credentials
Making API request
{ status: 200,
statusText: 'OK',
data:
{ message: 'Getting data!',
input:
{ resource: '/getdata',
path: '/getdata',
httpMethod: 'GET',
headers: [Object],
queryStringParameters: null,
pathParameters: null,
stageVariables: null,
requestContext: [Object],
body: null,
isBase64Encoded: false },
data: [ [Object] ] } }

@Guitarkalle That makes sense. This should be easy to fix. In these two instances:

https://github.com/AnomalyInnovations/aws-api-gateway-cli-test/blob/master/index.js#L179
https://github.com/AnomalyInnovations/aws-api-gateway-cli-test/blob/master/index.js#L187

We need to pass in a second argument of null to the console.dir() call (https://nodejs.org/api/console.html#console_console_dir_obj_options).
Do you want to try it and submit a PR?

@jayair
Sure, give me a few days and I can find some time to look at it:)

I fixed and created a PR for it

@bveenvliet That's awesome! Thanks!

i found this issue too haha.  Any chance of releasing a new version with the PR applied please?
Otherwise is there any other tool people are using for debugging the API output? I love the ease of cognito testing with this utility!


#1168 Fix uncaught CancellationException when stopping reporter with …

Fix uncaught CancellationException when stopping reporter with externally managed executor:

Do not try to wait the end of current reporting phase, when stopping reporter with externally managed executor, because JDK ScheduledExecutorService does not provide this feature, and handy implementation of this feature requires big amount of complex concurrent code.
Do not mock executors(where sensible) in order to make tests more realistic.

Thank you!


Change class name generation from type not from response field name

Currently the names for generated classes is taken from responseName (capitalized).
If we change it to take name from type we can avoid of nested class names collision and make it more clear correlation between GraphQL schema type naming and java classes. For example:
query {
  shop {
    collections{
      edges {
        cursor
         node {
          title
          image {
            src
          }
          products{
            edges {
              cursor
              node {
                title
                images(first: 1) {
                  edges {
                    node {
                      src
                    }
                  }
                }
              }
            }
          }
        }
      }
    }
  }
}

Right now we will have next classes:
Query.Data.Shop.Collection.Edge.Node.Product.Edge1.Node1.Image1.Edge2.Image2

If we change to take name from type we will get this:
Query.Data.Shop.CollectionConnection.CollectionEdge.Collection.ProductConnection.ProductEdge.Product.ImageConnection.ImageEdge.Image

I don't really see how this would solve nested class name collisions, because type names are even less unique than response names. A response name is guaranteed to be unique within a selection set, but you can select multiple fields that return the same type.

Yeah I guess you are right, there is no silver bullet to fix this, just wanted to eliminate this Edge1, Edge2 issue. Closing

Is there a way to map these types to their generated class names? It seems really fragile to name the types like Node, Node1, Node2 etc. because if you introduce a new node part of the query, with a new node between any of the existing ones, the names will change.
We are using extension functions to map comments on different things into a native comment type, and we've currently got duplicate code like fun ExampleQuery.Node1.toComment(): Comment = ... with the exact same implementation for each NodeN type. Maybe this is the wrong approach entirely - or maybe there's a better way, to inform Apollo when nodes map to the same internal type it can use the same classes?


Provide an ability to change the default colors of themes

Note: for support questions, please use stackoverflow. This repository's issues are reserved for feature requests and bug reports.

I'm submitting a ...

[ ] bug report
[x] feature request
[ ] support request => Please do not submit support request here, see note at the top of this template.

Do you want to request a feature or report a bug?
feature

What is the current behavior?
Currently there's no way to change themes colors via UI

If the current behavior is a bug, please provide the steps to reproduce, ideally also a screenshot or gif if it's a style issue
--

What is the expected behavior?
It's expected to be able to change colors for themes via UI

What is the motivation / use case for changing the behavior?

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

Solved with #588.


Upgrade to using netbox-5.0.0-beta.161

Upgrade to the latest version, to reduce migration headache in the future

Removed the accidental commit for the netbox postgres database.sql dump


Do not support "static propTypes"?

@shx996 It is support， look at it

change Code Style  'default' to 'class'


Google SSO

Description
Google SSO login for ThirdEye dashboard.

Uses ember-simple-auth and torii frameworks.

Extended ThirdEyeDashboardModule to support multiple idps

ThirdEyeGoogleAuthenticator Implements Authenticator interface. Supports ThirdEyePrincipal properties and sessionDTO save

Imported Google client library

Enable flag from thirdeye/thirdeye-dashboard/config/dashboard.yml authEnabled: true.

New configuration options - thirdeye-dashboard/config/dashboard.yml

Setup properties

authProvider: google # others can be ldap, azure, github, etc
tokenURL: "https://oauth2.googleapis.com/token"

TODOs:

[ ] Login disable support in Local deployment
[x] google secrets load from env variables in torii providers in thirdeye-frontend/environment.js file

Notes:

To support other IDP, eg: Azure AD

create ThirdEyeAzureAuthenticator.java , import azure dependencies and handle access token creation
enable torii-azure adapter in thirdeye-frontend/config/environment.js and torii.js
create action for azure-login

Need Frontend Enginerring support. torii framework in environment.js file is unable to read env variables. tried using dotenv and process.env.KEY. still its not working.

Following environment variables are to be created on stage and prod envs

MYSQL_USER
MYSQL_HOSTNAME
MYSQL_PORT
THIRDEYE_DATABASE
MYSQL_PASSWORD
DRUID_USERNAME
DRUID_PASSWORD
POSTGRES_HOSTNAME
POSTGRES_PORT
POSTGRES_DATABASE
POSTGRESQL_USER
POSTGRESQL_PASSWORD
GOOGLE_CLIENT_ID
GOOGLE_AUTH_REDIRECT_URL
GOOGLE_CLIENT_SECRET

Following environment variables are to be created on stage and prod envs

MYSQL_USER
MYSQL_HOSTNAME
MYSQL_PORT
THIRDEYE_DATABASE
MYSQL_PASSWORD
DRUID_USERNAME
DRUID_PASSWORD
POSTGRES_HOSTNAME
POSTGRES_PORT
POSTGRES_DATABASE
POSTGRESQL_USER
POSTGRESQL_PASSWORD
GOOGLE_CLIENT_ID
GOOGLE_AUTH_REDIRECT_URL
GOOGLE_CLIENT_SECRET


feat: Show API name in reporters

Issue
https://github.com/gravitee-io/issues/issues/APIM-3932
Description
A small description of what you did in that PR.
Additional context

Gravitee.io Automatic Deployment
🚀 A prerelease version of this package has been published on Gravitee's private artifactory, you can:

use it directly by updating your project with version: 5.4.0-apim-3932-Show-API-name-in-reporters-SNAPSHOT
download it from Artifactory here

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

5.4.0
GitHub release

Your semantic-release bot :package::rocket:


Leftwards arrow in syncAfterBuild message.

I just want to be able to quickly understand what the state is by looking on logs without reading words.

It is pretty hard to understand this when you read the text.

Ok, let's close it for now.


Use a Func for isRunning in Program.cs

Since the isRunning variable is updated using the same method call and comparison, it would make more sense to save it as a delegate and just call it twice.
Alternatively, we should investigate if there is still any use for the comparison being run a second time, as the newer bot structure may not require such a check.

Switched to a local function in https://github.com/the-mighty-mo/BotBase/commit/dfb2f859198dfa69c4410c2d8f2b4b90bd5655ca. Looking into if we need to call it twice.

For now the mentioned fix is fine. We can revisit this later if we decide.


⚠ Bofrost.at has degraded performance

In 9f3af1d, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1361 ms

Resolved: Bofrost.at performance has improved in 84589fd after 7 minutes.


Added new site: Chromatic Live

I think there’s inconsistent whitespace here but YOLO!!!!

@zachleat Damn I would have fixed that whitespace for you! Thanks for merging! 💟


Blue best color

Bro make a module that sets system accent to blue

Sure, maybe with next update i will add Pixel blue


optimizer: remove panic and return error if got invalid expression

Fixed panic if Optimizer can't optimize the expression. It happens because the user can define an invalid query that can be optimized, but if Parser fails, Optimizer calls panic.
Related issue: https://github.com/VictoriaMetrics/VictoriaMetrics/issues/4422
Signed-off-by: dmitryk-dk<EMAIL_ADDRESS>

@dmitryk-dk , thanks for the fix!


Test ArgoCD health script with ArgoCD sources

[x] wait for #285

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


Extra "" characters listed for paths to files

Near the top of the article we list a path for files that include extra "" :
The XSDs for all EAP methods are shipped in the box and can be found at the following locations:
C:\Windows\schemas\EAPHost
C:\Windows\schemas\EAPMethods

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

ID: c8b1b81d-3c35-0315-d149-f10af3a409b2
Version Independent ID: c34e7acb-ece7-d76f-fb41-470e19119ee0
Content: VPNv2 CSP - Windows Client Management
Content Source: windows/client-management/mdm/vpnv2-csp.md
Product: w10
Technology: windows
GitHub Login: @ManikaDhiman
Microsoft Alias: dansimp

@tomdan-MSFT . Thanks for pointing out.


Get net_tcp.c to compile under Fedora 21

Here is an example build under Fedora COPR using the update .spec file: https://copr.fedoraproject.org/coprs/portante/rsyslog-v8.15/build/148888/


Rendering Point Clouds

Currently there is no means of rendering point clouds in unity. Something like pcx or unity's p.system would be useful. I took a pass at implementing this type of object into the converter in this pr. #32

Thanks dude, this is a busy week, hope to give it a looksie soon! :)

#32


Ai generated template

Added buttons for social media links
Improved mobile compatibility
re arranged columns to stacked vertical
nav bar aligned to the left side of the page
fixed nav bar hover problem when on mobile


pull request for e2e test at 01/22/2017 08:15:12

:white_check_mark: Validation status: passed
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.


cleanup: perform cleanups for clippy

Make cargo clippy happy and remove unwanted code.

This PR should be good but I still need to perform testing so don't merge till i test every case.
/hold

@flouthoc: PR needs rebase.

ackd i'm testing this. hold still

@baude Sure.

@flouthoc is this ready to go?

@flouthoc please wrap this up

@baude Apologies for the delay. This should be good to go now.

/lgtm

/hold cancel


Create issue for failed actions triggered by branch/release trigger

There is an interface for showing status/logs for branch/release builds, but there is no space for more details (what we put to the comments in PRs), no notification and no way to retrigger.
Long-term solution of this #780

AC:

[ ] create an issue in case of an error

Issue has a consistent name (e.g. [packit] build for branch master failed) and can be reused/reopened.

[ ] putting a /packit build comment will retrigger the build (same for tests)
[ ] allow user to create this issue as well (ideally, retrigger also without need to create a comment) (nice-to-have = can be done later)

Counter proposal:

Use commit comments instead. The problematic part is the matching from commit to trigger. (One commit can be HEAD of multiple branches and also a release.)

This will take some time but we still want to do it.

Yes, we want this. Would be nice to implement re-triggering together with this.

This would be tough work but would improve the code and will allow a lot of UX improvements. So, reopening, but not sure when we have some time for that.


->join() is gone

https://github.com/facebook/hhvm/commit/80d0af7 kills ->getWaitHandle()->join(). Need to replace the two calls to it with \HH\Asio\join()
vendor/facebook/xhp-lib/src/core/Element.php:23:53,56: Could not find method join in an object of type WaitHandle (Typing[4053])
  /tmp/hh_server/hhi_1b7ab862/interfaces.hhi:831:36,45: This is why I think it is an object of type WaitHandle
  /tmp/hh_server/hhi_1b7ab862/classes.hhi:109:16,25: Declaration of WaitHandle is here
vendor/facebook/xhp-lib/src/core/Primitive.php:22:53,56: Could not find method join in an object of type WaitHandle (Typing[4053])
  /tmp/hh_server/hhi_1b7ab862/interfaces.hhi:831:36,45: This is why I think it is an object of type WaitHandle
  /tmp/hh_server/hhi_1b7ab862/classes.hhi:109:16,25: Declaration of WaitHandle is here

New tag please :)
Changes that'll be included: https://github.com/facebook/xhp-lib/compare/2.3.0...master


into_inner function for GraphicsContext

It may be desired to destroy the graphics context while preserving the window.

I think this is irrelevant now since we don't own the window


Question about KE

Thank you for your outstanding work, but I have an error when testing the KE method, it seems that the code from src.data.mmkb_dataset import CaptionDataset is not included in the program.

Thank you for pointing out this issue! I have uploaded these codes just now and you can check it.


add stat for total destination rules

We have stat to indicate total VS but not DR. These two are most frequently used Istio resources. So adding a stat for DR.

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

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-istiodremote-mc_istio
1351711ab5938f682f9f309eaff6646af216e7fd
link
true
/test integ-pilot-istiodremote-mc_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-pilot-istiodremote-mc_istio

K8S actually does have these (or maybe it's etcd), so you could probably
monitor at that level
On Thu, Mar 17, 2022, 6:00 AM Zhonghu Xu @.***> wrote:

@.**** commented on this pull request.
In pilot/pkg/model/push_context.go
https://github.com/istio/istio/pull/37967#discussion_r829088924:

@@ -599,6 +599,12 @@ var (
"Total virtual services known to pilot.",
)

// totalDestinationRules tracks the total number of destination rules
totalDestinationRules = monitoring.NewGauge(

I think this is not core istio's business., but the responsibility of a
company monitoring system. Otherwise we could include so many things in
istio
—
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/37967#discussion_r829088924, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXNHD2ERI7HGINS3GTDVAMUF5ANCNFSM5Q3GI5FA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because your review was requested.Message ID:
@.***>

Ok. I will close this then


With relative times, tooltip shows in browser language instead of document language

When using relative times, times older than a month are show absolutely, localized using the document's language (good!); however, the title is localized using the browser language (bad!). This should consistently use the document language.
https://github.com/github/time-elements/blob/4414cbacdb1e0cf95b7e99960897bb0c55736323/src/extended-time-element.ts#L64

@JasonBarnabe did you open a PR?

In v3.2.0 they both now use the same way to determine the local; looking for the closest element with a lang attribute. So I believe this fixes the issue. Let me know if you run into any other issues.


I sy/token sorting

Hello @iSyqozz and thank you for the PR! Unfortunately, this issue has already been resolved with #263. I encourage you to keep contributing and open up new PRs! Thanks again for your work


addresses #32

@manulera it looks like you've hoisted a couple functions that now no longer have access to the variables in the scope they were hoisted from.
Out of curiosity what are you planning on using that function for?

As you can see the way it is written currently it is not actually returning anything, it is just pushing locations onto the current feature.
Thanks!

Sorry about this @tnrich, I replied on the issue #32


Retiring this gem

GOV.UK is considering retiring this gem in favour of a Ruby-specific gem for Rubocop and using NPM packages to lint CSS and Javascript. The internal GOV.UK RFC is here: https://github.com/alphagov/govuk-rfcs/pull/100.
Opening this issue for visibility, since this gem is used across GDS and government.

Was a decision made to retire the gem? If so, where can I find the alternative?

Was a decision made to retire the gem? If so, where can I find the alternative?

As above, as this gem is a wrapper around Rubocop and scss_lint, the general trajectory is to use those tools directly (or alternatives to those tools directly). A gem may still be used for providing common configuration.

A gem may still be used for providing common configuration.

It's that gem (common configuration) we are after. Is that something you will publish?

It's that gem (common configuration) we are after. Is that something you will publish?

In some cases, rather than using the govuk-lint-ruby script within this gem, we're just using the configuration already.
This is one example of how you can do this: https://github.com/alphagov/content-data-api/blob/master/.rubocop.yml#L1-L2

Should the GDS Way be updated to reflect this? Currently it seems to suggest just using this gem https://gds-way.cloudapps.digital/manuals/programming-languages/ruby.html

Should the GDS Way be updated to reflect this new advice? Currently it seems to suggest just using this gem https://gds-way.cloudapps.digital/manuals/programming-languages/ruby.html

Probably.

This has been deprecated

💯


100% CPU utilisation when used as a JSR-107 compliant cache for Spring framework 4.1.4

I have the following Java config setup in my Spring container:
@Configuration
@EnableCaching
public class FXCacheConfig {

@Value("${" + FXConstants.CACHE_RATES_TTL + "}")
private int ratesCacheTTL;

@Bean
public JCacheManagerFactoryBean cacheFactory() {
    return new JCacheManagerFactoryBean();
}

@Bean
public CacheManager jCacheManager(javax.cache.CacheManager jsr107CacheManager) {

    MutableConfiguration<Object, ExchangeRates> config = new MutableConfiguration<>();
    config.setStoreByValue(false);
    config.setExpiryPolicyFactory(CreatedExpiryPolicy.factoryOf(new Duration(TimeUnit.MILLISECONDS,
            this.ratesCacheTTL)));
    jsr107CacheManager.createCache(FXConstants.CACHE_RATES_NAME, config);

    return new JCacheCacheManager(jsr107CacheManager);
}

}

I've then run this setup on Jetty 9 and JBoss 8.2. On both servers the CPU goes to 100% (and beyond). If I take this out and replace it with the equivalent Guava or ConcurrentMapCache implementations as per the Spring documents, the server uses <5%.
I've tried to profile the server but have not been able to isolate the problem. All I can tell so far is that with EHCache 3, my CPUs max out.
I've also tried removing the cache setup from the above code, and removing all the @Cache... annotations so there is nothing left but the raw setup. CPUs still 100%+.
Any ideas what to try next?
Derek

Did you take some thread dumps when your app was running with ehcache and maxing out the CPU? JTop (https://code.google.com/p/hatter-source-code/wiki/jtop) can also be helpful figuring out what threads are consuming the CPU.

Hi, I've run jProfiler and YourKit. This is where it gets weird, I cannot find any threads that are chewing CPU. Everything appears ok and EHCache doesn't appear in any of the dumps I've taken. But the presence of EHCache is all that is required to max out the CPUs. If I swap to JDK or Guava caches, the system stops chewing CPU. I'll give JTop a try. Thanks.

Maybe you could share the strip down project? See if it can be reproduced?!
Could be environmental...
On Sun, Apr 26, 2015 at 8:25 PM Derek Clarkson<EMAIL_ADDRESS>wrote:

Hi, I've run jProfiler and YourKit. This is where it gets weird, I cannot
find any threads that are chewing CPU. Everything appears ok and EHCache
doesn't appear in any of the dumps I've taken. But the presence of EHCache
is all that is required to max out the CPUs. If I swap to JDK or Guava
caches, the system stops chewing CPU. I'll give JTop a try. Thanks.
—
Reply to this email directly or view it on GitHub
https://github.com/ehcache/ehcache3/issues/387#issuecomment-96451511.

Tried JTop and I'm still rather confused. My system is showing me two different stories. Using normal Top, my system is showing massive CPU usage by the java process:
Processes: 274 total, 3 running, 13 stuck, 258 sleeping, 1402 threads                                                                                10:49:49
Load Avg: 2.37, 2.08, 2.00  CPU usage: 13.76% user, 11.71% sys, 74.51% idle  SharedLibs: 141M resident, 0B data, 15M linkedit. MemRegions: 82423 total, 3578M resident, 87M private, 932M shared.
PhysMem: 8113M used (1352M wired), 70M unused. VM: 732G vsize, 1353M framework vsize, 250967(0) swapins, 394040(0) swapouts. Networks: packets: 9753285/4027M in, 9342538/2382M out.
Disks: 2247821/37G read, 2374601/64G written.

PID    COMMAND      %CPU      TIME     #TH   #WQ  #PORT MEM    PURG   CMPRS  PGRP  PPID  STATE    BOOSTS           %CPU_ME %CPU_OTHRS UID  FAULTS    COW     MSGSENT    MSGRECV    SYSBSD
32721  java         169.4     11:49.28 38/1  1    207-  522M+  16K    0B     32418 32418 running  *0[86]           0.00000 0.00000<PHONE_NUMBER>2+<PHONE_NUMBER>+   4434023+   70116760+
0      kernel_task  4.3       60:55.07 116/8 0    2     1035M  0B     0B     0     0     running   0[0]            0.00000 0.00000    0    81889     0       451839037+ 432287466+ 0
32969  top          3.2       00:07.84 1/1   0    146+  4428K  0B     0B     32969 32911 running  *0[1]            0.00000 0.00000    0    432856+   92      1647541+   823469+    234068+ 3

Then she I run Jtop, it shows me almost no usage (Same result from JProfiler and YourKit):
Heap Memory: INIT=128.00M  USED=259.05M  COMMITED=486.50M  MAX=910.50M
NonHeap Memory: INIT=23.44M  USED=44.71M  COMMITED=77.56M  MAX=560.00M
GC PS Scavenge  VALID  [PS Eden Space, PS Survivor Space]  GC=35702  GCT=50667
GC PS MarkSweep  VALID  [PS Eden Space, PS Survivor Space, PS Old Gen, PS Perm Gen]  GC=2  GCT=201
ClassLoading LOADED=7770  TOTAL_LOADED=7775  UNLOADED=5
Total threads: 20  CPU=4 (0.23%)  USER=3 (0.20%)
NEW=0  RUNNABLE=13  BLOCKED=0  WAITING=2  TIMED_WAITING=5  TERMINATED=0

RMI TCP Connection(1)-<IP_ADDRESS>  TID=291397  STATE=RUNNABLE  CPU_TIME=4 (0.20%)  USER_TIME=3 (0.18%) Allocted: 253.65K
sun.management.ThreadImpl.dumpThreads0(Native Method)
sun.management.ThreadImpl.dumpAllThreads(ThreadImpl.java:446)
me.hatter.tools.jtop.rmi.RmiServer.listThreadInfos(RmiServer.java:59)
me.hatter.tools.jtop.management.JTopImpl.listThreadInfos(JTopImpl.java:48)

Abandoned connection cleanup thread  TID=8  STATE=TIMED_WAITING  CPU_TIME=0 (0.02%)  USER_TIME=0 (0.01%) Allocted: 0
java.lang.Object.wait(Native Method)
java.lang.ref.ReferenceQueue.remove(ReferenceQueue.java:135)
com.mysql.jdbc.AbandonedConnectionCleanupThread.run(AbandonedConnectionCleanupThread.java:43)

JMX server connection timeout 291571  TID=291571  STATE=TIMED_WAITING  CPU_TIME=0 (0.00%)  USER_TIME=0 (0.00%) Allocted: 1.59K
java.lang.Object.wait(Native Method)
com.sun.jmx.remote.internal.ServerCommunicatorAdmin$Timeout.run(ServerCommunicatorAdmin.java:168)
java.lang.Thread.run(Thread.java:744)

 qtp1265909690-3210073  TID=3210073  STATE=TIMED_WAITING  CPU_TIME=0 (0.00%)  USER_TIME=0 (0.00%) Allocted: 0
sun.misc.Unsafe.park(Native Method)
java.util.concurrent.locks.LockSupport.parkNanos(LockSupport.java:226)
java.util.concurrent.locks.AbstractQueuedSynchronizer$ConditionObject.awaitNanos(AbstractQueuedSynchronizer.java:2082)
org.eclipse.jetty.util.BlockingArrayQueue.poll(BlockingArrayQueue.java:389)

It appears that the Java tooling is not aware of something that's chewing CPU, but OS X is.

I've now created a vanilla project. Unfortunately due to my client's rules regarding IP issues, security etc, I cannot upload a project. However I've stripped all the dependencies, and removed all the code and other files until all that exists are two files. A application loader and the cache config. Here is the app loader:
public class TestApplicationLoader implements WebApplicationInitializer {

@Override
public void onStartup(ServletContext container) {

	// Create the dispatcher servlet's Spring application context
	AnnotationConfigWebApplicationContext dispatcherContext = new AnnotationConfigWebApplicationContext();
	dispatcherContext.register(TestConfig.class);

	// Register and map the dispatcher servlet
	ServletRegistration.Dynamic dispatcher = container.addServlet("test", new DispatcherServlet(
			dispatcherContext));
	dispatcher.setLoadOnStartup(1);

	// Add our mappings.
	dispatcher.addMapping("/*");

}

}

And here is the cache test config:
@Configuration
@EnableWebMvc
public class TestConfig {

@Bean
public javax.cache.CacheManager ehCache() {
	return Caching.getCachingProvider().getCacheManager();
}

@Bean
public CacheManager jCacheManager(javax.cache.CacheManager jsr107CacheManager) {
	MutableConfiguration<Object, Object> config = new MutableConfiguration<>();
	jsr107CacheManager.createCache("test", config);
	return new JCacheCacheManager(jsr107CacheManager);
}

}

There are no other files. The gradle dependencies are:
compile	"org.springframework:spring-webmvc:4.1.4.RELEASE",
		"org.springframework:spring-context-support:4.1.4.RELEASE",
		"org.ehcache:ehcache:3.0.0.m1",
		"javax.cache:cache-api:1.0.0",
		"javax.servlet:javax.servlet-api:3.1.0"

Notice that @EnableCaching is not present either. So now all we have is the presence of two beans. Still using at least 200% CPU for no reason.
I just did so more testing. I've discovered that if I remove the declaration of the cache then the CPU goes back to normal. i.e. the second bean becomes:
@Bean
public CacheManager jCacheManager(javax.cache.CacheManager jsr107CacheManager) {
	//MutableConfiguration<Object, Object> config = new MutableConfiguration<>();
	//jsr107CacheManager.createCache("test", config);
	return new JCacheCacheManager(jsr107CacheManager);
	}

So this is something to do with the adding of a cache declaration. Is there anything wrong with the way I've done it?
ciao
Derek

More news, we have just tried the code on a JBoss/Linux box and it is doing the same thing. 100% CPU as measure by Top, next to no CPU as measured by JTop. This leads me to the conclusion that something in the new EHCache code is triggering CPU usage underneath Java where Java profilers cannot see it.

I managed to reproduce the problem and indeed, there seems to be a bug in ehcache 3 that is the cause of this excessive CPU consumption.
It seems that the statistics thread pool mistakenly was created in a way that it never keeps its threads around, hence every single statistic work ehcache does internally ends up creating a new thread each time. It is that thread creation that consumes unordinate amounts of CPU time, and since those threads only end up doing very minimal work this explains why they don't show up on the jtop radar: most CPU actually is consumed by the OS to create the thread, not to execute any java code.
We're going to fix this the for the next release, but in the meantime, there fortunately is a workaround: just set the org.ehcache.statisticsExecutor.poolSize system property to 1 and the problem should go away.
Thanks for reporting this bug!

Cool. Thanks for looking into that. I'll try the workaround.

Excellent. Work around fixes the issue.

Hi, I am using spring boot 1.5.x with jcache as caching interface. I too was facing the issue of close to 100% CPU and set the org.ehcache.statisticsExecutor.poolSize property to 1 in my application.properties. Unfortunately, the issue still persists. Now I think I am missing something here. maybe properties in the said file is not a system property. I even tried setting the environment variable but nothing. Help appreciated. Thanks

@nikhil-sahu Are you sure it is the same issue? This has been fixed for a while (3.0.0m2 if I'm not mistaken).

@henri-tremblay I upgraded spring-boot from 1.4.0 to 1.5.8. The only major change is Ehcache 2 to 3. The CPU spiked since day 1 of deployment. Also one of the upstream servers saw a spike in requests(the response is cached). I'll try to pinpoint what exactly is causing the issue and see what comes up.

@nikhil-sahu It will be really nice if you can do a profiling or are able to provide a stripped down application for us to reproduce the problem. Otherwise it is pretty hard for us to help you. The cache configuration might also give us a hint. You probably should put that in another issue since I'm pretty sure it is not related to this one. Or better, on the mailing list and we will tell you if it should be turned into an issue.

Ok sure. Thanks


[gdal] New feature "cfitsio"

Describe the pull request
A new feature "cfitsio" for gdal port

What does your PR fix?

Refs https://github.com/microsoft/vcpkg/issues/20875

Which triplets are supported/not supported? Have you updated the CI baseline?

all

Does your PR follow the maintainer guide?

Yes

Thanks for the support @dg0yt

I took the opportunity to add another feature "hdf5", integrated into the same PR to avoid merge conflicts requiring manual action due to versioning

I would also suggest to not change a PR once it has the "info-reviewed" label, because it will trigger a new review cycle.

I would prefer to not add an "hdf5" feature now. hdf5 is not enabled in static linkage on windows, so the feature will not deliver what it promises. I'm handling that as soon as the vcpkg team manages to review and merge my other PRs (pkgconfig for nmake in proj, libspatialite, etc.)

:+1:

I would also suggest to not change a PR once it has the "info-reviewed" label, because it will trigger a new review cycle.

I understand that and was considering it too. In my experience for small things like this, the review cycle is normally short, while the time to merge is larger. As I'm currently not that often at a keyboard, the manual interaction required by the vcpkg versioning strategy makes it harder to "fire and forget".

Thanks for the new feature!


Create low volume dashboards from transactions explorer feed

This loads all records from the transactions explorer feed and,
depending on the data present, creates modules and a dashboard.
https://www.agileplannerapp.com/boards/15088/cards/9254
This also gives us a base to build on for updating all existing dashboards
from the feed. The spreadsheet merge test is disabled until we
need merge functions.
Overall this is fairly low-quality code, but it should suffice as a first iteration
for creating dashboards.

Otherwise, :+1:

Thanks, updated. Also changed the criteria for creating modules after discussing with @tombooth and @easternbloc.


PCSF00021 [LittleBigPlanet™ PlayStation®Vita]

App summary

App name: LittleBigPlanet™ PlayStation®Vita
App serial: PCSF00021
App version: 1.0

Vita3K summary

Version: v0.1.5
Build number: 2570-ffa66d88
Commit hash: https://github.com/Vita3K/Vita3K/commit/ffa66d88de3a1b25bed47b5b85074b3b99778ee6
CPU backend:  Dynarmic

Test environment summary

Tested by: 7GEN emulation news (https://vk.com/rpcs3)
OS: Windows 10
CPU: Intel i7 7700k
GPU: NVIDIA 1060 6G
RAM: 16 GB

Issues
Ingame with broken graphics
Screenshots
Video:
https://vk.com/video-75040291_456267156
Screenshot:

Log
Recommended labels

A?
B?
C?

no write state of game inside title, and before create new issue, check if id of game already exist, here is doublon


[BUG] long package names shortened with "..." causes package not found error

Please confirm these before moving forward

[X] I have searched for my issue and have not found a work-in-progress/duplicate/resolved issue.
[X] I have tested that this issue has not been fixed in the latest (beta or stable) release.
[X] I have checked the FAQ section for solutions.
[X] This issue is about a bug (if it is not, please use the correct template).

UniGetUI Version
3.1.1
Windows version, edition, and architecture
Windows 11 Pro 24H2 x64 (Build 26100.1742)
Describe your issue
winget update --id "DBBrowserForSQLite.DBBrowserForSQL…" --exact --source winget --accept-source-agreements --silent --disable-interactivity --accept-package-agreements --force --include-unknown
Package name  is to long so its shortened with "..." at the end. it seems like its using the shown paket-id instead of the full id
Steps to reproduce the issue
Update DB Browser for SQLite
UniGetUI Log
Starting package update operation for package id=DBBrowserForSQLite.DBBrowserForSQL… with Manager name=Winget
Given installation options are <InstallationOptions: SkipHashCheck=False;InteractiveInstallation=False;RunAsAdministrator=False;Version=;Architecture=;InstallationScope=;InstallationScope=;CustomParameters=;RemoveDataOnUninstall=False>
Process Executable     : C:\Program Files\WingetUI\Assets\Utilities\gsudo.exe
Process Call Arguments : "C:\Program Files\WingetUI\winget-cli_x64\winget.exe"  update --id "DBBrowserForSQLite.DBBrowserForSQL…" --exact --source winget --accept-source-agreements --silent --disable-interactivity --accept-package-agreements --force --include-unknown
Working Directory      : C:\Users\ralfr
    |    \
    |   ███████████▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒  1024 KB / 2.72 MB
    |   ██████████████████████▒▒▒▒▒▒▒▒  2.00 MB / 2.72 MB
    |    |
    | No installed package found matching input criteria.
Process Exit Code      : -1978335212
Process End Time       : 14.09.2024 09:39:41

Package Managers Logs
Manager WinGet with version:
Bundled WinGet CLI Version: v1.8.1911
Microsoft.WinGet.Client PSModule version: ""
Using bundled WinGet helper (CLI parsing)

——————————————————————————————————————————

Logged subprocess-based task on manager Winget. Task type is FindPackages
Subprocess executable: "C:\Program Files\WingetUI\winget-cli_x64\winget.exe"
Command-line arguments: " source list"
Process start time: 14.09.2024 09:26:39
Process end time:   14.09.2024 09:26:39

-- Process STDOUT
 ...

Return code: SUCCESS (0)

——————————————————————————————————————————

Logged subprocess-based task on manager Winget. Task type is RefreshIndexes
Subprocess executable: "C:\Program Files\WingetUI\winget-cli_x64\winget.exe"
Command-line arguments: " source update --disable-interactivity"
Process start time: 14.09.2024 09:26:41
Process end time:   14.09.2024 09:26:42

-- Process STDOUT
 ...

Return code: SUCCESS (0)

——————————————————————————————————————————

Logged subprocess-based task on manager Winget. Task type is ListInstalledPackages
Subprocess executable: "C:\Program Files\WingetUI\winget-cli_x64\winget.exe"
Command-line arguments: " list  --accept-source-agreements"
Process start time: 14.09.2024 09:26:41
Process end time:   14.09.2024 09:26:45

-- Process STDOUT
 ...

Return code: SUCCESS (0)

——————————————————————————————————————————

Logged subprocess-based task on manager Winget. Task type is FindPackages
Subprocess executable: "C:\Program Files\WingetUI\winget-cli_x64\winget.exe"
Command-line arguments: " source list"
Process start time: 14.09.2024 09:26:42
Process end time:   14.09.2024 09:26:42

-- Process STDOUT
 ...

Return code: SUCCESS (0)

——————————————————————————————————————————

Logged subprocess-based task on manager Winget. Task type is ListUpdates
Subprocess executable: "C:\Program Files\WingetUI\winget-cli_x64\winget.exe"
Command-line arguments: " update --include-unknown  --accept-source-agreements"
Process start time: 14.09.2024 09:26:42
Process end time:   14.09.2024 09:26:46

-- Process STDOUT
 ...

Return code: SUCCESS (0)

——————————————————————————————————————————

Logged subprocess-based task on manager Winget. Task type is ListInstalledPackages
Subprocess executable: "C:\Program Files\WingetUI\winget-cli_x64\winget.exe"
Command-line arguments: " list  --accept-source-agreements"
Process start time: 14.09.2024 09:26:52
Process end time:   14.09.2024 09:26:56

-- Process STDOUT
 ...

Return code: SUCCESS (0)

——————————————————————————————————————————

Logged subprocess-based task on manager Winget. Task type is RefreshIndexes
Subprocess executable: "C:\Program Files\WingetUI\winget-cli_x64\winget.exe"
Command-line arguments: " source update --disable-interactivity"
Process start time: 14.09.2024 09:27:06
Process end time:   14.09.2024 09:27:07

-- Process STDOUT
 ...

Return code: SUCCESS (0)

——————————————————————————————————————————

Logged subprocess-based task on manager Winget. Task type is ListUpdates
Subprocess executable: "C:\Program Files\WingetUI\winget-cli_x64\winget.exe"
Command-line arguments: " update --include-unknown  --accept-source-agreements"
Process start time: 14.09.2024 09:27:07
Process end time:   14.09.2024 09:27:11

-- Process STDOUT
 ...

Return code: SUCCESS (0)

——————————————————————————————————————————

Logged subprocess-based task on manager Winget. Task type is ListInstalledPackages
Subprocess executable: "C:\Program Files\WingetUI\winget-cli_x64\winget.exe"
Command-line arguments: " list  --accept-source-agreements"
Process start time: 14.09.2024 09:38:49
Process end time:   14.09.2024 09:38:52

-- Process STDOUT
 ...

Return code: SUCCESS (0)

——————————————————————————————————————————

Logged subprocess-based task on manager Winget. Task type is RefreshIndexes
Subprocess executable: "C:\Program Files\WingetUI\winget-cli_x64\winget.exe"
Command-line arguments: " source update --disable-interactivity"
Process start time: 14.09.2024 09:39:13
Process end time:   14.09.2024 09:39:14

-- Process STDOUT
 ...

Return code: SUCCESS (0)

——————————————————————————————————————————

Logged subprocess-based task on manager Winget. Task type is ListUpdates
Subprocess executable: "C:\Program Files\WingetUI\winget-cli_x64\winget.exe"
Command-line arguments: " update --include-unknown  --accept-source-agreements"
Process start time: 14.09.2024 09:39:14
Process end time:   14.09.2024 09:39:17

-- Process STDOUT
 ...

Return code: SUCCESS (0)

——————————————————————————————————————————

Relevant information
No response
Screenshots and videos

same with the following packages

Are you using Bundled WinGet instead of System WinGet?

yes, bundled WinGet CLI Version: v1.8.911
But PSModule shows "" in Version, maybe something wrong here?

No, this should be fine. What happens if you disable bundled WinGet?

☝Works, even the View isn't shortened
(and more information if this is something you could add this as a feature 🤔)

System Winget Version

Do you need any more Informations?

I mean, by default the setting is to have bundled WinGet disabled, since it uses the COM APIs (and not cli parsing, which has the ... issue), and this setting shouldn't be toggled unless you are experiencing issues with System WinGet...


feat: generalized Evaluator and Tensor interfaces

Update the evaluator and tensor's interfaces for graph capturing evaluators.

Add eval_function to Evaluator and calls it from functions
Add rank to Tensor
Make all helper functions in evaluator private

@gramalingam so this part fails with the api change in the evaluator:
    # This is a bit fragile. We want to get the ONNXFunction object representing
    # the outer-scope ONNXScript function from the execution stack. The caller of
    # @graph is the original script function (cumulative_sum in the above example),
    # and the caller of that function is the wrapper function/method in the
    # corresponding OnnxFunction object.
    # Currently, there is no support for eager-mode execution of nested functions,
    # so we don't need to handle doubly nested functions (e.g., a function defined
    # inside Sum in the above example).

    function_frame = sys._getframe(1)  # pylint: disable=protected-access
    wrapper_frame = sys._getframe(2)  # pylint: disable=protected-access
    onnx_function = wrapper_frame.f_locals["self"]
    nested_functions = onnx_function.function_ir.nested_functions

I fixed it by updating the frame it looks for.

I am going to merge this to process with the graph capturing evaluator. LMK if changes need to be made and I will send a patch


Add organisation ID FK to profiles, profiles_readonly, messages and replies and config

Closed in https://github.com/PostHog/squeak/pull/62


SYSTEM_RUN_ERR -- image exceeds 5 MB maximum

I currently have this error and just wondering if anyone has a workaround or a way to troubleshoot the error.
TIA
Error code
{
SYSTEM_RUN_ERR: {
err: Error: Error: 400 {"type":"error","error":{"type":"invalid_request_error","message":"messages.0.content.1.image.source.base64: image exceeds 5 MB maximum: 6279864 bytes > 5242880 bytes"}}
at .../cofounder/api/build.js:107:17
at process.processTicksAndRejections (node:internal/process/task_queues:105:5)
at async .../cofounder/api/build.js:84:25
at async .../cofounder/api/node_modules/p-queue/dist/index.js:187:36,
id: 'op:LLM::GEN'
}
}

I have the same error in addition to reaching Claude error
type: 'rate_limit_error',
message: 'Number of request tokens has exceeded your daily rate limit (https://docs.anthropic.com/en/api/rate-limits); see the response headers for current usage. Please reduce the prompt length or the maximum tokens requested, or try again later. You may also contact sales at https://www.anthropic.com/contact-sales
to discuss your options for a rate limit increase.'
}

yes - issue was api serving back files bigger than claude limit
serverside fix otw


Update kafka-avro-serializer to 6.2.5

Updates io.confluent:kafka-avro-serializer from 6.2.3 to 6.2.5.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.confluent", artifactId = "kafka-avro-serializer" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "io.confluent", artifactId = "kafka-avro-serializer" }
}]

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

Superseded by #646.


upgraded to qwik version 1.4.0

What is it?

[ ] Feature / enhancement
[ ] Bug
[ ] Docs / tests
[x] Other

Why is it needed?
To possibly fix #594
Checklist:

[x] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[x] I have ran pnpm changeset and documented my changes
[ ] I have add necessary docs (if needed)
[ ] Added new tests to cover the fix / functionality

@wmertens @maiieul
Can you please fix the types on Tab?
Thanks!


Blank cape

It renders the cape but the cape has no texture
i have my cape repo seperate from my mod repo

Well that would indicate the mod can't access wherever you are pointing the images to.. Make sure you're using github's "raw file" feature..

i am and my image is on dropbox, my json is on my githup repo named capes

Make sure both are the raw files and have no outter frame or anything. Often times Dropbox will put an HTML frame around the image if it's not the raw version link.


KAFKA-7883 add schema.namespace support to SetSchemaMetadata SMT in Kafka Connect

When you use Kafka Connect and Avro Converter with Schema Registry you may need to transform your source schema namespace. Currently SetSchemaMetadata provide a way to transform the namespace by specifying in the schema.name attribute both actual namespace being concatenated with the actual schema name:
{
  "transforms" : "TransformSchema",
  "transforms.TransformSchema.type" : "org.apache.kafka.connect.transforms.SetSchemaMetadata$Value",
  "transforms.TransformSchema.schema.name" : "my.new.namespace.NewSchemaName"
}

But this way you are forced to specify the schema name even when you want to preserve the original schema name. In some cases you just may need to replace possibly an empty namespace of your source with the actual namespace fitting "your.java.package" and leave schema name as it is.
Moreover as mentioned in KAFKA-7883, when use JDBC connector with multiple tables in the source database what you may want is to preserve the table names but specify some non-empty namespace for them.
To support this we have to provide a way to append a schema namespace to the schema name (doesn't matter being updated or not).
In this PR a new schema.namespace attribute is added to SetSchemaMetadata letting developer to decide either to transform namespace only or transform both schema name and schema namespace by using either schema.namespace together with schema.name or just single schema.name attribute. An example:
{
  "transforms" : "TransformSchema",
  "transforms.TransformSchema.type" : "org.apache.kafka.connect.transforms.SetSchemaMetadata$Value",
  "transforms.TransformSchema.schema.namespace" : "my.new.namespace",
  "transforms.TransformSchema.schema.name" : "NewSchemaName",
  "transforms.TransformSchema.schema.version" : 42
}

I really want this! This is exactly what our team needs to resolve issues with Avro namespaces.

@kkonstantine could you please find a time to review my request? I believe it can be a useful feature and it should fill in the gaps in the current SetSchemaMetadata SMT API. Thank you!

For context, we have had to go from 2 connectors to a.... huge number of connectors as a workaround, which has been a crazy amount of configuration for us, based on how we currently use kafka connect with yaml configuration (think of strimzi or the kafka confluent k8s operator). All each overriding the entire Schema name and being explicit while also introducing tabe whitelisting.... This pull request is a fantastic solution that would allow us to go back to 2 connectors.

I've marked this as needing a KIP, though I'm not yet sure whether that is the case. These SMTs are technically part of the API, since they appear in STM configurations, which are considered part of the public API. I'll keep looking into this to figure out if this is really the case.

Would I be correct with the following example too?
{
  "transforms" : "TransformSchema",
  "transforms.TransformSchema.type" : "org.apache.kafka.connect.transforms.SetSchemaMetadata$Value",
  "transforms.TransformSchema.schema.namespace" : "added.namespace"
}

I've marked this as needing a KIP, though I'm not yet sure whether that is the case. These SMTs are technically part of the API, since they appear in STM configurations, which are considered part of the public API. I'll keep looking into this to figure out if this is really the case, but it's probably safer to assume a KIP is needed.

@rhauch thank you for your review, I've really missed the fact I've broken current behavior on empty schema name!
I've fixed this and added more corner tests cases on the schema update. Also I've made an effort to prettify our tests to make them looking more uniform and readable.

@rhauch @mnegodaev  this is looking pretty good now. Would others need to have reviewed this before being merged in? I'm not too clued in on the open source contributions here but it looks like what @mnegodaev has done is excellent work that may solve a lot of issues for people using kafka with avro. :)

I am not familiar with Connect code -- @rhauch @kkonstantine @mimaison should be able to help.

@mnegodaev It looks like this is something you would add to this PR a link to a KIP.

@mnegodaev It looks like this is something you would add to this PR a link to a KIP.

@pjmagee, I'll try to do may best. Thanks!

@rhauch @mimaison @kkonstantine @pjmagee
If you guys don't mind I'm going to wait till Wednesday and then call a [VOTE] to have the proposal adopted. Thanks!

@rhauch @mimaison @kkonstantine @pjmagee I've started a vote for the KIP-855: https://lists.apache.org/thread/b0cdofxhl7p84k14wvhtrhd0r1z8qzpk
Please vote for it if you're agree.

@mimaison Hey, It looks like @mnegodaev  has followed all the right procedures and we have the KIP. As someone with more experience on this repo, what are our next steps?

Hi @hachikuji Sorry to pull you in on this PR - I notice you have been active on some merged PR's quite recently. We have had some input from others regarding the process around creating a KIP, which is now completed for this feature, along with an explanation as to why @mnegodaev  has added this feature, along with myself having a usecase for this feature. What would be the next steps in being able to get this merged? It seems like a elegant solution and change to the SMT to support namespaces. Would it be possible for you to maybe include others who may be able to get this merged? I am rather new to this repo so I am unaware of how best to proceed.

@pjmagee I commented in the discuss thread last week, see https://lists.apache.org/thread/92y0qqlwb75xqkxnzwbw7jywfdw8gs8k. Can you reply to my question there?

@pjmagee I commented in the discuss thread last week, see https://lists.apache.org/thread/92y0qqlwb75xqkxnzwbw7jywfdw8gs8k. Can you reply to my question there?

I've fixed this typo into KIP, thanks for finding this.

whats the status of this?

To be able to merge this, the associated KIP must be voted.
In the discussion thread for the KIP, a few questions have not been answered yet by the author. Until these are answered, it's unlikely there will be any votes, so this will stay stuck.

Hi guys, I'll try to answer the questions in the near time, thanks.
ср, 15 февр. 2023 г. в 15:51, Mickael Maison @.***>:

To be able to merge this, the associated KIP
https://cwiki.apache.org/confluence/display/KAFKA/KIP-855%3A+Add+schema.namespace+parameter+to+SetSchemaMetadata+SMT+in+Kafka+Connect
must be voted.
In the discussion thread
https://lists.apache.org/thread/3hkd9lljobf9rl56ogjpcbo4ldoxcz5n for
the KIP, a few questions have not been answered yet by the author. Until
these are answered, it's unlikely there will be any votes, so this will
stay stuck.
—
Reply to this email directly, view it on GitHub
https://github.com/apache/kafka/pull/11442#issuecomment-1431141433, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AE2RTBY2JST3W745OFVSLJ3WXSYLTANCNFSM5G3C2JEA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Specify correct path to 'Always on'

The setting 'Always on' is currently found under Configuration -> General settings.

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

ID: 3e26e5a4-7b72-ba7e-cfed-e31a4a8aaef9
Version Independent ID: dc783fe3-bd1a-c25e-2b92-903595c0822e
Content: Application performance FAQs - Azure App Service
Content Source: articles/app-service/faq-availability-performance-application-issues.md
Service: app-service
GitHub Login: @genlin
Microsoft Alias: genli

Hi @petkit, Thanks for the feedback. We will update this doc shortly.

Thanks for catching this error in the doc! I have submitted a PR to fix the issue, the change is merged internally and will go-live in the public docs shortly.


CR does not insert new line when completion popup is visible.

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md
[X] I have updated 'mini.nvim' to latest version

Module(s)
mini.completion
Description
When completion is enabled, any time the completion popup is visible, pressing  closes the popup but does not go to the next line. :imap <CR> shows nothing.
Neovim version
NVIM v0.7.0-dev+1376-g973e91007
Steps to reproduce
minimal.lua:
vim.api.nvim_command("packadd! mini")
require("mini.completion").setup()

nvim -nu minimal.lua
Trigger completion (CTRL-space)
Press enter

Expected behavior
Popup goes away and newline is inserted.
Actual behavior
Popup goes away, but no newline is inserted.

That is not really a bug: built-in Neovim (and Vim) completion behaves the same way. As 'mini.completion' is designed to use built-in mechanisms as much as possible, this can indeed be solved only by remapping <CR> in Insert mode. In turn, this is a very popular key for mapping in Insert, so early on I decided to not make any such mappings inside plugin.
What I think is more appropriate here, is to document common code snippets used for making these mappings. I did so for <Tab> and <S-Tab>, but forgot for <CR>. So thank you for reminder!


Azure AppService Plan always computes a new name when it's Windows

There's an odd issue occurring with the AppService.Plan resource, where having kind: "windows" forces the resource to generate a new name every time pulumi runs
Code snippet
const appServicePlan = new azure.appservice.Plan(`${prefix}-app-plan`, {
    kind: "windows",

    location: resourceGroup.location,
    resourceGroupName: resourceGroup.name,

    // Consumption Play
    sku: {
        size: "Y1",
        tier: "Dynamic",
    },
});

The newly generated name is most likely a symptom not a cause.  I assume some other aspect of the resource is marking it as needing to be replaced (which then requests a new name for the replacement).
Can you share the results of a pulumi preview --diff using the latest CLI version?

Hi @w0rldart
Thanks for raising the issue here. Unfortunately, you cannot use a Y1 SKU and a Windows AppService Plan. Y1 is reserved for FunctionApp kind as it's a dynamic tier for Function consumption
I have been able to recreate this exactly and Y1 only works when you use the following:
const appServicePlan = new azure.appservice.Plan(`${prefix}-app-plan`, {
    kind: "functionapp",

    location: resourceGroup.location,
    resourceGroupName: resourceGroup.name,

    // Consumption Play
    sku: {
        size: "Y1",
        tier: "Dynamic",
    },
});

I have also been able to confirm that I cannot choose Y1 as a SKU in the Azure console
I hope this helps
Paul


Include titles in RFC

Currently our RFC don't have titles. I suggest to change the template to include a title for the RFC:
# Title

## Summary

...

## Motivation

...

The format is referred from Rust and crossbeam, and the file name has already shown the title, I think it is ok.
If you really want to add a title, you can include some fields like https://github.com/rust-lang/rfcs/blob/master/0000-template.md

Oh. So it seems that these RFCs don't have titles, but only unique identifiers (names) to be easily referred. I think this is a good idea. Closing.


race in "NIC is really absent after detach" check

Description
Sometimes NIC detach fails on relatively loaded hosts.
To Reproduce
Attach NIC then try to detach it
Expected behavior
The NIC should be detached clearly. Also, if the interface is already not available the script must return success.
(pull request is on the way)
Details

Affected Component: [Network]
Hypervisor: [KVM]
Version: [5.4+]

Additional context
After initial fail each try to delete the nic fails because the NIC is deleted.
Reproduction using the shell
[root@a1 ~]# time virsh detach-interface one-300 --type bridge --mac 02:00:51:0b:f2:8e; time virsh domiflist one-300
Interface detached successfully

real	0m5.027s
user	0m0.016s
sys	0m0.004s
Interface  Type       Source     Model       MAC
-------------------------------------------------------
one-300-0  bridge     br61       virtio      02:00:0a:02:01:c8
one-300-1  bridge     br219      virtio      02:00:51:0b:f2:8e

real	0m0.019s
user	0m0.013s
sys	0m0.005s

Running virsh domiflist again (a little bit later) the nic is detached:
[root@a1 ~]# time virsh domiflist one-300
Interface  Type       Source     Model       MAC
-------------------------------------------------------
one-300-0  bridge     br61       virtio      02:00:0a:02:01:c8

real	0m0.017s
user	0m0.012s
sys	0m0.005s

After the manual removal the NIC could not be deleted until the VM is powered down (and the delete operation is executed without calling vmm_mad/detach_nic.

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

ref https://dev.opennebula.org/issues/5171
Hey wasn't this bug reported to libvirt team?

ref #314


patient-ssapp browse trials "LEARN MORE" button vertical centering

The clickeable prototype specifies:

The current v0.4.14 patient-ssapp displays:

This issue is specifically about vertical centering of the "LEARN MORE" button.
(The search form and  "REMOVE SELECTED" mechanism is missing, but these are not the scope of this issue).

Hi! @lehialessandro
Thanks for the proposed fix.
The solution presented in b26c34377d374b6fe08d547eac88401081a3f009 vertically aligns all the row elements, including the text. In the clickable prototypes, the texts are aligned to the top.
Can we achieve that easily ? Or we just leave it as is, and say it is like that ?

Hi! @joaoluis-pdm
I'll add both texts and checkbox aligned to the top.

Thanks @lehialessandro
Closing the issue.


Dell XPS 17 9700

Not ready to go, but thought I'd put what I have up here just in case anyone wants to use it as a starting point.
Fixes #290

So thanks to @workflow, it seems that we have figured out how to get audio working for this laptop. Unfortunately, his fix seems to require a kernel at 5.11 or newer, but the current nixos lts is only at 5.10. However, we should ALSO be able to get it to work with by building https://github.com/maaarghk/soundwire-dkms/commits/master
which I got working in nixpkgs here:  https://github.com/IvanMalison/nixpkgs/tree/soundwire
I guess the question I have is:
What should we put in this configuration? Is it a requirement that it works with the current nixos stable release?
For unstable users, its actually probably better to provide a version where we do not add the extra kernel module and the extra package, while of stable users, it will be a requirement to add it. Curious about how we should proceed here.

Interestingly, the default linux kernel is still 5.10 even on master and in unstable See (https://github.com/NixOS/nixpkgs/blob/28135ad08a5d3b40df1d68a686e8f65a106def3e/pkgs/top-level/all-packages.nix#L21487)
That means that no matter what, we probably need to add something like:
  boot.kernelPackages = pkgs.linuxPackages_latest;

Would it be better to use something a bit less recent like
  boot.kernelPackages = pkgs.linuxPackages_5_12;

Since we are going to be adding a kernel patch (really just modifying kernel flags), we actually need a kernel recompilation anyway:
  boot.kernelPatches = [{
    name = "enable-soundwire-drivers";
    patch = null;
    extraConfig = ''
      SND_SOC_INTEL_USER_FRIENDLY_LONG_NAMES y
      SND_SOC_INTEL_SOUNDWIRE_SOF_MACH m
      SND_SOC_RT1308 m
    '';
    ignoreConfigErrors = true;
  }];

It looks like there is some precedent for bumping the kernel version here, e.g.:
https://github.com/NixOS/nixos-hardware/blob/360b5a7bedf955275fa3fc89bcbe28a2a85bf62b/lenovo/thinkpad/e495/default.nix#L10

Yes. Just set a newer kernel if current lts version is too old.

Yes. See https://github.com/NixOS/nixos-hardware/pull/291/files#r693242215

I tried this, and it works great with regards to audio! But is there a way to get the brightness control of the panel to work?

I tried this, and it works great with regards to audio! But is there a way to get the brightness control of the panel to work?

Yes, but this really depends a lot more on your desktop environment. What are you using?

I see, thank you for replying! I'm using xmonad, but with the plasma KDE environment.

I see, thank you for replying! I'm using xmonad, but with the plasma KDE environment.

I use xmonad as well. You can take a peek at what I do here:
https://github.com/IvanMalison/dotfiles/blob/af36fddc4164a8ea6f0d020a1a96dc6b80a3a66d/dotfiles/config/xmonad/xmonad.hs#L1057
Essentially that uses this script: https://github.com/IvanMalison/dotfiles/blob/af36fddc4164a8ea6f0d020a1a96dc6b80a3a66d/dotfiles/lib/bin/brightness.sh#L3
Which ultimately calls this:
https://github.com/IvanMalison/dotfiles/blob/af36fddc4164a8ea6f0d020a1a96dc6b80a3a66d/dotfiles/lib/bin/brightness_manager.py#L7
which is just a python script that basically writes to the file that manages brightness

@peterstorm I'm honestly kind of surprised that using the KDE environment doesn't make these keys work though. When I use stock KDE brightness keys are bound.
PErhaps when you use XMonad as WM the keypressses are intercepted though.

@peterstorm I'm honestly kind of surprised that using the KDE environment doesn't make these keys work though. When I use stock KDE brightness keys are bound.
PErhaps when you use XMonad as WM the keypressses are intercepted though.

You seem to be correct. Just booted up without xmonad (because I can't get it to work after switching to flakes and Home Manager, hehe) and brightness controls work! I see you also are a guru in xmonad flakes, any chance I can reach out to you somewhere for help there? 😅

You seem to be correct. Just booted up without xmonad (because I can't get it to work after switching to flakes and Home Manager, hehe) and brightness controls work! I see you also are a guru in xmonad flakes, any chance I can reach out to you somewhere for help there? 😅

Sure https://matrix.to/#/#taffybar:matrix.org might be a good place.

Moving this to draft mode so it's obvious this is not ready for review.

I cant speak to SND_SOC_RT1308 kernel patch but SND_SOC_INTEL_USER_FRIENDLY_LONG_NAMES and SND_SOC_INTEL_SOUNDWIRE_SOF_MACH have been added to the default nixos kernel and no longer needed

@IvanMalison  are you still working on this? I might close the pull request otherwise to encourage someone else to pick it up.

Isn't the change that is needed just 1 line or so?

Isn't the change that is needed just 1 line or so?

Yeah. But no one bothered to do this changed and now it has a merge conflict. Please reopen when you are ready with the changes.

I cant speak to SND_SOC_RT1308 kernel patch but SND_SOC_INTEL_USER_FRIENDLY_LONG_NAMES and SND_SOC_INTEL_SOUNDWIRE_SOF_MACH have been added to the default nixos kernel and no longer needed. Works for the XPS 9710 which has similar hardware AFAIK

Is that nixos-unstable? 5.16 kernel on latest nixos-21.11 still seems to require it.

Potentially only on unstable just now: https://github.com/NixOS/nixpkgs/blob/dd697d414853a99d0d5f05ee23430923c6da1176/pkgs/os-specific/linux/kernel/common-config.nix#L308

@Mic92 @IvanMalison I re-opened it here, with the opinionated bootloader parts removed.
Also changed Nvidia to offload by default.


Consider uploading to official decky store?

It would be awsome if players could install this plugin from decky store directly, not by entering commands

Please be aware that we do not allow private repositories, "black-box" binaries, deliberately obfuscated code and any other items, processes etc that would undermine the team's ability to verify the functionality of the plugin and or any underlying software etc.

Due to Clash Premium is not an open-source Project, It is not allowed to submit it to decky's store.


Incorrect javascript syntax highlighting for destructored function parameters in arrow functions

From @tomsiwik on November 8, 2017 15:48

VSCode Version: 1.17.2
OS Version: Ubuntu 14.04 x64

Steps to Reproduce:

Change color-scheme to monokai (best to showcase as default scheme doesn't highlight this part)
Compare syntax-highlighting for the following methods:

// 1. Correct highlighting
doSomething = (param, { a = "default" } = {}) => {
    //noop
}

// 2. Incorrect highlighting
doSomething = (param, {
    a = "default"
} = {}) => {
    //noop
}

// 3. Correct highlighting
doSomething(param, {
    a = "default"
} = {}) {
    //noop
}

// 4. Correct highlighting
doSomething(param, { a = "default" } = {}) {
    //noop
}

// 5. Correct highlighting (notice config module)
doSomething(param, {
    a = config.get("a")
} = {}) {
    //noop
}

// 6. Correct highlighting (notice config module)
doSomething(param, { a = config.get("a") } = {}) {
    //noop
}

// 7. Incorrect highlighting (notice config module) while 1. does
doSomething = (param, { a = config.get("a")  } = {}) => {
    //noop
}

I'm surprised this even breaks github's code highlighting. Does this belong here what syntax highlighter does vscode and github use that behaves the same?

Reproduces without extensions: Yes
Copied from original issue: Microsoft/vscode#37829

This is a similar issue to #481 and #539 and some others. We currently do not handle multiline arrow function declarations well

Hey thanks. Can I ask why this is typescript related and what syntax highlighter is used that causes this same issue in github's markdown? Is typescript's syntax highlighter the same as native js in vscode?

Our JavaScript grammar is just a modified version of the TypeScript grammar from this repo. I can't speak to what github does but in general JavaScript's syntax can making parsing ambiguous cases like arrow functions difficult, especially when these are split over multiple lines

Case 7 is now fixed but Case 2 will remain broken because of tmLanguage grammar not being able to lookup beyond current line.

Thanks a lot for looking at it. I appreciate the nice answers :)


SDK content diffs - .NET 9 Preview 1 - dotnet-watch

Microsoft and source-built SDKs contain the following difference related to dotnet-watch content and packages. There should be no unintentional differences.
All intentional differences should be captured in SDK diff exclusion file:
Files only present in Microsoft-built SDK:
./sdk/x.y.z/DotnetTools/dotnet-watch/x.y.z/tools/netx.y/any/Microsoft.CodeAnalysis.ExternalAccess.RazorCompiler.dll
./sdk/x.y.z/DotnetTools/dotnet-watch/x.y.z/tools/netx.y/any/Microsoft.Extensions.DependencyInjection.Abstractions.dll
./sdk/x.y.z/DotnetTools/dotnet-watch/x.y.z/tools/netx.y/any/Microsoft.Extensions.Logging.Abstractions.dll
./sdk/x.y.z/DotnetTools/dotnet-watch/x.y.z/tools/netx.y/any/System.Security.Cryptography.Pkcs.dll
./sdk/x.y.z/DotnetTools/dotnet-watch/x.y.z/tools/netx.y/any/System.Security.Cryptography.Xml.dll

Files only present in Source-built SDK:
./sdk/x.y.z/DotnetTools/dotnet-watch/x.y.z/tools/netx.y/any/BuildHost-netcore/Microsoft.CodeAnalysis.Workspaces.MSBuild.BuildHost.deps.json

Related to https://github.com/dotnet/source-build/issues/4013

./sdk/x.y.z/DotnetTools/dotnet-watch/x.y.z/tools/netx.y/any/Microsoft.CodeAnalysis.ExternalAccess.RazorCompiler.dll is expected per https://github.com/dotnet/source-build/issues/4016#issuecomment-1908996093


Allow Rooty Soil to remember what block it was.

Describe the bug
Not necessarily a bug, but a huge flaw with rooty soils. Any addon that allows planting trees on anything other than vanilla dirt, gets an annoying behavior where the rooty soil converts back into vanilla dirt, I propose allowing rooty soil to remember what block it was before it became rooty soil, so that when it dies it returns to being the correct block.
This affects countless addons, like DTNatura, DTBiomes O' Plenty, and worst of all, TFC Dynamic Trees. I would rather not have different "soils" convert to vanilla dirt when they should not.
To Reproduce
Plant a tree from any DT version/addon onto a non vanilla "soil", kill the tree and watch as it becomes dirt.
Screenshots
Non vanilla dirt.

Kill the tree.

Bad dirt.

Additional context
It's really just not that enjoyable to have blocks you shouldn't, or soils you can't plant on.
An alternative solution could be to allow disabling the use of rooty soils entirely, which means players could plant on the same block over and over, without it converting to dirt, nor rooty soil.
Tested on version: 1.12.2-0.9.7.

In order to accomplish this I would have to make every single rooty dirt block a tile entity that holds a reference to it's original dirt.  Or similarly make the bottom truck hold the data in the same way and redesign how the branch block works.  It's not really something I'm willing to sacrifice in order for dirt to be perfect.  I'm open to suggestions though.

I'm unsure why you would have to redesign the branch block. (but then again I'm not even sure how DT works for the most part)
Would it be tough to allow planting on any "soil" type block (as a config) and not convert that block to rooty soil when the tree grows? This would eliminate the need to rewrite the block to prevent the soil changing to Vanilla dirt.

Agree with solution of removing rooty soil, if it is only for decorative purposes then maybe this can be done without converting block to rooty soil, just by adding rendering overlay on top of it (I'm not sure how this can be implemented but I think it is possible).

The rooty dirt block is not just decoration.  It has indispensable purpose.  I'm not redesigning the entire mod just because some add-on DT mods aren't handling the soil exactly right.

If I could please mention though. DynamicTrees rooty soil seems to be used to prevent easily knocking down the tree, yeah? If so, BetterWithMods manages to make normal dirt not break so easily without the addition of a new block for it's HCStumping feature.
Unless it has another purpose I'm unsure of. Please do let me know.

It contains data for the amount of soil fertility.  This data is used to keep the growth under control.  Without this data the trees will keep growing and become distorted or simply waste cpu cycles growing a tree that is already fully grown.  I added it because it was an integral part of how the system works.  The fact that it prevents the player from easily breaking the dirt block and being decorated with a root texture are just added benefits.

Just to clarify thing

I would have to make every single rooty dirt block a tile entity that holds a reference to it's original dirt.
But isn't it a tile entity already? Or it has tile entity, but not all the time, and to implement this feature you have to make it to be tile entity all the time?

Oh okay, I see. As you mentioned before though, couldn't it be stored within the tree "trunk" itself? If so, is it just too much of a technical issue to prioritize at this time? If so, if it can be gotten to eventually I don't mind waiting on it.
Sorry for keeping this issue "open" when it's been officially closed.

That would require a massive rewrite and I'm not rewriting this, how it works now is fine.  When the mod eventually is ported to later MC versions then it'll be rewritten anyway because of how blocks work in the new versions.


fix(versions): update Activiti/activiti-cloud-full-chart-security versions

UpdateBot pushed helm dependency: activiti-cloud-full-example to: 7.1.148

UpdateBot commands:
updatebot push-version --kind helm activiti-cloud-full-example 7.1.148


Incorrect Package Name in Readme

Hey, just a heads up.
Adding the crate as a dependency according to the README fails since package.name in the manifest is different from the one from the README.md

100% right, this is because the crate got renamed. Solved.


feat: error handling in controller REST API

different HTTP Status code for error scenarios
generic error message body
closes #756

Signed-off-by: sudesh.shetty<EMAIL_ADDRESS>

Ignore codecov pkg/restapi/operation/didexchange/didexchange.go I replaced error handling lines which aren't covered since before. No easy way to reproduce error on json umarshal of request body.


Windows not meant to maximise can still be supersized.

SuperSize version: 0.1.0
Expected behaviour: Non-maximisable windows should not be able to be supersized.
Actual behaviour:  Said windows (like the SuperSize configuration dialog) get supersized anyway, breaking layout with no way to restore them.

This issue is now tracked by #6


Merge the current version of Steel into master

This PR merges the current snapshot of Steel, including the various examples and libraries recently developed.
Changes are restricted to ulib/experimental and examples/steel, and as such shouldn't impact any existing F* development.
The only changes outside of these folders are small changes to ulib/Makefile and ulib/Makefile.extract, to extract the Steel framing tactics for native execution, and a slight reordering of targets in the top-level Makefile, to build binary libs before ulib/ (introduced in commit 8a009a7137)

Looks good to me as well, merging, thanks!