Investigate why two validation runs on the same model return slightly different statistics

validation with batch of size 1 - works perfectly
validation with batch of size 10 - from time to time returns values that differ
First I was thinking that the issue related to lack of weighted averaging when we are not dropping the last batch. Sadly, the issue remained even when dropping last batch/limiting size of set to batch.
To Reproduce
mip-offline-trainer --c configs/vision/simplecnn_mnist.yaml
Validation problem section:
validation:
problem:
name: MNIST
batch_size: 10
use_train_data: True
resize: [32, 32]
sampler:
name: SubsetRandomSampler
indices: [55000, 55010]
dataloader:
drop_last: True
================================================================================
[2018-11-08 12:36:07] - INFO - OfflineTrainer >>> Validating over the entire validation set (10 samples in 1 episodes)
[2018-11-08 12:36:07] - INFO - OfflineTrainer >>> episode 000858; episodes_aggregated 000001; loss 0.0016851807; loss_min 0.0016851807; loss_max 0.0016851807; loss_std 0.0000000000; epoch 00; acc 1.0000000000; acc_min 1.0000000000; acc_max 1.0000000000; acc_std 0.0000000000; samples_aggregated 000010 [Full Validation]
[2018-11-08 12:36:07] - INFO - Model >>> Model and statistics exported to checkpoint ./experiments/MNIST/SimpleConvNet/20181108_123547/models/model_best.pt
[2018-11-08 12:36:07] - INFO - OfflineTrainer >>>
[2018-11-08 12:36:07] - INFO - OfflineTrainer >>> Training finished because Epoch limit reached
[2018-11-08 12:36:07] - INFO - OfflineTrainer >>> Validating over the entire validation set (10 samples in 1 episodes)
[2018-11-08 12:36:07] - INFO - OfflineTrainer >>> episode 000858; episodes_aggregated 000001; loss 0.0016851809; loss_min 0.0016851809; loss_max 0.0016851809; loss_std 0.0000000000; epoch 00; acc 1.0000000000; acc_min 1.0000000000; acc_max 1.0000000000; acc_std 0.0000000000; samples_aggregated 000010 [Full Validation]

validation set: 10 OK
================================================================================
[2018-11-08 12:55:28] - INFO - OfflineTrainer >>> Starting next epoch: 0
[2018-11-08 12:55:28] - INFO - OfflineTrainer >>> loss 2.3090281487; episode 000000; epoch 00; acc 0.0312500000; batch_size 000064
[2018-11-08 12:55:30] - INFO - OfflineTrainer >>> episode 000084; episodes_aggregated 000085; loss 0.6594894528; loss_min 0.1157590970; loss_max 2.3090281487; loss_std 0.5856138468; epoch 00; acc 0.7722426653; acc_min 0.0312500000; acc_max 0.9687500000; acc_std 0.2274699211; samples_aggregated 005440 [Epoch 0]
[2018-11-08 12:55:30] - INFO - OfflineTrainer >>> Validating over the entire validation set (10 samples in 1 episodes)
[2018-11-08 12:55:30] - INFO - OfflineTrainer >>> episode 000084; episodes_aggregated 000001; loss 0.0505415723; loss_min 0.0505415723; loss_max 0.0505415723; loss_std 0.0000000000; epoch 00; acc 1.0000000000; acc_min 1.0000000000; acc_max 1.0000000000; acc_std 0.0000000000; samples_aggregated 000010 [Full Validation]
[2018-11-08 12:55:30] - INFO - Model >>> Model and statistics exported to checkpoint ./experiments/MNIST/SimpleConvNet/20181108_125528/models/model_best.pt
[2018-11-08 12:55:30] - INFO - OfflineTrainer >>>
[2018-11-08 12:55:30] - INFO - OfflineTrainer >>> Training finished because Epoch limit reached
[2018-11-08 12:55:30] - INFO - OfflineTrainer >>> Validating over the entire validation set (10 samples in 1 episodes)
[2018-11-08 12:55:30] - INFO - OfflineTrainer >>> episode 000084; episodes_aggregated 000001; loss 0.0505415723; loss_min 0.0505415723; loss_max 0.0505415723; loss_std 0.0000000000; epoch 00; acc 1.0000000000; acc_min 1.0000000000; acc_max 1.0000000000; acc_std 0.0000000000; samples_aggregated 000010 [Full Validation]
[2018-11-08 12:55:30] - INFO - OfflineTrainer >>> Experiment finished!

batch size: 10 NOT OK!!
loss 0.0698996484 vs loss 0.0698996559
================================================================================
[2018-11-08 12:57:14] - INFO - OfflineTrainer >>> Starting next epoch: 0
[2018-11-08 12:57:14] - INFO - OfflineTrainer >>> loss 2.3131735325; episode 000000; epoch 00; acc 0.1406250000; batch_size 000064
[2018-11-08 12:57:16] - INFO - OfflineTrainer >>> episode 000084; episodes_aggregated 000085; loss 0.5389420390; loss_min 0.0600712560; loss_max 2.3131735325; loss_std 0.5660359263; epoch 00; acc 0.8187500238; acc_min 0.0468750000; acc_max 1.0000000000; acc_std 0.2081174403; samples_aggregated 005440 [Epoch 0]
[2018-11-08 12:57:16] - INFO - OfflineTrainer >>> Validating over the entire validation set (10 samples in 1 episodes)
[2018-11-08 12:57:16] - INFO - OfflineTrainer >>> episode 000084; episodes_aggregated 000001; loss 0.0698996484; loss_min 0.0698996484; loss_max 0.0698996484; loss_std 0.0000000000; epoch 00; acc 1.0000000000; acc_min 1.0000000000; acc_max 1.0000000000; acc_std 0.0000000000; samples_aggregated 000010 [Full Validation]
[2018-11-08 12:57:16] - INFO - Model >>> Model and statistics exported to checkpoint ./experiments/MNIST/SimpleConvNet/20181108_125714/models/model_best.pt
[2018-11-08 12:57:16] - INFO - OfflineTrainer >>>
[2018-11-08 12:57:16] - INFO - OfflineTrainer >>> Training finished because Epoch limit reached
[2018-11-08 12:57:16] - INFO - OfflineTrainer >>> Validating over the entire validation set (10 samples in 1 episodes)
[2018-11-08 12:57:16] - INFO - OfflineTrainer >>> episode 000084; episodes_aggregated 000001; loss 0.0698996559; loss_min 0.0698996559; loss_max 0.0698996559; loss_std 0.0000000000; epoch 00; acc 1.0000000000; acc_min 1.0000000000; acc_max 1.0000000000; acc_std 0.0000000000; samples_aggregated 000010 [Full Validation]
[2018-11-08 12:57:16] - INFO - OfflineTrainer >>> Experiment finished!


Issue 211. Test for readonly properties

Ok, I remember that readonly didn't work for me, but now they are working, so I created a test case for it. In order for it to work we need to add php 8.1 in CI matrix settings.
But php 8.1 will not work with current lower versions.
The questions is - do you want to add php 8.1 support? If no, I can just add the test and it will be waiting for the version upgrade.
There is another question: how should I go about testing this behavior: with marshaller or with payload converter? I created both, but I think that one of them is enough. Or both is good?

What was changed
Added test that covers readonly properties issue
Why?
Because issue is open and I though readonly properties would not work, but surprisingly they are working just fine
Checklist

Closes issue 221

How was this tested:
With unit test

@shanginn have you tried functional tests with your changes? It fails, I think there are errors when running worker under RR.
https://github.com/temporalio/sdk-php/runs/7752350196?check_suite_focus=true

I did check it under my branch, and it went fine

I rebased (or do you prefer merge?) this branch to the current master. in my fork all the test are good

😢


http.proxy service is changed to accept only with URIs ending with /

This addresses https://github.com/kaazing/tickets/issues/920

I believe a "/" path gets injected for an accept that is missing a path, but not for a connect that is missing the path.  This would cause confusion if someone configured the proxy with a
<accept>http://kaazing.com:80</accept>
<connect>http://localhost.com:8080</connect>

as the error message would only complain about the connect.  I would suggest confirming with a test, and then changing the behavior of the connect url to also inject the path if it is http.  So effectivly the config above would be valid as the / path is implied.

Updated the PR for second review

Thanks @jitsni, LGTM, just waiting for travis to pass


Bucket name used in functional test should have a prefix

Bucket names used in functional test should have a prefix. This is required by mint to cleanup left over buckets. Currently aws ruby functional test uses a random string as the bucket name. It needs to add a prefix.

closing as fixed by #207


Security Fix for Man-in-the-Middle - huntr.dev

https://huntr.dev/users/alromh87 has fixed the Man-in-the-Middle vulnerability 🔨. alromh87 has been awarded $25 for fixing the vulnerability through the huntr bug bounty program 💵. Think you could fix a vulnerability like this?
Get involved at https://huntr.dev/
Q | A
Version Affected | ALL
Bug Fix | YES
Original Pull Request | https://github.com/418sec/em-imap/pull/1
GitHub Issue | https://github.com/ConradIrwin/em-imap/issues/25
Vulnerability README | https://github.com/418sec/huntr/blob/master/bounties/rubygems/em-imap/1/README.md
User Comments:
📊 Metadata *
em-imap is a gem that allows you to connect to an IMAP4rev1 server in a non-blocking fashion.
Affected versions of this package are vulnerable to Man-in-the-Middle (MitM). The hostname in a TLS server certificate is not verified. An attacker can acquire the identity of a trusted server and implement malicious data.
Bounty URL: https://www.huntr.dev/bounties/1-rubygems-em-imap/
⚙ Description *
SSL validation was not implemented making em-imap vulnerable to MiM attacks, fixed adding validations.
💻 Technical Description *
ssl_verify_peer param is eneabled when calling connect from EventMachine::Connection, this will call ssl_verify_peer and ssl_handshake_completed of the connection module for the calling programm to implement validation logic.
Validation of Server certificate was implemented using openssl based on code from:
https://github.com/lostisland/faraday/commit/63cf47c95b573539f047c729bd9ad67560bc83ff
🐛 Proof of Concept (PoC) *

