Grid layout for group creation window?

Open up a secondary issue for the suggestion of grid pattern layout just to keep it organized.

Originally posted by @PikeNote in https://github.com/PikeNote/taskbar-groups-pike-beta/issues/31#issuecomment-1427327268

Suggestion: Grid layout for group creation, since groups are always displayed as a grid. Concept:

v2 concept

deletion buttons should probably be smaller, less virulent shade of red. EDIT function is provided by context menu. Grid helps with layout and can be used eventually for dragging around to increase icon spacing or resize horizontal/vertical layout, eliminating some of the wordiness at the bottom of the page. Icon size could probably be replaced by "Scroll on an icon to increase size" or context menu option. Colour/opacity options could be put under an eyedropper button? Recommend adding a text-box, slightly lighter than background, with flashing cursor and placeholder name. "Cancel" unnecessary, X in top right already does that.

Could even have grid and list layouts, toggled by button, which changes how the programs display in the editor and in the resulting group.

Related: https://github.com/tjackenpacken/taskbar-groups/issues/49


Reduce permissions scope

There is no need for this plugin to "read and change data on all [a user's visited] web sites." At the very least, it should be limited to the web sites it supports, which if I'm not mistaken is a half dozen or so known domains.

Except that MDH works in a surprising number (and variable number) of places, so it's not possible to enumerate them. See:
https://github.com/adam-p/markdown-here/wiki/Compatibility
However, I keep meaning to look into whether there's some mechanism where the extension can request permissions to a site when the user clicks the button. It's not like MDH needs access before then. So I'm leaving this issue open.


Super cool!

Just wanted to share that this is super cool. I've been using Expo and wanting to do an Android (and iOS but Ill try later) widget for a long time, but it was always to hairy and busy to figure out a clean way. Will definitely be using this soon thanks!
Just a side question: Do you know of any lib out there for iOS by chance?
I would love to donate to this package, do you have plans to open sponsorship? Just wanted to say thanks!

Thanks!
Let me know if you build something, I would love to see it.
I don't know of any similar lib for iOS.
No plans for sponsorship right now, since GitHub sponsors is not available in my country.

Ah thank you. That's silly of Github. Hopefully soon they open up one, I'll also be on the lookout.

By the way, quick question: will it say anything if I use it with Expo Go on iOS?

By the way, quick question: will it say anything if I use it with Expo Go on iOS?

What do you mean by "say anything"?
It won't work on iOS, but should not throw errors.

It won't work on iOS, but should not throw errors.

Yeah, i was wondering if it would give me an error on iOS. Thanks for the reply!


ModuleNotFoundError: No module named 'tensorflow.contrib'

It seems that this code was developed against older version.
Apparently in tensorflow 2.4.0 there is no tensorflow.contrib module.

DID THE ISSUE GET SOLVED?

Not yet solved, any solution?


https   request    SSL routines:OPENSSL_internal:WRONG_VERSION_NUMBER

This is a HTTPS request, the certificate created by ourselves, using the okhttp3.8.0 version to respond is ok, but the handshake failed using the 3.8.1 version, and the error message is as follows
: javax.net.ssl.SSLHandshakeException: Handshake failed
at com.android.org.conscrypt.OpenSSLSocketImpl.startHandshake(OpenSSLSocketImpl.java:429)
at okhttp3.internal.connection.RealConnection.connectTls(RealConnection.java:281)
at okhttp3.internal.connection.RealConnection.establishProtocol(RealConnection.java:251)
at okhttp3.internal.connection.RealConnection.connect(RealConnection.java:151)
at okhttp3.internal.connection.StreamAllocation.findConnection(StreamAllocation.java:195)
at okhttp3.internal.connection.StreamAllocation.findHealthyConnection(StreamAllocation.java:121)
at okhttp3.internal.connection.StreamAllocation.newStream(StreamAllocation.java:100)
at okhttp3.internal.connection.ConnectInterceptor.intercept(ConnectInterceptor.java:42)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:92)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:67)
at okhttp3.internal.cache.CacheInterceptor.intercept(CacheInterceptor.java:93)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:92)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:67)
at okhttp3.internal.http.BridgeInterceptor.intercept(BridgeInterceptor.java:93)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:92)
at okhttp3.internal.http.RetryAndFollowUpInterceptor.intercept(RetryAndFollowUpInterceptor.java:120)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:92)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:67)
at okhttp3.RealCall.getResponseWithInterceptorChain(RealCall.java:185)
at okhttp3.RealCall$AsyncCall.execute(RealCall.java:135)
at okhttp3.internal.NamedRunnable.run(NamedRunnable.java:32)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1133)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:607)
at java.lang.Thread.run(Thread.java:761)
Caused by: javax.net.ssl.SSLProtocolException: SSL handshake aborted: ssl=0xe327b780: Failure in SSL library, usually a protocol error
error:100000f7:SSL routines:OPENSSL_internal:WRONG_VERSION_NUMBER (external/boringssl/src/ssl/tls_record.c:192 0xe334faf3:0x00000000)
at com.android.org.conscrypt.NativeCrypto.SSL_do_handshake(Native Method)
at com.android.org.conscrypt.OpenSSLSocketImpl.startHandshake(OpenSSLSocketImpl.java:357)

It seems unlikely the changes between OkHttp 3.8.0 and 3.8.1 could cause this. I suspect the issue is elsewhere in your HTTPS config. Unfortunately I don't think there's anything we can do in OkHttp to fix this.

Closing because I don’t think this is actionable. If I’m wrong, please provide an executable test case!

That's the way it is:
Okhttp uses the 3.8.0 version, and the same code can ask for success
Okhttp uses the 3.8.1 version, and the same code feedback handshake fails, and the log is the code I posted above

Now we are changing certificates, and then try again, there is a problem to ask you

Oh, I made a mistake
The last version we used was 3.4.1, not 3.8.0
So what's the difference between 3.4.1 and 3.8.1?

https://github.com/square/okhttp/blob/master/CHANGELOG.md
My guess is it's this. Work around by creating a ConnectionSpec that supports TLSv1.

“OkHttp no longer recovers from TLS handshake failures by attempting a TLSv1 connection.”

No, I tried this, but still prompted a handshake failure

I just tried again, plus the TLS encryption suite was set up, and I don't know why it wasn't set up before

thank you！
But I have a question,  Why did you do that?

TLSv1 is obsolete and security experts worry about potential compromises like Heartbleed soon becoming possible.


Correct requirements

The required version of Golang is 1.21.
jq is not optional, it is a requirement.

Addresses guacsec/guac#1068

Thank you @lumjjb  and @pxp928
This PR should now be ready.


[etlas] lib directory doesn't exist on windows

On *nix-based systems: ~/.etlas/lib/*eta*/
On Windows: %APPDATA\etlas\*eta*\
On Windows, it's missing lib in the directory tree. This makes it annoying for tooling for handling that special case, so we should align the Windows path with the Unix path.

This was done to shorten paths on Windows which is a serious problem. etlas should expose a command to get all the paths to the important places.


Use AppScan instead of ASoC

 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.


[sw/silicon_creator] Watchdog updates

This PR addresses some open points from (#10632). Please see individual commits for details.

disassembly


Unexpected: The specified task is missing an execution

VSCode Version: 1.45.1
OS Version: macos-10.15

Steps to Reproduce:
I can only reproduce this in CI (using vscode-test), using Github Action (macos-latest).  I cannot reproduce using linux or windows. I also cannot reproduce on my MacOS 10.15 desktop machine.
I'm effectively doing:
vscode.tasks.executeTask(
  new vscode.Task(
    {
      type: 'tasktype',
    },
    'Task Name',
    'tasktype',
    new vscode.ShellExecution('./command')
  )
);

And I get this error & stack.
 Unexpected: The specified task is missing an execution: Error: Unexpected: The specified task is missing an execution
at e.getTaskExecution (/Users/runner/runners/2.262.1/work/vscode-gradle/vscode-gradle/extension/.vscode-test/vscode-1.45.1/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:863:72)
at e.$onDidStartTaskProcess (/Users/runner/runners/2.262.1/work/vscode-gradle/vscode-gradle/extension/.vscode-test/vscode-1.45.1/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:861:200)
at e._doInvokeHandler (/Users/runner/runners/2.262.1/work/vscode-gradle/vscode-gradle/extension/.vscode-test/vscode-1.45.1/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:788:757)
at e._invokeHandler (/Users/runner/runners/2.262.1/work/vscode-gradle/vscode-gradle/extension/.vscode-test/vscode-1.45.1/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:788:449)
at e._receiveRequest (/Users/runner/runners/2.262.1/work/vscode-gradle/vscode-gradle/extension/.vscode-test/vscode-1.45.1/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:787:120)

at e._receiveOneMessage (/Users/runner/runners/2.262.1/work/vscode-gradle/vscode-gradle/extension/.vscode-test/vscode-1.45.1/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:785:970)
at /Users/runner/runners/2.262.1/work/vscode-gradle/vscode-gradle/extension/.vscode-test/vscode-1.45.1/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:784:213
at e.fire (/Users/runner/runners/2.262.1/work/vscode-gradle/vscode-gradle/extension/.vscode-test/vscode-1.45.1/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:46:67)
at v.fire (/Users/runner/runners/2.262.1/work/vscode-gradle/vscode-gradle/extension/.vscode-test/vscode-1.45.1/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:232:615)
at /Users/runner/runners/2.262.1/work/vscode-gradle/vscode-gradle/extension/.vscode-test/vscode-1.45.1/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:957:179
at e.fire (/Users/runner/runners/2.262.1/work/vscode-gradle/vscode-gradle/extension/.vscode-test/vscode-1.45.1/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:46:67)
at v.fire (/Users/runner/runners/2.262.1/work/vscode-gradle/vscode-gradle/extension/.vscode-test/vscode-1.45.1/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:232:615)
at t.PersistentProtocol._receiveMessage (/Users/runner/runners/2.262.1/work/vscode-gradle/vscode-gradle/extension/.vscode-test/vscode-1.45.1/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:237:17)
at /Users/runner/runners/2.262.1/work/vscode-gradle/vscode-gradle/extension/.vscode-test/vscode-1.45.1/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:234:155
at e.fire (/Users/runner/runners/2.262.1/work/vscode-gradle/vscode-gradle/extension/.vscode-test/vscode-1.45.1/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:46:67)
at p.acceptChunk (/Users/runner/runners/2.262.1/work/vscode-gradle/vscode-gradle/extension/.vscode-test/vscode-1.45.1/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:230:129)
at /Users/runner/runners/2.262.1/work/vscode-gradle/vscode-gradle/extension/.vscode-test/vscode-1.45.1/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:229:483
at Socket.t (/Users/runner/runners/2.262.1/work/vscode-gradle/vscode-gradle/extension/.vscode-test/vscode-1.45.1/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:238:565)
at Socket.emit (events.js:203:13)
at addChunk (_stream_readable.js:295:12)
at readableAddChunk (_stream_readable.js:276:11)
at Socket.Readable.push (_stream_readable.js:210:10)
at Pipe.onStreamRead (internal/stream_base_commons.js:166:17)

I've tried a bunch of things to find the cause of this error, but have not had any luck. I don't understand how/why this error occurs, and I'm unable to catch it. Can you offer any advice and what I can do to debug this?

Does this issue occur when all extensions are disabled?: Yes

Hi @alexr00
Unfortunately I'm seeing this race condition again. While the fix for https://github.com/microsoft/vscode/issues/96643 solved the race condition for the state of my code at that time, after moving some code around i'm seeing the race condition again.
Unfortunately I can't provide a repo because it's racy and it looks like it depends on the load on the machine, and will only show when the machine is under heavy load and running slowly.
I can reliably reproduce this in my CI and local (running tests on a specific branch of my extension).
I have however recorded a video: https://i.imgur.com/8MzWs6T.mp4
This appears to be the sequence of events:

$onDidStartTask is called, which calls getTaskExecution passing in an execution object
$onDidStartTaskProcess is called, which calls getTaskExecution passing in execution as a string, but at this point the execution has not been saved to the _taskExecution map, so when it tries to get the task execution (string) from the _taskExecution map, it doesn't exist and throws an error.

Thanks for all the details. The problem seems pretty clear, so I've pushed a fix. However, I can't repro the issue, so I can't verify it. Please let me know if the next insiders build still has this problem!

Sure will do! How can i check when the next insiders build will be released?

The unreleased label on this issue will be removed when it's in the insiders build. That's probably the easiest way to check. Generally, they are released every 24ish hours.

It doesn't appear a new insiders was released in the past 24hrs. I'll check again tomorrow.

hi @alexr00 I have verified this change fixes the bug 🎉
Using insiders, I'm no longer getting this error in CI nor in local.
I'll continue to run my CI tests using insiders until 1.47.0 is released. Thanks!


Added support for release event (Github)

As a github developer (both public and Enterprise) I would like to be able to catch and run CI on the release event, allowing to add artifacts for release via the github GUI.
Reasons:

It is easy for all users (management/lower level devs) to release, revert, get artifacts etc from release page.
It is a part of the github UI.

Changes:

Added global webhook "release"
Added support for parsing webhook "release" to the github vendor.

No release commit. We are using the /refs/tags/... Ref.
No branch is associated.

this is a new feature and because of feature freeze will be assigned to v1.0.0 :)

