Suggested Option: Only update Ark on Major Version Changes

Hi guys,
Just chatting with my friend about this today. Ark has released 2 minor version updates in the span of only a few hours today. So, my server is set to check for Ark updates, and mod updates, every 2 hours. As a result, my server has booted players a few times today, to preform these updates, which is somewhat annoying to some of the players. Which, raises this question:
Is there any current way to configure this tool to ONLY update on major version updates to Ark, rather than every version update? Or, if this is not already implemented, is this something that could be added?
Thanks for your efforts on creating this tool. It has been a godsend for my admin-ing duties. Cheers guys!

This would have to use an external service (such as the ARK wiki) to keep track of what the latest version is, as Steam only provides a Build ID and not the version.

http://ark.gamepedia.com/Web_API#version
I have been following http://arkdedicated.com/version. As what I have been keeping my eye on that the version number bumps up the minor version number on hotfixes too.
And at least couple months back the version number from time to time was not in sync; there was new release in steam but http://arkdedicated.com/version reported the previous version number. I'm not sure how it is these days. So might not be the most reliable solution to check if the update is major or not.

@Lihis Thank you for sharing those links.
It looks like major versions are reported here: http://arkdedicated.com/version/major
Not sure how actively it's updated, so, I am not sure if that is a reliable means to this end.

@radonn24 I have not paid attention to the major version so I do not know does it share the same behaviour as I wrote on last part of my previous message.


Entity to manage lanch configuration assigned to auto scaling groups

auto scaling groups in command line

Not applicable.


Added Pusher.js in Real-Time category

Pusher does a good work for transferring data through Web Sockets in real time.
By submitting this pull request, I promise I have read the contribution guidelines twice and ensured my submission follows it. I realize not doing so wastes the maintainers' time that they could have spent making the world better. 🖖
⬆⬆⬆⬆⬆⬆⬆⬆⬆⬆

Pusher is a paid and commercial service. Likely the PR will be put down.


zh-TW addition with some minor translation addon

Rejecting because this requires cherry-picking and conflict resolution.
You did not pull the updates from the main repository and your commits include changes (that were already made in the main repo) to files from the English translation.


Vision API failing on Standard Environment Appengine

Are the cloud vision python clients ready for use on Standard Environment on AppEngine?
Steps as below :
pip install --upgrade google-cloud --target ./lib

Redeployed the project and during the invocation of the API I get the below stacktrace.
    from google.cloud import vision
  File "/base/data/home/apps/s~app-test/20170202t220112.400250114231361871/lib/google/cloud/vision/__init__.py", line 21, in <module>
    from google.cloud.vision.client import Client
  File "/base/data/home/apps/s~app-test/20170202t220112.400250114231361871/lib/google/cloud/vision/client.py", line 22, in <module>
    from google.cloud.vision._gax import _GAPICVisionAPI
  File "/base/data/home/apps/s~app-test/20170202t220112.400250114231361871/lib/google/cloud/vision/_gax.py", line 17, in <module>
    from google.cloud.gapic.vision.v1 import image_annotator_client
  File "/base/data/home/apps/s~app-test/20170202t220112.400250114231361871/lib/google/cloud/gapic/vision/v1/image_annotator_client.py", line 31, in <module>
    from google.gax import api_callable
  File "/base/data/home/apps/s~app-test/20170202t220112.400250114231361871/lib/google/gax/__init__.py", line 36, in <module>
    import multiprocessing as mp
  File "/base/data/home/apps/s~app-test/20170202t220112.400250114231361871/lib/multiprocessing/__init__.py", line 87, in <module>
    import _multiprocessing
ImportError: dynamic module does not define init function (init_multiprocessing)

The cloud vision python clients seem to be doing fine on a compute engine box.

google.gax does not work at all on GAE Standard (because the gRPC transport is not supported).
Set the GOOGLE_CLOUD_DISABLE_GRPC environment variable to 'true' and it should fall back to the HTTP transport and work without issue.
Feel free to re-open if this does not fix the problem for you.

Added below to app.yaml and redeployed using gcloud but still the same. It's unable to detect the environment variable.
env_variables:
APP_ENV: 'test'
GOOGLE_CLOUD_DISABLE_GRPC: 'true'

Anyone know what is the recommended way now to use Vision API from appengine Python standard environment.


Updated genes list

It's been a while, and much has changed. For instance, now, we have a field called gene family to make it easier to group genes! I'm also including details of which genes are automated (controlled partially, or entirely, by programmatic rules).

Bump! @reganartinfo


feat: bump the peer dependency version of react-native-performance

Details

This PR updates the version of react-native-performance peer dependency from 4.0.0 to 5.1.0.
Related Issues

https://github.com/Expensify/App/issues/26987
Automated Tests

N/A
Linked PRs

https://github.com/Expensify/App/pull/27289

Not sure why @marcochavezf was not assigned here, do we not have react-native-onyx repo as part of the automation?

Not an emergency merge, tests were passing.

@allroundexperts You will now need to update the onyx version in App repo.

@allroundexperts Can you link the PR that updates react-native-performance in the App? Asking as it is blocking one of my PR


System.Exception: TODO: No service for type 'Microsoft.Framework.DependencyInjection.ServiceLookup.IServiceManifest' has been registered.

Running 1.0.0-alpha4-10353 on Linux/Mono 3.12.1 . When I try to run
k web
Then I get this:
System.Exception: TODO: No service for type 'Microsoft.Framework.DependencyInjection.ServiceLookup.IServiceManifest' has been registered.
  at Microsoft.Framework.DependencyInjection.ServiceProviderExtensions.GetRequiredService (IServiceProvider provider, System.Type serviceType) [0x00000] in <filename unknown>:0
  at Microsoft.Framework.DependencyInjection.ServiceProviderExtensions.GetRequiredService[IServiceManifest] (IServiceProvider provider) [0x00000] in <filename unknown>:0
  at Microsoft.AspNet.Hosting.HostingServices.Import (IServiceProvider fallbackProvider) [0x00000] in <filename unknown>:0
  at Microsoft.AspNet.Hosting.HostingServices.Create (IServiceProvider fallbackServices, IConfiguration configuration) [0x00000] in <filename unknown>:0
  at Microsoft.AspNet.Hosting.Program.Main (System.String[] args) [0x00000] in <filename unknown>:0
  at (wrapper managed-to-native) System.Reflection.MonoMethod:InternalInvoke (System.Reflection.MonoMethod,object,object[],System.Exception&)
  at System.Reflection.MonoMethod.Invoke (System.Object obj, BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00000] in <filename unknown>:0

The weird thing is that I get this on Windows too, so it doesn't seem to be a Linux specific thing. Judging by the TODO comment this looks like something that needs to be implemented.

I don't know if this is related, but after I installed the dev RC-1, my older projects that target beta-1 no longer will compile in visual studio.  All of the assembly neutral interfaces are missing (Such as IServiceCollection and IHostingEnvironment).  I have made sure to select the beta-1 runtime in the "Target KRE version".
What's strange is that I can build from the command line as long as I make sure to "kvm use 1.0.0-beta1".  I have even tried making the old beta-1 runtime the default from the command line, but visual studio still compiles the project as though it's using the rc-1 runtime.  I've tried restarting visual studio, etc...
If I delete the .kproj.user file (that contains the selected runtime) it reverts the "Target KRE version" to the default that I've selected (beta-1).
Note that it's conceivable that the build errors that I'm getting have nothing to do with which target kre version I've selected, but it seems unlikely since I can reproduce the exact same errors from the command line if I choose the rc-1 runtime.
Has anybody successfully gotten Visual Studio to be able to switch back and forth between the beta-1 and rc-1-nightly runtimes?

Stuff changed in the latest runtime that will make older deps fail to compile. Basically a compiler change that;s no backwards compatible.

Thanks for the amazingly prompt response, but I'm not quite sure I understand:
I can successfully change the KRE runtime from the command line from the rc-1 version that I installed recently back to the beta-1 one, and then successfully build my older project.  I would expect that choosing the "Target KRE Version" in visual studio 2015 would have exactly the same effect, but it somehow now seems permanently stuck on the RC-1 KRE, even when I've changed the default KRE back to beta-1.
I understand that I changed the actual "kvm.cmd/kvm.ps1" scripts when I installed the RC-1 runtime, but it seems bizarre that an incompatibility with Visual Studio 2015 would manifest itself in such a specific and non-sensical way.

UPDATE:
Visual Studio 2015 appears to always use the compiler that's associated with the KRE that is marked with the ALIAS of "default".  Changing the "Target KRE Version" does not appear to be able to change the version of the compiler that's used.  So it appears that for now, the only solution to this issue is to change the "default" alias to the desired KRE and restart visual studio.

I am having this problem with beta2 on Ubuntu 14.10. Ran the dev kvm script and did upgrades and and everything. neither k web or k kesterel work
root@vnext:~/Home/samples/HelloMvc# k kestrel
System.DllNotFoundException: libdl
  at (wrapper managed-to-native) Microsoft.AspNet.Server.Kestrel.Networking.PlatformApis/LinuxApis:dlopen (string,int)
  at Microsoft.AspNet.Server.Kestrel.Networking.PlatformApis+LinuxApis.LoadLibrary (System.String dllToLoad) [0x00000] in <filename unknown>:0

web
root@vnext:~/Home/samples/HelloMvc# k web
System.TypeInitializationException: An exception was thrown by the type initializer for HttpApi ---> System.DllNotFoundException: httpapi.dll
  at (wrapper managed-to-native) Microsoft.Net.Http.Server.UnsafeNclNativeMethods/HttpApi:HttpInitialize (Microsoft.Net.Http.Server.UnsafeNclNativeMethods/HttpApi/HTTPAPI_VERSION,uint,void*)
  at Microsoft.Net.Http.Server.UnsafeNclNativeMethods+HttpApi.InitHttpApi (UInt16 majorVersion, UInt16 minorVersion) [0x00000] in <filename unknown>:0


build(dev-deps): added mermaid plugin to docs

Issue #, if available: N/A but discussed directly with maintainers
Description of changes:
This PR adds the plugin that allows to use mermaid diagrams in the MKDocs documentation as discussed internally.

Note: the diagram above is provided only as an aid for review, it is not part of the actual commit.
Additionally I also updated the year in copyright notice that is in the footer of the docs to the correct year (it was still 2021).
Checklist

[x] Meet tenets criteria
[ ] Update tests
[x] Update docs
[x] PR title follows conventional commit semantics

Breaking change checklist

RFC issue #:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #1070 (abaeed3) into develop (2a3ff9a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #1070   +/-   ##
========================================
  Coverage    99.96%   99.96%
