Add features from Azure CLI into Knack

This PR:

Adds support for positional arguments with the .positional helper method on ArgumentsContext.
Removes the necessity for the type field in help.py. This information can be inferred from the class, so specifying it causes unnecessary crashes.
Adds support for examining the result of a command after a call to invoke. The raw object, error (if any) an exit code are accessible.
Adds support for accessing the command instance from inside custom commands by putting the special argument cmd in the signature.
Fixes an issue with the default config directory. It use to be .cli and is now .{cli_name}.

@williexu I'll make a PR for the CLI to use this version and some of the logic will be removed (some will still need to be overridden).  Primarily though, these are features that partner teams have requested and/or features I want to use in azdev.

This also looks good to me! (y)


Code Coverage Options With Combined Coverage Results

Context
I know it is possible to get code coverage using the custom parameters option like this:
- name: Code Coverage
  uses: game-ci/unity-test-runner@v2
  id: tests
  env:
    UNITY_LICENSE: ${{ secrets.UNITY_LICENSE }}
  with:
    artifactsPath: code-coverage-artifacts
    githubToken: ${{ secrets.GITHUB_TOKEN }}
    checkName: Code Coverage Test Results
    customParameters: -debugCodeOptimization -enableCodeCoverage -coverageOptions generateHtmlReport;generateBadgeReport;assemblyFilters:+<assets> -coverageResultsPath code-coverage-artifacts

This works great for getting the coverage results for a single editmode or playmode test run, but it is unable to get the combined coverage results from running something similar to the following script:
./Unity.exe -projectPath <project-path> -batchmode -testPlatform editmode -runTests -debugCodeOptimization -enableCodeCoverage -coverageResultsPath <results-path> -coverageOptions "generateAdditionalMetrics;assemblyFilters:+<assets>"

./Unity.exe -projectPath <project-path> -batchmode -testPlatform playmode -runTests -debugCodeOptimization -enableCodeCoverage -coverageResultsPath <results-path> -coverageOptions "generateAdditionalMetrics;assemblyFilters:+<assets>"

./Unity.exe -projectPath "C:/Users/Joshua/Documents/Code Projects/GrizzledOwl/common-ci-cd" -batchmode -debugCodeOptimization -enableCodeCoverage -coverageResultsPath <results-path> -coverageOptions "generateHtmlReport;generateBadgeReport;assemblyFilters:+<assets>" -quit

In a lot of cases this combined report is what a lot of people will want when looking at code coverage.
With the information from this the people/teams using this script would be able to use that metric to enforce a minimum code coverage standard on their repositories.
Suggested solution
In order to support the above, I would suggest adding in a "codeCoverage" option flag to the action. I would have it work as follows:

When the "testMode" option is "editmode" - creates the code coverage report for editmode tests.
When the "testMode" option is "playmode" - creates the code coverage report for playmode tests.
When the "testMode" option is "all" - generate combined coverage result.

I would probably add a flag "coverageOnly" so that the test results could be skipped if they aren't needed.
Not only would this support the feature, but it would make it a little clearer to people who are trying to use this action how to get coverage results. In my opinion, it would also make their GitHub actions a little bit more readable.
I would also add an option for "coverageReportType" that would let you specify which reports you are wanting. This would have three options:

html - generates the html coverage report
badge - generates the coverage summary badges
both - generates both the html report and summary badges

I would have this default to the "both" option.
I would add an option for "assemblyFilters" that would let you define what assemblies to include. This would just place your options into the correct place.
To round this out, I would add "coverageOptions". This would be the direct input for the coverageOptions that will be in the batch mode call. Honestly, with this coverageReportType and assemblyFilters wouldn't be needed, but since those options are the ones that are used most often, I think it makes sense to specify them up front in order to make it easier and more readable for people using the action.
Finally, I would add a "minimumCoverage" option. It wouldn't be required, would default to 0, and the allowed values would be a number between 0 and 100. If this is provided, when the coverage results are in, the script would perform a check to see if the coverage meets or exceeds the provided value. If it does, it passes. If it doesn't, it fails.
Using these options would look something like this:
- name: Code Coverage
  uses: game-ci/unity-test-runner@v2
  id: tests
  env:
    UNITY_LICENSE: ${{ secrets.UNITY_LICENSE }}
  with:
    artifactsPath: code-coverage-artifacts
    githubToken: ${{ secrets.GITHUB_TOKEN }}
    checkName: Code Coverage Test Results
    testMode: all
    codeCoverage:
    coverageOnly:
    coverageReportType: both
    assemblyFilters: +<assets>
    # coverageOptions: verbosity? generateTestReferences? sourcePaths? etc...
    customParameters: -debugCodeOptimization -burst-disable-compilation
    minimumCoverage: 80

Considered alternatives
I don't really have any alternatives, outside of writing my own custom script, to handle these things.
Additional details
I thought about writing my own custom script to handle these things, but it seems like something others would want as well. I am already using game-ci in my other GitHub actions for my projects, so I wanted to see if this was a feature that would be desired here.
I have seen other code coverage issues and pull requests, but I wasn't sure if they were just dropped because they only covered the basic solution that can be handled with customParameters, or if directly supporting coverage wasn't desired for this repository.
I was planning on forking the repository and implementing this myself, but I wanted to discuss my solution, and see if this was even desired.

I've been looking at the repo and it looks like it could probably be added as a flag to the end of this file, https://github.com/game-ci/unity-test-runner/blob/main/dist/steps/run_tests.sh
Maybe add a new flag like "-enableCodeCoverage"
And at if both "$PLAY_MODE" = "true" and "$EDIT_MODE" = "true" and the code coverage flag is enabled (as a separate clause added at the end), we could add a clause to add custom flags based on if code coverage is included.
I might fork this repo tomorrow to see if I can get it working on my own project for validation. I'll post an update if I come up with anything.

I'm working on drafting a PR for this addition right now, my first time using node in a while so code might be a bit off.
I don't think it will work yet (still need to test it) but working on just setting out the boiler plate code with three parameters

enableCodeCoverage (optional flag, default of disabled)
assemblyFilters (optional, requires enableCodeCoverage)
coverageResultsPath (optional, requires enableCodeCoverage)

https://github.com/nicholas-maltbie/unity-test-runner/tree/nicholas-maltbie/codecoverage-support
I'll work on testing it, but have to get on a flight soon so will probably have some update in the next few days! hopefully can test it with my project soon.

Oh, I was planning on working on the solution for this myself. I was just checking to see if this was a desired change since there have been other issues and pull requests related to code coverage.

Alright, I debugged the changes and finishing up all the additions for these requirements of code coverage. Here is a link to the PR, I included some tests and docs to make sure it works as expected. I'll describe the scenarios that I tried to include (I think I have all the ones you outlined form this issue). If you see any issues or have any suggestions, let me know @jhornsb2 :)
https://github.com/game-ci/unity-test-runner/pull/182

@nicholas-maltbie I just got the chance to look through all of the changes. Thank you for implementing it. It looks great and should cover exactly what I need.

@nicholas-maltbie, @davidmfinol Thanks for your work on the code coverage. I tried to run game-ci/unity-test-runner@v2 in a self-hosted windows runner. I did not provide coverageOptions parameter so the default value is used. I can see the tests are run successfully and the test artifact result is expected. Also, code coverage folder is generated with content. However, I got 0% coverage for all files. When I run the test from the cmd with the same set of parameters locally I get the expected code coverage result which is not 0% for some files. Do you guys have any idea why this is happening? Thanks.

Hi @spencer-cheng , i ran into this issue as well. I found that explicitly including the assemblies I want to generate coverage options for such as 'assemblyFilters:+my.assembly.*' may help. Are there coverage files generated in the output folder that or is the result just empty?

@nicholas-maltbie Thanks for your reply. I tried to explicit include them. But it does not help.
I did more tests in the past two days. It works when i run it on "ubuntu-latest".
However test same code does not when i choose "windows-2019". Similarly, if i run it using self-hosted windows 2019 runner, the action does not work. If i change to use "run unity command line directly" in step instead of using unity-test-runner action on my self-hosted runner, the coverage data is generated correctly as expected.
I guess there might be a problem with code coverage's windows implementation.  Thanks.

@spencer-cheng , there are a few differences between unity when it runs on windows versus Linux in CLI mode. Are you able to produce coverage files properly when running on your local windows machine via cli directly? It might be good to isolate if it's a windows problem when running with CLI or a problem with the way the test runner is configured.
If you're able, could you share the configuration you're using to generate test coverage? Having this workaround of using the run unity command line directly may prove useful to others if this issue takes a while to resolve for windows containers.


Regression in dgeqrf_ QR factorization since 0.3.21 - Output contains NaN values

This was discovered from a PyTorch unit test failure on aarch64 ( link will be attached )
Code to reproduce the problem is attached
reproducer.zip
See attached code for more details
   dgeqrf_(&m, &n, input_buffer, &lda, tau_buffer, workspace, &lwork, &info);

    for(int i = 0; i < TAU_STRIDE; i++){
        if(std::isnan(tau_buffer[i]))
            throw std::invalid_argument( "TAU contains NaN" );
    }

Under libopenblas/openblas 0.3.20 there is no NaN contained in the tau output, however in 0.3.21+ the output contains NaN values. It appears this is when the implementation was changed from Fortrain to C.

The implementation as such did not change, 0.3.21 only introduced an optional C translation of the LAPACK code that can be used when no Fortran compiler is available. If you see it "only" in CI, it suggests that your setup lost its fortran compiler in about the same timeframe and you wouldn't be able to compile 0.3.20 there now.
I cannot reproduce this on x86_64 with the current develop branch, will try aarch64 shortly.

Also not reproducible on NeoverseN1 with either NeoverseN1 or ARMV8 target (and NOFORTRAN=1 of course). If anything, this could implicate one of the newer SVE kernels (N1 not having SVE), but 0.3.21 did not have any additions or changes there.

Ah apologies about that, I missed that crucial peiece of information. This issue appeared on a Neoverse-v1 machine. I will check n1 from my side, but i don't think there is an issue as it came through a pytorch unit test failure on v1.
I agree with you, this is suggestive of an SVE related issue.

