Integration tests

Depends on #5. Run with it/run. Not yet documented.

It would be useful to separate the components of this test that are S3-specific from the components that  are not. Then we can provide a property checks suite to anyone writing a connector. And for that reason, I think avoiding the specs2 dep makes sense :)
Though make sense to factor this out like that after we've written a couple more integration tests for specific lwc's. Do we want the general test infrastructure (not connector dependent) in its own repo or in quasar?

@alissapajer I don't see any need for property testing here. I initially looked at that approach, but realized:
a) The only law-like thing I have is "paths returned by children must also return true for exists"; but using randomly generated paths makes no sense here
b) The tests don't depend on generated data; all we do is pull down the full filesystem, compare it with what we believe should be present, and run some extra exists calls.
Also, what we have in here is entirely non-specific to S3; all that's connector-specific is the definition of "expected" TestFS values.
I believe that as long as we aren't factoring out other, more change-sensitive things (like assembleLWC) it makes sense to do what I have in the LWC template right now: https://github.com/edmundnoble/lwc.g8/blob/master/src/main/g8/it/src/main/scala/quasar/physical/LWCTests.scala

I'm :+1: to not factoring this out yet; that's just something that at some point I think will make sense, so we can continue to consider when to do it as work progresses.


Update README.md

Updating WSK CLI build instructions.

Partially closes #1

@pritidesai Please take the doc of openwhisk-wskdeploy as the reference, though it is not perfect yet: https://github.com/openwhisk/openwhisk-wskdeploy/blob/master/README.md.
We need to at least add how to build the project, how to use the cli, how to build Cross Compile Binary with Gradle/Docker, etc.

@pritidesai I can merge this patch for now, but I will propose a more comprehensive change to the readme doc.


Block comments in RTL don't have expected position

Expected Behavior
Please describe what should happen
Comment is attached just to the right of the block in LTR:

In RTL , it should appear just to the left.
Actual Behavior

Steps to Reproduce
load https://scratch.mit.edu/projects/230672117/

This is specifically an issue for when block comments are loaded from a 2.0 project.


Unhandled promise rejection when shard takes too long to log in ("Client took too long to become ready")

This may be related to discord.js instead of the bot itself. Difficult to replicate.
(node:11632) UnhandledPromiseRejectionWarning: Error: Shard 0's Client took too
long to become ready.
    at Timeout.setTimeout [as _onTimeout] (C:\Content\_Dev\Discord-Trivia-Bot\no
de_modules\discord.js\src\sharding\Shard.js:89:31)
    at ontimeout (timers.js:482:11)
    at tryOnTimeout (timers.js:317:5)
    at Timer.listOnTimeout (timers.js:277:5)
    at emitWarning (internal/process/promises.js:65:17)
    at emitPendingUnhandledRejections (internal/process/promises.js:109:11)
    at runMicrotasksCallback (internal/process/next_tick.js:124:9)
    at _combinedTickCallback (internal/process/next_tick.js:131:7)
    at process._tickCallback (internal/process/next_tick.js:180:9)
(node:11632) Error: Shard 0's Client took too long to become ready.
    at Timeout.setTimeout [as _onTimeout] (C:\Content\_Dev\Discord-Trivia-Bot\node_modules\discord.js\src\sharding\Shard.js:89:31)
    at ontimeout (timers.js:482:11)
    at tryOnTimeout (timers.js:317:5)
    at Timer.listOnTimeout (timers.js:277:5)
(node:11632) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
    at emitWarning (internal/process/promises.js:92:15)
    at emitPendingUnhandledRejections (internal/process/promises.js:109:11)
    at runMicrotasksCallback (internal/process/next_tick.js:124:9)
    at _combinedTickCallback (internal/process/next_tick.js:131:7)
    at process._tickCallback (internal/process/next_tick.js:180:9)

The shard does fix itself after a few seconds, so in most cases manual action is not required until unhandled promise rejections are entirely deprecated.

Confirmed still occurring as of Dec 7, 2018.

Stale

Stale


Include compliance documentation and required data sources

Proposed changes
Include compliance documentation and auto-generated NIST 800-53 reference based on OpenControl-formatted compliance content at https://github.com/docker/compliance. Currently, the yaml data sources are manually copied to /_data/compliance/, but once a more formalized process for pulling in external content is established, this can be modified.
Related issues
Address #3206.

Please don't use docker as a keyword. :) Otherwise this LGTM but @johndmulhausen should review.

Ah, ok. No prob. Will update.

LGTM now, even with documentation in keywords... I'll let @johndmulhausen have the final say.

Improvements I think are worth making:

Remove "documentation" as a keyword
Instead of using includes, just use a layout; see for example how the files under /samples/ use a different layout that processes the information. It creates cleaner shell pages than having to define all these include statements. We should be doing that in the CLI docs but we aren't yet
These tabs aren't syncing with each other and they probably should: https://deploy-preview-3223--docker-docs.netlify.com/compliance/reference/800-53/cp/#cp-1-contingency-planning-policy-and-procedures
It seems like the opening salvo paragraphs are a bit jumbled. For example take this: https://deploy-preview-3223--docker-docs.netlify.com/compliance/reference/800-53/ia/#ia-1-identification-and-authentication-policy-and-procedures
... Is it really supposed to use what looks like "put the real information here" markers like [Assignment: organization-defined frequency]? Are these from a template that isn't filled out yet? Also, wouldn't this be better formatted as a list with proper indenting? There are both alpha and numeric list items whizzing by in an unbroken stream of text making a big word blob.

Thanks @johndmulhausen ... working on the cleanup now. The NIST 800-53 descriptions are currently being parsed from the included _data/compliance/nist80053.yml file, which is based on the OpenControl schemas defined here. This was mainly to simplify the parsing code against one schema instead of multiple schemas. However, given that the schema has yet to be finalized and to clean up the description layout, I'll go ahead and instead parse the descriptions based on the original NIST 800-53 Rev 4 xml definition here. This should give us better separation of paragraphs and lists.
As far as the [Assignment: organization-defined frequency] markers, these are actually part of the NIST 800-53 specification itself and whose contents are replaced by either the organization's specific requirement, or that of another standard that inherits the 800-53 controls (e.g. FedRAMP). I can clarify this in the index docs for the reference.

@anweiss Any news here?

Working through it. Backlogged at the moment.

Ping :)

Sorry, haven't had a chance to look at it. Just pushed some new commits, but running in to some issues with refactoring using layouts per @johndmulhausen's recommendation.

For some reason, I'm unable to pull any of the Liquid variables/data from within the nist80053.html layout that I created. It inherits from the docs layout. Also getting an Angular module injection error:
jquery.min.js:2 Uncaught Error: [$injector:modulerr] http://errors.angularjs.org/1.5.11/$injector/modulerr?p0=Docker&p1=Error%3A…%2Fdbepenphjfofmnjmlacfcdehikakmaap%2Fjs%2Flib%2Fangular.min.js%3A21%3A332)
    at angular.min.js:6
    at angular.min.js:40
    at q (angular.min.js:7)
    at g (angular.min.js:39)
    at fb (angular.min.js:43)
    at c (angular.min.js:21)
    at Gc (angular.min.js:21)
    at se (angular.min.js:20)
    at HTMLDocument.<anonymous> (angular.min.js:323)
    at j (jquery.min.js:2)

@johndmulhausen PTAL

Squashed in to single commit

How are you feeling about this PR, @anweiss ?

We've decided to externalize these docs in the compliance repo -> https://github.com/docker/compliance ... will create a new PR that will support fetching these, in a manner similar to that which the CLI docs are pulled in


[WIP]Release 0.17.2

重构操作入口，https://github.com/ice-lab/iceworks/issues/721#issuecomment-774904161 列表中有这些入口的说明 https://github.com/ice-lab/iceworks/pull/776
监听创建文件夹和文件操作，如果在特定的文件路径内则自动补全 React/Rax Component 代码片段

Codecov Report

Merging #777 (52a0cda) into master (0f0f948) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #777   +/-   ##
=======================================
  Coverage   57.14%   57.14%
=======================================
  Files           1        1
  Lines          70       70
  Branches       18       15    -3
=======================================
  Hits           40       40
  Misses         30       30

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 0f0f948...52a0cda. Read the comment docs.


Windows port

is there any windows port or anyway to build this on windows ?

Windows port is in development and will be available eventually.

i somehow got able to compile it without the support for ffmpeg and statically (no hotreload since dll are 🐛)

Windows? whats that? Never heard of it

I managed to build on Windows. If you want to know how, the steps are easy.

Profit!

Also, a side note, inside the MSYS MingW64 window, Ctrl+Ins = copy and Shift+Ins = paste

Windows build is available at https://github.com/tsoding/musializer/releases/tag/alpha1


Add FUTURE Grants to the list of Grants extracted and documented

The tool should add output SHOW FUTURE GRANTS ON command to the reports

Fixed in https://github.com/Snowflake-Labs/sfgrantreport/releases/tag/2<IP_ADDRESS> release


Delta to Axis (inverted y-axis)

Is there a way to invert y-axis?

Use negative values on sensitive (multiplier)


sayonara:  init at 1.0.0-git5-20180115

Motivation for this change
Fix #34024
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build sayonara

Failure on x86_64-darwin (full log)
Partial log (click to expand)
  No known features for CXX compiler

  "Clang"

  version 5.0.1.

-- Build files have been written to: /tmp/nix-build-sayonara-player-1.0.0-git5-20180115.drv-0/sayonara-player/build
builder for '/nix/store/bk9brppzyhlk8345b664lyg58rc19nc9-sayonara-player-1.0.0-git5-20180115.drv' failed with exit code 1
error: build of '/nix/store/bk9brppzyhlk8345b664lyg58rc19nc9-sayonara-player-1.0.0-git5-20180115.drv' failed

Success on x86_64-linux (full log)
Partial log (click to expand)
shrinking /nix/store/qw3d5hs2i5v9qzilxxwvir2x6wap0j9b-sayonara-player-1.0.0-git5-20180115/bin/sayonara
shrinking /nix/store/qw3d5hs2i5v9qzilxxwvir2x6wap0j9b-sayonara-player-1.0.0-git5-20180115/lib/sayonara/libsayonara_somafm.so
shrinking /nix/store/qw3d5hs2i5v9qzilxxwvir2x6wap0j9b-sayonara-player-1.0.0-git5-20180115/lib/sayonara/libsayonara_soundcloud.so
gzipping man pages under /nix/store/qw3d5hs2i5v9qzilxxwvir2x6wap0j9b-sayonara-player-1.0.0-git5-20180115/share/man/
strip is /nix/store/adidfx4pa7vmvby0gjqqmiwg2x49yr27-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/qw3d5hs2i5v9qzilxxwvir2x6wap0j9b-sayonara-player-1.0.0-git5-20180115/lib  /nix/store/qw3d5hs2i5v9qzilxxwvir2x6wap0j9b-sayonara-player-1.0.0-git5-20180115/bin
patching script interpreter paths in /nix/store/qw3d5hs2i5v9qzilxxwvir2x6wap0j9b-sayonara-player-1.0.0-git5-20180115
checking for references to /tmp/nix-build-sayonara-player-1.0.0-git5-20180115.drv-0 in /nix/store/qw3d5hs2i5v9qzilxxwvir2x6wap0j9b-sayonara-player-1.0.0-git5-20180115...
postPatchMkspecs
/nix/store/qw3d5hs2i5v9qzilxxwvir2x6wap0j9b-sayonara-player-1.0.0-git5-20180115