Add a fake DNS entry to /etc/hosts.
echo "<IP_ADDRESS> test.imap.gmail.com" | sudo tee -a /etc/hosts

Create a certificate.
openssl req -x509 -newkey rsa:2048 -keyout key.pem -out cert.pem -days 365 -nodes

Listen on port 443 with TLS enabled.
openssl s_server -key key.pem -cert cert.pem -accept 443

Create sample client:

#clientPoc.rb
require 'rubygems'
require 'em-imap'

EM::run do
  client = EM::IMAP.new('test.imap.gmail.com', 993, true)
  client.connect.errback do |error|
    puts "Connecting failed: #{error}"
  end.callback do |hello_response|
    puts "Connecting succeeded!"
    puts hello_response
  end.bothback do
    EM::stop
  end
end

Run the test client
ruby clientPoc.rb
Client will connect ignoring the self signed certificate

🔥 Proof of Fix (PoF) *
After fix Invalid certificate is detected and connection terminated

Valid certificate for wrong server will also be detected and connection aborted

👍 User Acceptance Testing (UAT)
After fix functionality is unafected

Thanks!


Override default rabbmitq image for periodic CI

This adds the rabbmitmq and rabbmitmq-cell1 image override so we use the right registry, namespace and tag under test in the periodic job.

going to rebase so update incoming.
I still have draft on this because we are still not successfully overriding the rabbit container
Andrew posted a fix which is needed first there https://github.com/openstack-k8s-operators/openstack-operator/pull/475

@jistr @fao89 @cescgina hi folks please have a look when you get some time we can merge this now
after the fix at https://github.com/openstack-k8s-operators/openstack-operator/pull/475 merged we can now override the rabbit image.
Here in the pull request, it defaults to quay.io/current-podified:
        * https://logserver.rdoproject.org/55/155/68fc680ea521ac3f8d573c9437c346682a88334d/github-check/data-plane-adoption-github-rdo-centos-9-crc-single-node/8ae277a/controller/pod/rabbitmq-server-0-describe.txt
        *     Image:         quay.io/podified-antelope-centos9/openstack-rabbitmq@sha256:fdd6a92fe08d0e50257e9197b13a819519b121c59e5878dbbe38baa430b8d161

And in the periodic/rdo test it takes quay.rdoproject.org with the specified hash (podified-ci-testing)
        * https://logserver.rdoproject.org/55/155/68fc680ea521ac3f8d573c9437c346682a88334d/github-check/data-plane-adoption-github-rdo-centos-9-crc-single-node/8ae277a/controller/pod/rabbitmq-server-0-describe.txt
        *     Image:         quay.io/podified-antelope-centos9/openstack-rabbitmq@sha256:fdd6a92fe08d0e50257e9197b13a819519b121c59e5878dbbe38baa430b8d161


Rename google.com/pay to Google Pay from Pay with Google

The https://google.com/pay payment method is known as Google Pay since Chrome M65. Change documentation referencing the payment method and its label to match current Chrome UI.
CC: @petele @agektmr

Screenshot needs updating. Text in background reads "Android Pay" (should be Google Pay) with old Chrome Payment Request dialog in foreground.
/images/deep-dive/pr-demo-pwg-and-cards-short-blackout.png

:tada: This has been pushed live to https://developers.google.com/web/


Bugfix: Bathymetry regridding and B grid velocity regridding.

at this step the intermediate 'topog_raw.nc' file already has vertical coordinate named elevation. Threw an error when vcoord had different name

Test fail. Probably the black?
Also, could we change the PR name into something more descriptive?
Otherwise I'm happy to approve as soon as I see tests pass.

oh wait, I got confused.... you merged a different PR on this branch?
I think the black formatting is still an issue

Oops sorry Navid you were so speedy! I failed to see that you'd already approved changes
I made another PR to fix an issue with the regridder (Luwei found the bug) and figured they could go together but maybe that was a mistake

Can we enhance the tests so they catch these things or you think it's not worth it?

Yes tests definitely need enhancing to include more edge cases. They're pretty basic for now and even that has been very helpful.

Oops sorry Navid you were so speedy! I failed to see that you'd already approved changes
I made another PR to fix an issue with the regridder (Luwei found the bug) and figured they could go together but maybe that was a mistake

All good! :)
thanks @luweiyang

Yes tests definitely need enhancing to include more edge cases. They're pretty basic for now and even that has been very helpful.

Well, I was suggesting we enhanced in this PR :)
Perhaps raise an issue to list the things we can improve them.


Kibana 6.2.1 Crash

Hello,
I have installed plugin in Kibana 6.2.1 but it is crashing as soon as I am clicking on Visualization.

Kindly help,
Thanks,
Naveesh

closing, duplicate of https://github.com/nreese/kibana-time-plugin/issues/39


[BUG] Some search terms break search

Describe the bug
If I open the package window and enter specific terms to search, errors are posted every second or so and no results appear.
To Reproduce
Steps to reproduce the behavior:

Open the search window.
Type Microsoft.MixedReality.QR.
Press enter/return.

Expected behavior
The expected results appear.
Desktop (please complete the following information):

OS: Linux x86-64
Unity Version: 2021.3.6f1
Package Version 2.3.2

Additional context
The error that is posted is the following:
You cannot call GetLast immediately after beginning a group.
UnityEngine.GUILayoutUtility:GetLastRect ()
kumaS.NuGetImporter.Editor.NuGetImporterWindow:OnGUI () (at Library/PackageCache/org.kumas.nuget-importer@2.3.2/Editor/NuGetImporterWindow.cs:427)
UnityEngine.GUIUtility:ProcessEvent (int,intptr,bool&) (at /home/bokken/buildslave/unity/build/Modules/IMGUI/GUIUtility.cs:189)