PS: https://woodpecker-ci.org/docs/administration/vcs/overview#supported-features should be extended

Is there a difference between release and tag in Github / Gitea etc? Is it possible to create multiple releases for a single tag?

Codecov Report

Merging #764 (8ffb392) into master (f16525f) will increase coverage by 0.25%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #764      +/-   ##
==========================================
+ Coverage   52.23%   52.49%   +0.25%
==========================================
  Files          86       86
  Lines        5940     5972      +32
==========================================
+ Hits         3103     3135      +32
  Misses       2665     2665
  Partials      172      172

Impacted Files
Coverage Δ

server/model/const.go
66.66% <ø> (ø)

server/remote/github/parse.go
90.06% <100.00%> (+2.67%)
:arrow_up:

Continue to review full report at Codecov.

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

Hi, that is OK. that version is getting bigger ;) I'm using my own build for now since Bloomberg requires some custom networking. I am though making the users do some loops around the when: thing... so I am waiting for 1.0.0.

this is a new feature and because of feature freeze will be assigned to v1.0.0 :)

Hi, that is OK. that version is getting bigger ;) I'm using my own build for now since Bloomberg requires some custom networking. I am though making the users do some loops around the when: thing... so I am waiting for 1.0.0.

@LamaAni Could you try to add the release event for Gitlab and Gitea as well, so we have it for our 3 most used SCMs? If you need help doing that just ask, please.

@LamaAni Could you try to add the release event for Gitlab and Gitea as well, so we have it for our 3 most used SCMs? If you need help doing that just ask, please.

Hi, I could try and add that. But I cant test that at the moment and before I do I would need a sample json. Can you post one?

PS: https://woodpecker-ci.org/docs/administration/vcs/overview#supported-features should be extended

Fixed

@anbraten Do you have that JSON? Or we can do that in another PR...

Gitlab

Content-Type: application/json
User-Agent: GitLab/14.8.0-pre
X-Gitlab-Event: Release Hook
X-Gitlab-Event-UUID: 579ec8e1-b801-4a48-8ed8-xxx

{
  "id": 4268085,
  "created_at": "2022-02-09 20:19:09 UTC",
  "description": "new version desc",
  "name": "Awesome version 0.0.2",
  "released_at": "2022-02-09 20:19:09 UTC",
  "tag": "0.0.2",
  "object_kind": "release",
  "project": {
    "id": 32521798,
    "name": "ci",
    "description": "",
    "web_url": "https://gitlab.com/anbratens-test/ci",
    "avatar_url": null,
    "git_ssh_url": "git@gitlab.com:anbratens-test/ci.git",
    "git_http_url": "https://gitlab.com/anbratens-test/ci.git",
    "namespace": "anbratens-test",
    "visibility_level": 0,
    "path_with_namespace": "anbratens-test/ci",
    "default_branch": "main",
    "ci_config_path": "",
    "homepage": "https://gitlab.com/anbratens-test/ci",
    "url": "git@gitlab.com:anbratens-test/ci.git",
    "ssh_url": "git@gitlab.com:anbratens-test/ci.git",
    "http_url": "https://gitlab.com/anbratens-test/ci.git"
  },
  "url": "https://gitlab.com/anbratens-test/ci/-/releases/0.0.2",
  "action": "create",
  "assets": {
    "count": 4,
    "links": [

    ],
    "sources": [
      {
        "format": "zip",
        "url": "https://gitlab.com/anbratens-test/ci/-/archive/0.0.2/ci-0.0.2.zip"
      },
      {
        "format": "tar.gz",
        "url": "https://gitlab.com/anbratens-test/ci/-/archive/0.0.2/ci-0.0.2.tar.gz"
      },
      {
        "format": "tar.bz2",
        "url": "https://gitlab.com/anbratens-test/ci/-/archive/0.0.2/ci-0.0.2.tar.bz2"
      },
      {
        "format": "tar",
        "url": "https://gitlab.com/anbratens-test/ci/-/archive/0.0.2/ci-0.0.2.tar"
      }
    ]
  },
  "commit": {
    "id": "0b8c02955ba445ea70d22824d9589678852e2b93",
    "message": "Initial commit",
    "title": "Initial commit",
    "timestamp": "2022-01-03T10:39:51+00:00",
    "url": "https://gitlab.com/anbratens-test/ci/-/commit/0b8c02955ba445ea70d22824d9589678852e2b93",
    "author": {
      "name": "Anbraten",
      "email"<EMAIL_ADDRESS>    }
  }
}

Gitea

Request URL: http://xxx
Request method: POST
Content-Type: application/json
X-GitHub-Delivery: c00b8c17-f187-4d5a-bd17-xxx
X-GitHub-Event: release
X-Gitea-Delivery: c00b8c17-f187-4d5a-bd17-xxx
X-Gitea-Event: release
X-Gitea-Signature: xxx
X-Gogs-Delivery: c00b8c17-f187-4d5a-bd17-xxx
X-Gogs-Event: release
X-Gogs-Signature: xxx
X-Hub-Signature: sha1=e4d4286d15c3f9870fe2c239067eae12xxxx
X-Hub-Signature-256: sha256=74515aa84467fdb69b68133d67a17a07096b5e745f3xxxxx

{
  "action": "published",
  "release": {
    "id": 48,
    "tag_name": "0.0.5",
    "target_commitish": "main",
    "name": "Version 0.0.5",
    "body": "",
    "url": "https://git.xxx/api/v1/repos/anbraten/demo/releases/48",
    "html_url": "https://git.xxx/anbraten/demo/releases/tag/0.0.5",
    "tarball_url": "https://git.xxx/anbraten/demo/archive/0.0.5.tar.gz",
    "zipball_url": "https://git.xxx/anbraten/demo/archive/0.0.5.zip",
    "draft": false,
    "prerelease": false,
    "created_at": "2022-02-09T20:23:05Z",
    "published_at": "2022-02-09T20:23:05Z",
    "author": {"id":1,"login":"anbraten","full_name":"Anton Bracke","email":"anbraten@noreply.xxx","avatar_url":"https://git.xxx/user/avatar/anbraten/-1","language":"","is_admin":false,"last_login":"0001-01-01T00:00:00Z","created":"2018-03-21T10:04:48Z","restricted":false,"active":false,"prohibit_login":false,"location":"world","website":"https://xxx","description":"","visibility":"public","followers_count":1,"following_count":1,"starred_repos_count":1,"username":"anbraten"},
    "assets": []
  },
  "repository": {
    "id": 77,
    "owner": {"id":1,"login":"anbraten","full_name":"Anton Bracke","email":"anbraten@noreply.xxx","avatar_url":"https://git.xxx/user/avatar/anbraten/-1","language":"","is_admin":false,"last_login":"0001-01-01T00:00:00Z","created":"2018-03-21T10:04:48Z","restricted":false,"active":false,"prohibit_login":false,"location":"world","website":"https://xxx","description":"","visibility":"public","followers_count":1,"following_count":1,"starred_repos_count":1,"username":"anbraten"},
    "name": "demo",
    "full_name": "anbraten/demo",
    "description": "",
    "empty": false,
    "private": true,
    "fork": false,
    "template": false,
    "parent": null,
    "mirror": false,
    "size": 59,
    "html_url": "https://git.xxx/anbraten/demo",
    "ssh_url": "ssh://git@git.xxx:22/anbraten/demo.git",
    "clone_url": "https://git.xxx/anbraten/demo.git",
    "original_url": "",
    "website": "",
    "stars_count": 0,
    "forks_count": 1,
    "watchers_count": 1,
    "open_issues_count": 2,
    "open_pr_counter": 2,
    "release_counter": 4,
    "default_branch": "main",
    "archived": false,
    "created_at": "2021-08-30T20:54:13Z",
    "updated_at": "2022-01-09T01:29:23Z",
    "permissions": {
      "admin": true,
      "push": true,
      "pull": true
    },
    "has_issues": true,
    "internal_tracker": {
      "enable_time_tracker": true,
      "allow_only_contributors_to_track_time": true,
      "enable_issue_dependencies": true
    },
    "has_wiki": false,
    "has_pull_requests": true,
    "has_projects": true,
    "ignore_whitespace_conflicts": false,
    "allow_merge_commits": true,
    "allow_rebase": true,
    "allow_rebase_explicit": true,
    "allow_squash_merge": true,
    "default_merge_style": "squash",
    "avatar_url": "",
    "internal": false,
    "mirror_interval": ""
  },
  "sender": {"id":1,"login":"anbraten","full_name":"Anbraten","email":"anbraten@noreply.xxx","avatar_url":"https://git.xxx/user/avatar/anbraten/-1","language":"","is_admin":false,"last_login":"0001-01-01T00:00:00Z","created":"2018-03-21T10:04:48Z","restricted":false,"active":false,"prohibit_login":false,"location":"World","website":"https://xxx","description":"","visibility":"public","followers_count":1,"following_count":1,"starred_repos_count":1,"username":"anbraten"}
}