Success on aarch64-linux (full log)
Partial log (click to expand)
shrinking /nix/store/mb53xs3frahjq56wh7nkrqib76b2569v-sayonara-player-1.0.0-git5-20180115/lib/sayonara/libsayonara_somafm.so
shrinking /nix/store/mb53xs3frahjq56wh7nkrqib76b2569v-sayonara-player-1.0.0-git5-20180115/lib/sayonara/libsayonara_soundcloud.so
shrinking /nix/store/mb53xs3frahjq56wh7nkrqib76b2569v-sayonara-player-1.0.0-git5-20180115/bin/sayonara
gzipping man pages under /nix/store/mb53xs3frahjq56wh7nkrqib76b2569v-sayonara-player-1.0.0-git5-20180115/share/man/
strip is /nix/store/skd6ix5ipkyhxzq7naylj4digawakl4j-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/mb53xs3frahjq56wh7nkrqib76b2569v-sayonara-player-1.0.0-git5-20180115/lib  /nix/store/mb53xs3frahjq56wh7nkrqib76b2569v-sayonara-player-1.0.0-git5-20180115/bin
patching script interpreter paths in /nix/store/mb53xs3frahjq56wh7nkrqib76b2569v-sayonara-player-1.0.0-git5-20180115
checking for references to /build in /nix/store/mb53xs3frahjq56wh7nkrqib76b2569v-sayonara-player-1.0.0-git5-20180115...
postPatchMkspecs
/nix/store/mb53xs3frahjq56wh7nkrqib76b2569v-sayonara-player-1.0.0-git5-20180115

@joachifm, all fixed now, I believe.

@GrahamcOfBorg build sayonara

Success on x86_64-linux (full log)
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/abaxv089731bqyjzp1vppcm8k59gni92-sayonara-player-1.0.0-git5-20180115
shrinking /nix/store/abaxv089731bqyjzp1vppcm8k59gni92-sayonara-player-1.0.0-git5-20180115/bin/sayonara
shrinking /nix/store/abaxv089731bqyjzp1vppcm8k59gni92-sayonara-player-1.0.0-git5-20180115/lib/sayonara/libsayonara_soundcloud.so
shrinking /nix/store/abaxv089731bqyjzp1vppcm8k59gni92-sayonara-player-1.0.0-git5-20180115/lib/sayonara/libsayonara_somafm.so
gzipping man pages under /nix/store/abaxv089731bqyjzp1vppcm8k59gni92-sayonara-player-1.0.0-git5-20180115/share/man/
strip is /nix/store/adidfx4pa7vmvby0gjqqmiwg2x49yr27-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/abaxv089731bqyjzp1vppcm8k59gni92-sayonara-player-1.0.0-git5-20180115/lib  /nix/store/abaxv089731bqyjzp1vppcm8k59gni92-sayonara-player-1.0.0-git5-20180115/bin
patching script interpreter paths in /nix/store/abaxv089731bqyjzp1vppcm8k59gni92-sayonara-player-1.0.0-git5-20180115
checking for references to /build in /nix/store/abaxv089731bqyjzp1vppcm8k59gni92-sayonara-player-1.0.0-git5-20180115...
postPatchMkspecs

Failure on x86_64-darwin (full log)
Partial log (click to expand)
  No known features for CXX compiler

  "Clang"

  version 5.0.1.

-- Build files have been written to: /tmp/nix-build-sayonara-player-1.0.0-git5-20180115.drv-0/sayonara-player/build
builder for '/nix/store/nggy3hgc36b1gd8vw5rgf8maykhcmh6h-sayonara-player-1.0.0-git5-20180115.drv' failed with exit code 1
error: build of '/nix/store/nggy3hgc36b1gd8vw5rgf8maykhcmh6h-sayonara-player-1.0.0-git5-20180115.drv' failed

Success on aarch64-linux (full log)
Partial log (click to expand)
shrinking /nix/store/hh4crnm5sc2y20ww9071ipcqrs3j6pyp-sayonara-player-1.0.0-git5-20180115/lib/sayonara/libsayonara_somafm.so
shrinking /nix/store/hh4crnm5sc2y20ww9071ipcqrs3j6pyp-sayonara-player-1.0.0-git5-20180115/lib/sayonara/libsayonara_soundcloud.so
shrinking /nix/store/hh4crnm5sc2y20ww9071ipcqrs3j6pyp-sayonara-player-1.0.0-git5-20180115/bin/sayonara
gzipping man pages under /nix/store/hh4crnm5sc2y20ww9071ipcqrs3j6pyp-sayonara-player-1.0.0-git5-20180115/share/man/
strip is /nix/store/skd6ix5ipkyhxzq7naylj4digawakl4j-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/hh4crnm5sc2y20ww9071ipcqrs3j6pyp-sayonara-player-1.0.0-git5-20180115/lib  /nix/store/hh4crnm5sc2y20ww9071ipcqrs3j6pyp-sayonara-player-1.0.0-git5-20180115/bin
patching script interpreter paths in /nix/store/hh4crnm5sc2y20ww9071ipcqrs3j6pyp-sayonara-player-1.0.0-git5-20180115
checking for references to /build in /nix/store/hh4crnm5sc2y20ww9071ipcqrs3j6pyp-sayonara-player-1.0.0-git5-20180115...
postPatchMkspecs
/nix/store/hh4crnm5sc2y20ww9071ipcqrs3j6pyp-sayonara-player-1.0.0-git5-20180115

@joachifm, do you feel there are any other blockers remaining?


Evaluate with multi GPU and mini batch

Hi all,
I found that this code still does not implement evaluation on multi GPU, I have tried the same manner as in train.py, but meet infinite recursioon problem during forwad propagation with data_parallel.py.
Hope somebody could help, or will the evaluation using multi GPU will be supported?

Okay, I'm looking forward for that!

Hi,
Please check out the latest version of this repo. You'll find what you want :)


Update README.md

Thanks for the merge! Here's an update to the readme to really get this over the finish line :)

Hey this is super cool. I dislike having my external dependencies directly checked into my repo and I also dislike git submodules. I use Cmake's FetchContent system that was introduced a couple of versions ago for dealing with this in (IMO) a very elegant way. So I thought I'd share that CMake file with others here.
My only complain would be I wish @zeux would maybe tag versions so I didn't have to pull master ;)
Hope this is useful to someone.
message(STATUS "\nVOLK configuration:")

# NOTE: volk setup below will transitively have a dependency
#   on and link against Vulkan::Vulkan. By linking against
#   volk library there is no need to also link against the
#   Vulkan::Vulkan library.

FetchContent_Declare(volk
    GIT_REPOSITORY https://github.com/zeux/volk.git
    GIT_TAG origin/master
    GIT_SHALLOW ON
    GIT_PROGRESS ON
)

FetchContent_GetProperties(volk)
if (NOT volk_POPULATED)
    FetchContent_Populate(volk)

    if (CMAKE_SYSTEM_NAME STREQUAL Windows)
        set(VOLK_STATIC_DEFINES VK_USE_PLATFORM_WIN32_KHR)
    elseif (CMAKE_SYSTEM_NAME STREQUAL Darwin)
        set(VOLK_STATIC_DEFINES VK_USE_PLATFORM_MACOS_MVK)
    elseif (CMAKE_SYSTEM_NAME STREQUAL Linux)
        set(VOLK_STATIC_DEFINES VK_USE_PLATFORM_XLIB_KHR)
    endif()

    set(VOLK_PULL_IN_VULKAN ON CACHE BOOL "" FORCE)
    add_subdirectory(${volk_SOURCE_DIR} ${volk_BINARY_DIR} EXCLUDE_FROM_ALL)
endif()