I don't know if it's a coincidence, but if I search for just QR, I do not get these errors, but it gets stuck on "Organizing datas", presumably because there are some packages it can't fetch something for:
HttpRequestException: Response status code does not indicate success: 404 (Not Found).
System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode () (at <a07a075638bc4ba8988cc7cdce08093f>:0)
System.Net.Http.HttpClient.GetByteArrayAsyncCore (System.Threading.Tasks.Task`1[TResult] getTask) (at <a07a075638bc4ba8988cc7cdce08093f>:0)
kumaS.NuGetImporter.Editor.PackageDataExtentionToGUI.GetIcon (System.String url) (at Library/PackageCache/org.kumas.nuget-importer@2.3.2/Editor/PackageDataExtentionToGUI.cs:650)
kumaS.NuGetImporter.Editor.PackageDataExtentionToGUI.GetIcon (System.String url) (at Library/PackageCache/org.kumas.nuget-importer@2.3.2/Editor/PackageDataExtentionToGUI.cs:667)
kumaS.NuGetImporter.Editor.PackageDataExtentionToGUI.GetIcon (kumaS.NuGetImporter.Editor.DataClasses.Datum data) (at Library/PackageCache/org.kumas.nuget-importer@2.3.2/Editor/PackageDataExtentionToGUI.cs:236)
kumaS.NuGetImporter.Editor.NuGetImporterWindow.UpdateData () (at Library/PackageCache/org.kumas.nuget-importer@2.3.2/Editor/NuGetImporterWindow.cs:332)
UnityEngine.Debug:LogException(Exception)
kumaS.NuGetImporter.Editor.<UpdateData>d__34:MoveNext() (at Library/PackageCache/org.kumas.nuget-importer@2.3.2/Editor/NuGetImporterWindow.cs:341)
System.Runtime.CompilerServices.AsyncTaskMethodBuilder:SetException(Exception)
kumaS.NuGetImporter.Editor.<GetIcon>d__10:MoveNext() (at Library/PackageCache/org.kumas.nuget-importer@2.3.2/Editor/PackageDataExtentionToGUI.cs:242)
System.Runtime.CompilerServices.AsyncTaskMethodBuilder:SetException(Exception)
kumaS.NuGetImporter.Editor.<GetIcon>d__16:MoveNext() (at Library/PackageCache/org.kumas.nuget-importer@2.3.2/Editor/PackageDataExtentionToGUI.cs:675)
System.Reflection.MethodBase:Invoke(Object, Object[])
Microsoft.MixedReality.Toolkit.Utilities.SyncContextUtility:ExecuteContinuations() (at Library/PackageCache/com.microsoft.mixedreality.toolkit.foundation@2.8.0/Core/Utilities/Async/Internal/SyncContextUtility.cs:53)
UnityEditor.EditorApplication:Internal_CallUpdateFunctions() (at /home/bokken/buildslave/unity/build/Editor/Mono/EditorApplication.cs:356)

This error occurs when the search results show that there are packages in the development version, but none of them are in the stable version. (I'll fix it.)
I thought I fixed the 404 issue with this commit(https://github.com/kumaS-nu/NuGet-importer-for-Unity/commit/d322086a4b1dcae1a4197d7da726deeb035425aa#diff-e42742dcb744df86ea36fa18292a0c557ae8cbe4d5b458a2217dbbe6160067f9R612)... I'll check it out.

In previous versions of Unity, it would throw an exception, and I would grab it. But apparently, the new version of Unity changed to output the error directly to the console without throwing an exception.

Just a hunch, but looking at https://github.com/kumaS-nu/NuGet-importer-for-Unity/commit/d322086a4b1dcae1a4197d7da726deeb035425aa#diff-e42742dcb744df86ea36fa18292a0c557ae8cbe4d5b458a2217dbbe6160067f9R612 (not sure if that is the latest code), are you sure that the message didn't just change and it's now hitting the throw below, rethrowing the exception?
Also, you might want to catch HttpException e instead and check for e.StatusCode == HttpStatusCode.NotFound (or similar) instead to avoid the message changing in the future :slightly_smiling_face: .


More than one migrations configuration type was found in the assembly 'NuGetGallery'

I'm setting up Nuget Gallery for first time and am following the steps in this document: https://github.com/NuGet/NuGetGallery/wiki/Hosting-the-NuGet-Gallery-Locally-in-IIS
I have successfully installed IIS and created the Website http://localhost/Gallery
I have also successfully installed SQLExpress2014 and am using the following as my connectionstring for the destination database:
"Data Source=.\SQLEXPRESS;Initial Catalog=master;Integrated Security=True"
But when I ran Update-Database command to create the DB I get below error:
PM>  Update-Database -StartUpProjectName NuGetGallery -debug -verbose
Using StartUp project 'NuGetGallery'.
Using NuGet project 'NuGetGallery'.
Specify the '-Verbose' flag to view the SQL statements being applied to the target database.
System.Data.Entity.Migrations.Infrastructure.MigrationsException: More than one migrations configuration type was found in the assembly 'NuGetGallery'. Specify the name of the one to use.
at System.Data.Entity.Utilities.TypeFinder.FindType(Type baseType, String typeName, Func2 filter, Func2 noType, Func3 multipleTypes, Func3 noTypeWithName, Func3 multipleTypesWithName)    at System.Data.Entity.Migrations.Utilities.MigrationsConfigurationFinder.FindMigrationsConfiguration(Type contextType, String configurationTypeName, Func2 noType, Func3 multipleTypes, Func3 noTypeWithName, Func`3 multipleTypesWithName)
at System.Data.Entity.Migrations.Design.ToolingFacade.BaseRunner.FindConfiguration()
at System.Data.Entity.Migrations.Design.ToolingFacade.BaseRunner.GetMigrator()
at System.Data.Entity.Migrations.Design.ToolingFacade.UpdateRunner.Run()
at System.AppDomain.DoCallBack(CrossAppDomainDelegate callBackDelegate)
at System.AppDomain.DoCallBack(CrossAppDomainDelegate callBackDelegate)
at System.Data.Entity.Migrations.Design.ToolingFacade.Run(BaseRunner runner)
at System.Data.Entity.Migrations.Design.ToolingFacade.Update(String targetMigration, Boolean force)
at System.Data.Entity.Migrations.UpdateDatabaseCommand.<>c__DisplayClass2.<.ctor>b__0()
at System.Data.Entity.Migrations.MigrationsDomainCommand.Execute(Action command)
More than one migrations configuration type was found in the assembly 'NuGetGallery'. Specify the name of the one to use.
Can anyone help?   I have followed through this document https://github.com/NuGet/NuGetGallery/issues/1108 but I cannot see how to 'exclude' /src/Migrations/configuration.cs without deleting it and if I do that the project refuses to build.

Use Update-Database -ConfigurationTypeName MigrationsConfiguration. We have a second migrations configuration in the admin area for a support requests database which you likely won't need, and which is likely to be pulled out of the gallery code base at some point.
Closing the issue, but feel free to reply here if you encounter further issues.

Boom... Thank you Xavier ;-) I confirm that that's got it.

@xavierdecoster Can you update the gallery readme?

Done https://github.com/NuGet/NuGetGallery/commit/4e3f1fd75218ee8e060770850a20539465234321


Fix #857 Collecting all values for given key in object array returns empty Collection

resolve issue https://github.com/json-path/JsonPath/issues/857 in this way

There is too much formatting in this PR. IMHO, we should focus on fixes.

There is too much formatting in this PR. IMHO, we should focus on fixes.

OK，I will remove   formatting in this PR.
Thanks


Stop/Pause music when trailer is focus

How do I make it stop or pause when a trailer from extrametadata is in fullscreen ?

Only way is to give themes control over music in a similar manner to ExtraMetadataLoader.
I have a story for that already.


Change escape double quotes to backticks

The change in commands.js (changing from double quotes to backticks) does not seem to work, are you sure it's working for you? It gives an ER_BAD_FIELD_ERROR. Keeping it with double quotes work

I just rechecked my logs, and that is an incorrect format for selecting. I had thought that would be the correct way to escape, but it seems to only be so in particular cases.


declare SMTPClientSettings parameter of sendMail as 'in'

as it is not modified.

Is it possible to get some feedback on this?
My motivation for this:

store the settings in a global variable and reuse it -> with current API, it has to be mutable. Therefore it needs to be constructed in static this()
once you want to send a mail on server startup, it runs in shared static ctor and therefore before any static ctor is executed and results in a segfault.

With this param declared const, one can store the settings in a immutable field and initialise it in shared static ctor.

Sorry, missed this. Should be fine!


Give line numbers for uninitialized fields when reporting error on an initializer

This adds the line numbers for uninitialized fields when reporting error on an initializer. As a response to issue #95.
Summary of changes:
-Edit function  errMsgForInitializer() in ErrorBuilder.
-Create a new function GetLineforField() which gives back a source line number.
-Adjust the expected error messages in the test cases.

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

Thanks for the first contribution @Lycheeicy!

Thanks for the first contribution @Lycheeicy!

My pleasure!


Push state option

First try at implementing an option to support push state by rewriting all request URLs to use /index.html instead of the URL requested.
Does this conflict with other options? I'm concerned about changing the requested URL.
This was originally requested as part of  indexzero/http-server#80

Noticed that this doesn't work well if you're including JS files as well. Going to update to make it work as a fallback so if a file exists it is served.

Updated logic in 5dd25b5.

+1 for the merge.

I agree with @indexzero, this is a better implementation than #205. +1 for the merge.

Would be nice to see merged.

+1

+1

:shipit:

+1

Would be very nice to see this merged.

@indexzero Could you merge this?

Nice implementation that doesn't disable directory listing. It looks like somebody has published it to npm: https://www.npmjs.com/package/spa-http-server.

+1 for the merge!

+1

Used the spa-http-server npm package as suggested and then enabled the --push-state option. Works like a charm. Thanks!!

Has anyone here tried it along with setting proxy option for http-server (-P www.something.com)?
It works for me as long as I am not setting the proxy.

How to use spa-http-server in SPA mode:

Install it with npm i spa-http-server -g. It installs 2 global binaries hs and http-server.
Use it http-server --push-state.

This is what you should see:

@ekulabuhov, thanks for this! Where is the code for this spa-http-server fork?
Could this be merged over? It's an amazing feature and spa-http-server works like a charm for me!

We're adding a "self-proxying" approach to the README to address this issue, see #513
That should address this sort of usage without breaking the focus of this project on providing accurate HTTP responses. spa-http-server is available as a fork which focuses on SPA usage.


Applying tokenizers before compression

Dear authors,
Thanks so much for the interesting work!
I am currently trying to replace the language models in your code with GPT-2; however, it seems that the encoding after tokenization is not as impressive as the results you showed in the paper. I wonder if your finding only applies to pure ASCII (i.e., w/o tokenization) regarding image compression.
I also noticed that in Table 2, you tested different sizes of BPE on a text dataset. I wonder if you have any results on images?
Thanks so much for your time. I'd appreciate any feedback you may have.
best,

Hi @anianruoss ,
Thanks for the answer, and sorry for the confusion. I actually want to know your opinions on whether your findings can be generalized when using pre-trained text tokenizers for other modalities, such as images.
I am currently trying to compress images using GPT -2 but have failed to reproduce the performance or even match the PNG codec. Given that, I am curious if tokenization would be the main cause of the failure.
best,


Delete assets/css/style.scss

Closes #
📑 Description

✅ Checks

[ ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] All the tests have passed

☢ Does this introduce a breaking change?

[ ] Yes
[ ] No

ℹ Additional Information

Summary by CodeRabbit

Bug Fixes

Removed outdated CSS styles that were impacting the web application's appearance.

Chores

Eliminated unnecessary style file to streamline the codebase.

