correctly marshal constant arrays

 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.:x: IIFE sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@IIFE I've merged one of your PR-s and there conflicts now, please resolve.

@ddobrev @tritao I've updated this PR with latest changes from master.
I've purposely only put tests for CLI because C# generator appears to have issues with the test cases. Obviously it needs to be looked at and fixed, but probably best done as a separate PR so that we're not holding off the CLI fixes.
Thanks.

@tritao @ddobrev Would it be possible to progress this PR please? Let me know if you have any questions.
Thanks.

@tritao @ddobrev Would it be possible to progress this PR please? Let me know if you have any questions.
Thanks.

Sorry for the delay in processing this one.
Looks mostly good, I have just one question I left as a review.

@tritao No problem. Hopefully my answer to your question makes sense.

Thanks again for the contribution.


增加代码风格检查

暂时没加goimport风格检查

Codecov Report

Merging #199 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #199      +/-   ##
==========================================
+ Coverage   73.02%   73.02%   +<.01%
==========================================
  Files         203      203
  Lines       17454    17454
==========================================
+ Hits        12745    12746       +1
+ Misses       4041     4038       -3
- Partials      668      670       +2

Impacted Files
Coverage Δ

g/container/gtree/gtree_redblacktree.go
96.95% <0%> (-0.41%)
:arrow_down:

g/os/gspath/gspath_cache.go
88.63% <0%> (+6.81%)
:arrow_up:

Continue to review full report at Codecov.

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


Generate takes very long time to finish

I have created the following application:
from cot import Collection
import os

os.environ["OPENAI_API_KEY"] = "<my_api_key>"

dataset = Collection(["med_qa"])
config = {
    "instruction_keys": ['qa-01'],
    "cot_trigger_keys": ['kojima-01'],
    "answer_extraction_keys": ['kojima-A-D'],
    "api_service": "openai",
    "engine": "text-davinci-003",
    "temperature": 0.35,
    "max_tokens": 512,
    "verbose": False,
    "warn": True
}

dataset_subset = dataset.select(split="train", number_samples=20, random_samples = True, seed = 0)

dataset_subset.generate(config = config)

Running the application, however, takes a very long time to finish (>> 30 minutes). During this time it progresses through the dataset, but each step takes a long time to finish and it seems like the application lags out at some point and doesn't continue generating.
After a while, the following message/warning appears:
Retrying langchain.llms.openai.completion_with_retry.<locals>._completion_with_retry in 4.0 seconds as it raised APIError: Bad gateway. {"error":{"code":502,"message":"Bad gateway.","param":null,"type":"cf_bad_gateway"}} 502 {'error': {'code': 502, 'message': 'Bad gateway.', 'param': None, 'type': 'cf_bad_gateway'}} {'Date': 'Fri, 26 May 2023 13:26:41 GMT', 'Content-Type': 'application/json', 'Content-Length': '84', 'Connection': 'keep-alive', 'X-Frame-Options': 'SAMEORIGIN', 'Referrer-Policy': 'same-origin', 'Cache-Control': 'private, max-age=0, no-store, no-cache, must-revalidate, post-check=0, pre-check=0', 'Expires': 'Thu, 01 Jan 1970 00:00:01 GMT', 'Server': 'cloudflare', 'CF-RAY': '7cd652355b51c2c3-VIE', 'alt-svc': 'h3=":443"; ma=86400'}.
Retrying langchain.llms.openai.completion_with_retry.<locals>._completion_with_retry in 4.0 seconds as it raised APIError: Bad gateway. {"error":{"code":502,"message":"Bad gateway.","param":null,"type":"cf_bad_gateway"}} 502 {'error': {'code': 502, 'message': 'Bad gateway.', 'param': None, 'type': 'cf_bad_gateway'}} {'Date': 'Fri, 26 May 2023 13:31:56 GMT', 'Content-Type': 'application/json', 'Content-Length': '84', 'Connection': 'keep-alive', 'X-Frame-Options': 'SAMEORIGIN', 'Referrer-Policy': 'same-origin', 'Cache-Control': 'private, max-age=0, no-store, no-cache, must-revalidate, post-check=0, pre-check=0', 'Expires': 'Thu, 01 Jan 1970 00:00:01 GMT', 'Server': 'cloudflare', 'CF-RAY': '7cd65a493845c2c3-VIE', 'alt-svc': 'h3=":443"; ma=86400'}.

NOTE: reducing the number of samples SOMETIMES lets the method finish. (tried with 1 and 5)

Hey!
would you try testing our code by changing two lines in the config:
"api_service": "mock_api",
"engine": "",
Then it just runs our code and inserts a mock message. That should be very fast, within seconds, if not I will have a more detailled look at it.
For now, I believe the problem is at the openai API...
Maybe just try again in a few hours or tomorrow.

Sorry for getting back this late.
I tried rerunning the code with the mock API, and this works almost instantly.
And it really seems that last week the issue was with openAI as the generation works fairly fast today. (finishing 5 samples in ~1 minute)


Adding yaml for DMC sample browser: 11on12 and Bundles.

As per email convo (about the "products" yaml), we probably want to close this one and do another one later?

I just did another commit to the same pr. @stevewhims please let me know what you think of these changes.


2.3.1. Backend payment request sent, hissoty item has strange link. Was payment email sent?

Backend payment request sent, hissoty item has strange link. Was payment email sent?

Hi Sean,
Looks like your magento version is not able to show the payment link but only the text. Yes the payment url is in the order confirmation that is send to the customer.
Regards,
Rik

why close it? We use magento <IP_ADDRESS>
On Wed, Jul 15, 2015 at 12:46 PM, Rik ter Beek<EMAIL_ADDRESS>wrote:

Closed #366 https://github.com/Adyen/magento/issues/366.
—
Reply to this email directly or view it on GitHub
https://github.com/Adyen/magento/issues/366#event-356317999.

Hi Sean,
Because your question is answered. I think url is not showing because your Magento is not allowing links in the comment history.
Regards,
Rik
Adyen

It was not a question / something is wrong it seems
Are you saying: this is a Magento setting to allow or not allow links in
history? (I remember seeing links here before ... )
And if html is not allowed, should the message then not be something else?
thx
On Wed, Jul 15, 2015 at 2:24 PM, Rik ter Beek<EMAIL_ADDRESS>wrote:

Hi Sean,
Because your question is answered. I think url is not showing because your
Magento is not allowing links in the comment history.
Regards,
Rik
Adyen
—
Reply to this email directly or view it on GitHub
https://github.com/Adyen/magento/issues/366#issuecomment-121600630.

Hi Sean,
In your screenshot it looks like valid html code but i do not see the full link. I have no idea why it is not showing I have <IP_ADDRESS> as well and I see the link.
Regards,
Rik
Adyen

Hi Rik,
the link seems to be valid html indeed
FYI some remarks
The customer claims not to have received the payment link.
Is there a status overview where we can monitor payment request emails?
(are they sent yes or no)
Also it states "Customer not notified", and though I understand Magento did
not do it => Adyen did! So was the e-mail sent?
And last: is there a way to send a reminder? or does Adyen do this
automatically?
Is the field set to allow html? Or does one need to set this in config
manually? or on attribute level? (we have been using this install since
magento 1.3 so maybe somethings have changed)
[image: Inline image 1]
On Mon, Jul 20, 2015 at 9:25 AM, Rik ter Beek<EMAIL_ADDRESS>wrote:

Hi Sean,
In your screenshot it looks like valid html code but i do not see the full
link. I have no idea why it is not showing I have <IP_ADDRESS> as well and I see
the link.
Regards,
Rik
Adyen
—
Reply to this email directly or view it on GitHub
https://github.com/Adyen/magento/issues/366#issuecomment-122785070.


Regression on PPC64LE cross-compilation build

The regression happened after merging #16854
There are breakages in evp_test with aes-128-gcm and also in ssl tests with aes-256-gcm ciphers.
See https://github.com/openssl/openssl/runs/4915736203?check_suite_focus=true
@dannytsen can you please look?

Unless this is resolved quickly we would have to revert.

@t8m I'll take a look.

@t8m The failure looks strange.  In evp_test with aes-128-gcm,it failed in these instances with the mismatched last byte.  I have no clues at this point and have to look into it further.
    # aes-128-gcm is fetched
    # ERROR: (memory) 'expected == got' failed @ test/evp_test.c:108
    # --- expected
    # +++ got
    # 0000: 56b3373ca9ef6e4a 2b64fe1e9a17b614 25f10d47a75a5fce 13efc6bc784af24f
    # 0020: 4141bdd48cf7c770 887afd573cca5418 a9aeffcd7c5ceddf c6a78397b9a85b49
    # 0040: 9da558257267caab 2ad0b23ca476a53c b17fb41c4b8b475c b4f3f7165094c229
    # 0060: c9e8c4dc0a2a5ff1 903e501511221376 a1cdb8364c5061a2 0cae74bc4acd76ce
    # 0080: b0abc9fd3217ef9f 8c90be402ddf6d86 97f4f880dff15bfb 7a6b28241ec8fe18
    # 00a0:-3c2d59e3f9dfff65 3c7126f0acb9e642 11f42bae12af462b 1070bef1ab5e3606
    # 00a0:+3c2d59e3f9dfff65 3c7126f0acb9e642 11f42bae12af462b 1070bef1ab5e3625
    #                                                                        ^^
    #
    # INFO:  @ test/evp_test.c:3365
    # ../../test/recipes/30-test_evp_data/evpciph_aes_common.txt:860: Source of above error (aligned in-place, fragmented); unexpected error VALUE_MISMATCH
    # INFO:  @ test/evp_test.c:589
    # aes-128-gcm is fetched
    # ERROR: (memory) 'expected == got' failed @ test/evp_test.c:108
    # --- expected
    # +++ got
    # 0000: 56b3373ca9ef6e4a 2b64fe1e9a17b614 25f10d47a75a5fce 13efc6bc784af24f
    # 0020: 4141bdd48cf7c770 887afd573cca5418 a9aeffcd7c5ceddf c6a78397b9a85b49
    # 0040: 9da558257267caab 2ad0b23ca476a53c b17fb41c4b8b475c b4f3f7165094c229
    # 0060: c9e8c4dc0a2a5ff1 903e501511221376 a1cdb8364c5061a2 0cae74bc4acd76ce
    # 0080: b0abc9fd3217ef9f 8c90be402ddf6d86 97f4f880dff15bfb 7a6b28241ec8fe18
    # 00a0: 3c2d59e3f9dfff65 3c7126f0acb9e642 11f42bae12af462b 1070bef1ab5e3606
    # 00c0: 872ca10dee15b324 9b1a1b958f23134c 4bccb7d03200bce4 20a2f8eb66dcf364
    # 00e0:-4d1423c1b5699003 c13ecef4bf38a3b6
    # 00e0:+4d1423c1b5699003 c13ecef4bf38a325
    #                                      ^^
    #
    # INFO:  @ test/evp_test.c:3365
    # ../../test/recipes/30-test_evp_data/evpciph_aes_common.txt:869: Source of above error (aligned in-place, fragmented); unexpected error VALUE_MISMATCH
    # INFO:  @ test/evp_test.c:589
    # aes-128-gcm is fetched
    # ERROR: (memory) 'expected == got' failed @ test/evp_test.c:108
    # --- expected
    # +++ got
    # 0000: 56b3373ca9ef6e4a 2b64fe1e9a17b614 25f10d47a75a5fce 13efc6bc784af24f
    # 0020: 4141bdd48cf7c770 887afd573cca5418 a9aeffcd7c5ceddf c6a78397b9a85b49
    # 0040: 9da558257267caab 2ad0b23ca476a53c b17fb41c4b8b475c b4f3f7165094c229
    # 0060: c9e8c4dc0a2a5ff1 903e501511221376 a1cdb8364c5061a2 0cae74bc4acd76ce
    # 0080: b0abc9fd3217ef9f 8c90be402ddf6d86 97f4f880dff15bfb 7a6b28241ec8fe18
    # 00a0: 3c2d59e3f9dfff65 3c7126f0acb9e642 11f42bae12af462b 1070bef1ab5e3606
    # 00c0: 872ca10dee15b324 9b1a1b958f23134c 4bccb7d03200bce4 20a2f8eb66dcf364
    # 00e0: 4d1423c1b5699003 c13ecef4bf38a3b6 0eedc34033bac190 2783dc6d89e2e774
    # 0100:-188a439c7ebcc067 2dbda4ddcfb27946 13b0be41315ef778 708a70ee7d75165c
    # 0100:+188a439c7ebcc067 2dbda4ddcfb27946 13b0be41315ef778 708a70ee7d751625
    #                                                                        ^^
    #