I should probably start tagging versions although the Vulkan SDK updates so frequently that doing so is tedious :(


Mistake on file navigation.yml

Previous wrong name was navigations.yml

Thanks for the typo fix.


Support different colors

At the moment, the background always seems to be white (unless I m missing something).
It would be nice, if the color could be either set via a parameter or in the plugin configuration (similar to admonition).

In case this is still relevant for someone:
I am using a CSS snippet ("Options" → "Appearance" → "CSS snippets"). Simply put a .css file with the following content into your snippets folder:
.auto-card-link-container {
    border-color: var(--background-modifier-border);
}
.auto-card-link-container:hover {
    border-color: var(--background-modifier-border-hover);
}

.auto-card-link-error-container {
    background-color: var(--background-modifier-error);
}
.auto-card-link-error-container:hover {
    background-color: var(--background-modifier-error-hover);
}

.auto-card-link-card {
    color: var(--text-normal);
    background-color: var(--background-secondary);
}

.auto-card-link-description {
    color: var(--text-muted);
}

This should ensure that card links work in basically all light and dark themes. However, which exact variables you use is up to personal preference (e.g., --background-secondary vs. --background-primary-alt).

Gonna share mine here as well:
.auto-card-link-container {
    border-width: 2px;
    border-color: var(--text-faint);
    transition: border-color .3s;
}

.auto-card-link-container:hover {
    border-color: var(--text-accent);
}

.auto-card-link-error-container {
    background: var(--background-error);
}

.auto-card-link-card {
    background: var(--background-secondary);
    color: var(--text-normal);
}

.auto-card-link-card:hover {
    text-decoration: none;
}

.auto-card-link-main {
    padding: 20px 10px;
}

.auto-card-link-description {
    color: var(--text-muted);
}

.auto-card-link-host {
    color: var(--text-faint);
}

You may find all available variables here and the original stylesheet here

When used inside a canvas, I would expect the card's background and accent colors to match that of the canvas node's theme. Instead it uses the non-themed default background and accent:

By default the card uses --background-primary-alt as background, which is set by the active theme and usually is a solid color. What you describe could be done by adding some transparency to the background.
I've noticed that in a canvas card the colors of text, code blocks, and other text elements don't change when setting a theme, so I don't know if the cardlink should.
Try with this snippet
.auto-card-link-card {
  background: rgba(0, 0, 0, 0.05);
}

Thanks! I got want I wanted with the following css:
.auto-card-link-card {
    background: rgba(0, 0, 0, 0.2);
    .auto-card-link-host,
    .auto-card-link-title {
        color: rgb(var(--canvas-color));
    }
    .auto-card-link-thumbnail {
        width: 33% !important;
        min-width: 33% !important;
        max-width: 33% !important;
    }
}


WIP: Add initial rust module

Here is the first attempt. Currently it just imports an empty module. Basically, it's to help up ensure that the module builds well on all our workstations, CI, and packages well.
Questions: how to put the module into PYTHONPATH of the build?

In current commit, you need PYTHONPATH=build/rustlib to make it work
It looks like cython modules are just put into sources. So should I just copy the module somewhere to edb/edgeql/_rust_lexer.cpython-bla-bla.so ?

Notes to myself:

Update submodule url to public URL before merging in

@tailhook Is it possible to use https://pypi.org/project/setuptools-rust/ to integrate this properly?

Also: https://github.com/getsentry/milksnake

Add setuptools-rust. Milksnake is for cffi bindings (I think we'd be better with normal cpython bindings they should be faster and we don't have to support multiple pythons).


[CTIS] Change V17 name with new format

Description
V17 output field names were changed with the new format. Update microdata output names (V17 is not used anywhere else at the moment).
Changelog

responses.R

@krivard This is ready to merge and deploy.


What improvements are needed to faciliate a developer using the ticketing repo as a boilerplate app

We have a lot of disorganized "copy pasta" and commented out code that would be good to clean up. Even just a day or two on code quality improvements would benefit us and other devs using this template.

We might also want to refactor and cleanup the Stripe connection logic when creating a new event. I've noticed some weird behavior that we can probably fix while cleaning things up too. I'll receive toast messages related to Stripe when I don't expect it.

After looking through the utils folder, I saw a lot of variables which depends on selected network. For example lines like this
export const NETWORK_NODE_URL =
  NETWORK_ID === 'mainnet' ? 'https://rpc.mainnet.near.org' : 'https://rpc.testnet.near.org';

happen quite often.
Proposal 1
Why don't we split env file into mainnet.env and testnet.env and put there all related variables?
Pros:

clean and understandable variables;
no need to duplicate if ... else ... structure depending on network;
all environment variables are configurable;

Cons:

a bit complicate configuration (I mean two .env files instead of one);
maybe something else?

Also, worth to mention that along with adding two .env files requires to extend our scripts by adding this:
// package.json file
"scripts": {
    //... here comes existed scripts
    "dev": "env-cmd -f ./testnet.env next dev",
    "dev:mainnet": "env-cmd -f ./mainnet.env next dev",
    //... other scripts
}

We can archive this using env-cmd module. WDYT @calebjacob @charleslavon @jackson-harris-iii ?

Why don't we split env file into mainnet.env and testnet.env and put there all related variables?

@shelegdmitriy It's a good question. I see pros and cons both ways. If we wanted to move towards all of these values being configured in the env file, I'd vote for only having a single .env file. We could do something like this in our example file where you would copy to your .env file and uncomment/comment the block of values you're wanting to reference for mainnet vs testnet:
# NEXT_PUBLIC_NETWORK_ID=mainnet
# NEXT_PUBLIC_HOSTNAME_KEYPOM_EVENTS_CONTRACT_ID=ticketing-v1.keypom.near
# NEXT_PUBLIC_HOSTNAME_KEYPOM_MARKETPLACE_CONTRACT_ID=marketplace-v1.keypom.near
...

NEXT_PUBLIC_NETWORK_ID=testnet
NEXT_PUBLIC_HOSTNAME_KEYPOM_EVENTS_CONTRACT_ID=1717013187496-kp-ticketing.testnet
NEXT_PUBLIC_HOSTNAME_KEYPOM_MARKETPLACE_CONTRACT_ID=1717013187496-marketplace.testnet
...

utils folder contains a lot of helper functions and I believe it's worth to add jest tests for it.

@shelegdmitriy Also a good suggestion. I'd vote for reorganizing/refactoring our utils folder and then add tests. For example, we have a lot of copy/paste code that was grabbed from another project (some of it is commented out or maybe not used at all). We also have filenames like utils/common and utils/helpers which can be pretty confusing and not very descriptive.

I'd vote for reorganizing/refactoring our utils folder and then add tests. For example, we have a lot of copy/paste code that was grabbed from another project (some of it is commented out or maybe not used at all). We also have filenames like utils/common and utils/helpers which can be pretty confusing and not very descriptive.

@calebjacob this is exactly what I'm trying to accomplish. Of course, I wasn't going to write tests until I got these files in a good shape 🙂

As per our discussion with @charleslavon, one of the options is to create a page or at least add a mention about ticketing app on docs.near.org somewhere under Web3 Applications? Pros of this would be an ability to find this repo using search not only on docs website but on dev.near.org also.
Along with this, seems like we did pretty good job by reorganizing utils folder and it seems like a good time to cover utils with tests.

@calebjacob @jackson-harris-iii I wonder do you guys have some ideas how we can improve our repo even more?


chore: install tests print output on failure

Previously the output of subprocesses would only be printed if
DEBUG was set to true in the test-install.ts file.  This
is not helpful for tests that fail on CI.
Now if tests pass then no process output is printed to the
console.  This makes it easy to see that the tests pass.
However, if a test fails, the output of processes invoked in that
test are automatically printed to the console to help determine
the cause of the error.

PTAL

Note: This PR contains some of the code that was in PR #74 since the code should be in its own PR.  As such, PR #74 should not land until this PR lands.


email one time passcode enabled by default

@msmimart,
Can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Docs Build status updates of commit 98f88b1:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/active-directory/fundamentals/whats-new-archive.md
:warning:Warning

Details

articles/active-directory/external-identities/one-time-passcode.md
:white_check_mark:Succeeded

articles/active-directory/fundamentals/whats-new-archive.md

Line 2039, Column 364: [Warning: bookmark-not-found] Cannot find bookmark '#note-for-azure-us-government-customers' in 'articles/active-directory/external-identities/one-time-passcode.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Change made via private repo.
#please-close


Feature Request: optionally return 404 when protected files exist but auth fails

When accessing a protected file with an invalid token, the API returns a 403:
❯ curl http://localhost:8090/api/files/11h2ewlken3gnc0/de0t70pp59xyfz7/my-private-file.doc?token=foobar
{"code":403,"message":"Insufficient permissions to access the file resource.","data":{}}

This currently makes clear the resource exists, making fingerprinting possible.
Presumably changing this to a 404 by default is not an option for compatibility or other reasons, but it'd be nice if returning a 404 could be enabled optionally.

I've just changed it in the refactored version since I'm currently working on the tests for the new file APIs. It will be available with the next v0.23.0 release (there are no ETAs yet) but even as it is file fingerprinting is very difficult because by default all uploaded files have a random part.

I've just changed it in the refactored version since I'm currently working on the tests for the new file APIs

Superb 💟

but even as it is file discovery is very difficult because by default all uploaded filenames have a random part (and you need to know from somewhere the targeted record id in the first place).

👍 agreed.
Taking the chance to send my warmest thanks for such an awesome, simple, pain-free and solid software.


[DNM] initial commit for mobile app

Initial commit. I'll start following the commit message template once we actually do something.

Also, the app/.DS_Store file shouldn't be committed and be purged? Looks like it might have been committed after you added that pattern in gitignore

Side note, can this be formatted withprettier with the same rules as the web app?

Merging this :)


@orta => [Internal WebView] Don’t show stuck progress indicators or broken pages.

Fixes #738.

This is definitely going to need more beta testing, me thinks.

Gonna do some merging on my own, as I want to do a release.


Structure of the model

Time to do: 1 hour
According to the structure of the model, everything looks as if the agent is part of the user, and not a dedicated middleware for users and ML models.

Added part of the business process diagram to Google drive

https://drive.google.com/file/d/1cJOEaBVuPXZISKOlaLOA4ey2iONKZvJv/view?usp=sharing


fabric8 and k8s discovery treat instanceId (when not present in the underlying service) in a different manner

When discovering services, we return a List<ServiceInstance>. That ServiceInstance has a field called: instanceId.
In both clients, we compute that field from:
V1Endpoints -> V1EndpointSubset -> V1EndpointAddress -> V1ObjectReference -> getUuid

Both fabric8 and k8s treat this the same way, except when this Uuid is not present.

fabric8 sets this instanceId to null
k8s sets it to an empty String : ""

Since we started to delegate to the same code in commons, this is how it was observed.

In this particular case, I don't see a big issue here, imo: they should both set it to null, denoting that it is not present. wdyt?

Agree


tool/app permission bug

A tool/app associated with a res should display on that res' landing page only if the tool/app is public or the current user owns this tool.

There are also in between cases.  If a tool is not public, but has been shared with a user or a group the user is part of then that tool should be accessible to the user.  Basically access to tools should follow the same rules as access to other resources in terms of sharing.

closed.


Fix: Verify s3 object exists or not

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: bkhadars
To complete the pull request process, please assign mkumatag after the PR has been reviewed.
You can assign the PR to them by writing /assign @mkumatag in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

and the description with the background and the reason we are doing this as well.

/assign @bkhadars

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: bkhadars, mkumatag
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [mkumatag]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Commenting out a cell causes page to break

I have an element in the page like this:

Calls remaining

    ${[...data].filter(d => mutual_aid ? true : d.In_town).length.toLocaleString("en-US")}

but when I comment this out in markdown, like

I get an error in the console after I refresh the page (it doesn't break on live reload). Then all elements lower down on the page don't work. Error looks like:
client.js:133 Uncaught TypeError: Cannot read properties of null (reading 'classList')
    at define (client.js:133:24)
    at dashboard:183:1

and yes, you can see that the line in question from client.js
  const loading = root.classList.contains("observablehq--loading");

will get an error because the element id is not selected previously by document.querySelector because it's commented out from the html.
Seems like the issue is that the JS is sill picking up this element, and isn't respecting the html comment in the markdown the same way that the browser it (which doesn't render it).
Also seems pretty easy to reproduce. I'm loving framework and this tripped me up for longer than I'd like to admit!

yes. but I think it's a duplicate of #375

@Fil thanks. I had tried searching but that only has pictures, not the text, of the error message so that's why I didn't find it 😂
this issue should help point searchers back over there!


Helm readme file generator

Description
Workflow to generate documentation README file from the values file of DD
Please see example
Test results
Please see the result https://github.com/dsever/django-DefectDojo/blob/helm_readme_file/helm/defectdojo/README.md
@alles-klar   @Maffooch do you see this as useful, I've borrowed it from  the bitnami labs looks useful to me to standardize helm

Seems very useful to me! I have done a helm install before, but I imagine this would make the process much easier for newcomers

@dsever great one! I think it is very useful.

Closing due to stale. Please open again if you'd like to pick this one up again :)


Add mikkkole's file

Add mikkkole's filebbb

Add mikkkole's file

Add mikkkole's file

Add mikkkole's file


Query won't work

I build container.
After start, auth is succes.
I try get QUERY OCI, but won't got compartments etc.
But in the log i see it is success.

 INFO: cert_bundle=None INFO: submitted 1 list resources requests INFO: processed 1 list resources responses INFO: found 15 active compartments INFO: >>>>> Queried Regions [{'is_home_region': True, 'region_key': 'FRA', 'region_name': 'eu-frankfurt-1', 'status': 'READY'}] INFO: >>>>>>>>> Regions: [   {     "display_name": "EU Frankfurt",     "id": "eu-frankfurt-1",     "is_home_region": true,     "key": "FRA",     "name": "eu-frankfurt-1",     "region_key": "FRA",     "region_name": "eu-frankfurt-1",     "status": "READY"   } ]
