Fix release empty state

Fixes

https://issues.redhat.com/browse/RHTAPBUGS-460
Description

Updates ReleaseList Empty state
Type of change

[ ] Feature
[x] Bugfix
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Screen shots / Gifs for design review

How to test or reproduce?

Browser conformance:

[ ] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge

/approve


Video page

Coverage decreased (-0.2%) to 85.922% when pulling 611523ce713d567b620b195f8f0db3522321a8ae on video-page into 2814cb04ef9b75145e8c3fd2841d96c4205df9dc on master.

Coverage decreased (-0.2%) to 85.93% when pulling f2f1d664d7a8fb27dda0a451a715c7b7cbc5c01f on video-page into 2814cb04ef9b75145e8c3fd2841d96c4205df9dc on master.

Coverage decreased (-0.2%) to 85.93% when pulling f9ee44136106cff28a3e3f68f4035a5d03710480 on video-page into 469c596f3d3b42d9de6a83ff4a8a6e33a701a70c on master.

Coverage remained the same at 86.118% when pulling 21ecb6e1ca25afd7ffe8efcd94d9bd6e1a2974e1 on video-page into 469c596f3d3b42d9de6a83ff4a8a6e33a701a70c on master.

Coverage decreased (-0.2%) to 85.892% when pulling 42b16f2e72d6b1b7417e9241b670618339707970 on video-page into 469c596f3d3b42d9de6a83ff4a8a6e33a701a70c on master.

Coverage decreased (-0.2%) to 85.93% when pulling e5b3514fae2b7238f6b86a5e1bba6415aece953f on video-page into 469c596f3d3b42d9de6a83ff4a8a6e33a701a70c on master.

Coverage decreased (-0.2%) to 85.892% when pulling e5b3514fae2b7238f6b86a5e1bba6415aece953f on video-page into 469c596f3d3b42d9de6a83ff4a8a6e33a701a70c on master.

Coverage decreased (-0.1%) to 85.972% when pulling 139d580530ab8373417195e4400d01d270dd3efc on video-page into 469c596f3d3b42d9de6a83ff4a8a6e33a701a70c on master.

Coverage increased (+0.06%) to 86.178% when pulling f25a01eba448ab927877d828eb3c05cf01a5fe87 on video-page into 469c596f3d3b42d9de6a83ff4a8a6e33a701a70c on master.

Coverage decreased (-0.05%) to 86.063% when pulling 9383e96bb3a5c7eaad4ee70fc84e0efd5ef27ec8 on video-page into 469c596f3d3b42d9de6a83ff4a8a6e33a701a70c on master.

Coverage decreased (-0.2%) to 85.934% when pulling 89aa4c8be3befd0bcc636b331de9a3ffde465a23 on video-page into 469c596f3d3b42d9de6a83ff4a8a6e33a701a70c on master.

Coverage increased (+0.04%) to 86.157% when pulling 8d3d8a7f54b0713c665e451193e607487cc338c3 on video-page into 469c596f3d3b42d9de6a83ff4a8a6e33a701a70c on master.

LGTM.  We should make sure we've tested AVC video and series page and this won't cause any issues there.

Coverage decreased (-0.2%) to 85.934% when pulling 78dd192705d76840b10592263ac2bf35224d151e on video-page into 501c6a7486f1d0f92e3d8991a05ea574026c6f17 on master.


I need an HTML, css and JavaScript calculator code pls

You can write me on facebook or my twitter:
@soyReynald it's my twitter. I can help you with that code.

I can help with that

i already made a calculator as personal project with css , js , and html.
so i thing i can help.

@namanjain2001, @Maikaelj -  How can I contact you .??

I can help on frontend Part

https://www.makeuseof.com/build-a-simple-calculator-using-html-css-javascript/
You can find 1 here. Hope it can help. If you would like something more specific let us know.


MAINT: Attempts to resolve issues with pushes to test pypi.

What happens if we install this as editable? Does it drag along the version from the scm?

As before, can't tell if this worked without merging, so here I go.


Add aoe-elo.com

Something that's missing:
https://www.aoe-elo.com
AoE II player rating, based only on tournament games
Cheers

Add that url for you @simonsan . Thanks

Add that url for you @simonsan . Thanks


[Housekeeping] Add sample for CharactersValidationBehavior

Summary
It's just housekeeping work :) Need to implement a sample page for CharactersValidationBehavior following the example as done for other behaviors.
API Changes
None
Who Will Do The Work?

[ ] I am willing to take this on myself
[X] Just putting this out there for someone to pick up

I've tested the CharactersValidationBehavior and made a Sample page for this.
Did you want me to made a PR of it ? @AndreiMisiukevich

I've tested the CharactersValidationBehavior and made a Sample page for this.
Do you want me to make a PR of it ? @AndreiMisiukevich

Yes, please :)
Thanks for the help

Please keep in mind that you needn't have to localize your sample anymore. Just put text directly to the page


PWS should handle certain redirects differently

http://goo.gl/yRcWu0 points to http://www.randomwebsite.com/cgi-bin/random.pl, which redirects to a random website. The physical web server will cache the site it redirects to the first time, and then vend that same site over and over. What we really want is for the link to identify itself as a random-website redirector. Can we do that?
This is a somewhat silly example of a legitimate use case, where people might be redirected to substantially different content depending on various factors, and what you want to index is not the final destination, but the redirecting page's content.

The goo.gl link returns a 301 (Moved Permanently), while randomwebsite.com returns a 302.
However, goo.gl also retuns a cache-control headers of Cache-Control: no-cache, no-store, max-age=0, must-revalidate, so, perhaps we should not be storing either of them.

Redirects are no longer cached!
curl -k -s -H "Content-Type: application/json" -d '{"objects":[{"url":"http://goo.gl/yRcWu0"}]}' http://url-caster.appspot.com/resolve-scan


Package Fails to Precompile