@anbraten I'll update the PR when I get a chance. I'm having network issues with docker service in services. Something about the dns resolve.

Added support for gita and gitlab. @anbraten @6543 These are not tested since I dont have the means.

linter fail :D

linter fail :D

On?

Hi, Sorry for the late responses. I have set time aside to fix my PR's in about a couple of weeks. I'll fix these then, @6543

dont worry we also dont rush - proper solutions are better than thing merged in a hurry


Not able to change width and height of camera view

Issue Description
[Not able to resize cameraview]
<CameraKitCamera
ref={(cam) => (this.camera = cam)}
cameraOptions={{
flashMode: 'off',
      }}
      style={{

        backgroundColor: colors.transperent,
        marginBottom: moderateScale(450),
        position: 'absolute',
        height:
          Dimensions.get('window').height -
          moderateScale(isIphoneX() ? 308 : 270),
        width: moderateScale(200),

      }}
      resetFocusTimeout={0}
      resetFocusWhenMotionDetected={true}
      saveToCameraRoll={false}
    />

[FILL THIS OUT - It will be extremely helpful]

Environment

React Native Navigation version:5.7.1
React Native version: 0.63.2
Platform(s) (iOS, Android, or both?): [Android and iOS]
Device info : (in all device same issue)

following.

Will look into this.

any update here ?

any update here ?

I'm not able to change width and height of the CameraScreen too<EMAIL_ADDRESS>


Erroneous warning when symbol not specified

Running magic trace without -symbol prints out
[Couldn't find symbol. Will still snapshot on end]

even though that is expected.

Fixed in #79


[libvpx] build failure

Host Environment

OS: Windows
Compiler: MSVC 19.24.28314

To Reproduce
Steps to reproduce the behavior:
./vcpkg install libvpx
Failure logs
Starting package 2/4: libvpx:x64-windows
Building package libvpx[core]:x64-windows...
-- Note: libvpx only supports static library linkage. Building static library.
-- Using cached C:/vcpkg/downloads/libvpx-1.8.1.tar.gz
-- Using source at C:/vcpkg/buildtrees/libvpx/src/libvpx-1-87e9c0e1cf
-- Acquiring MSYS Packages...
-- Acquiring MSYS Packages... OK
-- Acquiring MSYS Packages...
-- Acquiring MSYS Packages... OK
-- PERL_EXE_PATH ; C:/vcpkg/downloads/tools/perl/perl/bin
-- Generating makefile
-- Generating MSBuild projects
-- Building C:/vcpkg/buildtrees/libvpx/x64-windows/vpx.vcxproj for Release
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:72 (message):
Command failed: msbuild C:/vcpkg/buildtrees/libvpx/x64-windows/vpx.vcxproj /p:Configuration=Release /t:Rebuild /p:Platform=x64 /p:PlatformToolset=v142 /p:VCPkgLocalAppDataDisabled=true /p:UseIntelMKL=No /p:WindowsTargetPlatformVersion=10.0.19025.0 /m /p:WholeProgramOptimization=false
Working Directory: C:/vcpkg/buildtrees/libvpx/x64-windows-rel
Error code: 1
See logs for more information:
C:\vcpkg\buildtrees\libvpx\build-x64-windows-rel-out.log
Call Stack (most recent call first):
scripts/cmake/vcpkg_build_msbuild.cmake:119 (vcpkg_execute_required_process)
ports/libvpx/portfile.cmake:73 (vcpkg_build_msbuild)
scripts/ports.cmake:94 (include)
build-x64-windows-rel-out.log
Additional context
Seems like the build process can't find yasm, even though I've added it to PATH

@longnguyen2004 Thanks for reporting this issue.
Could you provide the configure log C:\vcpkg\buildtrees\libvpx\configure-x64-windows-out.log ?
Thanks.

Current workaround: patch the generator script to include the full path to the yasm binary
I'll checkout master again to get the error

yasm should be installed first in VCPKG_PATH/downloads/tools/yasm, so I think you may installed yasm manually.

Nope I didn't, I let vcpkg download it. But I noticed that yasm is contained inside a versioned subfolder (VCPKG_PATH\downloads\tools\yasm\<IP_ADDRESS>). Don't know if that affects the configure process or not
configure-x64-windows-out.log
Here's the configure log, looks alright I think. I've tried adding the yasm dir to PATH but that doesn't seem to change anything. I even tried manually building the generated project in VS but no luck

@dan-shaw Could you have a look?
Thanks.

Probably just a problem with my previous Windows installation, upgraded to Windows 10 and everything seems good

@JackBoosY
MSBuild not find yasm in folder VCPKG_PATH\downloads\tools\yasm\<IP_ADDRESS>
readme file

[1]: http://www.tortall.net/projects/yasm
[2]: For Visual Studio the base yasm binary (not vsyasm) should be in the
     PATH for Visual Studio. For VS2017 it is sufficient to rename
     yasm-<version>-<arch>.exe to yasm.exe and place it in:
     Program Files (x86)/Microsoft Visual Studio/2017/<level>/Common7/Tools/

copy msvc2019 yasm binary, build ok
Microsoft Visual Studio\2019\Enterprise\Common7\Tools

On VS2019 and look into "Current" folder like:
C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\MSBuild\Current\Bin
C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\MSBuild\Current\Bin
Place yasm.exe there.
Regards

@maddinthegreat
https://github.com/microsoft/vcpkg/pull/10952/commits/7185c476ae3a169a106d0d4e86a1772387b2c1c9
https://github.com/microsoft/vcpkg/pull/10952/commits/4a62f86d6b61461739359c24d2905c0713d83a0e
add /p:UseEnv=True
?

@Voskrese: have to use msbuild in "C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\MSBuild\Current\Bin" explicitly.
Regards


Could you guys please support typescript?

Feature request.. Please provide typescript types :)

+1 we're building integrations to your API in AWS using Typescript and not having the types to help keep developers from making dumb mistakes is slowing things down considerably.

Typescript support can be generator with tsc.
There at least one wrong jsdoc type. the method 'serviceAccountCredentialsBuilder()' should return 'ServiceAccountCredentialsBuilder;'
It is only a hour work to support typescript

Thanks @burtonator @Anderman for bringing this request up. Currently, this request is not in our roadmap. We would request you to use our community forum to detail out your requirements along with the use cases, for our Product Management and Tech Evangelist group to evaluate them further.
Link to the community forum: https://community.adobe.com/t5/document-services-apis/ct-p/ct-Document-Cloud-SDK.
Hope this helps.

@burtonator @sfwhite I created a PR in DefinitilyTyped repro. You can review and test and approve it to speedup the PR. see the review link. Examples how to use can you find in the testscript

for Typescript you can use npm install --save @types/adobe__pdfservices-node-sdk


Strata/Platform/Host - UI: minor updates

Issue:

bcgov/entity/issues/24807
bcgov/entity/issues/24944
bcgov/entity/issues/24897

Description of changes:

fix strata application payload
log user out and redirect to housing page when declining tos
fix modal close button not working

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

Temporary Url for review: https://strr-platform-dev--pr-394-gcxeaqr3.web.app

Temporary Url for review: https://strr-strata-hotel-dev--pr-394-fsh76qnt.web.app

Temporary Url for review: https://strr-hosts-dev--pr-394-vfbzccb2.web.app

Temporary Url for review: https://strr-platform-dev--pr-394-gcxeaqr3.web.app

Temporary Url for review: https://strr-strata-hotel-dev--pr-394-fsh76qnt.web.app

Temporary Url for review: https://strr-platform-dev--pr-394-gcxeaqr3.web.app

Temporary Url for review: https://strr-strata-hotel-dev--pr-394-fsh76qnt.web.app

Temporary Url for review: https://strr-hosts-dev--pr-394-vfbzccb2.web.app

Temporary Url for review: https://strr-platform-dev--pr-394-gcxeaqr3.web.app

Temporary Url for review: https://strr-strata-hotel-dev--pr-394-fsh76qnt.web.app

Temporary Url for review: https://strr-hosts-dev--pr-394-vfbzccb2.web.app


Iss 100: Implement interfaces and constants for generic API methods

Addresses #100

Added interfaces SetWriteOption, SetExOption, and added argument GET in Set API to replace SetOptions struct.
Added interface ExpireOptions to replace ExpireOptions, PExpireOptions, ExpireAtOptions, and PExpireAtOptions structs.
Added GetExOption interface to replace GetExOptions struct.
Adjusted logic in Set, Expire, ExpireAt, PExpire, PExpireAt to accept new interfaces.
Fixed various method calls that broke with new interface implementation.

Working on documentation

Added new struct SETOptions to use as a cleaner way to pass in options to SET API.
Adjusted documentation to reflect changes.
Added EXPIREAT and PEXPIREAT pages to the docs.


Error in the toolbar.js

Hi,
I am using Django 2.1.2.
I try to show information with djdt (sql, parameters or static files) but I have the following error:
Uncaught TypeError: fetch is not a function     at ajax (toolbar.js:45)     at HTMLAnchorElement.<anonymous> (toolbar.js:88)     at HTMLUListElement.<anonymous> (toolbar.js:7)
In information page (gray, yellow), the spinner continue to run (acts like fetching data).

You

I am using Chrome Version 73.0.3683.103 (Build officiel) (64 bits)
This is the latest version.
I have the same problem on Safari.
On Ubuntu 18.04 with Chrome & Firefox the problem is the same