:white_check_mark: Build Sankhya-SDK-dotnet 3.0.955 completed (commit https://github.com/guibranco/Sankhya-SDK-dotnet/commit/08e72f730e by @gstraccini[bot])

:x: Build Sankhya-SDK-dotnet 3.0.968 failed (commit https://github.com/guibranco/Sankhya-SDK-dotnet/commit/bc36b06f6d by @guibranco)


[Python] Improve Python consistency: "variable in function should be lowercase" (N806)

What's in this pull request?
Background
See discussion with @modocache and @nadavrot in #1369.
This is PR is part 3 of 3 in the series of PRs covering the Python naming consistency fixes discussed in #1369.
The previous parts were:

Part 1. argument_name, see #1374
Part 2. function_name(…), see #1399

Rationale
The repo contains roughly 80 Python scripts. snake_case naming is used for local variables in all those scripts. This is the form recommended by the PEP 8 naming recommendations (Python Software Foundation) and typically associated with idiomatic Python code.
However, in 11 of the 80 scripts there were at least one instance of camelCase naming prior to this commit.
This commit improves consistency in the Python code base by making sure that these eleven remaining files also follow the variable naming convention used for Python code in the project.
References

PEP 8: https://www.python.org/dev/peps/pep-0008/
pep8-naming: https://pypi.python.org/pypi/pep8-naming

Resolved bug number: –

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

@modocache Do you have time to review? That would be appreciated! :-)

Looks good to me, based on a quick scan through the source files. I'd merge this if CI passes! :100:

@swift-ci please test

@modocache Thanks! Unfortunately I don't think commit access is a sufficient condition for triggering @swift-ci :-)   Seems like these commands can only be issued by Apple employees at the moment.

@swift-ci Please test

@practicalswift Yeah, I type in "@swift-ci please test" as a sort of bat signal for Apple org members to trigger CI for me. :joy: Thanks, @trentxintong!

@jckarter please swift-ci

Thanks, @practicalswift! :100:
(Aside: I wish more of the Python code in utils was tested as part of the validation tests, like utils/swift_build_support is.)


Invalid error text? ("loginUrl must use Http ...")

public loadDiscoveryDocument(fullUrl: string = null): Promise<object> {
         ......
        if (!this.validateUrlForHttps(fullUrl)) {
            reject('loginUrl must use Http. Also check property requireHttps.');
            return;
        }

Must be "issuer" instead of "loginUrl" in error text?
May be additional check for empty issuer will be usefull?

shouldn't this be how user allows http or https even url is local. I still face this error when I have url as https://<IP_ADDRESS>:5001 and set the requireHttps to true. In this case I run on local https address but still get this error. Sounds like this is forcing me to use http on local.

Thx for this. Yes, the message was wrong. It should be issuer must use Https. It's fixed in the next version.


Add homepage, fix lint issues, delete copied code

Description
I grabbed James's commits from his branch & updated it with Reece's changes to make sure it all worked, am putting up this PR to get Sonar to do some more checks - have fixed linting issues. James's PR as follows:
Description:
Implemented a basic homepage with "Popular Categories" and "Featured Today" components. Clicking on any category navigates to the search results page. Click logo will now navigate to homepage.

Notes:
The data for "Featured Today" is currently hard-coded. If anyone has ideas for either storing this data in a database and retrieving it randomly each time or calling the API endpoint each time the homepage loads, this could be left as an issue for version 2.

Works on my end. Regarding the issue, can't we just leave it open? Or do we have to suggest a solution?

Works on my end. Regarding the issue, can't we just leave it open? Or do we have to suggest a solution?
What issue do you mean sorry?


Fix - getRelated() on null for relationship discovery

Namaskar,
This PR addresses an issue by introducing an additional instanceof check for the $relation to see if it's a valid relationship, in the EloquentEntitySet driver. If not, bail early.
https://github.com/flat3/lodata/blob/1c3b45a06da9fceb8698a1925f63eb12ebb01c01/src/Drivers/EloquentEntitySet.php#L517
I'm not sure as to why even in case of a null exception, the get_class is not bailing, but doing a check to bail seems to work.
Here's a stack trace where I was able to encounter this issue: https://flareapp.io/share/J7oxAYx5
Here's the offending model snippet, the ... denotes parts that have nothing to do with the bug:
...

use Flat3\Lodata\Attributes\LodataRelationship;
use Illuminate\Database\Eloquent\Factories\HasFactory;

class Companies extends Model
{
    use HasFactory;
    use InputScopeTrait;

    protected $table = 'companies';

    protected $fillable = [
        ...
        'domain_id',
        'organization_id',
    ];

    protected $guarded = ['id'];

    public $timestamps = true;

    /**
     * Define the relationship where each row belongs to one domain.
     *
     * @return \Illuminate\Database\Eloquent\Relations\hasOne
     */
    #[LodataRelationship]
    public function domain()
    {
        return $this->hasOne(Domains::class, 'id', 'domain_id');
    }

    /**
     * Define the relationship where each row belongs to one organization.
     *
     * @return \Illuminate\Database\Eloquent\Relations\hasOne
     */
    #[LodataRelationship]
    public function organization()
    {
        return $this->hasOne(Organizations::class, 'id', 'organization_id');
    }

    ...

    /**
     * Define the relationship where each row can have multiple payment information.
     *
     * @return \Illuminate\Database\Eloquent\Relations\hasMany
     */
    #[LodataRelationship]
    public function paymentInformation()
    {
        return $this->hasMany(PaymentInformation::class, 'owner_id', 'id')
            ->where('owner_type', 'Organization')
            ->where('domain_id', $this->domain_id)
            ->where('organization_id', $this->organization_id);
    }
}

OdataCompanies Class:
...
namespace App\Providers\Odata;

use Illuminate\Support\ServiceProvider;
use App\Models\Account\Companies;

/**
 * Lodata Service Provider
 * @package App\Providers
 */
class ODataCompanies extends ServiceProvider
{
    public function boot()
    {
        if (app()->runningInConsole() || app()->environment('testing')) {
            return;
        }

        \Lodata::discover(Companies::class);
    }
}

Essentially loaded into as a provider.
Additional information:

PHP 8.2.15 (built: Jan 20 2024 14:17:05) (NTS)
Laravel Framework 10.43.0
MySQL Community version 8@latest

Note it's essentially the stock Laravel 10.x sail docker image.

Thanks @coderkoala !


Container gone after docker stop

Whenever I use docker stop to stop a certain container it's gone and not visible through docker ps -a.
Executing a docker start again then results in an error from the daemon: 'No such container ', so the container seems to be gone.
I'm running docker 17.06.2-ce-win27 (13194) stable on Windows 10 enterprise.
Anyone else had the same issue?
Help is very much appreciated!

If you have -rm as Parma for your docker run command, then that would be by design

@tfenster I'm not using -rm as parameter!
docker run     --rm
-m 3G     --name $hostname
--hostname $hostname     -v $PSScriptRoot\my:c:\run\my
-e Accept_eula=Y     -e Auth=Windows
-e username=richardr     -e password=$passplain
${NAV_DOCKER_IMAGE}

@tfenster You're right, my bad, thanks for the quick reply!

@richardrobberse Thanks for asking this question... 😅
@tfenster Thanks for answering this question... 🥳

@richardrobberse Thanks for asking this question... 😅
@tfenster Thanks for answering this question... 🥳


add backgroundCursorColor

backgroundCursorColor is essential for EditableText
otherwise it will give runtime EXCEPTION
flutter: 'backgroundCursorColor != null': is not true.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.lizhuo.huang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank you for your commit.


Make CMAKE_DEBUG_POSTFIX truly optional

The missing piece to fix  #385
Add DEFINED to the conditional to allow the user to set CMAKE_DEBUG_POSTFIX to blank which will disable the _d suffix on Windows builds. Othwerise, the default jansson cmake behavior is unchanged.

Coverage decreased (-0.8%) to 95.585% when pulling 62aaa1db576e117ecf2634ffb2e23ff28a68a7eb on KOConchobhair:optional-cmake-debug-postfix into b333f3656d6a408fe38737982d854b80ea870803 on akheron:master.

LGTM!  It is a better way to set the value of CMAKE_DEBUG_POSTFIX.
It's also seen in curl project like 73a3b3e0


Installation Instructions Need Simplification & Clarification

The blog post talks about downloading and saving the .tbx file and adding it to ArcToolbox. However, this doesn't get the associated python script. I've had two users recently ask me why the tool doesn't work and it was because they didn't have the python script on their computer.
Also, the path to the python script in AGRC Geocode Tools.tbx seems to be pointed at a local directory on @keatonwalker's machine. Could we make this a relative path?
Finally, the installation instructions on the repo talk about running build.py and install.py. A completely different installation method than is referenced in the post.
I would suggest packaging the .tbx and .py file into a .zip file and making that the distribution mechanism. Then the installation would be a simple as unzipping the files onto your computer and adding the toolbox to ArcToolbox.

I thought the downloadable version still had the script imported but maybe it doesn't. I will look at it.
We talked about making an automatic update mechanism for toolboxes at one time and that is why the build.py and install.py exist.
We need to find a better way to distribute toolboxes if we are going to keep making them. At one time we decide any sort of custom installation process was to big a burden for the end users, so any installation will need to be very straight forward.

I believe that ESRI has a suggested structure for sharing geoprocessing tools. You may want to check that out. I like the .zip file idea the best if relative paths work.


fix: create filepath instead of trimming string

Fixes #463

@diegorqc fyi

closing this as stale - if you made any progress on the discussed changes/testing please just open a new PR. Thanks
(note this should be obsolete with different handling that will be introduced in v2)


RDP Config reports version 1.5 after installing version 1.6

Repro steps:

Install version 1.6
Run RDP Config
Notice that the GUI reports RDPWrap version 1.5

I saw the same thing.  Running the "update.bat" upgrades from v1.5 to 1.6.

Seems to still be an issue. Even after running update.bat, RDPCheck.exe still indicates v1.5. Also, choosing properties on rdpwrap.dll indicates version <IP_ADDRESS>

This is not an issue, internal library version is 1.5, it doesn't need to recompile since all important settings are stored in INI file.
1.6 is the version of the final release package.

Added to FAQ in README.
This won't be fixed for now.


Docs cognito

Signed-off-by: Sam Lock<EMAIL_ADDRESS>Description

Add tutorial/integration docs for Cerbos + AWS Cognito demo
Checklist

[x] The PR title has the correct prefix
[ ] PR is linked to the corresponding issue
[x] All commits are signed-off (git commit -s ...) to provide the DCO

This PR needs to be deployed first


Predicate inference fails for empty object and nullish values

🔎 Search Terms
"infer predicate object types", "infer predicate empty object", "infer predicate empty object null", "{} | null"
🕗 Version & Regression Information

This is a type narrowing issue
This is the behavior in every version I tried

⏯ Playground Link
https://www.typescriptlang.org/play/?ts=5.5.2#code/MYewdgzgLgBApgWwA5QJ4DkCuAbbMC8MA2gN4C+ANDGDtgLoB0AZgJbZRwBOAFBAQHwwAhEIgBKANwwA9NPgAPJHGAcAJjHJE6AKBh7ZMAHoB+baEixEKVAFUwquKzBx1hUpRiZ7jls9WNWdi5eAWFRSRk5OEVlNQ0yLV19ORMzcGgFJGwWYBYoLFwWCAALAmJyKhpcALYOHj58QT4hfEIq7AiDEABrJMijU3MMqzQsBAAjLjL3KgAGGqD60JFxKS7evX7UoctkNAAVTkwoYtRp8gW6kMaw1f7opRUXeMTNg2303esAMQBDbAgZzcF2YtWCDUE4TWURiT3UzgAblxXskBmkLNRwABRPYYWjnGBMEAgABc8RgHnal3Byyh-R6fXeg0+1FoRVKbnaVC8DicLmpSxuK06MMecURyJ0bxSpiAA
💻 Code
const emptyNull = [{}, null].filter(s => !!s); // expected {}[]
   // ^? ({} | null)[]
const emptyUndefined = [{}, undefined].filter(s => !!s); // expected {}[]
   // ^? ({} | undefined)
const emptyTruthy = [{}].filter(s => !!s); // expected {}[]
   // ^? {}[]
const emptyFalsy = [{}].filter(s => !s); // expected never[]
   // ^? {}[]
const nullish = [null, undefined].filter(s => !!s); // expected never[]
   // ^? (null | undefined)[]
const explicitNullish = [{}, null].filter(s => s !== null); // ok
   // ^? { foo: {}; }[]
const nonEmptyNull = [{ foo: {} }, null].filter(s => !!s); // ok
   // ^? { foo: {}; }[]
const emptyNumber = [{}, 0].filter(s => !!s); // ok
   // ^? {}[]

🙁 Actual behavior
Predicate inference fails for the cases where {} is declared along with a nullish value, using a predicate to test for truthiness or falsiness (!!x / !x). It infers a type that includes the nullishness.
It also fails to infer the predicate for cases where array only contains nullish values.
It works if the object has at least one property, or using explicit bullishness check.
🙂 Expected behavior
The return type should reflect the truthiness of {}, as it is neither a nullish or falsy value and should pass the truthiness test.
Additional information about the issue
The section "This is a gentle nudge away from truthiness footguns" in https://github.com/microsoft/TypeScript/pull/57465 explains that object types should have no footguns:

If you're working with object types, on the other hand, there is no footgun
and a truthiness test will infer a predicate:
const datesTruthy = [new Date(), null, new Date(), null].filter(d => !!d);
//    ^? const datesTruthy: Date[]

That's why I expected the same for {} and the opposite with nullish-only values.

Primitives are assignable to {}, unlike (most) other object types.

And because "" and 0 both match {}, you can't say that !x implies that x is null.  And without direct support for #15048 you can't use a type predicate for this at all. You'd like to say that (x: {} | null) => !!x returns (x is {}) else false or something, and redefine filter() so it only cares about the true side of the type guard. But we don't have that so 🤷♂


await response for: What permissions do you want to grant to this new layer version?

Scenario:

executing amplify push
updates to layer function exists
updates to other functions exist

The CLI does not wait for a response to the question "What permissions do you want to grant to this new layer version?"
Amplify CLI Version
4.41.2
To Reproduce

make changes to layer
make changes to function
amplify push
wait
notice the push does not continue

Expected behavior
CLI should wait for user to respond to the question instead of exiting.
Desktop

OS: Windows 10 WSL 2
Node Version. v12.18.3

Hi @jgroom33, I wasn't able to reproduce this on mac. It might be WSL specific, but do you see any similar issues when using Amplify CLI? The code that waits on this response isn't fundamentally different from other question answer flows.

@jhockett I have not noticed any other similar issues.
The workaround is pretty simple: amplify push layers on their own.  This is probably good practice anyway, so I don't think this is worth any extra effort.

Hi all!
We have an alpha build of our redesign of Lambda layers in Amplify CLI available to try. This includes several bug fixes and new features. It also includes a one-way migration step for existing layer resources. Lambda layer resources pushed with this build will not work with any other version of the Amplify CLI.
DO NOT USE THIS IN PRODUCTION!
To test it out: npm i -g @aws-amplify/cli@not-for-production-layers
Notable changes:

Complete redesign of layer version tracking. New code does not make assumptions about what layer versions exist in the cloud and always pulls the latest data from the Lambda service
Option to automatically update dependent functions to the latest deployed Lambda layer version
Added ability to delete individual layer versions
Layers now have a dependency installation step that will install dependencies based on what's in package.json for NodeJS or Pipfile for Python
Support editing layer version descriptions
Removes local version state tracking from amplify-meta.json and team-provider-info.json

We'd love to hear any community feedback!


Reminder Feature

A reminder feature using JARVIS

This would be the first module that would require a backend! 😎
@harshitdx29 Would you be willing to pick it up?

This would be the first module that would require a backend! 😎
@harshitdx29 Would you be willing to pick it up?

Yes sure! Glad you liked the suggestion. 👍

Yes sure! Glad you liked the suggestion. 👍

Is firebase a good fit for storing data in this case?

Haven't thought over this feature. Give me a couple of days to be really able to come up with a solution for this.

Hi Swapnil,
I was thinking about this feature and have come up with a design for it for the first phase.
We can have a get call say /reminder which will render a form having fields like "remind me to" which will take a string as input and two other fields as "date" and "time". Once the form is submitted the values can be stored in database. And also there would be a background task running asynchronously which will find all the reminders for a specific period of time(eg (CurrentTime to CurrentTime+ 10 minutes) and send them a notification/sms.
We  can use firebase as a database here because there wont be any joins or transactional events.
Let me know what do you feel about it.

I didn't understand what you meant by ...have a get call say /reminder which.... Users interact with the bot on messenger platform, there's not a website that they can hit and fill a form. So it would be a query like remind me to do something at 10pm tonight, from which we can extract relevant data and store it in firebase.
The background task is the harder of the two steps; taking care of all possible scenarios.. like if there was some problem in sending the notification, we should retry, etc. The reminders would be sent via messenger itself.

There would be a lot of parsing required for the query as the user can enter time and date in various formats and overall language as well.
Eg: 1) Remind me to do something at 10 pm tonight
2) Remind me to do something at 2200 hrs today.
3) Remind me to do something at 10pm on 10th jan 2017 (various date formats here.)
4) At 2 o clock in the afternoon on 10th Jan 2017 remind me to do something.
And yes agreed background step is the harder step because we want to send reminder at a fixed time before the event.