Looks like some dependencies were changed, so QuantEcon and Games.jl stopped working for me (even after Pkg.rm() and re-installing the packages)
Error:
julia> using QuantEcon
[ Info: Precompiling QuantEcon [fcd29c91-0bd7-5a09-975d-7ac3f643a60c]
WARNING: both FFTW and Util export "Frequencies"; uses of it in module Periodograms must be qualified
ERROR: LoadError: LoadError: UndefVarError: Frequencies not defined
How do I get it to work again?

@frawas I cannot reproduce the error. Would you try update and precompile again?

Still nothing! I'm unfamiliar with this error or what is causing it, I have been using the package as well as Games.jl fine.
julia> Pkg.update("QuantEcon")
Updating registry at C:\Users\Farah\.julia\registries\General
Updating git-repo https://github.com/JuliaRegistries/General.git
Resolving package versions...
Installed TimerOutputs ─ v0.5.2
Installed HTTP ───────── v0.8.8
Updating C:\Users\Farah\.julia\environments\v1.1\Project.toml
[no changes]
Updating C:\Users\Farah\.julia\environments\v1.1\Manifest.toml
[cd3eb016] ↑ HTTP v0.8.7 ⇒ v0.8.8
[a759f4b9] ↑ TimerOutputs v0.5.1 ⇒ v0.5.2
julia> using QuantEcon
[ Info: Precompiling QuantEcon [fcd29c91-0bd7-5a09-975d-7ac3f643a60c]
WARNING: both FFTW and Util export "Frequencies"; uses of it in module Periodograms must be qualified
ERROR: LoadError: LoadError: UndefVarError: Frequencies not defined
Stacktrace:
[1] top-level scope at none:0
[2] include at .\boot.jl:326 [inlined]
[3] include_relative(::Module, ::String) at .\loading.jl:1038
[4] include at .\sysimg.jl:29 [inlined]
[5] include(::String) at C:\Users\Farah.julia\packages\DSP\wwKNu\src\DSP.jl:1
[6] top-level scope at none:0
[7] include at .\boot.jl:326 [inlined]
[8] include_relative(::Module, ::String) at .\loading.jl:1038
[9] include(::Module, ::String) at .\sysimg.jl:29
[10] top-level scope at none:2
[11] eval at .\boot.jl:328 [inlined]
[12] eval(::Expr) at .\client.jl:404
[13] top-level scope at .\none:3
in expression starting at C:\Users\Farah.julia\packages\DSP\wwKNu\src\periodograms.jl:186
in expression starting at C:\Users\Farah.julia\packages\DSP\wwKNu\src\DSP.jl:13
ERROR: LoadError: Failed to precompile DSP [717857b8-e6f2-59f4-9121-6e50c889abd2] to C:\Users\Farah.julia\compiled\v1.1\DSP\OtML7.ji.
Stacktrace:
[1] error(::String) at .\error.jl:33
[2] compilecache(::Base.PkgId, ::String) at .\loading.jl:1197
[3] _require(::Base.PkgId) at .\loading.jl:960
[4] require(::Base.PkgId) at .\loading.jl:858
[5] require(::Module, ::Symbol) at .\loading.jl:853
[6] include at .\boot.jl:326 [inlined]
[7] include_relative(::Module, ::String) at .\loading.jl:1038
[8] include(::Module, ::String) at .\sysimg.jl:29
[9] top-level scope at none:2
[10] eval at .\boot.jl:328 [inlined]
[11] eval(::Expr) at .\client.jl:404
[12] top-level scope at .\none:3
in expression starting at C:\Users\Farah.julia\packages\QuantEcon\vVeHL\src\QuantEcon.jl:12
ERROR: Failed to precompile QuantEcon [fcd29c91-0bd7-5a09-975d-7ac3f643a60c] to C:\Users\Farah.julia\compiled\v1.1\QuantEcon\V0Mv9.ji.
Stacktrace:
[1] compilecache(::Base.PkgId, ::String) at .\loading.jl:1197
[2] _require(::Base.PkgId) at .\loading.jl:960
[3] require(::Base.PkgId) at .\loading.jl:858
[4] require(::Module, ::Symbol) at .\loading.jl:853

I can confirm using it with  project environment works fine. This is likely an issue of having a polluted environment. The recommended way to use Julia is with project environments which avoid those issues.

For those that have the issue, I would suggest using project environments (see Working with environments).
Your global environment file should probably only include packages for cross projects, such as IDE related packages (e.g., Atom, Juno, IJulia), package development (e.g., Coverage, Documenter, DocumenterTools), and tooling (e.g., BenchmarkTools, Coverage, Revise, Weave).
(v1.3) pkg> st
    Status `~/.julia/environments/v1.3/Project.toml`
  [c52e3926] Atom v0.11.3
  [6e4b80f9] BenchmarkTools v0.4.3
  [a2441757] Coverage v0.9.3
  [e30172f5] Documenter v0.23.4
  [35a29f4d] DocumenterTools v0.1.2
  [7073ff75] IJulia v1.20.2
  [e5e0dc1b] Juno v0.7.2
  [295af30f] Revise v2.3.0
  [40c74d1a] TableView v0.4.1
  [44d3d7a6] Weave v0.9.1
  [0f1e0344] WebIO v0.8.11

Those would be available for every environment. Other packages should be added to the project environment.
(v1.3) pkg> activate .
Activating new environment at `Path/to/my/ProjectName/Project.toml`

(ProjectName) pkg> st
    Status `Path/to/my/ProjectName/Project.toml`
  (no changes since last commit)

(ProjectName) pkg> add QuantEcon
  Updating registry at `~/.julia/registries/General`
  Updating git-repo `https://github.com/JuliaRegistries/General.git`
 Resolving package versions...
  Updating `Path/to/my/ProjectName/Project.toml`
  [fcd29c91] + QuantEcon v0.16.2
  Updating `Path/to/my/ProjectName/Manifest.toml`
  [621f4979] + AbstractFFTs v0.5.0
  [ec485272] + ArnoldiMethod v0.0.4
  [7d9fca2a] + Arpack v0.3.1
  [4fba245c] + ArrayInterface v1.2.1
  [9e28174c] + BinDeps v0.8.10
  [b99e7846] + BinaryProvider v0.5.8
  [631607c0] + CMake v1.1.2
  [d5fb7624] + CMakeWrapper v0.2.3
  [49dc2e85] + Calculus v0.5.0
  [bbf7d656] + CommonSubexpressions v0.2.0
  [34da2185] + Compat v2.2.0
  [8f4d0f93] + Conda v1.3.0
  [717857b8] + DSP v0.6.1
  [9a962f9c] + DataAPI v1.1.0
  [864edb3b] + DataStructures v0.17.5
  [01453d9d] + DiffEqDiffTools v1.4.0
  [163ba53b] + DiffResults v0.0.4
  [b552c78f] + DiffRules v0.1.0
  [31c24e10] + Distributions v0.21.8
  [7a1cc6ca] + FFTW v1.1.0
  [1a297f60] + FillArrays v0.7.4
  [f6369f11] + ForwardDiff v0.10.6
  [d25df0c9] + Inflate v0.1.1
  [682c06a0] + JSON v0.21.0
  [093fc24a] + LightGraphs v1.3.0
  [d3d80556] + LineSearches v7.0.1
  [1914dd2f] + MacroTools v0.5.2
  [fdba3010] + MathProgBase v0.7.7
  [e1d29d7a] + Missings v0.4.3
  [d41bc354] + NLSolversBase v7.5.0
  [76087f3c] + NLopt v0.5.1
  [77ba4419] + NaNMath v0.3.2
  [429524aa] + Optim v0.19.4
  [bac558e1] + OrderedCollections v1.1.0
  [90014a1f] + PDMats v0.9.10
  [d96e819e] + Parameters v0.12.0
  [69de0a69] + Parsers v0.3.8
  [f27b6e38] + Polynomials v0.6.0
  [85a6dd25] + PositiveFactorizations v0.2.3
  [27ebfcd6] + Primes v0.4.0
  [1fd47b50] + QuadGK v2.1.1
  [fcd29c91] + QuantEcon v0.16.2
  [3cdcf5f2] + RecipesBase v0.7.0
  [189a3867] + Reexport v0.2.0
  [ae029012] + Requires v0.5.2
  [79098fc4] + Rmath v0.5.1
  [699a6c99] + SimpleTraits v0.9.1
  [a2af1166] + SortingAlgorithms v0.3.1
  [276daf66] + SpecialFunctions v0.8.0
  [90137ffa] + StaticArrays v0.12.1
  [2913bbd2] + StatsBase v0.32.0
  [4c63d2b9] + StatsFuns v0.9.0
  [30578b45] + URIParser v0.4.0
  [81def892] + VersionParsing v1.1.3
  [2a0f44e3] + Base64
  [ade2ca70] + Dates
  [8bb1440f] + DelimitedFiles
  [8ba89e20] + Distributed
  [b77e0a4c] + InteractiveUtils
  [76f85450] + LibGit2
  [8f399da3] + Libdl
  [37e2e46d] + LinearAlgebra
  [56ddb016] + Logging
  [d6f4376e] + Markdown
  [a63ad114] + Mmap
  [44cfe95a] + Pkg
  [de0858da] + Printf
  [3fa0cd96] + REPL
  [9a3f8284] + Random
  [ea8e919c] + SHA
  [9e88b42a] + Serialization
  [1a1011a3] + SharedArrays
  [6462fe0b] + Sockets
  [2f01184e] + SparseArrays
  [10745b16] + Statistics
  [4607b0f0] + SuiteSparse
  [8dfed614] + Test
  [cf7118a7] + UUIDs
  [4ec0a83e] + Unicode

(ProjectName) pkg> add https://github.com/QuantEcon/Games.jl
  Updating git-repo `https://github.com/QuantEcon/Games.jl`
 Resolving package versions...
  Updating `Path/to/my/ProjectName/Project.toml`
  [d5cce6e7] + Games v0.0.0 #master (https://github.com/QuantEcon/Games.jl)
  Updating `Path/to/my/ProjectName/Manifest.toml`
  [6e4b80f9] + BenchmarkTools v0.4.3
  [e2554f3b] + Clp v0.7.0
  [3da002f7] + ColorTypes v0.8.0
  [861a8166] + Combinatorics v0.7.0
  [53c48c17] + FixedPointNumbers v0.6.1
  [d5cce6e7] + Games v0.0.0 #master (https://github.com/QuantEcon/Games.jl)
  [4d00f742] + GeometryTypes v0.7.6
  [c8e1da08] + IterTools v1.2.0
  [4076af6c] + JuMP v0.20.1
  [b8f27783] + MathOptInterface v0.9.7
  [774612a8] + ParameterJuMP v0.1.2
  [67491407] + Polyhedra v0.5.6

julia> using QuantEcon, Games
[ Info: Precompiling QuantEcon [fcd29c91-0bd7-5a09-975d-7ac3f643a60c]
[ Info: Precompiling Games [d5cce6e7-4428-536b-a17f-94dfa0e3a07a]

This is probably fixed by JuliaRegistries/General#5778


[AIRFLOW-3398] Google Cloud Spanner instance database query operator

Jira

[x] My PR addresses the following Airflow Jira issues and references them in the PR title.

Description

[x] Here are some details about my PR, including screenshots of any UI changes:

Created an operator to enable executing arbitrary DML query (INSERT, UPDATE, DELETE) in a Transaction in Cloud Spanner:
CloudSpannerInstanceDatabaseQueryOperator
Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

test_gcp_spanner_operator.py
Commits

[x] My commits all reference Jira issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.
All the public functions and the classes in the PR contain docstrings that explain what it does

Code Quality

[x] Passes flake8

Codecov Report

Merging #4314 into master will increase coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4314      +/-   ##
==========================================
+ Coverage   78.02%   78.09%   +0.07%
==========================================
  Files         201      201
  Lines       16466    16466
==========================================
+ Hits        12847    12859      +12
+ Misses       3619     3607      -12

Impacted Files
Coverage Δ

airflow/configuration.py
89.24% <0%> (+4.3%)
:arrow_up:

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 457ad83...9c7ae48. Read the comment docs.


npm ERR! notarget No compatible version found<EMAIL_ADDRESS><2.0.0'

Error as above when attempting to install watchify, which uses deps-sort.
Latest appears to be 1.0.1: https://github.com/dominictarr/JSONStream/releases

https://github.com/substack/module-deps/issues/85


Update reference guide

Add new overview sections for SCDF, SCSt, and SCT
Add a note about empty application registry
Update README

Resolves spring-cloud/spring-cloud-dataflow-server-cloudfoundry#95

minor changes to wording after squashing commits - merged as b062d091f4aec1242f1580fa6d5dfac4fdd1dc45


fix: upgrade mobx and mobx-react dependencies to latest stable versions

Hi, thanks for the great project!
I'm working on an app with mobx 3.0 & mobx-react 4.1, so the current mobx-react-router release doesn't work because of dependencies to older versions.
This simple PR updates those dependencies to latest stable versions, all tests are still passing without any other changes and it seems to work in real use as well without any deprecation messages.

Ah cool, didn't realise new versions had been released. Thanks for this! I'm not actually using MobX too much at the moment so the help is appreciated 👍


Replace existing website with a Jekyll bootstrapped version

Provides a Docker based development environment that encapsulates all of the dependencies required for Jekyll/GitHub Pages to function. See README for details around getting the development environment working locally.
Resolves https://github.com/azavea/operations/issues/153

Testing
See the Getting Started section of the README. Ensure that you can navigate to http://<IP_ADDRESS>:4000/ locally and see the Minima themed Jekyll site.

Retargeted against develop so that nothing happens to the current site until we're ready. At the point, we'll merge all changes into master.


Handling Broken Document Links

In instances where an instance of a document (Doc A) is linked to some other document (Doc B), and Doc B is then deleted, Prismic doesn't kill the reference to Doc B in Doc A. The user must then edit Doc A and remove the broken link manually (Prisimc informs the user that the link is broken).
We'll have to do some extra checks to see that document links are valid. If they're not, it breaks on boot when the data gets passed to infer-graphql-type.js:221 in gatsby.

Running some tests with current data now - I'll report back soon 👍

I keep running into instances where gatsby hangs on checking pages in certain cases, and it's hard to debug what causes this. One way for sure I can make this happen is by adding the check for value.isBroken and returning undefined in normalizeLinkField (normalize.js:43).
if (!value.type || !value.id || value.isBroken) return undefined

Well, forget that - turns out there's some issue with a template in the project that I'm testing on. No more crashes dealing with the data, though. PR coming up.

Handled by 18250b207e6195d8695a9457ae35c98019b17d5f


Fix broken links in flux installation guide

Fix broken links for controlling flux and more sophisticated setups

btw, would ../using.md etc work?

@dholbach  @rade   Changed links to relative links.  ../using.md  does seem to work

Thanks for the fix! Approved.


nikon nd2 to tif converter

Thanks @tdincer.  Are you planning on using tifffile or scanreader to read in the output of this function?  If scanreader, did you happen to test if it loads correctly?

@kabilar When I used scanreader, it didn't read because the tif file lacks some header information. So, I'll use tifffile.
Here is the actual complaint: ScanImageVersionError('Could not find ScanImage version in the tiff header')

Thanks for testing, @tdincer.  Perhaps we should add an Outputs section in the docstring and make note that this file should be read in with tifffile and not scanreader, since we quite often use scanreader.

FISSA's last release is buggy. We need to git clone + install it. Let's not version lock until they release a new one.

Ready to merge!


[BUG]

Describe the bug
When installing it, when I try to run this command: ./run.ps1 , PowerShell shows me this error.
To Reproduce
Steps to reproduce the behavior:

run this ./run.ps1
See error

Expected behavior
A clear and concise description of what you expected to happen.
alpaca-electron should have run after running that command.
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: Windows 11
Version [e.g. 1.0.5]

Additional context
Add any other context about the problem here.

The error message you're receiving is indicating that the execution policy on your system is preventing you from running unsigned scripts.
You can change the execution policy to allow running unsigned scripts, but please be aware that this can have security implications, especially if you run scripts from untrusted sources.
To change the execution policy for the current session only, you can run: Set-ExecutionPolicy -ExecutionPolicy Bypass -Scope Process
But even when I do that it says "Unsupported platform: Windows_NT"...

Still giving me the same error.

Still giving me the same error.

This might be a case where it just can't find the ./run.ps1 file? :/ idk I am not involved. I can't get it to run either.
I reached out to Albert for more clarity on where these files are supposed to be placed? Are they to be placed in the root of the alapaca-electron folder? Hopefully he will reply here, otherwise I will let you know if he gets back to me.

Hello! Firstly, thank you for giving the .ps1 script or program a try on Windows. Unfortunately, it appears that the conversion from the shell script to PowerShell script using AI didn't go smoothly. As a result, I've made the decision to rework the script from scratch. This might take some time, but I want to ensure its functionality. While waiting, you might want to explore using WSL2 with the GUI feature, WSLg. I appreciate your patience, and I'll keep this issue open until the script can run seamlessly on both native Windows 10 and 11. Thank you!

Hi there,
I am delighted to share with you the current strides made towards resolving this issue. The advancements are delineated as follows:

Our endeavors extend to bolster Windows support, albeit excluding arm64 Windows due to resource constraints for development.
Subtle refinements have been woven into the central codebase (index.js) to foster greater compatibility with Windows NT, particularly pertaining to directory path resolution.
The script responsible for initiating Windows execution has been reset, primed for tailored rewriting to align with Windows specifics.
Streamlining the launch procedure has been a priority, ensuring that valuable time is not squandered on processes like excessive repackaging.
Advancements have been made in procuring acceleration resources such as CUDA, OpenCL, OpenBLAS, and Windows-specific binaries. These resources are curated from the official Falcon (ggllm.cpp) and LLaMa GitHub Release pages. It is pertinent to note that Windows compatibility is currently confined to the Falcon and LLaMa ggml models exclusively.

While I extend an invitation for you to explore the repository, navigate to the 'usr' directory, and initiate 'npm start,' I must underscore that, owing to ongoing development, the functionality might not be assured at this juncture. Your patience and understanding are deeply appreciated.


GDScript 2.0: Can't write multiline assert statements

Godot version
4.0.beta7.official
System information
Ubuntu
Issue description
Attempting to write an assert statement on multiple lines causes the following error:
Expected expression to assert
Steps to reproduce
Example:
# This is fine
get_node(
    ""
)

# and this is fine
assert(true, "")

# but his is not
assert(
    true,
    ""
)

Minimal reproduction project
N/A

Duplicate of https://github.com/godotengine/godot/issues/66535.


fix(helm): vc expiration date

Description
The default operation for the vc_expiration that did not work as intended. Instead, always the default was used.
Pre-review checks
Please ensure to do as many of the following checks as possible, before asking for committer review:

[x] DEPENDENCIES are up-to-date. Dash license tool. Committers can open IP issues for restricted libs.
[x] Copyright and license header are present on all affected files

@DominikPinsel please change the merge branch to develop.

@DominikPinsel let's merge this next. You know the drill.


Figure out what to do with deprecated APIs that only differ by completion: vs completionHandler:

Swift treats the calls as ambiguous when using the "normal" elided-closure syntax.

OK, so given these decls:
typedef void (^TestCallback)(NSError * _Nullable error);

- (void)testMethod:(BOOL)arg completion:(TestCallback)completion;
- (void)testMethod:(BOOL)arg completionHandler:(TestCallback)completionHandler /*NS_SWIFT_UNAVAILABLE("")*/ /*NS_SWIFT_DISABLE_ASYNC*/;

and this Swift code:
    testObj.testMethod(false) { error in }
    testObj.testMethod(false, completion: { error in })
    testObj.testMethod(false, completionHandler: { error in })
    do {
        try await testObj.testMethod(false)
    } catch {
    }

calls 1 and 4 error out with "Ambiguous use of 'testMethod'" and items 2 and 3 are fine (but warn about "Consider using asynchronous alternative function").  This is where we are right now.
If I uncomment the NS_SWIFT_DISABLE_ASYNC, then calls 3 and 4 are completely fine, call 2 is fine but has that "alternative function" warning, and call 1 errors out.
If I instead uncomment the NS_SWIFT_UNAVAILABLE, then call 3 errors out, calls 1 and 2 are fine but have the warning about "alternative function", and call 4 is fine.
There does not seem to be a way to make all four calls work, unfortunately.  Given how Swift is usually written, we're going to take the "make call 3 not work and everything else work" option, so add NS_SWIFT_UNAVAILABLE in all the cases where we have two selectors that differ only in completion vs completionHandler.

Similarly, given these decls:
- (void)testMethod2WithCompletion:(TestCallback)completion;
- (void)testMethod2WithCompletionHandler:(TestCallback)completionHandler /*NS_SWIFT_UNAVAILABLE("")*/ /*NS_SWIFT_DISABLE_ASYNC*/;

and this Swift code:
    testObj.testMethod2() { error in }
    testObj.testMethod2(completion: { error in })
    testObj.testMethod2(completionHandler: { error in })
    do {
        try await testObj.testMethod2()
    } catch {
    }

the same behavior is observed.


Add metadata to ClusterWeight

This may be a little more controversial than #81, but the idea is similar. We'd like to be to filter a weighted cluster by endpoint label, and as nearly as I can tell, this needs to happen in the ClusterWeight.
We could probably get by baking this in the Route metadata, but it's really specific to the weighted cluster, so it would be a little klunky.

Can you describe a specific use case? I'm a little confused by this one.

Can you describe a specific use case? I'm a little confused by this one.

Certainly, and it may be that I just misunderstand how this is all wired up.
Imagine a single service, with multiple instances, each labeled with version=<something> (this part is handled in #81)
We'd like to initially route 100% of traffic to version=v1 and then incrementally route traffic to version=v2.
So your ForwardAction looks something like:
ForwardAction:
  weighted_clusters:
    - name: my_service
      weight: 35
      metadata:
        version: v1
    - name: my_service
      weight: 65
      metadata:
        version: v2

Obviously you can also model this with a new service for each version, but that explodes pretty quickly if you want to route on several dimensions (eg stage, version, partition, etc).
We can also probably figure out a way to encode this into the Route metadata, but I think this is the right place to inject this.

Whether we make N clusters, or internally inside the load balancer deal with N subsets, there really isn't that much of a difference from Envoy perspective in terms of resource requirements. I would argue that having N clusters is much simpler to understand for this use case.
I would like to hear from Istio people on this because they are already dealing with this and do the same thing. @louiscryan @rshriram @kyessenov

Note the general assumption regarding Envoy API at this point is that in complex deployments, config is machine generated. I'm not saying that means we shouldn't potentially move subsetting capabilities into the load balancer, but just pointing that out.

Yeah we can definitely generate denormalized clusters to support this use-case, if that's the consensus. It really depends on how you want to generalize subsetting.

The rule you described above is exactly what we have in Istio ( @9len ). We statically generate clusters. And as @mattklein123 says, there really isn't much of a overhead, whether its 100 or 2000 clusters (I have tested this). If anything, you get detailed stats. The config is loaded via *DS, so changes get propagated quickly as well.
I don't see much use for pushing management plane stuff down into the data plane. This is just going to complicate the code a lot (computing clusters, pre-generating subsets and so on). It makes it even more harder to debug things when traffic is not arriving at your instance. Atleast with a config that you generate at the management plane, you can subject it to sanity checks, and have the ability to grok over the config.

fwiw, you don't necessarily have to use Istio if you are not running on K8S. You could easily generate denormalized clusters, as you stated.

I agree that it potentially complicates the load balancer, and that we can statically produce the necessary rules; my main concern is there being multiple implementors of the *DS APIs who potentially have to implement the same behavior to support what seems to me a common use-case.
Certainly we can live without this if the general consensus is that it overly/unnecessarily complicates the data plane.

One other potential advantage to having this within Envoy is that you can produce a sufficiently detailed log-line to explain why a particular instance was selected. You can do this as a log consumer with access to richer data, but there are race conditions (eg cluster or route is removed before log line is processed)

In thinking about it more, let's just go ahead and add metadata here too (I assume it already exists at the primary route level, if not we need it there). It's a pretty straightforward and self contained problem to add a new load balancer implementation that can do internal subsetting based on label. I don't know that I would make this impl the default, but if people want to use it, it can be available. This seems like a good compromise.

On Wed, Jun 21, 2017 at 12:43 AM Matt Klein<EMAIL_ADDRESS>wrote:

In thinking about it more, let's just go ahead and add metadata here too
(I assume it already exists at the primary route level, if not we need it
there). It's a pretty straightforward and self contained problem to add a
new load balancer implementation that can do internal subsetting based on
label.

You would need dynamic (at the time of forwarding) sub setting wouldn't you?
What happens if I split up a subset (of labels) into smaller sets and/or
merge them with other subsets? I could do this via RDS update. Correct me
if I am wrong, but this feels like you  would also need additional code to
handle things like retries to non existent subsets in the lb pool.
I don't know that I would make this impl the default, but if people want to

use it, it can be available. This seems like a good compromise.

A more flexible option might be to introduce filters in the cluster manager
or in the LB phase atleast. Metadata, headers, etc could go in and the
filter could output a host.
This would allow folks to maintain their own implementations and data
structures without complicating the rest of Envoy code base. I could use
the same framework for my shard router usecase.
—

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lyft/envoy-api/issues/91#issuecomment-309961498, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AH0qdxKZemHMvWidByI1csGMXTxRD-P1ks5sGJ9wgaJpZM4OAIO0
.

--
~shriram

You would need dynamic (at the time of forwarding) sub setting wouldn't you? What happens if I split up a subset (of labels) into smaller sets and/or
merge them with other subsets? I could do this via RDS update. Correct me
if I am wrong, but this feels like you  would also need additional code to
handle things like retries to non existent subsets in the lb pool.

No, it would happen in an eventually consistent way as all LB updates do today. It's very straightforward to write a LB that looks at host metadata and sets up subsets in some way that makes sense (and for example if there are too many subsets falls back to a slow path in some cases). There are no changes required to RDS, etc. See below.

A more flexible option might be to introduce filters in the cluster manager
or in the LB phase atleast. Metadata, headers, etc could go in and the
filter could output a host.

This is already the way it works. Each LB is an interface that already has the capability to take hint information. The router would pass metadata as the hint, the load balancer is free to do with it what it wants.
Anyhow, I would prefer to not totally sidetrack this issue, but I do think that if people are willing to do the work we can support both manager and envoy side subsetting with little risk to either path due to the architecture. Adding metadata here keeps both options option.


Urlbar broken in Fx74 beta

Sanity checks (must complete)

[x] I have read and followed the installation instructions in the README
[x] I have not modified the userChrome.css file
[x] I have tested the latest release or commit on master

To Reproduce
Firefox 74
Screenshots

System info

OS: Windows 10
Firefox version: 74

To clarify, this is the focused state of the URL bar.
Reproducible by cycling through the UI elements with the TAB key.

Until this is fixed you can disable the new urlbar by setting browser.urlbar.update1 to false in about:config.

Should be fixed now; you'll need to switch to 74 branch.

Should be fixed now; you'll need to switch to 74 branch.

Thanks !
While the url bar in focused state seems to be fixed, there's an other bug with urlbar results/predictions. While typing if results/predictions are large enough, the width of the bar will change.

I'm unable to reproduce that issue. Could you provide a screenshot?

URLBar without results/predictions : https://prnt.sc/r2d3ch
With prediction : https://prnt.sc/r2d3o0
As you can see the width changed. And it does that while typping

This is the exact urlbar glich I have with firefox nightly and we discussed in #201.
I can confirm the 74 branch fix it.
I also have the urlbar width changes. It seems to be related to the suggestions text length: the bar width is relative to the suggested text width. As shown on @Write screenshots, when I have long url in the suggestions, the urlbar is wider, but when the suggestions become smaller, the urlbar follows and become smaller.

I'm unable to reproduce the width change on macOS. Are you using Windows 10 and do you have any spacers in your nav bar? Looks like there might be spacers from the screenshot but I've tested with spacers and still don't get the problem on my end.

I'm unable to reproduce the width change on macOS. Are you using Windows 10 and do you have any spacers in your nav bar? Looks like there might be spacers from the screenshot but I've tested with spacers and still don't get the problem on my end.

Yes i'm on W10
Not sure what spacers is, is it the 'flex spaces' you can add to add blank space ?
Anyway i tried to re-enable browser.urlbar.update1 to test, it again.
I removed the spacers and it seems working correctly (the width isn't changing).
It seems to only bug when spacers are added to the left of the search bar.

I'm unable to reproduce the width change on macOS. Are you using Windows 10 and do you have any spacers in your nav bar? Looks like there might be spacers from the screenshot but I've tested with spacers and still don't get the problem on my end.

Linux on my side. I updated to latest nightly 76.0a1 (2020-04-03) and switched back to master. The problem persists...
Removing "flexible space" from the Customize tab fix the bug tho.

Let's move tracking of this flexible width bug to #223, as the original bug has been resolved.


Bug in Shop Page

In Shop page there no need to show tv shows two times and home btn is not working.
If some one already working on that then ok, otherwise assign to me.
https://github.com/MAVRICK-1/e-commerce_website/assets/120244528/6a5ec081-c47b-496a-893d-9398d8a09abd

@GOURAVSINGH19 okay go ahead

I have a question for If we show groceries , fashion and electronics Individually  in header then why we show made  shop?
@MAVRICK-1

@GOURAVSINGH19 i didn't understood your question , could you plz elaborate

Like in this screenshot we have groceries, electronic and fashion nav tags then why we show it again when hover on shop..maybe now you understand what I want to say..

@MAVRICK-1

Hey I will be happy to work on it !! Please assign this issue to me


The mstsc client exits because of the network disconnection, but the xrdp process does not exit

Hello xrdp team, the mstsc client exited due to network reasons, but the corresponding process of xrdp did not exit, so there will be more and more xrdp processes, so I would like to ask if there is any way to solve it.

Hi @lee415835450
Two solutions for you:-

Make sure the session is sending small amounts of data on a regular basis (e.g. a clock in the status bar). This will result in missing peers being detected fairly quickly.
Set the tcp_keepalive option in /etc/xrdp/xrdp.ini to set a TCP-level keepalive.

 You can tune this at the OS level. See tcp(7) for more details on this.

Hi @lee415835450
Two solutions for you:-

Make sure the session is sending small amounts of data on a regular basis (e.g. a clock in the status bar). This will result in missing peers being detected fairly quickly.
Set the tcp_keepalive option in /etc/xrdp/xrdp.ini to set a TCP-level keepalive.

 You can tune this at the OS level. See tcp(7) for more details on this.

Thank you very much for your reply, I will seriously consider your reply, and then try to solve this problem.


Please move the component in the first column instead of the last

Fixed in https://github.com/mozilla/relman-auto-nag/commit/46816d6e477e7c45544dc0ea28d1a9d616315cfb.


Hello Everyone,

Description of problem

What did you do?
What happened?
What did you expect to happen?
How can someone reproduce the problem?

Command used and debugging output

Platform and version information

Your OS:
Your Ruby version: <!-- type ruby --version at the command prompt -->
Your version of Puppet:
Your version of octocatalog-diff: <!-- type octocatalog-diff --version -->

Do the tests pass from a clean checkout?

Anything else to add that you think will be helpful?

Done


Feature: SyncListStruct

Issue Type

[ ] Bug
[x] Enhancement / Feature
[ ] Question
[ ] Other

Description
HLAPI's SyncListStruct can be used to synchronize lists of structs from server to clients. This would be a great addition for MLAPI.

Agreed. The plan is to change the current SyncedVar system on how it handles arrays and lists. Arrays ara going to be fixed size and only send the diff. Lists are going to have a history since last dirty similar to SyncList in HLAPI, I will be sure to add struct and class support to that. It's just a matter of using the BinarySerializer to grab the fields by Reflection.
Currently, we support arrays only and we send the whole array every time (I know, it's bad)

Moving this feature forward, it requires a bit of rework on the SyncVar system. Implementing this into the current system would mean (a lot) of boxing. Should be easy enough to work around.

https://github.com/TwoTenPvP/MLAPI/commit/45aeb253532384179c95230045dfd38fd3dd555e for now, any type can be sent as SyncedVar. As long as the type is a struct, class with primitive types / arrays or the type itself is a array. SyncListStruct is still not implemented as it will come with the optimization of the array system.

This is what I have managed to do so far:

All custom structs, arrays of them and arrays of primitives properly sync with the following limitations:
Arrays gets fully resynced, no diff checking. Ex, changing one value in a array with 100 values will resend all 100 values. Custom structs have no limitations, they are flattened by the binary serializer and treated as if each field was it's own syncedvar (but they still share dirty status etc).
The only thing that remains is optimized sending of Arrays, I just need to have a copy of the old array which is what I am currently working on.

Looks really promising

https://github.com/TwoTenPvP/MLAPI/commit/79b1a31f49a6e68d1d95582af2f55b13f0175dd4
Diff sending is now done. Try to not resize the arrays. The system works on the assumptions that your arrays are fixed size. If it changes size you resend the whole array and some nasty allocations etc happens in the MLAPI. You are better off to write like a SyncedVarWrapper if for lists if you want that functionality.
By not resizing you might waste some bandwidth to new players by telling them about unused data but you save a lot by not having to resend it to everyone.
Closing this, let me know if there are additionals or if something is not working. Only tested it briefly.


Mac OS - after closing a report appears

Bug description
Case 1:
When the user sends all the coins available on the balance using the "add all" button, a report appears after closing the wallet.
Report content
30_09(2).zip
Case 2:
When try is a connection to the btc\ltc or qtum node and at this time change the network(connect to different providers), my wallet hangs(the list of transactions is not displayed). after closing the wallet, a report appears.
Report content
Report.zip

checked


Issues launching debug daemonset in OCP 4.12 related to security constraints

Today, we have launched the first OCP 4.12 daily job and we have tested some workloads on top of it, including the workloads we use ot test CNF Cert Suite. You can see that job here.
Digging into the tnf results, we observed that there were a lot of unexpected results (some tests failed or were skipped and they should have passed), all of them caused by debug daemonset wrong behavior.
Taking a look into the events that happened in the cluster (it can be seen here, in events.txt file), we can see these issues related to the debug daemonset:
default                                            32m         Warning   FailedCreate                                      daemonset/debug                                                       Error creating: pods "debug-rtcj7" is forbidden: violates PodSecurity "restricted:latest": host namespaces (hostNetwork=true, hostPID=true), privileged (container "container-00" must not set securityContext.privileged=true), allowPrivilegeEscalation != false (container "container-00" must set securityContext.allowPrivilegeEscalation=false), unrestricted capabilities (container "container-00" must set securityContext.capabilities.drop=["ALL"]), restricted volume types (volume "host" uses restricted volume type "hostPath"), runAsNonRoot != true (pod or container "container-00" must set securityContext.runAsNonRoot=true), runAsUser=0 (container "container-00" must not set runAsUser=0), seccompProfile (pod or container "container-00" must set securityContext.seccompProfile.type to "RuntimeDefault" or "Localhost")
openshift-kube-controller-manager                  32m         Normal    CreatedSCCRanges                                  pod/kube-controller-manager-master-2                                  created SCC ranges for tnf namespace
default                                            32m         Warning   FailedCreate                                      daemonset/debug                                                       Error creating: pods "debug-8l6pg" is forbidden: violates PodSecurity "restricted:latest": host namespaces (hostNetwork=true, hostPID=true), privileged (container "container-00" must not set securityContext.privileged=true), allowPrivilegeEscalation != false (container "container-00" must set securityContext.allowPrivilegeEscalation=false), unrestricted capabilities (container "container-00" must set securityContext.capabilities.drop=["ALL"]), restricted volume types (volume "host" uses restricted volume type "hostPath"), runAsNonRoot != true (pod or container "container-00" must set securityContext.runAsNonRoot=true), runAsUser=0 (container "container-00" must not set runAsUser=0), seccompProfile (pod or container "container-00" must set securityContext.seccompProfile.type to "RuntimeDefault" or "Localhost")
default                                            32m         Warning   FailedCreate                                      daemonset/debug                                                       Error creating: pods "debug-dpkrr" is forbidden: violates PodSecurity "restricted:latest": host namespaces (hostNetwork=true, hostPID=true), privileged (container "container-00" must not set securityContext.privileged=true), allowPrivilegeEscalation != false (container "container-00" must set securityContext.allowPrivilegeEscalation=false), unrestricted capabilities (container "container-00" must set securityContext.capabilities.drop=["ALL"]), restricted volume types (volume "host" uses restricted volume type "hostPath"), runAsNonRoot != true (pod or container "container-00" must set securityContext.runAsNonRoot=true), runAsUser=0 (container "container-00" must not set runAsUser=0), seccompProfile (pod or container "container-00" must set securityContext.seccompProfile.type to "RuntimeDefault" or "Localhost")
default                                            32m         Warning   FailedCreate                                      daemonset/debug                                                       Error creating: pods "debug-2rqvh" is forbidden: violates PodSecurity "restricted:latest": host namespaces (hostNetwork=true, hostPID=true), privileged (container "container-00" must not set securityContext.privileged=true), allowPrivilegeEscalation != false (container "container-00" must set securityContext.allowPrivilegeEscalation=false), unrestricted capabilities (container "container-00" must set securityContext.capabilities.drop=["ALL"]), restricted volume types (volume "host" uses restricted volume type "hostPath"), runAsNonRoot != true (pod or container "container-00" must set securityContext.runAsNonRoot=true), runAsUser=0 (container "container-00" must not set runAsUser=0), seccompProfile (pod or container "container-00" must set securityContext.seccompProfile.type to "RuntimeDefault" or "Localhost")
default                                            32m         Warning   FailedCreate                                      daemonset/debug                                                       Error creating: pods "debug-ghjw9" is forbidden: violates PodSecurity "restricted:latest": host namespaces (hostNetwork=true, hostPID=true), privileged (container "container-00" must not set securityContext.privileged=true), allowPrivilegeEscalation != false (container "container-00" must set securityContext.allowPrivilegeEscalation=false), unrestricted capabilities (container "container-00" must set securityContext.capabilities.drop=["ALL"]), restricted volume types (volume "host" uses restricted volume type "hostPath"), runAsNonRoot != true (pod or container "container-00" must set securityContext.runAsNonRoot=true), runAsUser=0 (container "container-00" must not set runAsUser=0), seccompProfile (pod or container "container-00" must set securityContext.seccompProfile.type to "RuntimeDefault" or "Localhost")
default                                            32m         Warning   FailedCreate                                      daemonset/debug                                                       Error creating: pods "debug-rf2l8" is forbidden: violates PodSecurity "restricted:latest": host namespaces (hostNetwork=true, hostPID=true), privileged (container "container-00" must not set securityContext.privileged=true), allowPrivilegeEscalation != false (container "container-00" must set securityContext.allowPrivilegeEscalation=false), unrestricted capabilities (container "container-00" must set securityContext.capabilities.drop=["ALL"]), restricted volume types (volume "host" uses restricted volume type "hostPath"), runAsNonRoot != true (pod or container "container-00" must set securityContext.runAsNonRoot=true), runAsUser=0 (container "container-00" must not set runAsUser=0), seccompProfile (pod or container "container-00" must set securityContext.seccompProfile.type to "RuntimeDefault" or "Localhost")
default                                            32m         Warning   FailedCreate                                      daemonset/debug                                                       Error creating: pods "debug-cg58w" is forbidden: violates PodSecurity "restricted:latest": host namespaces (hostNetwork=true, hostPID=true), privileged (container "container-00" must not set securityContext.privileged=true), allowPrivilegeEscalation != false (container "container-00" must set securityContext.allowPrivilegeEscalation=false), unrestricted capabilities (container "container-00" must set securityContext.capabilities.drop=["ALL"]), restricted volume types (volume "host" uses restricted volume type "hostPath"), runAsNonRoot != true (pod or container "container-00" must set securityContext.runAsNonRoot=true), runAsUser=0 (container "container-00" must not set runAsUser=0), seccompProfile (pod or container "container-00" must set securityContext.seccompProfile.type to "RuntimeDefault" or "Localhost")
default                                            32m         Warning   FailedCreate                                      daemonset/debug                                                       Error creating: pods "debug-kk22c" is forbidden: violates PodSecurity "restricted:latest": host namespaces (hostNetwork=true, hostPID=true), privileged (container "container-00" must not set securityContext.privileged=true), allowPrivilegeEscalation != false (container "container-00" must set securityContext.allowPrivilegeEscalation=false), unrestricted capabilities (container "container-00" must set securityContext.capabilities.drop=["ALL"]), restricted volume types (volume "host" uses restricted volume type "hostPath"), runAsNonRoot != true (pod or container "container-00" must set securityContext.runAsNonRoot=true), runAsUser=0 (container "container-00" must not set runAsUser=0), seccompProfile (pod or container "container-00" must set securityContext.seccompProfile.type to "RuntimeDefault" or "Localhost")
default                                            32m         Warning   FailedCreate                                      daemonset/debug                                                       Error creating: pods "debug-nnhnd" is forbidden: violates PodSecurity "restricted:latest": host namespaces (hostNetwork=true, hostPID=true), privileged (container "container-00" must not set securityContext.privileged=true), allowPrivilegeEscalation != false (container "container-00" must set securityContext.allowPrivilegeEscalation=false), unrestricted capabilities (container "container-00" must set securityContext.capabilities.drop=["ALL"]), restricted volume types (volume "host" uses restricted volume type "hostPath"), runAsNonRoot != true (pod or container "container-00" must set securityContext.runAsNonRoot=true), runAsUser=0 (container "container-00" must not set runAsUser=0), seccompProfile (pod or container "container-00" must set securityContext.seccompProfile.type to "RuntimeDefault" or "Localhost")
default                                            5m2s        Warning   FailedCreate                                      daemonset/debug                                                       (combined from similar events): Error creating: pods "debug-r5rsl" is forbidden: violates PodSecurity "restricted:latest": host namespaces (hostNetwork=true, hostPID=true), privileged (container "container-00" must not set securityContext.privileged=true), allowPrivilegeEscalation != false (container "container-00" must set securityContext.allowPrivilegeEscalation=false), unrestricted capabilities (container "container-00" must set securityContext.capabilities.drop=["ALL"]), restricted volume types (volume "host" uses restricted volume type "hostPath"), runAsNonRoot != true (pod or container "container-00" must set securityContext.runAsNonRoot=true), runAsUser=0 (container "container-00" must not set runAsUser=0), seccompProfile (pod or container "container-00" must set securityContext.seccompProfile.type to "RuntimeDefault" or "Localhost")

My impression is that it is hitting some security issues caused by OCP 4.12. In theory, there are several changes in terms of security policies in OCP 4.12 (e.g. https://connect.redhat.com/en/blog/important-openshift-changes-pod-security-standards), and I suppose the debug daemonset specification should be reviewed to deal with them.

@ramperher If I push up a branch, can you point to it and test?

@ramperher If I push up a branch, can you point to it and test?

Sure, I can give a try in an OCP 4.12 cluster :) I'll take a look!

@sebrandon1 , as discussed in https://github.com/test-network-function/cnf-certification-test/pull/548, I think we don't need to change anything in the debug daemonset specification, after testing in OCP 4.12. It's just a matter of adding these two labels to default namespace (where debug daemonset is deployed):
              pod-security.kubernetes.io/enforce: privileged
              pod-security.kubernetes.io/enforce-version: latest

However, as discussed in the PR, I feel that, maybe, it is too extreme to set such privileges in default namespace, which can be used by other workloads, utilities, etc. Having said that, would it be a problem if debug daemonset is deployed in a specific namespace rather than in default namespace? Just to isolate the permissions we give to specific namespaces.
Then, second question I have: can it be included, in the README, this particularity about labelling the namespace where debug daemonset is deployed when working in OCP 4.12? Or should we wait until OCP 4.12 is finally released?

I believe this change covers the security policies needed for the daemonset:
https://github.com/test-network-function/privileged-daemonset/pull/52
We can close this until we it any more (if any) problems.


Thread Local Storage Leak in openSSL 1.1.0b

It seems any TLS being allocated from the call stack of ERR_get_state() is not getting freed. I used an OPENSSL1.1.0b sample  demos\bio\server-cmod.c and it reproduces the issue.
Any tips to resolve this issue is appreciated.

	testConsoleApplication.exe!CRYPTO_THREAD_init_local(unsigned long * key, void (void *) * cleanup) Line 93	C
	testConsoleApplication.exe!err_do_init() Line 637	C
	testConsoleApplication.exe!err_do_init_ossl_() Line 634	C
	testConsoleApplication.exe!CRYPTO_THREAD_run_once(long * once, void (void) * init) Line 82	C
	testConsoleApplication.exe!ERR_get_state() Line 643	C
	testConsoleApplication.exe!ERR_put_error(int lib, int func, int reason, const char * file, int line) Line 358	C
	testConsoleApplication.exe!BIO_new_file(const char * filename, const char * mode) Line 75	C
	testConsoleApplication.exe!def_load(conf_st * conf, const char * name, long * line) Line 140	C
	testConsoleApplication.exe!NCONF_load(conf_st * conf, const char * file, long * eline) Line 217	C
	testConsoleApplication.exe!CONF_modules_load_file(const char * filename, const char * appname, unsigned long flags) Line 129	C
	testConsoleApplication.exe!main(int argc, char * * argv) Line 120	C++
	[External Code]

//demos\bio\server-cmod.c
#include <stdio.h>
#include <signal.h>
#include <openssl/err.h>
#include <openssl/ssl.h>
#include <openssl/conf.h>

int main(int argc, char *argv[])
{
    unsigned char buf[512];
    char *port = "*:4433";
    BIO *in = NULL;
    BIO *ssl_bio, *tmp;
    SSL_CTX *ctx;
    int ret = 1, i;

    ctx = SSL_CTX_new(TLS_server_method());

    if (CONF_modules_load_file("cmod.cnf", "testapp", 0) <= 0) {
        fprintf(stderr, "Error processing config file\n");
        goto err;
    }

    if (SSL_CTX_config(ctx, "server") == 0) {
        fprintf(stderr, "Error configuring server.\n");
        goto err;
    }

    /* Setup server side SSL bio */
    ssl_bio = BIO_new_ssl(ctx, 0);

    if ((in = BIO_new_accept(port)) == NULL)
        goto err;

    /*
     * This means that when a new connection is accepted on 'in', The ssl_bio
     * will be 'duplicated' and have the new socket BIO push into it.
     * Basically it means the SSL BIO will be automatically setup
     */
    BIO_set_accept_bios(in, ssl_bio);

 again:
    /*
     * The first call will setup the accept socket, and the second will get a
     * socket.  In this loop, the first actual accept will occur in the
     * BIO_read() function.
     */

    if (BIO_do_accept(in) <= 0)
        goto err;

    for (;;) {
        i = BIO_read(in, buf, sizeof(buf));
        if (i == 0) {
            /*
             * If we have finished, remove the underlying BIO stack so the
             * next time we call any function for this BIO, it will attempt
             * to do an accept
             */
            printf("Done\n");
            tmp = BIO_pop(in);
            BIO_free_all(tmp);
            goto again;
        }
        if (i < 0) {
            if (BIO_should_retry(in))
                continue;
            goto err;
        }
        fwrite(buf, 1, i, stdout);
        fflush(stdout);
    }

    ret = 0;
 err:
    if (ret) {
        ERR_print_errors_fp(stderr);
    }
    BIO_free(in);
    exit(ret);
    return (!ret);
}

How did you measure that memory leak?  Cleanup is now done with the atexit() procedure OPENSSL_cleanup(), found in crypto/init.c.  It appears to me that it does call error string unloading and err cleanup.  However, this happens after exit from main.

I used Application Verifier and it reported a TLS leak. However I put breakpoint at threads_win.c where the TlsAlloc and TlsFree happens. And noticed TlsFree did not happen for the allocation that occured from Err_get_state() stack as I had shown in my previous thread.

It seems error string unloading and err cleanup does not free the allocated TLS

Richard already pointed out that it is done after main via an atexit call.  Do you not see that happening?

I could see atexit call , which does OPENSSL_cleanup. However did not see the TlsFree for the one which was allocated during Err_get_state. Also Appverifier complains about it. You can try with the program that I submitted in my earlier thread, the issue is reproducible.

Please can you try again with a more up to date version of OpenSSL? There have been a number of fixes to the auto-init/deinit code that impact Windows specifically since 1.1.0b.

Could you pl. point me to changelog that has fixes for auto-init/deinit code.

I checked with latest OpenSSL 1.1.0e and similar issue I see. If you track the number of TlsAlloc vs TlsFree, they don't match. If the openssl is just used inside an exe with statically linking, Appverifier does not complain. And I could see atexit call and no of  TlsAlloc and TlsFree still does not match .
However if I am using openssl inside a dll, and the calling application does call FreeLibrary() then I see complain from AppVerifier on TLS leak. However I could see atexit call when FreeLibrary() is invoked and could see similar pattern on non-matching TlsAlloc and TlsFree.
Is there a way of doing explicit uninit in openssl1.1.0 or any other workaround to resolve this issue.

Hmmmm...  when I grep for err_thread_local, I can see it being initialised, but never cleaned up.
: ; git grep -n err_thread_local
crypto/err/err.c:124:static CRYPTO_THREAD_LOCAL err_thread_local;
crypto/err/err.c:622:    CRYPTO_THREAD_set_local(&err_thread_local, NULL);
crypto/err/err.c:640:    return CRYPTO_THREAD_init_local(&err_thread_local, NULL);
crypto/err/err.c:650:    state = CRYPTO_THREAD_get_local(&err_thread_local);
crypto/err/err.c:657:        if (!CRYPTO_THREAD_set_local(&err_thread_local, state)) {

I think its corresponding state is cleaned up as below, instead of doing TlsFree on err_thread_local.
CRYPTO_THREAD_set_local(&err_thread_local, state)
ERR_STATE_free(state);
If this is legal way of freeing the TLS resource then I feel Application verifier does not understand this.It just assumes to have matching number of TlsAlloc and TlsFree

No. state is not the thread local storage itself, err_thread_local is. So it looks like you found something indeed, unless @mattcaswell has another explanation...

Please try the small patch from #3035 and see if that makes it better.

Yes the patch makes it better, I don't see any more TLS leak from App Verifier. Before consuming this fix any more test the change need to go through at OpenSSL development.

Before consuming this fix any more test the change need to go through at OpenSSL development.

Err, was that meant to be a question?
All that's needed is for another team member to approve it, then it goes in and will be part of the next 1.1.x release.

I would like to see the change gets approved by a team member and goes to next 1.1x release. How much time it will take tentatively ?

The approval is probably a matter of today or so.  The next release, whichever it is, hasn't been scheduled yet.  Can you live with this little patch for the time being?

I think this fixes only the symptom:
Previously all threads were able to call
ERR_remove_state(0), but now this function is empty.
If a server creates many thread and does that over months
error state memory will pile up, or did I miss something ?
I see there is a OPENSSL_thread_cleanup, that is
called in the DLL_THREAD_DETACH, but IIRC
ERR_remove_state was alled from DLL_THREAD_DETACH
in 0.9.8 until it was suddenly removed, because
it requires mutexes and caused dead-locks,
are you sure that this issue was not re-introduced ??

The library automatically deallocates thread resources when it detects that a thread has stopped. Additionally applications can call OPENSSL_thread_stop() directly:
The OPENSSL_thread_stop() function deallocates resources associated
with the current thread. Typically this function will be called automatically by
the library when the thread exits. This should only be called directly if
resources should be freed at an earlier time, or under the circumstances
described in the NOTES section below.

I can't think of why it should cause dead-locks. We use real thread local storage now, rather than the pretend version we had in 0.9.8.

Only if LoadLibrary/FreeLibrary is called in a critical section, that is also used in OPENSSL_thread_stop.
ERR_remove_state was removed in 87d14a3625, probably the reason was something different...


host/cli: Include artifact URIs in flynn-host inspect output

This will help diagnose #2797 (i.e. is the image artifact set as the file artifact and vice versa, or does the job just have an image artifact and no file artifacts).

LGTM


Track the tutorial progress and quiz results and adopt the final message

Currently the final message is very prominent and shows the results of the quizzes (if quizzes).

[x] The grade of the results should be less prominent next to the result
[ ] Results (and grades) should only be shown if all quizzes are done
[ ] Add closings when all steps are done.
[x] Allow unlimited blocks in the workspace after leaving a tutorial

and 4. tasks done and work.


fix(radix): font-face weight declaration

In order to simplify font-family declarations across the project, the best way to declare font-faces from the same family (in this case UntitledSans regular and medium) is to declare them with the same font-family name.
This simplifies things a lot because the developer using the theme doesn't need to think if the normal or medium font-family must be used, it'll always the normal one and by declaring the font-weight the browser picks up the correct font-face. Conceptually makes sense: is the same font-family, only that the weight is different.
This is a really good (short) read about the subject https://css-tricks.com/whats-deal-declaring-font-properties-font-face/
Before
fontFamily: bold ? 'medium' : 'normal',
fontWeight: '500',

After
fontFamily: 'normal',
fontWeight: '500',

BTW, I love what you are doing with Radix 😄

Brilliant, thanks @aarongarciah!
We were talking about this only last week, so your timing is good. 😄

@peduarte This came up in a convo with @colmtuite IIRC.
Separately, I like the use of a medium fontFamily theme in a generic component library because it allows from simple swap-outs that allow for interesting brand modifications. eg. A/B testing font pairing for header vs body fonts... but those considerations are directly relevant to Radix 😄

Separately, I like the use of a medium fontFamily theme in a generic component library because it allows from simple swap-outs that allow for interesting brand modifications. eg. A/B testing font pairing for header vs body fonts... but those considerations are directly relevant to Radix 😄

You want me to keep the medium font family declaration in the theme file?

Hey, thank you. Yeah we were just talking about doing this last week 😁
I’ve got a PR opened right now, I’ll probably it get it merged tomorrow morning, I’ll let you know so you can resolve the conflicts that will happen here.

Ok! Medium font-face declaration recovered https://github.com/modulz/radix/pull/130/commits/a3f346504b96d398c9407f72c253529bb758b239

My PR #128 has now been merged, keen to get this in once conflicts have been resolved 🙌

Conflicts resolved 😄

I haven't read this thread at all. I just saw people talking about font weight.
Just dropping in to mention that we can't remove any font-weight declaration because we need it for when the font hasn't loaded yet. The fallback font should be in the correct font-weight.
Maybe you guys aren't changing that. If not, carry on!

@colmtuite All good. font-weight declarations are staying, what we removed is manually having to set UntitledSans-Medium when the font-weight is 500 - the Medium font should now be loaded automatically based on the weight.

@aarongarciah Thanks for this 👍 🚀


Announce release of code

Announce availability on Sparkfun's product page in the comment section.

Put a comment on the Sparkfun product page.


Search does not run if keyword is empty - needs an alert

Currently, the search does not run when the keyword is empty. However, the user is still able to click search and the search bar collapses. This appears to be buggy to the users, because a user gets the impression that they can run the search, except nothing actually happens.

Recommendation:

If there is no keyword and the search will not run, when the user clicks the "Search" button, the search bars should not collapse into the single search bar. This gives users the false impression that their search will run.
When a user clicks search and the keyword is empty, user should be alerted that they need to fill in a keyword. For the time being, can we just use a system alert that says "Please enter a keyword to search"? - System alert like below:

^ @jparlina this string "Please enter a keyword to search" will need to be translated if this is implemented. @rasmus-storjohann-PG or whoever is developing this will need to confirm if this copy is adequate for an alert, or if other copy is needed.

Ideally, the search button should appear inactive (deactivated/greyed out) until the search can be run. Do we currently have a deactivated button state built? I believe this was mocked up but not sure if it is implemented anywhere.

Acceptance Criteria

[ ]  Search bars don't collapse when the user clicks "Search" button and then keyword search is empty
[ ]  Clicking "Search" button when the keyword search is empty gives the user a system alert that they need to enter a keyword.
[ ]  Optional: inactive state for search button when the keyword search is empty

Steps for QA:

I've talked to Cherrie about this. Due to simplicity we've decided to make these buttons disabled (non clickable) and reduce opacity when we have no text in the field. The popup goes against the idea that the button is disabled so we're not moving forward with it.

Staging version 1.27.0:
Pass


Any way to roll great weapon master?

Reroll a 1 or 2, forced to use the new roll even if its a 1 or 2?
I've tried /r 1d12r1r2, but that doesnt work perfectly. A 1 that rerolls as a 1 will stay as 1, but a 1 that rerolls a 2 will reroll again.

brute force way, you can command to reroll the same number multiple times. unless you roll ones and twos 6 (or more) times in a row, you'll get what you need. though 1 in 50000 times will screw up. Do /r $GWM = 1d12r1r2r1r2r1r2r1r2r1r2r1r2 and use /r $GWM

I had the same issue and had to play around with it to find a solution. /r 1d12r<2 should work.

and keep the reroll no matter what

In practice I'm sure you wouldn't do that. You're at a table, you roll 1, reroll it to 1, reroll it to 1, reroll it to 1, reroll it to 1, reroll it to 1, reroll it to 1, reroll it to 1, reroll it to 1, reroll it to 1, reroll it to 1, reroll it to 1, reroll it to 1, reroll it to 1, reroll it to 1, reroll it to 1, reroll it to 1, reroll it to 1 and so on until the game session ends? No way.
#199

@ArtemGr I may not have worded that clearly. I don't mean "keep rerolling no matter what," I meant "keep the die that was rerolled no matter what," as in, don't reroll it again. I was just telling OP that the function /r 1d12r<2 should work for their needs.

I see, thanks!


#62 Bug-fix, refactoring

Addressed #62 by having a toast display "Please fill in all fields!" if not all fields have been filled out. I've also done a lot of string refactoring to fit naming conventions and also refactored some class names.

The bugfix changes can be found in RegisterActivity and the StringHelper classes, the rest is refactoring.


Discrete switch for watchs history

See https://github.com/nteract/hydrogen/pull/143 and https://github.com/nteract/hydrogen/pull/175
It's more intuitive than the scrollbar.
I think it looks a bit cluttered. But I couldn't come up with a better layout.

I think it would be great to have a scrollbar behind counter too: it would be hard to rewind 1000 history entries by clicking.
Cluttered? Yes, But it looks better than it was. Thanks for your work

Thanks for the feedback.
I thought about that too.
The problem with the scrollbar for me is that it won't appear on Mac OS because by default scrollbars are hidden on Mac OS. And I don't know how to force show them.
Would an additional skip to start/end button do the trick?

Would an additional skip to start/end button do the trick?

No. If I have thousands result items.

default scrollbars are hidden on Mac OS.

And previous scrollbar too?

OK, I understand. So something like alt-clicking on the button for skipping 10 results at a time wouldn't work to.
I try adding the scrollbar behind the counter.

And previous scrollbar too?

Yes.

default scrollbars are hidden on Mac OS.

did you try to change your atom theme?
I think it should be .darwin specific css rule.

I added the scrollbar again.

Very slick.

Let scrollbar be as tall as history controll.

I made the scrollbar bigger:

On Mac the scrollbar is shown, depending on your systemwide settings. If not it doesn't break the layout:

Great! I'll test now

Nice! I hope it works on Windows and Linux as well.

It works! Let's merge
@lgeiger Is there any way to make scrollbar responsive when mousedown triggered on div.counter?

@slavaGanzin Stackoverflow to the rescue. Just one line CSS ;)
I didn't know this is possible.

@lgeiger Looks like I forget how to use google, I asked after lurking at stackoverflow.

:tada: feel free to make a release, you have the power

@rgbkrk I dont know how :)

After pulling master / syncing up with master, and making sure that your remote is configured for this source repo:
apm publish minor
Depending on the changes, you can also do major or patch (like npm, with less steps).

@rgbkrk Thanks for clarification

Let me know if you want me to publish, I'm happy to as well.

Though it looks like there's a new related PR from @lgeiger, so we can hold off til you want to cut the release.


Click a link, except ajax response Dom, but there is no response Dom

Reproduce:
1 Get a link.
link = document.querySelector() .( this link is an ajax request,  and except response some data to modify the dom, add new element in dom.)
2 create mouse event:
var ev = document.createEvent("MouseEvents");                                                                          ev.initMouseEvent("click", true, true)
3 link.dispatchEvent(ev)
And there is no new element create. I try in chrome console the same step, it work correcly.
I found the ajax request was sent, but will follow a page.onNavigationRequested. It seems the page was reopened.
use ubuntu server 12.04, without graphics.

I have the same problem at ubuntu server 12.04 without graphics. Both 1.9.8 and 2.0 version.
I try the solution https://newspaint.wordpress.com/2013/04/05/waiting-for-page-to-load-in-phantomjs/,but it not works.
But 2.0 version for mac work perfect.


Add Helm

Closes #15
Summary
This PR adds helm to our ks8 deployment. The benefits of helm are

Share your own applications as Kubernetes charts
Create reproducible builds of your Kubernetes applications
Intelligently manage your Kubernetes manifest files
Manage releases of Helm packages

Please visit: https://github.com/kubernetes/helm to learn about helm
How to test

helm init to install tiller on your cluster in gcloud (Tiller runs inside of your Kubernetes cluster, and manages releases (installations) of your charts.)
helm install data8-jhub/
Test that the deployment works

@SamLau95 this PR is ready to review. I decided not to break up seperate-proxy into it's own subchart and didn't add too much customization because I think it's worth more discussion before deciding what to configure.
However, this PR works and will allow us to iterate/refine it in the future.

Todos after merging:

Change docs in the README to deploy using helm
Possibly write a tutorial on how to deploy and manage helm deployments
Split out other components of the deployment (eg. proxy, etc.)


Add badges in readme

There is no readme and no badges in readme.md i can make new readme with badegs.
Please assign this issue me to under Gssoc'24 , i would like to work on this

hey, can you assign me this issue if issue is still not solved? This is my first time in open source and I really want to showcase my skills.


tpmtest: Use inttypes 64bit format specifier for clock values.

long unsigned int is a different size on 64bit and 32bit platforms. Since
the clock values are fixed width we shouldn't be using architecture
dependant format spacifier anyways.
Signed-off-by: Philip Tricca<EMAIL_ADDRESS>

this fixes build breakage on 32bit systems

merged in f73ea368bbfa11ff9747d5bd91738cf6588297e9


TsLint= >Implement lifecycle hook interface OnInit for method ngOnInit

Keep tslint checks upto date with latest implementations.
Running
angular-cli: 1.0.0-beta.24
node: 4.6.0
os: darwin x64
Description
Tried to utilize latest Angular's addition for the inheritance
Angular Commit
where lifecycle hook interface is implemented on the parent level and Child component just override the method and it seems tslint is not uptodate with this .
"use-life-cycle-interface": true,

It should probably check all the way up to the parent if the interface is implemented.
Thank you,
Frank
Please see https://github.com/angular/angular-cli/issues/4060

Seems duplicate of https://github.com/mgechev/codelyzer/issues/64.


Feature/ott 1778

remove package-lock.json

Closed due to inactivity, please restore if still needed.


No validation error is logged when trying to create swapchain without enabled extension

When calling vk:create-swapchain-khr on a device without vk:+khr-swapchain-extenstion-name+ enabled SBCL signals a sb-sys:memory-fault-error ("bogus stackframe").
I would have expected a validation error message, but got nothing.
Test this using the C API to find out if this is the correct behavior or not. If not -> fix!
e.g.:
CL-USER> (vk-samples:12-init-frame-buffers)
; Evaluation aborted on #<SB-SYS:MEMORY-FAULT-ERROR {10042DF123}>.

validation errors are logged to inferior-lisp and not via the debug utils messenger
-> document this!

I guess that's the reason why:
https://stackoverflow.com/a/40180199


unable to pull service with custom registry mirror

RancherOS Version: (ros os version)
1.5.4
Where are you running RancherOS? (docker-machine, AWS, GCE, baremetal, etc.)
Vsphere
Hi, I am trying to enable the volume-nfs service and the nfs kernel-modules in my rancher provisioned kubernetes kluster on vsphere. The kernel modules where straight forward with sudo ros config set rancher.modules "['nfs']" but I am having major issues with the service.
I can't seem to be able to tell rancherOS to pull the service image via our internal docker registry. I have tried to set the registry_mirror for both the user docker and system docker via ros config set and I am able to do both sudo system-docker pull rancher/os-volumenetshare and docker pull rancher/os-volumenetshare.
This is as far as I have been able to figured out from https://github.com/rancher/os-services/blob/master/v/volume-nfs.yml but now I am stuck. Any help on how this is suppose to work?

The issue is not present if I tell the cluster the our internal coorporate proxy configuration via the ros config rancher.network.


:sparkles: Feat: Support remappings.txt

Description
Concise description of proposed changes
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1332  👈
#1331
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @roninjin10 and the rest of your teammates on  Graphite


feat(MetadataLoading) : enable metadata loading from images

Context
The goal of this PR is to allow the MetadataProvider to read the metadata of dicom images directly without making an extra WADO-RS Metadata request to the PACS.
Internaly it uses dataSetCacheManager of dicomImageLoader to access the DICOM dataset. Then dcmjs parse the dataset and add the metadata to the MetadataStore.
This PR allow the DicomJSON source to provide only the image UID + instance number without the need of putting all the image metadata into the final .json.
You can enable or disable this behavior with the variable automaticallyLoadDicomMetadata in the config file.
Changes & Results

Testing

This can be tested using a regular JSON DataSource where only SeriesInstanceUID, StudyInstanceUID, SOPClassUID and InstanceNumber must be present.
Checklist
PR

[x] My Pull Request title is descriptive, accurate and follows the
semantic-release format and guidelines.

Code

[x] My code has been well-documented (function documentation, inline comments,
etc.)

Public Documentation Updates

[] The documentation page has been updated as necessary for any public API
additions or removals.

Tested Environment

[x] OS:  Windows 10
[] Node version:
[x] Browser: Chrome 115.0.5790.110, Firefox 115.0.3

so this is only for the dicomJSON right? since for the dicomweb there is no extra call to the PACS and the metadata provider already have all the necessary information from the instance call

Maybe we missed something in the recent changes, but in the previous versions, OHIF was doing a WADO-RS Metadata request before retrieving the instances, so we thought this could be useful as well for the dicomweb mode, as long as you know the studies, series and instances UIDs.
But in our case, yes, we are mostly interested in reducing the metadata in the JSON datasource. Actually, we are trying to open OHIF from a DICOM Key Object Selection (KOS), which usually does not contain all the images metadata (rows, column, ...).

I see.
We indeed fire a metadata for wadors since that is needed to layout the viewer before the instance call for the dicomweb.
I will review your PR, the immediate thing that comes to my attention is the added dependency of cornerstone and dcmjs into the core
import dicomImageLoader from '@cornerstonejs/dicom-image-loader';
import dcmjs from 'dcmjs';

which ideally should not happen. But I will give my in-depth comments today/tomorrow
Thanks!

can you give this a look please? @moalsad

can you give this a look please? @moalsad

I agree that dicomImageLoader + dcmjs should be placed somewhere else more specific to the (JSON) data source.

Replaced by https://github.com/OHIF/Viewers/pull/3935


Separate out NATS code base into a seperate projects.

Right now Kafka and NATS triggers are on the same repo as Kubeless. We should be able to have triggers on different repos.
We should put at least NATS on its on repo

https://github.com/kubeless/kubeless/issues/695 is opened as well. We could close one of the bugs.

close this one and move #695 to the board, please. Thanks!

Closing in favour of #695, which is bit more broder in context.


I took this example from the middle of a larger file, so I know that the odd indentation is not related to this being the last form in a file.
Notably, this altered version is correctly formatted as I expect:
(ns example)

(defn rgb->css
  ([r g b] (str))
  ([r g b a] (str)))

I suspect the bug is connected to parentheses appearing inside string literals.
I observed this behavior on v0.4.1 of the formatter.

Great catch! Thank you for the helpful bug report 🤓 Fixed with PR-100

Published v0.5.0 with this fixed


insert all rows at once untested

should be way more efficient.

Note however that this breaks on certain servers:
http://stackoverflow.com/questions/16491039/maximum-query-length-to-avoid-epipe-error-in-node-js
So it should be optional I guess.

@atom Thats great.  Ill review that soon. And create tests.


flutter config --enable-[android][ios] not working

I'd like to be able to add Android/iOS build at a later time, only setting up web initially. There seem to be an issue on adding Android/iOS support after the project has been created.
Steps to Reproduce

Create a new Flutter Project
Only have web enabled

Add Android config with flutter config --enable-android
I've tried restarting Android Studio. Running flutter clean, flutter pub get, and flutter pub upgrade has no effect.

Any idea on what might've gone wrong? Is there other ways to update the Flutter project config? The only workaround I can think of at present is to just create a new Flutter project and migrate the existing code from lib/
Expected results:
Developers should be able to add/remove Android/iOS builds from the project with an update of the config.
Actual results:
Using flutter config --enable-[android][ios] to add mobile support to the Flutter project doesn't work.

Logs

[✓] Flutter (Channel stable, 2.2.3, on macOS 11.5.1 20G80 darwin-x64, locale en)
    • Flutter version 2.2.3
    • Framework revision f4abaa0735 (6 weeks ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Library/Internet Plug-Ins/JavaAppletPlugin.plugin/Contents/Home/bin/java
    • Java version Java(TM) SE Runtime Environment (build 1.8.0_221-b11)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0.1, Build version 12A7300
    • CocoaPods version 1.10.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] VS Code (version 1.59.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.25.0

[✓] Connected device (2 available)
    • AOSP on IA Emulator (mobile) • emulator-5554 • android-x86    • Android 9 (API 28) (emulator)
    • Chrome (web)                 • chrome        • web-javascript • Google Chrome 92.0.4515.131

! Doctor found issues in 1 category.

config is a global setting, not a project setting. There's no config opt-in for iOS and Android because support for those platforms isn't prerelease.
The command you are looking for is flutter create --platforms=android,ios . in your project directory.

Thanks! The command generated the missing Android/iOS files flutter create --platforms=android,ios [PROJECT]


Installing Error CS0117

Hi! I am trying to install the package through github url but the unity gives the following error:
Library\PackageCache\com.openai.api.unity@ca61ef4aaf\Runtime\Scripts\Api\Base\AApiResource.cs(209,54): error CS0117: 'UnityWebRequest' does not contain a definition for 'PostWwwForm'
I am not sure what is going on, will it be a problem with the Unity version?

This is fixed as of v0.2.12.


dashboard not automatically refreshed

If I create a new repo, and click "Codepod" in the breadcrumbs to navigate back to the dashboard, the new repo will not appear on my dashboard until I manually fresh (via the browser) the dashboard page. I am not sure whether we do not have autorefresh or the auto-refresh frequency is too low. But at least every time I access it.

Yes, I'm aware of this. This is a bug somewhere in React re-rendering logic. It should be instantly refreshed.

i can take a look

PR #422 should fix the issue

#422 doesn't fix this issue. (And #422 introduced a regression, fixed in #425).
This issue is weird, it works well (to refresh the dashboard) in the local development env, but doesn't work on the production server.

Possible solutions:

use the full query {query: ..., variables: ...} instead of the string "GetDashboardRepos" in the refetchQueries option. Ref: https://github.com/apollographql/apollo-client/issues/5419#issuecomment-598065442
update GraphQL cache manually instead of relying on refetchQueries, similar to what Haochen did in https://github.com/codepod-io/codepod/pull/364#discussion_r1257722308


GPP API v1.1

Will this package be updated to support v1.1 of the GPP API (changes outlined in this pr - https://github.com/InteractiveAdvertisingBureau/Global-Privacy-Platform/pull/70)

Yes, this will be updated shortly following finalization

Hi, is there any time estimate for when this will be available?


Resize Error in 0.7.x on playback

Testing in 0.7.x with Persona 0.1.56.  The poster resizes to 100% width:

But the video is left-aligned and bound by the original height instead of the 100% width:

+1 that's happening with me on latest nodebb and persona

That is not really a plugin problem. You can add the following lines in your custom CSS to solve the issue :
.topic .posts .content iframe {
    max-width: 100%;
}

With persona theme, it is set at 85% by default.

Closing as not a plugin issue.


Unique cleanup

Clean up a couple of deprecated features used in Unique.

Current coverage is 88.83% (diff: 100%)

Merging #4763 into master will decrease coverage by 0.03%

@@             master      #4763   diff @@
==========================================
  Files           121        121
  Lines         74221      74285    +64
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          65958      65991    +33
- Misses         8263       8294    +31
  Partials          0          0

Powered by Codecov. Last update 96f092b...d75387b

@tmccombs this is getting there, could you respond to @JackStouffer and take this to the finish line? Thx!

Auto-merge toggled on


Create v1.10.2.md

Add 1.10.2 release notes.

Do not merge, @artursouza has thoughts around patching 1.10.1.

Should this be merged and 1.10.2 cut again?

I will still do 1.10.2 because @mukundansundar pointed out that users might not pick up the fixed 1.10.1 release if they don't set their PodTemplates to always pull image.


chore: improve naming of build scripts

darwin/configure-electron.sh -> darwin/electron-configure-package-darwin.sh
darwin/installer-zip.sh -> darwin/electron-installer-app-zip.sh
darwin/installer-dmg.sh -> darwin/electron-installer-dmg.sh
darwin/sign.sh -> darwin/electron-sign-app.sh
linux/configure-electron.sh -> linux/electron-configure-package-linux.sh
linux/installer-appimage.sh -> linux/electron-installer-appimage.sh
linux/installer-deb.sh -> linux/electron-installer-debian.sh
unix/create-asar.sh -> unix/electron-create-asar.sh
unix/create-electron-app.sh -> unix/electron-create-resources-app.sh
unix/download-electron.sh -> unix/electron-download-package.sh

Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

@lurch Looking forward to naming suggestions. I prefixed most with electron since they are very electron specific, and by having a prefix in place, we can create a whole new suite of "cli" specific scripts.

In the future, I'm also looking forward to join all scripts in a single directory, since a lot of stuff will be able to get re-used. What do you think? Can you think of any reason to not do so?

electron-create-resources-app.sh sounds like a strange name?

:+1: for re-use :)

electron-create-resources-app.sh sounds like a strange name?

Yeah, but seems to be the way electron calls it, and despite strange, it will probably be familiar to people working on electron. So when you package Electron applications, you create a directory called app (you should compress to app.asar for proper releasing) which lives inside a "resources" directory. Therefore its called "the resources app", so its not confused with OS X *.apps.

Talking about re-use, I'd be so awesome if we could write both Windows and UNIX scripts in the same language (e.g: bash), since that would allow us to re-use even more, but that doesn't look straightforward. We could require Windows users to build on something like Cygwin, but we rely on "Visual Studio" cmd.exe to access command line build utilities, and I'm not sure how easy it'd be to make use of those from outside (since the Visual Studio console is pre-configured with lots of stuff to just work).
Windows 10 ships experimental bash support, which would be the ideal case, but we still need to support older Windows versions, and the bash support is very unstable as far as I've heard.
Something like https://github.com/BYVoid/Batsh looks perfect, but that one is unmaintained, and doesn't build at all in OS X.
Thoughts?

and despite strange, it will probably be familiar to people working with electron

Fair enough.

Something like https://github.com/BYVoid/Batsh looks perfect

Cool, that looks neat! As you say, shame that it's unmaintained :-(  (and obviously being written in OCaml means it's unlikely to get 3rd-party contributors)
If we wan't to go with "native scripting" then I guess using bash on Linux / OSX and bat on Windows is a pain we'll just have to put up with. OTOH, when we start shifting more stuff over to Makefiles, they're easier to make cross-platform.
Alternatively, given that node and python are build-dependencies, I guess theoretically there's no reason that the build scripts couldn't be rewritten in those?

Yeah, to be honest, I'm very tempted to use the C preprocessor or something like gpp and create some macros that translate basic functionality to either bash or batch... Maybe a small side project :P

Alternatively, given that node and python are build-dependencies, I guess theoretically there's no reason that the build scripts couldn't be rewritten in one of those?

The problem is that some scripts will feel very awkward on something else (for example see the AppImage installer generation script). Bash makes perfect sense for it, while using something else might lead to an unmaintainable mess.

There are huge delays in OS X builds in Travis at the moment, so I'll merge without waiting for the OS X builds (which can take up to ~10 hours). I tested it myself on my MacBook Pro.


Annoying Background

hello sir, any fix for this problem? those white background are really annoying me

@dimasrr3 It has been fixed, try it.

umm sorry, but if im using default fulldisplayapp, those disk didnt rotate. and then when im using modified fulldisplayapp it will, but still had those white background

this one is default fulldisplay app

and this one with modified fulldisplay app
and if im using both, it will rotate, but there's no button for choosing upcoming and previous song

ok im trying reinstall spicetify,

it works, but now i should use 2 fulldisplayapp... the original one, and modifier, as you can see, any fix maybe so i can only use 1 instead 2 fulldisplayapp?

still exist

and sometimes the problem's dissapeared, but after using several times and check again, those problem back
im trying to using 2 fulldisplay app, the original and the modified one, the original one didnt had this problem, but those disc didnt rotate.. the modified one is rotating but had those white background, kinda annoying


added 'affinity' and 'tolerations' configuration hooks to coredns

Added affinity and tolerations configuration options to cert-manager chart to fix: https://github.com/helm/charts/issues/7042

/assign @munnerz

Do we need lgtm label for this?

Also you will need to rebase and fix the conflict

Hi there -
Thanks for your pull request. I'm afraid we don't accept changes to the cert-manager Helm chart in this repository, in favour of making the changes in the upstream jetstack/cert-manager repo.
If you wouldn't mind making this same change there, it will then be synced across automatically. This allows us to fully test our Helm chart/deployment code before it is published to users 😄
You can find a copy of the chart here: https://github.com/jetstack/cert-manager/tree/master/contrib/charts/cert-manager
Again, thanks very much for doing this work and I'd love to be able to accept this upstream 😄
/close

... seems like someone has disabled the close plugin for this repo :\ can this be re-enabled so I am able to manage PRs?
/cc @mattfarina

Ok, here it is: https://github.com/jetstack/cert-manager/pull/869


Missing code linting dependencies in setup.py

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[ ] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[x] Tried basic troubleshooting (if a bug/error)

[x] Restarted Spyder
[ ] Reset preferences with spyder --reset
[ ] Reinstalled the latest version of Anaconda
[ ] Tried the other applicable steps from the Troubleshooting Guide

[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
There are a few missing packages in setup.py that are necessary for code styling/linting messages, namely:

pydocstyle
autopep8
pycodestyle

This probably doesn't pose a problem most of the time with pip or Anaconda installs, but occasionally these dependencies may not be always pulled in on all Linux distributions making use of setup.py for packaging, such as is the case with Fedora.
A Fedora bug has already been opened on this matter:
https://bugzilla.redhat.com/show_bug.cgi?id=2034385
What steps reproduce the problem?

(On Fedora) Install Spyder through the package manager.
Turn on all linting options in Spyder's settings.
Observe the lack of code styling warnings for my bad Python code.

What is the expected output? What do you see instead?
Code styling warning messages.
 Those are transitive dependencies that come through the python-lsp-server package in our setup.py:
https://github.com/spyder-ide/spyder/blob/8454770df17a32a4b89b6237cf97b2f1a4f96f42/setup.py#L230
The all extra requirement is what brings them:
https://github.com/python-lsp/python-lsp-server/blob/963fd4de7f1651e0ef0a11516f51626453d1230b/setup.cfg#L24-L33
so we don't need to declare them. Please inform that to the Fedora maintainers.


Fundamental of OOP's

Description 📜
Defining what exactly OOP's is  in C++ and enlisting the features of OOP's

Domain of Contribution 📊

[x] C/CPP

/assign

/assign