audacity: 3.1.3 -> 3.2.1, add aarch64-darwin support

Description of changes
As the title says. Also added myself as a maintainer.
https://github.com/audacity/audacity/releases/tag/Audacity-3.2.1
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

audacity-gtk2 was the only way to run audacity on plasma due to https://github.com/NixOS/nixpkgs/issues/149812
it's sad I can't use audacity anymore on NixOS

@rapenne-s Sorry for my being on GNOME. Could you please test if adding wrapGAppsHook to nativeBuildInputs works?

@rapenne-s Sorry for my being on GNOME and Sway. Could you please test if adding wrapGAppsHook to nativeBuildInputs works?

This fixes it :+1: :rocket:


Faulty redirect to start page after deletion of an admin on course page

Describe the bug
When deleting an admin from a course page the deletion causes a faulty redirect.
To Reproduce
Steps to reproduce the behavior:

Go to a course page
Add an admin
Delete it
Get redirected to the start page

Expected behavior
The page should stay the same, ideally not even reload.
Screenshots

Desktop (please complete the following information):

OS: Manjaro GNOME
Browser: Firefox 105.0.3
Version [e.g. 22]

Seems to be not a problem on dev, just on the branch I've accidentially checked out.

Looks like this happens because admin removed themselve from the course admin list. As they are not an admin now, staying on the course administration page doesn't make sense for them.

So this is unintended intended behavior? Also, I can't reproduce this issue as it wouldn't let me remove the last admin.

This behavior is intended imo

I mean this case is kinda weird, since I actually had admin rights and was not a lecturer, so it doesn't make much sense to add an admin user. Maybe we should just prohibit adding admin users as lecturers for a course?
I think this is fine for the PR, but possibly general edge case.


SecRuleEngine off - still performance numbers drop

Hi
modsecurity version - 2.7.5
nginx versoin - 1.4.2
owasp rules set - 2.2.8
I am using modsecurity as WAF inline with nginx serving as reverse proxy.
I am trying to measure the performance of nginx with mod security with SecRuleEngine - off,detection,on and without modsecurity.
What i find is as soon as modsecurity is introduced in nginx.conf ,though SecRuleEngine  is set to off,the performance - transaction per second - TPS drops.There is no significant difference between SecRuleEngine  being set to on/off/detection in TPS measurement.
I could see in the modsecurity debug log that the transactions are not processed
My question is :
What i dont understand is,even though the transactions are not processed in modsecurity when SecRuleEngine is off  why the performance should drop when it is introduced in nginx?
configuration                                                       TPS
nginx only                                                             3594
nginx+modsec+SecRuleEngine-off               1399
nginx+modsec+SecRuleEngine-detection    342
nginx+modsec+SecRuleEngine-on                 580
Load generator/Stress tool used  - weighttp(WG)
Configuration:
The following configuration is kept on for all the below given tests
total request - 500000
connection rate - 100/s
keep alive - on
file size fetched - 1kb
cpu - 1
All the fetches are genuine ,No threats were injected
VM size - small
nginx running process - 1 each master,worker,cache manager process

Marking as won't fix in 2.x. That is already Ok in libmodsecurity.
https://github.com/SpiderLabs/ModSecurity/tree/v3/master


CSV decode error

I am using Tensorflow 0.9.0. I have CUDA Driver Version = 7.5 and CUDNN 4 on Ubuntu 14.04
This maybe related to another issue I found here
I have a simple csv file which has a single line like this
"field with
newline",0

where the newline has been added by pressing Enter key in vim on Ubuntu.
I am able to read this file in pandas using the read_csv function where the text field is shown as containing a single \n character.
But when I try to read it in tensorflow, I get the following error:
tensorflow.python.framework.errors.InvalidArgumentError: Quoted field has to end with quote followed by delim or end
My tensorflow code to read the csv uses this function to read a single row.
def read_single_example(filename_queue, skip_header_lines, record_defaults, feature_index, label_index):
    reader = tf.TextLineReader(skip_header_lines=skip_header_lines)
    key, value = reader.read(filename_queue)
    record = tf.decode_csv(
            value,
            record_defaults=record_defaults)
    features, label = record[feature_index], record[label_index]
    return features, label

If I read using pandas and replace all newlines with spaces, the tensorflow code is able to parse the csv successfully.
But it will be really helpful if newlines can be handled within the Tensorflow CSV pipeline itself.

I tried this on Tensorflow 0.10 and the issue is still there. Please advise on possible ways to solve the problem.

@jmchen-g, it looks like you wrote the CSVReader original. It seems like it is possible that each row of a csv may need to consume an arbitrary number of lines, because embedded unescaped newlines are allowed in the format.  https://tools.ietf.org/html/rfc4180.  Section 2 talks about CRLF's even being allowed.

Isn't the use of TextLineReader the problem?

Good point... to expand on @vrv's suggestion... He pointed out ot me that maybe WholeFileReader could possibly work. Can you try that and let us know if that solves your problem?

I tried this in Tensorflow 0.10. I changed to use WholeFileReader instead of TextLineReader.
I get the following error tensorflow.python.framework.errors.InvalidArgumentError: Unquoted fields cannot have quotes/CRLFs inside
Also I have another question about using a WholeFileReader. If the input csv file is large wouldn't WholeFileReader bring all of the data into memory, causing an out of memory error?

Yeah WholeFileReader is not a great solution for large files, but it at least allowed testing whether the CSVReader was robust to CRLFs. In any case it seems like a bug in the csv reader. Assigning to @jmchen-g.

This is not a bug but intended behavior. In decode_csv, the input is expected to be a tensor of string/record, each element string is expected to be a record. Inside the record, it is already supported that you can have '\n' in a string field. It is expected that some pre-processing is done to satisfy this requirement.
If we don't have such constraint, the example below will have a problem to decide if 5 is from the second or third record:
"
1,2,3, (first record)
,4,  (second)
5,6,7 (third)
"

@jmchen-g, That reasoning  makes sense from a simplicity of implementation stand point, but then we are technically not adhering to the RFC.
https://tools.ietf.org/html/rfc4180
Section 2  item 6 specifically mentions this case. If we aren't going to change it, I suggest we just add this deviation from the standard in the documentation to make it easier for users to know to do the preprocessing. It is also true that CSV is a very de facto standard anyways and there are many variants.

hi @aselle and @jmchen-g , I think I understand that if some preprocessing is done to extract records from the file and feed it to decode_csv.
Currently there is no CSVReader which can read a single csv record for decode_csv. I think all that is required is to wrap TextLineReader, and read till a \n, determine if we are within a quoted field. Will counting number of double quotes and seeing if it is odd suffice to validate this?
Then if we are inside quoted field read more lines until number of quotes becomes even. When that happens return the record.
I don't know how to write this class and replace TextLineReader with this. Can you guide me to write it?

@aselle We already allow section 2 item 6 case in the current implementation. The only issue here is that the current op expect a tensor of records, each of which is a record.
You will need to either preprocess the csv to generate the tensor of records or have a new op that takes the whole file and process on the fly. It is better to just convert it directly instead of composing tensor of records and do another traversal of it in this op.

I hit this issue too. I was hoping to be able to use the in-graph loading of data to simplify a multi-GPU pipeline.
If decode_csv() really needs just one record then the reader needs to itself do some parsing of the CSV just to see when a record ends, which seems a bit counter to the current separation of reader/decoder. Using WholeFileReader would work for me (I can just split up the inputs into multiple files) and I was thinking of having a go making decode_csv() handle multiple input/output records.
An alternative would be to have something like a CSVLineReader that just has a simple "am I inside a quoted field" check and is otherwise like TextLineReader. Sounds from the comments above that this might be a more welcome addition?

@darrengarvey I suppose you could go ahead and have a CSVLineReader. Note that this will complicate things when we allow splittable files.

@drpngx - Can you give any more information on what you mean by "splittable files"?
In the end I've been going more in the direction of writing (a much more complicated) input function that feeds this data into a tf.learn Estimator using multiple threads in Python that feed data into a queue. This is basically done but seemingly very complicated to get right.

"Splittable file" is hadoop parlance for files that can be split for parallel processing. An example of that is a simple file, say, that has fixed-length records. Each thread can skip to an offset and read from there. An example of a non-skippable file is stream-compressed file.

I experienced this error when moving my code to windows ( running tensorflow in Anaconda3). In my case, I also work with csv data created by myself.  However, the error occurs only when I create the csv data running my code in Windows, so it is definitely related to the way the OS ends each line of the file. The question is, what extra consideration should I make in order to create the file  in Windows that does not have this problem?  To create the file  I use the csv library available in python.

@slothkong please ask this question on stackoverflow.

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!

Here's the updated link for tf.data.experimental.CsvDataset


Update A Wall of Roses to Ver. 1.1

Design Team require an update for A Wall of Roses:

A Wall of Roses (Ver. 1.1)
X Cost | Loyal [tyrell] Event
Reaction: After an attacking character is removed from a challenge, choose X defending characters (cannot be 0). If you win this challenge, gain 2 power and stand each of those characters.

For reference, here is the previous version of this card:

A Wall of Roses (Ver. 1.0)
X Cost | Loyal [tyrell] Event
Reaction: After an attacking character is removed from a challenge, choose X defending characters. If you win this challenge, gain 2 power and stand each of those characters.

General steps

[ ] Update the code for this card (within 'server/game/cards/{current pack}')
[ ] (Optional) Test the implementation locally
[ ] Create Pull Request to develop branch, linking this issue in the description as 'Closes: #[this issue]'

Alternatively, it will also be automatically closed once this card is marked as "Implemented" on the spreadsheet.

The card technically already does this (the minimum to spend is already 1). Closing without changes


Bug: "string".gsub(/\0*\s*/, "") is just eating CPU

You can try this:
crystal eval 'puts "Parsing"; "string".gsub(/\0*\s*/, ""); puts "Done" '
You will see Parsing and 100% CPU load from compiled program.
Crystal 0.7.3 [bf72b07] (Sun Jun  7 16:17:59 UTC 2015)
Darwin Kernel Version 15.0.0: Wed May 27 20:17:31 PDT 2015; root:xnu-3<IP_ADDRESS>.13~1/RELEASE_X86_64 x86_64

Seems like it's related to the #787

Seems like * (zero-or-more) is breaking it.

This is fixed in HEAD. You can check this using the latest standard library. Just clone the repo and from that directory do:
bin/crystal eval 'puts "Parsing"; "string".gsub(/\0*\s*/, ""); puts "Done" '