@t8m I may have found the problem.  It looks like it's counter overflow.  Will have a solution.

@t8m I believe I have fixed the problem.  Here is the test result.  I'll double check.  So, What should do next?
All tests successful.
Files=241, Tests=3277, 387 wallclock secs ( 6.48 usr  0.34 sys + 362.64 cusr 21.22 csys = 390.68 CPU)
Result: PASS

@t8m Should I create another PR for the fix?  Thanks.

Yes, please create a PR for the fix.

Closing this as it's been fixed.


Insert KaTeX autorenader at last of doc

Resolve #1

Now module level document work, but auto-render works only if the #[katexit] is added to the page.


Can we expect a 1.18 release?

Ofc when I find the time

The axolotls are begging for you to do it and so do the floating islanders.

Can I do some build system magic?

The port is going to be non-trivial because biomes are now 3D so it'll take some time

Also, check the pull requests bit for I added java 17 compatibility.


feat: print report to console when reporting is not specified (#172)

Possible solution for #172 print to console

@KengoTODA thoughts on this?

@jscancella sorry for my late response.
this PR breaks backward compatibility, so I wanna release this as a part of v5 release.
Could you review and merge #555 first? Then we can release v5.0.0-beta1 with this feature.

I've merged these changes to the beta branch manually. Thanks for your contribution!


Flexible indexes: how to handle possible dimension vs. coordinate name conflicts?

Another thing that I've noticed while working on #5692.
Currently it is not possible to have a Dataset with a same name used for both a dimension and a multi-index level. I guess the reason is to prevent some errors like unmatched dimension sizes when eventually the multi-index is dropped with renamed dimension(s) according to the level names (e.g., with sel or unstack). See #2299.
I'm wondering how we should handle this in the context of flexible / custom indexes:
A. Keep this current behavior as a special case for (pandas) multi-indexes. This would avoid breaking changes but how to support custom indexes that could eventually be used like pandas multi-indexes in sel or stack?
B. Introduce some tag in xarray.Index so that we can identify a multi-coordinate index that behaves like a hierarchical index (i.e., levels may be dropped into a single index/coordinate with dimension renaming)
C. Do not allow any dimension name matching the name of a coordinate attached to a multi-coordinate index. This seems silly?
D. Eventually revert #2353 and let users taking care of potential conflicts.

Both 1 and 2 are now supported with v2023.8.0, so I think we can close this.


[QUESTION] docker 镜像 不支持Ubuntu吗，我看介绍是支持两个系统的，但是所有的tag都是alpine

Before you submit this issue, you has been search all existed issues and search the documentation

[] I've been search all existed issues
[] I've been read all documentation

Describe your question

(php8.1)


Routes with aliases and named child routes cause warnings

Vue.js / vue-router versions
>= 2.1.2
Reproduction Link
https://jsbin.com/satalijiga/edit?js,console
Steps to reproduce
Give a child route a name, add an alias to the parent
What is actually happening?
The console is blasted with warnings about duplicate named route definitions. This happened at 2.1.2

"[vue-router] Duplicate named routes definition: { name: "test.subpage", path: "/subpage" }"

Updated to<EMAIL_ADDRESS>problem still exists.

Create a new issue with a repro. The one provided before doesn't produce any warning now. You can use this template: http://jsfiddle.net/posva/9r6xhqbp/

@yyx990803 Why is this closed? I have the same issue. At least it should be answered or referenced

Evan closed it because the issue was fixed in the commit https://github.com/vuejs/vue-router/commit/67307680c2a0a70d47d6a11697ec27e6876917e2


What does ns stand for?

There are many config files under ./src/config/algs and python modules under ./src/modules which has suffix ns. I go through the paper, code comment, README and even PyMARL project, but failed to find its meaning. What does it stand for?

Hi,
ns stands for non-shared parameters.


Version Upgrade failure to 1.26.2 with spring 3.1.x

Hi recently i try to upgrade my armeria version to 1.26.2 with spring 3.x
my application seems launch successfully with log open port 8080, 50051(gprc)
but when i try to make a connection to application with grpc stub or postman reflection
it throw error like belows
[C] 2023-11-15T19:33:44.448+09:00|WARN|armeria-common-worker-kqueue-3-4||Exceptions.java:115|[id: 0xad4406ae, L:/[0:0:0:0:0:0:0:1%0]:50051 ! R:/[0:0:0:0:0:0:0:1%0]:65298][h1c] Unexpected exception:
java.lang.NoSuchMethodError: 'io.netty.handler.codec.http2.AbstractHttp2ConnectionHandlerBuilder com.linecorp.armeria.server.Http2ServerConnectionHandlerBuilder.decoderEnforceMaxRstFramesPerWindow(int, int)'
	at com.linecorp.armeria.server.Http2ServerConnectionHandlerBuilder.<init>(Http2ServerConnectionHandlerBuilder.java:45)
	at com.linecorp.armeria.server.HttpServerPipelineConfigurator.newHttp2ConnectionHandler(HttpServerPipelineConfigurator.java:248)
	at com.linecorp.armeria.server.HttpServerPipelineConfigurator.access$800(HttpServerPipelineConfigurator.java:108)
	at com.linecorp.armeria.server.HttpServerPipelineConfigurator$Http2PrefaceOrHttpHandler.configureHttp2(HttpServerPipelineConfigurator.java:702)
	at com.linecorp.armeria.server.HttpServerPipelineConfigurator$Http2PrefaceOrHttpHandler.decode(HttpServerPipelineConfigurator.java:671)
	at io.netty.handler.codec.ByteToMessageDecoder.decodeRemovalReentryProtection(ByteToMessageDecoder.java:529)
	at io.netty.handler.codec.ByteToMessageDecoder.callDecode(ByteToMessageDecoder.java:468)
	at io.netty.handler.codec.ByteToMessageDecoder.channelInputClosed(ByteToMessageDecoder.java:426)
	at io.netty.handler.codec.ByteToMessageDecoder.channelInputClosed(ByteToMessageDecoder.java:393)
	at io.netty.handler.codec.ByteToMessageDecoder.channelInactive(ByteToMessageDecoder.java:376)
	at com.linecorp.armeria.server.HttpServerPipelineConfigurator$Http2PrefaceOrHttpHandler.channelInactive(HttpServerPipelineConfigurator.java:658)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelInactive(AbstractChannelHandlerContext.java:305)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelInactive(AbstractChannelHandlerContext.java:281)
	at io.netty.channel.AbstractChannelHandlerContext.fireChannelInactive(AbstractChannelHandlerContext.java:274)
	at io.netty.handler.logging.LoggingHandler.channelInactive(LoggingHandler.java:206)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelInactive(AbstractChannelHandlerContext.java:303)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelInactive(AbstractChannelHandlerContext.java:281)
	at io.netty.channel.AbstractChannelHandlerContext.fireChannelInactive(AbstractChannelHandlerContext.java:274)
	at io.netty.channel.DefaultChannelPipeline$HeadContext.channelInactive(DefaultChannelPipeline.java:1405)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelInactive(AbstractChannelHandlerContext.java:301)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelInactive(AbstractChannelHandlerContext.java:281)
	at io.netty.channel.DefaultChannelPipeline.fireChannelInactive(DefaultChannelPipeline.java:901)
	at io.netty.channel.AbstractChannel$AbstractUnsafe$7.run(AbstractChannel.java:813)
	at io.netty.util.concurrent.AbstractEventExecutor.runTask$$$capture(AbstractEventExecutor.java:174)
	at io.netty.util.concurrent.AbstractEventExecutor.runTask(AbstractEventExecutor.java)
	at io.netty.util.concurrent.AbstractEventExecutor.safeExecute$$$capture(AbstractEventExecutor.java:167)
	at io.netty.util.concurrent.AbstractEventExecutor.safeExecute(AbstractEventExecutor.java)
	at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:470)
	at io.netty.channel.kqueue.KQueueEventLoop.run(KQueueEventLoop.java:300)
	at io.netty.util.concurrent.SingleThreadEventExecutor$4.run(SingleThreadEventExecutor.java:997)
	at io.netty.util.internal.ThreadExecutorMap$2.run(ThreadExecutorMap.java:74)
	at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
	at java.base/java.lang.Thread.run(Thread.java:833)

version 1.25.x not working like this.
so i think some change after 1.26.0 caused this problem.
i found some features from lease note
Server
  .builder()
  .http2MaxResetFramesPerWindow(100, 10)
  .build();

is this can make this problems?
please let me know how to handle this problems.
thanks!

i think if support default value for this option
would help to prevent unexpected server failure of person who try to version upgrade like me

Hi, in general armeria has a tight dependency with netty so it is recommended that you match the version of Armeria with Netty.
https://github.com/line/armeria/blob/e59527529d9d49cc16aaaac0d1d32f1beaa05ae1/dependencies.toml#L87
Can you try with Netty version 4.1.100.Final?

i also find bom of netty version 4.0 from guide page
i try to add that dependency and let you know the result!
i also try to see pr that i mentioned feature but it already have it's own default settings. so i also agree the reason is not from there
anyway! i will update after my try!
thanks for reply!

@jrhee17
    implementation(platform("io.netty:netty-bom:4.1.100.Final"))
    implementation("com.linecorp.armeria:armeria:$armeriaVersion")
    implementation("com.linecorp.armeria:armeria-spring-boot3-webflux-starter:$armeriaVersion")
    implementation("com.linecorp.armeria:armeria-grpc:$armeriaVersion")

this is what i used
but also same error shown