Later i got error in log:
No dns record found for https://identity.retrieving.oci.oraclecloud.com
`
INFO: >>>>> Queried Regions [{'is_home_region': True, 'region_key': 'FRA', 'region_name': 'eu-frankfurt-1', 'status': 'READY'}]
INFO: >>>>>>>>> Regions: [
{
"display_name": "EU Frankfurt",
"id": "eu-frankfurt-1",
"is_home_region": true,
"key": "FRA",
"name": "eu-frankfurt-1",
"region_key": "FRA",
"region_name": "eu-frankfurt-1",
"status": "READY"
}
]
INFO: Using Profile : DEFAULT
INFO: OCIConnection: Passed Config <class 'dict'> {'region': 'Retrieving'}
INFO: OCIConnection: Passed Config <class 'dict'> dict_keys(['region']) {'region': 'Retrieving'}
INFO: OCI_CLI_AUTH = Undefined
INFO: Request : ['Retrieving']
INFO: Request : ['Retrieving']
INFO: Request : True
INFO: cert_bundle=None
[2023-08-28 07:01:31 +0000] [18] [ERROR] Socket error processing request.
Traceback (most recent call last):
File "/usr/local/lib/python3.8/site-packages/oci/_vendor/urllib3/connection.py", line 179, in _new_conn
conn = connection.create_connection(
File "/usr/local/lib/python3.8/site-packages/oci/_vendor/urllib3/util/connection.py", line 77, in create_connection
for res in socket.getaddrinfo(host, port, family, socket.SOCK_STREAM):
File "/usr/lib64/python3.8/socket.py", line 918, in getaddrinfo
for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
socket.gaierror: [Errno -2] Name or service not known
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.8/site-packages/oci/_vendor/urllib3/connectionpool.py", line 708, in urlopen
httplib_response = self._make_request(
File "/usr/local/lib/python3.8/site-packages/oci/_vendor/urllib3/connectionpool.py", line 391, in _make_request
self._validate_conn(conn)
File "/usr/local/lib/python3.8/site-packages/oci/_vendor/urllib3/connectionpool.py", line 1045, in _validate_conn
conn.connect()
File "/usr/local/lib/python3.8/site-packages/oci/_vendor/urllib3/connection.py", line 363, in connect
self.sock = conn = self._new_conn()
File "/usr/local/lib/python3.8/site-packages/oci/_vendor/urllib3/connection.py", line 191, in _new_conn
raise NewConnectionError(
oci._vendor.urllib3.exceptions.NewConnectionError: <oci.base_client.OCIConnection object at 0x7f7199af4700>: Failed to establish a new connection: [Errno -2] Name or service not known
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.8/site-packages/oci/_vendor/requests/adapters.py", line 447, in send
resp = conn.urlopen(
File "/usr/local/lib/python3.8/site-packages/oci/_vendor/urllib3/connectionpool.py", line 790, in urlopen
retries = retries.increment(
File "/usr/local/lib/python3.8/site-packages/oci/_vendor/urllib3/util/retry.py", line 597, in increment
raise MaxRetryError(_pool, url, error or ResponseError(cause))
oci._vendor.urllib3.exceptions.MaxRetryError: OCIConnectionPool(host='identity.retrieving.oci.oraclecloud.com', port=443): Max retries exceeded with url: /20160918/tenancies/ocid1.tenancy.oc1..****(Caused by NewConnectionError('<oci.base_client.OCIConnection object at 0x7f7199af4700>: Failed to establish a new connection: [Errno -2] Name or service not known'))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.8/site-packages/oci/base_client.py", line 669, in request
response = self.session.request(
File "/usr/local/lib/python3.8/site-packages/oci/_vendor/requests/sessions.py", line 536, in request
resp = self.send(prep, **send_kwargs)
File "/usr/local/lib/python3.8/site-packages/oci/_vendor/requests/sessions.py", line 652, in send
r = adapter.send(request, **kwargs)
File "/usr/local/lib/python3.8/site-packages/oci/_vendor/requests/adapters.py", line 527, in send
raise ConnectionError(e, request=request)
oci._vendor.requests.exceptions.ConnectionError: (MaxRetryError("OCIConnectionPool(host='identity.retrieving.oci.oraclecloud.com', port=443): Max retries exceeded with url: /20160918/tenancies/ocid1.tenancy.oc1..*******(Caused by NewConnectionError('<oci.base_client.OCIConnection object at 0x7f7199af4700>: Failed to establish a new connection: [Errno -2] Name or service not known'))"), 'Request Endpoint: GET https://identity.retrieving.oci.oraclecloud.com/20160918/tenancies/ocid1.tenancy.oc1..******See https://docs.oracle.com/en-us/iaas/Content/API/Concepts/sdk_troubleshooting.htm for help troubleshooting this error, or contact support and provide this full error message.')
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.8/site-packages/gunicorn/workers/sync.py", line 135, in handle
self.handle_request(listener, req, client, addr)
File "/usr/local/lib/python3.8/site-packages/gunicorn/workers/sync.py", line 193, in handle_request
util.reraise(sys.exc_info())
File "/usr/local/lib/python3.8/site-packages/gunicorn/util.py", line 641, in reraise
raise value
File "/usr/local/lib/python3.8/site-packages/gunicorn/workers/sync.py", line 178, in handle_request
respiter = self.wsgi(environ, resp.start_response)
File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 2213, in call
return self.wsgi_app(environ, start_response)
File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 2193, in wsgi_app
response = self.handle_exception(e)
File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 2190, in wsgi_app
response = self.full_dispatch_request()
File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1486, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1484, in full_dispatch_request
rv = self.dispatch_request()
File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1469, in dispatch_request
return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)
File "/github/oci-designer-toolkit/okitweb/okitOci.py", line 299, in ociQuery
response = query.executeQuery(regions=[regions] if regions else None, compartments=[compartments] if compartments else None, include_sub_compartments=sub_compartments)
File "/okit/visualiser/query/ociQuery.py", line 209, in executeQuery
discovery_client = OciResourceDiscoveryClient(self.config, signer=self.signer, cert_bundle=self.cert_bundle, regions=regions, include_resource_types=self.SUPPORTED_RESOURCES, compartments=compartments, include_sub_compartments=include_sub_compartments)
File "/okit/visualiser/discovery/oci_discovery_client.py", line 695, in init
self.tenancy = self.get_tenancy(tenancy_override)
File "/okit/visualiser/discovery/oci_discovery_client.py", line 818, in get_tenancy
tenancy = identity.get_tenancy(self.config["tenancy"]).data
File "/usr/local/lib/python3.8/site-packages/oci/identity/identity_client.py", line 8192, in get_tenancy
return retry_strategy.make_retrying_call(
File "/usr/local/lib/python3.8/site-packages/oci/retry/retry.py", line 308, in make_retrying_call
response = func_ref(func_args, **func_kwargs)
File "/usr/local/lib/python3.8/site-packages/oci/base_client.py", line 531, in call_api
response = self.request(request, allow_control_chars, operation_name, api_reference_link)
File "/usr/local/lib/python3.8/site-packages/circuitbreaker.py", line 159, in wrapper
return call(function, args, **kwargs)
File "/usr/local/lib/python3.8/site-packages/circuitbreaker.py", line 170, in call
return func(args, **kwargs)
File "/usr/local/lib/python3.8/site-packages/oci/base_client.py", line 692, in request
raise exceptions.RequestException(e)
oci.exceptions.RequestException: (MaxRetryError("OCIConnectionPool(host='identity.retrieving.oci.oraclecloud.com', port=443): Max retries exceeded with url: /20160918/tenancies/ocid1.tenancy.oc1..(Caused by NewConnectionError('<oci.base_client.OCIConnection object at 0x7f7199af4700>: Failed to establish a new connection: [Errno -2] Name or service not known'))"), 'Request Endpoint: GET https://identity.retrieving.oci.oraclecloud.com/20160918/tenancies/ocid1.tenancy.oc1..****See https://docs.oracle.com/en-us/iaas/Content/API/Concepts/sdk_troubleshooting.htm for help troubleshooting this error, or contact support and provide this full error message.')` #

@nyirimark can you try the 0.54.3 release

Thanks @toxophilist !
Look like it is working now!  - Solved
Just little issue found at launch, in right top corner is now retriving always

@nyirimark try the 0.54.4 release

@nyirimark try the 0.54.4 release

Working ;)


Accommodate Jasmine's done.fail(e)

Should be able to call done.fail(e) if function fail exists, in the case of Jasmine.
Related to #16 and #12

I am refactoring the library to make it framework agnostic, you can follow the discussion here
https://github.com/arnaudbenard/redux-mock-store/issues/22
I won't merge new code until I release the new version.
Thank you for the contribution. :+1:


[gettext] build failure

GetText build is failing at the start of the building phase with an autoconf syntax error (on AC_SUBST).
autoconf, automake, autoconf-archive, libtool and m4 have been installed with homebrew.
I don't know if I am missing any other tool.
Host Environment

OS: macOS 11.7.4
Compiler: Clang 13.0

To Reproduce
Steps to reproduce the behavior:
./vcpkg install gettext
Failure logs

Building gettext[core]:x64-osx...
-- Using cached gettext-0.21.1.tar.gz.
-- Extracting source /Volumes/ExtremeSSD/actions-runner/_work/altitude/altitude/vcpkg/downloads/gettext-0.21.1.tar.gz
-- Applying patch android.patch
-- Applying patch uwp.patch
-- Applying patch win-gethostname.patch
-- Applying patch 0003-Fix-win-unicode-paths.patch
-- Applying patch rel_path.patch
-- Using source at /Volumes/ExtremeSSD/actions-runner/_work/altitude/altitude/vcpkg/buildtrees/gettext/src/gettext-0-de39ca5451.clean
-- Found external ninja('1.11.1').
-- Getting CMake variables for x64-osx-dbg
-- Getting CMake variables for x64-osx-rel
-- Configuring x64-osx-dbg
-- Configuring x64-osx-rel
-- Building x64-osx-dbg
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:134 (message):
Command failed: /usr/bin/make V=1 -j 5 -f /Volumes/ExtremeSSD/actions-runner/_work/altitude/altitude/vcpkg/ports/gettext/Makefile build-intl
Working Directory: /Volumes/ExtremeSSD/actions-runner/_work/altitude/altitude/vcpkg/buildtrees/gettext/x64-osx-dbg/
See logs for more information:
/Volumes/ExtremeSSD/actions-runner/_work/altitude/altitude/vcpkg/buildtrees/gettext/build-x64-osx-dbg-out.log
/Volumes/ExtremeSSD/actions-runner/_work/altitude/altitude/vcpkg/buildtrees/gettext/build-x64-osx-dbg-err.log
Call Stack (most recent call first):
scripts/cmake/vcpkg_build_make.cmake:149 (vcpkg_execute_build_process)
scripts/cmake/vcpkg_install_make.cmake:2 (vcpkg_build_make)
ports/gettext/portfile.cmake:85 (vcpkg_install_make)
ports/gettext/portfile.cmake:134 (build_libintl_only)

scripts/ports.cmake:147 (include)

build-x64-osx-dbg-err.log
build-x64-osx-dbg-out.log
config-x64-osx-dbg-out.log

Any other package managers installed? Can you disclose PATH?

No. Here's the PATH :

/usr/local/opt/bison/bin:/usr/local/bin:/usr/local/sbin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/share/dotnet:~/.dotnet/tools:/Library/Apple/usr/bin:/Users/IVAO/.local/bin

Hm, I'm surprised that it re-run confiure during build. This isn't normal.
/Library/Developer/CommandLineTools/usr/bin/make -C intl all
cd .. && /Library/Developer/CommandLineTools/usr/bin/make  am--refresh
CDPATH="${ZSH_VERSION+.}:" && cd ./../src/gettext-0-de39ca5451.clean/gettext-runtime && /bin/sh '/Volumes/ExtremeSSD/actions-runner/_work/altitude/altitude/vcpkg/buildtrees/gettext/src/gettext-0-de39ca5451.clean/build-aux/missing' autoconf

And it somehow reads like it considers autoconf missing. Is this an ARM mac?

Nope, x86