It should work fine. Note that for this you don't need to compile a new compiler, bin/crystal will pick the new standard library.


crypto/x509: support OIDs for tcg-kp (<IP_ADDRESS>)

What version of Go are you using (go version)?

$ go version
go version go1.16.6 linux/amd64

Does this issue reproduce with the latest release?
Yes, according to the sources.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/user/.cache/go-build"
GOENV="/home/user/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/user/repos/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/user/repos/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.16.6"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build169247540=/tmp/go-build -gno-record-gcc-switches"

What did you do?
I'm playing around TPM Endorsement Key certificates. One of the things I want to archive is to verify a EK certificate against the root CA. The function looks like
func (v *Verifier) VerifyEK(ekPem []byte) (bool, error) {
	block, _ := pem.Decode(ekPem)
	if block == nil {
		return false, errors.New("failed to parse certificate PEM")
	}

	ek, err := x509.ParseCertificate(block.Bytes)
	if err != nil {
		return false, fmt.Errorf("failed to parse certificate: %w", err)
	}

	opts := x509.VerifyOptions{
		Roots: v.ekRoots,
		Intermediates: v.ekIntermediates,
	}
	if _, err := ek.Verify(opts); err != nil {
		return false, fmt.Errorf("failed to verify certificate: %w", err)
	}

	return true, nil
}

What did you expect to see?
I expect all ExtKeyUsages to be correctly parsed with ParseCertificate.
What did you see instead?
UnknownExtKeyIsage of the x509.Certificate object (ek) is not empty. It contains asn1.ObjectIdentifier  <IP_ADDRESS>.1.
The reason
crypto.x509 has extKeyUsageOIDs.

extKeyUsageOIDs contains the mapping between an ExtKeyUsage and its OID

There is no any tcg-kp OID.
The proposal
Add  tcg-kp OIDs to extKeyUsageOIDs. However, I understand that it can be excessive as there are plenty of other OIDs people may wish to add.

cc @FiloSottile


[Meta issue] Restructure request parameters

This is a meta issue for tracking improvements for the structure of request parameters.
Please don't add comments to this issue--rather open new tickets, if they apply, and reference this as the parent.

Link to parent #560

Unlinking this from #560, it's worthy of it's own meta issue.

Oh, they're restructured.


Update 0315-placeholder-types.md

add placehholder link
#-hidden-code /* or i should write address instead? ) */ #-end-hidden-code

This change isn't adding anything


MAINT: remove scipy.weave submodule

scipy.weave has been deprecated since 0.15.0. It has been available as a standalone package (https://github.com/scipy/weave) for 2.5 years now. So I think it's time to remove it.
I will mention this on the mailing list as well.

LGTM, +1 for merge but you somehow already need a rebase

yeah, release notes are annoying for merging.
will rebase once the mailing list message has gone through, so that we can merge quickly after it

Three +1's in total and no complaints. Here is the mailing list thread for reference: https://mail.scipy.org/pipermail/scipy-dev/2016-September/021534.html

Merging before it needs another rebase then.


Documentation for deploying app versions

Is your feature request related to a problem? Please describe.
Add documentation, like a slide deck, for how to deploy the main, staging, and testing instances of the app.
Describe the solution you'd like
A clear and concise description of what you want to happen.
How important is this feature? Select from the options below:
• 🌗 Medium - can do work without it; but it's important (e.g. to save time or for convenience)
When will use cases depending on this become relevant? Select from the options below:
• Short-term - 2-4 weeks

Documentation for shinyapps.io

Related to https://github.com/orgs/Sage-Bionetworks/projects/42/views/10?pane=issue&itemId=7041054


Question: Should we consider fs/promises a builtin module?

See title

Yes, and all the other sub-exports, like timers/promises, etc.

I'll work on it.
Don't switch to ESM before fixing it, since we are going to use it in eslint-plugin-unicorn and ESLint rules don't support ESM. 😄

Don't switch to ESM before fixing it, since we are going to use it in eslint-plugin-unicorn and ESLint rules don't support ESM. 😄

👍


Make help legend less busy

*Overwrite old text with spaces
*Nicer line breaks
*Corrected / added missing commands

I think you can still have just 2 assignments per command, I was just thinking to split the 2 lines strings in slightly different places.

I was thinking it would be ideal to do the help message to the side of the table/phrase number indicator, but honestly haven't checked if it would fit well.
Another option would be to replace the notes on the bottom of the screen (but leave the map intact), thoughts?

I was thinking it would be ideal to do the help message to the side of the table/phrase number indicator, but honestly haven't checked if it would fit well.
Another option would be to replace the notes on the bottom of the screen (but leave the map intact), thoughts?

Going from the leftmost border makes the most sense to me, frees up the most amount of space to write too :)

I think the top is good but bottom replacing the notes display makes alot of sense too as you'd never be using the notes display while entering FX commands, but really I'm happy with either.


[BUGFIX] defaulting partial ControllerConfiguration logging options

The current defaulting logic for the ControllerConfiguration logging options fails in case a partial logging configuration is provided.
(it only defaults the logging options in case no logging options are provided, and does not default any of the logging option fields in case a partial configuration is provided)
Introduced in https://github.com/cert-manager/cert-manager/pull/5337, this has not been released yet.
This is how you can test the logging options:
$ cat config.yaml
apiVersion: controller.config.cert-manager.io/v1alpha1
kind: ControllerConfiguration
logging:
  format: json
$ _bin/server/controller-linux-amd64 --config config.yaml
{"ts":1692259054053.9272,"caller":"clientcmd/client_config.go:618","msg":"Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.\n","v":0}
{"ts":1692259054053.972,"caller":"clientcmd/client_config.go:623","msg":"error creating inClusterConfig, falling back to default config: unable to load in-cluster configuration, KUBERNETES_SERVICE_HOST and KUBERNETES_SERVICE_PORT must be defined\n","v":0}
{"ts":1692259054054.059,"caller":"app/start.go:152","msg":"cert-manager: Failed to run the controller","err":"error creating rest config: invalid configuration: no configuration has been provided, try setting KUBERNETES_MASTER environment variable"}

Kind
/kind bug
Release Note

NONE

Is that same struct also embedded in the webhook configuration?

Not yet, I want to do that in this PR: https://github.com/cert-manager/cert-manager/pull/6243


Question: Significance of yuv420p

In many of the ffmpeg commands format=yuv420p is used. What does this mean? Is it the input resolution? If so, why create 720p output from 420p input?

Thats just ensuring that the color space is yuv4:2:0.  its not changing the resolution.

Yes that makes sure the video is 4.2.0 8 bit color.
On Fri, Dec 14, 2018, 3:47 PM e3z8<EMAIL_ADDRESS>wrote:

Thats just ensuring that the color space is yuv4:2:0. its not changing the
resolution.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/streamlinevideo/streamline/issues/18#issuecomment-447512067,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACdZaWQuVzIaNoQx3ccBVWFbmTl6tEiMks5u5Dh2gaJpZM4ZUPhd
.

Thanks @e3z8  and @colleenkhenry.


Textureview does not scale properly when resizing in pip mode

When using mSession with texture view options true, and the conatiner view is resized, content in textureview is not scaled. While in GLSurfaceView the problem does not occur.
mSession = new Session.Builder(requireContext(), OpenTokConfig.API_KEY, OpenTokConfig.SESSION_ID)
                        .sessionOptions(new Session.SessionOptions() {
                            @Override
                            public boolean useTextureViews() {
                                return true;
                            }
                        })
                        .build();

I see such an issue was reported as fixed in 2.14.1

When resizing a TextureView containing an OpenTok video, the resized view does not scale the whole video correctly and results in a cropped video. However, when resizing a GLSurfaceView containing an OpenTok video, the resized view produces a properly scaled video. This issue was introduced in v2.14.0.

I'm using 2.15.3 and this is still an issue for me. Please update us with an ETA on the fix

This has been fixed with 2.16.1 release.


More Information on Working with Airflow Scheduler

Several questions came up when we were exploring Airflow and I believe this could also be of interest to others and might be part of a later version of the documentation.

what is the proper way to terminate the scheduler
a) for all running dags
b) for specific running dags
what is the proper way to restart the scheduler
a) for all running dags
b) for specific running dags

There's a new toggle on the latest version of the UI that looks like this:

In older version there was a pause checkbox when clicking the magnifying glass by your DAG in the my page.
You can also kill the scheduler process SIGTERM / SIGINT , the state is stored in the metadata database so it's ok.
It could be nice to have some master toggle switch, but it's typically something that never happens here expect when we do major infrastructure migrations and then I'll just kill the scheduler manually.


Add no-code contribution - Skrift Article

Hello @andrewmckaskill ,
Thanks for your no-code contribution with this Skrift article, it was a great read 👍
I'll merge it right away 😁
Cheers!


Add React Native clients

This would make it a truly universal app and provide a good example on how to structure a universal application which includes android and ios.

+1

+1

+1

+1

+1

AFAIK, React Native components are substantially different than DOM components. While there could be a lot of code reuse, having a Native client is a totally different target, and will most likely have a decent amount of separate concerns.
"Universal" in this respect is meant to describe universal DOM rendering, not universal in the "write once, run everywhere" sense. I'm interested to see some approaches, but it may confuse the purpose of this repository.

I agree with @quicksnap. I had not noticed the misunderstanding of "universal". This app is "universal" in that the DOM is rendered on both the client and the server with the same code.
A react native client doesn't make any sense in this context, since it would share none of the code with the server.

those interested in "universal" redux w/ native and web clients should check out https://github.com/este/este


Keybindings for new and native menu

👋🏼,
I previously raised a question on how one would setup Tab and S-Tab keybindings for both the new cmp menu and the native menu. I just played around and this is working nicely for me:
local tab_complete = function(fallback)
    if vim.fn.pumvisible() == 1 then
        feedkey("<C-n>", "n")
    elseif cmp.visible() then
        cmp.select_next_item()
    elseif has_words_before() then
        cmp.complete()
    else
        fallback()
    end
end

local s_tab_complete = function(fallback)
    if vim.fn.pumvisible() == 1 then
        feedkey("<C-p>", "n")
    elseif cmp.visible() then
        cmp.select_prev_item()
    else
        fallback()
    end
end

Just wanted to share for searchability. Please close whenever you feel like it (if you think this looks sane).

It works for me.

https://github.com/hrsh7th/nvim-cmp/issues/231

