System.MissingMethodException Method not found: 'Void Microsoft.VisualStudio.CodeCoverage.Shim.1017596829__coverage_runtime_method__32412_19337__(UInt64)'.

Description
When using the /Enablecodecoverage option on vstest.console.exe,  we occasionally see a MissingMethodException  about the injected Shim method for the code coverage.  This appears during calls than span across a new AppDomain.
i.e.
System.MissingMethodException
Method not found: 'Void Microsoft.VisualStudio.CodeCoverage.Shim.2218451032__coverage_runtime_method__56282_18307__(UInt64)'.
Please this this related post we created about 4 years ago. This post is slightly different because it's dealing with IIS,  but might be related due to dealing with AppDomains.
https://social.msdn.microsoft.com/Forums/vstudio/en-US/3f7415ed-7e17-496b-b705-3055c48a4406/iis-codecoverage-occasional-method-not-found-microsoftvisualstudiocodecoverageshim?forum=vsunittest
Steps to reproduce

Develop a unit test that makes a call across an app domain.  You'll occasionally see this exception (not always).

Expected behavior

There should be no exception

Actual behavior

We see the MissingMethodException

Diagnostic logs
Example failed test stack trace:
Failed   SaveTwitterSearchData
Test method UnitTests.SocialTests.SaveTwitterSearchData threw exception:
System.ServiceModel.FaultException`1[CompAnalytics.Contracts.Fault.ExecutionFault]: Exception has been thrown by the target of an invocation.
Stack Trace:
Server stack trace:
System.Reflection.TargetInvocationException
Exception has been thrown by the target of an invocation.
at System.RuntimeMethodHandle.InvokeMethod(Object target, Object[] arguments, Signature sig, Boolean constructor)
at System.Reflection.RuntimeConstructorInfo.Invoke(BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
at System.RuntimeType.CreateInstanceImpl(BindingFlags bindingAttr, Binder binder, Object[] args, CultureInfo culture, Object[] activationAttributes, StackCrawlMark& stackMark)
at System.Activator.CreateInstance(Type type, BindingFlags bindingAttr, Binder binder, Object[] args, CultureInfo culture, Object[] activationAttributes)
at System.Activator.CreateInstanceFromInternal(String assemblyFile, String typeName, Boolean ignoreCase, BindingFlags bindingAttr, Binder binder, Object[] args, CultureInfo culture, Object[] activationAttributes, Evidence securityInfo)
at System.AppDomain.InternalCreateInstanceFromWithNoSecurity(String assemblyName, String typeName, Boolean ignoreCase, BindingFlags bindingAttr, Binder binder, Object[] args, CultureInfo culture, Object[] activationAttributes, Evidence securityAttributes)
at System.AppDomain.InternalCreateInstanceFromWithNoSecurity(String assemblyName, String typeName, Boolean ignoreCase, BindingFlags bindingAttr, Binder binder, Object[] args, CultureInfo culture, Object[] activationAttributes, Evidence securityAttributes)
at System.Activator.CreateInstanceFrom(AppDomain domain, String assemblyFile, String typeName, Boolean ignoreCase, BindingFlags bindingAttr, Binder binder, Object[] args, CultureInfo culture, Object[] activationAttributes)
at CompAnalytics.Execution.Isolation.SandBox..ctor(ExecutionContext context) in C:\agent_work\1\s\Product\CompAnalytics.Execution\Isolation\SandBox.cs:line 91
at CompAnalytics.Execution.Isolation.SandBox.Create(ExecutionContext context) in C:\agent_work\1\s\Product\CompAnalytics.Execution\Isolation\SandBox.cs:line 52
at CompAnalytics.Execution.Isolation.SandBoxExtension.Create() in C:\agent_work\1\s\Product\CompAnalytics.Execution\SandBoxExtension.cs:line 15
at CompAnalytics.Execution.CodeModuleExecutor.Execute(IExecutionContext context) in C:\agent_work\1\s\Product\CompAnalytics.Execution\CodeModuleExector.cs:line 76
at CompAnalytics.Execution.ExecutionContext.ExecuteModuleWithRetry(ModuleExecutor executor, Module module) in C:\agent_work\1\s\Product\CompAnalytics.Execution\ExecutionContext.cs:line 1066
at CompAnalytics.Execution.ExecutionContext.ExecuteModule(ModuleExecutor executor, Module module, ExecutionCallback postExecutionCallback) in C:\agent_work\1\s\Product\CompAnalytics.Execution\ExecutionContext.cs:line 900
System.MissingMethodException
Method not found: 'Void Microsoft.VisualStudio.CodeCoverage.Shim.2218451032__coverage_runtime_method__56282_18307__(UInt64)'.
at CompAnalytics.Execution.Isolation.RemoteSandBox..ctor(IsolatedTraceRedirector traceRedirector)
at CompAnalytics.IServices.ClientMessageInspector.AfterReceiveReply(Message& reply, Object correlationState) in C:\agent_work\1\s\Product\CompAnalytics.IServices\ErrorHandler.cs:line 161
at System.ServiceModel.Dispatcher.ImmutableClientRuntime.AfterReceiveReply(ProxyRpc& rpc)
at System.ServiceModel.Channels.ServiceChannel.HandleReply(ProxyOperationRuntime operation, ProxyRpc& rpc)
at System.ServiceModel.Channels.ServiceChannel.Call(String action, Boolean oneway, ProxyOperationRuntime operation, Object[] ins, Object[] outs, TimeSpan timeout)
at System.ServiceModel.Channels.ServiceChannel.Call(String action, Boolean oneway, ProxyOperationRuntime operation, Object[] ins, Object[] outs)
at System.ServiceModel.Channels.ServiceChannelProxy.InvokeService(IMethodCallMessage methodCall, ProxyOperationRuntime operation)
Environment
Windows Server 2012 R2
Test Execution Command Line Tool Version 14.0.25463.0

We are seeing this out of the blue starting today. Nothing has changed in the code or the build environments.

We are also getting the same error for our gated builds and it started out of nowhere.
VSTS test task fails with below error for 50% of unit test cases:
Error: System.MissingMethodException: Method not found: 'Void Microsoft.VisualStudio.CodeCoverage.Shim.4251430275__coverage_runtime_method__28318_17131__(UInt64)'
Any update is highly appreciated.

Also having nearly half our builds on Azure Devops fail with this issue.

We did a release of 16.7.1 yesterday, are your builds picking up that version of TestPlatform installer task?

This is same issue as https://github.com/microsoft/azure-pipelines-tasks/issues/12808
You can find workaround there.
@ShreyasRmsft  what is the issue here?

We are still experiencing build failures during test phase running code coverage in Azure devops as part of the PR pipeline. The workaround suggests providing path to empty directory for adapters and we have tried adding -a ./path/to/empty/dir but that didn't help.
Roughly 50% of our builds are now failing.
This is excerpt from one of failures.
2020-10-29T07:55:46.0883405Z [command]"C:\Program Files\dotnet\dotnet.exe" test D:\a\1\s\Tests\IntegrationTests\IntegrationTests.csproj --logger trx --results-directory D:\a\_temp --no-restore --configuration Release "--collect:Code Coverage" -s CodeCoverage.runsettings
2020-10-29T07:55:54.1023258Z Test run for D:\a\1\s\Tests\IntegrationTests\bin\Release\netcoreapp3.1\IntegrationTests.dll(.NETCoreApp,Version=v3.1)
2020-10-29T07:55:54.2460466Z Microsoft (R) Test Execution Command Line Tool Version 16.7.0
2020-10-29T07:55:54.2461279Z Copyright (c) Microsoft Corporation.  All rights reserved.
2020-10-29T07:55:54.2461620Z
2020-10-29T07:55:54.4303001Z Starting test execution, please wait...
2020-10-29T07:55:55.3598112Z
2020-10-29T07:55:55.3599439Z A total of 1 test files matched the specified pattern.
2020-10-29T07:55:57.8844196Z An exception occurred while invoking executor 'executor://xunit/VsTestRunner2/netcoreapp': Method not found: 'Void Microsoft.VisualStudio.CodeCoverage.Shim.2481202972__coverage_runtime_method__41931_20370__(UInt64)'.
2020-10-29T07:55:57.9940605Z M�i�c�r�o�s�o�f�t� �(�R�)� �C�o�v�e�r�a�g�e� �C�o�l�l�e�c�t�i�o�n� �T�o�o�l� �V�e�r�s�i�o�n� �1�6�.�0�.�3�0�3�1�9�.�2�0�0�
2020-10-29T07:55:57.9941354Z �
2020-10-29T07:55:57.9941560Z �
2020-10-29T07:55:57.9942237Z �C�o�p�y�r�i�g�h�t� �(�c�)� �M�i�c�r�o�s�o�f�t� �C�o�r�p�o�r�a�t�i�o�n�.� � �A�l�l� �r�i�g�h�t�s� �r�e�s�e�r�v�e�d�.�
2020-10-29T07:55:57.9942739Z �
2020-10-29T07:55:57.9942932Z �
2020-10-29T07:55:57.9943371Z �
2020-10-29T07:55:57.9943647Z �
2020-10-29T07:55:58.1713110Z �Results File: D:\a\_temp\VssAdministrator_WIN-I03V8NT17AF_2020-10-29_07_55_58.trx
2020-10-29T07:55:58.1713746Z
2020-10-29T07:55:58.1714225Z Attachments:
2020-10-29T07:55:58.1714878Z   D:\a\_temp\89168bd3-ad1a-4562-bbdd-1da12060b5ef\VssAdministrator_WIN-I03V8NT17AF_2020-10-29.07_55_55.coverage
2020-10-29T07:55:58.1731074Z Test Run Failed.
2020-10-29T07:55:58.1994938Z C:\Program Files\dotnet\sdk\3.1.403\Microsoft.TestPlatform.targets(32,5): error MSB4181: The "Microsoft.TestPlatform.Build.Tasks.VSTestTask" task returned false but did not log an error. [D:\a\1\s\Tests\IntegrationTests\IntegrationTests.csproj]
2020-10-29T07:55:58.3311826Z ##[error]Error: The process 'C:\Program Files\dotnet\dotnet.exe' failed with exit code 1

Start of the log file with version:
2020-10-29T07:54:24.3728850Z ##[section]Starting: dotnet test Release 2020-10-29T07:54:24.3911353Z ============================================================================== 2020-10-29T07:54:24.3911686Z Task         : .NET Core 2020-10-29T07:54:24.3912011Z Description  : Build, test, package, or publish a dotnet application, or run a custom dotnet command 2020-10-29T07:54:24.3912343Z Version      : 2.175.0 2020-10-29T07:54:24.3912567Z Author       : Microsoft Corporation 2020-10-29T07:54:24.3912901Z Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/build/dotnet-core-cli 2020-10-29T07:54:24.3913274Z ============================================================================== 2020-10-29T07:54:25.5543381Z [command]C:\windows\system32\chcp.com 65001 2020-10-29T07:54:25.5717495Z Active code page: 65001 2020-10-29T07:54:25.6071009Z Info: .NET Core SDK/runtime 2.2 and 3.0 are now End of Life(EOL) and have been removed from all hosted agents. If you're using these SDK/runtimes on hosted agents, kindly upgrade to newer versions which are not EOL, or else use UseDotNet task to install the required version.
Any updates or help is appreciated.

This is same issue as microsoft/azure-pipelines-tasks#12808
You can find workaround there.
@ShreyasRmsft what is the issue here?

@ShreyasRmsft  Any updates?

FYI
After downgrading packages in our test projects issue has not reproduced:
    <PackageReference Include="Microsoft.AspNetCore.Mvc.Testing" Version="3.1.9" />   ->   "3.1.4"
    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="16.7.1" />            ->  "16.6.1"
    <PackageReference Include="xunit.runner.visualstudio" Version="2.4.3">            ->  "2.4.1"

This is still an active issue for my company something like half of our builds are failing with this error and having to go in and manually re-run them is a lot of overhead.

I also worked around this by downgrading Microsoft.NET.Test.Sdk from 16.8.3 to 16.6.1

I also worked around this by downgrading Microsoft.NET.Test.Sdk from 16.8.3 to 16.6.1

There is 1 more way of fixing the issue in case of .NET Core, .NET Standard and .NET 5: you can start using unverifiable probes. Unverifiable probes don't require Microsoft.VisualStudio.CodeCoverage.Shim dependency.
To enable unverifiable probes please set in your runsettings:
<?xml version="1.0" encoding="utf-8"?>
<!-- File name extension must be .runsettings -->
<RunSettings>
  <DataCollectionRunSettings>
    <DataCollectors>
      <DataCollector friendlyName="Code Coverage" uri="datacollector://Microsoft/CodeCoverage/2.0" assemblyQualifiedName="Microsoft.VisualStudio.Coverage.DynamicCoverageDataCollector, Microsoft.VisualStudio.TraceCollector, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a">
        <Configuration>
          <CodeCoverage>
......
            <UseVerifiableInstrumentation>False</UseVerifiableInstrumentation>
......
          </CodeCoverage>
        </Configuration>
      </DataCollector>
    </DataCollectors>
  </DataCollectionRunSettings>
</RunSettings>

If you don't use runsettings you can add  -- DataCollectionRunSettings.DataCollectors.DataCollector.Configuration.CodeCoverage.UseVerifiableInstrumentation="False" suffix to your command. For example:
dotnet test --collect "Code Coverage" --diag "log.txt" -- DataCollectionRunSettings.DataCollectors.DataCollector.Configuration.CodeCoverage.UseVerifiableInstrumentation="False"

We will make unverifiable probes default for all .NET Core and .NET 5 projects in VS 16.10.

There is 1 more way of fixing the issue in case of .NET Core, .NET Standard and .NET 5: you can start using unverifiable probes. Unverifiable probes don't require Microsoft.VisualStudio.CodeCoverage.Shim dependency.
To enable unverifiable probes please set in your runsettings:
<?xml version="1.0" encoding="utf-8"?>
<!-- File name extension must be .runsettings -->
<RunSettings>
  <DataCollectionRunSettings>
    <DataCollectors>
      <DataCollector friendlyName="Code Coverage" uri="datacollector://Microsoft/CodeCoverage/2.0" assemblyQualifiedName="Microsoft.VisualStudio.Coverage.DynamicCoverageDataCollector, Microsoft.VisualStudio.TraceCollector, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a">
        <Configuration>
          <CodeCoverage>
......
            <UseVerifiableInstrumentation>False</UseVerifiableInstrumentation>
......
          </CodeCoverage>
        </Configuration>
      </DataCollector>
    </DataCollectors>
  </DataCollectionRunSettings>
</RunSettings>

If you don't use runsettings you can add  -- DataCollectionRunSettings.DataCollectors.DataCollector.Configuration.CodeCoverage.UseVerifiableInstrumentation="False" suffix to your command. For example:
dotnet test --collect "Code Coverage" --diag "log.txt" -- DataCollectionRunSettings.DataCollectors.DataCollector.Configuration.CodeCoverage.UseVerifiableInstrumentation="False"

We will make unverifiable probes default for all .NET Core and .NET 5 projects in VS 16.10.

@jakubch1 Seemingly after your proposed workaround, our pipelines no longer collect any coverage information:
"Coverage for changed lines cannot be determined. Coverage data not found."
The tests are run as before, and the output of the test process implies that the results are published
Our CodeCoverage.runsettings:
<?xml version="1.0" encoding="utf-8"?>
<RunSettings>
  <DataCollectionRunSettings>
    <DataCollectors>
      <DataCollector friendlyName="Code Coverage" uri="datacollector://Microsoft/CodeCoverage/2.0" assemblyQualifiedName="Microsoft.VisualStudio.Coverage.DynamicCoverageDataCollector, Microsoft.VisualStudio.TraceCollector, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a">
        <Configuration>
          <CodeCoverage>
            <!-- Match the path of the source files in which each method is defined: -->
            <Functions>
              <Exclude>
                <!-- Removes Migrations namespace from code coverage-->
                <Function>^Persistence.Migrations\..*</Function>
              </Exclude>
            </Functions>
            <Sources>
              <Exclude>
                <!-- Removes translation files from code coverage -->
                <Source>.*\\Translations\\.*</Source>
                <Source>.*Designer.cs</Source>
              </Exclude>
            </Sources>
            <!-- We recommend you do not change the following values: -->

            <!-- Set this to True to collect coverage information for functions marked with the "SecuritySafeCritical" attribute. Instead of writing directly into a memory location from such functions, code coverage inserts a probe that redirects to another function, which in turns writes into memory. -->
            <!-- Disabled due to https://github.com/microsoft/vstest/issues/2459 -->
            <UseVerifiableInstrumentation>False</UseVerifiableInstrumentation>
            <!-- When set to True, collects coverage information from child processes that are launched with low-level ACLs, for example, UWP apps. -->
            <AllowLowIntegrityProcesses>True</AllowLowIntegrityProcesses>
            <!-- When set to True, collects coverage information from child processes that are launched by test or production code. -->
            <CollectFromChildProcesses>True</CollectFromChildProcesses>
            <!-- When set to True, restarts the IIS process and collects coverage information from it. -->
            <CollectAspDotNet>False</CollectAspDotNet>
          </CodeCoverage>
        </Configuration>
      </DataCollector>
    </DataCollectors>
  </DataCollectionRunSettings>
</RunSettings>

And the pipeline:
# Build and test ASP.NET projects.
# Add steps that publish symbols, save build artifacts, deploy, and more:
# https://docs.microsoft.com/azure/devops/pipelines/apps/aspnet/build-aspnet-4

pool:
  vmImage: 'windows-2019'

variables:
  buildPlatform: 'Any CPU'
  buildConfiguration: 'Release'

steps:

- task: DotNetCoreCLI@2
  inputs:
    command: 'restore'
    feedRestore: 'env-common'
    includeNuGetOrg: true
  displayName: 'dotnet restore'

- task: DotNetCoreCLI@2
  displayName: 'dotnet build $(buildConfiguration)'
  inputs:
    command: build
    arguments: '--configuration $(BuildConfiguration) oursolution.sln --no-restore'

- task: DotNetCoreCLI@2
  displayName: 'dotnet test $(buildConfiguration)'
  inputs:
    command: test
    arguments: '--no-restore --configuration $(BuildConfiguration) --collect:"Code Coverage" -s "CodeCoverage.runsettings"'
    projects: 'Tests/**/*.csproj'

@spaasis  could you please post also output from dotnet test command? Could you please run it locally and see if .coverage file is generated?

@jakubch1  sure, here's the raw output from the pipeline (it's a bit verbose..): https://pastebin.com/bijKZ5d8
Running dotnet test --no-restore --configuration Release --collect:"Code Coverage" -s "CodeCoverage.runsettings" does generate a code coverage result in e.g. /Tests/IntegrationTests/TestResults/

@spaasis in both places code coverage report was generated:
2021-02-03T10:26:09.7518355Z Attachments:
2021-02-03T10:26:09.7518874Z   D:\a\_temp\8e991836-327b-44b2-82ec-991573fa95ac\VssAdministrator_fv-az350-317_2021-02-03.10_26_04.coverage

could you please send me those .coverage files for investigation? [You can add it in pipeline to artifacts if it's not accessible.]

@jakubch1 Here you go - seems to open correctly in VS and have (maybe) proper data.
20210204.1.Release.Any CPU.9239.zip
Could the issue be then in the Pipeline reporter?
Our azurepipelines-coverage.yml:
coverage:
  status:           #Code coverage status will be posted to pull requests based on targets defined below.
    comments: on    #Off by default. When on, details about coverage for each file changed will be posted as a pull request comment.

@ganesp @ShreyasRmsft  could you please help on this? I've checked that coverage file is generated and it contains data for example:
        <source_file id="5" path="D:\a\1\s\Core\Application\Cases\Commands\CreateOrUpdateCaseCommand.cs" />

Huh.. All of a sudden one pipeline reported the coverage successfully..
Maybe something changed on Azure's end?

@ganesp to help out here.

For a couple days our code coverage reports have worked as expected

We are still experiencing this sporadically for our DevOps pipeline running .NET 5.

@acn-sbuad what version of TestPlatform do you use? Did you put
<UseVerifiableInstrumentation>False</UseVerifiableInstrumentation>

to your runsettings?

Why would the verifiable instrumentation work sometimes and not work other times?  In all the scenarios that folks have listed, it's not like it never works or always works.  It's an  occasional failure of the missing method exception.

FYI
After downgrading packages in our test projects issue has not reproduced:
    <PackageReference Include="Microsoft.AspNetCore.Mvc.Testing" Version="3.1.9" />   ->  "3.1.4"
    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="16.7.1" />            ->  "16.6.1"
    <PackageReference Include="xunit.runner.visualstudio" Version="2.4.3">            ->  "2.4.1"

In my case I only needed to downgrade Microsoft.NET.Test.Sdk (from 16.8.3) to 16.6.1

I had this error pop up for 1 of my 5 solutions being build in DevOps self hosted docker agent, but only some of the time.
Upgrading nugets related to testing helped me.
Eg:
  --was 16.5.0 in this solution
 --was 2.4.0
 ...   --was 2.4.0
I also added missing .csproj properties:
AnyCPU
false

Closing this issue. In latest VS and stable Microsoft.CodeCoverage packages we use non verifiable instrumentation.

FYI
After downgrading packages in our test projects issue has not reproduced:
    <PackageReference Include="Microsoft.AspNetCore.Mvc.Testing" Version="3.1.9" />   ->  "3.1.4"
    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="16.7.1" />            ->  "16.6.1"
    <PackageReference Include="xunit.runner.visualstudio" Version="2.4.3">            ->  "2.4.1"

In my case I only needed to downgrade Microsoft.NET.Test.Sdk (from 16.8.3) to 16.6.1

How do I know the correct version should be 16.6.1 instead of 16.8.3? Is there any method for reference?
Thanks!


Update store locator JS

closes #129
closes #120

I've been bouncing back and forth between a fresh install, and Perlick. Works well in both instances.
Perlick required minimal work in config to integrate


moonlander: init at unstable-2021-05-23

Description of changes
Fixes: #230028

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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2181

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

1 package built:

moonlander


Amend configs to use Pydantic-style discriminator

Remove Sinks that were used prior to isolated devices being supported

Required https://github.com/dls-controls/tickit/pull/154

@abbiemery @tpoliaw @callumforrester this is ready for review now that the eiger changes are in and everything is behaving.


Add option to save current terminal config

Right now, you can specify the terminals you want restored in a JSON file. But it would be great if I could open my terminals, organise them (change directories, name, etc) and then run a command to save that current configuration to the JSON file.

I second this. It will be extremely useful feature.


avatar

If the user has been removed, the avatar is not deleted from the table system_files.

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

I fixed this issue.


Output code warnings in yellow instead of red

Addresses issue #3401.

Added test.

thanks @jez9999


[MDM-404] When an admin creates a user account, show the user's default password

In EMM 1.1.0, when an admin would create a new user, it would show the admin the user's default password along with the QR code for downloading the app.This functionality appears to have been removed in 2.0.0, opting to instead only send the user their password via email. Unfortunately this will not work for anyone intending on using this software on a network without internet access, like myself. Please enable this functionality again!Reference: https://wso2.org/jira/browse/MDM-404

IoT 3.1.0 onwards we do have password reset functionality and hence for the newly created user, the admin user can provide the password for that user after the successful user creation. Please refer the attached screenshot. This functionality doesn't require, any internet connection for you to proceed with this operation. As this requirement is already handled with different feature, I'm closing this issue.


A few small bugs

Issue 1: Default Checkbox State for Implant Categories
Description: When adding a product, all the implant category checkboxes are checked by default, which may lead to unintentional selections.
Steps to Reproduce:
Go to the "Product" section, press "+"
Observe that all implant category checkboxes are pre-checked.

Expected Behavior: The checkboxes for implant categories should be initially unchecked, allowing the user to select the relevant categories.
Issue 2: Missing "Save" Button for Upgrade Editing
Description: When editing an upgrade, there is no "Save" button. Instead, users need to press the 'X' button to close the editing window, which can be confusing.
Steps to Reproduce:
Go to the "My Upgrade" section, go to "Edit upgrade"
Observe that there is no explicit "Save" button.

Expected Behavior: There should be a clear "Save" button to confirm and save the changes made to an upgrade.
Issue 3: Lack of Image Caching for Self-Added Products
Description: The images for self-added products are not cached, resulting in slower loading times, especially on slow internet connections.
Steps to Reproduce:
Add a self-created product.
Reload the page or navigate away and return.
Observe that the product image takes a while to load.

Expected Behavior: Images for self-added products should be cached to improve page loading performance.
Issue 4: Accidental Removal of Upgrades
Description: The "remove upgrade" button is easily pressed by accident, and there is no confirmation dialog to confirm the deletion of the upgrade.
Steps to Reproduce:
Go to the upgrades section, click "Edit upgrade" and scroll down
Click the "remove upgrade" button.

Expected Behavior: There should be a confirmation dialog asking the user to confirm the deletion of the upgrade.
Issue 5: Overlapping Default Image and Symbol
Description: The default image overlaps with a symbol in the bottom-right corner (as shown in the provided screenshot).
Steps to Reproduce:
Create a new product.

Expected Behavior: The default image should not overlap with any symbols.
Issue 6: Missing "About" Section for App Version Information
Description: There is no "About" section where users can view the installed app version, which can be important for troubleshooting and support purposes.
Steps to Reproduce:
Navigate through the app interface.
Look for an "About" or "Version" section.

Expected Behavior: There should be an "About" section that displays information about the installed app version.
Additional Notes:
Screenshots have been provided for Issue 5.
This report has been submitted on 21.10.23.
Device Information: [star2lte SM-G965F Android 11]
Zinc version <IP_ADDRESS>.2

Issue 1, fixed

Issue 2, fixed

Issue 3, won't fix now. Moved to its own ticket

Issue 4, fixed with "Hold buttons" for upgrades and products as well.

Issue 5, fixed.

Issue 6, fixed with version display.


pack switch nightly-240524 : error compilong Pretty.idr

Hello! After a fresh install of pack (nightly-240909), I tried
pack switch nightly-240519
and got the error :
[ build ] 126/281: Building Idris.Pretty (src/Idris/Pretty.idr)
[ build ] Error: Couldn't parse any alternatives:
[ build ] 1: Unexpected end of expression.
[ build ]
[ build ] Idris.Pretty:311:5--311:17
[ build ]  307 |     prettyPrec d (PDelayed _ _ ty) = parenthesise (d > startPrec) $ "Lazy" <++> prettyPrec appPrec ty
[ build ]  308 |     prettyPrec d (PDelay _ tm) = parenthesise (d > startPrec) $ "Delay" <++> prettyPrec appPrec tm
[ build ]  309 |     prettyPrec d (PForce _ tm) = parenthesise (d > startPrec) $ "Force" <++> prettyPrec appPrec tm
[ build ]  310 |     prettyPrec d (PAutoApp _ f a) =
[ build ]  311 |     parenthesise (d > startPrec) $ group $ prettyPrec leftAppPrec f <++> "@" <+> braces (pretty a)
[ build ]            ^^^^^^^^^^^^
[ build ] ... (2 others)
make: *** [Makefile:73: /home/zlizmk/.pack/.tmp0/idris2-compiler/build/exec/idris2] Error 1
[ fatal ] Error when executing system command.
          Command: "make" "idris2-exec" "PREFIX=/home/zlizmk/.pack/install/cf68e995c414912c338ae694a74fd833a6c4cee2/idris2" "SCHEME=chezscheme"
          Error code: 2

As a side remark : I have looked for a way to directly install nightly-240519, but  could not find how to do this. Version nightly-240519 is the version deployed at my college and I need the same version on my computer.
Thank you.

Try pack --bootstrap switch nightly-240519. This will use the release version of the compiler rather than the current one, but will increase the time of switching.

Indeed, it worked, many thanks, and sorry if that was documented somewhere and I'd missed it.

Pack should actually do this automatically: If building Idris without bootstrapping fails, bootstrapping should be tried instead. I opened a PR for this: #306.


.NET Core September 2019 Update - 2.2.7 and 2.1.13

2.1.13 Release Notes
2.1.13 Download
2.2.7 Release Notes
2.2.7 Download
Blog posts
.NET Core
Please report any issues you find with 2.1.13 or 2.2.7, either responding to this issue, creating a new issue or creating a new issue in one of the following repos:

aspnet/Home - for ASP.NET Core questions
aspnet/EntityFrameworkCore - for Entity Framework Core questions
dotnet/cli - for CLI tools and questions
dotnet/corefx - for API issues and questions
dotnet/coreclr - for runtime issues
dotnet/core-setup - for installer issues

hi @leecow the first four links above don't go to the right place (404)
Also the release notes need a   for the <h3> on CVE-2018-8269: Denial of Service Vulnerability in OData for mark down to render properly
👍

Thanks, @williamdenton and sorry for that. Links fixed.

I couldn't see a discussion issue for https://github.com/dotnet/announcements/issues/121, so I'll ask here instead. For clarity, is 3.0 preview 9 unaffected by CVE-2019-1301?

These release notes state that System.Net.Sockets is vulnerable at 4.3.0 and secure at 4.3.1 however there is no version 4.3.1 published on Nuget.org.  What am I missing here?

How can I see the complete list of latest (secure) individual package versions across corefx and aspnetcore for a given .net core release number?  In other words, my projects are on net core 2.1.13, net standard 2.0 and net framework 461 but my nuget package references in csproj files are all over the place.  I don't want my team's blindly going to 2.2 or 3.0 versions of packages so they cannot simply grab the latest version.  And the package manager is not enforcing any target limitations on the versions it will accept.  Thank you for any help.
Bryan

@leecow @vivmishra Any help here?

@scalablecory  & @davidsh for System.Net.Sockets version question.
@bmorenc, we have the list of packages that were updated for a release (see the bottom of the release notes) - this may help you to at least know what changed. 2.1.13 Release Notes
Just to be clear -- from our perspective the latest set of 2.1 and 3.0 packages will always be the most secure and recommended. In only a very odd case (say where we have uncovered a bad regression) would we not recommend picking the latest.
Cc @terrajobst for guidance on complete list of packages.

@vivmishra thank you for responding and pulling in others for the security patch question on System.Net.Sockets.
Understanding which versions are associated with say 2.1.13 is quite easy for the packages that follow the framework versioning (Microsoft.Extensions.DependencyInjection 2.1.1).
The particularly challenging ones are those that don't such as the System.Net.Sockets example. I've been doing some more exploring on my end and it seems like even the latest 3.1.x based Nuget packages contain the older DLLs compatible with previous targets ensuring that no matter which target including multiple targets, I'll have the latest dll for each.  However, if I pull say System.componentmodel.annotations 4.7.0 but I am only targeting netstandard 2.0 and not netstandard2.1, how do I really know I am actually using an older version and which version that is?
I very much appreciate your help.  My questions here are related to our security and license compliance process which relies on a high level of version knowledge and trace-ability with our dependencies.  Since we consume a net core macro version for our runtime and SDK, but each project itself references the nuget packages directly by version, it is a challenge to know exactly what version of any given package is going to be in use.

Closing in favor of #4119


Posts are Being Processed Multiple Times

For whatever reason, posts are being commented on more than once. According to feedback: "It happens 2, 3, sometimes even 4 times."  I have only seen it three times max, but if people are saying four, I would be believe them.
Turning off the bot until this is fixed. I don't know what is wrong but I am setting up a check at the end (near the -1 cut) to go through all the recent comments for /u/DealWithItbot and remove dups. This will work for now. The -1 cut is also not working although I think that's related to #33.

The last 9 posts are three sets of three duplicates. This is bad.

These duplicates do not show up in karma.txt

I need to test this more verbosely, but, the bot has been running for a good hour without any duplicate problems.


Any plans to updated the SSL certificate?

Currently, the website is showing a secure error that might be related to the SSL certificated

This is on purpose I believe. If you explicitly add https:// in front of the URL it should give you a valid certificate. You seem to be looking at the plain HTTP version, which many browsers call "Not secure" now.
Both HTTP and HTTPS are accessible, because this way you can test with both protocols.


(SERVER-2550) Add testutils for generating cert and CRL chains

see individual commit messages for details

okay this is updated, I'll squash whenever y'all are ready.

when running write-cert-chain-and-crls I get:
WARNING: Illegal reflective access by clojure.lang.InjectedInvoker/0x00000008401e8040 (file:/home/justin/.m2/repository/org/clojure/clojure/1.10.1/clojure-1.10.1.jar) to method sun.security.x509.X509CertImpl.getSubjectX500Principal()

Which usually means that clojure needs a type hint at one of the invocation of .getSubjectX500Principal to understand which method signature to call.

huh, that does not happen when I run it..
user=> (require '[puppetlabs.ssl-utils.testutils :as t])
nil
user=> (t/write-cert-chain-and-crls 3 "crl_validation")
nil

okay @justinstoller fixed


Fix issue 9656 optimized

Does not break public API and should pass all tests!

Optimized version of https://github.com/yiisoft/yii2/pull/9658

Mehh, typo, unfortunately I can't change a commit message safely can I?
Anyway is it mergeable wtih the wrong commit message? ;-)

It's not an issue :)

We need a CHANGELOG line

This should be ready (assuming tests complete successfully).

@samdark It's good as for me (except minor changelog notice).

Merged with minor adjustments: https://github.com/yiisoft/yii2/commit/4a464afe9b4103d96c61f77ea6e093e45ee837c9

Thank you!


Add rest of the network handling

See https://github.com/mhsjlw/pocket-minecraft-protocol/issues/2 and https://github.com/PocketMine/RakLib/tree/master/server (or https://github.com/DragonetMC/DragonProxy/tree/269f01370624ae105e1af29fa52d13c2d8425c86/src/main/java/org/dragonet/raknet/server if java > php)

[x] add special packets
[ ] add server functionality

Basically we need to have all the functionnality of that server/ dir.

First we need all these packets in protocol.json https://github.com/PocketMine/RakLib/blob/master/server/ServerHandler.php#L82

First one is "packet_encapsulated":["container",[{"name":"identifier","type":["pstring",{"countType":"i8"}]},{"name":"flags","type":"i8"},{"name":"buffer","type":"restBuffer"}]]

https://github.com/PocketMine/RakLib/blob/a5cca45a922ca193a616069f885866efc7158c67/RakLib.php#L61 is a good ref for "special packets"

Files to reimplement:

[x] ServerHandler.php by special_packet_serializer.js
[x] RakLib.php and protocol/ dir by protocol;json
[x] Binary.php by datatypes.js
[ ] UDPServerSocket.php
[ ] SessionManager.php
[ ] Session.php
[ ] ServerInstance.php
[ ] RakLibServer.php

19:06  encapsulated packet have a "seq number"
19:06  that seq number go into ack packets
19:07  that way the opposite side knows whether we got the encapsulated packet or not
19:07  that means we need to know which seq numbers we sent
19:07  and which packets are confirmed
19:07  hence queues

mostly done, remaining : https://github.com/mhsjlw/node-raknet/issues/9

also #14

It seems like unconnected_ping handling is the responsability of raknet and not mcpe, see https://github.com/PocketMine/RakLib/blob/a5cca45a922ca193a616069f885866efc7158c67/server/SessionManager.php#L176


Redesign menu interface to include the full email address of accounts to prevent confusion

Checklist

[X] I have used the search function to see if someone else has already submitted the same feature request.
[X] I will describe the problem with as much detail as possible.
[X] This issue only contains a request for one single feature, not multiple (related) features.

App version
8.0
Problem you are trying to solve
The new menu design makes it difficult to see which account I am selecting, resulting in me needing to cycle through multiple accounts to find the one that I want to select.
This is caused by the fact that accounts are now selected via a "thumbnail" style circle with some initials in them. I'm not sure how the initials are generated, perhaps from the domain name of the account? If a user has multiple accounts under the same or similar domain names then the initials will be duplicated across accounts adding to the confusion. The account colour coding can help but puts additional mental burden on the user to remember these, when the interface should be assisting the user in the task of selecting the account to view.

I feel that this is a slight usability regression compared to previous versions, which clearly displayed the full address of the account to select via a picker.

Suggested solution
Redesign the menu so that each account's full email address is displayed during the selection process. This will prevent users from having to cycle through accounts to get the one they want.
Screenshots / Drawings / Technical details
I suggest removing the vertical sidebar of "thumbnails" and replacing the top view of the menu bar (containing the current account address and name) with a picker. This will accomplish two tasks with a single UI element: (1) the picker displays accounts unambiguously to support the task of selecting an account (2) the currently selected item provides a visual indication of which account is selected and gives the user an idea of the internal state of the system.

Would https://github.com/thunderbird/thunderbird-android/issues/8381#issuecomment-2428881786 alleviate your concerns? A major redesign in the account selection would require substantially more work, but could certainly be useful when more space is available, such as on tablets.

Thanks for such a swift reply! That means a lot for such a busy project.
The comment you've linked to does sound like a move in the right direction but my core issue is that I think the "thumbnail" style account picker is maladapted for purpose because it shifts additional burden onto the user rather than using the space effectively to communicate with the user.
Systems should ideally support recognition rather than making the user recall certain elements, even initials/colours.
I understand that there would be substantial work to redesign the look of the menu in the accounts section, and therefore this is not likely to be resolved quickly due to that.

but could certainly be useful when more space is available, such as on tablets.

Agreed, but I think an eventual redesign would be useful on smaller devices such as smartphones. I understand that the view components are likely quite complex and thus represent quite a lot of effort; but chopping away the vertical sidebar to free up some room gives back some horizontal space with which to display a full email address in a picker component.
As noted I understand this is a lot of work, so I really empathise if this needs to be closed as "won't fix" based on the work you've already done in resolving the other issue. However it would be appreciated if this could be communicated to whomever is responsible for the design/UI/UX, to see what their analysis is, for when the interface is updated in the future (e.g. when a new Android design language is released etc.)

I agree that there are scenarios where displaying the full account name would work better, though we also recognize that some users may prefer a more succinct identifier as it is currently. The planned addition of customizable images should offer flexibility while keeping the interface streamlined. We’ll revisit the account picker’s design in 2025, keeping these considerations in mind.


Insomnia lagging, high CPU usage and very slow in general

Expected Behavior
Expected behavior is that i should be able to open folders under a second, send requests, export my workspace and open my workspace.
Actual Behavior
Folders take about 10 seconds to open, requests send after waiting for around 15s, exporting a workspace results in an error, opening a workspace does absolutely nothing at all.
Reproduction Steps

Open random workspace
Open folder
Send request
Try to export workspace

Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
This error arises after i try to export the workspace:
TypeError: Cannot destructure property 'requestTree' of 'Nr(...)' as it is undefined.
at sH (file:///C:/Users/Jozef/AppData/Local/insomnia/app-2023.5.2/resources/app.asar/index-205ae130.js:193:6824)
at XA (file:///C:/Users/Jozef/AppData/Local/insomnia/app-2023.5.2/resources/app.asar/index-205ae130.js:116:19553)
at H7 (file:///C:/Users/Jozef/AppData/Local/insomnia/app-2023.5.2/resources/app.asar/index-205ae130.js:118:44101)
at U7 (file:///C:/Users/Jozef/AppData/Local/insomnia/app-2023.5.2/resources/app.asar/index-205ae130.js:118:39819)
at Vge (file:///C:/Users/Jozef/AppData/Local/insomnia/app-2023.5.2/resources/app.asar/index-205ae130.js:118:39745)
at ax (file:///C:/Users/Jozef/AppData/Local/insomnia/app-2023.5.2/resources/app.asar/index-205ae130.js:118:39596)
at QC (file:///C:/Users/Jozef/AppData/Local/insomnia/app-2023.5.2/resources/app.asar/index-205ae130.js:118:35959)
at rL (file:///C:/Users/Jozef/AppData/Local/insomnia/app-2023.5.2/resources/app.asar/index-205ae130.js:118:36764)
at qu (file:///C:/Users/Jozef/AppData/Local/insomnia/app-2023.5.2/resources/app.asar/index-205ae130.js:116:3288)
at file:///C:/Users/Jozef/AppData/Local/insomnia/app-2023.5.2/resources/app.asar/index-205ae130.js:118:34286
at node:internal/process/task_queues:140:7
at AsyncResource.runInAsyncScope (node:async_hooks:204:9)
at AsyncResource.runMicrotask (node:internal/process/task_queues:137:8)
Insomnia Version
2023.5.2
What operating system are you using?
Windows
Operating System Version
Windows 10 Enterprise 22H2 19045.3324
Installation method
downloaded from official insomnia website
Last Known Working Insomnia version
2023.4.0
Current workaround
Disable automatic update in insomnia
Roll back to version 2023.4.0:
https://github.com/Kong/insomnia/releases/tag/core%402023.4.0

Seeing similar behavior since updating to 2023.5.2.
https://github.com/Kong/insomnia/assets/31408341/e997b3ee-491a-484e-afec-8a7aa88bc78b

@Jozefwl @Juuldamen you should see better performance in latest patch release 2023.5.4

@Jozefwl @Juuldamen you should see better performance in latest patch release 2023.5.4

Hello, I've updated to 5.4 and even though it says 'fixed issue with export being blank' etc. issues with my Insomnia client still persist, I've confirmed this with a colleague and he has the same issue: Workspace document won't open and export is still blank.

@Jozefwl for the export double check you are on the latest version. Check in About Insomnia menu option which version you have

You should see requests on the export menu:


⚠ [warning] Releases in progress (Robot Testing Framework, YARP and iCub)

We are about to release Robot Testing Framework, YARP and iCub, please do not update your robot until new instructions.

Is it safe now? Given https://github.com/robotology/QA/issues/352 @traversaro

Still don't, we released master (3.1.2) we have still to release devel (3.2.0).

Confirming that it is now safe to upgrade.
Sorry for the delay


Feature: flatFractions display flag

This addresses issue #626
The flatFractions display flag is used to display fractions horizontally, it renders them as Latex: \left. numerator \middle/ denominator \right..
Sometimes, the numerator or denominator contain other operations, and it's nice to display brackets around them to make it clear that for example in (a + b)/4 the complete numerator is a + b. To identify these cases, I looked for a whitespace in the numerator and denominator. This solution might not be very elegant and although I've tested it with several situations, I might be forgetting some edge cases. Tell me what you think of it.
Also, my editor linted and removed trailing whitespaces in the modified files. I've put this in two separate commits, so that it is easy to revert if you'd rather not include this in the PR. I also advise you to read the PR commit by commit as this linting thing is hiding the main changes I've made.

Thanks for this. I'm not keen on using the presence of spaces to decide whether to bracket - I'd rather keep a precise handle on when we need brackets. I said in the issue that we could reuse the logic from treeToJME, but now I look at it, there's already a function texifyOpArg which adds brackets based on operator precedence, which we could use directly. So you'd return something like
'\\left. ' + texifyOpArg(thing,texArgs,0) + ' \\middle/ ' + texifyOpArg(thing,texArgs,1) + ' \\right.'

I'm happy with removing the trailing whitespace - thanks for making it a separate commit!

There are a few other places where fractions can be rendered - do a search/replace for \frac in jme-display.js. In particular, texRationalNumber needs to obey the new flag, and diff and partialdiff might as well do the same.

OK I'll add these changes in the coming days! Should be pretty easy if the function is already there :)

So I used texifyOpArg to apply brackets to numerator and denominator.
I added the flag in diff and partialdiff, always adding brackets to texArgs[0] in the case where args[0] is not of type 'name'. Otherwise for something like diff(2x, y, 1), we would have \left. \mathrm{d}2x \middle/ \mathrm{d}y \right., and texifyOpArg wouldn't change this.
I added the flag to texRationalNumber, but I'm not satisfied with result for mixedFractions that use a mixedInteger, it gives something like 1(\left. 1 \middle./ 2 \right.) (for 1.5). Thought you might have ideas on how to prettify this? My personal opinion is that it will be hard to make flat fractions natural for the mixed fractions notation, I'd be in favor of not displaying flat fractions at all in that case.

Thanks for this! I've merged it in, after making a little change: I think the best thing for the combination of mixedFractions and flatFractions is to put a space between the integer and fraction parts, rather than putting the fraction in brackets.
I've added your username to the CONTRIBUTORS.md file. Please put your real name in if you want.

Cool! Thanks for reviewing and helping on this :)


Fix example in guvectorize docstring.

In the guvectorize docstring the  example add_2d_array function does not compile as it is written because the function signature is missing it's third argument, the output.  This PR adds the argument in numba/npyufunc/decorators.py and in the example notebook, tutorials/Numpy and numba.ipynb, where the incorrect docstring also appears.

Thanks for the patch, well caught. Fix looks good.


Version 1.1 shows "1.0" on the about screen

This is obviously not a serious issue, but: the about screen for version 1.1 wasn't updated to reflect the new version. It still says "1.0"

Fixed.


Bad file names lead to mysterious lake error messages

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Check that your issue is not already filed.
Reduce the issue to a minimal, self-contained, reproducible test case. Avoid dependencies to mathlib4 or std4.

Description
Having a forbidden character in a file name leads to a misleading error message.
Context
This was discussed at least here and there. It affected Terence Tao there.
Steps to Reproduce
Create a project foo containing Foo/main-lemmas.lean and Foo/main.leanand put import Foo.main-lemmas in Foo/main.lean.
Expected behavior: [Clear and concise description of what you expect to happen]
The ideal behavior would be: everything works. An acceptable situation would be an error message saying Foo.main-lemmas is not a valid name for a Lean file and the file should be renamed or used with French quotes.
Actual behavior: [Clear and concise description of what actually happens]
The error message is
error: build cycle detected:
  +Foo.main:lean.precompileImports
  +Foo.main:lean.precompileImports

(both when invoked from the command line and from VSCode).
Versions
Lean (version 4.3.0-rc1, commit baa4b68a7192, Release)
Linux
Additional Information
This is first a parsing error. The import parsing stops before the dash and sees a circular import of Foo.main in Foo.main.
Impact
Add :+1: to issues you consider important. If others are impacted by this issue, please ask them to add :+1: to it.

Proposal: restrict header grammar so as to not allow additional tokens on the same line as the last import


git town-repo: support GitLab

@JaKXz would you be willing to try to implement this? I don't use Gitlab and I would rather had someone who does implement it.
I believe implementing this would only require adding a driver and some feature tests. If we are going to add support for GitLab we should also implement git-town-new-pull-request.
If you're willing to implement this please read through our contributing docs on how to get setup. I'm also here to help if you need any assistance

Fixed in #790


feat: add gotify/cli

https://github.com/aquaproj/aqua-registry/pull/6186 gotify/cli: A command line interface for pushing messages to gotify/server
$ aqua g -i gotify/cli

Thanks!

Released. https://github.com/aquaproj/aqua-registry/releases/tag/v3.52.0


Update documentation on guidance for HNSC when using SPSite

Details of the scenario you try and problem that is occurring:
I am creating 4 Host Named Site Collections under a main Web Application using the SPSite keyword.
While the sites are created and reported in SP Central Administration, and a HOSTS file edit has been made to route the DNS names used to the Single Server Farm server, I was getting a HTTP 404 error when trying to access the URL.
My fix: The fix came when I examined IIS Bindings for the Web Application, and added the Bindings for the URL of the Site Collections manually. Then the site collections were displayed. However, this step is not presently documented in the SPSite GitHub documentation.
Ideal fix: I think this step to create the additional bindings in IIS should be part of the SharePointDSC or added in notes with reference to how to use another DSC Module (possibly xWebAdministration) to ensure the change happens.
Attaching the DSC script used to configure the Single Server Farm and create the web application and host named site collections
sp2016singleserver.zip
The DSC configuration that is using the resource:
Single Server Farm created using SharePointDSC with site collections created using the SPSite keyword
Version of the Operating System and PowerShell the DSC Target Node is running:
Target system is running Window Server 2012 R2 running Powershell v5.0.10586.20151218
Version of the DSC module you're using:
SharePointDSC v<IP_ADDRESS>

Hi @nissan, this is correct. This is exactly the same behavior when a site collection from Central Administration. You have to do this manually, via xWebAdministration (when using DSC) or by using a wildcard domain (for example add *.domain.com to your IIS websites and then create new site collections in site1.domain.com and site2.domain.com).
I think it is a good addition to include this in the documentation to prevent others from spending time on troubleshooting.

Actually, when I updated my SharePointDSC to 1.6.0 and removed the "HostHeader" field in creating my web application, the new site collections worked correctly (once the DNS entries were made in my AD to point the hostnames to the SP WFE, something that can also be achieved with a HOSTS file entry on the machine browsing the site if you are only testing). If the HostHeader field is present, it binds the IP to the web application name, and prevents IIS from resolving any of the site collection URLs. However this is actually what should be the documentation enhancement for both SPSite and SPWebApplication (to remove the HostHeader parameter if planning to use Host Named Site Collections). There are documentation references for this host header parameter concern on MSDN and TechNet

Exactly if you want to use Host Named Site Collections, you have to create the SPWebapplication without HostHeader parameter. But it's a good point, we have to add this tips in documentation.
In term of best practices, never change manually the IIS bindings because these bindings are saved in SharePoint_Config_Db, so If you add a new server in your farm, the IIS Bindings will be settings with the informations  saved in this database.

The points by @ykuijs and @luigilink are correct here. SPSite will never touch the IIS bindings as we have no way to do that on every server from that one resource. The advice should be to use the xWebsite resource in xWebAdministration to manage bindings on each WFE server in the farm. So lets go ahead and add that to the documentation (I feel the doco change needs to go on both SPWebApplication and SPSite to make it super obvious) and go from there

Will be included in my bugfix PR


modify hmget command

when the filed is not exist，old hmget will return “”，but should return nil
old blackwidow return "", when the filed is not exist
modify blackwidow add status，pika can check the status

#388  fixed


Add shell completion automatically to rstuf

Test the CLI: pip install -e .

Isn't required in that case to have it in the pyproject.toml?

Isn't required in that case to have it in the pyproject.toml?

Yes, I forgot to add it there, thanks!

Not sure why, it doesn't work on my terminal (ksh) 😞

I don't know why, but after I install rstuf cli with pip install -e . still the terminal doesn't suggest autocomplete options for rstuf.
I am using zsh.

 If it's not zsh it's probably because bash is set to the default shell.
In any case, I think there's an easy fix for this. I pushed the change

Can you run on your shell: echo $SHELL? If it's not zsh it's probably because bash is set to the default shell.

(.env) ➜  cli [17:47]:  echo $SHELL
/bin/zsh

In any case, I think there's an easy fix for this. I pushed the change, can you retry?

I tried it again, but sadly it doesn't work.
When you have time we can debug this together.


Build of the Model image fails from Flatpack Install

When building the model image for chatbot from Podman Desktop UI (executed from flatpak):
STEP 5/7: RUN pip install --no-cache-dir --upgrade -r /locallm/requirements.txt
Collecting llama-cpp-python (from -r /locallm/requirements.txt (line 1))
  Downloading llama_cpp_python-0.2.44.tar.gz (36.6 MB)
     ━━━━━━━                                  6.5/36.6 MB 610.5 kB/s eta 0:00:50
ERROR: Exception:
                 Traceback (most recent call last):
                                                     File "/opt/app-root/lib64/python3.9/site-packages/pip/_vendor/urllib3/response.py", line 438, in _error_catcher
                                                              yield
                                                                     File "/opt/app-root/lib64/python3.9/site-packages/pip/_vendor/urllib3/response.py", line 561, in read
                                                                    data = self._fp_read(amt) if not fp_closed else b""
               File "/opt/app-root/lib64/python3.9/site-packages/pip/_vendor/urllib3/response.py", line 527, in _fp_read
                  return self._fp.read(amt) if amt is not None else self._fp.read()
                                                                                File "/usr/lib64/python3.9/http/client.py", line 463, in read
                                            n = self.readinto(b)
                                                                  File "/usr/lib64/python3.9/http/client.py", line 507, in readinto
                             n = self.fp.readinto(b)
                                                      File "/usr/lib64/python3.9/socket.py", line 704, in readinto
            return self._sock.recv_into(b)
                                            File "/usr/lib64/python3.9/ssl.py", line 1275, in recv_into
                                                                                return self.read(nbytes, buffer)
                                   File "/usr/lib64/python3.9/ssl.py", line 1133, in read
                                                                                return self._sslobj.read(len, buffer)
                        socket.timeout: The read operation timed out
                                                                    During handling of the above exception, another exception occurred:
                             Traceback (most recent call last):
                                                                 File "/opt/app-root/lib64/python3.9/site-packages/pip/_internal/cli/base_command.py", line 180, in exc_logging_wrapper
                                                                                status = run_func(*args)
                                                                                File "/opt/app-root/lib64/python3.9/site-packages/pip/_internal/cli/req_command.py", line 245, in wrapper
                                                                                return func(self, options, args)
                                     File "/opt/app-root/lib64/python3.9/site-packages/pip/_internal/commands/install.py", line 377, in run
                                     requirement_set = resolver.resolve(
                                                                          File "/opt/app-root/lib64/python3.9/site-packages/pip/_internal/resolution/resolvelib/resolver.py", line 95, in resolve
                                                                                result = self._result = resolver.resolve(
                            File "/opt/app-root/lib64/python3.9/site-packages/pip/_vendor/resolvelib/resolvers.py", line 546, in resolve
                                  state = resolution.resolve(requirements, max_rounds=max_rounds)
                                                                                File "/opt/app-root/lib64/python3.9/site-packages/pip/_vendor/resolvelib/resolvers.py", line 397, in resolve
                                                                                self._add_to_criteria(self.state.criteria, r, parent=None)
                                                           File "/opt/app-root/lib64/python3.9/site-packages/pip/_vendor/resolvelib/resolvers.py", line 173, in _add_to_criteria
                                                                          if not criterion.candidates:
                                                                                File "/opt/app-root/lib64/python3.9/site-packages/pip/_vendor/resolvelib/structs.py", line 156, in __bool__
                                                                                return bool(self._sequence)
                                File "/opt/app-root/lib64/python3.9/site-packages/pip/_internal/resolution/resolvelib/found_candidates.py", line 155, in __bool__
                                                           return any(self)
                                                                             File "/opt/app-root/lib64/python3.9/site-packages/pip/_internal/resolution/resolvelib/found_candidates.py", line 143, in <genexpr>
                                                                                return (c for c in iterator if id(c) not in self._incompatible_ids)
                                                                    File "/opt/app-root/lib64/python3.9/site-packages/pip/_internal/resolution/resolvelib/found_candidates.py", line 47, in _iter_built
                                                                                candidate = func()
           File "/opt/app-root/lib64/python3.9/site-packages/pip/_internal/resolution/resolvelib/factory.py", line 182, in _make_candidate_from_link
                                              base: Optional[BaseCandidate] = self._make_base_candidate_from_link(
          File "/opt/app-root/lib64/python3.9/site-packages/pip/_internal/resolution/resolvelib/factory.py", line 228, in _make_base_candidate_from_link
                                                  self._link_candidate_cache[link] = LinkCandidate(
                                                                                File "/opt/app-root/lib64/python3.9/site-packages/pip/_internal/resolution/resolvelib/candidates.py", line 290, in __init__
                super().__init__(
                                   File "/opt/app-root/lib64/python3.9/site-packages/pip/_internal/resolution/resolvelib/candidates.py", line 156, in __init__
                                                        self.dist = self._prepare()
                                                                                File "/opt/app-root/lib64/python3.9/site-packages/pip/_internal/resolution/resolvelib/candidates.py", line 222, in _prepare
                                                                                dist = self._prepare_distribution()
                                     File "/opt/app-root/lib64/python3.9/site-packages/pip/_internal/resolution/resolvelib/candidates.py", line 301, in _prepare_distribution
                                                                       return preparer.prepare_linked_requirement(self._ireq, parallel_builds=True)
                                           File "/opt/app-root/lib64/python3.9/site-packages/pip/_internal/operations/prepare.py", line 525, in prepare_linked_requirement
                                                                    return self._prepare_linked_requirement(req, parallel_builds)
                         File "/opt/app-root/lib64/python3.9/site-packages/pip/_internal/operations/prepare.py", line 596, in _prepare_linked_requirement
                                                   local_file = unpack_url(
                                                                             File "/opt/app-root/lib64/python3.9/site-packages/pip/_internal/operations/prepare.py", line 168, in unpack_url
                                                                                file = get_http_url(
                                                                                File "/opt/app-root/lib64/python3.9/site-packages/pip/_internal/operations/prepare.py", line 109, in get_http_url
            from_path, content_type = download(link, temp_dir.path)
                                                                     File "/opt/app-root/lib64/python3.9/site-packages/pip/_internal/network/download.py", line 147, in __call__
                                                                          for chunk in chunks:
                                                                                File "/opt/app-root/lib64/python3.9/site-packages/pip/_internal/cli/progress_bars.py", line 53, in _rich_progress_bar
     for chunk in iterable:
                             File "/opt/app-root/lib64/python3.9/site-packages/pip/_internal/network/utils.py", line 63, in response_chunks
                                     for chunk in response.raw.stream(
                                                                        File "/opt/app-root/lib64/python3.9/site-packages/pip/_vendor/urllib3/response.py", line 622, in stream
                                                                         data = self.read(amt=amt, decode_content=decode_content)
                         File "/opt/app-root/lib64/python3.9/site-packages/pip/_vendor/urllib3/response.py", line 587, in read
                        raise IncompleteRead(self._fp_bytes_read, self.length_remaining)
                                                                                File "/usr/lib64/python3.9/contextlib.py", line 137, in __exit__
                                                    self.gen.throw(typ, value, traceback)
                                                                                File "/opt/app-root/lib64/python3.9/site-packages/pip/_vendor/urllib3/response.py", line 443, in _error_catcher
                                                                                raise ReadTimeoutError(self._pool, None, "Read timed out.")
                                                     pip._vendor.urllib3.exceptions.ReadTimeoutError: HTTPSConnectionPool(host='[files.pythonhosted.org](http://files.pythonhosted.org/)', port=443): Read timed out.
Error:building at STEP "RUN pip install --no-cache-dir --upgrade -r /locallm/requirements.txt": while running runtime: exit status 2

Looks like an HTTP error can you build an image that install packages

It is reproducible only on DV's machine for now

It is reproducible only on DV's machine for now

DV's network connection is known worldwide for not being reliable :)

But the build passes when it runs the build from the podman cli podman build ...

Okay I just tested with Podman Desktop 1.9.1 installed on Fedora 39 with Flatpak, used the latest tag
ghcr.io/containers/podman-desktop-extension-ai-lab:nightly
Cicking on Chatbot, Start AI App, the checkout of the repository actually worked this time
But indeed Building llamacpp-server turns red after a bit of time, error:
"Something went wrong while building the image: building at STEP "RUN pip install --no-cache-dir --upgrade -r /locallm/requirements.txt": while running runtime: exit status 2"

Closing as it's related to timeout


Outdated data

Not sure which version of tz_world you are using at the moment, but I just wanted to let you now that tz_world is not being maintained any more (and there were lots of changes since may 2016!).
As stated on http://efele.net/maps/tz/world/ there now is another alternative. This new one works for timezonefinder (but data is different as in tz_world!).

Thanks for this. We are still using the old data.


ghcWithPackages: set withLLVM if hostPlatform is not x86_64

Resolves #116235.
This change fixes tests.haskell.shellFor on aarch64 which convinces me that this is a good change to make.
For GHC 9.2 we'll may have to adjust this again though.

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@ofborg build tests.haskell.shellFor

@oborg test agda

@ofborg build nixosTests.agda

Resolves #111738 as well.

This looks good to me (although I don't have much experience with aarch64)

Is there anything that could break from this?
What about darwin do we need LLVM there?

Is there anything that could break from this?

I don't think so, this only puts LLVM programs in PATH.

What about darwin do we need LLVM there?

https://github.com/NixOS/nixpkgs/issues/116235#issuecomment-799034030

I must admit I haven't touched or used the stock ghc in nixpkgs for a long time, since we ship our own ones in haskell.nix to be independent of what ever nixpkgs does. For anything that's not x86, or explicitly set to not use LLVM we do enable useLLVM. (https://github.com/input-output-hk/haskell.nix/blob/bab2eaea62be97649e0d38ef9e3bd313b92cf21d/compiler/ghc/default.nix#L22-L52)
In general GHC up to 9.2 has Native Code Gens for: PowerPC, SPARC, X86 only. From 9.2 this will also include AArch64, but no AArch32.


Text is replaced, not inserted.

The existing text is replaced by the new text. No insertion takes place. Ending with \r or \r\n has no effect.

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

ID: 8e650714-c739-1c42-ef89-c7dcc8333483
Version Independent ID: c82c6e17-3322-63dd-3fbc-b05cacf0fdab
Content: RichEditTextDocument.SetText(TextSetOptions, String) Method (Microsoft.UI.Text) - Windows UWP applications
Content Source: winui-api-build/xml/Microsoft.UI.Text/RichEditTextDocument.xml
Product: uwp
Technology: winui
GitHub Login: @Karl-Bridge-Microsoft
Microsoft Alias: kbridge

Thanks @MichaelBakkerDP - sorry for the delayed response. The topic has been updated.


Revisit crash_loop_limit property

In https://github.com/redpanda-data/helm-charts/pull/718 we set the default value of crash_loop_limit to 5 to avoid accidentally accumulating many short log segments. While an excellent mitigation, having to wait an entire hour in the case of an accidental misconfiguration is a non starter.
We should either set crash_loop_limit to null and rely on Kubernetes backoffs or add in a bash shim that will wait to start the Redpanda process for a few seconds if the crash limiter file exists. The latter would provide enough of a window of opportunity to A) Log what is happening and B) exec in and delete the file.
cc @vuldin for thoughts as we had to recently suffer due to this feature :) and @JakeSCahill who is writing up documentation related to this.
JIRA Link: K8S-110

Talked with @mattschumpert about this one and I think we're going to go with a bandaid solution for now. We should add in a bash script at the start of redpanda that checks if the crash_loop_file exists. If so, log out a warning explaining what it means and the kubectl command that users can run to remove the file then sleep for ~30-60s to give plenty of time to remove said file.

I think we should be completely OK to parse this internal format since this is about a redpanda-specific feature and this is a redpanda-specific helm chart.  For the rest I suggest speaking with @mmaslankaprv  about what we can do here

Our best bet in that case is probably to put something into rpk? Otherwise we're going to be parsing binary in bash which I'd really rather not depend on. Even in the case of RPK, the definition lives in redpanda and seems specific to how C++ marshals structs. Keep me posted on Michał's opinion!

It feels Redpanda is going too far into managing runtime concerns that are usually up to a process supervisor to decide such as systemd, kubelet, etc.

@chrisseto  this needs to go back through @mmaslankaprv for a re-thinking of how this should work in K8s.
We've already proven we NEED crash loop detection on by default (that part is not negotiable).
For the rest, If we want to change the approach of that file being there, not being there, what it looks like, or , please work with @mmaslankaprv on what we should do.  maybe worth a live discussion if needed

FWIW, we had to disable this feature in Azure because it was getting on the way. So, I'm not sure it is truly needed, and if it is, it probably points to a design  or implementation smell in Redpanda.

Do we have an idea of how many restarts are too many? I know we touched on this @mattschumpert but I don't recall the number. If we could raise the default in the chart to something that's less likely to trigger (5 is VERY low) when taking into account Kubernetes backoffs, we can move this ticket a bit further into the backlog.
Just as a spitball, what if we went to 50?

@c4milo brand new clusters never need it.
you should also speak to @mmaslankaprv who has the experience to explain why it's needed

@chrisseto the number was driven by incidents and there is no sense in making it very high as you just cause more damage and get into unrecoverable situations.  I'll stop here and let you speak with the replication team to design jointly the property solution

@mattschumpert Just to note that the current implementation also causes significant issues in incidents, where it requires a good deal of extra work to undo the crash loop protection to bring a cluster back online after an issue is resolved. This is true in Cloud, but doubly so in Self-Hosted envs where the same tools may not be available.

@mattschumpert thanks for the additional insight on the default!
There might be an easier answer now that I think about it. The node/broker config is just a configmap being mounted to the Pod and one way to get RP out of a crash loop is to change the node/broker config. In theory, one could just increment the number in the configmap (or even make a simple white space change IIUC) to recover a crashing node.
If we test that and it works as expected/well, do we think a docs update would be a possible resolution to this issue?


Fix error spelling of kubectl

fixed #15914

/assign @mistyhacks

@sftim thanks，i have created a new pull request #16957 to target the master branch.And this will be closed.


Failed to read partition information

Run against an environment for the first time, Successful on some disks (both 'Success' and 'Ignored') however a large number of disks are reporting the following message:

Failed to read partition information for disk - ""The input name ""Type"" cannot be resolved to a property

PowerShell is being elevated as Administrator, AD account executing script has Full Control NTFS permissions, and all disks (Ignored, Success, and Failed to read) are all on same server/share and were created identically.
Happy to provide any output if requested, I do not see any existing submissions or notes in the ReadMe for the above message.
I have also tested and verified ShrinkFSL.exe is able to mount and compact a VHDX that errors in Invoke-FslShrinkDisk

Just realized Issue #12 is identical to this one


icons-updated

issue number #1212
Description:
Our website's social media icons are essential for connecting with our audience and sharing content. To enhance user engagement and elevate the overall aesthetics, I recommend improving the responsiveness and vibrancy of these icons.
Solution:
To achieve this, I've made the following enhancements:
Aligned the icons' colors with their respective logos for better visual recognition.
Implemented a smooth transition of 0.3 seconds with a hover effect, enhancing responsiveness and providing users with intuitive feedback when interacting with the icons.
These changes result in a more vibrant and user-friendly experience, encouraging increased engagement with our social media channels.

Can you mark it as Hacktoberfest accepted??
On Mon, Oct 16, 2023, 10:27 AKSHITA GUPTA @.***> wrote:

Merged #1223 https://github.com/akshitagupta15june/PetMe/pull/1223 into
main.
—
Reply to this email directly, view it on GitHub
https://github.com/akshitagupta15june/PetMe/pull/1223#event-10660138808,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3JRPHPPDDV32UIHWGL2WMTX7S5DZANCNFSM6AAAAAA6ABBYZQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Still Unclear How to Install MSBuild without Visual Studio

The article mentions that MSBuild can be installed through .Net SDK, which is false. After .Net SDK installation my hard drive does not have MSBuild. Download page for Visual Studio says Build Tools require a Visual Studio license. Since MSBuild is licensed as MIT it is unclear what the right way of installing the tool is.

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

ID: 911666fa-55ac-cbc1-ce79-66c058859ad9
Version Independent ID: 571eec5d-8fb9-2401-fc4c-8a7896723eb7
Content: MSBuild - MSBuild
Content Source: docs/msbuild/msbuild.md
Product: visual-studio-windows
Technology: msbuild
GitHub Login: @ghogen
Microsoft Alias: ghogen

@AlanMiltonETS -- Alan, thank you for your thorough feedback.  Please consider these resources:

Ask a question or find an answer on the Microsoft Q&A page for Visual Studio.
Check the Visual Studio MSDN Forum.
Stack Overflow
Reddit

@ghogen -- Gordon, please look into this issue.

@AlanMiltonETS @owenn
You have a point though that MSBuild.exe is literally not installed when you install the .NET SDK. Instead, MSBuild is installed as MSBuild.dll and the command to invoke it is dotnet build. Any command-line arguments that dotnet build does not process are passed on to MSBuild, so it can substitute for MSBuild.exe whenever required. However, it is only the .NET Core version of MSBuild.
I have some updates to this article in the works to clarify this.


Replace io/ioutil with os or io package

Signed-off-by: yanggang<EMAIL_ADDRESS>We should clean all io/util package according the url: https://go.dev/doc/go1.16#ioutil
/kind cleanup

Please rewrite your commit message with a leading capital and no full stop, and as an imperative:
Replace io/ioutil with os or io package

Please rewrite your commit message with a leading capital and no full stop, and as an imperative:
Replace io/ioutil with os or io package

@skitt
Thanks for review.
Update the pr again.


test environemnt

Hi,
i am busy coding tests for my app in which I need to mock flutter_uesr_agent. the problem is that after its initialization in blocTest function, when I want to have access one of the properties, it gives me MissingPluginException(No implementation found for method getProperties on channel flutter_user_agent) error.
Do I miss something here?

 How you're testing (is it running on a device or not) etc.
This doesn't sound like an issue with the library, and most likely something to do with how you're running your tests. I'd recommend posting to stackoverflow instead of posting here.
The GitHub issues are typically for opening bug reports or feature requests as I do maintain this project in my spare time as a hobby and this seems like a general development question.
I'm not as involved in Flutter as I previously was, and I haven't delved in testing in Flutter. My intuition is telling me that you're probably trying to run the tests on a device (not Android/iOS) which is why it's complaining about a missing plugin implementation.
If not, you can set up a debugger and see what the code is doing, and then if you find that there's an issue with the code, feel free to open a ticket/pull request addressing the problem.


Proposed Recipes for GPM_3IMERGDF_06

Dataset Name
GPM_3IMERGDF_06
Dataset URL
https://disc.gsfc.nasa.gov/datasets/GPM_3IMERGDF_06/summary?keywords=GPM_3IMERGDF_06
Description
GPM IMERG Final Precipitation L3 1 day 0.1 degree x 0.1 degree V06 (GPM_3IMERGDF)
The Integrated Multi-satellitE Retrievals for GPM (IMERG) is the unified U.S. algorithm that provides the multi-satellite precipitation product for the U.S. GPM team. This dataset is the GPM Level 3 IMERG Final Daily 10 x 10 km (GPM_3IMERGDF) derived from the half-hourly GPM_3IMERGHH. The derived result represents the final estimate of the Daily accumulated precipitation. The dataset is produced at the NASA Goddard Earth Sciences (GES) Data and Information Services Center (DISC) by simply summing the valid precipitation retrievals for the day in GPM_3IMERGHH and giving the result in (mm).
License
Unknown
Data Format
NetCDF
Data Format (other)
No response
Access protocol
HTTP(S)
Source File Organization
There is one file per day. Each file has one timestep. Files are organized in folders by year and month.
Example URLs
https://gpm1.gesdisc.eosdis.nasa.gov/data/GPM_L3/GPM_3IMERGDF.06/2000/09/3B-DAY.MS.MRG.3IMERG.20000901-S000000-E235959.V06.nc4

Authorization
NASA Earthdata Login
Transformation / Processing
No response
Target Format
Zarr
Comments
No response

Thanks for posting @briannapagan! I've been on vacation but will try to spend some time on this this coming week.


Small images proof of concept

Just a small proof of concept I wanted to show:
USING: accessors kernel kernel.private math.ranges memory
quotations sequences slots.private system words ;
IN: test27

: main-word ( -- ) 99 (exit) ;

: clear-specials ( start end -- )
    [a,b] [ f swap set-special-object ] each ;

: strip-word ( id-quot word -- )
    f >>props f >>name f >>vocabulary def<< ;

: small-image ( -- )
    ! Stripped and pre-compiled identity quot.
    [ ] dup jit-compile dup f swap 1 set-slot

    [ word? ] instances [ strip-word ] with each

    !  Startup quot, global and shutdown quot
    OBJ-STARTUP-QUOT OBJ-SHUTDOWN-QUOT clear-specials

    ! JIT removal, must be done almost last.
    JIT-PROLOG JIT-DECLARE-WORD clear-specials

    ! Entry points and signal handlers we can do without
    LAZY-JIT-COMPILE-WORD FFI-LEAF-SIGNAL-HANDLER-WORD clear-specials

    ! PIC things, megamorphic caches, undefined and stderr
    PIC-LOAD OBJ-STDERR clear-specials
    "whatever.image" save-image-and-exit ;

MAIN: small-image

If you run the vocab it produces a factor image called whatever.image which is 1320 bytes on 32 bit and 1888 bytes on 64 bit. You can run it with ./factor -i=whatever.image but it doesn't do anything useful except exit with code 99. I'm thinking that the normal deploy images can probably be shrunk a lot too.

I would love to shrink the deploy images, thats a current big pain point.
But part of that is we haven't given the deploy framework much love lately.

At some point we can dig into why deploy sizes are so big and then we don't have to keep bumping the test sizes all the time, hah!  I still like having the test sizes be the way they are because then we know when deploy gets bigger.


Use minimist to parse arguments

Changes as suggested in #6080

Validates whether the user provides multiple arguments taking --scripts-version option into consideration.

@mrmckeb What do you think?

Hi @jamesgeorge007, this looks interesting.
It is probably a bit much to add a dependency for a small change like this, especially when it doesn't break anything if a user passes 'unknown' options.
I think it's worth investigating if this can be handled with Commander, rather than by adding another dependency - https://www.npmjs.com/package/commander.

@mrmckeb What about these manual workarounds?

Hi @jamesgeorge007, I appreciate the effort, but I feel that there's little value to add additional validation, but there is risk to editing these flows - especially when we don't have thorough test coverage of all commands, expected orders, etc.
There's a chance that the validation will catch something it shouldn't, and I think that's a worse outcome than an ignored flag.


Feature/user service

User Service
Added

add friend service
remove friend service
update profile service
delete account service
sign up service
log in service
test suite to test user service
document user service using jsdoc

closes #118

This documentation is beautiful. There are still a lot of things to learn from you. Thanks for writing the remaining user services. Below, I will list my notes and any requested changes:

To be honest, I am also learning from you. You have used many patterns that I have not tried before or previously did not prefer. It is a good experience to get out of my comfort zone. Thanks and keep it up! 👍🏾


Create ksp-block-stig-rhel-v-230330.yaml

https://www.stigviewer.com/stig/red_hat_enterprise_linux_8/2021-12-03/finding/V-230330
Not exactly addresses what's specified in STIG though.

stig-V-230330 states that
RHEL 8 must not allow users to override SSH environment variables.
Fix
Configure RHEL 8 to allow the SSH daemon to not allow unattended or automatic logon to the system.

Add or edit the following line in the "/etc/ssh/sshd_config" file:

PermitUserEnvironment no

The SSH daemon must be restarted for the changes to take effect. To restart the SSH daemon, run the following command:

$ sudo systemctl restart sshd.service

Since right now this is not in the Scope of KubeArmor or Cilium. Hence closing the PR


 Let's get rid of it.
Should be something like this in the task:
options.addStringOption('Xdoclint:none')

@JakeWharton  It will work with https://github.com/Vorlonsoft/GradleMavenPush. Please read README.md before use it.
You can set DOCLINT_CHECK (default is "false") to "true" or "false"


Feature/senior data engineer learning objectives

I know there were quite a few recommended certifications, is it worth adding them below as you have for DE?


Make compatible with graphql-js v0.5.0

The third argument to the resolve functions is now a context, and what was formerly the third argument (the info containing rootValue and so on) is now the fourth argument.
See:
https://github.com/graphql/graphql-js/releases/tag/v0.5.0
Also squashes in additional test fixes (additional types declarations) from @taion:
https://github.com/graphql/graphql-relay-js/commit/73c5c3fefb9f7ec5ee963f54efab73ae1068fb20
These are required as noted in the v0.5.0 release notes.

Types can now be explicitly provided to new GraphQLSchema({ types: [...] }). Types which implement an interface but are otherwise not referenced as a field return type are no longer automatically added to the Schema!

Haven't run the tests yet as my local set-up is currently broken, but we'll need something like this.

I'm taking a look at the remaining test failures now. I have all but 1 fixed on https://github.com/taion/graphql-relay-js/commit/40b0ef52b027cba44792ae389e09e403871d536b; they came from the change to require types to be specified explicitly for GraphQL schemas.
Let me know if you want me to PR against your branch, or submit a different PR, or if you just want to cherry-pick from my branch once I'm done.

I've fixed the remaining test failures on https://github.com/graphql/graphql-relay-js/commit/73c5c3fefb9f7ec5ee963f54efab73ae1068fb20. Let me know how I should proceed, as above.

Thanks @taion. I'll squish in your changes and update this PR.

BTW, the second change there is from an update to the signature of graphql.

Coverage remained the same at 99.779% when pulling 4ed95bdb2fb2d3ec604e7957927567a7b43e8891 on wincent:graphql-5.0-compat into 84a126866a01f78513bc3bf32413aaaae83e7931 on graphql:master.

Thanks for your help, @taion. Do you have any ideas on how best to warn the community that 0.5.0 is likely to break their schemas? I threw around some ideas for a https://github.com/relayjs/relay-codemod with @wincent, but one will be extremely hard to write when you consider that most people's schemas will be split across multiple files.

I'm actually in a state of blind panic over the changes I'm going to need to make to my schemas, since I've been using rootValue extensively.
I don't know if this addresses your issues directly, but I'd consider cutting a backward compatible release of graphql-js with deprecation warnings, with the following properties:

Move graphql() to take an object rather than an argument list; warn and use the old syntax if > 1 argument is supplied, else use the new list
For the backward compat release, restrict context to be a POJSO; in __DEV__, make the 3rd arg be something like { ...deprecationWrap(info), ...context }, where deprecationWrap wraps all properties with accessors that emit deprecation warnings
Warn on missing types (I know nothing about the code involved here, so I don't know if this is possible)

Separate from codemods, as a user, this would let me just chase down all relevant deprecation warnings. I still have no idea how much people use codemods, and I don't know if e.g. any of the ones I've written for React Router have actually seen use.

Err, obviously that syntax above wouldn't work for the deprecation-wrapped info. The thought there is to preserve the availability of all properties defined on info, but hidden behind a deprecation wrapper in __DEV__, except in the case that the property gets shadowed by one on context.

(And accepting an object for graphql() means I won't need to inexplicably pass in a null placeholder for rootValue goes whenever I want to specify variables)


Idea: Unit cost system involving food

I recently had the idea to revise unit costs by adding a food cost to recruiting (some) soldiers.
Some examples of what that might look like:

Monks: cost 5 gold and 1 bread (instead of currently 10 gold)
Arabian archers: cost 65 gold and 2 bread (currently 75 gold)
Spearmen: cost 1 spear, 2 gold and 1 apple (currently 8 gold and 1 spear)
Horse archers: cost 60 gold and 4 apples (currently 80 gold)
Macemen: cost 1 mace, 5 gold and 3 food (any type) (currently 20 gold and 1 mace)
~(possibly 1 wheat for horse archers / knights?)~

Now, I know this would be quite some work to implement, might be very difficult to achieve (especially with displaying the food cost ingame), and would be a rather impactful change. So why even bring it up?

Workers (even unassigned peasants) have a food cost over time, soldiers have none at all. Requiring a bit of food to recruit kinda feels a bit better in that regard, imho. A food cost over time would probably not fit the game balancing at all, so this might just be as close as it gets.
Currently, it is way easier (and faster, also for the AI) to recruit large amounts of troops with mercenaries than with European troops, as the latter need weapons while the former only need gold which can be stockpiled in "infinite" amounts.
With a food requirement for all (or most) troop types, Mercenaries would also require goods that cannot simply be stored in unlimited quantities, and potentially food purchases, making it slightly harder to spam them endlessly (although there are still less food types and food storage is larger, compared to weapon types and weapon storage).
Currently, beer production is a rather cheap and easy way to fully replace food altogether. With the proposed changes of troops requiring food to be recruited, food would become a bit more important: Either, you need to buy the food (more expensive) or produce it (cheaper, but requires more preparation and is a bit less flexible). This would also make troop costs a bit more "flexible" / dynamic as an effect.
This opens up a new "decision space" for strategic planning: If I want to focus mainly on troop types X, Y and Z, possibly I should focus more on food production of the corresponding food type(s).
Do I want to set up some (generally less productive) dairy farms to be able to recruit some troop types cheaper (and more conveniently), or do I really want an all-out bread production for high profits, but then have to buy cheese for gold? May it be better to have a few apple orchards in addition to my wheat and hops farms, and switch apple consumption off, so I can recruit some troop types cheaper when I need them, or do I want to use those apples for an added +1 popularity? Or possibly alternate?
This also affects how expensive different unit types are for different AI characters (and human players with different economic setups).
E.g. (with example costs from above), a single spearman would effectively cost 6 gold for Rat (2 gold and 1 apple that Rat could have sold for 4 gold otherwise), but it would cost 10 gold for anyone without apple production (2 gold + 8 gold for buying 1 apple). A horse archer would effectively be just 76 gold for Caliph (60 gold + 4 apples otherwise sold for 16 gold), but cost a Wazir or Saladin 92 gold (60 gold + 32 gold for buying 4 apples).
This would be similar to the current price ranges for European troops - producing weapons is (in the medium to long run) considerably cheaper than buying them all the time. But it would also allow for some "meta balancing" to subtly buff troops or AI character types using inferior food production types.

Now, I expect this suggestion to be controversial at best, but I think it might be useful to give it a thought. Cheers! ;)

Pretty controversial idea. 50x50
Hmm... I think this will greatly complicates the gameplay.
I think this idea will be difficult to implement. It may even be impossible ...

Interesting idea, although I am worried that it might weaken some of the Arabian AIs too much if overall mercenary prices increase, such as Wazir who needs to create a lot of horse archers to defend his lands. What do you propose for Arabian swordsmen? @Heroesflorian

While i like the idea of using food as recruitment costs. Its probably a better idea to not specify the food type as you don't want to restrict the use of units too much by the economy set up of the ai. On top of all that as you said yourself its probably faaaar faaar of from being possible to be realized which ontop of that probably alienate a lot of players as it touches a pretty core system of the game. granted it would be optional but it sounds like a lot of effort to make something potential nieche.

I am worried that it might weaken some of the Arabian AIs too much if overall mercenary prices increase

The example values provided above were just that - example values. Having said that, with the "right" food type being produced by an AI, troop costs would be slightly lower than before. I'm sure it would be possible to find a balance that works there.
Plus, with the patch, nothing prevents e.g. Wazir from building a few apple orchards in addition to his wheat farms.

its probably a better idea to not specify the food type

Well, maybe. I did that in the macemen example above. But I find the option for requiring certain food types interesting, so I wanted to have that mentioned as an idea as well.

What do you propose for Arabian swordsmen?

I haven't thought about it yet.

it sounds like a lot of effort to make something potential nieche.

Probably, yeah. I didn't say "do it now!" though, I just wanted to see a bit of discussion and give "food for thought".


Queue list buggy

It shows the current song on the first and current playing position!

What is the issue ?


Improvements and fixes

Cleaned up and refactored the application (PSR-2, better file/class/folder structure, support for .env configuration, etc.). Added some useful new features like possibility to define filename and sub-directory for file being uploaded, support for multiple file types, etc.. All of the changes are backwards compatible and there is no need to change anything when updating from older version (beside running 'composer install').

Wow this is some serious work you did there. Thank you!
Will test it out and get back to you

Okay I thought long about this but I have chosen not to merge your pull request.
The thing is that I hate external dependencies and since your fork needs composer with some huge libraries I decided against it.
I do realize that pictshare needs a massive code rewrite since this thing has become a frankensteins monster over the last years and I will rewrite it cleaner when I have more time.
Still thank you so much for your work


Add an option that disabled navigation keys

Optionally disables arrows, page up/down, home and end keys
While I think it is nice to have the keyboard interaction available, it is also good to be able to turn it off.
The motivation for this is that we found that users in equal part either wanted to edit the date as text with keyboard functions (e.g. arrow keys) or wanted to click a date. So in our case, disabling the navigation only allows the former without affecting the latter. This PR makes that possible.

Current coverage is 91.01% (diff: 92.00%)

Merging #702 into master will increase coverage by 0.01%

@@             master       #702   diff @@
==========================================
  Files            13         13
  Lines           589        590     +1
  Methods          17         17
  Messages          0          0
  Branches        106        107     +1
==========================================
+ Hits            536        537     +1
  Misses           13         13
  Partials         40         40

Powered by Codecov. Last update 88c5a28...84eff0e

Thanks for reviewing. This should address all the comments. I decided that it made sense to drop the checks for the effect of the keyboard navigation (e.g.):
data.copyM.add(1, 'years')
expect(result.format(data.testFormat)).to.equal(data.copyM.format(data.testFormat))

...as the fact that it is not calling the callback handler at all makes this moot.

Thanks!


"No internet connection" in Android's WIFI OS with mitmproxy, however interception works with other tools.

Steps to reproduce the problem:
mitmproxy
Install the mitmproxy certificate through http://mitm.it/
Start mitmproxy with mitmproxy -p 8080
Set proxy settings on Android Marshmallow or Nougat
Open the application
Get prompted with the error
Charles
Install the Charles proxy certificate through http://chls.pro/ssl
Start Charles
Set proxy settings on Android Marshmallow or Nougat
Open the application
See traffic in Charles
Any other comments?
I cannot see any traffic from the application in mitmproxy what so ever. Nothing. Could this be because the application blocks the mitmproxy certificate somehow?
System information
Mitmproxy version: 2.0.2 (release version)
Python version: 3.6.1
Platform: Darwin-15.0.0-x86_64-i386-64bit
SSL version: OpenSSL 1.1.0e 16 Feb 2017
Mac version: 10.11.1 ('', '', '') x86_64
Also installed and tried on an Ubuntu server.
-- Work around, which isnt a good fix, is to re-create a certificate manually and then re-import it, which is an advanced operation for many users. Since this definitely has something to do with the way mitmproxy is handling certs, I believe this bug is valid.
Thanks.

Thanks. It looks like your mitmproxy is outdated. Could you please upgrade an try again?