That's odd. If https://github.com/advisories/GHSA-xpw8-rcwv-8f8p is included than a NoSuchMethodError shouldn't be thrown.
What dependency management are you using? Can you confirm that the correct version is used at runtime? If gradle, I guess you can use something like ./gradlew dependencies

i found same issue as mine from stackoverflow
https://stackoverflow.com/questions/77307795/how-to-force-specific-version-of-a-transitive-dependency-netty-codec-http-in-g
it seems like spring dependency management plugin forced override netty version to 4.1.97
and my spring boot version is 3.1.3
after i upgrade spring boot version to 3.1.5 which described use netty 4.1.100 here (https://docs.spring.io/spring-boot/docs/current/reference/html/dependency-versions.html#appendix.dependency-versions.coordinates)
after that it successfully fetch netty version 4.1.100 final...
i think this situation is not from here.
but every user who integrated with spring boot 3 should know!
now it's working find!

For spring-boot 2.x user:  we should use spring boot 2.7.17 even though armeria 1.26.3 depends on 2.7.16.


Dse 5.0.0

@kow3ns if this is merged to master will vinilla Apache Cassandra still work?
Uber shared today that they have multiple production deployments on Cassandra 2.2.X. For new deployments they plan to use 3.0.X but they still need to maintain the existing deployments on 2.2.X.
Bottom line - we'll need a strategy for supporting 2.2.X, 3.0.X, and DSE 5.X.

👍

retest this please

retest this please

Going by how a lot of the code changes appear to be config related, might be able to have an empty/optional 'dse' section in the default yaml configs, and then fill in the config objects from the DSE_ envvars dynamically in code? This could also allow dynamically switching between 'DSE mode' vs 'Apache mode' since it'd mostly be handled in code anyway.
I was able to do something similar for kafka's handling of KAFKA_OVERRIDE_* values, where it doesn't need any specific override settings in the default yaml, and instead just automatically grabs the override values from envvars when the config doesn't have any explicitly set: https://github.com/mesosphere/dcos-kafka-service/pull/242/files#diff-4042f7d0bef53bd920ae2006b10e595eR51

retest this please

retest this please

retest this please

retest this please

retest this please


[MsgProtocol]: Add Host,Scheme and PATH info in message.

In viewer part we use http headers Host to get the host info. I think we should add Host and Scheme in the message so that the viewer part don't need to use
request["headers"]["Host"]

Will also be fixed in #87

Closed via PR #103.


Use maxdepth=3 in the API index

This makes the API index page list every function in the spec. On the NumPy list, @mattip requested a page that lists every API function. This is an easy way to achieve this.

I like this idea. My first instinct was that it was very verbose and lost the overview, however on second thought the overview with top-level section is still in the left navbar:

And getting the overview of all objects with full signatures is nice. Let's go with this, we can iterate if needed.


ERROR the request does not contains a client id parameter

code

@lucahttp thanks for raising the issue. I will take this up and push the changes asap.

@lucahttp can you try the below url for userFlowUrl
https://<host>.b2clogin.com/<host>.onmicrosoft.com/oauth2/v2.0/authorize?p=B2C_1_signinup&client_id=<client_id>&nonce=defaultNonce&redirect_uri=<redirect_uri>&scope=openid&response_type=id_token&prompt=login
in our next release, we will concatenate using client_id and redirect url and you wouldn't have to do this anymore. Please expect the release by end of this week.


Intermittent E2E test error due to 'failed to reconcile Rollout's Metrics Service.'

Seeing this intermittent E2e test error in a couple of PRs. More specifically, the mechanism for detecting unexpected errors is flagging these lines.
We should either add them to the ignore list, or figure out why they are being output:
2024-07-22T16:28:53.6773824Z 2024-07-22T16:28:53Z	INFO	rollouts-controller	Creating Service argo-rollouts-metrics
2024-07-22T16:28:53.6919812Z 2024-07-22T16:28:53Z	ERROR	rollouts-controller	Error creating Service	{"Name": "argo-rollouts-metrics", "error": "services \"argo-rollouts-metrics\" already exists"}
2024-07-22T16:28:53.6923050Z github.com/argoproj-labs/argo-rollouts-manager/controllers.(*RolloutManagerReconciler).reconcileRolloutsMetricsService
2024-07-22T16:28:53.6928676Z 	/home/runner/work/argo-rollouts-manager/argo-rollouts-manager/controllers/resources.go:433
2024-07-22T16:28:53.6930839Z github.com/argoproj-labs/argo-rollouts-manager/controllers.(*RolloutManagerReconciler).reconcileRolloutsManager
2024-07-22T16:28:53.6933080Z 	/home/runner/work/argo-rollouts-manager/argo-rollouts-manager/controllers/reconcile.go:125
2024-07-22T16:28:53.6935049Z github.com/argoproj-labs/argo-rollouts-manager/controllers.(*RolloutManagerReconciler).Reconcile
2024-07-22T16:28:53.6937264Z 	/home/runner/work/argo-rollouts-manager/argo-rollouts-manager/controllers/argorollouts_controller.go:144
2024-07-22T16:28:53.6939243Z sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile
2024-07-22T16:28:53.6941285Z 	/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.16.3/pkg/internal/controller/controller.go:119
2024-07-22T16:28:53.6943251Z sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler
2024-07-22T16:28:53.6945440Z 	/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.16.3/pkg/internal/controller/controller.go:316
2024-07-22T16:28:53.6947403Z sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
2024-07-22T16:28:53.6949658Z 	/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.16.3/pkg/internal/controller/controller.go:266
2024-07-22T16:28:53.6951613Z sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
2024-07-22T16:28:53.6953678Z 	/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.16.3/pkg/internal/controller/controller.go:227
2024-07-22T16:28:53.6956379Z 2024-07-22T16:28:53Z	ERROR	rollouts-controller	failed to reconcile Rollout's Metrics Service.	{"error": "services \"argo-rollouts-metrics\" already exists"}
2024-07-22T16:28:53.6959451Z github.com/argoproj-labs/argo-rollouts-manager/controllers.(*RolloutManagerReconciler).reconcileRolloutsManager
2024-07-22T16:28:53.6961772Z 	/home/runner/work/argo-rollouts-manager/argo-rollouts-manager/controllers/reconcile.go:126
2024-07-22T16:28:53.6963903Z github.com/argoproj-labs/argo-rollouts-manager/controllers.(*RolloutManagerReconciler).Reconcile
2024-07-22T16:28:53.6966075Z 	/home/runner/work/argo-rollouts-manager/argo-rollouts-manager/controllers/argorollouts_controller.go:144
2024-07-22T16:28:53.6967925Z sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile
2024-07-22T16:28:53.6970117Z 	/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.16.3/pkg/internal/controller/controller.go:119
2024-07-22T16:28:53.6972120Z sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler
2024-07-22T16:28:53.6974229Z 	/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.16.3/pkg/internal/controller/controller.go:316
2024-07-22T16:28:53.6976438Z sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
2024-07-22T16:28:53.6978557Z 	/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.16.3/pkg/internal/controller/controller.go:266
2024-07-22T16:28:53.6980386Z sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
2024-07-22T16:28:53.6982422Z 	/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.16.3/pkg/internal/controller/controller.go:227

This appears to be resolved.


TS new 2.X version not working

Hi,
not sure what you change after version 2.X but I install the new version of dropzone and its start complaning that I should install @types/dropzone-ui, but it looks like that something is wrong.
Could not find a declaration file for module 'dropzone-ui'. '/xxx/node_modules/dropzone-ui/build/index.js' implicitly has an 'any' type.
  Try `npm i --save-dev @types/dropzone-ui` if it exists or add a new declaration (.d.ts) file containing `declare module 'dropzone-ui';`

When I open the npm_modules the structure not look right I add the screenshot of that. Could you help me with that ?

Hi @ssimk0 , thanks for the comment. There was a problem on the build process we didn't notice. This was produced due to oneof the dependencies that makes the build file and didn't make the type definitions.
Really sorry about that.
Could you please reinstall the package? There is a new version (2.1.1) that fixes this problem.

Hi,
@JinSSJ3 thanks new version is working :+1:


MAV_DO_CHANGE_SPEED not applied correctly

This:
https://github.com/PX4/Firmware/blob/b4755297ecf3d91b2aa0d6d08d78e35f6692177d/src/modules/fw_pos_control_l1/FixedwingPositionControl.cpp#L774-L788
makes so that a MAV_DO_CHANGE_SPEED command is only applied when changing waypoints.

How does that make it only apply when changing? The mission_throttle is passed to TECS each iteration here. https://github.com/PX4/Firmware/blob/b4755297ecf3d91b2aa0d6d08d78e35f6692177d/src/modules/fw_pos_control_l1/FixedwingPositionControl.cpp#L807

it sets it hard: mission_throttle = _parameters.throttle_cruise;
Then only checks if there was a possp throttle set
So updating mission_throttle doesn't do anything
https://github.com/PX4/Firmware/blob/3c18be387cc47c9875137745b171e7ccec901f2e/src/modules/navigator/navigator.h#L204
same for airspeed

Ps: that is called from here in response to a do_change_speed command: https://github.com/PX4/Firmware/blob/3c18be387cc47c9875137745b171e7ccec901f2e/src/modules/navigator/navigator_main.cpp#L506

Hey, this issue has been closed because the label status/STALE is set and there were no updates for 30 days. Feel free to reopen this issue if you deem it appropriate.
(This is an automated comment from GitMate.io.)

This is still an issue

I use firmware1.4.4, and I don't want to use the other version, how can I use do_change_speed command to change the mission speed?

@goodman110110110 Please do not append to old and somewhat unrelated issues.
You really should update to the newest version, but to answer your question, you need to check if the mission item is supported in the version you want to use. If not then you'll have to add support in the code yourself. This will effectively be a custom message. See https://dev.px4.io/en/middleware/mavlink.html

Thankyou @hamishwillee


Add instructions for compiling dynamically loaded module

Dynamic modules are more flexible. This should be documented in the readme :)

Thanks


cannot load underlying module for 'IKEventSource'

After cleaning and  reinstalling the pods, cleaning derived data, and rebuilding, I cannot get rid of this error:

Please update pod again


Upgrade to Scala Native 0.3.6

Very impressive speedups!

     RscNativeSchedule
-    82.200 ms
-    50.133 ms
+    52.571 ms
+    38.234 ms

     RscSchedule
-    335.085 ± 1.007 ms
-    10.471 ± 0.007 ms
+    339.965 ± 1.698 ms
+    10.619 ± 0.007 ms

     ScalacNamer211
@@ -91,13 +91,13 @@ To reproduce, run `bin/bench` (this will take a while).

     RscNativeTypecheck
-    173.264 ms
-    106.148 ms
+    99.707 ms
+    71.346 ms

     RscTypecheck
-    454.622 ± 2.483 ms
-    32.246 ± 0.013 ms
+    460.705 ± 2.718 ms
+    32.498 ± 0.017 ms

     ScalacTyper211
