Windows Server 2019 - Ubuntu - WslRegisterDistribution failed with error: 0x80070001

On Windows Server 2019 Standard (64bit OS) - Build 17763.615, when trying to install Ubuntu on WSL (by following https://docs.microsoft.com/en-us/windows/wsl/install-on-server), I have this error:
Installing, this may take a few minutes...
WslRegisterDistribution failed with error: 0x80070001
Error: 0x80070001 Incorrect function.

Your Windows build number:  (Type ver at a Windows Command Prompt)

systeminfo | Select-String "^OS Name","^OS Version"

OS Name:                   Microsoft Windows Server 2019 Standard
OS Version:                10.0.17763 N/A Build 17763`

What you're doing and what's happening:  (Copy&paste the full set of specific command-line steps necessary to reproduce the behavior, and their output. Include screen shots if that helps demonstrate the problem.)

PS C:\Windows\system32> Enable-WindowsOptionalFeature -Online -FeatureName Microsoft-Windows-Subsystem-Linux

Path          :
Online        : True
RestartNeeded : False

Invoke-WebRequest -Uri https://aka.ms/wsl-ubuntu-1604 -OutFile Ubuntu.appx -UseBasicParsing
Rename-Item Ubuntu.appx ./Ubuntu.zip
Expand-Archive Ubuntu.zip D:\Ubuntu
Remove-Item .\Ubuntu.zip
cd D:\Ubuntu

$userenv = [System.Environment]::GetEnvironmentVariable("Path", "User")
[System.Environment]::SetEnvironmentVariable("PATH", $userenv + ";D:\Ubuntu", "User")

.\ubuntu1604.exe
Installing, this may take a few minutes...
WslRegisterDistribution failed with error: 0x80070001
Error: 0x80070001 Incorrect function.`

Logs:
logman.zip

I have fixed my issue.
It seems that I can't use WSL on an NVMe disk.
On a classical disk, on the same server, it works.

I have fixed my issue.
It seems that I can't use WSL on an NVMe disk.
On a classical disk, on the same server, it works.

I have the same issue on Windows Server 2019. However, my OS runs on a SATA SSD rather than a nvme drive.

I get this error when installing WSL on a server which is a hosted VM. So the disk is neither an NVMe nor a Sata SSD. It's a file on a hypervisor somewhere in a cloud or something.
And why should any of that make any difference at all? What kind of hocus pocus nonsense would the type of disk make which would prevent running a virtual OS inside another OS?
It's the 21st century. It's VM's all the way down.

Had same problem (HDD Windows installation – success, NVMe – failure), was solved like this:

Open msconfig (e.g. via Win + R)
Select "Services" tab
Mark "Hide all Microsoft services"
Click on the "Disable all" button
Apply changes and reboot the system
Install WSL in the standard way
Revert services changes and reboot again

Which services were disabled

Intel(R) Content Protection HDCP Service
Intel(R) Dynamic Tuning Technology User Interface Service
Intel(R) Graphics Command Center Service
Intel(R) Innovation Platform Framework Service
Intel(R) Innovation Platform Framework User Interface Service
Intel(R) Dynamic Application Loader Host Interface Service
Intel(R) Management Engine WMI Provider Registration
ImDisk RamDisk starter
ImDisk VirtualDisk Driver Helper
Mozilla Maintenance Service
Realtek Audio Universal Service


Made nltk/__init__.py equivalent to setup.py

Fixes #2509
Hello!
Pull request overview

Made nltk/__init__.py equivalent to setup.py in terms of project description, classifiers, etc.

Changes
I simply updated all dunder variables in nltk/__init__.py to be equivalent to their corresponding variables in setup.py. This includes changes to:

Long description.
Maintainer.
Trove classifiers.

Note
I simply copied the values from setup.py, but as a consequence two names are removed from the maintainer list here!

Tom Aarsen

Thanks @tomaarsen, good catch!

Would it be possible to reuse this string in setup.py directly? I dunno if that'll work, but seems like a good way to reduce duplication. Just to minimize the number of things we have to keep consistent...


ERROR: Unsupported URL: http://www.digitaltutors.com/tutorial/1491-Rigging-Quadrupeds-in-3ds-Max#play-39380

Any update on digital tutors ? trying right now to do it but it doesnt work

Is it supported now? or no fix for it?

I hope that some one can make it.
I have digitaltutors account and it can not be used in pluralsight.com in youtube-dl.
although in browser, it redirects to pluralsight from the digitaltutors user page.

@masneg
Could you provide example URLs? I'm not a dev, but I'm interested in able to have it working on youtube-dl.


Fix issues link on Accessibility page

Description
The Accessibility page links to https://github.com/pixelandtonic/spoke-and-chain/issues and should be updated to https://github.com/craftcms/spoke-and-chain/issues.

Fixed in bbf9cc533614e829853828361ecc274b5ae98e43.


Login screen going back to welcome screen unexpectedly

Environment Information
Browser: Google Chrome v58 for Mac
Samples: https://github.com/auth0-samples/auth0-ios-objc-sample
Xcode: 8.3.2
Description
When logging in with any account, the app goes back to the welcome screen and the user is not able to log out in a normal manner.

Which chapter?

It happens across all chapters. Also, no errors are showing up in the logs.
Success Login Log:
{   "date": "2017-08-16T18:56:54.738Z",   "type": "s",   "connection": "Username-Password-Authentication",   "connection_id": "con_dqfXgB4vPZurm7D4",   "client_id": "WLk5ZaXxbmC0sHnntBoZrKeDHjtg79bK",   "client_name": "swift-test",   "ip": "<IP_ADDRESS>",   "user_agent": "Mozilla/5.0 (iPhone; CPU iPhone OS 10_3_1 like Mac OS X) AppleWebKit/603.1.30 (KHTML, like Gecko) Version/10.0 Mobile/14E8301 Safari/602.1",   "details": {     "prompts": [       {         "name": "lock-password-authenticate",         "initiatedAt":<PHONE_NUMBER>046,         "completedAt":<PHONE_NUMBER>404,         "timers": {           "password": 126,           "rules": 324         },         "connection": "Username-Password-Authentication",         "elapsedTime": 358       }     ],     "initiatedAt":<PHONE_NUMBER>064,     "completedAt":<PHONE_NUMBER>737,     "elapsedTime": 15673,     "stats": {       "loginsCount": 36     }   },   "user_id": "auth0|5911d7344baa935a253da4a6",   "user_name"<EMAIL_ADDRESS>  "strategy": "auth0",   "strategy_type": "database",   "auth0_client": {     "name": "lock.js",     "version": "10.7.3",     "lib_version": "7.5.0"   },   "log_id": "49574419589858254662355359013975577209727105990834257954" }
Success Exchange Log:
{   "date": "2017-08-16T18:56:54.979Z",   "type": "seacft",   "description": "",   "connection_id": "",   "client_id": "WLk5ZaXxbmC0sHnntBoZrKeDHjtg79bK",   "client_name": "swift-test",   "ip": "<IP_ADDRESS>",   "user_agent": "00-Login/1 CFNetwork/811.4.18 Darwin/16.7.0",   "user_id": "",   "user_name": "",   "auth0_client": {     "name": "Auth0.swift",     "version": "1.7.0",     "swift-version": "3.0"   },   "log_id": "49574419589858254662355359013381994632296322928614572066" }
Video
App going back to welcome screen.zip

This is the correct behaviour, you notice that it tells you auth was successful when presents the access token value. In this sample chapter there is no profile page, it's a basic auth test.
As you said it effects all chapters, I checked chapter 03, upon auth you will be presented with a user profile page. There was no problem here during authentication and I was presented with my profile.


No Equation of Time correction in sun orbit code

Need to add the Equation of Time (EOT) correction to MAPL Sun Orbit code. This expresses the fact that apparent solar time, which governs the actual position of the sun, is not exactly the same as mean solar time (almost UTC).

Basic EOT machinery is now in place in MAPL.
Next will call that machinery from MAPL_SunGetInsolation (in solar and other places) with resource selectable EOT logical.

@dr0cloud I believe we can close this issue right? All your EOT work is now in 2.1?

This is fixed by PR #249 which is the new orbital system.


PromQL: Allow label append to groups of metrics

Proposal
From an IRC conversation with @ecraven, we had an idea to simplify some expressions by allowing labels to be applied to groups of metrics surrounded by ().
For example:
foo{env="prod"} / bar{label="prod"}

Becomes:
(foo / bar){env="prod"}

The effect for other labels applied to foo and bar would be to append the list of label filters to any interior ones.
For example:
(foo{blah="blah"} / bar{blah="blah"}){env="prod"}

Would translate to:
foo{blah="blah",env="prod"} / bar{blah="blah",env="prod"}

I'm not sure about this. Given how the rest of the language works, I'd expect this to evaluate all of foo{blah="blah"} / bar{blah="blah"} and then restrict it down. I'm also generally against features that are merely another way of writing something that we already support.

The reason I tried this was a form like
((foo - bar - baaz) / foo) where I'd have to repeat the labels four times :-/

@SuperQ we're doing our weekly bug scrub ATM; I suspect you're still interesting in this?

I still think it might be a nice way to simplify queries.

At some point I was thinking that we could do that automatically in the promql engine itself but apparently remore read prevents that. In which case the parser is the place to do it. But how do you see that?
Does (foo / bar){env="prod"}
become foo{env="prod"} / bar{label="prod"}
or does  (foo / bar){env="prod"} do
foo / bar then selects {env="prod"}, which means that we can use that on functions too?

We could recursively add the outer selector to every selector inside the respective subtree. That should be possible to do in the engine and would be similar to how we already treat subqueries.
The interesting part would be how to deal with functions like label_replace.

This is a form of templating, which can always be done by configuration management. We shouldn't ever be making PromQL more complex to substitute for a lack of templating on the part of the user or merely to reduce apparent duplication.
The semantics here are also unclear, as there's two different things that this syntax could mean as seen above. Things like label_replace only make it more complicated.

Does (foo / bar){env="prod"}
become foo{env="prod"} / bar{label="prod"}

This should be the one if we are going ahead with it. The other one would have a lot of unnecessary calculations.

Just had a better example of how this is very unclear semantically come to mind. What would sum(foo){a="b"} do?
I don't think we should be adding confusing new syntax and non-obvious semantics to cover things that are already possible.

What would sum(foo){a="b"} do?
sum(foo{a="b"})?

That's not what it looks like though, I'd expect it to return nothing as the output of sum(foo) won't have an a label. This would get even more fun when vector matching is involved.

yeah this does not look sane

Anything in curlies would only apply to the scope of the parentheses directly in front of them. If users try to select on non-existing labels, they would receive an empty return set no matter which foot-gun they would be using.

Yeah, this seems both confusing to read, and a concept that doesn't really fit to the rest of the language, where parent nodes do not affect the evaluation of child nodes, only the other way around.

where parent nodes do not affect the evaluation of child nodes, only the other way around.

Don't subqueries affect their child nodes?

Don't subqueries affect their child nodes?

They don't, it's an independent evaluation.

@brian-brazil are we going ahead with this? else we can close the issue since it seems less likely as per the discussions.

While there's a definite strong feeling here that this should not be implemented, there's not a consensus.

Hello from the bug scrub! Closing as this is a somewhat controversial feature request and also there was no discussion in the last four years. Please follow up with new proposal if you want to work on this issue.


Support for Infrastructure as Code (terraform, etc...)

So it dawned on me, that say you wanted to represent not just your application, but the external dependencies for it (so, the miraculous things that happen in AWS, GCP, etc...), I wasn't entirely sure how to represent this! I'm opening this issue as more so a collection of ideas, no concrete feeling that I'll come out of it with YES THIS IS HOW, but more so to see how people think about it?
This could be slightly duplicative of the intent of something like terraform, too! Terraform is meant to manage your IaC, and it creates files that describe what it plans to do, or state, (terraform plans, terraform state, etc...)
Mostly curious as to others thoughts around this. How would YOU describe them?

I would probably document them as services. Or is there some required information missing from that type?

I can think of e.g. the google_container_clusters resource from the google terraform package, which describes the Kubernetes version being used, which can then be linked to CVEs. Same for Cloud SQL (postgresql) etc.

@hazcod I don't use terraform myself, so I might be misunderstanding your example.
But, if it's an external service I would document it as a service. Ideally you would additionally add an external reference to the BOM of that service. Which is a supported external reference type.

Yes I can see many use cases for this as vulnerabilities etc could live in the underlying infra such as an Ansible play, Terraform, CloudFormations etc and also the specific underlying OS instances, packages, and configuration.

Checkov supports CycloneDX and obviously IaC, but I haven't seen how they output CycloneDX. Does anyone have an example of infrastructure represented in CycloneDX from checkov?

Let me join this discussion.
IaC only defines the infrastructure of the product.
Probably If it is created as a BOM specification, it is necessary to abstract the product.
A product is
・Service {Operating-system { Applications {Libraries}}}
・Devices (L2, L3 switch e.g. load balancer, machine)
・Network (IP address, domain, port)
If it is possible to define the SaaSBOM specification and how these Services or Applications are connected to each other, it should be possible to create a BOM from the IaC.

v1.5 will introduce a new external reference called codified-infrastructure which will allow a CDX BOM to reference terraform or other IaC spec.


Added spoolman metrics

Added metrics from Spoolman module, such as current spool ID and connected status&

I also use Spoolman with my Klipper install so I will give this a test and provide any feedback.

I have already do PR to Spoolman with simillar feature. For both it view as:


fixed npm install error that comes in docker file

Hi @Kanika637 , i have fixed the error of npm install.
It was coming because react corousel slider was installed in the project. It was the latest version 2.13.0 which is not compatible with react 17.
It was also not used anywhere in the project.
I have uninstalled it and deleted ffrom package.json.
please check if it is working!!

Hey @harshit0571, but react carousel slider is used for the carousel on the website if I am not wrong!

the bootstrap carousel is being used.
no one is using this package.

@harshit0571 , not able to solve the error


appleseed.studio's Save Project As... does not save search paths

If a project containing search paths is opened in studio and saved as with a new name,
search paths are not saved.

It seems like this was intentional:
https://github.com/appleseedhq/appleseed/commit/a745600d90e2e95d5e7f73a7995072b0ba19d2b3

Fixed in 9b61581ab9404627fdada8eb241715d1e8066961.


Unified file path checker

Currently almost every function has a different way to check if a file exists or not, and to upload it or not to the MAPDL instance. A first atemp was made with ansys.mapd.core.misc.load_file but this should be improved and extended.
Features

[ ] It should detect if in local or remote.
[ ] In case of local, it should pass the full path to the MAPDL instance. Not uploading the file.
[ ] Raise warning when file is present in several places.
[ ] When in remote, it should upload the file to the instance and notifies if already exists.

This was actually fixed in #783


Add Java checkstyle

In order to have a common appearance and general rules in the whole extension, we should add a light Java check style and import order.
Documentation:

https://checkstyle.org/
https://github.com/revelc/impsort-maven-plugin

Checkstyle proposal:
My proposal is something very light, that could be extended(or change) in the future if we see any inconvenience.
<module name="Checker">
    <property name="severity" value="error"/>
    <module name="TreeWalker">
        <!-- Checks for imports                              -->
        <!-- See https://checkstyle.org/config_imports.html -->
        <module name="AvoidStarImport"/>
        <module name="RedundantImport"/>
        <module name="UnusedImports"/>
        <module name="RequireThis">
            <property name="validateOnlyOverlapping" value="true"/>
        </module>
    </module>
</module>

Imports (https://checkstyle.org/checks/imports/index.html):

RedundantImport

Checks for redundant import statements. An import statement is considered redundant if:

It is a duplicate of another import. This is, when a class is imported more than once.
The class non-statically imported is from the java.lang package, e.g. importing java.lang.String.
The class non-statically imported is from the same package as the current package.

AvoidStarImport

Checks that there are no import statements that use the * notation.

UnusedImports

Checks for unused import statements. An import statement is considered unused if:

It is not referenced in the file. The algorithm does not support wild-card imports like import java.io.*;. Most IDE's provide very sophisticated checks for imports that handle wild-card imports.
The class imported is from the java.lang package. For example, importing java.lang.String.
The class imported is from the same package.
Optionally: it is referenced in Javadoc comments. This check is on by default, but it is considered bad practice to introduce a compile-time dependency for documentation purposes only. As an example, the import java.util.List would be considered referenced with the Javadoc comment {@link List}. The alternative to avoid introducing a compile-time dependency would be to write the Javadoc comment as {@link java.util.List}.

RequireThis (https://checkstyle.sourceforge.io/checks/coding/requirethis.html)
Import sort proposal:
impsort-maven-plugin  in order to automatically sort/validate the order of all the import
Proposal:

java
jakarta
org
io

formatter-maven-plugin in order to automatically (re)format/validate a maven project(including source code). Keeping a coding style format consistent and following certain specifications is an important requirement in preventing unnecessary conflicts and code diffs.
Note: all of this plugins should be triggered with a mvn clean install


Remove Goerli from Juno codebase

Context: the Goerli network is being deprecated, but Juno tests still rely on it. The tests that rely on Goerli should be updated to use the Sepolia network instead.
RPC based tests that require Goerli:

[ ] TestClassV0
[ ] TestTransaction
[ ] TestHeight
[ ] TestBlockByNumberAndHash
[ ] TestEvents
[ ] TestChainId
[ ] TestBlockTransactionCount
[ ] TestBlockWithTxHashes
[ ] TestEvents
[ ] TestRpcBlockAdaptation

Goerli is used in other non-rpc tests (sn2core, blockchian, cmd, etc). These should also be updated, but the RPC tests might be a good place for newcomers to start
Note: any newcomers that are interested in working on this issue, should 1) open a new issue stating which test they want to fix, 2) create a new working branch, 3) implement the fix, 4) create a new PR, and assign someone form the Juno team to review it.
Note: these tasks need done before April 11th (Goerli testnet support will be removed April 11, 2024.)

