Root level deploy.cmd/deploy.sh scripts cannot be invoked with -version

Describe the bug
Because the root level deployment scripts act as a pass-through to the ./deploy/scripts/deploy.ps1 file and hard code "-version preview", end users are unable to leverage them if/when they would like to specify a specific version of the stack to deploy
deploy.sh

Line 69: pwsh -File $curdir/deploy/scripts/deploy.ps1 "$@" -version preview

deploy.cmd

Line 33: %PWSH% -ExecutionPolicy Unrestricted ./deploy.ps1 %* -version preview

We were hoping to use these top level deployment scripts in CI/CD to stage the Azure Powershell environment, but need to specify a version
To Reproduce
Steps to reproduce the behavior:

Go to the root of the workspace
Invoke the deploy.sh or deploy.cmd script with the -version flag (e.g. ./deploy.cmd -version 2.7.105)
See error

Expected behavior
Users should be able to provide any/all input parameters that they would by directly invoking the deploy.ps1 script
Instead of hard coding the -version preview at this level, could we specify a default value in deploy.ps1 similar to $type and $environmentName?
Screenshots

Hi @jleger-ptc , thanks for reporting the issue. It has been fixed now.


Tapping placeholder does not focus cursor on Safari

It's impossible to focus on an editor by clicking on the placeholder in Safari, you can only focus on it by clicking after the placeholder, which is both incredibly unintuitive, and for inline simpla-text impossible.
This effects production simpla-text, so is urgent and breaking
Screencap repro:

Have not tested whether #16 fixes yet

See attached commit. Initially thought it was just forcing the selection that would fix it, but nope. Not really sure why but adding a zero width space to before the actual placeholder meant Safari would focus on it. Logically, no idea why, but figured it was something to do with Safari needing an extra piece of DOM to sit the cursor next to.

That seems to be the case for most things that happen to work on Safari - "no idea why this works, but hey, it's Safari". Coolio.


Could we change one letter ... ?