So the parsing would be handled by wit.ai so you need not do anything. I'll train it and you'll have the required data in entities. 😄

Awesome then 😆  will start working on it soon 👍


Cleaned up API endpoint documentation to point to DS overview

What does this PR (Pull Request) do?
Removes a lot of boilerplate explanations of DS fields in the API docs, and changes them to reference the definitions provided in the Delivery Service overview. There's still boilerplate, but it's much smaller and should only need to be updated in one place from now on.

[x] This PR is not related to any Issue

It also fixes a bunch of incorrect notices regarding tenancy, and adds some missing fields to various DS-related endpoints. One of which was not properly formatted - so it fixed that too.
Which Traffic Control components are affected by this PR?

Documentation

What is the best way to verify this PR?
Build the documentation, then read the docs to be sure they are correct and make sense. No tests, just a docs change.
The following criteria are ALL met by this PR

[x] I have explained why tests are unnecessary
[x] This PR includes documentation
[x] An update to CHANGELOG.md is not necessary
[x] This PR includes any and all required license headers
[x] This PR does not include a database migration
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3853/
Test PASSed.


{CosmosDB} Typo "Azure CosmosDB"→"Azure Cosmos DB"

Description

Typo "Azure CosmosDB"→"Azure Cosmos DB"
https://docs.microsoft.com/en-us/cli/azure/cosmosdb/locations?view=azure-cli-latest#az_cosmosdb_locations_show
https://github.com/Azure/azure-cli/blob/dev/src/azure-cli/azure/cli/command_modules/cosmosdb/_help.py
#PingMSFTDocs
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

CosmosDB


Add signatures for MDE, Elastic

Hi!
This is a PR to add EDR signatures for MDE and Elastic.
For MDE, a small detail:

Check for the strings sensendr and sensetvm: these are the running processes SenseNDR.exe and SenseTVM.exe.
The main binary seems to be MsSense.exe, but it's installed as a service Sense in all (?) Windows installations. The service is stopped when MDE is not used, but it would still trigger an mssense detection string, leading to a false positive.
Instead, we check for the processes SenseNDR.exe and SenseTVM.exe that run only when MDE is actually in use.

Cheers!

Nice trick for MDE indeed. Thanks for your contribution! :)


Installation on Ubuntu