Hi, all the tests concerning the rpc package do not include any of the goerli networks anymore.
However the following tests are still using the goerli network as a reference:

[ ] TestTransaction

sn2core_test
feeder_test

[ ] TestEvents

blockchain_test

[ ] TestDeclareV0TransactionHash

transaction_test

[ ] TestBlockHash

block_test

[ ] TestNetwork

network_test

[ ] TestCoreContractAddress

network_test

[ ] TestExecute

vm_test

[ ] TestV1Call

vm_test

Can i take this one up?

Hi @rianhughes
William here from Costa Rica 🇨🇷
Wondering if you guys still need some help with this
I did a quick search on the project and seems like there are 100 results when you search for Goerli

I am willing to help either remove that code OR replacing with Sepolia if possible


Copyedits

Cross fingers - hope no build errors!

Not sure what I broke here, but it seems the table is fixed? Need some help here.

You triggered an error in another of our build checks, for the YAML frontmatter:
https://travis-ci.org/aliemteam/aliemcards/builds/231012371#L577

Codecov Report

Merging #183 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #183   +/-   ##
=======================================
  Coverage   60.16%   60.16%
=======================================
  Files          16       16
  Lines         241      241
  Branches       50       50
=======================================
  Hits          145      145
  Misses         96       96

Continue to review full report at Codecov.

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


[BUG] Cognitive Language Conversations rejecting input

Library name and version
Azure.AI.Language.Conversations 1.1.0-beta.1
Describe the bug
I'm trying to test the new Conversation Summarization feature documented here, but even calling it with the sample input is coming back with a 400 (Unsupported feature).  I've created a language resource in EastUS in my Azure tenant, and it's set to Standard pricing-is there something else that I might be missing here?  I've got the 1.1.0-beta1 nuget package installed.
Expected behavior
Summary should return
Actual behavior
Call returns a 400 (invalid operation)
Reproduction Steps
Repros with sample code here: https://docs.microsoft.com/en-us/azure/cognitive-services/language-service/summarization/quickstart?tabs=conversation-summarization&pivots=programming-language-csharp
Environment
.net 6, win 11 21h2, vs 2022 Enterprise 17.2.5

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Cognitive Services:0.08884846,Service Bus:0.068317525,Azure.Core:0.05241282'

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Not all regions support conversational summarization and PII. I'll get someone from the service team that can better help you.

Conversation summarization is gated you need to fill in onboarding form to get whitelisted

From your link, I found the link to the form: https://aka.ms/applyforconversationsummarization/

@chrisbardon I tested from the same region (East US) and see the form URL is contained in the error message as well.