Could you please check this file [VCPKG_ROOT]/buildtrees/gettext/src/gettext-0-de39ca5451.clean/Makefile.am is exist?
automake-1.16: error: no 'Makefile.am' found for any configure output
automake-1.16: Did you forget AC_CONFIG_FILES([Makefile]) in configure.ac?

Yes, it exists.

@Abs-nth You are facing a different issue.

I am getting the same error on Linux:
error: building gettext:x64-linux failed with: BUILD_FAILED
Has anyone ever managed to install COLMAP from source? It doesn't build on Windows or Linux or MacOS for me. 🤬🤬🤬🤬

And it somehow reads like it considers autoconf missing

Did you try to install autoconf from brew?
(It may appear unexpecte at this point, but many ports need it. This might explain why the problem doesn't appear e.g. in CI.)


Channel persistance error

When refreshing the browser window when I already had a channel, I got this error:
INFO mutiny-core/src/logging.rs:27
2023-05-04 17:56:33.541 INFO  [lightning::chain::chainmonitor:672] Persistence of new ChannelMonitor for channel 3b12cdc7b5568df5822bf9bb28636b696f4f3cc36c5e2a747bd1181a54108d87 completed
<EMAIL_ADDRESS>INFO mutiny-core/src/logging.rs:27
2023-05-04 17:56:33.544 INFO  [lightning::chain::chainmonitor:672] Persistence of new ChannelMonitor for channel 3b7f68ee28cd348756ae7cfa2d571af05faa325b7c527a8b6b61de80eb33cddc completed
<EMAIL_ADDRESS>INFO mutiny-core/src/logging.rs:27
2023-05-04 17:56:33.548 INFO  [lightning::chain::chainmonitor:672] Persistence of new ChannelMonitor for channel 59af7e2297e0df696a017a71ad4dc8a720f14f96ea0a44596997d3c4d91a5379 completed
<EMAIL_ADDRESS>ERROR mutiny-core/src/logging.rs:29
2023-05-04 17:56:33.549 ERROR [lightning::chain::chainmonitor:653] Failed to add new channel data: channel monitor for given outpoint is already present
<EMAIL_ADDRESS>ERROR mutiny-core/src/logging.rs:29
2023-05-04 17:56:33.550 ERROR [lightning::chain::chainmonitor:653] Failed to add new channel data: channel monitor for given outpoint is already present

Nothing seems wrong though, Just don't know why it tried to persist 3 times (2 of which produced the failure)

Haven't seen this and VSS should fix? Good to close?


Added powershell script for Windows

So that's the powershell script I mentioned here. I'm not sure how the command in the plugin.yaml has to be so that it uses cnr.sh or cnr.ps1 depending on the OS.
The powershell script was also requested here #14

I talked to upstream and there isn't anyway to pivot on which script depending on OS.
I did see on stackoverflow there's some nasty hacks to embed bash/cmd into one script using some hacks...
I'm not quite sure the best way to move forward here.

@jzelinskie we can just release two zip file, containing different plugin.yaml, 1 for linux* 1 for windows.

@jzelinskie @sbueringer , latest appr release will allow to directly use the cli without a shell-script.
on the plugin.yaml command will be appr helm .
And because there is no more shell/ps script required, we should probably merge both repo too:
appr and appr-helm-plugin.

@ant31 Sounds good and it's obivously the better solution.


Events when turning on and off the extension

In Follcker there is the possibility of leaving the extension active or not, and this means that when you open the following page it appears or not whether the users presented there follow you.
The problem is that when turning on the extension with the page open, the statuses are not loaded, despite the function that causes it to be called in the event, the same happens with the function that is called to clear the descriptions when the extension is turned off with the page already open.
The function loaded on the turn on/off is:

browser.runtime.onMessage.addListener(function(request) {
    removeElements();

    if (request.action === "on"){
        trackFollowers();
    }
});

All of this is generated in background.js

I already solved this!

I already solved this!


cargo-metadata doesn't track target-specific dependencies

While the target configuration for a dependency is included in cargo-metadata's output (e.g. cfg(windows)), it's not all that easy to filter down to the dependencies that are enabled for a given target. It'd be nice if cargo-metadata could optionally perform the target-specific filtering.

cargo-tree still needs this before it can stop using cargo-the-library.

I'm going to close, as I think this is now implemented via the --filter-platform flag (#7376).  Alternatively, the cargo-platform crate can be used to parse the expressions and compare against the output of rustc --print=cfg.


Error with jquery promise

I am trying to test following code. Basically server rejects this request since some parameter were not sent as required by api specification.
Api.createRecord use jquery.ajax under the hood and returns a jquery promise when called.
    describe("Creating test record with incorrect data", ()=> {
        const params      = {tId:"12323","col":"colValue"};
        it('Should not create new record', (done) => {
            const result = Api.createRecord(params);
            console.log(result);
            expect(result)
                .to.eventually.equal(null)
                .notify(done);
        });
    });

When I run this code I get following error.
   Error: done() invoked with non-Error: {"readyState":0,"status":0,"statusText":"SyntaxError"}
      at mightThrow (node_modules/jquery/dist/jquery.js:3583:29)
      at Window.process (node_modules/jquery/dist/jquery.js:3651:12)
      at Timeout.callback [as _onTimeout] (node_modules/jsdom/lib/jsdom/browser/Window.js:524:19)

I am not able to find out what is the problem. Whether there is some problem in my code or somewhere else.

It appears jQuery is rejecting the promise with a non-Error, instead of fulfilling it with null. So the test is failing as expected.

This is how I am sending ajax request.
    sendRequest(URL, dataToSend) {
        return $.ajax({
            type        : 'POST',
            url         : URL,
            crossDomain : true,
            data        : JSON.stringify(dataToSend),
            dataType    : 'json'
        }).fail((responseData) => {
            if (responseData.responseCode) {
                console.error(responseData.responseCode);
            }
        });
    },

is there some problem in above code that it handles failures instead of it getting thrown?

I'm not sure, sorry. I don't really have time to debug your codebase. Maybe someone else who is monitoring this issue tracker will, or perhaps you could turn to StackOverflow or your coworkers.

In mocha docs I found the solution.

In Mocha v3.0.0 and newer, returning a Promise and calling done() will result in an exception, as this is generally a mistake:

const assert = require('assert');

it('should complete this test', function (done) {
  return new Promise(function (resolve) {
    assert.ok(true);
    resolve();
  })
    .then(done);
});

I changed my test case to this one and it worked.
           expect(result).to.be.rejected;
I am really sorry to bother you all the problem was actually at my end only.


Revisit state tracking in the parser vs scanner

For a better distribution of responsibilities, the state tracker needs some refactoring.
Token lookahead is necessary to determine the proper grammar production. This needs to exist in the parser as the scanner has no business regarding the production. The same goes for the extraneous location tracking for syntax node's location information.
The implementation of the reader algorithm for the pure tokenizer (i.e. tokenize API) shall not be in the parser as well. One day we will separate the tokenizer and find a new home for it, but for now it is sufficient to include it as part of the scanner.

Wonder if we need to think about how this effects template token production. } is ambiguous without a object/template stack.

Wonder if we need to think about how this effects template token production

There should not be any conflict, template is handled solely in the scanner.


Allow to pass supplementary arguments and relax device constraints