@@ -142,7 +142,7 @@ To reproduce, run `bin/bench` (this will take a while).
   * Scala Native has clearly succeeded in its goal of speeding
     up startup time of Scala applications. In cold benchmarks that are
     representative of running programs from command line, Rsc Native has
-    a ~1.6x edge over vanilla Rsc.
+    a ~4.6x edge over vanilla Rsc.


Full Channel Description, VideoCount and Subscribers

Hello! I used SearchYouTubeChannelAsync to get information about a channel, is there a way to obtain the full description of the channel instead of its short version? Also, I'm getting 0 for the number of subscribers and the number of videos on the channel.

@slagusev hey there! regarding your first issue, I stopped working on this project a long time ago, but I was planning to introduce a GetChannelMetadataAsync function when I stopped. if you're willing to wait a while I'll try to implement this asap to get the full description. regarding your second issue, since this search function is done through scraping the site, you'll have instances where data is presented like this

As you can see sometimes youtube sends channels without their subscriber count, so the program can't do much about it.

Thank you for the response! It would be great to see further development of the library.

ill leave this open so ill remember to work on this


Update the URL of Pipenv documentation page

Hi, Pipenv Documentation page has moved to here:
https://pipenv.readthedocs.io/en/latest/
P.S. Thanks for useful fish config file! :)

@ParthS007 Thank you for merging!


I cannot save my profile

Subject: I cannot save my profile
Description:
idk
Results:
I did fill my profile many times but I cannot save it
Backend version: develop-64e6cf65
Frontend version: develop-6d22ac7f
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Page: https://couchers.org/profile/edit/about
User: alborrico (31776)

Button at the bottom


feature request: double legend position in sc.pl.umap

[x] Additional function parameters / changed functionality / changed defaults?

...
Hi scanpy team
Thank you for the excellent package! I would like to request a new legend_loc parameter for the sc.pl.umap function.
If sc.pl.umap could plot the cluster names on the "right margin", but at the same time assign a number to each cluster and plot that number "on data", we should be able to distinguish the clusters better.. Currently the labels sometimes take up too much of space when I put the legend "on data". On the other hand the the colors get confusing for >15 clusters clusters if I put the legend on "right margin".
Best,
Vivek

You would like the legend on the right to have both the short and long names, while the "on data" annotation would only have the short names – right?
I'm not sure I can think of a great way to do this without making the process complicated. It may be easier for you to add the text labels to the plot yourself.
Here is some code for adding the labels with "adjustText" (which needs some parameter fiddling to look nice)
def gen_mpl_labels(
    adata, groupby, exclude=(), ax=None, adjust_kwargs=None, text_kwargs=None
):
    if adjust_kwargs is None:
        adjust_kwargs = {"text_from_points": False}
    if text_kwargs is None:
        text_kwargs = {}

    medians = {}

    for g, g_idx in adata.obs.groupby(groupby).indices.items():
        if g in exclude:
            continue
        medians[g] = np.median(adata.obsm["X_umap"][g_idx], axis=0)

    if ax is None:
        texts = [
            plt.text(x=x, y=y, s=k, **text_kwargs) for k, (x, y) in medians.items()
        ]
    else:
        texts = [ax.text(x=x, y=y, s=k, **text_kwargs) for k, (x, y) in medians.items()]

    adjust_text(texts, **adjust_kwargs)

which can be simplified if you're alright with just plotting on the medians.
I personally think interactivity and hover-over becomes quite useful at this point, though that can be difficult to scale and share.

It will be great if this function can be added.

@hyjforesight, the adjustText functionality? As discussed in the linked issue, without fiddling the adjust text labels often have very poor placement.
If it looked decent at least half the time, I'd be up for it, but this solution looks bad 90% of the time (until you fiddle with parameters).

I meant apart from the long label at "right margin", basically adding a number to each label and display that number "on data".
Right margin would look like this:

Cell type A
Cell type B
Cell type C

And the UMAP will simply show 1, 2, 3 ... on the plot
The issue is adding the text label on the plot make them overlap with each other, but the corresponding numbers (which would be automatically generated when one runs sc.pl.umap) would do just fine.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 879 ms

Resolved: Bofrost.at performance has improved in d3e7ad1 after 40 minutes.


initial version of the xamarin component

fixes issue FH-2472
how to test
Download the xamarin-component command line tool and build a xam file by:
> .\xamarin-component.exe package .\fh-dotnet-sdk\component

@rsmeral care to take look?

@edewit Sure, I'll review it soon. Is there any reason why this should be in fh-dotnet-sdk, and not in doxy? I'd put it in doxy with all the other documents.

@rsmeral cause this will be part of the xamarin component library publishing our sdk there as a component, for me it makes more sense to have it be part of the sdk, WDYT?


Search pill looks misaligned on webrowser

steps to reproduce:

On chat.zulip.org, go to search box on top of the page
search for 'group'
select this

the pill looks misaligned here

using chrome latest version

Zulip Server and web app version:

[ ] Zulip Cloud (*.zulipchat.com)
[ ] Zulip Server 8.0+
[ ] Zulip Server 7.0+
[ ] Zulip Server 6.0+
[ ] Zulip Server 5.0 or older
[ ] Other or not sure

can someone add UI label , thanks

@zulipbot claim

I am not able to recreate this in any browser.
For chrome version:

@codewithnick I can't replicate either... What font size settings are you using?

To be clear, this is not a group pill. It's just a user with "GROUP" in their name.

I started a CZO thread.

Fixed in #30966.


Revert daprsystem config validation in CI

Description
Revert daprsystem config validation in CI
Issue reference
#1408
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

/ok-to-test


chore: configured eslint, prettier and commitizen tools

#12
Layers affected by changes? (backend, frontend, async service)
backend
Changes description:
Configured eslint, prettier and commitizen tools
Review checklist
It should be used only by the reviewer.

[ ] Source code in English
[ ] Use of clean code principles
[ ] No code duplication

@eduardopetri please resolve the conflicts.


nix-env -i firefox provides invalid download link for SDK