========================================
  Files          119      119
  Lines         5365     5365
  Branches       612      612
========================================
  Hits          5363     5363
  Partials         2        2

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 2a3ff9a...abaeed3. Read the comment docs.

Thank you so much @dreamorosi !!!!


Error with v-text

When i use v-text and refresh page i get an error "Cannot read property 'replace' of undefined"

When I turn to this page with any other, then everything is ok. And when i change v-text to v-html everything is fine.

@alexchopin I do not need help, it's a bug

Set a default value to s param:
function escape(s= "") {
...
}

@NicoPennec i think you should say this to the developers nuxt, because this is not a bug in my code. As you can see, this error in node_modules folder.

vue-server-renderer module come from Vue Core: https://github.com/vuejs/vue/tree/dev/packages/vue-server-renderer
Can you share your complete components/forms/vinput.vue file? the template is truncated on your screenshoot...

link to https://github.com/nuxt/nuxt.js/issues/1327

@NicoPennec
vinput.vue file.

as I said, the problem is solved, when replacing the v-text with v-html, but still, it's a bug

It seems that the bug is probably on your side.
if errors is an empty array, then errors[0] will be undefined, because the v-if condition is true => errors is not null...
retry with v-if="errors.lenght" v-text="errors[0]"

@NicoPennec
Okay, but in other places where I also use v-text, I get same error. And as you wrote before, to fix this bug, you need to fix the build.js file in nuxt core and add
function escape(s = "") {
  ...
}

@NicoPennec
And if you don't know, how Vue.js and Nuxt.js works, please don't answer. You've already recommended that I fix the file in the node_modules folder, and if i change my condition in v-if, nothing will change. Thanks.

vue-server-renderer module come from Vue Core

I think you @NicoPennec is in a right direction. If we ever should consider not "handling null values with v-text" directive as a bug, we need to make minimal a reproduction code and report it to the correct repository (vuejs/vue in this case) not here, as it is not directly something with Nuxt. I'll try to make a PR fix for this soon.

@Exitialis
You are right, the label "help wanted" is a not good choice for your issue.
People (like @NicoPennec) from the Vue.js/Nuxt.js community should not spent their free time to resolve your issue posted in the wrong repository.

@NicoPennec @alexchopin @pi0
I did not want to offend anyone, and I also did not know that it was a bug with vue.js, since before updating the version of Nuxt to the RC version, everything worked fine. I apologize if I offended anyone.

Closing since it's related to Vue-Server-Renderer directly (https://github.com/vuejs/vue/issues/6572), in the meantime, make sure to not use null value into v-text @Exitialis by using a computed property.
Thanks to @NicoPennec for his help on this issue.


fix(collaborator): show all collaborators in assignee dropdown.

@vikram-raj snapshot fabric8-ui is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-1997-fabric8-planner.int.rdu2c.fabric8.io


Migration SFN is not robust to mid-run deployments

At present, a single invocation of the schema migration SFN is not robust to deployments that occur while the step function is running. One or more task definitions that the step function depends on will be expired / rendered no longer active. It might be worthwhile determining how to make sure task definitions are not immediately expired if they are updated in order to prevent the SFN run from being disrupted. This would allow us to continue to deploy as normal when schema migrations are running.

@nayib-jose-gloria / @Bento007 : Moved this over to epic #544 for your consideration. Please close if not relevant.

Now that we can freeze deployment and plan to move to a different migration paradigm, this work is not worth the effort.


Fixed PSScriptAnalyzer errors in examples. Fixes #14

Hi @nanalakshmanan, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

closing per author's request


Add basic UFW rules

Rules included:

Default deny policy
SSH allowed
WireGuard allowed

Other service-specific rules are loaded from roles/ufw/vars/main.yml, which will have a list of dictionaries, where each dict corresponds to options for the ufw module.

I'm not sure if this will work or how to test it
https://github.com/python-discord/infra/pull/16/files#diff-68eba24c77e511acfa744d0aeb3f24a1c7b57d56dcb27f455ba0500ceab3eeebR18-R20


cryptsetup: 2.4.3 -> 2.5.0

Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

@OfBorg build cryptsetup pkgsStatic.cryptsetup

The aarch64 failure seems to be unrelated to the change:
error: derivation '/nix/store/cc547z47b82jvwdmdnbn98z7h00pkzk1-python3-minimal-3.10.6.drv' requires non-existent output 'out' from input derivation '/nix/store/2zs22w5bglhh10j2j4cm9zbfxbh1hvnx-expat-2.4.8.drv'


fix: chore msg factories

Closes #320
Adds all missing msg factories

Summary by CodeRabbit

New Features

Introduced new encoding and decoding functionality for transaction messages in the devgas, inflation, oracle, sudo, and tokenfactory modules.
Added new message types and factories for creating transaction messages related to fee shares, inflation parameters, oracle transactions, sudo operations, and token factory actions.

Refactor

Reordered exports in the GraphQL query index to optimize module accessibility.
Updated message factory in encode-types.ts to support new modules.

Tests

Modified test names and variables in epochs.test.ts to align with updated request and response naming conventions.

Documentation

Adjustments and clarifications in documentation to reflect new features and refactoring efforts.

Failure is sonarqube

LGTM


Update ImageDataGenerator to allow channelwise standardization