You should query cabal for the bin path and add that to PATH during installation, because otherwise one of the gtk*hs packages doesn't build and the installation fails.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/29015042-installation-on-ubuntu?utm_campaign=plugin&utm_content=tracker%2F332399&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F332399&utm_medium=issues&utm_source=github).

Duplicate #618.


energy volume curve - use axes = ("x", "y", "z") rather than axes = ["x", "y", "z"]

@liamhuber I have to admit I am not really sure how the final type hint should look like. So if you want to make a suggestion or a stacked pull request I am happy to wait with merging this one, otherwise I would go ahead and merge this one as is.

The hints are non binding in python anyhow; this one is by no means wrong, perhaps just narrower than the reality of the options; and I won't be able to get to it super fast -- so I'm fine with merging as is. Would you do me a favour and raise an issue linked to this and assign me to it though? Then I can get back to it.


[Bug] Currency Code Sorting is Broken

Hi,
I noticed that the Currency datatype sort appears to be broken. I confirmed it is broken as well on the web demo https://weblaze.dev/pluto_grid/build/web/#feature/currency-type-column when you click on the Euro symbol.
Can you please have a look and advise?
Warm regards,
Zambet

Bounce, bad boy

Bounce,  bad bot


Google ReCaptcha may not be checking if captcha is completed

User reports captcha having no effect on spam
 Front conversations

This was a configuration issue.

@BenParizek Can you please elaborate? We're having the same problem...

@rbrv We'll be releasing an update soon to try to address some of this but here are some things to consider:

Currently, this Captcha doesn't make the reCAPTCHA field required if a user does not fill it out. You will need to add the code below to the template to help make that happen. We'll be adding an option to output that code in the settings in the next release. We had previously removed it because another user reported it conflicting with other things they were doing.

{% css %}
#g-recaptcha-response {
    display: block !important;
    position: absolute;
    margin: -78px 0 0 0 !important;
    width: 302px !important;
    height: 76px !important;
    z-index: -999999;
    opacity: 0;
}
{% endcss %}

Also note that reCAPTCHA has settings on the Google side where you can increase the Security Preference: https://developers.google.com/recaptcha/docs/settings
If those items still don't help solve things for you, let us know. The next release of Sprout Forms will have improved Spam logging to help diagnose more about what characterizes your spam traffic.

As we weren't sure when there would be a fix, we've had to switch to FreeForm for this project.
Will try again on our next project. Thanks @BenParizek!

@rbrv fyi, yesterday we released Google ReCaptcha For Sprout Forms v1.0.6 which adds the option to enable/disable the output the CSS above and make the field required on the front-end.
We also released Sprout Forms v3.6.0 which adds spam management features including flagging entries that fail a Captcha check with a spam status in the database and filtering them out of the main listing views, additional tracking metadata like IP Address and a new Spam Log report to view data on what Captchas have failed and why.


Update koa changelog

Description
Preparing the changelog entry and README warning message for #1455 release.
This is similar to the actions we took when shopify_app had a CSRF vulnerability: https://github.com/Shopify/shopify_app/commit/778244e0c9b404f06daf45e7106cd6a984f41fb3
Type of change

[x] Patch: Bug/ Documentation fix (non-breaking change which fixes an issue or adds documentation)

Checklist

[x] I have added a changelog entry, prefixed by the type of change noted above

I directly introduce those changes in the publish commit.


Add arguments for contract's init method

Allow to pass initialization arguments, when deploying a contract.

Hello, is there any progress on this issue?


Monitor custom metrics

Hi
I'm using kafka <IP_ADDRESS> and I'm trying to add additional metrics to the index.html + config lists.
I can see that the UI show the expected metrics but in the logs I dont see the lines as well as the default metrics, does this project support that?
Thanks
D.

@panda87 If you update report.metrics.list  in https://github.com/linkedin/kafka-monitor/blob/master/config/kafka-monitor.properties and start Kafka-Monitor using ./bin/kafka-monitor-start.sh config/kafka-monitor.properties, you should see metrics value in the console, no?

I tried but I don't see anything in the console log.
Do you have metrics name that I can test with?

@panda87 Can you provide instructions to reproduce this, i.e. the link to your clone of the repository that includes your change to the config file, and the comments you used to start Kafka-Monitor and so on.


chore: add shell flag to prepare script

This should fix the script failing to run on windows.
I had added this flag in earlier work when we wanted a more complex prepare script but we abandoned that, however we should still keep this flag to fix windows use cases.

One question, is there any harm to passing {shell: true} on platforms other than Windows?

No harm, but someone astutely made the point that it adds expense where none is needed, I was inclined to agree. I'll merge this now just to unblock those using the git link dep style, we can simplify it back to shell: true in your PR if we wish. :) ty for speedy ✅


Pack x, Pick y naming

The app uses the term Booster mostly.
In the interface, when users are picking cards I prefer to use Pack instead as players are used to discuss picks with phrases like p1p1, second picked z in pack 3, z was quite late in pack 1 etc.
Pack #x, Pick #y seems to be the default wording in the Magic community.

Yeah I'm pretty inconsistent with this, I've only ever use "Booster" in French ^^
"Your Booster" is fine thought?

"Your Booster" is fine thought?

Yeah, I think so. Otherwise it's also a repetition right next to each other.


AuthStateListener is called multiple times.

Android device: Sony D6503
Android OS version: 4.4.4
Google Play Services version: 9.2.56(038-124593566)
Firebase/Play Services SDK version: 9.0.2

Describe the problem:
AuthStateListener is called multiple times.
Steps to reproduce:

addAuthStateListener onStart activity
removeAuthStateListener onStop activity

Observed Results:
07-08 23:36:16.804 29229-29229/com.domain D/FirebaseApp: Notifying auth state listeners.
07-08 23:36:16.804 29229-29229/com.domain D/FirebaseApp: Notified 0 auth state listeners.
07-08 23:36:16.814 29229-29229/com.domain D/FirebaseApp: Notifying auth state listeners.
07-08 23:36:16.814 29229-29229/com.domain D/FirebaseApp: Notified 0 auth state listeners.
Expected Results:
07-08 23:36:16.804 29229-29229/com.domain D/FirebaseApp: Notifying auth state listeners.
07-08 23:36:16.804 29229-29229/com.domain D/FirebaseApp: Notified 0 auth state listeners.

@mific I can confirm that this is happening, not 100% sure why but I will find out.  In the meantime, what problem is this causing for you?  Maybe we can find a workaround.

Just heard back, this is the intended behavior.  We notify listeners once per internal events, so in this case it was likely a TOKEN_CHANGE event followed by a USER_INFO_CHANGE event.  Unfortunately we don't expose event type via the SDK so if this is bothering you, I'd suggest keeping track of a timestamp and ignoring events that are too close to each other,

Thanks for your comments. I added additional checks. Where i can see full list of internal evens for FirebaseAuth.AuthStateListener?

@mific we don't publish such a list since it should not matter to your app.  You should only react to the information passed to the listener.

I can reproduce the same issue. onAuthStateChanged is called between 4 to 16 times.

Any resolutions for this so far?

Hi. I have the same issue
In AuthStateListener I am trying to get the auth token. Tokens are fetched multiple times and have a different values sometimes.
I cant store the first token permanently because expiration time is not known
It seems the knowledge about the internal event types would be not superfluous

@ramonz what sort of issues is this causing in your application?  Ideally when using the Firebase Android SDKs you should not need to worry about tokens at all.

Hi again. I need a token to iteracting with my own server app. To get it I use a code like:
class MyActivity extends AppCompatActivity implements FirebaseAuth.AuthStateListener {

    @Override
    public void onStart() {
        super.onCreate();
        FirebaseAuth.getInstance().addAuthStateListener(this);
    }

    @Override
    public void onAuthStateChanged(@NonNull FirebaseAuth firebaseAuth) {
        FirebaseUser user = firebaseAuth.getCurrentUser();
        if (user != null) {
            Log.d("TAG", "onAuthStateChanged:signed_in:" + user.getUid());
            user.getToken(true)
                    .addOnCompleteListener(new OnCompleteListener<GetTokenResult>() {
                        @Override
                        public void onComplete(@NonNull Task<GetTokenResult> task) {
                            if (task.isSuccessful()) {
                                String idToken = task.getResult().getToken();
                                Log.d("TAG", "User token is: " + idToken);
                            }
                        }
                    });
        }
    }
}

And this code causes multiple token requests to google server. In what place of app I could get the token only once and after it refreshed?

I can confirm that this is happening. We should either get a reason for the change, or get only one change. An authentication state change to a developer means either the user has signed in, or has signed out.

I use the auth listener callback to detect whether there is already an opened session.
It works perfectly, so i thought. Now i am faced with a device on which that callback is triggered on app start telling me that there is a signed in user. It does so after a fresh install of an app (or data/cache clear). Further that UID seems to not exist. I cant find it in the Firebase Web interface.
that exprrience is based on a custom app and not the quckstart sample. I did not test the sample with that 'problematic' device yet

I'm getting three calls all at the same time
09-23 14:15:52.584 28961-28961/com.gpease.one D/FirebaseApp: Notifying auth state listeners.
09-23 14:15:52.585 28961-28961/com.gpease.one D/FirebaseApp: Notified 0 auth state listeners.
09-23 14:15:52.585 28961-28961/com.gpease.one D/SignInActivity: onAuthStateChanged:signed_in:vGEYuxs3VTXKnfQ94xXs2m3ZJlE3
09-23 14:15:52.597 28961-28961/com.gpease.one D/FirebaseApp: Notifying auth state listeners.
09-23 14:15:52.598 28961-28961/com.gpease.one D/FirebaseApp: Notified 0 auth state listeners.
09-23 14:15:52.598 28961-28961/com.gpease.one D/SignInActivity: onAuthStateChanged:signed_in:vGEYuxs3VTXKnfQ94xXs2m3ZJlE3
09-23 14:15:52.607 28961-28961/com.gpease.one D/SignInActivity: onAuthStateChanged:signed_in:vGEYuxs3VTXKnfQ94xXs2m3ZJlE3