Reproduced now on my phone :) I have a hunch that it could be the DNRM2 kernel (which isn't even SVE at the moment, just a different "big server" implementation), will see in a moment

DNRM2 it is indeed, I had already retired this particular implementation of NRM2 on the Apple M "Vortex" targets earlier. Will create a PR later today.


Update index.md

Checklist

[ ] Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
[ ] Any modified or new methods or classes have helpful JSDoc and code is thoroughly commented
[ ] The description lists all applicable issues this PR seeks to resolve
[ ] The description lists any configuration setting(s) that differ from the default settings
[ ] All tests and CI builds passing
[ ] The description lists all relevant PRs included in this release (remove this if not merging to master)
[ ] e2e tests are all passing (remove this if not merging to master)
[ ] Code coverage does not significantly worsen (ideally it improves) (remove this if not merging to master)

Description
Please explain the changes you made here and, if not immediately obvious from the code, how they resolve any referenced issues (and PRs if merging to master). Be sure to include all issues being resolved and any special configuration settings that are need for the software to run properly with these changes.

@ritesh-warade-ibigroup, I'm going to change your base branch to dev. FYI, all PRs should typically be made to dev (after a few stack up, we generally make a PR from dev to master).


Incorrent value of maxModelTokens for gpt-3.5-turbo

Verify latest release

[X] I verified that the issue exists in the latest chatgpt release

Verify webapp is working

[X] I verify that the ChatGPT webapp is working properly for this account.

Environment details
node version: v16.16.0
macOS 12.4
Describe the Bug
https://github.com/transitive-bullshit/chatgpt-api/blob/main/src/chatgpt-api.ts#L60 sets maxModelTokens = 4000.
But gpt-3.5-turbo max tokens setting is 4096.
Should we set maxModelTokens to 4096?

This project is undergoing a major revamp; closing out old issues as part of the prep process.
The chatgpt package is pretty outdated at this point. I recommend that you use the openai package or the openai-fetch package instead.


[ROCm] ci build and dockerfile changes

Install system GCC libraries
Remove docker image
apt update

Created from: rocm/jax#161

@superbobry  could you please approve? thanks!

@superbobry @dfm can someone merge please?

It looks like there was a merge conflict when this was submitted internally and then it must have gotten lost in the shuffle. Thanks for the ping! @Ruturaj4 — Can you rebase, and then I'll get this merged?

@dfm Just rebased! Could you please merge! thanks.


Broken Paella Player Layout

Watching a single stream presenter video in Paella player leads to the video being displayed as a tiny thumbnail only, making the player completely unusable:

To test this, ingest a video using:
curl -f -i -u admin:opencast https://develop.opencast.org/ingest/addMediaPackage/fast \
  -F 'flavor=presentater/source' \
  -F mediaUri=https://data.lkiesow.io/opencast/test-media/sintel_trailer-1080p.mp4 \
  -F title=presenter-only \
  -F creator=lk

This happens on develop (2020-11-05) and thus likely on r/9.x. Tested in a current Chromium and Firefox.

@miesgre, do you by any chance have time to take a look at this?

@lkiesow The problem is you named you flavor presentater and by default paella only recognize presenter and presentation
See:
https://github.com/opencast/opencast/blob/c910481af7ceb838bf739e7a7f9b6a0d6fe36d7d/etc/ui-config/mh_default_org/paella/config.json#L145-L151
You can configure your installation adding a new videoSet with the flavor presentater
{ "icon":"professor_icon.svg", "id":"presentater", "content":["presentater"]},

Good catch.
But then Paella should not show the video at all, should it?
It does show the video. Just very small.

In general, in paella player we don't add code that controls those failures derived from a bad configuration of the player, because we understand that this type of failures should never happen. The video is shown like this simply because after it is added to the player, a failure occurs when applying the layout, and that failure is not being controlled.
However, I have added an issue in the paella player repository to show this kind of error in the javascript console, to make it easier for website administrators to debug it:
https://github.com/polimediaupv/paella/issues/628

I've dropped the „needs attention“ label since it definitely isn't that important given the current findings.

Closing this since Paella 7 is replacing Paella 6 soon anyway


e2e tests don't use currently-installed versions of packages

In our GHA workflow, we are manually installing htmltools from github, but the e2e tests aren't recognizing that installed copy of htmltools.
Error here:
https://github.com/rstudio/py-shiny/actions/runs/4359881025/jobs/7622174378#step:13:23
Installation of correct version of htmltools here:
https://github.com/rstudio/py-shiny/actions/runs/4359881025/jobs/7622174378#step:5:28

Related #414

@jcheng5 May we remove tox as a testing wrapper? If so, we can just use the installed packages with pytest.
I do remember you saying that tox can test in multiple platforms, but we have this with GHA testing. So I do not believe the duplication of e2e dependencies and headache of trying to get GitHub dependencies installed within tox.
I've added a make test-e2e command in https://github.com/rstudio/py-shiny/pull/418/files#diff-76ed074a9305c04054cdebb9e9aad2d818052b07091de1f20cad0bbac34ffb52


When MPAS throws an exception, status is set to success

I just started a run (using my own conda environment) and even though MPAS threw an exception and no diagnostics were produced, both the atm and ocnice diagnostics reported success.
MPAS-Analysis submodule:  30b394a73fc47f5d8f25f9c1e5b5685ed529cc74 python/MPAS-Analysis (v0.5)
Traceback (most recent call last):
  File "python/MPAS-Analysis/run_mpas_analysis.py", line 13, in <module>
    import matplotlib as mpl
ImportError: No module named matplotlib
ocnice diags complete

Additionally when the atm diags were trying to run, turns out I put my data in the wrong directory, but when it exited it also reported it was complete
/p/cscratch/acme/baldwin32/aprime_test_20171107/20171011.beta2_FCT2-icedeep_branch.A_WCYCL1850S.ne30_oECv3_ICG.edison/run for 20171011.beta2_FCT2-icedeep_branch.A_WCYCL1850S.ne30_oECv3_ICG.edison does not exist! Please check.
Exiting atmosphere diagnostics...

atm diags complete

Status of atmospheric diagnostics: 0
 (0-->success, -1-->diags not invoked, -2-->batch_script, 3-->init error)
Status of ocean/ice diagnostics: 0
 (0-->success, -1-->diags not invoked, -2-->batch_script, 3-->init error)

If either one of these fails, the script should probably exit with an error code, or at least set their status reported to the user to something other then success.

I also have seen cases where scripts within the atmosphere diagnostics fail but I don't see any indication of a problem until I look in the log file.  The only other way to know something went wrong is that there is no plot in the end.

We have addressed as much of this issue as possible in #24. I will close this now and we can open a new issue if other instances occur, so that we can address them specifically.


Implement Layouts and Views for Blog App to match the Wireframe Designs

Implement Layouts and Views for Blog App
This pull request adds the necessary views to the Blog App, matching the Microverse wireframes. It includes the integration of layouts and templates to manage shared content across different views.
Features Implemented

Developed views for users and posts, adhering to project wireframes.
Utilized layouts for shared content like headers and footers.
Embedded Ruby code within HTML to dynamically render content.

Technical Adjustments

Arranged .html.erb files to match the expected wireframe structure.
Styled views to ensure a user-friendly interface in line with wireframe aesthetics.
Maintained code simplicity and readability throughout the implementation.

Preview

Standards Compliance

Addressed all linter errors
Followed Microverse's Gitflow protocols for structured version control.
Ensured code is well-documented for professional repository maintenance.

Review Request
The pull request focuses on functional views and navigational flow. Feedback on these aspects would be greatly appreciated to refine the app's user interface.
I look forward to your constructive feedback. Thank you for reviewing my code!

Hello Reviewer!

I wanted to point out that I'm aware of the discrepancy between the positioning of the .post--body element in the "Blog Single Post" page and the wireframe design specifications. Due to time constraints, as I need to move on to the next project before the end of the day, I would prefer not to make this adjustment at the moment.
I assure you that this is on my radar and I intend to revisit this styling issue as soon as I complete the necessary work on the upcoming project. I'm committed to maintaining high standards in my work, and I appreciate your understanding.
Thank you for your review and for considering this aspect of my submission. I look forward to implementing all the required changes following your feedback once my immediate project commitments are met.


Adding ability to override specific styles

Added the ability to pass props for specific styles to override.  Added styles are as follows:
popoverStyle
containerStyle
rowTextStyle
seperatorStyle
If not defined, plugin uses defaults.

Pushed these changes on a separate commit


How to set array of unique values?

We have different platforms for our app and we want to provide user property across them. So we need an array of unique values, for this we're using:
const value = ... // value - 'a' | 'b' | 'c'
const identify = new Identify();
identify.append('Property', value);

But when we check the analytics in Amplitude we see:
Property: ['a', 'a', 'c', 'b', 'a']
How we can set unique values to array in user property?
"@amplitude/react-native": "^2.5.0",
"react-native": "0.68.1"

Hi @Deodes.
You can use identify.postInsert instead. If there is any duplication, it will do no operation.
Thanks.

@yuhao900914
You sent the link to docs for browser, but we use legacy react-native SDK and there is no identify.postInsert method. We see it in new react-native SDK(Beta), but we afraid to use it because it's just Beta

Yea, currently react-native doesn't have that method. I will create a ticket and work on that soon. I will let you know after that has been done.

@yuhao900914
Thank you very much, I really appreciate that you respond quickly and try to solve the problem as soon as possible!

Hi @Deodes, we just released the new version to add the preInsert and postInsert. @amplitude/react-native v2.14.0
Please have a try.
Thanks.

Hi @yuhao900914, thank you very much for such a quick solution, testing will begin soon!


(Java) Java 17 sealed keywords not implemented

With the release of Java 17 being an LTS, it seems proper to support the new sealed keyword.
However, the new non-sealed keyword brings some issues, that while I have attempted to fix, seem best suited for someone more familiar with the highlightjs library.
Sample Code or Instructions to Reproduce
sample js fiddle
Expected behavior
the sealed keyword should be highlighted, and as more hyphenated keywords may come to java in the future, some structure
may be useful for non-sealed and the like.

I think what you have already can be converted into a workable PR.

Ping.

Ping.


Envelope for moving vehicle + load pattern

Hi!
First off, thanks for a nice package, i have been looking for this!
I am trying to use pycba for the quick predesign of a large number of bridges. For this, I need to obtain envelope results for a udl load (through load patterning) combined with a moving vehicle load. I have obtained the max/min values already, but would like to get the envelope plotted as well.
The way i think this should work is to generate envelopes for both the load pattern and than add (augment) it with the moving load envelope. However, I can't seem to get it to work.... Can you give me some pointers?
Thanks in advance!
Ronald

Hi @Grefron , thanks for the nice comments 😊
Augmenting the envelopes should work. Could you share an minimal (non)working example of what you're trying please?

Hi @Grefron : I added a short example of this use case:
https://ccaprani.github.io/pycba/notebooks/envelopes.html#Example-4:-Augmenting-Envelopes

Hi @ccaprani, thanks for the example, it showed exactly what I needed and what i did wrong: I tried to augment an envelope with a beam result, which doesn't work, obviously.


Added Home button while viewing a specific track

PR Fixes:

1 Added home button to the BlogAppBar so that it will be easy to come out of the tracks easily

Resolves #[Issue Number if there]
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I assure there is no similar/duplicate pull request regarding same issue

https://github.com/code100x/daily-code/pull/492#issue-2403841965

closing due to inactivity


Result of Display as -> Enum Member gets 'stuck' after renaming enum members

Version and Platform (required):

Binary Ninja Version: 3.5.4387-dev Personal (71942341)
OS: Windows
OS Version: 10.0.25375.1
CPU Architecture: x64

Bug Description:
When using Display as -> Enum Member on some lines and later renaming the enum members, the lines that had their display changed get 'stuck' - they continue to use the old names, and BN just completely refuses to change their display to anything else, too.
Steps To Reproduce:

Grab this BNDB.
Navigate to 0x7ff69bb40c08.

Expected Behavior:
The enum member references should update with renames, and also not get 'stuck'.
Screenshots:

Yes this is an annoying bug you have to re-analyze the function an that should fix it I believe. I think this is a duplicate of https://github.com/Vector35/binaryninja-api/issues/4341

Reanalyzing the function unfortunately does not seem to resolve it.


Add markdownlint

Ensure npm run lint runs all linting scripts using npm-run-all
Add markdownlint which runs under lint:docs
Autofix common markdown issues like extra blank lines, extra spaces, inconsistent formatting, etc
Convert the double trailing space used to force a newline to a backslash which is much less fragile as mentioned here: https://github.com/ota-meshi/eslint-plugin-regexp/pull/493#discussion_r1032930506

All of these changes should make it easier and safer for us to tweak and improve the docs going forward like in my upcoming PR:

https://github.com/ota-meshi/eslint-plugin-regexp/pull/493

Semi-unrelated: Maybe we could use prettier to format markdown files? This would also have the nice side effect that prettier would also format MD code blocks.

CI should be fixed.

Thank you @bmish!


[feature] Prevent duplicate stylesheets

Currently no checks are made as to whether a stylesheet has already been injected with the given href. This feature prevents this.

excellent. thank you!


Add markdown support in Linguist

This project doesn't have any languages associated with it in the GitHub linguist. This PR should change that. It is a really small change, I just feel it will make the project more perfect!

I've made it minimal, but done - thanks!


Undo service_type transformation in stg_rate_sheet_daily and fix service_type filters in cost_per_query and hourly_spend

This PR undoes the CASE statement that forced the daily rate sheet service_type to either STORAGE or COMPUTE in stg_rate_sheet_daily and fixes all service_type filters in the package accordingly.
I cross referenced the hard-coded service_type values with snowflake.organization_usage.rate_sheet_daily, snowflake.account_usage.metering_history and snowflake.account_usage.metering_daily_history. I also did manual validation checks to ensure that the compute spend values are being populated in the daily_spend model.

Thanks for your contribution @stumelius! This is included in release 5.1.0 :)


