Make Executor::SetMonitorCallback take std::function as parameter

I change it as the way implemented in KVStore->set_updater, because it is necessary for supporting JNI callback function (scala-package in progress).
In order to support JNI callback, we have to bind the function with a jobject, (see https://github.com/javelinjs/mxnet/blob/scala-package-cc/scala-package/native/src/main/native/ml_dmlc_mxnet_native_c_api.cc#L217), which is impossible when we deal with a pure C function ptr.
And here is what I'm going to do once this PR is accepted: https://github.com/javelinjs/mxnet/blob/scala-package-cc/scala-package/native/src/main/native/ml_dmlc_mxnet_native_c_api.cc#L466
Please let me know if there exists any better solution.
This PR also includes a typo fixing and a minor bug in example/autoencoder.

I think the reason I didn't use std::function is because when the call back takes more than 1 argument, std:function doesn't work. Could you verify if it's still the case?

Try to allow the function take an additional pointer, then you can use that additional pointer to hold the closure object, see the way updater is dealt

@piiswrong I'm not sure whether I misunderstand what you meant by 'the call back takes more than 1 argument', since it does take two argument (const char* and void*) and I tested it by hand which worked. Moreover, monitor_callback_ is std::function, and what SetMonitorCallback in graph_executor.h used to do is simply converting C func PTR to std::function implicitly using std::function::operator=
And I didn't change the exposed c api, it is still pure C function ptr.

@tqchen Sorry I don't understand how to hold an java object by an additional pointer in pure c function, (this jobject will not be passed as an argument when executor calls back). KVStore::updater is using std::function, and it is the easiest way to bind an object so far I see.
Let me know if I missed anything, thank you.

@tqchen I reviewed c api again. I think I got what you mean. Did you suggest that, instead of invoking set_updater directly, we can pass jobject to c_api::MXKVStoreSetUpdater as the last argument ?
So by using this approach, besides changing SetMonitorCallback in executor.h (change argument type), we have to change MXExecutorSetMonitorCallback in c_api.h as well, add an additional pointer to its arguments, right?

@javelinjs Yes, the object pointer is passed as updater_handle

@tqchen Could you take a look again?

LGTM @piiswrong please make another review and merge it in

Can anyone help me to rerun the test? Everything went well on my own branch

Can anyone help me to rerun the test? Everything went well on my own branch

looks good. monitor is not in unit tests. Did you make sure that this works for python?

looks good. monitor is not in unit tests. Did you make sure that this works for python?

@piiswrong Yes, I printed something here https://github.com/dmlc/mxnet/blob/master/python/mxnet/monitor.py#L44 and then add Monitor(1000).install(exe) after this line https://github.com/dmlc/mxnet/blob/master/example/autoencoder/solver.py#L75 , ran example/autoencoder/mnist_sae.py and it worked.

@piiswrong Yes, I printed something here https://github.com/dmlc/mxnet/blob/master/python/mxnet/monitor.py#L44 and then add Monitor(1000).install(exe) after this line https://github.com/dmlc/mxnet/blob/master/example/autoencoder/solver.py#L75 , ran example/autoencoder/mnist_sae.py and it worked.


Visualization Support

The Python version of Langgraph added support for visualizing a StateGraph as PNGs - is this planned for JS as well? https://github.com/langchain-ai/langgraph/blob/main/examples/visualization.ipynb

I'd like to know this as well. Seems like a big limitation of the JS implementation.

Seem to be added already: https://github.com/langchain-ai/langgraphjs/pull/241


Create shellSort.java

Shell Sort

Make shell sort

Closes: #[issue number that will be closed through this PR]
Describe the changes you've made
Give a clear description what modifications you have made
Type of change
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Screenshots(optional)

Original
Updated

original screenshot
updated screenshot

We have found same code in GFG


How to tunneling raw tcp over websocket, any example?

Title: How to tunneling raw tcp over websocket, any example?
Description:
Some gw in product will disable connect method for private tcp proxy service for security. websocket is ok.
gw will refuse http connect method using tcp-over-http
how to tunnel tcp(socket servie, like android debug bridge service) over websocket with envoy, any examples?

hi @wuhuizuo there are some example configurations in the websockets sandbox here:
https://www.envoyproxy.io/docs/envoy/latest/start/sandboxes/websocket
not sure if i understand the question exactly but there is an example of tcp proxying a web socket without termination included here:
https://www.envoyproxy.io/docs/envoy/latest/start/sandboxes/websocket#step-5-test-proxying-wss-passthrough

hi @wuhuizuo there are some example configurations in the websockets sandbox here:
https://www.envoyproxy.io/docs/envoy/latest/start/sandboxes/websocket
not sure if i understand the question exactly but there is an example of tcp proxying a web socket without termination included here:
https://www.envoyproxy.io/docs/envoy/latest/start/sandboxes/websocket#step-5-test-proxying-wss-passthrough

https://github.com/envoyproxy/envoy/pull/14552 might help. It supports TCP over HTTP POST.

https://github.com/envoyproxy/envoy/pull/14552 might help. It supports TCP over HTTP POST.

hi @wuhuizuo there are some example configurations in the websockets sandbox here:
https://www.envoyproxy.io/docs/envoy/latest/start/sandboxes/websocket
not sure if i understand the question exactly but there is an example of tcp proxying a web socket, without termination, included here:
https://www.envoyproxy.io/docs/envoy/latest/start/sandboxes/websocket#step-5-test-proxying-wss-passthrough

thanks for your reply, but I need an example of tcp tunneling with websocket prototol. not proxy websocket service with tcp proxy。
wlhee's comment may be closed.

@wuhuizuo if you have resolved your issue, could you close this ticket

@wuhuizuo if you have resolved your issue, could you close this ticket

ok.

tunneling by HTTP POST is blocked by my company's firewall. The request is chunked-encoded, the the firewall buffer the request, and then timeouted for waitting the rest request fragments.
I have no right to setting the firewall.
kubectl is using websocket to portward pod or service's tcp port, and kubectl port-forward is allowed.
I think i need reopen the issue, maybe this issue should be treated as a feature requirement.
@phlax

@alyssawilk do you know if Envoy can support this ?

The last folks who ran into issues with CONNECT being unsupported added POST tunneling which is another option you might consider.
https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/http/upgrades
Envoy doesn't currently allow upgrades from raw TCP to websocket, though it could be implemented fairly easily as another option on the CONNECT path, especially if you crib off of the PR which went from CONNECT-only to CONNECT-or-POST.

@alyssawilk I have tried, it used empty http chunked body to establish the tunnel connect, firewall will failed with timeout.The firewall want to buffer the full request body.

I am wondering if the grpc streaming request is able to go through the firewall at your side. The underlying of grpc request is also POST method.
If grpc request is allowed, you should be able to borrow something from grpc request


⚠ ClamAV has degraded performance

In 0cccbd8, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 52 ms

Resolved: ClamAV performance has improved in 6f899d2.


Linker error caused by info!(...) macro.

Recently tried to build the crate, it seems newer versions of Rust / LLD break the build.
The warning / error is as follows:
lld: warning: libtests.a(tests.rcgu.o): undefined symbol: ___chkstk_ms
error: link failed

I've narrowed down the issue to the usage of the info! macro for logging:
info!("UEFI {}.{}.{}", major, minor / 10, minor % 10);

The issue might somehow be related to stack checks, considering the name of the missing function name.

Fixed in 3083747b6ab067.


Use Remix presets for virtual routes

This would allow us to remove a lot of hacky code to add virtual routes:
// vite.config.js
import {defineConfig} from 'vite';
import {hydrogen} from '@shopify/hydrogen/vite';
import {oxygen} from '@shopify/mini-oxygen/vite';
import {vitePlugin as remix} from '@remix-run/dev';
import tsconfigPaths from 'vite-tsconfig-paths';

export default defineConfig({
  plugins: [
    hydrogen(),
    oxygen(),
    remix({presets: [hydrogen.preset()]}),
    tsconfigPaths(),
  ],
});

However, there are currently some issues:

Small bug: https://github.com/remix-run/remix/pull/9194
I think all of the virtual routes would now go through the user's root loader. In the previous implementation, we could use a custom virtual root without any heavy requests. Therefore, virtual routes are probably slower now because user roots tend to contain queries for cart, header, etc. that we don't need. Is there any way to add a route and skip the root?
Not sure if it's possible to disable virtual routes with a CLI flag or when it's a build (instead of serving), since the Remix presets don't have access to the CLI flag or Vite info (I think?). This means we are now adding /graphiql and /subrequest-profiler to builds with this approach. We could probably just rely on process.env.NODE_ENV to at least avoid it in production builds.

Other thoughts:

It would be great if Remix exposed an API to add presets programmatically (Vite plugin's api: {}). That way we could add the preset only for development, and if the user doesn't pass --disable-virtual-routes to our CLI. Also, you wouldn't need to specify hydrogen.preset() in vite.config.js.

Thinking about this more, maybe we could create a secondary Remix app with only our virtual routes, and use the Node handler instead of workerd for this 🤔
Will check this idea soon.


[Storage Common] Replace old RFC1123 impl which cannot be built until GCC 5

we need to support GCC 4.8 https://azure.github.io/azure-sdk/cpp_implementation.html#linux

Hi @JeffreyRichter about this CommonHeadersRequestPolicy, originally @katmsft and I thought we might have some headers that apply to all requests of all services. But now I find that we don't have any other such headers except for x-ms-date. So I want to rename it to something like DatePolicy or DateHeaderPolicy. What do you think?

We are going to need this too at some point, see https://github.com/Azure/azure-sdk-for-cpp/blob/master/sdk/core/azure-core/src/http/retry_policy.cpp#L45

We are going to need this too at some point, see https://github.com/Azure/azure-sdk-for-cpp/blob/master/sdk/core/azure-core/src/http/retry_policy.cpp#L45

So why not Core team write a DateTime class that provides conversion from RFC-1123 and ISO8601. Both Core and storage team can benefit from it.

Hi @JeffreyRichter about this CommonHeadersRequestPolicy, originally @katmsft and I thought we might have some headers that apply to all requests of all services. But now I find that we don't have any other such headers except for x-ms-date. So I want to rename it to something like DatePolicy or DateHeaderPolicy or DataTimePolicy. What do you think?

First, I don't think you need to support x-ms-date at all; instead just use the standard HTTP date header. And the policy should set this and so it doesn't have to be exposed to customers at all.
And the Core team will create a DateTime class (copied from CPP REST with some renamed functions) which can be used.

@JeffreyRichter Does storage respect the http Date header?  From my past experience with the API there were certain APIs which had different behaviors based on if you did(n't) send the x-ms-date header.

I seem to remember someone (maybe it was you) saying this to me before. Is it that x-ms-date supports higher resolution or some thing? Whichever header we decide to use, it should not be exposed to a customer.

I seem to remember someone (maybe it was you) saying this to me before. Is it that x-ms-date supports higher resolution or some thing? Whichever header we decide to use, it should not be exposed to a customer.

This policy that sets x-ms-date header isn't exposed to customers now, it's for internal use only. And i'm looking for suggestions about the name, I think CommonHeadersRequestPolicy is not descriptive.

I guess I don't understand why there is a name at all. This looks fine to me:  request.AddHeader(c_HttpHeaderXMsDate, datetimeStr);

I guess I don't understand why there is a name at all. This looks fine to me: request.AddHeader(c_HttpHeaderXMsDate, datetimeStr);

We have to update the date on every retry, so it has to be a per-retry policy

Does the policy only update the x-ms-date header or does it do more than this?

Since this policy is Storage specific and might do all kinds of stuff over time, I suggest you just call it the StoragePerTryPolicy. Then, over time you can shove more stuff (headers or query parameters) into to without adding additional policies to the pipeline.

Since this policy is Storage specific and might do all kinds of stuff over time, I suggest you just call it the StoragePerTryPolicy. Then, over time you can shove more stuff (headers or query parameters) into to without adding additional policies to the pipeline.

Sure.


Failed writing shader header files

Hi, when trying to compile the samples, I get an error related to the shaders:

1>------ Build started: Project: pbr_uwp, Configuration: Debug ARM64 ------
1>failed writing F:\work\OpenXR-MixedReality\obj\Debug\ARM64\pbr_uwp\CompiledShaders\PbrPixelShader.h
1>failed writing F:\work\OpenXR-MixedReality\obj\Debug\ARM64\pbr_uwp\CompiledShaders\PbrVertexShader.h
1>failed writing F:\work\OpenXR-MixedReality\obj\Debug\ARM64\pbr_uwp\CompiledShaders\HighlightPixelShader.h
1>PbrResources.cpp
1>F:\work\OpenXR-MixedReality\shared\pbr\PbrResources.cpp(9,10): fatal error C1083: Cannot open include file: 'PbrPixelShader.h': No such file or directory
1>C:\Program Files (x86)\Windows Kits\10\bin\10.0.18362.0\XamlCompiler\Microsoft.Windows.UI.Xaml.Common.targets(482,5): error MSB4181: The "CompileXaml" task returned false but did not log an error.
1>Done building project "pbr_uwp.vcxproj" -- FAILED.
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========

Any idea what's wrong?
The BasicXrApp compiles and runs on Hololens 2 without any problem. Which additional information do you need to help?

I think this might be related to a recent Visual Studio update causing the shader compiler to no longer create any missing directories as needed. Until we get an update out to work around this, you can create the folder "F:\work\OpenXR-MixedReality\obj\Debug\ARM64\pbr_uwp\CompiledShaders" to work around it.

Yes, you were right. Thanks a lot!

@jworch FYI,  Bryce made a fix for this issue and I merged the fix into latest master day.  If you sync to latest, you shouldn't see above errors even without the workaround above.  Please let us know if you still hit any issue.

@yl-msft Verified: The fix works for me. Just tried it. Thx a lot.


remove spaces in license

This brings to light the need for better licenses, though. There are a lot of silly ones in the package.xml files like "ToDo" that are obviously not real licenses. Any ideas as to how this should be dealt with?

Multiple licenses apply. It's a legal requirement of the license to distribute all licenses with it, so it's entirely possible to have a single file under multiple licenses, then the rest of a repo under a different one. It's kinda strange, but so it is.

Ok then I'll remove ||

Sounds like a good idea to me.


Eventhub Component in beta and not stable

Hi,
Reviewing the supported pubsub components I see the Azure Eventhubs component is beta state.
https://docs.dapr.io/reference/components-reference/supported-pubsub/
I was wondering if anyone should shed light on what aspect of the stable requirements are not met by the component

@artursouza, @berndverst - Can you provide specifics on what is needed to flip this component from beta to stable?
Also, this question might come up for more components. Should we consider adding current status of component and the needed criteria for a component to move to the next state - example: conformance/cert tests, docs, etc?

With 1.8 release, EventHubs can be announced as stable candidate, as per the criteria defined here: https://docs.dapr.io/operations/components/certification-lifecycle/#stable ;  the component has certifications tests as noted here https://github.com/dapr/components-contrib/tree/master/tests/certification/pubsub/azure/eventhubs;
Also carrying the good documentation on its usage.  @berndverst - being the maintainer of the Repo to confirm.

Stable for reading (maybe) maybe but not for writing
https://github.com/dapr/components-contrib/issues/1764
and
https://github.com/dapr/components-contrib/issues/1598

still broken


Fixed losses docs

Renamed objectives.md -> losses.md
Replaced the terms "objective" with "loss"
Changed (nb_sample, nb_classes) to  (num_samples, num_classes)
Removed losses.md from auto-generation since no content is auto-generated

Oh, I had actually updated this one yesterday, but somehow I had forgotten to push the files. Anyway, thank you!

Always happy to help !

@fchollet It still isn't showing the available loss functions. It isn't autogenerated I guess. The version from Keras 1 had static content for that particular doc.

I just fixed it.


Extend keptn delete service with --purge flag

As a user, I can use the keptn delete service CLI command to delete an onboarded service. However, this command does not delete the Keptn CloudEvents related to this service. Consequently, I get the entire service history when onboarding a previously deleted service.
Improvement
I propose to add the --purge flag to the keptn delete service command in order to remove the service-related events from the data store.
Tasks

Add the flag to the command: https://github.com/keptn/keptn/blob/master/cli/cmd/delete_service.go#L68
Provide in-code documentation of this flag
Extend the Keptn API endpoint (/project/{projectName}/service/{serviceName}) by an additional query parameter: purge=true
Delete events from MongoDB
Clean-up of materialized views

Definition of Done

[ ] keptn delete service xyz -p=abc --purge removes every event related to the service. Consequently, another keptn onboard service xyz does not show the history of the previously deleted service xyz.
[ ] Closed #1795

I would like to work on this, can you please assign this to me.

Thanks! If you have any questions, don't hesitate to ask us on Slack (I recommend @warber @bacherfl as the primary contacts for this issue)

Hi @ankitjain28may  are you still looking into this issue? If not, we would like to re-assign it.

@christian-kreuzberger-dtx I am looking into this issue, can you please help me with the doc where I can find the routes for the API?

That's a bit hidden :D You can find it in shipyard controller
https://github.com/keptn/keptn/blob/master/shipyard-controller/handler/servicehandler.go

@christian-kreuzberger-dtx I am trying to reproduce the above issue but facing this issue while deleting the existing service
I have the project name grafana and service with name grafana with deployment strategy direct. Now I am trying to delete it via keptn CLI version 0.8 (latest master) but I create a service via keptn CLI v0.7.2
Delete project was unsuccessful
Error: Delete project was unsuccessful. could not delete service grafana from stage production: Stage  production does not exist.

Not sure what is going on there, but delete project should not be needed for this task anyway, right?

Not deleting the project, deleting the service just to see what happens. Let me try it with CLI v0.7.x

I've tried both with a recent build (see https://github.com/keptn/keptn/releases ) and delete service aswell as project seems to work fine.

Cool, let me try again then. Thanks @christian-kreuzberger-dtx


Updating phenotype data for several genes

very interesting! I really admire how you already planned ahead for all of this situations.
I'm planning to add other functional impact of some other genes, such as: CACNA1S, RYR1, CFTR, CYP4F2, G6PD, IFNL3. I see the phenotype prediction page, as well as the categories in PhenotypeMethod of gene-table.tsv. I want to add this case to our discussion as well:

CFTR: https://www.pharmgkb.org/chemical/PA165950341/guidelineAnnotation/PA166114461
In this guideline, the order of clinical action based on on genotyping will be:

G551D Het/Hom > F508del Homozygous > Some other variants Het/Hom.
However, the meanings of those variant functions have not been defined in the list FUNCTION_ORDER yet. Do you think it's OK to add more entry for CFTR variant functionality score?
Originally posted by @NTNguyen13 in https://github.com/sbslee/pypgx/issues/35#issuecomment-990869341

For case such as CACNA1S and RYR1, I'm planning to add their PhenotypeMethod as Diplotype, and then add all of the possible combinations of 2 alleles to the diplotype-table.csv file (they must be sort using sort_alleles method, of course). Finally, I will add their phenotype based on PGKB annotation. Is this OK to do so?

@NTNguyen13,
Thanks for asking before making PRs :)

Could you tell me where you are planning to obtain diplotype-phenotype mapping for CACNA1S and RYR1? It doesn't look like such data exists in CPIC or PharmGKB. It'd be great if you can share some links first.
Same question as above for CYP4F2, G6PD, and IFNL3.
For CFTR I looked at the link you attached (thanks!) and it looks like we can use diplotype-phenotype mapping for phenotype prediction (ivacaftor recommended vs. not recommended). Is that what you have in your mind? If so, you will need to update the diplotype-table.csv file, not  api.core.FUNCTION_ORDER. The latter is used to give ordering between different enzymatic functions (e.g. No Function > Normal Function).

I think it's important that we have these discussions sooner than later because I hate to see you wasting your precious time.

For CACNA1S and RYR1, I think I can use the Uncertain Susceptibility and Malignant Hyperthermia Susceptibility phenotype from here. For example:
CACNA1S Diplotype: c.520C>T/Reference => Malignant Hyperthermia Susceptibility
CACNA1S Diplotype: Reference/Reference => Uncertain Susceptibility
RYR1 Diplotype: Reference/Reference => Uncertain Susceptibility
RYR1 Diplotype: c.742G>C/Reference => Malignant Hyperthermia Susceptibility

Similarly, IFNL3 diplotype can be determined using this guide:
IFNL3 Diplotype: rs12979860CT =>  unfavorable response
IFNL3 Diplotype: rs12979860TT =>  unfavorable response
IFNL3 Diplotype: rs12979860CC =>  favorable response

In CFTR case, I'm concern about is when 1 individual has multiple alleles detected per chromosome (thanks again for your correction), such as F508del and G551D. The first issue is that, at this moment, all CFTR alleles are "Unknown Function" in allele-table.csv, so the function sort_alleles will not prioritize any of them. The second issue is the G551D allele should have the highest priority among all CFTR alleles(from my understanding from the guideline, but its functionality meaning is not quite match with the current entries in FUNCTION_ORDER list.
To sum up, for CFTR, I think we should fill up its Function column in allele-table.csv, and the way we fill should be reflected in FUNCTION_ORDER list.
Example of expected behavior: an individual with the diplotype of F508del, G551D/F508del, G511D should have higher functional priority than F508del, so the result of sorting and prioritizing process will be G551D/F508del => this individual can use ivacaftor as normal.

Hi, as I saw your work on IFNL3, CACNA1S and RYR1, I'm also curating data for G6PD genes according to this guideline. At first I tried to do it with PhenotypeMethod as Diplotype, but 186 alleles will yield 17205 combinations, which make the diplotype-table.csv quite large. Additionally, this variant is on chrX, which leads to a discrepancy in the guideline for 2 genders.
I think that I can assign the activity score using this strategy based on Class I, Class II, Class III and Class IV:

AS = 0 for Class I, Class II, Class III
AS = 1 for Class IV

One male will always be assumed to carry the B alleles (Class IV), so we will only deal with 3 cases:

2 Class IV: AS equals to 2 => normal
1 Class I-III: AS equals to 1 => variable/unknown
2 Class I-III: AS equals to 0 => Deficient/Deficient with CNSHA

What do you think about my strategy? I can make the PR if you need my help.

@NTNguyen13,
Thanks for the links and additional information!

I updated phenotype data for CACNA1S, IFNL3, and RYR1. Please let me know if you have any questions/concerns about this update.
I still couldn't find phenotype data for CYP4F2 and G6PD. If you can share links containing a phenotype table for these genes, that'd be great.
For CFTR I agree that we'd need to update both allele-table.csv and FUNCTION_ORDER. And that's exactly what I did for CACNA1S, IFNL3, and RYR1. However, I'm not so sure about giving more priority to G551D than F508del. In the example you gave, if an individual is homozygous for F508del ivacaftor is not recommended because "ivacaftor does not potentiate CFTR function in F508del-CFTR homozygotes (probably due to inadequate CFTR expression at the cell membrane)" according to the CPIC paper. Also, CFTR variants are pretty rare so I'm not sure how likely we will find multiple variants in one haplotype (chromosome). If such haplotype exists (e.g. a haplotype with both G551D and F508del), then I think it should be reported to the world so that it can be defined as a novel haplotype. Were you asking these questions as hypothetical? Or do you actually have a sample with the F508del,G551D/F508del genotype?

Yes, my question is purely hypothetical, I didn't observe it among 1000 samples of our project.

@NTNguyen13, thanks for the prompt answer. If that's the case, then I don't think we need to worry about the multiple-alleles-in-one-haplotype problem for CFTR right now :) And because of this, we don't need to update allele-table.csv for CFTR. That being said, do you want to make a PR updating phenotype data for CFTR? You can look at CACNA1S, IFNL3, and RYR1 to see which files need to be updated. But no pressure whatsoever. Just let me know if you want to give it a shot. Otherwise, I can take care of it pretty easily.
In the case of G6PD, I will have to review it more carefully because, as you said, it's on the X chromosome. Therefore, updating phenotype data for G6PD requires not just addition of some new data but also potentially new structuring of the code to, for example, allow specification of sex, etc. for phenotype prediction. I will get back to you on this soon.

Hi, I made the PR for CFTR. I tried to follow your format, but there may be something that I didn't notice.

I noticed that the original pypgx/api/data/diplotype-table.csv does not have the newline character at the end of file, while my file in #37 have it, so git diffed the whole file. I removed it and created a new PR #38


Incorrect handling of width/height attribute when the value is in percent (e.g. "100%")

Given a SVG tag like:
<svg width="100%" ...>...</svg>

This code will get the width as 100 pixels instead of the real width:
width = parseInt(clone.getAttribute('width') ||
      box.width ||
      clone.style.width ||
      window.getComputedStyle(el).getPropertyValue('width'));

Because parseInt("100%") is 100. I have a patch enhancing the options parameter so you can explicitly give a width/height but I'm not sure this is how you will like to handle this issue.

Sounds like it should first check whether clone.getAttribute('width') ends with '%' and if so bypass that value and use the box's width.
Something like the following should work:
(clone.getAttribute('width').match(/%$/) && parseInt(clone.getAttribute('width'))) ||
  box.width ||
  clone.style.width ||
  window.getComputedStyle(el).getPropertyValue('width');

Sounds good. Should I submit a PR?

Yes please. Thanks!


How do I subscribe to performance metrics data in Unity

As title， How do I subscribe to performance indicator data in Unity, I just want to get the foc and eng data, is it in DataStreamManger.cs or PMDataBuffer.cs?

Hi @tearoue323 ,
To get the performance metrics data you can use EmotivUnityItf.cs and follow the below steps:
`

call Init() to set application configuration ( clientId, client Secret...) and if you want to save data to buffer before obtaining the data , please set isDataBufferUsing = true
call Start() to start a websocket and authorizing
CreateSessionWithHeadset(string headsetId)  to create a working session with your headset
call SubscribeData(List() {"met"}) to subscribe performance metrics
If you use Data buffer -> you need to call GetPMData(string label) to get the pm data. If you don't use the data buffer -> you will get the data at OnPerfDataReceived()
`
You can reference to https://github.com/Emotiv/cortex-v2-example/blob/master/unity/Assets/SimpleExample.unity for more detail
Thank you


[Solution] for exploding layouts

Not an issue, but hopefully useful for the next poor soul wondering why using this in a grid causes his layout to continuously explode.
I placed several of these sliders inside a responsive layout (material-ui grid) and watched in awe as the page kept re-rendering several times each second, which in of itself was quite a spectacular way to fail, but I digress.
Nothing, aside from wrapping the slider with a static-width div, stopped the sliders from unleashing Armageddon.
Since this kills the responsiveness, which is usually undesired, at least to me, I spent a couple hours respectfully cursing at whoever came up with CSS, and finally found a solution that actually works.
Wrapping the slider with a div, having the following style, will forcefully limit it to the width of the parent element.
display: 'grid',
grid-template-columns: 'minmax(0, 1fr)'

Note that this only works because of grid-template-columns: 'minmax(0, 1fr)'.

I had the same issue using this library and I have no idea why this works but it does. Yay css 🎉

I have no idea why this works but it does. Yay css 🎉

Glad to see that my comment helped you.
The fact that this works while a simple fr1 does not, is caused by the CSS grid default behavior. Apparently fr1 is being interpreted as minmax(auto, 1fr), which forces columns to grow with their children. Strange choice as far as default behaviors for layout components are concerted, at least IMO.
The following answer on Stackoverflow has more insight: https://stackoverflow.com/a/52861514


AnimationTree animation flickering

Godot version:
3.1.2 >
OS/device including version:
Linux
Issue description:
All the animations that execute the AnimationTree are flickering IF you leave one animation in the AnimationPlayer with the AutoPlay flag. You have no error and this mistake is hard to identify and resolve. It's better to deactivate that flag automatically when the AnimationTree is activated maybe

Thanks for saving my day :D
It should have a warning or something like you described...

@grevius5 @igoracmelo Please upload a minimal reproduction project to make this easier to troubleshoot.

Here is a minimal repro on Godot 3.5.1:
v3.5.1 - Infinite animation flicker on Auto Advance.zip

Just open the project, select the AnimationTree and preview the changes to see transition flicker.
Here is a minimal repro on Godot 4 rc2:
v4.0.rc2 - Infinite animation flicker on Auto Advance.zip

Do the same to see the flicker.
The concept is the same in both, you can see green arrows meaning transitions are set to Auto mode, and since they have no condition, they keep switching to the next state indefinitely.
In Godot 4, auto-advance bool has been replaced with Advance > Mode enum, but the concept is the same. When set to Auto with no expression, which is the default, the transition immediately applies. If a circular transition chain exists, this causes infinite flickering.
It is however worse in Godot 4 because Auto is the default mode (although you can click on the revert button to switch back to Enabled, which suggests that Enabled should be the correct default value).
In Godot 3, as we said we can add some warning, esp. if a circular path is detected, switch mode is Immediate and conditions are empty.
In Godot 4, I see several improvements possible:
a. Make Enabled the default mode for new transitions, so animation only changes on call to travel()
b. On new transition creation, fill the Auto transition condition with a parameter yet to be set like transition_parameter, or set the transition expression with something falsey (like... false)
c. When Auto transition condition/expression is empty, consider it never fulfilled (akin to Unity Animator Controller which shows a warning when there is no transition condition as it will never happen). Invite user to set condition to true if they really want that.
d. As suggested for Godot 3, show a warning if a transition combines Switch Mode Immediate (or Sync) and Advance Mode Auto with no condition (or condition hardcoded to "true"). Invite user to set Switch Mode to AtEnd for true auto-chained transition if they want (useful for natural transition animations like IdleToCrouch -> Crouch)
Btw the doc about transition condition and expression is incomplete, and naming ambiguous. I though that Condition that the expression, but in fact it's more like a bool parameter name you'd have for a transition condition as in Unity's animator controller; while expression is an arbitrary expression, but I don't know how to access external vars from this context. But that will be for another ticket.

OK, I see why in Godot 4, Auto is the default mode: it's for the Start to first animation transition to work immediately.
Then we could make Enabled the default, except when transitioning from Start (for the first time), in which case it'd be Auto.


port variable not parsed when using system properties in json file

Hi,
I have been working with a json file which has a variable configured for the server port number. However it does not get resolved to the actual value I have added to the System using the -Dport=1234 method. When looking at the source code I see that this is indeed not resolved.
sniplet from the Server.java (com.googlecode.jmxtrans.model)
....
this.alias = resolveProps(alias);
this.host = resolveProps(host);
this.port = port;
this.username = resolveProps(username);
this.password = resolveProps(password);
....
I am wondering if this is done with a reason, and if so would like to know the reason. If this is a bug could it please be fixed? I can also contribute the fix myself if preferred.
kind regards and keep up the good work !
Vincent

That's definitely a bug. I'd be happy to accept a PR.
I'd like to rewrite part of the configuration parsing to make sure variable substitution is done at load time with a strategy similar to what we have in jmxtrans-embeded : https://github.com/jmxtrans/embedded-jmxtrans/blob/master/src/main/java/org/jmxtrans/embedded/util/json/PlaceholderEnabledJsonNodeFactory.java. If you want to port this approach to jmxtrans, feel free to have a go at it !


App_id for another app

How's app_id retrieved for another app?

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

ID: 24460ebf-4e79-a810-457f-538462186a0a
Version Independent ID: 062493b6-0b9b-a8d9-7466-878a381be604
Content: App metadata API for advertising networks - Windows UWP applications
Content Source: windows-apps-src/monetize/app-metadata-api-for-advertising-networks.md
Product: uwp
GitHub Login: @Karl-Bridge-Microsoft
Microsoft Alias: kbridge

Thanks for taking the time to let us know about this and I sincerely apologize for the delayed response. Though the scope of our feedback channel here on GitHub covers specific documentation fixes, I can help redirect you to the right support channel to get an answer to your question. The community of experts and Microsoft Engineers over at Microsoft Q&A offer timely answers to a multitude of technical questions surrounding windows-uwp.
If you do create a thread over at Microsoft Q&A, it would be fantastic if you could share that link here so I can track the progress and potentially incorporate the technical details into our documentation. Though I'm marking this Issue as closed now, please treat this thread as an open thread for follow-ups. Thanks!

Thanks for taking the time to let us know about this and I sincerely apologize for the delayed response. Though the scope of our feedback channel here on GitHub covers specific documentation fixes, I can help redirect you to the right support channel to get an answer to your question. The community of experts and Microsoft Engineers over at Microsoft Q&A offer timely answers to a multitude of technical questions surrounding windows-uwp.
If you do create a thread over at Microsoft Q&A, it would be fantastic if you could share that link here so I can track the progress and potentially incorporate the technical details into our documentation. Though I'm marking this Issue as closed now, please treat this thread as an open thread for follow-ups. Thanks!


Crash on ios when printing an array containing [deleted object]

Godot version:
3.2.1
OS/device including version:
ios 13.3.1
Issue description:
On an ios device, printing an array containing a [deleted object] crashes with the following bt.
Running the same from editor show: [[Deleted Object], [Deleted Object], [ColorRect:3180]] with no errors.
thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x4a8)
  * frame #0: 0x0000000101d63134 gameone`Object::to_string() + 32
    frame #1: 0x0000000101dc88e8 gameone`Variant::stringify(List<void const*, DefaultAllocator>&) const + 328
    frame #2: 0x00000001013f6960 gameone`GDScriptFunctions::call(GDScriptFunctions::Function, Variant const**, int, Variant&, Variant::CallError&) + 4732
    frame #3: 0x00000001013f1924 gameone`GDScriptFunction::call(GDScriptInstance*, Variant const**, int, Variant::CallError&, GDScriptFunction::CallState*) + 23756
    frame #4: 0x00000001013eb2e8 gameone`GDScriptFunctionState::resume(Variant const&) + 128
    frame #5: 0x00000001013eb6e8 gameone`GDScriptFunctionState::_signal_callback(Variant const**, int, Variant::CallError&) + 408
    frame #6: 0x0000000101d5d75c gameone`Object::call(StringName const&, Variant const**, int, Variant::CallError&) + 304
    frame #7: 0x0000000101d67bd0 gameone`Object::emit_signal(StringName const&, Variant const**, int) + 756
    frame #8: 0x0000000101d688c8 gameone`Object::emit_signal(StringName const&, Variant const&, Variant const&, Variant const&, Variant const&, Variant const&) + 100
    frame #9: 0x00000001016de590 gameone`SceneTree::idle(float) + 164
    frame #10: 0x000000010137bb84 gameone`-[AppDelegate drawView:] + 4008
    frame #11: 0x0000000101378dbc gameone`-[GLView drawView] + 268
    frame #12: 0x0000000103cfd5f8 GPUToolsCore`-[DYDisplayLinkInterposer forwardDisplayLinkCallback:] + 168
    frame #13: 0x00000001c793d130 QuartzCore`CA::Display::DisplayLink::dispatch_items(unsigned long long, unsigned long long, unsigned long long) + 632
    frame #14: 0x00000001c7a07000 QuartzCore`display_timer_callback(__CFMachPort*, void*, long, void*) + 264
    frame #15: 0x00000001c0f4d8ac CoreFoundation`__CFMachPortPerform + 172
    frame #16: 0x00000001c0f7707c CoreFoundation`__CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE1_PERFORM_FUNCTION__ + 56
    frame #17: 0x00000001c0f767a8 CoreFoundation`__CFRunLoopDoSource1 + 444
    frame #18: 0x00000001c0f7167c CoreFoundation`__CFRunLoopRun + 2168
    frame #19: 0x00000001c0f70adc CoreFoundation`CFRunLoopRunSpecific + 464
    frame #20: 0x00000001caf11328 GraphicsServices`GSEventRunModal + 104
    frame #21: 0x00000001c507e63c UIKitCore`UIApplicationMain + 1936
    frame #22: 0x000000010137a400 gameone`main + 124
    frame #23: 0x00000001c0dfa360 libdyld.dylib`start + 4