@sQVe I tried this mapping and it worked fine.
  mapping = {
    -- other mappings
    ["<Tab>"] = cmp.mapping(function(fallback)
      if cmp.visible() then
        cmp.select_next_item()
      else
        fallback()
      end
    end, { "i" }),
    ["<S-Tab>"] = cmp.mapping(function(fallback)
      if cmp.visible() then
        cmp.select_prev_item()
      else
        fallback()
      end
    end, { "i" }),
  }

I don't think it's a problem with cmp alone, so I think you should try to create a minimal reproduce config.


Tables in extensions should take full width

I have developed Azure Data Studio Managed Instance extension that executes some queries and shows them in simple table.
When the queries are executed, the tables that are displayed are very narrow. As an example Logs and recommendations takes 30% of width and the rest of the window is empty. On the screenshots I have manually increased width to 100%.
The definition of logs widget is here
Tables should take full width and at least the last columns should fill entire space.

Thanks for your feedback @JocaPC! I'll re-tag this as an enhancement.


Adds job opportunities for nested

Please review our Contribution Guidelines to ensure your changes abides by it.
Review your changes and complete the checklist below.

[X] Company name
[X] Industry
[X] Company location
[X] Links
[X] Company description
[X] How Elixir is used

I'm not sure why the build failed?

I think we're underselling the extent to which we use Elixir - outside of data science and alongside JavaScript it's our primary language, and we essentially have no legacy in other languages to maintain.

Good point Phil, have updated see what you think.

@doomspork Thanks Phil is happy merge away!

Thanks @Adzz 😁


Missing error code: 0x80090016

Having big troubles with a AAD joined Windows 10 machine and setting a PIN code, throws error 0x80090016 tried unjoining and re-joining many times but doesn't work.

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

ID: 6b4f9133-887f-17a6-95fc-3712de8a3fa1
Version Independent ID: 09d34ce8-1590-8202-3e70-e6299dadbb65
Content: Windows Hello errors during PIN creation (Windows 10)
Content Source: windows/security/identity-protection/hello-for-business/hello-errors-during-pin-creation.md
Product: w10
Technology: windows
GitHub Login: @mapalko
Microsoft Alias: mapalko

@officedocsbot assign @jvsam

Hi @redforce123, apologies for the inconvenience. We would love to help you with your concern, however, this GitHub-based documentation feedback system is intended to help improve the quality of the technical documentation of Microsoft Docs. If you have feedback, suggestions related to this documentation, or ideas to make it better for users, please let us know and we will submit it for review.
What we recommend regarding your issue is to open a service/support ticket in your tenant. There are also other support resources that you can explore like the Microsoft Tech Community or the Technet forums where you can post your question and get help from the community.
Kindly keep us informed if you find any relevant information that can be incorporated into this documentation.
Thanks for being part of the Microsoft Docs community!

@azscloudpro : On one hand, the reason for the error codes missing on this page has often been explained as intermittent or unclear errors without a constant reason or solution. This error code hex value is also used by several other applications or software solutions, both on Windows Servers and Windows 10.
If you are able to produce a useful error message consistent, it could be helpful to report that error message here, in case it can be used to add the entry in the table.
In the meanwhile, I am fairly sure that a web search will be a faster way to find a solution, like this page:

https://www.thewindowsclub.com/error-code-0x80090016-for-pin-and-microsoft-store

What that page says about Windows Hello PIN error 0x80090016 and its cause, it seems to be the corruption of the contents of the NGC folder. The NGC folder is responsible for storing certain files which are crucial for the system’s security. The most probable reason behind the corruption of the folder is when a third-party software modifies the files. Other than this, third-party software products induce certain background processes which could cause this error.

Hi again @azscloudpro, we hope you were able to contact Windows 10 Support to resolve your concern, or checked the information provided by @illfated. We will now close this issue, however, feel free to re-open if you have suggestions or ideas to improve the quality of this documentation.
Thanks for being part of the Microsoft Docs community!

@officedocsbot close


Link "Using Saved Settings" directs to 404 - Page Not Found

The last link on this page, under section "See Also" (link "Using Saved Settings") is broken. It takes me to a 404 - Page not found error page.

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

ID: 3dce95ba-a175-0884-d851-173c7c13e897
Version Independent ID: 711c1256-ed89-152c-3fca-bf11363bdc00
Content: SaveValues Property - Business Central
Content Source: dev-itpro/developer/properties/devenv-savevalues-property.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Thanks for the feedback! The links have been fixed and the updated article will be published soon. In the meantime, here are a couple links: https://docs.microsoft.com/en-us/dynamics365/business-central/reports-saving-reusing-settings and Using default values - predefined settings


Uncaught Error: Model is disposed!

Error: Model is disposed!
at W.ib (out/vs/workbench/workbench.desktop.main.js:728:263)
at W.findMatches (out/vs/workbench/workbench.desktop.main.js:731:3497)
at p.D (out/vs/workbench/workbench.desktop.main.js:1374:14253)
at <anonymous> (out/vs/workbench/workbench.desktop.main.js:1374:13511)
at <anonymous> (out/vs/workbench/workbench.desktop.main.js:149:15982)

Go to Errors Site