Looked at all the rules now and FWIW, think they're all excellent - specially Be Excellent To One Another. But there's the issue for me, embedded in link [1] which ==> SafeSpaces policy.
Could we call it SafeSpace policy ? This is Hackspace, not Hackspaces, albeit there's going to be 2.5 accessed by a flight of stairs.
SafeSpaces , plural as here, connotes all sorts of very recent political things, like no-platforming people in institutions of Higher Education (until not so long ago spaces for open and free debate and discussion) ; areas on campuses reserved for people of a certain race ; and erasing / denying history by tearing down sculptures and etc., etc.
I don't know if the phrase "SafeSpaces Policy" was intended to evoke this overtly political aspect, to be embedded in our rules ?
(Can't fault the actual policy)

The policy stands by its content and not its name - I'm pretty sure that "SafeSpace" has all the same connotations to those that cry "PC gone mad!" in response to any perceived opposition to their (offensive) views. I still like the name "Don't Be a Dick policy"

I'm happy to change to either.  It wasn't chosen because of the recent political university issues, in fact those policies were named after the same tech-conference policies we drew on to create this.
That said, we don't suffer from the same issues as universities - they are public spaces so have a need to allow free speech.  We are a private space and don't have any such requirement.
We obviously work very hard to ensure that there are freedoms in the space, but we draw the line at true "free speech" - as per this policy, there are restrictions on what you can say when at the space.
As the consultation ends today, and this policy is not technically a "rule" - it has been added as an appendix to keep it all in one place and allow easy updating, feel free to continue this discussion on the google group if you would like to.

I think Safe Spaces (plural) makes sense as we don't just do things in the physical building. We have an extensive online presence and do external events, both of which the policy should also apply to.

Yup; I take all those very fair points. I guess I'm over-sensitive about the phrase "safe spaces" in contemporary political ... 'discourse'


Add package.json

Added package.json so that we can install this package via npm.
By the way, I couldn't run rake build because I don't have project-kit.
I googled a minute but couldn't find out project-kit.
Should I replace Rakefile with npm scripts?

Should I replace Rakefile with npm scripts?
Sure, why not


Update GCP Dataproc custom image instruction and sample

Update instruction and example to create GCP dataproc custom image

@aroraakshit  @viadea @sameerz FYI.

build


Error in gui.lua on some display

Fixes #1

Bug was due to not rounding result of Button x position calculation


Variable in server url not being interpolated

Description
When the api spec in use specifies variables in the server url and the values are being passed when instantiating the connexion.FlaskApp, interpolation is not executed. See simple example below.
Expected behaviour
====== test session starts =====
test_app.py .
====== 1 passed ======

Actual behaviour
pytest test_app.py
============================================================================ test session starts ============================================================================
platform linux -- Python 3.11.8, pytest-8.1.1, pluggy-1.5.0
rootdir: /home/jvasik/dev/connexion-dummy
configfile: pyproject.toml
plugins: anyio-4.3.0
collected 1 item

test_app.py F                                                                                [100%]

================================================================================= FAILURES ==================================================================================
________________________________________________________________________________ test_route _________________________________________________________________________________

    def test_route():
        app_client = create_app((Path(os.getcwd()) / "api.yml").as_posix())
        response = app_client.get("test/v1/")
>       assert response.status_code == 200
E       assert 404 == 200
E        +  where 404 = <Response [404 Not Found]>.status_code

test_app.py:23: AssertionError
----------------------------------------------------------------------------- Captured log call -----------------------------------------------------------------------------
ERROR    connexion.middleware.exceptions:exceptions.py:84 HTTPException(status_code=404, detail='The requested URL was not found on the server. If you entered the URL manually please check your spelling and try again.')
========================================================================== short test summary info ==========================================================================
FAILED test_app.py::test_route - assert 404 == 200
======================================================================= 1 failed, 7 warnings in 0.79s =======================================================================

Full log
pytest test_app.py
============================================================================ test session starts ============================================================================
platform linux -- Python 3.11.8, pytest-8.1.1, pluggy-1.5.0
rootdir: /home/jvasik/dev/connexion-dummy
configfile: pyproject.toml
plugins: anyio-4.3.0
collected 1 item

test_app.py F                                                                                [100%]

================================================================================= FAILURES ==================================================================================
________________________________________________________________________________ test_route _________________________________________________________________________________

    def test_route():
        app_client = create_app((Path(os.getcwd()) / "api.yml").as_posix())
        response = app_client.get("test/v1/")
>       assert response.status_code == 200
E       assert 404 == 200
E        +  where 404 = <Response [404 Not Found]>.status_code

test_app.py:23: AssertionError
----------------------------------------------------------------------------- Captured log call -----------------------------------------------------------------------------
ERROR    connexion.middleware.exceptions:exceptions.py:84 HTTPException(status_code=404, detail='The requested URL was not found on the server. If you entered the URL manually please check your spelling and try again.')
============================================================================= warnings summary ==============================================================================
../../.local/share/virtualenvs/connexion-dummy-LMhlc2UK/lib64/python3.11/site-packages/connexion/json_schema.py:16
../../.local/share/virtualenvs/connexion-dummy-LMhlc2UK/lib64/python3.11/site-packages/connexion/json_schema.py:16
  /home/jvasik/.local/share/virtualenvs/connexion-dummy-LMhlc2UK/lib64/python3.11/site-packages/connexion/json_schema.py:16: DeprecationWarning: jsonschema.RefResolver is deprecated as of v4.18.0, in favor of the https://github.com/python-jsonschema/referencing library, which provides more compliant referencing behavior as well as more flexible APIs for customization. A future release will remove RefResolver. Please file a feature request (on referencing) if you are missing an API for the kind of customization you need.
    from jsonschema import Draft4Validator, RefResolver

../../.local/share/virtualenvs/connexion-dummy-LMhlc2UK/lib64/python3.11/site-packages/connexion/json_schema.py:17
  /home/jvasik/.local/share/virtualenvs/connexion-dummy-LMhlc2UK/lib64/python3.11/site-packages/connexion/json_schema.py:17: DeprecationWarning: jsonschema.exceptions.RefResolutionError is deprecated as of version 4.18.0. If you wish to catch potential reference resolution errors, directly catch referencing.exceptions.Unresolvable.
    from jsonschema.exceptions import RefResolutionError, ValidationError  # noqa

../../.local/share/virtualenvs/connexion-dummy-LMhlc2UK/lib64/python3.11/site-packages/connexion/validators/form_data.py:4
../../.local/share/virtualenvs/connexion-dummy-LMhlc2UK/lib64/python3.11/site-packages/connexion/validators/form_data.py:4
  /home/jvasik/.local/share/virtualenvs/connexion-dummy-LMhlc2UK/lib64/python3.11/site-packages/connexion/validators/form_data.py:4: DeprecationWarning: Accessing jsonschema.draft4_format_checker is deprecated and will be removed in a future release. Instead, use the FORMAT_CHECKER attribute on the corresponding Validator.
    from jsonschema import ValidationError, draft4_format_checker

../../.local/share/virtualenvs/connexion-dummy-LMhlc2UK/lib64/python3.11/site-packages/connexion/validators/json.py:6
../../.local/share/virtualenvs/connexion-dummy-LMhlc2UK/lib64/python3.11/site-packages/connexion/validators/json.py:6
  /home/jvasik/.local/share/virtualenvs/connexion-dummy-LMhlc2UK/lib64/python3.11/site-packages/connexion/validators/json.py:6: DeprecationWarning: Accessing jsonschema.draft4_format_checker is deprecated and will be removed in a future release. Instead, use the FORMAT_CHECKER attribute on the corresponding Validator.
    from jsonschema import Draft4Validator, ValidationError, draft4_format_checker

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
========================================================================== short test summary info ==========================================================================
FAILED test_app.py::test_route - assert 404 == 200
======================================================================= 1 failed, 7 warnings in 0.79s =======================================================================

Steps to reproduce
Worked with: pip install connexion==2.14.2 flask==2.2.5
Does not work: pip install connexion==3.0.6 flask==3.0.3
Execute the test: pytest test_app.py
api.yml:
openapi: "3.0.0"
info:
  description: "..."
  version: "1.0.0"
  title: "..."

servers:
  - url: "/{{path_prefix}}/v1"

paths:
  /:
    get:
      operationId: route.index
      responses:
        "200":
          description: "..."

route.py:
def index():
    return "Hello World!"

test_app.py:
import os
import json
from pathlib import Path

import yaml

from connexion import FlaskApp

def create_app(spec_path: str):
    with open(spec_path, "r") as f:
        specs = yaml.safe_load(f)

    openapi_args = {"path_prefix": 'test'}
    connexion_app = FlaskApp(__name__, specification_dir="", arguments=openapi_args)
    connexion_app.add_api(specs)
    return connexion_app.test_client()

def test_route():
    app_client = create_app((Path(os.getcwd()) / "api.yml").as_posix())
    response = app_client.get("test/v1/")
    assert response.status_code == 200
    assert json.loads(response.content) == "Hello World!"

To prove the failure of interpolation, change:
response = app_client.get("test/v1/")
to:
response = app_client.get("{path_prefix}/v1/")
The test will now pass.
Additional info:
Output of the commands:

python --version:
Python 3.11.8
pip show connexion | grep "^Version\:":
Version: 3.0.6

Fix: if pass the arguments for substitution by Jinja to the add_api method instead, it works.
Error:
connexion_app = FlaskApp(__name__, specification_dir="", arguments=openapi_args)
connexion_app.add_api(spec_path)

Success:
connexion_app = FlaskApp(__name__, specification_dir="")
connexion_app.add_api(spec_path, arguments=openapi_args)

Don't know if this is intentional, there's not a word on this in the 3.0 docs.


[WIP]  Improvements to the Documentation

[x] Add doc page about Minicli runtimes
[ ] Update docs about services to include new feature that allows callbacks as a service
[ ] Add tutorial about creating a GH Action

Merging this to unblock more docs work


Improve command line interface for monthly and annual data

This adds some DWIM-magic to improve behaviour when acquiring data from the monthly and annual time resolutions.
It is mostly about appropriate time range floor/ceil computations but also adds the respective names for DATE_FROM and DATE_TO columns within monthly and annual data as suggested within #59.

That looks good!

Thanks.

Would you mind adding a test for mktimerange and adjust the type hints for the dates (guess it should be str) and return type (Tuple[...])?

Sure, I've just adjusted the PR according to your suggestions.


Implement with viewpager

How to implement this with view pager?

Hi,
What do you mean exactly? StepperLayout uses a ViewPager internally

Hello,
I mean how to enable the swipe functionality? Because in the example app, swipe is disabled. Thanks!

OK, I get it now.
Short answer: swipe should not happen in this component.
Long answer:
Swiping is disabled in the library on purpose (https://github.com/stepstone-tech/android-material-stepper/blob/master/material-stepper/src/main/java/com/stepstone/stepper/internal/NonSwipeableViewPager.java).
The reason for this is because we want to do some verification before going to the next step. This is not explained explicitly in the documentation for this component (https://material.io/guidelines/components/steppers.html) however I have never seen such a behavior in this type of component.
Could you explain why would you like to be able to swipe across the steps?

I get it now why the library don't have a swipe functionality! thanks! great library also! 👍


[Azure DevOps]:

Inputs
Provide the following required inputs:
Organization: valet-testing
Project: build-playground
Available commands
The following commands can be executed by adding a comment to this issue:

/audit
/dry-run --pipeline-type pipeline|release --pipeline-id :pipeline-id
/migrate --pipeline-type pipeline|release --pipeline-id :pipeline-id --target-url :github-repository-url

Note: The pipeline-type option will default to pipeline if omitted. If any remaining options are missing, the command will not be successful.

/audit

/audit


docs(misc): document model export and claim/unclaim with body

updated the doc and added the event type origin

I updated line 43 and 43, to document the optional body
I also documented the model export

I fixed a few more typos and updated titles

👍

Just waiting for @mvonthron review if he dares :)

LGTM


[storage] Introduce Archive

@coderabbitai summary
TODO:

[x] Support Blob cloning for concurrent access: #153
[x] Add tests for storage::archive
[x] Add extensive documentation for both Journal and Archive (like p2p::authenticated): #160
[x] Switch to lazy index pruning to reduce RAM usage: #158
[x] Don't allow pruning backwards (can't prune at a lower index): #158
[x] Avoid reading entire journal on replay with skip reads: #159
[x] Handle byte corruption differently than truncation (don't try to fix): #159

@coderabbitai review


GitHub features we're missing

We just moved to GitHub for issue tracking to reduce friction for contributors. However, this choice was made at the expense of core contributors who could work more efficiently using a more powerful tool like Phabricator or Monorail.
Here's a list of important missing features we encountered:

Recursive issue dependencies. We used to use issue dependencies as a powerful project management tool in Phabricator:

It's possible to roughly approximate this by manually curating lists of blockers, but that's nowhere as useful. We could build a bot which edits issues to insert a similar tree of issues.

Issue resolutions (i.e. Wontfix, Resolved, Invalid...), rather than just "Closed".

Ability to watch labels to be notified only about specific components in our monorepo, rather than watching the whole repo. This isn't just impossible to do in the UI, you can't even filter email messages since they don't include the labels!

Hotlists for triage/grooming.

Marking issues as private (for security issues to be unrestricted eventually, or collaborating with customers - we could abuse Security Advisories for some of that, but those aren't usable for day-to-day issue tracking).

A Write role equivalent only for issue tracking. Some triage tasks like creating labels require Write permissions, which also allows creating releases/tags or merging PRs.

Go project discussion: https://github.com/golang/go/issues/39055

Complex queries for labels (like searching for "needs triage" OR "hotlist-123").

Custom linkify for issues (we currently have to write out monogon-dev/monogon#123 for fully qualified issue references, rather than being able to use a custom format like mng.link/123).

UI annoyances:

The issue list is hard to navigate with many issues and labels. Compare the Kubernetes issue tracker to the Gerrit one - the dense Monorail UI looks a bit unfriendly, but is much more useful for triage and backlog management (and note the grid view and how any label can be made into a column!)
No way to mass edit issues (both Monorail and Phabricator have nice batch edit UIs).

None of these features appear to be on the public roadmap: https://github.com/github/roadmap/issues
(we use Gerrit for code review, so this list doesn't include missing features related to that)
CC @q3k

I signed us up for the new issues beta: https://github.com/features/issues


feat(blobs): Allow blobs (which are by definition > 128KB)

📝 Summary
Currently we are blocking all tx over 128KB, which blocks blob txs.
⛱ Motivation and Context
Arbitrum might want to send blob txs to protect to skip the public mempool: https://flashbots.slack.com/archives/C0689NRL552/p1719345980974439
📚 References
Before:
> go run cmd/blob-sender/main.go -r https://rpc-holesky.flashbots.net --random-privkey --to 0x4bE0Cd2553356b4ABb8b6a1882325dAbC8d3013D -b "0xdeadbeef"

ERRO[0000] Error while sending transaction: invalid target for large tx  client="https://rpc-holesky.flashbots.net"

After:
> go run cmd/blob-sender/main.go -r https://rpc-holesky.flashbots.net --random-privkey --to 0x4bE0Cd2553356b4ABb8b6a1882325dAbC8d3013D -b "0xdeadbeef"
TX 1: 0x77c3481cbacdd3e463aa14e79153c2cf2f6bd66981fd99ab74b48298aca2d136

INFO [06-25|22:12:41.952] [sendRawTransaction] sending raw transaction uid=39a86cd3-12fa-4ae7-8096-ca266135ece6 rpc_method=eth_sendRawTransaction txHash=0x77c3481cbacdd3
INFO [06-25|22:12:41.957] [sendTxToRelay] sending transaction to relay uid=39a86cd3-12fa-4ae7-8096-ca266135ece6 rpc_method=eth_sendRawTransaction txHash=0x77c3481cbacdd3
INFO [06-25|22:12:42.015] [sendTxToRelay] sending transaction      uid=39a86cd3-12fa-4ae7-8096-ca266135ece6 rpc_method=eth_sendRawTransaction txHash=0x77c3481cbacdd3e463
INFO [06-25|22:12:43.498] [sendTxToRelay] Sent                     uid=39a86cd3-12fa-4ae7-8096-ca266135ece6 rpc_method=eth_sendRawTransaction txHash=0x77c3481cbacdd3e463
INFO [06-25|22:12:43.498] Request finished                         uid=39a86cd3-12fa-4ae7-8096-ca266135ece6 rpc_method=eth_sendRawTransaction duration=1.751

✅ I have run these commands

[ ] make lint
[ ] make test
[ ] go mod tidy

Closing this as it makes more sense to send blobs directly via the relay API instead of through the RPC endpoint.


gradlew fails with Java versions newer than 17

Expected Behavior
According to the DITA-OT prerequisites, Java versions newer than version 17 should be usable to build the DITA-OT:

To build and run DITA-OT, you’ll need Java Development Kit (JDK), version 17 or newer.

Actual Behavior
When OpenJDK 18 is used, the initial gradlew command fails with the following error:
* What went wrong:
Could not compile settings file '/home/chrispy/git/dita-ot/settings.gradle'.
> startup failed:
  General error during conversion: Unsupported class file major version 62

When OpenJDK 19 is used, a similar error occurs:
* What went wrong:
Could not compile settings file '/home/chrispy/git/dita-ot/settings.gradle'.
> startup failed:
  General error during conversion: Unsupported class file major version 63

Possible Solution
If this is a known limitation, the prerequisites statement should be updated.
Steps to Reproduce

Install OpenJDK 19:
sudo apt install openjdk-19-jdk
Select version 19:
sudo update-alternatives --config java
In a clone of the DITA-OT development repository, build the DITA-OT:
rm -rf ~/.gradle && git clean -fdx && ./gradlew

Copy of the error message, log file or stack trace
$ ./gradlew
Downloading https://services.gradle.org/distributions/gradle-7.2-bin.zip
..........10%...........20%...........30%...........40%...........50%...........60%...........70%...........80%...........90%...........100%

Welcome to Gradle 7.2!

Here are the highlights of this release:
 - Toolchain support for Scala
 - More cache hits when Java source files have platform-specific line endings
 - More resilient remote HTTP build cache behavior

For more details see https://docs.gradle.org/7.2/release-notes.html

Starting a Gradle Daemon, 3 stopped Daemons could not be reused, use --status for details

FAILURE: Build failed with an exception.

* Where:
Settings file '/home/chrispy/git/dita-ot/settings.gradle'

* What went wrong:
Could not compile settings file '/home/chrispy/git/dita-ot/settings.gradle'.
> startup failed:
  General error during conversion: Unsupported class file major version 63

  java.lang.IllegalArgumentException: Unsupported class file major version 63
        at groovyjarjarasm.asm.ClassReader.<init>(ClassReader.java:196)
        at groovyjarjarasm.asm.ClassReader.<init>(ClassReader.java:177)
        at groovyjarjarasm.asm.ClassReader.<init>(ClassReader.java:163)
        at groovyjarjarasm.asm.ClassReader.<init>(ClassReader.java:284)
...omitted...

Environment

DITA-OT version: develop branch
Operating system and version: Ubuntu 22.04 LTS

what happens when you try a more recent version of Gradle, such as 8.x?

@kirkilj - I'm not sure how I would use a different version than what's included with the DITA-OT repository. I usually just clone it and run the ./gradlew wrapper script in the root directory.

Can't reproduce this on a Mac.

Reproduced with Docker image docker:22.04.

@infotexture are you able to test if docs build will work with Gradle 7.6.1. We could update Gradle in both core and docs repos for 4.1 release.

✅ @jelovirt Yes, bumped both wrappers locally and tested docs distribution build.
Still works fine with OpenJDK 17, so no regressions. Didn't test with Java 18 or 19, but since Gradle 7.6 adds support for Java 19, we should be fine.
I'll submit PRs with the wrapper changes for both repos.


YARN-10752. Shaded guava not found when compiling with profile hbase2.0.

Fix a classpath typo.
Verified by running mvn clean install -Dhbase.profile=2.0 -DskipTests -DskipITs

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
21m  3s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
pathlen
0m  0s
/results-pathlen.txt
The patch appears to contain 1 files with names longer than 240

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ branch-3.3 Compile Tests _

+1 :green_heart:
mvninstall
33m 36s

branch-3.3 passed

-1 :x:
compile
1m 45s
/branch-compile-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2.txt
hadoop-yarn-server-timelineservice-hbase-server-2 in branch-3.3 failed.

+1 :green_heart:
checkstyle
0m 24s

branch-3.3 passed

-1 :x:
mvnsite
0m 25s
/branch-mvnsite-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2.txt
hadoop-yarn-server-timelineservice-hbase-server-2 in branch-3.3 failed.

+1 :green_heart:
javadoc
0m 26s

branch-3.3 passed

-1 :x:
spotbugs
0m 25s
/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2.txt
hadoop-yarn-server-timelineservice-hbase-server-2 in branch-3.3 failed.

+1 :green_heart:
shadedclient
16m 47s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 28s

the patch passed

+1 :green_heart:
compile
0m 17s

the patch passed

-1 :x:
javac
0m 17s
/results-compile-javac-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2.txt
hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2 generated 1 new + 13 unchanged - 4 fixed = 14 total (was 17)

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 13s

the patch passed

+1 :green_heart:
mvnsite
0m 20s

the patch passed

+1 :green_heart:
javadoc
0m 17s

hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2 generated 0 new + 2 unchanged - 2 fixed = 2 total (was 4)

+1 :green_heart:
spotbugs
0m 42s

the patch passed

+1 :green_heart:
shadedclient
15m 45s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
0m 21s

hadoop-yarn-server-timelineservice-hbase-server-2 in the patch passed.

+1 :green_heart:
asflicense
0m 33s

The patch does not generate ASF License warnings.

93m 54s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2957/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2957

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux a74676a45fe7 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.3 / 57db7841287e51013f40c62f91b9df057c3e0199

Default Java
Private Build-1.8.0_282-8u282-b08-0ubuntu1~18.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2957/1/testReport/

Max. process+thread count
746 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-timelineservice-hbase/hadoop-yarn-server-timelineservice-hbase-server/hadoop-yarn-server-timelineservice-hbase-server-2 U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-timelineservice-hbase/hadoop-yarn-server-timelineservice-hbase-server/hadoop-yarn-server-timelineservice-hbase-server-2

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2957/1/console

versions
git=2.17.1 maven=3.6.0

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Merged. Thank you @jojochuang, @qizhu-lucas, and @ayushtkn.


[AVM Module Issue]: I am unable to configure the options for Standby Agent mode

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
Bug
(Optional) Module Version
main
(Optional) Correlation Id
No response
Description
When I create the managed DevOps pool with this code, everything is created perfectly, but the Standby Agent Mode remains set to OFF. I am unable to switch it to 'Manual' in order to finely configure the pool's behavior:
module "managed_devops_pool" {
  source = "github.com/Azure/terraform-azurerm-avm-res-devopsinfrastructure-pool?ref=main"

  resource_group_name                      = module.default_resource_group.resource_group_name
  location                                 = module.default_resource_group.location
  name                                     = "my-mdp"
  dev_center_project_resource_id           = azurerm_dev_center_project.main.id
  version_control_system_organization_name = var.version_control_system_organization_name
  version_control_system_project_names     = var.version_control_system_project_names

  agent_profile_kind                       = "Stateful"
  maximum_concurrency                      = 4
  agent_profile_resource_predictions_manual = {
    time_zone = "Romance Standard Time"
    days_data = [
      {
        "08:00:00" = 1
        "17:00:00" = 0
      }
    ]
  }

  fabric_profile_sku_name                    = "Standard_DS1_v2"
  fabric_profile_os_disk_storage_account_type = "Standard"
  fabric_profile_images                       = [
    {
      aliases              = ["ubuntu-22.04/latest"]
      well_known_image_name = "ubuntu-22.04/latest"
    }
  ]
}

Thank you for your help.

Per our email thread. You appear to be missing the agent_profile_resource_prediction_profile = "Manual" variable per the example here: https://github.com/Azure/terraform-azurerm-avm-res-devopsinfrastructure-pool/blob/main/examples/standby-agent-manual/main.tf


[SPARK-25903][Core] TimerTask should be synchronized on ContextBarrierState

What changes were proposed in this pull request?

BarrierCoordinator sets up a TimerTask for a round of global sync. Currently the run method is synchronized on the created TimerTask. But to be synchronized with handleRequest, it should be synchronized on the ContextBarrierState object, not TimerTask object.
Why are the changes needed?

ContextBarrierState.handleRequest and TimerTask.run both access the internal status of a ContextBarrierState object. If TimerTask doesn't be synchronized on the same ContextBarrierState object, when the timer task is triggered, handleRequest still accepts new request and modify  requesters field in the ContextBarrierState object. It makes the behavior inconsistency.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Test locally

cc @ jiangxb1987 @cloud-fan

Test build #111191 has started for PR 25897 at commit a5b5f70.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16270/
Test PASSed.

Test build #111191 has finished for PR 25897 at commit a5b5f70.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111191/
Test FAILed.

retest this please.

Test build #111197 has started for PR 25897 at commit a5b5f70.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16276/
Test PASSed.

Test build #111198 has started for PR 25897 at commit a5b5f70.

Test build #111198 has finished for PR 25897 at commit a5b5f70.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111198/
Test FAILed.

Test build #111197 has finished for PR 25897 at commit a5b5f70.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111197/
Test PASSed.

cc @jiangxb1987

good catch! LGTM. Why do we need [test] in the title?

Because SPARK-25903 is about one flaky test in BarrierTaskContextSuite. I found this issue when looking into the reason. Should I create another JIRA?

Sure. Removed. Thanks.

thanks, merging to master/2.4!

Thank you, @viirya and @cloud-fan !


⚠ Secret Website 3 has degraded performance

In 35b5a45, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2514 ms

Resolved: Secret Website 3 performance has improved in 966f5de.


Sorting names breaks something

On myndzi's fork, sorting the user names freezes the entire client

chunk-vendors.js:7 TypeError: Cannot read property 'key' of undefined
    at ri (chunk-vendors.js:7)
    at S (chunk-vendors.js:7)
    at j (chunk-vendors.js:7)
    at S (chunk-vendors.js:7)
    at j (chunk-vendors.js:7)
    at S (chunk-vendors.js:7)
    at j (chunk-vendors.js:7)
    at S (chunk-vendors.js:7)
    at j (chunk-vendors.js:7)
    at S (chunk-vendors.js:7)

may not be an issue


Update CLI extensions available doc

Update CLI extensions available doc.
Triggered by Azure/azure-cli-extensions - TRAVIS_BUILD_ID=462230570
https://github.com/Azure/azure-cli-extensions/commit/3a193fd9da3d0162602d2f900826b826115ae827

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-extensions-list.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Error when running si update

What happened?
si update failed to perform an update due to an unexpected error:
> si update
System Initiative Launcher is in "local" mode

Container update found for systeminit/postgres
Container update found for systeminit/otelcol
Container update found for systeminit/nats
Container update found for systeminit/jaeger
Container update found for systeminit/veritech
Container update found for systeminit/sdf
Container update found for systeminit/pinga
Container update found for systeminit/council
Container update found for systeminit/web
Launcher update found: from 20230816.151117.0-sha.fae4e1c44 to bin/si/binary/20230818.205543.0-sha.9629dfb23

Updating the containers will destroy your data!
> Are you sure you want to update the binary and the containers listed above? Yes
Stopping Container: local-web-1
Stopping Container: local-sdf-1
Stopping Container: local-pinga-1
Stopping Container: local-veritech-1
Stopping Container: local-council-1
Stopping Container: local-otelcol-1
Stopping Container: local-nats-1
Stopping Container: local-postgres-1
Stopping Container: local-jaeger-1
All system components stopped...

The credentials are never sent back to System Initiative and can be inspected at the location:
/home/salmon/.local/share/SI/si_credentials.toml

Checking that the system is able to interact with the docker engine to control System Initiative...
┌──────────────────────┬────────────┐
│ Docker Engine Active ┆     ✅     │
└──────────────────────┴────────────┘
All containers downloaded

Starting systeminit/jaeger:stable as local-jaeger-1
Starting systeminit/postgres:stable as local-postgres-1
Starting systeminit/nats:stable as local-nats-1
Starting systeminit/otelcol:stable as local-otelcol-1
Starting systeminit/council:stable as local-council-1
Starting systeminit/veritech:stable as local-veritech-1
Starting systeminit/pinga:stable as local-pinga-1
Starting systeminit/sdf:stable as local-sdf-1
Starting systeminit/web:stable as local-web-1
All system components running... System Initiative is alive!

You can now use the `si launch` command to open the System Initiative web portal
Downloading new binary
Overwriting current binary
Error:
   0: io: Invalid cross-device link (os error 18)
   1: Invalid cross-device link (os error 18)

Location:
   bin/si/src/main.rs:128

  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ BACKTRACE ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
                                ⋮ 3 frames hidden ⋮
   4: si::main::{{closure}}::hc959437c33741098
      at <unknown source file>:<unknown line>
   5: tokio::runtime::park::CachedParkThread::block_on::h8657001296f26bdc
      at <unknown source file>:<unknown line>
   6: tokio::runtime::scheduler::multi_thread::MultiThread::block_on::hc81638d8fa067ff8
      at <unknown source file>:<unknown line>
   7: tokio::runtime::runtime::Runtime::block_on::haa13a5c81da961a0
      at <unknown source file>:<unknown line>
   8: si::main::h9c3529cb47f1a178
      at <unknown source file>:<unknown line>
   9: std::sys_common::backtrace::__rust_begin_short_backtrace::hb6319b36f3c9c391
      at <unknown source file>:<unknown line>
  10: std::rt::lang_start::{{closure}}::hadca2ee8395f889c
      at <unknown source file>:<unknown line>
  11: std::rt::lang_start_internal::h76f3e81e6b8f13f9
      at <unknown source file>:<unknown line>
  12: main<unknown>
      at <unknown source file>:<unknown line>
  13: __libc_start_call_main<unknown>
      at <unknown source file>:<unknown line>
  14<EMAIL_ADDRESS>      at <unknown source file>:<unknown line>
  15: _start<unknown>
      at <unknown source file>:<unknown line>

How can we reproduce this?
All I did was run si stop and si update.
Operating system
Linux
Architecture
x86_64 / amd64
What browsers are you seeing the problem on?
Firefox
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Additional context
No response

Thanks for the report! After double checking our code, I see that we aren't performing a final atomic move from within the same parent directory of the original si binary. If I was to guess, I would wager that your temp directory is a separate volume mount from that of si's destination.
I'll see about fixing this up and link a PR

Unfortunately, because the updated "update" logic is in a new binary release, you'll have to re-install si with:
curl -sSfL https://auth.systeminit.com/install.sh | sh

or manually from the download page.
Hope this does the trick, happy Friday!

Done. I'd like to test it, but given it's an installer bug, I'll have to wait for the next release. Thanks!

Yeah...I was in my terminal thinking the exact same thing!

As one last data point, I did manage to trigger the same error using an older version of the launcher, installed in $HOME/.local/bin/si which is a different fs mount than my /tmp fs

I think you are right:
╰─ df -h
Filesystem             Size  Used Avail Use% Mounted on
devtmpfs               4.0M     0  4.0M   0% /dev
tmpfs                   32G   27M   32G   1% /dev/shm
tmpfs                   13G  2.4M   13G   1% /run
/dev/dm-0              1.9T  870G  991G  47% /
/dev/dm-0              1.9T  870G  991G  47% /home
/dev/nvme1n1p2         974M  324M  583M  36% /boot
tmpfs                   32G   17M   32G   1% /tmp
/dev/nvme1n1p1         599M   43M  557M   8% /boot/efi
tmpfs                  6.3G  2.4M  6.3G   1% /run/user/1000


[release-1.13] Revert selecting virtual services

Manual cherrypick required.
#37813 failed to apply on top of branch "release-1.13":
Applying: Revert "Added selecting virtualservice (#33764)"
Applying: Expand tests
Using index info to reconstruct a base tree...
M	pilot/pkg/networking/core/v1alpha3/sidecar_simulation_test.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/networking/core/v1alpha3/sidecar_simulation_test.go
CONFLICT (content): Merge conflict in pilot/pkg/networking/core/v1alpha3/sidecar_simulation_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0002 Expand tests
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

Seeing issues with virtual services not working properly with http match and added CORS policy in v1.13.x, but works fine in 1.12.5 - unsure if related but wanted to document somewhere.
ex:
http:
    - match:
        - uri:
            prefix: /service-name
      route:
        - destination:
            host: service-name
            port:
              number: 8080
      corsPolicy:
        allowOrigins:
          - exact: https://service.io/
          - exact: http://localhost:3001/
        allowHeaders:
          - authorization
          - content-type
          - sentry-trace
        allowMethods:
          - GET
          - POST
          - PUT
          - DELETE
          - OPTIONS


Sending an ArraySegment object

If we are populating a very large buffer and want to send that via a websocket;
we need that buffer to be of "right" size. To doi that, we need to either create a new byte[] for each message, or truncate the large buffer to the message size. If we do the truncration thing for a byte[] object, what we get is an ArraySegment<byte> object, not a new byte[] object. So, it should be possible to pass an ArraySegment instead of just a byte array for binary data.

:white_check_mark: Build Fleck 0.0.61-ci completed (commit https://github.com/statianzo/Fleck/commit/ba7ddb68ff by @kuzux)

:white_check_mark: Build Fleck 0.0.61-ci completed (commit https://github.com/statianzo/Fleck/commit/ba7ddb68ff by @kuzux)

Could you provide context beyond the new code?

Here's the case where that would be useful:
In a codebase using Fleck, I was creating a buffer, filling the buffer in native code and then sending that buffer. Something like
byte[] buf  = new byte[MaxSize];
while(some_condition) {
    // ...
    int bufsize = fillBuffer(buf); // native method called with p/invoke
    byte[] sendBuf = new byte[bufsize];
    buf.CopyTo(sendBuf, 0);
    conn.Send(sendBuf);
    // ...
}

So, I want to send just a portion of that buffer (the part filled by the native code). To extract that part as a new byte array, the only "solution" I was able to find (to creating a smaller byte array from a larger one) was allocating a new buffer and copying the data to that (Causes latency spikes when working with buffers of large data).
The way to do that is, I believe, using an ArraySegment<T> object, which is just a separate "view" to the existing underlying array.

And you’ve found a measurable improvement with this code?
I ask because it still writes to a memory stream that gets written to an array.
To avoid allocation of the intermediate array, I believe you’d need to write directly to the socket.

About measurable improvement: Well there was measurable (in fact noticeable) improvement when I eliminated other allocate & copy operations elsewhere in the code. So I decided to go on an allocation-hunting spree, essentially. This was the low-hanging fruit.
I know about the memory stream allocation, but eliminating that as well would probably take way too much modification in the source, which I wasn't comfortable doing. After all, having 2 allocation & copy operations made a significant improvement over having 3. Having 1 would still be better than having 2 of them (Even if it's not as good as having 0 :D)

:white_check_mark: Build Fleck 0.0.62-ci completed (commit https://github.com/statianzo/Fleck/commit/4d644b9f28 by @kuzux)

:white_check_mark: Build Fleck 0.0.62-ci completed (commit https://github.com/statianzo/Fleck/commit/4d644b9f28 by @kuzux)

+1 for this, definitely a needed feature

@kuzux Don't forget to add the Send method to the IWebSocketConnection!


Search file on API

Hi there thank for open this section, I'm really confused using vuex recently. i need more know about vuex.
My problem is i want search a data using API, example im using API from TMDB, let me know how to use vuex using query.
thank you

That isn't really a Vuex thing and besides, you might not need to use Vuex except when really necessary.
It really depends on how you want the search to be executed. The simplest approach would be to make the request as soon as the enter key or the submit button is pressed.
The other approach would be to fetch on almost every keystroke (that is, throttle key inputs).
So, if you intend to make use of Vuex, you can have an action to make the API request and save the search result in the state object then you can populate the page with the search results from the state.
If you need to learn to use Vuex well (you probably should learn to use it well), you can check out their official guide.

That isn't really a Vuex thing and besides, you might not need to use Vuex except when really necessary.
It really depends on how you want the search to be executed. The simplest approach would be to make the request as soon as the enter key or the submit button is pressed.
The other approach would be to fetch on almost every keystroke (that is, throttle key inputs).
So, if you intend to make use of Vuex, you can have an action to make the API request and save the search result in the state object then you can populate the page with the search results from the state.
If you need to learn to use Vuex well (you probably should learn to use it well), you can check out their official guide.


Instance class change to F4_HIGHMEM

Strangely, it seems according to the node docs that this is F4_1G in the YAML.
Also had to opt yaml files out of prettier.
Test Plan

Deploy this commit SHA (or one including it)
Fire a request at it (see our Confluence docs)
See that the instance spins up with the higher memory limit in the stackdriver logs

I think this is fine, but we'll want to monitor this for a few days once it goes live to make sure we're not increasing costs disproportionately by doing this.

Yup, I expected that would be what we do. If it turns out costly, we can do a new PR to put it back to F4 and we'll ensure file caching works differently for the gateway.


Fix BL scraper

New column in the table

Is Geheilte geschätzt recovered? I am not sure.

Yes it's recovered. It's an assumption, but I guess it's like that in all cantons, no one really measures the recovered cases. There is a description of this concept from BL.

Please take another look.


Update Docusaurus to 1.9.0

Summary

Update Docusaurus to 1.9.0
Enable collapsible side bar
Clean up some hrefs that were pointing to .html extensions. This should have been done when the cleanUrl option was enabled but it still works because we generate both /path/index.html and /path.html in v1. This will break in v2 so better change it now.
Enable the scroll-to-top button

Whaaaat, thanks @yangshun!

Thanks so much @yangshun, this is so much better :)
I'm wondering if there is a way to split up the Docs and API pages into two separate top-level entry points both with their own sidebars? For Docs, it should only have the Guides sections and for API it should have the Components and API sections. Is this possible?

Thank you, @yangshun!

Welcome! I'm always available to answer any Docusaurus-specific questions. Cheers! 🎉

I just noticed that this also collapses the sidebar in the blog section. Is it possible to always expand it there?

This is a limitation of the collapsible sidebar. I'll modify Docusaurus core to not collapse for blogs.

PR for the fix in https://github.com/facebook/Docusaurus/pull/1519. Will update RN website when that PR is merged.


lets add newline so its more obvious from tools like fabric8-maven-plugin whats happening

[merge]


DAW automation lanes display

In the LV2, the automation lane options appear to be stuck on the initial "Galactic" setting.
This only appears to be a problem with the display, as the automation itself still works -- just need to visually line up which "knob" you're on.  :-)

The vst3 clap and au do the right thing in the hosts I’ve tried. Can you try any of those formats? I think ardour supports vst3 yeah?
if so I can flag this as a problem with juce and maybe they can fix it

Oh right LV2 has that weird thing where it puts the parameter names in the ttl file at build time.
I wonder if LV2 just can't handle engine morphing track names.
This implementation in the JUCE LV2 wrapper
 void audioProcessorChanged (AudioProcessor*, const ChangeDetails&) override {}

is very different than the VST3 and CLAP ones though. So I think this is either a JUCE shortcoming or an LV2 format shortcoming.
Curious if the VST3 does the right thing in Ardour?

Ah, yep!  VST3 version works as expected.  :+1:   I'll chalk that up to LV2 limitations and go that route.

You may want to let the ardour team know. If they can tell you what to put inside that block perhaps the Juce team would be interested! It may not be an LV2 limitation as much as a JUCE implementation of LV2 incompleteness. But it might be an LV2 limitation!
(I have no idea how LV2 works really so wouldn't know where to start).
I'm going to close this since there's nothing I can change to fix it. But feel free to point folks here if they have ideas.


Clear basic auth credentials

It seems that Just records credentials with ".ForSession" persistence.
Is there a way I can clear these? Ideally, can I make it use "NSURLCredentialsPersistence.None"?

@igorpan this is possible now in version 0.4.4.

@dduan That is awesome! Great job!


CLAS-INTF refactor changed by/change since

Hello,
Continuation on the issue #21. Please note that I'm only refactoring when we have cases like IF clas, do that, IF intf, do else.
@nununo If you have the availability, could you please test this? I adapted some already existing calls so that it is compatible with 7.02. Testing: Just pull a repository with both classes and interfaces.
Still missing:

delete
serialize

cool, thanks

@EduardoCopat are you in Karlsruhe this week, found your name in the app? If so lets meet up on Thursday and say hi :smile:

@larshp I already favorited your session, but I was going to make a surprise hehe.


XSS Injection problem

Please add quote string with slashes and protection against XSS attacks.
You can reproduce that critical issue but storing in the edit form simple XSS:

We've got some escaped code from ours WP Devs in the DB and it ruin JS scripts:

you can protect it by using a mutator within your model. not sure we are going to do this right now.

I am agree with @MaxKosenko there is xss vulnerability here.


Issue #171

Fixing newsletter form alignment

Coverage remained the same at 16.59% when pulling ccf159047c16ec499072a0b6b7199dbe3d992a8e on marceloperini:master into 830eec573b4b971f16c3334cf8c60c3b2b26281a on gorails:master.


audiofile: update to work on osx

Add missing OSX frameworks for audiofile so it can build.
Dependency of youtube-dl as an example.

@pikajude Can you review?

Looks like @ocharles basically committed this in https://github.com/NixOS/nixpkgs/commit/981e8d55824717960809179a47bfeb81c8c1ee70
Closing this out then.


Remove private default parameters.

There are several constructors in the Flutter SDK with private defaults.
I'm working on a visual Widget builder that is scraping the Flutter SDK and needs to have access to every default value. Due to several defaults being private I'm not able to use those widgets without unnecessary edge-case handling.

_DefaultHeroTag at material/floating_action_button.dart
_defaultHeroTag (_HeroTag) at cupertino/nav_bar.dart.
ForcePressGestureRecognizer._inverseLerp at gestures/force_press.dart used by ForcePressGestureRecognizer for interpolation
_kDefaultSemanticIndexCallback at widgets/sliver.dart used by SliverChildBuilderDelegate & SliverChildListDelegate & SliverChildListDelegate.fixed for semanticIndexCallback

I see two possible solutions:
1: making each default public.
2: hiding the default so that a null would make the widget take on the default.
Edit:

Rephrased to exclude defaults that can be evaluated to constants in a trivial way and 2 more private defaults added.

I'm closing this issue as stale


Output isn't preserved

I have the following code:
new Listr ([
  {
    title: 'Whatever',
    task: async ( ctx, task ) => task.output = 'My Output'
  }
], { collapse: false, clearOutput: false }).run ();

My understanding is that this way all outputs should be preserved, but this isn't the case, when the task ends the output isn't preserved.
The problem seems to be in this line:
https://github.com/SamVerschueren/listr-update-renderer/blob/9d3a4e39deee2d006f7420d7ff808fcb18fcc782/index.js#L21
Even though there's an output, and I'm passing collapse: false, once the task is completed the following evaluates to false:
(task.isPending() || task.isSkipped() || task.hasFailed())
I think this bit of code should be deleted, and replaced with something like:
(task.isPending() || task.isSkipped() || task.hasFailed()) || !options.collapse.
What do you think?

I think this might be the root issue to a lot of the collapse bugs.  Is there any update on this?

Ran into this today.  Locally made the change in OP's commit and it seems to behave as expected now.  Would really like to see an official release to fix this.


Start using sdc submit button in the demo app.

Describe the bug
SDC module submit button is visible in demo app, but it does not perform submit operation.
To submit the form user has to click on submit menu which is currently present in the app menu bar.
Component
reference app
To Reproduce
Steps to reproduce the behavior:

Go to the demo app
Click on + icon to add patient
Now, it shows submit button at the bottom and in the app bar.

Expected behavior
Form should be submitted on click event of sdc submit button.
Remove app bar submit menu.
Screenshots
If applicable, add screenshots to help explain your problem.

Smartphone (please complete the following information):

Device:  Pixel4a emulator
Android version: Android api 30
Build number: na
Version na

Additional context
Add any other context about the problem here.
Would you like to work on the issue?
Please state if this issue should be assigned to you or who you think could help to solve this issue.

@aditya-07
Can you please add the project name? (I don't have permission to add it.)

@santosh-pingle added the project name!


ReactiveStatusHandler is overrided in one Configuration but for all clients

1st client configuration
    @Bean
    public ErrorDecoder errorDecoder() {
        return new MyErrorDecoder1();
    }

    @Bean
    public ReactiveStatusHandler reactiveStatusHandler(ErrorDecoder errorDecoder) {
        return ReactiveStatusHandlers.errorDecoder(errorDecoder);
    }

2st client configuration overrides only ErrorDecoder
    @Bean
    public MyErrorDecoder2 errorDecoder(ObjectMapper objectMapper) {
        return new MyErrorDecoder2(objectMapper);
    }

When 1st and 2nd clients are used together 2nd client uses MyErrorDecoder1 but I expect it to use MyErrorDecoder2with default ReactiveStatusHandler.
Both clients contain @ReactiveFeignClient with appropriate 'configuration'.
)

Resolved by split AutoConfiguration and Configuration for client.


Integrates the end-to-end testing with the distro launcher

This seems bigger than it really is. Let's go by parts:

ubuntu-desktop-installer submodule is updated to latest main so we can incorporate the automation recently merged there. See PR https://github.com/canonical/ubuntu-desktop-installer/pull/1173
A small change in meta\UbuntuPreview\Preprocessor.props allows defining OOBE_E2E_TESTING through msbuild CLI (by setting the property OOBE_E2E_TESTING to any value other than empty - /p:OOBE_E2E_TESTING=1 for example). That will only make sense in the context of running the end-to-end tests.
AppConfig struct and a function to provide a global static constexpr instance carries the pieces of data that change between production and e2e testing.
Fixes and enhancements on the launcher code revealed by the e2e testing experimentation. (2ceb9e3b640bbe641e8c3f8f8400f62a910515ef, a3e0f476881acfca9293b9a631449d5f740d3e41 and 57f465b0cbbb1b1eddc84456916cb585bc0f7754)

We are almost done. With those changes in place it's possible to generate an appx ready for end-to-end testing by following the recipe below on a Developer Powershell:

 go build .\wsl-builder\prepare-build
 .\prepare-build Ubuntu-Preview ...
cp e2e\ui-driver\ui-driver.props DistroLauncher-Appx\
# Patching the solution to not complain about lack of ARM64 artifacts.
Set-Content -Path "DistroLauncher.sln" -Value (get-content -Path "DistroLauncher.sln" | Select-String -Pattern 'ARM64' -NotMatch)
msbuild .\DistroLauncher.sln /t:Build /m /nr:false /p:Configuration=Debug /p:AppxBundle=Always /p:AppxBundlePlatforms="x64" /p:UapAppxPackageBuildMode=SideloadOnly -verbosity:normal /p:OOBE_E2E_TESTING=1
.\AppPackages\Ubuntu\Ubuntu_<version>_Debug_Test\Install.ps1 -Force
$env:LAUNCHER_REPO_ROOT=$(pwd).Path
cd e2e
go test .\test-runner --distro-name Ubuntu-Preview --launcher-name ubuntupreview.exe

This will soon be turned into a script, on another PR, once experimentation with the parameters get more mature.

Jammy was moved to the new pipeline. I didn't check yet the results, but it is the reason why the CI is failing to build a package.


Rename whitelist to allowList in Microsoft.Azure.Management.SecurityCenter

https://github.com/Azure/azure-sdk-for-net/tree/master/sdk/securitycenter/Microsoft.Azure.Management.SecurityCenter/src/Generated/AdaptiveApplicationControlsOperations.cs
https://github.com/Azure/azure-sdk-for-net/tree/master/sdk/securitycenter/Microsoft.Azure.Management.SecurityCenter/src/Generated/AdaptiveApplicationControlsOperationsExtensions.cs
https://github.com/Azure/azure-sdk-for-net/tree/master/sdk/securitycenter/Microsoft.Azure.Management.SecurityCenter/src/Generated/IAdaptiveApplicationControlsOperations.cs
https://github.com/Azure/azure-sdk-for-net/tree/master/sdk/securitycenter/Microsoft.Azure.Management.SecurityCenter/src/Generated/Models/AppWhitelistingGroup.cs
https://github.com/Azure/azure-sdk-for-net/tree/master/sdk/securitycenter/Microsoft.Azure.Management.SecurityCenter/src/Generated/Models/AppWhitelistingGroups.cs
https://github.com/Azure/azure-sdk-for-net/tree/master/sdk/securitycenter/Microsoft.Azure.Management.SecurityCenter/src/Generated/Models/AppWhitelistingIssueSummary.cs
https://github.com/Azure/azure-sdk-for-net/tree/master/sdk/securitycenter/Microsoft.Azure.Management.SecurityCenter/src/Generated/Models/PathRecommendation.cs
https://github.com/Azure/azure-sdk-for-net/tree/master/sdk/securitycenter/Microsoft.Azure.Management.SecurityCenter/tests/AdaptiveApplicationControls/AdaptiveApplicationControlsTests.cs

Not sure if "Security" is the right tag here... also + @taklei just in case. Thanks

Thank you Mariana. We are working to fix it

This appears to have been completed.
https://github.com/Azure/azure-sdk-for-net/tree/master/sdk/securitycenter/Microsoft.Azure.Management.SecurityCenter/src/Generated/Models/AppWhitelistingGroup.cs  returns 404, the updated content is under https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/securitycenter/Microsoft.Azure.Management.SecurityCenter/src/Generated/Models/AdaptiveApplicationControlGroup.cs


CrashLoopBackoff error when I use my own artifactory image and deployment

I've beem trying to configure artifactory-oss so I can set admin password upon creating the container and linking to google binary storage. I'm using k8s so I used kompose to translate between docker-compose and kubernetes deployment. I have my own artifactory image on gcp and that is the one I use while creating the pod.
The problem that i'm having is that all my container works, but the artifactory. It keeps giving me de crashLoopBackof error.  This is the deployment:
apiVersion: v1
kind: Pod
metadata:
creationTimestamp: null
labels:
io.kompose.service: artifactory
name: artifactory
spec:
containers:

env:

name: DB_PASSWORD
value: password
name: DB_TYPE
value: postgresql
name: DB_USER
value: artifactory
image: gcr.io/emb-infra/artifactory-teste:latest
name: artifactory
ports:
containerPort: 8081
resources: {}
restartPolicy: Always
status: {}

apiVersion: v1
kind: Service
metadata:
annotations:
kompose.cmd: kompose convert
kompose.version: 1.17.0 (a74acad)
creationTimestamp: null
labels:
io.kompose.service: artifactory
name: artifactory
spec:
ports:

name: "8081"
port: 8081
targetPort: 8081
selector:
io.kompose.service: artifactory
status:
loadBalancer: {}

apiVersion: v1
kind: Pod
metadata:
creationTimestamp: null
labels:
io.kompose.service: nginx
name: nginx
spec:
containers:

env:

name: ART_BASE_URL
value: http://artifactory:8081/artifactory
name: SSL
value: "true"
image: docker.bintray.io/jfrog/nginx-artifactory-pro:5.4.4
name: nginx
ports:
containerPort: 80
containerPort: 443
resources: {}
restartPolicy: OnFailure
status: {}

apiVersion: v1
kind: Service
metadata:
annotations:
kompose.cmd: kompose convert
kompose.version: 1.17.0 (a74acad)
creationTimestamp: null
labels:
io.kompose.service: nginx
name: nginx
spec:
ports:

name: "80"
port: 80
targetPort: 80
name: "443"
port: 443
targetPort: 443
selector:
io.kompose.service: nginx
status:
loadBalancer: {}

apiVersion: v1
kind: Pod
metadata:
creationTimestamp: null
labels:
io.kompose.service: postgresql
name: postgresql
spec:
containers:

env:

name: POSTGRES_DB
value: artifactory
name: POSTGRES_PASSWORD
value: password
name: POSTGRES_USER
value: artifactory
image: docker.bintray.io/postgres:9.6.11
name: postgresql
ports:
containerPort: 5432
resources: {}
restartPolicy: OnFailure
status: {}

apiVersion: v1
kind: Service
metadata:
annotations:
kompose.cmd: kompose convert
kompose.version: 1.17.0 (a74acad)
creationTimestamp: null
labels:
io.kompose.service: postgresql
name: postgresql
spec:
ports:

name: "5432"
port: 5432
targetPort: 5432
selector:
io.kompose.service: postgresql
status:
loadBalancer: {}

There are no logs generated. The only info I have is the pod describe:
Name:               artifactory
Namespace:          default
Priority:           0
PriorityClassName:
Node:               gke-kube-phpipam-pool-1-6b696b70-jr2z/<IP_ADDRESS>
Start Time:         Fri, 07 Jun 2019 14:16:48 -0300
Labels:             io.kompose.service=artifactory
Annotations:        kubectl.kubernetes.io/last-applied-configuration:
{"apiVersion":"v1","kind":"Pod","metadata":{"annotations":{},"creationTimestamp":null,"labels":{"io.kompose.service":"artifactory"},"name"...
kubernetes.io/limit-ranger: LimitRanger plugin set: cpu request for container artifactory
Status:             Running
IP:                 <IP_ADDRESS>
Containers:
artifactory:
Container ID:   docker://36b1fa43367a4e2858b55a84ae705d6e0f52542984784a3a809b52566363565d
Image:          gcr.io/emb-infra/artifactory-teste:latest
Image ID:       docker-pullable://gcr.io/emb-infra/artifactory-teste@sha256:40d354b007cc34755c19a5f645c83b1e5c6b38abc2ebceb5eecdea18d4427a30
Port:           8081/TCP
Host Port:      0/TCP
State:          Waiting
Reason:       CrashLoopBackOff
Last State:     Terminated
Reason:       Completed
Exit Code:    0
Started:      Fri, 07 Jun 2019 14:17:31 -0300
Finished:     Fri, 07 Jun 2019 14:17:31 -0300
Ready:          False
Restart Count:  3
Requests:
cpu:  100m
Environment:
DB_PASSWORD:  password
DB_TYPE:      postgresql
DB_USER:      artifactory
Mounts:
/var/run/secrets/kubernetes.io/serviceaccount from default-token-5dn7s (ro)
Conditions:
Type              Status
Initialized       True
Ready             False
ContainersReady   False
PodScheduled      True
Volumes:
default-token-5dn7s:
Type:        Secret (a volume populated by a Secret)
SecretName:  default-token-5dn7s
Optional:    false
QoS Class:       Burstable
Node-Selectors:
Tolerations:     node.kubernetes.io/not-ready:NoExecute for 300s
node.kubernetes.io/unreachable:NoExecute for 300s
Events:
Type     Reason     Age                From                                            Message

Normal   Scheduled  60s                default-scheduler                               Successfully assigned default/artifactory to gke-kube-phpipam-pool-1-6b696b70-jr2z
Normal   Pulling    18s (x4 over 60s)  kubelet, gke-kube-phpipam-pool-1-6b696b70-jr2z  pulling image "gcr.io/emb-infra/artifactory-teste:latest"
Normal   Pulled     18s (x4 over 59s)  kubelet, gke-kube-phpipam-pool-1-6b696b70-jr2z  Successfully pulled image "gcr.io/emb-infra/artifactory-teste:latest"
Normal   Created    18s (x4 over 59s)  kubelet, gke-kube-phpipam-pool-1-6b696b70-jr2z  Created container
Normal   Started    17s (x4 over 59s)  kubelet, gke-kube-phpipam-pool-1-6b696b70-jr2z  Started container
Warning  BackOff    2s (x6 over 57s)   kubelet, gke-kube-phpipam-pool-1-6b696b70-jr2z  Back-off restarting failed container

Please move to the latest  helm charts for  Kubernetes here This repo is under deprecation and hence closing this issue.


can I apply react-draggable and react-resizable on a div at the same time?

if I want a div can be draggable and resizable at the same time. Can I do this ? :
<Draggable>
  <ResizableBox width={200} height={200} draggableOpts={{...}}
        minConstraints={[100, 100]} maxConstraints={[300, 300]}>
      Contents
   </ResizableBox>
</Draggable>

I was struggling a bit with this as well. The key for me was to add a cancel prop. I have a small prototype you can check out here: https://codesandbox.io/s/react-resizable-react-draggable-zue38

Still not sure how to persist location o Draggable after re-renders. Check out my question on https://stackoverflow.com/questions/67775912/react-draggable-save-position-of-element-after-re-render

There is a package that handles alot of this automatically called react-rnd you can see here


Problem sending/receiving emails with attachments

From @nzola:

Lokole system cannot transfer email with attachment neither from Internet or from the Lokole device. Only simple email without attachment can be transferred What is happening right now, all the emails with attachment from the Lokole device are transferred to the sent box after sync, but never show up at yahoo or gmail. They are swallowed by the Main server. All the emails with attachment from yahoo or gmail are sent, but do not show up on Lokole device and never bounce back.

Lots of exceptions of this type in AppInsights:
TypeError:
   at log_exception (UnknownUnknown: /app/opwen_email_server/utils/log.pyUnknown: 82)
   at __call__ (UnknownUnknown: /app/opwen_email_server/actions.pyUnknown: 37)
   at _action (UnknownUnknown: /app/opwen_email_server/actions.pyUnknown: 92)
   at _store_inbound_email (UnknownUnknown: /app/opwen_email_server/actions.pyUnknown: 100)
   at _to_id (UnknownUnknown: /app/opwen_email_server/actions.pyUnknown: 117)
   at to_json (UnknownUnknown: /app/opwen_email_server/utils/serialization.pyUnknown: 15)
   at dumps (UnknownUnknown: /usr/local/lib/python3.6/json/__init__.pyUnknown: 238)
   at encode (UnknownUnknown: /usr/local/lib/python3.6/json/encoder.pyUnknown: 199)
   at iterencode (UnknownUnknown: /usr/local/lib/python3.6/json/encoder.pyUnknown: 257)
   at default (UnknownUnknown: /usr/local/lib/python3.6/json/encoder.pyUnknown: 180)


Adding a StreamOutput so that compiled eo examples can be output to the console.

This was useful for me as I was working out what the compiler does exactly. It also shows some deficiencies in the current compiler.
The command line argument "--demo" can now be used to show the various eo example files and their compiled output.
I had to duplicate the resource eo files, which is a shame, but I don't yet know how to avoid that.

I'd made these changes anyway for my own benefit and learning. If they're not useful more generally then that's fine. Maybe they'll just be useful for a while until we improve the tests?

Here's what the output looks like in IntelliJ:
=====================
EOLANG:
type Book:
    Text text()

JAVA:
public interface Book {
    Text text();
}
=====================

=====================
EOLANG:
type Car:
    Money cost()
    Bytes picture()
    Car moveTo(Coordinates coords)
JAVA:
public interface Car {
    Money cost();
    Bytes picture();
    Car moveTo(final Coordinates coords);
}
=====================

=====================
EOLANG:
object fibonacci as Int:
  Int @n
  ctor():
    fibonacci:
      1
  Int int():
    if:
      firstIsLess:
        @n
        2
      1
      plus:
        @n
        fibonacci:
          minus:
            @n
            1
JAVA:
line 6:2 extraneous input ' ' expecting DEDENT
line 5:2 missing NEWLINE at ' '
line 6:2 extraneous input 'Int' expecting {<EOF>, 'type', 'object', LONAME}
line 6:9 extraneous input '(' expecting {<EOF>, 'type', 'object', LONAME}
line 10:8 extraneous input ' ' expecting {<EOF>, 'type', 'object', LONAME}
line 14:8 extraneous input ' ' expecting {<EOF>, 'type', 'object', LONAME}
line 17:12 extraneous input ' ' expecting {<EOF>, 'type', 'object', LONAME}
public final class fibonacci implements Int {
 private final Int n;

public fibonacci(final Int n) {
 this.n = n;
}
}
=====================

=====================
EOLANG:
type Number:
    Integral integral()
    Decimal decimal()

type Text:
    Number length()
    Collection lines()
JAVA:
public interface Text {
    Number length();
    Collection lines();
}public interface Number {
    Integral integral();
    Decimal decimal();
}
=====================

=====================
EOLANG:
type Pixel:
  Pixel moveTo(Integer x, Integer y)

JAVA:
public interface Pixel {
    Pixel moveTo(final Integer x, final Integer y);
}
=====================

=====================
EOLANG:
object zero as Money, Int:
    Int @amount
    Text @currency
    ctor(Int amount, Text currency)

JAVA:
public final class zero implements Money, Int {
 private final Int amount;
private final Text currency;

public zero(final Int amount, final Text currency) {
 this.amount = amount;
this.currency = currency;
}
}
=====================

@pa9ey good idea, but the problem with file duplication is so obvious that we have to fix it. I believe, we have to do it all a bit differently. I should be able to do:
$ java -jar eolang.jar --demo

This should output all .eo files available in src/main/resources/eo. For example:
$ java -jar eolang.jar --demo
test.eo
car.eo
simple/fibonacci.eo

Then, I should be able to do:
$ java -jar eolang.jar --demo simple/fibonacci.eo

And get what you're printing now.
And, we have to use the same files in our tests. Just remove them from src/test/resources and point tests to src/main/resource (should happen automatically, if you don't change their names and paths).

@yegor256 Agreed. You know how much I dislike pointless duplication. ;)
I'll have a crack at sorting it all out.

@yegor256 A request from a Java/Open Source noob who is a bit unsure of himself...

I have an almost ready fix for this that is very if-else-y, ugly, and horribly procedural, and has some code stolen straight from a stackoverflow answer, but it would at least work. I could submit this soon. (I could even add a puzzle for making it more OO.)

I could tidy up the whole thing and make it more OO but this pull request might have to sit around for a while.

I could do 2. but submit the fix from 1. first for comment (I'm learning here and would welcome advice), and then the whole fix could be pulled when I've made it more OO.

I don't particularly mind either way, but which would you prefer? :)

@yegor256 Oh boy.  I should have known that Qulice wasn't going to let me get away with the ugly procedural version... 😆
The static analysis is absolutely brutal. I might have some quibbles with it, and there might be some things I don't understand yet... but I'm impressed.

@pa9ey - I don't understand - As far I see, all checks have passed = Qulice "approved" your code. Am I right?
BTW It is not the prettiest code and has one big problem - mixing levels of abstraction: in one method you are low-level programmer who manipulate with stdout and then, you are high level programmer who just create objects and delegate the tasks to them.
Anyway, your code is better than 95% of code in the world - it needs refactoring but it is easy to refactor.

@pchmielowski I'm referring to the stuff I'm working on right now. I've been trying to fix it according to this comment: https://github.com/yegor256/eo/pull/116#issuecomment-275997419. But I've been trying to achieve No.1 in my list of options above. I just found it interesting that I couldn't actually achieve that because Qulice won't let me. It's looks like it's going to force me to be more OO. That's kinda cool. I'm an advocate of static analysis in theory (although circumstances at work haven't let me try it in practice), I just didn't realise it could be so brutal - and perhaps so effective. :)

@yegor256 and @pchmielowski: I'm gonna go for option 2 then... But like I said, this PR might have to sit here a while... All good though. I'm learning stuff.

Closing this PR, and will resubmit when I've done a proper solution.


Shuffle the array.

Add the solution for the problem shuffle the array
Given the array nums consisting of 2n elements in the form [x1,x2,...,xn,y1,y2,...,yn].
Return the array in the form [x1,y1,x2,y2,...,xn,yn].
Example:
Input: nums = [2,5,1,3,4,7], n = 3
Output: [2,3,5,4,1,7]
Explanation: Since x1=2, x2=5, x3=1, y1=3, y2=4, y3=7 then the answer is [2,3,5,4,1,7].

I want to work on this issue..I can provide the solution in C/C++/Pyhton


Customizing transformers

Hi @thierryvolpiatto . I am looking for some guidance on how is the best way to customize helm transformers to add icons(refer to the README gif how it looks like). For sources that are created lazily, I am using helm-setup-user-source (e. g. helm-buffers-source) and for some sources, I directly modify the source since it is not created lazily(e. g. helm-source-recentf).
I haven't found a way to update the functions that create the sources inline(e. g. helm-locate-library). Is there a better way to do what I am trying to achieve?

Ivan Yonchovski<EMAIL_ADDRESS>writes:

Hi @thierryvolpiatto . I am looking for some guidance on how is the
best way to customize helm transformers to add icons(refer to the
README gif how it looks like). For sources that are created lazily, I
am using helm-setup-user-source (e. g. helm-buffers-source) and for
some sources, I directly modify the source since it is not created
lazily(e. g. helm-source-recentf).

You can also use helm-setup-user-source for such source.

I haven't found a way to update the functions that create the sources
inline(e. g. helm-locate-library).

There is no ways to modify such source, we need to define a class and
run (helm-make-source "foo" 'foo-class ...) to allow using
helm-setup-user-source.
If you have a list of commands that define source inline you want to
modify, let me know I will build classes so that you can modify
them.
--
Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997

You can also use helm-setup-user-source for such source.

when I do
    (cl-defmethod helm-setup-user-source :after ((source helm-not-loaded-source)))

I get Debugger entered--Lisp error: (error "Unknown specializer helm-not-loaded-source")
signal(error ("Unknown specializer helm-not-loaded-source"))
error("Unknown specializer %S" helm-not-loaded-source)

If you have a list of commands that define source inline you want to
modify, let me know I will build classes so that you can modify
them.

You can also use helm-setup-user-source for such source.

when I do
    (cl-defmethod helm-setup-user-source :after ((source helm-not-loaded-source)))

I get
 Debugger entered--Lisp error: (error "Unknown specializer helm-not-loaded-source")
  signal(error ("Unknown specializer helm-not-loaded-source"))
  error("Unknown specializer %S" helm-not-loaded-source)

If you have a list of commands that define source inline you want to
modify, let me know I will build classes so that you can modify
them.

I

Ivan Yonchovski<EMAIL_ADDRESS>writes:

You can also use helm-setup-user-source for such source.

when I do
(cl-defmethod helm-setup-user-source :after ((source helm-not-loaded-source)))

I get Debugger entered--Lisp error: (error "Unknown specializer helm-not-loaded-source")
signal(error ("Unknown specializer helm-not-loaded-source"))
error("Unknown specializer %S" helm-not-loaded-source)

Hmm! Don't know. If I eval this I have the same error, but it is normal
as helm-not-loaded-source is not a valid class.
With recentf, there is may be a problem as the source is initialized at
load time by a defcustom, so may be your defmethod is not already known
at this time. This can be fixed if needed.
--
Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997

With recentf, there is may be a problem as the source is initialized at
load time by a defcustom, so may be your defmethod is not already known
at this time. This can be fixed if needed.

Yes - this is what I was referring to.
As a side note, what about adding an extension point just before the source has been used? It will make enabling and disabling much easier and won't(?) require reworking places which do not define classes. WDYT?

Ivan Yonchovski<EMAIL_ADDRESS>writes:

With recentf, there is may be a problem as the source is initialized at
load time by a defcustom, so may be your defmethod is not already known
at this time. This can be fixed if needed.

Yes - this is what I was referring to.
As a side note, what about adding an extension point just before the
source has been used? It will make enabling and disabling much easier
and won't(?) require reworking places which do not define
classes. WDYT?

Sorry I don't understand what you want to do here.
--
Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997

@thierryvolpiatto please ignore - I decided to put advice over helm-make-source which solved all of my issues.


Add MQ dataplan segmentation

Similar to ICD services - see
https://github.com/terraform-ibm-modules/terraform-ibm-cbr/blob/e6e1e54d7c12a6354a844c35f9ec7774bb527afa/modules/fscloud/main.tf#L335

Available in https://github.com/terraform-ibm-modules/terraform-ibm-cbr/releases/tag/v1.23.1, closing this.


Error in intent schema

"WhatsNextIntent" should be changed to "NextEventIntent" in IntentSchema.json

Fixed intent schema.


Error: (HTTP code 400) unexpected - Invalid reference format

Good afternoon,
I've encountered an issue when attempting to start the Docker container in my environment. After initiating the container, it automatically restarts after a few seconds due to an error. This is causing inconvenience in the proper functioning of the MQTT service in my Home Assistant instance.
MqDockerUp restarts and does not function correctly, thereby preventing the MQTT service from receiving data from all containers as it should.
I have attached a screenshot showing the specific error I am experiencing.
Any help or suggestions to resolve this issue would be greatly appreciated!

Thanks you for your assistance!

Same problem with release 1.9.0

Same issue here

Interestingly, mine also provides a 400 error directly after checking the Immich containers. Could that be part of the issue?
17.12.2024 02:51:47 [warn] Failed to find new digest for image ghcr.io/immich-app/immich-server:release
17.12.2024 02:51:47 [error] MqDockerUp stopped due to an error
17.12.2024 02:51:47 [error] object
17.12.2024 02:51:47 [error] Error: (HTTP code 400) unexpected - invalid reference format
    at /app/node_modules/docker-modem/lib/modem.js:382:17
    at getCause (/app/node_modules/docker-modem/lib/modem.js:417:7)
    at Modem.buildPayload (/app/node_modules/docker-modem/lib/modem.js:378:5)
    at IncomingMessage.<anonymous> (/app/node_modules/docker-modem/lib/modem.js:346:16)
    at IncomingMessage.emit (node:events:529:35)
    at IncomingMessage.emit (node:domain:489:12)
    at endReadableNT (node:internal/streams/readable:1400:12)
    at processTicksAndRejections (node:internal/process/task_queues:82:21)
12/16/2024 06:51:47 PM Container stopped


gsoler2 Acrolinx quality push: AD-Forest-Recovery-Verify-Replication.md

#assign: @robinharwood, @Xelu86
@robinharwood @Xelu86
Can you review the proposed changes?
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Minor Acrolinx edits, merging now.


kubevpn listen on 6060 cause port conflict

Port 6060 is a well-known port. It is used for golang pprof. So kubevpn's port will be conflict with other golang programs. It is better to use ports that are rarely used.

already change port to 32345 , commit: https://github.com/wencaiwulue/kubevpn/commit/a545f3a9589809b20e62950eb213a1680d37367a


using tooltip dynamically as part of innerHTML for a div

I am setting the contents of a div dynamically based on a simple JavaScript function, but the tooltip does not display using tooltipster (just as the default browser behavior when encountering a title=). Other tooltips on this webpage do use tooltipster.
function setAwards() {
var awardsdiv=document.getElementById("awards"); awardsdiv.innerHTML="";
if (document.setvalues.superstar.checked == true) awardsdiv.innerHTML=awardsdiv.innerHTML + "<img class="tooltip" title="Super Star (3.-8. by Mar 15th)" width="16" src="/pix/superstar.png"  alt="[ Super Star ]" />";
[... stuff removed ...]
}
Is there a way to make this work?
Thank you!

Hi, please head over to Srackoverflow for this question, thank you.


Vertical rhythm between a field of radios or select boxes isn't right

Describe the bug
The vertical rhythm of these boxes doesn't feel right. It should be 0.5 rem. See screen shot

@will-madebymoon this will be resolved when #263 is merged


Setting the User-Agent header

Issue: https://github.com/aspnet/SignalR/issues/299

@mikaelm12,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


(fix) Refactor pkg/version/openebsVersion.go

Refactor: pkg/version/openebsVersion.go
Issue closed : https://github.com/litmuschaos/litmus/issues/786

Thanks for the PR @rampreeth, Can you resolve the conflict in the file

Looks like some changes have been made to the same file I was working on, shall I still resolve the conflict and push my changes?

Have resolved and made changes accordingly.

@chandankumar4 , thank you for the suggestions. Have made the changes.

Thanks for taking part in Hacktoberfest 2019. Please fill up this form to claim your swag https://hubs.ly/H0l5ldm0


value is shown instead of text

hi
i encountered a very strange issue! i wonder if you could help?
in one tree i used two php snipets to load data and strangely the second group of data ( i mean those loaded from the second php) show the value part of option instead of text part.
i checked and the syntax for both php snipets are the same.

i found that i was passing null for TEXT and correcting that solved my problem


Inactive repo 'core-eng' should be archived

https://github.com/dotnet/org-policy-violations/issues/5151

Release Note Category

[ ] Feature changes/additions
[ ] Bug fixes
[ ] Internal Infrastructure Improvements

Release Note Description

Archiving seems good to me!

Archived (again)


Update Dockerfile

opensource is where this material goes
Signed-off-by: Mary Anthony<EMAIL_ADDRESS>

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "moxiegirl-patch-1" git@github.com:docker/swarm.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

LGTM (@moxiegirl: just need to sign-off the commit and we good , sorry :)
ping @thaJeztah :cocktail:

change LGTM after sign off :innocent:

We can close this one after including #1493


1040 search function

Thank you for your Pull Request! We have developed this task checklist from the Development Process Guide to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your code as well as making sure the admiral codebase remains robust and consistent.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the devel branch until you have checked off each task.

[ ] Code is formatted according to the tidyverse style guide
[ ] Updated relevant unit tests or have written new unit tests - See Unit Test Guide
[ ] If you removed/replaced any function and/or function parameters, did you fully follow the deprecation guidance?
[ ] Update to all relevant roxygen headers and examples
[ ] Run devtools::document() so all .Rd files in the man folder and the NAMESPACE file in the project root are updated appropriately
[ ] Address any updates needed for vignettes and/or templates
[ ] Update NEWS.md
[ ] Build admiral site pkgdown::build_site() and check that all affected examples are displayed correctly and that all new functions occur on the "Reference" page.
[ ] Address or fix all lintr warnings and errors - lintr::lint_package()
[ ] Run R CMD check locally and address all errors and warnings - devtools::check()
[x] Link the issue so that it closes after successful merging.
[ ] Address all merge conflicts and resolve appropriately
[ ] Pat yourself on the back for a job well done!  Much love to your accomplishment!

Taking this for a spin - I see that the References are aligned differently then the current version of the website.  Can you see if it is possible to align them to be the same?

This is so cool that we can search so easily!!

I put search: exclude: ['news/index.html'] in the yml to ignore the changelog.  I think this would be helpful as I don't think a lot of people want to search the changelog/news.

@PoojaKumari05  The change in the look for the Reference Tab was determined to not to be an issue.  Rather have better search capability then the exact same layout.  Are you still able to push this along?

@PoojaKumari05 The change in the look for the Reference Tab was determined to not to be an issue. Rather have better search capability then the exact same layout. Are you still able to push this along?

@bms63 Hi Ben, as per the new bootstrap 5 update, the font size of the themes are larger than before, this is the reason the descriptions in References is falling to next line. If this is good to go with, I think we can closed this PR.


Added Vagrantfile for build environment with Vagrant

Proposed Changes
I've added a Vagrantfile to the vagrant subdirectory. One may create a working environment by the command vagrant up and run the Ansible playbook by the command vagrant provision --provision-with ansible.
I also added a section within the README.md regarding the usage of Vagrant and I made the retry_count in the Verification section of the master role customizeable (default is still 20).
I made the experience, that the ubuntu2110 Box Image of Vagrant is not running into a VirtualBox issue so often than other Ubuntu images do. It still happens from time to time, making the VM unresponsive. It is described here.
Checklist

[x] Tested locally
[x] Ran site.yml playbook
[ ] Ran reset.yml playbook
[x] Did not add any unnecessary changes
[ ] 🚀

Thank you!


[Agent] kafka support topic in produce and fetch

This PR is for:
#4029
#3922

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

I tried to fix the problem #3922 here as well

@mickeyzzc Note that this pr intersects with your, so you can rebase your pr
https://github.com/deepflowio/deepflow/pull/4507

@liqian1834 here.

@mickeyzzc 这个 mr 的 commit 太多，可以合并成一个

@mickeyzzc 这个 mr 的 commit 太多，可以合并成一个

done

现在有两个 commit，好像带了很多别的代码，麻烦再重新 rebase 一下；另外还要处理一下 check 里的失败

@mickeyzzc 这个 mr 的 commit 太多，可以合并成一个

done

现在有两个 commit，好像带了很多别的代码，麻烦再重新 rebase 一下；另外还要处理一下 check 里的失败

抱歉，近期忙其他事情没有及时跟进问题。不过我已经看到有其他pr在解决同类问题。#4905 。我就不继续重复了。这个pr可以关闭了。
不过我看到历史的pr都是在解决单个问题，比如#4905 只解决了produce版本9的topic name问题。但是还是没有处理fetch 12版本的情况。这可能会成为bug。


nixos/version: add ANSI_COLOR

Description of changes
Ages ago, some application (I don't remember what it was) was crashing because ANSI_COLOR wasn't set in /etc/os-release. That field is optional according to the spec and obviously the application(s) in question should be fixed, but there is no harm in adding it in anyway.
I have no idea what happened with #282520 - I have rebased PRs in the past without this issue. Apologies to all involved and thanks for limiting the damage @drupol.
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

It's totally fine, it's the one who does nothing, to whom nothing ever happens! Thanks for submitting it again properly.


IT gate discount not reflected in tech browser

It still gives the full 25% discount in the ship designer, but it shows the full cost in dialogs and tech browser descriptions.
(This is visual only bug fyi)

Looking into it getPlayerCost is outdated and needs to be updated.
https://github.com/sirgwain/craig-stars/blob/12ce8f9fcdb35e249b134182276852ca0ff73f7f/frontend/src/lib/types/Player.ts#L363


Multiple threads - RouterOsApiParsingError

Can a single connection be used from multiple threads simultaneously?  I'm currently getting a routeros_api.exceptions.RouterOsApiParsingError whenever I try to do this.
Is there any documentation for this library somewhere?

You can't use single connection for multiple threads. It would be possible to modify library to make it happen, in fact there is already foundation for that. This library is using Mikrotik async API, so making it multi-thread would be quite simple:

you would need to add mutex for writing to connection
you would need to add condition with some logic for reading from from

We don't need this feature so we probably won't implement it by ourselves.
There is no documentation whatsoever. We have created this library for our personal needs and shared on Github in case anyone would be interested in using it or contributing. Originally we have used Python code from Mikrotik wiki, but this code worked awful, so we have created this library.
If you would like to contribute by writing some documentation or making this library thread safe it would be great.


Improve handling of spec types in composer

Previously we used regex on key names to determine if the contents was an entity spec.  This meant some (eg multi-group) didn't work and in some cases could make the UI unusable or broken.
This checks the actual type defined for the key name and of the value if the type is Object and so supports entity spec
either if:

it uses the DSL word
the key type matches
the key name matches (fallback for where type is unknown/not-loaded and value not set)

and in multiple places:

in model
in designer view canvas
in spec editor

Also makes minor related improvements to UI and code.

Includes and builds on https://github.com/apache/brooklyn-ui/pull/332


chore(ci): [NMP-27] Feature** Added eslint(airbnb) and prettier to project

Description
Added Eslint and prettier to have an easier time identifying and rectifying potential errors
Type of changes
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist

[x] I have read the CONTRIBUTING doc
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have already been accepted and merged

Further comments

Thanks for the PR!
Deployments, as required, will be available below:

Frontend
Backend

Please create PRs in draft mode.  Mark as ready to enable:

Analysis Workflow

After merge, new images are deployed in:

Merge Workflow

@LocalNewsTV I fixed configuration for eslintrc to follow airbnb style guide.

I did some researching and referencing of what you did @LocalNewsTV from your past capstone, and added a couple of comments to provide understanding on what each line is doing. I've run npm run lint and it is giving me my lint problems. We'll be adding more once we get into creating the project further. Meanwhile, I'll settle for this initial settings for mostly TypeScript configurations.
Once approved I'll resolve conflicts with package dependencies. I'm wondering if resolving it needs to be on vscode, and merge it there to resolve conflicts on package.json and do a npm install to lock the dependencies.

@kcaparas1630
If you restart VSC after installing the dependencies that were missing it will pick them up on load. I've tested that it runs on my local
I don't know why they don't get picked up from the initial installation, It might just be how the extensions are configured, this isn't unique to your repo.
There is also Prettier and ESLint extensions for VSC :)
You can also add recommended VSC Extensions to your repo, if you want to alert people developing to install them Here's more details.