That's very surprising. I'm running similar versions without problems. Could it be you're setting window.fetch to some variable in your code somewhere, e.g. fetch = "bla" without var? What is the value of fetch when you enter it in the console?
fetch should be available for years now (https://caniuse.com/#search=fetch)

You've right
I am using django notification module and in the template I write fetch=12 to get the 12 notifications.
This is why the fetch doesn't work.
Thank you for your answer, I close the issue.


[Bug] [Task] HiveCli SQL execution failed, but the task node shows success.

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
In the shell node, multiple SQL files are called using hive -f, and each file contains multiple SQL statements. When one of the SQL statements fails, the subsequent SQL statements continue to execute, and the status of the shell node is marked as successful.

What you expected to happen
In the shell node, after invoking SQL via hive -f, if there are failures in the SQL execution, the subsequent SQLs should not continue to execute, and the status of the shell node should be marked as failed.
How to reproduce
If there are 5 SQL commands in a Hive SQL file and the second command fails, the third, fourth, and fifth commands will still be executed. If these commands are executed successfully, the node will be marked as having executed successfully.
Anything else
No response
Version
3.2.x
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

Yes, this should be a bug, we might need to split the sql, and check each sql status after finished.

Yes, this should be a bug happens when the script contains multiple sqls, we might need to split the sql, and check each sql status after finished.

The HiveCli SQL run base on shell, When build the shell ,use set -e ,could fix this ?

Yes, this should be a bug happens when the script contains multiple sqls, we might need to split the sql, and check each sql status after finished.

The HiveCli SQL run base on shell, When build the shell ,use set -e ,could fix this ?

I have not test this, but it might not fix this due to the multiple SQLS are been executed by one command, rather than multiple commands.

Yes, this should be a bug happens when the script contains multiple sqls, we might need to split the sql, and check each sql status after finished.

The HiveCli SQL run base on shell, When build the shell ,use set -e ,could fix this ?

I have not test this, but it might not fix this due to the multiple SQLS are been executed by one command, rather than multiple commands.

Oh,This doesn't look like a very good solution for one command .I will check it .

Yes, this should be a bug happens when the script contains multiple sqls, we might need to split the sql, and check each sql status after finished.

use multi shell command also need check each command status,we produced bugs in prod env


doc: prepare go document

Prepare for go document.
Signed-off-by: Sam Yuan<EMAIL_ADDRESS>

@bestbeforetoday , @channingduan , @davidkhala please free to comments here.

making it to draft to make it same structure with https://github.com/hyperledger/fabric-gateway/blob/main/go.mod#L1

@bestbeforetoday , I just refactored package structure to https://github.com/hyperledger/fabric-gateway please review.

merge it as it is.


Implementation is different with the paper?

Hi, thank you for sharing you work.
I read the TPAMI paper, but i didn't found the two stream architecture in this repositories. The paper propose  Pose Adaptive Convolution and Projected Normalized Coordinate Code which are not contained in this repositories either.
Well, from the main.py, it just crops face region in image and feeds to mobilenet. I wonder why using simple regression  strategy can produce promising results as you provided? Is data augmentation matters?

Well, good question, you are the first one to propose it.
To be honest, the experiments in this project's conclusion is not completely consistent with the original paper.

I use the 3x4 matrix (11 degrees if more precise) form to substitute the original R (rotation, 3 degrees) and t_2d (2 degrees) in the original paper.
In this repo, VDC finetuned from WPDC loss is better than WPDC loss, while there is no such experiment in the original paper. The original paper's conclusion is that WPDC is better than VDC.
PAC and PNCC are all implemented, but the two-stream cascade structure does not bring any performance improvement under my circumstance.
The augmentation part has no big difference with the original paper. But it matters.

Why simple regression strategy can produce promising results? The method still needs improvement.

@cleardusk Thank you for your quick reply. I am very interesting in PAC and PNCC since it may work in human pose estimation.
From the params.py, you load some .npy file. How do you generate those file?
From main.py, you said the network's output control shape, pose and expression of the face. I wonder how they can control the expression, because as I know, there is no such label in the training set.

I use BFM face model, the network's output is pose, shape, expression parameter. 3DDFA is parameter regression.
I will update the code of PAC and PNCC, but it may require a little time to release as I am busy these days.
*.npy are generated from BFM (original form is *.mat)

Well, good question, you are the first one to propose it.
To be honest, the experiments in this project's conclusion is not completely consistent with the original paper.

I use the 3x4 matrix (11 degrees if more precise) form to substitute the original R (rotation, 3 degrees) and t_2d (2 degrees) in the original paper.
In this repo, VDC finetuned from WPDC loss is better than WPDC loss, while there is no such experiment in the original paper. The original paper's conclusion is that WPDC is better than VDC.
PAC and PNCC are all implemented, but the two-stream cascade structure does not bring any performance improvement under my circumstance.
The augmentation part has no big difference with the original paper. But it matters.

Why simple regression strategy can produce promising results? The method still needs improvement.

I also wonder why regression strategy works but your result proved it, nice work. What I care about is that since your current implementation only need parameters as the ground truth, does this mean I can prepare the training data by just taking down the first 40 attributes  of the shape info and the first 20 attributes of the expression info with 12 elements of the projection matrix from 300W_LP dataset?

Yes, but if you do some pre-process to the original image, the corresponding param should be modified, such as scale, offset.

@cleardusk How do you generate param_all_norm.pkl? Do you release the code for generating param_all_norm.pkl?

Sorry for that, the training details will be updated in the next two months.

@cleardusk How do you generate param_all_norm.pkl? Do you open source the code to create the param_all_norm.pkl


Render-Target Camera and Screen fixes

Fixes broken RT Screen material variants dropdown
Fixes RT Camera darkening when halos are rendered
Fixed some cases when last frame from disabled RT Camera are shown

Halo fix detours Garry's mod hook for drawing halos. Proper PR to garrysmod repo needed.

Are you waiting for me to finish #2578 before merging this PR and #2577?

No, just giving other reviewers a chance to look

@stepa2 does this PR fix this?


Fix #116 Always create $DATA_PATH

In case no datasets were downloaded, we still need the path for
Notebooks.

@paskino, this overwrites some of your recent changes. Do you see any issues with WORKING_DIR?

looks good to me. did you test if the mkdir -p doesn't complain too loudly when it wants to create it in a read-only location?

It will still crash if the location doesn't exist, but will be happy if it exists, even if we don't have permission. I think this is the desired behaviour


SCRUM-2784 make short citation show up next to the entry box when sel…

…ected

Not different. I accidentally closed the first one :)


Allow token from secretKeyRef for the Hook resource

What problem are you facing?
I would like to create a Gitlab hook with a token specified in a kubernetes secret. Currently I would have to create a composition with the secret to use patching. This method also displays the token directly on the hook resource.
How could Crossplane help solve your problem?
Introduce a spec.forProvider.tokenRef field for this functionality.

thats a duplicate for this Issue https://github.com/crossplane-contrib/provider-gitlab/issues/133

Yes it is, thanks for your contribution.


python 3.9 not resolved correctly

ammar@pc:~$ rez-env python-3.9

You are now in a new environment.

running rez-config v1.6.9

requested packages (mode=latest, time=1677032475: Wed Feb 22 10:21:15 +08 2023):
lin64
python-3.9

resolved packages:
lin64               /server/rez/packages/lin64
python-3.9.0        /server/rez/packages/python/3.9.0

number of failed attempts: 0

context file:
/tmp/.rez-context.LA6TKxtRdM

> ammar@pc:~$ python -V
Python 2.7.17
> ammar@pc:~$ python3 -V
Python 3.6.9

my system is :
Ubuntu 18.04
rez-config v1.6.9
default python
python v2.7
python3 v3.6.9
how to make python3.9 load correctly,
thanks

Hello @ammar0466 , can you show us the package definition (package.py) of your python-3.9.0 rez package please? Basically the output of rez-view python-3.9 -f py?

server/rez/packages/python/3.9.0
|_ package.py
|_ package.yaml
|_ /bin
|   |_ rezpython  (on file detail, Type: "link to /usr/bin/python3")
|_ /lin64
|_ /win64
package.py
name = "python"

version = "2.9.0"

authors = ['ammar']

variants = [
    ["lin64"],
    ["win64"]
]

requires = [
    'site_env'
]

# @late()
def commands():
    if this.index == 0:
        #   linux
        #
        env.PATH.append("{ROOT}/bin")
    else:
        #   windows
        pass # on windows we have python installed locally

package.yaml
config_version : 0

name: python

version: 3.9.0

uuid: de78a12d-8dc7-4b2d-b00b-62d264043923

authors:
- ammar

description: >
 This is a dummy package to be used as variant setter for other python packages.
 This version of python was system default for Ubuntu 18.04

commands:
- export PATH=$PATH:!ROOT!/bin

ammar@pc:~$ rez-view python-3.9 -f py
rez-view: command not found

I see env.PATH.append("{ROOT}/bin"). Try {root} instead of {ROOT}.
Also, was this created using rez-bind? And what's the version of Rez you are using?

/server/software/lin64/rez/1.6.9/bin/rez

still same , do i need to rebuild python package after edit {root}.

rez old version.
found another way to install , using rez version 2.112.0
https://github.com/j0yu/rez-python


Create Github Action to Build Docker Image

For our initial testing of the flux integration, we need to see what payload is getting passed through and when, we have the Go App for this but now need to put it inside Kubernetes beside flux so therefore need it pushed to a container registry

PR raised: https://github.com/keptn-sandbox/keptn-flux-integration/pull/8


⚠ Samsung has degraded performance

In 15fec85, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2267 ms

Resolved: Samsung performance has improved in 1533307.


allow multiline managed by ansible

ansible_managed is allowed to be a multiline string.
the PR lets every line start with a "#".

Successfully merging this pull request may close these issues:  #336


default topic should not be deleted

We should have a default topic that can't be deleted. This helps when deleting topics. All articles under the deleted topic can then be moved to the default topic.
The behavior of deleting topic is not defined yet. I suggest to move all the articles to the default topic.

@MinweiShen Thanks, but it'd be great if the logic could be moved to the backend. For now you're hiding the delete button for the default topic. If the action could be prevented in the backend that'll be better. If we just hide the delete link anyone could simulate the request from another client and delete that topic.
Also, yes, moving all the articles under a deleted topic to a default topic is the apt way to go about it. Let's get on it :smile:

Sounds correct. What about now? I modified the code.

Merged. Thanks :smile:


multi-host parallel training cannot recognize all gpu

Hi I am following https://github.com/hpcaitech/ColossalAI/tree/main/examples/images/diffusion and trying to use 2 hosts with each 1 T4 GPU. I changed
lightning:
trainer:
accelerator: 'gpu'
devices: 2
And had error:

lightning.fabric.utilities.exceptions.MisconfigurationException: You requested gpu: [0, 1] But your machine only has: [0]
lightning.fabric.utilities.exceptions.MisconfigurationException: You requested gpu: [0, 1] But your machine only has: [0]

Is there other config I should assign to recognize multi-host?

We have fixed it and will release a detailed blog with the AWS team together. Thanks.

https://aws.amazon.com/cn/blogs/china/run-colossal-ai-based-distributed-finetune-tasks-on-sagemaker/
We are happy to release this blog together. Thanks.


BLE GATT Object Transfer Service

PR changes:

Added Object Transfer Service library
Added Peripheral OTS sample which demonstrates how to use service library.

The sample lacks some mandatory components (Readme and yaml files). I'm planning to add them once review for the library is finished.

@Thalley, @joerchan, thank you for your reviews - I addressed all your comments.
I have been marking them as resolved so that it is easier for me to track remaining work items. However, if you think that something is still missing after my response or fix in code, you can unresolve your review comment.

I tagged the OTS API as experimental so that we can still make changes to the API after it is merged.

This PR contains commits from another PR in order to work:
https://github.com/zephyrproject-rtos/zephyr/pull/27830
They will be removed after linked PR is merged.

This PR contains commits from another PR in order to work:
#27830
They will be removed after linked PR is merged.

One of them are also part of https://github.com/zephyrproject-rtos/zephyr/pull/27840

@Thalley, in the last patch, I addressed all comments except the unresolved one from your last review round.

I would appreciate if we could get this in before the Zephyr freeze. There are still two unresolved comments regarding naming discussion. I hope we can resolve them soon.