Moving this bug to NixOS/nixpkgs from NixOS/nix https://github.com/NixOS/nix/issues/1628
I am NIX beginner.
My aim: update Firefox.
What I typed:
bash <(curl https://nixos.org/nix/install)
. /home/dhp/.nix-profile/etc/profile.d/nix.sh
nix-env -i firefox

Result:
Unfortunately, we cannot download file jdk-8u144-linux-x64.tar.gz automatically.
Please go to http://www.oracle.com/technetwork/java/javase/downloads/jdk8-downloads-2133151.html to download it yourself, and add it to the Nix store
using either
  nix-store --add-fixed sha256 jdk-8u144-linux-x64.tar.gz
or
  nix-prefetch-url --type sha256 file:///path/to/jdk-8u144-linux-x64.tar.gz

I hate this problem, but, I know Oracle is very boring about direct download, and nobody ever could workaround this legal issue (especially, I also have it in Gentoo). So, let's hope my old version of Firefox still functions, let's hope that on fresh install it's possible to either use an other browser to download the file, or have a light Firefox to download this dep. => needing a browser to install a browser can quickly become an egg-chicken issue.
The big issue I have met here is that, as of today, october 23rd 2017, the given link to oracle website only offers me SDK 151 and 152; what means, if I download them ... nix-store won't be able to put the file where it's expected, and the dep will remain unsatisfied.
Several solutions:

find a way to provide a link that would directly point to the good SDK version
tell if any recent version will do (otherwise, I will assume that non matching number will break the dep, and, I will assume that version 151 will NOT satisfy the need for jdk-8u144-linux-x64.tar.gz
can't find any other alternative for this issue :/

Technical details
$ lsb_release -a
LSB Version:    n/a
Distributor ID: Gentoo
Description:    Gentoo Base System release 2.2
Release:        2.2
Codename:       n/a

$ nix-env --version
nix-env (Nix) 1.11.15

Nixpkgs version: (run nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion)

error: getting status of â/home/dhp/firefoxâ: No such file or directory

Sandboxing enabled: (run grep build-use-sandbox /etc/nix/nix.conf)

returned error 2

Hmm... I'm pretty sure firefox does not depend on Oracle's JDK. In fact, it doesn't depend on any JDK. Something else is going on.
That message is produced by the Oracle JDK package. You can see it here in the call to requireFile: https://github.com/NixOS/nixpkgs/blob/7ed75dc283d0892505ddf40f59b11649c8581870/pkgs/development/compilers/oraclejdk/jdk-linux-base.nix#L91

Of course, you are talking about this:
$ cat ~/.nixpkgs/config.nix
{
    allowUnfree = true;
    firefox = {
         enableGoogleTalkPlugin = true;
        enableAdobeFlash = true;
        jre = true;
#       icedtea = true;
    };

    chromium = {
         enablePepperFlash = true; # Chromium's non-NSAPI alternative to Adobe Flash
         enablePepperPDF = true;
   };
}

which explains why there is no egg-chicken problem, but, still does not solve the main issue.
I have also ended up in searching manually the required file, and Google gave me http://www.oracle.com/technetwork/java/javase/downloads/java-archive-javase8-2177648.html?printOnly=1 ... which still does not really solve the bug: nix-env provides an invalid link. But this archive link could help you fixing it, and develop a long term workaround.

The fix for the URL was merged 7 days ago: https://github.com/NixOS/nixpkgs/pull/30536
It's in the Nixpkgs unstable channel right now, so you should have access to it: https://github.com/NixOS/nixpkgs-channels/commit/f11948d31e11b02ee68391f76caafb7630b4d3af


Move time calculation to MDW

Context information
Some adapters are using block height to calculate time, But some needs to be calculated because there is no time entry in response
Steps to reproduce
see usage formatBlockDiffAsDatetime in adapters.js
What is expected?
Move other calculations to backend, remove formatBlockDiffAsDatetime() then
This Issue is followup of https://github.com/aeternity/aescan/issues/100
MDW Tracking https://github.com/aeternity/ae_mdw/issues/831

It's solved on MDW side now, ready to implement on our side

Followup Tracking
https://github.com/aeternity/ae_mdw/issues/1425

The followup is ready to be implemented.

It's only partly fixed  here https://github.com/aeternity/aescan/pull/489 as some values from MDW are missing. So this issue should not be closed and needs further adjustment
https://github.com/aeternity/ae_mdw/issues/1425#issuecomment-1701006129
https://github.com/aeternity/ae_mdw/issues/1425#issuecomment-1700994875

last 2 values are fixed on mdw side, waiting for deployment

It still has not been developed. Pinged again

@janmichek to check if it's fixed or something is still missing

Still not fully resolved.
Status: https://github.com/aeternity/ae_mdw/issues/1425#issuecomment-1880384717


Resolve Ubuntu GPG key errors

The following errors in the VM startup script cause the deployment to fail.
Repository 'http://security.debian.org/debian-security buster/updates InRelease' changed its 'Suite' value from 'stable' to 'oldstable'
Repository 'http://deb.debian.org/debian buster InRelease' changed its 'Version' value from '10.4' to '10.10'
Repository 'http://deb.debian.org/debian buster InRelease' changed its 'Suite' value from 'stable' to 'oldstable'
Repository 'http://deb.debian.org/debian buster-updates InRelease' changed its 'Suite' value from 'stable-updates' to 'oldstable-updates'

The reason is because the current Debian base image is Debian 7 or 8, and the most current release is Debian 10.
Once I upgraded to Debian 10 and added one additional command to update the GPG keys, the error stopped and the deployment was successful.
Increment the Marketplace release to 0.7.3.

https://github.com/GoogleCloudPlatform/datashare-toolkit/pull/553


Private stores in a computed callback

Private Stores: defensively support re-entry when a store is defined within a computed callback
Use cases for private stores include:

A computed callback calling a helper function that instantiates a
Cancelling a run

size-limit:

Atom: + 3 B
All: + 44 B

Ported from https://github.com/nanostores/nanostores/pull/203

const $path = atom('')
const $userCancel = atom(false)

const $download = computed(task => {
  if (task() && task().path !== $path()) {
    task().stream.cancel('path changed')
    task.save(null)
  }
  if (!$path()) return null

  const stream = task()?.stream || createStream()
  computed(() => {
    if ($userCancel() && stream) {
      stream.cancel('user cancelled')
      $userCancel.set(false)
    }
  })()

  return { path: $path(), stream }

  function createStream() {
    return new ReadableStream({
      async start(controller) {
        this.responsePromise = fetch(`https://downloads.com/${$path()}`)
        const response = await this.responsePromise
        await response.body.pipeTo(new WritableStream({
          write(content) {
            controller.enqueue(content)
          }
        }))
        controller.close()
      },
      async cancel(reason) {
        console.warn('Stream cancelled:', reason)
        const response = await this.responsePromise
        await response.body.cancel(reason)
      }
    })
  })
})

Too complex API for the simple nature of Nano Stores.
As I mention before, it is a great feature for separated state manager with this design from the beggining.


Request for more detailed instructions

when using the Azure portal, the instructions state "navigate to aks cluster" , when I search for AKS cluster, no results are found. Please provide more detailed instructions about how to navigate to an AKS cluster.
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: a14a3f84-28b4-0a2a-4da7-47cb4d66689f
Version Independent ID: d1ffdd88-ab9a-9f55-8689-526e7415b326
Content: Kubernetes on Azure tutorial - Upgrade a cluster - Azure Kubernetes Service
Content Source: articles/aks/tutorial-kubernetes-upgrade-cluster.md
Service: azure-kubernetes-service
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@DeanGross - If you are not familiar working in the Azure portal, there are a few different ways to select any one of your Azure resources. Check out the following portal overview article that can help you get comfortable with it. You can search by resource (in this case, your cluster name), or navigate to Kubernetes services to see all AKS cluster deployed in your selected subscription, or navigate to the resource group it's deployed in and select it from there.
As a common practice in our documentation, we don't explicitly state how to navigate to the resource in the portal since there are multiple methods.
We will consider your feedback further and evaluate if we should be more descriptive with that step.
Thanks for your submission. #please-close


ConditionalApplyPlugin: Allow every apply to be optionally skipped

As a developer, I want to be able to arbitrarily "deploy" to any environment that I want, so that I can fix problems in a emergency, and not be "slowed down" by the pipeline.

In the master branch, there are 2 options - confirm that you want to apply, or abort
What if we gave customers a third option - "skip".
If the user chooses skip, then "apply" is not run for the current environment, and instead simply continues through the pipeline as though "apply" had run

Example code:
Jenkinsfile.init(this)
ConditionalApplyPlugin.enableSkipApply()

def validate = new TerraformValidateStage()
def deployQa = new TerraformEnvironmentStage('qa')
def deployUat = new TerraformEnvironmentStage('uat')
def deployProd = new TerraformEnvironmentStage('prod')

// When prompted to confirm "apply qa", user now has a 3rd option to simply skip over, and continue to uat
validate.then(deployQa)
// When prompted to confirm "apply uat", user now has a 3rd option to simply skip over, and continue to prod
        .then(deployUat)
// When prompted to confirm "apply prod", user now has a 3rd option to simply skip over and finish the pipeline
        .then(deployProd)
        .build()


Clarify Helm support under /manifests

manifests/UPDATING-CHARTS.md states that Helm installation is being deprecated, but we have indicated elsewhere that it is not (see https://discuss.istio.io/t/istios-helm-support-in-2020/5535).  Can we clarify our communications around helm and its future?

Any updates on this?


Documentation - how is it different from Tensorboard

It's not clear to me, how is this different from TensorBoard? TensorBoard is usable with PyTorch. So, why should one switch to TensorWatch?

From what I understand, the only advantage is the ease of use of TensorWatch with notebooks.
Since I don't really work with notebooks, I was wondering if there is any other advantage


Sample values in Chart activity are not initialized

At first launch, the Chart activity use sports name to initialize values.
When launch from the app in a default English language, values are not initialized (EgLabelX).

Full step to reproduce:
Capture vidéo du 2024-12-30 18-02-31.webm

The issue is begin caused because the the dictionary is not fully loaded in SugarL10.js when the acitivity starts. One thing we can do is add a loader. If it is fine to add a loader @llaske I will raise a pr for this or should i look for other alternatives ?

@AdityaKrSingh26 have you tried this loader implementation? If yes, can you just test does it work in firefox? I tried a delay but it isn't working with firefox. If you got the same issue with firefox ig this implementation wouldn't work. I have an alternative fix.


PLY Header Invalid when importing to UE5.4 via XV3DGS

Hi there!
I'm trying to load a Gaussian Splat .ply that was created in Postshot then cropped in Supersplat to UE5 via the XV3DGS plugin. The uncropped .ply loads fine, but after saving from supersplat (1.7.2) I get an invalid headers error. (see screenshot)

In previous versions of supersplat (0.19.2) I was able to save from supersplat without this issue occuring.
Many thanks for your help!
Nathan

Hi, are you able to post an example of the original and modified version?

here's a folder with 3 files - the original, one modified in 0.19.2 (works in UE5), and one modified in 1.7.2 (has the invalid headers issue).
https://drive.google.com/drive/folders/1DhZwTLSPPT797ZHs-VwtRVnaa1pD5OJj?usp=sharing

Actually I found the issue. Fix will be deployed sooon.

perfect thank you so much! really appreciate it!

Fixed in #231 and deployed to prod.


Add CCS parameters to headers or form_data

This PR adds the CCS (Cache Credential Service) parameters to the headers or form_data when the login_hint or account object are available.
It also introduces a configuration parameter to prevent adding headers when CORS preflight requests can be prevented.

Coverage increased (+12.2%) to 96.024% when pulling 11f7ea9689d4df030dd405414276ed80adb4209d on ccs-feature into b42e87fa3746923e90f4997277dc8d764b2e6a8d on dev.

Codecov Report

Merging #3636 (08ce96e) into dev (21123e6) will not change coverage.
The diff coverage is 92.85%.

Flag
Coverage Δ

*Carryforward flag

msal-angular
95.81% <ø> (ø)

Carriedforward from 104e809

msal-browser
84.55% <94.11%> (ø)

Carriedforward from 104e809

msal-common
83.32% <92.00%> (ø)

Carriedforward from 104e809

msal-core
73.02% <ø> (ø)

Carriedforward from 104e809

msal-node
78.48% <ø> (ø)

msal-react
93.77% <ø> (ø)

Carriedforward from 104e809

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files
Coverage Δ

lib/msal-browser/src/app/ClientApplication.ts
90.60% <ø> (ø)

lib/msal-common/src/account/ClientInfo.ts
100.00% <ø> (ø)

lib/msal-common/src/config/ClientConfiguration.ts
100.00% <ø> (ø)

lib/msal-common/src/client/BaseClient.ts
75.86% <66.66%> (ø)

lib/msal-browser/src/cache/BrowserCacheManager.ts
93.56% <87.50%> (ø)

.../msal-common/src/client/AuthorizationCodeClient.ts
85.31% <88.88%> (ø)

...wser/src/interaction_handler/InteractionHandler.ts
86.20% <100.00%> (ø)

...browser/src/interaction_handler/RedirectHandler.ts
92.59% <100.00%> (ø)

lib/msal-browser/src/utils/BrowserConstants.ts
100.00% <100.00%> (ø)

...b/msal-common/src/client/ClientCredentialClient.ts
82.53% <100.00%> (ø)

... and 17 more


Vite doesn't inline SVG assets

https://github.com/vitejs/vite/blob/480367b83a5d418e76a4a6bccd004abb97413c22/src/node/build/buildPluginAsset.ts#L60-L63
Hi. I noticed that svg files is explicitly excluded in the assetsInlineLimit feature, and I'm wondering why that is so. Renaming the extension to .SVG  (thus bypassing the .endsWith check) works fine in my project.
This limit exists since the first time the feature was added, and I'm unable to find any kind of comments or documentation about it. Could someone shed some light on this please? Thanks!

Look like !id.endsWith(.svg)  should be removed.. @antfu @aleclarson what do you think?

We don't want to use base64 with svg files. See here: https://css-tricks.com/probably-dont-base64-svg/
Basically, base64 is larger in size, and svg files don't need it to be inclined. In fact, svg files even compress better without base64, because there's more repetition.
If svg files aren't inclined currently, they should be. Just without base64.

@aleclarson Thanks for yor clarify.

Are there plans to support inlining svg without base64?

Are there plans to support inlining svg without base64?

Can you open a new issus to discuss it.

Basically, base64 is larger in size, and svg files don't need it to be inlined. In fact, svg files even compress better without base64, because there's more repetition.

"compress better without base64", everything does.... base64 is an encoding useful for content like binary which can't be represented in ASCII/UTF-8 text content, but has a drawback of 6 bits for every 8-bits, roughly 1.33x increase in weight.
Inlining content via a base64 data URI still has it's benefits despite this.

What you're referring to is inline SVG element, which isn't always appropriate (eg favicon).
In some cases you need more accurate / crisp rendering and need the SVG content to be the src of an <img> element instead. As img is a replaced element, it has some drawbacks vs inlining SVG into the DOM, however if you're scaling the content, it will be properly buffered into pixel width/height even if it'd otherwise be a floating point value. I've had this with logos that involve some vectorized text which suffer aliasing issues at certain scales on low DPI displays as there is no shape hinting.
You can workaround that by providing an integer width/height, but you must know that in advance and it comes with some other disadvantages if you want responsive content. Additionally the img element allows providing the alt attribute which has value beyond screen readers (more so if the file isn't inlined/embedded but remains an external file reference).


[Feature Request] An option to disable change directory for all recipes in justfile

Hello. just is a great tool and I really like it. However this automatic change directory behavior and using [no-cd] in all my recipes is a bit annoying (but still acceptable).
Would you consider adding some setting to disable this behaviour? Something like set no-cd and new attribute [do-cd] or something like this?
My use case.
All projects I'm working on are in ~/Projects.
~/Projects/my-project-1
~/Projects/my-project-2
...

I want to reuse one justfile located in ~/Projects/justfile. All those projects contain terraform configuration that I'm working on. When I want to change infrastructure of some project I will:

Go into ~/Projects/my-project-2
Make changes in infra config
Run validate, fmt, plan
If all is okay then apply infrastructure

There are some other common actions I do on all projects also but this is not important.
To work with terraform I need to be inside the directory that contains configuration. To make it work with just each recipe needs to have this [no-cd] added.
My working justfile looks like similar to this:
set quiet := true

[private]
[positional-arguments]
run-tf *args='':
    # this is actually more complicated command using docker with volumes and whatnot
    # but for this example I simplified it
    terraform "$@"

[no-cd]
init:
    just run-tf init

[no-cd]
upgrade:
    just run-tf init -upgrade

[no-cd]
validate:
    just run-tf validate

[no-cd]
plan:
    just run-tf plan

[no-cd]
apply:
    just run-tf apply

[no-cd]
destroy:
    just run-tf destroy

[no-cd]
cli:
    just run-tf bash

[no-cd]
format:
    just run-tf fmt -recursive .

[no-cd]
output *args='':
    just run-tf output "$@"

alias console := cli

All is working fine but would be great if I could just use something like set no-cd and remove all those [no-cd] lines. If I wanted to change dir to jsutfile dir I could then add [do-cd] in recipe and it would work the standard way.
Would you consider adding setting like this?
Thanks

I think set no-cd would be reasonable. The working directory logic has been recently refactored into a single function, ExecutionContext::working_directory, which has access to settings, so it should actually be a pretty simple PR.

I have started tacking this. I think it can be combined with #2082, to avoid multiplying attributes beyond necessity:

set no-cd makes that by default working directory isn't modified
[cd(DIR)] sets alternative working directory for a recipe with or without global no-cd setting
[cd] undoes global set no-cd for a marked recipe (instead of the suggested [do-cd]
setting [cd] and [no-cd] on the same recipe should cause an error on startup saying they are incompatible.

@artm That all seems reasonable to me!


Add Encoding.calcCharCountForTokens method

A very common use case for token counting is when chunking a long text to fit in a model context window. In order to efficiently use jtokkit library for this purpose, we need to be able to count number of characters for given token count. I added Encoding.calcCharCountForTokens method that does that.
Please, review and accept the pull request if it makes sense.
I updated the implementation for the latest code.

problematic: not accounting for "multi-token-characters"
maybe alternative solution: re-decode the encoded tokens to a String and use .length() of this.


Formatting corrupted when Unicode East Asian Ambiguous characters are wide

Category

[x] fzf binary
[ ] fzf-tmux script
[ ] Key bindings
[ ] Completion
[ ] Vim
[ ] Neovim
[ ] Etc.

OS

[ ] Linux
[x] Mac OS X
[ ] Windows
[ ] Windows Subsystem for Linux
[ ] Etc.

Shell

[s] bash
[ ] zsh
[ ] fish

Under macOS built-in Apple Terminal, if you have the setting "Unicode East Asian Ambiguous characters are wide" checked, and are using the --preview option, the formatting is corrupted. You get something like this:

Of course, the workaround is to uncheck the option, but that option is needed in many situations. East Asian users need it for their text. See Unicode® Standard Annex #11: East Asian Width.
In particular, if the command prompt or command line has an ambiguous width character and that option is not set, then searching and editing command lines can fail because the cursor is not properly placed.
See https://github.com/bpython/bpython/issues/254 for how a similar issue was resolved in another project. This utility might be a help: https://github.com/mattn/go-runewidth/blob/master/runewidth_posix.go

Since fzf has no way to know if the option is enabled or not, the only option is to add a new option to draw borders in ASCII characters instead of Unicode box drawing characters as in https://github.com/bpython/bpython/commit/566dfe5a5fab815fdd8d04d6d3d131333c2a5800. Also related: https://github.com/workshopper/workshopper/issues/73
By the way, I'm also an East Asian user, but I use iTerm2 and never had the issue. Have you tried using other terminal emulators?

Since fzf has no way to know if the option is enabled or not, the only thing we can do is to add a new option to draw borders in ASCII characters instead of Unicode box drawing characters as in bpython/bpython@566dfe5.

Yes, but I think that is not an unreasonable option to add, preferably via both a command line option and a single-function environment variable. (The problem with FZF_DEFAULT_OPTS is that it is difficult to add or change options to it, for example changing the color based on day or night, because doing that the easy way wipes out any other default options you have set.)

By the way, I'm also an East Asian user, but I use iTerm2 and never had the issue. Have you tried using other terminal emulators?

Yes. "Ambiguous width characters" is a very well known issue and has more to do with the version of Unicode support the OS has installed rather than anything specific to individual terminal programs.


find definition seems like it could be more specific

If I have (possibly different) struct baz in foo.hxx and bar.hxx, included and used by foo.cxx and bar.cxx respectively, I'd hope issuing a find-definition (via xref-find-definitions) would find the correct one. (Similarly xref-find-references on one of the definitions ought to just find the actual uses.)
However, I find in both cases I get a list of both definitions (or both references). (This isn't a case where  both are being linked together: I put them in different libraries; they're just in the same CMake project.)
(Obviously it would be odd to start here. The actual problem occurs because in the project there was a fork of some libraries, used by different programs.)
(I note that a not-dissimilar project rtags handles this correctly, identifying (IIUC) symbols relative to the header in which they're declared. (That's not perfect, obviously; you can have the same symbol declared once in a header and defined differently in different files. It seems good enough, however.))

This is likely the heuristic-based find definition; is the file correctly indexed? ie, do outline/references/code completion work?

The USR of a struct/class is global (e.g. c:@S@A) and it does not include filenames.
I recently added multiple definition support (by turning optional<Def> def; to std::forward_list<Def> def;. As a result, you may see multiple definition as xref-find-definitions targets and removing one of the definitions will not make the entity undefined.
The rtags behavior seems like a heuristic that I am not willing to add.
In addition, compile_commands.json does not include linking commands. If multiple linked executable share functions with the same name, there is no way to differentiate which function should be the definition for a textDocument/definition request.

Ah, I didn't read closely enough. If there are indeed multiple definitions and you want to jump to the correct one, I agree, this is a wontfix - cquery is doing the correct behavior by showing all instances.

OK, it's a reasonable design choice.
I don't agree with the rtags behaviour being "heuristic", though: it's finding the actual declaration of the symbol being pointed at rather than all of them which happen to have the same name.

There are many intricacy that make indexing not align (well) with actual compilation/linking behavior.
e.g.
"top-level cv-qualifiers on the parameters are discarded in function declaration" in human readable manner, the follow foo is really the same thing. But they have different USR and you may not be able to find the definition given the declaration.
// a.h USR c:@F@foo#1I#
int foo(const int a);

// a.cc USR c:@F@foo#I#
int foo(int a) {}

There can be version scripts in linking stage to change function names.
Symbols can be marked weak so that they may be overriden by definitions in other shared objects.
The rtags behavior is "heuristic" but it might be appealing. Maybe we can have an initialization option in config.h to make textDocument/definition do such heuristics.

I was too quick in claiming that it's not a mere heuristic. For sure it is, sometimes.
It does seem like for the specific example I gave it's reasonable to argue that it's not a heuristic at all. That is, for finding a struct definition where the definition is in the same translation unit; offering me a list of that one together with one with the same name which isn't in the same TU seems like something people wouldn't want by default.
For declarations (of types, functions, etc.) things can certainly be more fiddly. Heuristics such as if the declaration is in a header which is also included in a single definition then that's most likely what's wanted feels like it might be good enough.
With DXR and rtags I've sometimes wanted something a bit more general, since it's not uncommon for include files to get copied around. It would be nice to be able to give some kind of rule that says when two headers are really the same. (Not really a problem with cquery at present, obviously.)


What is the plan for 2.13.0?

It looks like there have been a number of fantastic changes since 2.12.0 was released (about 14 months ago).  It would be w if a new version was released.  Are there any blockers for that?

Bump.  Can we get a new version?

Would also help us to release a new version, particularly with the relaxed rake version dependency

This is becoming a nightmare for us at the moment. If we could get a release simply to fix the rake issue, that would be great.

You can do gem 'sneakers', github: 'jondot/sneakers' until a new release

@thisismydesign that works, but it doesn't have any semver conventions, so if you update your bundle and a breaking change is merged you maybe caught off guard.

so if you update your bundle and a breaking change is merged you maybe caught off guard

Can be pinned to a ref:
gem 'sneakers', github: 'jondot/sneakers', ref: '31d0cb25'

Hm, is there really not a way to create a new release? the last one is from 2019? Do you need more maintainers for this project?

ping @gabrieljoelc

@h0jeZvgoxFepBQ2C the contributors (including myself) have been dormant for a while unfortunately. We likely could use more contributors who are still actively using Ruby. However, I don't own the project so not sure I can make the final call on a contributor.
I will try to refresh myself on the release this weekend

Ok this sounds great, thank you so much!

I would produce a release but not sure if I have the permissions both on GH and RubyGems

@michaelklishin not sure I do either. I couldn't remember if it was in the CI or not but I guess it's not. :(

My best suggestion would be to push this repo to a new one under the ruby-amqp GitHub org but it won't help us with RubyGems access for releases.

@jondot can you please add me as a maintainer on RubyGems and perhaps transfer the repo to github.com/ruby-amqp? 🙏

Any update on this? I completely understand that this requires assigning maintainer rights, and that without it there is no way to proceed, therefore I am sorry to bump this thread up again. Just doing it because by now the Rake version dependency is making Sneakers incompatible with many other popular gems (like http, for example).

Any news on this? This projects definitely needs new release. I'm happy to help if welcomed.

Any news? I see @jondot and @gabrieljoelc as admins on rubygems. Maybe add @michaelklishin on there?

Any news? I see @jondot and @gabrieljoelc as admins on rubygems. Maybe add @michaelklishin on there?

I did my best to move forward with Sneakers and I think we should consider it abandoned with no way to continue for now. We can try to plan fork and rename.

I agree, the original author has moved on.
I can set up a repo under ruby-amqp for a new project based on the current state of Sneakers. I don't have a better name than "kicks", though.

Here's a new blank repo https://github.com/ruby-amqp/kicks (since the name is a decent pun on the original one and it is not taken on RubyGems, @simi was invited to it.
We can do the following to proceed, since the MIT license permits us to do so:

Push this repo as is there
Update the README to credit @jondot for the original Sneakers project
Ask authors of open PRs to re-submit them there
Bump the major version to indicate the shift to the new repo with a new set of maintainers.
Release the current state of the project without renaming anything as kicks X.0
If we choose to — and I don't think we have to or should — we can rename the modules and so on to Kicks

I do not volunteer to be a maintainer but since I work on RabbitMQ and (as time permits) Bunny, you can be sure that I at least will be around to grant permissions to those willing to maintain Kicks.
I will also be around to not allow for any decisions that Team RabbitMQ would find questionable in such a library/tool. Or at least serve as a "litmus test" for ideas and changes.
If you want to be on the Kicks maintainers team (for real, maintaining OSS software can be nerve wrecking and time consuming), just start a Discussion in that repo.
HTH.

@michaelklishin great, I'll reopen some crucial PRs in there and let's aim to release new version soon. See you in there!

@simi awesome! I'm working on some README updates as outlined above.

Discussions are now enabled in ruby-amqp/kicks.


Can't launch SonarQube with plugins

Describe the bug
Hello,
When I try to integrate plugins installation within the configuration files, the Sonarqube can't more be launched with the helmfile apply command (it infinitely reboots and displays a 503 error). I've got the bug with standard and LTS versions.
Versions used:

Chart Version: 3.0.0 and 1.0.26
SonarQube Version: those included on the charts, i haven't touched it
Helm Version: 3.8.2
Kubernetes Version: 1.21

Custom Values
Please add a minimal value example to reproduce this error
#Help page : https://github.com/SonarSource/helm-chart-sonarqube/tree/master/charts/sonarqube
ingress:
  enabled: false
  annotations:
    nginx.ingress.kubernetes.io/proxy-body-size: "8m"
  hosts:
    - name: url
      path: "/"
  tls:
  - hosts:
    - url
    secretName: url
plugins:
  install:
    - "https://github.com/elegoff/sonar-rust/releases/download/v0.0.9/community-rust-plugin-0.0.9.jar"
    - "https://github.com/mc1arke/sonarqube-community-branch-plugin/releases/download/1.11.0/sonarqube-community-branch-plugin-1.11.0.jar"
sonarProperties:
  sonar.web.javaAdditionalOpts: "-javaagent:./extensions/plugins/sonarqube-community-branch-plugin-1.11.0.jar=web"
  sonar.ce.javaAdditionalOpts: "-javaagent:./extensions/plugins/sonarqube-community-branch-plugin-1.11.0.jar=ce"
account:
  adminPassword: secretpassword
  currentAdminPassword: secretpassword

Expected behavior
A clear and concise description of what you expected to happen.
I'd like to use the two plugins I added on my values.yaml file.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Do you have any logs to share?
Otherwise try to use jvmOpts and jvmCeOpts instead of sonarProperties:sonar.web.javaAdditionalOpt and sonarProperties:sonar.ce.javaAdditionalOpts


Unable to import Electron modules

I'm unable to import Electron modules, such as dialog, in the example project.
I installed and saved the module:
npm install electron --save

Required Electron in Home.js
// app/components/Home.js
import React, { Component } from 'react';
import { Link } from 'react-router';
import styles from './Home.module.css';
import { dialog } from 'electron';

But it fails to compile. Here's the error output:
ERROR in ./~/electron/index.js
Module not found: Error: Cannot resolve 'file' or 'directory' ./lib-cov/electron in /Users/jcummins/workspace/js/electron-react-boilerplate/node_modules/electron
 @ ./~/electron/index.js 2:4-33

ERROR in ./~/drip/index.js
Module not found: Error: Cannot resolve 'file' or 'directory' ./lib-cov/drip in /Users/jcummins/workspace/js/electron-react-boilerplate/node_modules/drip
 @ ./~/drip/index.js 2:4-29

ERROR in ./~/tea-concat/index.js
Module not found: Error: Cannot resolve 'file' or 'directory' ./lib-cov/concat in /Users/jcummins/workspace/js/electron-react-boilerplate/node_modules/tea-concat
 @ ./~/tea-concat/index.js 2:4-31

I have tried various other ways of importing the module, such as const {dialog} = require('electron'), but I get the same error.

I will update repo next week to be compatible with electron 0.35. For now, you can upgrade webpack-target-electron-renderer to 0.2 to use electron module.

Thanks a bunch, @chentsulin. That fixed the problem.


Update deprecated uuid

nyc uses a deprecated version of uuid which triggers npm warnings:

npm WARN deprecated<EMAIL_ADDRESS>Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.

Could you please update to v8?
Older related issues from March 2020:

https://github.com/istanbuljs/nyc/issues/1299
https://github.com/istanbuljs/istanbul-lib-processinfo/issues/16

Are we ok to deprecate support for node 10 to make this pass?

It seems this has resolved itself with the release of<EMAIL_ADDRESS>last year. Installing<EMAIL_ADDRESS>or later today in a fresh project results in (only<EMAIL_ADDRESS>being installed. Note that<EMAIL_ADDRESS>still has a direct dependency on<EMAIL_ADDRESS>and I did not test anything pre-v15.


AttributeRouting - Route Template - Changed Behavior?

In our Api, we have an Attribute route with a template that looks like this:
[Route("{something}/foo")]
public IAction GetStuff(int something) { ... }

This previously worked fine but now the Route is no longer matched.
Has the default Routing behavior changed so that named tokens are no longer matched automagically?

I thing I worked it out.  It appears that the behavior is now stricter in that you have to specify the type of the token:
e.g.: THIS WORKS:
[Route("{something:int}/foo")]
public IAction GetStuff(int something) { ... }

Can you provide a more complete code sample? We didn't make any intentional behavior changes here.

I don't have time to set up a canonical repro, but what we have is essentially this:
[Route("api/myentity")]
public class MyEntityController : MyBaseController<MyEntity>
{
    [Route("{something}/foo")]
    public IAction GetStuff(int something) { ... }
}

This worked fine previously by now requires the type descriptor to be applied to '{something}'.
It's odd because Swagger finds it and accurately displays the GET route, but when we invoke calls against it, the default routing provider is unable to find the route so it falls through to our default app.Run catch all handler.

Further info...
It appears to be only an issue where the token is not at the end of the route template.
E.g.
//OK
[Route("foo/{something}")]

//NOT OK
[Route("{something}/foo")]

Thanks, I'll take a look at this

So I tried this out on newest bits and I am able to hit this action:
http://localhost:23687/api/myentity/5/foo
    [Route("api/myentity")]
    public class MyEntityController : Controller
    {
        [Route("{something}/foo")]
        public IActionResult GetStuff(int something)
        {
            return Content("Hello");
        }
    }

Can you provide more information?
Is there a github repo I can look at? What version of MVC?

Closing because we cannot reproduce this. If you see this on a new build of ASP.NET 5 / MVC 6 please re-open or log a new issue.


Futures transactions do not appear on overview (testnet)

Received futures transactions do not appear in QT on Overview > Recent transactions.

fixed in develop


Add contributors list to Readme

Description:
As this repo is solely built by open source contributors, we need to thank them by highlighting them in Repo's Readme
Embed contributors list to readme with help of https://contrib.rocks/preview?repo=jsvigneshkanna%2Ftailwind_ui_components
PR Requirement:

Please check the above requirements are met
Always create new branch on forked repo from master to avoid and error while merging the fix
Please Star this repo while forking as that would increase this repo's visibility

Cheers 🎉

hi @jsvigneshkanna please assign this to me

Hi @najeebkp ,
you can work on this issue, assigning your name
Please don's forget to star this repo to make it more cooler
Cheers 🍻


Build failed on yosemite

I ran the following command line on Yosemite:
Zeppelin-With-R $mvn package install -DskipTests -Pspark-1.5
[INFO] --- exec-maven-plugin:1.2.1:exec (default) @ zeppelin-zrinterpreter ---
ERROR: dependency ‘evaluate’ is not available for package ‘rzeppelin’

removing ‘/shared/Zeppelin-With-R/r/lib/rzeppelin’
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Zeppelin ........................................... SUCCESS [ 19.627 s]
[INFO] Zeppelin: Interpreter .............................. SUCCESS [ 16.560 s]
[INFO] Zeppelin: Zengine .................................. SUCCESS [  8.415 s]
[INFO] Zeppelin: Spark dependencies ....................... SUCCESS [01:17 min]
[INFO] Zeppelin: Spark .................................... SUCCESS [ 30.803 s]
[INFO] Zeppelin: R Interpreter ............................ FAILURE [ 17.001 s]

You need to install the R package "evaluate".  You can do that from R with install.packages("evaluate") from R.  "evaluate" is listed as a mandatory dependency in the README.  I suggest taking a look at it - there are some other things you'll want to install to get real use out of this.


Fixed #4772 create unique connection for each notebook by using noteb…

Fixed #4772 create unique connection for each notebook by using notebook path as uri
Disconnect sqlConnection from SqlKernel to ensure no connection left after the notebook is closed.

What happens when the path changes (i.e. we go from untitled to saved notebook)? Does the connection still work?

It will be a new doc from untitled to saved. So it created a new connection when run

Not sure why it was closed. Should be open


typo: convertion -> conversion

:x: Validation status: errors
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Split public and browser WS server

Clear and concise description of the problem
Currently, browser and public-facing API share the same web socket reporter. We should limit public usage and move browser methods into their own reporter so it's easier to manage and not break public API.
Suggested solution
Move browser methods from packages/vitest/src/api/setup.ts into their own reporter (or better just as create a separate rpc).
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

@userquin I would recommend doing this. We have a workaround with browserPromise, but in fact, we can just use a regular RPC. Here is the line that setups a WebSocket:
https://github.com/vitest-dev/vitest/blob/db22c6772519f62a6435a9cac21017d4b56441aa/packages/vitest/src/integrations/browser/server.ts#L63
If we create a separate RPC, we can share it with the browser provider and reuse it in the pool to wait for a response.


Add 'Remove "-image" to background' option or remove '-image' suffix to background property to loadingIndicator

What problem does this feature solve?
I would like to use gradient and images as a background of loadingIndicator.
However, the background property of the loadingIndicator template is currently attached with -image suffix and it can not be done.
https://github.com/nuxt/nuxt.js/blob/dev/lib/app/views/loading/circle.html#L3
What does the proposed changes look like?
I'd like to Implement function and submit a Pull Request.
The function content is to delete the -image suffix, or add the option of {image: true} to lodindIndicator.
Is this reasonable?

This feature request is available on Nuxt community (#c7512)

@potato4d Do it :relaxed:

@manniL
If this is reasonable, I will do it myself!

👍

@Atinux
Thank you for reply.
The -image I refer to is the style of loadingIndicator templates.
e.g.) https://github.com/nuxt/nuxt.js/blob/dev/lib/app/views/loading/chasing-dots.html#L3

Oh I see, so I suggest removing -color and only keeping background: <%= options.background %>;.
This way, you will be able to define background: image('...') :)

@Atinux
It's breaking changes, Can I do it?

It's not a breaking change, you can do background: blue;, see https://www.w3schools.com/CSSref/css3_pr_background.asp

So of course you can make a PR :)

@Atinux
OK, then I will send it.
Thanks!


Exception / Segfault then inserting a column with all NULL value

Trying to insert a table which contained a column with only NULL values caused turbodbc to report a casting error (most likely because it left the python type as object).
Trying to insert it via arrow instead of numpy caused a segmentation fault.

@pacman82 Would you be able to produce a stacktrace?


Migrate PL website & curriculum Site to team Fleek

** Describe the use case**
Migrate both the Launchpad website and Curriculum site to a shared team Fleek account
Fleek Migration Guide
** Describe the problem**
Context:
Jeff Everett
Hi Lindsay-- as we've continued to add stuff to pathfinder account, there is a pretty strong preference to not share that account.
Can I help get whatever site(s) Marco needs to work with moved to your own fleek account instead?
I've gone through this with another team and had some initial feedback-- the fact fleek is able to basically just update an existing site to be owned by another account means we don't have to do any changes in git or DNS, so it's pretty straightforward.: https://www.notion.so/protocollabs/Fleek-account-migration-guide-c091cd0c0d6d426092f46474f40bcf0e
If there's an urgency around fixing something, we can potentially add Marco to pathfinder, but we're really trying to avoid creating the same situation that made us need to move away from the old shared protocol team account to begin with.
Proposed Solution
How can this Problem be solved?

@walkerlj0 needs to talk to Molly/ Walker & Johnny about this need, and see if
A) We migrate away from Pathfinder
B) Talk to Johnny about how we set up docs

LP now has a Fleek account that hosts both repos.


Build failure with USE_STATIC_LIBCXX option set to OFF

Describe the bug
I'm trying to integrate filament with another project built against shared c++ library. To be able to do that, I'll have build filament with shared c++ library too. filament's main CMakeList.txt has provided that option: USE_STATIC_LIBCXX. It is turned on by default. When I turn it off and build the filament again, I encounter many build failures. All of them are missed definition of some standard C functions, like memset, strlen and such. I gave it a try and it is actually not so hard to fix, just include proper headers (such as <cstring> <cstddef>)
To Reproduce
Steps to reproduce the behavior:

Modify /CMakeList.txt to turn off USE_STATIC_LIBCXX option:
change line option(USE_STATIC_LIBCXX "Link against the static runtime libraries." ON)
to:     option(USE_STATIC_LIBCXX "Link against the static runtime libraries." OFF)

build.sh release

Expected behavior
Build completes with no error.
Desktop (please complete the following information):

OS: Ubuntu 20.04
GPU: N/A
Backend: N/A

Additional context
Here's the clang version that I'm using:
$ clang --version
clang version 10.0.0-4ubuntu1
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin

Thanks for reporting this. I'm not sure why using a static libc++ would affect the required includes. We like to keep our includes clean anyway but the main issue is that we won't test this forward so this is likely to break again.

patch.txt
Created a patch file to address the build error. It is diff againts release v1.9.15.
Tested with both USE_STATIC_LIBCXX set to ON and OFF. seems build fine. hope that's helpful.

It's very helpful thank you!

Thanks for the contribution. Though it no longer applies, the hope is that it can spur interest in providing some relief from the huge, slow-linking static libraries.


Idea .:. Comment Block

What problem does this feature solve?
Hi,
Situation...
We're working with several persons on the same project.
We're searching for 'the best way' to comment our .vue files in the file itself.
We know we can use the normal HTML comment tags in our components (e.g. on top in the section.
We are wondering if this is the best and recommended solution....
Or ...
Wouldn't it be an idea to have a <comment> block available (like we have <template>, <script> and <style>) where we can type 'free' (meaning, non HTML) text
(and of course, this block is completely skipped during 'compile')
Thanks in advance for any reply or suggestion...
Kind regards,
Johnny
What does the proposed API look like?
<comment>
This is some general information about this .vue file.
Text here can be FREE (ascii) text
</comment>

We know we can use the normal HTML comment tags in our components (e.g. on top in the section.
We are wondering if this is the best and recommended solution....

Yes. Not in JS, you can use // and /*

Hi,
I don't want to comment the code itself per line. (HTML or JS)
The question is in fact:
How can I 'describe' my components in a general way...
Grtz,
Johnny

you can do that in many ways. Using comments at the top of the script tag would be my go. But generally, just comment your component as you would comment code

I think the question was about a new block on single-file templates. Custom blocks are already possible and vue-loader has documentation regarding them here: https://vue-loader.vuejs.org/en/configurations/custom-blocks.html
So you can already use a  block and it will be ignored by vue-loader

Thanks everybody for your time to reply.
@pandemm
Indeed, a 'Custom block' will do the trick. (I didn't know about their existence)
Thank you for the link, and pointing me in the right direction.
Kind regards

how can i comment a block of code in vue.js


Update nomination so that wallet correctly displays account balance

this is due to the fact that nearcore now keeps account balance in units of
yoctonear

@janedegtiareva Should we update this to use the nearlib function? (Once that PR is merged)

Looks good overall, but I assume we wait with merge for now (before nearcore is updated)?

Looks good overall, but I assume we wait with merge for now (before nearcore is updated)?

It can be merged right now - the current version of nearlib has constants for attoNear. When we ship core change, we just need to update the constant in nearlib and bump the nearlib version.

@janedegtiareva @kcole16 found interesting bug: after I send 0.1 NEAR once, next time I try to send 10 NEAR – I have 0.1 stuck. Same goes for follow up screen and I think the amount actually sent as well. Also I feel like we should remove that second number altogether, not sure why it's displayed.
Note, that I haven't tested yet if it's a regression from production wallet.

@janedegtiareva @kcole16 found interesting bug: after I send 0.1 NEAR once, next time I try to send 10 NEAR – I have 0.1 stuck. Same goes for follow up screen and I think the amount actually sent as well. Also I feel like we should remove that second number altogether, not sure why it's displayed.
Note, that I haven't tested yet if it's a regression from production wallet.
UPD: Looks like it doesn't happen on production. So this shouldn't be merged for now.

I can't repro the bug! How do you navigate to the send page again? I go through "Go to dashboard -> send money", and I see 0 as the amount to send.

Repros for me.
Also:

after sending money first time "Send Money" button doesn't work - you can only go to dashboard.
at registration I'm getting 10.1 NEAR ???
after sending 0.1, I get now balance 9.900000999999996562 shown which is super unclear. Especially given I had 10.1 before --- where is ~0.1 more?


Allow tags to be ended before the end of a string

Describe the solution you'd like
The current #tag parser looks for " " and # to find the end of a tag. This is creating a list of tag mutations that makes using tags as filters a challenge as you only get partial results back by selecting the main topic or you need to select multiple tags for the same topic.

Ideally there would be a way to mark the end of the tag so that the user can specify what part of the string should be used as the tag

Proposed solution
Add a text combination that indicates the end of a tag mid string

Proof of concept
I created a PR that allowed for this here (https://github.com/usememos/gomark/pull/4)
Couple of design decisions

Used ]] as tag end indicator similar to embedded content. This indicator can be anything
Decided to create an additional inline tag parser that gets evaluated before the og tag parser. Thought this would allow for the idea of indicating a tag end to be extended at a later stage to allow for e.g. # to be used in tags (like mentioned here: https://github.com/usememos/memos/issues/3787). This could be simplified by combining the 2 parsers

Implementation

The pr works with the current UI so tag suggestions and editing works out of the box
Type of feature
User Interface (UI)
Additional context
No response

Refer to https://github.com/usememos/gomark/issues/5


Configurability enhancements - table_1_trans section

Add validations to the table_1_trans section data to support more robust configurability.
The following tickets are part of this main ticket:

[x] #594
[x] #595
[x] #596
[x] #597
[x] #598
[x] #599
[x] #600
[x] #601
[x] #602
[x] #603
[x] #604
[x] #605

This issue will be addressed in cQube v5.0.

This is related to cQube v4.1 beta


No-Cache for Notifications

#629
Added a never_cache decorator for IE as it seems to be running into caching issues
Changelog:

Added never_cache decorator to list()

 Add Pragma to app | Adding Pragma may mitigate caching issues with IE |1|


degrade to armv8.0

Signed-off-by: Lloyd-Pottiger<EMAIL_ADDRESS>What problem does this PR solve?
Issue Number: close #6254
Problem Summary:
What is changed and how it works?

disable sve on ARM by default
fix the broken test cases introduced by https://github.com/pingcap/tidb/pull/37384 and https://github.com/pingcap/tidb/pull/38447 that change the default value of tidb_enable_clustered_index from INT_ONLY to ON

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-test

/run-all-tests

/run-integration-test

/run-all-tests

/rebuild

/rebuild

/build

/rebuild

/run-all-tests

/run-integration-test

/merge

/cherry-pick release-6.4


failure to install graph cli dependencies via yarn

Do you want to request a feature or report a bug?
I tried to install a subgraph in my folder and for that i tried installing graph cli and it errors out.
What is the current behavior?
the yarn-error.log file reports this error
Trace:
  Error: connect ETIMEDOUT 2405:200:1607:2820:41::36:443
      at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1300:16)

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
i tried the command in format
graph init --contract-name RandomWinnerGame --product hosted-service GITHUB_USERNAME/Learnweb3  --from-contract YOUR_RANDOM_WINNER_GAME_CONTRACT_ADDRESS  --abi ./abi.json --network mumbai graph

and then after a lot of warnings this error is thrown.
What is the expected behavior?
should install all the dependencies and create a subgraph in the directory.

Same issue here. Trying both with yarn and npm. Also tried this solution from Stackoverflow adding the --prefix /usr/local parameter and it didn't work

Can anyone post a link where this issue is continued? I am having the same problem. @evaporei

Closing this as I think the underlying issue is the same as https://github.com/graphprotocol/graph-tooling/issues/1052, let's continue investigation there


fix bundler cache

oh looks like this is just a test?

@bingbongle just a test :p  (I have to open PRs to trigger the worfklow). attempting to resolve https://github.community/t/bundler-bundle-not-using-cache-in-github-actions/117686


Documentation: Check spelling on CONTRIBUTING.md

There are a few typos and misspellings on the CONTRIBUTING.md page.
I'm reserving this issue for any first-timers or code newbies who want to practice using git and making pull requests.
If you need help comment below and I'll help you to get started!

I can do that. 👍

Happy to do this one, let me know :)

I've never made a pull request and would like to start here - please choose me! :)

@Jared-Peacock @tomasswood
It looks like someone else already made this edits. I'm sure there are other errors elsewhere if you wanted to take a look.


How to change cache path when evaluating multi-models

Hi, I have noticed that the following codes control for the cache file, https://github.com/tatsu-lab/alpaca_eval/blob/4ced94f63cec6de3ac9e69ee59634d43e1bcba5c/src/alpaca_eval/annotators/base.py#L508
and I want to know that if I simultaneously start evaluation for multiple models with the same annotator, will the cache for these multiple models be stored in the same file? Could there be cache conflicts? Additionally, how do I specify the cache file path from the command line?”
For example, my program submits the following three command lines simultaneously. These three models use the same annotator for evaluation, so the cache file names appear to be the same.
alpaca_eval --model_outputs {model1} --annotators_config {annotators} --output_path {output_path1}
alpaca_eval --model_outputs {model2} --annotators_config {annotators} --output_path {output_path2}
alpaca_eval --model_outputs {model3} --annotators_config {annotators} --output_path {output_path3}

Can I specify the storage path by using a --cache-file option?

It's possible but unlikely that you will get caching conflicts. In particular, I reload the cache right before appending the new annotations. Note that worse case if there's a caching conflict, you will lose some of the cached annotations which isn't too bad (you will still have all the annotations & score, but if you want to reannotate it will call GPT4 again).
For the CLI use --caching_path {path}


Cache executables and add them to the sandbox

@soenkehahn I merged this now, but please feel free to still give it a review.