Steps to reproduce:
Minimal reproduction project:

Reproducible with disabled DEBUG_ENABLED compilation flag.
The reason for crash seems to be in condition inside variant.cpp file which return true for already deleted object, which it shouldn't: https://github.com/godotengine/godot/blob/3.2/core/variant.cpp#L1599
With DEBUG_ENABLED flag enabled, once object is already deleted the condition fails and returns [Deleted Object] or [Object:null] in case of iOS.
In gist there are logs for deleted object state right before the deletion and right after. The ok.txt is right before the deletion, and crash.txt is the state right after:
https://gist.github.com/naithar/95c1d375302e2f33f28318976a8025c2
This could probably be reproducible not only on iOS, but only on 3.2 release, as master seems to be handling objects in stringify method differently.

I was able to reproduce this crash on MacOS 10.15.4 and Windows 10 in builds exported with 3.2.1. While sometimes this issue does not happen immediately it still happens on this systems with both debug and release builds.
Minimal project is here at delete_check branch: https://github.com/naithar/gd_richedit_test/tree/delete_check

Fixed by #39736.


See console on failed jobs

When a hangfire job fails it gets placed in the 'Failed' job group.
If you go to that failed job and click it for details, it shows you the exception information.
It would be great if it would also show the output from the console that happened too!
Is that possible?

