yarn: There are no scenarios

Running bootstrap.sh seems to setup the repository correctly, but then just ends with an error message from yarn.
00h00m00s 0/0: : ERROR: There are no scenarios; must have at least one.

I have never used xterm.js. How does one launch it?

Ive never seen that error message, so I am not quite sure what it means. Looking at SO indicates, that you most likely have an outdated yarn installed. You can either update your global yarn (how depends on how it got installed in the first place), or try to use npm install instead:
git clone --depth 1 --branch 4.16.0 https://github.com/xtermjs/xterm.js.git
cd xterm.js
rm -rf .git
cd addons
git clone https://github.com/jerch/xterm-addon-image
cd ..
cp -avx addons/xterm-addon-image/overwrite/* .
# for npm install we have to patch typescript version
sed -i 's/"typescript": "^4.4.4"/"typescript": "4.4.4"/g' package.json
npm install

If that all succeeds you can start playing around with it (run these commands from base repo folder):

yarn start or npm start to get the demo running (point browser to localhost:3000)
yarn test-api or npm run test-api for addon tests
yarn watch or npm run watch to run the TS compiler in watch mode (helpful if you want to make source changes)

And to build the final addon bundles:
cd addons/xterm-addon-image
yarn package   # or npm run package

which creates the browser bundles under addons/xterm-addon-image/lib.
I admit that this tight integration with the base repo is subpar, but it is currently the only way to pull the symbols correctly for the tests and the demo. If I find the time I might further detangle it. Further note that the base repo demo still lacks proper flow control (we discussed that already in the PR thread), thus will stop working for fast succession of big images (see console.log if the demo behaves weird).
If thats still failing, just ping me back.

@hackerb9 Just release an npm version, maybe thats easier to play with. Gonna close the issue, feel free to reopen if something is still missing.


Jane/ptkb 4682/grafana tls

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] DCO signed
[ ] Chart Version bumped
[ ] Variables are documented in the README.md

Apologies, meant to merge into my fork, not here yet


Revisit wording for "Help" in FZ editor

we should revisit this
Initially when I was trying to use the merge feature I was selecting in the middle of the two zones just because the description wasn't as clear?
Instead of saying "To merge zone, select and click "merge"
Say "To merge zone, click and drag from one zone to the second zone"

Visual feedback should been added to know when the left mouse click is held down. (Similar blue color you get when you drag a window into a zone)

EDIT: PowerToys v0.31.2
Originally posted by @komawoyo in https://github.com/microsoft/PowerToys/issues/9670#issuecomment-778486223

There was already https://github.com/microsoft/PowerToys/issues/9441

Still very confusing, had to come here to figure out how to merge zones. Why not say "click inside zone and drag across to another zone to merge zones"?


Indicated Successful Check

Summary
Adds a successful check message after no errors were found
Implements #8553

Test Plan
Ran a check on a test file with cargo run -p ruff_cli -- check test.py --no-cache and outputted as expected.
Ran the same check with cargo run -p ruff_cli -- check test.py --no-cache --silent and the command was gone as expected.

I'm wondering how to add the counting for the number of files checked - I see that it lives in check.rs here but this stat isn't available in diagnostics, is it?

Also in the case of tests like this:

Do we even want to output a successful check?

No, we should fail there but currently don't

This might cause issues with pre-commit since that framework uses "any message at the output" as error. Or bash pre-commit hooks.

No, we should fail there but currently don't

@konstin, by "fail", do you specifically mean that we should be exiting with a non-zero exit code? And perhaps continuing to have nothing dumped to stdout?

by "fail", do you specifically mean that we should be exiting with a non-zero exit code?

yes

Slight preference for something like "All checks passed" over something that contains the word error, otherwise i'm on board.

I like "All checks passed" although it's a little outside our normal terminology

Is there an option to disable this? All my other linters are silent on success, so ruff sticks out like a sore thumb. -q does not work because it would also suppress errors which should stay visible.
poetry run ruff check src
All checks passed!          // why so verbose?
poetry run yamllint .
npx eslint ui
npx stylelint ui

I'm not sure what to say here. A configuration option for this seems excessive and it is significantly friendlier for Ruff to indicate that it did something. If you want to open an issue requesting a configuration option, we can consider it. I'd want to see significant community feedback that this is important first though.

Sorry, I figured out that -q actually works as expected and suppresses it just fine, so nothing to do and sorry for the noise :)

Oh great! :)

that framework uses "any message at the output" as error

https://github.com/aspect-build/rules_lint/blob/main/docs/lint_test.md is one such tool. It is used by Bazel users to assert "no warnings were produced by the tool". So this change broke our tests. From what I can tell, ruff is the only linter we integrate that prints on success.

issue requesting a configuration option, we can consider it. I'd want to see significant community feedback that this is important

Here's some other linter decisions on this:

https://github.com/golangci/golangci-lint/issues/121 - always be silent following https://www.linfo.org/rule_of_silence.html
https://github.com/eslint/eslint/issues/3495#issuecomment-133820141 - eslint is incredibly widely adopted and the person commenting here is one of the most prolific and respected developers in the Node.js ecosystem.
From the (still open?) issue requesting this change, the author observes
"On the other hand, several linters are silent if they find nothing (pycodestyle, flake8, isort, etc.)."

So I think rather than add an option, ruff should go back to the old silent-on-success behavior.

@alexeagle would you mind creating a new issue for this to avoid having discussions on merged PRs. Thank you

@MichaReiser thanks for the quick reply, since the issue requesting this change is still open, I've just commented there: https://github.com/astral-sh/ruff/issues/8553#issuecomment-2022771565

Thanks for moving the discussion!


[libSyntax] Re-enable the UnknownSyntaxTests

These appear to have been commented out for years for no particular reason.

@swift-ci Please smoke test


Staging to main: Merge multiple Dockerfiles into a single one

Merge multiple Dockerfiles into a single one
Signed-off-by: Simon Zhao<EMAIL_ADDRESS>Co-authored-by: Miguel Fierro<EMAIL_ADDRESS>Description

Related Issues

References

Checklist:

[ ] I have followed the contribution guidelines and code style for this project.
[ ] I have added tests covering my contributions.
[ ] I have updated the documentation accordingly.
[ ] I have signed the commits, e.g. git commit -s -m "your commit message".
[ ] This PR is being made to staging branch AND NOT TO main branch.

I checked and the GPU tests that were failing now are working: https://github.com/recommenders-team/recommenders/actions/runs/11831613396
Everything is green.


Convertted depth image should be an uint16 array

Thanks for these functions! They are really useful.
However, in the depth_2_color_space function, the returned depth gets converted to uint8 which I think is not optimal.  The original data is in uint16 range and numpy astype may cause loss in precision, even plain erroneous depth value.
Best regards!

As i don't have a Kinect device to test it now, i cannot be sure if the uint16 is fitted for the depth image. I remember when i tested it that the uint16 resulted in an error depth image but i may be wrong. You can test it out and make a pull request if you think that this is correct.

I am pretty sure it's uint16. The C++ SDK uses UINT16 array to store the depth image. See AccessUnderLyingBuffer and CopyFrameDataToArray. The error depth image might be due to visualization issues. Sometimes the uint16 data get clipped when visualization.

Thanks for the update. I have changed the code to uint16 for the depth image and the aligned image.


Implementing override functionality for XR

This PR adds a bit of cleanup to to the render target API and adds 3 overrides to it: render_target_set_override_color, render_target_set_override_depth and render_target_set_override_velocity.
When setting an override, output is directed to these textures instead of our build in textures (it's not smart enough yet to lazy create these and skip if an override is present, may add that later).
The use case here is to prevent overhead copying data from the build in buffers to the output buffers for XR applications. Especially with XR these are large buffers and on mobile hardware like the Quest copying 6 large textures (3 textures x 2 eyes) has a measurable detrimental effect on performance.
For non XR this opens the door to render fullscreen viewports directly into our swapchain, important for Android, though some research needs to be done to verify this works fine for 2D rendering. I think it does, Reduz has some doubts but this is a matter of checking for situations where this is not possible.
Also important to note, in our use cases we're always cycling through images in swap chains as the images are passed to the XR compositor while we've started rendering the next frame. Hence each frame the overrides are updated in a round robin fashion. This is now possible because of earlier work where we're caching framebuffers and we'll end up cycling through 3 sets of framebuffers as we are rendering.
Note:

Color buffer override is always used with OpenXR
Depth buffer override is used if depth layer or spacewarp is available
Velocity override is used for spacewarp only (atm).
This PR implements depth layer support in OpenXR but Spacewarp we'll implement later, this just builds the foundation.

Known issues:

The texture object currently created for each viewport to allow viewport textures still points to the build in texture to which we no longer render when overrides are used (will deal with this in a separate PR in due time)
This PR has minor compatibility breakage mainly as we're retiring the old overrule function. The only thing effected by this is our Godot 4 OpenVR plugin though that is being superseded by our build in OpenXR functionality anyway.

Todos (these will be done in new PRs after this is merged):

if MSAA is used, we do not have a depth and velocity texture resolve in the mobile renderer yet
velocity is currently only enabled for TAA, it also needs to be enabled when the velocity texture override is specified
velocity is not yet implemented in the mobile renderer and needs to be added and enabled if a velocity texture override is specified

@kisg you might want to try this out together with your Oculus Quest changes, this should have a measurable performance improvement on the Quest.

@clayjohn note that I have removed the old implementation from the GLES3 driver. It currently was a copy of the Godot 3 implementation which no longer applies. We'll have to see in due time what we're going to do with GLES3, it will likely only have an impact for webXR and for webXR we may not require the overrule logic at all because webXR supplies framebuffers we can't use, and thus we're stuck with a copy anyway.

We'll have to see in due time what we're going to do with GLES3, it will likely only have an impact for webXR and for webXR we may not require the overrule logic at all because webXR supplies framebuffers we can't use, and thus we're stuck with a copy anyway.
edit @dsnopek in due time you, me and clay should have a discussion of what we should do with webXR and getting it to work again.

@BastiaanOlij Yes, we should chat about this! But here's a preview of my thoughts on this:
I mentioned this on another issue/PR somewhere, but using the new-ish WebXR Layers spec we can get WebXR to give us two 2D texture arrays (one for the color buffer and another for the depth buffer).
The WebXR Layers spec is supported in Chrome and on the Oculus Browser, but there's also a polyfill that can be used for browsers that don't support it, which will create its own 2D texture arrays and just copy from them to the framebuffer from WebXR, meaning it should be safe to rely on this spec.
So, I think the key to WebXR is (1) having multiview support in the opengl3 driver and (2) using the new WebXR Layers spec rather than "original" method which just gives us that framebuffer.

We'll have to see in due time what we're going to do with GLES3, it will likely only have an impact for webXR and for webXR we may not require the overrule logic at all because webXR supplies framebuffers we can't use, and thus we're stuck with a copy anyway.
edit @dsnopek in due time you, me and clay should have a discussion of what we should do with webXR and getting it to work again.

@BastiaanOlij Yes, we should chat about this! But here's a preview of my thoughts on this:
I mentioned this on another issue/PR somewhere, but using the new-ish WebXR Layers spec we can get WebXR to give us two 2D texture arrays (one for the color buffer and another for the depth buffer).
The WebXR Layers spec is supported in Chrome and on the Oculus Browser, but there's also a polyfill that can be used for browsers that don't support it, which will create its own 2D texture arrays and just copy from them to the framebuffer from WebXR, meaning it should be safe to rely on this spec.
So, I think the key to WebXR is (1) having multiview support in the opengl3 driver and (2) using the new WebXR Layers spec rather than "original method" which just gives us that framebuffer.

That sounds very promising, that approach is much more in line with what we're doing.

Needs rebase.

Thanks!


chore: fix typo in MockRender.md

v14.7.0 has been released and contains a fix for the issue. Feel free to reopen the issue or to submit a new one if you meet any problems.


Display indicator for selected object

Currently, this only works for tiles. Should work for units structures etc too.

Probably use #637 here as well.


[BUG]: "conda activate" can not work in the ".bashrc" environment.

Athena OS version
v23.06.23
Which environment or technology are you using?

[X] Baremetal
[ ] VirtualBox Virtual Machine
[ ] VMware Virtual Machine
[ ] Hyper-V Virtual Machine
[ ] QEMU/KVM
[ ] Parallels
[ ] UTM
[ ] Container
[ ] WSL

Kernel version
6.4.11-zen2-1-zen
Involved software
miniconda
Repro steps
For all version of miniconda:

step 1: wget -c https://repo.anaconda.com/miniconda/Miniconda3-latest-Linux-x86_64.sh
step 2: bash Miniconda3-latest-Linux-x86_64.sh
step 3: conda init bash
step 4: conda create -n test && conda activate test

Expected behavior
the prompt changes from (base) to (test)
Actual behavior
the prompt is still (base)
Diagnostic logs
i find the reason why the prompt doesn't change is that the line in .bashrc
export PROMPT_COMMAND='source ~/.bashrc no-repeat-flag'
conda adds the conda init lines in ~/.bashrc , but this command execute for every time a command is finished in bash.
so the prompt doesn't change.
if [[ $1 != no-repeat-flag  ]]; then
    __conda_setup="$('/home/kumquat/miniconda3/bin/conda' 'shell.bash' 'hook' 2> /dev/null)"
    if [ $? -eq 0 ]; then
        eval "$__conda_setup"
    else
        if [ -f "/home/kumquat/miniconda3/etc/profile.d/conda.sh" ]; then
            . "/home/kumquat/miniconda3/etc/profile.d/conda.sh"
        else
            export PATH="/home/kumquat/miniconda3/bin:$PATH"
        fi
    fi
    unset __conda_setup
fi
# <<< conda initialize <<<
conda activate $CONDA_DEFAULT_ENV

The code is my solution to this issue

Thank you very much. Here https://athenaos.org/en/support/troubleshooting/#tools added the details you reported in the issue. I'm not sure to add that code directly on .bashrc because Athena has not Conda installed by default.

Thank you for your attention。


Support factory pattern for deployment

While trying out, I realized that the deployed contract addresses are not stored when using a factory contract for deployment. Additionally, the visualization failed with the following error:
TypeError: Cannot read properties of undefined (reading 'id')
    at file:///Users/maguroid/ghq/github.com/maguroid/hardhat-ignition-demo/cache_hardhat/visualization/index.html:326:356
    at Array.map (<anonymous>)
    at lEn (file:///Users/maguroid/ghq/github.com/maguroid/hardhat-ignition-demo/cache_hardhat/visualization/index.html:326:208)
    at bPe (file:///Users/maguroid/ghq/github.com/maguroid/hardhat-ignition-demo/cache_hardhat/visualization/index.html:70:20007)
    at N7t (file:///Users/maguroid/ghq/github.com/maguroid/hardhat-ignition-demo/cache_hardhat/visualization/index.html:72:44535)
    at M7t (file:///Users/maguroid/ghq/github.com/maguroid/hardhat-ignition-demo/cache_hardhat/visualization/index.html:72:40195)
    at Uhn (file:///Users/maguroid/ghq/github.com/maguroid/hardhat-ignition-demo/cache_hardhat/visualization/index.html:72:40118)
    at gge (file:///Users/maguroid/ghq/github.com/maguroid/hardhat-ignition-demo/cache_hardhat/visualization/index.html:72:39966)
    at iNe (file:///Users/maguroid/ghq/github.com/maguroid/hardhat-ignition-demo/cache_hardhat/visualization/index.html:72:36270)
    at A7t (file:///Users/maguroid/ghq/github.com/maguroid/hardhat-ignition-demo/cache_hardhat/visualization/index.html:72:35207)
    at le (file:///Users/maguroid/ghq/github.com/maguroid/hardhat-ignition-demo/cache_hardhat/visualization/index.html:57:1666)
    at MessagePort.pe (file:///Users/maguroid/ghq/github.com/maguroid/hardhat-ignition-demo/cache_hardhat/visualization/index.html:57:2056)

Here is my module for reproduction:
https://github.com/maguroid/hardhat-ignition-demo/blob/factory-pattern/ignition/modules/Vault.ts
I want to express my gratitude to the entire team for their hard work and support throughout this project. Your dedication has been invaluable.

@maguroid thanks for the kind words ... and the repo to help us reproduce the issues.
On the visualization error. I was able to get a minimally reproducible example where the module id of a submodule matches the module id of the root module. In this case both the vault module and the vault factory module have the same id. If I rename the vault factory module I get a visualization.
This is missing validation, you should have got a clear and comprehensible error on running visualize. I have opened a separate issue for this bug here: https://github.com/NomicFoundation/hardhat-ignition/issues/608
Can I confirm @maguroid that that the deployed contract addresses are not stored when using a factory means you where expecting that the contractAt that wrap the readEventArguments from the factory deploys would be recorded into deployed_addresses.json?
So something like:
  const univ3Usdc = m.contractAt('Vault', m.readEventArgument(create1, 'VaultCreated', 'vault', { id: 'read1' }), { id: 'create1' })
  const mockUsdc = m.contractAt('Vault', m.readEventArgument(create2, 'VaultCreated', 'vault', { id: 'read2' }), { id: 'create2' })

  return {
    vaultFactory,
    univ3Usdc,
    mockUsdc,
  }

Shows the contracts deployed through the factory in the deploy command:

But the returned contractAt are missing (i.e. Vault#create1 and Vault#create2) from the deployed_addresses.json

@kanej
Thank you for the prompt response! I made a mistake by not specifying the ID, but I have now confirmed that the results you provided are consistent with my findings.

But the returned contractAt are missing (i.e. Vault#create1 and Vault#create2) from the deployed_addresses.json

Yes. It would be greatly appreciated if this issue could be resolved...

I think it makes sense to include contractAt addresses in deployed addresses; but let me check with the rest of the team and see if we can think through unintended consequences.

This is a bug, we suspect that our check for recording the deployed address should include the contractAt init message:
https://github.com/NomicFoundation/hardhat-ignition/blob/5a54ac58643cbf3b87d6adf9086b08710b3fc944/packages/core/src/internal/execution/future-processor/future-processor.ts#L142


FINERACT-1932: Add starter Configuration class for self-shareaccounts module

Description
Describe the changes made and why they were made.
Ignore if these details are present on the associated Apache Fineract JIRA ticket.
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[x] Write the commit message as per https://github.com/apache/fineract/#pull-requests

[x] Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

[x] Create/update unit or integration tests for verifying the changes made.

[x] Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

[x] Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/legacy-docs/apiLive.htm with details of any API changes

[x] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the developer mailing list for guidance, if required.)

FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.

@galovics I have no idea what is this failure related to.
Is it due to to my PR or some error in the CI?


Issue about train_database.yaml

Hi, thank you for opening your great work.
When I run sh scripts/scannet/scannet_val.sh, the code exit by reporting generate data/processed/scannet/train_database.yaml first.
I want to know how can I generate this file?
Thank you very much!

We use Mask3D codebase and follow it to process data for train and evaluate.  This file will be generated after you run Mask3D data preprocess.


Add suffixedWith option to StatsDReporter

This allows appending a specific suffix to all reported metrics, similar to the existing prefixedWith option.

It's not clear to me what's up with that travis-ci build- if there's something that needs to be done, just let me know.


[Question] TTY inaccessible from Hyprland?

Issue
I'm able to switch between different TTY(num) sessions when I'm in TTY0 with WIN+ALT+F(num), e.g: TTY(1,2,3,x,..).
However, as soon as I enter Hyprland this is no longer possible. I'm guessing this is because mainMod key is set to WIN button (SUPER).
 This can happen when you receive a notification in Brave browser without a notification daemon or sometimes if you try to launch Hyprland with 10bit colors with a Nvidia card.
Question
How can you access different TTY sessions when inside Hyprland? E.g could it be possible with key binds?

the key shortcut to change ttys was never win+alt+f(num), no clue what you are on about, it's ctrl+alt+f(num).
Furthermore, if hyprland freezes and keybinds stop working, how do you expect tty switching to work? It can't.


⚠ SceneTime has degraded performance

In 2a39ed3, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1143 ms

Resolved: SceneTime performance has improved in 75e342a after 6 minutes.


Make Hyper/HTTP support an optional feature

I would like to request you make Hyper/HTTP support an optional feature, as the language presents a good platform to use for a bunch of project ideas I have, but Hyper (thanks to OpenSSL), is difficult and unpredictable to build on Windows machines, and I think making it an optional feature would both allow for easy building on windows, and the ability to lock down a potential security hole in applications that don't need HTTP support, while also giving the developer the choice to use other HTTP libraries if they so wish (or even simply implement tighter controls on when/where HTTP is allowed).

Turns out it was another crate pulling in OpenSSL, but the point still stands regarding giving the choice to the developer whether to include HTTP support

I agree. It could be moved to a separate library.

Not sure its enough functionality to warrant moving it to an additional library, feature gating it should be sufficient and minimally affect users

One idea is to make HTTP support a default feature.

Yeah that would work

One problem is when you want HTTP support for some part of a project while not permitting it for other parts.

I think the scope of that issue is beyond that of the language itself and should probably be implemented higher up where appropriate decisions can be made based on the individual use cases (perhaps by (re)implementing the HTTP support themselves)

Or alternatively, allowing intrinsics to be overriden would enable the developer to simply override HTTP support with a dummy function in areas where it should be disabled, though that would warrant a separate discussion

It is already possible to override an intrinsic, simply by declaring a new function with same name. I forgot that.


Use internal download mirror for Fedora images

Reduce stress on upstream mirrors and speed up builds by using an
internal mirror to download packages in CI.
Signed-off-by: Major Hayden<EMAIL_ADDRESS>

This one is broken due to internal mirror issues for Fedora. Helpdesk ticket has been opened.


Add Continuous Integration (CI)

This PR adds Continuous Integration (CI) for SPARTACUS-Surface. The software is automatically compiled and tested at every push commit or pull request. In case of compile- run-time errors a notification is sent out. Currently, checks are only performed on the latest Ubuntu LTS with gfortran for Debug and Release mode but can be easily expanded if needed.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[CARBONDATA-2369] updated the document about AVRO to carbon schema converter

[CARBONDATA-2369] updated the document about AVRO to carbon schema converter
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed? Added new interface.

[ ] Any backward compatibility impacted? NA

[ ] Document update required? updated

[ ] Testing done. NA

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4659/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/5814/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/4866/

LGTM

LGTM

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4681/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/4885/


Update ru.toml

Minor translation fixes

Hey @ViktorOn,
We cannot just copy your patch and open a new one as our own. Please understand that without CLA signed, the changes are eligible for us to review.
It's been a long time since inactive, feel free to /reopen it when you are ready.
/close


Passing function results

Quick question, passing the result of one function to the next in resolutionType: 'sequential' is not supported, right?
Thx!

Hey @gurkerl83 - that's correct. Piping the results from one function to the next is not currently supported. That being said, I could certainly see there being use cases for this. Happy to hear out any proposal for this addition. Otherwise I'd be happy to explore this myself when I get some spare time.


Change status access from header to attribute.

This is to unify all the times a status is used from the response (containing header info) of an HTTP request.

@jonparrott @nathanielmanistaatgoogle I was considering adding transport.get_status(response) and transport.get_header(response), these are essentially the only remnants of httplib2 outside of the transport module. (The only other is that http.request returns response, content and that content is always bytes.)

@jonparrott @nathanielmanistaatgoogle Can you weigh in on transport.get_status(response) and transport.get_header(response)?

Let's table it until we actually try using a different transport.

SGTM. I'm more concerned about flagging / fencing off all the httplib2 features.


Images during search

This patch allows the images to be show during find-as-you-type filtering.
However, it requires the chosen:filter event to be fired from Chosen, which is in a PR here: https://github.com/harvesthq/chosen/pull/2373

This one doesn't seem to be fixed with both master branches from Chosen and Image-Select
Displayed select without typing

### As soon as I start typing

Same here with both master branches

Still waiting for Chosen to merge https://github.com/harvesthq/chosen/pull/2373


Create Staging environment for testing container hardening

As an assurance engineer I need an environment to test out any hardening changes made to the current container scanning pipeline. This is a predecessor to Demonstrate gating an image on late CVE#2540
Acceptance Criteria

[ ] When the container scanning pipeline is run, a staging environment is created with two staging tasks, one for concourse and another for s3 hardening.

Security considerations
This will aid in improving security as it is a predecessor to a ticket that will add gating
Implementation sketch

[x] Update terraform in ecr.tf to add the new tasks
[x] Update pipeline to utilize the new tasks

Closing this in favor of scanning containers in pipeline before they are pushed. Will make another ticket to update ADR.


[WIP]284 update readme

@mbonsma . Please check Is this how you want it? I have used the all contribution bot

Codecov Report

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

@@           Coverage Diff           @@
##           master     #288   +/-   ##
=======================================
  Coverage   22.89%   22.89%
=======================================
  Files          21       21
  Lines         760      760
=======================================
  Hits          174      174
  Misses        586      586

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 bc05d76...b317bec. Read the comment docs.


feat(linter/typescript): support consistent-type-imports rule

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

Current dependencies on/for this PR:

main

PR #2021

PR #2022   👈

This stack of pull requests is managed by Graphite.

Blocked by #2023

See PR https://github.com/oxc-project/oxc/pull/3895


Fix nit errors in readme.md

First PR, fixing nit issues in README.md

Thanks so much for the fix :)
I don't have the DCO bot in place yet, can you just edit the commit message with "Signed-off-by: [Your Name]" (https://github.com/axiomic-ai/axiomic/blob/main/CONTRIBUTING.md); or use 'git commit -a -s' to automatically do it.
Sorry I don't have the automation in place for this, will get it going shortly.


browserslist target throws when Babel can understand it

Bug report
What is the current behavior?
Using the following .browserslistrc that was generated a few weeks back with npx browserslist --mobile-to-desktop "> 0.5%, last 2 versions, Firefox ESR, not dead, not IE 11" throws.
and_chr 91
and_ff 89
and_qq 10.4
and_uc 12.12
android 91
baidu 7.12
chrome 90
edge 91
firefox 78
ios_saf 12.4
kaios 2.5
op_mini all
op_mob 76
opera 76
safari 14
samsung 13.0

If the current behavior is a bug, please provide the steps to reproduce.
$ git clone https://github.com/eps1lon/webpack-targets-browserslist
$ yarn
$ yarn start

What is the expected behavior?
Webpack is able to understand this browserlist since Babel is able to. See https://github.com/babel/babel/issues/12123 for more context.
Other relevant information:
webpack version:  5.52.0
Node.js version: 12.22.6
Operating System: Linux 5.4 Ubuntu 20.04.3 LTS (Focal Fossa)
Additional tools<EMAIL_ADDRESS><EMAIL_ADDRESS>

mobileToDesktop: Use desktop browsers if Can I Use doesn’t have data about this mobile version. For instance, Browserslist will return chrome 20 on and_chr 20 query (Can I Use has only data only about latest versions of mobile browsers). Default is false.

I don't think we should enable it by default, better to throw error on wrong browserslist entries, it will allow developer validate which browser should be used
/cc @ai

mobileToDesktop is a very specific option just for Babel because they have a limited database (without data for mobile browsers)
For instance, we do not use it in Autoprefixer since we are using prefixes accroding to Can I Use database which have this data.
But how webpack is using Browserslist?
I am not fully get the issue? What is an error if somebody is throwing something on this config?

@ai no magic just load config and pass it to browserslist https://github.com/webpack/webpack/blob/main/lib/config/browserslistTargetHandler.js#L54

I am not fully get the issue? What is an error if somebody is throwing something on this config?

browserslist just throw an error about unknown entry due android 91

This is probably the question, should we use mobileToDesktop option or not?

This is probably the question, should we use mobileToDesktop option or not?

I think not. It is better to use mobileToDesktop option on Babel level on top of browsers from webpack

Because mobileToDesktop is a specific Babel hack. It could leads to compatibility data losses for other tool

Because mobileToDesktop is a specific Babel hack. It could leads to compatibility data losses for other tool

Could you explain this with a concrete example? "it's a babel hack" is not really helpful since what constitutes a hack is subjective.
@JLHwung Could you maybe shine some light on why you use mobileToDesktop in Babel? As a Webpack and Babel user, an inconsistent behavior between these two libraries is the worst solution. Would be nice if we could agree on the best behavior.

Mobile browsers are not 100% equal to desktop browsers. Sometimes some feature is available only in desktop browser. With mobileToDesktop you may have wrong compatibility data because you converted all mobile browsers to desktop browsers.
It is not so critical for JS, since JS VM are more often similar between mobile and desktop. But it is more critical for data formats.
Simple non-real example:

User has .browserslistrc with last 1 FirefoxAndroid version
With mobileToDesktop option, Browserslist will return ff 92. Without this option, it will return and_ff 92.
If we have a webpack loader for video compression, with mobileToDesktop option loader will use AV1 (because desktop version of Firefox supports AV1). But it will be wrong because mobile version doesn’t support it.

@ai Maybe it is good idea to add this to browserslist docs

Babel uses mobileToDesktop because they do not have support data for ESx features for mobile browsers. They have only data about desktop users.
For JS runtime, it is more likely to assume that desktop and mobile versions will have the same ES features. It is a hack (since we can’t 100% guarantee the same version of JS VM in mobile and desktop), but it is pretty safe to use it in this context.

Seems like I was wrong in previous comment. I deleted it.
How mobileToDesktop option works:

By default, last 2 and_ff versions will return only and_ff 90. Only 1 version because Can I Use have only data for latest mobile Firefox version.
With mobileToDesktop, last 2 and_ff versions will return and_ff 91, and_ff 90. This option is assuming that mobile browsers have the same versions that desktop browsers and is using desktop browsers’ data.

The problem with mobileToDesktop is that browsers is not correct Can I Use versions. Autoprefixer (who uses Can I Use to get data about the prefixes) will not be able to use and_ff 91 data.
Here is an example:

User is using last 1 and_ff versions
With mobileToDesktop Browserslist returns and_ff 91.
Autoprefixer doesn’t find any vendor prefixes for and_ff 91 (because Can I Use data has only and_ff 90 data) and will not insert vendor prefixes for mobile Firefox.

I improved docs https://github.com/browserslist/browserslist/commit/49af84894d7eb8a043f2b063f62efed71bb9ca09

So I think we're doing it right, there is a chance of a difference in JS syntax/API and this can lead to the generation of incorrect code

So I think we're doing it right, there is a chance of a difference in JS syntax/API and this can lead to the generation of incorrect code

In one hand, yeap we can’t use non-mobileToDesktop result in Babel.
In other hand, using mobileToDesktop for everyone will lead to unknown version error.
Maybe the best option is to read Browserslist config, but do not convert it to browser versions. Pass it as queries, so every tool will convert it to versions according to their options.

@ai we only pass query or config to browserslist and nothing more

@alexander-akait in this case I do not understand where the error is coming from

and lots of bugs in versions, so this was rewritten to this

Got it. The error happened because .browserslistrc was generated from --mobile-to-desktop output.
@eps1lon you should not do it. It is better to keep origin queries in .browserslistrc. Your Browserslist config is invalid. For instance, Autoprefixer will not work with it.

@alexander-akait should we close this in favor of #14226 ?

https://github.com/webpack/webpack/issues/14226 is not related to this, but we can close it, using mobileToDesktop is not safe and can lead to wrong generation of runtime
Anyway feel free to feedback


I don't think this injector is used anywhere

I think the only number that is used here is this one
https://github.com/mpraglowski/cqrs-es-sample-with-res/blob/master/lib/domain/services/number_generator.rb
and a Fake one in tests, of course


memory leak when implement rnn attention decoder

System information
== cat /etc/issue ===============================================
Linux quad 4.4.0-72-generic #93-Ubuntu SMP Fri Mar 31 14:07:41 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
VERSION="16.04 LTS (Xenial Xerus)"
VERSION_ID="16.04"
== are we in docker =============================================
No
== compiler =====================================================
c++ (Ubuntu 4.9.4-2ubuntu1~16.04) 4.9.4
Copyright (C) 2015 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
== uname -a =====================================================
Linux quad 4.4.0-72-generic #93-Ubuntu SMP Fri Mar 31 14:07:41 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
== check pips ===================================================
numpy (1.12.1)
protobuf (3.3.0)
tensorflow-gpu (1.1.0)
== check for virtualenv =========================================
True
== tensorflow import ============================================
tf.VERSION = 1.1.0
tf.GIT_VERSION = v1.1.0-rc0-61-g1ec6ed5
tf.COMPILER_VERSION = v1.1.0-rc0-61-g1ec6ed5
Sanity check: array([1], dtype=int32)
== env ==========================================================
LD_LIBRARY_PATH is unset
DYLD_LIBRARY_PATH is unset
== nvidia-smi ===================================================
Tue May  9 12:16:54 2017
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 375.39                 Driver Version: 375.39                    |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  GeForce GTX TIT...  Off  | 0000:05:00.0     Off |                  N/A |
| 22%   47C    P0    76W / 250W |      0MiB / 12205MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+
|   1  GeForce GTX TIT...  Off  | 0000:06:00.0     Off |                  N/A |
| 22%   60C    P2   129W / 250W |  11713MiB / 12207MiB |     80%      Default |
+-------------------------------+----------------------+----------------------+
|   2  GeForce GTX TIT...  Off  | 0000:09:00.0     Off |                  N/A |
| 22%   49C    P0    83W / 250W |      0MiB / 12207MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+
|   3  GeForce GTX TIT...  Off  | 0000:0A:00.0     Off |                  N/A |
| 24%   63C    P2   117W / 250W |  11713MiB / 12207MiB |     70%      Default |
+-------------------------------+----------------------+----------------------+
+-----------------------------------------------------------------------------+
| Processes:                                                       GPU Memory |
|  GPU       PID  Type  Process name                               Usage      |
|=============================================================================|
|    1     21558    C   python3                                      11709MiB |
|    3     21346    C   python3                                      11709MiB |
+-----------------------------------------------------------------------------+
== cuda libs  ===================================================
/usr/local/cuda-7.5/doc/man/man7/libcudart.7
/usr/local/cuda-7.5/doc/man/man7/libcudart.so.7
/usr/local/cuda-7.5/lib64/libcudart_static.a
/usr/local/cuda-7.5/lib64/libcudart.so.7.5.18
/usr/local/cuda-7.5/lib/libcudart_static.a
/usr/local/cuda-7.5/lib/libcudart.so.7.5.18
/usr/local/cuda-8.0/doc/man/man7/libcudart.7
/usr/local/cuda-8.0/doc/man/man7/libcudart.so.7
/usr/local/cuda-8.0/lib64/libcudart_static.a
/usr/local/cuda-8.0/lib64/libcudart.so.8.0.27
Describe the problem
When we try to implement a complex network which contain a rnn attention decoder, It will consume all the memory after several days. I extract the decoder in a test file, the memory still grow in a slower speed. also found that if change softmax to sigmoid, memory doesn't leak.
Source code / logs
test code:
# __author__ = "liusiye"
# -*- coding: utf-8 -*-
import tensorflow as tf
from tensorflow.contrib.rnn import GRUCell, MultiRNNCell
from os import getpid
import psutil
import gc
import tensorflow as tf
import numpy as np

process = psutil.Process(getpid())
B, T, H = 20, 60, 256
layer_num = 4

def apply_attention(encoding, rnn_output):
    ''' encoding: [t, b, h1]
        rnn_output: [b, h2]
    '''
    T, B, H1 = encoding.get_shape().as_list()
    _, H2 = rnn_output.get_shape().as_list()
    with tf.variable_scope('attention'):
        w_encoder = tf.get_variable(
            name='W_encoder',
            shape=[H1, H1],
            initializer=tf.random_uniform_initializer(-0.01, 0.01))
        w_decoder = tf.get_variable(
            name='W_decoder',
            shape=[H2, H1],
            initializer=tf.random_uniform_initializer(-0.01, 0.01))
        w_attention = tf.get_variable(
            name='W_attention',
            shape=[H1, 1],
            initializer=tf.random_uniform_initializer(-0.01, 0.01))
    r_decoder = tf.matmul(rnn_output, w_decoder)  # [b, h1]

    r_encoder = tf.matmul(tf.reshape(encoding, [-1, H1]), w_encoder)
    r_encoder = tf.reshape(r_encoder, [T, B, H1])
    # [t, b, h] -> [t * b, h] -> [t, b, h]

    r_attention = tf.tanh(r_encoder + r_decoder)  # [t, b, h1]
    attention = tf.matmul(tf.reshape(r_attention, [-1, H1]), w_attention)
    attention = tf.nn.softmax(tf.reshape(attention, [T, B]), dim=0)
    #attention = tf.nn.sigmoid(tf.reshape(attention, [T, B]))
    encoding = tf.transpose(encoding, perm=[2, 0, 1])  # [t, b, h1]->[h1, t, b]

    context = tf.reduce_sum(encoding * attention, axis=1)  # [h1, b]
    return tf.transpose(context)  # [b, h1]

def rnn_attention_decoder_test():
    encoding = tf.get_variable(name='encoding', shape=[T, B, H], dtype=tf.float32)
    rnn_outputs = []  # t * [b, h]
    scope = tf.get_variable_scope()

    zero_input = tf.constant(0, shape=[B, H], dtype=tf.float32)
    cell = tf.contrib.rnn.MultiRNNCell([tf.contrib.rnn.GRUCell(H) for i in range(layer_num)], state_is_tuple=True)
    state = cell.zero_state(B, tf.float32)
    with tf.variable_scope(scope) as outer_scope:
        for t in range(T):#T):
            attention_input = zero_input
            rnn_input = apply_attention(encoding, attention_input)
            rnn_output, state = cell(rnn_input, state)
            rnn_outputs.append(rnn_output)
            outer_scope.reuse_variables()
    return tf.stack(rnn_outputs, axis=1), state  # t * [b, h] -> [b, t, h]

with tf.Session() as sess, tf.variable_scope('model'):
    with tf.variable_scope('model', reuse=None):
        tensor_lists_test = rnn_attention_decoder_test()
    init_op = tf.group(
        tf.global_variables_initializer(),
        tf.local_variables_initializer()
    )
    sess.run(init_op)
    sess.graph.finalize()
    for step in range(100000):
        after = process.memory_percent()
        if step > 0:
            print("MEMORY CHANGE %.7f -> %.7f" % (before, after))
        before = process.memory_percent()
        sess.run(tensor_lists_test)
        gc.collect()

log:
2017-05-09 09:27:55.435870: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.1 instructions, but these are available on your machine and could speed up CPU computations.
2017-05-09 09:27:55.435903: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.2 instructions, but these are available on your machine and could speed up CPU computations.
2017-05-09 09:27:55.435909: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use AVX instructions, but these are available on your machine and could speed up CPU computations.
2017-05-09 09:27:55.435913: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use AVX2 instructions, but these are available on your machine and could speed up CPU computations.
2017-05-09 09:27:55.435916: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use FMA instructions, but these are available on your machine and could speed up CPU computations.
2017-05-09 09:27:55.732204: I tensorflow/core/common_runtime/gpu/gpu_device.cc:887] Found device 0 with properties:
name: GeForce GTX TITAN X
major: 5 minor: 2 memoryClockRate (GHz) 1.2405
pciBusID 0000:09:00.0
Total memory: 11.92GiB
Free memory: 11.81GiB
2017-05-09 09:27:55.732232: I tensorflow/core/common_runtime/gpu/gpu_device.cc:908] DMA: 0
2017-05-09 09:27:55.732238: I tensorflow/core/common_runtime/gpu/gpu_device.cc:918] 0:   Y
2017-05-09 09:27:55.732247: I tensorflow/core/common_runtime/gpu/gpu_device.cc:977] Creating TensorFlow device (/gpu:0) -> (device: 0, name: GeForce GTX TITAN X, pci bus id: 0000:09:00.0)
MEMORY CHANGE 1.1297021 -> 1.3666840
MEMORY CHANGE 1.3666840 -> 1.3666840
MEMORY CHANGE 1.3666840 -> 1.3697929
MEMORY CHANGE 1.3697929 -> 1.3697929
MEMORY CHANGE 1.3697929 -> 1.3729200
MEMORY CHANGE 1.3729200 -> 1.3733208
MEMORY CHANGE 1.3733208 -> 1.3733208
MEMORY CHANGE 1.3733208 -> 1.3737215
MEMORY CHANGE 1.3737215 -> 1.3768487
MEMORY CHANGE 1.3768487 -> 1.3799758
MEMORY CHANGE 1.3799758 -> 1.3803644
MEMORY CHANGE 1.3803644 -> 1.3834976
MEMORY CHANGE 1.3834976 -> 1.3834976
MEMORY CHANGE 1.3834976 -> 1.3838862
MEMORY CHANGE 1.3838862 -> 1.3838862
MEMORY CHANGE 1.3838862 -> 1.3842870
MEMORY CHANGE 1.3842870 -> 1.3846878
MEMORY CHANGE 1.3846878 -> 1.3850885
MEMORY CHANGE 1.3850885 -> 1.3850885
MEMORY CHANGE 1.3850885 -> 1.3850885
MEMORY CHANGE 1.3850885 -> 1.3850885
MEMORY CHANGE 1.3850885 -> 1.3850885
MEMORY CHANGE 1.3850885 -> 1.3854771
MEMORY CHANGE 1.3854771 -> 1.3854771
MEMORY CHANGE 1.3854771 -> 1.3854771
MEMORY CHANGE 1.3854771 -> 1.3858779
MEMORY CHANGE 1.3858779 -> 1.3858779
MEMORY CHANGE 1.3858779 -> 1.3858779
MEMORY CHANGE 1.3858779 -> 1.3858779
MEMORY CHANGE 1.3858779 -> 1.3858779
MEMORY CHANGE 1.3858779 -> 1.3858779
MEMORY CHANGE 1.3858779 -> 1.3862665
MEMORY CHANGE 1.3862665 -> 1.3866673
MEMORY CHANGE 1.3866673 -> 1.3866673
MEMORY CHANGE 1.3866673 -> 1.3870680
MEMORY CHANGE 1.3870680 -> 1.3870680
MEMORY CHANGE 1.3870680 -> 1.3870680
MEMORY CHANGE 1.3870680 -> 1.3870680
MEMORY CHANGE 1.3870680 -> 1.3870680
MEMORY CHANGE 1.3870680 -> 1.3870680
MEMORY CHANGE 1.3870680 -> 1.3870680
MEMORY CHANGE 1.3870680 -> 1.3874688
MEMORY CHANGE 1.3874688 -> 1.3874688
MEMORY CHANGE 1.3874688 -> 1.3878695
MEMORY CHANGE 1.3878695 -> 1.3878695
MEMORY CHANGE 1.3878695 -> 1.3882581
MEMORY CHANGE 1.3882581 -> 1.3882581
MEMORY CHANGE 1.3882581 -> 1.3886589
MEMORY CHANGE 1.3886589 -> 1.3886589
MEMORY CHANGE 1.3886589 -> 1.3890597
MEMORY CHANGE 1.3890597 -> 1.3894604
MEMORY CHANGE 1.3894604 -> 1.3894604
MEMORY CHANGE 1.3894604 -> 1.3898612
MEMORY CHANGE 1.3898612 -> 1.3898612
MEMORY CHANGE 1.3898612 -> 1.3902619
MEMORY CHANGE 1.3902619 -> 1.3906627
MEMORY CHANGE 1.3906627 -> 1.3906627
MEMORY CHANGE 1.3906627 -> 1.3906627
MEMORY CHANGE 1.3906627 -> 1.3906627
MEMORY CHANGE 1.3906627 -> 1.3906627
MEMORY CHANGE 1.3906627 -> 1.3906627
MEMORY CHANGE 1.3906627 -> 1.3906627
MEMORY CHANGE 1.3906627 -> 1.3906627
MEMORY CHANGE 1.3906627 -> 1.3937898
MEMORY CHANGE 1.3937898 -> 1.3937898
MEMORY CHANGE 1.3937898 -> 1.3937898
MEMORY CHANGE 1.3937898 -> 1.3937898
MEMORY CHANGE 1.3937898 -> 1.3937898
MEMORY CHANGE 1.3937898 -> 1.3937898
MEMORY CHANGE 1.3937898 -> 1.3937898
MEMORY CHANGE 1.3937898 -> 1.3937898
MEMORY CHANGE 1.3937898 -> 1.3941906
MEMORY CHANGE 1.3941906 -> 1.3945913
MEMORY CHANGE 1.3945913 -> 1.3945913
MEMORY CHANGE 1.3945913 -> 1.3945913
MEMORY CHANGE 1.3945913 -> 1.3945913
MEMORY CHANGE 1.3945913 -> 1.3945913
MEMORY CHANGE 1.3945913 -> 1.3945913
MEMORY CHANGE 1.3945913 -> 1.3945913
MEMORY CHANGE 1.3945913 -> 1.3945913
MEMORY CHANGE 1.3945913 -> 1.3945913
MEMORY CHANGE 1.3945913 -> 1.3945913
MEMORY CHANGE 1.3945913 -> 1.3945913
MEMORY CHANGE 1.3945913 -> 1.3945913
MEMORY CHANGE 1.3945913 -> 1.3945913
MEMORY CHANGE 1.3945913 -> 1.3945913
MEMORY CHANGE 1.3945913 -> 1.3945913
MEMORY CHANGE 1.3945913 -> 1.3945913
MEMORY CHANGE 1.3945913 -> 1.3945913
MEMORY CHANGE 1.3945913 -> 1.3945913
MEMORY CHANGE 1.3945913 -> 1.3945913
MEMORY CHANGE 1.3945913 -> 1.3945913
MEMORY CHANGE 1.3945913 -> 1.3949921
MEMORY CHANGE 1.3949921 -> 1.3949921
MEMORY CHANGE 1.3949921 -> 1.3949921
MEMORY CHANGE 1.3949921 -> 1.3949921
MEMORY CHANGE 1.3949921 -> 1.3949921
MEMORY CHANGE 1.3949921 -> 1.3949921
MEMORY CHANGE 1.3949921 -> 1.3949921
MEMORY CHANGE 1.3949921 -> 1.3949921
MEMORY CHANGE 1.3949921 -> 1.3949921
MEMORY CHANGE 1.3949921 -> 1.3949921
MEMORY CHANGE 1.3949921 -> 1.3949921
MEMORY CHANGE 1.3949921 -> 1.3949921
MEMORY CHANGE 1.3949921 -> 1.3949921
MEMORY CHANGE 1.3949921 -> 1.3949921
MEMORY CHANGE 1.3949921 -> 1.3949921
MEMORY CHANGE 1.3949921 -> 1.3949921
MEMORY CHANGE 1.3949921 -> 1.3949921
MEMORY CHANGE 1.3949921 -> 1.3949921
MEMORY CHANGE 1.3949921 -> 1.3953929
MEMORY CHANGE 1.3953929 -> 1.3953929
MEMORY CHANGE 1.3953929 -> 1.3953929
MEMORY CHANGE 1.3953929 -> 1.3953929
MEMORY CHANGE 1.3953929 -> 1.3953929
MEMORY CHANGE 1.3953929 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3957936
MEMORY CHANGE 1.3957936 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3961944
MEMORY CHANGE 1.3961944 -> 1.3965951
MEMORY CHANGE 1.3965951 -> 1.3965951
MEMORY CHANGE 1.3965951 -> 1.3965951
MEMORY CHANGE 1.3965951 -> 1.3969959
MEMORY CHANGE 1.3969959 -> 1.3969959
MEMORY CHANGE 1.3969959 -> 1.3969959
MEMORY CHANGE 1.3969959 -> 1.3969959
MEMORY CHANGE 1.3969959 -> 1.3969959
MEMORY CHANGE 1.3969959 -> 1.3969959
MEMORY CHANGE 1.3969959 -> 1.3969959
MEMORY CHANGE 1.3969959 -> 1.3969959
MEMORY CHANGE 1.3969959 -> 1.3973967
MEMORY CHANGE 1.3973967 -> 1.3973967
MEMORY CHANGE 1.3973967 -> 1.3973967
MEMORY CHANGE 1.3973967 -> 1.3973967
MEMORY CHANGE 1.3973967 -> 1.3977974
MEMORY CHANGE 1.3977974 -> 1.3977974

by the way , if we implement softmax like this, the problem doesn't exists.
original softmax in TF
attention = tf.nn.softmax(tf.reshape(attention, [T, B]), dim=0)

mannul softmax
attention = tf.matmul(tf.reshape(r_attention, [-1, H1]), w_attention)
attention = tf.reshape(attention, [T, B])
attention_exp = tf.exp(attention)  # [T ,B]
attention_sum = tf.reduce_sum(tf.exp(attention), axis=0)  # [B]
attention = attention_exp / attention_sum  # [T, B]

@ebrevdo Can you take a look?  This seems like a reasonably minimized memory leak bug.

It has been 14 days with no activity and this issue has an assignee.Please update the label and/or status accordingly.

Nagging Assignee: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 140 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 155 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 170 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 185 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 200 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 215 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 230 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 245 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 260 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 275 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 290 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 305 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 320 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

This is a stale issue. Please check the issue with latest TensorFlow. If the issue still persists in the newer version of TF, please feel free to reopen it by providing details about the issue and a standalone code to reproduce the issue. Thanks!


UnboundLocalError: local variable 'beta1' referenced before assignment

When I run 'python tools/train.py configs/detr/detr_r50_8x2_150e_coco.py‘，An UnboundLocalError occured. Please help me.
My env:
GCC: gcc (GCC) 6.4.0
PyTorch: 1.8.0+cu111
TorchVision: 0.9.0+cu111
OpenCV: 4.5.1
MMCV: 1.2.4
MMCV Compiler: GCC 6.4
MMCV CUDA Compiler: 11.1
MMDetection: 2.11.0+
Error Information:
Traceback (most recent call last):
File "tools/train.py", line 187, in
main()
File "tools/train.py", line 183, in main
meta=meta)
File ".../mmdetection/mmdet/apis/train.py", line 170, in train_detector
runner.run(data_loaders, cfg.workflow)
File ".../lib/python3.7/site-packages/mmcv/runner/epoch_based_runner.py", line 125, in run
epoch_runner(data_loaders[i], **kwargs)
File ".../lib/python3.7/site-packages/mmcv/runner/epoch_based_runner.py", line 51, in train
self.call_hook('after_train_iter')
File ".../lib/python3.7/site-packages/mmcv/runner/base_runner.py", line 307, in call_hook
getattr(hook, fn_name)(self)
File ".../lib/python3.7/site-packages/mmcv/runner/hooks/optimizer.py", line 34, in after_train_iter
runner.optimizer.step()
File ".../lib/python3.7/site-packages/torch/optim/optimizer.py", line 89, in wrapper
return func(*args, **kwargs)
File ".../lib/python3.7/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
return func(*args, **kwargs)
File ".../lib/python3.7/site-packages/torch/optim/adamw.py", line 117, in step
beta1,
UnboundLocalError: local variable 'beta1' referenced before assignment

It seems a bug of PyTorch 1.8.0.
https://github.com/pytorch/pytorch/issues/55740

Thanks for your reply. Actually, this is indeed an error in pytorch1.8.0. And I have solved this problem successfully by your link.

Thanks for your reply. Actually, this is indeed an error in pytorch1.8.0. And I have solved this problem successfully by your link.

are you think it right to just decreases one tab, although it can be run.


Updated Test workflow to use re-usable version

refs<EMAIL_ADDRESS>
this commit switches CI to using a re-usable workflow to reduce
duplication and make repositories easier to maintain

Codecov Report

Merging #182 (f25d68a) into main (a380150) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #182   +/-   ##
=======================================
  Coverage   82.14%   82.14%
=======================================
  Files          49       49
  Lines        3568     3568
  Branches      507      480   -27
=======================================
  Hits         2931     2931
  Misses        634      634
  Partials        3        3

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 a380150...f25d68a. Read the comment docs.


Shred created by Machine.add() is lost

To reproduce: upload a hello.ck file,  execute "Machine.add("hello,ck")" from editor.
The shred related to hello.ck is not visible in the visualizer and also cannot be removed by the "Remove" button on the top.

Hi @lmaxwell, with WebChucK IDE, the shred table will only show threads that are running and started by you. Machine.add shreds, Machine.eval() or even sporked shreds won't show up. To remove shreds from the Machine, you can do Machine.removeLastShred(), Machine.removeAllShreds(), or even Machine.clearVM().


Automated cherry pick of #2301: fix image head no field tenant

Cherry pick of #2301 on release/2.11.
#2301: fix image head no field tenant

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@13913fc). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             release/2.11   #2303   +/-   ##
==============================================
  Coverage                ?   7.09%
==============================================
  Files                   ?     452
  Lines                   ?   71318
  Branches                ?       0
==============================================
  Hits                    ?    5060
  Misses                  ?   65842
  Partials                ?     416

Flag
Coverage Δ

#aFlag
7.09% <ø> (?)

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 13913fc...5db5ff3. Read the comment docs.

/lgtm

/approve


Houdini: Refactor Extractors to use ROP input dependency ordering

Changelog Description
Render ROPs in the order of the Input Dependencies in the ROP networks.
Additional info
Examples:

Note: Due to this now being a Context Plug-in this will not mark any of its logs related to an instance. It might be nice @iLLiCiTiT to have a mechanism during publishing to be able to e.g. do:
class Plugin(pyblish.api.ContextPlugin):
    def process(self, context):
        with context.as_instance(instance):
            self.log.info(f"Log for {instance}")
            raise PublishValidationError(f"Error for {instance}")

So that for an artist we could still show logs/errors as related to a particular instance.
However, since that's currently not possible logs and errors will be shown on the Instance.

This currently does not alter anything for Deadline submissions - there is no defined "ordering" or "job dependencies" in that area that uses this sorting. That's to be implemented.
Testing notes:

Launch Houdini
Publish all types of instances; with and without input dependencies in /out

I believe this PR shares similar bugs with the publish button.
this method rop.inputDependencies() has problems with Karma and HDA instances as follows

Karma is a subnetwork, and the method retrieves another node inside the subnetwork.DEBUG: Ignoring instance karma_ropTest as it is marked for farm rendering...
DEBUG: Defined rendering order:
DEBUG:     0. /out/karma_ropTest/lopnet/rop_usdrender
DEBUG:     1. /out/vdbcacheTest
DEBUG:     2. /out/pointcacheBgeoTest

HDA doesn't have that method.Traceback (most recent call last):
  File "E:\Ynput\OpenPype\.venv\lib\site-packages\pyblish\plugin.py", line 527, in __explicit_process
    runner(*args)
  File "E:\Ynput\OpenPype\openpype\hosts\houdini\plugins\publish\extract_rop.py", line 53, in process
    sorted_rops = get_sorted_rops(rop_nodes)
  File "E:\Ynput\OpenPype\openpype\hosts\houdini\api\lib.py", line 1057, in get_sorted_rops
    in enumerate(rop.inputDependencies())
AttributeError: 'ObjNode' object has no attribute 'inputDependencies'

for the publish button I didn't need the dependencies in the correct order so my simple solution to check the inputs was sufficient https://github.com/ynput/OpenPype/pull/5891
I can tweak my solution a little to return the correct order.
import hou

def find_rop_inputs_chain(node):
    """Find rop inputs chain.
    This function retrieves input nodes and their inputs recursively.
    Arguments:
        node (hou.Node) The node at which the find function starts.
    Returns:
        list the paths of all input nodes.
    """

    all_input_nodes = []
    input_nodes = node.inputs()
    for input_node in input_nodes:
        all_input_nodes += find_rop_inputs_chain(input_node)
        all_input_nodes.append(input_node.path())

    return all_input_nodes

node = hou.node('/out/staticMeshObjTest')
all_nodes = find_rop_inputs_chain(node)
all_nodes.append(node.path())
print(all_nodes)

test 1
['/out/usdTest', '/out/reviewTest', '/out/staticMeshSopTest', '/out/staticMeshObjTest']

test 2
['/out/pointcacheABCTest',
 '/out/mantra_ropTest',
 '/out/usdTest',
'/out/karma_ropTest',
'/out/vdbcacheTest',
'/out/cameraTest',
'/out/pointcacheBgeoTest',
'/out/imagesequenceTest',
'/out/reviewTest',
'/out/staticMeshSopTest',
'/out/staticMeshObjTest']

@MustafaJafar I'm pretty sure your function will fail for these cases:

Because in that case there will be rops where a rop will be closer to the 'final' node you're getting the inputs for than for another ROP - meaning you'd still need to sort them if you want to detect the order for all ROPs intead of just iterating inputs for one ROP. You'd still need to identify the full ordering for each, right?
Also, be wary with that function - it is possible in the out network to create cyclic dependencies (not sure why you'd do that) but it'd make your function basically recurse itself over and over.

HDA doesn't have that method.
    Traceback (most recent call last):       File "E:\Ynput\OpenPype\.venv\lib\site-packages\pyblish\plugin.py", line 527, in __explicit_process         runner(*args)       File "E:\Ynput\OpenPype\openpype\hosts\houdini\plugins\publish\extract_rop.py", line 53, in process         sorted_rops = get_sorted_rops(rop_nodes)       File "E:\Ynput\OpenPype\openpype\hosts\houdini\api\lib.py", line 1057, in get_sorted_rops         in enumerate(rop.inputDependencies())     AttributeError: 'ObjNode' object has no attribute 'inputDependencies'

What the hell? But... could you even .render() that? I wonder how Deadline deals with a dependency like that.
Deadline's code is in repository\submission\Houdini\Main\SubmitDeadlineRop.py which does also seem to indeed iterate .inputs() but because of that also has to handle very specific cases, like from a switch node pick only the active input, etc.
As can be seen there, that will grow complicated quickly. Also, Fetch nodes then get special treatment as well. Anything that is not a ROP and not a Fetch seems to be discard/ignored, so likely the ObjNode you're hitting issues with here would by Deadline just be ignored and only considered if it were a RopNode.

Karma is a subnetwork, and the method retrieves another node inside the subnetwork.
    DEBUG: Ignoring instance karma_ropTest as it is marked for farm rendering...     DEBUG: Defined rendering order:     DEBUG:     0. /out/karma_ropTest/lopnet/rop_usdrender     DEBUG:     1. /out/vdbcacheTest     DEBUG:     2. /out/pointcacheBgeoTest

I mean, it's not 'wrong' but it does feel a bit unexpected. And it would the rop to instance code I wrote for this PR at least.

I also did some simple test in my side, the input ordering looks good but I do have some questions on the refactoring after looking at Mustafa's test.
Do you think it's better idea to have the custom OP parameters (or a custom OP node, but it's not really for publishing but setting up and collecting custom data for publishing) for input dependency ordering? This is for just in case the users have relatively complex graph in their /out and the ROP extractors can't recognize the ordering.

Repeating what I said here https://github.com/ynput/OpenPype/pull/5891#issuecomment-1812676450 and I will add some more info

HDA doesn't have that method.
Traceback (most recent call last): File "E:\Ynput\OpenPype\.venv\lib\site-packages\pyblish\plugin.py", line 527,

You are looking at OBJ HDAs and not ROP HDAs. ROP HDAs do have that method and that's what we should be looking at supporting if we want an alternative to the Deadline Submission ROP. We don't need dependency graph evaluation or farm submission on the OBJ context!

Because we're splitting OpenPype into ayon-core and individual host addons, this PR would have to be re-created to target one of those.
We're closing it down, but we'll he happy for a new PR to ynput/ayon-core or the host addon repository once it's up.


Data type icons a little closer to Tableau's

Tableau is increasingly becoming a standard in data visualization & analysis so I'd like to borrow their icon language since more people will be familiar with it.
Attached are Tableau's images for Numbers, Text, Dates & Geographic data (lats or longs).
To be clear - I'm not saying use these particular images, just maybe use the same visual language but different icons.

these are the glyphs that come with bootstrap: http://getbootstrap.com/components/#glyphicons
there aren't many icons that match tableau exactly - might need to have aidan design a few.

How about searching the noun project - https://thenounproject.com/? Maybe
something could surface from there and the Engagement Lab could purchase
the icons (usually only like $2 or something)

Assistant Professor of Civic Media and Data Visualization, Emerson College
Fellow, Emerson Engagement Lab
Research Affiliate, MIT Center for Civic Media
www.kanarinka.com | @kanarinka |<PHONE_NUMBER>
On Wed, Nov 18, 2015 at 10:45 AM Jay Vachon<EMAIL_ADDRESS>wrote:

these are the glyphs that come with bootstrap:
http://getbootstrap.com/components/#glyphicons
there aren't many icons that match tableau exactly - might need to have
aidan design a few.
—
Reply to this email directly or view it on GitHub
https://github.com/c4fcm/DataBasic/issues/48#issuecomment-157753646.

oooh wonderful thanks!

oooh wonderful thanks!


Disabling Grafana's anonymous user breaks Cluster Dashboard page

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Install rancher monitoring with grafana[grafana.ini][auth.anonymous].enabled set to false.
Open the Rancher Cluster Dashboard page in Cluster Explorer.

Result:
The user is logged out and redirected to the login page every time the Cluster Dashboard page is opened (which is the default landing page for the Cluster Explorer, so it's very annoying/hard to work around).
Other details that may be helpful:
Inspecting network logs in chrome, it looks like something on the dashboard page is trying to load some grafana data through it's proxy. For example: https://<RANCHER_DOMAIN>/api/v1/namespaces/cattle-monitoring-system/services/http:rancher-monitoring-grafana:80/proxy/api/dashboards/uid/rancher-cluster-nodes-1.
When the default view for Grafana requires authentication, Grafana returns a 401 Unauthorized. My guess is that Rancher's code sees this 401 response and assumes its own session has expired, so it kicks the user back to the login page.
All the other pages seem to work, for instance navigating directly to pods works fine: https://<RANCHER_DOMAIN>/dashboard/c/local/explorer/pod
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): v2.5.8
Installation option (single install/HA): HA / rke/ rancher-monitoring v14.5.100

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Custom/bare-metal
Machine type (cloud/VM/metal) and specifications (CPU/memory): metal / Various x86_64 mini-pc's (2-6 cores/32-64GB each)
Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.7", GitCommit:"1dd5338295409edcfff11505e7bb246f0d325d15", GitTreeState:"clean", BuildDate:"2021-01-13T13:23:52Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.6", GitCommit:"8a62859e515889f07e3e3be6a1080413f17cf2c3", GitTreeState:"clean", BuildDate:"2021-04-15T03:19:55Z", GoVersion:"go1.15.10", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

Docker version 20.10.2, build 20.10.2-0ubuntu1~20.04.2

Hi @CoopeyB , did you manage to solve this issue or find a viable workaround?
I see the same problem on a cluster where anonymous auth was not disabled by default, running rancher v2.5.8.
Thanks,
Marco

HI @mrcdb, nope, I haven't found any workarounds other than re-enabling the anonymous user, which is acceptable in my environment (but I can see this being an unacceptable solution in others).

Hi all,
We have the same issue , could be great to have a workaround for this 👍
Best Regards,


unittest: isFalse matcher should match '== false'

The current behavior matches '!= true' which feels *very* broken.
I guess one could have a notTrue matcher for this.
I just ran a test that I expected to fail because I changed the return type from bool to int, but it still passed because 64 != true.
This is pretty bad IMHO.

This issue has been moved to dart-lang/test#167.


How to customize the color of the action button?

https://github.com/afollestad/material-dialogs/issues?q=is%3Aissue+is%3Aclosed+action+button+color


planner: add Prune function to remove shares

Nothing uses this change yet, but it adds a function to the internal planner
library to Prune shares from an otherwise populated smbcc config.
Once the operator supports adding multiple shares to one instance we'll also
need a way to remove them.
With unit tests.

/jenkins retest sink-clustered/mini-k8s-1.24

/retest centos-ci/sink-clustered/mini-k8s-1.24

I would pay money for some kind of feedback that I've issued the correct command to rerun the jenkins tests.


Node install locally as well as globally

Hi There,
**i just noticed that my node is installed in my root directory as well as home directory. Kindly let me know which one is good to have and how to uninstall other one alone. Is it harm to keep node in both directory.
The purpose of Node installation is for Appium Automation.**  Please advise
In Root Directory:

USCHAMNMOG0JK78:~ root# pwd
/var/root
USCHAMNMOG0JK78:~ root# node --version
v12.7.0
USCHAMNMOG0JK78:~ root# which node
/usr/local/bin/node

In Home Directory:

USCHAMNMOG0JK78:~ vz8zw5$ pwd
/Users/vz8zw5
USCHAMNMOG0JK78:~ vz8zw5$ node --version
v12.6.0
USCHAMNMOG0JK78:~ vz8zw5$ which node
node_modules/.bin/node
USCHAMNMOG0JK78:~ vz8zw5$

your home directory is a project?
Without knowing the shell env it is hard to say what your problem is..  Your question is more suited to the node project.
I will give it a blast..
You installed node via npm in your home directory i.e. via npm install node
and you got https://www.npmjs.com/package/node installed.
That this is also on your shell path (outside of package.json scripts) means you also have something adding $HOME/node_modules/.bin to your PATH in your shell profile.
You need to decide which node you want... the os distributed version or a custom own version...
if you want the os distributed version node you should remove ~/node_modules/.bin/node.
Then also possibly remove ~/node_modules and ~/package-lock.json et al.. who knows


Updated for 0.19

Definitely interested in seeing this released if possible.

It's now published to package.elm-lang.org


"window.ipfs" from service worker

Hello,
Is there a way to access window.ipfs from a service worker?
I tried with self.ipfs and it returns undefined
Thanks!

Unfortunately no. AFAIK WebExtensions are unable to inject content scripts into Service Worker context, that is why there is no self.ipfs.
There is ongoing research on "Universal IPFS API Provider" and ways we can expose API coming from js-ipfs,  HTTP API or proxy created by IPFS Companion.
Places to follow:

https://github.com/ipfs/in-web-browsers/issues/137

poc with sw+iframe+postMessage: https://github.com/Gozala/lunet/

Closing due to https://github.com/ipfs-shipyard/ipfs-companion/issues/589#issuecomment-712356448


Improve property names

For example, "start" and "end" are not very informative for the Y position properties for an area element.  Top and Bottom or some other, semantically richer names would be helpful.

Agreed, handled with ac0dc43!


Update Readme.md with Undo and Redo commands

Updated app name and download link in readme.md
Proofread readme.md

lgtm!


Terrain rotated 90 degrees CW when generating.

Trying to generate the terrain for the University of Louisville, and every time I spawn in it generates the terrain 90 CW. Do you need to be facing a certain direction when you tp to your destination?

When you generate the world you should do orientation: upright. Also, it depends on the projection. Mercator and equatorial keep everything facing the right direction but other ones have different orientations depending on where you are. You can check the generated map in the menu to tell if its facing the way you want.

Thanks, I'll give it a shot.


junk in request

Consider https://github.com/libuv/libuv/blob/v1.x/samples/socks5-proxy/client.c#L311 we are experiencing issues where the client may send the SOCKS request AND initial upstream data before receiving a success/fail message from the SOCKS Proxy for performance optimisation. Similar to Google's pre-fetch.
How might one edit this code so that if size != 0 that the additional data / size may be stored/cached and later sent upstream once a successful connection is made?

instead of return do_kill(cx); would this be acceptable?
outgoing->t.buf[0] = incoming->t.buf[incoming->result - size - 1];
outgoing->result = size;

and then within conn_connect_done
if (c->result > 0) {
  printf("additional upstream data in connect request %u\n", (unsigned) c->result);
  conn_write(c, c->t.buf, c->result);
  return;
}

@saghul I remember you looked at this code earlier, any ideas?

@saghul Any ideas? We've moved buf outside of the conn.t union and conn_write the additional buf once connection is made, but we still see the odd instance of the data not being piped correctly.


Support for WSL/Bash on Windows

Currently opn seems to be unable to open browsers in WSL/Bash on Windows.
I also added the directory with chrome.exe to PATH but it still fails to start it.
Also drive C: is mounted to /c now (see https://blogs.msdn.microsoft.com/commandline/2018/02/07/automatically-configuring-wsl/) and
with built-in path translation the same path to chrome.exe as in Windows cmd should be usable.

Addendum: I just tried this with master and it seems to work.


Remove non supported components

Remove non supported components:

AliabaTarget
ConfluentTarget
HasuraTarget
TektonTarget

This is a breaking change and will require manual removal of CRDs at existing installations.

There is a also manual removal of CRDs and a pending PR for docs.

Docs update
https://github.com/triggermesh/docs/pull/348
@sameersbn ^^


[API] [Event-comment-controller] 401 code is returned in requests where authorization is not required.

Environment: Windows 11,
Google Chrome Version 111.0.5563.147 (64-bit)
Reproducible: Always
Preconditions
Website https://greencity.greencity.social/swagger-ui.html#/ is opened.
Steps to reproduce

Chose to GET /econews/comments/actives?ecoNewsId=151 AND /econews/comments/countreplies/0?page=0&size=5&sort=desc
Click the 'Send' button.
Check responses.

Actual result
Status code 401 Unauthorized
Expected result
Status code 404

retest please


Add missing cosmosdb dependency in template

Add missing cosmosdb dependency in template
📲 What
Dependency for CosmosDb was missing the in the template when CosmosDb was selected.
🤔 Why
This is required so that we have the depedency referenced in the final built template.
🛠 How
Minor code changes in the template.
👀 Evidence
Screenshots / external resources / links / etc.
Link to documentation updated with changes impacted in the PR
🕵 How to test
Notes for QA
✅ Acceptance criteria Checklist

[ ] Code peer reviewed?
[ ] Documentation has been updated to reflect the changes?
[ ] Passing all automated tests, including a successful deployment?
[ ] Passing any exploratory testing?
[ ] Rebased/merged with latest changes from development and re-tested?
[ ] Meeting the Coding Standards?

/azp run

/azp run

/azp run

/azp run

/azp run


Add support for signing assemblies during build (#78)

@cwe1ss I was just typing out the same message, I've just pushed the update to include the PublicSign attribute.

Tests now pass, but we need to be aware there are some consequences described here.
From that link:

However, since the binary is not actually fully signed, it cannot be installed to the GAC, cannot be used with shadow copying, and cannot be loaded in a partial trust context on full framework.

Travis CI is just a "best effort" build environment anyway. I have to deploy to NuGet from a Windows machine as we also target "net45". I'm currently deploying from my local desktop but I want to set up AppVeyor at some point.

That's fair - I'm familiar with AppVeyor and can look into that integration if you like? I'll create a new issue for it.

I'll merge this as there's (hopefully) not much to object. Just comment here or create a new issue if you have any further comments.
Thanks @MikeGoldsmith !


Fixed ci config

[x] Enable yarn installation and cache with appveyor for faster install times
[x] Use clone depth of 1 for faster clone
[x] Fast finish for faster failure reporting

AppVeyor fails?

Looks like its failing because of danger (yarn run danger run -- --verbose). Is it required?

Yeah that's required, but should only run on Travis, not on AppVeyor.

Removed yarn run danger run -- --verbose from appveyor config:

[x] Enable yarn installation and cache with appveyor for faster install times
[x] Use clone depth of 1 for faster clone
[x] Fast finish for faster failure reporting

@mxstbr Ready for merge?

This is awesome, thanks so much @amilajack! Much appreciated ✨


Window hiding doesn't work as expected in fullscreen kiosk mode

The latest window hide changes cause some edge case issues that we need to handle in the fullscreen kiosk mode.
If you hide a window that has previously been in kiosk mode, it will still keep you in kiosk mode with no active window on macOS.
If you enable a window that had not previously been in kiosk mode, the window will have a title bar when you made it a kiosk mode.
Both of these issues go away after an app restart, but they should be cleaned up.

We fixed this in the latest version.


ObjectPaste class miss some details when sample_2d=True

Thanks for your great work!
ObjectPaste class miss some details when sample_2d=True,
sampled_dict = self.db_sampler.sample_all(                 gt_bboxes_3d.tensor.numpy(),                 gt_labels_3d,                 gt_bboxes_2d=gt_bboxes_2d,                 img=img,             )
and db_sampler def sample_all(self, gt_bboxes, gt_labels, img=None):,  Could you update the db_sampler method for the baseline model?

Multi-modal copy-paste (i.e. MoCa or PointAugmenting) is not used in this work. You can assume that sample_2d is False.

Multi-modal copy-paste (i.e. MoCa or PointAugmenting) is not used in this work. You can assume that sample_2d is False.

thanks for your help!


Restore values after restart Home Assistant

Nice work. I don't know if it's possible, but a property to restore attribute values after restarting Home Assistant would be nice.
The variables component (https://github.com/rogro82/hass-variables) has such a feature.

Hello,
Thanks for your feed back!
I will do something respect to restore feature !

Wow, thanks for the quick answer.
Something like this would be nice:
restore_energy_charge: true
restore_energy_discharge: false
restore_battery_charge: true
restore_battery_discharge: true

Thanks in advance!

Hello,
I check the source and I have already put a restore feature. The restore is done at each restart.
I have implemented no option, since I recompute everything.
    async def async_added_to_hass(self):
        """Handle added to Hass."""
        # restore from previous run
        await super().async_added_to_hass()
        state_recorded = await self.async_get_last_state()
        if state_recorded:
            ....

Hello,
Perhaps you wanted to have a specific beahaviour ?
Tell me.

No news.
I close the issue.
If you need a specific beahviour tell me.


Only shows blackscreen when I start

However, app seems to be responsive. If I type Alt + Shift + Esc it closes.
From the docker build I learned I should see a mouse pointer. Is the graphics for that mouse pointer not included in the repos or something like that? sway works on my system and it also makes use of wlc. So I think the error is not on wlc. Ideas?

Can you give us some log output (for now setting $WAY_COOLER_LOG=trace should print to stdout, we're working on it)?
The docker is an old version, we had similar issues when upgrading.
We had a similar bug before - does the screen turn funny colors if you open a program (we had it turn gray when we opened dmenu). The issues seem to be with the background (the Rust wayland crate doesn't always work, currently it's just powering backgrounds), we'll see if the issue is there in the logs.
If you want to test it without setting the background, setting the way_cooler.background to an invalid color would cause the background to not be set.

does the screen turn funny colors if you open a program (we had it turn gray when we opened dmenu)

I didn't manage to open any program. Alt+d and Alt+Enter makes no effect on visual output.
The same error also happens if I open way-cooler as a X client, so I'm not sure if it's a bug related to Wayland or internal to way-cooler. I'll generate the logs for you.

It's almost certainly Way Cooler that's the problem, we were having similar problems on different devices ever since we implemented basic background support. Hopefully we can pin down what the issue is.
Another thing you can try doing is opening Cargo.toml and removing any mention of dlopen. It's supposed to dynamically load the library, but I was also having issues with that on one my Arch machines so try that.

Sorry about the delay, I was reading the logs to be sure no sensitive information about me was being sent (it stores my UID which I don't like, but I'll let you guys see it anyway).
way-cooler-wayland.txt
way-cooler-x11.txt

If you want to test it without setting the background, setting the way_cooler.background to an invalid color (such as the string "black" or "asdf") would cause the background to not be set.

It didn't help.

Another thing you can try doing is opening Cargo.toml and removing any mention of dlopen.

It made a difference. I still can't seem to use it properly (no mouse pointer or open programs when I hit Alt+d/Alt+Enter), but the screen "turn funny colors".

Ok, we think we might have zeroed in on the issue. Rust-wlc is only up-to-date with wlc 0.0.3, and right now the wlc in the Arch repos is on 0.0.5. There may have been some breaking changes between those versions that have affected Way Cooler. Can you try linking it against that version of the library?

Can confirm, was running on my machine with old wlc, updated and it broke. We will update rustwlc right away, we had been putting it off for the Reddit announcement.

Ok, we think we might have zeroed in on the issue. Rust-wlc is only up-to-date with wlc 0.0.3, and right now the wlc in the Arch repos is on 0.0.5. There may have been some breaking changes between those versions that have affected Way Cooler. Can you try linking it against that version of the library?

Will do. Just need a few minutes to build the correct package version.

Can confirm, was running on my machine with old wlc

Can confirm as well. I just downgraded wlc to version 0.0.3 and I've managed to have way-cooler from most recent git version running sound.

We will update rustwlc right away

That sounds great. Thank you.
:)

Awesome to hear that we tracked this down. This means the docker image can be updated to the right version as well, for people who want to try out Way Cooler but don't have the benefit of running Arch to try your PKGBULID :). We should have it updated by tonight (EST), we'll close this issue when it's updated.

Thanks again.

Confirmed on my machine (Arch Linux) as well.
Downgraded wlc to 0.0.3
way-cooler now works correctly

rust-wlc has been updated, as has Way Cooler (including the docker image!), so the PKGBUILD should just work now, no need to downgrade to v0.0.3. If there any other problems, feel free to comment and reopen this thread, I'm going to close it as fixed for now.

Fixes pushed to v0.3.1

Fixes pushed to v0.3.1

Hi guys, sorry about the delay to update the ArchLinux recipes (a.k.a. AUR packages, PKGBUILD...), but I was testing many different build options to find out why the binary being produced was still showing the "same" (just the blackscreen part) problem.
After a while I found cargo install never produced an usable binary while cargo build was okay. I don't know why, but cargo install does something different that makes the generated binary unusable (app looks responsible, but only shows blackscreen).

Hmm, interesting... Did you try removing dlopen from Cargo.toml again like last time? That wasn't having issues on our machines but we've had some trouble working with it, and that seems to cause a change last time.
You might also want to double check it's linking to the right library when ldd when you use install it.
We didn't try testing cargo install, we'll give it a try and get back to you

Hmm, interesting. On my machine when I use cargo install I can open terminals and do things but a lot of the keybindngs aren't working correctly. Moving between windows works but I can't nest them or open dmenu...It's definitely opening the right configuration file too since it's symlinked in ~/.config/way-cooler/init.lua.
Yours sounds like a different issue though, since you can't even open any programs. Not sure if it'll provide any more info, but maybe some more logs can help us track down what's wrong with Way Cooler installing on your machine? Feel free to purge any mention of your UID from it, that only has to do with setting up the IPC socket and shouldn't be causing problems.
@SnirkImmington, I know you changed some xkb things in the latest release, is it possible there was a regression there?

I'm on a tight schedule this week, so I can only get back to play with this project which has a high chance to be my next Desktop Shell and replace my years of loyalty to Enlightenment next week.
I hope to contribute back possibly (a lock screen maybe, stuff like that... and tracking down bugs).

That's excellent to hear, happy to hear you're excited about the project. We'll continue to try and debug this on our side, and track down the other issue you pointed out in #56 and the issue I found in #57. These sound like recent regressions, and hopefully shouldn't be hard to track down. By the time you're ready next week we hopefully will have resolved these issues.

I can reproduce the cargo install issue on my machine now, will report back if I find the issue.

Hm, frankly @vinipsmaker I cannot reproduce the issue. I thought I did before, but that was just an out of date binary from a few months ago I had on my machine.
When you get the chance, can you send us the output of ldd <way-cooler-binary> (if it lists wlc pointing to a generic libwlc.so, use ls -l to give us what the library it ultimately points to to verify you are using the right version of wlc), the Way Cooler log, and your Cargo.toml and Cargo.lock. That would be the best way to narrow down what your environment is like and why it's behaving the way it is.

When you get the chance, can you send us the output of [...]

ldd-out.txt
ls-out.txt

the Way Cooler log, and your Cargo.toml and Cargo.lock

Will have time next week.

This was the failing command:
cargo install way-cooler --root "$pkgdir"

However, I can install way-cooler with the following command, running from the right folder:
cargo install --root "$pkgdir"

I guess I was pulling the latest version from crates.io. It's working okay now.

Good to hear, hopefully this problem will never happen again if/when we switch to statically linking wlc and distributing the full linked binary


Removes wp-embedded-content with a converter

fixes https://github.com/frontity/saturn-theme/issues/78

:tada: This PR is included in version 0.8.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Can't delete releases after upgrade from v20160512 to v20160624.1

Hi,
I upgraded from v20160512 to v20160624.1 via backup restore and wanted to clean up some releases. But I just get "controller: resource not found" no matter what application I try?

➜  ~ flynn -a myapp release
ID                                    Created
2173b16c-3999-4779-b363-ea6e9c285d1c  5 weeks ago
53ee727a-43f0-46b8-aecf-0c8608bca885  6 weeks ago
➜  ~ flynn -a myapp release delete 53ee727a-43f0-46b8-aecf-0c8608bca885
Are you sure you want to delete release "53ee727a-43f0-46b8-aecf-0c8608bca885"? (yes/no): yes
controller: resource not found

Is there a corresponding 404 in the controller web process log?

I think yes?

2016-06-25T18:40:50.510472Z app[web.Flynn1-f90d0e08-b63d-4979-bf05-8915a40cf486]: t=2016-06-25T18:40:50+0000 lvl=info msg="request started" component=controller req_id=6596aedf-4414-4f2a-a340-031e49201859 method=DELETE path=/releases/e7047194-0bfb-4057-8b21-e6721cdcbb38 client_ip=<IP_ADDRESS>
2016-06-25T18:40:50.524595Z app[web.Flynn1-f90d0e08-b63d-4979-bf05-8915a40cf486]: t=2016-06-25T18:40:50+0000 lvl=info msg="request completed" component=controller req_id=6596aedf-4414-4f2a-a340-031e49201859 status=404 duration=13.971428ms

/cc @lmars

@philiplb what version of the Flynn CLI are you using (i.e. flynn version on your local machine)?
The path in the logs is /releases/xxx but it should be /apps/xxx/releases/xxx (changed in fe07945).

That was it, I was on an old CLI version. Thank you! :)


linux: remove hardcoded path to fix CommandNotFound

I got Err(CommandNotFound) on Gentoo GNU/Linux because iwlist lives in /sbin

I have the same problem in Ubuntu (16.04.2). But maybe it's better to use only "iwlist" without touching the user env.

@alopatindev  really sorry i missed this issue. merging the fix. thank you. will publish a release shortly.
cc @jesusprubio

v0.3.5 has been published to crates.io cc @alopatindev @jesusprubio


Fixes for elf_aux_info support

Remove the unnecessary use of the sys/elf.h header

Add RISC-V support

Could you please explain in which scenario these changes are needed? The old code worked for me on RISC-V.


WorkIcon doesnt show in my code

Hi friends
I was trying to put the Workicon in my code I tried also to import but it doesnt work.
I was also trying to add this icon from semantic ui but it doesnt work. Can you please tell how to include this icon in your library?
https://react.semantic-ui.com/elements/icon/
Thanks

Hi,
Thank you for your question. Could you provide an full example ?

I installed the component and where do I find  in your demo?