PS D:\TestLanguageServiceSDK> dotnet run
Unhandled exception. Azure.RequestFailedException: Invalid request has been sent.
Status: 400 (Unsupported Feature)
ErrorCode: InvalidRequest
Content:
{"error":{"code":"InvalidRequest","message":"Invalid request has been sent.", "innererror":{"code" : "UnsupportedFeature", "message":"The conversation summarization is a gated public preview. To request access please fill out this form  https://aka.ms/applyforconversationsummarization."}}}
Headers:
apim-request-id: REDACTED
Strict-Transport-Security: REDACTED
X-Content-Type-Options: REDACTED
Date: Thu, 14 Jul 2022 00:39:53 GMT
Content-Length: 290
Content-Type: application/json
at Azure.Core.HttpPipelineExtensions.ProcessMessageAsync(HttpPipeline pipeline, HttpMessage message, RequestContext requestContext, CancellationToken cancellationToken)
at Azure.Core.ProtocolOperationHelpers.ProcessMessageAsync[T](HttpPipeline pipeline, HttpMessage message, ClientDiagnostics clientDiagnostics, String scopeName, OperationFinalStateVia finalStateVia, RequestContext requestContext, WaitUntil waitUntil, Func`2 resultSelector)
at Azure.AI.Language.Conversations.ConversationAnalysisClient.StartAnalyzeConversationAsync(WaitUntil waitUntil, RequestContent content, RequestContext context)
at Azure.AI.Language.Conversations.ConversationAnalysisClient.AnalyzeConversationAsync(WaitUntil waitUntil, RequestContent content, RequestContext context)
at Program.$(String[] args) in D:\TestLanguageServiceSDK\Program.cs:line 68
at Program.(String[] args)
PS D:\TestLanguageServiceSDK>

So it is- I have no idea how I missed that.  Filling out form now.


fix: Core shouldn't check user roles in case authorization by API key…

…… (#181)

/deploy-review


Keyboard doesn't work in Slim login manager

Issue description
Keyboard does not work in Slim using commit 82bcfef109ecf58ac1503e4cab15ae53dd524f4b or after. I determined that commit using git bisect. The one right before it works fine.
Steps to reproduce
Using the below configuration.nix and 82bcfef109, run nixos-rebuild build-vm and run the VM. When the VM gets to a log in screen, typing in the login box doesn't work. On my laptop I also can't switch to another VT using ctrl+alt+F2. I'm not sure how to do that in the VM.
configuration.nix
{
  services.xserver = {
    enable = true;
    displayManager.slim.enable = true;
  };
}

Technical details

System: (NixOS: nixos-version, Ubuntu/Fedora: lsb_release -a, ...)$ nixos-version
17.03.git.58dcda6 (Gorilla)

Nix version: (run nix-env --version)$ nix-env --version
nix-env (Nix) 1.11.6

Nixpkgs version:$ nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion
"17.03pre99759.f66d782"

Could this be an instance of #22290? That should be verifiable by trying with that particular commit reverted.

We could reproduce this with slim. It is in fact due to https://github.com/NixOS/nixpkgs/commit/82bcfef109ecf58ac1503e4cab15ae53dd524f4b.

Reverted for now in https://github.com/NixOS/nixpkgs/commit/24f3abdafbf0a4840d63baa91e60ba34fad00d4d.


部署报错

[+] Building 0.1s (2/2) FINISHED
=> [internal] load build definition from Dockerfile.base                                                                                0.0s
=> => transferring dockerfile: 696B                                                                                0.0s
=> [internal] load .dockerignore                                                                                0.0s
=> => transferring context: 2B                                                                                0.0s
failed to solve with frontend dockerfile.v0: failed to create LLB definition: dockerfile parse error line 11: FROM requires either one or three arguments
看了一下其它人的 Issues 说是 docker版本的问题，我尝试过从官网下载最新的版本，但是也无法正常部署，不知道哪个版本可以正常编译？
Cloud integration: v1.0.31
Version:           20.10.24
API version:       1.41
Go version:        go1.19.7
Git commit:        297e128
Built:             Tue Apr  4 18:21:21 2023
OS/Arch:           darwin/arm64
Context:           default
Experimental:      true

试一下这个 #11 (comment)

出现了这个错误
  => ERROR [5/5] RUN python /get-models.py &&     rm -rf /get-models.py     1.9s
------
 > [5/5] RUN python /get-models.py &&     rm -rf /get-models.py:
#0 1.545 Traceback (most recent call last):
#0 1.545   File "/usr/local/lib/python3.8/dist-packages/sklearn/__check_build/__init__.py", line 44, in <module>
#0 1.545     from ._check_build import check_build  # noqa
#0 1.545 ImportError: /usr/local/lib/python3.8/dist-packages/sklearn/__check_build/../../scikit_learn.libs/libgomp-d22c30c5.so.1.0.0: cannot allocate memory in static TLS block
#0 1.545
#0 1.545 During handling of the above exception, another exception occurred:
#0 1.545
#0 1.545 Traceback (most recent call last):
#0 1.545   File "/usr/local/lib/python3.8/dist-packages/transformers/utils/import_utils.py", line 1146, in _get_module
#0 1.545     return importlib.import_module("." + module_name, self.__name__)
#0 1.545   File "/usr/lib/python3.8/importlib/__init__.py", line 127, in import_module
#0 1.545     return _bootstrap._gcd_import(name[level:], package, level)
#0 1.545   File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
#0 1.545   File "<frozen importlib._bootstrap>", line 991, in _find_and_load
#0 1.545   File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
#0 1.545   File "<frozen importlib._bootstrap>", line 671, in _load_unlocked
#0 1.545   File "<frozen importlib._bootstrap_external>", line 848, in exec_module
#0 1.545   File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
#0 1.545   File "/usr/local/lib/python3.8/dist-packages/transformers/pipelines/__init__.py", line 60, in <module>
#0 1.545     from .document_question_answering import DocumentQuestionAnsweringPipeline
#0 1.545   File "/usr/local/lib/python3.8/dist-packages/transformers/pipelines/document_question_answering.py", line 29, in <module>
#0 1.545     from .question_answering import select_starts_ends
#0 1.545   File "/usr/local/lib/python3.8/dist-packages/transformers/pipelines/question_answering.py", line 8, in <module>
#0 1.545     from ..data import SquadExample, SquadFeatures, squad_convert_examples_to_features
#0 1.545   File "/usr/local/lib/python3.8/dist-packages/transformers/data/__init__.py", line 26, in <module>
#0 1.545     from .metrics import glue_compute_metrics, xnli_compute_metrics
#0 1.545   File "/usr/local/lib/python3.8/dist-packages/transformers/data/metrics/__init__.py", line 18, in <module>
#0 1.545     if is_sklearn_available():
#0 1.545   File "/usr/local/lib/python3.8/dist-packages/transformers/utils/import_utils.py", line 565, in is_sklearn_available
#0 1.545     return is_scipy_available() and importlib.util.find_spec("sklearn.metrics")
#0 1.545   File "/usr/lib/python3.8/importlib/util.py", line 94, in find_spec
#0 1.545     parent = __import__(parent_name, fromlist=['__path__'])
#0 1.545   File "/usr/local/lib/python3.8/dist-packages/sklearn/__init__.py", line 81, in <module>
#0 1.545     from . import __check_build  # noqa: F401
#0 1.545   File "/usr/local/lib/python3.8/dist-packages/sklearn/__check_build/__init__.py", line 46, in <module>
#0 1.545     raise_build_error(e)
#0 1.545   File "/usr/local/lib/python3.8/dist-packages/sklearn/__check_build/__init__.py", line 31, in raise_build_error
#0 1.545     raise ImportError("""%s
#0 1.545 ImportError: /usr/local/lib/python3.8/dist-packages/sklearn/__check_build/../../scikit_learn.libs/libgomp-d22c30c5.so.1.0.0: cannot allocate memory in static TLS block
#0 1.545 ___________________________________________________________________________
#0 1.545 Contents of /usr/local/lib/python3.8/dist-packages/sklearn/__check_build:
#0 1.545 _check_build.cpython-38-aarch64-linux-gnu.so__pycache__               __init__.py
#0 1.545 setup.py
#0 1.545 ___________________________________________________________________________
#0 1.545 It seems that scikit-learn has not been built correctly.
#0 1.545
#0 1.545 If you have installed scikit-learn from source, please do not forget
#0 1.545 to build the package before using it: run `python setup.py install` or
#0 1.545 `make` in the source directory.
#0 1.545
#0 1.545 If you have used an installer, please check that it is suited for your
#0 1.545 Python version, your operating system and your platform.
#0 1.545
#0 1.545 The above exception was the direct cause of the following exception:
#0 1.545
#0 1.545 Traceback (most recent call last):
#0 1.545   File "/get-models.py", line 1, in <module>
#0 1.545     from transformers import AutoTokenizer, AutoModelForSeq2SeqLM, pipeline
#0 1.545   File "<frozen importlib._bootstrap>", line 1039, in _handle_fromlist
#0 1.545   File "/usr/local/lib/python3.8/dist-packages/transformers/utils/import_utils.py", line 1136, in __getattr__
#0 1.545     module = self._get_module(self._class_to_module[name])
#0 1.545   File "/usr/local/lib/python3.8/dist-packages/transformers/utils/import_utils.py", line 1148, in _get_module
#0 1.545     raise RuntimeError(
#0 1.545 RuntimeError: Failed to import transformers.pipelines because of the following error (look up to see its traceback):
#0 1.545 /usr/local/lib/python3.8/dist-packages/sklearn/__check_build/../../scikit_learn.libs/libgomp-d22c30c5.so.1.0.0: cannot allocate memory in static TLS block
#0 1.545 ___________________________________________________________________________
#0 1.545 Contents of /usr/local/lib/python3.8/dist-packages/sklearn/__check_build:
#0 1.545 _check_build.cpython-38-aarch64-linux-gnu.so__pycache__               __init__.py
#0 1.545 setup.py
#0 1.545 ___________________________________________________________________________
#0 1.545 It seems that scikit-learn has not been built correctly.
#0 1.545
#0 1.545 If you have installed scikit-learn from source, please do not forget
#0 1.545 to build the package before using it: run `python setup.py install` or
#0 1.545 `make` in the source directory.
#0 1.545
#0 1.545 If you have used an installer, please check that it is suited for your
#0 1.545 Python version, your operating system and your platform.
------
ERROR: failed to solve: executor failed running [/bin/sh -c python /get-models.py &&     rm -rf /get-models.py]: exit code: 1

cannot allocate memory in static TLS block
我当时改完这个文件也报了另外一个错，没注意那个错跟你这个一样不，升级了docker到version 20.10.24, build 297e128，就构建成功了

cannot allocate memory in static TLS block 我当时改完这个文件也报了另外一个错，没注意那个错跟你这个一样不，升级了docker到version 20.10.24, build 297e128，就构建成功了

还是不行，我把docker卸载重新安装了20.10.24版本，还是依然出现上面的错误，不知道和Apple silicon M1有没有关系

我也是 mac m1 max 同样的错误


Admonition Background Colours/Borders

Id like to request the ability to set the background colours and/or border settings of each admonition block please

#444 is a duplicate of this issue. Since it contains more context, I'm going to close this issue as a duplicate and let that one stand.


FEAT-#2606: Support creating DataFrame from remote partitions

Signed-off-by: Igoshev, Yaroslav<EMAIL_ADDRESS>
What do these changes do?
This PR introduces new API to create DataFrame from remote partitions (there is possibility to create DataFrame from remote partitions with their IPs). Also, the fix was added to unwrap_partitions when returning all partitions. Because we hold all partitions as 2D np.array the function now returns 2D list instead of 1D list.

The API is a little clunky, is there a shorter name?

I can propose two ways to try and improve the naming.
One is make a module named df or dataframe with a method from_partitions, so user code would be
from modin.api.dataframe import from_partitions
# do stuff
df = from_partitions(parts, axis=1)

Another is make a module named partition, then those functions would become modin.api.partitions.unwrap and modin.api.partitions.create_df.
However personally I don't think naming is really bad here - this is for fellow developers anyway, they should be fine with a little longer names if names are descriptive IMHO.

The API is a little clunky, is there a shorter name?

I can propose two ways to try and improve the naming.
One is make a module named df or dataframe with a method from_partitions, so user code would be
from modin.api.dataframe import from_partitions
# do stuff
df = from_partitions(parts, axis=1)

Another is make a module named partition, then those functions would become modin.api.partitions.unwrap and modin.api.partitions.create_df.
However personally I don't think naming is really bad here - this is for fellow developers anyway, they should be fine with a little longer names if names are descriptive IMHO.

The API is a little clunky, is there a shorter name?

As for me, it is most appropriate API. I agree with @vnlitvinov on: "However personally I don't think naming is really bad here - this is for fellow developers anyway, they should be fine with a little longer names if names are descriptive IMHO."

The API is a little clunky, is there a shorter name?

As for me, it is most appropriate API. I agree with @vnlitvinov on: "However personally I don't think naming is really bad here - this is for fellow developers anyway, they should be fine with a little longer names if names are descriptive IMHO."

@devin-petersohn , @vnlitvinov , just a friendly reminder.

@devin-petersohn , @vnlitvinov , just a friendly reminder.

@devin-petersohn , @vnlitvinov , please, look at #2643 . We need to synchronize these PRs.

@devin-petersohn , @vnlitvinov , please, look at #2643 . We need to synchronize these PRs.

@devin-petersohn , just a friendly reminder.

@devin-petersohn , just a friendly reminder.

Sorry for the delay, have been thinking on this. I think modin.api.pandas.dataframe or modin.api.pandas is probably confusing compared to modin.pandas. To me, modin.api is very generic and does not convey what it is doing.
I propose something different, which may also be clunky but should fit in with future plans:
from modin.distributed.dataframe.pandas import from_partitions, uwnrap_partitions

or
from modin.distributed.dataframe import from_partitions, uwnrap_partitions

Where from_partitions accepts an argument declaring the pandas API be used (or another one if that is requested).

Sorry for the delay, have been thinking on this. I think modin.api.pandas.dataframe or modin.api.pandas is probably confusing compared to modin.pandas. To me, modin.api is very generic and does not convey what it is doing.
I propose something different, which may also be clunky but should fit in with future plans:
from modin.distributed.dataframe.pandas import from_partitions, uwnrap_partitions

or
from modin.distributed.dataframe import from_partitions, uwnrap_partitions

Where from_partitions accepts an argument declaring the pandas API be used (or another one if that is requested).

To start off, I love the ample amounts of documentation and inline comments! This will make the lives of external developers using modin a lot easier!
We should avoid using api in the name here because it can be confusing. From an external developer's point of view, it may appear that all of the APIs that modin exposes (e.g., pandas, sql) go through this module. However, this is not the case. I also agree with @vnlitvinov about having developer in the name. However, my issue with api also extends to internal_api.
I like @devin-petersohn's suggestion of modin.distributed.dataframe because it allows for the flexibility to be able to return non-dataframe objects in the future (e.g., returning a SQL-like object). I also agree that the caller should specify the dataframe API that they want to use. Even though everyone will currently only want the pandas API over the pyarrow API, it will set a better precedent.

To start off, I love the ample amounts of documentation and inline comments! This will make the lives of external developers using modin a lot easier!
We should avoid using api in the name here because it can be confusing. From an external developer's point of view, it may appear that all of the APIs that modin exposes (e.g., pandas, sql) go through this module. However, this is not the case. I also agree with @vnlitvinov about having developer in the name. However, my issue with api also extends to internal_api.
I like @devin-petersohn's suggestion of modin.distributed.dataframe because it allows for the flexibility to be able to return non-dataframe objects in the future (e.g., returning a SQL-like object). I also agree that the caller should specify the dataframe API that they want to use. Even though everyone will currently only want the pandas API over the pyarrow API, it will set a better precedent.

@devin-petersohn , I see your idea. To me, from modin.distributed.dataframe.pandas import from_partitions, uwnrap_partitions looks more appropriate because we will probably want to add other implementations of a distributed dataframe (Dask/Ray/etc.). That is fair for an distributed array as well.

@devin-petersohn , I see your idea. To me, from modin.distributed.dataframe.pandas import from_partitions, uwnrap_partitions looks more appropriate because we will probably want to add other implementations of a distributed dataframe (Dask/Ray/etc.). That is fair for an distributed array as well.

Yes, the documentation page will be added as a part of #2643 .

Yes, the documentation page will be added as a part of #2643 .

@YarShev @devin-petersohn Is there a mechanism for directly creating a Series from partitions (other than creating a DF and getting the column)?

@fschlimb That question is better in the Discourse, so it is reverse searchable for  others with the same question: https://discuss.modin.org/


Iscsi auth using k8s secrets

This feature lets us map k8s secrets to VMs for iscsi auth.
Workflow

create a k8s secret containing a password field.  This password will be used by iscsi auth later on.

cat << END > mysecret.yaml
apiVersion: v1
kind: Secret
metadata:
  name: myK8sSecretID
  namespace: default
type: Opaque
data:
  password: $(cat "mySuperSecretPassword" | base64 -w0)
END

create a vm that references the name given to the k8s secret in the iscsi usage field

cat << END > myvm.yaml
kind: VM
metadata:
  name: testvm
  namespace: default
spec:
  domain:
    devices:
      disks:
      - Auth:
          secret:
            type: iscsi
            usage: myK8sSecretID
          username: myUsername
        type: network
        snapshot: external
        device: disk
        driver:
          name: qemu
          type: raw
          cache: none
        source:
          host:
            name: iscsi-demo-target.default
            port: "3260"
          protocol: iscsi
          name: iqn.2017-01.io.kubevirt:sn.42/2
        target:
          dev: vda
    memory:
      unit: MB
      value: 64
    os:
      type:
        os: hvm
    type: qemu
END

Add the secret and vm to the cluster.

kubectl create -f mysecret.yaml
kubectl create -f myvm.yaml

From there, the password field in the k8s secret will automatically be mapped to a libvirt secret when the VM is scheduled to a node allowing the iscsi auth to work without any further configuration.
Container Registry Disk Iscsi Auth
The second part of this PR involves auto generating k8s user/secret combos for container registry disks associated with a VM. This ensures CRD disks can only be accessed by their corresponding VM process.
In the future it would also be wise for us to introduce a network policy that further locks down CRD disks to the local node.

David, why not use K8S iSCSI w/chap and attach a volume to the VM's container and use it from there?
What's the benefit of involving libvirt here?
I'm looking at https://github.com/kubernetes/examples/blob/master/staging/volumes/iscsi/README.md and I'm not sure I'm seeing any differences in functionality?

hey @mykaul

I'm looking at https://github.com/kubernetes/examples/blob/master/staging/volumes/iscsi/README.md and I'm not sure I'm seeing any differences in functionality?

I can see why this looks similar. The big difference here in the document you're referencing k8s has an iscsi device backing a volume that is mounted into the container.
This quote from the doc you referenced might help with making the distinction.
"If you have followed the instructions in the links above you may have partitioned the device, the iSCSI volume plugin does not currently support partitions so format the device as one partition or leave the device raw and Kubernetes will partition and format it one first mount."
Using libvirt, we want to consume iscsi as a block device, and not a mounted filesystem.
In the future, it looks likely that k8s will offer block storage device access.  There's a good chance we will be able to support using k8s features like this at some point, but at the moment it doesn't appear to offer what we need for virtualization.
You've brought up a good point though. We need to start working with the upstream k8s community and make them aware of our use case. That could influence priorities of k8s supported features like this.

@davidvossel  , in general, I'd say that KubeVirt's strategic direction is to work with upstream Kubernetes and assist, develop and review features there so we can consume them natively from it. Of course, this may not always be feasible from both feature-set or time-perspective, but I think in cases where we diverge and implement our own we might be creating a gap.
K8S are indeed working on raw block @ https://github.com/kubernetes/community/pull/805 .
I wonder if a small change in https://github.com/kubernetes/kubernetes/blob/5644e3d77039df66406237e5fb966971522370d8/pkg/volume/iscsi/iscsi.go would allow not to format (for example, passing it without a mount option)...

Of course, this may not always be feasible from both feature-set or time-perspective, but I think in cases where we diverge and implement our own we might be creating a gap.

I agree, this is a tough balance for us right now.
@mykaul The doc you referenced about k8s iscsi support gave me an idea.
K8s requires iscsi auth secrets to be formatted like this. https://github.com/kubernetes/examples/blob/master/staging/volumes/iscsi/chap-secret.yaml
I'll just modify my PR to require the k8s format for the iscsi secrets.  In the future we'll be able to switch to k8s iscsi support transparently on our backend if we want.
Basically, this lets us move forward with iscsi auth, but do it in a way that lets us converge with k8s easily in the future if they provide iscsi block device access.

@rmohr @mykaul
I've rebased the patch and made a few changes.

libvirt secrets are stored in a cache which makes removal much more efficient.

We are now conforming our k8s secrets just like the iscsi chap secrets k8s natively supports. https://github.com/kubernetes/examples/tree/master/staging/volumes/iscsi. I updated the PR description to reflect this change as well.

Item 2 also fixes the weird situation where the username was put in the VM definition, but the password was in the k8s secret. Now both username and password are in the k8s secret. The k8s secret is referenced in the 'usage' field of the disk auth.  Behind the scenes we do all the magic to make that work in virt-handler when the VM is launching.
By formatting our secrets like this, we can silently begin using the native k8s iscsi support once we can have direct access to the block device (rather than k8s trying to mount it as a file system)

I assume similar to OpenStack[1] it does not support Discovery CHAP.
I assume similar to OpenStack[?] it does not support mutual auth.
Both are reasonable limitations, but perhaps should be documented somewhere.
[1] https://github.com/openstack/cinder/blob/master/cinder/volume/targets/iscsi.py#L153

I assume similar to OpenStack[1] it does not support Discovery CHAP.
I assume similar to OpenStack[?] it does not support mutual auth.
@mykaul
Right, the only thing we get with this feature is the target authenticates the initiator... so just client authentication. This is a limitation of how we're using libvirt for iscsi.

I'll note this. I need to move what I have in the PR description to a .md file in the doc's directory. I'll note this there.

another thing @rmohr has pointed out to me is that there is currently a 1:1 relationship between k8s secrets and vms... meaning because of the way libvirt secrets are managed and garbage collected, we can't safely use the same secret for multiple VMs.  I've figured out a way to fix this. I'll update the patch with that and the .md doc shortly.

patches updated.

added iscsi related documentation in /docs folder
added the ability to associate a single k8s secret with multiple VMs
improved libvirt secret cache by keying of domain name

retest this please

alrighty, got tests passing again
@rmohr @fabiand @mykaul How do you all feel about this patch now. Is this something everyone is comfortable with merging at this point?
We have a pretty solid plan for migrating to native k8s iscsi support in the future once k8s provides the actual block device rather than requiring volumes to be a mountable filesystem.
The k8s secret used for KubeVirt iscsi now matches exactly what is used in native k8s. This means we can pivot on the backend to use native k8s for iscsi in the future without impacting users.

retest this please

When it comes to the usage of the secret in the CRD (we should find a different acronym, now that CRDs are so popular in K8s) then it looks like a mechanism to ensure exclusive access to the freshly created iSCSI target.
To me this implementation looks redundant to the PV accessModes and the PVC concept.

@fabiand These are separate concepts solving separate problems. PVCs are solving a problem with exclusive access that we don't have with CRDs.  Iscsi auth is a security feature that both PVCs and CRDs need.
Access modes are a k8s cluster feature that (attempt to) provide guarantees about how volumes are accessed by pods.  ReadWriteOnce, ReadOnlyMany, ReadWriteMany...
For CRDs we don't have a situation where multiple pods/nodes/vms attempt to grab ownership of the same iscsi device. There is no resource contention. The CRD is directly correlated to a specific VM and no other VM is going to attempt to access another VMs CRDs. This is inherent to how the feature is designed. We don't need PVCs to express this for CRDs.  Basically we get it for free.
Authentication is something different. Auth gives us guarantees that nothing outside of the clusters control can access the CRDs.  For example, with auth we limit the ability for an unauthorized user or app to attempt to connect to the iscsi device. This is for security. Both CRDs and iscsi devices managed by PVs need auth as a security feature.

In order to avoid reimplemting this functionality and to strengthen Kubernetes, do you also see a way to achieve your goal by leveraging the Kubernetes PV/PVC, accessMode functionality?

sorry, I kind of implicitly answered this, but probably need to directly answer it.
PVCs are a feature for managing access to network block devices that there is contention for.  With the ReadWriteOnce access model, there is an assumption that there are potentially multiple consumers of a datasource that k8s needs to coordinate access to. Without contention, the access modes feature wouldn't be necessary.
We don't have that contention with a CRD.  The disk is actually a part of the thing that consumes it. The virt-launcher pod + VM process are an atomic unit. The CRDs are containers in the virt-launcher pod and directly tied to the VM lifecycle.  Because of all this, we're guaranteed a 1:1 ratio. no contention.
Here's an example that might make it more clear. If the VM consuming a CRD fails and is started again on another node, the CRDs between the first and second run are completely different. If the same VM definition happened to run twice for some reason because of a cluster network partition, the CRDs associated with each VM process are not shared. As long as we maintain the 1:1 relationship of virt-launcher pods to VMs, there will never be a time where a CRD is accessed by more than one VM process.
All that is to say, without data contention, the CRD feature doesn't gain anything of value by introducing the PVC concept.  We already have exclusive access for free.

latest update includes

kubevirt trunk rebase
iscsi auth documentation updates
iscsi PVs with auth now work

Despite that thee is agreement that the registry disk mechanism needs to be reworked once we have POCed some relevant mechanics (sharing discs between containres).

i created an issue related to this.  #373


Fix - #203 - Remove search from input bar

It is a Simple Fix to remove Search Term from Edit Text[DDGActionBarManager] When the result is opened in an External Browser.

Closing this PR. New PR will be raised as per the suggested changes.


Backend: Endpoint to retrieve specific product by id - GET api/products/{product_id}

User story
Who: As a developer
What: Able to retrieve information about a specific product.
Why: so the user can see the specific product details.
Acceptance criteria

Create an endpoint to retrieve all information about a specific product.

Check the wireframes  Display product page  here: https://www.figma.com/file/lMeDy1n608SLjsKYrKu1nc/HYF-WebShop?node-id=0%3A1
Implementation details
Create the GET endpoint for 'api/products/{product_id}'
Check the README file here https://github.com/HackYourFuture-CPH/fp-class18/#-working-with-api-documentation---swagger and this article should also help you with that https://blog.logrocket.com/documenting-your-express-api-with-swagger/

Already created an issue for retrieving product with productid


Modify condition

Issue: Now server returns a list of deployments with the latestRelease property for and metrics. CLI tries to add the new field metrics for the undefined property latestRelease.
Solution: Add one more condition for checking that the latestRelease isn't undefined.

/azp run


Fix Volume Drawer Crashing

Purpose
Volume drawer was crashing due to this being undefined
Notes

Changed multiple methods to arrow functions so that this would refer to the class

Coverage decreased (-0.02%) to 26.76% when pulling 5a4b446e86b3e34a4145ee3b5a754890540cae77 on martinmckenna:hotfix/fix-volume-drawer-crash into 40ba6fbb9fa2b1a1259cbc69a907b2901c348725 on linode:develop.


Feature request: daemonise caddy

It would be really useful in my opinion if Caddy could daemonise itself (automatically run in the background), being launched in the daemon via a -d flag, or similar.
What're other people's thoughts on this?

@kardianos was actually working on this before. I closed the PR because I think it needs a lot more testing and docs, since this is not a "light" change to make: https://github.com/mholt/caddy/pull/588
I also might need to be convinced that daemonizing is the job of the web server instead of the system/sysadmin. :wink:

Modern systems with for example systemd require that services are running in foreground, docker does that too

I agree that modern job systems have the service run in the foreground.
On Unix type system, no special change is needed except to obey signals and have correct service files placed.
On Windows, participation is required for the service callback and registration is even more arcane.

I could revive that PR if interest, I just kinda ran out of time then. If there are a few others who could help test, I could work on it again.

Lastly, I will note for the average user creating a service (systemd, upstart) file on linux is often beyond what they want to research and create.

@kardianos You're right, which is why I support the change overall, but it will require a lot of testing for potential "gotchas" since there's a lot riding on a feature like that. Right now if Caddy goes down in production it's not Caddy's fault, but if we integrate this capability, we incur a heavy responsibility. And possible maintenance burden. It will have to be tested on many platforms over the course of a long time, I think.

I agree with you in idea I think, but you may be overstating the effect of this change. I think more then anything this is a potential mindset change that you are now "endorsing" then a technical change.
You could add a disclaimer above the flag and in the help message :).

@mholt also consider making deb/rpm packages for easier updating

On Windows, I prefer to use something like nssm to turn my Go binary into service since I don't have to implement the service logic in my source code.
I believe leaving the service logic to another tool (e.g. systemd, upstart, supervisord, etc) that do it well is the way to go.

If we are going to implement this, I think we need to clarify the use cases and whether there are better solutions.
If we just want Caddy to run in the background, why not just caddy & or start /b caddy to send it to background and forget about it?
We probably also want to keep Caddy running, so what happens when Caddy crashes?
At this point we would need something else to monitor our process and what would that be?
If we end up requiring a 3rd party to keep us alive, do we really need our own service logic?

@tpng I don't have a dog in this, however as a windows user I don't see a reason to use nssm. If I run a program as a service it supports windows callbacks.

Maybe a good first step would be adding documentation for adding a caddy
service via nssm. People may still want a native service mode, but that
will are least show people how it could be done without it.

@kardianos Yes, that's just my use case and how I am deploying Go binary to Windows Server.
@captncraig I think we can add a section in the User Guide for deployment recipes on different platform.

Nssm and a native service don't mix. Do one or another. Such a first step
is the step.
On Fri, Apr 8, 2016, 08:47 Craig Peterson<EMAIL_ADDRESS>wrote:

Maybe a good first step would be adding documentation for adding a caddy
service via nssm. People may still want a native service mode, but that
will are least show people how it could be done without it.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/mholt/caddy/issues/732#issuecomment-207488573

I see. Is the native service implemented only for Windows or for all platforms? If it is only for Windows, will it affect other platforms?

@tpng From a Go perspective, you can easily check the architecture and adjust depending on that.

From the previous PR https://github.com/mholt/caddy/pull/588, it seems to support most platforms, so I have no further concerns in this case.

Been looking into this quite a bit and talking some friends with more experience than I on the matter. I don't think Caddy is in the business of daemonizing.
To run Caddy in the background, use &. This is how I run all my sites on DigitalOcean. I've found that on AWS you have to use nohup as well because for some reason the AMI sends Caddy a SIGHUP randomly... anyway, Caddy (core -- can't speak for plugins) doesn't crash in production on its own, so & is a pretty good way to go, honestly.
To make Caddy a service on Linux/BSD, use something like systemd, upstart, or another process/service manager.
To make Caddy a service on Windows, well:

If you’re a developer who wants to release a Windows Service that users can install and uninstall you should use InstallShield.

—MSDN
@kardianos

I agree with you in idea I think, but you may be overstating the effect of this change.

I don't think so. Remember, I'm the one who's liable if somebody's site goes down. :smile: If we don't even attempt this integration (which I still feel is out of scope for Caddy) then the responsibility rests with the sysadmin, which I believe is where it should be.
@mnzt I just now saw your addendum to the original post about it being an add-on. I'm working on restructuring Caddy so plugins (no longer to be called "add-ons" -- finally!) can have a lot more power and flexibility, including the ability to manage the process. Well, I can't promise that yet, but it's what I'm hoping for. (Already some things I was hoping for out of this aren't going to be happening due to technical complications. :disappointed:) But if it works out, add-ons could certainly integrate with the system more than Caddy core is willing to do.
If you'd like to share a way to run Caddy as a service/daemon, you can contribute to the dist/init folder in this repo, as it will be going out with all Caddy releases in the future, starting with the next release.
Thanks for the discussion! Going to close this now but not to shut down the discussion, just to dequeue it from my "todo" list.

Sounds good @mholt! Looking forward to the new plugin system.
Is there a way we can get entries added into the docs? Personally I think it'll be beneficial to have a How to run Caddy in the background on the FAQ

Sure, submit a PR to https://github.com/caddyserver/caddyserver.com


[WIP] Fx79 Frame: Reverse aileron outputs

FX-79 Buffalo Flying Wing as shown below had reversed MAIN 1/2 (it has its own mixer)

@bkueng This modification does the correct thing for the Buffalo and some of the other cases, but as you can see it gets some wrong - ie the other ones below that are empty of MAIN outputs now.
Also, I had to add the thruster MAIN option to the init file, and I would have expected this would be fetched from the default configuration if I did not define it.
Have I done it wrong, or is the toolchain misbehaving?

The grouping logic is as following:

if all airframes specify the same channels for MAIN or AUX, or some leave it empty, they are grouped as common outputs (the assumption is that airframes that don't specify the MAIN outputs for example, they are the same as for the others in that vehicle group).
if the first case does not hold, there are no common outputs, and only Specific is used for each airframe.

As you can see there is no notion of 'default' here - we would need to introduce that if you want it.
Your change switches from case 1 to 2, and some of the frames like Viper don't specify their MAIN at all. This is the reason why you don't see anything.
The simple solution is to add all outputs explicitly for these airframes.

Also, I had to add the thruster MAIN option to the init file, and I would have expected this would be fetched from the default configuration if I did not define it.

The level of granularity is either MAIN or AUX, so either all or none of the MAIN outputs need to be specified - the parser does not know that you want to add a throttle channel if you don't specify it.

@bkueng Thanks for the explanation - I have fixed the other init files as you suggested so this is ready for re-review.
Regarding the module/docs algorithm, I think what you have done is fine.
What doesn't make sense is the mixer file for this particular vehicle setting the outputs differently :-( Where possible IMO outputs should be consistent across vehicles.


Error: did not find an active interpreter

Probably benign, but then maybe should not be reported (or less detail).
2021.4.765268190
plaforms {'win32', 'darwin', 'linux'}

Error
did not find an active interpreter
Error: did not find an active interpreter
 at f.getInitializationOptions (/out/client/extension.js:59:202153)

MPLS will be deprecated soon. So, we won't be addressing this.


caniuse.com Hacked?

In the Google result page I have experienced some strange redirectons.
Here is a page which redirected: https://caniuse.com/pointer-events/embed
To: http://lab5upport305111234.bid/index/?2471509871574
Raw response:
<script>window.location.replace("http://lab5upport305111234.bid/index/?2471509871574");window.location.href = "http://lab5upport305111234.bid/index/?2471509871574";</script><!doctype html>
<!--[if lte IE 9]><html dir="ltr" lang="en-US" class="old-ie  no-js"><![endif]-->
<!--[if gt IE 9]><!--><html dir="ltr" lang="en-US" class="no-js"><!--<![endif]-->
<head>
	<meta charset="utf-8" />
	<title>Can I use CSS pointer-events</title>

	<meta name="viewport" content="width=device-width">

    <link rel="stylesheet" href="http://fonts.googleapis.com/css?family=Open+Sans:300,700">
	<link rel="stylesheet" href="/css/dark.css" media="all">

Keep redirecting to:
top.lovike.xyz

@Fyrd ?

I don't think it happens when you access the site via HTTPS, however, Google search results redirect to HTTP instead of HTTPS.

Spent the past two days on this, cautiously optimistic that it's been solved now. Also defaulting to HTTPS by default so that should help.


[SKIP CI] Updating the script that copies changes to gh-pages from vmware/master

Following changes have been made to the script:

Renamed the script to updateGHpages.sh
Delete the md files from jekyll-docs folder before copying the changes from vmware/master
Fail the script if any command fails
Added more comments for running the script

Renamed the script to updateGHpages.sh

How about renaming to publish_document.sh/update_gh_pages.sh?

Addressed all the review comments. Merging the changes.


Question about document

Hey Jared,
The package look awesome, some great work here.
I can see that there is some signs of docs but looking into the package.json file I don't see a script to build the docs.
Could you please advise how to build the docs for the components?
Would also love to contribute if possible.
Cheers

Hi Praney,
The docs are a work in progress! I started work on a docs site here: https://github.com/jaredLunde/curls-app
As you can see by the last commit date I haven't moved on it in a while and the build is current failing. Knowing that other people have looked at the project is motivating so I will spend some nights this week getting my ducks in a row.
My big concern earlier in the year was that I hadn't written any production apps with it but that has changed now that I have several under my belt.
If you would like to follow along with the progress, I have just created a roadmap to v1 here: https://github.com/jaredLunde/styled-curls/projects
As far as contributing goes, I can fill you in on the patterns I've been using and some of the challenges I'm encountering if you want to have a conversation about that sometime. Perhaps there are things I could be doing better. At the moment my main concern is just fixing the docs site build and getting it live.
Thanks for taking an interest in the project! I'll update here when I fix that build.

Hey Jared,
Thanks for coming back so quickly. I tried the curl-app but the build fails, tried spending an hour trying to fix missing dependencies and npm packages, but could it be that you might have been using local packages that are different than those in the config?
Would you like to consider something like storybook or react-styleguidist for the docs I could help out with it.
It would be great to catch up sometime, I am down-under in Melbourne Australia GMT+10, not sure what time zone you are in. I could be reached with my username various social sites or gmail if you would want to ping and then can setup up a suitable time.
Cheers,
Praney

I did mention in my original comment that the curls-app build is failing. Sorry I didn't make it more clear as that would have saved you some time :( It's hard for me to diagnose because it was so long ago and it wasn't a part of my normal build process - was just tossing something together largely for the purpose of benchmarking direct function calls vs functional components. I'll set up a proper build environment early next week or Sunday night when I get back from the mountains.
I looked at storybook and react-styleguidist and at first glance react-styleguidist looks most desirable to me. In general I like the idea of most of the documentation coming from the code comments because you're killing two birds with one stone that way.
I'm in Mountain time so we've got 8 hours ahead of space between us as you're 16 hours ahead of me. I usually work daily between 9a-5p + 10p-12a MT.
My email is here: https://github.com/jaredLunde
Twitter: https://twitter.com/jaredLunde


[Bug Report] Custom component with el-table-columns: columns are displayed in the wrong order

Element UI version
2.13.0
OS/Browsers version
Any
Vue version
2.6.10
Reproduction Link
http://jsfiddle.net/vbu2he80/7/
Steps to reproduce
Create a custom component with el-table-columns inside
Add it to a table
What is Expected?
The columns order is as specified
In the fiddle it should be: Date, Name, Address
What is actually happening?
The order is different.
In the fiddle it actually is: Name, Address, Date
Seems there was a similar issue already solved: https://github.com/ElemeFE/element/issues/1638

As temporary solution I've tried this: https://github.com/ElemeFE/element/issues/1638#issuecomment-349989437
But it isn't working also

I have the same problem


subset_warming_levels does not verify that the full timeseries exists

Setup Information

xscen version: 0.8.3

Description
As long as start_yr is within ds, subset_warming_level will return something.
https://github.com/Ouranosinc/xscen/blob/8bb4c0a6342fc845d9462eed808ece75caa9cb98/xscen/extract.py#L1267
We should also check for end_yr.
Steps To Reproduce
Let's say that I have a dataset from 1965-2017 with CMIP6_CanESM5_ssp585_r1i1p1.
subset_warming_level(ds, wl=0.5, window=20)
<xarray.Dataset> Size: 393kB
Dimensions:       (warminglevel: 1, time: 240, station_id: 393)
Coordinates:
  * warminglevel  (warminglevel) object 8B '+0.5Cvs1850-1900'
  * time          (time) datetime64[ns] 2kB 1971-01-01 1971-02-01 ... 1990-12-01

subset_warming_level(ds, wl=2.0, window=20) Only 3 years in the dataset!
<xarray.Dataset> Size: 90kB
Dimensions:       (warminglevel: 1, time: 48, station_id: 393)
Coordinates:
  * warminglevel  (warminglevel) object 8B '+2Cvs1850-1900'
  * time          (time) datetime64[ns] 384B 2014-01-01 ... 2017-12-01

subset_warming_level(ds, wl=3.0, window=20)
None

Additional context
I'm sure that we tested that at some point...
Contribution

[ ] I would be willing/able to open a Pull Request to address this bug.

This is making me think that this might be a problem everywhere we cut a period. It never fails or warn. For example, in train we ask for a period, but it will be invisible in the output if the full period was not actually there...
At least in climatological_op, we have min_periods that checks. Maybe this is what you remember testing ?

This is making me think that this might be a problem everywhere we cut a period. It never fails or warn.

Yeah, we should indeed open an issue for that.


Am I able to obtain lava?

This isn't really an issue, more of a question. But I have no idea on how I'm able to obtain lava, if I'm not able to, I sincerely recommend adding a recipe for lava/lava buckets.

As the advancement says: "Fill your bucket with 20 super-heated Cobblestones".
So you need to put 20 Cobblestones in a basin, super heat that with a blaze burner with straw and Biofuel and mix it!

How the straw and biofuel should be held or placed so that the blaze can get hot enough to create lava


Submodules updates

Needs to be rebased on top of main once #180 gets merged, but updates the ml4gw ref and implements support for TensorRT in the pipeline via ML4GW/pinto#28

Rerunning everything with the new modules now to confirm it works then I'll merge


Added example to allow custom script on specific site

Added example to allow custom script on specific SharePoint site
Set-PnPSite <SiteUrl> -NoScriptSite $false

Allows custom script on a specific site. See Allow or prevent custom script for more information.
updated hyperlink to docs.microsoft.com in Example 6 and in parameter -NoScriptSite Allow or prevent custom script

Docs Build status updates of commit 3bcae97:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-pnp/Set-PnPSite.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@get-itips Please guide. Thanks

Hello @Markus-Hanisch Markus, how are you? Hope you are doing good. 👍
Even though the PnP documentation articles are on this repository, I need to kindly redirect you to open PnP cmdlets issues/pull requests here, in that repository, where a member of the PnP community will be able to correct the article/review your PR with your feedback.
The documentation for PnP cmdlets is then synced from that repo into this one.
Thank you!

okay done
https://github.com/pnp/powershell/pull/733


Tabular Display Example For DM24

As an enhancement, please modify the DM24 data display to match the uploaded example.
TabularDM24displayEnhancementExample.docx

This should result in about a 50% reduction in printed lines, when compared to adding a 4th numerically sorted list for DM58 support, and keeping the existing three


fix: clear state on empty state

fixes https://github.com/keboola/storage-api-php-client/issues/134


Add edit xml functionality on license request page

For issue #76

Tested this out locally - merging into the master and deploying.
@tjasmith Thanks for the contribution!


Syntax error: Invalid character

Here you can look at my polyfill.js
Here is my configuration:
/* Polyfill service v3.17.0
 * For detailed credits and licence information see https://github.com/financial-times/polyfill-service.
 *
 * UA detected: ie/11.0.0
 * Features requested: Array.prototype.find,Array.prototype.includes,Object.assign,String.prototype.includes,fetch,matchMedia
 *
 * - Array.prototype.find, License: CC0
 * - Array.prototype.includes, License: CC0
 * - Object.assign, License: CC0
 * - String.prototype.includes, License: CC0
 * - Event, License: CC0 (required by "Promise", "fetch", "XMLHttpRequest")
 * - Promise, License: MIT (required by "fetch")
 * - fetch, License: MIT */

I have an issue:
SyntaxError: Invalid character
   at Anonymous function (http://bmb-dev-weu-m2ga-feature-bump-277-vars-json-proper-folder.azurewebsites.net/bundles/index.js?_etag=c47a8b91546b89642a890f7900ab9f2d:79:13166)
   at g (http://cdn.polyfill.io/v2/polyfill.min.js?features=Object.assign,Array.prototype.find,Array.prototype.includes,String.prototype.includes,fetch,matchMedia:3:6472)
   at s (http://cdn.polyfill.io/v2/polyfill.min.js?features=Object.assign,Array.prototype.find,Array.prototype.includes,String.prototype.includes,fetch,matchMedia:3:4990)
   at Anonymous function (http://cdn.polyfill.io/v2/polyfill.min.js?features=Object.assign,Array.prototype.find,Array.prototype.includes,String.prototype.includes,fetch,matchMedia:3:8746)
   at r (http://cdn.polyfill.io/v2/polyfill.min.js?features=Object.assign,Array.prototype.find,Array.prototype.includes,String.prototype.includes,fetch,matchMedia:3:5124)

I found that it happens in Promise section. In minificated function (even if I remove .min from URL).
What's it: !*** ./src/global.js ***! and !*** ./src/yaku.js ***! ? I think it can be a source of my problem.

@slawiko I set up a JSBin to load your code in IE11 and it does not produce an error:
http://jsbin.com/qumewixoxu/1/edit
I suspect the error is triggered by something in your own code.  If you can post a JSBin link to a reduced test case, we'll investigate the cause, as there might be a bug we should report to the upstream repo.  Feel free to reopen with that info.
Some polyfills are minified regardless of whether you request minified or not, because the original source that we import from is already minified.


Store delta of rows every time the leaderboards are calculated

[ ] Create a history table to store delta
[ ] Update standings service to return delta of last - current standings for each rank


Active window changes from Terminal to SublimeText after hitting enter

Windows Version
Microsoft Windows [version 10.0.26100.2033]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[x] WSL 2
[ ] WSL 1

Kernel Version
2.15.153-1
Distro Version
Ubuntu 24.04
Other Software
WSL Terminal
Sublime Text
Repro Steps
enter commands like git add, git commit
Expected Behavior
The current window should still be the one i'm typing in
Actual Behavior
The active window switches from the terminal to Sublime , preventing me from entering commands after commands
Diagnostic Logs
No response

(Trying to collect the logs, as it seems that the issue doesn't happen when running the collect-wsl-logs.ps1)


Save New Patient Screen after KoboCollect

The user will forget to open the KoboCollect app when they press 'Save' on the New Patient Screen. Desired behavior:

'Save' saves the patient to the database (same as before), but does not close the screen. If save is successful, the save button becomes inactive (greyed out and cannot be pressed anymore).
Maybe show an information/animation somehow that the user should now open the KoBoCollect app to fill in the baseline assessment.
Maybe add an additional 'Finish' button at the bottom which only becomes active if the 'Open KoBoCollect' button has been pressed. The 'Finish' button closes the screen.

→ An even better idea from a UI perspective could be the use of steppers. I.e. first page for entering patient characteristics, second page for KoBoCollect button, third page for finishing.

Implemented as part of Release 0.2.0.


Accept &str as an input to raw_memory::get

Just an API niceness since we don't need to own the string. The value will need to be copied anyways when sending to JavaScript, so we might as well accept a borrowed string.
CC #17. I'd forgotten about this detail before.

👍


Build cache

Test

Woops, did not mean to create the PR here. Meant to test out the new draft PR feature on my on fork...


seqToseq demo Pretrained model can't be downloaded

I want to download the pretrained model by the script "wmt14_model.sh" in the seqToseq demo.  The command is "wget https://www.googledrive.com/host/0B7Q8d52jqeI9ejh6Q1RpMTFQT1k/wmt14_model.tar.gz --no-check-certificate". I tried many kinds of VPNs but failed to download. So I wonder if you can provide a download site which can be reached in China without VPN.

You can have a look in wmt14_model.sh, it says:
# you can also directly download from https://pan.baidu.com/s/1o8q577s

@luotao1 @dayhaha It should be added to Paddle CDN. I will handle it.


#143305379 Set up Authentication

What does this PR do?
set up login and sign up
Description of Task to be completed?
Set up both user sign up and login
How should this be manually tested?
What are the relevant pivotal tracker stories?
#143305379

Coverage decreased (-0.9%) to 80.882% when pulling 076b69457368e9248c62ec3d01e8b1569ca81acf on feature/143305379/configure-login into 5d11c3c91c2285bedbc012bacb9473e17e61c2d5 on feature/143305379/configure-frontend-doc-routes.


Expose City and Role constructors from SalaryCalculator module

So that in tests we can use them directly. No need to make wrapper
functions - type constructors are functions themselves.

Thanks 😊


Feature: Inform user to take a note of the energies when leaving the app

See https://stackoverflow.com/questions/58199849/how-to-detect-that-a-client-closed-the-browser-on-blazor-server-side-web-socket?msclkid=edf2d229b6b011eca6e511374e896d28

Unless you manage to fix #96. Because leaving the app can only accomplished via Dispose() and that seems to be an unreliable solution when it comes to users' interactions.

Maybe this sample will help: https://github.com/ctrl-alt-d/BlazorConfirm


Isnt teams supported by WVD in Machine install mode

I thought that teams is available for wvd now, which should be installed in machine mode? Also, does the xml template have a line to disable teams if its not supported?

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

ID: dbf7e52a-b2b1-0372-0e62-12f6733fc699
Version Independent ID: a21ed743-3fbc-0d20-6947-188744e0a2f7
Content: Install Office on a master VHD image - Azure
Content Source: articles/virtual-desktop/install-office-on-wvd-master-image.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

Looks like there is some support for it now.
https://docs.microsoft.com/en-us/azure/virtual-desktop/teams-on-wvd


Bug: Model->insert() method returns 0 after successful database insertion if table's primary key does not auto increment.

PHP Version
8.0
CodeIgniter4 Version
4.1.1
CodeIgniter4 Installation Method
Composer (using codeigniter4/appstarter)
Which operating systems have you tested for this bug?
Linux
Which server did you use?
apache
Database
MariaDB
What happened?
I'm not sure whether this acutally is a bug or whether it was intentionally designed this way. However, if one uses $model->insert() to insert a record into a table with a non-auto-incrementing primary key, the return value will be 0 after a successful insertion. Obviously, in this scenario one will have had to include the primary key value within the record being inserted so one can test for successful insertion by adding a subsequent query to check for the existence of a record with said value as primary key, e.g.
if ($model->find(PRIMARY_KEY_VALUE_OF_INSERTED_RECORD !== null)
If one is using the return value of ->insert() to perform error handling in case of a failed insertion, this can be cause for confusion.
Steps to Reproduce
Create a table with a primary key that doesn't auto-increment. Create a model linked to this table. Insert a record using ->insert() and check the return value. Check the table itself to confirm whether insertion was successful.
Expected Output
I would expect ->insert() to return a truthy value after any successful insertion, regardless of whether an auto-incrementing primary key is being used or not.
Anything else?
No response

Can't reproduce in the latest develop.
create table test (
  id int(11) not null,
  name varchar(255) not null,
  primary key (id)
);

<?php
namespace App\Models;
use CodeIgniter\Model;
class TestModel extends Model
{
    protected $table = 'test';
    protected $useAutoIncrement = false;
    protected $allowedFields = ['id', 'name',];
}

<?php
namespace App\Controllers;
use App\Models\TestModel;
class Home extends BaseController
{
    public function index()
    {
        $model = model(TestModel::class);
        $id = $model->insert(['id' => 2, 'name' => 'test']);
        var_dump($id); exit;
    }
}

$ php public/index.php
int(2)

My apologies. I hadn't set $useAutoIncrement to false.
On Wed, Dec 15, 2021 at 9:21 AM kenjis @.***> wrote:

@dragonbiketoursvn https://github.com/dragonbiketoursvn Thank you for
reporting.
I can't reproduce with the latest develop.
It might be fixed.
Update to v4.1.5, and check it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codeigniter4/CodeIgniter4/issues/5466#issuecomment-994228842,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARFAJJWSXFKNDM7SZJAEWP3UQ73S5ANCNFSM5KAEXZEA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@dragonbiketoursvn No problem.
Nobody's perfect.


Consistent trailing punctuation in README lists.

We need sync for all line. Dot or none, we can't do like that.

Thank you! 😄
I looked at markdownlint to see if it could enforce this particular rule, among others. Sadly not, this is the only related rule I could find, but it applies to headings and not lists.

That's okay, I might be the one who enforced that rule for the team. Wish can help you.

@luanldt Issues tagged with Good First Issues are a great starting point. Also, if you face any issues while developing, feel free to reach out to the community on Slack!


Wrong documentation

Describe the bug
The documentation give wrongful statement in Parsing Guide section. moment() and moment.utc() doesn't behave like described. So either it's a bug or the documentation is outdated.
To Reproduce
Steps to reproduce the behavior:

run moment.utc('2021-03-04T17:00:00+02:00').toLocaleString()
result is Thu Mar 04 2021 15:00:00 GMT+0000

Expected behavior in documentation
Thu Mar 04 2021 17:00:00 GMT+0000
Please run the following code in your environment and include the output:
console.log((new Date()).toString())
console.log((new Date()).toLocaleString())
console.log((new Date()).getTimezoneOffset())
console.log(navigator.userAgent)
console.log(moment.version)

"Mon Apr 05 2021 18:18:26 GMT+0000 (Coordinated Universal Time)"
"4/5/2021, 6:18:26 PM"
0
"Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Firefox/78.0"
"2.18.1"

False alert


helm completion fails in zsh

The helm completions fail in zsh source <(helm completion)
It specifically fails on
if [[ $(type -t compopt) = "builtin" ]]; then
    complete -o default -F __start_helm helm
else
    complete -o default -o nospace -F __start_helm helm
fi```

I was able to reproduce this bug as well

This is not technically a bug with Helm itself, but rather with Cobra. In that issue they mention a workaround that kubectl uses to fix this here.
@technosophos do we want to keep this open to try and implement this fix ourselves or just wait for Cobra to fix it?

Given that the Cobra bug has been open for two years, maybe we should fix it here. :-|

@technosophos, @thomastaylor312 -- Gonna take a look at a fix for this. Thanks!


sp-core: remove useless bounded module

the file primitives/core/src/bounded.rs was forgotten to be deleted when using the bounded-collections crate

bot merge


Update Redux commit hash

Summary

This PR updates the redux commit to include this changes
Ticket Link

https://mattermost.atlassian.net/browse/MM-22201
Related Pull Requests

Has server changes (please link here)
Has redux changes (please link here)
Has mobile changes (please link here)

https://github.com/mattermost/mattermost-redux/pull/1051

/update-branch

@ethervoid can you please resolve the conflicts?

@enahum given that this https://github.com/mattermost/mattermost-webapp/pull/4947 PR has been merged and includes a most recent commit should I point this branch to release-5.21 and only merge it for that version?

@ethervoid I’ll ping you in about 1.5h and walk you through it

@amyblais this should go in 5.21

@ethervoid
Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Fetching origin
+++ Updating remotes done...
+++ Creating local branch automated-cherry-pick-of-#4928-upstream-release-5.21-1583239272
Branch 'automated-cherry-pick-of-#4928-upstream-release-5.21-1583239272' set up to track remote branch 'release-5.21' from 'upstream'.
+++ Downloading patch to /tmp/4928.patch (in case you need to do this again)

+++ About to attempt cherry pick of PR. To reattempt:
  $ git am -3 /tmp/4928.patch

Applying: Update Redux commit hash
Using index info to reconstruct a base tree...
M	package-lock.json
M	package.json
Falling back to patching base and 3-way merge...
Auto-merging package.json
CONFLICT (content): Merge conflict in package.json
Auto-merging package-lock.json
CONFLICT (content): Merge conflict in package-lock.json
Patch failed at 0001 Update Redux commit hash
Use 'git am --show-current-patch' to see the failed patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

+++ Conflicts detected:

UU package-lock.json
UU package.json

+++ Aborting in-progress git am.

+++ Returning you to the master branch and cleaning up.


Jasmine

This patch adds client-side testing support for travis. I've modified the makefile and package.json to enable a test runner. In order to support es6, I need to pass the --harmony flag to node and so I used mauvm's jasmine/es6 recipe.
Unit tests are ran using jasmine 2.0. Because some complex code nesting causes a segfault in node 5, I've opted to use node 6, the latest version of node, over nvm which travis supports.
The travis file has also been modified to run npm install in the client dir prior to running tests. This is needed to install the jasmine dev dependency.
I've also added a small change for clarity: I modified the webpack rules to use the app name in the outputs.
Development workflow change
This breaks backwards compatibility with developers who have been using node.js 5.0 or prior, as it drops the --harmony_destructuring flag. In order for you to run the client tests on your local machine, you need to enable node 6.3 or later using nvm:
nvm install 6.3
nvm use 6.3

LGTM


function syllable_count returns a float value

I find that syllable_count function returns non-integer result, and is always a multiple of 0.9
e.g. syllable_count('Hello') -> 1.8
e.g. syllable_count('how') -> 0.9
e.g. syllable_count('are') -> 0.9
e.g. syllable_count('you') -> 0.9
what does 0.9 syllable mean? shouldn't it be just 1 syllable
moreover, the syllable_count does not seem to follow the rule of summation, by calculating each word's syllable count, I expect the total syllable count of 'hello how are you' to be 1.8 + 0.9 + 0.9 + 0.9 = 4.5
but syllable_count('hello how are you') -> 5.4
Is this a common problem? Attached the screenshot as proof of the issue.

This is updated. Thanks


error ESP32

in microWire.cpp
'TWBR' was not declared in this scope
'TWSR' was not declared in this scope
'TWCR' was not declared in this scope
'TWEN' was not declared in this scope
'TWIE' was not declared in this scope
platformio, lolin 32d pro board

you cant use microwire on esp32 or other non-AVR platform

Any solution for run GyverOLED on esp32?

there is no problem with esp32


[SPARK-19003][DOCS] Add Java example in Spark Streaming Guide, section Design Patterns for using foreachRDD

What changes were proposed in this pull request?
Added missing Java example under section "Design Patterns for using foreachRDD". Now this section has examples in all 3 languages, improving consistency of documentation.
How was this patch tested?
Manual.
Generated docs using command "SKIP_API=1 jekyll build" and verified generated HTML page manually.
The syntax of example has been tested for correctness using sample code on Java1.7 and Spark 2.2.0-SNAPSHOT.

Can one of the admins verify this patch?

Thanks @srowen  for review and comments, made changes and committed on branch.

Test build #3518 has started for PR 16408 at commit ebb5e5d.

Test build #3518 has finished for PR 16408 at commit ebb5e5d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged to master/2.1


[Bug]: Reading from BigQuery provides inconsistent schemas

What happened?
When doing a BigQuery Read like
p | beam.io.ReadFromBigQuery(
    table='apache-beam-testing:beam_bigquery_io_test.taxi_small',
    output_type='BEAM_ROW')

the TIMESTAMP fields are converted to fields of schema type Field{name=event_timestamp, description=, type=LOGICAL_TYPE<beam:logical_type:micros_instant:v1>, options={{}}} whereas in Java they are converted into (incompatible) fields of schema type Field{name=event_timestamp, description=, type=DATETIME, options={{}}}.
The Python one is probably the one that is wrong here. In addition, one cannot write elements of this type to another BigQuery table as one gets
  File "/Users/robertwb/Work/beam/incubator-beam/sdks/python/apache_beam/io/gcp/bigquery_file_loads.py", line 261, in process
    writer.write(row)
  File "/Users/robertwb/Work/beam/incubator-beam/sdks/python/apache_beam/io/gcp/bigquery_tools.py", line 1432, in write
    return self._file_handle.write(self._coder.encode(row) + b'\n')
  File "/Users/robertwb/Work/beam/incubator-beam/sdks/python/apache_beam/io/gcp/bigquery_tools.py", line 1379, in encode
    return json.dumps(
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/json/__init__.py", line 234, in dumps
    return cls(
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/json/encoder.py", line 199, in encode
    chunks = self.iterencode(o, _one_shot=True)
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/json/encoder.py", line 257, in iterencode
    return _iterencode(o, 0)
  File "/Users/robertwb/Work/beam/incubator-beam/sdks/python/apache_beam/io/gcp/bigquery_tools.py", line 152, in default_encoder
    raise TypeError(
TypeError: Object of type 'Timestamp' is not JSON serializable [while running 'WriteToBigQueryHandlingErrors/WriteToBigQuery/BigQueryBatchFileLoads/ParDo(WriteRecordsToFile)/ParDo(WriteRecordsToFile)']

Issue Priority
Priority: 2 (default / most bugs should be filed as P2)
Issue Components

[X] Component: Python SDK
[ ] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[X] Component: IO connector
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

This occurs when using the BigQuery streaming write API too. My bigquery.Timestamp schema field type is translated to DATETIME which is causing an input schema output schema mismatch error.
The thread here seems to suggest it's been fixed but this might be in a pre-release not current (2.49.0 at the time of writing). Do we have an ETA on this fix?
We can use the legacy streaming API but data in the streaming buffer is not queryable for ~2-3 minutes removing the possibility of any sort of real time queries.

@JoeCMoore For BigQuery streaming write API use case please try add
LogicalType.register_logical_type(MillisInstant)

before pipeline creation. This would solve the issue.
This was due to #22679 makes the source-of-truth for shcema translation to be in python side, and then logical types with same language types has conflict (MillisInstant, MicrosInstant). This workaround was used to fix tests: https://github.com/apache/beam/commit/b0484e792bc615b7494b5195e3ef3d464ec7e426

@Abacn is there a way to fix this long term? it seems like we should just have millisinstant


Reuse / Fork base Jupyter JVM kernel

Such as https://github.com/twosigma/beaker-notebook-private/tree/master/kernel/base/src/main/java/com/twosigma
Or the potentially future base implementation from https://groups.google.com/forum/#!topic/jupyter/O1KnaEPqCM4

So, this just happened: Beaker is now a set of JVM extensions for Jupyter. This might be a very good thing for our project...

Very good news. Looks like they are also going to re-use the following https://github.com/jupyter/jvm-repr
See here for details : https://groups.google.com/forum/#!topic/jupyter/OaaqlsuMaWM

Here is what we want to fork/re-use : https://github.com/twosigma/beakerx/tree/master/kernel/base/src/main/java/com/twosigma/jupyter

master now uses the BeakerX base kernel. Code can't be executed yet since Evaluator is not implemented. I am working on it !


Fix Pub/Sub extended properties tests errors

This adds additional mocks in the PubSubExtendedBindingsPropertiesTests so the test does not do real checks for the existence for topics/subscriptions (it is only interested in verifying the parsing and setting of the pub/sub extended properties).

There is some failure on Travis:
[ERROR]   PubSubMessageChannelBinderEmulatorTests>AbstractBinderTests.testSendAndReceive:198 » ClassCast
[ERROR]   PubSubMessageChannelBinderEmulatorTests>AbstractBinderTests.testSendAndReceiveNoOriginalContentType:403 » ClassCast

Doe it say anything to you?
Thanks

There is some failure on Travis:
[ERROR]   PubSubMessageChannelBinderEmulatorTests>AbstractBinderTests.testSendAndReceive:198 » ClassCast
[ERROR]   PubSubMessageChannelBinderEmulatorTests>AbstractBinderTests.testSendAndReceiveNoOriginalContentType:403 » ClassCast

Doe it say anything to you?
Thanks

Yeah we thought it was going to be solved by this; however it appears to be a separate issue. Investigating now.

@artembilan We're looking into it.


New sample: Display route later

Description
Create a new sample display route layer.
Type of change

New sample implementation

Platforms tested on

[x] WPF .NET 6
[x] WPF Framework
[x] WinUI
[x] MAUI WinUI
[x] MAUI Android
[ ] MAUI iOS
[ ] MAUI MacCatalyst

Checklist

[x] Self-review of changes
[ ] All changes work as expected on all affected platforms
[x] There are no warnings related to changes
[x] Code is commented and follows .NET conventions and standards
[x] Codemaid and XAML styler extensions have been run on every changed file
[x] No unrelated changes have been made to any other code or project files
[ ] Screenshots are correct size and display in description tab (800 x 600 on Windows, 600 height mobile screenshots for MAUI)

Did not want to take MAUI screenshot in Android considering that we're switching to Windows for screenshots - awaiting screenshot tool.


[Suggestion] Cross-Platform Support Implementation

I have a suggestion for making PySilon work cross-platform: You can automatically detect the infected operating system and only enable windows-specific features if they are supported.
To detect if it the operating system is Windows, you can use the platform module:
import platform

def is_windows():
	return platform.system() == "Windows"

Yeah that's possible, usually people who make malware like that would use os.name and if it returns 'nt' you know you're on windows. But you would be missing quiet a few of things if you changed to linux. Let me rank it:
Impossible:

registry injection
mic recording

Very hard:

key logger
processes
reverse shell
webcam
grabber (the locations for passwords are in completely different places)
file delete (there is a lot of management using rwx, so file access could be difficult quiet often)
reverse shell

Doable:

file upload
file download
file exploring
screenshots

It might not be worth the effort, but making the compiler work on linux that would be good I think

Hey! I am really sorry for my recent absence. I will look forward to fix the issues in the following week.


for adoption, consider supporting pip-tools files too, controlled by environment variable (or auto detected)

It would be great to be able to incorporate requirements.in files from pip-tools

@kennethreitz is this supported with the new pip-tools dependency? Or was it closed for other reasons/

closed because it's not happening :)


W32/Exploit.gen is Symbol desktop wallet virus? whats going on?

Panda security says its virus and block from installing, any idea why?

any extra logs from Panda?

This is well-known False-Positive. just search for W32/Exploit.gen and you'll have ton of issues...
They somehow managed to mark 1Password using same mechanism...