This is somewhat related to the issue  here (https://github.com/fchollet/keras/issues/2559#issuecomment-222501765), but also looks to be on a to-do list in the random_transform method of the ImageDataGenerator.
Two notes:

If we call samplewise_center without passing the boolean for channelwise (which is the default on master right now) for grayscale images, this 0's out the image (this is the line that leads to that: https://github.com/fchollet/keras/blob/master/keras/preprocessing/image.py#L323). This is noted in the issue linked above. It might be worth putting in some kind of warning or something like that in the future to let people know.
In the random_transform method, there is "channel-wise normalization" on the to-do (https://github.com/fchollet/keras/blob/master/keras/preprocessing/image.py#L400). I'm not certain whether that refers to what I've done here (I think yes, but it's in the random_transform method). So, that might be something that could be removed if this PR is accepted.

i understand and agree with the channelwise case implementation.
but for channelwise=False why do you standardize over the channels (as in master today) instead of over the whole 3d (channelsrowscols)? i just don't understand the usecase for the former, and i think the latter is more useful.


Logout Exception for ADFS identity provider

We integrated a partner ADFS system to our B2C login as per the documentation. Everything looks good and is working. However, our partner has contacted us and said that they are seeing exceptions in their logs. These exceptions happen during logouts.
Any idea how I can solve this? Shouldn't WantsSignedRequests be true by default and also sign logout requests? The partner federationmetadata specifies WantAssertionsSigned="true", but no other Wants* are present.

The verification of the SAML message signature failed.
Message issuer: https://mytenantinfo/B2C_1A_TrustFrameworkBase
Exception details:
MSIS7074: SAML authentication request for the WebSSO profile must specify an issuer with no NameQualifier, SPNameQualifier or SPProvidedId properties.
This request failed.
User Action
Verify that the message issuer configuration in the AD FS configuration database is up to date.
Configure the signing certificate for the specified issuer.
Verify that the issuer's certificate is up to date.
Verify the issuer and server message signing requirements.
Encountered error during federation passive sign-out.
Additional Data
Exception details:
Microsoft.IdentityServer.RequestFailedException: MSIS7054: The SAML logout did not complete properly.
at Microsoft.IdentityServer.Web.Protocols.Saml.SamlProtocolHandler.BuildSamlLogoutResponse(SamlContext samlContext, Boolean partialLogout, Boolean& logoutComplete)
at Microsoft.IdentityServer.Web.Protocols.Saml.SamlProtocolHandler.ProcessSignOut(SamlContext samlContext, String redirectUri, List`1 iFrameUris, Boolean partialLogout)

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

ID: e14a5d4c-5125-8ed2-7fda-38aee1296b76
Version Independent ID: 054d298b-937e-9f01-ae56-7025e47e4eec
Content: Add ADFS as a SAML identity provider by using custom policies - Azure AD B2C
Content Source: articles/active-directory-b2c/identity-provider-adfs2016-custom.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

@LXBdev,  Thanks for the feedback! We are currently investigating and will update you shortly.

@LXBdev, Thank you for reaching out. Looking at the error, it looks like somehow the logout request to ADFS was not correct and hence ADFS threw that error. One thing that I can think of to check here is if the certificate (public key) added under the Signature tab of the B2C relying party properties in ADFS is matches the certificate that is present in the B2C.
If this is also correct, then to troubleshoot this issue would need deeper analysis which wont be possible through the forum. We would have to collect some logs like a Fiddler trace to understand the requests and then troubleshoot the issue accordingly.
I would suggest you to create a case with the Microsoft Support team to help you on this. In case you face difficulties opening a case, please do drop an email to us on azcommunity[at]microsoft[dot]com along with the following details:

Tenant Name/TenantID:
Subscription ID:

Do not forget to share the url of the following thread so that its easier for us to identify and help you better.

@LXBdev, Just wanted to followup with you to check if you got a chance to create a case of if you have sent an email to the community on the email id mentioned above. Do let us know if any more help is required from our end on this.

@LXBdev, Closing the thread for now, since no response.


Topic exists exception thrown when using EmbeddedKafkaRule

In what version(s) of Spring for Apache Kafka are you seeing this issue?
3.0.4. Issue was discovered in 2.2.15.RELEASE, but verified present in 3.0.4.
Describe the bug
Sometimes topics created in an earlier test are still present in later tests, even though a new EmbeddedZookeeperServer is created. This causes an exception to be thrown when the EmbeddedKafkaRule calls afterPropertiesSet() on the EmbeddedKafkaBroker (when the EmbeddedKafkaBroker calls createTopics()). I'm not sure if the issue is in this project, or in Kafka itself, but I'm starting here. I spent a considerable amount of time debugging but have been unable to identify the root cause of the problem. A new Zookeeper server is started for each test, and should have clean log and snapshot directories, but somehow, sometimes it doesn't.
To Reproduce
embedded-kafka-rule-test.zip
I've attached a zip file containing a small project reproducing the issue. Note that it doesn't occur 100% of the time. If it consistently works for you, try increasing the LOAD_FACTOR constant in AbstractEmbeddedKafkaRuleTest. Increasing the number of topics / messages appears to more reliably reproduce the issue. Naturally, it may appear contrived, but I started seeing this issue with only 2 topics, and 1 or 2 message in each topic. It failed about 20% of the time on my machine.
Expected behavior
Tests should not fail with "topic already exists" errors.
Sample
Minimal reproducible sample

Yes. You test is very hard to reproduce just because you use that "test-topic-" + UUID.randomUUID().
Also I don't see how EmbeddedKafkaBroker.afterPropertiesSet() can fail if you start it without any topics upfront: new EmbeddedKafkaRule(1, true, 1).
I also think, even if you have that public static EmbeddedKafkaRule embeddedKafka in the abstract class, every single extension of that class would get its own EmbeddedKafkaRule instance.
So, if you believe that two of those EmbeddedZookeeperServer looks into the same target Kafka dir, then it is indeed the problem in the Kafka Client test library or even downstream in the java.nio.file.TempFileHelper.
Please, consider to make your test as reliably reproducible as possible to give us a chance to dig more what is going on.
Also consider to try to upgrade to the latest Spring for Apache Kafka: perhaps some fixes has made it into any of transitive dependencies.

@artembilan I generate random topics for every test. This is the practice recommended here: https://stackoverflow.com/a/60064493/1303158. The test prints out all the topic name it creates, so you can see that when it fails, the "topic exists" error that is thrown by the second test is a topic that is created for the first test.
I also don't see how EmbeddedKafkaBroker.afterPropertiesSet() can fail without any topics upfront, however that is the behaviour that my demo exhibits. As I said, more load increases the likelihood of failure, so I suspect that there's some kind of race condition - I speculate that some asynchronous operation called by EmbeddedKafkaBroker.destroy() does not have enough time to complete, and so some element of Kafka or Zookeeper carries over into the next test run.
Using public static EmbeddedKafkaRule embeddedKafka means there is only one instance of the EmbeddedKafkaRule no matter how many subclasses are instantiated. The static field is on the abstract superclass, not on each one of the subclasses.
...
I spent some more time investigating and discovered that a KafkaProducer was still running after the test completes, it appears as though this is the root cause of the issue. So finally I realised that I must call either ProducerFactory.reset() or DefaultKafkaProducerFactory.destroy() after I'm finished with the KafkaTemplate. My tests are now working as expected.
I'm closing this issue as there's nothing really to fix.


Creating the Collapsible component

An interactive component which expands/collapses a panel.

https://github.com/Mobilecn-UI/nativecn-ui/assets/43105344/c746c0d3-2326-4fee-aa54-02f726c9b778

Hey @Victorcassiano, thanks for the PR. Please add description and screenshots.

Hey @Victorcassiano, thanks for the PR. Please add description and screenshots.

Ok, I'll add it, sorry for not adding it sooner. 😅😅

@sumansid Is everything ok with the PR or is something else missing?

@Victorcassiano Can you follow this style?
Collapsible
Also, please write comments in english since many of the users are english speakers.

@Victorcassiano Can you follow this style?
Collapsible
Also, please write comments in english since many of the users are english speakers.

Comments in the code you say?, if so I forgot to remove them hehe

@sumansid Is there anything else missing?

@Victorcassiano
As mentioned before please follow Shadcn's style for the collapsible :Collapsible component
Also, would be nice if you can update the docs as well in the .mdx file

@Victorcassiano Checking in to see if you needed any help with the documentation.

@Victorcassiano Checking in to see if you needed any help with the documentation.

Yes, I need it, would you happen to have a model for me to follow?

Sure, feel free to check out the .mdx files. Here's a PR that got merged recently with a documentation.
https://github.com/Mobilecn-UI/nativecn-ui/pull/25


feat request: support http/https download function

For example,
import megfile

path = "https://dl.fbaipublicfiles.com/detectron2/ImageNetPretrained/MSRA/R-50.pkl"
megfile.smart_copy(path, local_path)
# or new interface like: megfile.smart_cache(path, local_path)

@LoveEatCandy

This feature had supported, do you have any questions about it? Please show your code and errorstack.

This issue will be closed, if u still have any questions, u can reopen it.


[Improvement] Correct error message of ReflectUtils's invokeAs when method not found

Why are the changes needed?
Currently, overloaded methods are considered the same and deduplicated in ReflectUtils, thus not easy to tell why no method is found.
This PR fixes the problem by adding the argument lists. In addition, it fixes the problem that the arg classes are not printed correctly.
How was this patch tested?

[x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Merged. Thanks to all!

@link3280 don't forget to update the milestone and assignee

@link3280 don't forget to update the milestone and assignee

Thanks for reminding me. Updated.

@link3280 don't forget to update the milestone and assignee

Maybe we could check these requirements in the request merging scripts.

Maybe we could check these requirements in the request merging scripts.

Nice idea~

@link3280 don't forget to update the milestone and assignee

Maybe we could check these requirements in the request merging scripts.

Created PR #5152 for this.


File lifecycle primitives: Read & Write

Finish file lifecycle primitives so we can read and write files.
FUSE operates on blocks of a file. So every request will be for a Read or Write of a length of bytes at an offset. This maps cleanly to Go's io.ReaderAt and io.WriterAt interfaces. Read operations via FUSE will only read up to the Size attribute of the file, so we need to know the file size before performing any reads.
This has several use cases our primitives need to satisfy:

I have a file where I know the size. Primary operations will be ReadAt and WriteAt to make updates to it.
I have read-access to a log or other data source with unknown size that I can only read via a Reader or download fully. When FUSE opens the file we want to buffer the source and compute the size. Then ReadAt and WriteAt operations occur on that file.
I'm operating on a file through an actual filesystem. It makes sense to mirror Open, ReadAt, WriteAt, Sync, Close so we get consistent behavior.
Writing to an entry with no persistent content like a message queue. This can mirror WriteAt and ignore the specified offset.
Update a file. I'm not clear what operations are involved when an editor saves a file, so this may take some experimentation. Likely candidates are WriteAt and Sync or Close.

The file lifecycle has several points that could be useful

Open: buffer data to compute its size, open a file descriptor. Additionally we need to take into account whether an open for write is truncating the file.
ReadAt: read a block of data from a file or data source.
WriteAt: write a block of data to a file or data source.
Sync: batch buffered writes.
Close: release a file descriptor.

The file descriptor use-cases don't seem compelling enough to add Close. It's not clear that we need Sync yet. And Open is primarily a helper for data that we need to buffer up-front.
So I propose that we make ReadAt and WriteAt the primary patterns for plugins. As raised in https://github.com/puppetlabs/wash/issues/162, we can use Content as a helper concept for files that buffer everything in a single operation. ReadAt may be (initially) uncached while Content should be cached. Content will be called when a file is opened and the result used to update
type Contentable struct {
  Content(context.Context) ([]byte, error) // or the old SizedReader
}

type Readable struct {
  Read(context.Context, []byte, int64) (int, error)
}

type Writable struct {
  Write(context.Context, []byte, int64) (int, error)
}

External plugins would be able to implement any of these interfaces as well. Only Content is prefetchable (see External Plugins docs). This represents a breaking change to external plugins.
External plugin invocations would be
<plugin_script> read <path> <state> <offset> <size> # data written to stdout and must match supplied size
<plugin_script> write <path> <state> <offset> # data supplied on stdin
<plugin_script> content <path> <state> # data written to stdout

Looks good. Only thing I'd propose is to keep the Go interfaces as symmetric with external plugins as possible and to remove Content. So something like
Read(ctx, offset, size) ([]byte, fullContent, err)
Write(ctx, data, size)

Also, any reason why we're passing data to STDIN in external plugin write ? Would be nice to keep the invocation symmetric with other stuff.

We pass data to STDIN with write because it's expensive to encode binary data on the command-line (and command-line length is limited).

Could we break this issue up a bit? I can see at least two chunks:

Refactor the Read interfaces. Two cases that need to be handled: (1) Files were size is known -- these should have ReadAt semantics and (2) Files were size is known after downloading all content -- the Download interface.

Properly breakdown all the write scenarios and provide the appropriate implementations.

1 and 2 could also be broken up.
I'm mostly asking because I think breaking stuff up would make it easier for me to think about what's going on too. Read seems to make sense, but I'm having a bit of a hard time understanding all the Write stuff to give any valuable feedback on the design.

I'm tempted to leave the Block Write semantics for later. They're an optimization for writing large files, and probably wouldn't see any benefit for many types of updates.

I'm ok with that.


fix migrateConfig for io.containerd.cri.v1.images

Adjust code sequence in migrateConfig to migrate pinned_images and registry first even when containerd does not exist.
Fix: #10120

cc @dmcgowan


Unit Testing

Currently there is no unit tests for core bot functionality. Cog-based unit tests seem a little bit out of scope for the most part, but having tests for the core bot would be useful.

Just seems wildly unrealistic due to the nature of relying on an external service.


added new feature to check the winning side of trade

Description
API now returns a new columns which checks and tells which side of trade is better
Type of change
Please delete options that are not relevant.

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

Change Status

[x] Complete, tested, ready to review and merge
[ ] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

How Has This Been Tested?

tested on local machine

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My code has been reviewed by at least one peer
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] There are no merge conflicts

Looks good!


runewatch: v2.0.1 release

v2 includes the following changes:

use github as host for WDR/RW watchlist
include WDR as a source for reported users
make clan chat alerting toggleable
add option to disable/notifiy screenshots

Note: The warning upon plugin installation has been removed as we now store the watchlist on GitHub so users are not directly accessing RuneWatch's public API.

@RuneWatch can you confirm this code change looks good to you?

@Nightfirecat - We are happy with these changes. - Thanks for the confirmation check. I can include some transcripts of the UD between while-loop, runewatch & wdr, if necessary, to confirm all parties are happy.


Issue with PR #68

Hello!
I am using the IOSLinkedInAPI Pod in my companies project.
Today we tried to issue an update with the fix for the added characters in the state. (#71)
Now we experience issues added by PR #68.
Xcode complains that it can't find AFHTTPManager (which should get evaluated by the checks added in #68).
Anyone know how we could fix this? For now we reverted that commit in our fork.
Our project:
Deployment Target: iOS8
Using updated and forked IOSLinkedInAPI

The referred issue is just a spell correction in the Readme.md file...?
Do you mean: https://github.com/jeyben/IOSLinkedInAPI/pull/70 ?

Oh, yes. Sorry, Meant #70

Same issue here. I'm using AFNetworking 2.0 (both AFNetworking/AFHTTPRequestOperationManager.h and AFNetworking/AFHTTPSessionManager.h are present in AFNetworking 2.6.3)


fix: Use the incoming request to determine the host and port

This fixes issues when the hapi server is running on a different host/port to the external interface, e.g. within Docker.
Fix #497

Coverage remained the same at 100.0% when pulling 83b7d748d9aa23a822426625369c3efd8c84ee28 on jameswragg:master into 24e942c543a19a6fd76ba637aa98b65235493ed4 on glennjones:master.

Coverage remained the same at 100.0% when pulling 83b7d748d9aa23a822426625369c3efd8c84ee28 on jameswragg:master into 24e942c543a19a6fd76ba637aa98b65235493ed4 on glennjones:master.

@robmcguinness I've added some tests into the branch for James

thanks!


A basic svelte checker

Based mostly off the typescript checker and svelte-check's index.ts.

not stale.


Feat/sample file format + metadata propagation

Changed the expected input format of the samplesheet as discussed on notion.
We now expect a CSV file, with exactly 5 columns per sample (header required):

familyID
sampleID
sequencingType
file

A metadata block (familyID, sampleID, sequencingType, number of samples) now follow the channel for each function across the pipeline.
The pipeline still works with the same functions as before, we still need to separate them in modules.

Tested on juno


docs: Please introduce the system architecture

Current State
readme.md
Desired State
add system architecture

Thank you for your suggestion!
Indeed, adding a system architecture section will help improve the clarity of the project and make it easier for contributors to understand the overall design.
Now, this has been added to our to-do list and we will prioritize it. If you have any specific ideas or examples about the architecture, please feel free to share them!
Thank you again for your feedback!


Update multi_index.hpp example documentation

Change Description
Updated the example documentation for the multi_index.hpp constructor to match the actual code specification.
Consensus Changes
None
API Changes
None
Documentation Additions
It might be useful to update the rest of the example documentation as well.

As you can see #6329, contracts in eos repository will be removed.

eoslib is now here:
https://github.com/EOSIO/eosio.cdt


Remove CancelExecutionStep

It seems that in its current form cancelling the execution in the execution engine is pointless. Instead, we should at some point cancel the execution when the cancellation token is triggered the same way that we handle timeouts.

I see some benefit in not keeping the task history if the task is cancelled. If I hit Ctrl+C during the build, I'd probably want to have the interrupted task to be re-executed next. Let's leave everything as it is.
Please reopen if you disagree.


CAMEL-18932: Upgrade to slf4j-api 2.0.6

Description

Target

[ ] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[ ] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[ ] I checked that each commit in the pull request has a meaningful subject line and body.

[ ] I formatted the code using mvn -Pformat,fastinstall install && mvn -Psourcecheck

@davsclaus I added the updates

Thanks a lot

Seems to be good.


feat: RPC - starknet_getClassHashAt

Get the contract class hash in the given block for the contract deployed at the given address

Hello, may I take this one?


Fix search label width issue

What
Change the label in the search component to put the max-width: 68% onto it only when it appears 'inline' as fake placeholder text.
Why
Fixes https://github.com/alphagov/govuk_publishing_components/issues/4202
Visual Changes

Before
After

Nice work mate 👍🏻


chore(update): is-glob

/cc @es128 @phated i need some feedback on this update (tests failed) due ? character.
What we can do:

Modify https://github.com/es128/glob-parent/blob/master/index.js#L20 regexp for backward compatobility
Release major version and remove https://github.com/es128/glob-parent/blob/master/test.js#L50 and https://github.com/es128/glob-parent/blob/master/test.js#L55 tests

? is valid character for windows

/cc @es128 @phated friendly ping

@evilebottnawi we are working on it. Sorry for the delay.

@evilebottnawi i removed it in the PR

@paulmillr thanks

Closing in favor of #21


Any tips on making pre-roll race conditions smoother?

I've noticed the following sequence a lot in testing. It makes for a very distracting effect on pre-roll ads:

Player is started by click/ touch
Play button disappears
First frame of content loads!?!
Play button re-appears!?!
Screen goes black
Pre-roll plays
Pre-roll finishes
On returning to content play button is still visible!?!
Content plays

Here's an example of the above sequence:
https://www.dropbox.com/s/qndbesx6x48kqkq/screencast 2018-07-07 12-36-18.mp4?dl=0
IMHO, if pre-roll is available the sequence should be:

Player is started by click/ touch
Play button disappears
Screen goes black
Pre-roll plays
Pre-roll finishes
Content plays

Perhaps some of this can be improved by simple CSS and JS rules?

Hmm yours is the only implementation for which I've seen this - are you using additional plugins besides videojs-ima? I'm wondering if another plugin is potentially interfering with our flow.

Not that I'm aware of. Our lead developer think's it's because we're using custom poster images for our videos.

Hmm our examples are using posters as well, and we're not seeing the same issue. Is this page hosted anywhere that I can get access to it to test?

Sure, thanks: https://www.destinationtips.com/videos/12-things-not-thailand/

I can see what's happening, but I'm not sure why. It looks like your player is entering a paused state while the pre-roll is loading, which causes the play button to show. When the pre-roll finishes, you see the content with the play button for a second, then the player goes from paused to playing and the play button disappears. Would you be able to specify debug: true in your video.js plugin options so we can get an event log for contrib-ads? Thanks!

I'll add this today and revert back here…

I tested out your page with videojs-contrib-ads 6.0 and I'm no longer seeing the issue - are you able to update to that version and see if it fixes it on your end as well?

Thanks, @shawnbuso. I don't think we're using videojs-contrib-ads at all. I thought that lib was for writing plugins. I'll speak with our dev who is back to work on Monday…

videojs-contrib-ads is a dependency of videojs-ima :)

Right, I see it now, in our code ( videojs.ads.min.js ).  And we're already on 6.0.0.

Yea you're right - I wonder if this is a race condition of some kind, and maybe that's why my proxying a different file seemed to fix it.

Looks like our dev figured it out. Your example videos start with a single black frame. Ours don't. You can see the flickery effect in the side by side comparison here:
https://staging.destinationtips.com/examples/simple/
Should we and other users start producing videos that start with a single black frame?

Hi @jnthnclrk,
I wanted to check in on the status of your issue here.
I was looking into this and did not see the behavior you were describing even with a video with a non-black first frame. I did however see some similar behavior when throttling the network connection to slower speeds.
Let me know any changes on your end, if the issue is still a problem.

No longer an issue. Go ahead and close this.


Add support for firefox

Could you possible add this extension to Firefox or as a userscript. Added it as a temporary extension but it does not work.

Thank you for the suggestion but I have no plans to add FF support, at least not in the near future. I will tinker in my spare time anyway and if I manage to make a fully functional port - I will publish it.


Auto-Scaling Issues, terminating not found by master node and terminating on their own

Bug description and how to reproduce:
After getting my stack to successfully create by pre-installing unbound, I now have an issue with the autoscaling group. After immediately SSHing into my master node, I run sinfo and get this output:
PARTITION AVAIL  TIMELIMIT  NODES  STATE NODELIST
compute*     up   infinite      0    n/a

Also, I notice that the EC2 compute nodes in my auto-scaling group immediately terminate after initializing, with this message as justification:
Description:DescriptionTerminating EC2 instance: i-0aaafaa8d795cd925
--
Cause:CauseAt 2019-07-17T18:24:28Z instance i-0aaafaa8d795cd925 was taken out of service in response to a user request.

This happens even if I suspend termination and replace unhealthy actions for the auto-scaling group.
Looking into my slurmctld.log files in the master nodes, I find that the compute nodes are now not able to be found by the master node. If I run srun -N 2 -n2 hostname, the job gets stuck on waiting for node:
srun: Required node not available (down, drained or reserved)
srun: job 2 queued and waiting for resource

and the logs below reflect the changes as a result of requesting a job.
Environment:
AWS ParallelCluster version 2.4
[aws]
aws_region_name = us-east-1

[cluster HPC]
key_name = *****
vpc_settings = vpc
pre-install = s3://****/install_unbound.sh
scaling_settings = custom
base_os: ubuntu
scheduler: slurm
initial_queue_size = 2
maintain_initial_size = true
placement_group = DYNAMIC
placement = compute
master instance type = c5.large
compute instance type = c5n.large

[vpc vpc]
vpc_id: vpc-*****
master_subnet_id = subnet-*****
vpc_security_group_id = sg-*****

[scaling custom]
scaledown_idletime = 30

[global]
cluster_template = galaxy-HPC
update_check = true
sanity_check = true

[aliases]
ssh = ssh {CFN_USER}@{MASTER_IP} {ARGS}

Referenced Issue
#1192
Logs
master_cfn_init_log.txt
master_jobwatcher.txt
master_sqswatcher_log.txt
master_slurmctld_log.txt
compute_nodewatcher.txt

I noticed errors in the /var/log/amazon/ssm directory that asked for permissions for ssm and ec2messages, which I've added. The update SQSwatcher file is attached here.
updated_master_sqswatcher_log.txt

By looking at the logs It seems like the issue is still related to the networking/dns setup. The master node is not able to resolve the compute nodes short name (see error: Failed when connecting to host ip-10-42-65-198 with error: [Errno -2] Name or service not known), hence it's unable to restart slurm on the compute nodes to complete the setup.
On the other side compute nodes are terminating themselves after a certain timeout since they are not attached correctly to the scheduler.

I think the missing ec2.internal in /etc/resolv.conf is causing this problem. And the search domain in /etc/resolv.conf are managed through the domain name DHCP Options Set config

In addition to installing unbound as referenced in #1192 , I added to the bottom of the preinstall script, the script to add ec2.internal as referenced in #597 , and now my master node can recognize and send jobs to the compute nodes. I will test further if anything is not working, but it looks functional, I will keep you posted!

@jcpasion I think we're trying to build something similar. Do keep us posted here :)

My cluster was working fine all throughout yesterday. However, my network has a protocol that shuts down all EC2 instance outside of work hours. My Master and Compute Nodes were shut down, and upon wake up, they failed to connect as they did before, resulting in the repeated terminating and creation of new compute nodes. Maybe this issue is related to the timeout issues of not being connected to the scheduler again.

Generally speaking if you want to terminate the master node of your cluster you should have a graceful shutdown where you stop the cluster to terminate all compute nodes and have them removed from the master node and then you stop the master node, otherwise you risk to get into an inconsistent state.
Regarding the issue you are facing, are you sure the /etc/resolv.conf didn't get restored to its initial value after the restart? I'm afraid something got reverted with the restart and the master cannot resolve the compute nodes anymore, hence those are self terminating periodically.


[bug] metamask does not open when running from chrome extension

Is there an existing issue for this?

[X] I have searched the existing issues

RainbowKit Version
1.0.0
wagmi Version
1.0.5
Current Behavior
Maybe this isnt possible cause of security reasons, idk, however when trying to open the metamask wallet from within a custom page or new tab of a chrome extension it doesnt work. nothing loads, just the QR code sits there.
Maybe because its not a normal url its hosted on?  I'm happy to put together a chrome extension dummy app to reproduce if needed. Thinking it may be a simple answer though like its just not possible.
screencast-bpconcjcammlapcogcnnelfmaeghhagj-2023.05.19-21_02_23.webm
Expected Behavior
the popup should open
Steps To Reproduce
run rainbowkit from within chrome extension
Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
No response

This is unfortunately a Chrome limitation. An extension content script or inpage script (which hosts the MetaMask window injector) cannot run in the context of another extension. You can test this by looking for a successful injection of window.ethereum; it will be unset in these contexts. You'll want to instead kick the user to a web URL where window.ethereum will be properly set by the wallet.


Add new should have a text file or csv file option

R parser would need to know that {r ...} is a special block/syntax construct, Probably attached to AST as a separate data type since it is not a regular language syntax and then completion source could use it.

@MikhailArkhipov i think above comment is on the wrong issue.

Right. About this - don't you get CSS/Text/XML/XSLT in Add New Item? In File | New they are in General

You are right. I get CSS/Text/XML/XLT. I think we can close this.


[Question] Why does using multiple gpus lead to slower performance?

When I set tensor_parallel_size to > 1, the wall time increases though everything else is down. Am I doing something wrong in my setup where using multiple gpus is actually slower than using one?
vllm = LLM(
    model="mosaicml/mpt-7b-instruct",
    trust_remote_code=True,
    dtype="float16",
    tensor_parallel_size=1,
    gpu_memory_utilization=.95,
)

CPU times: user 3.66 s, sys: 262 ms, total: 3.93 s
Wall time: 1.11 s

vllm = LLM(
    model="mosaicml/mpt-7b-instruct",
    trust_remote_code=True,
    dtype="float16",
    tensor_parallel_size=2,
    gpu_memory_utilization=.95,
)

CPU times: user 65.5 ms, sys: 32.2 ms, total: 97.7 ms
Wall time: 1.27 s

I'm facing the same issue. Seems like there isn't much discussion going on so I'm going to link a bunch of issues together to hopefully get the ball rolling.
https://github.com/vllm-project/vllm/issues/689
https://github.com/vllm-project/vllm/issues/935
https://github.com/vllm-project/vllm/issues/732
https://github.com/vllm-project/vllm/issues/367

I also got the same problem !

The cost of inter-GPU communication outweighs the parallelisation of the compute for smaller models

There has also been significant reduction in the TP overhead since this issue was opened. Would suggest trying the same comparison with the latest version, and also with https://github.com/vllm-project/vllm/pull/3466.

i also notice this problem. Use one card i take 15 mins a round. While use 4 card, my time consumption increases 30%.


Easy One - Avoid Exception when AppSettings File is missing

Here's a quick code change that I'd like to submit to Stride codebase -- just a few lines of code, to eliminate a pointless FileNotFound Exception at startup, from calling "YamlSerializer.Load".
Here's the code "as is now" from AppSettingsProvider:

Here's the code after I edited it:

What I did matches how it's done from the only other place in code that we call this method, where the File.Exists is checked beforehand to avoid using Exception for Program Flow:

So this just requires a few lines of code to change, and poof, it behaves the SAME but avoid throwing an exception.

It behaves the same for the vast majority of cases but it's not correct as the file could be moved between the Exists test and the read inside of Load. Yes it is very unlikely to ever happen but this is also not a bug, so if changes have to be made they must either improve the program or at the very least can't introduce a potential bug, you'll have to find another way.

OK, then I'd like to modify my request to simply insert the "File.Exists()" call to prevent exceptions from being "normal program flow".   Then the exception would only happen that 1 in a billion times where the File was deleted inside this 100 nanosecond window....
So this provides 100% same safety, while NOT using Exceptions for normal program flow.

LGTM and that exception is a nuisance.  I'd say go ahead and make a PR for the change.


add: task-specific environment variables

History

pdm

https://pdm-project.org/latest/usage/scripts/#env
https://pdm-project.org/latest/usage/scripts/#env_file

rye

https://rye.astral.sh/guide/pyproject/#env
https://rye.astral.sh/guide/pyproject/#env-file

We can also support command-line arguments:

-e VAR=value, --env VAR=value
--env-file PATH

May we should also do --dotenv to just load a nearby .env file?
But this could just as easily be solved with --env-file, so probably not.


Handle "default" routes

In ASP.NET, there were default documents that would be queried for /. An example of this is:
        <defaultDocument enabled="true">
            <files>
                <add value="Default.htm" />
                <add value="Default.asp" />
                <add value="index.htm" />
                <add value="index.html" />
                <add value="iisstart.htm" />
                <add value="default.aspx" />
            </files>
        </defaultDocument>

This can be setup similarly as:
...

var app = builder.Build();

app.Services.GetRequiredService<IHostApplicationLifetime>().ApplicationStarted.Register(() =>
{
    RouteTable.Routes.MapPageRoute("MainPage", "/", "~/Default.aspx");
});

However, it would be better add some sort of helper method for this. A proposal would be:
public static class RoutingServiceExtensions
{
     public static ISystemWebAdapterBuilder AddRouting(this ISystemWebAdapterBuilder builder);
+    public static ISystemWebAdapterBuilder AddRouting(this ISystemWebAdapterBuilder builder, Action<RouteCollection> configure);
+    public static ISystemWebAdapterBuilder AddRouting(this ISystemWebAdapterBuilder builder, string defaultRoute);
}

where this new API would be called as early as possible and called to set up configured routes.

There's a built in method to do this:
app.UseDefaultFiles("/Default.aspx");

Actually, this only ends up serving static content, so it won't work for our needs


Is discovery compatible to discovery-swarm?

Can a client using discovery find all clients using discovery-swarm?

I'd be curious about this as well.

I'd be curious about this as well.

The two are not compatible no

The two are not compatible no


Add minimum size validation for header parameter

Throw exception on values that are less than the minimum acceptable size of 32 bytes.

Can someone review?


Problem with selected data

Dear,
I got a problem when loaded data. The ng-select can not set default value. But other can be set default value.
All of ng-select i used same code. But only the last one get problem.
Thank for your support.

@khoapossible Can you make plker(Demo) for that issue?

Closing this issue because of lacking response. @khoapossible please let me know if you still need help.


Replace folder_id argument of upload document with the document object

This lets us pass arbitrary options to the CreateDocument call in Landax, including folder id. The argument is optional as well, since Landax doesn't require it.

Have you testet uploading a document without folder? It is a criteria in Landax that all documents needs to be in a folder, so if you dont pass a folder id, I wonder where it goes or if it is uploaded at all.

So why not make folder id a required parameter, I see this as the most relevant user case for Landax use.


Quick load @export custom resource shows no result

Godot version
4.0-beta10_win64
System information
Windows 10, RTX 3050 Laptop GPU
Issue description
Quick loading not working with custom resources.

Steps to reproduce
See images above.
Minimal reproduction project
MRP.zip

Duplicate of #67119


fix: make index and error documents configurable

This should resolve https://github.com/jariz/gatsby-plugin-s3/issues/365

Sorry, I accidentally closed this PR. I was trying to get update the branch.
Could you please re-create this PR?
I'm sorry for the long delay in reviewing this, I'll review it as a priority once you re-open it. Thanks!

Hi @JoshuaWalsh please see https://github.com/JoshuaWalsh/gatsby-plugin-s3/pull/454


webview

I want to add events to the webview loaded by html to send communications,but how do I get the elements in webview?

GitHub issues are for feature requests and bug reports, questions about using Electron should be directed to the community or to the Slack Channel.


The creation time of the tidb_cdc.syncpoint_v1 table of the secondary cluster does not meet expectations

What did you do?

Set the syncpoint interval to 5 minutes, and complete the creation of changefeeds for three clusters at 12:32:30
Primary cluster: tidb_cdc.syncpoint_v1 was created at 04:35:00
Secondary cluster: created at 2024-07-02 04:45:22, and it takes 13 minutes from create changefeed to tidb_cdc.syncpoint_v1
This delay is greater than 2 syncpoint intervals
The ticdc config is as follows
bdr-mode = true
force-replicate = true
enable-sync-point = true
sync-point-interval = "5m"

What did you expect to see?
The creation time of the tidb_cdc.syncpoint_v1 table of the secondary cluster within 1~2 syncpoint intervals
What did you see instead?

more than 2 syncpoint intervals
the changefeed status is warning
the changefeed lag is very large

Versions of the cluster
ticdc git_hash 2db092244d18c043f7f8ecc85a598680b33cc324
tidb git_hash 585c631742acb338c9bd0913a89efc3efb5b884a
current status of DM cluster (execute query-status <task-name> in dmctl)
No response

/assign @CharlesCheung96

/severity major

/area ticdc

/remove-area dm

/label affects-8.2.0

/label affects-8.2

[2024/07/02 04:40:08.276 +00:00] [ERROR] [changefeed.go:325] ["an error occurred in Owner"] [namespace=default] [changefeed=cluster-a-cluster-b] [error="[CDC:ErrMySQLTxnError]MySQL txn error: failed to create syncpoint table;: Error 8263 (HY000): The operation is not allowed while the bdr role of this cluster is set to secondary."]

The root cause is that ticdc did not set write source when creating syncpoint table.


(MAINT) Extract puppet server config file name

This commits moves the name of the config file to options.rb so we can
set it to the PE-specific name in pe-puppet-server-extensions.

Once this is merged I'll need to do a voom build of puppet-server and then upgrade the dependency in pe-p-s-e

Looks good to me...


Wik feature morphing v2

PR Checklist
If you are contributing to Javis.jl, please make sure you are able to check off each item on this list:

[x] Did I update CHANGELOG.md with whatever changes/features I added with this PR?
[x] Did I update the TODO.md (if applicable)?
[x] Did I make sure to only change the part of the file where I introduced a new change/feature?
[x] Did I cover all corner cases to be close to 100% test coverage (if applicable)?
[x] Did I properly add Javis dependencies to the Project.toml + set an upper bound of the dependency (if applicable)?
[x] Did I properly add test dependencies to the test directory (if applicable)?
[ ] Did I check relevant tutorials that may be affected by changes in this PR?
[x] Did I clearly articulate why this PR was made the way it was and how it was made?

Link to relevant issue(s)
Closes #80
How did you address these issues with this PR? What methods did you use?
It's now possible to morph a shape into a different one with having fill for example with:
Action(
            26:125,
            morph(astar, acirc; action = :fill);
            subactions = [SubAction(1:100, color_anim, sethue())],
        ),

where the color is set with a SubAction. Needs #153
Example:

Haha next time you should use other polygons than I do :stuck_out_tongue_winking_eye:


Formatting of description field in META file is a bit odd sometimes

  package "options" (
    directory = "options"
    version = "dev"
    description = "Options
                   for
                   Lwt
                   Camlp4
                   syntax
                   extension
                   (deprecated;
                   use
                   lwt.ppx)"

Indeed, I added a heuristic to only reformat certain variables


Why does the accuracy of "gpt-3.5-turbo" fall in GSM8K ?

Hi, I found that the accuracy of gpt-3.5-turbo in GSM8K was 78.9 in and before commit f007faf, but it decreased to 74.9 after commit c420693.
What fators lead to this accuracy shift?
After checking the code in gpt3.5turbo_gsm8k_complex.ipynb, I noticed 74.9 corresponds to the accuracy of Complex Prompt, Dialog In-Context Learning while 78.9 corresponds to the accuracy of Complex Prompt Greedy Decoding.  Is this the problem ?

Yep 78.9 is after prompt engineering using complex prompt while 74.9 is from Baseline Prompt which is a more fair comparison.
Cheers


Build Krew archive

Unlike in 0.6+, this change doesn't attempt to build the manifest as a
part of Cloud Build. That can be done by hand after the release; too
much has changed in the makefiles between the two versions. So this just
focuses on creating the archive.
Tested: this is virtually impossible to test given the state of the
build process in v0.5 (it's much better in v0.6). I'll revert the
changes to cloudbuild.yaml if I can't build a release candidate, and the
changes to the hack/krew*.yaml files don't matter to anyone.

/assign @yiqigao217
/assign @rjbez17
/cc @GinnyJI
/cc @ahmetb

/lgtm


Idea: add demo folder

A demos folder should be added in the repo or in a related repo.
The code for the demos listed in the following url unfortunately have their javascript minified, and since the code is not posted elsewhere, this isn't particularly friendly to newcomers.
http://binarymuse.github.io/ngInfiniteScroll/demos.html

You're totally right ! I won't be able to do that before few weeks, so PR are welcome :)


feat: add Pulumi Aquasec provider to Pulumi Registry

Hi,
Thi PR adds the Pulumiverse Pulumi Aquasec provider to the Pulumi Registry.
FYI @ringods

Waiting to merge, for the release of the provider https://github.com/pulumiverse/pulumi-aquasec

Hi @ringods, this should be fine to merge. The release of the aquasec provider went through

@dirien could you rebase this PR?

@ringods done 😄


Include module without electron cause error

Including the module without Electron cause error. Can be useful including it  in a "progressive web app" where some build haven't electron code. Example web, android, ios, desktop with electron.
It can be solved creating a empty object electronRequire

Your MR worked for me.  Thanks @gimox

👍

Remember to check if ipcRenderer has electron object doing something like:
// es6 syntax

if (angular.isFunction(this.ipcRenderer.on)) {
      this.ipcRenderer.on('hoto:home', (event, arg) => {
            this.$state.go('main.home');
      });


[5.5] Schedule && Session store not set on request

Laravel Version: 5.5.*
PHP Version: 7.2
Database Driver & Version: none

Description:
When I used timed task to send email and make a mistake(mail use laravel5.5):
In Request.php line 419:
Session store not set on request.

$schedule->call(function () {

            $service = new PackageService();
            $service->dailyEmails();

        })->everyMinute();//->dailyAt('08:00');

public function dailyEmails()
    {
        $toemail =<EMAIL_ADDRESS>
        $subject = 'test';

        $data = [];

        $this->sendmail($toemail, $subject, $data, 'Admin.work.mailtpl.packagedaliy');

        //return view('Admin.work.mailtpl.packagedaliy', compact('data'));
    }

 /**
     * @param $tomail
     * @param array $data
     * @param string $subject
     * @param string $template
     */
    protected function sendmail($tomail, $subject = '', $data = [], $template = 'Admin.work.mailtpl.common')
    {
        Mail::send($template, ['data' => $data], function ($mail) use ($tomail, $subject) {
            if (is_array($tomail) && count($tomail) >= 2) {
                $tomails = collect($tomail);
                $mail->to($tomails->first())->cc($tomails->forget(0)->toArray())->subject($subject);
            } else {
                $mail->to($tomail)->subject($subject);
            }
        });
    }

Steps To Reproduce:
artisan
php artisan schedule:run

I don't see what issue you're trying to report. Could you expand on how you reproduce the problem, and perhaps add a full stack trace?

It's not clear what you are asking. Please open a new ticket with a clear description of the problem, and how to reproduce it.


Add test case for namespaced exception catching

@st0012 how about suggestions?

@ko1 @ono-max sorry that I missed the suggestions. have applied them.


start removing unwanted nullish coalescisors - ?. ?? etc.

Version
@nuxtjs/i18n:  7.2.0
nuxt:  2.15.8
typescript - 4.5.4
-- any downgrade is not expected.
Nuxt configuration
Please change to [x] if relevant for this issue:

[x] Applies to a site deployed to a static server (site generated with nuxt generate)
[x] Applies to a site deployed to a server with a Node backend

@nuxtjs/i18n configuration

Reproduction Link

https://typescript.nuxtjs.org/guide/setup/
Steps to reproduce
See:

What is Expected?
Grace end of build and perfect execution.
What is actually happening?
Unsuccessful build of our core modules is a showstopper in the middle og our project schedule.

The version of much that you are using is configured to handle that syntax just fine.
It sounds like you have customized nuxt in a way that breaks support for that.
It's not the module's issue.

Also, did you see the info box within the page that you've linked?

@rchl
Please define "customization" of your own. Additionally, do explain why on the earth you think there is an engineer who may not have sufficient knowledge of specific softwaer which he/she is going to implement/deploy never read its documentation prior to be in the house.
I appreciate your feedback but, for now, it is too hard to understand for me as a non-English speaker

Please provide a reproduction repo. It doesn't reproduce for me with a fresh project created using nuxt-app preset (as described in https://nuxtjs.org/docs/get-started/installation and with typescript enabled).


tooltip texts seems to be cut off at the top in setup your dojo form

Technical spec:

You'll be looking for anything using tooltip-html-unsafe in this file.
The CSS file is here

@tangentfairy appears to be already fixed!?!

@odowdj I think someone fixed this at the last hackathon when they spotted it but didn't close the issue. Thanks :)


Add copy to clipboard external address

Codecov Report

Merging #2183 (3bcc118) into master (cef1c24) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 3bcc118 differs from pull request most recent head 036c59c. Consider uploading reports for the commit 036c59c to get more accurate results

@@           Coverage Diff           @@
##           master    #2183   +/-   ##
=======================================
  Coverage   61.86%   61.86%
=======================================
  Files          79       79
  Lines        3852     3852
  Branches      247      247
=======================================
  Hits         2383     2383
  Misses       1450     1450
  Partials       19       19

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 cef1c24...036c59c. Read the comment docs.


Restore all the way back to original c524xlarge as m58xlarge just not stable on Windows

Description
Restore all the way back to original c524xlarge as m58xlarge just not stable on Windows.
Issues Resolved
https://github.com/opensearch-project/opensearch-build/issues/3816
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Close now as it actually connects after nearly 20min.

Re-open as the connection is not stable.

https://build.ci.opensearch.org/job/packer-build/83/console


TextLine.firstNonWhitespaceCharacterIndex

Definition is "The offset of the first character which is not a whitespace character as defined by /\s/"
It may now be too late to change, but I found it weird that for empty or whitespace-only lines, 0 the length of the string is returned. 0 implies that there is a non-whitespace character at position 0, whereas in fact there is no non-whitespace character. Therefore I'd expect the value returned in this case to be -1 or something.
If it has to stay that way then there should really be a note in the docs I guess.
Edit: I had it slightly wrong first time: for empty lines, it returns 0, but for lines with whitespace chars it returns the length of the string. Sure I can see the logic, though I still think it's not the value people would expect (and leadingWhitespaceLength would then have been a more appropriate name).

great find.

It's a pity we cannot make a proper change unless we break the API. For now, adding an explicit comment...

Verified comment.


Refactor: clean trainer device & distrib setters

What does this PR do?

Fixes # (issue) <- this links related issue to this PR
Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes [if needed]?
[x] Did you write any new necessary tests [no need for typos, docs]?
[x] Did you verify new and existing tests pass locally with your changes?
[x] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified
[x] Check that target branch and milestone are aligned!

Did you have fun?
Make sure you had fun coding 🙃

Hello @Borda! Thanks for updating this PR.

In the file pytorch_lightning/accelerators/accelerator_connector.py:

Line 315:17: W503 line break before binary operator
Line 320:17: W503 line break before binary operator
Line 332:17: W503 line break before binary operator

@tchaton @awaelchli just thinking is this valid configuration ('ddp_cpu', None, None) for ddp_backend, gpus, num_processes?

As a user input? No, I don't think so. When requesting ddp_cpu explicitly, we would need to know how many processes. Also, be careful with the changes here, it needs to match the previous behavior exactly. The order of the if-elif blocks is important.

As a user input? No, I don't think so. When requesting ddp_cpu explicitly, we would need to know how many processes. Also, be careful with the changes here, it needs to match the previous behavior exactly. The order of the if-elif blocks is important.

As a user input? No, I don't think so. When requesting ddp_cpu explicitly, we would need to know how many processes.

so it seems we had a bug there... :D
I have added if num_processes is None then sent nb CPU available

Also, be careful with the changes here, it needs to match the previous behavior exactly. The order of the if-elif blocks is important.

yes, that is why I change only setting and not reading in this PR and no tests shall be changed either...
the reading change is in #5300

As a user input? No, I don't think so. When requesting ddp_cpu explicitly, we would need to know how many processes.

so it seems we had a bug there... :D
I have added if num_processes is None then sent nb CPU available

Also, be careful with the changes here, it needs to match the previous behavior exactly. The order of the if-elif blocks is important.

yes, that is why I change only setting and not reading in this PR and no tests shall be changed either...
the reading change is in #5300

@tchaton how are these plugins selected?

@tchaton how are these plugins selected?

@SeanNaren mind check why the parity params are different for ddp_cpu?
# Assert model parameters are identical after fit
        for ddp_param, custom_param in zip(ddp_model.parameters(), custom_plugin_model.parameters()):
>           assert torch.equal(ddp_param, custom_param), 'Model parameters are different between DDP and Custom plugin'
E           AssertionError: Model parameters are different between DDP and Custom plugin
E           assert False
E            +  where False = <built-in method equal of type object at 0x11d2d95b0>(Parameter containing:\ntensor([[ 0.1203,  0.0808, -0.0999,  0.1504, -0.1179,  0.0486, -0.1525,  0.1665,\n          0.207... -0.0007,\n         -0.0356, -0.2548,  0.0780, -0.1915, -0.1204, -0.1929,  0.1851, -0.1996]],\n       requires_grad=True), Parameter containing:\ntensor([[ 0.1277,  0.1138, -0.0707,  0.1564, -0.0783,  0.0421, -0.1193,  0.1352,\n          0.181...  0.0362,\n          0.0102, -0.1289,  0.1082, -0.1586, -0.0546, -0.1568,  0.1198, -0.1302]],\n       requires_grad=True))
E            +    where <built-in method equal of type object at 0x11d2d95b0> = torch.equal

@SeanNaren mind check why the parity params are different for ddp_cpu?
# Assert model parameters are identical after fit
        for ddp_param, custom_param in zip(ddp_model.parameters(), custom_plugin_model.parameters()):
>           assert torch.equal(ddp_param, custom_param), 'Model parameters are different between DDP and Custom plugin'
E           AssertionError: Model parameters are different between DDP and Custom plugin
E           assert False
E            +  where False = <built-in method equal of type object at 0x11d2d95b0>(Parameter containing:\ntensor([[ 0.1203,  0.0808, -0.0999,  0.1504, -0.1179,  0.0486, -0.1525,  0.1665,\n          0.207... -0.0007,\n         -0.0356, -0.2548,  0.0780, -0.1915, -0.1204, -0.1929,  0.1851, -0.1996]],\n       requires_grad=True), Parameter containing:\ntensor([[ 0.1277,  0.1138, -0.0707,  0.1564, -0.0783,  0.0421, -0.1193,  0.1352,\n          0.181...  0.0362,\n          0.0102, -0.1289,  0.1082, -0.1586, -0.0546, -0.1568,  0.1198, -0.1302]],\n       requires_grad=True))
E            +    where <built-in method equal of type object at 0x11d2d95b0> = torch.equal

So ddp_cpu can now only work if we set the number of processes above 1, and if specified automatically allocate all CPU resources. I think this is fine, just ensuring there is no case where we'd like to keep the current behaviour!

happy to verify, but no idea when you do not want to use max resources...?

So ddp_cpu can now only work if we set the number of processes above 1, and if specified automatically allocate all CPU resources. I think this is fine, just ensuring there is no case where we'd like to keep the current behaviour!

happy to verify, but no idea when you do not want to use max resources...?


Dataset-wide _FillValue

I'm looking at a netCDF file that has the variable
	float T_20(time, depth, lat, lon) ;
		T_20:name = "T" ;
		T_20:long_name = "TEMPERATURE (C)" ;
		T_20:generic_name = "temp" ;
		T_20:FORTRAN_format = "f10.2" ;
		T_20:units = "C" ;
		T_20:epic_code = 20 ;

and global attributes
// global attributes:
		:platform_code = "8n90e" ;
		:site_code = "8n90e" ;
		:wmo_platform_code = 23007 ;
		:array = "RAMA" ;
		:Request_for_acknowledgement = "If you use these data in publications or presentations, please acknowledge the GTMBA Project Office of NOAA/PMEL. Also, we would appreciate receiving a preprint and/or reprint of publications utilizing the data for inclusion in our bibliography. Relevant publications should be sent to: GTMBA Project Office, NOAA/Pacific Marine Environmental Laboratory, 7600 Sand Point Way NE, Seattle, WA 98115" ;
		:Data_Source = "Global Tropical Moored Buoy Array Project Office/NOAA/PMEL" ;
		:File_info = "Contact<EMAIL_ADDRESS>;
		:missing_value = 1.e+35f ;
		:_FillValue = 1.e+35f ;
		:CREATION_DATE = "13:05 28-JUL-2017" ;
		:_Format = "classic" ;

Problem description
In this case the _FillValue and missing_value attributes are set for the entire dataset and not each individual variable. decode_cf_variable thus fails to insert NaNs.
I'm not sure that this is standards-compliant but is this something we could support?

I'm pretty sure this isn't CF conventions compliant :). We could definitely do this in theory but I'm not sure it's worth the trouble. You could also fix this in a preprocessing step (i.e., by setting decode_cf=False, manually copying the attributes and then using xarray.decode_cf()).

That's an easy enough solution. Thanks.

AFAIK, storing the fill value in the global attributes does not follow any widely adopted convention (e.g. CF). Certainly possible to support but since we decode one variable at a time, I'm not sure our current approach would work. I think we'd need to know if there is a convention for this or if this particular dataset is just improperly formatted.

Hmmm, my notifications are coming though with quite the lag today. I just saw @shoyer's comment. Sounds like we're all on the same page.


CIP Slides

I hope so! It was an experiment to get the slides into the CIP. If you can resolve/archive them to clean things up, by all means.

Great okay thanks


Correction for raw affiliation Hybrid Team, University of Rennes, Irisa, UMR CNRS,Rennes,France,6074

Correction needed for raw affiliation Hybrid Team, University of Rennes, Irisa, UMR CNRS,Rennes,France,6074
raw_affiliation_name: Hybrid Team, University of Rennes, Irisa, UMR CNRS,Rennes,France,6074
new_rors: 015m7wh34;02feahw73;00myn0z94
previous_rors: 015m7wh34;02feahw73
works_examples: W4391331237
contact: efbd8dd19ae3cfbee7df552f573cacec:b3a45a1895ea3f41fcd42c672879236d @ univ-rennes.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


Fixed auth local dir variable

Is the nginx_auth_permissions_local_path needed now? Can it be removed?
LGTM otherwise.


Saving videos on latest twitter version doesn’t work

When trying to save videos on latest version of twitter 7.49 it doesn’t let you download video it doesn’t show the quality section on which one to download it just says videodownloadercn and above that there is no options to choose from the resolution and download

fixed in version 1.2.1.
just update


fix: Added zia_url_filtering_rule new attribute source_ip_groups and SDK upgrade to v2.5.0

fix: Added zia_url_filtering_rule new attribute source_ip_groups and SDK upgrade to v2.5.0

Codecov Report
Attention: Patch coverage is 33.33333% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 72.08%. Comparing base (bf8af44) to head (2bef59c).

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #340      +/-   ##
==========================================
- Coverage   72.46%   72.08%   -0.38%
==========================================
  Files          79       79
  Lines       12494    12500       +6
==========================================
- Hits         9054     9011      -43
- Misses       2833     2861      +28
- Partials      607      628      +21

Files
Coverage Δ

zia/resource_zia_dlp_web_rules.go
56.66% <33.33%> (-2.14%)
:arrow_down:

zia/resource_zia_url_filtering_rules.go
50.82% <33.33%> (-1.18%)
:arrow_down:

... and 16 files with indirect coverage changes


Station/popup

318 issue closed
Hello !!! @dhairyagothi sir , I have successfully added the load on website pop up . I hope you will like it.  i have added two files pop.css foe styling and pop.js under component and updated index.html. The load on website pop up will grab attention of new users Please merge a PR and if you can increase my level that would be highly appreciable.
Related Issue
load on website pop up added.
Description of Changes
New feature load on website popup added with the cross sign and a signup box.
Type of Pull Request

[ ] Bug Fix
[X] New Feature
[ ] Performance Improvement
[ ] Documentation Update
[ ] Code Refactoring
[ ] Security Enhancement
[ ] Other (Specify): _______________

(Please check relevant boxes by entering an "X" inside them)
Checklist for Contributor
use'X' in [ ] to check it .

[X] I have reviewed my code before submission.
[X] I have read and followed the project's Contribution Guidelines.
[X] I have tested the code thoroughly.
[X] I have attached related issue numbers, screenshots, or videos (if applicable).
[X] I have added comments to my code where necessary for clarity.
[X] I have checked the existing code base and my changes do not introduce any breaking changes.
[X] My PR addresses only one issue; I have created separate PRs for other issues.
[X] I have ensured that the PR does not include unrelated files.
[X] I am mindful of the project's coding standards and best practices.

Thank you
Dakshata
Gssoc'24 Extnd , Hacktoberfest

Thanks for contributing @daky2024

My pleasure @dhairyagothi sir.


partial bump of cda

Refactor and change behavior when reconnecting with password based login

/test e2e-dpu-operator-tests


Add fsgcl.jp

Resubmitted due to a name error.
website URL: https://www.nsg.gr.jp/list/006.html
IT related course example: https://wiz.ac.jp/course/info_technology/info_high/
ownership: https://wiz.ac.jp/tgs2021/portfolio/08/img/portfolio.pdf

@maaeda lib/domains/fsgcl.txt: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.


⚠ Bofrost.de has degraded performance

In 0fc92ab, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1543 ms

Resolved: Bofrost.de performance has improved in 380e2fe after 6 minutes.


Loss decreasing slowly while moving the GO flag out of ground truth.

I found that the GO flags have been remove while caculating the loss function, but why? In my experiments, the speed of convergency including the GO flags is faster.
May be a naive question, thanks.

I honestly don't remember if there was a particular reason behind it, but please do try to experiment, and if you see better speed with the same result, submit a PR. :)


SmokeTests not catching missing template nuget packages in the SDK archive

/templates folder from the produced sdk archive in dotnet/sdk's sdk-source-build leg:

LKG source-build /templates folder:

The smoke tests ran but didn't fail.
Live built SDK = download link
LKG = dotnet-sdk-9.0.100-preview.5.24257.1-centos.9-x64.tar.gz (https://dev.azure.com/dnceng/internal/_build/results?buildId=2446449&view=results)

In the smoke tests ran link, I see a number of failing SmokeTests in the Tests view.  @ViktorHofer, What are you expecting?

I was thinking about the SDK diff test that indicates when an asset is missing / different from the msft SDK. I think that's part of the smoke tests test library. That one didn't fail in the PR.

The SDK diff tests don't run in PR validation because files are changing too frequently.  The test is very noisy and provides marginal value.

The SDK diff tests are broken right now with the installer-sdk repo consolidation, so I reference a failed job.

I see, ok that explains it then. Thanks.


formly-field directive apiCheck failed!

Hi,
While running my app i am getting following error from angularformly,
Error: angular-formly: formly-field directive apiCheck failed! value at Argument 1 cannot have extra properties: label, placeholder.It is limited to $$hashKey, type, template, templateUrl, key, model, className, expressionProperties, data, templateOptions, wrapper, modelOptions, watcher, validators, noFormControl, hide, hideExpression, ngModelAttrs, elementAttributes, optionsTypes, link, controller, validation, formControl, value, runExpressions, resetModel, updateInitialValue, initialValue, defaultValue  formly-field-directive-validation-failed
I have following order:

And in my app:
angular.module('myApp', [ 'formly'])
Do you know how i can resolve this issues?

It's because you did something like this:
{
  label: 'some label',
  placeholder: 'some placeholder'
  // more configuration options
}

When it should have been something like this:
{
  // some configuration options
  templateOptions: {
    label: 'some label',
    placeholder: 'some placeholder'
  }
}

Change that and try again. If it doesn't fix the problem. Re-open this issue.
Also, I highly recommend you go through all the resources available on the Learn angular-formly page.

This is what we have done before while formly came, and that was working completely.
https://github.com/CA-Labs/fire-survey/blob/master/forms/survey.json
So, with your explanation i undrestood we should add templateOptions for all the parts, am i right?
Another problem about type:text..which now is undefined.. what is the replacement for type 'text'?

We have implemented before this app with angularformly which was working correctly.
https://github.com/CA-Labs/fire-survey/blob/master/forms/survey.json
Now with your examples i understood i should add "templateOptions" for all cases. It seems is working except for custom templates such as "type": "multi-checkbox", etc.
To solve this problem i have added
.run(function(formlyConfig) {     formlyConfig.setType(      {       name: 'multi-checkbox',       templateUrl: './polls/partials/multi-checkbox-template.html'     })})
and i have changed Q2 to:
{             "key": "Q2",             "templateOptions": {               "label": "In what languages does your firm provide live chat support?",               "options": [                 {                   "name": "english",                   "value": "eng"                 },                 {                   "name": "french",                   "value": "frnch"                 },                 {                   "name": "more",                   "value": "more"                 }               ]             },             "type": "multi-checkbox"           }
and i don't get any output for that. How i can fix it?
Thank you for your reply in advance.

Please spend some time watching the lessons on the page I linked to you and
looking at the examples. They will answer most of your questions.

Kent C. Dodds

(Sent from my mobile device, please forgive typos or brevity)
On May 24, 2015 6:10 AM, "maryam"<EMAIL_ADDRESS>wrote:

We have implemented before this app with angularformly which was working
correctly.
https://github.com/CA-Labs/fire-survey/blob/master/forms/survey.json
Now with your examples i understood i should add "templateOptions" for all
cases. It seems is working except for custom templates such as "type":
"multi-checkbox", etc.
To solve this problem i have added
.run(function(formlyConfig) {
formlyConfig.setType(
{
name: 'multi-checkbox',
templateUrl: './polls/partials/multi-checkbox-template.html'
})})
and i have changed Q2 to:
{
"key": "Q2",
"templateOptions": {
"label": "In what languages does your firm provide live chat support?",
"options": [
{
"name": "english",
"value": "eng"
},
{
"name": "french",
"value": "frnch"
},
{
"name": "more",
"value": "more"
}
]
},
"type": "multi-checkbox"
}
and i don't get any output for that. How i can fix it?
Thank you for your reply in advance.
—
Reply to this email directly or view it on GitHub
https://github.com/formly-js/angular-formly/issues/313#issuecomment-105007463
.

I've followed your link completely. I listened to your videos as well. I found one simple example for custom template and i tried to generalized that with my example as i presented to you in above.
Also i changed it into "formlyConfigProvider", in any case with what i presented to you i can not have any output from custom html. Moreover i want to emphasize that before we have done this project correctly and it was working. Please give me hint if there is any problem in above comment.

I've followed your link completely. I listened to your videos as well. I found one simple example for custom template and i tried to generalized that with my example as i presented to you in above.
Also i changed it into "formlyConfigProvider", in any case with what i presented to you i can not have expected output from custom html. Also i have added wrapper
wrapper: ['bootstrapLabel', 'bootstrapHasError'], the only change is showing the question without any options.
Moreover i want to emphasize that before we have done this project correctly and it was working. Please give me hint if there is any problem in above comment.

Sorry, I don't have time to help right now, but I'm sure that if you asked
this on stack overflow someone could help you there.

Kent C. Dodds

(Sent from my mobile device, please forgive typos or brevity)
On May 24, 2015 10:26 AM, "maryam"<EMAIL_ADDRESS>wrote:

I've followed your link completely. I listened to your videos as well. I
found one simple example for custom template and i tried to generalized
that with my example as i presented to you in above.
Also i changed it into "formlyConfigProvider", in any case with what i
presented to you i can not have expected output from custom html. Also i
have added wrapper
wrapper: ['bootstrapLabel', 'bootstrapHasError'], the only change is
showing the question without any options.
Moreover i want to emphasize that before we have done this project
correctly and it was working. Please give me hint if there is any problem
in above comment.
—
Reply to this email directly or view it on GitHub
https://github.com/formly-js/angular-formly/issues/313#issuecomment-105033729
.

ok.thx


Compile with openssl fails

Hi,
just gave it a try. Maybe because this option defaults to off it never was tested (at least by me).
cd /home/werner/pupnp/upnp && /usr/bin/c++ -DDEBUG -DSTATS -DUPNP_STATIC_LIB -I/home/werner/VBox/pupnp/upnp/src/threadutil -I/home/werner/VBox/pupnp/upnp/inc -I/home/werner/VBox/pupnp/upnp/src/inc -I/home/werner/pupnp -I/home/werner/pupnp/upnp/inc -I/home/werner/VBox/pupnp/ixml/inc -I/home/werner/VBox/pupnp/ixml/src/inc -g -pthread -o CMakeFiles/upnp_static.dir/src/api/upnpapi.cpp.o -c /home/werner/VBox/pupnp/upnp/src/api/upnpapi.cpp
/home/werner/VBox/pupnp/upnp/src/api/upnpapi.cpp: In function ‘int UpnpInitSslContext(UpnpLib*, int, const SSL_METHOD*)’:
/home/werner/VBox/pupnp/upnp/src/api/upnpapi.cpp:443:14: error: invalid use of incomplete type ‘UpnpLib’ {aka ‘struct s_UpnpLib’}
  443 |         if (p->gSslCtx) {
      |              ^~
In file included from /home/werner/VBox/pupnp/upnp/src/inc/VirtualDir.h:4,
                 from /home/werner/VBox/pupnp/upnp/src/inc/upnpapi.h:40,
                 from /home/werner/VBox/pupnp/upnp/src/api/upnpapi.cpp:43:
/home/werner/VBox/pupnp/upnp/inc/upnp.h:409:16: note: forward declaration of ‘UpnpLib’ {aka ‘struct s_UpnpLib’}
  409 | typedef struct s_UpnpLib UpnpLib;
      |                ^~~~~~~~~
/home/werner/VBox/pupnp/upnp/src/api/upnpapi.cpp:451:10: error: invalid use of incomplete type ‘UpnpLib’ {aka ‘struct s_UpnpLib’}
  451 |         p->gSslCtx = SSL_CTX_new(sslMethod);
      |          ^~
In file included from /home/werner/VBox/pupnp/upnp/src/inc/VirtualDir.h:4,
                 from /home/werner/VBox/pupnp/upnp/src/inc/upnpapi.h:40,
                 from /home/werner/VBox/pupnp/upnp/src/api/upnpapi.cpp:43:
/home/werner/VBox/pupnp/upnp/inc/upnp.h:409:16: note: forward declaration of ‘UpnpLib’ {aka ‘struct s_UpnpLib’}
  409 | typedef struct s_UpnpLib UpnpLib;
      |                ^~~~~~~~~
/home/werner/VBox/pupnp/upnp/src/api/upnpapi.cpp:452:15: error: invalid use of incomplete type ‘UpnpLib’ {aka ‘struct s_UpnpLib’}
  452 |         if (!p->gSslCtx) {
      |               ^~
In file included from /home/werner/VBox/pupnp/upnp/src/inc/VirtualDir.h:4,
                 from /home/werner/VBox/pupnp/upnp/src/inc/upnpapi.h:40,
                 from /home/werner/VBox/pupnp/upnp/src/api/upnpapi.cpp:43:
/home/werner/VBox/pupnp/upnp/inc/upnp.h:409:16: note: forward declaration of ‘UpnpLib’ {aka ‘struct s_UpnpLib’}
  409 | typedef struct s_UpnpLib UpnpLib;
      |                ^~~~~~~~~
/home/werner/VBox/pupnp/upnp/src/api/upnpapi.cpp: In function ‘int UpnpFinish(UpnpLib*)’:
/home/werner/VBox/pupnp/upnp/src/api/upnpapi.cpp:526:13: error: ‘gSslCtx’ was not declared in this scope
  526 |         if (gSslCtx) {
      |             ^~~~~~~
make[2]: *** [upnp/CMakeFiles/upnp_static.dir/build.make:238: upnp/CMakeFiles/upnp_static.dir/src/api/upnpapi.cpp.o] Fehler 1
make[2]: Verzeichnis „/home/werner/pupnp“ wird verlassen
make[1]: *** [CMakeFiles/Makefile2:327: upnp/CMakeFiles/upnp_static.dir/all] Fehler 2
make[1]: Verzeichnis „/home/werner/pupnp“ wird verlassen
make: *** [Makefile:160: all] Fehler 2

Seems like spaces get swallowed.
The ^ points to -> in p lines and s_UpnpLib in the others.

If you wrap your code in triple backticks it should keep the whitespace:
```
your output here!
```

Tried that with the backticks on the first and last line. Now it's better

Please check now after #286 .

Didn't change.

You get the same error? The code you previously pasted no longer exists.

Maybe I pulled to early. Works now. Maybe we should just set all disabled option to on in the tests to get it all coverred.

You've got a point. Do a test and a PR.
I did a test called "OpenSSL build (ubuntu-20.04)", but that is autotools based, maybe you can do the same, cmake based and with all options enabled.
Also be sure to include the sanitizer flags, we currently have no CMake sanitizer build, that could be the one. Maybe a "SANITIZER" option in CMakeLists.txt.

I'm sure that works the same way, I just don't think I have time for it today. Will be done asap.

Closing because it seems to be compiling ok now.


More info at: https://travis-ci.com/Qiskit/qiskit-terra/jobs/322175298

    qiskit.providers.ibmq.api.exceptions.RequestsApiError: '501 Server Error: Not Implemented for url: [secure]/version'

Duplicated #4215