I believe the intended behavior of the onAuthStateChanged should be what it's name says: fire only when auth state has actually changed. Telling us to just check timestamps and ignore events that are close together is just a workaround.
Would love to see a proper fix.

For me everytime I logout then login again the auth listener fires one extra time. I am starting an activity if my user is not null and since this happening multiple times my apps NavigationDrawer is not working correctly among other things. I think since it's firing multiple times my MainActivity also register multiple listeners causing my onNavigationItemSelected to fire many times on each selection which cause some weird behavior.

For me my listener getting called one extra time on every signout. I found that it is actually the onCreate of my activity that getting called more and more. I wonder if Firebase is the cause of this, as seen here using AsyncTasks can cause issues like this where onCreate is fired multiple times. http://stackoverflow.com/a/22224038/804440

Found what line causing the problem. After running auth = FirebaseAuth.getInstance(); in onCreate() my activities starting to run multiple times. Is this the correct behaviour?

Firebase: Hey, there are changes on the user's auth state.
Devs: Ok. Cool. What is it?
Firebase: You don't need to know. It's only matter for us.
Devs: Ow... Ok. See you later.
... in just one second (or even less?)...
Firebase: Hey, there are changes on the user's auth state.
Devs: Ok. Cool. What is it now?
Firebase: You don't need to know. It's only matter for us.
Devs: ....
Seriously. Us developers only need to know when user is signed in, refreshed tokens (if any) and signed out. And that's basically it. And yet the library spamming us with too many changes that they deem it unnecessary for us. If it unnecessary for us, then why bother informing us (by triggering that event in rapid succession)? Isn't this just hogging down the app performance?

this behaviour makes using firebase for my case pretty useless. so annoying and time wasting because you need to add different variables to track extra state of the app. Also because how could we know when exactly those events are triggered and what is minimum delta time between them?

To me, it seems that this is the same architecture as callbacks for event listeners. As per documentation, they are called twice, once they are created and once when the data changes. Seems that this follows the same pattern.
My workaround is that I introduced a static boolean value that tracks if firebase auth has executed its operation the first time.
My authStateListener looks like this:
private FirebaseAuth.AuthStateListener authStateListener = new FirebaseAuth.AuthStateListener() {
    @Override
    public void onAuthStateChanged(@NonNull FirebaseAuth firebaseAuth) {

        if (!isFirebaseCalledOnce) {

            isFirebaseCalledOnce = true;
            FirebaseUser firebaseUser = firebaseAuth.getCurrentUser();
            if (firebaseUser == null)
                startActivityForResult(AuthUI.getInstance()
                        .createSignInIntentBuilder()
                        .setProviders(Arrays.asList(new AuthUI.IdpConfig.Builder(AuthUI.EMAIL_PROVIDER).build(),
                                new AuthUI.IdpConfig.Builder(AuthUI.GOOGLE_PROVIDER).build(),
                                new AuthUI.IdpConfig.Builder(AuthUI.FACEBOOK_PROVIDER).build(),
                                new AuthUI.IdpConfig.Builder(AuthUI.TWITTER_PROVIDER).build()))
                        .setLogo(R.drawable.ic_splash)
                        .setTheme(R.style.SplashThemeBase)
                        .build(), AUTH_REQ);
            else
                processUserState();

        }

    }

};

It used to cause the processing of user state to run twice, which meant creating the same activity twice.

This is a helpful site. Dont add listener to onCreate() http://www.c-sharpcorner.com/article/google-firebase-login-authentication-implementation-in-android/

Call the removeAuthStateListener after you send the intent/ action
private FirebaseAuth.AuthStateListener mAuthListener = new FirebaseAuth.AuthStateListener() {         @Override         public void onAuthStateChanged(@NonNull FirebaseAuth firebaseAuth) {             if(firebaseAuth.getCurrentUser() == null){                 startActivity(new Intent(MainActivity.this, LoginActivity.class));                 finish();                 mAuth.removeAuthStateListener(mAuthListener);             }         }     };

private FirebaseAuth.AuthStateListener mAuthListener = new FirebaseAuth.AuthStateListener() {
@Override
public void onAuthStateChanged(@NonNull FirebaseAuth firebaseAuth) {
if(firebaseAuth.getCurrentUser() == null){
mAuth.signOut();
startActivity(new Intent(MainActivity.this, LoginActivity.class));
finish();
mAuth.removeAuthStateListener(mAuthListener);
}
}
};

Call the removeAuthStateListener after you send the intent/ action
private FirebaseAuth.AuthStateListener mAuthListener = new FirebaseAuth.AuthStateListener() {     @Override     public void onAuthStateChanged(@NonNull FirebaseAuth firebaseAuth) {         if(firebaseAuth.getCurrentUser() == null){         startActivity(new Intent(MainActivity.this, LoginActivity.class));         finish();         mAuth.removeAuthStateListener(mAuthListener);         }     } };


installtion issue.

why its not in AUR?
im not gonan install CARGO/rust for it

Hi, I'm not the creator of allmytoes.
You can grab a prebuilt binary from here if you want to avoid compiling it yourself. You may also write a PKGBUILD for it and submit it to the AUR.


Add Missing Dependency

That does not help with
An error occurred

learning_to_paint.D1 >! Traceback (most recent call last):
learning_to_paint.D1 >!
  File "/tmp/milabench/1930/venv/torchbench/lib/python3.10/site-packages/voir/forward.py", line 81, in __call__
    yield ov.phases.load_script
learning_to_paint.D1 >!
  File "/tmp/milabench/1930/venv/torchbench/lib/python3.10/site-packages/voir/phase.py", line 183, in _step_one
    next_phase = gen.throw(ph.exception)
learning_to_paint.D1 >!
  File "/tmp/milabench/1930/milabench/milabench/instruments.py", line 383, in profile_gpu
    yield ov.phases.load_script
learning_to_paint.D1 >!
  File "/tmp/milabench/1930/venv/torchbench/lib/python3.10/site-packages/voir/overseer.py", line 55, in run
    func = find_script(script, field)
learning_to_paint.D1 >!
  File "/tmp/milabench/1930/venv/torchbench/lib/python3.10/site-packages/voir/overseer.py", line 90, in find_script
    exec(code, glb, glb)
learning_to_paint.D1 >!
  File "/tmp/milabench/1930/code/torchbench/run.py", line 16, in <module>
    from torchbenchmark import load_model_by_name
learning_to_paint.D1 >!
  File "/tmp/milabench/1930/code/torchbench/torchbenchmark/__init__.py", line 16, in <module>
    from components._impl.workers import subprocess_worker
learning_to_paint.D1 >!
  File "/tmp/milabench/1930/code/torchbench/components/_impl/workers/subprocess_worker.py", line 18, in <module>
    from components._impl.workers import subprocess_rpc
learning_to_paint.D1 >!
  File "/tmp/milabench/1930/code/torchbench/components/_impl/workers/subprocess_rpc.py", line 22, in <module>
    import psutil
learning_to_paint.D1 >!
ModuleNotFoundError: No module named 'psutil'
learning_to_paint.D1 >!


🚚 Template organization

Describe the bug
The templates used in c-pac are currently pulled from a variety of sources, several of which are non-canonical. This should be resolved.
To reproduce
No response
Expected behavior
All templates pulled from canonical sources into a common location, which is then pulled directly into C-PAC.
Acceptance criteria
Clear provenance on where all templates/data sources reside and came from.
Screenshots
No response
C-PAC version
No response
Container platform
No response
Docker and/or Singularity version(s)
No response
Additional context
No response

### GitHub-native task list for those not using ZenHub to view the epic
- [ ] https://github.com/fcp-indi/c-pac_templates/issues/2
- [ ] https://github.com/fcp-indi/c-pac_templates/issues/3
- [ ] https://github.com/fcp-indi/c-pac_templates/issues/4
- [ ] https://github.com/fcp-indi/c-pac_templates/issues/10
- [ ] https://github.com/fcp-indi/c-pac/issues/1738
- [ ] https://github.com/fcp-indi/c-pac_templates/issues/12
- [x] https://github.com/fcp-indi/c-pac_templates/issues/14


inbox: Improve look of empty inbox view.

dark:

light:

discussion: https://chat.zulip.org/#narrow/stream/101-design/topic/Empty.20inbox

How about hsla(147, 40%, 55%, 1) for the dark mode green?

New color and increased the space between the texts.

Changed the "recent conversations" capitalization to match what we do in the spectators experience, and marked to merge once CI passes. Thanks @amanagr and @terpimost!


[Feature Suggestion] Add external index manipulation method

Add an initial index final variable.

Add a method or public function in order to manipulate _selectIndex in widget,
So we can set the selected index without user tap

Agree, there has to be a parameter to set index programmatically,
Initially selected index is not always 0.


Email address

We would like to link email address from GitHub public profile to SMAL IDAP email attribute.

will be tackling this shortly

