chore: created a temporary downtime page

Description
A temporary downtime page as we do our database migration. This PR will not get merged. It will remain in draft, and we will deploy this branch first to beta to test, and then prod just before we begin the database migration.
Once the database migration is complete in production, we can just rollback to the previous frontend deploy on Netlify.
Related Tickets & Documents
Relates to #4156.
Mobile & Desktop Screenshots/Recordings

Steps to QA
To test, navigate to any page and it redirects to /downtime.html.
e.g.
https://deploy-preview-4157--oss-insights.netlify.app/ -> https://deploy-preview-4157--oss-insights.netlify.app/downtime.html
https://deploy-preview-4157--oss-insights.netlify.app/u/nickytonline -> https://deploy-preview-4157--oss-insights.netlify.app/downtime.html
https://deploy-preview-4157--oss-insights.netlify.app/workspaces/a5b9c1f5-8f01-4f61-a6b7-09943ef9e069 -> https://deploy-preview-4157--oss-insights.netlify.app/downtime.html
https://deploy-preview-4157--oss-insights.netlify.app/workspaces/explore -> https://deploy-preview-4157--oss-insights.netlify.app/downtime.html
Tier (staff will fill in)

[ ] Tier 1
[ ] Tier 2
[ ] Tier 3
[x] Tier 4

[optional] What gif best describes this PR or how it makes you feel?

@BekahHW @isabensusan @open-sauced/engineering, feel free to suggest better wording.

The 1 page is in their. Just struggling to get a normally simple global redirect to work in the netlify.toml. 😅

This should be a single page deployable from a downtime branch: this way, we can simply flip the deployment of app.opensauced.pizza on Netlify to the downtime branch.
If that doesn't track with how we'd execute on this, let me know.

Yeah I can do that. It's simpler. Update incoming.

https://github.com/open-sauced/app/tree/nickytonline/downtime-page

Great: can we set the target branch to be a new downtime branch instead of beta? That way, we can merge to that branch and target it for Netlify deploys if needed!


pacman.conf missing cachedir.

Notice lately that aur sync -c -D /tmp/$(uname -m) -T ${pkg} keeps downloading all dependencies instead of using on-disk cache. I have my cache moved to home, but there is no corresponding cachedir option in chroot pacman.conf.
/etc/pacman.conf:
...
CacheDir    =  /home/_pacman_cache/pkg/
...

/tmp/x86_64/root/etc/pacman.conf:
...
[options]
HoldPkg = pacman
HoldPkg = glibc
Architecture = auto
SigLevel = PackageRequired
SigLevel = PackageTrustedOnly
SigLevel = DatabaseOptional
SigLevel = DatabaseTrustedOnly
LocalFileSigLevel = PackageOptional
LocalFileSigLevel = PackageTrustedOnly
[core]
Server = http://mirror.dkm.cz/archlinux/$repo/os/$arch
...

Is "lately" after pacman 5.1 and recent updates to devtools?

Seems that way.

I think https://git.archlinux.org/devtools.git/commit/?id=5b3c14454a9c1ec00a3ef11f3f599281127b383d should be applied to https://git.archlinux.org/devtools.git/tree/arch-nspawn.in#n56 as well, since the pacman -v trick no longer works in pacman 5.1. @eli-schwartz

What do
pacconf --config=/usr/share/devtools/pacman-extra.conf
pacconf --config=/usr/share/devtools/pacman-extra.conf --raw

return on your system?

pacconf
[options]
RootDir = /
DBPath = /var/lib/pacman
CacheDir = /home/_pacman_cache/pkg/
CacheDir = /home/_pacman_cache/pkg
CacheDir = /home/_pacman_cache/aur
HookDir = /etc/pacman.d/hooks
GPGDir = /etc/pacman.d/gnupg
LogFile = /var/log/pacman.log
HoldPkg = pacman
HoldPkg = glibc
IgnorePkg = seahorse
IgnorePkg = cura
Architecture = x86_64
Color
CheckSpace
ILoveCandy
CleanMethod = KeepCurrent
SigLevel = PackageRequired
SigLevel = PackageTrustedOnly
SigLevel = DatabaseOptional
SigLevel = DatabaseTrustedOnly
LocalFileSigLevel = PackageOptional
LocalFileSigLevel = PackageTrustedOnly
RemoteFileSigLevel = PackageOptional
RemoteFileSigLevel = PackageTrustedOnly
[core]
...

pacconf --config=/usr/share/devtools/pacman-extra.conf
[options]
RootDir = /
DBPath = /var/lib/pacman
CacheDir = /var/cache/pacman/pkg
HookDir = /etc/pacman.d/hooks
GPGDir = /etc/pacman.d/gnupg
LogFile = /var/log/pacman.log
HoldPkg = pacman
HoldPkg = glibc
Architecture = x86_64
CleanMethod = KeepInstalled
SigLevel = PackageRequired
SigLevel = PackageTrustedOnly
SigLevel = DatabaseOptional
SigLevel = DatabaseTrustedOnly
LocalFileSigLevel = PackageOptional
LocalFileSigLevel = PackageTrustedOnly
[core]
Usage = All
Server = http://mirror.dkm.cz/archlinux/core/os/x86_64
...

pacconf --config=/usr/share/devtools/pacman-extra.conf --raw
[options]
HoldPkg = pacman
HoldPkg = glibc
Architecture = auto
SigLevel = PackageRequired
SigLevel = PackageTrustedOnly
SigLevel = DatabaseOptional
SigLevel = DatabaseTrustedOnly
LocalFileSigLevel = PackageOptional
LocalFileSigLevel = PackageTrustedOnly
[core]
...

pacconf.txt
pacconf--config=pacman-extra.conf.txt
pacconf--config=pacman-extra.conf--raw.txt

Tell me why it doesn't work.
$ pacman -v 2>&1 | grep '^Cache Dirs:' | sed 's/Cache Dirs:\s*//g'
/var/cache/pacman/pkg/  /var/cache/pacman/custom/  /home/bartoszek_testcase/_pacman_cache/pkg/

I mean yes, it's sort of ugly...

You could just use pacman-conf and use mapfile (I have no idea why everything is deliberately split on white space in devtools) for the first part and cache_dirs+=("$OPTARG") for the second. Anyway I doubt anyone is interested in anything that makes devtools easier for external use.

Looks like arch-nspawn will replace whatever I put in CacheDir with whatever it thinks is best: https://git.archlinux.org/devtools.git/tree/arch-nspawn.in#n95
Can we just remove devtools support already from aurutils?

Anyway with 6036e5b832222b10c0f51a5a07533fca41ead022 all should be working now.


why  clEnqueueReadImage  is not  linkable in the  built  lib.