Error: Model is disposed!
[at W.ib in src/vs/editor/common/model/textModel.ts:416:10](https://vscode.dev/github/microsoft/vscode/blob/35419fc016b8d3d5dbc59aa11a0be957b0897309/src/vs/editor/common/model/textModel.ts#L416:10)
[at W.findMatches in src/vs/editor/common/model/textModel.ts:1129:8](https://vscode.dev/github/microsoft/vscode/blob/35419fc016b8d3d5dbc59aa11a0be957b0897309/src/vs/editor/common/model/textModel.ts#L1129:8)
[at p.D in src/vs/workbench/contrib/extensions/browser/fileBasedRecommendations.ts:202:17](https://vscode.dev/github/microsoft/vscode/blob/35419fc016b8d3d5dbc59aa11a0be957b0897309/src/vs/workbench/contrib/extensions/browser/fileBasedRecommendations.ts#L202:17)
[at <anonymous> in src/vs/workbench/contrib/extensions/browser/fileBasedRecommendations.ts:131:32](https://vscode.dev/github/microsoft/vscode/blob/35419fc016b8d3d5dbc59aa11a0be957b0897309/src/vs/workbench/contrib/extensions/browser/fileBasedRecommendations.ts#L131:32)
[at handler in src/vs/base/common/async.ts:512:3](https://vscode.dev/github/microsoft/vscode/blob/35419fc016b8d3d5dbc59aa11a0be957b0897309/src/vs/base/common/async.ts#L512:3)

/duplicate

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


#118 - Add support for yield statement

Hello!

Type: new feature
Link to issue: #118

In raising this pull request, I confirm the following:

[x] I have read and understood the Contributing Guidelines
[x] I have checked that another pull request for this purpose does not exist.
[x] I wrote some tests for this PR.

Thanks

@Jeckerson can you add a test with lambda func, please.


Fix O->T and T->O size in fwd open with heartbeat assemblies

Hi, been working on implementing a CIP adapter for a customer using the OpENer stack.
I found that OpENer rejected O->T and T->O of heartbeat assemblies.  From the little I've read and understood of CIP, heartbeat assemblies have a zero-length data payload, yet come with a CIP sequence number, that never increments (because no changes to non-existing data).
The enclosed patch drops the code that resets the offset to 0 for heartbeats.  This works with OpENer, but may not work with other adapter implementations.

It's been over a year and not even a reply, so I'm closing this PR due to lack of interest and will at some point have to fork your project for my customers.


bin/gftools-update-families.py refactor

@eniallator said:
I want to change fontbakery-update-families so we can remove lib magic and also making the interface nicer. What I'm thinking of doing is using a before and after list of file names and comparing instead of assuming there's another file within the directory named the same but with "-old" at the end. Also just checking if me changing the interface will impact any scripts setup or not @felipesanches
Originally posted at https://github.com/googlefonts/fontbakery/issues/1514

To be tackled after #10 gets merged

I'm fairly sure this has rotted; update-families is no longer a thing.


support for Jasmine 2.4

Following http://jasmine.github.io/2.4/introduction.html and the guidelines for contributing.

Looks pretty good! I commented on a few of the anys that should probably be mixed, but pretty much any functions that take an arbitrary value should be mixed.
After that, I think this is good to go :)

thanks!

back atcha! fastest PR reply I've ever gotten on a public project.


Binaries on Linux should be relocatable within the install package

Expected Behavior

Test coverage showing that once an application is installed the original binaries can be deleted while keeping the installed application working.
Moreover, the binaries fetched from cache should work when installed in a different location.
Current Behavior

macOS handle relocation even though we have absolute paths in the binary. However, Linux is true to the absolute path and will fail if the original binaries can't be found.
Context

Steps to Reproduce (for bugs)

Your Environment

Build scan URL:

PR: https://github.com/gradle/gradle/pull/6176 contains what needs to be fixed.

@lacasseio I left comments on the PR


Fix moar tests

Codecov Report

Merging #167 (960a84d) into main (89f749f) will increase coverage by 0.18%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #167      +/-   ##
==========================================
+ Coverage   92.20%   92.38%   +0.18%
==========================================
  Files          32       32
  Lines        3425     3467      +42
==========================================
+ Hits         3158     3203      +45
+ Misses        267      264       -3

Impacted Files
Coverage Δ

crates/crochet/tests/integration_test.rs
98.39% <100.00%> (+0.09%)
:arrow_up:

crates/crochet_infer/src/lib.rs
98.92% <100.00%> (+0.01%)
:arrow_up:

crates/crochet_infer/src/unify.rs
96.39% <100.00%> (+0.03%)
:arrow_up:

crates/crochet_infer/src/types/type.rs
56.07% <0.00%> (+2.80%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us.


Windows: Slash/backslash problem with SASS

Seems the --load-path parameter isn't working when there are slashes and backslashes mixed. I've been able to fix this by exchanging
$root = $asset->getSourceRoot();
$path = $asset->getSourcePath();

by
$root = str_replace('\\', '/', $asset->getSourceRoot());
$path = str_replace('\\', '/', $asset->getSourcePath());

in SassFilter->filterLoad().
Tested with Symfony 2.0.0-RC4 under Windows 7 x64.

Confirmed with Symfony 2.0.6 / Compass 0.11.5 under Windows 7 x64.


gpioCfgSetInternals now returns PI_INITIALISED after gpioInitialise

This resolves #455 with the discussion with @guymcswain that
gpioCfgSetInternals() should return an error if gpioInitialise()
has been called previously to report that it's no longer possible,
for example, to disable signal handling. If successful it returns
0 as before. Tested it by calling gpioCfgSetInternals, gpioInitialise,
gpioCfgSetInternals and reports 0 and -32 which looks OK.

Fixed also doc that set_PWM_range() returns the actual range. This resolves #453.


crash if export x file

I can load fbx models with animation, but crash when exporting to x file (both xp and win7).

Could you upload the file that crashed somewhere? Thanks!

http://www.augmreal.com/downloads/test/pig.zip

same issue here, different fbx model

Same issue for me as well. Though I'm 3 years late to the party here...


fix(ui): ensure onboarding next button fully clickable

Description:
Ensure onboarding next button fully clickable
Motivation and Context:
This addresses a bug in which the Next button in several of the onboarding steps is not fully clickable (specifically, the left hand edge of the button). This happens because the form wizard content is being absolutely positioned over the top of the buttons.
This affects the steps that are wrapped in the Container element such as ConnectionType, BackupSetup and BackupSetupLocal
How Has This Been Tested?
Manually
Types of changes:
UI fix
Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[x] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage remained the same at 20.23% when pulling a36a2fc0aed4f5b0aa652b959c50821165ff7e33 on mrfelton:fix/onboarding-next-button into f283421b5fc62fabeca1e5c1c1ec7e4b09158f0f on LN-Zap:next.


📝 update schema loader documentation

Description
Update schema loader documentation with a example of module option based on comment from @JamieS1211 in #815.
Checklist

[ ] My changes follow the contributing guidelines of this project.
[ ] I have performed a self-review of my code.
[ ] I have made corresponding changes to the documentation.
[ ] I have added tests that prove my fix is effective or that my changes work.
[ ] New and existing unit tests pass locally with my changes.

Looks fantastic hopefully it will help people.
Just a double check that the token in the example is randomly generated placeholder and not one of yours that you should keep secure?


Start-commit <repo> <branch>should create the branch if it doesn't exist

My understanding of the reasoning for -b is that if a user mistypes the branch name then they'll create a new branch instead of creating a commit on an existing branch.
I don't think it makes sense to try to be typo-proof here. If a branch it accidentally created, its not like any pipelines are triggered or anything bad happens -- you can delete it and do the right thing when you realize your mistake.
The fact that start-commit is a different command whether you're creating the first commit on a branch or later commits is a bit weird. Another argument to remove -b is that put-file -c current creates a branch if it doesn't exist (and I think we want to keep that functionality). I can't think of a reasonable explanation why -c its ok, but for start-commit its not. It should probably work for both.

This is fixed in 1.4


Add ZHA Texas Instruments CC device support

Proposed change

Add initial Beta support for variation of Texas Instruments CC devices to use them in ZHA integration.
Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New integration (thank you!)
[x] New feature (which adds functionality to an existing integration)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests

Example entry for configuration.yaml:

# Example configuration.yaml
zha:
  radio_type: ti_cc
  usb_path: /dev/ttyACM0

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request:

Checklist

[x] The code change is tested and works locally.
[x] Local tests pass. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist
[x] The code has been formatted using Black (black --fast homeassistant tests)
[ ] Tests have been added to verify that the new code works.

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

[ ] Documentation added/updated for www.home-assistant.io

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

[x] The manifest file has all fields filled out correctly.
Updated and included derived files by running: python3 -m script.hassfest.
[x] New or updated dependencies have been added to requirements_all.txt.
Updated by running python3 -m script.gen_requirements_all.
[x] Untested files have been added to .coveragerc.

The integration reached or maintains the following Integration Quality Scale:

[ ] No score or internal
[ ] 🥈 Silver
[ ] 🥇 Gold
[ ] 🏆 Platinum

Lgtm.
Are you using it in production in your environment?

Can we merge this PR? :)

@springstan @Adminiuga The documentation should have been updated before this PR was merged. There is no docs PR linked to this PR, nor has the checkbox been checked.
@sanyatuning, could you please open up a PR on our documentation to ensure this new feature is documented? Thanks! 👍
https://github.com/home-assistant/home-assistant.io

Can we merge this PR? :)

That's why I was asking 😅

Doh, yeah, I missed it.
Can pro-bot add docs-missing label for new features, but no PR checkbox?

The problem is that new features do not have to require docs changes 🤷♂

Ok, since there was no response with a PR, I've opened up: https://github.com/home-assistant/home-assistant.io/pull/12051


Feature/nginx proxy

Creates an nginx proxy for marathon at port 18080
Creates an nginx proxy for mesos leaders at port 15050
Puts iptables rules on marathon:8080 and mesos:5050

Spinning up a cluster to test.

Tests I plan to run

[ ] firewall rules created
[ ] mesos port 5050 blocked
[ ] mesos port 15050 forces https
[ ] mesos port 15050 forces authentication
[ ] marathon port 8080 blocked
[ ] marathon port 18080 forces https
[ ] marathon port 18080 forces authentication
[ ] marathon proxy between nodes works
[ ] mesos master/follower communication works
[ ] mesos <-> marathon communication works

Looks really good so far. Main issue is that mesos-leader redirect in the web UI goes to the leader at port 5050. I think this is a problem within how mesos does redirects internally.
But overall this is huge a nice improvement over security that we should merge it.


Validate the response header and body for the root URL

michael-lazar/gemini-diagnostics
[ResponseFormat] Validate the response header and body for the root URL
Request URL
  'gemini://localhost:9999/\r\n'
Response header
  '30 gemini://localhost:9999/index\r\n'
Status should return a success code (20 SUCCESS)
  x Received status of '30'
There should be a single space between <STATUS> and <META>
  ✓ '0 g'
Mime type should be "text/gemini"
  x None
Header should end with "\r\n"
  ✓ '\r\n'
Body should be non-empty
  x ''
Body should use consistent line endings

Should be handled by handler callback


Unhandled exception. System.TypeInitializationException: The type initializer for 'Kusto.Cloud.Platform.Utils.NativeThread' threw an exception.

My Aspnet core web app uses "Microsoft.Azure.Kusto.Ingest" NuGet package for routine ADX data ingestion. As part of our dotnet 8 migration, I recently upgraded the NuGet package from version "11.2.2" to "12.0.0". While the updated NuGet version functions smoothly on the local windows environment, deployment to Azure ACA containers (docker image targets a Linux OS) results in failure. The exception seems to originate from the Kusto.Cloud.Platform.Utils.NativeThread class.
Unhandled exception. System.TypeInitializationException: The type initializer for 'Kusto.Cloud.Platform.Utils.NativeThread' threw an exception.
---> System.DllNotFoundException: Unable to load shared library 'kernel32.dll' or one of its dependencies. In order to help diagnose loading problems, consider using a tool like strace. If you're using glibc, consider setting the LD_DEBUG environment variable:
/usr/share/dotnet/shared/Microsoft.NETCore.App/8.0.0/kernel32.dll.so: cannot open shared object file: No such file or directory
/app/kernel32.dll.so: cannot open shared object file: No such file or directory
/usr/share/dotnet/shared/Microsoft.NETCore.App/8.0.0/libkernel32.dll.so: cannot open shared object file: No such file or directory
/app/libkernel32.dll.so: cannot open shared object file: No such file or directory
/usr/share/dotnet/shared/Microsoft.NETCore.App/8.0.0/kernel32.dll: cannot open shared object file: No such file or directory
/app/kernel32.dll: cannot open shared object file: No such file or directory
/usr/share/dotnet/shared/Microsoft.NETCore.App/8.0.0/libkernel32.dll: cannot open shared object file: No such file or directory
/app/libkernel32.dll: cannot open shared object file: No such file or directory
...
at Kusto.Cloud.Platform.Utils.NativeThread.GetCurrentThread()
at Kusto.Cloud.Platform.Utils.NativeThread..ctor()
--- End of inner exception stack trace ---
at Kusto.Cloud.Platform.Utils.NativeThread.GetThreadDescription()
at Kusto.Cloud.Platform.Utils.PeriodicNonoverlappingTimer.OnTimerCallback(Object context)
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)

Upon investigation, it appears that the "12.0.0" version introduced the Kusto.Cloud.Platform.Utils.NativeThread class, which engages in unmanaged operations with a dependency to load window specific "kernel32.dll" library. Any guidance on resolving this issue would be greatly appreciated.

The new NuGet 12.0.1 fixes this for me.

This issue was fixed in 12.0.1


Error with rollup?

I keep getting this error when I try to run,
Error: Error transforming C:\wamp64\www\50\src\main.js with 'babel' plugin: It looks like your Babel configuration specifies a module transformer. Please disable it. If you're using the "es2015" preset, consider using "es2015-rollup" instead. See https://github.com/rollup/rollup-plugin-babel#configuring-babel for more information
By the way I am glad that you started this,  I hope that we can get it working.

FYI,  I am getting the same error with your starter setup as well.  I will keep digging.

I'm guessing from your path that you're running on Windows?  I have to admit, I didn't test this on Windows :)  But I can verify that I get the same error when I do try running on my Windows machine, so I don't have to hit you with the "Works on my machine" line!  LOL
I'll take a look at what's going on.  I know I saw this error in Linux when I was initially getting things set up so hopefully it's nothing too major.

Looks like the rollup plugin I was using for ES2015 was no longer maintained and there had been changes since the newest version was released. So I removed the reference and manually configured babel (which was only a few extra lines). I was able to get this running on Windows, so let me know if it works for you!

I will try it out now

It is working!  Good job.

Oh wow,  I really like that once you build the routes,  you can use normal links to the pages.  That is way cool.

Thanks!  And I'm glad it's working for you.  I'm trying to model this after react-router, so I can't take all the credit :P


VACNet

If you have been using nskinz, do not use it anymore as it has been detected by vacnet for some reason... the account that ive been only using nskinz on just got banned by ow... (i averaged like 10 kills a game) i would use caution

Did you happen to use any version before 3e49191f56df89f771db534d9467eda246e7dd4d on the account?
Those are indeed detected by the hook check

does the new source fix the hookcheck stuff

Yes.
Please only submit issues present in mainline version next time, thanks.


[New Port Request]

Library name: zbar
Library description: qrcode/ barcode ocr same with z-xing
Source repository URL: https://github.com/ZBar/ZBar
Project homepage (if different from the source repository):
https://github.com/ZBar/ZBar
Anything else that is useful to know when adding (such as optional features the library may have that should be included):

Related to PR https://github.com/microsoft/vcpkg/pull/16113

If you wish to add this port in the future, you can reopen this issue, but we're closing it for now.


Vickel: Blacklist 1-866-884-3315

Vickel requests the blacklist of the number 1-866-884-3315. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by double-beep in Charcoal HQ


Auto-select LSBMode on Decode; Various fixes/improvements

All methods appear to operate as intended for en/decode regardless of LSBMode flag with and without metadata saved to the image. The calculation of optimal LSBMode no longer includes iteration. Return values from decode methods are now caught by conditionals, even the LSBMode error which should never occur.
Edit: Errors now support multi-line messages
Closes #5

      Instead of trying to read the metadata from the image when the method LSBDecode or LSBCDecode is used, add a "fileSize" flag. Like what you talked about in #5. My reasoning behind this is, the LSBDecode and LSBCDecode methods are suppose to be more manual. The autoDecode method is the method that is suppose to be more automatic. The LSBDecode and LSBCDecode are there in case if the metadata is corrupted. And if the metadata is corrupted then the try and except at line 56 is going to always return an exception.
      The way the fileSize argument should work is, if the fileSize argument is provided, have the program automatically find the optimal LSBMode using the file size provided. The fileSize argument should work like a replacement incase if the LSBMode argument isn't provided. The LSBMode and the fileSize argument cannot be used at the same time. So in conclusion, yes, I am asking you to delete what you have and implement this instead. Sorry.

Once you implement that, I will give the "Ok" and merge this with the main branch.

That wasn't changed with this, though. Since the fileSize arg was fully implemented, we had always been trying to read it from metadata when a decode method was called directly (#13). I feel it is better to attempt to do the same here rather than put more (possibly unnecessary or damaging) arguments on the user initially. File size is also a failsafe estimate, the user should not be encouraged to use it unless absolutely necessary, ie these checks should be in place even if the user provides a file size estimate flag, which makes this method more intuitive. I would be willing to add an override which bypasses these for incorrect metadata, but removing them entirely seems like a bad idea.

I don't understand what you mean by, "...the fileSize arg was fully implemented...". There is no fileSize argument in the program. And again, the whole point of the LSBDecode and the LSBCDecode methods are to replace the autoDecode method incase if it fails. The only way the autoDecode can fail is, if the metadata is corrupted (not including fails from any of the decode methods inside of the autoDecode function), so adding those extra lines for grabbing metadata for the LSBDecode and LSBCDecode methods are unnecessary in my opinion, and I don't think it should be included for the LSBDecode and LSBCDecode methods.

The file size argument for the decode functions, included in #13. Predating that, the argument was never used despite being present in LSBDecode. I recognize that the decode methods don't have a very practical use for images with metadata, given autoDecode, but that doesn't mean a user should not be able to execute them directly to the fullest extent if they choose. It would seem to be intentionally decreasing the versatility of the direct decode methods. The user enters the fileSize either way, as a flag for the program or as input when metadata reading fails. It's just like file formats. It would seem there is not reason not to use png, but it would be nice to support other formats for users who decide they are better suited for their purpose. Checking metadata has a negligible effect on performance and increases the methods' odds of success when called directly.

parser = argparse.ArgumentParser(description="A simple program that can be used to encode data into images using steganography.")

    # Adds arguments to parser.
    parser.add_argument("--method", "-m", type=str, choices=["append", "LSBEncode", "LSBDecode", "LSBCEncode", "LSBCDecode", "dataToChannel", "channelToData", "autoDecode"], help="The method that will be used to encode/decode the data.", required=True)

    parser.add_argument("--input", "-i", type=str, help="The path to the image you want to decode/encode.", required=True)
    parser.add_argument("--data", "-d", type=str, help="The path to the data you want to encode into the output image.")

    parser.add_argument("--LSBMode", "-l", type=int, choices=range(1, 9), help="Which LSB mode to encode/decode the image with.")
    parser.add_argument("--channel", "-c", type=int, help="The channel you would like to use to encode/decode the data")

    parser.add_argument("output", type=str, help="The path of where the encoded or decoded data will be saved to. If the \"autoDecode\" method is selected, provide the file path with no extension (the correct extension will be automatically put at the end of the file path).")

    args = parser.parse_args()

I don't see a fileSize argument.
I also don't see the need for the user to access the LSBMode and LSBCMode directly, if it's just going to do the same thing as autoDecode, but with more steps. Using the metadata to automatically select the fileSize defeats the whole purpose of it being a more direct way to access the function through the CLI.
My suggestion is add fileSize as an argument flag, and have it be used with the LSBDecode and LSBCDecode methods. Make the fileSize argument optional.

"The file size argument for the decode functions, included in #13. "
Pulled straight from #13 file changes, previously LSB had accepted an argument for this parameter, but it was never actually used:
`    elif args.method == "channelToData":
returnValue = ImageSteganography.channelToData(args.input, args.output, args.channel - 1)
->
elif args.method == "channelToData":
try: fileSize = Utils.readMetadata(args.input)[2]
except: fileSize = -1
    returnValue = ImageSteganography.channelToData(args.input, args.output, args.channel - 1, fileSize)`

Like I was saying, I'd be willing to add an override option for incorrect metadata or bypassing input for integration with other programs, but I'd be very hesitant to intentionally remove features that improve user experience because we don't see a need for it.  Just because it doesn't have apparent utility for us does not mean users will not attempt it and possibly run into issues of misuse. An override flag would have more utility overall and could accept a dict of all current and future variables in a similar position of being unsafe when used improperly. Of all the flags to have, file size is most likely to both be unknown and cause frustration. LSBMode is at least reasonably finite.
At least consider the option of override as a balance between experienced users requiring the absence of metadata reading but want direct access to the functions and newer users who may become easily frustrated and could use the extra guidance or safety net.

So, by having an "override flag", are you talking about an optional "--fileSize" argument flag where you can input the file size of the encoded data? If so, I think that is fine. I still want you to get rid of this though:
        print("Missing \"LSBMode\" flag, attempting to read metadata...")
        try:
            args.LSBMode = Utils.readMetadata(args.input)[1][0]
        except:
            try:
                fileSize = int(float(input("Unable to read LSBMode, enter estimated size of data in bytes, scientific notation using \"e\" is permitted: \n")))
                args.LSBMode = -1
                os.system("cls" if os.name == "nt" else "clear")
            except:
                raiseErrorAndExit("Either LSBMode or fileSize must be provided for decoding images without metadata.")

(Lines 55-64)
I don't want the LSBDecode and LSBCDecode methods accessing metadata, because the whole point of them is to have a way to bypass the metadata, and as a plan B incase if the metadata is corrupted.

I am referring to a --override flag that skips all metadata checks and accepts a dict. And as I said above, all decode methods already access metadata for fileSize and have since PR 13. --override would be closer to what you desire and supports any future options that put the user in a similar position

I want to flesh this out a little more, maybe pre-calculate LSBMode to force LSBDecode and LSBCDecode not to return all data in the image as a protection when --override is enabled.
This seem alright, though, @catjacks38? All metadata checks are removed when --override is enabled.

Just add an optional "--fileSize" argument flag., and get rid of the metadata checks for the LSBDecode and LSBCDecode methods. Adding an override flag is pointless. In my opinion it adds more steps for the user to do.

What about the ones that have been in since #13 in every decode method? They're caught by try-excepts, same as LSBMode, and --override improves user experience all around for those using LSBDecode and LSBCDecode with metadata. It's not a matter of performance either, because now override cuts out all the processing. This has more longevity and will decrease flag count as the program grows. This gives direct decode methods the opportunity to get even closer to the bare-bones functions, exactly what you wanted in the first place. It also cuts out metadata reading in channelToData, which is more than what you asked.

That's great, but I still want you to do the optional "--fileSize" argument flag thing instead. Also thanks for letting me know about some of the changes you made in #13; I would like you to also remove all of the metadata reading in the LSBDecode and LSBCDecode methods. I missed them when I was reviewing the code.

Provide some reasoning for your "want" without just repeating what you've said before. I have addressed all those points multiple times now and explained how override fulfills all of them to an equal extent, if not with more versatility and room for future design. I feel that I am providing all of the justification but getting very little reasoning from your end other than "because I said so," which is not a compelling argument and does not encourage collaboration, growth, or improvement. I also don't feel that you are truly reading or considering my point of view, seeing as you did not explore #13 until now which I mentioned very early on. I'm not unreasonable; I just do not currently see the pros of your method outweighing the cons. If you could sufficiently justify the advantages in terms of performance, user friendliness, versatility, future-proofing, complexity, or some other metric, I would be far more inclined to agree. I request review on these PRs not because I have to do so to merge (as I am a collaborator) but because I want the project to grow in quality through evidence-based discussion.
As soon as the override flag is added, even with an empty dict, all metadata reading is sufficiently removed. There is no concern of performance or outcome disruption in that regard.
In fact, any user which desires the version you describe could simply alias the program to include --override "{}", excepting the rare need for fileSize to be included.
The only con to this is from my view is a few extra characters which I don't see as being a real issue. "--override" is capable of including future variables in a similar position which would cut down on characters in the long run if anything, as a new flag would not have to be added every time. I have described my points about user friendliness and versatility in the past as well.

So the override argument flag would work like this, "--override "?

As I said before, it accepts a dict containing precise or unsafe variables to be assigned, currently fileSize is the only one of it's kind, having an indefinite number of values. I already pushed the PoC commit. You can see it yourself.

Override flag doesn't work when I do this, --overide {"fileSize" : 10}. Why not?

What do you mean by "doesn't work"?

Well you see, I kinda spelled override wrong, and didn't realize it until now.

Ok, so it still gives me an error. Says I have an "invalid loads value". This is what I'm doing, --override "{'fileSize' : '10'}". Doesn't the loads type for argparse mean I need to input a JSON string?

Aight figured out the issue.

Yes, valid json must use double quotes.

And you gotta use escape characters for the quotes inside of it.

Oh, I thought that I had mentioned that before.

I don't understand the point of the bypassNullLSB variable. Explain.

It bypasses the protection mechanism for LSB decoding methods. It's in the README Don't think I put it in for regular LSBDecode last night tho, might've just put it in for LSBC. Will check when I'm home from work

What protection mechanism? It is also never explained in the README the purpose of it for either LSB or LSBC methods.

In README.md since last commit:
"Presently, fileSize in bytes (float -> int) and bypassNullLSB (bool) are the only arguments accepted, fileSize being treated as an estimate excepting the use of bypassNullLSB or the presence an LSBMode flag."
I meant I only implemented this for LSBCDecode, not LSBDecode:
    elif mode == -2:
        mode = fileSize*8 // len(flattened) + 1

Otherwise, a null LSBMode with fileSize will output a file with the maximum fileSize given the LSBMode (as a protection mechanism), and a warning is displayed informing the user that this may not be the mode used for encoding. This is not just for human error, but also concerns about parsing scientific notation using float and the way Windows chooses to not use the correct units (B when iB should be used). Also since a lack of override specifically requests an "estimate."

"Presently, fileSize in bytes (float -> int) and bypassNullLSB (bool) are the only arguments accepted, fileSize being treated as an estimate excepting the use of bypassNullLSB or the presence an LSBMode flag." This never explains what it does. So, if you could add an explanation of the purpose of the bypassNullLSB variable to the README, that would be poggers.

I considered the "excepting the use of bypassNullLSB" to be sufficient in showing the estimate -> final shift, but I will add a more verbose explanation as you desire.

Aight, that should be enough explanation on the bypassNullLSB variable. Will finish reviewing code later.

The changes seem fine to me; merged with main branch.


Create docker image for import tool

We may need to do this to run the importer on the RSP
docker images (well, singularity images) are recommended for running jobs on PSC
looking into this for COIaC engagement.

What are the requirements for this? Would it be somehow different from just the following?
FROM python:3
RUN pip install hipscat-import


iotedge-cli and iotedge-daemon are preventing us from build offline

Describe the bug
When trying to build an offline build and making use of BB_NO_NETWORK the iot-edge recipes will fail. We have a local source mirror so it should fetch any of the SRC_URI from there but it dose not my guess is that the reason is that both the daemon and cli recipe depends on some other repos
    git://github.com/Azure/hyperlocal-windows;protocol=https;branch=main;name=hyperlocal-windows;destsuffix=hyperlocal-windows \
    git://github.com/Azure/mio-uds-windows.git;protocol=https;branch=main;name=mio-uds-windows;destsuffix=mio-uds-windows \
    git://github.com/Azure/tokio-uds-windows.git;protocol=https;branch=main;name=tokio-uds-windows;destsuffix=tokio-uds-windows \

and that AUTOREV is used.
SRCREV_FORMAT .= "_hyperlocal-windows"
SRCREV_hyperlocal-windows = "${AUTOREV}"
EXTRA_OECARGO_PATHS += "${WORKDIR}/hyperlocal-windows"
SRCREV_FORMAT .= "_mio-uds-windows"
SRCREV_mio-uds-windows = "${AUTOREV}"
EXTRA_OECARGO_PATHS += "${WORKDIR}/mio-uds-windows"
SRCREV_FORMAT .= "_tokio-uds-windows"
SRCREV_tokio-uds-windows = "${AUTOREV}"
EXTRA_OECARGO_PATHS += "${WORKDIR}/tokio-uds-windows"

resulting in that it will always try and fetch it from an external source. We get the following error
ERROR: iotedge-cli-1.0.10-r0 do_fetch: Network access disabled through BB_NO_NETWORK (or set indirectly due to use of BB_FETCH_PREMIRRORONLY) but access requested with command git -c core.fsyncobjectfiles=0 ls-remote https://github.com/Azure/hyperlocal-windows  (for url https://github.com/Azure/hyperlocal-windows)
ERROR: iotedge-cli-1.0.10-r0 do_fetch: Function failed: base_do_fetch

We can see in the DL_DIR that the packages that are causing the problems are always fetched instead of just setup as a link pointing to the SOURCE_MIRROR location
lrwxrwxrwx  1 ext ext     115 Mar  1 10:46 git2_anongit.freedesktop.org.pkg-config.tar.gz -> /cache/sources/git2_anongit.freedesktop.org.pkg-config.tar.gz
-rw-r--r--  1 ext ext   25361 Mar  1 10:56 git2_github.com.Azure.hyperlocal-windows.tar.gz
-rw-r--r--  1 ext ext       0 Mar  1 10:56 git2_github.com.Azure.hyperlocal-windows.tar.gz.done
-rw-r--r--  1 ext ext   99160 Mar  1 10:56 git2_github.com.Azure.mio-uds-windows.git.tar.gz
-rw-r--r--  1 ext ext       0 Mar  1 10:56 git2_github.com.Azure.mio-uds-windows.git.tar.gz.done
-rw-r--r--  1 ext ext   22635 Mar  1 10:56 git2_github.com.Azure.tokio-uds-windows.git.tar.gz
-rw-r--r--  1 ext ext       0 Mar  1 10:56 git2_github.com.Azure.tokio-uds-windows.git.tar.gz.done
lrwxrwxrwx  1 ext ext     114 Mar  1 10:46 git2_github.com.besser82.libxcrypt.git.tar.gz -> /cache/sources/git2_github.com.besser82.libxcrypt.git.tar.gz
lrwxrwxrwx  1 ext ext     123 Mar  1 10:46 git2_github.com.cryptodev-linux.cryptodev-linux.tar.gz -> /cache/sources/git2_github.com.cryptodev-linux.cryptodev-linux.tar.gz
lrwxrwxrwx  1 ext ext     105 Mar  1 10:46 git2_github.com.file.file.git.tar.gz -> /cache/sources/git2_github.com.file.file.git.tar.gz
lrwxrwxrwx  1 ext ext     111 Mar  1 10:46 git2_github.com.libgit2.libgit2.git.tar.gz -> /cache/sources/git2_github.com.libgit2.libgit2.git.tar.gz

To Reproduce
Steps to reproduce the behavior:

Follow the offline build
Make sure you have BB_NO_NETWORK set to 1
Make sure you have a local SOURCE_MIRROR_URL e.g SOURCE_MIRROR_URL ?= "file:///cache/sources"
Run bitbake -c fetch iotedge-cli
See error

Expected behavior
I would have expected that the fetch task would use the SOURCE_MIRROR and would have found the necessary sources in there.
Logs
If applicable, add logs to help explain your problem.
Build System (please complete the following information):

OS: Ubuntu
Version 20.04

Target System (please complete the following information):

OS: GNU/Linux
Version Na

Additional context
I suspect it has something todo with the usage of AUTOREV but I am not sure. But I do think it is bad practice to use AUTOREV unless you absolutely must. I don't think AUTOREV is needed in this case since we have a versioned recipe. Specifying a sha would be a better solution in my opinion. I have tried setting the SRCREV to a sha and based on my tests I think it works. I can push a change for this but I would like to know if I have missed something.

Hi @manszigher -
I just merged PR #79 that should address this.  Please try this out and let me know.
Thanks!

Hi thanks,
I will try it out but would that again just end up with taking the latest from main? I think it would make more sense to pick a sha. I can see when I am building that cargo is actually trying to get a specific sha from the project so not really sure why the recipe is not just specifying the sha we need from the start.


full disk because of many left .tmp directories

We face TSDB storage filling up because of .tmp directories that are left. We understand that these tmp directories result from failed compaction or retention, but even though the Prometheus pods are healthy the left tmp dirs are not rotated / removed at any time later.
What did you do?
cleaning up the tmp dirs manually - but that is not acceptable for a fleet of clusters
What did you expect to see?
eventually Prometheus removes / rotates these tmp directories, but they keep on lingering forever
What did you see instead? Under which circumstances?
The tmp directories are never cleaned up
Environment
kube-prometheus running quay.io/prometheus/prometheus:v2.22.1

System information:
Linux 4.14.198-152.320.amzn2.x86_64 x86_64

Prometheus version:
prometheus, version 2.22.1 (branch: HEAD, revision: 00f16d1ac3a4c94561e5133b821d8e4d9ef78ec2)
build user:       root@516b109b1732
build date:       20201105-14:02:25
go version:       go1.15.3
platform:         linux/amd64

Prometheus configuration file:

prometheus --log.level=debug --web.console.templates=/etc/prometheus/consoles --web.console.libraries=/etc/prometheus/console_libraries --storage.tsdb.retention.si
ze=90GB --config.file=/etc/prometheus/config_out/prometheus.env.yaml --storage.tsdb.path=/prometheus --storage.tsdb.retention.time=1y --web.enable-lifecycle --storage.tsdb.no-lockfile --web
.external-url=/prometheus --web.route-prefix=/prometheus --storage.tsdb.min-block-duration=2h --storage.tsdb.max-block-duration=2h

@universam1 thanks - fighting with that as well

Can you please tell us what those files are excactly? And provide us with the Prometheus logs? Thanks!

Can you please tell us what those files are excactly? And provide us with the Prometheus logs? Thanks!

tmp directories like /prometheus/01EQ0DZ14E04F7P51Q3NA1562G.tmp

Do you see the same under previous versions of Prometheus?

cc @bwplotka I think you could poke around the code :)

Can you please tell us what those files are excactly? And provide us with the Prometheus logs? Thanks!

tmp directories like /prometheus/01EQ0DZ14E04F7P51Q3NA1562G.tmp
Which logs would you like to see? No errors logged in this regard, just missing function we believe

logs around when that file was created

Do you see the same under previous versions of Prometheus?

Yes, we hoped the latest version would be different but it does not

logs around when that file was created

We saw OOMKilled (which is a different thing), but these .tmp dirs are left untouched. They are already way older than the retention

Looking at your last comment, we think that the issue is the oomkilled, in such cases it can happen that there are leftovers .tmp files. The actual issue is that Prometheus did not have enough memory.

Looking at your last comment, we think that the issue is the oomkilled, in such cases it can happen that there are leftovers .tmp files. The actual issue is that Prometheus did not have enough memory.

Right, and that is what I stated as a fact at the TO. However that is not the concern of this issue. It is the mere fact that Prometheus will never clean up such artifacts at any time later, and by mere probability every storage will eventually fill up.
Hence a function is missing that automatically purges or processes such artifacts

Mmmh which version created thos tmp files?
#7772 should have covered this.

Mmmh which version created those tmp files?
#7772 should have covered this.

Interesting! We were running 2.15.2 at that time

@bwplotka do you think we should delete pre-2.21 .tmp blocks as well? I'd say yes..

I'd like to pick this up. @roidelapluie

do you think we should delete pre-2.21 .tmp blocks as well? I'd say yes..

I would say yes too since deleting old .tmp dirs manually would be tedious if you have lots of Prometheus running.
@AdaephonBen sure go ahead!

I'd like to pick this up

I'd like to pick this up

Awesome thanks 😊

Awesome thanks 😊


Whitespace diffs require second formatting run to stabilize

I'm using:

SQLcl: Release <IP_ADDRESS> Production Build: <IP_ADDRESS>1.1718
sql_developer/trivadis_advanced_format.xml
sql_developer/trivadis_custom_format.arbori

Sometimes packages require a second run of the formatter before subsequent runs don't make any changes. I believe this is only for whitespace diffs, but it would be nice if they could be eliminated.
I've noticed this in a few packages, but there's one I can point two that's publically available:
https://github.com/OraOpenSource/oos-utils/blob/master/source/packages/oos_util_crypto.pkb
Granted, that's a fairly complex package, but I do see this in some of our internal ones that are not as complex.

This is a bug. It's not related to the complexity of the package. I extracted the following code from the function randombytes:
begin
   for i in 0 .. 255
   loop
        t_arcfour.j := mod( t_arcfour.j + t_arcfour.s(i) + t_s2(i), 256 );
        t_tmp := t_arcfour.s(i);
        t_arcfour.s(i) := t_arcfour.s( t_arcfour.j );
        t_arcfour.s( t_arcfour.j ) := t_tmp;
   end loop;
end;
/

The first formatting result is (intermediate, incomplete):
begin
   for i in 0..255 loop
      t_arcfour.j                      := mod(t_arcfour.j + t_arcfour.s(i) + t_s2(i), 256);
      t_tmp                            := t_arcfour.s(i);
      t_arcfour.s(i)                   := t_arcfour.s(t_arcfour.j);
      t_arcfour.s(t_arcfour.j)       := t_tmp;
   end loop;
end;
/

And the second formatting result is (OK, stabilized):
begin
   for i in 0..255 loop
      t_arcfour.j                      := mod(t_arcfour.j + t_arcfour.s(i) + t_s2(i), 256);
      t_tmp                            := t_arcfour.s(i);
      t_arcfour.s(i)                   := t_arcfour.s(t_arcfour.j);
      t_arcfour.s(t_arcfour.j)         := t_tmp;
   end loop;
end;
/

I believe it is a bug in SQLDev's callback function pairwiseAlignments (see this blog post for more information about this callback function). pairwiseAlignments calculations the whitespaces to be inserted between nodes in a top-down approach. This does not work in all constellations. However, we decided to use this callback function in the Arbori program. Therefore I consider it also a bug of our project. It should be solvable with an alternative implementation of pairwiseAlignments in JavaScript.
As you figured out yourself, the workaround is to format the code twice to get a stable result.

The result of pairwiseAlignements is stable when the left-hand node does not contain unwanted whitespaces.
For example the following would work in one go (no whitespaces in parenthesis of line 7):
begin
   for i in 0 .. 255
   loop
        t_arcfour.j := mod( t_arcfour.j + t_arcfour.s(i) + t_s2(i), 256 );
        t_tmp := t_arcfour.s(i);
        t_arcfour.s(i) := t_arcfour.s( t_arcfour.j );
        t_arcfour.s(t_arcfour.j) := t_tmp;
   end loop;
end;
/

Nice find! Sounds like a timing thing, no? Should pairwiseAlignements be called after unwanted whitespace is removed?
I filed a bug for the one that Jeff said I should. Do you think this is a bug in SQLDev or no?

Sounds like a timing thing, no? Should pairwiseAlignements be called after unwanted whitespace is removed?

You cannot solve this problem by moving the call of pairwiseAlignments to the end of the Arbori program, because the real serialization happens afterwards. And my comment regarding "unwanted whitespaces" was misleading. More precise is probably "whitespace mismatch" which considers missing "wanted whitespaces" as well as in the following example:
begin
   x(1+2+3) := 'a';
   x(3 + 4 + 5) := 'b';
   x( 3  +  4  +   5) := 'c';
end;
/

On line two there where missing whitespaces in the left node.
On line three the whitespaces on the left side match the final ones.
And on line four there are too many whitespaces.
As a result first formatting result looks like this:
begin
   x(1 + 2 + 3)         := 'a';
   x(3 + 4 + 5)     := 'b';
   x(3 + 4 + 5) := 'c';
end;
/

The second formatter call fixes that issue:
begin
   x(1 + 2 + 3)     := 'a';
   x(3 + 4 + 5)     := 'b';
   x(3 + 4 + 5)     := 'c';
end;
/

So the code in pairwiseAlignment must properly consider the complete logic applied in the serializer step that runs after the Arbori program when calculating the length of the predecessor nodes. Currently a simplified logic is applied which works in most cases but not in all cases for the first invocation of the formatter.

Do you think this is a bug in SQLDev or no?

Yes, it is a SQLDev bug. pairwiseAlignments is not behaving as it should. The example in this comment should make that clear.
However, in this case we could solve the problem ourselves in the Arbori program, since we can "override" pairwiseAlignments via JavaScript.

@dmcghan my previous comment was not accurate. It's a bug in the callback function paddedIdsInScope (the callback function pairwiseAlignments looks good).
Sorry for the confusion.

Got it. I'll file another bug. Thanks!

Or maybe not. I just tried to reproduce in SQLDev, but it worked as expected there - just one pass needed. I'll hold off on filing another bug in SQLDev for now. Let me know what you think...

I used the default formatting settings in SQLDev 20.4. The only thing I changed was the setting for alignments of the "Assignment Operator :=" (well, we want to align things right? and that's part of the standard SQLDev formatting features).

Then I formatted the following code:
begin
   x(1+2+3) := 'a';
   x(3 + 4 + 5) := 'b';
   x( 3  +  4  +   5) := 'c';
   x( 3  +  4  +   5   +  6 ) := 'd';
end;
/

The result was:
BEGIN
    x(1 + 2 + 3)             := 'a';
    x(3 + 4 + 5)         := 'b';
    x(3 + 4 + 5)   := 'c';
    x(3 + 4 + 5 + 6) := 'd';
END;
/

The subsequent formatter call produce this result:
BEGIN
    x(1 + 2 + 3)         := 'a';
    x(3 + 4 + 5)         := 'b';
    x(3 + 4 + 5)         := 'c';
    x(3 + 4 + 5 + 6)     := 'd';
END;
/

So I still consider it a SQLDev bug.

Got it, that's what I missed. I just filed the bug. Thanks again!

Thank you, Dan. Very much appreciated.

I expect that the SQLDev team will fix this bug. In the meantime we have to call the formatter twice for a stable result. I consider that acceptable and don't plan to re-write paddedIdsInScope in JavaScript. Hence I close this issue.


torch.concat missing

url :https://github.com/salesforce/DeepTime/blob/main/models/modules/regressors.py
code:
    def get_weights(self, X: Tensor, Y: Tensor, reg_coeff: float) -> Tensor:
        batch_size, n_samples, n_dim = X.shape
        ones = torch.ones(batch_size, n_samples, 1, device=X.device)
        X = torch.concat([X, ones], dim=-1)   # this

        if n_samples >= n_dim:
            # standard
            A = torch.bmm(X.mT, X)
            A.diagonal(dim1=-2, dim2=-1).add_(reg_coeff)
            B = torch.bmm(X.mT, Y)
            weights = torch.linalg.solve(A, B)
        else:
            # Woodbury
            A = torch.bmm(X, X.mT)
            A.diagonal(dim1=-2, dim2=-1).add_(reg_coeff)
            weights = torch.bmm(X.mT, torch.linalg.solve(A, Y))

        return weights[:, :-1], weights[:, -1:]

It's coming in the next release. In the meantime, use torch.cat (the former is an alias for the latter).


Removed repeated text, minor formatting correction

Removed repeated text and fixed bullet points

Docs Build status updates of commit 1b39f8c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2022-ps/remoteaccess/Install-RemoteAccess.md
:white_check_mark:Succeeded
View (WindowsServer2022-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

Hello @isjwuk Chris
Thanks for your contribution!
Actually, the markdown generator for this file (PlatyPS) is inserting two blocks of yaml per parameter set because different "required" property on each parameter set (vpnType is required if used if vpn parameter set)
The edit that is required is merging both blocks into one using for example
Parameter Sets: Vpn, MultiTenant


Build on RHEL

Enable build on RHEL if TARGET=rhel.

The build scripts will be executed twice on the same machine, once for
CentOS and one for RHEL
docker login is required to run docker build
The image target is different if TARGET=rhel
There is a separate Dockerfile for the RHEL build

@gastaldi @rhoads-zach: Can you please review and merge?

LGTM, merging.

merged: 642b72a2d293fd1d2366edcf4e5ce3d26bfb60f5


Issue 660: Refactor the UserController to use form validation

Related issue: #660
Methods modified in UserController to follow the controllers guide and to use form validation.
The proper tests cases have been modified and new cases have been added.

Thanks for these changes!


Add Belmont, California

To<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Defund the BPD
Message (Don’t forget to fill in your info)
To the Mayor Lieberman, Vice Mayor Stone, the Belmont City Council, Belmont City Manager, and Belmont Finance Manager:
My name is [YOUR NAME], and I am a resident of Belmont. I am writing to demand that the Belmont City Council adopts a People’s Budget that prioritizes community wellbeing, and redirects funding away from the police.
This past week our nation has faced protests calling for a rapid and meaningful reconsideration of the role of policing in communities, as well as an end to racism and anti-Blackness in America. The Bay Area, including our small town, has been at the forefront of much of this action. Accordingly, it has come to my attention that the budget for 2021 is being decided as these protests continue.
Belmont PD has been a waste of our resources. In 2019 the budget for Law Enforcement and Crime Control was $10,052,972. This means that in 2019 Belmont PD cost us taxpayers twice as much as we spent on Street Maintenance, Parks and Open Spaces, Recreation Programs, Senior Services, and Affordable Housing combined. The Belmont PD budget is excessive for a small town, with a very low crime rate. Also, despite spending an astonishing amount on police, the crime rate rises every year. In light of these facts, I am alarmed to see that Belmont’s projected budget for 2021 shows an increase in spending on police.
We can be a leader in this issue, if you and our other city officials have the courage to step up. I join the calls of those across the country to defund the police. I demand a budget that adequately and effectively meets the needs of at-risk Belmont residents during this trying and uncertain time, when livelihoods are on the line. I call on you to slash the Belmont PD budget and instead meaningfully reallocate funds towards social programs and resources that support affordable housing, education, health care, child care, and other critical community needs. We demand a budget that supports community wellbeing, rather than empowers the police forces that tear them apart.
As the City Council, the budget proposal is in your hands. It is your duty to represent your constituents. I am urging you to completely revise the budget for the 2020-2021 fiscal year.
We can be a beacon for other cities to follow, if only we have the courage to change.
Sincerely,
[YOUR NAME] [YOUR ADDRESS] [YOUR EMAIL] [YOUR PHONE NUMBER]

Minor copy-edits below, including replacing language of "crime" (see our style guide https://github.com/defund12/defund12.org/blob/gh-pages/EMAIL_TEMPLATE_STYLE_GUIDE.md )
Follow up with any concerns.
Ready for PR.
To<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Defund the BPD
Dear Mayor Lieberman, Vice Mayor Stone, the Belmont City Council, Belmont City Manager, and Belmont Finance Manager,
My name is [YOUR NAME], and I am a resident of [YOUR DISTRICT/NEIGHBORHOOD]. I am writing to demand that the Belmont City Council adopts a budget that prioritizes community wellbeing, and redirects funding away from the police.
This past week our nation has faced protests calling for a rapid and meaningful reconsideration of the role of policing in communities, as well as an end to racism and anti-Blackness in America. The Bay Area, including our small town, has been at the forefront of much of this action. Accordingly, it has come to my attention that the budget for 2021 is being decided as these protests continue.
Belmont PD has been a waste of our resources. In 2019 the budget for Law Enforcement and Crime Control was $10,052,972. This means that in 2019 Belmont PD cost us taxpayers twice as much as we spent on Street Maintenance, Parks and Open Spaces, Recreation Programs, Senior Services, and Affordable Housing combined. The Belmont PD budget is excessive for a small town, and I am alarmed to see that Belmont’s projected budget for 2021 shows an increase in spending on police despite historic unemployment, economic instability, and popular resistance to widespread police abuses.
We can be leaders on this issue, if you and our other city officials have the courage to step up. I join the calls of those across the country to defund the police. I demand a budget that adequately and effectively meets the needs of at-risk Belmont residents during this trying and uncertain time, when livelihoods are on the line. I call on you to meaningfully restrict the Belmont PD budget and instead meaningfully reallocate funds towards social programs and resources that support affordable housing, education, health care, child care, and other critical community needs. We demand a budget that supports community wellbeing, rather than empowering the police forces that tear us apart.
As the City Council, the budget proposal is in your hands. It is your duty to represent your constituents. I am urging you to completely revise the budget for the 2020-2021 fiscal year.
We can be a beacon for other cities to follow, if only we have the courage to change.
Sincerely,
[YOUR NAME] [YOUR ADDRESS] [YOUR EMAIL] [YOUR PHONE NUMBER]

On it!


Name?

Any suggestions?
I think everyone so far assumes "ElixirScript", but I'm not 100% sold on that. It may end up being the name though.
I like "ExScript" but a quick Google search showed there is already a project with that name. I also like "ElixirJS".

ElixirScript seems the best option. I'm a :-1: for ExScript.

I like ElixirJS too

ExScript has fallen out of favor with me too.
ElixirScript is in the lead followed by ElixirJS. It's already ElixirScript in the code. It probably is the best option, but wanted to get some feedback to see what people were thinking.

After thinking about it some more, I think ElixirScript is the best choice. People already call it that and it's that name in the code already. I'm going to change the repo name today to match.


run tests on SLE Base Container (using SLE BCI repository)

In addition to openSUSE BCI image, the testsuite should also be running on SLE Base Image, which is now setup with SLE_BCI repository. No SCC enablement is needed.

Frederic Crozat @.***> writes:

In addition to openSUSE BCI image, the testsuite should also be running on SLE Base Image, which is now setup with SLE_BCI repository. No SCC enablement is needed.

What do you want tested in these images? Which requirements should they
fulfill?

I think that's a great question, my suggestion is to test the zypper install lines from the github.com/rancher/rancher/package/Dockerfile* and Dockerfile.dapper  for now, to ensure that we don't break that?

we could also test that without registration and with registration we get the same base result, and maybe also that we get the "extra" with registration.

As a last comer to the party I agree with @dirkmueller
I'd like to see rancher/rancher being one of the platforms used to validate our SLE BCI images. Similarly to what we previously did with rancher/fleet.


no attribute 'load_kwargs' if you use JSONFormField

If I try to use JSONFormField as a field for one of my forms, I run into an
AttributeError
'JSONFormField' object has no attribute 'load_kwargs'

in fact load_kwargs is initialized in model's field JSONFieldBase, but not in the form's field.

My PR https://github.com/bradjasper/django-jsonfield/pull/129 should fix it

PR #129 should be merged in version 2.0.0 on PyPI. Please reopen this issue if there is still a problem. Thanks!


install as many dependencies as possible with conda, then pip: a small example please ?

https://climetlab.readthedocs.io/en/latest/installing.html

we recommend installing as many ❓ dependencies as possible with conda, then install CliMetLab with pip

Before I bore you with logs from conda install / pip, could anyone give an example of a small conda install
e.g. cfgrib eccodes
after which pip install climetlab works ?
https://xkcd.com/1987

You do not have to install any dependencies with conda. pip install climetlab should work out of the box.
Depending on the other packages that you have, depending on their versions, depending on your system, depending when/how you will update them, you may want to install some dependencies with conda to solve dependency issues. "Some dependencies" means here "some of the packages in the setup.py" https://github.com/ecmwf/climetlab/blob/82c38b9f23e53b2294f11b2da344bd37dd17b4aa/setup.py#L48

Hi @floriankrb,
did you try import eccodes after pip-only install ?
ecmwflibs/__init__.py:142: UserWarning: ecmwflibs universal: cannot find a library called eccodes
  warnings.warn(f"ecmwflibs universal: cannot find a library called {name}")

The problem is that pip install eccodes installs .py with no error, but not libeccodes.dylib;
for that, you have to either get it from conda, or build it from source
and setenv ECCODES_DIR /opt/miniconda3 or the like.
The Right Thing to do would be to build libeccodes.dylib under eccodes/.dylibs, like numpy; see
https://stackoverflow.com/questions/47380150/dealing-with-dylibs-when-distributing-python-packages
https://github.com/matthew-brett/delocate
What do you think ?

These are the installations supported directly by climetlab https://github.com/ecmwf/climetlab/actions/runs/2550905617
they are tested for each release under standard environments with github actions.
I do not know which platform you are using. Guessing from the fact that you got ecmwflibs universal, I am assuming this is not vanilla linux, nor macos, nor windows, or that you have special tweaks. In such cases, yes, you can install eccodes differently (conda, build from source, other), and make sure ecmwflibs finds it.


Use sphinx Note box rather than the text "Note:"

Fixes #23
This calls out the notes and looks much better.

:shipit:


Managed Script for "Restart default dns pods in openshift-dns namespace"

What type of PR is this?
This script will trigger a rollout of daemonset, controlling dns default pods. Such action might be required when a customer is not able to perform it themself due to not having enough permissions etc, but dns-default pods need to be restarted.
What this PR does / Why we need it?
We have a use case scenarios where a customer is required to restart dns default e.g. after applying a dns forwarding (if this script is considered as a valid then it can be applied to apiserver etc which will reduce the number of requests for SRE) pods and can't do it to multiple reasons (e. g. not having enough permissions, not being able to log in or not having enough expertise). Having this script will let MCS to perform this on behalf of the customer. This script will be run by the CRU member only.

/retest

I can see you submitted the PR from your local main branch which is not the best practice, better to avoid submitting from main but creating a feature or topic branch (e.g restart_dns_pods) for the changes and submit a PR from there.

Also, please test your managed script against an actual OSD/ROSA cluster in staging environment to make sure it worked properly.


add admin page to delee app, cluster and app namespace

Codecov Report

Merging #1278 into master will decrease coverage by 12.31%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master    #1278       +/-   ##
=============================================
- Coverage     48.83%   36.52%   -12.32%
+ Complexity     1725      134     -1591
=============================================
  Files           369       40      -329
  Lines         10451     1076     -9375
  Branches       1070      139      -931
=============================================
- Hits           5104      393     -4711
+ Misses         4969      627     -4342
+ Partials        378       56      -322

Impacted Files
Coverage Δ
Complexity Δ

...ework/apollo/spring/util/BeanRegistrationUtil.java

...rip/framework/apollo/enums/PropertyChangeType.java

...ework/apollo/configservice/util/NamespaceUtil.java

...lo/common/customize/TomcatContainerCustomizer.java

...rip/framework/apollo/internals/AbstractConfig.java

...p/framework/apollo/biz/auth/WebSecurityConfig.java

...service/service/config/ConfigServiceWithCache.java

...apollo/portal/listener/AppInfoChangedListener.java

...ollo/adminservice/AdminServiceHealthIndicator.java

...ework/apollo/biz/service/NamespaceLockService.java

... and 316 more

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 0c4069f...cff5d8c. Read the comment docs.


Using Lodash with MutatingArrayObserver doesn't trigger a binding update

I'm submitting a bug report

Library Version:
1.2.1

Please tell us about your environment:

Operating System:
Windows [10]

Node Version:
7.3.0

NPM Version:
3.10.10

Browser:
all

Language:
all

Current behavior:
Using lodash's _.pull(items, item) removes the item from the underlying array, but doesn't trigger a binding update.
Expected/desired behavior:
If I instead do a regular items.splice(items.indexOf(item),1) then the item is removed correctly, and the binding is updated.

What is the expected behavior?
http://plnkr.co/edit/YbA5vtLtsUXLP91GffeJ?p=preview

What is the motivation / use case for changing the behavior?
Lodash is a commonly used library and one would think that mutating operations on an array is a pretty common scenario.

If it's mutating them by using the index directly, we can't detect that change.

Ultimately it's just using splice, just like the working code...
https://github.com/lodash/lodash/blob/master/.internal/basePullAll.js

Aurelia modifies the splice method in order to detect changes. However, if lodash is loaded before aurelia it will save a reference to the native splice before aurelia can modify it: https://github.com/lodash/lodash/blob/65b148a3b2c8a86da92f51add871b04afddaba59/.internal/basePullAll.js#L7. So, an workaround would be to load lodash after aurelia. There is a change on lodash's master branch that removes the saved reference.