net/http: improve useFakeNetwork check to allow browsers to run tests in wasm

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

$ go version
go version devel +1531192272 Mon May 27 17:58:39 2019 +0000 linux/amd64

net/http/rountrip_js.go has this code:
// useFakeNetwork is used to determine whether the request is made
// by a test and should be made to use the fake in-memory network.
func useFakeNetwork() bool {
	return len(os.Args) > 0 && strings.HasSuffix(os.Args[0], ".test")
}

This code has gone through several iterations. First it was:
host, _, err := net.SplitHostPort(req.Host)
if err != nil {
	host = req.Host
}
if ip := net.ParseIP(host); ip != nil {
	return ip.IsLoopback(ip)
}
return host == "localhost"

Then, it was changed to:
return len(os.Args) > 0 && path.Base(os.Args[0]) == "node"

And finally,
return len(os.Args) > 0 && strings.HasSuffix(os.Args[0], ".test")

The latest version will always return true if any code is invoked with go test. That prevents wasm codepath from being taken if the test is being run using a binary (by replacing the -exec flag) which spins up a browser, and loads the wasm file in it and captures the logs. Essentially a test environment for wasm code inside the browser.
I stumbled onto this while writing a tool like that. The current workaround from my end is to rename the binary to .wasm if it ends with .test. But ideally, this should be fixed in the code.
@johanbrandhorst @neelance @bradfitz

What do you propose?

Actually, I can't recall why the 2nd version was changed. The only difference between 2nd and 3rd is that from catching all node-spawned processes, now it catches all tests under js,wasm flag.
Perhaps -
len(os.Args) > 0 && path.Base(os.Args[0]) == "node" && strings.HasSuffix(os.Args[0], ".test")

sounds better ? Since it filters both node processes and tests, allowing browser tests to run.

The reason it was introduced was to allow the gerrit tests to pass, so if such a change would still allow the gerrit tests to pass while allowing tests to be run in the browser then I think that sounds good.

I meant what exactly changed in the logic from v2 to v3 to allow trybots to pass. AFAIU, v3 generalizes the logic to follow a non-wasm path for all tests. But v2 already allowed node. So how did it fail ? (assuming wasm trybots failed)
Anyways, I am afk at the moment, but I will check if my suggestion passes both node and browser tests.


Is it possible to export releases for deployment projects

Is it possible to export releases for deployment projects to grafana to build release overview dashboard ?
We have several deployment projects with deployment environment. It would be great to export release version ( release names ) for each environment and project to build deployment overview dashboards.

I will take a look at
On Mon, Feb 10, 2020 at 08:39 Frintrop<EMAIL_ADDRESS>wrote:

Is it possible to export releases for deployment projects to grafana to
build release overview dashboard ?
We have several deployment projects with deployment environment. It would
be great to export release version ( release names ) for each environment
and project to build deployment overview dashboards.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/AndreyVMarkelov/bamboo-prometheus-exporter/issues/11?email_source=notifications&email_token=AAPXM4A4KJJCXLBGP3IDHDLRCF7NRA5CNFSM4KSRRD62YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IMJ4EXQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPXM4CMVOUA2HKVEFJRXLTRCF7NRANCNFSM4KSRRD6Q
.

--
-- Andrey Markelov


Allow loading from model with empty weights.

Description
(Brief description on what this PR is about)
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

Curious to know, why is this being merged to other branches? #16061 seems to be tackling similar issue right?

@ChaiBapchya Thanks for pointing it. Closing this request out and look forward to getting the master one pushed.


AppleBitz New Crypto Market

https://applebitz.org/

First of all, hi.
I see you offer bitcoin but I can't see monero listed as a mean of payment.
Could you clarify?

Please clarify how you support XMR @AppleBitz

The website is not accessible anymore, and the submitter has not respond in a month. This should be closed. @erciccione? @SamsungGalaxyPlayer?

+resolved


Fixed #334 - Added support for HttpServerOptions on the configuration…

… level

We worked on the same issue :) Please update port and it will be ready.

@tomaszmichalak - done


Adding a new TileDirection.INVALID value causes existing code to crash

I attempted to add TileDirection.INVALID, and without changing any other code, the game crashes with the following error when trying to move a unit to a new tile:
System.ArgumentOutOfRangeException: Specified argument was out of the range of valid values. (Parameter 'Invalid TileDirection')
   at C7GameData.Tile.HasRiverCrossing(TileDirection dir)

This is strange because I did not reference the INVALID enum anywhere in code other than adding it to the enum declaration.

I imagine some code is iterating over the enum values and calling HasRiverCrossing for each. It's probably a good idea for most enums to have a default or invalid value though.

Looks like it's GameMap's computeNeighbors. Should be easy to add a check if you want to add an Invalid TileDirection

Good find. I wonder if there's a common pattern to deal with this kind of thing.


added diagram

#approved but can't we get someone with a modicum of artistic ability to make these diagrams?

Oh please.  Merging.


change because error: E: The method driver /usr/lib/apt/methods/https…

issues: https://github.com/apache/incubator-superset/issues/6256
Got error
E: The method driver /usr/lib/apt/methods/https could not be found.
E: Failed to fetch https://dl.yarnpkg.com/debian/dists/stable/InRelease
E: Some index files failed to download. They have been ignored, or old ones used instead.

when without apt-transport-https
and got warning
debconf: delaying package configuration, since apt-utils is not installed

without apt-utils

Codecov Report

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

@@           Coverage Diff           @@
##           master    #6258   +/-   ##
=======================================
  Coverage   77.02%   77.02%
=======================================
  Files          64       64
  Lines        9500     9500
=======================================
  Hits         7317     7317
  Misses       2183     2183

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 59109f6...e5c46c0. Read the comment docs.

Do we know what actually requires these deps? If so, can we add a comment above the line you added explaining why this is required?

@mistercrunch  I see , i will add comment above the line


Rendering subTemplates in Templates {{>subTemplate}} - anyone got this working?

Hi guys,
does anyone know a hack, how to render subTemplates in .html templates??? It seems like {{>subTemplate}} is NOT supported, which is a same as I'll start keeping it unDRY from know on... damn...
Any suggestions?

Sadly, if the meteorhacks:ssr doesn't support it, this package can't either :/ Perhaps switching to Jade as template engine will support it.

Ha, well.. yeah so if we want to stay with Blaze we'd have to officially fork meteorhacks:ssr... sounds like a fun and time-intensive thing to do! :-/ Right now I plan to stay with Blaze so no need for Jade... 👍

seems like there even is an existing fork https://github.com/meteorhacks/meteor-ssr/pull/40#issue-124286924, which has NOT been included... it just lies around there since dez 2015

Since meteorhacks is NOT developed anymore, how about fetching it into the lookback package (including the pull-request from above)... so we have it in "our codebase" and complete control

I or Lookback have no immediate need for sub-templates right now, so feel free to pull down this package at your end and experiment with the changes you're suggesting :) And then submit a PR if everything works smoothly, and I'll have a look at it.

Ok, i might give it a go, but only after I experience 3 times in my code repeating myself. :-O
Which way would you recon? I guess the most elegant way would be to create a fork of meteorhacks:ssr and publish it as an athmosphere package and then make it a dependence in lookback?

Since the SSR package hasn't (or won't likely) get any updates, I think it's safe to fork it into a new package, yes. Make sure that feature branch is update with master of the SSR package, etc.

OK, I found a way to do this without too much work:

create 1 file per subtemplate within /private/*
compile the subtemplate (and give it a name) by SSR.compileTemplate('subTemplate', Assets.getText('test_subtemplate.html'))
use it within your main template via "{{>subTemplate data=dataContext}}"

Of course it would be really cool to support this with standard blaze-style, BUT it works for now

Cool!


Fixes crash when storage isn't available at bootstrap

Self-tracing during Finatra bootstrap when cassandra was down caused the
process to crash. This makes it a log warning instead.
16:29:18.435 [main] WARN  c.t.z.builder.QueryServiceBuilder - Unable to trace bootstrap due to NoHostAvailableException(All host(s) tried for query failed (tried: localhost/<IP_ADDRESS>:9042 (com.datastax.driver.core.TransportException: [localhost/<IP_ADDRESS>:9042] Cannot connect)))

Note: api calls will fail w/ HTTP 500 until storage is available.
Fixes #1007
Fixes https://github.com/openzipkin/docker-zipkin/issues/74

PS the reason this works is cassandra's repository is already lazy wired. the only thing dereferencing it at bootstrap was self-tracing.
https://github.com/openzipkin/zipkin/blob/master/zipkin-cassandra/src/main/scala/com/twitter/zipkin/cassandra/CassandraSpanStoreFactory.scala#L47


Added freegeek/models.py with Appointment, Station, Factory models. I…

…ssue #21

Appointments should also have "slots", each with a "proficiency" requirement for the slot to be filled, and the "slots" should be able to be associated with a "user", or be "empty".


Fixed screenshot when scene has multi-cameras. Issue #9201

Issue #9201

Any update on this? want to resolve the conflicts?

Conflict fixed.
On my side, I would like to know about the call of scene.render() as it has been pointed here. Even though there is a solution that moves the calling of scene.render() before changing the cameras, it still unanswered for me if it is mandatory to call scene.render() after changing the camera.

Better just keep the scene.render() call & go with this one
https://playground.babylonjs.com/#Y0CG29#4
Now thinking it's to avoid problems, if the function is called before any rendering has been done, like from PG without delay

yes @gabrielheming yes this is mandatory to update everything (matrices, viewport, cache, etc..)


please help errors and i can't build. update the code and the .exe

when i run it i get this errors:
"
[2024-10-23.10:11:00] [Info] Compiling OpenCL kernels...
2 errors generated.
[2024-10-23.10:11:00] [Info] Error: <kernel>:1935:34: error: passing '__generic unsigned int *' to parameter of type 'unsigned int *' changes address space of pointer
                                 ripemd160sha256NoFinal(hash, digestOut);
                                                              ^~~~~~~~~
                             <kernel>:291:67: note: passing argument to parameter 'digest' here
                             void ripemd160sha256NoFinal(const unsigned int x[8], unsigned int digest[5])
                                                                                ^
                             <kernel>:1949:34: error: passing '__generic unsigned int *' to parameter of type 'unsigned int *' changes address space of pointer
                                 ripemd160sha256NoFinal(hash, digestOut);
                                                              ^~~~~~~~~
                             <kernel>:291:67: note: passing argument to parameter 'digest' here
                             void ripemd160sha256NoFinal(const unsigned int x[8], unsigned int digest[5])"

用cuBitCrack.exe

Make sure you have installed all required dependencies for the program.

what GPU are you using? I found in my case I get the same errors if i try to run the cl version on my RTX 3050, I don't get them on the intel CPU. CUDA version seems to work fine on 3050.


Istio sidecar/envoy updating application listen ports in kubernetes

Bug description
Hello,
We are facing issue with Istio sidecar injection with our GRPC app in kubernetes. We have created Ingress Gateway, Virtual Service and Destination Rule. Gateway is publicly exposed, Service has port mapping where request should route and also have destination rule for traffic routing.
So now what happens is as soon as we inject the istio sidecar it stops one of the port 50051 which our app is listening and that breaks things to work. Below is the sample yaml file for reference.
Please let us know how to fix this.
Expected behavior
Listener ports shouldn't be changed
Steps to reproduce the bug
Deployment.yaml
apiVersion: extensions/v1beta1
kind: Deployment
metadata:
annotations:
deployment.kubernetes.io/revision: "2"
creationTimestamp: 2019-05-02T06:09:38Z
generation: 3
labels:
app: example-dev
name: example-dev
namespace: default
resourceVersion: "2734130"
selfLink: /apis/extensions/v1beta1/namespaces/default/deployments/example-dev
uid: ddc75612-6ca0-11e9-966f-42010a8000a3
spec:
progressDeadlineSeconds:<PHONE_NUMBER>
replicas: 1
revisionHistoryLimit: 10
selector:
matchLabels:
app: example-dev
strategy:
rollingUpdate:
maxSurge: 1
maxUnavailable: 1
type: RollingUpdate
template:
metadata:
creationTimestamp: null
labels:
app: example-dev
spec:
containers:
- env:
- name: bt_hhd_instance
valueFrom:
configMapKeyRef:
key: bt_hhd_instance
name: dev-ws-configmap
- name: bt_ssd_instance
valueFrom:
configMapKeyRef:
key: bt_ssd_instance
name: dev-ws-configmap
- name: is_staging
valueFrom:
configMapKeyRef:
key: is_staging
name: dev-ws-configmap
- name: username
valueFrom:
secretKeyRef:
key: username
name: credentials
- name: password
valueFrom:
secretKeyRef:
key: password
name: credentials
image: gcr.io/example/svc-example@sha256:e5b6643534f723d423165386421d4bd2d01ce4fb6c6b75cc300b28f69134d012
imagePullPolicy: IfNotPresent
name: svc-example-sha256
ports:
- containerPort: 80
name: http2
protocol: TCP
- containerPort: 8080
name: http
protocol: TCP
- containerPort: 50051
name: grpc
protocol: TCP
resources: {}
terminationMessagePath: /dev/termination-log
terminationMessagePolicy: File
dnsPolicy: ClusterFirst
restartPolicy: Always
schedulerName: default-scheduler
securityContext: {}
terminationGracePeriodSeconds: 30

example-dev-gateway.yaml
apiVersion: networking.istio.io/v1alpha3
kind: Gateway
metadata:
name: example-dev-gateway
spec:
selector:
istio: ingressgateway
servers:

port:
number: 50051
name: grpc
protocol: GRPC
hosts:

'*'

port:
number: 8080
name: http
protocol: TCP
hosts:

'*'

example-dev-virtualservice.yaml
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
name: example-dev-virtualservice
spec:
gateways:

example-dev-gateway
hosts:
'*'
tcp:
match:

port: 50051
route:
destination:
host: example-dev-service
port:
number: 50051

example-dev-destinationrule.yaml
apiVersion: networking.istio.io/v1alpha3
kind: DestinationRule
metadata:
name: example-dev-destinationrule
spec:
host: example-dev-service
trafficPolicy:
loadBalancer:
simple: ROUND_ROBIN
Version (include the output of istioctl version --remote and kubectl version)
ISTIO_VERSION=1.0.5
How was Istio installed?
We have installed the istio by following this article:
https://medium.com/google-cloud/istio-grpc-loadbalancing-with-gcp-internal-loadbalancer-ilb-1b058b517640
Apart from this we have also tried with enabling the feature of istio from GKE itself, both cases result is same.
Environment where bug was observed (cloud vendor, OS, etc)
Google Kubernetes Engine
Affected product area (please put an X in all that apply)
[X] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
Additionally, please consider attaching a cluster state archive by attaching
the dump file to this issue.

Please open an issue against the Istio project. Thank you.


Custom HTTP methods

Is there a way to call custom HTTP methods like SUBSCRIBE, NOTIFY with this?

Hi @arjunmenon, Is this fixed?

I have moved on to okhttp

For future coders... https://github.com/kevinsawicki/http-request/issues/144 https://github.com/kevinsawicki/http-request/issues/91 https://github.com/kevinsawicki/http-request/issues/53
I need to use PATCH method. Thus, we got such a workaround. Can use any protocols
HttpRequest request;
if (method.equals("PATCH")) {
    request = new HttpRequest(url, "POST");
    HttpURLConnection conn = request.getConnection();
    Field statFinField = HttpURLConnection.class.getDeclaredField("method");
    statFinField.setAccessible(true);
    statFinField.set(conn, "PATCH");

    statFinField = HttpsURLConnectionImpl.class.getDeclaredField("delegate");
    statFinField.setAccessible(true);
    DelegateHttpsURLConnection z = (DelegateHttpsURLConnection)statFinField.get(conn);

    Field delegateField = HttpURLConnection.class.getDeclaredField("method");
    delegateField.setAccessible(true);
    delegateField.set(z, "PATCH");

    statFinField.set(conn, z);

    statFinField = HttpRequest.class.getDeclaredField("requestMethod");
    statFinField.setAccessible(true);
    statFinField.set(request, "PATCH");

}
else {
     request = new HttpRequest(url, method).basic(this.user, this.password).accept("application/json");
}


Comments: link to reported comments shows no comments

https://unacms.com/d/can-t-find-reported-contetnt
I think solution would be to show all comments when module isn't selected, then both links (to all comments and all reports) from Dashboard will start to work.

@AntonLV
Link to reported comments - https://ci.una.io/test/cmts-administration?order_field=reports&order_dir=desc - isn't showing sorted grid


Added NERDTreeHighlightCursorcolumn option to toggle cursorcolumn

Description of Changes
Closes #

added the else case for g:NERDTreeHighlightCursorline, to make it work properly
added g:NERDTreeHighlightCursorcolumn, because I use cursorcolumn in my config but there was no option to disable it in the NERDTree

New Version Info
Author's Instructions

[X] Derive a new MAJOR.MINOR.PATCH version number. Increment the:

MAJOR version when you make incompatible API changes
MINOR version when you add functionality in a backwards-compatible manner
PATCH version when you make backwards-compatible bug fixes

[X] Update CHANGELOG.md, following the established pattern.

Collaborator's Instructions

[ ] Review CHANGELOG.md, suggesting a different version number if necessary.
[ ] After merging, tag the commit using these (Mac-compatible) bash commands:git checkout master
git pull
sed -n "$(grep -n -m2 '####' CHANGELOG.md | cut -f1 -d: | sed 'N;s/\n/,/')p" CHANGELOG.md | sed '$d'
git tag -a $(read -p "Tag Name: " tag;echo $tag) -m"$(git show --quiet --pretty=%s)";git push origin --tags

Thanks for the submission, but I'm not going to merge this. I don't want to keep adding options to NERDTree for every possible configuration. What you're doing here can easily be accomplished in your .vimrc file like so:
autocmd FileType nerdtree setlocal nocursorline nocursorcolumn

ohh.... I didn't know that.
thanks :)


feat(text_input): For ignored events, pass Ignored

For a small side-project of mine I wanted to implement the following feature: On my GUI, I have three text inputs on top and I wanted that pressing F1 would focus the first text input, F2 would focus the second text input ect, even if another text input is focused.
That was working fine, when no other text input was focused, but as soon as one was focused the Event of pressing F1, F2, F3 was just caught by the text-input without doing anything with them or passing them as ignored. Currently only Tab, Up and Down-events are not caught by the text-input.
In this PR, for all Events of type Event::Keyboard(keyboard::Event::KeyReleased{...}) or Event::Keyboard(keyboard::Event::KeyPressed{...}) that are not explicitly handled, the text input will return event::Status::Ignored.
For my local example this worked fine without impacting the functionality of the TextInput but I don´t know if that collides with the overall architecture of the crate.

I have nothing against this PR, but I find the motivation interesting: why do you use three text inputs instead of one?

I have nothing against this PR, but I find the motivation interesting: why do you use three text inputs instead of one?

Do you mean my side project? There I want to design workouts for my bike-trainer. A workout consists of individual efforts, which is basically holding a wattage for a specific time-span. So for an effort you need potentially three inputs: length in minutes, wattage in the beginning, wattage in the end (in between it is linearly interpolated). So I naively used hree text-inputs :shrug:

The implementation here doesn't seem correct. There are definitely keyboard events related to text input that we are now marking as ignored. See #652 for the reasoning behind the current implementation.
You can listen to captured events with the events_with subscription. Does that not work for you?


Authentication: set actor based on Sandstorm headers

https://docs.sandstorm.io/en/latest/developing/auth/ documents the headers.
Discord conversation: https://discord.com/channels/823971286308356157/996877076982415491/1017843070097698946

These look like the relevant headers:

X-Sandstorm-Username
X-Sandstorm-User-Id
X-Sandstorm-Permissions
X-Sandstorm-Preferred-Handle
X-Sandstorm-User-Picture
X-Sandstorm-User-Pronouns

Everything except X-Sandstorm-Tab-Id.

I'm going to map these to keys in the "actor" dictionary like so:

X-Sandstorm-Username -> username
X-Sandstorm-User-Id -> id
X-Sandstorm-Permissions -> permissions
X-Sandstorm-Preferred-Handle -> preferred_handle
X-Sandstorm-User-Picture -> `picture'
X-Sandstorm-User-Pronouns -> pronouns

Note that username is treated specially by this Datasette function, used to show the user's logged in state in the navigation bar: https://github.com/simonw/datasette/blob/5aa359b86907d11b3ee601510775a85a90224da8/datasette/utils/init.py#L1036-L1040
def display_actor(actor):
    for key in ("display", "name", "username", "login", "id"):
        if actor.get(key):
            return actor[key]
    return str(actor)

X-Sandstorm-Username is percent encoded, needs to be decoded.

I'm going to turn permissions into a list.


programming-language agnostic, not programming language-agnostic

The introductory paragraph currently reads:
"The OpenAPI Specification (OAS) defines a standard, programming language-agnostic interface description for HTTP APIs..."
I think it should rather read:
"The OpenAPI Specification (OAS) defines a standard, programming-language agnostic interface description for HTTP APIs..."
It currently reads as if "programming" were a separate item, as though the punctuation were "a standard, programming, language-agnostic interface description".

no, you should read it chunked like this: "... (programming language)-agnostic ..."


🌐 Add Turkish translation for docs/tr/docs/tutorial/path-params-numeric-validations.md

🌐 Add Turkish translation for docs/tr/docs/tutorial/path-params-numeric-validations.md
Original File
Discussion: https://github.com/tiangolo/fastapi/discussions/9193

📝 Docs preview for commit 6ba5f62901e0861134f0a78f4929c3506d27622a at: https://22817ce6.fastapitiangolo.pages.dev

📝 Docs preview for commit 348fa3ff6d95d4739c98c4ff0821d206d9191670 at: https://c9d3cef1.fastapitiangolo.pages.dev

Do you happen to know three other Turkish developers you can ask a PR review from?

Do you happen to know three other Turkish developers you can ask a PR review from?

I am going to look into this situation, thank you for noticing the lack of Turkish support for PRs.


I was struggle in section 'Set up an Azure Pipelines release'

Now Azure DevOps being used so I lost some track to follow your instruction regarding to create the release as Azure DevOps did not have the option for 'Azure Resource Group Deployment'. I did try another option but did not work as guideline.
Could you update the guide, I am quite keen to implement this to my project.

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

ID: 1e22e4d6-366f-fbf8-e141-8905ca7ab7ae
Version Independent ID: 5be2acf2-66e7-cadf-6666-5a85a65b9771
Content: Continuous integration and delivery in Azure Data Factory
Content Source: articles/data-factory/continuous-integration-deployment.md
Service: data-factory
GitHub Login: @douglaslMS
Microsoft Alias: douglasl

@nhoomtham . Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

@nhoomtham We have assigned the issue to the content owner to review and update the doc as appropriate.
@craigg-msft Can you please have a look at this one ?

@gauravmalhot For awareness.

@nhoomtham
We sincerely apologize for the delayed response to this issue. I am working in collaboration with the owners of the GitHub Issues related to Data Factory to help either get them resolved or to determine how to update our documentation.
As such, is this GitHub Issue still a concern for you? If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion, and we will reopen the issue.
Due to the age of this issue, we will continue to close this issue.
To see what the community is wanting, please visit the UserVoice site.
Thank you.
@djpmsft for awareness
#please-close


When unloading and loading a new scene (Non additive), managers do not reinitialize properly

Does this affect the legacy HoloToolkit (master) or the Mixed Reality Toolkit (mrtk_release)?
mrtk_development
Describe the bug
GazeInput is not registered on Start. RaiseSourceDetected is called in GazeProvider#OnStart but await WaitUntilInputSystemValid does not continue.
When quitting application Assertion fails:
Assertion failed: Gaze was never registered with the Input Manager!
UnityEngine.Debug:Assert(Boolean, String)
Microsoft.MixedReality.Toolkit.SDK.Input.MixedRealityInputManager:RaiseSourceLost(IMixedRealityInputSource, IMixedRealityController) (at Assets/MixedRealityToolkit-SDK/Features/Input/MixedRealityInputManager.cs:521)
Microsoft.MixedReality.Toolkit.SDK.Input.GazeProvider:OnDisable() (at Assets/MixedRealityToolkit-SDK/Features/Input/GazeProvider.cs:343)

To Reproduce
Steps to reproduce the behavior:

Start a scene with MixedRealityManager and a GazeProvider with Default Cursor
Cursor is not visible
Cursor does not appear on object with collider

Actual behavior
Cursor stays in the middle of the head and is inactive
Unity Editor Version
2018.1.9f2
Mixed Reality Toolkit Release Version
vNext (mrtk_development)
Rev: 7c960a6e46c331171900f24e09e3935e6ccc583a

Tested in a freshly created project with nothing than a cube in the scene, and there it works.

Retested and could reproduce in multiple scene scenarios (Networkmanager loads next scene).
This is related to: https://github.com/Microsoft/MixedRealityToolkit-Unity/issues/2653
Is there a quick workaround please? (e.g. remove the wait and let gracefully re-register the input?)

Did you mark the camera as persistent in your camera profile?

Yes, tick is set and camera is not destroyed.

You only have one Mixed Reality Manager and no camera's in your other scenes?

Yes. Additionally there is an error:
XR: OpenVR Error! OpenVR failed initialization with error code VRInitError_Init_HmdNotFound: "Hmd Not Found (108)"!
despite there is no OpenVr controller mapping set. Don't know if this could be related.

So it happens when you're targeting standalone?

No, same for UWP build target. Tested in Editor. Scripting Backend is .Net not IL2CPP.
OpenVR Error only with Standalone Target.

I got it running when I replace the WaitUntil with a Coroutine:
        private void RaiseSourceDetected() {
            StartCoroutine(RaiseSource());
        }

        private IEnumerator RaiseSource() {
            while (InputSystem == null) yield return null;
            InputSystem.RaiseSourceDetected(GazeInputSource);
            GazePointer.BaseCursor?.SetVisibility(true);
        }

Let me test this out. I'd like to keep the async. Can you write me a simpler network scene setup I can repro this bug with on one of your banches?

GazeTest.zip
Totally agree. I attached a simple scene, just open the SampleScene and start. There is a script on the NetworkManager, ("SceneLoad") loading the offline scene.
When in the offline scene you cannot see the cursor and when you stop an exception occurs.
For me it seems like a unity bug with the WaitUntil() due the input system is already there but the waituntil does not continue (maybe wrong instance?).

Can you also post a copy of the exception you're running into please?

Yeah the old uNET stuff isn't very friendly as they unload everything in the scene. I thought we solved this at the summit by marking the main camera as don't destroy on load (which is optional). I'll take a look after work later today.

Likely this is an issue with more than just the input manager, but may be something we need to stress test with all managers, so I updated the issue title.

The error is:
failed: Gaze was never registered with the Input Manager! UnityEngine.Debug:Assert(Boolean, String) Microsoft.MixedReality.Toolkit.SDK.Input.MixedRealityInputManager:RaiseSourceLost(IMixedRealityInputSource, IMixedRealityController) (at Assets/MixedRealityToolkit-SDK/Features/Input/MixedRealityInputManager.cs:521) Microsoft.MixedReality.Toolkit.SDK.Input.GazeProvider:OnDisable() (at Assets/MixedRealityToolkit-SDK/Features/Input/GazeProvider.cs:343)
When putting breakpoints in the RaiseSourceDetected:

When RaiseSourceDetected() is called from GazeProvider#OnStart() I get already the direction vector of: Debug.LogWarning(InputSystem.GazeProvider.GazeDirection); before the WaitUntilInputSystemValid.

I removed the networkmanager and in the end it boils down that just a SceneManager.LoadScene("offline"); can reproduce the issue.
I'm really curious what's under the hood of WaitUntil() because in the end it's just an IEnumerator too.

WaitUntil is unity's built in yield instruction.

@wassx I was able to fix this by moving the LoadScene to the update and loaded after pressing space bar.
again, be sure to have the camera profile persistence enabled.
    private void Update()
    {
        if (Input.GetKeyUp(KeyCode.Space))
        {
            SceneManager.LoadScene("offline");
        }
    }

Likely the system breaks bc we load a scene before the whole system starts up.
There's no real way to protect against that.

Ok, still had troubles with the offline scene. Now I added a "WaitForSeconds" in the Start of our networkmanager before loading the offline scene. This would be a workaround, but very ugly. Would it be possible to have an event from the MxrMgr when its init is done?


please have a review to the structures before further coding

Is this related to #9251?

Is this related to #9251?

It is not used currently, you can close it.


Add a "Select all" button to the dimension detail table when searching for values

@ericpgreen2 @jkhwu  Where do you think the Select all button should be ?

Details here - https://www.notion.so/Select-all-shortcut-1e20bcd2e2514ab5bdcd7b500a965909

Closed by #3364


fix: care non-null terminated chunk data

I encountered a broken JSON audit log when used sanitizeArg on the body.
{"transaction":{"time":"17/May/2019:03:58:01 +0000","transaction_id":"AcVcAcAcAcAcAzAcAcWQ4cAc","remote_address":"<IP_ADDRESS>","remote_port":37868,"local_address":"<IP_ADDRESS>","local_port":80},"request":{"request_line":"POST / HTTP/1.1","headers":{"Host":"test.modsec.com","User-Agent":"curl/7.54.0","Accept":"*/*","Content-Type":"application/json","Content-Length":"41"},"body":["{\"password\":\"****\",\"user_id\":\"../../../\"}f�#IV"]},"response":{"protocol":"HTTP/1.1","status":405,"headers":{"Content-Type":"text/html","Content-Length":"173","Connection":"keep-alive"},"body":"\r\n<title>405 Not Allowed</title>\r\n\r\n
405 Not Allowed
\r\n
nginx/1.15.0\r\n\r\n\r\n"},"audit_data":{"messages":["Warning. Unconditional match in SecAction. [file \"/etc/nginx/modsecurity.d/rules/modsecurity.conf\"] [line \"265\"] [id \"101\"]"],"handler":"IIS","stopwatch":{"p1":45,"p2":78,"p3":1,"p4":0,"p5":2,"sr":0,"sw":0,"l":0,"gc":0},"response_body_dechunked":true,"producer":"ModSecurity for nginx (STABLE)/2.9.3 (http://www.modsecurity.org/)","server":"ModSecurity Standalone","sanitized":{"args":["password"]},"engine_mode":"ENABLED"}}