Dear ,  I  built a program with   arm_compute-v22.05-bin-android-arm64-v8a-cl\lib\arm64-v8a-cl\libarm_compute.so  ,   it got an error  as below:
undefined reference to `clEnqueueReadImage'
clang++.exe: error: linker command failed with exit code 1
other  cl  apis   has  no problem , like  clCreateImage.
is  the function  clEnqueueReadImage  forbidden  in  the  arm  system?  it  is ok in windows X86 .

Hi @lyz9801
Could you please share the command you used to build the program?
Please see the documenation on how to build the examples:
https://arm-software.github.io/ComputeLibrary/latest/how_to_build.xhtml#S1_2_2_examples
Hope this helps

Hi,  @morgolock
I  havn't  built  the program yet,  just directly  use  the  library  in  the hompage ,   the  linked address is  https://github.com/ARM-software/ComputeLibrary/releases/download/v22.05/arm_compute-v22.05-bin-android-arm64-v8a-cl.tar.gz
in   ComputeLibrary-main/src/core/CL/OpenCL.cpp,  there is an implementation of  clCreateImage,   but   I cannot find clEnqueueReadImage ,  which is also a standard  api   in  https://www.khronos.org/files/opencl30-reference-guide.pdf .  I‘m not sure  whether  this api  is supported in the embedded computing   system.     As in  the  intel X86 platform, the api  can be found in https://gitlab.freedesktop.org/beignet/beignet/-/blob/master/src/cl_api_mem.c

Hi @lyz9801

I havn't built the program yet, just directly use the library in the hompage , the linked address is https://github.com/ARM-software/ComputeLibrary/releases/download/v22.05/arm_compute-v22.05-bin-android-arm64-v8a-cl.tar.gz

Could you please elaborate and explain how do you use the library?
What device and OS are you using?


Contracts header with assertion macros specific to Sparrow

This provides macros (extracted from my library NoContracts) for contract assertions checks, named with SPARROW_ so that control over enabling these assertions is let to the user.
The checks are enabled by default and all C assert(...) calls have been replaced.

OK I made a big replace operation to do that so now I'll review each assert see if the other option isnt better 👍🏽


R75: Data Services: Monitoring: Monitoring services shall calculate the total number of query counts over time

Tools: MySQL

Apache Derby is the SQL tool used for now, as it doesn't require a server install, and we only need rudimentary, single-class, SQL functionality

Added the endpoint monitor/querytimes to provide this.


Changes from apache/tvm for to neo-ai/tvm 1.11

Thanks for contributing to TVM!   Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers by @ them in the pull request thread.

Looks good but some tests are failing, could you take look @ylc ?


Technical set up for ongoing training

Title of ticket:
Description
There are some technical tasks involved with training FOI users on the pilot system
Dependencies
DOD

[ ] Create a namespace / staging environment
[x] Set up IDIRs for all users in keycloak

Closing since #1687 was created


addSample function of SampledProperty bug

hi
When adding a new position(sample property), there is a bug that does not work properly.
There is a bug that changes existing values instead of being added.

Can you please provide a Sandcastle code example that recreates this issue?

I'm sorry..
my mistake..
not bug. thank you.

No worries, thanks for confirming!


Disabling several dates

I want to disable a range of dates like the format below:
disable[0] = { start: '2015-07-03', end: '2010-07-11' };
disable[1] = { start: '2010-07-17', end: '2010-07-20' };
disable[2] = { start: '2010-08-01', end: '2010-08-02' };
How can I do this using Kalendae blackout function?

This is dry coded and might have errors, but I believe it should do the job.
function (date) {
  var ok = true;
  disable.forEach(function (d) {
    if (!ok || date >= (new Date(d.start)) && date <= (new Date(d.end))) ok = false;
  }
  return ok;
}

The code below is working good except to dates that are in between different months.
var disable = [];
disable.push({ start: '2015-06-28', end: '2010-06-29' });
disable.push({ start: '2015-06-30', end: '2010-07-01' }); // Dates in between different months doesn't works.
disable.push({ start: '2015-07-03', end: '2010-07-05' });
disable.push({ start: '2010-07-17', end: '2010-07-20' });
disable.push({ start: '2010-08-01', end: '2010-08-15' });
var mo = Kalendae.moment,
i=disable.length,
dd = Kalendae.moment(date).date(),
mm = Kalendae.moment(date).month(),
start,
stop;
while (i--) {
start = mo(disable[i].start);
stop = mo(disable[i].end);
if (mm == start.month() && mm == stop.month() && dd >= start.date() && dd <= stop.date()) return true;

}
return false;
credits to: https://github.com/ChiperSoft/Kalendae/issues/18


[BUG] Can't use for in a custom widget that delegates to box

Checklist before submitting an issue

[X] I have searched through the existing closed and open issues for eww and made sure this is not a duplicate
[X] I have specifically verified that this bug is not a common user error
[X] I am providing as much relevant information as I am able to in this bug report (Minimal config to reproduce the issue for example, if applicable)

Description of the bug
When defining a custom widget that delegates its children into a box, using that widget with a for as child fails:
error: This widget can only be used as a child of some container widget such as box
  ┌─ /nix/store/xi81k4i5a1pa8l6hqajg1088lf0bflz1-hm_eww/eww.yuck:5:10
  │
5 │   (v-box (for workspace in workspaces (label :text {workspace})))
  │          ───────────────────────────────────────────────────────
  │
  → Hint: try wrapping this in a `box`

Reproducing the issue
(defwidget v-box [] (box :orientation "v" (children)))

(defvar workspaces '["foo"]')
(defwindow taskbar
  (v-box (for workspace in workspaces (label :text {workspace})))
)

Expected behaviour
A widget wrapping a box should be allowed to accept for as children.
Additional context
No response

Same


Can't scroll to newly added plugin

Environment

OS Version: Ubuntu 20.04
Source or binary build?

Source, main branch, latest commit 5555853ce9ad2ae15860c2d3fc7f79e12247940a
Description

Expected behavior: Scroll to the newly added plugin on the righthand pane
Actual behavior: Can't scroll until the window has been resized

Most likely introduced by https://github.com/ignitionrobotics/ign-gui/pull/194
Steps to reproduce

ign gazebo shapes.sdf
Click any plugin from the menu bar
Attempt to scroll

Output

This corner case was overlooked in #194.
When fixing this, we should make sure we don't break the use cases that were fixed then:

If the window is not large enough to fit all plugins, squish them all to their minimum height and make the split scrollable.
If the window is large enough to fit all plugins, they are stretched to fill the entire height, and their proportion can be changed dragging the separator.
Collapsed plugins behave consistently with the above
Adding / removing plugins also behaves as expected


fix: Ensure persistent volumes are detached before deleting node

Fixes #N/A
Description
Fixes 6+ minute delays for disrupted EBS-Backed stateful workloads when starting on their new node.
For more context see RFC for solving 6+ minute delays for disrupted stateful workloads
TLDR:
In order for a stateful pod to smoothly migrate from terminating node to new node...

Consolidation event starts
Stateful pods must terminate
EBS CSI Node pod must unmount all filesystems (NodeUnpublish & NodeUnstage RPCs)
EBS CSI Controller pod must detach all volumes from instance
Karpenter terminates EC2 Instance
Karpenter ensures Node object deleted from Kubernetes

Problems:
A. If 2 doesn't happen, today there's a 6+ minute delay in stateful pod migration because Kubernetes is afraid volume still attached and mounted to instance (6+ min delay)
B. If 3 doesn't happen, the new stateful pod can't start until consolidated instance is terminated which auto-detaches volumes (1+ min delay)
Solution:
Wait for volumeattachment objects associated with drainable pods & non-multi-attach volumes before deleting the node.
How was this change tested?
Manual: Create statefulset + nodepool. Have nodes expire every 3 minutes. Check that stateful pods migrate to new node and start running in under a minute.
Also tested that we do not block deletion when there are stateful workloads that tolerate all taints, or Node terminationGracePeriod elapsed.
Additional Notes
Note 1: Must add read permissions for volumeattachments to clusterrole-core.yaml in karpenter-provider-aws
  - apiGroups: ["storage.k8s.io"]
    resources: ["storageclasses", "csinodes", "volumeattachments"]
    verbs: ["get", "watch", "list"]

Note 2: Separate PR to add e2e tests in karpenter-provider-aws: https://github.com/aws/karpenter-provider-aws/pull/6484
Note 3: It was decided in the RFC we will block node deletion for all volumeattachments, regardless of CSI Driver. In the future, we may decide to inject a list of CSI Drivers via the cloud provider instead.
Note 4: There might be some rare cases where EBS CSI Node pod can get killed before it unmounts volumes. The solution would be karpenter (or some reliable automation) tainting node with nodeshutdown:NoExecute once node is terminated, as discussed in RFC. In design meeting consensus was that this could be added later if customers run into it.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

In the future we should consider adding an e2e test to karpenter-provider-aws that tests that a stateful workload on a consolidating node is migrated and starts running on second node < 6 minutes. @jmdeal mentioned that EBS CSI Driver is already installed on e2e environment. Thoughts?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

49 of 67 (73.13%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.04%) to 77.922%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/controllers/node/termination/terminator/terminator.go
18
20
90.0%

pkg/operator/operator.go
0
3
0.0%

pkg/utils/node/node.go
7
12
58.33%

pkg/controllers/node/termination/controller.go
15
23
65.22%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
8319

Relevant Lines:
10676

💛  - Coveralls

/remove-lifecycle stale

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

23 of 68 (33.82%)  changed or added relevant lines in 4 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.3%) to 78.009%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/operator/operator.go
0
3
0.0%

pkg/utils/node/node.go
8
13
61.54%

pkg/controllers/node/termination/controller.go
9
17
52.94%

pkg/utils/volumeattachment/volumeattachment.go
6
35
17.14%

Files with Coverage Reduction
New Missed Lines
%

pkg/controllers/node/termination/controller.go
2
58.87%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
8996

Relevant Lines:
11532

💛  - Coveralls

/hold
Need to Confirm the following through Manual tests:

multi attach volume doesn’t block node deletion
In line volume doesn’t block node deletion (because no volumeattachment objects)
static pod doesn't block node deletion (because no volumeattachment objects?)

Feel free to still lgtm or have other teammates review if code looks fine @jmdeal @tallaxes @jackfrancis

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

1 of 69 (1.45%)  changed or added relevant lines in 4 files are covered.
9 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.5%) to 77.625%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/operator/operator.go
0
3
0.0%

pkg/utils/node/node.go
0
13
0.0%

pkg/controllers/node/termination/controller.go
1
18
5.56%

pkg/utils/volumeattachment/volumeattachment.go
0
35
0.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/test/expectations/expectations.go
2
94.64%

pkg/controllers/provisioning/scheduling/preferences.go
7
86.67%

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
8805

Relevant Lines:
11343

💛  - Coveralls

/retest

/assign @jmdeal

/hold
Re-doing manual tests now that Karpenter is v1.

No 1 minute EC2 delay
multi attach volume doesn’t block node deletion
Node TGP Works properly
workload tolerating Disruption taint does not block node deletion

/retest

/hold Wait for E2E tests to complete in the CloudProvider repo

/unhold e2es are passing in the AWS provider repo

/unhold
^^


Move dist from repo

Good repositories avoid generated content. There is many reasons for it, but I think main reasons is:

Some user may make mistake and fix for dist content.
dist makes diffs much harder to read

So we should add dist to .gitignore

Thanks @ai, you're right! I've just updated ignore files

You need to delete dist first.
git rm -R dist/

of course :) it's done.


Fixes for using node version 4.

Are there unit tests that need updating as well?

Thanks for fixing!

@benbp don't think I need to add more unit tests. I could for createSalt but createHash already has tests. I will if requested.

Oh I guess I meant did we need to update existing ones? If not, great. :+1: when the build is passing.

+1

Even it has been merged, I still want to raise a :+1: , glad we have move to Node 4 ,thank for your wok @rolandpoulter


start stop instance

Towards https://github.com/giantswarm/giantswarm/issues/1886
CLI utility and library for spinning and setting up access to a remote minikube. This build uses it: https://circleci.com/gh/giantswarm/shipyard/17

I am not into the bigger picture here. The PR is quite overwhelming for me because I miss all the details. I think it makes sense to use https://github.com/spf13/cobra for the CLI magic. When building CLI tools it would be cool to align the project structure with others. Maybe check https://github.com/giantswarm/opsctl. I am not sure my review is useful besides that.

I think it makes sense to use https://github.com/spf13/cobra for the CLI magic

mmm i'm afraid of pulling too much deps, we are using just two flags atm, maybe when it gets a bit hairier? :)

maybe when it gets a bit hairier?

Roll how you prefer.

Thanks Tim!!

@xh3b4sd i think the better example now is https://github.com/giantswarm/shipyard/blob/master/pkg/shipyard/shipyard_test.go#L24, a shipyard instance is created and later used in https://github.com/giantswarm/shipyard/blob/master/pkg/shipyard/shipyard_test.go#L47, only the aws credentials environment variables are needed. Using it this way there's no need to change circle.yaml files and it integrates with architect, the tests are run as part of the existing suites.
Let me know if i can help setting it up.

This looks like I have to integrate it into my golang test which implies running all the AWS magic even when I am testing on my local laptop, where I have minikube already. Do I see this right? This is actually why I asked to a circle config and architect integration because I want to keep the local testing as easy as possible and when pushing to Github the AWS magic should be used automatically. I do not want to rewrite my go tests. That is not a nice integration IMO.

mmm using the same tests with both the local and the remote minikube without changing the tests can be challenging. I think that, with a small change in the tests, we could use both, just checking for env vars set in the CI https://circleci.com/docs/1.0/environment-variables/#basics in that case if they exists use the remote minikube, if not use the local one. This way architect and circle.yaml files stay the same.
I can include that code in shipyard too in order to make things easier.


ERROR: Files could not be pruned

I'm getting this error when I'm trying build both https://github.com/ungoogled-software/ungoogled-chromium-windows/commit/33b93e3e655c64115c636097aa2bf39f8db9b6da and https://github.com/ungoogled-software/ungoogled-chromium-windows/commit/27fc223e7cf87bc4b2245afeb495bd52288b9ae3 while it attempts to Extracting archive:\build\download_cache\node-v16.17.0-win-x64.zip.
This is the full error-

ERROR: Files could not be pruned: {'third_party/harfbuzz-ng/src/test/fuzzing/fonts/clusterfuzz-testcase-minimized-hb-repacker-fuzzer-4579249263345664', 'third_party/harfbuzz-ng/src/test/fuzzing/fonts/clusterfuzz-testcase-minimized-hb-subset-fuzzer-4552226966994944', 'third_party/harfbuzz-ng/src/test/fuzzing/fonts/clusterfuzz-testcase-minimized-hb-subset-fuzzer-5029952234586112', 'third_party/harfbuzz-ng/src/test/fuzzing/fonts/clusterfuzz-testcase-minimized-hb-subset-fuzzer-5458896606855168', 'third_party/harfbuzz-ng/src/test/fuzzing/fonts/clusterfuzz-testcase-minimized-hb-subset-fuzzer-6187272924692480', 'third_party/dawn/third_party/gn/webgpu-cts/cache.tar.gz', 'third_party/harfbuzz-ng/src/test/fuzzing/fonts/clusterfuzz-testcase-minimized-hb-subset-fuzzer-5191907895279616', 'third_party/boringssl/src/pki/testdata/ssl/certificates/cronet-quic-leaf-cert.key', 'third_party/harfbuzz-ng/src/test/fuzzing/fonts/clusterfuzz-testcase-minimized-hb-subset-fuzzer-5419002026131456', 'third_party/harfbuzz-ng/src/test/fuzzing/fonts/clusterfuzz-testcase-minimized-hb-repacker-fuzzer-6490945267564544', 'third_party/harfbuzz-ng/src/test/fuzzing/fonts/clusterfuzz-testcase-minimized-hb-draw-fuzzer-5692635449524224', 'third_party/harfbuzz-ng/src/test/fuzzing/fonts/clusterfuzz-testcase-minimized-hb-shape-fuzzer-4787105656864768', 'third_party/harfbuzz-ng/src/test/fuzzing/fonts/clusterfuzz-testcase-minimized-hb-repacker-fuzzer-6697168080338944', 'chrome/test/enterprise/e2e/connector/common/demo_agent/agent.zip', 'third_party/flatbuffers/src/tests/alignment_test_after_fix.bin', 'third_party/devtools-frontend/src/test/unittests/fixtures/traces/mainWasm_profile.json.gz', 'third_party/vulkan_memory_allocator/bin/Shader.frag.spv', 'third_party/vulkan_memory_allocator/bin/VmaSample_Release_vs2019.exe', 'third_party/harfbuzz-ng/src/test/fuzzing/fonts/clusterfuzz-testcase-minimized-hb-subset-fuzzer-5842152921628672', 'third_party/flatbuffers/src/tests/alignment_test_before_fix.bin', 'third_party/harfbuzz-ng/src/test/fuzzing/fonts/clusterfuzz-testcase-minimized-hb-subset-fuzzer-6032126569742336', 'third_party/harfbuzz-ng/src/test/fuzzing/fonts/clusterfuzz-testcase-minimized-hb-shape-fuzzer-5114131137822720', 'net/data/ssl/certificates/cronet-quic-leaf-cert.key', 'third_party/vulkan_memory_allocator/bin/Shader.vert.spv'}

Seems to be a problem with the files I had, I deleted and git cloned the repo again and it works fine so far


bluetooth: host: Runtime HCI_LE_Create_Connection timeout

Is your enhancement proposal related to a problem? Please describe.
I have a common GATT central application that is expected to connect to many different peripheral applications. Consider 2 applications, one which advertises at 10Hz, another at 1Hz.
A timeout of 500ms is likely sufficient for connecting to application A, while 5s may be desired for application B.
Waiting for 5 seconds for application A is a waste of time, as if you haven't connected in 500ms, you likely wont connect in 5 seconds (device has probably moved out of range).
Time spent initiating a connection that will never happen is time that you are not scanning for useful information from other devices.
Describe the solution you'd like
An additional interface bt_conn_create_timeout or similar, that updates the timeout for initiating connections.
Describe alternatives you've considered
Currently I am implementing this functionality by waiting on a semaphore that is given in the connection callback, where the wait duration is my desired timeout. Connection is then canceled through bt_conn_disconnect if the semaphore times out.

I will have a go at implementing this

@JordanYates Have you considered to use a whitelist and bt_conn_create_aute_le for this?

@joerchan I hadn't previously, but it does not really fit the model of what we are trying to achieve.
We aren't interested in connecting to ANY device running application A or B. There can easily be over 50 devices  within bluetooth range, none of which I want to connect to. At some point in time, in response to an external command or as a result of the data that one particular device has sent, I want to connect to a specific device. Using the whitelists appears to add overhead without solving my actual problem, which is receiving a timely response that I can't connect to a device. Sometimes there is a user (often me) in the loop of this process, making the timely response more important.


Add seed to VI test

Adds a seed to a VI test that's failing non-deterministically.

Nevermind, @devmotion has this in #1134.


Trojan in launch exe?

Windows defender marks and quarantines LaunchParadoxModManager.exe for containing the trojan Wacatac.

Just checked the other issues and found the other one about this, opening with the bat seems to work


The inferred type of 'createBuiltMeshHTTPHandler' cannot be named without a reference to ....

When running codegen to create typescript client, I am getting one error, and that is:
The inferred type of 'createBuiltMeshHTTPHandler' cannot be named without a reference to<EMAIL_ADDRESS>This is likely not portable. A type annotation is necessary.
🎁 error
🎁 error 5676 export function createBuiltMeshHTTPHandler() {
🎁 error                      ~~~~~~~~~~~~~~~~~~~~~~~~~~
🎁 error

Then I go to .graphclient/index.ts and find this piece of code:
export function createBuiltMeshHTTPHandler() {
  return createMeshHTTPHandler({
    baseDir,
    getBuiltMesh: getBuiltGraphClient,
    rawServeConfig: undefined
  })
}

And I annotate it with:
import type { MeshHandler, YamlConfig } from '@graphql-mesh/types'
....
export function createBuiltMeshHTTPHandler(): MeshHandler {
  return createMeshHTTPHandler({
    baseDir,
    getBuiltMesh: getBuiltGraphClient,
    rawServeConfig: undefined
  })
}

Error is no more. I'd be happy to contribute to the project and fix this myself, but I don't have time to dig deep, so if you can give me some pointers where to look I can do it.

Please create a reproduction on CodeSandbox or StackBlitz. Thanks!

Closing the issue for now. Feel free to create a new one with a reproduction if you still have the problem. Thanks.


feat(Shape): add Shape component

Fixes #196

@layershifter Could you guide me on this one, if possible? Thanks!

Codecov Report

Merging #1323 into master will decrease coverage by -0.21%.
The diff coverage is 16.66%.

@@            Coverage Diff             @@
##           master    #1323      +/-   ##
==========================================
- Coverage   99.75%   99.54%   -0.21%
==========================================
  Files         140      141       +1
  Lines        2402     2408       +6
==========================================
+ Hits         2396     2397       +1
- Misses          6       11       +5

Impacted Files
Coverage Δ

src/modules/Shape/Shape.js
16.66% <16.66%> (ø)

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 17a5775...71c8431. Read the comment docs.

@layershifter Could you give me some feedback when you get a chance?

Let's start from Shape sub-components and some props:
<Shape.Side active header>Did you know? This side starts visible.</Shape.Side>
//
Did you know? This side starts visible.

<Shape.Side>
  <Shape.Content center>4</Shape.Content>
</Shape.Side>

<Shape.Side>
  <Shape.Content center='4' />
</Shape.Side>

<Shape.Side center content='4' />

//

//

//
4

//

//

<Shape cube />
//

<Shape type='auto' />
//

<Shape type='square' />
//

<Shape type='auto' />
//

@layershifter Can we come up with a list of sub-components and props? I think that would make this easier!

I suggest you an update in readme.md marking shape as supported.

@tarang9211 please see https://github.com/Semantic-Org/Semantic-UI-React/pull/1323#issuecomment-281065710 for subcomponents and props.

@levithomason yup, will have a look. I'm just swamped in the week, will be closing out my open issues by Saturday.

Any updates on this ? :eyes:

Hello everyone! Would love to see this component make it into the react implementation! Thank you all for your hard work ! Making the world a prettier place!

Note, I've just rebased this branch to the latest master to keep it from getting to stale.

Has this been forgotten ?

It appears @tarang9211 has gotten busy.  We'll gladly take another PR picking up where this one left off.

Thank you for such a quick response, I pulled the fork from tarang but unless i'm not understanding something i don't think we were that close to completion.
For better understanding of the library and its component, i'll try and give it a shot. Really just at initial stage but i got it to show in the docs with example having shape components and 2 sub components being side and content.
https://github.com/madwill/Semantic-UI-React/commit/12dc99dd09136a64102d82ac7527bda9dc9ffa25
I confess having currently no idea how to manipulate it to get the semantic ui shape behavior. I'll dig into the official code for it .shape() and .shape('flip X'); as well as other components using javascript for behavior within Semantic-UI-React to see how it already handle such cases.
I can not make promises as i don't feel confident enough right now but if you could point to a component you feel would be a good example of javascript manipulation without jquery it it may accelerate / make it possible.
Thank you very much

Hey guys, I'm really sorry for being away! Been extremely busy with work and personal commitments. Happy to help whenever needed.

Turns out is a surprisingly complex component :
https://github.com/Semantic-Org/Semantic-UI/blob/master/src/definitions/modules/shape.js
The way it invokes methods, using query arguments in order to allow for a .shape('flip up') makes it so that its not just re-implementation, it would require some re-engineering.
With the 6 types of animations requiring adjustments based on component size, its a longer task than expected.
I get why it stalled. Jquery less semantic UI is an ambitious project considering its implementation. It would even say it has to lose a bit of its "semantic-ism".
This feels like a lot for a newcomer so i'll let my head cool down.   I don't think i'll be the one creating the pull request.
Perhaps the flip / stage section will become a bit more clear as i take a step back.

What if we simplify it a bit ? for the sake of quick implement-ability.
Top component (shape) would have 2 props being side and direction

    /** Currently displayed side, */
    side: PropTypes.number,

     /** Direction the shape should rotate. */
    direction: PropTypes.oneOf(['right', 'left', 'up', 'down', 'over', 'back']),

Usage would be something like this
       <Shape type='auto' side={1} direction='left'>
          <Shape.Side >
            <Image src='https://react.semantic-ui.com/assets/images/avatar/large/matthew.png' />
          </Shape.Side>
          <Shape.Side>
            <Image src='https://react.semantic-ui.com/assets/images/avatar/large/elliot.jpg' />
          </Shape.Side>
        </Shape>

User would have to change side by himself with his own controls and state.
We would trigger animation in a life cycle function such as
 componentWillReceiveProps(nextProps) {
    if (nextProps.side !== this.props.side) {
      this.startAnimating()
    }
  }

User would still have the liberty to change the "direction" props if that's something someone would want and i would have to make a big "switch" statement in startAnimating to create the correct animation depending on our props and type
What would be the point of
<Shape.Content center='4' />
It feels like an extra layer that could be avoided, if you look at usage within semantic-ui itself there is no side-content divs. Content is directly written in the side div.

This side starts visible.

This is yet another side

This is the last side

Let me know what you think

@madwill Thanks for a detailed suggestions :+1:
My initial idea was about componentWillReceiveProps, too. I'll be glad to see changes that realizing this idea.

I take the PR on myself, #1751


chore: update emails

update email cases for digital credentials to only be CC the dc team when production is originally submitted or added

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 71.24%.  Consider uploading reports for the commit 7b79578 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #1189      +/-   ##
==========================================
+ Coverage   71.23%   71.24%   +0.01%
==========================================
  Files         301      301
  Lines       10397    10396       -1
  Branches     2417     2417
==========================================
+ Hits         7406     7407       +1
+ Misses       2991     2989       -2

Flag
Coverage Δ

unitests
71.24% <100.00%> (+0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


FP=1 set in ACK even no pending message available for end device

We have RCP design where host and radio controller communicated over SPI through spinel interface.
We have observed, Frame pending bit set to 1 in ACK of reply received from end-device.
During Debugging, Host added entry in SAM table when it has message for child. Host is removed entry from SAM table after received ACK of sent message. Due to high latency from received ACK to host and send remove entry from SAM table command, Radio received response from child before removing entry from SAM table and that is why ACK has frame pending bit is set.

Expected behavior : Host should clear entry from SAM table before received reply from end device.
Proposed solution:  Handling remove entry from SAM table from Sub mac itself instead of handling from host.

In the scenario you describe above, if the SED receives an ACK with FP=1 and the parent does not have any messages queued, the host should send an empty data frame indicating that it has no more messages queued.
For your proposed solution, if the radio clears the SAM table entry and the host has more messages queued, this will cause the SED to stop polling for messages and cause high latency.

Yes,  you are correct. Host sent empty data frame indicating that is has no more messages queued but my concern is for RCP design regarding to clear quickly entry from SAM table.
I think it is not good to set FP=1 as we don't have messages in queue. It is just due to entry is removed lately.  Can't we have optimize on host side or some way to clear entry quickly?

The radio co-processor needs explicit indication from the host that there are no more frames pending.
Maybe that explicit signal can just be the host setting/clearing the FP bit in the data frame that it is sending via indirect transmission. If the radio co-processor is sending an indirect transmission with FP bit clear, then it can remove the Source Address Table entry.
@abtink , thoughts?


Cannot log after tests are done. Did you forget to wait for something async in your test?

I get this error when i run test:
import { cleanup, render } from "@testing-library/react";
import { createMemoryHistory } from "history";
import React from "react";
import { Router } from "react-router-dom";
import TestPublications from "../Post.TestPublications";

afterEach(cleanup);

const setPublicationId = () => {

};

test('Async function', async () => {
  const history = createMemoryHistory();
  const { getByText } = render(
    <Router history={history}>
      <TestPublications setPublicationId={setPublicationId} />
    </Router>
  );
});

the test passes but i get message : "Cannot log after tests are done. Did you forget to wait for something async in your test?"

@fardin03, why do you think this is remx issue?

@fardin03, why do you think this is remx issue?

@andykog  sorry I'm new here and this is not what i intended.

@fardin03, no problem, closing this issue as not related to remx. You may check here the advises on how to fix this.


fr/es translations for "thumbnailsView"

Per #827
Not in this PR but I could also add new strings and french and spanish translations for:
"load": "Load"
"load": "Charger"
"load": "Cargar"

(per #729)
and also
"dropToLoad": "Drop to Load Manifest"
"dropToLoad": "Déposer ici pour ajouter l'objet"
"dropToLoad": "Soltar aquí para añadir el objeto"

@regisrob Those extra translations look good - can you please add before I merge? Thanks!

Ok ;) I've just added the new strings


Keep tile ratio

Hi!
Thanks for the awesome plugin! I have one issue, I'd like to have tiles with a fixed ratio - small square (1x1), wide (1x2) and tall (2x1). A grid should take the whole viewport width and tiles should keep the ratio as their width changes. Right now my tiles are sometimes longer, sometimes shorter than others (e.g. wide tile is not exactly 2x wider than square one).
Is there any solution for that?

Hi
you can set it with fixSize param:
 //fixSize: 0, resize but keep ratio = no fill gap;

example:
wall.reset({
 ...
 fixSize: 0
 ...
})

Best

Not always working, see #234


"following": false

Hi
This is not a issue , but want to understand the API , In this API Strucutre https://github.com/gothinkster/realworld/tree/master/api
In Profile API what does following : false means ?
{
"profile": {
"username": "jake",
"bio": "I work at statefarm",
"image": "https://static.productionready.io/images/smiley-cyrus.jpg",
"following": false
}
}

following:false

means that the authenticated user is not following the profile you are viewing. That following references whether or not you are following that profile.

Hi
Thanks @Cameron-C-Chapman  .Just to make sure I understood, the authenticated user who is requesting GET profile , is he access this user ( in thi case jake) .
What about if the user access his own GET profile API ?

If you view your own profile you will see following: false since you can't follow yourself. If you view a different users profile you will see if you are or are not following that user.
I'm going to go ahead and close this. If you have more questions or notice any problems with the API feel free to open other issues or I can open this one back up if it's related.


[Enhancement] Provide ability to customize different material textfield/Entry properties

Summary
Currently it is relatively hard to change the cursor color of an Entry, either needs custom renderer or digging into platform styles. It would be nice to have an effect (applied in xaml as in code) that enables the users to change the cursor color inside an Entry on all target platforms.
Also provide the ability to customize different properties that material textfield offers directly from xaml.
API Changes
Something like:
xct:CursorEffect.Color="red"
xct:CursorEffect.IndicatorUnFocusedColor="red"
Intended Use Case
Less use of custom renderers, more easier to change the styling dynamically.
Preliminary List of properties

Cursor color.
Activation indicator when not focused (when focused I believe it takes PlaceholderColor).

Reference
https://material.io/components/text-fields
Who Will Do The Work?

[ ] I am willing to take this on myself
[ ] Just putting this out there for someone to pick up

Thanks! However, we are no longer adding new features to Xamarin Community Toolkit, focusing on the .NET MAUI Community Toolkit.
Please open a New Feature Discussion to implement this feature in the .NET MAUI Community Toolkit.
I've posted more information about the Future Of Xamarin Community Toolkit here: https://devblogs.microsoft.com/xamarin/the-future-of-xamarin-community-toolkit/?WT.mc_id=mobile-0000-bramin


pod/extend pod jobs

In order to get out of the Prow CI, we have to port some other jobs to
Zuul pods.
In order to avoid code ducplication, some changes were needed in the
pre-commit.yml playbook - for instance, it got renamed with a more
generic name, and it's now getting the make target directly from the
job definition.
In addition, two checks that were in pre-commit are now standalone:

check_zuul_files
check_k8s_snippets_comment

This ensures we get consistent logging and related artifacts in Zuul, as
well as ensuring we trigger them only when needed (those are small jobs,
but still - let's save resources)
As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running

/test all

ansible-test is failing (hence the voting: false) in zuul env while it's working in Prow, and locally. I'll need to debug this in order to switch that job to "voting". Seems a checksum isn't matching :(.

recheck

ansible-test is failing (hence the voting: false) in zuul env while it's working in Prow, and locally. I'll need to debug this in order to switch that job to "voting". Seems a checksum isn't matching :(.

yeah something in /tmp/logs/ci_script_012_run_using_chdir.log is different, size of the file too. I guess something different in the environments but I'll run a recheck and check that file live for clues.

If we are getting rid of the prow job and the file looks fine we should just update the checksum

Thank you @lewisdenny for your check! I could make tests converge, so we're green on both end!

/lgtm

/lgtm

/approve


Add Enums type as a select

Enums would be nice to have select instead of normal input

done with v0.8.0


Fix #6346  UI - convert all cron editor jsx file to tsx file

Describe your changes :

I worked on converting the cron editor JSX file to a TSX file and added types for the same.

Type of change :

[x] Improvement

Frontend Preview (Screenshots) :

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers
@open-metadata/ui

@Sachin-chaurasiya Since it's bigger change let's guard this PR with some unit tests. Basic and positive will work

Yes, we already have unit tests.


Add config annotations for tables and schemas

Codecov Report

Merging #2746 into master will decrease coverage by 0.17%.
The diff coverage is 52.97%.

@@            Coverage Diff             @@
##           master    #2746      +/-   ##
==========================================
- Coverage   68.53%   68.35%   -0.18%
==========================================
  Files         876      893      +17
  Lines       42288    42876     +588
  Branches     5748     5832      +84
==========================================
+ Hits        28980    29306     +326
- Misses      11445    11625     +180
- Partials     1863     1945      +82

Impacted Files
Coverage Δ

.../com/linkedin/pinot/common/config/QuotaConfig.java
44% <ø> (+8%)
:arrow_up:

.../linkedin/pinot/common/config/TableTaskConfig.java
31.57% <ø> (ø)
:arrow_up:

.../com/linkedin/pinot/common/data/TimeFieldSpec.java
92.59% <ø> (ø)
:arrow_up:

...inkedin/pinot/common/config/TableCustomConfig.java
16.12% <ø> (ø)
:arrow_up:

...com/linkedin/pinot/common/config/TenantConfig.java
25.71% <ø> (+2.85%)
:arrow_up:

...om/linkedin/pinot/common/config/RoutingConfig.java
45.45% <ø> (ø)
:arrow_up:

...inot/common/config/ReplicaGroupStrategyConfig.java
50% <ø> (ø)
:arrow_up:

...n/config/SegmentsValidationAndRetentionConfig.java
46.59% <ø> (+1.13%)
:arrow_up:

.../linkedin/pinot/common/data/DateTimeFieldSpec.java
94.11% <ø> (ø)
:arrow_up:

...din/pinot/common/config/ColumnPartitionConfig.java
62.79% <0%> (-4.71%)
:arrow_down:

... and 69 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 769b55a...aee2adf. Read the comment docs.


Is it possible to remember gallery features and calculate the distance ?

Hello again.
Thank you for supporting me every time.
I've used this model so far and it's quite amazing.
But as I wrote codes like #3 you helped me before, the process time takes a bit long when there are over thousands of query images because it calculate all gallery features each time when one of query features.
So now, I want to write codes like

calculate gallery features
calculate query features
compute the distances between 1 and 2

Is it possible to write codes like this?
Thank you

Yes.
What you have proposed is the correct approach, and the features of one test sample should be calculated only once.
This is also how the existing code is implemented. Please check this snippet for more details.
Xingyang

Thank you for your prompt response.
I'll write codes , refering to your instructions.
Thank you

@kazuki-can
Hi. I am glad that you were able to run this project
But I have the problem of incompatibility between the versions and I can't run Google colab
Can you guide me?


Updated readme

please feel free to add additional notes if I missed anything.

I went ahead and merged without approval because I wanted to be sure to have a completed readme in the repo


Growth Maturity and Decline Status Notes

To keep the metrics committee up to date we think our working updates to metrics definitions being available, along with status indicators, makes the process more transparent and enables broader engagement.
If you prefer us to create another branch we could do that in the future.

Thank you for the updates and thoughts. I agree to keep this in the master branch and branch out at this point.
Where you have implementations of the activity metrics, could you include screenshots and sample code that is used in Augur?
activity-metrics-list.md:
How does the proposed column contribute to the CHAOSS community? It appears to have meaning mainly within your project team.

  I did it directly on one of Sean's questions, but please let me know if there's a better way....

 I did it directly on one of Sean's questions, but please let me know if there's a better way....

I have been using comments and reviews on pull requests. I only change files if I think the change makes the suggested edits more suitable for merging.


Git is not working properly on slaves

docker run -it --rm openshift/jenkins-slave-nodejs-centos7 bash
$ git clone https://github.com/tnozicka/nodejs-ex.git /home/jenkins/nodejs-ex

ends up with error:
Cloning into '/home/jenkins/nodejs-ex'...
remote: Counting objects: 393, done.
remote: Total 393 (delta 0), reused 0 (delta 0), pack-reused 393
Receiving objects: 100% (393/393), 119.53 KiB | 0 bytes/s, done.
Resolving deltas: 100% (161/161), done.
fatal: unable to look up current user in the passwd file: no such user
Unexpected end of command stream

Same for maven one:
docker run -it --rm openshift/jenkins-slave-maven-centos7 bash
$ git clone https://github.com/tnozicka/nodejs-ex.git /home/jenkins/nodejs-ex

Cloning into '/home/jenkins/nodejs-ex'...
remote: Counting objects: 393, done.
remote: Total 393 (delta 0), reused 0 (delta 0), pack-reused 393
Receiving objects: 100% (393/393), 119.53 KiB | 0 bytes/s, done.
Resolving deltas: 100% (161/161), done.
fatal: unable to look up current user in the passwd file: no such user
bash-4.2$ Unexpected end of command stream

Surprisingly, this is not a problem when running Jenkins on oc cluster up. When using ADB/CDK all attempts to checkout repositories on slaves fail with these errors.
(There is probably a different policy to get UID from or something like that.)
I am not sure about the proper solution here, but we had a hack in our images before summit to fix it: https://github.com/tnozicka/dockerfiles/blob/master/openshift-nodejs-builder/Dockerfile#L3
Running the commands like
GIT_COMMITTER_NAME='<unknown>' GIT_COMMITTER_EMAIL='<unknown>' git clone https://github.com/tnozicka/nodejs-ex.git /home/jenkins/nodejs-ex

helps.
@rupalibehera @bparees

The image isn't intended to be run that way, the problem is this:
$ docker run -it --rm openshift/jenkins-slave-nodejs-centos7 bash
bash-4.2$ whoami
whoami: cannot find name for user ID 1001

the slave startup script that is invoked when you use this image as a proper slave uses nss wrapper to setup a fake passwd file that ensures the uid can be resolved.
if you want to use this image manually, you need to do the same steps:
bash-4.2$ USER_ID=$(id -u)
bash-4.2$ GROUP_ID=$(id -g)
bash-4.2$  NSS_WRAPPER_PASSWD=/tmp/nss_passwd
bash-4.2$     NSS_WRAPPER_GROUP=/etc/group
bash-4.2$
bash-4.2$     cat /etc/passwd | sed -e 's/^default:/builder:/' > $NSS_WRAPPER_PASSWD
bash-4.2$
bash-4.2$     echo "default:x:${USER_ID}:${GROUP_ID}:Default Application User:${HOME}:/sbin/nologin" >> $NSS_WRAPPER_PASSWD
bash-4.2$
bash-4.2$     export NSS_WRAPPER_PASSWD
bash-4.2$     export NSS_WRAPPER_GROUP
bash-4.2$     export LD_PRELOAD=libnss_wrapper.so

which results in:
bash-4.2$ whoami
default

@bparees This was just an example, but it happens to real slaves as well. The error is exactly the same. It only manifests in ADB because of the admission policy.

if you want to use this image manually, you need to do the same steps:

Actually the script was called. I did not change the the entrypoint at all, I changed only command. And entrypoint sources this nss wrapping script.
The problem is the condition. You are refusing to generate the entry for id=1001 but the entry is not there!
That's most probably why it's not working in ADB because it allows you to run it as the default user 1001 as specified in your Dockerfile. There is no entry in /etc/passwd for user 1001 and you are refusing to generate it.

@tnozicka yeah normally our base images define uid 1001 as a real use, but that wasn't the case for the slave images.  i'll remove the conditional.


[Bug] When all the rows are hidden the user must not be displayed with number of records or pages in the Deploy mode

Steps to reproduce:

Add a table
Hide all column
Now click on deploy
and observe

Observation:
It is observed that the "Number of records and pages" are displayed to the user
Expectation:
It is expected not to show the records when all the rows are hidden which leads to confusion by showing the details

@prapullac hiding all columns does not mean the data set is empty. According to me, the behaviour is right, since its the columns which are hidden and not the rows!

@somangshu: Since the data is hidden, showing the records or pagination would lead the user to confusion and search for the data in the table rather this is a rare scenario where all the columns are hidden which ends have data not being added.

I would add an error if all the columns are hidden in the edit mode, but anything extra seems not worth it.
Can't this be done by the dev, by an if condition in the visible property of the table which devs can define? Like if data is empty then show, else hide or any other rules. Don't think we need to solve this.
@somangshu

@dilippitchika today the header show/hide option is not JS convertible, we should definitely provide that and leave the onus of this error on the dev

Hey @somangshu was referring to the isVisible property for the table. Dev can hide the table if the data is empty. Won't that solve it?

Hiding the whole component from the viewport might not solve users problem because the user is still not informed wether there is an error or not.
also to the original point, I feel hiding all the columns does not mean that the data set is empty and hence this is actually a non-issue. However, providing the dev an option to choose to do this is good enough I feel


state.can docs update

Add warning that state.can will execute transition guards
Updates state.can examples to use the full event object due to valid confusion in the discord discussion below and the fact that TypeScript demands the full event object.

Based on the discussion at https://discord.com/channels/795785288994652170/1022469795582591057/1022481693929635880

Just a couple of readability suggestions

I'm so used to Prettier.


maxMessagePublishBufferSizeInMB does not consider the messages handled by Protocol Handler

Is your enhancement request related to a problem? Please describe.
We use maxMessagePublishBufferSizeInMB to limit the direct memory usage for reading messages from producers, default value is 1/2 of direct memory.
If we enable any Protocol Handler in broker such as KoP, the direct memory to buffer the pending messages from Kafka producers is not limited by maxMessagePublishBufferSizeInMB. And if we have Pulsar producers and Kafka producers sending message together with higher throughput than BookKeeper IO, broker OOM will happen.
Although we could add another configration in KoP to limit the direct memory usage for KoP, we could not use the entire memory space since we will have two separate configration for pulsar and KoP.
Describe the solution you'd like
To make effective use of memory, using a LongAdder in BrokerService to record the current pending bytes for entire broker, include both pulsar pending messages and Protocol Handler's pending messages.
When handling producer message, we could check if  pending bytes over maxMessagePublishBufferSizeInMB and stop reading from producer both in broker and Protocol Handler side.

I want to work on it. Thx

@gaozhangmin It should be better to start with a PIP, to make everyone on the same page for this issue.

@gaozhangmin It should be better to start with a PIP, to make everyone on the same page for this issue.

OK, I will create a new PIP later.


Fix Llama Benchmarking 8b f16 tests to run on presubmit

Modify all the changes merged into the sharktank repo to make Llama Benchmarking test work.

@sogartar fyi


Race condition in msm/sort.cuh

There is a race condition between lines 267 and 278 in the msm/sort.cuh file. If a warp with a larger warpid executes line 278 before the warp with warpid=0 executes line 267, the calculation result will be wrong. In fact, on the ampere architecture, due to the scheduling strategy of the warp scheduler, it is impossible for warp (id>0) to execute line 278 before warp(id=0) executing line 278 and therefore the test always passes. However, there is indeed a logical error.

Thanks! I have to ask, did you spot it by eyeballing or is there some tool that spots potential race conditions? If the former, I have to praise the keenness, and if the latter, I'd like to know which one is it:-) Thanks again!

I spotted it by eyeballing

In addition, to correct my previous statement, in fact, on the Ampere architecture, warps in the same block (1024 threads) may not be scheduled by the same warp scheduler, so the scheduling strategy of the warp scheduler may not necessarily guarantee that warps (id>0) will update the counters after warps (id=0) read them.


Problem loading model - 'Failed to load function from bytecode:' Error while loading model file

Hi,
So i'm currently trying to load a network model via torch on an Nvidia TX1. When I try to load the model

net = torch.load('modelfile.t7','ascii')

I get the following error:

The model loads fine on my Ubuntu 14.04 desktop, so I tried loading the same model, converting it to binary and then trying to load the converted file

net = torch.load('modelfile.bin')

But i still get a similar error:

I've noticed that a few people have had the same errors in the past but most people seem to have been able to get past this by using an 'ascii' version of the model since it's platform independent (?). I seem to have had no luck with that. The other set of individuals who faced this problem were on a 32bit system. But my Nvidia TX1 is currently running on Ubuntu 16.04 (64bit).
For anyone willing to recreate these results:
I installed JetPack (JetPack-L4T-2.3.1-linux-x64.run) and verified that my installation of CUDA 8.0 and OpenCV is functional.
For Torch, I used dusty-nv's installation script
https://github.com/dusty-nv/jetson-reinforcement
The installation script in particular is https://github.com/dusty-nv/jetson-reinforcement/blob/master/CMakePreBuild.sh
It all looks pretty straightforward.
The model file in specific is https://s3.amazonaws.com/mc-cnn/net_kitti_fast_-a_train_all.t7
Any tips on how to fix this problem is gladly appreciated. If anyone has any ideas on how I can tweak the model on my desktop machine to make it work here I'd love to hear it.
Thanks in advance,
Shreyas

Shreyas, I don't have a Jetson nearby, so I'm afraid I can't really help you much. If you find a solution please let me know.

Hi @jzbontar thanks for responding to this. So I looked into it some more and it seems as though only the fast models suffer from this problem. The slow models (I tried kitti & kitti2015) seem to load just fine.
Was there a difference in the way these models were exported? I looked through the code and I noticed both were saved as ascii models.

Hi,
So I managed to figure out (at least somewhat) what was causing the problem and fix it.
So on taking a closer look at the fast architecture model specification and ASCII model, I noticed what seems to be a pointer to a file/function located on your computer [ @jzbontar ]. That seems to be what is making the ASCII file unreadable on the TX1. For a reason unknown to me, that slips through undetected on my desktop.
Model specification:
My hypothesis is that the accUpdateGradParameters : function : X is the cause of this problem.

Can also be verified by looking at the ASCII version of the fast architecture model(s)

The fix is to either load this model onto a computer that accepts it, remove those parameters and resave to an ASCII file or to just re-train these models from scratch.
Here is what the model specification looks like on a working model file:

If anyone is interested in freshly re-trained models that work on the TX1, I've attached links to them below:
KITTI 2012 Fast Architecture Models

net_kitti_fast_-a_train_all.t7

net_kitti_fast_-a_train_tr.t7
Thanks for the help and I hope this post helps anyone else stuck in the same boat.


Support multiple key management vault integration

Microsoft Azure support was our test & trail implementation for external key management but it is too closely coupled with core application. We should decouple (#506) and refactor to make it easier to integrate and support out key management vaults like AWS, Hashicorp Vault etc.

Hashicorp has done a fair amount of refactoring on this front, and has been merged. Any future vaults which need refactoring will have it done as part of their ticket.


Avoid importing package.json into bundle

To get access to the current version, deck.gl imports package.json.

Since our package.json has grown big, this adds a couple of unnecessary kilobytes to the bundle
Can also cause issues for users that redefine json loaders, see #1087

The best solution is probably to write our own small webpack plugin to generate a minimal version file from the info in package.json.

Because we already have the global version defined here, we should be able to use this instead of importing the package in core/lib/init
Another point, I think VERSION is too generic it might conflict with user code / libs, we should change it to DECK_VERSION or similar

Agreed. And we can do the same change in luma.gl, should give us some bundle size savings, every bit helps.


[5.9] Set tabnav to the tab with latest title update

Explanation: Added logic to select tabs when new tabs are added / removed
Scope: TabNavigator component only
Issue: rdar://108013117
Risk: Low
Testing:

All unit tests should still pass.
Verify that for documentation pages using TabNavs, this change doesn’t introduce regression

Reviewer: @mportiz08 @dobromir-hristov @marinaaisa
Original PR: https://github.com/apple/swift-docc-render/pull/601

@swift-ci test


Rolling Update of ServerPools

Let's say I deploy a cluster "daniel" in AWS with two ServerPools: " daniel.master" and "daniel.node-original" Later, I want to update the instance of the nodes.
Here's one way I can make this happen:

Add a third ServerPool ("daniel.node-new") to my profile
Change the MaxCount of the "daniel.node-original" to 0
kubicorn apply

The downside is that the original nodes are not replaced gradually by new ones. One way to address this is to have kubicorn gradually scale the "original" ServerPool down and the "new" one up. (Reminds me of the rolling-update behavior of ReplicationController).
Is this something within scope for kubicorn? If so, I'll sketch out a design.

We would love a sketch if at all possible, I think this would be a huge win for the project.

Thanks for the feedback!
What I have in mind right now is to shamelessly crib from how an update to a Deployment is applied. Simply put, the Deployment spec is edited, and the Deployment controller orchestrates the rolling update itself.
Because kubicorn is a tool rather than a service, an explicit apply step will need to follow the edit step.
I'll keep updating this issue with more details. I'll also give some thought to how to add this capability incrementally to different clouds.

Rolling ServerPool Update Design Proposal
Motivation
Replace instances in one ServerPool with instances from another incrementally, maintaining constant cluster capacity while instances are replaced. Replacing instances is necessary to change instance type or operating systems.
Process

kubicorn edit mycluster
1.a. Add a ServerPool with name equal to newPool
1.b. Set newPool.minCount and newPool.maxCount equal to 0
kubicorn apply --rolling-update --from oldPool --to newPool

This design is meant to use kubicorn's edit-apply workflow. The rolling update changes how apply works only on the two explicitly named ServerPools.
The end result is that oldPool is scaled to the original maxCount of newPool (typically 0), and newPool is scaled to the original maxCount of oldPool.
At each iteration of the rolling update algorirthm, the newPool is scaled up by one instance and the change reconciled; then the oldPool is scaled down by one instance and that change reconciled. If either reconcile step returns an error, the rolling update undoes its work: The oldPool is immediately scaled up to its original size and the change reconciled; the newPool is immediately scaled down to its original size and that change reconciled.
Known Limitations
The rolling update algorithm will ensure that the newPool is scaled to the correct size, but it cannot guarantee that the instances are healthy. That is because a ServerPool is currently reconciled without  checking the health of its instances.
Additional Ideas

It might be even more convenient if steps 1.a and 1.b could be done with commands, e.g. kubicorn edit --copy oldPool newPool and kubicorn edit --set newPool.minCount=0 --set newPool.maxCount=0, respectively.
The rolling update above has an implicit "batch size" of one, but this size can be exposed as an option.


multiarch images on docker hub

Do you want to request a feature or report a bug?
Feature
What did you expect to see?

Please create an image for ppc64le.  FYI, docker machine works well if ppc64le hardware isn't available:
https://developer.ibm.com/linuxonpower/2017/06/08/build-test-ppc64le-docker-images-intel/

I second this. For the time-being I build the image locally :(

I updated the title to one that makes a bit more sense.  We are still primarily interested in having a ppc64le image available as a part of any multiarch builds.

We already have a multi-arch image: https://hub.docker.com/_/traefik?tab=tags
The official image is built by Docker it-self with a special process (it's a not the basic Docker Hub process),  so we have to check the possibilities.
You find here our base repo to create the official image https://github.com/containous/traefik-library-image/
We already have an issue on this repo about the ppc64le topic (https://github.com/containous/traefik-library-image/issues/73).
So I will close this issue, I will invite you to follow the issue on the image repository https://github.com/containous/traefik-library-image/issues/73

Thank you.  I didn't realize.  It's a bit confusing since this repo is still getting updates:  https://hub.docker.com/r/containous/traefik
It might be helpful to put a quick blurb there pointing out where the official images are.

Never mind.  I see the blurb now.  I didn't notice when I previously scanned.


Parse build ios framework build mode from params

Description
Use getBuildInfo to parse all the extra options. Migrate off BundleBuilder and use the build system for all App.framework modes.
Related Issues

getBuildInfo will allow us to add null-safety options easier (PR forthcoming) https://github.com/flutter/flutter/pull/59773
Continue migration from BundleBuilder to assemble.

Tests
BuildIOSFrameworkCommand remains pretty unit-untestable at the moment.  Fortunately, the build_ios_framework_module_test integration already validates every expected output file for this command.  It passes on this PR.
Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

Actually I'm going to split this into two PRs: one for getBuildInfo, one for BundleBuilder.

getBuildInfo at https://github.com/flutter/flutter/pull/59874

BundleBuilder removal at https://github.com/flutter/flutter/pull/60041.


Defender for Identity Timestamp values do not match in Sentinel and Microsoft 365 Defender

Describe the bug
Timestamp column values for Defender for Identity do not match in Sentinel and in Microsoft 365 Defender
To Reproduce
Steps to reproduce the behavior:

Have a Sentinel workspace receiving events in Defender for Identity tables (e.g. IdentityQueryEvents or IdentityLogonEvents)
Go to Logs tab in Sentinel.
Take one event from IdentityQueryEvents or IdentityLogonEvents tables with:

union IdentityQueryEvents, IdentityLogonEvents
| summarize take_any(TimeGenerated, Timestamp, ReportId) by Type

Search for the same ReportId in Advanced Hunting in Microsoft 365 Defender with:

union IdentityQueryEvents, IdentityLogonEvents
| search "<found 1st ReportId in step 3>" or "<found 2nd ReportId in step 3>"

Observe Timestamp values might not match.

Expected behavior
Please, could Timestamp column have the same values in Sentinel and in Microsoft 365 Defender?
Additional context
Microsoft 365 Defender has an advantage over Sentinel for these events. Sentinel does not have the real value where an event might have happened.
The following 4 events where generated consecutively, a logon and a LDAP query are generated from one single action, and in Microsoft 365 Defender the Timestamp values show clearly this (the LDAP query is "generated" 1 second after the LogonSuccess).

BUT in Sentinel the Timestamp column does not have the same values. The order of the events is not kept. Thus, in Microsoft 365 Defender I can create a query that could match easily logon and LDAP query events, but in Sentinel I can't create it.

I would like Sentinel to show the real values from Defender for Identity, to be able to develop better detections rules, please.

Hi @ep3p, Thanks for flagging this issue, we will investigate this issue and get back to you with some updates by 20-10-2023. Thanks!

Hi @ep3p, just want to know, you are using the below (MicrosoftThreatProtection) data connector or any other standalone one?

Please let us know which data connector you are using. Thanks!

@v-sudkharat I believe that is the only connector to be able to receive Defender for Identity tables.
Yes, we are using that connector.

Hi @ep3p, Thanks for clarify. We are reaching out to data connector team. Once we receive an update from the team, we will update you. Thanks!

Hi @ep3p, we received the response from concern team, the Timestamp is populated by the “Timestamp” field as it appears in the data sent by M365D API. Could you please create a support case for this issue? so, team can investigate this issue and if required they can connect with you to get more details that are needed about the issue and assist you.
Please let us know once you raise a Support Case, so we can close this issue from GitHub. Thanks!

I have opened a support case, Support request number:<PHONE_NUMBER>003360
You may close this issue, thank you very much for helping us.

Hi @ep3p, Thanks for confirmation, closing this issue. If you still need support for this issue, feel free to re-open it any time. Thank you for your co-operation.


[victor] High verbosity when conda activate/deactivate their custom env

Context
Ref https://2i2c.freshdesk.com/a/tickets/607
It has been reported that lots of INFO messages get generated every time their user defined conda environment gets activated/deactivated, which is also every time they open a new terminal. Example below:

Proposal
No response
Updates and actions
I've been able to track this down to https://github.com/conda/conda/issues/6945, which from what I understand means:

the INFO messages are to inform the users about conda compiler being used and not system one
the INFO messages might be related to newer compiler versions
should have been solved by conda by not piping this meesages to the user when CONDA_BUILD is not set, which is true for the victor hub, but doesn't look to be working

Does anyone on the @2i2c-org/engineering team has any thoughts about fixing this? Thanks

Thanks a lot @pnasrat!

as the upstream bug suggests there isn't a plan to actually fix this.

They do say that The compilers only echo this info now when CONDA_BUILD is set to 1 in https://github.com/conda/conda/issues/6945#issuecomment-389773340
And according to these conda docs CONDA_BUILD is set always set to 1 during build. But I believe that's it, then the var appears to be unset.

cc @yuvipanda who was suggesting just downloading from mozilla in the most recent note as the state of snap in ubuntu means this is going to be non-trivial via apt.txt

@yuvipanda do you want to send your note in https://2i2c.freshdesk.com/a/tickets/527 as a ticket response, maybe?

I came back to this issue today and tried various ways to surpress the warning. Apparently, conda supports an env var called CONDA_QUIET, but only for conda > 4.2 and we're at conda 4.1.*.
It's not clear to me what is the file that we need to edit to redirect conda activate/deactivate output to /dev/null every time a new terminal is opened. If I add it to ~/.bashrc, it still shows because I believe it's set from somewhere else also, which takes precedence or gets called later (maybe from /srv/conda/etc/profile.d/conda.sh)?
Does anyone on the @2i2c-org/engineering has any advice about this one? Wee need to follow up to https://2i2c.freshdesk.com/a/tickets/607

So, there's an /etc/profile.d/activate-conda.sh that's referenced from /etc/profile where the notebook env gets activated each time a new terminal is opened.

Mmm... I do not think you will be able to do much on this one until it gets fixed upstream (which seems unlikely).

Yes, agree. I just wanted to explore how to give them the option to redirect the output to /dev/null when a new terminal is opened and the conda env activated, if they wished. But I don't think that's possible either... Is the answer to this one "we can't do anything about it? :/ "

Is the answer to this one "we can't do anything about it? :/ "

Unless someone else has further ideas (which do not add a lot of future pain), I think that is the case...

I'm closing this as I've updated https://2i2c.freshdesk.com/a/tickets/607 with the resolution here.


[BUG] build-docs.sh is failing to build local docs

Is this issue already tracked somewhere, or is this a new report?

[X] I've reviewed existing issues and couldn't find a duplicate for this problem.

Current Behavior
In trying to build docs locally with ./scripts/build-docs.sh I get a ModuleNotFoundError: No module named 'mkdocs.tests'.
This may be related to https://github.com/mkdocs/mkdocs/issues/3702
Expected Behavior
I expect build-docs.sh to generate html locally for the docs
Steps To Reproduce
I'm using the earthaccess-dev environment that I just ran mamba env update -f ci/environment-dev.yml
The current mkdocs associated versions are:
$ mamba list | grep mkdocs
mkdocs                    1.6.1              pyhd8ed1ab_0    conda-forge
mkdocs-autorefs           1.2.0              pyhd8ed1ab_1    conda-forge
mkdocs-get-deps           0.2.0              pyhd8ed1ab_0    conda-forge
mkdocs-jupyter            0.22.0             pyhd8ed1ab_0    conda-forge
mkdocs-material           8.5.11             pyhd8ed1ab_0    conda-forge
mkdocs-material-extensions 1.3.1              pyhd8ed1ab_0    conda-forge
mkdocs-redirects          1.2.1              pyhd8ed1ab_0    conda-forge
mkdocstrings              0.26.0             pyhd8ed1ab_0    conda-forge
mkdocstrings-python       1.10.9             pyhd8ed1ab_0    conda-forge

Running $ ./scripts/build-docs.sh
I get the following Traceback
Traceback (most recent call last):
  File "/home/apbarret/mambaforge/envs/earthaccess-dev/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/home/apbarret/mambaforge/envs/earthaccess-dev/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/home/apbarret/mambaforge/envs/earthaccess-dev/lib/python3.10/site-packages/mkdocs/__main__.py", line 370, in <module>
    cli()
  File "/home/apbarret/mambaforge/envs/earthaccess-dev/lib/python3.10/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/home/apbarret/mambaforge/envs/earthaccess-dev/lib/python3.10/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "/home/apbarret/mambaforge/envs/earthaccess-dev/lib/python3.10/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/apbarret/mambaforge/envs/earthaccess-dev/lib/python3.10/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/apbarret/mambaforge/envs/earthaccess-dev/lib/python3.10/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/home/apbarret/mambaforge/envs/earthaccess-dev/lib/python3.10/site-packages/mkdocs/__main__.py", line 285, in build_command
    cfg = config.load_config(**kwargs)
  File "/home/apbarret/mambaforge/envs/earthaccess-dev/lib/python3.10/site-packages/mkdocs/config/base.py", line 374, in load_config
    errors, warnings = cfg.validate()
  File "/home/apbarret/mambaforge/envs/earthaccess-dev/lib/python3.10/site-packages/mkdocs/config/base.py", line 231, in validate
    run_failed, run_warnings = self._validate()
  File "/home/apbarret/mambaforge/envs/earthaccess-dev/lib/python3.10/site-packages/mkdocs/config/base.py", line 188, in _validate
    self[key] = config_option.validate(value)
  File "/home/apbarret/mambaforge/envs/earthaccess-dev/lib/python3.10/site-packages/mkdocs/config/config_options.py", line 186, in validate
    return self.run_validation(value)
  File "/home/apbarret/mambaforge/envs/earthaccess-dev/lib/python3.10/site-packages/mkdocs/config/config_options.py", line 1065, in run_validation
    self.load_plugin_with_namespace(name, cfg)
  File "/home/apbarret/mambaforge/envs/earthaccess-dev/lib/python3.10/site-packages/mkdocs/config/config_options.py", line 1103, in load_plugin_with_namespace
    return (name, self.load_plugin(name, config))
  File "/home/apbarret/mambaforge/envs/earthaccess-dev/lib/python3.10/site-packages/mkdocs/config/config_options.py", line 1121, in load_plugin
    plugin_cls = self.installed_plugins[name].load()
  File "/home/apbarret/mambaforge/envs/earthaccess-dev/lib/python3.10/importlib/metadata/__init__.py", line 171, in load
    module = import_module(match.group('module'))
  File "/home/apbarret/mambaforge/envs/earthaccess-dev/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/home/apbarret/mambaforge/envs/earthaccess-dev/lib/python3.10/site-packages/mkdocs_jupyter/plugin.py", line 10, in <module>
    from mkdocs.tests.base import get_markdown_toc
ModuleNotFoundError: No module named 'mkdocs.tests'

Environment
- OS: Ubuntu 22.04.4 LTS
- Python: Python 3.10.14

Additional Context
No response

Try bumping mkdocs-jupyter to version 0.24.8, which I believe should get past this problem.

I think my issue was that I only updated my environment with ci/environment-dev.yml and did not do the poetry install step.  I thought that only applied to setting up the environment.  This may be moot once @MattF-NSIDC tears out Poetry.

I'll close this for now.


How to create multiple graphs for the same entity

Can't find any doc about how to create multiple graphs for the same entity ? Tried to duplicate the conf from the graph property, but it throws a configuration error

Hi,
Can you post the configuration you have tried?

I was wondering if this is possible and if it even makes sense? For example I have a process that can be of three different types. For Type A of the process I need to follow a set of transitions, for Type B I want it to follow a certain set of transitions, etc. This would be one table I suppose with a FK to a type table. How this library would map to that model though I do not know. For now I had to create three different tables for each process even though they are somewhat similar.. I had one guy busting me on the design though I don't think it looks too bad. Anyways, was just wondering if this guys question is feasible, sharing multiple graphs for one table.


Fix frame-decode errors.

This pull request is for both https://github.com/cisco/openh264/issues/3710 and https://github.com/cisco/openh264/issues/3713.
I am not very sure that these are the correct fixes even though the frame decode errors are indeed resolved, so please review them. Especially for https://github.com/cisco/openh264/issues/3713.

Does this PR need rebasing?

I had synchronized to this master and rebased to the master in my forked repository. So, "Files changed" includes only the commit 1792403 and f7d35f2. However, "Commits" also shows previous five commits for unknown reason.
What was wrong in my steps?

I had synchronized to this master and rebased to the master in my forked repository. So, "Files changed" includes only the commit 1792403 and f7d35f2. However, "Commits" also shows previous five commits for unknown reason.
What was wrong in my steps?

Seems to be a Merge in there somehow, not sure how it got there.
But you can checkout a clean master and cherrypick the two patches and push --force to this branch again


Pre-trained models

could you share the download link of pre-trained models，thanks！

The model weights have always been available under Releases:
https://github.com/Qualcomm-AI-research/FFNet/releases/tag/models


Custom Folder Templates

What feature or improvement do you think would benefit Files?
There are 5 default folder templates: generic items, Music, Videos, Pictures and documents. They change the folder view based on the content of the folder.
These are not enough, i would like to have the ability to add more templates and customize them for Sfx, Project Files, Assets, Textures...
I would also like it if Files can recognize if the details view, preview pane, Item Checkboxes, Hidden Items ,Compact view... are on or off when making a new template and automatically turning on or off when that template is selected
Requirements

This proposal will accomplish X
This proposal will accomplish Y
This proposal will accomplish Z
-Add the ability to create new templates
-Make the templates remember settings like preview pane

Files Version
3.1
Windows Version
22H2
Comments
No response

Thanks for the feedback, We're tracking this in #5564. You can check there or click subscribe in the right sidebar for updates.

This is probably a little more advanced than what's being tracked in #5564 but we might as well track them together. Can you add a comment to #5564?


Creating vm from restore is failing for VMs with managed disk

faced issue when i deploy vm from restore( a VM with managed disk).  first i had to create manged disk from restore then only deployment template will work.

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

ID: e50dfe50-c177-f976-753b-9fa689d20f92
Version Independent ID: 7f787259-8ce5-9ba6-8df7-58ab0b89dd5c
Content: Back up and recover Azure VMs using Azure Backup with PowerShell
Content Source: articles/backup/backup-azure-vms-automation.md
Service: backup
GitHub Login: @dcurwin
Microsoft Alias: dacurwin

i havent read document properly.

@laxmihub I'm glad to hear that you resolved your issue. Please feel free to reach out if you have any feedback for this documentation.


bug(rust/gui-client/windows): Internet stops working in certain conditions

Describe the bug
In some conditions, the computer's Internet stops working.
We aren't sure yet if it's NRPT DNS or routes or a third thing. We've only encountered it while working on higher-priority bugs, so we haven't had time to debug and document it fully.
To Reproduce
Not sure yet. Seen it a couple different ways:

While doing fault injection and pretending make_tun bailed out in #6548's branch, I clicked "Sign In" but the Firezone page didn't load.
It happened on Jamil's laptop after he had closed the lid (maybe Windows suspended?)
It also happened on my Windows dev laptop when I closed the lid yesterday.

Expected behavior
Expect the Internet to work normally when Firezone is not open.
Screenshots / Logs
None yet.
Platform (please complete the following information)

Component (i.e. macOS client / Linux client / Gateway / Admin portal): Windows GUI Client
Firezone Version (e.g. 1.0.0 or N/A): Not sure
OS and version: (e.g. Ubuntu 22.04 or N/A): main? Some PR branches?
Deployment method: (e.g. Docker / Systemd / App Store or N/A): Local dev builds mostly. Maybe 1.3.0?

Additional context
Seen on my Windows dev laptop and Jamil's Windows laptop.

I think the issue that happened on my laptop was because IPv4 was disabled and github.com has only A records.


$FlowFixMe - TODO T29156721 React.forwardRef is not defined in Flow, yet.

[x] Review the documentation: https://facebook.github.io/react-native
[x] Search for existing issues: https://github.com/facebook/react-native/issues
[x] Use the latest React Native release: https://github.com/facebook/react-native/releases

Description
I found this comment out and I'm wondering I should remove this comment out or fix something.
    // $FlowFixMe - TODO T29156721 `React.forwardRef` is not defined in Flow, yet.
    ViewToExport = React.forwardRef(View);
    ViewToExport.displayName = 'View';

https://github.com/facebook/react-native/blob/v0.58.0-rc.1/Libraries/Components/View/View.js#L50-L52
Because React.forwardRef()'s arguments are defined in React v16.6.3.
export default function forwardRef<Props, ElementType: React$ElementType>(
  render: (props: Props, ref: React$Ref<ElementType>) => React$Node,
) {

https://github.com/facebook/react/blob/v16.6.3/packages/react/src/forwardRef.js#L12-L14
May I have some info about this comment out? @yungsters

I'm having this issue with typescript and react native


Fetch in background

When fetching data after phase changes, ensure it fetches without showing a loading indicator.

Appears to be the case already.


Eli oat badge fix

Can you make a PR that only modifies your entry?

Lol, I hope so! I thought that is what I was doing, but I guess I done and goofed! I'll give it another try!


feat: add paru as new providor

I'm submitting a

[ ] bug fix
[x] feature
[x] documentation addition

What is the current behaviour?
Comtrya does not have a provider for Paru
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem
Not a bug
What is the expected behavior?
Addition of Paru to the list of available providers
What is the motivation / use case for changing the behavior?
Paru is my preferred AUR helper and I'm sure others would want the option to use Paru over Yay
Please tell us about your environment:
Version (comtrya --version): comtrya 0.9.0
Operating system: Arch

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 0% with 77 lines in your changes missing coverage. Please review.

Project coverage is 26.16%. Comparing base (b574682) to head (e3d7c32).
Report is 64 commits behind head on main.

Files with missing lines
Patch %
Lines

lib/src/actions/package/providers/paru.rs
0.00%
76 Missing :warning:

lib/src/actions/package/providers/mod.rs
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #496      +/-   ##
==========================================
- Coverage   28.59%   26.16%   -2.44%
==========================================
  Files          85       87       +2
  Lines        2301     2511     +210
==========================================
- Hits          658      657       -1
- Misses       1643     1854     +211

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


feat: Add support for synchronizing to a local slave instance

Purpose
This PR adds support for synchronizing from a remote DynamoDB to a local instance by specifying a localhost URL for the parameter --slave-profile.
See #27
Key Changes

Updated cli.js to check for localhost in the --slave-profile
Updated Synchronizer.js to add construction of local client as well as selection of correct configuration for DynamoDB
Updated docs

Hi @echernyavskiy I checked the CI error and it is because of the commit messages (sorry, I did not read the rules before I made the initial commits!).
What would you like to do on that front?

Hi @echernyavskiy I checked the CI error and it is because of the commit messages (sorry, I did not read the rules before I made the initial commits!).
What would you like to do on that front?

Looks like Travis jobs failed because commitlint wasn't happy. If I understand correctly, commit messages must start with a type in order to satisfy this check, so you just need to push a valid commit here.
I'm not a maintainer of this project, just guessing here.

@echernyavskiy @CharlieDigital please just rebase and squash all these commits down into one commit, with an appropriate message. More info at: https://github.com/silvermine/silvermine-info/blob/master/commit-history.md

HI @jthomerson - any way we can get this merged into master and added to the latest NPM release?

@CharlieDigital can you please rebase to merge the two commits so that the build will pass? Alternatively, you can click they button that will allow me to push to your branch so I can update this PR.
git fetch --all
git rebase -i 80edaf9a1b5b5119fc241a336c3f59cf461bf07f
# in your editor, change "pick" to "fix" on the second commit
git push --force

Or:
git reset HEAD^
git add --all
git commit --amend
git push --force

@jthomerson added you to the repo

coverage: 0.0%. remained the same
when pulling 7ef89f21cc42250aaac3fd13c5d721efc3677315 on CharlieDigital:master
into 80edaf9a1b5b5119fc241a336c3f59cf461bf07f on silvermine:master.

@CharlieDigital @echernyavskiy I just rewrote this PR to (1) not overload the --slave-profile parameter, and (2) thereby reduce complexity of the code so that there were less places that needed to know about the local endpoint (only where we're creating a DDB SDK instance).
I've done a basic test of syncing a remote table to a local table. I have not been able to do a test of using remote masters and slaves, especially in separate accounts. Is that something one of you could test? Just a smoke test of overall master/slave credentials management?
If someone can verify that things are still working as they were before, I'll merge and cut a release. Thanks!

coverage: 0.0%. remained the same
when pulling 95d4595de375bb191eaa638649bd243de6813e1f on CharlieDigital:master
into 80edaf9a1b5b5119fc241a336c3f59cf461bf07f on silvermine:master.


Fix DID truncation and scrolling on mobile screen sizes

Fixed by https://github.com/xch-dev/sage/pull/53


fixed reduced price in admin order detail

ping @tonipinel please, can you check?

It's ok for me!


feat(en): metric scraping and grafana for docker compose image

I've added prometheus and grafana do EN docker compose with a simple dashboard like this:

I didn't add snapshots applier metrics as it turned out prometheus server is only started after the recovery finishes. I will create a separate PR fixing that and adding additional panels if I have enough time before I'm OOO


Update Composer dependencies (2021-12-09-00-03)

Visual regression test passed!
View the visual regression test report


Add a License

OSS is awesome! Promote contributions by adding a license.

Yeah, I forgot about the license. Adding the MIT license right now.


Add netlify.toml file

Moves the netlify configuration into a project file, to allow people without access to the netlify account to tweak it. Also allows people forking the repo to set up their own netlify which has the same configuration.

Looks like it worked, and the build logs indicate it built from the config file.


linux 用中文问就会报错Sorry, but it looks like your connection has been lost.

Sorry, but it looks like your connection has been lost.

Make sure bing.com and its subdomains are proxied.


Changed caution block link colour in dark mode

Changed the link colour in caution blocks to black when using dark mode

Found bug on the following page:
https://coolify.io/docs/knowledge-base/docker/custom-commands

Thanks for the PR!


Performance of InputMenu with more than 1000 entries in an object

Description
Is there a way to improve the performance of InputMenu with an object of more than 1000 entries? At 1000 it is already slow but at 5000 the whole page freezes and I have to hit refresh

@patrickhoehn I have the same issue, have you found a solution?

virtual scroll should be added to InputMenu and others 🙏🏻

@jfboily Unfortunately not yet

Closing in favor of https://github.com/nuxt/ui/issues/280. This should be fixed in https://github.com/nuxt/ui/issues/1289.


Missing files when deploying using build-context local

When using "--build-context local" to deploy a job, all files with file formats stated in the .gitignore are not included in the image which is normally desired. However, this also ignores files that were added with "add -f" to the repo and may be required by the job.

The reason it works like that is that the local build context is implemented for rapid developments sake, I.E, debugging code and such without having to care about the quality as you explore, because it's not going to end up in the git history. When all is said and done though, Racetrack does expect your code to live in a git repository eventually - so obeying .gitignore is very much on purpose.
I can't really find a purpose for git add -f, so I'd be happy to learn what the use case here is - it seems antithetical to good developer ergonomics to have a non-representative .gitignore file.

It is exactly testing where I am at.
My situation is, say you have several files (e.g. pickles) that generally not all go to the repo until they are tested and decisions have been made on which one should be commited. The file format might generally be excluded, because many such objects exist in the development process. One of those was chosen to be included in the repo.
Now I have new changes and I do not want to commit to the repo yet, but I would like to test the deployment, so I opted for the local build-context option. Then the deployment tries to look for the necessary file but of course it was not included.
Do you have any suggestions on what would be better practice?

That setup sounds like you should have feature branches or similar instead, that way other people can review and test the code as well. Then your .gitignore could be representative of the real code (since it'd just be the .gitignore of the feature branch, you could change it however you wanted), and this problem would be bypassed.
Besides, git add -f still adds the files to the repository as far as I can read, so one of us is misunderstanding what git add -f does.
I'll get a hold of the lead dev, but from my understanding this is Racetrack working as intended, and your setup not aligning with that. When it comes to this project though, I've mostly just lurked the issues without touching the code for a while now though, so I may very well be wrong in that assessment.

@juan-s-cruz I understand your problem. However, as @LookACastle said, local build context is implemented for rapid development sake. It's an experimental feature and it should be used with caution. Keep in mind that it packs the whole repository state, so if you're going to put large pickles there, the things may get heavy and sluggish pretty fast. Thus, the recommended way is to use git build context instead.
Then, you won't face this issue, because files seen by git will be always the same no matter where it's accessed, including files added forcefully. I presume since you did git add -f, you have probably also made a commit, so it's all in the git already and using git build context would fit to your workflow.
Although it makes local build context a bit inconsistent, at this point we don't plann to scan the whole git repository in search of the forcefully added files, this level of integration with git would be too intrusive in my opinion.
In a nutshell, if you want to be in sync with git, use "git" build context.