AR path snapping

If the GPS indicates that the user is walking in the direction of a nearby path, then they are likely walking on that path.
Adjust the user's location to be on the path (can apply to other tools than just AR). Allow toggling this behavior via a setting.

A simple, naive method could be that if a path is within the GPS accuracy radius, then the user likely on that path and it should be snapped to the nearest point.


provide the expected input to allow the container inspector to continue its execution

Expected Behavior
Hello
I'm new to slim and want to use it in my CICD pipeline.
When I try to resize a python image, it stops and shows me this message:
cmd=build info=continue.after mode='enter' message='provide the expected input to allow the container inspector to continue its execution'
cmd=build prompt='USER INPUT REQUIRED, PRESS <ENTER> WHEN YOU ARE DONE USING THE CONTAINER

And after I press Enter, it will build the container successfully.

The problem is I'm not going to be there when the pipeline executes the command to press Enter! is there any flag or option to automate this?

The command:
docker-slim --report=off build --target 2b7ca628da40 --tag python:test1 --show-blogs --http-probe=false

app='docker-slim' message='Join the Gitter channel to ask questions or to share your feedback' info='https://gitter.im/docker-slim/community'
app='docker-slim' message='Join the Discord server to ask questions or to share your feedback' info='https://discord.gg/9tDyxYS'
app='docker-slim' message='GitHub Discussions' info='https://github.com/docker-slim/docker-slim/discussions'
cmd=build info=exec message='changing continue-after from probe to nothing because http-probe is disabled'
cmd=build info=exec message='changing continue-after to enter'
cmd=build state=started
cmd=build info=params continue.mode='enter' rt.as.user='true' keep.perms='true' tags='python:test1' target.type='image' target='2b7ca628da40'
cmd=build state=image.inspection.start
cmd=build info=image id='sha256:2b7ca628da40dc0cdfc6bf518b0f8cc0c5c3b83e89960e265e540cc7502ceee5' size.bytes='919788091' size.human='920 MB'
cmd=build info=image.stack index='0' name='' id='sha256:2b7ca628da40dc0cdfc6bf518b0f8cc0c5c3b83e89960e265e540cc7502ceee5'
cmd=build state=image.inspection.done
cmd=build state=container.inspection.start
cmd=build info=container status='created' name='dockerslimk_3860711_20230109063239' id='4511bc9fe6f561100f925809a26a7abac6a1a4de0969c57664023f076de30d10'
cmd=build info=container status='running' name='dockerslimk_3860711_20230109063239' id='4511bc9fe6f561100f925809a26a7abac6a1a4de0969c57664023f076de30d10'
cmd=build info=container message='obtained IP address' ip='<IP_ADDRESS>'
cmd=build info=cmd.startmonitor status='sent'
cmd=build info=event.startmonitor.done status='received'
cmd=build info=container name='dockerslimk_3860711_20230109063239' id='4511bc9fe6f561100f925809a26a7abac6a1a4de0969c57664023f076de30d10' target.port.list='' target.port.info='' message='YOU CAN USE THESE PORTS TO INTERACT WITH THE CONTAINER'
cmd=build info=continue.after mode='enter' message='provide the expected input to allow the container inspector to continue its execution'
cmd=build prompt='USER INPUT REQUIRED, PRESS <ENTER> WHEN YOU ARE DONE USING THE CONTAINER'

Specifications

Version: 1.39.1
Platform: Ubuntu 20.04

me too