Add the dpdk_supl_args argument to pass arguments such as ["--no-huge","--no-pci"] allowing to try Retina in restricted environment, i.e. with Docker using PCAP virtual device (https://doc.dpdk.org/guides/nics/pcap_ring.html).
Also, if RSS is not supported do not fail but warn the user.
Similarly, if some statistics are not available, do not fail but warn
the user.
Verify that the MTU is in the device constraint. If setting it is not
supported, do not fail but warn user.
Verify the VLAN_STRIP offload is possible. If not, just don't activate
it.

Looks good to me, just need to run cargo fmt for formatting.


Cannot read property 'childNodes' of undefined

This error has popped up in Sentry, originating from Angular source code:

The offending line is: https://github.com/angular/angular.js/blob/master/src/ng/compile.js#L2043
It seems under some circumstances node may be undefined, after which this error is triggered when attempting to access node.childNodes. A guard should probably be put in place to ensure node is not undefined, and if it's a use case error, a more meaningful error should probably be triggered by Angular itself.
This issue has been seen on Angular 1.5.8 with Chrome mobile v44.0.2403, on Android 6.0.1.

This sounds very similar to #13960. It could be due to the same underlying issue: An out-of-sync nodeList.length (i.e. a situation where nodeList[nodeList.length - 1] === undefined, which should be impossible).
Unfortunately, we can't start putting guards all over the place in order to work around possible browser bugs. Do you have a way to reliably-ish reproduce this (I assume not 😞)?
It would be interesting to know what nodeList and stableNodeList look like and how the latter is populated here - but maybe that's too much to ask :smiley:

Yeah, I was afraid it might just be a fluke or browser specific issue. No reliable way to reproduce I'm afraid :) I was going to ignore the error, but figured I'd report it to see if it's something that Angular could do something about it or not.
Sounds similar to the issue you referred to, which also has been captured by Sentry. I guess capturing every single error from a large variety of browsers will inherently trigger some fluke situations/errors that we can't really do anything about.

I had this issue when using $compile on html that contained a directive that used $templateCache.  I got rid of the $templateCache and templateUrl and just replaced it with plain old template:'<div>...', and the error went away.

Closing due to lack of reproduction.


Children fn api

This updates the implementation with the latest changes to the result item renderer via a children render function.
Should resolve #3

Also note that children is just a prop so you can implement in either way:
<Omnibar children={ResultItem} />

or
<Omnibar>{ResultItem}</Omnibar>

There is quite the difference between example 2 and 3,
is there a check that children is a function?

Yup. You can verify here where ResultRenderer is a function.
These are technically equivalent:
<Omnibar>
    {({ item, ...rest }) =>
{item.title}
}
</Omnibar>

// just your average stateless functional component
const ResultRenderer = ({ item, ...rest }) =>
{item.title}
;

<Omnibar>
    {ResultRenderer}
</Omnibar>


Continously not able to load data from the API

Always this error comes. I have put my token v2 as well but it fails

It works now


Problem installing on Pop!_OS 19.04

Problem encountered on https://dotnet.microsoft.com/download/linux-package-manager/ubuntu19-04/sdk-2.2.108
Operating System: Linux Pop!_OS 19.04 - x64
I tried to download and install the latest version of dotnet core on a fresh install of Pop!_OS 19.04. Previously I had successfully install dotnet core on pop 18.04 following the Ubuntu instructions, however after updating to 19.04 I encountered a problem when running "sudo apt-get update." I'm getting the following error:

Err:8 https://packages.microsoft.com/ubuntu/19.04/prod disco/main amd64 Packages
File has unexpected size (8562 != 8396). Mirror sync in progress? [IP: <IP_ADDRESS> 443]
Hashes of expected file:

Filesize:8396 [weak]
SHA512:3cc3bfb2c60ba8f3fc52a7dde70c6f968fe0d270dfb2b2b5d741d28be072477e6644e42b39ba6793408a54a6448bcee68d3a3b2fac06aae5f0d50a4e926d66c9
SHA256:01c480722e325f82384a0fda852b58c8cf7e5181d609cbd1068ec5f72a89d572
SHA1:fed05cf0ba4157167e441745fb1887131be9105e [weak]
MD5Sum:b6a978f64f602f94ff5a03a69cf2f96e [weak]
Release file created at: Tue, 16 Jul 2019 22:15:00 +0000
Get:9 http://us.archive.ubuntu.com/ubuntu disco-backports InRelease [88.8 kB]
Fetched 190 kB in 1s (206 kB/s)
Reading package lists... Done
E: Failed to fetch https://packages.microsoft.com/ubuntu/19.04/prod/dists/disco/main/binary-amd64/Packages.bz2  File has unexpected size (8562 != 8396). Mirror sync in progress? [IP: <IP_ADDRESS> 443]
Hashes of expected file:
- Filesize:8396 [weak]
- SHA512:3cc3bfb2c60ba8f3fc52a7dde70c6f968fe0d270dfb2b2b5d741d28be072477e6644e42b39ba6793408a54a6448bcee68d3a3b2fac06aae5f0d50a4e926d66c9
- SHA256:01c480722e325f82384a0fda852b58c8cf7e5181d609cbd1068ec5f72a89d572
- SHA1:fed05cf0ba4157167e441745fb1887131be9105e [weak]
- MD5Sum:b6a978f64f602f94ff5a03a69cf2f96e [weak]
Release file created at: Tue, 16 Jul 2019 22:15:00 +0000
E: Some index files failed to download. They have been ignored, or old ones used instead.

I tried googling for a solution, but to no avail. Please help!

Just ran into this issue. I am also on POP_OS! 19.04

Same here with ubuntu 19.04

Taken today @ 3:30 pm

The dotnet core team has resolved the issue. Here is the link


Adds support for property origin tracking in config server/client.

Adds a new format if requested using v2 Accept header, otherwise the
format is backwards compatible so older clients will work with new
servers.
fixes gh-866
/cc @twicksell

Codecov Report

Merging #870 into master will decrease coverage by 1.04%.
The diff coverage is 46.8%.

@@            Coverage Diff            @@
##           master    #870      +/-   ##
=========================================
- Coverage   75.74%   74.7%   -1.05%
=========================================
  Files          83      86       +3
  Lines        2771    2842      +71
  Branches      397     408      +11
=========================================
+ Hits         2099    2123      +24
- Misses        515     561      +46
- Partials      157     158       +1

Impacted Files
Coverage Δ

...oud/config/server/resource/ResourceController.java
92.5% <ø> (ø)
:arrow_up:

...nfig/server/environment/EnvironmentRepository.java
0% <0%> (ø)

.../server/environment/JdbcEnvironmentRepository.java
80.95% <0%> (-1.98%)
:arrow_down:

...cloud/config/environment/EnvironmentMediaType.java
0% <0%> (ø)

...server/environment/VaultEnvironmentRepository.java
50.53% <0%> (-0.55%)
:arrow_down:

...ud/config/environment/PropertyValueDescriptor.java
0% <0%> (ø)

...fig/server/config/ConfigServerHealthIndicator.java
82.35% <100%> (ø)
:arrow_up:

...environment/MultipleJGitEnvironmentRepository.java
76.35% <100%> (ø)
:arrow_up:

.../config/server/environment/EnvironmentCleaner.java
100% <100%> (ø)
:arrow_up:

...nt/EnvironmentRepositoryPropertySourceLocator.java
100% <100%> (ø)
:arrow_up:

... and 10 more

Continue to review full report at Codecov.

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

It seems like we need this feature, and this looks like a good start, and we clearly have all the data we need. I don't like the format of the JSON though, and it seems like we could maybe even make a more incremental change that didn't require a new media type (or maybe not, but it's worth thinking about). Repeating the file location/name in every property value seems wasteful, and we could use integer fields for the line and column values, to make them more machine readable, and then convert to canonical "Boot" style format in the client.

I agree we could probably not repeat the location, but I don't think we can go straight to integers. Ints would work for the file based env repos, like git, but wouldn't work for jdbc or vault. Also, git blame info could go on each origin for git.

@dsyer @ryanjbaxter @OlgaMaciaszek @marcingrzejszczak @TYsewyn I'm reviving this again.
Two options I can see for the format:

file not repeated

{
    "label": null,
    "name": "foo",
    "profiles": [
        "cloud"
    ],
    "propertySources": [
        {
            "name": "https://github.com/spring-cloud-samples/config-repo/application.yml (document #1)",
            "source": {
                "application.domain": {
                    "origin": "27:11",
                    "value": "${APPLICATION_DOMAIN:cfapps.io}"
                },
                "endpoints.restart": {
                    "origin": "29:12",
                    "value": "enabled"
                }
         }
    ]
}

or repeated
{
    "label": null,
    "name": "foo",
    "profiles": [
        "cloud"
    ],
    "propertySources": [
        {
            "name": "https://github.com/spring-cloud-samples/config-repo/application.yml (document #1)",
            "source": {
                "application.domain": {
                    "origin": "[https://github.com/spring-cloud-samples/config-repo/application.yml]:27:11",
                    "value": "${APPLICATION_DOMAIN:cfapps.io}"
                },
                "endpoints.restart": {
                    "origin": "[https://github.com/spring-cloud-samples/config-repo/application.yml]:29:12",
                    "value": "enabled"
                }
         }
    ]
}

Once we get the paired down one (I think I prefer it), we could provide different fields for origin, line and column for origins that support it and an opaque string for others.

+1 to the paired down version

I agree on the paired down version, only question that I have now is whether or not we still need to have that  (document #1) suffix in the property source its name since we now have the exact location of the property within that file.

That comes from spring boot actually

Here's a sample client error message
***************************
APPLICATION FAILED TO START
***************************

Description:

Binding to target org.springframework.boot.context.properties.bind.BindException: Failed to bind properties under 'democonfigclient' to com.example.democonfigclient.DemoconfigclientApplication$Config failed:

    Property: democonfigclient.message
    Value: hello from dev profile
    Origin: https://github.com/spring-cloud-samples/config-repo/foo-dev.yml:3:27
    Reason: size must be between 0 and 10


fix(storybook-addon): update src path also for stories from v7

Fix for issue #1195

@fyodorovandrei locally i gett following build error when running
nx run-many --target=build

    ✖  nx run storybook-addon:build
       Compiling TypeScript files for project "storybook-addon"...
       error TS5090: Non-relative paths are not allowed when 'baseUrl' is not set. Did you forget a leading './'?

@ScriptedAlchemy


allow service MTU override through web ui

add UI component to select the connection MTU. Dropdown with options: 9000 (Jumbo) and 1500. Default is "9000"

add field to Connection (and SimpleConnection) to record the MTU. Integer.

add database field to connection table.

add PSS field for MTU to MxVPLS and AluVPLS classes under shared

PSS field parameter values are: Connection MTU + 100 (for MX), +114 (ALU).

add template variable for MTU to build-mx-vpls-service.ftl, build-alu-vpls-service.ftl

Fixed in v1.0.26


[Bug]: Cross linking navigation doesn't work correctly

Steps to repro:

On the ForYou screen, tap on any topic chip within a news resource card. The topic detail screen opens for that topic. It is in the Interests nav graph.
Tap on ForYou in the nav bar/rail

Expected result:

The ForYou screen is shown again

Actual result:

Nothing happens

I have the same problem.This problem is caused by the Navigation library and I've never been able to figure it out.

It worked!Thank a lot!


feat: certificate support

@matt-FFFFFF  - Based on the feedback from the customer,  requesting two features
Support for certificates

Work underway in associated branch

@matt-FFFFFF - Doing issue Triage and it appears there was a release back in November of last year that was supposed to close this issue.  Can this issue be closed or is there additional work to be done?

Hi I have key and secret support but not cert - still needs adding

Hi,
Please don't forget about certificates outputs.
Thank you!

Hi experimenting with this I don't think it belongs in the module. Certs are a data plane resource and it's perfectly possible to add certificates to a KV outside this module, using the outputs.


Obliterate Stripe::reset_agg_buf_pos

There was only one use of Stripe::reset_agg_buf_pos, and this gets rid of it by moving the code to flush the aggregate buffer from CacheDir.cc into Stripe.cc and AggregateWriteBuffer.cc.
I also removed two #defines from P_CacheVol.h that were supposed to be moved to AggregateWriteBuffer.h, but got left in due to a rebase mistake and were duplicated in both places.

[approve ci autest]

[approve ci autest]


Questions regarding LICENSE page

Here's the link:
https://github.com/github/open-source-guide/blob/gh-pages/LICENSE
Lack of Copyright Notice
There appears to be no copyright notice on this page.
Compare with the following - all of which have copyright notices somewhere on the page

https://github.com/github/choosealicense.com/blob/gh-pages/LICENSE.md
https://github.com/github/gh-ost
https://github.com/github/orchestrator/blob/master/LICENSE

Combined Creative Commons Explanation and License May Be Confusing
The text on the LICENSE page appears to be a word for word copy of
https://creativecommons.org/licenses/by/4.0/legalcode
If you read this page you will note that it combines both an explanation of the license and the license itself.
It might be clearer to people if the page had just the license text and linked back to the original CC page with the full CC explanation.
Software License
Where is it?

A copyright notice is not required. License notices are at https://github.com/github/open-source-guide/#licenses
The other repos you point at are using licenses that come in the form of a template. Many licenses (basically, the longer ones) aren't templates, and are intended to be used verbatim, with notices elsewhere, such as a README at the project level, or in file headers.
Closing as I don't think any change needs to be made, but feel free to continue to discuss if anything above doesn't make sense to you.

At least our legal department (Germany) would prefer (or requires us) to provide a copyright information. In 341 is a link to List of parties to international copyright agreements and TRIPS - Agreement on Trade-Related Aspects of Intellectual Property Rights  is mentioned.
But is unclear to me if this means that it is agreed that a copyright holder information is not needed?


Update plugin.json

Reorder restricted key permissions, change plans for products (plans removed, products needed now), and add index to name of group type index field.

@ivanagas could this be closed?


Investigate Form Recognizer Live test failures on Linux env

Form Recognizer live tests recently started failing with Temporary failure in name resolution error only on Linux setup env.
Would it be caused by the recent update to update the Linux OS images to Ubuntu 18.04 for the tests pipeline done in this #10469 .

cc: @mitchdenny @JimSuplizio @weshaggard

One difference between recently passing and recently failing runs appears to be that the ubuntu version changed. One would expect DNS resolution to work fairly consistently between versions of ubuntu.
However, in a test example running ubuntu 16 today the Linux jobs succeed and note no DNS failures: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=371880&view=logs&j=4d5db6ce-0b7f-527e-b115-2367ee6e1fef&t=57e6bea7-024e-5230-7282-6992d68e7015
Running against master (ubuntu 18) just now (in the hopes of this issue being transient) shows the same DNS failures: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=371919&view=logs&j=4d5db6ce-0b7f-527e-b115-2367ee6e1fef&t=57e6bea7-024e-5230-7282-6992d68e7015

Closed with #10576


Reformat Issue description and Open on click

This is the first PR like this for me, and I'm not a JS/TS developer, so let me know anything I could have done better.
I liked your extension, but there were a couple of things that didn't seem to fit my expectations set from your description.
This PR contains 3 main updates:

Update Issue description format. (See screenshot for example)
I like having the status in the description and I'm guessing other's would too
Use a task_complete icon when the status is Done. It's a box with a green check mark. This lets you quickly identify what is done vs. what is not.
Open issue in browser on click. This seemed to be the intended behavior, but was not what happened when I used the extension.

Screenshot:

Thanks in advance for doing this!

@avondohren  thank you very much for your contribution! I am really bad at seeing notifications so just saw this now. I'll review your PR over the 4th of July weekend and get back to you on any suggestions I might have for you.

@crhistianramirez Update the Done logic check and uploaded green versions of all the icons. Take a look and let me know how you like it!

@avondohren  hey hey! Look at you.
The changes look great! I'll get this merged in and publish a new release. Thanks for contributing, you made our extension better :)

v1.2.0 is out! 🎉


Change allowed values in PSS Seccomp section

According to the document, it seems runtime/default is for annotation and RuntimeDefault is correct for securityContext. Or, is runtime/default still valid for securityContext as well?
https://kubernetes.io/docs/tutorials/clusters/seccomp/#create-pod-that-uses-the-container-runtime-default-seccomp-profile

Duplicate of https://github.com/kubernetes/website/pull/28393


Add support for Toggle Radio and Checkbox

Closes #189

Any chance this gets merged and released before November? We have a rollout deadline I'm keeping an eye on. Given the historical release cadence, I'll assume "no chance" unless I hear back.

Thanks for the patch.
Please could you add docs and tests? Maybe the patch that added 'switches' could be inspiration. https://github.com/django-crispy-forms/crispy-bootstrap5/pull/162/files#diff-70937d46e98a34a699d83ee6562bcb31d7af839dd634e56060a1e7772995c62f


Messaging doesn't run stably on iOS 14

Unity editor version: 2019.3.12f1
Firebase Unity SDK version: 6.15.2
Source you installed the SDK: .unitypackage
Problematic Firebase Component: Messaging
Other Firebase Components in use: none
Additional SDKs you are using: Agora Engine
Platform you are using the Unity editor on: Windows
Platform you are targeting: iOS, Android
Scripting Runtime: IL2CPP

Since the realase of iOS 14 we noticed problems with the operation of the messaging module. We are using it to send informations beetwen two users in our application that share the view from the camera on the back of the mobile device.
The problem manifests itself in such a way that messages do not always reach the addressee, even though no errors are recorded throughout the process. Sometimes everything works fine and the next time you connect to the user, it "hangs up" after a few sent messages. Sometimes messages doesn't arrive in the initial phase when an attempt is made to connect to the other user.
This issue only occure on iOS 14, application is working fine on the device with iOS system lower then 14 and on devices with Android system.
We will be grateful for advice on how to improve the stability of the messaging module under iOS.

HI @RafalPucilowski ,
Thanks for sending a report in. Can I have you do a few things:
First, you left out a few fields in the bug report template that would really help. Can you tell us:

Have you been able to reproduce this issue with just the Firebase Unity quickstarts (this GitHub project)?
What's the issue repro rate? (eg 100%, 1/5 etc)

Getting accurate (>50%) repro steps in the sample app for messaging will help us get to the source of the issue as fast as possbile!
Second: Can you send us some sample code to see exactly what you're trying to do? It sounds like you might be running into issues with Upstream Messages but it'd be great to see the firebase bits to verify everything looks right at first glance.

Hi @patm1987,
thank you very much for your reply.
About first question, we didn't try to reproduce the issue on the Firebase Unity quickstart project because we use the backend to send messages. I should have mentioned this earlier, for which I am sorry. The whole process looks something like that: when user A wants to send a message to user B, it sends UnityWebRequest on the backend (all the required data are included in this request), which then sends a notification to the appropriate user via Firebase. The issue seems to be a problem with receiving messages, but I think we can't rule out the possibility that the sending is failing for some reason.
About second question, it's hard to say. It gives the impression that the message or data limit is running out. So at the app start it is fine but at some point the messages stop coming, after some time everything returns to normal (as if the limit was restarted) and the whole cycle repeats. If I had to define the rate, I would say 50%.
I will talk to our web developer and client today (to make sure they don't mind sharing the code) and we will try to prepare a sample code for you to check.
Have a great day.

Note that if it makes you feel more comfortable, you can always file a bug report. This will go through an internal process which would help keep your code secret and the support agents there may be able to examine your project setup to see if there's anything obviously wrong. If you go that route, feel free to link this GitHub issue in your report!

Hi,
sorry for the silence on our side but it's been a very busy week for me on other projects. We were able to contact the client and he agreed to try to reproduce the bug in your quickstart project and share the code from the main application.
I also talked to a colleague who was the original developer of the application and it turns out that due to latency only some types of messages are sent through our backend and so far we have not noticed that they cause any problems. So, only those sent via Cloud Messaging cause issue.
Below is one of the methods that cause trouble:

private void SendChangeCameraFreezeStateRequest ()
{
MessageDataBase messageData = new MessageDataBase("changeCameraFreezeState", Time.realtimeSinceStartup);
BasicMessage message = new BasicMessage(
"/topics/" + CallController.receiveCall.streamer,
1,
"high",
messageData
);
StartCoroutine(ChangeCameraFreezeStateRequest.POST(uri, headers, JsonUtility.ToJson(message)));
}

uri is equal to "https://fcm.googleapis.com/fcm/send", and 2 headers are set:
key: Content-Type
value: application/json
and
key: Authorization
value: here is our server key taken from the Firebase console
The POST method looks like this:

public IEnumerator POST(string uri, Header[] headers, string body = null)
{
byte[] data = null;
if (body != null)
{
data = Encoding.UTF8.GetBytes(body);
}
    using (UnityWebRequest unityWebRequest = UnityWebRequest.Post(uri, body))
    {
  if (data != null && data.Length > 0)
  {
  	unityWebRequest.uploadHandler = new UploadHandlerRaw(data);
  }

            yield return Request(headers, unityWebRequest);
    }

}

and Request method looks like this:

private IEnumerator Request(Header[] headers, UnityWebRequest unityWebRequest)
{
SetHeaders(headers, unityWebRequest);
yield return unityWebRequest.SendWebRequest();
CheckRequest(unityWebRequest);
}

below are two more methods used in Request()

private void SetHeaders(Header[] headers, UnityWebRequest unityWebRequest)
{
foreach (Header header in headers)
{
unityWebRequest.SetRequestHeader(header.Key, header.Value);
}
}

private void CheckRequest(UnityWebRequest unityWebRequest)
{
if ((unityWebRequest.isNetworkError == true) || (unityWebRequest.isHttpError == true))
{
Fail(unityWebRequest);
}
else
{
Success(unityWebRequest);
}
}

CheckRequest() invoke Success() at the end even when the news stops arriving.
If you see anything that may be causing this issues or have any questions please let me know. Next week we will try to do tests on your project. If we manage to reproduce the bug, we will probably do a bug report.
Thank you very much for your time.

Hi @patm1987 ,
we reported a bug - case 00089714 - with video and simple project.
Looking forward for your findings.

Hi @RafalPucilowski
Where can we find the video and simple project for this case 00089714? Did you report it through a different channel?
FWIW, could you try the latest 6.16.0 and see if the instability is resolved, since the Pod version was upgraded from 6.24.0 to 6.32.2.
I also noticed this weird issue with push notification #820 and was trying to reach out to FCM team about this.
Please let us know if upgrading works for you.
Shawn

Hi Shawn,
we send the Bug Report but the issue is still not resolved. Right now we will be checking if the latest iOS update has changed anything.
Regarding your suggestion, we used the 6.16.0 in test project.
Bellow is the link to the video and the description what to look for.
https://drive.google.com/file/d/1gxd_jTIhnyVXeX82d9doAQciiiYyvyxu/view?usp=sharing
Phones from left:

iPhone 7 (iOS 13.4.1)
iPhone 8 (iOS 13.3)
iPhone XS (iOS 14.0.1)
Samsung Galaxy S7 (Android 8.0)

The flow of the application is very simple, it attach to a message event, joins the room, and increases the counter when it receives any message. Of course all devices runs the same application. You can see that all devices receive messages (on iOS sometimes one message appears 3 times) but only counter on the iPhone XS stops at some point.
I will find out if we still have a project available online and send it to you later.
If you have any thoughts on that issue, please let me know.
Rafal

Hi,
here is the link to the test project:
https://drive.google.com/file/d/1vcqc3FgE_b3Ccmtcedl1us4Kt0C8oxc0/view?usp=sharing
Just find the Manager object in the scene and set the Authorization key and you should be able to build and run the app.
Let me know when you reproduce the problem.
Rafal

Definitely let us know here if you make any progress in the bug report.
I have some more things for you to try:

can you try 6.16.1 just to see if anything there fixed your issue?
Can you gather debug logs with -FIRDebugEnabled set as an argument?
I assume that you have since this is iOS 14 specific, but have you uploaded your APNs certificate/auth key?
Is swizzling enabled (ie: things may fail if you explicitly disable it in your Info.plist. If you don't know what this is then you should be good)
Can you reproduce this issue by directly sending messages via APNs?

A quick update, I did some digging over the weekend and found that there were some token generation issues that were fixed recently (an example issue: https://github.com/firebase/firebase-ios-sdk/issues/6862 ).
That particular bug looks like it might have been fixed in 7.0.0 and the Firebase Unity SDK ships with the iOS pod at version 6.32.2 with Unity SDK 6.16.1. I'd still recommend upgrading to the latest SDK then, if that doesn't work, locate your MessagingDependencies.xml file and change the line:
<iosPod name="Firebase/Messaging" version="6.32.2" minTargetSdk="8.0">

to:
<iosPod name="Firebase/Messaging" version="7.0.0" minTargetSdk="8.0">

to see if that works.
An important note: the Unity SDK is typically pinned to a specific iOS SDK. Modifying this field will put you into an unsupported configuration, until a new version of the Unity Firebase SDK is released. This may work, but only do this if it fixes your issue and do some extra testing around messaging.

Hi @patm1987
thank you very much for your time spent on the issue. We conducted tests on our other project, that is during development, which also uses Firebase communication for iOS, and also on the test project, and I can confirm that the issue has not occurred so far.
By the end of the week, I will update the Unity SDK on my project and change the iOS pod to 7.0.0.
Everything indicates that we will finally be able to end this thread 👍

Hi @RafalPucilowski ,
Any update from your side?

Hi @chkuang-g,
I got some update. Our QA spend couple of hours on testing application with suggested setup (Unity SDK 6.16.1 and iOS pod 6.32.2).
He wasn't able to use iOS pod 7.0.0 because as far as I can remember, the app was unstable and there were crashes.
Fortunately it works much better with iOS pod 6.32.2.
We tested the connection between two iPhone devices (on system versions 13.4 and 14.2), which was stable and did not lose any messages sent.
Unfortunately, problems appeared when trying to use the device on Android. Our tester was unable to run the build. Immediately at the start there was a crash and this error:
2020-11-20 12:47:16.005 21754-21754/? E/AndroidRuntime: FATAL EXCEPTION: main     Process: com.ForExperience.Ar4Vision, PID: 21754     java.lang.RuntimeException: Unable to instantiate activity ComponentInfo{com.ForExperience.Ar4Vision/com.google.firebase.MessagingUnityPlayerActivity}: java.lang.ClassNotFoundException: Didn't find class "com.google.firebase.MessagingUnityPlayerActivity" on path: DexPathList[[zip file "/data/app/com.ForExperience.Ar4Vision-yLF_UXMkFQo8gz2SCosX0Q==/base.apk"],nativeLibraryDirectories=[/data/app/com.ForExperience.Ar4Vision-yLF_UXMkFQo8gz2SCosX0Q==/lib/arm64, /data/app/com.ForExperience.Ar4Vision-yLF_UXMkFQo8gz2SCosX0Q==/base.apk!/lib/arm64-v8a, /system/lib64, /system/vendor/lib64]]         at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:3020)         at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:3257)         at android.app.servertransaction.LaunchActivityItem.execute(LaunchActivityItem.java:78)         at android.app.servertransaction.TransactionExecutor.executeCallbacks(TransactionExecutor.java:108)         at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:68)         at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1948)         at android.os.Handler.dispatchMessage(Handler.java:106)         at android.os.Looper.loop(Looper.java:214)         at android.app.ActivityThread.main(ActivityThread.java:7050)         at java.lang.reflect.Method.invoke(Native Method)         at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:494)         at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:965)      Caused by: java.lang.ClassNotFoundException: Didn't find class "com.google.firebase.MessagingUnityPlayerActivity" on path: DexPathList[[zip file "/data/app/com.ForExperience.Ar4Vision-yLF_UXMkFQo8gz2SCosX0Q==/base.apk"],nativeLibraryDirectories=[/data/app/com.ForExperience.Ar4Vision-yLF_UXMkFQo8gz2SCosX0Q==/lib/arm64, /data/app/com.ForExperience.Ar4Vision-yLF_UXMkFQo8gz2SCosX0Q==/base.apk!/lib/arm64-v8a, /system/lib64, /system/vendor/lib64]]         at dalvik.system.BaseDexClassLoader.findClass(BaseDexClassLoader.java:134)         at java.lang.ClassLoader.loadClass(ClassLoader.java:379)         at java.lang.ClassLoader.loadClass(ClassLoader.java:312)         at android.app.AppComponentFactory.instantiateActivity(AppComponentFactory.java:69)         at android.app.Instrumentation.newActivity(Instrumentation.java:1215)         at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:3008)         at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:3257)          at android.app.servertransaction.LaunchActivityItem.execute(LaunchActivityItem.java:78)          at android.app.servertransaction.TransactionExecutor.executeCallbacks(TransactionExecutor.java:108)          at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:68)          at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1948)          at android.os.Handler.dispatchMessage(Handler.java:106)          at android.os.Looper.loop(Looper.java:214)          at android.app.ActivityThread.main(ActivityThread.java:7050)          at java.lang.reflect.Method.invoke(Native Method)          at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:494)          at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:965)
Finally, I shared my last build with him (which used the last Unity SDK). I remember having the same problems when I built it, but I believe changing the Player settings in Unity helped. Surprisingly, this time using the same settings did not help.
The connection between iPhone with iOS14.2 and Samsung S8 (possibly with the latest Android) was stable for the first 5 minutes of testing. Later on, the same issues with not arriving messages surfaced (interestingly, we haven't noticed this issue in cross-platform communication so far).
Then we checked the iPhone connection again and here too there was a problem. After further testing, we found that after restarting both applications, the iPhones could communicate stably with each other again.
Everything indicates that as long as there is no communication between different platforms (iOS <-> Android), sending and receiving messages works stably.

Hi @RafalPucilowski,
6.16.* builds of the Firebase Unity SDK was tested and built with Pods older than Firebase iOS SDK 7.0.0, so the instability that you mentioned is expected. Forutnately we released Firebase Unity SDK 7.0.0 and 7.0.1 which do use Firebase iOS SDK 7.0.0.  Could you try to update to the latest Unity SDK and try again?
Thanks!

Hi @DellaBitta,
thanks for the update, we definitely will check new Unity SDK and let you know if issue is still present.
Best regards.


Fix rollback steps

Description
Fix rollback steps in flannel migration doc
Release Note

None required

Deploy preview for calico ready!
Built with commit a5435eabf5cf3d74f52387f6001eb7a28ecd842e
https://deploy-preview-2842--calico.netlify.com


The selectedvalue value is incorrect while running the onclose event

Bug type
Component
Component name
MudSelect
What happened?
I want to query the selected value in the onclose event, but it is set only after the onclose event is called, so it always shows the previous value
Expected behavior
The correct value must be set at the moment the onclose event is called
Reproduction link
https://try.mudblazor.com/snippet/GYccOeOpdbuicAyO
Reproduction steps

Select any value in the dropdown
The onclose event is called and reads the selected value that does not match the actual value

...
Relevant log output
No response
Version (bug)
6.0.9
Version (working)
No response
What browsers are you seeing the problem on?
Chrome
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

I think its the aim of OnClose. Its the only place when you catch previous value. For new value you can use ValueChanged event. Am i wrong @henon ?

It seems to be always one behind.
Correct, which also affects the Required parameter.

@henon @mckaragoz So we want to run CloseMenu/OnClose methods AFTER we set the value, but before any ValueChanged events?

@henon @mckaragoz So we want to run CloseMenu/OnClose methods AFTER we set the value, but before any ValueChanged events?

I think OnClose run after setting value but not sure which one call first, ValueChanged or OnClose? I think valuechanged should fire first.

Appears to be working in v7.13.0. Please let us know if this is still a problem so we can re-open the issue


Build overrides for displaying a Version indicator

TL;DR
I need to know what version of the function is running for  --runtime go113
I need to be able to do something like:
CGO_ENABLED=0 GOOS=linux GOARCH=amd64 go build -ldflags -X main.Version=$(git describe --always) -o server *.go

Hi @kaihendry
Could you elaborate on the request a bit more. Did you mean you would like an output with what is the current deployed version after the deployment is complete?

Hi @kaihendry
Could you elaborate on the request a bit more. Did you mean you would like an output with what is the current deployed version after the deployment is complete?

Closing due to lack of response.


Details about normalizing each channel for rectified flow training.

I found that it's mentioned in report_03 that
New VAE and fps conditioning: we replace the original VAE with ours and add fps conditioning to the timestep conditioning, we train for 25k steps. Note that normalizing each channel is important for rectified flow training.
I failed to find how you normalized each channel in details, maybe four 0.18215-like scale factors for four channels? Could any one lend a hand?

normalizing each channel?nn. Ln?


MatMul op results mismatch (NPU/Numpy)

I attempted to measure the performance between NPU and Numpy-based dot product computations using float16 type but I found that results mismatch occurs between NPU/CPU numpy. I am using NPU on ARL.
To Reproduce
Steps to reproduce the behavior:

Copy the code below and to run it python <filename.py>

from intel_npu_acceleration_library.backend import MatMul
import numpy as np
import time

inC = 8
outC = 8
batch = 1

X1 = np.random.uniform(-1, 1, (batch, inC)).astype(np.float16)
X2 = np.random.uniform(-1, 1, (outC, inC)).astype(np.float16)

mm = MatMul(inC, outC, batch, profile=False)

start_time = time.perf_counter()
result = mm.run(X1,X2)
end_time = time.perf_counter()
print(f"Intel NPU Acceleration Library Time: {end_time - start_time} * 1000:.6f millisecs")

start_time = time.perf_counter()
np_res = np.dot(X1, X2)
end_time = time.perf_counter()
print(f"Numpy Library Time: {end_time - start_time} * 1000:.6f millisecs")

print("NPU Result: ", result)
print("Numpy Result:", np_res)

Expected behavior
Output mismatch between numpy/NPU occurs
>python matmul.py
Intel NPU Acceleration Library Time: 0.177700 ms
Numpy Library Time: 0.015700 ms
NPU Result:  [[ 1.178  -0.572   1.957  -0.4443 -0.549   0.7744 -0.2756 -0.997 ]]
Numpy Result: [[-0.1885    1.905    -1.779    -0.8945    0.866    -0.9365    1.248
  -0.006233]]

>python matmul.py
Intel NPU Acceleration Library Time: 0.189100 ms
Numpy Library Time: 0.016700 ms
NPU Result:  [[-0.3157  2.506   0.3499  0.631  -0.1031  0.5913 -1.599   1.001 ]]
Numpy Result: [[-1.137    0.09534  0.12366 -1.748    0.981    0.2004  -0.2607  -2.086  ]]

>python matmul.py
Intel NPU Acceleration Library Time: 0.177000 ms
Numpy Library Time: 0.013300 ms
NPU Result:  [[ 0.7153 -1.251   0.2106 -0.409  -0.4336  0.2329  1.653  -1.58  ]]
Numpy Result: [[-1.764   1.005  -0.837  -1.518   0.8794  0.427   0.1887 -1.153 ]]

Desktop (please complete the following information):

OS: Win11Enterprise

Hi, mm.run(X1,X2) is equivalent to np.dot(X1, X2.T), if you use that the math checks out.
Also, external AI accelerators (like GPUs and NPUs) are more effective when offloading large operations, you won't see speedups in multiplying a 8x8 matrix with a vector (as explained very nicely here )
Here an example with a medium size matrix matrix operation that have a significative speedup:
from intel_npu_acceleration_library.backend import MatMul
import numpy as np
import time

inC = 1024
outC = 1024
batch = 256

X1 = np.random.uniform(-1, 1, (batch, inC)).astype(np.float16)
X2 = np.random.uniform(-1, 1, (outC, inC)).astype(np.float16)

mm = MatMul(inC, outC, batch, profile=False)

start_time = time.perf_counter()
result = mm.run(X1,X2)
end_time = time.perf_counter()
print(f"Intel NPU Acceleration Library Time: {(end_time - start_time) * 1000:.6f} ms")

start_time = time.perf_counter()
np_res = np.dot(X1, X2.T)
end_time = time.perf_counter()
print(f"Numpy Library Time: {(end_time - start_time) * 1000:.6f} ms")

print("NPU Result: ", result)
print("Numpy Result:", np_res)

attached some code you can use for reference, it returns the following on an ARL machine
Intel NPU Acceleration Library Time: 0.831700 ms
Numpy Library Time: 2027.385200 ms
NPU Result:  [[ 18.16   -12.62     6.703  ...   8.24     4.71    -5.74  ]
 [  5.82   -15.77   -11.47   ...  -6.914   11.86    -0.8833]
 [  3.604   -7.562    6.15   ...  14.19     8.88    13.63  ]
 ...
 [ -7.438   -8.72     0.1948 ...   5.684   -1.962   -0.7773]
 [  4.727   19.52   -13.34   ...   4.973   -4.89    18.12  ]
 [ -8.625    4.54     7.22   ... -11.734   14.914  -19.64  ]]
Numpy Result: [[ 18.16   -12.62     6.703  ...   8.24     4.71    -5.74  ]
 [  5.82   -15.77   -11.47   ...  -6.914   11.86    -0.8833]
 [  3.604   -7.562    6.15   ...  14.19     8.88    13.63  ]
 ...
 [ -7.438   -8.72     0.1948 ...   5.684   -1.962   -0.7773]
 [  4.727   19.52   -13.34   ...   4.973   -4.89    18.12  ]
 [ -8.625    4.54     7.22   ... -11.734   14.914  -19.64  ]]

Also, consider that first time you compile results are skewed because of first inference latency

Thank you will check this and confirm at my end.


Revved gatk public with ensuing refactoring in Mutect

@davidbenjamin A lot of TODOs here.  These should be addressed and removed soon (or file issues).  You can merge otehrwise after addressing a few comments.