If there's a Processing state in history, console should be shown automatically.
But console is currently expired independently from the job itself (after 24 hours by default), so it may be already deleted when you check the failed job. I'm currently working on a solution for that.

Thank you for your efforts :smile: @pieceofsummer

Unless the console data expires as soon as the job fails, this is not working.   In my case, I did a write to the console right before throwing an exception.   When on the job failure screen, it only shows the exception and not the console.  Pretty sure the console has not expired yet since the job just failed in my tests?

@waynebrantley do you possibly mean "Failed Jobs" list?

Console is really not shown there, because there's no way to customize list entries. But if you click on the job id to see details, console will be there.

Yes, sorry... I mean failed job list.
When you expand it as you show and indicate it does not show the console..that would be the ideal place to show it if possible.
As you indicate, clicking on the job brings you the full screen.  (I didn't know it even had that) So, that is at least a way to see the output since it cannot display on the 'Failed jobs' screen.


Data race on MachineTasks

In TaskMachineScheduler, the MachineTasks field is accessed without a lock. I don't know if this matters though. If it doesn't matter, you could add a comment.

I don't think it matters because the TaskMachineScheduler is only used during bug-finding (and we might end up completely changing this in the near future -- its very experimental) -- on that thought, I should probably disable this feature and only use it via a command line option.

Task scheduling is optional and can be enabled using /tpl, see de10ed77387d068fc326d3f3f33eee46e0301e70


chore: release npm packages

<EMAIL_ADDRESS>

Releasing npm pacakge for #2814
Followed the process on release.md. Although I don't think the tags pushed to the vercel/turbo because this is a forked repo. I'm unsure what to do about that.

Thanks @jooshua-inglis! The team has to do the releases for now - I'll push this one this afternoon or tomorrow AM.

Shipped this at 0.0.5 @jooshua-inglis, sorry for the delay


Create "gold-standard" dataset

Upload short test input video
Upload small number of result frames to compare result against

Using Ultrasound dataset and model


Minor updates on Forms and Filters

Add the possibility to filter BGP session with the site parameter as it's present on the model + serializer
Updating tags use on form as they are already present on class NetBoxModelForm (https://github.com/netbox-community/netbox/blob/develop/netbox/netbox/forms/base.py)
Added the possibility to filter prefix list by their ID

It's easier to use the filterset from netbox https://github.com/netbox-community/netbox/blob/v3.6.4/netbox/netbox/filtersets.py#L269  and thanks to inheritance the q method is already present. no need to implement it in the query.


Add KIP-51 support

https://issues.apache.org/jira/browse/KAFKA-3316

@andrewstevenson, what do you think: we now have the command: ls to list active connectors names. On the one hand, I'd say that with the new API endpoints ls could show available connector names, and introduce ps to show  active connectors. On the other hand, that breaks the current behaviour of the CLI, where ls lists active connectors.

I think ps to show active and ls to show available inline with linux.

Exactly, my thoughts too.


ondrop event not getting fired

Problem
I'm trying to implement drag & drop on the web, but ondrop is not getting called.
                        ondrop: move |event| {
                            debug!("on drop");
                        },
                        ondragenter: move |event| {
                            event.stop_propagation();
                            debug!("ondragenter");
                        },
                        ondragover: move |event| {
                            event.stop_propagation();
                            debug!("ondragover");
                        },

I think I need to call event.preventDefault() on the JS event (not the Rust event). However, it's not clear to me how to do so.
Expected behavior
ondrop should be called.
Environment:

Dioxus version: 0.5.0
Rust version: 1.77
OS info: macOS
App platform: web

Questionnaire

[x] I'm interested in fixing this myself but don't know where to start
[ ] I would like to fix and I have a solution
[ ] I don't have time to fix this right now, but maybe later

I had the same problem not long ago.
Indeed the problem is that you have to preventDefaults() on the on ondragover event of the target.
In Dioxus you do this be setting an attribute on the rsx Element instead of calling a function on the event.
div {
    prevent_default: "ondragover",
}

Last time I checked i didn't find any documentation for this and found out about this attribute after reading a code snipped in a random issue.
It seems to be documented now:
Event Handlers

I had the same problem not long ago. Indeed the problem is that you have to preventDefaults() on the on ondragover event of the target. In Dioxus you do this be setting an attribute on the rsx Element instead of calling a function on the event.
div {
    prevent_default: "ondragover",
}

Last time I checked i didn't find any documentation for this and found out about this attribute after reading a code snipped in a random issue. It seems to be documented now: Event Handlers

Thank you, that works for me 👍

We now support .prevent_default() thanks to #2792 and have updated the relevant examples!


XHR request to query data triggered on SHIFT key press

Just noticed that at least with versions 4.0.3 and 4.0.6-rc.0, when SHIFT key is pressed down in searchbox then XHR request is triggered. This naturally causes undesired extra queries into backend.
Howto
0) reload page

Open select2 box (with XHR and searchbox)
When search field has focus, press SHIFT key down
=> XHR request is done on every press
type some letter and press SHIFT key
=> SHIFT key press does not generate extra XHR requests
Delete letter typed in step (3)
When search field has focus, press SHIFT key down
=> Redundant XHR requests are not generated
close and reopen select2 box
When search field has focus, press SHIFT key down
=> Redundant XHR requests are not generated

So it seems that issue occurs when box is opened first time. Seems to happen with various meta keys.

I can't seem to replicate this in the docs.  Are you certain that this isn't due to some other interaction in your code?  Have you tried creating a minimal, complete, and verifiable example?

Based into simple trivial example, you decided that "issue does not exist"

Just change example in https://select2.org/data-sources/ajax to be:
$('.js-data-example-ajax').select2({
  ajax: {
    url: 'https://api.github.com/search/repositories',
    dataType: 'json',
    minimumInputLength: 0,
    // Additional AJAX parameters go here; see the end of this chapter for the full code of this example
  }
});

and issue is triggered

So the issue is because of minimumInputLength: 0?


feat: bump yeoman v6

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 24 (0.0%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.6%) to 30.19%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/base-generator.ts
0
2
0.0%

src/husky/index.ts
0
2
0.0%

src/readme/index.ts
0
2
0.0%

src/prettier/index.ts
0
3
0.0%

src/vitepress/index.ts
0
5
0.0%

src/app/index.ts
0
10
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.6%

Covered Lines:
551

Relevant Lines:
1965

💛  - Coveralls


Term matrix issue

Hello,
I'm trying to use term matrix, but I get an error with the "pair wise entity search"
this is the URL it's trying to access
https://amigo.geneontology.org/amigo/search/bioentity?q=*:*&fq=taxon_subset_closure_label:"Eukaryota"&fq=taxon_subset_closure_label:"Vertebrata <vertebrates>"&fq=taxon_subset_closure_label:"Homo sapiens"&fq=type:"protein"&fq=isa_partof_closure:"GO:0000940"&fq=isa_partof_closure:"GO:0048870"

@Antonialock I'm sorry you've run into a problem here. Can you give me the inputs and path you took to get to this page with the error?

Unable to simulate working backwards from the GOlr URL:


Bitmap opaque_to_polygons documentation

Let's add documentation to opaque_to_polygons
Context: link.

Is there anything else missing here? Should I have clicked somewhere else to re-request the review or wait more patiently? (not super familiar with github review tools)


(Sometimes?) if the server is started as a systemd service then it can't find rust-analyzer

Jul 23 20:12:23 pop-os ra-multiplex[3051108]: Caused by:
Jul 23 20:12:23 pop-os ra-multiplex[3051108]:     0: spawning langauge server: server="rust-analyzer", args=[], cwd="/home/amedhi/dev/tail-sim"
Jul 23 20:12:23 pop-os ra-multiplex[3051108]:     1: No such file or directory (os error 2)

You would get an error message like the above in systemctl status ra-multiplex
This is solved by pass_environment = ["PATH"] in the ra-multiplex config

"Sometimes?" Does this really not happen every time? I'd expect this to be a difference between your systemd-user PATH and the path of your editor, probably your rust-analyzer is installed in ~/.rustup/... or ~/.cargo/bin and you only add that to your PATH in the shell initialization script, which doesn't get included in systemd-user environment.

It worked fine for months and then last week I started having the problem of it not finding rust-analyzer.

Hmm, and the only thing that change was a new version of ra-multiplex? The way we use tokio::process::Command::envs should not change the environment if pass_environment is empty (the default).

I've added more context to the error message from the description here https://github.com/pr2502/ra-multiplex/commit/247736b359fc46c19b0f7fef19504861c4b38332, that should help you figure out what's wrong with the PATH in the systemd service.


python312Packages.tencentcloud-sdk-python: 3.0.1160 -> 3.0.1161

Diff: https://github.com/TencentCloud/tencentcloud-sdk-python/compare/refs/tags/3.0.1160...3.0.1161
Changelog: https://github.com/TencentCloud/tencentcloud-sdk-python/blob/3.0.1161/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

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

4 packages built:

python311Packages.tencentcloud-sdk-python
python311Packages.tencentcloud-sdk-python.dist
python312Packages.tencentcloud-sdk-python
python312Packages.tencentcloud-sdk-python.dist


LSP-volar Installed successfully, but not available !

my environment

SublimeText 3122 ( latest third version )

LSP 0.14.3 ( the highest version LSP matching ST3 )

LSP-volar 1.1.13 ( Install a higher version of lsp-volar, the python console will report an error, which will be described in detail later )

lsp-utils setting:

{
    "nodejs_runtime": ["local"]
}

local node version
Sublime Text Installation Directory/Data/Package Storage/lsp_utils/node-runtime/16.15.0

language server version
Sublime Text Installation Directory/Data/Package Storage/LSP-volar/16.15.0

Screenshot below

When I install a higher version of LSP-volar on Sublime Text 3211, clone a specific version through git, as shown below

the python console will report an error, the error is as follows

All in all, the highest version of lsp-volar installed on Sublime Text 3211 is, 1.1.3. Versions higher than this will report similar errors in the python console, which will eventually cause the language server to fail to install

My best advice to you is to install Sublime Text 4. LSP works a way better in ST4.
Is there any reason why you still stick with ST3?

My best advice to you is to install Sublime Text 4. LSP works way better in ST4.
Is there any reason why you still stick with ST3?

The only reason I use ST3 is for learning and research purposes. ((Of course, I will whisper that I only purchased the ST4 license key a few days ago)
I used to use VSCode but got a bit tired of it permanently. I wanted to experience something new, so I switched editors. I'm a new player in Sublime Text, and I'm not very familiar with the environment here.
I found that many packages and plug-ins are developed based on ST2 and ST3. At present, there are very few packages developed based on ST4 (most of which I don’t need). So a natural idea is that these old packages and plug-ins can be used normally in ST4. ?
Currently I have 2 sublime versions installed on my computer, one is the latest ST4 and the other is the latest ST3. I found that for new players of sublime text, if they only use the latest ST4, they will not know the following things

Why is the TypeScript package not allowed to be installed by package control in ST4?
(Because ST4 has built-in TS support, if it is installed, a duplicate code prompt will appear. The package control detects the compatibility of the plug-in and the Sublime Text version for us, avoiding errors)

Why does ST4 not install Babel, JSCustom and other modern JavaScript plugins?
(Because ST4 has built-in modern JS language support, there will be conflicts if installed)

Why are there 2 or even 3 versions in the information description of many packages? (one compatible with ST4, one compatible with ST3, if there is a third, then compatible with ST2)

... and so on.
All in all, I will know more if I use multiple versions of sublime text at the same time.

The following is the running status of LSP-volar on the latest ST4, everything is ok. But I think, as a plug-in labeled ST3, it should not fail to run in the ST3 environment

This is a bug but there is really no reason to use ST3 these days. You haven't really provided any convincing argument why you would use ST3. All plugins that worked in ST3 should work just fine with ST4 and the TypeScript package was always pretty bad. Use LSP-typescript instead.


Accepts shell connections from unkown hostnames

This is useful for sqld running locally.
ps: A revert of merge commit a9bc7c60b72b44c8a104a4898e40c3a0521da653
Closes #440

ack

@haaawk why is CI not running here?


Error compiling  vai_library src code

I compile according to the official process :
https://github.com/Xilinx/Vitis-AI/tree/master/src/vai_library
The following question arises:

Hi @jzymessi , AILibrary code need to be compiled in the conda base env. Thanks!

Thanks!


Parse functionality for .ac.uk domains

Add parsing for .ac.uk with tests for jisc.ac.uk, go files formatted.

It seems that you have replaced the 4 spaces to tab in the code, It's very kind of you to take it back.

Hi Li
The code was passed through goimports which uses the default golang formatting.
I can take a look at changing this however best to stick to using go standard formatting?
All the best
Darren

Hi Darren,
Thank you very much for your pr and comment.
For the formatting, it is just everyone has his own Hamlet, that is no problem for you using tabs, you can keep your fork working.
Thank you.

Thanks.


BUILD_EXTENSIONS=True make install fail

Installing text-generation-launcher v1.1.1 (/home/luhao/text-generation-inference/launcher)
Updating crates.io index
warning: use of deprecated unit variant Quantization::Bitsandbytes: Use eetq instead, which provides better latencies overall and is drop-in in most cases
--> launcher/src/main.rs:56:27
|
56 |             Quantization::Bitsandbytes => {
|                           ^^^^^^^^^^^^
|
= note: #[warn(deprecated)] on by default
warning: text-generation-launcher (bin "text-generation-launcher") generated 1 warning
Finished release [optimized + debuginfo] target(s) in 0.74s
Replacing /root/.cargo/bin/text-generation-launcher
Replaced package text-generation-launcher v1.1.1 (/home/luhao/text-generation-inference/launcher) with text-generation-launcher v1.1.1 (/home/luhao/text-generation-inference/launcher) (executable text-generation-launcher)
if [ "$BUILD_EXTENSIONS" = "True" ]; then cd server/custom_kernels && python setup.py install; else echo "Custom kernels are disabled, you need to set the BUILD_EXTENSIONS environment variable to 'True' in order to build them. (Please read the docs, kernels might not work on all hardware)"; fi
running install
/root/anaconda3/lib/python3.11/site-packages/setuptools/_distutils/cmd.py:66: SetuptoolsDeprecationWarning: setup.py install is deprecated.
!!
    ********************************************************************************
    Please avoid running ``setup.py`` directly.
    Instead, use pypa/build, pypa/installer or other
    standards-based tools.

    See https://blog.ganssle.io/articles/2021/10/setup-py-deprecated.html for details.
    ********************************************************************************

!!
self.initialize_options()
/root/anaconda3/lib/python3.11/site-packages/setuptools/_distutils/cmd.py:66: EasyInstallDeprecationWarning: easy_install command is deprecated.
!!
    ********************************************************************************
    Please avoid running ``setup.py`` and ``easy_install``.
    Instead, use pypa/build, pypa/installer or other
    standards-based tools.

    See https://github.com/pypa/setuptools/issues/917 for details.
    ********************************************************************************

!!
self.initialize_options()

It's just a warning and does not affect usability.


[CARBONDATA-1406] Fixed the issue with the Quotation marks

LGTM
The " " is been removed from the heading as it is not the standard to have " " in headings.
@chenliang613 kindly review and merge

LGTM


Cannot select node in Service Catalogs tree

Error caught: [NoMethodError] undefined method 'get_options' for nil:NilClass in 'prov_set_show_vars'
Only RedHat and Windows ones cannot be selected in Infrastructure as a Service.
@miq-bot assign @ZitaNemeckova
@miq-bot add_label ui, bug

Seems my DB had some errors that produced it. Cannot reproduce with new one.


Address issues with HTTP testing

This can be closed in favour of #540, I merged these changes in to that branch to allow for testing.


docs(formatters): Update stylish example

PR checklist

[ ] Addresses an existing issue: #0000
[ ] New feature, bugfix, or enhancement

[ ] Includes tests

[x] Documentation update

Overview of change:
#1738 added support for rule severity but forgot to update the example for the stylish formatter.
Is there anything you'd like reviewers to focus on?
Do I need to run an additional doc script or is this something only someone with publishing rights can do?
Maybe other formatters also have outdated examples.

CHANGELOG.md entry:

Thanks for your interest in palantir/tslint, @eps1lon! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.


feat: SPIM configuration

This PR allows the configuration of the frequency, mode and over-read character of the SPIM interfaces.

Oops, I forgot to fix the examples

Looks good to me now.
The changes with the embedded-hal can be added once its new version is live.


fix: Import from server_deps

I think this module is exported by server_deps.ts

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Update Country db path to map to default country constant

Hey Andy,
Just fixing an issue that I've run across where the package is trying to open the City MaxMind db by default instead of the Country db
Thanks,
Laura

You should find this fix now in 3.2.1.


update devcontainer setup

Will require copying the added file to devcontainer.json and rebuilding the VSCODE devcontainer only.

Mostly adding you guys on a PR to this so that you're aware of what you need to do when you pull master next


build: update notice to 2023

It's 2023 :tada:
CI is failing because the year is not up to date.

:broken_heart: Tests Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2023-01-01T03:04:59.991+0000

Duration: 8 min 46 sec

Test stats :test_tube:

Test
Results

Failed
1

Passed
203

Skipped
2

Total
206

Test errors
Expand to view the tests failures
Test / Matrix - PLATFORM = 'ubuntu-18 && immutable' / Test / Test_handleInfoRequest – github.com/elastic/apm-aws-lambda/apmproxy

Expand to view the error details
 Failed

Expand to view the stacktrace
 === RUN   Test_handleInfoRequest
    logger.go:130: 2023-01-01T03:12:33.650Z	INFO	Extension listening for apm data on :1234
    receiver_test.go:204:
        	Error Trace:	receiver_test.go:204
        	Error:      	Received unexpected error:
        	            	Post "http://<IP_ADDRESS>:1234/intake/v2/events": EOF
        	Test:       	Test_handleInfoRequest
    logger.go:130: 2023-01-01T03:12:33.650Z	DEBUG	server closed
--- FAIL: Test_handleInfoRequest (0.00s)

Steps errors
Expand to view the steps failures

Running Go tests

Took 1 min 31 sec . View more details here
Description: gotestsum --format testname --junitfile junit-report.xml -- -v ./...

:robot: GitHub comments
Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

:broken_heart: Tests Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2023-01-01T03:04:59.991+0000

Duration: 8 min 46 sec

Test stats :test_tube:

Test
Results

Failed
1

Passed
203

Skipped
2

Total
206

Test errors
Expand to view the tests failures
Test / Matrix - PLATFORM = 'ubuntu-18 && immutable' / Test / Test_handleInfoRequest – github.com/elastic/apm-aws-lambda/apmproxy

Expand to view the error details
 Failed

Expand to view the stacktrace
 === RUN   Test_handleInfoRequest
    logger.go:130: 2023-01-01T03:12:33.650Z	INFO	Extension listening for apm data on :1234
    receiver_test.go:204:
        	Error Trace:	receiver_test.go:204
        	Error:      	Received unexpected error:
        	            	Post "http://<IP_ADDRESS>:1234/intake/v2/events": EOF
        	Test:       	Test_handleInfoRequest
    logger.go:130: 2023-01-01T03:12:33.650Z	DEBUG	server closed
--- FAIL: Test_handleInfoRequest (0.00s)

Steps errors
Expand to view the steps failures

Running Go tests

Took 1 min 31 sec . View more details here
Description: gotestsum --format testname --junitfile junit-report.xml -- -v ./...

:robot: GitHub comments
Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

/test

/test


lsp_finder error when file is not saved

Just started using the awesome Lspsaga lsp_finder
It's working great so far, one issue that is bothering me.
If I were to edit the file without saving, and execute lsp_finder, I'm getting the following error
E5108: Error executing lua: ...te/pack/packer/start/lspsaga.nvim/lua/lspsaga/finder.lua:608: Vim(edit):E37: No write since last change (add ! to override)
stack traceback:
        [C]: in function 'nvim_command'
        ...te/pack/packer/start/lspsaga.nvim/lua/lspsaga/finder.lua:608: in function 'open_link'
        ...te/pack/packer/start/lspsaga.nvim/lua/lspsaga/finder.lua:377: in function <...te/pack/packer/start/lspsaga.nvim/lua/lspsaga/finder.lua:376>

Is this a bug? or it's by design that we have to save the file first before executing lsp_finder?

@edward-s  which open type edit?

this is the issue im facing, lsp_finder errors if i do not save after editing the buffer, "asdf" in the example

works now, i uninstalled and re-installed neovim 0.8 nightly.
the title now disappears correctly, and no more error in lsp_finder in unsaved buffer


Adding min and max target temperatures attributes on the underlying entity for a vThern over climate (air conditioner)

I need to lower the min and max temperature that the vTherm sets on the underlying climate.
Blowing too cold or too hot air from an air conditioner isn't really comfortable especially in a small room.
For instance in heating mode, the vTherm sets the target temperature up to 30°C but 26°C would be definitely more comfortable even though heating the room is slower...
Thanks a lot for your great project.

Hello @ngthwi ,
VTherm sets the setpoint to the value you set. How is that possible that you obtain 30° if you don't request this value ?

Here's an example:

Current temp: 16°C
vTherm setpoint: 19°C
The underlying thermostat increases its own setpoint (19 to 30°C) until the the vTherm setpoint us reached (sorry for oversimplifying the actual process).
Isn't how is supposed to work Versatile Thermostat on an air conditioner?

Hello, ok I suppose you use self-regulation in a strong way. If you want to lower the max, then use lower level of self-regulation.
Please post me your VTherm attributes so that I can confirm you are in that case.

min_temp: 16
max_temp: 22
target_temp_step: 0.1
fan_modes: Auto, Silence, 1, 2, 3, 4, 5
preset_modes: none, frost, eco, comfort, boost
swing_modes: Off, Vertical, Horizontal, 3D
current_temperature: 19.1
temperature: 21
fan_mode: Auto
hvac_action: heating
preset_mode: none
swing_mode: 3D
is_on: true
hvac_mode: heat
type: null
is_controlled_by_central_mode: false
last_central_mode: null
frost_temp: 7
eco_temp: 19
boost_temp: 20
comfort_temp: 19
frost_away_temp: 0
eco_away_temp: 0
boost_away_temp: 0
comfort_away_temp: 0
power_temp: null
target_temperature_step: 0.1
ext_current_temperature: 10
ac_mode: true
current_power: null
current_power_max: null
saved_preset_mode: none
saved_target_temp: 20
saved_hvac_mode: null
motion_sensor_entity_id: null
motion_state: null
power_sensor_entity_id: null
max_power_sensor_entity_id: null
overpowering_state: null
presence_sensor_entity_id: null
presence_state: null
window_state: off
window_auto_state: off
window_bypass_state: false
window_sensor_entity_id: null
window_delay_sec: null
window_auto_enabled: false
window_auto_open_threshold: null
window_auto_close_threshold: null
window_auto_max_duration: null
window_action: window_turn_off
security_delay_min: 60
security_min_on_percent: 0.5
security_default_on_percent: 0.1
last_temperature_datetime: 2024-12-22T02:41:56.234417+01:00
last_ext_temperature_datetime: 2024-12-22T02:41:56.234417+01:00
security_state: false
minimal_activation_delay_sec: 10
device_power: 3.5
mean_cycle_power: null
total_energy: 0.18
last_update_datetime: 2024-12-22T02:42:57.252460+01:00
timezone: Europe/Paris
temperature_unit: °C
is_device_active: true
device_actives: climate.salon
nb_device_actives: 1
ema_temp: 19.1
is_used_by_central_boiler: false
temperature_slope: 0
hvac_off_reason: null
max_on_percent: null
have_valve_regulation: false
is_over_climate: true
start_hvac_action_date: 2024-12-22T02:42:57.248178+01:00
underlying_entities: climate.salon
is_regulated: true
regulated_target_temperature: 20
auto_regulation_mode: auto_regulation_expert
regulation_accumulated_error: 0
auto_fan_mode: auto_fan_high
current_auto_fan_mode: auto_fan_high
auto_activated_fan_mode: null
auto_deactivated_fan_mode: null
auto_regulation_use_device_temp: true
auto_start_stop_enable: false
auto_start_stop_level: auto_start_stop_none
auto_start_stop_dtmin: null
auto_start_stop_accumulated_error: 0
auto_start_stop_accumulated_error_threshold: null
auto_start_stop_last_switch_date: null
follow_underlying_temp_change: false
friendly_name: vTherm Salon
supported_features: 441 ```

https://github.com/user-attachments/assets/d7e7c90a-9f47-4a40-bb82-a81af43aa243

In this example, I'd like the underlying temperature (on the left) not to go over 26°C.
https://github.com/user-attachments/assets/bac7aa82-d59b-4d17-bd73-a1fbd458f094

Hello, you use the expert regulation mode auto_regulation_mode: auto_regulation_expert with certainly some very strong regulation values. You can adjust the offset_max so that the maximum value will be contained.
But the objective of the expert regulation mode is to have a large offset_max so you have some contradictory injunctions in your case.

Thanks for your quick answer.
Where can I set offser_max?
I don't see it among the attributes...
About auto_regulation_expert, isn't that the correct setting when there a" > 5°C" gap between the temperature sensors of the underlying climate (air con) and the "good" (external) sensors?

Despite having set offset_max:  6
I have the underlying target temperature more than 6°C over the vTherm setpoint...

Here's my auto_regulation_expert setting:
  auto_regulation_expert:
    kp: 0.6
    ki: 0.1
    k_ext: 0.0
    offset_max: 6
    stabilization_threshold: 0.1
    accumulated_error_threshold: 80

Yes, you have to uncheck this: auto_regulation_use_device_temp: true
Else VTherm will add the compensation offset.

I will close the issue because it is not an issue but wa can continue the discussion if you need it.

Yes, you have to uncheck this: auto_regulation_use_device_temp: true Else VTherm will add the compensation offset.
It is rarely a good idea to cumulate the both.

That would defeat the purpose of using your excellent Versatile Thermostat, wouldn’t it? ;)
Here’s my situation: I need a thermostat that displays accurate temperature setpoints and values because the ones on my air conditioner (Daikin) are completely off and unreliable.
The only issue I’m facing is that the maximum temperature setpoint on the underlying thermostat is too high to be comfortable. If I’m under the air stream, 30°C feels too hot, while 26°C is fine.
If offset_max is the correct setting to cap the maximum setpoint on the underlying thermostat, why does it show 27°C when offset_max: 6, while the vTherm setpoint is set to 19°C?

Ce serait contre-productif d’utiliser ton excellent Versatile Thermostat, non ? ;)
Mon utilisation est d'avoir d’un thermostat qui affiche des consignes et des valeurs de température précises, car celles de mon climatiseur (Daikin) sont totalement incorrectes et peu fiables.
mon souci est que la consigne maximale de température sur le thermostat sous-jacent est trop élevée pour être confortable. Si je suis sous le flux d’air, 30 °C est trop chaud, tandis que 26 °C est acceptable.
Si offset_max est le bon paramètre pour limiter la consigne maximale, pourquoi le thermostat sous-jacent affiche-t-il 27 °C avec offset_max: 6, alors que la consigne du vTherm est réglée à 19 °C ?

Parce-que tu as sélectionné :

Donc tu ajoutes à la régulation la compensation de la température interne de ton device qui comme tu le dis toi même est fausse. Décoches cette option et laisse l'auto-régulation sur expert et ça devrait être bon.
Comme dit dans la doc: https://github.com/jmcollin78/versatile_thermostat/blob/main/documentation/fr/over-climate.md#compenser-la-température-interne-du-sous-jacent
Attention toutefois, certaines températures internes varient tellement vite et sont tellement fausses qu'elles faussent totalement le calcul. Dans ce cas, là, il vaut mieux désactiver cette option.


plugins/dashboard: switch to mkNeovimPlugin and update

Switch nvimdev/dashboard-nvim to use mkNeovimPlugin and update how it is configured from the old dashboard.<option> to the current setup(config) style (this changed upstream Jan 2023).
Added a few test-cases based on examples in upstream's README, as well as documented defaults and a couple undocumented defaults lifted from the lua implementation.

[x] define test cases for documented examples & defaults
[x] ensure old options are deprecated correctly
[x] define (most) settings options
[x] define (most) theme-specific options
[ ] check if nvim-web-devicons is an actual requirement (it's in upstream's examples)

It hasn't been easy to document the options or establish their defaults, as the upstream documentation is kinda vague. For some options I've had to dig into the source code to find the actual defaults.
Fixes #1432

I've left a couple TODO/FIXME comments because upstream's documentation is kinda vague in places.
There's also some options that are theme specific (the plugin has two "themes"). Some of these options I've not made options for, but they are included in at least one of the tests.
I suspect some of them would benefit from being added to the documented options.

@RAVENz46 @sachinchaudhary1808 would you be able to review this PR as users of the plugin?

@RAVENz46 @sachinchaudhary1808 would you be able to review this PR as users of the plugin?

i don't even know how to review a pr lol

@RAVENz46 @sachinchaudhary1808 would you be able to review this PR as users of the plugin?

i don't even know how to review a pr lol

If you just want to test it out with your config locally, you can run this:
nix flake lock --override-input nixvim github:MattSturgeon/nixvim/dashboard

If you want to read through the docs generated by this PR, you can build them locally:
nix build github:MattSturgeon/nixvim/dashboard#docs

The docs will be a bunch of HTML files inside ./result, which can be opened in a web browser.

i haven't checked much but i think it works fine... do  i have to unovvride things in lock file after testing ?

do i have to unovvride things in lock file after testing ?

Next time you run nix flake update the override will get dropped. Alternatively you can use git restore to return flake.lock to an unmodified state.

do i have to unovvride things in lock file after testing ?

Next time you run nix flake update the override will get dropped. Alternatively you can use git restore to return flake.lock to an unmodified state.

okey so u meant review by u mean testing of the plugin ? (if that then for me it works)

what about those lines are they fine ?

okey so u meant review by u mean testing of the plugin ? (if that then for me it works)

Glad to hear it's working. Thanks for testing!
A review can be any form of feedback, depending on how much time and expertise you have available.

okey so u meant review by u mean testing of the plugin ? (if that then for me it works)

Glad to hear it's working. Thanks for testing!
A review can be any form of feedback, depending on how much time and expertise you have available.

what about those lines are they fine ?

Just looks like some of your recently opened files have particularly long filepaths.

i don't think so see this

this is what i'm using....
    dashboard = {
      enable = true;
      header = [
        ""
        "⡿⠋⠄⣀⣀⣤⣴⣶⣾⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣦⣌⠻⣿⣿"
        "⣴⣾⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣦⠹⣿"
        "⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣧⠹"
        "⣿⣿⡟⢹⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⡛⢿⣿⣿⣿⣮⠛⣿⣿⣿⣿⣿⣿⡆"
        "⡟⢻⡇⢸⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣣⠄⡀⢬⣭⣻⣷⡌⢿⣿⣿⣿⣿⣿"
        "⠃⣸⡀⠈⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣧⠈⣆⢹⣿⣿⣿⡈⢿⣿⣿⣿⣿"
        "⠄⢻⡇⠄⢛⣛⣻⣿⣿⣿⣿⣿⣿⣿⣿⡆⠹⣿⣆⠸⣆⠙⠛⠛⠃⠘⣿⣿⣿⣿"
        "⠄⠸⣡⠄⡈⣿⣿⣿⣿⣿⣿⣿⣿⠿⠟⠁⣠⣉⣤⣴⣿⣿⠿⠿⠿⡇⢸⣿⣿⣿"
        "⠄⡄⢿⣆⠰⡘⢿⣿⠿⢛⣉⣥⣴⣶⣿⣿⣿⣿⣻⠟⣉⣤⣶⣶⣾⣿⡄⣿⡿⢸"
        "⠄⢰⠸⣿⠄⢳⣠⣤⣾⣿⣿⣿⣿⣿⣿⣿⣿⣿⣧⣼⣿⣿⣿⣿⣿⣿⡇⢻⡇⢸"
        "⢷⡈⢣⣡⣶⠿⠟⠛⠓⣚⣻⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣇⢸⠇⠘"
        "⡀⣌⠄⠻⣧⣴⣾⣿⣿⣿⣿⣿⣿⣿⣿⡿⠟⠛⠛⠛⢿⣿⣿⣿⣿⣿⡟⠘⠄⠄"
        "⣷⡘⣷⡀⠘⣿⣿⣿⣿⣿⣿⣿⣿⡋⢀⣠⣤⣶⣶⣾⡆⣿⣿⣿⠟⠁⠄⠄⠄⠄"
        "⣿⣷⡘⣿⡀⢻⣿⣿⣿⣿⣿⣿⣿⣧⠸⣿⣿⣿⣿⣿⣷⡿⠟⠉⠄⠄⠄⠄⡄⢀"
        "⣿⣿⣷⡈⢷⡀⠙⠛⠻⠿⠿⠿⠿⠿⠷⠾⠿⠟⣛⣋⣥⣶⣄⠄⢀⣄⠹⣦⢹⣿"
        ""
      ];
      center = [
        {
          icon = "  ";
          desc = "Find File                               ";
          action = "Telescope find_files find_command=rg,--hidden,--files";
          shortcut = "SPC f f";
        }
        {
          icon = "  ";
          desc = "Recently opened                         ";
          action = "Telescope oldfiles";
          shortcut = "SPC f r";
        }
        {
          icon = "פּ  ";
          desc = "File browser                            ";
          action = "NvimTreeToggle";
          shortcut = "SPC f o";
        }
        {
          icon = "  ";
          desc = "Search for word                         ";
          action = "Telescope live_grep";
          shortcut = "SPC f g";
        }
        {
          icon = "  ";
          desc = "Help                                    ";
          action = "Telescope help_tags";
          shortcut = "SPC f h";
        }
        {
          icon = "  ";
          desc = "Commands                                ";
          action = "Telescope commands";
          shortcut = "SPC ?  ";
        }
      ];
    };

The vertical lines? That looks like an unrelated plugin, maybe indent-blankline?

The vertical lines? That looks like an unrelated plugin, maybe indent-blankline?

yeah u are right...

https://github.com/nix-community/nixvim/pull/1590#issuecomment-2140893749

this is what i'm using....

Does that config produce some "renamed option" warnings (it should)?
Based on your screenshot, it looks like your center config isn't being applied :thinking:

#1590 (comment)

this is what i'm using....

Does that config produce some "renamed option" warnings during the nix build (it should)?
Based on your screenshot, it looks like your center config isn't being applied 🤔

yeah yeah rename thing actually i copied the other user who raised the issues's config
idk about center thing

Might be an upstream issue, or I might be misinterpreting their docs. I think that pink line is supposed to get replaced by the "center" actions:

ok so this is how it looks like

and this is config
    dashboard = {
      enable = true;
      settings.config = {
        header = [
          ""
          "⡿⠋⠄⣀⣀⣤⣴⣶⣾⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣦⣌⠻⣿⣿"
          "⣴⣾⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣦⠹⣿"
          "⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣧⠹"
          "⣿⣿⡟⢹⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⡛⢿⣿⣿⣿⣮⠛⣿⣿⣿⣿⣿⣿⡆"
          "⡟⢻⡇⢸⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣣⠄⡀⢬⣭⣻⣷⡌⢿⣿⣿⣿⣿⣿"
          "⠃⣸⡀⠈⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣧⠈⣆⢹⣿⣿⣿⡈⢿⣿⣿⣿⣿"
          "⠄⢻⡇⠄⢛⣛⣻⣿⣿⣿⣿⣿⣿⣿⣿⡆⠹⣿⣆⠸⣆⠙⠛⠛⠃⠘⣿⣿⣿⣿"
          "⠄⠸⣡⠄⡈⣿⣿⣿⣿⣿⣿⣿⣿⠿⠟⠁⣠⣉⣤⣴⣿⣿⠿⠿⠿⡇⢸⣿⣿⣿"
          "⠄⡄⢿⣆⠰⡘⢿⣿⠿⢛⣉⣥⣴⣶⣿⣿⣿⣿⣻⠟⣉⣤⣶⣶⣾⣿⡄⣿⡿⢸"
          "⠄⢰⠸⣿⠄⢳⣠⣤⣾⣿⣿⣿⣿⣿⣿⣿⣿⣿⣧⣼⣿⣿⣿⣿⣿⣿⡇⢻⡇⢸"
          "⢷⡈⢣⣡⣶⠿⠟⠛⠓⣚⣻⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣇⢸⠇⠘"
          "⡀⣌⠄⠻⣧⣴⣾⣿⣿⣿⣿⣿⣿⣿⣿⡿⠟⠛⠛⠛⢿⣿⣿⣿⣿⣿⡟⠘⠄⠄"
          "⣷⡘⣷⡀⠘⣿⣿⣿⣿⣿⣿⣿⣿⡋⢀⣠⣤⣶⣶⣾⡆⣿⣿⣿⠟⠁⠄⠄⠄⠄"
          "⣿⣷⡘⣿⡀⢻⣿⣿⣿⣿⣿⣿⣿⣧⠸⣿⣿⣿⣿⣿⣷⡿⠟⠉⠄⠄⠄⠄⡄⢀"
          "⣿⣿⣷⡈⢷⡀⠙⠛⠻⠿⠿⠿⠿⠿⠷⠾⠿⠟⣛⣋⣥⣶⣄⠄⢀⣄⠹⣦⢹⣿"
          ""
        ];
        center = [
          {
            icon = "  ";
            desc = "Find File                               ";
            action = "Telescope find_files find_command=rg,--hidden,--files";
            shortcut = "SPC f f";
          }
          {
            icon = "  ";
            desc = "Recently opened                         ";
            action = "Telescope oldfiles";
            shortcut = "SPC f r";
          }
          {
            icon = "פּ  ";
            desc = "File browser                            ";
            action = "NvimTreeToggle";
            shortcut = "SPC f o";
          }
          {
            icon = "  ";
            desc = "Search for word                         ";
            action = "Telescope live_grep";
            shortcut = "SPC f g";
          }
          {
            icon = "  ";
            desc = "Help                                    ";
            action = "Telescope help_tags";
            shortcut = "SPC f h";
          }
          {
            icon = "  ";
            desc = "Commands                                ";
            action = "Telescope commands";
            shortcut = "SPC ?  ";
          }
        ];
      };
    };

Might be an upstream issue, or I might be misinterpreting their docs. I think that pink line is supposed to get replaced by the "center" actions:

yeah

Ah, center relates to the "doom" theme, for the "hyper" theme it should be shortcut.
Also, the shortcut attributes in the individual entries should be renamed to keymap.

Ah, center relates to the "doom" theme, for the "hyper" theme it should instead be shortcut.
Also, the shortcut attributes in the individual entries should be renamed to keymap.

hmm, also alpha doesn't have much options  to configure ...(dashboard alternative)

typo:
pre-commit-run> error: `settigns` should be `settings`
pre-commit-run> --> plugins/utils/dashboard.nix:125:85
pre-commit-run> |
pre-commit-run> 125 | # Make an "action" submodule type, as used by `settings.config.shortcut` and `settigns.config.center`
pre-commit-run> | ^^^^^^^^
pre-commit-run> |
error: build of '/nix/store/gs928rham5fynsv6jsbcfszaadwp5rxk-pre-commit-run.drv' on<EMAIL_ADDRESS>failed: builder for '/nix/store/gs928rham5fynsv6jsbcfszaadwp5rxk-pre-commit-run.drv' failed with exit code 1;
last 10 log lines:
> - hook id: typos
> - exit code: 2
>
> error: `settigns` should be `settings`
> --> plugins/utils/dashboard.nix:125:85
> |
> 125 | # Make an "action" submodule type, as used by `settings.config.shortcut` and `settigns.config.center


RDT actuator

Hi,
Could you please elaborate on what the terminology "RDT" stands for, and possibly a bit more information about it?
Many thanks!

Hi,
RDT stands for Resource Director Technology; you can find more information over here here.
Cheers!

Hi @tmetsch,
Thank you for the reply! A further question is that does the intent-driven-orchestration implementation depend on Intel RDT tech?
Thanks!

Good question, so it is independent of it; If your platform supports it the planner can eventually make use of it. If your platform does not support it, the planner will not consider it either. On top of that, you can individually decide which actuators you want to have active in your system.
Hope that helps.

Thank you for the help. Appreciated.


Update architect-orb to 2.1.0

Update architect-orb to 2.1.0 which will use giantswarm.github.io as catalog base url.
Towards giantswarm/giantswarm#15898

hello good people @kubasobon @tomahawk28
can you assist me in getting this PR merged? Seems your code depends on<EMAIL_ADDRESS>which has some vulnerabilities. Failing check is https://app.circleci.com/pipelines/github/giantswarm/app-checker/238/workflows/8d839687-b4b6-4e46-b9a7-ae6ee81f3751/jobs/912/parallel-runs/0/steps/0-117
I'm unsure how to upgrade.

ping @kubasobon @tomahawk28

Hi, @ubergesundheit let me take a look. I will do that in another PR.

Very nice, thank you!


Add CI to the project

Description
I see that this project does not currently use CI. Do you think it would be a good idea to add github actions (it's free for open source) for test running.
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[x] enhancement (feature request)
[x] question
[ ] documentation related
[x] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Hi @simdani
We have considered this in the past, but ultimately decided against CI as we do not, yet, have extensive test coverage on this library.

Fair enough, then I'll close issue for now.


bower package not getting "installed"

Hi,
I try to install the bower package "fullcalendar" which works fine by using the commandline, but in combination with yii2 and your extension, it only creates an empty folder without any files inside?
Do you have an idea why this could happen? no case sensitive reasons...
Thanks Phil

Do you used the last versions of Composer and plugin (dev)?

yes

Can you give me your composer.json file? Because this bug is fixed by the issue #58.

ok, one sec
{
"name": "philippfrenzel/yii2fullcalendar",
"description": "Yii2 fullcalendar Widgets",
"keywords": ["yii2", "fullcalendar"],
"homepage": "http://www.frenzel.net/",
"type": "yii2-extension",
"license": "MIT License",
"authors": [
{
"name": "Philipp Frenzel",
"email"<EMAIL_ADDRESS>"homepage": "http://www.frenzel.net/",
"role": "Author Version Yii2"
}
],
"support": {
"source": "https://github.com/philippfrenzel/yii2fullcalendar"
},
"minimum-stability": "stable",
"require": {
"yiisoft/yii2" : "",
"yiisoft/yii2-jui": "",
"bower-asset/moment" : "",
"bower-asset/fullcalendar" : "2.2.@stable"
},
"autoload": {
"psr-4": { "yii2fullcalendar\": "" }
}
}

I just tested, and everything is OK.
Update your Composer:
$ composer self-update

and the plugin:
$ composer global update

And it should be good.

sorry still not working... thanks for support -> did all you said:

the funny thing is, that moment.js is working fine as it comes from the same "reference" file?...

Try with remove your vendor directory and relaunch composer install.

Works After a complete delete!;) thanks!


Fix removing mountpoints on container rm fail

Ensure that the the container's mountpoints are cleaned up if the
container is force removed.
Fixes #19672

I think the TP4 failure is timing related. You may need to put equiv to if daemonIsWindows then sleep 30 to allow the container to complete the shutdown in the restarter.

TP4 looks ok to me

It is now.


Update README.md to point people to contrib sooner (removing contrib from pinned)

@datametrician @kkraus14 ,  updated the text to be an introduction Notebooks Contrib.


Variant and Color options not working for Theme Configuration

Environment
export default defineAppConfig({
  ui: {
    primary: 'green',
    gray: 'zinc',
    button: {
      rounded: 'rounded-none',
      base: 'bg-white',
      variant: {
        solid: 'bg-green-500',
        none: 'text-red-500'
      },
      color: {
        white: {
          solid: 'text-blue-500'
        }
      }
    },
    input: {
      rounded: 'rounded-none',
      base: 'border-white border-2 uppercase font-black',
      default: {
        color: 'white',
      }
    }
  }
})

<template>
  <div class="p-8 space-y-2">
    <UButton color="white">Hi</UButton>
    <UButton variant="none">Hi</UButton>
    <UButton color="white">Hi</UButton>
  </div>
</template>

I must be doing something wrong here, the only styles that are applied are base to none and variant.none to none the base styles aren't applied to the solid button and neither are the color.white styles aren't applied either? Did I just read the documentation wrong?

Version
2.17.0
Reproduction
https://stackblitz.com/edit/nuxt-ui-f7qvdk?file=app.config.ts
Description
The style in nuxt.config.ts should be applied to the NuxtUI components, however some of them are not applied.
Additional context
No response
Logs
No response

It seems you're in dark mode, you need to override the dark: classes.

@benjamincanac I hope you don't mind me asking a couple more theme related questions, I'm flying through the UI updates now that I know how.
Just wondering is it possible to define a global rounded-* property as for each individual component I have to add rounded: rounded-none. I have done a google search and a look through the docs.

There is no way to configure the border radius of components globally, I thought about this a lot but most components have different radiuses. What is it you're trying to do exactly?

I have a design from my work that has no rounding on any buttons, containers etc. so just trying to remove the border radius from everything.

Unfortunately, you will have to override it per-component: https://github.com/nuxt/ui/blob/64e8a87073f45f915683fca7c55e67a0bd2a4ea2/src/runtime/ui.config/elements/button.ts#L4


Broken links in tutorial

The links pointing to https://gycos.github.io on page 1,  page 2 and page 3 of the tutorial appear to be broken. They result in an HTTP 404 status code.
I could fix the issue for you by pointing the links to https://docs.rs/cursive, but I'm not sure how to do that without hard-coding the version of the crate in the link.

Doh indeed, I missed that.
For now, I can we can "hard-code" to version 0, and it will use the latest version. Once we jump to 1.0.0, we may have to update the links, unless docs.rs has a new redirection mechanism.

Cool, it's fixed.


Slider inverted behaves the same way as dir='rtl'

Bug report
The slider inverted props inverts the filled Track and the scale.
I am using the shadcnui wrapper
Current Behavior
When I use this code:
<Slider.Root
          className="relative flex items-center select-none touch-none w-[200px] h-5"
          defaultValue={[5]}
          min={1}
          max={30}
          step={1}
          inverted={true}
          onValueChange={(e) => {
            setSliderValue(e[0]);
          }}
        >
          <Slider.Track className="bg-blackA7 relative grow rounded-full h-[3px]">
            <Slider.Range className="absolute bg-white rounded-full h-full" />
          </Slider.Track>
          <Slider.Thumb
            className="block w-5 h-5 bg-white shadow-[0_2px_10px] shadow-blackA4 rounded-[10px] hover:bg-violet3 focus:outline-none focus:shadow-[0_0_0_5px] focus:shadow-blackA5"
            aria-label="Volume"
          />
        </Slider.Root>

I expect to have the filled track inverting only. However, the values are also reverting. inverting behaves the same way as the dir='rtl'

Expected behavior
When I use this code with inverted={true} :
<Slider.Root
          className="relative flex items-center select-none touch-none w-[200px] h-5"
          defaultValue={[5]}
          min={1}
          max={30}
          step={1}
          inverted={true}
          onValueChange={(e) => {
            setSliderValue(e[0]);
          }}
        >
          <Slider.Track className="bg-blackA7 relative grow rounded-full h-[3px]">
            <Slider.Range className="absolute bg-white rounded-full h-full" />
          </Slider.Track>
          <Slider.Thumb
            className="block w-5 h-5 bg-white shadow-[0_2px_10px] shadow-blackA4 rounded-[10px] hover:bg-violet3 focus:outline-none focus:shadow-[0_0_0_5px] focus:shadow-blackA5"
            aria-label="Volume"
          />
        </Slider.Root>

I just want to invert the filled track in this way:

Reproducible example
CodeSandbox Template
"use client";

import * as React from "react";
import * as Slider from "@radix-ui/react-slider";

// If you aren't working in TypeScript, feel free to rename
// this file to `App.js` and reproduce your issue without
// types.

export function App() {
  const [sliderValue, setSliderValue] = React.useState(1);
  return (

Radix Primitives Template

        <Slider.Root
          className="relative flex items-center select-none touch-none w-[200px] h-5"
          defaultValue={[5]}
          min={1}
          max={30}
          step={1}
          inverted={true}
          onValueChange={(e) => {
            setSliderValue(e[0]);
          }}
        >
          <Slider.Track className="bg-blackA7 relative grow rounded-full h-[3px]">
            <Slider.Range className="absolute bg-white rounded-full h-full" />
          </Slider.Track>
          <Slider.Thumb
            className="block w-5 h-5 bg-white shadow-[0_2px_10px] shadow-blackA4 rounded-[10px] hover:bg-violet3 focus:outline-none focus:shadow-[0_0_0_5px] focus:shadow-blackA5"
            aria-label="Volume"
          />
        </Slider.Root>

  );
}

Suggested solution

Add a way to invert the filled track without inverting the scale.
Additional context

Your environment

Software
Name(s)
Version

Radix Package(s)
@radix-ui/react-slider
1.2.0

React
n/a
18

Browser
Google Chrome
126.0.6478.62

Assistive tech

Node
n/a
20

npm/yarn

Operating System
MacOS
14.5

This is by design, inverted inverts the direction of the entire slider. Sounds like you just want to switch up the styling of the track and range, without changing how the Slider actually works.


aplwrap: init at version 2.4

Motivation for this change
aplwrap is missing, so we're adding it
Things done
Created new package. Tested with:
$ nix run .#aplwrap

(aplwrap:77558): Gtk-WARNING **: 21:54:12.304: Theme parsing error: <data>:1:10: not a number

(aplwrap:77558): Gtk-WARNING **: 21:54:12.304: Theme parsing error: <data>:1:23: Using Pango syntax for the font: style property is deprecated; please use CSS syntax

(aplwrap:77558): Gtk-WARNING **: 21:54:12.304: Theme parsing error: <data>:1:86: 'ed2' is not a valid color name

(aplwrap:77558): Gtk-WARNING **: 21:54:17.343: Could not load a pixbuf from /org/gtk/libgtk/theme/Adwaita/assets/check-symbolic.svg.
This may indicate that pixbuf loaders or the mime database could not be found.

Played around within aplwrap (executed some APL commands) and works as expected. Gtk icons seem to work, too (after installing them in my profile).

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

Adding maintainer of gnuapl as reviewer since they might have some insight. @kovirobi

Sorry, I don't have much time these days to do much for NixOS, sadly, but the package looks good to me. That being said I'm just a contributor not a reviewer.

/marvin awaiting_changes

/status awaiting_changes

Oh, this was manually given the needs_reviewer status so I can't change it with marvin. That status is intended for the bot, see https://github.com/timokau/marvin-mk2#marvin---making-sure-your-pr-gets-a-review-and-your-reviews-dont-get-lost