Thanks @primetheus, this is important for my team as well.  For orgs that have linked SAML Identities, I would expect this tool to be able to link my GitHub account to my AD account using the email address shown on https://github.com/orgs/[org-name]/people/[username]/sso even if my AD username doesn't match my GitHub username.

@clgx-tform @timhirsh the ground work is laid for this one in #33. Initial testing worked, but we're looking at ways we can improve validation, particularly when users have private email addresses or the email isn't found in their account. The initial thought is that we can create an issue detailing the failure and listing the users, but I want to make sure that is a good approach. #33 introduces the ability to open issues on failure, and I'll be adding issue templates shortly as well. I may be able to have this feature kicked out by next week

There is currently a USERNAME_ATTRIBUTE variable that can be set. This will allow for a custom username attribute to map to the GitHub username. Emails are returned, but since synchronizing via email depends on users marking their email as public, it would make this somewhat unreliable to implement.

Hey @primetheus,
If we were to implement this for LDAP as well is this the only line we would need to change, other than updating the env to get it?
https://github.com/github/github-team-sync/blob/ff290cca32aba6ca8fd64b53b4995c4329016f13/githubapp/ldap.py#L77

@Chocrates no, that wouldn't be something we want to change...
https://github.com/github/github-team-sync/blob/main/githubapp/ldap.py#L80
☝🏾 already captures the email address for the user, and if we change that from uid to something else it will begin looking in LDAP for whatever attribute we set it to. For example, If we're using AD instead of OpenLDAP, the LDAP_USER_ATTRIBUTE will be sAMAccountName instead of uid.
Where we want to update is the logic found in https://github.com/github/github-team-sync/blob/main/app.py#L41-L78
☝🏾 is hardcoded to look for username within the dictionary ({"username": "<value>", "email": "<value>"}), and we'll want to make that a new setting that is passed into the method


Allow NaN in the images

Fixes Issue 31.

Coverage increased (+0.07%) to 82.12% when pulling 36ee72286400b638a0ce77d907ab556674b0abe2 on nan_in_elbo into 5dbb959e0dd56fa0e40eec79da1010930313a6d4 on master.


Think about extending the Nextflow tutorial

Think about extending the Nextflow tutorial to mirror the Snakemake tutorial, based on course feedback.

The Nextflow material will be extended, as per discussions 220124.


chore: add lint test

Description
This PR:

adds lint task.
adds yarn lint in Travis ci build
fixes lint errors

resolves #143

Your checklist for this pull request
:rotating_light: Please review this items for a good pull request. :four_leaf_clover:

I've read the project's Contributing Guidelines
My commits are well written and follow pagarme/git-style-guide
My changes are well covered by tests and logs
I've updated the project docs (if needed)
I fell safe about this implementation

In a good pull request, everything above is true :relaxed:

Só corrigir o typo na mensagem do commit ae95348 e ta show!

coirrigido =)


adding LTR MI tests

Adding clients + test file for Managed Instance Long Term Retention Tests

Added the missing models (also added many unrelated files in attempt to sync local repo). Will remove out-of-scope files
Here is the swagger PR: https://github.com/Azure/azure-rest-api-specs/pull/8266


fix(constants.py): update dev version

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


Zhipuai

Description
Added LLM related to zhipuai The function files of image loader and embedder. Relevant modules can be used through configuration files.
dependencies ： zhipuai>=2.0.1
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)
[x] Documentation update

How Has This Been Tested?
I only added three files and added three functions to the related schama. After detailed testing. The test files are tests.llm.test_zhipuai.py and the three test cases in the documentation
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)
[x] Documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[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] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

Maintainer Checklist

[x] closes #xxxx (Replace xxxx with the GitHub issue number)
[x] Made sure Checks passed

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

Hey @xunhanliu
Thank you for your contribution! At the moment, we’re focusing on integrations that are highly requested by the community. We appreciate your effort and will keep this in mind for future priorities.


Update bravepkg.sh

The appNewVersion variable is broken in the current label. This change fixes the problem.

Thank you!


RHSUSAF CH-53 Crates are in the wrong position.

Bug report

Basic Information

Mission version:
KP_TakistanACRE
Map used:
Takistan
Mods used:
9Liners & Notepad | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=1210050323
ace | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=463939057
ACE 3 Extension (Animations and Actions) | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=766491311
ACE Compat - RHS Armed Forces of the Russian Federation | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=773131200
ACE Compat - RHS United States Armed Forces | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=773125288
ACE Compat - RHS: GREF | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=884966711
ACE Interaction Menu Expansion | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=1376867375
ACEX | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=708250744
Achilles | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=723217262
ACRE2 | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=751965892
CBA_A3 | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=450814997
CUP Terrains - Core | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=583496184
CUP Terrains - Maps | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=583544987
Enhanced Movement | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=333310405
Enhanced Soundscape | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=825179978
F-15 Eagle | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=582478120
FA-18 Super Hornet | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=743099837
FIR AWS(AirWeaponSystem) | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=366425329
FIR Pilot&Crew Pack | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=433566590
JSRS SOUNDMOD | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=861133494
JSRS SOUNDMOD - RHS AFRF Mod Pack Sound Support | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=945476727
JSRS SOUNDMOD - RHS GREF Mod Pack Sound Support | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=1180534892
JSRS SOUNDMOD - RHS USAF Mod Pack Sound Support | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=1180533757
MCC Sandbox 4 - Mission Making The Easy Way | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=338988835
Philippine Army Uniforms | Local | http://www.armaph.net
Project OPFOR | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=735566597
RHSAFRF | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=843425103
RHSGREF | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=843593391
RHSSAF | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=843632231
RHSUSAF | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=843577117
ShackTac User Interface | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=498740884
USAF MOD Fighters Pack | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=1891343103
USAF MOD Main Pack | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=1891346714
USAF MOD Utility pack | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=1891349162
USS Nimitz | Steam | http://steamcommunity.com/sharedfiles/filedetails/?id=643530417
Dedicated or local game:
Dedicated
Source of the mission file (URL):
Recent Github link for KP_ACRE
Individual Things
Did you edit anything within the mission files?
Yes, rhs_usaf_des preset
If yes, what did you edit exactly?
Added vehicles included in RHS that are otherwise not included in the Vanilla release of KP GitHub
Short Issue Description
When crates are loaded into the CH53, the create position is outside of the body.
Expected Behaviour

When crates are loaded into the CH53, the create position should be within the fuselage.
Steps to Reproduce the Issue

Open Eden editor, force Zeus mode, plop down the CH53, plop down a supply crate, edit the init field of the CH53 and paste the following; ["rhsusf_CH53E_USMC_D", -11.0, [0,2.5,1.4]]
Use the attach to line to attach the crate to the CH53.
Screenshots, Serverlog or any other helpful Information

Fixed in 6fe4f99935a5ee1e943e4e654ecf7c0add34f750 will be available with 0.96.7 update.

Thanks for the update veteran29


Freq stats not being honored

If user sets stats on proc_freq, not being honored.  Using default.

SAS does not have a stats parameter on freq.  Uses options instead.  Would be confusing.


Support new hardware scsgate

Description:
Related issue (if applicable): fixes #
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:

[x] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Hello @terminet85,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

c64e2c4e1436a00ca9dcadb868a2afe4fae34367 - This commit has something that looks like an email address (emanuele@leuzzi.me). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:

Hello @terminet85,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

c64e2c4e1436a00ca9dcadb868a2afe4fae34367 - This commit has something that looks like an email address (emanuele@leuzzi.me). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:

Can you put this PR on hold?  It depends on the merge of https://github.com/flavio/scsgate/pull/3 which changes to the API of the flavio/scsgate library.
Let's wait for the code to be merged to see how the final API will look like.

Please feel free to make whatever you want. I'm not a coder. I committed this 'cause many people asked me this by email. Sorry if I made mistakes.


Login Issues

Hi there,
I can’t seem to find any way to reach out for support, so I’m blindly trying here.
I have been a keybase.io member for a looong time and way back when I chose to keep my private key in the system so I could use the browser interface. At some point I installed keybase on my phone, but never really used it there. Since that install I’ve upgraded phones and apparently the device key associated with it was lost.
Now, I’d like to install and use the apps on my phone and on my Mac, but, apparently it appears I’m required to authenticate with that now obsolete phone install before I can login with the apps or even on the command line. I can login to the website without issue, so I’m frustrated that I can not authenticate my devices, and I’m really not interested (nor do I think it is required) in resetting my keys.
Your help would be greatly appreciated.
Thank you,
Levi

See: #19597

Thanks, but that's just it... I don't understand why I would need to (and I don't want to) reset my keys. I have the ability to sign/encrypt with the keys I have access to through the web interface... I can even export them locally. Is there some way I can use these keys to enable my desktop/iOS apps too?

What should I do if I created an account, then gave the first device a name and submit but it reports some errors but I want to reset because I am not set a password and Keybase says that you can't reset accounts without email? Any feedback on my issue?


Remove pkg_resources usage (move to importlib.metadata & friends)

The simulator aims to have no dependencies at runtime, yet it does actually need setuptools at the moment. Now that importlib.metadata is in all supported versions of Python we should probably be able to use that instead. pkg_resources also seems like something the Python community is likely to try to deprecate over time, so there are other benefits from doing this.
One challenge I see here is ensuring we keep useful semantics, in particular working out what happens to PyTorch's +cpu versions on various platforms. That may be something we can either handle explicitly ourselves or ignore until it becomes an issue though.

I've just spotted that in the latest version of webots (possibly previously too?) there's a red warning about this. We therefore probably want to pick this up for SR2024, though at low priority.