OK, I did find a temporary solution for now:
Keep the httpProbe enabled, and add an option to the command that doesn't fail after the httpProbe check. (--http-probe-exit-on-failure=false)
Also, by default it will retry 5 times, so, it is a good idea to set it to 1 (Can't set it to 0 unfortunately) (--http-probe-retry-count=1)
Here is the command:
docker-slim --report=off build --target 2b7ca628da40 --tag python:test1 --show-blogs --http-probe-retry-count=1 --http-probe-exit-on-failure=false

@alifiroozi80 you should try the --continue-after parameter. Your original command disabled http probing and the app defaulted to the interactive mode instead. You can explicitly pick the continue mode that works better in CI environments. The README has a number of references about that parameter, but here's a high level overview to give you an idea:
You can pick one of these 'continue' modes:

enter - the main app waits until you press enter to finish executing (not great for CI environments)
signal - the main app waits until you send a signal to it (can be used in CI environments)
probe - with this mode the main app waits for the http probes to execute before it continues
exec - with this mode the main app waits for the exec param commands to execute
<timeout-number-in-seconds> - this is one of the modes you should try where you specify the number of seconds you want the main app to wait for the runtime inspection to execute and then the main app continues on its own without any interactions
container.probe - this is relevant when you are using the compose mode where one of the containers in your compose setup is a test container that you'd want to run against your app container

Thank you so much, @kcq.
This is the solution.
I added the --continue-after=10 to my command, and now everything is perfect.
Here is the complete command:
docker-slim --report=off build --target 2b7ca628da40 --tag python:test1 --show-blogs --http-probe=false --continue-after=10

thanks for your help @kcq


DBZ-6655 Log error and throw a DataException on a schema change record

closes: https://issues.redhat.com/browse/DBZ-6655

docs: https://github.com/debezium/debezium-connector-jdbc/pull/22

Applied to main (=2.4) and branch 2.3.


Enable [RETURN] key for search

[RETURN] key should be enabled after typing terms in the search box.

It of course is.
When you created this issue, you were asked to read this document. You seem to have though you are special and this doesn't apply to you, but you are not and it does. If you believe there's a bug, follow the instructions and provide a usable bug report. But this isn't one.

It isn't on MacOS.
Also, please be less arrogant when someone reports a bug. This is a translation tool, not the Linux kernel, and you are not Linus Torvalds.

@godzillante In a display of notable arrogance and self-centeredness, you don't bother with even the tiniest of courtesies of respecting the instructions, but then feel the need to preach to other people about arrogance. You're way out of line here.
This is indeed not Linux. What this is is 20 years of work that you got for free. I owe you nothing — I already gave you plenty. You might want to ponder that next time you feel like opening your mouth to attack an Open Source maintainer.


docs(watch): Update watchOptions types

The ignored property of watchOptions no longer accepts an array of anymatch patterns, and instead accepts an array of string glob patterns.
This PR updates the watchOptions.ignored types to reflect this change. Also adds an additional example.
Closes #11903

Whoops, I realize now my issue reference is for an issue in the webpack repo. This is the issue that I believe would be addressed by this PR: https://github.com/webpack/webpack/issues/11903


Error attempting to install release 0.1.0 in Chrome

Followed the non-developer instructions, error when loading the unpacked directory:

@tspoff Did the installation successfully after this error?

fixed


upgrade tty-table

Closing as I hadn't noticed someone sent basically the same PR just a bit before I did: https://github.com/changesets/changesets/pull/796


fix(@angular/cli): properly set multi-target command flag

Fixes: #10531

@clydin This doesn't fix the issue with the build command.
build.ts also needs:
this.multiTarget = true;

This results in an issue where if a 1.x CLI project is updated to 6.x, and then a library is added to it, then the build command fails
I believe it's 'cause when .angular-cli.json is migrated to angular.json, defaultProject is not set.

@shahmirn I just tested with a new updated project and I think you're right. defaultProject does not get set but should.


Add Wichita State University

the university official website URL, if it is different from the domain you are submitting

https://www.wichita.edu/

a URL of a page on the official website where a long-term (>1 year) IT related course is offered by the university

http://catalog.wichita.edu/undergraduate/engineering/school-computing/bs-computer-science/#requirementstext

a URL of a page or some other proof (.pdf or a screenshot are OK) showing that the university recognizes the domain which you are submitting as an official email domain for the enrolled students.

https://www.wichita.edu/services/its/Email.php

@willediger Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


Marker Lid validate for Inband updates

This commit adds changes to call the validate API
from phosphor-software-manager to validate the
marker lid. And further send the status of the validation to PHYP through a sensor event.

Did you get to test ibm-i?


Question: Way to visit directly to Sign Up tab?

Is there a way to directly navigate to /wp-login.php and have the Sign Up tab be displaying instead of the login form?
Would be great if you could pass a query parameter like /wp-login.php?initialscreen=signup or /wp-login.php?action=singup
I've looked through the documentation and have found that in Lock for Web this is an option, but im not sure if its implemented in wp-auth0
https://auth0.com/docs/libraries/lock/v11/configuration#initialscreen-string-

Just a suggestion if this is to be an enhancement, it might be nice to extend support for all Lock options. Something like:
/wp-login.php?auth0_options=eyJpbnRpYWxzY3JlZW4iOiJzaWdudXAifQ==
where the query param would be auth0_options and the value would be a base64 JSON object.


Add webpack as a dependency, otherwise it doesn't build without a globally installed webpack-cli

Thank you! My bad I didn't add it.


Getting started page should provide instructions on how to build from source

In the getting started page at https://pulsar.incubator.apache.org/docs/latest/getting-started/LocalCluster/, we have instructions on how to download both the source and binary distribution archive, though we don't have instruction on how to compile the source release.
Another minor thing would be to mention that you need JDK to compile source release and JRE (or JDK which includes it) to run the binary.

cc/ @lucperkins

Link to mailing list discussion<EMAIL_ADDRESS>

Fixed in #648


Public unread notifications: feature or bug?

Yo. I didn't want to post this on the forums.
Anybody with an access token can see how many unread notifications any user has.
{
  "id": 42245,
  "display_name": "KuroiKitsu",
  "anime_time": 0,
  "manga_chap": 0,
  "about": null,
  "list_order": 0,
  "adult_content": false,
  "forum_homepage": null,
  "following": false,
  "image_url_lge": "http://anilist.co/img/dir/user/reg/42245-76UbOGQ0alB5.png",
  "image_url_med": "http://anilist.co/img/dir/user/sml/42245-76UbOGQ0alB5.png",
  "image_url_banner": null,
  "title_language": "romaji",
  "score_type": 4,
  "custom_list_anime": [
    null,
    null,
    null,
    null,
    null
  ],
  "custom_list_manga": [
    null,
    null,
    null,
    "",
    ""
  ],
  "advanced_rating": false,
  "advanced_rating_names": [
    "Story",
    "Characters",
    "Visuals",
    "Audio"
  ],
  "notifications": 0
}

Feature or bug?

I guess I just kinda forgot about that, I suppose it is a feature, not that I can think of any real use for it on its own. I think it would be a bit of a stretch to call this a privacy issue as its only possible to actually view the notifications when authenticated as the user.
For now I'll leave it for consistences sake, I'll probably remove it when not authenticated whenever we have a big api version bump.

(...), I suppose it is a feature, not that I can think of any real use for it on its own.

Something similar to WhatsApp "double check".


fix: remove jsonwebtoken dep

:tada: This PR is included in version 1.1.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


BetterConllCorpusReader

Issue: #1962
I've implemented according to https://stackoverflow.com/a/46875652/610569

[CI: retest]

Sorry to put it this way but this is a really naive implementation and should be immediately rejected. The SO answer you link to (written by myself) shows how to get the job done without modifying the nltk codebase. In a patch request you need to modify CONLLCorpusReader itself, not derive a subclass!
Also, as my answer points out it is trivial to make the code accept multiple columns, and this should be done as well. (I left it out of the answer because the question did not need it.) This is how:
def _get_iob_words(self, grid, tagset=None, columns="chunk"):
        if isinstance(columns, str):
            columns = [ columns]
        pos_tags = self._get_column(grid, self._colmap['pos'])
        if tagset and tagset != self._tagset:
            pos_tags = [map_tag(self._tagset, tagset, t) for t in pos_tags]
        return list(zip(self._get_column(grid, self._colmap['words']), pos_tags,
                   *list(self._get_column(grid, self._colmap[c]) for c in columns)))

I.e. I changed column to columns, and modified the rest of the code to expect a list but accept a single argument as a special case. This matches the standard nltk handling, e.g., of fileids or categories. I haven't tested the code-- you can do that and submit a new pull request.
iob_words() only needs to rename column to columns, since all it does is pass this argument to _get_iob_words().

PS @alvations, what were you thinking? You didn't need me to spell all this out.

@alexisdimi sorry, I didn't look at the PR carefully. I think it's still worth extending the current CoNLL reader to support user defined columns built upon the solution in SO.

Hi, I have updated the code with Alexis input.
I am getting errors on my Windows machine when I try to use iob_words()
I will test on a different machine and see if I get the same error.
Also, I will redo this push, since I deleted the wrong line.
Thank you for the help!

I did some test with the original question.
>>> from nltk.corpus.reader import ConllCorpusReader
>>> a = ConllCorpusReader(r'/Users/jae/Documents/nltk_test/', r'test.txt', ('words', 'pos', 'chunk', 'ne'))
>>> a.iob_words()
[(u'WORD', u'POS', u'CHUNK'), (u'U.N.', u'NNP', u'I-NP'), ...]
>>> a._get_iob_words(a._grids())
[([u'WORD', u'POS', u'CHUNK', u'NE'], [u'U.N.', u'NNP', u'I-NP', u'I-ORG'], [u'official', u'NN', u'I-NP', u'O'])]
>>> a._get_iob_words(a._grids(), columns=['ne'])
[([u'WORD', u'POS', u'CHUNK', u'NE'], [u'U.N.', u'NNP', u'I-NP', u'I-ORG'], [u'Ekeus', u'NNP', u'I-NP', u'I-PER'])]
>>> a._get_iob_words(a._grids(), columns=['chunk', 'ne'])
[([u'WORD', u'POS', u'CHUNK', u'NE'], [u'U.N.', u'NNP', u'I-NP', u'I-ORG'], [u'official', u'NN', u'I-NP', u'O'], [u'Ekeus', u'NNP', u'I-NP', u'I-PER'])]

Is this expected behavior for this?
Thank you.

*list(...) should work on Python 2.7:
Python 2.7.13 (default, Dec 18 2016, 05:36:03)
[GCC 4.2.1 Compatible Apple LLVM 8.0.0 (clang-800.0.38)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> def f(*args):
...     print args
...
...
>>> f(*list('abcdefg'))
>>> ('a', 'b', 'c', 'd', 'e', 'f', 'g')

Apparently, string literals are unicode type.
def _get_iob_words(self, grid, tagset=None, columns='chunk'):
    ...
    print(type(columns)) # => <type 'unicode'>

So, rather than checking if column argument is unicode, I did check on list instead.

No need to be sorry.
Fixed.

I would suggest to set columns to None rather then checking whether a list is passed, what if a tuple is passed or a set or any other iterable?
Also, this is a private method, so taking care that an argument could be an item or a list is not necessary. (It might be a good idea for a public API.)
Many nltk methods accept a single string or a list of them, e.g. the fileids argument of corpus readers. This was all handled properly until Jaehoon got rid of my code, which set columns="chunk" as default and then checks for str (or the six construct for "string or unicode", for Python 2 compatibility):
if isinstance(columns, str):
    columns = [ columns ]

Anyway I take it back: Handling more than one extra column correctly is evidently not trivial. Maybe just integrate a (working) patch for one extra column, and the general case can be dealt with later.

So something along the lines of:
def _get_iob_words(self, grid, tagset=None, column='chunk'):
          pos_tags = self._get_column(grid, self._colmap['pos'])
          if tagset and tagset != self._tagset:
              pos_tags = [map_tag(self._tagset, tagset, t) for t in pos_tags]
          return list(zip(self._get_column(grid, self._colmap['words']), pos_tags,
               self._get_column(grid, self._colmap[column]))

The reason why I got rid of following code line, if isinstance(columns, str): is that, in Python 2.7.13, the code will return False for unicode. But, in Python 3.x, they would evaluate unicode as a str instance which the code would return as True.
Python 2
Python 2.7.13 |Anaconda custom (x86_64)| (default, Dec 20 2016, 23:05:08)
[GCC 4.2.1 Compatible Apple LLVM 6.0 (clang-600.0.57)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
Anaconda is brought to you by Continuum Analytics.
Please check out: http://continuum.io/thanks and https://anaconda.org
>>> a = "abc"
>>> b = u"abc"
>>> isinstance(a, str)
True
>>> isinstance(b, str)
False

Python 3
Python 3.6.4 (default, Mar  5 2018, 21:43:20)
[GCC 4.2.1 Compatible Apple LLVM 9.0.0 (clang-900.0.37)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> a = 'abc'
>>> b = u'abc'
>>> isinstance(a, str)
True
>>> isinstance(b, str)
True

I understand about str vs. unicode. The solution is in my previous comment, and uses six -- like all nltk code does for this purpose.

Ah, okay. So, do I one with this string check or just one custom column?

https://github.com/nltk/nltk/pull/2163 reformatted the return statement and explanded it into multiple lines. To resolve the conflict in this PR please take the code from develop and replace self._get_column(grid, self._colmap['chunk']), with *list(self._get_column(grid, self._colmap[c]) for c in columns)

I'll just close this request, since it seems like this is dead.


Thresholding at inference time

Hi, it is not very clear to me how to extrapolate a threshold at segmentation level to discriminate whether a pixel is classified as “normal” or “anomaly". The output anomaly map needs to be normalized with something like this?
normalize_anomalymap = (anomalymap- anomalymap.min()) / (anomalymap.max() - anomalymap.min())
Thanks

I ask for confirmation. My doubt comes from the fact that the probabilities within the anomaly map are not in the range [0,1]. It may be because the maps obtained in the various levels are summed with
anomaly_map = anomaly_map.sum(dim = 0)

Maybe you could set the threshold with the maximum value of the original training dataset samples (all normal).

As for the range of the anomaly map, you can just average the anomaly map, i.e., anomaly_map = anomaly_map.sum(dim=0)/4 (the number of intermediate visual features)

As for the range of the anomaly map, you can just average the anomaly map, i.e., anomaly_map = anomaly_map.sum(dim=0)/4 (the number of intermediate visual features)

This sounds reasonable to me.

Maybe you could set the threshold with the maximum value of the original training dataset samples (all normal).

This, on the other hand, does not seem correct. If, for example, even one prediction related to 'normal' image samples is incorrect, the threshold cannot be correct.
An approach that seems more reasonable is to test multiple thresholds (e.g., 0.4, 0.45, 0.5, 0.55, 0.6) since we are talking about probabilities in the range [0,1], and choose that one that maximizes one measure (e.g., accuracy).

As for the range of the anomaly map, you can just average the anomaly map, i.e., anomaly_map = anomaly_map.sum(dim=0)/4 (the number of intermediate visual features)

Could I ask why in your code you didn't average the anomaly map? averaging seems to be more reasonable when you calculating the pixel-level metric base on anomaly map


[Bug]: Logical table extraction error,

ODC version
4.3.2
OB version
no related
What happened?
After the table creation ticket is executed, click Extract Tables and click Refresh to report an error in the Logic Table Management page
What did you expect to happen?
Logical table extraction is normal
How can we reproduce it (as minimally and precisely as possible)?
After the table creation ticket is executed, click Extract Tables and click Refresh to report an error in the Logic Table Management page
Anything else we need to know?
No response

dirty DDL during dev period.


perf(vite): start warmups after nitro build

🔗 Linked issue
https://github.com/nuxt/nuxt/issues/26211
https://github.com/nuxt/nuxt/issues/27106
📚 Description
Investigating reports of slow startup time for Nuxt, I think there are a couple of ways we can improve things:

use Vite's own warmup mechanism.
ensure we do not block the nitro build with the warmup's heavy fs read operations (it's not a technical block as they run in parallel, but I think in practice it delays it)

Before
After

We definitely warm up fewer files this way (for example, in the test fixture I'm running, we don't warm up plugin files) and this may be a problem.
I'm also - related to https://github.com/nuxt/nuxt/issues/26211 - trying to track down an issue on Windows machines where vite warmup seems to be non-effective in the server build - even passing entry file to server.warmup doesn't yield transform results for key files. (Maybe a Vite bug - cc: @antfu?)

Looks very nice!
Re: Vite warmup: I have not been using Windows for a long while to tell if it's a Vite bug. I guess it would need a reproduction to make sure it's about vite (or maybe our path normalization?)


two point in ways are very close error when connecting a new way to end of existing ways

Description
When I draw a line that ends at the node of another line and then click again to end this line it seems to create another node. Then I get an error that says "Two Points in sidewalk are very close together."
https://user-images.githubusercontent.com/5851115/212987732-67645ae5-34d2-4cb8-823f-c54212739d6f.mov
Version
2.00-alpha.3.3
What browser are you seeing the problem on? What version are you running?
Chrome v109.0
Steps to reproduce
When I am creating sidewalks I like to create a new sidewalk for each block. So I create a line from one corner of the block to the other. I make that line into a sidewalk. Then I create another line from another corner that should connect to the line I just created. The light comes on when I hover over the endpoint so I think I am connecting them. Then I click again to end the line. When I click again it is making another node on top of the node I just created. See movie in description.
The browser URL at the time you encountered the bug
https://mapwith.ai/rapid-v2-alpha3#background=Bing&datasets=fbRoads,msBuildings&disable_features=boundaries&map=20.06/44.08721/-103.20381

Yep, this has been fixed as part of #807 and will be pushed live soon! Thanks for the bug report...


Fix v1.3

Pls work

Codecov Report

Merging #88 (d52e8e6) into master (cec8e56) will increase coverage by 0.33%.
The diff coverage is 72.24%.

@@             Coverage Diff              @@
##             master      #88      +/-   ##
============================================
+ Coverage     68.99%   69.32%   +0.33%
- Complexity      427      495      +68
============================================
  Files            76       82       +6
  Lines          1406     1666     +260
  Branches        142      182      +40
============================================
+ Hits            970     1155     +185
- Misses          393      447      +54
- Partials         43       64      +21

Impacted Files
Coverage Δ

...c/main/java/seedu/address/model/person/IsDone.java
71.42% <ø> (ø)

.../java/seedu/address/model/util/SampleDataUtil.java
0.00% <ø> (ø)

src/main/java/seedu/address/ui/FullPersonCard.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/MainWindow.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/PersonCard.java
0.00% <0.00%> (ø)

...rc/main/java/seedu/address/ui/PersonListPanel.java
0.00% <0.00%> (ø)

...n/java/seedu/address/storage/CsvAdaptedPerson.java
87.93% <68.42%> (-9.57%)
:arrow_down:

...seedu/address/model/person/interests/Interest.java
72.72% <72.72%> (ø)

.../address/model/person/interests/InterestsList.java
73.80% <73.80%> (ø)

src/main/java/seedu/address/model/person/Age.java
78.57% <78.57%> (ø)

... and 11 more

Continue to review full report at Codecov.

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


Add image via url instead of upload

i would like to ask for url instead of ask for uploading a file, is it possible to have a custom function but not after asking for the file upload ? i'm taking the handleImageAdded example, but that function is triggered after the upload, i would like to do it before the upload pop up window..
thanks

This issue is stale because it has been open for 60 days with no activity.

This issue was closed because it has been inactive for 7 days since being marked as stale.


do not use the path/filepath package

the io/fs package is platform independent, so paths are slash-separated on all systems, even Windows.

https://pkg.go.dev/io/fs#ValidPath
ValidPath reports whether the given path name is valid for use in a call to Open.
Path names passed to open are UTF-8-encoded, unrooted, slash-separated sequences of path elements, like “x/y/z”. Path names must not contain an element that is “.” or “..” or the empty string, except for the special case that the root directory is named “.”. Paths must not start or end with a slash: “/x” and “x/” are invalid.
Note that paths are slash-separated on all systems, even Windows. Paths containing other characters such as backslash and colon are accepted as valid, but those characters must never be interpreted by an FS implementation as path element separators.

On the other hand, the path/filepath package is dependent. Paths are slash-separated on unix-like system, but they are backslash-separated on Windows.

https://pkg.go.dev/path/filepath
Package filepath implements utility routines for manipulating filename paths in a way compatible with the target operating system-defined file paths.
The filepath package uses either forward slashes or backslashes, depending on the operating system. To process paths such as URLs that always use forward slashes regardless of the operating system, see the path package.

You should not use the path/filepath package for handling paths for fs.FS.

@shogo82148
Thank you for your review.

On the other hand, the path/filepath package is dependent. Paths are slash-separated on unix-like system, but they are backslash-separated on Windows.

I didn't implement the code considering the Windows environment.

You should not use the path/filepath package for handling paths for fs.FS.

Your review is correct. Be careful from the next time.


Troubleshoot Helm error when using Azure Issuer

Fixes https://github.com/cert-manager/cert-manager/issues/4867
There's a particularly confusing message when using an Azure Issuer in Helm with the AKS Kubelet identity. This is just something that requires documenting how to get around.

I've tried to maintain the style. Happy to make amends with guidance if necessary

Updated PR to include @irbekrm's suggestion and improved wording. I've changed a couple of stylistic things and a couple of typos from their suggested changed, but mostly imported as-is

Thanks for making the changes @MrSimonEmms !
Just one more small nit- can we move this addition to the pod identities section (so around line 136)?
At the moment it has been added to kubelet identity section which should not require ambient credentials flag

@irbekrm that's not a nit at all - I'd gone code-blind and put it in the wrong section. Good spot

Awesome, this now looks great to me!
Thanks again for taking the initiative to document this bit!
/ok-to-test
/lgtm
/approve


Error message is displaying at improper place thus making UI not so good looking

I want to work on this issue under JWOC

plz provide screen short of the issue u want to solve

There, the alert message is not in the proper place

Can u update the code . and see if its still required

Sorry, I didn't get you

I want to work on this under jwoc

@ManasaKaza its already been days , I will have to assign it to @khushimarothi if its not done by tonight

I want to work on this issue under JWOC

I would like to work on this under JWOC


Optimizing Boyer–Moore majority vote algorithm

In code present in java folder https://github.com/TesseractCoding/NeoAlgo/blob/master/Java/Algorithms/Moore_Algorithm.java complexity of this algorithm is Quadratic.
but according to Boyer–Moore majority vote algorithm the time complexity should be linear.
I would like to fix this .

@iamrajiv @atarax665 Please assign this to me.

@iamrajiv @atarax665 Please assign this to me.


Keep Getting: Uncaught TypeError: Cannot set properties of undefined (setting 'omap')

Issue summary
I am keep getting this error
Uncaught TypeError: Cannot set properties of undefined (setting 'omap')
    at jquery.terminal.js:264:7
    at jquery.terminal.js:227:9
    at jquery.terminal.js:183:1
(anonymous) @ jquery.terminal.js:264
(anonymous) @ jquery.terminal.js:227
(anonymous) @ jquery.terminal.js:183

...when trying to import jQuery Terminal.
Steps to reproduce
Just getting started.
Either import using CDN links, or import as ESM.
Browser and OS
Windows 10
Webview2 (MS Edge)
Additional notes
Using Svelte as a frontend!

Do you include jQuery? It looks like it's not defined. Can you create a reproduction of the issue? Something I can test?

I did included the jQuery lib already, because I also have other libraries that depends on the jQuery. Those libraries work tho.
I cannot reproduce the issue because Svelte REPL doesn't support TypeScript:
https://svelte.dev/repl/860f5ef210634b16803ea32af65b67ba?version=3.55.1
Maybe it only happen with TypeScript?

I was looking at the REPL, sorry I know nothing about Svelte you will need to debug this yourself.

But it looks like the scripts are just added to the document and the next script doesn't wait for the previous script as it happens with normal HTML. Any reason why you use the Svelte compiler with jQuery plugins? It's like a combination of Angular and React in one project (and I don't talk about micro frontends).

I was trying a new framework on my project, but still stuck with some old libraries that still depends on the jQuery (It would take awhile for those libraries to get a new ESM version).
Can you point me to the line of code where the $.omap was extended from? (Where is the $.omap was defined?)
Thanks,

It's at the top of the file, it's a utility that is used like Array::map but for objects. Just after UMD Module definition. To get this error you need to execute JS file without including jQuery. Which I think happens because loading scripts run in parallel. So jQuery Terminal is executed before jQuery is fully parsed and executed.
To fix this I can only think of one hack, that I use to load jQuery in my bookmarks:
    script('https://cdn.jsdelivr.net/npm/jquery');
    (function delay(time) {
        if (typeof jQuery === 'undefined') {
            setTimeout(delay, time);
        } else {
            next(jQuery, function() {
                if (orig_jQuery) {
                    window.jQuery = orig_jQuery;
                    if (dolar) {
                        window.$ = window.jQuery;
                    }
                }
            });
        }
    })(500);

See the rest of the code here: https://github.com/jcubic/lips/blob/master/lib/js/bookmark.js
The alternative is to search for how to load scripts is a sequence in Svelte.

This line I supposed?
https://github.com/jcubic/jquery.terminal/blob/29791ab401fddd8805d7efa4fdbeb038f2ed676e/js/jquery.terminal.js#L264
Thanks for the suggestions!

Solved by adding this line:
$ = window.jQuery

window.jQuery must defined in vite.config.ts
to line number 231 https://github.com/jcubic/jquery.terminal/blob/29791ab401fddd8805d7efa4fdbeb038f2ed676e/js/jquery.terminal.js#L231

True! but I have tried many way. I guess this is the only way for me.

If this has nothing to do with svelte (most likely) If I will have time I will test using the library with just Vite.

I am still very confused of how the root.jQuery is undefined at this line https://github.com/jcubic/jquery.terminal/blob/ef9b63834a9bc2679fae45fd7d308873304016c3/js/jquery.terminal.js#L193
After a few console.log, the root variable should point to window object at that line, and when I do:
console.log(window.jQuery) // <==== This clearly display jQuery is already defined
console.log(root.jQuery) // <=== This is undefined, wth? Should ``root`` be the SAME as ``window``?

Please provide a reproduction of the issue. Share your code somehow e.g. on GitHub

I got a minimal code for you to try here...
xdot-controller-v2.zip
To run the app you need to do a little bit of setup.

Download and unzip file.
In the editor, CD to the main folder: xdot-controller-v2.
You need to install Wails, follow these step: https://wails.io/docs/gettingstarted/installation
Now you can run the app with command: wails dev.
When the app run, you can right click to open the dev tool just like in the browser.

I think this is the only way that I can address the error.
Thanks,

The problem is that you're using import, jQuery Terminal is written as UMD module and you need to run something like this:
const jQuery = require('jquery');
const terminal = require('jquery.terminal');
terminal(jQuery);

it will not work with import, since it's not supported. You need to figure out how to include the library in some other way.
I think that the problem is that you have two version of jQuery one that is hardcoded by your build and one that is loaded by the terminal.

I also think that you can create minimal reproduction, your backend code with go doesn't really matter. The problem is with Svelte. You can probably create new Svelte project put your init code.
You need to learn how to reproduce the issues when you reporting bugs. You don't need 90% of the code you send to reproduce the issue.

I was not able to debug the code because the file was missing, got null as the filename in dev tools. But it seems that the jQuery Code runs after the jQuery Terminal. That's probably why jQuery is missing.

The problem is that you're using import, jQuery Terminal is written as UMD module

I know this lib is in UMD, but why is the src/libs/fomantic-ui/fomantic-ui.js works. It is also in UMD, and required jQuery.

I was not able to debug the code because the file was missing, got null as the filename in dev tools. But it seems that the jQuery Code runs after the jQuery Terminal. That's probably why jQuery is missing.

Were you be able to run wails dev?
What file was missing? Can you share the screenshot of the error?
Thanks,

If you have another library that requires jQuery then probably it loads one jQuery and jQuery Terminal loads another copy.

I have no idea, there are too much happening at the same time.

I will try to create a new svelte project and see what is needed. Will add that to Wiki, when I will solve the problem. Will also try to include two plugins if one will work out of the box.

That would be nice!
Thank you so much for the help!

I'm not sure about your setup, but here is the Svelte code that works:

Hello {name}!

Visit the Svelte tutorial to learn how to build Svelte apps.

I've just created a base project and added jQuery Terminal.

this also works but it's an invalid invocation.
terminal(jQuery)

I've noticed that the unit tests are using this but this is wrong.
This is probably why your code doesn't work with two jQuery plugins.

It works now! 😱
I didn't know we can import like:
import terminal from 'jquery.terminal';

... because my VSCode always give me this error:

How do I get rid of this error, by the way?
Also, please update the jquery.terminal.d.ts as it related to this issue: https://github.com/jcubic/jquery.terminal/issues/877
Thank you soooo much! This error giving me headache for months!!!

Already added it to Wiki Svelte.
I'm wondering about TypeScript, I used JS will check if importing will throw error also in Emacs.
As for the d.ts file it's already updated but I need to release a new version. Maybe will do that over the weekend. But I will need to move the milestone one version because I was planing fixing more issues.

The script works regardless, wondering if it something to do with VSCode IntelliSense?

Should this jquery.terminal.d.ts file has an export keyword to work in ES6 module type?

Hmmm... just found out there is a ticket for this
https://github.com/jcubic/jquery.terminal/issues/844

@KiddoV if you know how to fix it you can contribute. I will be able to release with the code I have on the devel branch. I didn't publish it yet.


No ORM in base, but as package.

@LuckyCyborg @daveismyname
As discussed yesterday, there should be no default ORM in the Nova base.
Or wasn't that the purpose of the new Packages?
@daveismyname

@tomvlk

As discussed yesterday, there should be no default ORM in the Nova base.

Tom, who decided that there should be no default ORM in the Nova base. ?!
#You?
Please note that I've stated clearly that we will take over and implement us the Nova\ORM, and @daveismyname asked clearly about.


srpc: add server support to typescript

This issue tracks:

Implement the server side in TypeScript (currently only has client).
Add ts-jest tests for TypeScript client + server integration.
Add an integration test for Go -> TypeScript with Go as the client.

See #6

Done.


Alias test execution on package setup

Rather than requiring users to run python setup.py pytest, allow the more familiar python setup.py test command to trigger pytest just the same.

Motivated initially by https://github.com/epigen/looper/issues/89


Need CVE disambiguation auto-linking

Description:
A CVE reference auto-linking should not link to a Github advisory if Github didn't assign the CVE ID (meaning the CVE came from somewhere else).
These kinds of CVE auto-linking should instead link to the MITRE (or other online CVE database) for that entry instead.
This will make it obvious that the CVE is bigger than the one project on github that is using the existing CVE id to update the impacted packages entries.
History:
The recent big CVE, CVE-2023-44487 (HTTP/2 Rapid Reset) was originally filed as a spec level CVE (against the HTTP/2 spec itself).
Over time, many projects have referenced CVE-2023-44487 in discussions, pull requests, issues, etc.
The use of CVE-2023-4487 auto-links to the https://github.com/advisories/GHSA-qppj-fm5r-hxr3 advisory, which is for a limited set of impacted packages on the swift-nio-http2 project. (see listing at https://nvd.nist.gov/vuln/detail/CVE-2023-44487)
See concern brought up at https://github.com/github/advisory-database/pull/2860
This results in confusing links to an unrelated project and advisory.
Examples in Discussions:

https://github.com/Azure/api-management-self-hosted-gateway/discussions/255

Examples in Release Notes:

https://github.com/jetty/jetty.project/releases/tag/jetty-12.0.2

Examples in Issues:

https://github.com/linkedin/cruise-control/issues/2070

Examples in PRs:

https://github.com/jetty/jetty.project/pull/10682
https://github.com/openshift/lvm-operator/pull/452
https://github.com/opendatahub-io/rest-proxy/pull/20
https://github.com/apache/httpd-site/pull/10

Some other github advisories referring to CVE-2023-44487

GHSA-qppj-fm5r-hxr3 - using the assigned CVE ID field as CVE-2023-44487
GHSA-xpw8-rcwv-8f8p - simply a reference in the description
GHSA-vx74-f528-fxqg - a reference in the description
GHSA-2m7v-gc89-fjqf - using CVE-2023-44487 as the assigned CVE ID

The github advisory database seems to be enforcing a 1:1 mapping of CVE ID to software project, see here https://github.com/github/advisory-database/pull/2868 The problem this creates is the NVD doesn't have this limitation. The CPE data for https://nvd.nist.gov/vuln/detail/CVE-2023-44487 spans a number of software projects including all of the referenced GHSA above.
This seems like something that could use a redesign to both address the disambiguation of links as well as making the GHSA metadata more accurate by allowing for multiple GHSA to reference the same CVE ID when it makes sense. The implication is downstream tools that consume the advisory-database currently have no way indicate these various GHSA all refer to https://nvd.nist.gov/vuln/detail/CVE-2023-44487 and when I'm reviewing my software for vulnerability to this issue it's much more cumbersome to correlate GHSA ids with randomized names. It also hurts my ability to communicate this with other members of my team and customers since they will all know the CVE ID for this, but few will know the GHSA.
I love the advisory-database and feel it provides amazing detail on fix versions in projects that may otherwise never file their own CVE. I just want to also be able to have those projects reference relevant CVE ID in a coherent manner.

Hey all, thank you for writing in about this issue. We're considering approaches to resolve it. I'll leave this issue open for now in case other want to chime in and comment.


[Feature request] add a way to open the three-lines menu with the keyboard

There is no way to open the 3-lines menu with the keyboard. I propose that it is added. Pressing the Alt key alone to open it would probably be best and in line with the classic notepad.

Agreed, I will do it.

@JasonStein Can I take a shot at this? And could I also take a shot at adding the ability to change the default keyboard shortcuts in settings?


Sui staking none adding stake

Hi
I have again tried today to stake further but no validator is working or allowing me to add funds :) i have tried all
Kind regards
Steve

Yep, seeing multiple reports of issues with this - seems to be affecting most if not all users

resolved thank you


force dropdown to close

I have made a button in my "noResults" slot which fires a method when clicked, in that method i'd like to close the dropdown afterwards, how can this be done? :)

This is a problem of the compilation scope. The slots are compiled using your parent scope, so they don’t have access to the Multiselect instance.
I would try changing the current focus to an element outside of Multiselect (this will force the dropdown to close).

Alright i'll do that, thanks for the response :)

No problem :)


rm role/binding for cluster-policy-controller leader lock, goes in kcm-o

role & rolebinding for cluster-policy-controller lease leader lock move to openshift/cluster-kube-controller-manager-operator.
PR to add to kcm-o: https://github.com/openshift/cluster-kube-controller-manager-operator/pull/299

/hold until this merges: openshift/cluster-kube-controller-manager-operator#299

/hold  cancel

/retest

/approve


Document diff mode

The diff mode feature I see on pull requests is very useful. Here's how I access it:

Modify a spec, build the diff. For instance, click on the "Diff" link in the "Preview | Diff" section of https://github.com/WICG/turtledove/pull/1180. (I'm not sure how to do this from the command line, it doesn't seem to be documented -- maybe it's another tool and not Bikeshed that does this, or maybe not? I couldn't figure it out looking at the GitHub build log for that PR.)

The Bikeshed documentation doesn't mention this at all though, AFAICT? Neither does bikeshed --help.
It would be nice to mention how to build a diff from the command line. Also, it would be nice to document keyboard shortcuts, like "j" and "k", and the next, back and 1 of N UI widget at the bottom right.
If this isn't a Bikeshed feature, do you know how the diff page gets built by the GitHub action?

I just ran into the exact same issue.
Ended up having to create a temporary pull request just to generate the diff document.

Right, it's not documented because that's not something I'm responsible for at all. ^_^ That's tobie's pr-preview.

Ah, interesting, looks like that GitHub action [0] invokes a service [1] that wraps a Perl script [2] that produces HTML that embeds this navigation script [3] that supports keyboard shortcuts "j", "k", and "escape" (to cancel movement) in addition to the UI buttons.
Presumably, you could run htmldiff.pl locally with the old and new HTML rendered specs to produce the same result.
[0] https://github.com/tobie/pr-preview
[1] https://services.w3.org/htmldiff
[2] https://github.com/w3c/htmldiff-ui/blob/main/htmldiff.pl
[3] https://github.com/w3c/htmldiff-nav/blob/ab28ebba333cc88407da0621f046c8f10eed1d52/index.js#L121


tf-module: control creation via variables

@sidick Please review PR and grant feasibility to control creation of policy from variables


Supertest minimal test case

It should be useful add a minimal test case for use an automated test suite (e.g. Supertest) to test the methods online.
Very useful project. Thank's

Can you give me more information about what you need?

Surely.
I use a Vite project, Vue3 in Typescript, I would test the backend API using Supertest framework and Vitest.
I imagine one thing as is :
  it('should post data to API Swagger service', async () => {
    const body = {
      "level": "CATALOG",
      "item": "MOTOR",
    }
    const response = await postGraphs(body);
    expect(response.statusCode).toBe(200);
  })

where postGraphs is your output method from a swagger definition.
Now, the result is
Failed to fully serialize error: 'get onreadystatechange' called on an object that is not a valid instance of XMLHttpRequest

I think is need to config the Axios instance, but I don't know how to.
Any suggestions, please ?

What is your axios version?

This seems to be the XmlHttpRequest issue which used by the Axios.


As a student, I can modify the deadline without deleting the task

... so that I will not waste time recreating the task

YEY!


Add ability to set skills.prof.minMillisToPrint for specific endpoints

currently, skills.prof.minMillisToPrint is used with CallStackProfAspect to show profiling for any method that takes longer than the specified time if the class is annotated with @skills.profile.EnableCallStackProf - add the ability to override that value for specific methods.

PR: #1243 1243


pull request for e2e test at 12/27/2016 07:26:07

:white_check_mark: Validation status: passed
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.


MINOR: updates docs for KIP-358

More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Call for review @guozhangwang @bbejeck @vvcephei @nizhikov

Hello, @mjsax
How can I view docs locally?
My browser shows a white screen when I open HTML file from sources.

@nizhikov you can follow this page to setup apache site locally from the source code: https://cwiki.apache.org/confluence/display/KAFKA/Contributing+Website+Documentation+Changes


Error: expect(jest.fn())[.not].toHaveBeenCalledTimes()  jest.fn() value must be a mock function or spy. Received:   function: [Function getItem]

🐛 Bug Report
I get
Error: expect(jest.fn())[.not].toHaveBeenCalledTimes()

jest.fn() value must be a mock function or spy.
Received:
  function: [Function getItem]

import localStorageWrapper from "./localStorage";

const mockStorage = {
  string: "some string value",
  object: `{"foo": "bar"}`
};
beforeEach(() => {
  window.localStorage = {
    getItem: jest.fn(key => mockStorage[key]),
    setItem: jest.fn()
  };
});

describe("get", () => {
  it("should get string item", () => {
    const key = "string";
    const result = localStorageWrapper.get(key);

    expect(window.localStorage.getItem).toHaveBeenCalledTimes(1);
    expect(window.localStorage.getItem.mock.calls[0][0]).toEqual(key);
    expect(result).toEqual(mockStorage.string);
  });

  it("should get object item", () => {
    const key = "object";
    const result = localStorageWrapper.get(key);

    expect(window.localStorage.getItem).toHaveBeenCalledTimes(1);
    expect(window.localStorage.getItem.mock.calls[0][0]).toEqual(key);
    expect(result).toEqual(JSON.parse(mockStorage.object));
  });
});

describe("set", () => {
  it("should set string item", () => {
    localStorageWrapper.set("foo", "bar");

    expect(window.localStorage.setItem).toHaveBeenCalledTimes(1);
    expect(window.localStorage.setItem.mock.calls[0]).toMatchSnapshot();
  });

  it("should set object item", () => {
    localStorageWrapper.set("some_object", { foo: "bar" });

    expect(window.localStorage.setItem).toHaveBeenCalledTimes(1);
    expect(window.localStorage.setItem.mock.calls[0]).toMatchSnapshot();
  });
});

To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Link to repl or repo (highly encouraged)
Please provide either a repl.it demo or a minimal repository on GitHub.
Issues without a reproduction link are likely to stall.
Run npx envinfo --preset jest
Paste the results here:

Jest uses jsdom under the hood for simulating dom. In browsers and in jsdom, you cannot just set window.localStorage to new object, because setter function for it is undefined.
Instead do this in order to override original window.localStorage
beforeEach(() => {
  const localStorageMock = {
    getItem: jest.fn(key => mockStorage[key]),
    setItem: jest.fn((args) => console.log(args)),
  };
  Object.defineProperty(window, 'localStorage', {
    value: localStorageMock,
    writable: true
  })
});

Your code was working in the older version of Jest, because jsdom did not have it's own implementation for localStorage. window.localStorage was undefined and you could assign whatever you wanted to it.

Nice answer @grosto, thanks!


Announcement draft

https://medium.com/@nopara73/introducing-zerolink-the-bitcoin-fungibility-framework-dc5338086198

[x] Fix typos
[x] Explain how it works in more detail

Announcement draft should be kept as short as possible. Details will be found on github page.

I am wondering if we even need a medium announcement. Maybe just stick with the GitHub?

I think that the Medium announcement is useful in terms of visibility if kept to a minimum that requires no updating afterwards.

Finished.


Removed Preview from the Title

Removed Preview from the Title

@ndolker,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@ndolker : Thanks for your contribution to the Azure documentation! The author, @TomShinder, has been notified to review your proposed change.

@sign-off


[fix] Add babelify to support non-es6 browsers

Should fix #5

Do I just press the merge button? :sweat_smile:

I'm a bit busy, I will try to look at it wednesday.

Lovely. I have confirmed that source mapping, it convert to ES5, works (except minor layout errors) in Safari.

Published as 0.14.0


Feature: linter specific values

Output of helm version:
version.BuildInfo{Version:"v3.0.2", GitCommit:"19e47ee3283ae98139d98460de796c1be1e3975f", GitTreeState:"clean", GoVersion:"go1.13.5"}
This is a feature request to solve common issues with linter. Currently having values not defined breaks helm lint. But for some values, it's dangerous to add defaults and the template designer wants the template to fail unless the user has reviewed and set this particular values.
I suggest that helm should read a special values file (e.g. "values-lint.yaml") in addition to values.yaml only if running helm lint to solve this.

Here's my understanding of this feature request:

values.yaml should be loaded first
lint-values.yaml should be loaded second, and merged over values.yaml
The -f values should be loaded third, merged over the set above
--set-style values should be merged after that

1, 3, and 4 exist already. So this would be adding 2. The danger here is that if lint-values.yaml overloads values.yaml, it may obscure problems with values.yaml (thus marking it as successful even if values.yaml is broken). So I will experiment with inverting that order.

This is blocked until #7984 is merged (since that has all the supporting code necessary for this PR)

closing as stale.


Error with contract verification

The smart contract was launched to the Rinkbey Testnet. However, the contract verification failed using the flat form of the smart contract. The following function returned the error.
  function useChainlinkWithENS(address ensAddress, bytes32 node) internal {
    s_ens = ENSInterface(ensAddress);
    s_ensNode = node;
    bytes32 linkSubnode = keccak256(abi.encodePacked(s_ensNode, ENS_TOKEN_SUBNAME));
    ENSResolver_Chainlink resolver = ENSResolver_Chainlink(s_ens.resolver(linkSubnode));
    setChainlinkToken(resolver.addr(linkSubnode));
    updateChainlinkOracleWithENS();
  }

The error read out is as follows.
DeclarationError: Identifier not found or not unique.
--> SmartContractLink_flat.sol:1025:5:
|
1025 | ENSResolver_Chainlink resolver = ENSResolver_Chainlink(s_ens.resolver(linkSubnode));
| ^^^^^^^^^^^^^^^^^^^^^

Etherscan verification error.

To resolve this Issue, I commented out two functions using the ENSResolver from the flat form of the smart contract.

This allowed Etherscan to verify the contract on Rinkbey.


feat improved some queries and deleted directories to simplify project

If you add "Resolves #33" to your comment it will auto-close that issue when this is merged.


CXF-8121 Improve STS REST interface

Thanks @dtsybulka, LGTM, @ashakirin anything you want to add / comment / change?

@reta, please hold off on the merge for a while, there are some things I need to check first with the PR.

@coheigea no objections, won't merge without your and @ashakirin approvals, thanks!

@coheigea could you please tell when you could check and review the PR? Thanks in advance.

@dtsybulka It's on my TODO list, but realistically it will be early in the new year before I can review it properly.

Hi @coheigea,
I hope you are fine. Could you please let me know if you had a chance to check the PR.
Best Regards,
Dmitry

Sorry, it'll be a few weeks before I can look at it, but it is on my radar for CXF 3.4.0.

Also, there is a merge conflict with CustomParameterTest.


Add canInitialise to GOVUKFrontendComponent

What
Add canInitialise to GOVUKFrontendComponent and associated test.
Why
If user extends GOVUKFrontendComponent they can define a function that will be executed before initialisation. If function returns false, then component will not initialise.
Fixes https://github.com/alphagov/govuk-frontend/issues/5225

Closing as we're picking a different route after noticing the overlap with the existing checkSupport.


Add onabort to XHR classes

04d870b added support for onabort in XHRs. The other on* events are declared on XMLHttpRequest and XMLHttpRequestEventTarget. This adds onabort there as well.
Adam Comella
Microsoft Corp.

lgtm!


.local names can only be resolved from wsl2 connected via Ethernet but not via WiFi

Environment
Windows build number: 2004
Your Distribution version: Ubuntu
Whether the issue is on WSL 2 and/or WSL 1: WSL2 only, it doesn't work on WSL1 at all

Steps to reproduce
Install Ubuntu on WSL2 and maybe install avahi-daemon as well. It's not required to start dbus and/or avahi-daemon. Connect you laptop via Ethernet to a network with other hosts that can be resolved using mDNS. Then just run ping abc.local in Ubuntu and it should just work fine. However when you disconnect the Ethernet cable and connect your host to the network via WiFi, then the ping command doesn't work anymore.
I then booted into Linux natively on the same machine and there I was able to resolve the host via Ethernet and via WiFi, so it's not a problem of the local WiFi router.
Expected behavior
I think .local names should resolve just fine.
Actual behavior
Resolving .local names from Ubuntu doesn't work when the host system is connected via WiFi instead of Ethernet.
Related issues
It sounds like #384 and https://github.com/microsoft/WSL/issues/3365 are related but they are already closed.

Just came across the same issue when trying to connect to my raspberry pi. Installed avahi-daemon and started the service, but still:
❯ ssh<EMAIL_ADDRESS>ssh: Could not resolve hostname raspberrypi.local: Name or service not known

So now it sometimes works via WiFi here as well. However I cannot say yet when it happens and when it doesn't happen. The only thing I can say is that I installed wireshark and it looked like the mDNS request went out but no response came back while a request from the Windows system directly (launching ping via cmd/powershell) would be properly answered by the other host.

I can't get it to work even on ethernet either. Same setup.
At daemon startup, this occurs:
Host name conflict, retrying with <HOSTNAME>-2

and apparently the original  maps to the host Windows machine,  which I can confirm by resolving to it from another machine on the local network (or from WSL for that matter).
Unanswered query (from WSL):
5 4.874430800 <IP_ADDRESS> → <IP_ADDRESS>  MDNS 83 Standard query 0x0000 A raspberrypi.local, "QM" question AAAA raspberrypi.local, "QM" question

Answered Windows host IP resolution (from WSL):
 34<PHONE_NUMBER>00 fe80::215:5dff:fefb:ca31 → ff02::fb     MDNS 103 Standard query 0x0000 A <HOSTNAME>.local, "QM" question AAAA <HOSTNAME>.local, "QM" question
 35<PHONE_NUMBER>00 <IP_ADDRESS> → <IP_ADDRESS>  MDNS 83 Standard query 0x0000 A <HOSTNAME>.local, "QM" question AAAA <HOSTNAME>.local, "QM" question
 36<PHONE_NUMBER>00 <IP_ADDRESS> → <IP_ADDRESS>  MDNS 135 Standard query response 0x0000 A, cache flush <IP_ADDRESS> AAAA, cache flush fe80::149d:d920:25b8:4b37 NSEC, cache flush <HOSTNAME>.local
 37<PHONE_NUMBER>00 fe80::149d:d920:25b8:4b37 → ff02::fb     MDNS 152 Standard query response 0x0000 A <IP_ADDRESS> AAAA fe80::149d:d920:25b8:4b37
 38<PHONE_NUMBER>00 <IP_ADDRESS> → <IP_ADDRESS>  MDNS 132 Standard query response 0x0000 A <IP_ADDRESS> AAAA fe80::149d:d920:25b8:4b37

I can't get it to work even on ethernet either. Same setup.
At daemon startup, this occurs:
Host name conflict, retrying with <HOSTNAME>-2

and apparently the original  maps to the host Windows machine,  which I can confirm by resolving to it from another machine on the local network (or from WSL for that matter).
Unanswered query (from WSL):
5 4.874430800 <IP_ADDRESS> → <IP_ADDRESS>  MDNS 83 Standard query 0x0000 A raspberrypi.local, "QM" question AAAA raspberrypi.local, "QM" question

Answered Windows host IP resolution (from WSL):
 34<PHONE_NUMBER>00 fe80::215:5dff:fefb:ca31 → ff02::fb     MDNS 103 Standard query 0x0000 A <HOSTNAME>.local, "QM" question AAAA <HOSTNAME>.local, "QM" question
 35<PHONE_NUMBER>00 <IP_ADDRESS> → <IP_ADDRESS>  MDNS 83 Standard query 0x0000 A <HOSTNAME>.local, "QM" question AAAA <HOSTNAME>.local, "QM" question
 36<PHONE_NUMBER>00 <IP_ADDRESS> → <IP_ADDRESS>  MDNS 135 Standard query response 0x0000 A, cache flush <IP_ADDRESS> AAAA, cache flush fe80::149d:d920:25b8:4b37 NSEC, cache flush <HOSTNAME>.local
 37<PHONE_NUMBER>00 fe80::149d:d920:25b8:4b37 → ff02::fb     MDNS 152 Standard query response 0x0000 A <IP_ADDRESS> AAAA fe80::149d:d920:25b8:4b37
 38<PHONE_NUMBER>00 <IP_ADDRESS> → <IP_ADDRESS>  MDNS 132 Standard query response 0x0000 A <IP_ADDRESS> AAAA fe80::149d:d920:25b8:4b37

Update:
Not running avahi-daemon in WSL at all, makes resolving .local addresses resolve successfully.
As long as MDNS support is enabled on the Windows host (which apparently it is) then trying to resolve a .local address directly using a normal DNS request
7 1.013978000 <IP_ADDRESS> → <IP_ADDRESS> DNS 77 Standard query 0xca9c A raspberrypi.local
8 1.013990900 <IP_ADDRESS> → <IP_ADDRESS> DNS 77 Standard query 0x4587 AAAA raspberrypi.local
9 1.027835500 <IP_ADDRESS> → <IP_ADDRESS>  MDNS 77 Standard query 0x0000 A raspberrypi.local, "QM" question
10 1.029148900 fe80::bd59:20fb:7514:15dc → ff02::fb     MDNS 97 Standard query 0x0000 A raspberrypi.local, "QM" question
11 1.032137000 <IP_ADDRESS> → <IP_ADDRESS> DNS 110 Standard query response 0xca9c A raspberrypi.local A <IP_ADDRESS>
12 1.034535100 <IP_ADDRESS> → <IP_ADDRESS>  MDNS 77 Standard query 0x0000 AAAA raspberrypi.local, "QM" question
13 1.035766800 fe80::bd59:20fb:7514:15dc → ff02::fb     MDNS 97 Standard query 0x0000 AAAA raspberrypi.local, "QM" question
14 1.039550500 <IP_ADDRESS> → <IP_ADDRESS> DNS 122 Standard query response 0x4587 AAAA raspberrypi.local AAAA fe80::c58c:12ee:af24:b1e8

prompts the host to issue an MDNS request and forward the reply as a DNS reply to the WSL DNS request. It works automagically.

Update:
Not running avahi-daemon in WSL at all, makes resolving .local addresses resolve successfully.
As long as MDNS support is enabled on the Windows host (which apparently it is) then trying to resolve a .local address directly using a normal DNS request
7 1.013978000 <IP_ADDRESS> → <IP_ADDRESS> DNS 77 Standard query 0xca9c A raspberrypi.local
8 1.013990900 <IP_ADDRESS> → <IP_ADDRESS> DNS 77 Standard query 0x4587 AAAA raspberrypi.local
9 1.027835500 <IP_ADDRESS> → <IP_ADDRESS>  MDNS 77 Standard query 0x0000 A raspberrypi.local, "QM" question
10 1.029148900 fe80::bd59:20fb:7514:15dc → ff02::fb     MDNS 97 Standard query 0x0000 A raspberrypi.local, "QM" question
11 1.032137000 <IP_ADDRESS> → <IP_ADDRESS> DNS 110 Standard query response 0xca9c A raspberrypi.local A <IP_ADDRESS>
12 1.034535100 <IP_ADDRESS> → <IP_ADDRESS>  MDNS 77 Standard query 0x0000 AAAA raspberrypi.local, "QM" question
13 1.035766800 fe80::bd59:20fb:7514:15dc → ff02::fb     MDNS 97 Standard query 0x0000 AAAA raspberrypi.local, "QM" question
14 1.039550500 <IP_ADDRESS> → <IP_ADDRESS> DNS 122 Standard query response 0x4587 AAAA raspberrypi.local AAAA fe80::c58c:12ee:af24:b1e8

prompts the host to issue an MDNS request and forward the reply as a DNS reply to the WSL DNS request. It works automagically.

I am having a similar issue. I can resolve the MDNS from Windows cmd, but not from Ubuntu/WSL2. The behavior is not consistent either. I have tried both stopping and starting avahi-daemon within Ubuntu/WSL2.
From Windows cmd:
>ping owrt.local

Pinging OWRT.local [<IP_ADDRESS>] with 32 bytes of data:
Reply from <IP_ADDRESS>: bytes=32 time=3ms TTL=64
Reply from <IP_ADDRESS>: bytes=32 time=2ms TTL=64

Ping statistics for <IP_ADDRESS>:
    Packets: Sent = 2, Received = 2, Lost = 0 (0% loss),
Approximate round trip times in milli-seconds:
    Minimum = 2ms, Maximum = 3ms, Average = 2ms

>ping backupbot.local

Pinging backupbot [<IP_ADDRESS>] with 32 bytes of data:
Reply from <IP_ADDRESS>: bytes=32 time=8ms TTL=64
Reply from <IP_ADDRESS>: bytes=32 time=2ms TTL=64
Reply from <IP_ADDRESS>: bytes=32 time=3ms TTL=64

Ping statistics for <IP_ADDRESS>:
    Packets: Sent = 3, Received = 3, Lost = 0 (0% loss),
Approximate round trip times in milli-seconds:
    Minimum = 2ms, Maximum = 8ms, Average = 4ms`

From Ubuntu on WSL2 (with and without avahi-daemon running):
$ sudo service avahi-daemon start
 * Starting Avahi mDNS/DNS-SD Daemon avahi-daemon                                                                [ OK ]
$ ping owrt.local
ping: owrt.local: Name or service not known
$ ping backupbot.local
ping: backupbot.local: Name or service not known
$ sudo service avahi-daemon stop
 * Stopping Avahi mDNS/DNS-SD Daemon avahi-daemon                                                                [ OK ]
$ ping backupbot.local
ping: backupbot.local: Name or service not known
$ ping owrt.local
PING OWRT.local (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from <IP_ADDRESS> (<IP_ADDRESS>): icmp_seq=1 ttl=63 time=4.23 ms
64 bytes from <IP_ADDRESS> (<IP_ADDRESS>): icmp_seq=2 ttl=63 time=1.91 ms
64 bytes from <IP_ADDRESS> (<IP_ADDRESS>): icmp_seq=3 ttl=63 time=5.07 ms
64 bytes from <IP_ADDRESS> (<IP_ADDRESS>): icmp_seq=4 ttl=63 time=6.03 ms
^C
--- OWRT.local ping statistics ---
4 packets transmitted, 4 received, 0% packet loss, time 3099ms
rtt min/avg/max/mdev = 1.906/4.307/6.026/1.525 ms

I am having a similar issue. I can resolve the MDNS from Windows cmd, but not from Ubuntu/WSL2. The behavior is not consistent either. I have tried both stopping and starting avahi-daemon within Ubuntu/WSL2.

Retracting my comment above about 'inconsistent behavior' - this is not specific to Ubuntu/WSL2 on my network and may be due to the version of avahi-daemon on one specific host (backupbot.local).
However, my notes above confirm the behavior noted by vjancik - that closing avahi-daemon in wsl2/ubuntu allows the .local name to resolve successfully, owrt.local in my example.


Fixed Jenkins script and cleaned-up pipeline scripts

Resolves issue #2 and cleans up documentation and extraneous code in pipeline scripts.

@hannes-ucsc I got a workaround going in terms of getting a custom version of toil-lib installed, but I'm still having issue with my general unit-test script. Are there local directories on Jenkins that I can work with in my unit-test?

In your tests use the tempfilepackage to get a temporary directory. In jenkins.sh set TMPDIR to point at the ephemeral volume. Just copy the relevant portions from Toil's jenkins.sh.

@hannes-ucsc I've already had TMPDIR set and exported in my jenkins.sh script. And I've switched my python test script to create a temp directory using tempfile instead of a hard-coded path. I'm running into an issue where an aws cp command isn't downloading the file to the temp directory since apparently the directory doesn't exist: OSError: [Errno 2] No such file or directory
I've just tested the commands in the python unittest script locally on an aws machine and it all works fine. What else am I missing?

The 'No such file or directory' exception is referring to the 'aws' program, not the destination file name or the directory. You should probably install the AWS CLI. You should install it into a separate virtualenv. Check out how toil-scripts' jenkins.sh installs s3am into venv and then puts the s3am binary on the PATH. You could also just use boto instead of calling out to a separate process.

Jenkins, test this please

This should build on every push without the trigger phrase. Let me know if it doesn't.

Jenkins, test this please.

@hannes-ucsc Ready for review! :)

I'm afraid my limited time doesn't permit me to do a full review. This isn't a project I am lead on. But it passes the tests, so this has my blessings.


AWS Lambda Mock

BEFORE SUBMITTING AN ISSUE CHECK THE FOLLOWING:

[X] I checked existing issues.
[X] This is not a support question (which can be answered on Slack).
[X] This issue is for the API / Game Engine code base (not Web Client, Lambda, or Bots)

FEATURE REQUEST?
A feature request constitutes any feature not currently supported by the API / Game Engine that does not fall under a Game Play Request.

Describe the feature you would like to see.
AWS Lambda mock should be accessible for local development without the use of AWS Keys

Why you believe this would improve the quality of the Wombat's API / Game Engine.
This would allow you to dev on the API without AWS keys.

Provide details on expected behavior.

Eric R. will be working on this.


Declarative syntax for autocompletions

I've been thinking of something like this for a while, but I always focused on the idea of automating the autocompletion scripts. It would be really cool to come up with a declarative syntax for defining function arguments that could be compiled into programmable completion in bash.
Here is a contrived example of how a function in bish could be annotated:
def ls(file=None, **kwargs) {
    """
    @file: File= - the name of the file to ls
    @-l - enable long output
    @-a - include hidden files
    @--longopt - longer option for example
    """
    return /bin/ls file? **kwargs
}

This would enable the following behavior:
> ls <Tab>
file - the name of the file to ls
-l  - enable long output
-a  - include hidden files
--longopt - longer option for example

> ls -<Tab>
-l  - enable long output
-a  - include hidden files
--longopt - longer option for example

> ls --long<Tab>
> ls --longopt
> ls --longopt<Tab>
--longopt - longer option for example

You can see a real world example of this type of autocompletion with tmux. Start some sessions and type tmux attach-session -<Tab>. I'm not sure if it would work to this degree in bash as I've only seen it in zsh, but it does offer some level of programmable autocompletion.

@bschlenk Interesting idea! Would the use-case be for using the bish REPL as a shell itself? Or are you thinking that the compiled bash script would register its autocompletion with the user's current shell (similar to Example J-1 in the document you linked)?
PS - you may have noticed, but I haven't had any time to devote to bish for quite a while now. If you'd like to take a crack at implementing this, please do!