I fixed this problem.

Merged! thanks!


Suggestion: declare module 'react' as React

TypeScript Version:  2.1.1 / nightly (2.2.0-dev.201xxxxx)
2.1.1
Code
Always we use some npm packages need to import like this:
var react = require('React');

In TypeScript way it is:
import * as React from 'react';

Many packages like this.
The main problem is, if I want to use React in TypeScript way, I may need a declaration like this:
declare module 'react' {
  //...
}

But this is not friendly to code auto completation, nothing refer that 'react' should be used as React.
So, what if we can assign a default name to import * from 'xxx' ? Like
declare module 'react' as React

We don't plan to add additional import syntax beyond what ES6 already has.
I can't say for sure but maybe you're confused about how all the importing stuff works -- there's nothing saying you need to import "react" as React. The second name is arbitrary and can vary from file to file.


require --force to bump python formulae with resources and formulae w/patches

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Running bump-formula-pr on a python formula with resources can lead to the resources getting out of date as described in https://github.com/Homebrew/brew/issues/8032.

@Homebrew/core thoughts? I'm open to whatever makes the most sense that both prevents unwanted bumps to happen but also doesn't put people in the habit of using --force unless they're really sure a formula should be updated.

I agree about not getting into the habit of using --force, maybe it would make sense to add a separate flag, like --verified or --checked

--skip-resource-checks?

--skip-resource-checks sounds good, but then would it also make sense to have a --skip-patch-checks flag as well? I'm trying to figure out the optimal balance between being helpful when suggesting/reviewing a PR but not being overly noisy/restrictive or getting people into a habit of doing --force, but also not adding a ton of new specific options.

I think resource and patch should be different. And force should ignore all.

I think it's OK to add a couple more flags in there. These kinds of checks will actually help a lot as we start efforts to e.g. automate running brew bump-formula-pr https://github.com/Homebrew/brew/issues/5725. This means that we'll get ultimately get fewer broken pull requests that maintainers have to check by hand.
If e.g. --skip-patch-checks or --skip-resource-checks gets added it would be a good idea to add that message to the pull request text, as well, to notify maintainers that the patch or resources might need some extra :eyes:

--force seems appropriate here rather than adding more flags. I think ideally we'd actually verify if they need to be updated and use homebrew-pypi-poet to do so (including it in Homebrew/brew to run directly).

There's a path forward for automatically resolving python dependent resources, but the larger question of other resources and patches remains.
There's a general consensus that there are some objective conditions where is undesirable to allow bump-formula-pr to run and that there should be a way to override it, but which conditions and how to override them is unclear. A downside of --force is that it not only would override this check, but anything else that could be overridden by the --force flag whereas the more targeted flags would only override the thing they're explicitly overriding. Adding a bunch more flags may be confusing though.
That being said, I'll leave this open for now until some sort of agreement can be reached.

How many PRs are we getting using this tool that aren't bumping the resources correctly? The way I see it there's three options:

--force or similar like this PR
leave this as-is until we have a better pypi-poet-like solution
don't allow these PRs to be created with this tool at all until we have a better pypi-poet-like solution

I can't give you numbers but I'm guessing about 40% of all PRs with resources in the formula don't check them. And that's only because it's usually a more experienced group that makes the remaining PRS.
I'm in favor of the version with split parameters because it's easier to understand and "less scary"

I can't give you numbers but I'm guessing about 40% of all PRs with resources in the formula don't check them.

Yeh, that doesn't seem good. Perhaps we should only allow brew bump-formula-pr usage on PRs with either no resources or resources where the version is the same as the main formula (and the resources can be bumped accordingly e.g. git.

@dtrodrigues can you take a look at the updated version of #8059? It's possible that it will solve this problem.
That PR has bump-formula-pr call the new update-python-resources automatically. If update-python-resources is unable to update the resources for whatever reason, it will fail (stopping bump-formula-pr as well) with a message prompting the user to manually update the resources.
If this kind of functionality is still wanted, a flag could be added that will cause update-python-resources to warn that it's unable to update resources but allow bump-formula-pr to continue and the PR to still be opened (maybe with a label saying that resource updates were skipped)

The work being done for update-python-resources will be more than sufficient to handle the reason I opened this PR in the first place. However, I think there are still two major cases where we don't want bump-formula-pr to be able to be blindly run:

formulae with patches
formulae with non-python resources

Patches is the case I feel most strongly about, since in theory, they should only exist until the next release because upstream will have merged them in. Of cours this isn't always the case, but it's worth flagging to the user to verify to prevent outdated patches from sticking around.

Since it's not as clear-cut when to alert users when bumping formulae with resources, I force-pushed this to be just formulae with patches since that's probably causing more consistent errors.

Would it make sense to still show a warning when --force is passed? I know if you pass --force because it's found an existing PR it will still warn you. I don't know if any other failures that can be overridden by --force do this.

Yeah, I think warning instead of erroring is fine here.

I've updated it to still show a warning about patches even if --force is passed in.

It might be an idea to benchmark how long it would take to do a brew install invocation that only tries to fetch and apply patches. That seems a better idea than just --force (which will likely just teach users "always run with --force").
I don't think we want/need people to be doing the entire install process locally before/when running brew bump-formula-pr but given we're running audit already I think adding fetch and apply_patches steps would be a reasonable check to have locally.

I agree that we don't want to get people in the habit of using --force, but just because a patch applies cleanly, doesn't mean it wasn't already merged in, so checking if applying patches succeeds would help, but would not be sufficient.
Most patches in homebrew should be ones that were accepted/merged upstream already, and there are a fair number of formulae with patches which include comments with phrasing like "Upstream patch, remove for next release", so I think causing users who are bumping formulae with patches to manually verify and use the --force flag prior to submitting shouldn't be overly burdensome.

but just because a patch applies cleanly, doesn't mean it wasn't already merged in

It does, actually 😉.

so I think causing users who are bumping formulae with patches to manually verify and use the --force flag prior to submitting shouldn't be overly burdensome.

Well, I'd argue you're only actually forcing them to use --force here. I'm not what about this step is going to encourage manual verification; you're just teaching users to run --force every time, no?

I think trying to see if any patches apply cleanly prior to bump-formula-pr is a good idea to investigate, but I don't have time to look into it in the short-term and it's diverged quite a bit from the original intent of this PR (requiring --force to bump python formulae with resources, or more generally, requiring passing of a flag to bump formulae when certain conditions are met), so I'm gonna close this.
I do like @SeekingMeaning and @jonchang's suggestion for a flag like --skip-patch-checks, but there doesn't seem to be consensus to use a new flag vs force vs neither.

Yeah, I think something like a warning would be appropriate. I agree that shouldn't be teaching contributors and maintainers to pass --force regularly.

Thanks anyway @dtrodrigues ❤


Rancher ingress and Traefik: How to properly disable rancher-ingress-controller

Rancher Version:
v1.2.0-pre2
Docker Version:
1.12.1
Let's try to be simple and comprehensive :wink: It will be about Rancher with Kubernetetes. I am a beginner in k8s and rancher so if you think I misunderstood some points.
I want to use Rancher+k8s for deploying a bunch of different apps. I will use namespaces for isolate each others.
Hosts are in a private cloud and public traffic will be routed by a historic nginx, configured by hand.
For exposing the app, I am thinking to create one or more Ingress in each namespace/app that should be exposed.
With Rancher ingress, it will create a new container binding a host port for each Ingress.
I would use host based rules for being able to bring all the rules together (all rules from all namespaces), expose a single port and is scaled at the number of hosts.
It is possible to accomplish with Rancher Ingress ?
I was able to produce that with Træfɪk, a modern reverse proxy. I simply deployed a template like this k8s.rc.yaml.
But it conflict with Rancher Ingress controller, for each Ingress created, Rancher create a load balancer and bind a host port.
It is possible to disable Rancher Ingress ?
I tried to stop rancher-ingress-controller in the system stack but, 1. it seems hacky, 2. it makes the Rancher UI display view "Setting up Kubernetes..." view.
The question on the forum (asked by my colleague): https://forums.rancher.com/t/rancher-ingress-and-traefik/3930
Thanks

@Fridus stopping an individual container won't help as service reconcile will restart it. You can stop ingress-controller service, and it will disable rancher ingress controller. Note that if you decide to upgrade k8s system stack to the latest template version, the service will be restarted, and you might need to start it again.

But if i disable rancher ingress controller the kubernetes view disappear and kubernetes installation get active because this service is not running.

Does rancher ingress obey kubernetes.io/ingress.class ?Perhaps that could be used? https://github.com/kubernetes/contrib/tree/master/ingress/controllers/nginx#running-multiple-ingress-controllers
https://github.com/kubernetes/contrib/blob/master/ingress/controllers/gce/BETA_LIMITATIONS.md#disabling-glbc

Looks like in v.1.1.4 it does not.

I think this should be revisited -- it appears there is no way to disable Rancher's k8s ingress integration without causing Catalog to fail (it returns to the "Setting up Kubernetes" screen)

In the latest k8s catalog (k8s v1.6.2), we add in a configuration option to remove the ingress controller.

@deniseschannon I see an option "Enable Kubernetes Add-ons", that disables all services.
Is there an other option with finer granularity to disable only the ingress controller?

@laszlocph This will be available when our template for k8s 1.6.x is released.

I just updated Rancher to 1.6.2 to have this. However I don't see the option ( 'ENABLE_RANCHER_INGRESS_CONTROLLER') in the UI when configuring a template env. ( not a problem in the end)
I edited the template in the API and set this:

_"type": "catalogTemplate",
"answers": {
"CONSTRAINT_TYPE": "none",
"CLOUD_PROVIDER": "aws",
"REGISTRY": "",
"ENABLE_RANCHER_INGRESS_CONTROLLER": "false",_

However the rancher-ingress-controller still started by default.

@mcedica - support for disabling the ingress controller is in Rancher 1.6.3, released 4 days ago (and not yet tagged stable). This issue is closed, so continuing conversation in it does not notify the Rancher team. If you find that you're still having issues when running 1.6.3 or later, please open a new issue.

How can we set the option?

Just want to confirm that this works for me.
You can edit the the Kubernetes environment template.

Click the first dropdown with your environments and click "Manage Environments".

On the environments view on the bottom you see the Environment Templates section, where click edit on the Kubernetes one.

On this view you can rename your template, etc, change orchestration type. Click "Edit Config".

Make sure it says "Kubernetes 1.6.6" on the top. You need Rancher v1.6.3 for this. Then find the "Enable Rancher Ingress Controller" radiobutton.

Once you updated this, the next deployment, or your next upgrade will adhere to this setting.
If you watch what the browser send to the Rancher API you will see that it's the ENABLE_RANCHER_INGRESS_CONTROLLER answer in the /v2-beta/projectTemplates/#{kubeTemplateId} endpoint.

Thx! Yes, I was testing with rancher 1.6.2 initially. In Rancher 1.6.4 I can not edit the default Kuberentes template, I see the option, but I get a 404 when saving the template with a PUT at https://ranchertest.io.nuxeo.com/v2-beta/projecttemplates/1pt1.
However I  have created a new  k8s template and I can disable the rancher ingress controller in the new template

@mcedica - I can't reproduce your 404 problem.  Which browser were you using?  Did you change anything else in the config or template beside setting ingress to false?

@laszlocph Its unclear to me how I would apply an updated template to an existing environment. I never understood the "templating" stuff in Rancher. Maybe I haven't found the documentation.
Anyway, I have followed your instructions to disable the Rancher ingress controller. How do I get my current k8s environment updated so that these changes will take effect?

Go to Kuberetes / Infrastructure Stacks. Under the "kubernetes" collapsable group, click on the "Up to date" or "Upgrade available" buttons on the right side.
It let's you pick a newer template if there is any, or allows you to fiddle with the settings of the current one. Now I have only upgraded in the past, so I don't know if changing the existing template's params will trigger a reconfiguration, but upgrade does. Hope it will turn the ingress off for you. You might have to delete the ingress controller manually after changing the template. If you do it now without changing the template, it will always redeploy it :)

Thanks! What threw me is that the button is actually called "up to date". I'd never thought of that being an entrypoint into changing things up. I'll add a separate bug for that, but I managed to remove the Rancher controller so I'm happy.


PLAT-43488: Force to update when jumping to 0 index

Enact-DCO-1.0-Signed-off-by: YB Sung<EMAIL_ADDRESS>Issue Resolved / Feature Added
When jumping to the target item far from a current item, we added a placeholder in VirtualList to keep focus in it. Then we deleted it when the target item got to focus. But when jumping to 0 index, there was still a placeholder even though the 0 index item got to focus.
Resolution
When getting to focus the 0 index item after jumping, this.nodeIndexToBeFocused became null. But render() in VirtualListBase was not called. On the other hand, when jumping to another index item, this.nodeIndexToBeFocused became null, the firstIndex of VirtualList's state changed, then the render() was called 1 or 2 times more. Finally, the placeholder could be deleted correctly.
So I tried to force render() when this.nodeIndexToBeFocused became null.
Additional Considerations
Links
PLAT-43488
Comments

I created the new PR #1026 instead of this PR.